Hotel Failing to Load. Invalid IL Code

I've been using the Hotel Plugin for about 3-months now and as of the Latest update to Rust the plugin has stopped working.
Here is the Error Message when Zone Manager try's to load Hotel.
[ZoneManager] Loading 'Hotel' to parent's request: 'Zone Manager v3.1.5 by k1lly0u'
Loaded plugin Zone Manager v3.1.5 by k1lly0u [512ms]

The current status is FAILURE: Invalid IL code in (wrapper dynamic-method) MonoMod.Utils.DynamicMethodDefinition:BuildingBlock.DoUpgradeToGrade_Patch0 (BuildingBlock,BaseEntity/RPCMessage): IL_02e8: ret
Check for possible errors on the log file
)
at MethodInfo HarmonyLib.PatchFunctions.UpdateWrapper(MethodBase original, PatchInfo patchInfo)
at MethodInfo HarmonyLib.PatchProcessor.Patch()
at MethodInfo HarmonyLib.Harmony.Patch(MethodBase original, HarmonyMethod prefix, HarmonyMethod postfix, HarmonyMethod transpiler, HarmonyMethod finalizer)

Hotel does not use harmony. I don’t think zone manager does either.  Are you running carbon. That looks like a bad harmony patchAdamOn Sep 11, 2024, at 7:08 PM, James R Elliott @.***> wrote:
I've been using the Hotel Plugin for about 3-months now and as of the Latest update to Rust the plugin has stopped working.
Here is the Error Message when Zone Manager try's to load Hotel.
[ZoneManager] Loading 'Hotel' to parent's request: 'Zone Manager v3.1.5 by k1lly0u'
Loaded plugin Zone Manager v3.1.5 by k1lly0u [512ms]
The current status is FAILURE: Invalid IL code in (wrapper dynamic-method) MonoMod.Utils.DynamicMethodDefinition:BuildingBlock.DoUpgradeToGrade_Patch0 (BuildingBlock,BaseEntity/RPCMessage): IL_02e8: ret
Check for possible errors on the log file
)
at MethodInfo HarmonyLib.PatchFunctions.UpdateWrapper(MethodBase original, PatchInfo patchInfo)
at MethodInfo HarmonyLib.PatchProcessor.Patch()
at MethodInfo HarmonyLib.Harmony.Patch(MethodBase original, HarmonyMethod prefix, HarmonyMethod postfix, HarmonyMethod transpiler, HarmonyMethod finalizer)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Yes, The Latest Version of Carbon.
Hotel Works but, when Zone Manager Calls it, I get the Error message I sent you.
So, I have no idea what changed in the Coding of Rust that needs to be adjusted in either Plug-in.
Also, I have No Harmony Mods in the Harmony Folder.
I'll check with k1lly0u also.


Build with vala 0.42.2

[   75s] /home/abuild/rpmbuild/BUILD/MarkMyWords-0.1.1/src/Widgets/Window.vala:12.5-12.30: error: Property Window.current_file' with custom get' accessor and/or set' mutator cannot have default' value
More info: https://build.opensuse.org/public/build/X11:Pantheon:Apps/openSUSE_Tumbleweed/x86_64/markmywords/_log

Same problem here. Im running GTK 3.24.5-1.

updated, should be fixed now. please reopen if the issue persists.


[AREA]: Introduce new products and services

Contact details
<EMAIL_ADDRESS>Organization type
Microsoft employee
End-to-end business process
Product and service lifecycle management
Specify the business process area name for the article.
Introduce new products and services
Enter any additional comments or information you want us to know about this business process area.
In progress
Specify the date you expect the article to be completed and ready for review.
8/31/2023
Code of Conduct

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

Split into two L2s


Removed json gem dependency

Not needed since ruby 1.9+ as it's including in the standard library
Similar pr sent to rails recently: https://github.com/rails/rails/pull/23453

Thanks, would you mind adding gem 'json', :platform => :ruby_18 to the Gemfile to ensure tests pass?  You'll want to rebase on master first as I've fixed the tests that are currently failing.

@domcleal adding that to the gem file won't actually fix it for people wanting to use ruby 1.8 though, if you want to fully support 1.8, which has been unsupported for nearly three years now, then don't merge this pr as it will fail to work for 1.8 users.

Yes, I'm happy for Ruby 1.8.7 users to add their own dependencies on the json gem if they want to use this, but I'd like to ensure the gem still runs on 1.8.7.  The Gemfile update is just needed for tests on 1.8.7 for spdx-licenses itself.
Though it's long since unsupported upstream, it's still common in certain places - this gem is used in the Puppet community, and 1.8's often still tested.

@domcleal cool, updated and rebased 👍

Merged, thanks @andrew!


Atom - Button

Decided not to do this. We can directly use the components of the selected component pack.


images: changed images to .png

Coverage remained the same at 90.663% when pulling 15adc8317515a1f244abb43269c44fd7a6341b3e on iMarbles:docs into 3343f939857a93f5ea48b0ac1c81030ee7701e09 on CS2103-AY1819S1-W16-2:master.


AutoComplete Breaks When Results is Null

After a null result dataset is gotten, further autocompletes are broken.
What should happen is that the null dataset is ignored and further autocompletes with data are shown normally.
What does happen is that further datasets that should have results are not presented as an uncaught error has already broken the autocomplete instance.
When dataresult result is a null rather than an empty array, this error occurs:
Cannot read properties of null (reading 'length')

Suggesting this edit to L1112 of jquery.easy-autocomplete.js:
if (listData.length === 0) { 									continue; 								}
with:
if (listData.length === 0 || typeof listData === "undefined") { 					 				continue; 								}


Patches aren't disabled in case of single player

Need to decide on possible solutions:

Drop single player support
Add patch global disable to PatchControl and use RunIfEnabled everywhere.

Decided to use RunIfEnabled approach quite some time ago.
Closing as it was resolved.


Subjectively fail transaction if contract bills RAM to another account within a notification handler

This PR introduces an optional (but enabled by default) subjective soft-fix for the issue of allowing notified contracts to charge RAM to any of the authorizers of the original action. It does so by subjectively (i.e. only during block production and not validation) failing a transaction if at any point in its execution an unprivileged contract executing under the context of handling a notification tries to bill additional RAM to an account other than itself (reducing RAM is still allowed).
This is a restriction in previously allowed behavior for a contract executing under the context of handling a notification. This means that if all active block producers of an EOSIO blockchain chose to enable this soft-fix, they would potentially restrict the ability of some existing contracts utilizing this behavior to fully function on that blockchain, at least until the contract code was updated to use a more appropriate pattern that does not rely on the restricted behavior. Note that block producers can still use a version of nodeos including these changes but without enforcing the soft-fix by setting the disable-ram-billing-notify-checks field in config.ini to true.
The most common pattern which utilizes the effected behavior is immediately creating a table row, if an appropriate one does not already exist, in response to receiving tokens from a user, and charging that user for the RAM costs of the table row. This is a pattern that was even used in an example exchange contract for depositing tokens into the exchange.
Unfortunately, supporting this particular pattern on EOSIO blockchains opens up contract developers and users to the potential of wasting their available RAM when interacting with accounts with malicious contracts deployed. While it is possible to mitigate the risk by carefully auditing the contracts deployed on accounts that will be notified as a result of a user executing an action, the existing behavior puts an unreasonable burden on contract developers and even users and can be easily argued to not be worth the benefits.
This is especially the case considering there is a straightforward alternative to the common pattern described above. Instead of creating a table row (and paying for that RAM then) in the notify handlers as a result of a transfer of tokens, the pattern could instead be:

The user first executes an action that creates a table row (perhaps initialized to a state that indicates it hasn't been fully initialized according to the application logic yet) of sufficient size to hold the later data that the table row will need to be mutated to hold.
Then, the transfer notification handler modifies that table row (which it will require to exist at that point) to hold the appropriate data.

Assuming the data modification of the table rows during the execution of the transfer notification handler does not require changing a primary key (there are future changes planned to address even this corner case) nor does it require increasing the size of the table row data, then the mutation should be successful even under the new stricter behavior. Note that these two actions can still occur atomically as before by putting them (in the right order) within the same transaction. Also note that another action may sometimes be desired to explicitly erase the table row (returning that RAM costs back to the user that payed for it) under certain circumstances; for example, if the table row tracks the balance of tokens deposited in the contract, then if all those tokens have been fully withdrawn, it would be sensible for the contract to allow the user to free that 0-balance table row.
Furthermore, the two-step pattern described above assumes the contract requires another user to pay for RAM costs of table row. If instead the contract can simply pay for the table row RAM costs itself, there is no need for any of the additional complication.

This is gonna break most existing game contracts... Some of them have even yielded contract control by setting it to eosio.code or block-hole public keys.

This is great.
I have been using atomic batching of transactions for row inserting + transfers and know that it works well. It's a small amount of added work for a huge benefit to the entire ecosystem.
As far as black hole contracts go, they can simply upload contracts to another account and change their UIs. Is this a hassle? Yes, but this fix takes out a user/dev experience issue which is more important than any single dapp imo. Those contracts are already most likely vulnerable to the ram exploit as well and need to be changed anyway.

Those contracts are already most likely vulnerable to the ram exploit as well and need to be changed anyway.

Good point. Those black hole contracts would have no need for RAM though, as all RAM usage are paid by users.
To me, the main downside of an action batching would be that users can no longer use my dApp with just wallet transfers. But anyways, I'm changing to using separate actions for all my contracts whether this PR is merged or not.

To me, the main downside of an action batching would be that users can no longer use my dApp with just wallet transfers.

Indeed this is the one sad reality about this.

Sorry, if this is the wrong place to comment, but if possible I would like high impact features like this to be opt-in rather than opt-out for BPs for a "deprecation period" (make disable-ram-billing-notify-checks "true" by default for first release and change it to "false" in the next one). Onus is on the BPs either way, but this might give DApps like ours a bit more time to adapt

Good news to hear that you guys have done something to resolve my issue #4824 raised one month ago.

One more question, If previous transactions had used this feature to use RAM. those transactions will fail to pass the verification when syncing from beginning if allow_ram_billing_in_notify is set to false


Timing information of individual notes ?

I'd like to do the following with the library and need you guidance as to where to start.
I have no problem building the library and linking it via CMake. It is the API that I need a little guidance with.
I'd like to

Load a MusicXML file (piano, two hands)
Find the beam (hands ?)
For each beam, iterate all the notes
For each note, extract the timing information for each note for further mathematical processing.

Which example or test code illustrate some of the above ?
What classes or methods should I dive further into ?
Cheers

Sorry I haven't answered yet. I'll try to get you an answer, but the short answer is that the tickTime field tells you when the note starts and duration tells you how long it is. Search for tickTime in the "api" directory.


feat: standalone markets page

This PR:

resolves #607
figma design
EPIC

Note:

This PR will serve as a base for other sub-issues related to this EPIC

[ ] #688
[ ] #689

Do not merge this PR until its sub-issues are developed
Screenshots:

I've requested some changes on the other PRs, when resolved I'll merge and I'll review this one as well.
Great job @ThomasRalee! Love it


Updating the file frontend-interview-preps/JavaScript/Programmes/Symbol.js

Updated the code in the aforementioned file.
Link to the issue: link

The example is quite not clear. I wrote this long back when I was doing something, which I can not recall why.
Would really appreciate if you can come up with a better example for this.

Um, I did some digging, and Symbol is not at all suited for hashing or encrypting passwords... as there are no ways to decrypt it.
We can use other hashing and salting methods for that.

Hey, the _password field wasn't intended to be hashed or something....was just a key to an object.
Symbol is used to create a unique variable name.
I just wanted a general example of it's usage.

Ohh, I misunderstood and thought that you wanted it to be a hashing function of some sort. My bad, I will fix it.

No issues. Request for re-approval once you fix it.

Hey, any plans of updating this?
You could just fix the first line (this.username = username) for now and maybe raise another PR when you come up with a better example.


PHOENIX-7156 enabling category-wise integration test execution capability

With the above change, we can select the integration tests which we want to run using following commands

To run the integration tests only for ParallelStatsEnabledTests
mvn integration-test -DskipParallelStatsDisabledTests=true -DskipNeedsOwnMiniClusterTests=true

To run the integration tests only for ParallelStatsDisabledTests
mvn integration-test -DskipParallelStatsEnabledTests=true -DskipNeedsOwnMiniClusterTests=true

To run the integration tests only for ParallelStatsDisabledTests
mvn integration-test -DskipParallelStatsEnabledTests=true -DskipParallelStatsDisabledTests=true

To run both ParallelStatsEnabledTests and ParallelStatsDisabledTests
mvn integration-test -DskipNeedsOwnMiniClusterTests=true

@stoty can you please provide the link to check jenkins pipeline failure

That result has aged out.
If you have ASF Jenkins access, you can trigger a re-run it from https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/view/change-requests/job/PR-1770/

@stoty I dont have access but I checked the link you have shared, it was triggered here. Everything passed but pipeline failed due to "Could not update commit status. Message: {"message":"Resource not accessible by integration","documentation_url":"https://docs.github.com/rest/commits/statuses#create-a-commit-status"}"

I have triggered the pipeline the https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/view/change-requests/job/PR-1770/3/
Will commit once after the build.

There are test case failures not related to this patch. Hence committing.


Issue 1724 - Porting logic of pre-handling minimatch patterns from azure-pipelines-task-lib

Issue - https://github.com/microsoft/build-task-team/issues/1724.
Ported logic of pre-handling minimatch patterns from azure-pipelines-task-lib (PS version / TS version).
The logic allows to handle negative minimatch patterns correctly.
Also reporting of excluded items was moved from GetFilteredItems to GetArtifactItems.
Checked cases:

download all (like **)
download all (like **) except 1or 2 file (like !**/file.1.nupkg)
download all (like **) except group of files (like !**/file.*.nupkg)
download all (like **) except directory (like !**/folder1/**)
download directory (like **/folder1/**) except 1 file (like !**/folder1/file.1.nupkg)
download group files (like **/file.*.nupkg) except 1 file (like !**/file.1.nupkg)
download all (like **) except directory (like !**/folder1/**) but with downloading 1 file from the directory (like !!**/folder1/file.1.nupkg)
using of whitespaces in start and end of pattern
using of sharp (#) in pattern
using of empty pattern

Please add tests to cover these changes.

Please add tests to cover these changes.

Done. Tests were added to this PR.

@denis-tikhomirov huge work, thanks! Let me take a look

Changes from the PR will be split to 3 parts for following sub-tasks:

porting logic of handling negative patterns for DownloadBuildArtifactsV1
adding of tests for ported logic
adding support of negative patterns to scenarios with using file share

Committed to master as:

https://github.com/microsoft/azure-pipelines-agent/commit/7ea02035e8403ff9dda97dd314159eaff3d96b74
https://github.com/microsoft/azure-pipelines-agent/commit/0c1261f003b1461ab65a149ad2ba9d42e6c05fa0
https://github.com/microsoft/azure-pipelines-agent/commit/7c376535b6290be95f0082a3a45f95ba0f1c9e5f


Fix memory leak and enable OkHttp support for ACR blob updates.

Description
Fix memory leak in ACR blob client API scenarios.
Enable support for OkHttp client.
Add tests for missing scenarios in the blob client.
Move to the latest core and netty\okhttp clients.
All SDK Contribution checklist:

[X] The pull request does not introduce [breaking changes]
[] CHANGELOG is updated for new features, bug fixes or other significant changes.
[X] I have read the contribution guidelines.

General Guidelines and Best Practices

[X] Title of the pull request is clear and informative.
[X] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[X] Pull request includes test coverage for the included changes.

Thanks for the feedback @srnagar, this PR was merged. I will open a new one for it.


South support was important when 1.4 was still the ESR, but it's gone now, so only Django versions with built-in migrations are supported. Rather than cleaning up the issues (e.g. #169) let's just kill the South migrations entirely.

:+1:

Fixed in #209


Engine freezes when calling Sprite.new() from joy_connection_changed callback

Godot version
v3.5.stable.official [991bb6ac7]
System information
Debian 11 - Intel Ivybridge Mobile - GLES3
Issue description
Pretty much what the title says, the engine freezes when I call Sprite.new() from a method that I previously connected to joy_connection_changed signal. No errors are shown.
Steps to reproduce

Run the minimal reproduction project, you will see the Godot icon moving around.
Connect or disconnect a gamepad and you will see the icon stops moving due to engine freeze.

Minimal reproduction project
minimal-reproduction-project.zip

I recall having a similar issue on my addon, which required a hack to bypass (waiting a frame):
https://github.com/rsubtil/controller_icons/blob/db639c0288e3992c6f861b4f91041adfb5a2e2f4/addons/controller_icons/ControllerIcons.gd#L102-L111
This happened on Godot 3, but I never tested if it still happens on Godot 4. I can try to see what's going on.
In the meantime, to prevent this, you can yield for a frame:
func _on_joy_connection_changed(device, connected):
    yield(get_tree(), "idle_frame")
    # Now run your code, it should not freeze anymore

@rsubtil Thanks for the fix!


Prefer React.createClass over react-mixin

@taion I'm a bit confused - why is this a better way to do it? I always thought it was just a preference thing

It's a little bit inelegant with react-mixin still because you have to manually bind the routerWillLeave hook. It's definitely up to user preference in the end, but on further thought and from talking to @ryanflorence, I think given a choice, I'd lean very slightly toward recommending using React.createClass over react-mixin to users who need advice on this in the first place.
I'm personally going to use react-mixin, but I think users checking these docs would be better served by dropping to React.createClass.

Ah, that totally makes sense then - it's also good to avoid using brand-new syntax (decorators are ES7) for general understanding.

Yup - I think we still want to point users at react-mixin, just... with a bit less enthusiasm.

@taion yeah, maybe just note it: "Note: If you're using ES7 decorators, you can try out react-mixin" or something like that

Well, the react-mixin thing works regardless. If users are comfortable/familiar with it, they'll know what to do. If not, then they probably should use createClass.


feat(extras.editor): add telescope media files

What is this PR for?
Support media file preview for telescope.
When developing, using Telescope to find files or search for PNG or other binary files sometimes encounters issues with previewing.
Therefore, one has to use VSCode or OS preview to view them, which requires initiating additional tool processes.
More information can be found here and here.
Does this PR fix an existing issue?

[x] No
[ ] Yes, #<issue_number>

ChangeLog

Create extra/editor/media.lua

Add nvim-telescope/telescope-media-files.nvim
2.1 Ensure all required prerequisites are installed
2.2 Verify that at least one of the required find commands is installed
2.3 Check optional prerequisites and include their filetypes if installed
2.4 Setup media_files extension for telescope

Checklist

[x] I've read the CONTRIBUTING guidelines.

Refs
https://github.com/nvim-telescope/telescope-media-files.nvim

Personally not a fan of this Extra and believe users should install it in their personal configuration if they want to. But the maintainer has the final say and will decide.

hey @arthur-mountain, ik there isn't but is there any way i can view images in a normal telescope window? like if im in a project and i do find_files do we have an option to preview images?

hey @arthur-mountain, ik there isn't but is there any way i can view images in a normal telescope window? like if im in a project and i do find_files do we have an option to preview images?

I think it's possible. telescope-media-files also has related issues, but no one has implemented it yet.
I tried to implement it here. However, it currently doesn't work.
Below is the code I implemented (Note: It doesn't work correctly 😢). You can refer to it or help me see how to complete it.
But this part should not be included in this PR, as the official recommendation is still to use the telescope media_files method.
local telescope_previewer_extend = function(telescope, plugin)
  local previewers = require("telescope.previewers")
  local ft = telescope.extensions.media_files.filetypes

  -- preview maker using vim.fn.job
  local new_preview_maker = function(filepath, bufnr, opts)
    print("fts: " .. vim.inspect(ft))
    print("filepath: " .. vim.inspect(filepath))
    print("bufnr: " .. vim.inspect(bufnr))
    print("options: " .. vim.inspect(opts))

    opts = opts or {}
    local isSupportedBinary = vim.list_contains(ft, filepath:match("^.+%.(.+)$"))

    if isSupportedBinary then
      vim.fn.jobstart({
        plugin.dir .. "/scripts/vimg",
        filepath,
        -- TODO: args for vimg
        --  refs: https://github.com/nvim-telescope/telescope-media-files.nvim/blob/master/lua/telescope/_extensions/media_files.lua#L35-L41
      }, {
        stdout_buffered = true,
        on_stdout = function(_, data)
          if data then
            vim.api.nvim_buf_set_lines(bufnr, 0, -1, false, data)
          end
        end,
        on_stderr = function(_, data)
          if data then
            vim.api.nvim_err_writeln(table.concat(data, "\n"))
          end
        end,
        on_exit = function()
          vim.api.nvim_buf_call(bufnr, function()
            vim.cmd("filetype detect")
          end)
        end,
      })
    else
      previewers.buffer_previewer_maker(filepath, bufnr, opts)
    end
  end

  -- preview maker using plenary.job
  local new_maker = function(filepath, bufnr, opts)
    print("fts: " .. vim.inspect(ft))
    print("filepath: " .. vim.inspect(filepath))
    print("bufnr: " .. vim.inspect(bufnr))
    print("options: " .. vim.inspect(opts))

    local isSupportedBinary = vim.list_contains(ft, filepath:match("^.+%.(.+)$"))

    if isSupportedBinary then
      local Job = require("plenary.job")
      local winc = vim.api.nvim_win_get_config(opts.winid)

      Job:new({
        command = plugin.dir .. "/scripts/vimg",
        args = {
          filepath,
          -- TODO: args for vimg;
          --   refs:  https://github.com/nvim-telescope/telescope-media-files.nvim/blob/master/lua/telescope/_extensions/media_files.lua#L35-L41
        },
        on_stdout = vim.schedule_wrap(function(_, data)
          print("output: " .. data)
          if vim.api.nvim_buf_is_valid(bufnr) then
            vim.api.nvim_buf_set_lines(bufnr, 0, -1, false, { data })
          end
        end),
        on_stderr = vim.schedule_wrap(function(_, data)
          if vim.api.nvim_buf_is_valid(bufnr) then
            vim.api.nvim_buf_set_lines(bufnr, 0, -1, false, { data })
          end
        end),
        on_exit = vim.schedule_wrap(function()
          if vim.api.nvim_buf_is_valid(bufnr) then
            vim.api.nvim_set_option_value("filetype", "text", { buf = bufnr })
          end
        end),
      }):start()
    else
      previewers.buffer_previewer_maker(filepath, bufnr, opts)
    end
  end

  telescope.defaults = vim.tbl_extend("force", telescope.defaults or {}, {
    buffer_previewer_maker = new_maker,
  })
end

In telescope plugin spec:
config = function(plugin)
  LazyVim.on_load("telescope.nvim", function()
    local ok, telescope = pcall(require, "telescope")
    if ok then
      telescope.extensions["media_files"] = vim.tbl_extend("force", telescope.extensions["media_files"] or {}, {
        filetypes = supportedFiletypeSet:toList(),
        find_cmd = findCmd,
      })
      -- telescope_previewer_extend(telescope, plugin)
    else
      LazyVim.error("Failed to load telescope.nvim")
    end
  end)
end,

I think it's possible. telescope-media-files also has related issues, but no one has implemented it yet.

There's this thing: https://github.com/nvim-telescope/telescope.nvim/wiki/Configuration-Recipes#use-terminal-image-viewer-to-preview-images
But Idk how to use it with ueberzug + not use it if it is not found in the system

I think it's possible. telescope-media-files also has related issues, but no one has implemented it yet.
@arthur-mountain
There's this thing: https://github.com/nvim-telescope/telescope.nvim/wiki/Configuration-Recipes#use-terminal-image-viewer-to-preview-images

But Idk how to use it with ueberzug + not use it if it is not found in the system

Thank you very much for the information! I didn't see that article. 🙃
However, I looked at the telescope source code and tried memo_hook in the initial implementation as well, but it doesn't seem to work.
I think my usage might be incorrect, but this could be explored further and a new PR could be opened.
Additionally, ueberzug is no longer being maintained, so i don't recommended to use it. You can try other options such as:

chafa, which is also used in telescope-media-files.
fzf, which supports image preview and has compatibility with two or three different terminals
...etc., for decoding image binaries that should be previewed in terminal tools.

Additionally, ueberzug is no longer being maintained, so i don't recommended to use it.

hey @arthur-mountain, i bet you didnt check: jstkdng/ueberzugpp

Additionally, ueberzug is no longer being maintained, so i don't recommended to use it.

hey @arthur-mountain, i bet you didnt check: jstkdng/ueberzugpp

Indeed, I couldn't find this project through my search.
The main hope is that it is safe and reliable, but the final decision will be left to the maintainers to judge.

Additionally, ueberzug is no longer being maintained, so i don't recommended to use it.

hey @arthur-mountain, i bet you didnt check: jstkdng/ueberzugpp

Indeed, I couldn't find this project through my search.
The main hope is that it is safe and reliable, but the final decision will be left to the maintainers to decide.

it is safe and reliable: i am using yazi.nvim and i can view images without installing image.nvim and i just have it configured in the main yazi config

not a fan of this. thank you anyway


Missing try-catch block in getBuildVersion function in gitutils.gradle file.

A try-catch block is missing int getBuildVersion function in the file gitutils.gradle : https://github.com/commons-app/apps-android-commons/blob/master/gitutils.gradle, due to which following error is occurring Gradle sync failed: CreateProcess error=2, The system cannot find the file specified
A try-catch block is used in the function getBranchName which is declared exactly below the getBuildVersion function, but I think it's not declared here by mistake.

This error never happens with me. What version of the studio are you using?

I am using Studio 3.0.1
This error occurred only when I first pulled the project from Github and tried to build the gradle in my system. For the first time, gradle built successfully only after I have added try-catch block there.
Now it's not showing any error even if I remove try-catch block. However, its a good practice to introduce try-catch block there.

I am not sure if it actually a good practice to have a try catch here. A try catch will let any errors happen silently and then any real issues would not be surfaced.
Open for discussion.

@maskaravivek what is your opinion for this discuttion?

As I said,

A try catch will let any errors happen silently and then any real issues would not be surfaced.

We should rather debug issues which come because of these util methods. :)

Closing it then


Adding OWNERS.md file

Fixes https://github.com/konveyor/ci/issues/25

Don't see risk in this PR, merging.


Single component to get the typical stuff, links, emails, phones

Thanks very much for this. It was badly needed. Everyone is using the unmaintained react-linkify and it has bugs.
I do miss the ease of react-linkify, where it got all the things you expected it to (links, phone, email - not jira, twitter, mentions etc), and could easily add properties like "target" blank, which you have, but just as example.
Is there a chance for a simpler linke-what-you-expect component?

This project provides various components for n number of cases this package can be used for. You can always create your own HOC which has links, phone, email combined into one component.
The goal of this project is to be highly customization so it's useful in many cases. In my experience, linking requirements have been very specific. For completely general cases other packages already exist.


autoenv_check_authz_and_run:20: = not found when entering into a new directory with .env file

I've just installed autoenv from source onto Ubuntu Trusty (commit 7858eaaf6b23a968e63fbb7584ff0df2795181ec) and it gives me this error when entering into a new directory with .env file:
autoenv:
autoenv: WARNING:
autoenv: This is the first time you are about to source /home/art/theproject/.env:
autoenv:
autoenv:     --- (begin contents) ---------------------------------------
autoenv:     # my token
autoenv:     TOKEN=*****
autoenv:
autoenv:     --- (end contents) -----------------------------------------
autoenv:
autoenv: Are you sure you want to allow this? (y/N) y
autoenv_check_authz_and_run:20: = not found

Hey, I've checked if it is work for commit c4bc3f5 (before) 9947b4a - Accept 'Y' or 'y' for prompt was merged. And it works. Seems commit 9947b4a broke something.

Nothing new on this?

I can confirm this issue still exists (installed on Arch Linux using antigen)

Getting the same issue.

I can confirm also that on the latest release, I have the same issue. reverting to  #c4bc3f5 fixes the issue for me

+1 I am getting this as well.

Fixed! https://github.com/kennethreitz/autoenv/pull/97

thanks @kennethreitz !

Works!


Issue #165 : Decouple event creation from event firing

From the code I have seen in CacheEventNotificationServiceImpl a lot of work remains to be done. It will probably be very useful for the development to find a way to represent the different conditions when a single listener of each type exist. And then see how these would compose when multiple listeners of different or same type are declared.

test this please

test this please

test this please

test this please

test this please

test this please

test this please

@rishabhmonga I added quite some comments that are again linked to having proper abstractions between the different objects interacting.
Could you do one more pass using the remarks I made and extending them to parts I did not go over?

sure, already I've taken care of a few of them

test this please

Closing as more work needs to happen to finalise - see #165


Unable to Edit Brew

I was attempting to share the brew with a friend to check on what needs to be edited and usability. However I am now unable to see the brew at all. I did notice at the top of the screen as seen in the attached image. I do not know if this is an added issue. Could use some help.

Additional Details
Share Link :
or
Brew code to reproduce : Click to expand
PASTE BREW CODE HERE
H1Gg0SEtDe

You've got a

Thank you this worked. Is there a way to have the broken page removed or deleted?

Haha.... no, I don't think there's a way to do that yet. I've got a couple copies of your thing on my account as well from me trying to figure out your broken page and I can't delete them. Might be good if there were a delete button right there on the profile page, huh?

yes, that would be wonderful

Pushed an update in v2.7.2 that adds a button on the user page to delete a brew.


Create a tendency chart

Idea
Each round has info about date when it occur. Using this info we could build a tendency chart like this one:

Implementation Steps
Try to use vue-chart library for this purpose, cause we're already using it for statistics chart. If you can't use it, then check out this library.
You can place new chart as a new tab Tendency

Hey, can I tackle this issue? If so please assign the Ticket to me :D

Sure!

Hey, which Icons do you use?

@Rayleigh3105  for the tab? I'm looking here https://materialdesignicons.com/
and I think of chart-line or chart-bell-curve-cumulative. But you can provide what you think suits the best

so you can use anything from this link, just prefix it with mdi-

Ah, okay. Never used Vue.js before (only Angular), I think its better to have basics in a Framework, where I want to implement something :D. That issue is a little bit to big for me, may you dissociate the issue, sorry?

sure, np:) if you want to learn more about Vue I'd recommend those resources:

https://www.vuemastery.com
https://vueschool.io/
https://michaelnthiessen.com/


Bug: fails when /api isn't defined

I have a simple project with no API. When this plugin added I'm getting this error
no such file or directory, lstat '<project-path>/pages/api`

As a workaround – I've just created api/index.ts

seems like fixed long time ago but not released to NPM

Anyway, bug found in this "unreleased" version. When there is no /api folder TypeSafeApiRoute is not created but still referenced in typeSafeUrl which causes TS to do not work.
it should be done like this
  export type TypeSafeApiRoute = never;

or TypeSafeApiRoute should not be mentioned at all


Mimick Typora specificity for color setting

Fix #50

Unfortunately this doesn't seem to fix the button hover color. I'll still merge it since that change can help, but I'm gonna see how I can fix that in a bit.


Port IMPROVER to Python 3

As an IMPROVER developer, I want to port our code to Python 3 in order to avoid problems when Python 2 is no longer supported in 2020.
Related issues: Scoping ticket: #487 and epic: #265
Acceptance criteria:

Following the suggestions in ticket #487, port the remaining code to Python 3.
Alter the Travis configuration to ensure that all new code/PRs are Python 3 compatible (do not merge into master until tests pass).

Ported to Python 3 in #558.


Install is broken

Hi,
https://github.com/SharpeRAD/Cake.Services/blob/master/src/Services/Manager/ServiceManager.cs#L886
indicates that you are missing the = sign as part of the argument.
USAGE:
        sc <server> create [service name] [binPath= ] <option1> <option2>...

OPTIONS:
NOTE: The option name includes the equal sign.
      A space is required between the equal sign and the value.
 type= <own|share|interact|kernel|filesys|rec|userown|usershare>
       (default = own)
 start= <boot|system|auto|demand|disabled|delayed-auto>
       (default = demand)
 error= <normal|severe|critical|ignore>
       (default = normal)
 binPath= <BinaryPathName to the .exe file>
 group= <LoadOrderGroup>
 tag= <yes|no>
 depend= <Dependencies(separated by / (forward slash))>
 obj= <AccountName|ObjectName>
       (default = LocalSystem)
 DisplayName= <display name>
 password= <password>

When I try to use this, it actually is executing
& "sc.exe" create "Name" -binPath "C:\Some\Path" -DisplayName "Foo"

What It should execute is
& "sc.exe" create "Name" binPath= "C:\Some\Path" DisplayName= "Foo"

Fixed in v0.2.7


chore: add ci config for unit test

This PR aimed to adding unit test CI check and report coverage for PR.
todo list

[x] add unit test
[x] add e2e test
[ ] report coverage for PR @narrowizard
[x] report coverage for branch @narrowizard
[ ] add coverage to README.md @narrowizard
[ ] create codecov account for uploading coverage info @UltimateBeaver

Report coverage for PR & Report coverage for branch & add to readme
We planed to use codecov to display coverage info in README.md and PR comments. For that purpose, we need install codecov app for lake repository, and add the upload token as a secret key called CODECOV_TOKEN. @UltimateBeaver

steps

Install code cov app for lake repository.

Sign in codecov.io with github account.

Setup repo at https://app.codecov.io/gh

Copy the upload token and create a secret called CODECOV_TOKEN for lake repo(on github).

LGTM


Fix for lodash 4.x

lodash 4.x does not alias _.unique -> uniq. May fix #239.

Thanks! I'll try to update the project definitions ASAP also to catch issues like this.


saving audio files?

I asked a question here: https://stackoverflow.com/questions/76163192/elevenlabs-python-how-to-set-a-storage-path-for-the-output-mpv-audio-files
I can't specify path to save audio files ,

Play is only to reproduce the audio, you can use the save function if you want to save it instead:
from elevenlabs import generate, save

audio = generate(
  text="Hi! My name is Bella, nice to meet you!",
  voice="Bella",
  model="eleven_monolingual_v1"
)

save(audio, "speech.mp3")


feat[web]: use the predicted denoised image for previews

Some schedulers report not only the noisy latents at the current timestep, but also their estimate so far of what the de-noised latents will be.
It makes for a more legible preview than the noisy latents do.
I think this is a huge improvement, but there are a few considerations:

Need to not spook @JPPhoto by changing how previews look.
Some schedulers (most notably DPM Solver++) don't provide this data, and it falls back to the current behavior there. That's not terrible, but seeing such a big difference in how previews look from one scheduler to the next might mislead people into thinking there's a bigger difference in their overall effectiveness than there really is.

My fear of configuration-option-overwhelm leaves me inclined to not add a configuration option for this, but we could.

Just tested this out. The new previews are 1000% better.
The samplers that do not provide this so far are

PLMS
dpmpp_2
k_dpm_2
k_dpm_2_a
k_dpmpp_2
k_huen

So apart from ddim, k_euler and k_euler_a .. None of them do. Is this intended?

Just tested this out. The new previews are 1000% better.
The samplers that do not provide this so far are

PLMS
dpmpp_2
k_dpm_2
k_dpm_2_a
k_dpmpp_2
k_huen

So apart from ddim, k_euler and k_euler_a .. None of them do. Is this intended?

klms is doing the new preview as well.
I like it very much. The only problem I found is that there is a moment at the very end when an unscaled 64x64 image flashes up for a fraction of a second before the full-size final image appears. This happens both with the predicted denoising and traditional previews. I recall that this was a problem for a while back in 2.2 and was only visible to me because of ssh latency. I can post a video if this isn't clear.

Looks wonderful, but has visual defect at very end when a miniature version of the final image flashes up just prior to displaying the full size image.

I've been seeing that on main for some days now. I don't think it's related to this branch at all.

looks good, though as i basically only ever use dpmpp2 i'll never see it :)

I feel that. It might be plausible to add this to DPM Solver++? Or, come to think of it, is this much different than the direct output of the U-net before the scheduler adjusts it for t[next]? In any case, there are some possible avenues for follow-up here.

looks good, though as i basically only ever use dpmpp2 i'll never see it :)

I feel that. It might be plausible to add this to DPM Solver++? Or, come to think of it, is this much different than the direct output of the U-net before the scheduler adjusts it for t[next]? In any case, there are some possible avenues for follow-up here.

yeah i think this is just the output of the unet. huh, now that i think about it the correct place to intercept/grab this data might be to keep a copy of the return value from do_diffusion_step - then it ought to work for all samplers

btw, https://github.com/invoke-ai/InvokeAI/pull/2940 may cause conflicts, depending on merge order. It's a minor change though.

@Kyle0654 I'd like to get this merged in, but there are conflicts with the invocations/generate.py file. I'm not sure which way to resolve some of these conflicts, so could you advise?

We've probably made changes. I think the best bet is probably to rebase on main then copy in the generate.py changes.


Force arrows are selectable but cannot be modified

File > Preview Experimental Data
click on arrowhead -> selection box appears but no properties appear in Navigator or Visualizer Controls box and arrowhead cannot be dragged

I still see this behavior.

That being said, I don't think this is a high priority issue for the 4.0 release, given other tasks on our plate. @aymanhab feel free to move off the 4.0 milestone.

The behavior described by Tom still exists. Also, when I select a force in the visualizer, the force in the navigator isn't highlighted (and vice versa)


ER: Validate if any *log contains "ORA-" before loading it

If any OP *log file contains the pattern "ORA-" the file should be skipped.  Those files should be flagged to the use with a message that tells them that those files were skipped most likely because there are errors in them and they should be manually reviewed. Exception to this rule are "patchlevel.log" and "alertlog.log".
Today we can only stop those files using:
grep "ORA-" *log|cut -d ":" -f1|uniq

Also, validate number o commas per line
for i in ls -l *awrhistosstat*.log|awk '{print $9}'|uniq; do
echo "Processing: "$i
linecounter=0
while read line; do
commascounter=echo -n "$line" | tr -cd , | wc -c
if [ $commascounter -gt 18 ]; then
echo "Filename: "$i", Line Number: "$linecounter", Comma Line Counter: "$commascounter
fi
linecounter=$((linecounter+1))
done < $i
done

Maybe validate only first X rows

Another possible validation is to check: If the files is empty


Models not available

Hi,
Thanks for the wonderful research. Also, thanks for sharing your code in such a timely manner, it is very helpful.
I am accessing your code and was trying to access the models from https://entuedu-my.sharepoint.com/:f:/g/personal/libo0013_e_ntu_edu_sg/ElEEAWuckgZKlDsK6inCa0UBs-KJcCG49ef30wmFmmW1og?e=2Gl0w0
But I could not find the models. Can you please help with this?
Regards
Harkirat

Thanks for your attention to this work. Since we are submitting our to a conference, we plan to release these models & records later.


Incidents not workable

Once we create an incident, it just spins for a long duration and its hit or miss if the incident sticks. Once they do stick, meaning we have incidents showing up on our dashboard, they cannot be edited or adjusted in any way. When you click the edit icon next to the incident, it just spins constantly with nothing opening.

Hi @PGSEOC ,
Thanks for reaching out to us. I have received the console logs in an email thread. Could you please confirm whether you are getting the error ("code":"Forbidden","message":"User is not an owner to perform migration of this Group to Team") for all the incidents that you have created so far or does this error pops up intermittently?
If all the incidents were failed with the above error, can you please try to replicate the issue again and share me the console logs and network trace?

Open the TEOC app in browser. Once the app is loaded completely, open developer tools then clear the console and network trace.
Then try to create an incident and after you receive the error message, save the console logs and also export the network trace as Har file. Please share those files for further troubleshooting on this issue.

Also let me know how these below settings are configured in your tenant.

Do you have any Group naming policy configured? (Azure Portal --> Groups --> Settings-->Naming Policy)
Do you have this setting "Users can create Microsoft 365 groups in Azure portals, API or PowerShell" set to No ? (Azure Portal --> Groups --> Settings-->General)
Do you have this setting "Create teams site under" set to /teams/? (SharePoint Admin Center --> Settings --> Site creation )

[Yesterday 3:27 PM] Knigge, Kelly-John W.
{"error":{"code":"Forbidden","message":"User is not an owner to perform migration of this Group to Team","innerError":{"message":"User is not an owner to perform migration of this Group to Team","code":"AccessDenied","innerError":{},"date":"2023-06-01T19:25:46","request-id":"e4b93a03-58ad-4ab9-b31d-da599e51c88b","client-request-id":"f481e83e-6db1-0f0f-1f49-1a1f4786264b"}}}
[Yesterday 3:28 PM] Knigge, Kelly-John W.
{"error":{"code":"NotFound","message":"Failed to execute MS Graph backend request GetGroupInternalApiRequest. The server failed to respond correctly. Response Code: NotFound, Reason: Reason: Not Found, Correlation headers: request-id: 28c546d3-51b9-465b-9f84-97499a53b3d8\r\nclient-request-id: 1784CA05C8834224B796C8F39A06C89D\r\nx-ms-ags-diagnostic: {"ServerInfo":{"DataCenter":"West US","Slice":"E","Ring":"4","ScaleUnit":"001","RoleInstance":"BY1PEPF00002561"}}\r\nx-ms-resource-unit: 1\r\nDate: Thu, 01 Jun 2023 19:25:35 GMT\r\n","innerError":{"message":"Failed to execute MS Graph backend request GetGroupInternalApiRequest. The server failed to respond correctly. Response Code: NotFound, Reason: Reason: Not Found, Correlation headers: request-id: 28c546d3-51b9-465b-9f84-97499a53b3d8\r\nclient-request-id: 1784CA05C8834224B796C8F39A06C89D\r\nx-ms-ags-diagnostic: {"ServerInfo":{"DataCenter":"West US","Slice":"E","Ring":"4","ScaleUnit":"001","RoleInstance":"BY1PEPF00002561"}}\r\nx-ms-resource-unit: 1\r\nDate: Thu, 01 Jun 2023 19:25:35 GMT\r\n","code":"ItemNotFound","innerError":{},"date":"2023-06-01T19:25:35","request-id":"eb77a965-0b9c-4595-a0f4-50a1997d9b48","client-request-id":"f9e707d2-93d2-2e13-80ef-4133599d216b"}}}

Hi @PGSEOC ,
I have received the console logs and HAR file from Avinash. Thanks for sharing those details.
I see from the trace, that the graph call fails for the below URL.
https://graph.microsoft.com/v1.0/sites/tenant.sharepoint.com:/sites/TEOC_10?$select=id
This is because of the managed path setting in the SharePoint Admin center,  the team site would have been created under /teams/ instead of /sites/ and hence it was throwing 404 not found for the above API call.
Please check this setting in your tenant, "Create teams site under" would have set to /teams/ (SharePoint Admin Center --> Settings --> Site creation)
We have fixed this bug in TEOC 2.0 version. So please follow the upgrade instructions from wiki and upgrade the app to 2.0, then it should be good. Thanks!

Thank you, our team said they will run the upgrade this week.
Thanks,
Kelly Knigge, MA, EMT, FPEM
Manager, Emergency Management
Peoples Gas System
O:<PHONE_NUMBER>
C:<PHONE_NUMBER>
<EMAIL_ADDRESS>@.***
@.***
From: v-prigunasek @.>
Sent: Monday, June 5, 2023 9:31 PM
To: OfficeDev/microsoft-teams-emergency-operations-center @.>
Cc: Knigge, Kelly-John W. @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-emergency-operations-center] Incidents not workable (Issue #104)
CAUTION - External Email
***** Don’t be quick to click! We're counting on you! This email is from an external sender! Don't click links or open attachments from unknown sources. To report a suspicious email, click the Forward to Phishing button within Outlook for analysis by our cyber security team. If the button is unavailable, forward the email as an attachment to<EMAIL_ADDRESS>  *****
Hi @PGSEOChttps://urldefense.com/v3/__https:/github.com/PGSEOC__;!!B7pEFg!ka_v-0q0Bz6x2Wj8QNoDTL0p1qS_6Vd7WapbRlz20Gg7kBzqnpmBV5AcAkKSE_Mnl3r_QVMXG-_rOlQIoSATVEaiVcCk$ ,
I have received the console logs and HAR file from Avinash. Thanks for sharing those details.
I see from the trace, that the graph call fails for the below URL.
https://graph.microsoft.com/v1.0/sites/tenant.sharepoint.com:/sites/TEOC_10?$select=idhttps://urldefense.com/v3/__https:/graph.microsoft.com/v1.0/sites/tenant.sharepoint.com:/sites/TEOC_10?$select=id__;!!B7pEFg!ka_v-0q0Bz6x2Wj8QNoDTL0p1qS_6Vd7WapbRlz20Gg7kBzqnpmBV5AcAkKSE_Mnl3r_QVMXG-_rOlQIoSATVBRIcpkQ$
This is because of the managed path setting in the SharePoint Admin center, the team site would have been created under /teams/ instead of /sites/ and hence it was throwing 404 not found for the above API call.
Please check this setting in your tenant, "Create teams site under" would have set to /teams/ (SharePoint Admin Center --> Settings --> Site creation)
We have fixed this bug in TEOC 2.0 version. So please follow the upgrade instructionshttps://urldefense.com/v3/__https:/github.com/OfficeDev/microsoft-teams-emergency-operations-center/wiki/Upgrade-to-Latest-Version__;!!B7pEFg!ka_v-0q0Bz6x2Wj8QNoDTL0p1qS_6Vd7WapbRlz20Gg7kBzqnpmBV5AcAkKSE_Mnl3r_QVMXG-_rOlQIoSATVPip8pZk$ from wiki and upgrade the app to 2.0, then it should be good. Thanks!
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/OfficeDev/microsoft-teams-emergency-operations-center/issues/104*issuecomment-1577761890__;Iw!!B7pEFg!ka_v-0q0Bz6x2Wj8QNoDTL0p1qS_6Vd7WapbRlz20Gg7kBzqnpmBV5AcAkKSE_Mnl3r_QVMXG-_rOlQIoSATVKYCIPbz$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/BAIDDOLBL4H7O5DUEOATOPDXJ2B5BANCNFSM6AAAAAAYXQIJXM__;!!B7pEFg!ka_v-0q0Bz6x2Wj8QNoDTL0p1qS_6Vd7WapbRlz20Gg7kBzqnpmBV5AcAkKSE_Mnl3r_QVMXG-_rOlQIoSATVJ-RghH6$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>
NOTICE: This email is intended only for the individual(s) to whom it is addressed and may contain confidential information. If you have received this email by mistake, please notify the sender immediately, delete this email from your system and do not copy or disclose it to anyone else. Although we take precautions to protect against viruses, we advise you to take your own precautions to protect against viruses as we accept no liability for any which remain.

Update did not fix this. I created a new issue...


Fix CheckboxWithLabel integration with Form

Problem
While refactoring AddDebitCard  to use our Form component, I noticed that CheckboxWithLabel only works when the shouldSaveDraft prop is passed.
Why is this important
The component should work with Form so it can be used in our refactors
Solution
Make it work without passing shouldSaveDraft

I'm focused on the NewDot API issues, but will get to this next.

Focused on API refactors. Gonna try to get to this next week.

Focused on API refactors. I'll try to get to this next week.


Error when loading valid draft7 json schema

When trying to compile a valid draft7 json schema like this one https://github.com/usnistgov/OSCAL/releases/download/v1.1.0/oscal_catalog_schema.json the tool returns an error as it cannot reference find valid ids on the schema.
After some troubleshooting, I think this might be related to here as in these cases, the definition both have an id and a reference to a common expression.
I already confirmed that changing it to if _, ok := m["$ref"]; ok && d.version <7  would work, but I'm not sure the impact this would have on this tool

EXPECTED_BEHAVIOUR
$ref keyword does not allow other keywords in draft7. it allows from draft 2019 onwards:
see  https://json-schema.org/draft/2019-09/release-notes.html#core-vocabulary
there is also test for this in draft 7: check this
the mentioned schema is using $id and $ref as siblings in draft07, which is not allowed. in draft07, if $ref is present, we ignore any other sibling keywords.


antedeluvians

Submitter: Eli
10 point version for npcs plz?

Thanks for the added input as far as user interest. This is a dupe, similar to what is requested in #174.
It is noted, but specifically this issue will be closed.


⚠ ClamAV has degraded performance

In d625260, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 80 ms

Resolved: ClamAV performance has improved in a2ef562.


Search in-progress UX

testing notes
designs desktop
de-prioritizing mobile/tablet for now because they involve adding the animated icon to the actual text fields so that it's visible higher up on the page; will come back to that later.
the animation and "Results are loading" text should appear when a search is in progress, fading back to the total results/zotero link when the results have loaded.
dev notes
I'd like to use animation, text, or some form of visual feedback to make it more clear to users when a search is in progress vs. when results have been loaded on the main archive search.
Maybe we can design something relatively low-impact that can make it into MVP.

I agree-- sometimes I can't tell that my results have updated. Some visual indicator would be helpful.

This is a wonderful idea! It's really helpful to the UX and clear. It actually works already on tablet too.
This is really granular, but on Firefox only it's a little more glitchy. On some keyword searches it'll flash, then go away and show the total number (4,946), then flash again, then go away and show the new number. Also when you clear a text field to "reset" the search, it flashes twice. On Chrome it works really smoothly.

this behavior is tricky to replicate for me, but I changed some simple things that I think might have resolved it. if not, we probably want to push it to icebox for awhile.

Keyword search and field clearing in FF are working a lot better now. Author is still a little flashy but it's fine.


Update cross-region-replication-introduction.md

I have a customer who requested we clearly state that the destination volume is read-only until failover in our documentation. I realize it's on the fail-over page, but it may be good on this page too as this explains CRR versus the how-to.
"Note: * The Azure NetApp Files destination volume is read-only until you fail-over to your destination region enabling the destination ANF volume for read and write. "

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

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

#hold-off

#hold-off

@aainyz - Thanks again for the input. The ANF PM agreed to adding your suggested change/reminder. However, the CRR Introduction article is not the suitable place for this info.  The Intro article is meant for high-level product info (supported regions, SLO, cost model, etc.) and not clarification of feature behavior.
A more suitable place for your suggested reminder is the CRR Considerations article, where we list behaviors and gotchas that users should know about (https://docs.microsoft.com/en-us/azure/azure-netapp-files/cross-region-replication-requirements-considerations ).
I'm adding the following bullet in the CRR Considerations article:

The replication destination volume is read-only until you fail over to the destination region to enable the destination volume for read and write.

This change is in PR 143772 (https://github.com/MicrosoftDocs/azure-docs-pr/pull/143772/files ). But due to system issues that are still being resolved, I'm not able to submit the PR.  Will do so when I can.    Thanks.

We'll close the PR. Thanks, all.
-PR review team

We'll close the PR. Thanks, all.
-PR review team


Fake/stub for Stored Procedures and Functions

It will be useful to be able to fake and/or stub stored procedures or functions to test some parts of your BI solution.
Stub: temporary replace SP/function by a fix result set
Fake: temporary replace SP/function by another T-SQL code.

WIll probably be included in v1.11, already started the development in branch "feature/Stub & Fake"


Develop checklist for build Conveyor release for multiple Elastic releases

As a core developer, I need a process such that I can accurately and repeatedly create multiple different intestances of a given release of Conveyor, such that they are configured for different versions of the Elastic stack

Goal of this issue is just to create the steps that you would replete to build multiple Elastic stack version - a likely future goal would be to build automation to do that.  To prepare for the future goal we want to collect information about any automation short-cuts - specifically interested in information on the ELK built in version setting functionality that @malave though he saw.

Talked to @malave and he pointed at these docs for the functionality thats built into kibana: https://github.com/elastic/kibana-plugin-helpers/blob/master/README.md#settings-for-build

Work in progress here https://github.com/samtecspg/conveyor/blob/master/docs/_developer_docs/new-build-version.md


Python parser

First draft for changes related to python unittest log parser
Pending:

More Unit test cases
Telemetry and Diagnostic logging.

@ShreyasRmsft Reacted to all your comments. Pushing this in, if you have any comments, I'll take them as part of next PR.


impl logic of self matching prevention

Description
impl logic of self matching prevention
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

In the current implementation of this code the prevention of self trading only cancels the last order placed that would cause it to self trade. Should we consider an implementation where both the bid / ask also get taken down? Or are we just assuming a user error here?

Can we also test the scenario where we have limit buy orders placed by Alice and then she tries to market sell? Then I think this covers all code paths.


Add github actions

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@dde02e2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master       #10   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         1
  Lines             ?        12
  Branches          ?         0
==========================================
  Hits              ?        12
  Misses            ?         0
  Partials          ?         0

Flag
Coverage Δ

unittests
100.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Add ability to play music through Spotify

Wraps up https://github.com/hnarayanan/shpotify into the plugin. 🎧  /cc @hnarayanan

Not exactly sure what this means but it's all very exciting!

This is so great, but I'd love this to be its own plugin, instead of adding a new parameter 👍

Move it to its own plugin! https://github.com/ohwutup/fastlane-plugin-spotify


chore: sync with main repo

chore: include main repo's changes
chore: update packages

Thanks for taking care of this.


Hyprpaper IPC not finding files

This issue seem to be with hyprpaper IPC command preload to find files.
echo 'preload ~/path/to/file.jpg'|socat - "/tmp/hypr/$HYPRLAND_INSTANCE_SIGNATURE/.hyprpaper.sock"

Expected output:
ok

Actual output;
no such file: /home/user/path/to/file.jpg'

Keeping in mind the path it returns as not found is actually the path to an existing file.
There are no issues with wallpaper and unload commands

I was able to resolve this issue in a project here using zig lang and from what I've seen it should be working fine. I am still unsure as why it is not working with socat but I think this issue in not relevant to hyprpaper hence I'm closing it.


feat: iOS preparing video progress

Hi,
We are using file picker to pick media files (video and picture).
In a previouse version we use the html input file to select media files from an iOS device and iOS on some file let show a progress dialog for preparing file. Now we are using the file picker with capacitor 4 (to have native path) and on iOS we can't see anymore the system progress dialog box.
I add a spinner starting at file picker call and ending at then or catch of the picker promise, but I can't show any progress percentage.
Is there a way to enable in file picker the system progress dialog?
Or to have a call back about preparing progress?
To show to final user the app is not blocked and is going on preparing file will be fine to avoid user to stop the app thinking to the app blocked.
Thank you in advance.

Thank you for the feature request.

Is there a way to enable in file picker the system progress dialog?

Unfortunately not. To be honest, I've never seen it before (or I didn't notice it). Can you share a screen recording?

Or to have a call back about preparing progress?

I definitely plan to implement this but it is not the highest priority at the moment (sponsoring the issue would be possible) as there are still some bugs.
For now I recommend showing a loading box without a percentage until the promise is resolved.

@robingenz I'm currently having this issue now where say I select 4 videos (each around 10 seconds long).  The picker dismisses and then it take a while for await FilePicker.pickMedia to resolve.  For now I'm just adding a spinner during this time but I noticed that when selecting images/videos via an html input type="file" on iOS, it displays a preparing loader within the picker and then dismisses.  Wondering if it's possible to do the same or if there is a way to speed this up.

Wondering if it's possible to do the same

As i said I definitely plan to implement this but it is not the highest priority at the moment as there are still some bugs.

or if there is a way to speed this up

Unfortunately not.

Unfortunately it look like the privacy enforcement of iOS return a virtual
url and the copy to temporary file at the end of file picking is the only
way to get access to native file.
Unfortunately native file manager has no progress callback.
I try to access directly to the returned url but it doesn't work, url
doesn't exist.
Finally I think the first implementation I did using html input file and
saving native file in chunk is the best possible solution for iOS, for
android OS the plugin is really nice and working really well.
This time iOS and it's privacy enforcement win against our apps, all
example from apple are about using picker selection directly in a swift
object..
Thanks Robin for the really good job you did.
--------------------------
dr. Massimo Liani
Goooled s.r.l.
Via Torricelli, 26
33080 - Porcia - PN
tel:3884484284
Il giorno sab 28 gen 2023 alle ore 15:38 Kyle Abens <
@.***> ha scritto:

@robingenz https://github.com/robingenz I'm currently having this issue
now where say I select 4 videos (each around 10 seconds long). The picker
dismisses and then it take a while for await FilePicker.pickMedia to
resolve. For now I'm just adding a spinner during this time but I noticed
that when selecting images/videos via an html input type="file" on iOS, it
displays a preparing loader within the picker and then dismisses. Wondering
if it's possible to do the same or if there is a way to speed this up.
—
Reply to this email directly, view it on GitHub
https://github.com/capawesome-team/capacitor-file-picker/issues/83#issuecomment-1407412004,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEC6WKEEKZUXBNILX2OZC63WUUVM3ANCNFSM6AAAAAAUHWA3NE
.
You are receiving this because you authored the thread.Message ID:
@.***>

would be best if this "progress" could also include the time it takes for files offloaded from the phone & only available in iCloud take to download before they get copied over to the app's storage.
This issue might be a solution to #92 as well.

There is now a new pickerDismissed listener that may already help some of you.
It will be called as soon as the picker modal disappears. You can then display a hint to the user until the promise is resolved.
The PR is still open, but there is already a dev build: #93

I'd love to see this behaving the same way as Cordova.  I'm currently migrating to Capacitor and was actually surprised this wasn't the same, as the "Converting..." progress bar appears to be native ios.
Without it, even with a spinner, the app feels much less responsive and I'm worried users will think it's frozen if they've opened a large video that takes 30 seconds or so to process.
Here's a video of how it behaves under Cordova using cordova-plugin-camera:
https://youtube.com/shorts/XvtaOKijLRY


Layer option

Thanks for developing this plug-in!
I just want to ask for an enhancement option, can KiBuzzard generate other than on the "F.SilkS' or 'B.SilkS' layer?
Sometimes I use label/polygon on the "F.Mask" layer above the copper pour.
Regards,
TJ.

I agree, sometimes on copper it is also required.

Layers are implemented now https://github.com/gregdavill/KiBuzzard/pull/73


Remove coverage profile from service pom

This PR updates the service poms to remove the coverage profile. The pom file required to create aggregate coverage report per service is now generated in the pipeline. So, the service poms can go back to being track agnostic and contain only the list of modules.

/azp run java - core

I'm going to override check enforcer. The resourcemanagerhybrid libraries and incorrect triggers caused the pipeline that should not have run to get run. Those triggers were fixed as part of this PR (thank you @srnagar)


If project has no image, dashboard tile squashed

Need spacing above the 'Manage' button.

v0.9.07


Is the order of virtual_hosts important for domain matching?

There is some confusion in the Istio community about if the ordering of virtual_hosts in a config.route.v3.RouteConfiguration will affect which route is matched.
While the envoy docs are clear that route is matched in-order, it's unclear about if the ordering of virtual_hosts will affect the order of domain matches.
I only see two possibilities:

Envoy matches the request host header against each  virtual_host by domain in-order

(meaning if the first virtual_host has a wildcard subdomain, any later specific subdomains will be effectively shadowed by the wildcard)

Envoy matches the request host header by using the virtual_host with the most precise domain match

Example:
What happens to traffic destined to http://app1.example.com, does it end up at outbound|80||app2.default.svc.cluster.local or outbound|80||app1.default.svc.cluster.local?
- name: http.80
  validateClusters: false
  virtualHosts:
    - domains:
        - '*.example.com'
        - '*.example.com:80'
      includeRequestAttemptCount: true
      name: '*.example.com:80'
      routes:
        - decorator:
            operation: app2.default.svc.cluster.local:80/*
          match:
            prefix: /
          metadata:
            filterMetadata:
              istio:
                config: /apis/networking.istio.io/v1alpha3/namespaces/default/virtual-service/app2
          route:
            cluster: outbound|80||app2.default.svc.cluster.local
            maxGrpcTimeout: 0s
            retryPolicy:
              hostSelectionRetryMaxAttempts: "5"
              numRetries: 2
              retriableStatusCodes:
                - 503
              retryHostPredicate:
                - name: envoy.retry_host_predicates.previous_hosts
              retryOn: connect-failure,refused-stream,unavailable,cancelled,retriable-status-codes
            timeout: 0s
    - domains:
        - app1.example.com
        - app1.example.com:*
      includeRequestAttemptCount: true
      name: app1.example.com:80
      routes:
        - decorator:
            operation: app1.default.svc.cluster.local:80/*
          match:
            prefix: /
          metadata:
            filterMetadata:
              istio:
                config: /apis/networking.istio.io/v1alpha3/namespaces/default/virtual-service/app1
          route:
            cluster: outbound|80||app1.default.svc.cluster.local
            maxGrpcTimeout: 0s
            retryPolicy:
              hostSelectionRetryMaxAttempts: "5"
              numRetries: 2
              retriableStatusCodes:
                - 503
              retryHostPredicate:
                - name: envoy.retry_host_predicates.previous_hosts
              retryOn: connect-failure,refused-stream,unavailable,cancelled,retriable-status-codes
            timeout: 0s

domain order does not matter.
@hzxuzhonghu is correct.


Upgrade to TypeScript 3.7

https://devblogs.microsoft.com/typescript/announcing-typescript-3-7/
Unfortunately, we will not be able to use the exciting new features
such as optional chaining and nullish coalescing until Node.js
supports them natively as we use a third-party compiler (sucrase)
rather than tsc to ensure fast build times.

Is that necessary? I just meant /** @typedef {import('../sim/prng').PRNG} PRNG */

Is that necessary?

No, that was just what I understood from your review.


Make naming conventions for namespaces, labels and other args consistent across utils

Is this a BUG REPORT or FEATURE REQUEST?
Choose one: BUG REPORT or FEATURE REQUEST

What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

cc: @shubhamchaudhary , can this be closed?

The experiment mandatory variables have been standardized over the past few releases.


add Bellcoin

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

All checks have passed BUT 1 neutral check :p please help!

I am going through the same, all check passed but I am not seeing any activity on it, I thought this would return with a fix or approval by now...

You need to open a New issue and link your PR to let Solana Developers manually merge it!! Hope this did help you!! If you do have any kind of questions, don't hesitate writing me!!

Your error is this: invalid asset path: assets/mainnet/BellLogo.png. Fix it and try again!!


Feature:  Bars

@bmcfee had some thoughts about this - I feel like we said that we would not use librosa for this?

This feature can be obtained by matching algorithm：
1. create a variety of tempo template set, for example, 2/4 template is {strong, weak, strong, weak, strong, weak}, 4/4 template is {strong, weak , The second strong, weak}；
2. use total beats   to match every templates and the one with highest match-rate maybe the music's time signature  , so that each cycle is a bar.

As per an email from @bmcfee, we can use an incoming Python lib from McFee Labs(tm) for this.  ETA is July 2017 or so.