Curriculum Review: Day 3 Code Profiling

Prior knowledge:
On the first line of the .ipynb I would specify that the output of the profiler is the output of running the code profile_orbitize.py in case someone is not in the class.
Content:
Regarding the content, first, I was not able to run profile_orbitize. This due to 2 reasons:

I think the command line should specify that it runs on python3
python3 -m cProfile -o profiler_output.txt profile_orbitize.py
I cloned orbitize as specified on the instructions and it did not import it. It import it as orbitize.orbitize if I move it to Day3. Maybe this is easy to solve, but it didn't runned for me on the first try.

Apart, I would add the link to the documentation of pstats in case someone is interested.
I like the activities. I remember I had a good time las year doing them. :D
In general everything is nice, maybe too much text on the notebook, but is okey, everything well explained.
Learning goals:
Good learning goals :D. I would specify it on the .ipynb first paragraph just for it to be clear. I mean, "Profile the run time of your code and identify the computational bottlenecks".
I think the objective is fulfilled with the lesson. Maybe even more than that, I mean, the activities give a better understanding on how the code works, not just on run time and bottlenecks, so that is very nice!

Ok I added these changes in PR #73. I think the current python configuration we have students use should work with python and not python3, and the orbitize import should work. I added a sentence to caution that their setup could be different and to ask a TA for help if needed.


fix: make the api error message more clear

The way it's written now it's not clear that this an "or" check. It sounds like
the package may be configured to use the azure version of openai instead of standard
openai. I think this wording makes it a bit more clear that langchain checked for both API
keys and didn't find any.

I like it, thank you!


TIM-278 Code: HTML/CSS User Profile

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#148  👈
#147
#146
#145
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tamrivera05 and the rest of your teammates on  Graphite


[vite 3.0.7] Vue is not correctly imported in production

Describe the bug
I import vue and a 3rd package portal-vue. It works fine in dev, but raises the following error if I preview the bundle built:
Vue.extend is not a function

This bug looks very similar to an issue #9481 I reported previously but occurs only since v3.0.6.
✔Vite v3.0.5 dev
✔Vite v3.0.5 production
✔Vite v3.0.6 v3.0.7 dev
❌Vite v3.0.6 v3.0.7 production
Reproduction
https://stackblitz.com/edit/vitejs-vite-e9wniz
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (4) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /bin/yarn
    npm: 7.17.0 - /bin/npm
  npmPackages:
    vite: ^3.0.7 => 3.0.7

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Same for react, since 3.0.6 we got a problem in production build which says "useQuery" is not defined. This is a library we use from "@tanstack/react-query" package.

Likely fixed by https://github.com/vitejs/vite/pull/9653, pending a new patch release

Running vite-ecosystem-ci so we can release 3.0.8

3.0.8, which includes #9653 is out. @kingyue737 would you try with 3.0.8?

Thanks for the patch! fixed in 3.0.8


Mineral Extraction

This PR requires #101, as it uses both Creep.prototype.recycle and Creep.prototype.transferAll.

Adds economic and room settings for the EXTRACT_MINERALS actions.
Once mineral extraction begins it will continue until the mineral is mined out and all of the spawned creeps have recycled themselves.
New role fracker is basically a super beefy miner, but it will spawn with all 50 parts and could be expensive (hence the economy check).
Right now it spawns at most three extractors (less if there isn't room for them) and will always spawn two haulers.
There are no containers used- each fracker holds its minerals for the haulers, and if it fills up it will stop extracting..
Roads will make this a lot better. We should add those.

resolves #69

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

3
0
0
3

Vote Breakdown

User
Yes
No
Abstain

tedivm
✔

TheRealMaxion
✔

ags131
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


New package: GridapBifurcationKit v0.2.1

Registering package: GridapBifurcationKit
Repository: https://github.com/bifurcationkit/GridapBifurcationKit.jl
Created by: @rveltz
Version: v0.2.1
Commit: 424d7c5451d5948fd5422f06c8c25b5d2d95aa50
Reviewed by: @rveltz
Reference: https://github.com/bifurcationkit/GridapBifurcationKit.jl/commit/424d7c5451d5948fd5422f06c8c25b5d2d95aa50#commitcomment-145433323
Description: Allow to use BifurcationKit.jl with Gridap.jl

Hi,
What is blocking the merge?

[noblock]
See item 2 in the bot comment above:

is scheduled to be merged when the mandatory waiting period (3 days) has elapsed.

Currently it's also blocked by your comment, see item 3.

[noblock]
lol I did not read that my comment would further delay it. Thank you for your answer


How should I set up the reference camera?

I use four cameras to extract the human skeleton and use the skeletal data to find the external parameters (position and orientation) of the cameras.
If the cameras are 1,2,3,4, I want to estimate the position and orientation of the other cameras based on camera 1.
In this case, which file should I modify to use camera 1 as a reference?

Once you have done the calibration, you can apply any rigid transformations to their rotations and translations without loss of generality.  So instead of modifying the code only for this, I would like to suggest applying the inverse of R and t of camera 1 to the others.  In other words, you can transform the other cameras in the camera 1 local coordinate system.

Thank you for your reply.
Indeed, by applying Camera 1's rotation matrix and translation vector to the external parameters of Cameras 2, 3, and 4, we can obtain the external parameters of each camera based on Camera 1. This is very informative.
However, what I would like to ask this time is about where the reference camera is specified. Do you know in which file the reference camera is set?


POST /refreshes fails (400) server does not exists

Post to  https://canadacentral.asazure.windows.net/servers/XXXX/models/YYYY/refreshes
fails with an 400
message=An internal error occurred.
error in the header:
x-ms-root-activity-id: 9e66e693-6c4a-4a2b-bb9a-39f5d5d8e932
x-ms-xmlaerror-extended: Either server 'asazure://asazurecanc1-canadacentral.asazure.windows.net/XXXX' does not exist or user is not the administrator of 'asazure://asazurecanc1-canadacentral.asazure.windows.net/XXXX'
in SSMS we connect to that server using asazure://canadacentral.asazure.windows.net/XXXX and it is working nicely.
As per the documentation, the ApplicationService was created and the clientid was granted server admin.
As per the documentation, the clientid was also granted rights for Azure Analysis Services: Read and Write all models.
Not sure has per the documentation what the Rollout parameter should be.
Not sure why the canadacenter part of the hostname is being replaced by asurecanc1-canadacenter.
asurecanc1-canadacenter does not appears to be visible anywhere in the azure portal

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

ID: 6f706b7d-0a82-4535-f7ec-3851bd204bf2
Version Independent ID: fefdb693-fb98-25a0-684c-d828bb709114
Content: Asynchronous refresh for Azure Analysis Services models
Content Source: articles/analysis-services/analysis-services-async-refresh.md
Service: azure-analysis-services
GitHub Login: @Minewiskan
Microsoft Alias: owend

@fgariepy-relation1 Thanks for bringing this to our attention. Your feedback  has been shared with the content owner for further review.

@fgariepy-relation1 - Please open a support issue for help with this.
#please-close


Inspector panel is too wide

Godot version:
Godot v3.1-stable (GLES3)
OS/device including version:
OS: Arch Linux x86_64
Kernel: 4.19.4-arch1-1-ARCH
GPU: NVIDIA GeForce 940MX
GPU: Intel HD Graphics 620 (secondary, unused)

Issue description:
Switching between the "Inspector" panel and any other panel resizes the dock incorrectly.
The "Inspector" panel seems to be the issue here, since all other panels don't have this issue.
Steps to reproduce:

Ensure the "Inspector" panel is in the same dock as any other panel ("Node", "FileSystem", "Scene" all work)
Switch between the 2 (or more) panels

Minimal reproduction project:
This happens with the default editor configuration

Duplicate of #25317?

Ah okay, I couldn't find any existing issues. Closing as duplicate.


File Classes Not Working: No FileClass View for a FileClass or Ability to Assing FileClass to Note

Hi,
I've recently started using Metadata Menu. I've enjoyed the Preset Field Settings Options and have recently tried to start using File Classes. However the fileClass functionality is not working.
There are two issues:

FileClasses within the defined fileClass folder are not accessible when selecting the menu option "Add file class to..."
There is no icon for any FileClass nor the ability to access the FileClass view.

More Details
This was done a Windows 10 PC on Obsidian v1.3.4.
I setup a fresh sandbox vault to help isolate any potential issues with my normal vault.
Within the vault, I did the following:

Installed Metadata Menu
Installed Dataview and enabled dataviewjs and inline queries
Created 4 notes: two example notes and two fileClass notes
Updated the fileClass settings

These are the changes I made to settings:

This is the the content of the four files:

When I right click on either the Basic or Music Note and select, "Add fileClass to Music Note" a search modal opens and says no result found.
For the fileClasses, either global or music, when I right click on them, I only seeing the following options.

There is also no icon next to the title of any of the fileClass notes.
I did manually add some info to the fileClasses. I wrote status:: {"type": "Input"} into the global fileClass and some other properties into the music fileClass. Unfortunately, this has not made any difference with the issue

I'm not sure what happened exactly.
But this morning when I started the computer, this issue was gone. 🤷🏻

I have this same issue.


[Task] Implement children property of Text component

What should be done
Implement the children property of the Text component.

We cannot implement this feature without reworking the widget parser on lenra_ui_runner.
I'm going to explain why.
There are two objects, the LenraText and the TextSpan which is a child of a LenraText. A TextSpan is not a widget which means that it cannot be parsed by the props_parser using the ChildrenWidgets type. The TextSpan is actually an object which means that it cannot be parsed as a property(not a widget) either.
We could fix this issue by having two components on Lenra, the text which will be the LenraText widget and the textSpan which will correspond to the children of the LenraText (the TextSpan). I don't think that this is nice to do because it adds complexity on a text component which should be the most simple component to use on the platform.

I found a solution by upgrading the Flutter SDK so that constructor-tearoffs is activated on Dart. This adds the possibility to use List in the props parser.


Picture in Picture mode

Picture in Picture mode: It'll allow smooth transition across different part of application, without interrupting the viewing.


🐛 GA title clipped and strange fonts

Area
Voting Pillar
Domain
sanchogov.tools
Which wallet were you using?
Eternl v<IP_ADDRESS>
Context
GAs should have all the necessary information clearly visible and should respect overall UI for consistency and good user experience.
Steps to reproduce

Go to sanchogov.tools
Connect the wallet
Go to this specific GA - https://sanchogov.tools/connected/governance_actions/5b06976ce88d1cc4a7a5cc2478e418c1bb18cd5538527d963a931593f021f533#0

Actual behavior

Name of the GA is clipped

Reasoning tab has some fonts that do not respect the font used in the application

Expected behavior

Name of the GA should be fully visible, at least while hovering
Font used in reasoning should be the same ones as used in the appllication

Metadata: https://github.com/IntersectMBO/governance-actions


Feature: Add payload option to trigger

☕ Purpose
During the implementation of a function on our infrastructure we stumbled upon the requirement to pass a payload in a CronJob trigger. On this PR I'm adding that feature.
🧐 Checklist

[x] Changed the controller to pass the entire CronJobTrigger object, instead of just the schedule
[x] Changed the utils to parse the `payload' option of the object (if exists)
[x] Updated the tests

🐞 Testing
You can test it with my custom image: odelucca/cronjob-trigger-controller
🔗 Related PRs
This PR requires that #11 should be merged first, because it is using that PR as it's base.

Thanks for the changes @odelucca! I am going to merge just this PR since it already contains the commits from #11


BrowserStack Integration

This PR adds a 'browserstack' driver which can be used to run tests on browserstack.
Additional support in spec/teaspoon_env.rb :
config.driver = :browserstack
config.driver_options = {
  capabilities: [{
    os: "OS X",
    os_version: "El Capitan",
    browser: "chrome",
    browser_version: ""
  }, {
    os: "Windows",
    os_version: "10",
    browser: "firefox",
    browser_version: "44"
  }],
  username: ENV['BROWSERSTACK_USERNAME'],
  access_key: ENV['BROWSERSTACK_ACCESS_KEY'],
  project: 'BrowserStack Project Name'
  build: 'BrowserStack Build Name',
}

The capabilities object is send as is to BrowserStack as desired_capabilities.
To access locally served pages on BrowserStack, we need to run a platform specific binary in the background. So, will have to run the binary from the command-line before running tests.

Thanks! This is great, and all your work here looks really good. I'll do an in depth dive later today or this weekend, but from a glance it looks like you've done some really hard and good work here. You noticed hound is doing its job trying to keep up some code consistency rules, and it looks like you're cleaning those up.
For this to be merged, I'd like to see a unit test of the new BrowserStack implementation so we can make sure not to break it in the future (an example: https://github.com/modeset/teaspoon/blob/master/spec/teaspoon/driver/selenium_spec.rb), and it looks like there's a broken spec around the CLI help -- the new browserstack driver correctly appears in the options displayed when running teaspoon --help, so the expectation in that spec should change to ensure it stays there. =)
As a follow up to this PR, I'd appreciate it if you updated the README, and added a wiki article with similar information to your PR description -- any third party things like browserstack can be complex to setup, so any information you have would be really valuable there.
If there's anything I can do to help, let me know.

As a follow up, when this is merged, I will update things like the repo description to include BrowserStack as a viable driver setup.

Thank you for being patient.
I have added the unit tests for the added driver. You might want to go through those.
Regarding the BrowserStackLocal binary -
There are some ways to go about this.
Issue - The tests require this platform specific binary running in the background.

There will be a published gem for this purpose. The source code is here. It does start the binary, stops it and adds the logs of the binary to a file. Will be efficient to use. But it might take some time for it to be published on rubygems.org and we will have to add it as a dependency.
A thin version of the gem file can be implemented here which downloads the binary (according to the platform its running on) and starts it before and stops it after tests.
We can just add a word of warning that this requires a specific binary running. But running the tests for the users won't be automatic and will require setup from user's part.

Pending: The wiki page for using browserstack driver. Will proceed after the above discussion is clear.
PS: Is there a way to run the houndci tests locally? (Before doing an actual push). I couldn't find one.

Ok, I'm out for a few days (surgery), but will review after that. I really appreciate your time.
To answer your questions:
Let's use option 3. That's the same as we do for selenium, phantom etc. There's some tooling in util that allows us to find that binary if it's in the path, so probably want to use that same process here.
No, there's no a nice way to check code styles without hound -- I have the rubocop config locally I think, but never pushed that up. Don't worry about them, it's not a bad thing, we just want to keep relatively consistent code style so you can just clean them up as you want. Those emails don't bother me; just it doing its job. :)

On Feb 29, 2016, at 3:31 AM, Harish Ved<EMAIL_ADDRESS>wrote:
Thank you for being patient.
I have added the unit tests for the added driver. You might want to go through those.
Regarding the BrowserStackLocal binary -
There are some ways to go about this.
Issue - The tests require this platform specific binary running in the background.
There will be a published gem for this purpose. The source code is here. It does start the binary, stops it and adds the logs of the binary to a file. Will be efficient to use. But it might take some time for it to be published on rubygems.org and we will have to add it as a dependency.
A thin version of the gem file can be implemented here which downloads the binary (according to the platform its running on) and starts it before and stops it after tests.
We can just add a word of warning that this requires a specific binary running. But running the tests for the users won't be automatic and will require setup from user's part.
Pending: The wiki page for using browserstack driver. Will proceed after the above discussion is clear.
PS: Is there a way to run the houndci tests locally? (Before doing an actual push). I couldn't find one.
—
Reply to this email directly or view it on GitHub.

My last response was truncated, please expand it to see the hound comments.

On Feb 29, 2016, at 3:31 AM, Harish Ved<EMAIL_ADDRESS>wrote:
Thank you for being patient.
I have added the unit tests for the added driver. You might want to go through those.
Regarding the BrowserStackLocal binary -
There are some ways to go about this.
Issue - The tests require this platform specific binary running in the background.
There will be a published gem for this purpose. The source code is here. It does start the binary, stops it and adds the logs of the binary to a file. Will be efficient to use. But it might take some time for it to be published on rubygems.org and we will have to add it as a dependency.
A thin version of the gem file can be implemented here which downloads the binary (according to the platform its running on) and starts it before and stops it after tests.
We can just add a word of warning that this requires a specific binary running. But running the tests for the users won't be automatic and will require setup from user's part.
Pending: The wiki page for using browserstack driver. Will proceed after the above discussion is clear.
PS: Is there a way to run the houndci tests locally? (Before doing an actual push). I couldn't find one.
—
Reply to this email directly or view it on GitHub.

Will add this page as a follow-up to the PR

Looks good! Thanks for the initial work, and being flexible and accommodating with the follow up requests.
I'll merge, give it some time on master, and then release the new gem version.
Follow up question: Do you work for BrowserStack, or was this something you wanted personally and were willing to share?

I work for BrowserStack :) Thanks! :+1:

Did you ever get that wiki page created? =)

Yup. Check here


[e2e]: generic workflow_dispatch main workflow_inputs slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/4299767246
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.workflow_inputs.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.workflow_inputs.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Mar  1 03:31:44 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/4299767246
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.workflow_inputs.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.workflow_inputs.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Mar  1 04:08:08 UTC 2023

Will try to send a fix, it's from the recent dependabot PRs

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/4299767246
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.workflow_inputs.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.workflow_inputs.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Mar  1 04:33:22 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/4300091572
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.workflow_inputs.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.workflow_inputs.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Mar  1 04:42:00 UTC 2023
Tests are passing now. Closing this issue.


Some characters on iOS do not look like Roboto

Steps to Reproduce
Run the app on the iOS simulator with a comma on the screen. Compare that comma to the comma at https://fonts.google.com/specimen/Roboto

I haven't checked many characters but this one caught my eye. It's possible they are all off.

This is not an issue on master.


Use static name value for azdata sudo commands

The name option doesn't allow non-alphanumeric characters except for space so was failing on insiders builds (which have - Insiders in the name)
`

Pull Request Test Coverage Report for Build 322594745

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
31 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.007%) to 39.739%

Files with Coverage Reduction
New Missed Lines
%

extensions/azurecore/src/azureResource/providers/database/databaseService.ts
1
0%

extensions/azurecore/src/extension.ts
30
0%

Totals

Change from base Build 322393054:
-0.007%

Covered Lines:
20533

Relevant Lines:
47406

💛  - Coveralls


Repo sync for protected CLA branch

The pull request is created from master637659614394703301 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Errors in documentation, no migration path for pre v1.0 users

The documentation (the README in the project root) contains a few errors and omissions:

It doesn't mention that one needs to add saml2_pro_auth to the INSTALLED_APPS list
It refers to the function_urls, which no longer exist

This second item is causing an issue for people migrating from pre 1.0 to post 1.0: the class-based views have changed the SAML URLs, which means the SSO provider must update their settings to match the new URLs. In some organisations this can be a bit problematic due to slow formal ITIL procedures.
Not sure if this is a bug, or simply not or badly documented, but I would have guessed that setting the SAML_ROUTE setting, and including the URLs for ACS and SLS under the SAML_PROVIDERS dictionary, would result in the system using the same URLs as previously under the function-based views. This doesn't seem to work. I already had them in my settings, and they remain unchanged. What's worse: the new URLs don't seem to work either. They either result in a 404 or a 500. Before I file a separate bug for that, can someone from the team enlighten me as to whether these settings should in fact override the URLs for the class-based views as well?

Apologies for the documentation errors, SAML_ROUTE is no longer used since it makes more sense for users of this package to just specify a route prefix in their own URL patterns.
path("sso/saml/", include((saml_urls), namespace="saml")),

or more simply:
path("sso/saml/", include("saml2_pro_auth.urls")),

Unfortunately, the old function_urls didn't fit with the substantial changes we needed to make in v1.0 to make this package work for our use cases internally. The rewrite of the views and some of the other fundamental changes we needed to make are why this was cut as a new major release. If you need support for the old function_urls or you would prefer querystring parameters for specifying provider or acs those could likely be implemented by inheriting and overriding some of the class-based view set up with a different set of URLs in your app.
I would also accept and consider any PR that mimics the old URL scheme as long as it doesn't impact complexity much further.
Lastly, if you are receiving 404 or 500's for the new views, please submit a ticket with as much information as you can provide and we will take a look, but it does sound like it is likely a configuration mismatch in your app or the IdP.

Apologies for the documentation errors, SAML_ROUTE is no longer used since it makes more sense for users of this package to just specify a route prefix in their own URL patterns.
path("sso/saml/", include((saml_urls), namespace="saml")),

or more simply:
path("sso/saml/", include("saml2_pro_auth.urls")),

Unfortunately, the old function_urls didn't fit with the substantial changes we needed to make in v1.0 to make this package work for our use cases internally. The rewrite of the views and some of the other fundamental changes we needed to make are why this was cut as a new major release. If you need support for the old function_urls or you would prefer querystring parameters for specifying provider or acs those could likely be implemented by inheriting and overriding some of the class-based view set up with a different set of URLs in your app.
I would also accept and consider any PR that mimics the old URL scheme as long as it doesn't impact complexity much further.
Lastly, if you are receiving 404 or 500's for the new views, please submit a ticket with as much information as you can provide and we will take a look, but it does sound like it is likely a configuration mismatch in your app or the IdP.

Hey, thanks for the response. No need to assist me with anything; as long as you're aware of the errors in the documentation, I'm good.
On my end, I won't be debugging this further. For now I'm going to stick with the previous version in use, as that one worked. It just lacked one feature of 1.0 which I can achieve through different means.

Hey, thanks for the response. No need to assist me with anything; as long as you're aware of the errors in the documentation, I'm good.
On my end, I won't be debugging this further. For now I'm going to stick with the previous version in use, as that one worked. It just lacked one feature of 1.0 which I can achieve through different means.


Refactor the entire codebase

I have observed that there are many unnecessary parts of the codebase that are commented out and is taking up a lot of space, here I propose to improve the readability of the codebase by getting rid of the comments and aligning the div containers better to improved readability and effective utilization of the space.

I would like to work on this issue under gssoc'23


Do i have option to use it without the option to selected image?

Hi @roysG!
Could you elaborate the problem you're having? Do you mean not defining selected property of media objects?

Closing for inactivity. Let me know if you have any update on the issue.

Sorry to re-open such an old issue but I think @roysG may have been trying to work out the same thing I am.
Basically the grid in react-native-photo-browser alone is one of the only react native plugins that do this really well. What I'd like to do is just have it so you select an image on the grid, without being shown the full-screen page.
Basically using the app as an image picker from a grid if that makes any sense?
I've tried overriding the onSelectionChanged but it doesnt appear to work like that.
Any tips or info would be greatly appreciated @halilb :)


Transport: allow [host]:port smtp syntax

Fix #241

Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Opening augeas with root /, lens path , flags 64
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Augeas version 1.12.0 is installed
Warning: Augeas[Postfix transport - maildev.test](provider=augeas): Loading failed for one or more files, see debug for /augeas//error output
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error = parse_failed
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error/pos = 30
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error/line = 1
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error/char = 30
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error/lens = /opt/puppetlabs/puppet/share/augeas/lenses/dist/postfix_transport.aug:55.10-.47:
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error/message = Iterated lens matched less than it should
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Will attempt to save and only run if files changed
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'set' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']", "maildev.test"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'set' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']/transport", "relay"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'rm' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']/nexthop"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'set' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']/host", "[localhost]"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'set' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']/port", "1025"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Closed the augeas connection
Error: /Stage[main]/Profile::Postfix/Postfix::Transport[maildev.test]/Augeas[Postfix transport - maildev.test]: Could not evaluate: Save failed, see debug output for details

Thanks for the output. Can you also paste the error that Augeas should be displaying right after that log?

there ist no augeas error output
Error: /Stage[main]/Profile::Postfix/Postfix::Transport[maildev.test]/Augeas[Postfix transport - maildev.test]: Could not evaluate: Save failed, see debug output for details
Debug: Postfix::Transport[maildev.test]: Resource is being skipped, unscheduling all events
Debug: Postfix::Hash[/etc/postfix/transport]: Resource is being skipped, unscheduling all events
Debug: Postfix::Map[/etc/postfix/transport]: Resource is being skipped, unscheduling all events

Ah right. Your problem is not with the Puppet code, it is that your file cannot be parsed by Augeas:
Warning: Augeas[Postfix transport - maildev.test](provider=augeas): Loading failed for one or more files, see debug for /augeas//error output
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error = parse_failed

Could you paste the content of your file?

i have deleted the file '/etc/postfix/transport' and started puppet again, now i get the error message
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Opening augeas with root /, lens path , flags 64
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Augeas version 1.12.0 is installed
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Will attempt to save and only run if files changed
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'set' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']", "maildev.test"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'set' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']/transport", "relay"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'rm' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']/nexthop"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'set' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']/host", "[example.com]"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): sending command 'set' with params ["/files/etc/postfix/transport/pattern[. = 'maildev.test']/port", "1025"]
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Put failed on one or more files, output from /augeas//error:
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error = put_failed
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error/path = /files/etc/postfix/transport/files/etc/postfix/transport/pattern
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error/lens = /opt/puppetlabs/puppet/share/augeas/lenses/dist/postfix_transport.aug:50.15-52.23:
Debug: Augeas[Postfix transport - maildev.test](provider=augeas): /augeas/files/etc/postfix/transport/error/message = Failed to match tree under /files/etc/postfix/transport/pattern

     { "transport" = "relay" }
     { "host" = "[example.com]" }
     { "port" = "1025" }

  with pattern

    { /transport/ = /([.0-9A-Z_a-z-]+)?/ }
      { /nexthop/ = /(\\[[.0-9A-Z_a-z-]+\\]|[A-Za-z](.*[^\t\n ])?)?/ }
      | { /host/ = /\\[[.0-9A-Z_a-z-]+\\]|[.0-9A-Z_a-z-]+/ }
      { /port/ = /[0-9]+/ }

Debug: Augeas[Postfix transport - maildev.test](provider=augeas): Closed the augeas connection
Error: /Stage[main]/Profile::Postfix/Postfix::Transport[maildev.test]/Augeas[Postfix transport - maildev.test]: Could not evaluate: Save failed, see debug output for details

I see the issue. Can you try again please?

That works, great!
cat /etc/postfix/transport
maildev.test [example.com]:1025

Great. I need to fix a few edge cases and clean up the code.

If that's still working @aba-rechsteiner, it should be ready actually. Can you check again with the last commit please?

ou no I have missed that
"transport" = "relay"
maildev.test [example.com]:1025

I don't have that syntax in the Augeas lens.
I have:
the.backed-up.domain.tld       relay:[their.mail.host.tld]
*            smtp:outbound-relay.my.domain
<EMAIL_ADDRESS>           smtp:bar.example:2025
<EMAIL_ADDRESS>[<IP_ADDRESS>]:587

but the pattern   transport:[host]:1234 syntax is not supported. Are you sure this is valid?

hmm.. yes you could be right

I'll merge this pr for now as it supports the smtp syntax as parsed by Augeas.
Feel free to open a new issue if you find out the syntax you want is valid.


Figure out why is screen closing

In the qlogin node, the screen sometimes quits when exiting the server window (but sometimes not!)
The catch is that if the window is closed abruptly - due to loss of connection, out-of-time for a VPN Connection window, or by using Ctrl+D, the screen closes. You hve to use exit commdn to exit the login node as well as the server window.


U-boot recipe out of date

The u-boot recipes require version 2019.07, but yocto has updated to 2019.10 - http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/recipes-bsp/u-boot

Out of date 6 days, since Oct 14.
One can checkout poky 65cc5c245 for now to use 2019.07.


Allow a user to 'execute advertisebeacon' after 5 months.

In code currently is code to allow a advertisement of a beacon on/after 5 months to use existing key pairs and make a seamless beacon transition.
However this only happens when the beacon timer which is run every 180 blocks tries to advertise your beacon for u if >= 5 months.
With 'execute advertisebeacon' it was set at 6 months as bFromService was false.
I've changed that bool to true as a wallet user should be able to do the advertisement manually on/after 5 months till 6 months period. I don't think its a great idea to make them have to keep there wallet fully unlocked for ~180 blocks to do the advertisement. 6 months is used for various code and functions related to beacons yet the idea of >= 5 months beacon period is only related to the advertisement of beacons itself.

With this change all the calls are from a "service". Maybe we should remove that flag from the AdvertiseBeacon function.

we can't, the service is used to determine whether or not we are using 5 months or 6 months in Beacon Age.
6 months for all beacon related stuff except re advertising beacons between 5-6 months. this is done so the beacon is returned "" as in no beacon during 5-6 month time so we can readvertise using existing keypair


Great library thanks!

Safari still refuses to download... tries to open up the file in a new tab or window. Any way to workaround this? I need to be able to save a client side code generated file in blob or objectURL format to a user's downloads folder.

i hear you, i'm mad at safari for being weird. it has slowly been getting better, it used to not support any of the technologies needed to pull this off well. it has been several months since i last played around with it, maybe the latest will finally let allow first-class downloads. i'll check into it and let you know what i figure out sometime this week when i get my mac back. thanks for the feedback.


Real set variable

fixes #237

Do not forget to update unit tests or add new like you say

I'm doing it, this is way I made this PR a draft in the first place.

Empty lines are here to make code more readable, why you remove them ?

O, wow sorry dude, but you made new line so often that for me it looked bad, so I thought that some thing was wrong.
I thought it was some formatter or code prettyfier go crazy

I added tests, but my code doesn't work and I don't know way.
Can you help me @theludovyc ?

I not forgot you 😊

@Jeremi360 You need to fix this bug, to fix yours https://github.com/rakugoteam/Rakugo-Dialogue-System/issues/99

@theludovyc
About https://github.com/rakugoteam/Rakugo-Dialogue-System/pull/240#issuecomment-1669652167 :
I tried to debug it with unit-test and also with debug with prints and like on your screenshot it only catches strings.
About #99 I will try to fix it also then.
When I try to run any of RKScripts examples in Godot 4.1.1 it always start for this 3 errors:

So which should we use instead call_deferred() or call_thread_group() ?

I found why we have these errors now https://github.com/godotengine/godot/issues/78214#issuecomment-1590802452

Fix is ready, I made a pr soon :) !

I found why we have these errors now godotengine/godot#78214 (comment)
Edit : Fix is ready, I made a pr soon :) !
Edit : here #241

@theludovyc o nice I will merge it soon! I'm sorry that I was last quiet, but there was a heat wave, and I can barely live and think in such conditions, and I also had to get back together mentally after I lost my job: /

partialy implemented by #263


🐛 PreStartup Error! Web firebase_core_web failing

│ [error] | 4:33:20 314ms | PreStartup Error!
│ Expected a value of type 'String', but got one of type 'Null'
│ StackTrace: dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 294:3       throw_
│ dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 127:3       castError
│ dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 818:12  cast
│ dart-sdk/lib/_internal/js_dev_runtime/patch/core_patch.dart 674:17                as
│ dart-sdk/lib/_internal/js_shared/lib/js_util_patch.dart 81:5                      getProperty
│ packages/firebase_core_web/firebase_core_web.dart 35:26                           _createFromJsOptions
│ packages/firebase_core_web/firebase_core_web.dart 27:39                           _createFromJsApp
│ packages/firebase_core_web/src/firebase_core_web.dart 322:12                      initializeApp
│ dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 45:50                <fn>
│ dart-sdk/lib/async/zone.dart 1407:47                                              _rootRunUnary
│ dart-sdk/lib/async/zone.dart 1308:19                                              runUnary
│ dart-sdk/lib/async/future_impl.dart 162:18                                        handleValue
│ dart-sdk/lib/async/future_impl.dart 846:44                                        handleValueCallback
│ dart-sdk/lib/async/future_impl.dart 875:13                                        _propagateToListeners
│ dart-sdk/lib/async/future_impl.dart 647:5                                         [_completeWithValue]
│ dart-sdk/lib/async/future.dart 529:22                                             <fn>
│ dart-sdk/lib/async/zone.dart 1407:47                                              _rootRunUnary
│ dart-sdk/lib/async/zone.dart 1308:19                                              runUnary
│ dart-sdk/lib/async/future_impl.dart 162:18                                        handleValue
│ dart-sdk/lib/async/future_impl.dart 846:44                                        handleValueCallback
│ dart-sdk/lib/async/future_impl.dart 875:13                                        _propagateToListeners
│ dart-sdk/lib/async/future_impl.dart 464:9                                         <fn>
│ dart-sdk/lib/async/zone.dart 1399:13                                              _rootRun
│ dart-sdk/lib/async/zone.dart 1301:19                                              run
│ dart-sdk/lib/async/zone.dart 1209:7                                               runGuarded
│ dart-sdk/lib/async/zone.dart 1249:23                                              callback
│ dart-sdk/lib/async/schedule_microtask.dart 40:11                                  _microtaskLoop
│ dart-sdk/lib/async/schedule_microtask.dart 49:5                                   _startMicrotaskLoop
│ dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 181:7                <fn>
│ }

Tried multiple versions of firebase, on latest and its breaking but i dont think it was a package upgrade that caused it. Would just prefer less useless errors.

@cyberpwnn
Can you check below similar issues and underlying comments to see if they help in your case or not ?
https://github.com/firebase/flutterfire/issues/12237
https://github.com/firebase/flutterfire/issues/12253

@cyberpwnn A new version has just been released that fixes this issue. Feel free to reopen if you still encounter the issue after updating.


Import @hapi/joi in Worker cause error

Hi, i use webpack and try to import @hapi/joi like this:
import {config, Endpoint, S3} from 'aws-sdk';
import moment from 'moment';
import Joi from '@hapi/joi';

It build successfully but in preview, it return error:
Error: Error("expected value", line: 2, column: 1)
If i remove import Joi part, everything goes well. Can you help to fixed this?

below my webpack.config.js:
module.exports = {
  entry: __dirname + '/index.js',
  mode: 'production',
  optimization: {
    minimize: false,
  },
  performance: {
    hints: false,
  },
  devtool: 'cheap-module-source-map',
  node: {
    net: 'empty',
    tls: 'empty',
    dns: 'empty',
  },
  output: {
    path: __dirname + '/dist',
    publicPath: 'dist',
    filename: 'worker.js',
  },
};

I put another post in cloudflare community for workers.

Thanks for the report. Could you please run the same command with a detailed stacktrace: RUST_BACKTRACE=1 wrangler preview and post it here?

this is stacktrace i get:
RUST_BACKTRACE=1 wrangler preview
⬇ Installing wasm-pack...
⬇ Installing wranglerjs...
✨   Built successfully.
Error: Error("expected value", line: 2, column: 1)

stack backtrace:
   0: backtrace::backtrace::trace::ha005f61e3cfbd07b (0x1050a1afd)
   1: backtrace::capture::Backtrace::new_unresolved::h339da43cb4cb2a6a (0x1050a0c67)
   2: failure::backtrace::internal::InternalBacktrace::new::h2f8d45be066dd26d (0x1050a04b8)
   3: <failure::backtrace::Backtrace as core::default::Default>::default::h174bed90f39d5e6b (0x1050a06b4)
   4: wrangler::commands::publish::preview::preview::h65c0824e07e11a23 (0x104d26e99)
   5: wrangler::main::hef4cb33fcad6dad6 (0x104d3b7c3)
   6: std::rt::lang_start::{{closure}}::h232ceef3a5e57413 (0x104d20692)
   7: std::panicking::try::do_call::h1252fc9a2ff235eb (0x105123937)
   8: ___rust_maybe_catch_panic (0x10512cfde)
   9: std::rt::lang_start_internal::h4c054360e442146c (0x10512441d)
  10: _main (0x104d3c0a8)

I merged a patch #394, would you mind installing wrangler from source and trying to run RUST_LOG=info RUST_BACKTRACE=1 wrangler preview? Thanks!

We just merged a patch #394, would you mind installing wrangler from source, trying to run RUST_LOG=info RUST_BACKTRACE=1 wrangler preview and post the result? Thanks!

below the result, it work ok.
RUST_LOG=info RUST_BACKTRACE=1 wrangler preview
[2019-08-05T17:25:26Z INFO  wrangler] Getting project settings
[2019-08-05T17:25:26Z INFO  wrangler::commands::build::wranglerjs] skipping npm install because node_modules exists
[2019-08-05T17:25:26Z INFO  wrangler::install] prebuilt artifact https://github.com/cloudflare/wrangler/releases/download/v1.1.0/wranglerjs-v1.1.0.tar.gz
⬇ Installing wranglerjs...
[2019-08-05T17:25:28Z INFO  wrangler::commands::build::wranglerjs] wranglerjs downloaded at: "/Users/binduni/Library/Caches/.wrangler/wranglerjs-1e3edcc6c25d97ac"
[2019-08-05T17:25:28Z INFO  wrangler::commands::build::wranglerjs] Running "npm" "install" in directory "/Users/binduni/Library/Caches/.wrangler/wranglerjs-1e3edcc6c25d97ac"

><EMAIL_ADDRESS>install /Users/binduni/Library/Caches/.wrangler/wranglerjs-1e3edcc6c25d97ac/node_modules/fsevents
> node install

node-pre-gyp WARN Using needle for node-pre-gyp https download
[fsevents] Success: "/Users/binduni/Library/Caches/.wrangler/wranglerjs-1e3edcc6c25d97ac/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64/fse.node" is installed via remote
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

added 391 packages from 217 contributors and audited 4226 packages in 18.347s
found 0 vulnerabilities

[2019-08-05T17:25:49Z INFO  wrangler::install] prebuilt artifact https://github.com/rustwasm/wasm-pack/releases/download/v0.8.1/wasm-pack-v0.8.1-x86_64-apple-darwin.tar.gz
⬇ Installing wasm-pack...
[2019-08-05T17:25:49Z INFO  wrangler::commands::build::wranglerjs] Running "/usr/local/opt/node@10/bin/node" "/Users/binduni/Library/Caches/.wrangler/wranglerjs-1e3edcc6c25d97ac" "--output-file=/var/folders/73/hynx970n2d94453__rdpx9cw0000gn/T/.wranglerjs_outputzNEj9" "--wasm-binding=wasmprogram" "--webpack-config=webpack.config.js"
✨   Built successfully, built project size is 773 KiB. ✨
[2019-08-05T17:25:55Z INFO  wrangler::commands::publish::upload_form] Webpack project detected. Publishing...
[2019-08-05T17:25:55Z INFO  wrangler::commands::publish::upload_form] Form { boundary: "874c94290d35479a8712ec4d73b2a571", parts: [("metadata", Part { value: Body { kind: b"{\"bindings\":[],\"body_part\":\"script\"}" }, mime: Some("application/json"), file_name: Some("metadata.json"), headers: {} }), ("script", Part { value: Body { kind: Reader { length: 5040774 } }, mime: Some("application/javascript"), file_name: Some("script.js"), headers: {} })] }
👷  GET https://00000000000000000000000000000000.cloudflareworkers.com 👷
👷  Your worker responded with:

Image Worker

Click the below button to generate a new QR code. This will make a request to your serverless function.

   👷

you might want to look into this issue: hapijs/joi#528 and into the slightly outdated joi-browser npm package for inspiration

hey @hbinduni ! I think there are a couple of things we can do to make this clearer:

You'll notice that there is an error printed after wrangler finishes building in the version with the hapi/joi package:

✨   Built successfully, built project size is 1 MiB. ⚠  Your built project has grown past the 1MiB size limit and may fail to deploy. ⚠  ✨
We can reduce a good deal of confusion by making this error more prominent.

It might make sense for wrangler to halt preview if your build exceeds the 1Mb limit imposed by the Workers platform.

As to your question, the limits on script size are imposed by the Cloudflare Workers platform, not by Wrangler; Wrangler just checks on your behalf. You can make adjustments to your custom Webpack config that reduce the size for Workers, such as disabling source maps, turning on minification, etc.


Update Donk mc.tokenlist.json

hello, your token doesnt currently meet the liquidity requirements to be tokenlisted, can revisit later. Keep it up!

Whats the liquidity need to be at to meet that requirement?
On Tue, Mar 12, 2024 at 6:02 PM unserhtain @.***> wrote:

hello, your token doesnt currently meet the liquidity requirements to be
tokenlisted, can revisit later. Keep it up!
—
Reply to this email directly, view it on GitHub
https://github.com/traderjoe-xyz/joe-tokenlists/pull/1238#issuecomment-1992658556,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BG3ZON3S6VBHLWTXCAJBM7DYX53OTAVCNFSM6AAAAABEQWC6UKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJSGY2TQNJVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

hello, your token doesnt currently meet the liquidity requirements to be tokenlisted, can revisit later. Keep it up!

Can we finally get this added? We're now 3 weeks old, listed on CoinGecko, and have done collectively over $3M in volume. Off to a slow start but we are growing and definitely deserve to be tokenlisted. We checksummed our address for you guys. It makes more sense to add us than not have us on there. You guys have some tokens that are lower liquidity than ours already on there. We have 13,000+ holders and are currently partnered with multiple communities that are tokenlisted already. We have a stacked Twitter community too, please check us out @DONKonAVAX. Our community is asking for this.

BLOB has the same liquidity as us. Why isn't DONK added yet? Can we please merge?


Build long prefix package and update to 20160523b

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I think we need to update the pinnings. Here is an old pinning PR ( https://github.com/conda-forge/texlive-core-feedstock/pull/7 ), but I think it is even out-of-date. Maybe worth checking this against the wiki.

Thanks @jjhelmus!

Appears the tests did not run on CircleCI. 😕 As a result, it appears to now fail on master as we have some dependencies not rebuilt with conda-build 2 yet.
Adding distribution /feedstock_root/build_artefacts/linux-64/texlive-core-20160523b-pl<IP_ADDRESS>_0.tar.bz2 to conda-forge's main channel
Packages with binary prefixes shorter than 255 characters:
/feedstock_root/build_artefacts/linux-64/texlive-core-20160523b-pl<IP_ADDRESS>_0.tar.bz2 (80 chars)

./ci_support/run_docker_build.sh returned exit code 1

Seems to be fontconfig that is the issue.
xref: https://github.com/conda-forge/conda-forge-build-setup-feedstock/issues/40#issuecomment-265469774

Restarting now that fontconfig version `2.11.`` is rebuilt.

texlive has now been rebuilt successfully.


docs: update stackable overview image

This PR updates the overview image for the SDP in the readmes of the operators.

Sure you can, @stackfab, just get rid of the rounded corners and it instantly becomes much less jarring, just like the old one:

Or you could replace the white background with a transparent one, but that would be much easier with access to the original source files instead of dragging things around in Krita for five minutes.

We can merge as is for now, and have a follow-up regarding the graphic in general (modernize, include source, dark mode)

I will provide an slightly updated version until noon (16/12/24)


Add matcher for URL type

Looks like we've forgot to add support for Swift URL type. Might be a good idea to throw in support for NSURL as well.

Looks like this will be fixed in #4.

Fixed by #4.


adding matt wrock as maintainer

:cake:

:metal:

:metal:

:dancers:

:tada: :cake:

:+1:

:+1:

:boom:


Flint Axe needs default enchantment

It's otherwise inferior to both flint hatchets.
Efficiency 1 & Unbreaking 1?

So, I believe I actually figured out how to adjust durability of tools; as such, I can make the flint axe have significantly more durability than hatchets by either adjusting the axe's durability up or the hatchets' durability down. Alternatively, or in addition to, I could, as an example, make the simple tools repairable in the crafting grid. The hatchets may be repairable with sticks and branches while the axe is repairable with flint (point in the hatchet's favor), but the axe retains its enchantments when repaired while the hatchets do not.
Stuff like this does involve intricate functions in CraftTweaker that are poorly supported, but I can make them work in the current version. As you know, I plan on offering Jared fixes if I can work them out in the form of pull requests.

This would be a superior option for the axe.
Repairing with Flint, Keeping it's Enchantments... If we had a way to enchant them early.

We have books from loot bags. I was considering allowing you to apply an enchantment this way without an anvil, somehow. I haven't worked out exactly how it'd work. It might be limited to a single enchantment, or will never allow an enchantment beyond level 1, or something along those lines.
Alternatively, the pre-iron anvil that PrimalCore might introduce?

Well, I think I can safely close this issue.


insert horizontal rule

Issue #24
add horizontal rule button on toolbar
Changes

add horizontal rule button on toolbar

fix element order and tag closing of the example html

Flags

Use a ready-to-use icon font instead of creating a SVG for toolbar icons
Position buttons and icons in the toolbar relative instead of absolute

@michael-grover what do we want to show in the button? -?

The issue referenced here https://github.com/accordproject/markdown-editor/issues/24#issuecomment-544270095 could be due to not including slate attributes into the react element. We may need to add it to the schema and make sure it is inline instead of a block?

@irmerk the horizontal rule as hr tag has be a block element.
Removing the horizontal rule by using the TextArea works now.
But in the example the horizontal rules (see: defaultMarkdown from examples/src/index.js) are not displayed in the TextArea. 😐

Also, mind if I mark this as ready for review?

@waahhhh can you try what I suggested here? https://github.com/accordproject/markdown-editor/pull/141#discussion_r338195199

@waahhhh everything looks good to me, so I think we'll just wait for a fix in markdown-transform and then merge this. Unless any other reviewers has a different opinion.

@waahhhh everything looks good to me, so I think we'll just wait for a fix in markdown-transform and then merge this. Unless any other reviewers has a different opinion.

Do you know what fix should be done? If I know what Slate DOM is expected, I can work on implementing it, but I'm not sure what Slate DOM is needed.

Ran this locally and noticed a bug: https://www.loom.com/share/ce5c913ebdea430da450bb748300cb78
When creating an HR, pressing ENTER after creates a second one. A further ENTER does not create another. However, then selecting the space before the HR, pressing right arrow, then pressing ENTER creates an HR on each press.

Tried this fix with 0.7.2 and it works https://github.com/accordproject/markdown-editor/pull/141#issuecomment-547056355
This is still an issue though: https://github.com/accordproject/markdown-editor/pull/141#issuecomment-547103764

Ran this locally and noticed a bug: https://www.loom.com/share/ce5c913ebdea430da450bb748300cb78
When creating an HR, pressing ENTER after creates a second one. A further ENTER does not create another. However, then selecting the space before the HR, pressing right arrow, then pressing ENTER creates an HR on each press.

@irmerk, good find! This happens when the button in the toolbar is marked as active and ENTER is pressed.
It seems that something in the method "onKeyDown" is not processed correctly. Moving the cursor to an empty line after a HR element and pressing BACKSPACE seems not to be processed correctly too.
I'll find out what's happening there.

Closing as stale, feel free to come back to it @waahhhh!


[9.x] Possibility to force safe retrieval in Form Request

The case
FormRequest values are usually accessed by using $request->value, i.e. by a magic method. If $request->value is accessed, it means that all defined values have been validated, however, it does not mean that the value itself has been validated and wanted by the application. It could be resolved by using safe or validated methods, but I cannot find an option to set this behavior by default when requesting values from a form request.
This PR adds the possibility to force a safe retrieval from the validated method when accessing values in Form Requests.
You can turn it on by overriding class property:
protected $safeInputElements = true;

This saves us from extracting validated input each time:
public function controllerMethod(FormRequest $request): mixed
{
    $request = $request->safe();
}

Breaking?
It's fully backward-compatible since this feature is optional and turned off by default. When disabled, the new __get just forwards the call to the parent.
Any thoughts?
What do you think about this? The PR will stay as a draft until I see the green light from the community, or close it if people don't want this feature. 🔨

I'm going to risk opening the idea for a review, as it seems to be liked by some people and doesn't have any downvotes at the moment.  🙂

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

Released this as a package if someone's interested in this feature:
https://github.com/michael-rubel/laravel-safe-request


Respect existing task family when creating new one

This ensures the service being updated uses the same task family as the previous
one.

Looks right to me


picamera image capture not working

When I run
#!/usr/bin/python
from time import sleep
import picamera

with picamera.PiCamera() as camera:

    camera.start_preview()
    #camera.start_recording('foo.h264')
    #camera.wait_recording(60)
    sleep(2)
    camera.capture('foo.jpg')
    camera.stop_preview()

I get
<module 'time' (built-in)>

Traceback (most recent call last):
  File "/home/pi/piccapture.py", line 11, in <module>
    camera.capture('foo.jpg')
  File "/usr/lib/python2.7/dist-packages/picamera/camera.py", line 1570, in capture
    encoder.start(output)
  File "/usr/lib/python2.7/dist-packages/picamera/encoders.py", line 1529, in start
    timestamp = datetime.datetime.now()
AttributeError: 'module' object has no attribute 'datetime'
>>>

If I comment out "camera.capture('foo.jpg')"
the code runs with no errors.
If I do a video capture everything works ok.
Please help!
looking at problems other people have had I have just run sudo apt-get dist-upgrade to make sure everything is up to date. Still no joy.

Would you happen to have a file named "datetime.py" somewhere in the same directory? If so: that's the issue - you're hiding the standard library's datetime module. You need to remove or rename datetime.py to something different (and if there's a datetime.pyc, just remove it).

Hi Dave,
After a long search I eventually found datetime.py, it is in
/usr/lib/python3.2
Is that the problem and is there a way to fix it?
Kindest regards
Ray
On 13/09/2016 03:19, Dave Jones wrote:

Would you happen to have a file named "datetime.py" somewhere in the
same directory? If so: that's the issue - you're hiding the standard
library's datetime module. You need to remove or rename datetime.py to
something different (and if there's a datetime.pyc, just remove it).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/waveform80/picamera/issues/326#issuecomment-246596554,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVIYQVwHEmN44GLmGGul-hKNgeIOnFsMks5qpk6YgaJpZM4J6_aI.

No, that's the system's built-in datetime module; that's supposed to be there (assuming you meant /usr/lib/python3.2/datetime.py). I mean a datetime.py (or a datetime.pyc) in the same directory that you're running your script from (/home/pi). Can you paste in the result of running the following from the command line:
ls /home/pi

It'll probably look something like the following but with some more things in it:
Desktop         Music              Public          userland
Documents       Pictures           Scratch         Videos
Downloads       oldconffiles       Templates

To copy and paste here, just select the text with the mouse then middle-click to paste it into a browser window (or a text editor or whatever you like).

Hello Dave,
Have listed my home directory and there they were! I was looking in the
directory of the picamera module. Sorry got the wrong end of the stick.
The script runs without errors now that I have followed the instructions
you gave in your first email.
Thank you very much.
kindest regards
Ray
On 14/09/2016 03:48, Dave Jones wrote:

No, that's the system's built-in datetime module; that's supposed to
be there (assuming you meant /usr/lib/python3.2/datetime.py). I mean a
datetime.py (or a datetime.pyc) in the same directory that you're
running your script from (/home/pi). Can you paste in the result of
running the following from the command line:
|ls /home/pi |
It'll probably look something like the following but with some more
things in it:
|Desktop Music Public userland Documents Pictures Scratch Videos
Downloads oldconffiles Templates |
To copy and paste here, just select the text with the mouse then
middle-click to paste it into a browser window (or a text editor or
whatever you like).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/waveform80/picamera/issues/326#issuecomment-246933113,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVIYQTwpoxzMXx8nsbSf2UcFZ5dCyjzHks5qp6awgaJpZM4J6_aI.

Ah, glad to hear it!


[stable/sonarqube] Add talha as a maintainer for stable/sonarqube

Signed-off-by: Rob Kernick<EMAIL_ADDRESS>
What this PR does / why we need it:
Add a maintainer in the effort the help with #13867
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x ] DCO signed
[x] Chart Version bumped
[ x] Variables are documented in the README.md
[x ] title of the PR contains starts with chart name e.g. [stable/chart]

/cc @rjkernick

/assign
/lgtm
nicely done, thanks!

@maorfr thanks for the quick turn around. How can we get him added as a collaborator as well?

its already done


Solution swamped with shader projects

With PR #1130, the solution (VS2022-speak) is filled with lots of top-level shader projects.
Is that intended?
Shouldn't we at least move them into some "shaders"-folder, or so?

100% not intended.  Somehow didn't happen when I reviewed it, but when doing a cmake clean and full rebuild of the project I get that too. Moving them into a folder would be an option, but I'd rather prefer not having them at all. Shaders are already accessible from the respective samples.
@gpx1000 : Any suggestions?

top level shader projects?  Does VS now make cmake tasks into shader projects?!?  ...  sorry not intended...  Maybe let's move them into folders for now, and let's look into making them subtasks for each sample; which probably is the right way to do it.


crypto/x509: CreateCertificate can panic in asn1 code

Please answer these questions before submitting your issue. Thanks!
(1) What version of Go are you using (go version)?
go version go1.7rc4 linux/amd64
(2) What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/jeff/go"
GORACE=""
GOROOT="/home/jeff/src/go"
GOTOOLDIR="/home/jeff/src/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build022291271=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"

(3) What did you do?
https://play.golang.org/p/P68H4kx8AF
Created a self-signed certificate. This goes away if I set a SerialNumber on the template. The documentation says that SerialNumber will be used from the template, but not that it is required (so it's reasonable to think that it may be randomly chosen if not set). Either way, I'd argue that there should be a check to detect this and return an error, rather than an undocumented panic situation.
(4) What did you expect to see?
An error.
(5) What did you see instead?
A panic.

/cc @agl

Here is a minimal code.
package main

import (
	"encoding/asn1"
	"math/big"
)

type A struct {
	B *big.Int
}

func main() {
	asn1.Marshal(A{})
}

Whether that's a minimal code example depends on where the problem lies. It may be appropriate for asn1 to panic on a null big.Int; it may be that the appropriate fix is for the x509 package to ensure it never is and error if so.

maybe yes, maybe no (I'm not a judge either:)
asn1.Marshal also undocumented panic at least.


Fix center alignment of text

Center alignment of text is declared to center in https://github.com/dcasati/kubernetes-PlantUML/blob/dee6e17448ddc348a8e5a739419e5b1946f8db53/dist/kubernetes_Common.puml#L13 and overwritten by this. Left aligned node names look quite strange, so changing this as well to center aligned.

Hi, is anyone looking at this?


[Eval Track Results] ValueError: Length of names must match number of levels in MultiIndex.

I run PYTHONPATH=. python tools/test.py [congfig] [ckpts] --eval bbox
and get the bellow erros when I eval the tracking results:
`Accumulating metric data...
Computing metrics for class bicycle...
Traceback (most recent call last):
File "tools/test.py", line 250, in
main()
File "tools/test.py", line 245, in main
score_dict = dataset.evaluate(outputs, **eval_kwargs)
File "/plugin/vip3d/dataset.py", line 1106, in evaluate
results_dict = self._evaluate_single(result_files)
File "plugin/vip3d/dataset.py", line 1028, in _evaluate_single
metrics = nusc_eval.main()
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/nuscenes/eval/tracking/evaluate.py", line 204, in main
metrics, metric_data_list = self.evaluate()
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/nuscenes/eval/tracking/evaluate.py", line 135, in evaluate
accumulate_class(class_name)
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/nuscenes/eval/tracking/evaluate.py", line 131, in accumulate_class
curr_md = curr_ev.accumulate()
File "envs/mmdet3d/lib/python3.7/site-packages/nuscenes/eval/tracking/algo.py", line 123, in accumulate
thresholds, recalls = self.compute_thresholds(gt_box_count)
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/nuscenes/eval/tracking/algo.py", line 308, in compute_thresholds
_, scores = self.accumulate_threshold(threshold=None)
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/nuscenes/eval/tracking/algo.py", line 278, in accumulate_threshold
events = acc.events.loc[frame_id]
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/nuscenes/eval/tracking/mot.py", line 61, in events
self.cached_events_df = MOTAccumulatorCustom.new_event_dataframe_with_data(self._indices, self._events)
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/nuscenes/eval/tracking/mot.py", line 36, in new_event_dataframe_with_data
idx = pd.MultiIndex.from_tuples(indices, names=['FrameId', 'Event'])
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/pandas/core/indexes/multi.py", line 202, in new_meth
return meth(self_or_cls, *args, **kwargs)
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/pandas/core/indexes/multi.py", line 560, in from_tuples
return cls.from_arrays(arrays, sortorder=sortorder, names=names)
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/pandas/core/indexes/multi.py", line 492, in from_arrays
verify_integrity=False,
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/pandas/core/indexes/multi.py", line 331, in new
result._set_names(names)
File "miniconda3/envs/mmdet3d/lib/python3.7/site-packages/pandas/core/indexes/multi.py", line 1431, in _set_names
"Length of names must match number of levels in MultiIndex."
ValueError: Length of names must match number of levels in MultiIndex.`

Hi, what is your package version of nuscenes-devkit and pandas? Will the same error occur when using the checkpoint we uploaded?

The version of nuscenes-deckit is 1.1.2 and pandas is 1.3.5.
I used the detr3d_resnet50.pth provided by you and then trained the vip3d 24 epochs, getting the above error when evaluating.

What about using pandas==1.1.5 ?

thanks, that solve the problem!


PostgreSQL enum support

Can PostgreSQL ENUM types be supported?
CREATE TYPE ledger_status AS ENUM ('Draft', 'Paid', 'Partial Payment', 'Unpaid');

CREATE TABLE ledger (
  id TEXT PRIMARY KEY,
  status ledger_status NOT NULL DEFAULT 'Draft'
)

$ typeorm-model-generator
...
[9:35:00 AM] Starting creation of model classes.
Unknown USER-DEFINED column type: ledger_status  table name: ledger column name: status
Error occured in typeorm-model-generator.
...

Implemented and released in<EMAIL_ADDRESS>


lscolors: init at 0.7.1

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Check all or will be closed.

Contributors, close this pull request, this person does not have checked all checkboxes.

Result of nixpkgs-review pr 114091 at 4fedda2c run on aarch64-linux 1

1 package built:

lscolors

Result of nixpkgs-review pr 114091 run on x86_64-linux 1

1 package built:

lscolors

Why you merge it, we need to merge us.

Reverted.


fix: MaxResultCount limit the maximum number of returned objects

Close: #3409
PR Checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/master/.github/Contributing.md.
What is the current behavior?

Issue Number: #3409
What is the new behavior?

Use MaxResultCount to limit the maximum number of returned objects
Return empty array when limit is zero

Does this PR introduce a breaking change?

[ ] Yes
[x] No

New Imports

[ ] Yes
[x] No

Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

Codecov Report

Merging #3416 (1160d5b) into master (3146899) will decrease coverage by 0.31%.
The diff coverage is 3.12%.

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


New task type: Route navigation

Expected behavior
Navigation task called "route navigation" - reaching a destination via marked route.
(see tasktypes.pdf)

Where can I find the tasktypes.pdf ?


allow up to sass-rails 5.0.x

sass-rails 5.0.2 came out.  Relaxed the dependencies so this can be used with sass-rails 5.0.x.

:100:

sass-rails 5.0.3 depend on sprockets 3.0.0. Could you please make it like in my version - https://github.com/mib32/compass-rails/blob/master/compass-rails.gemspec ?

ping and +1 for better dependencies in .gemspec

+1

Thanks for merging this. Any idea when 2.0.5 will be released to include this?

+1

+1

Hi. Any idea when 2.0.5 will be released to include this fix? Thanks!

Hello. Is this project still being maintained? It would be great to know if we can expect 2.0.5 to be released or not. If not, please let us know so we can point our gem to the master branch, or to our own fork.

This gem is a dependency of one I'm using in a dozen of apps (chosen-rails), hope to have soon a new release with support for new versions of sass-rails & sprockets.

@craigmcnamara Hi Craig. Are you still maintaining this project? We've been waiting patiently since April for a new release that includes this PR. It's preventing many of us from upgrading sass-rails. If you're not maintaining it anymore, could you please respond saying so? Thanks!

I'm not really maintaining this project past where my project compatibilities lie. It looks like a fine patch for master, but I can't cut a gem and certify that it will work for everyone. If you would like to become involved in maintaining and releasing compass-rails, reply to this thread and I'll get you hooked up with people that can give you the rights to make that happen.

Boosh! - https://rubygems.org/gems/compass-rails


Mutating singular relationships via direct http route

Addresses #144. When mutating a resources relationships on a hasOne relationship currently breaks in master. I've implemented the fix suggested by @jbockerstette and I've added tests to prove it now works for both POST'ing and DELETE'ing to both hasOne and hasMany relationships.

😄 👍


V3.5.0

Merging in changes for 3.5.0 update.

This pull request is now managed with CodeReviewHub. Refresh.


Inadequate consideration of image size when comparing images.

Describe the bug
I have two images of different sizes, one of which is less than the width of my phone's screen. When I try to use the Compare Pictures feature in the software, the software doesn't handle the pictures with sufficient consideration for size, making one smaller and one larger. I've included those two images and the screen recording of the comparison as an attachment for your reference. Thank you for your efforts and contributions! I use DeepL to translate, so please forgive me if there may be inappropriate statements.
To Reproduce
Just use the two images I provided as attachments.
Expected behavior
Correct "comparison" behavior. That is, making two images equal in size.
Screen Recording Video and Attachment
Click Here
Smartphone (please complete the following information):

Device: Xiaomi 12X
OS: Pixel Experience with Android 13

It's intended behavior

Just use different compare mode if this doesn't fit you, fe side by side is scaling up


Adding IT properly

@callebtc cannot find the package.lock file...  tried to look into package-lock.json and poetry.lock but did not find anything to update related to languages

Ill do, don't worry, thanks for contribution!

Codecov Report

Merging #1695 (e035915) into main (95071eb) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main    #1695   +/-   ##
=======================================
  Coverage   48.43%   48.43%
=======================================
  Files          47       47
  Lines        7047     7047
=======================================
  Hits         3413     3413
  Misses       3634     3634

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


SOR: Handle failing FX pools by adding latestFXPrice to it's poolTokens.

[ ] Find out if we need FX pools.
[ ] FX pool tokens are missing latestFXPrice - subgraph has it and it needs to be added to the DB as well.
[ ] Once done uncomment FX pool from the query filter: https://github.com/beethovenxfi/beethovenx-backend/blob/v3-canary/modules/sor/sorV2/sorV2.service.ts/#L361

FX pools have liquidity on Polygon and Avalanche


Set required cmake version

It seems to use: https://cmake.org/cmake/help/git-stage/policy/CMP0104.html

This is not needed actually.


fix: changes IsBundling default to false

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Closing because I think this is stale


More robust EA-MUS detection/parsing

Recreating this PR as I think I've added the needed bounds checking now, but this should help load EA-MUS songs whose first byte is not 0x7D.

Okay, I fixed these style cases by myself.

Sorry about that; I had ported over some changes that I had made when implementing your sequencer in my own project so didn't think about matching the case. I'll keep that in mind if I make future PRs.

By the way, I took your update into other projects where this thing is used, and ye, with your changes, there are other songs that started to work tested ("Ski or Die" and "The Immortal"). However, as I know, they have other banks, but I used Cartooners' bank to play them, and they worked, but with not exactly the same instruments. If you could help me to rip these banks, I'll be happy to include them in my libADLMIDI.

Cartooners' bank is placed here: https://github.com/Wohlstand/libADLMIDI/tree/master/fm_banks/misc_files
The code that parses it into the embedded database: https://github.com/Wohlstand/libADLMIDI/blob/master/utils/gen_adldata/file_formats/load_ea.h

I am not sure, are these banks the same, or do they have different structures?


[util] Remove workaround for D&D: ToEE

Just so we don't forget about this, since with d8vk it got moved to the d3d8 section. It's no longer needed as of https://github.com/doitsujin/dxvk/commit/9b877cf623d143ccea73016ce899384ce08977a0.

Will consolidate in an upcoming PR.


Add "code for transparent background image" to article

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @PanSem
As there has been no response from your end for a while, we are closing this PR.
Feel free to request this PR to be reopened or create a new PR, if you still wish to make the changes you originally added plus any additional change requests asked of you.  Also, remember to update your fork, so you have the latest version of master in case it has changed significantly.
Happy Coding!


Add optional k8s role binding to application chart

We have multiple applications that require access to the k8s API.
It would be great to have this as an option.

tbh, not sure if that's not better handled separate, like the ingress is as well.

you need to increment the chart version (again), @blenkjon - sorry :P


In small screen navbar is overlapped by home text

Hi,
Is this the issue you need resolving?

Hey @leetebbs,
Yes I need help to solve this issue, should I assign this issue to you ?

Hi
yes please
I have made a pr

Merged. Thanks!!. Good work.

Great stuff.
Thanks


Feature/fix job listing list item style bug

Hi Drew,
I think my first commit resolves the list item style issue. However, while browsing the other sample job descriptions I noticed a few other styling issues.
For example, the 'Test' job at 'Great Company' has bold, black links:

The link style is applied inline to the element. Do you think the Quill editor is applying it by default?
On line 48 of the JobTemplate, I tried resetting the style attribute to an empty string and fixed the issue:

But I'm concerned this approach could create other bugs! What do you think?

Hey @forresthayes I just have a couple questions:

The switch to child.hasChildNodes() in line 50 is because it returns a boolean (and the previous one isn't a function at all 🤦 )
What is the point of child.style = '' in line 48? As far as I can tell nothing should be applying in-line styles at all, so this seems unnecessary, however I'd love to know what the thinking was, perhaps to overwrite any that could be given?

Thanks for this, just wanting to make sure I understand 100% before I approve. 🚀

Sorry, I tried to explain that line in my comment above.
After I made the change to child.hasChildNodes(), I was checking other sample job posts in the dev environment to make sure I hadn't broken anything. Drew's checklist on the issue said all job-description text should be blue so it caught my eye when a post included bold, black links instead of the semibold teal specified in JobTemplate.js (line 36).
This screenshot is from the job post for Great Company, test, Front-end on Apr 20 (3rd to last post):

The change I made at line 48, child.style = '', is my attempt to address the issue. It seems to work for this case:

But I'm concerned this approach could cause other bugs and would love to hear what you guys think. I suspect Quill is applying the inline link style at some point. Can we configure it somehow?

So, it looks like some inline styles are coming through. My guess is that is leftover from where the job listing was originally copied from?
If so, then that may need to be a different task altogether (to strip styling in the quill wysiwyg)

@pickleat I think child.style = '' is just removing any inline styles (if it finds any) - which may be the fix we need to my previous comment?
Rather than engineering our wysiwyg to handle it, just rip em out with line 48 if it finds any?
I think that's whats happening?

I think that's right. I was just asking for clarification more than anything.

I think we are good to go here otherwise. Everything looked good locally!


After null safety migaration compiler can't infer types that it could before

I'm currently migrating my get_it_mixin package. After applying the changes I got a lot of errors in my tests that the compiler couldn't infer several types anymore. After making some of them nullable again most issues went away, but with one function that gets a handler passed the inference fails although it worked witout problems before:

  void registerStreamHandler<T extends Object, R>(
    Stream<R> Function(T) select,
    void Function(BuildContext? context, AsyncSnapshot<R> newValue,
            void Function() cancel)
        handler, {
    R? initialValue,
    String? instanceName,
  }) =>

Overall I feel migrating Function types quite difficult.
You can have a look at the current commit of the repository and comare it to the non safe master branch.
https://github.com/escamoteur/get_it_mixin/commit/cdc6627f392cd0e9239d21af3e0b0ebf6527a4d6
I'm running on latest master:
[✓] Flutter (Channel master, 1.26.0-18.0.pre.92, on macOS 11.1 20C69 darwin-arm, locale de-DE)
    • Flutter version 1.26.0-18.0.pre.92 at /Users/escmoteur/Entwicklung/flutter
    • Framework revision d968e33b81 (5 hours ago), 2021-02-01 07:21:22 -0500
    • Engine revision ce14c8a841
    • Dart version 2.12.0 (build 2.12.0-276.0.dev)

I really would like to have it working as before but I have no idea how to do it.

@leafpetersen If someone could give me some help here it would be awesome

You might well have relied on implicit downcasts in the pre-null-safety version, and those locations will now require an explicit cast (that seems likely to be the case for s.data). You can add those casts, which will preserve the static analysis that you had previously (the only difference is that the source code will explicitly show that there is a dynamic type check at this location, it used to be implicit).
You could also try to improve on the typing, e.g., by changing some declared types, type parameter bounds, adding some explicit type arguments that were previously inferred, etc, until you have a program with so few downcasts that you're satisfied. But you can't tell up front how many changes are required for that, so you'd probably want to do it in specific, limited steps.

Thanks, I have a closer look. I will close this for now. If I don't get along I'll comming back

This is an instance of this problem, unless I'm misunderstood your code.
TL;DR - Inference was never working there, but before we would default to dynamic for s and the code would just work (but with no static checking if you got it wrong, and no code completion).  Now we default to Object? for s which means the code gets statically rejected.
See that issue for more discussion, including what we can do to improve this at some point in the future.

@leafpetersen yout are absolutely correct. I tightened the types to extending Object while working on the null safety that was the real reason.
Still it would be really awesome if we could solve the issue you linked.


Possible to enable remote debugging port on built application with Webview2?

Hi, I have an application that I need to attach to for automation. The application has a login screen which is written as a UWP app but once you are logged in it switches over to Webview2. I need to attach to the Webview2 session after the login has completed.
I wanted to know if it is possible to enable remote-debugging on this application if not, is it possible to check if a remote-debugging port is enabled on the application without having access to the code?

Hi @omegadash - thanks for your question. Can you post your question on the following repo where the engineering team will respond?
https://github.com/MicrosoftEdge/WebView2Feedback


Update react-controls-platform-libraries.md

Fixed tiny spelling mistake

@CasparRubin
Thanks for pointing this out. I've fixed it directly.


MM-37701 - Fix: Active participants in stats graph is including bots

Summary

Exclude bots in the select statement

Ticket Link

https://mattermost.atlassian.net/browse/MM-37701

Checklist

[ ] Telemetry updated
[ ] Gated by experimental feature flag
[ ] Unit tests updated

@lieut-data Added bots to the stats tests. Thanks!


allow language to be selected in request

This makes it possible to switch language in yomichan without changing the add-on config every time.

This could also be used for issue #15 by allowing two audio sources for different languages.

Great, thanks!


python310Packages.zha-quirks: 0.0.94 -> 0.0.95

Changelog: https://github.com/zigpy/zha-device-handlers/releases/tag/0.0.95
Description of changes

Things done

Built on platform(s)

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

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

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

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

Result of nixpkgs-review pr 223769 run on x86_64-linux 1

8 packages built:

home-assistant-component-tests.hassio
home-assistant-component-tests.homeassistant_hardware
home-assistant-component-tests.homeassistant_sky_connect
home-assistant-component-tests.homeassistant_yellow
home-assistant-component-tests.otbr
home-assistant-component-tests.zha
python310Packages.zha-quirks
python311Packages.zha-quirks


Add rules in the initial popup when launching a game in singe mode #63

Hello @rennacarver , thanks for your PR.
Please, next time try to
* include some screenshots and description of your work even if it's simple. It facilitate the review.

* follow the naming convention of branches as demonstrated in [contrib-guide](https://github.com/Ramzi-Abidi/Pong/blob/main/CONTRIBUTING.md)

Ok, will do!


fixes

Signed-off-by: kumarabd<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Ready to merge?


[WIP] ERB Linter for Policial gem

What this will do
This PR will:
affect the policial gem | not affect the policial gem

add an ERB styleguide, Policial::StyleGuides::ERB to the Policial gem.

Policial::StyleGuides::ERB will use a lint gem erb-lint which is yet to be extracted from this code.
Policial::StyleGuides::ERB will behave exactly like the other Policial::StyleGuides and use the same interfaces.

require a new gem erb-lint as a dependancy.

erb-lint will allow custom linters to be written in the Shopify/shopify repo and used by referring to them in a config file .erb-lint.yml. (Won't touch this repo)
One of these described custom linters will be written for finding Legacy Classes. (Won't touch this repo)

What has been done

The interfacing of Policial::StyleGuides::ERB works with the Policial::StyleChecker caller and returns the proper created Policial::Violations.
The groundwork for the LegacyClass linter. Right now it only blacklists pp, ico, and ico-*.

In Action
Running Policial against the Test PR: https://github.com/justinthec/policialtest/pull/1/files
require 'octokit'
require 'policial'
Policial.style_guides = [Policial::StyleGuides::ERB]
octokit = Octokit::Client.new(access_token: 'nicetry:^)')
detective = Policial::Detective.new(octokit)
detective.brief(
  repo: 'justinthec/policialtest',
  number: 1,
  head_sha: '9fc41515fd42b25e81cac91075a1368500a8d56f'
)
detective.investigate
p "Errors:"
detective.violations.each {|violation| p "Line #{violation.line_number}: #{violation.message}"}

Output:
"Errors:"
"Line 1: Legacy class `ico` detected."
"Line 1: Legacy class `ico-status-warning` detected."
"Line 2: Legacy class `pp` detected."

Todo

[ ] Refactor ERBLint to support multiple linters using an ERBLint::Runner like SCSSLint does.
[ ] Encapsulate the LegacyClass code into one of these linter classes which will be written as a custom linter in the Shopify/shopify repo.
[ ] Pull out the entire ERBLint class into a separate gem erb-lint and include it as a dependancy.

Closing for now in favour of integrating erb_lint with policial.io instead.
https://github.com/Shopify/policial.io/pull/101


App Creation Error

Just an issue when trying to create an app within the orion cli. I get the following error upon creation
TypeError: Object # has no method 'decompress'

➜  Projects npm update -g orion-cli
➜  Projects orion create testApp
Succesfully created your app!

    cd testApp
    meteor

Works here, please give me reproduction steps and information about your node version and OS.

Feel free to write into this issue and I'll re-open


Frozen Flipper Zero (Reboot Need)

Using GPIO with the GpioModeInterruptRise mode causes the OK button to freeze when exiting the application.
https://github.com/EstebanFuentealba/Flipper-Zero-Game-Boy-Pokemon-Trading/blob/0c562bfdd04063439a7cdf12deb579a9c1234bd5/views/trade.cpp#L324-L329
The mandatory solution is to reboot the Flipper Zero.

https://github.com/flipperdevices/flipperzero-firmware/issues/2538


add htmlbars parser

parses attr=variable, using attr and variable classnames
makes as the only keyword, with each, if, etc. styled as
built_in
uses name for the name of tags
only allows dasherized names, built-ins, or any block-form to be a
name,
otherwise assumes variable. However, only uses template-tag, and all
tags are either template-tag or template-yield (see below)
adds template-yield, because when working with templates this is
very often the one thing I would like to be able to guickly find in
a tag soup. template-yield matches {{yield...}}
tags that have a . in the first token: {{bob.mary}} will get parsed as
{{bob.mary}}
hljs.QUOTE_STRING_MODE and hljs.NUMBER_MODE are recognised inside
tags
adds the ember 2.0 set of template helpers as built_ins

Would welcome some advice on how to set up the relevance.
For example, to make the existing handlebars win unless as or mut etc. are present.

Got to merging it, finally. I've reworked the definition quite heavily for better consistency with other template languages. One thing that still bother me is the 'built_in' keywords: this is used mostly for standard library objects in procedural languages, but in HTMLBars they can seemingly be everywhere. For example, in the snippet that we have as a test:

  {{!-- only output this author names if an author exists --}}
  {{#if author}}
    {{#playwright-wrapper playwright=author action=(mut author) as |playwright|}}

{{playwright.firstName}} {{playwright.lastName}}

    {{/playwright-wrapper}}
  {{/if}}
  {{yield}}

It's "action", "mut", "yield" (but not "as"). May be it's better to just call them "keyword", not "built-in".


Add CONTRIBUTING.md

Opening the PR to start collaboration. This is not yet ready to be merged to main.

@waelkdouh @pankajagrawal16 @tjoudeh
I am wondering whether we should collapse the Contribution section into CONTRIBUTING.md.
What do you think?

Sure. It makes sense. After all to contribute.. they need to endup in github repo. So might make sense.


Initialize static states when entering Playmode

This pull request inits Singleton states when entering playmode.  This allows projects to disable "Domain Reload" in the editor project settings for their projects.  It also disables "Domain Reload" for the example project.

I think there's some other underlying problems in here.  Going to kill this pull request and think it over.


Installable Web App shows address bar on nonstandart port

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.59.1
Python release (python3 --version):
Python 3.5.3
Component/platform:
Installable Web App (https://home-assistant.io/docs/frontend/mobile/)
Description of problem:
My homeassistant instance is accessible from the Internet with http(9090) and https(9093) ports.
If i try to Add to homescreen from Chrome using standart ports (tested with 80, 443, 8123) installed webapp looks like standalone app.
If i use 9093 or 9090 i see new opened tab in Chrome (each click on homescreen Assistant icon makes new tab)
I tried to make simple html page with manifest.json on port 9093 with  json from this page https://developers.google.com/web/updates/2014/11/Support-for-installable-web-apps-with-webapp-manifest-in-chrome-38-for-Android
The result is: webpage looks like standalone app.
Expected:
Installed Web App looks like standalone app while HASS accessible on non-standard ports.

Dont understand why, but if i remove icons from manifest.json, problem disappear.
I made alias with nginx to my custom manifest.json
   "name":"Home Assistant",
   "short_name":"MyAssistant",
   "orientation":"portrait",
   "start_url":"/",
   "theme_color":"hsl(var(--huesat) 36%)"
   "background_color":"#FFFFFF",
   "description":"Open-source home automation platform running on Python 3.",
   "dir":"ltr",
   "display":"standalone",
}


In-Page Pagination

Can we get in-page/post pagination feature for long content ?

Duplicates #465

@bep okay thank you, will check it out.