sfx not seeing uploads

Hello. Thank you so much for porting this to V3. The problem I am having is that when I upload an audio clip, I get an error saying "Error in command 'addsfx'. Check your console logs for details". The console will say something about the file not being found. The clip does get uploaded but if I try to play it the cog will say it doesnt exist. If I try the "allsfx" command it will tell me there are no files.
Thank you for your time.

Hey there. Can you post the error from the logs during upload? And which version of Red are you using?

I am using Red v3.1.6. Here is the log when I upload.
[2019-10-20 12:18:33] [ERROR] red: Exception in command 'addsfx'
Traceback (most recent call last):
File "c:\python 38\lib\site-packages\discord\ext\commands\core.py", line 79, in wrapped
ret = await coro(*args, **kwargs)
File "C:\apps\botbuffer\cogs\CogManager\cogs\sfx\sfx.py", line 214, in addsfx
audio_data = pydub.AudioSegment.from_file(filepath, format=file_extension[1:])
File "C:\apps\botbuffer\cogs\Downloader\lib\pydub\audio_segment.py", line 665, in from_file
info = mediainfo_json(orig_file)
File "C:\apps\botbuffer\cogs\Downloader\lib\pydub\utils.py", line 263, in mediainfo_json
res = Popen(command, stdin=stdin_parameter, stdout=PIPE, stderr=PIPE)
File "c:\python 38\lib\subprocess.py", line 854, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "c:\python 38\lib\subprocess.py", line 1307, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified
Also when the cog is loaded I get an error about ffmpeg not being installed.
C:\apps\botbuffer\cogs\Downloader\lib\pydub\utils.py:165: RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work
warn("Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work", RuntimeWarning)

Can you try installing ffmpeg? It's needed, because the bot has to add padding to every sound you upload to circumvent a limitation in the audio player of Red.
Uploading works fine for me.

Where is it looking for it?

Anywhere that's in your OS' search path. Look here: https://github.com/jiaaro/pydub#getting-ffmpeg-set-up

Adding ffmpeg to PATH worked. Thank you.
On a side note do you plan on adding the feature the cog used to have where if you typed part of a name it would randomly play a clip that started with those letters?

I had no idea the original cog had that functionality. I'll probably add it at some point in the future, but I don't really have time to work on my cogs at the moment.
PRs are welcome, though 😛


Missing reference to more useful secret function

Hi
I was just working my way through this article and trying to implement in Synapse and I feel that the getSecretWithLS function should be documented here.
At the moment the documentation calls out the getSecret function which requires the user to provide the key vault name, secret name, and optionally the linked service name. This is counter intuitive when previously in the docs it advises the user to create a linked service. To remedy this it would be useful for the documentation to draw attention to the getSecretWithLS method which takes the linked service name and the secret name. So something more like
import com.microsoft.azure.synapse.tokenlibrary.TokenLibrary

val secretValue: String = TokenLibrary.getSecretWithLS("<LINKED SERVICE NAME>", "<SECRET KEY>")
println(secretValue)

This allows for a more repeatable pattern to be implemented in notebooks allowing for a smoother delivery to production.

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

ID: 56d638cd-6592-ce95-90aa-dc8d6e4e90ba
Version Independent ID: 5360961f-6bfc-ecad-cf49-22eee26bd010
Content: Secure access credentials with Linked Services in Apache Spark for Azure Synapse Analytics - Azure Synapse Analytics
Content Source: articles/synapse-analytics/spark/apache-spark-secure-credentials-with-tokenlibrary.md
Service: synapse-analytics
Sub-service: spark
GitHub Login: @mlee3gsd
Microsoft Alias: martinle

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

We have assigned the issue to the content author . They will evaluate & update the document as appropriate

Apologies, as an update to this it's probably worth looking at how this is presented. Buried in one of the other pages is a link to the Microsoft Spark Utilities, which provides a wrapper over the top of the TokenLibrary and a lot of other useful features. This should probably be the preferred method of accessing secrets.

@himanshusinha-msft has there been any movement on this at all?

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


IG scraping limits

Does anybody keep track of scraping limits? How many requests per day can be made to internal api (e.g. stories) without getting blocked, using normal proxies (not residential)?

I've unfollowed 50 users every 30 to 60mins (random) until it reaches 500 unfollowed users on a day. When following users I do 25, same range, until 200 followed users on a day but, it is a little more tricky because sometimes IG ask you to complete a challenge which I haven't learn how to do yet. Also I try not execute more than 500 IG requests also on a day. Hope that experience helps.


a fix to AppID/FaceID handling on client app side issue #5

Hi Neb,
A suggestion to check the Trusted FacetIds

Hi Emerson,
I'll take a look and try to respond as early as of tomorrow.
Regards,
Neb.


Add all sub public.text

Many of the text files (like .js, .py, .sh) don't get a ql preview if any package associates with it but doesn't provide a ql plugin (ex. atom). So I've written a script which extracts all the UTIs which conform to public.text & public.unix-executable (with a few exceptions). I've added all of them to the supported UTIs.
This will effectively break qlhighlight for users who also use qlstephen, so the users will have to choose one of the two.

@tsdorsey @whomwah


cmd/analyze usability improvements

print errors for missing flags
better use of exit codes
[internal] consoliate context object initialisation

I think the headline news of this change is that you're now returning a nonzero exit code if the command fails; that should probably be in the commit subject line.

fair point, I'll add this to the final commit message and/or PR heading


Thanos expects rough play

Thanos creates a ConfigMap named user-workload-monitoring-config in the namespace openshift-user-workload-monitoring and expects users to modify it. The syncer avoids modifying it because that object lacks the annotation that the syncer uses to mark objects managed by the syncer. The syncer is trying to play too nicely.

@MikeSpreitzer is this issue closed as a matter of #1103 and #1105 merging?  Is there more to do?

Implemented by #1103 and #1105

Reopening to allow for a test first with upcoming release.

we are now using customization to achieve the post-edit.


[tests] unexpected fails

Describe the bug
Some tests fail even if the PR doesn't have any logical changes.
Besides manually rerun, are there anything we can do to enhance the stability of tests?
The typo fixes in #5446  should not cause any fails, however below fails are observed:

toranj-ncp: exit status 11

packet-verification Cert_5_5_02_LeaderReboot

cli-ftd test_router_reattach

Expected behavior
Consistently pass if no bugs newly introduced in the PR.

@jwhui @abtink @simonlingoogle @FiveDimensions

Another packet-verification 5.5.2 fail: https://github.com/openthread/openthread/pull/5425/checks?check_run_id=1021503850
Could you take a look? @FiveDimensions

Traceback (most recent call last):
------------------------------------------------------------------------------------------------------------------------
  File "test-001-get-set.py", line 47, in <module>
Starting 'test-001-get-set'
    wpan.Node.init_all_nodes()
Took too long to init node Node (index=1, interface_name=wpan1) (15.0565950871>15 sec)

Should toranj increase the timeout? @abtink

@simonlingoogle @FiveDimensions @abtink , I am experiencing an increase in transient failures. Any thoughts on what the issue might be?

@librasungirl , @simonlingoogle , regarding the toranj failure, we have seen this issue and have spent a lot of time on trying figure it out in past (when we were using travis CI). Based on the info this seems to be due to an issue (bad install) in dbus library itself (we have seen similar situation on actual devices as well). Increasing test timeout or increasing number test attempts won't impact and help in this case. A re-run (which re-builds and re-installs everything) seems to be the main option here.
I agree that we want to minimize the CI unreliable failures, but based on my observations the tornaj workflow is not failing that often compared to other transient failures.  For reference, the CI actions on my fork of OT, for the past ~20 branches/commits (~last week), I had to re-run simulation workflow about ~7-8 times, build workflow ~2-3 times but do not recall any re-kick of toranj. It may be good if we can collect info/stat about failures (on main fork, or may be on developer's fork (where we can have more control on tracking failures and re-kicks of CI workflows)).

I have submitted #5479 so that it is possible to investigate these issues using the failed log and PCAP.
Please take a look when you have time.

#5479 is a great idea to archive the test artifacts. Checked the case 5.5.2 test logs, failed in Step 9 Leader reattach and MLE Parent Request packet can not be found caused by packets order, enhanced this case in #5490 and rerun more than 5 times and no related failure found. Will keep on rerunning > 10 times then open to review


File without extension return 404

Are you accessing the CLI from the default port :4280 ?

[ ] No, I am using a different port number (--port) and accessing the CLI from that port
[x] Yes, I am accessing the CLI from port :4280

Make sure you are accessing the URL printed in the console when running!

Describe the bug
When fetching a file that doesn't have an extension (eg. foo), the CLI returns a 404.
To Reproduce
Steps to reproduce the behavior:

Create an HTML file with the following content:

foo:

Create an empty file foo (without an extension!)
Run swa start ./ --verbose silly
See error: file foo isn't found.
Accessing http://localhost:4280/foo also returns a 404

Expected behavior
The file foo should be fetched and accessed successfully.
Additional context
The issue seems to come from this function:
https://github.com/Azure/static-web-apps-cli/blob/bf4fbcb0e6af12695b8f24b886098be8df9e7c9c/src/msha/routes-engine/rules/routes.ts#L16-L35
Here are the logs from the swa command.
[swa] --------------------------------------------------------
[swa] ------------------- processing route -------------------
[swa] --------------------------------------------------------
[swa] processing /foo
[swa] checking for matching route
[swa] checking auth request
[swa]  - not an auth request
[swa] checking function request
[swa]  - not a function request
[swa] checking HTTP method: GET
[swa]  - method is valid (allow-list: GET,HEAD,OPTIONS)
[swa] checking for query params
[swa] checking rewrite auth login request
[swa] checking rewrite auth logout request
[swa] checking authorizations for route
[swa]  - no matching rule
[swa]  - access authorized
[swa] using userConfig
[swa]  - userConfig: <undefined>
[swa] checking storage content
[swa] finding file for request
[swa]  - requestPath: /foo/index.html
[swa]  - filePath: /Users/wassimchegham/swa-cli-issue-test/foo/index.html
[swa]  - exists: false
[swa]  - filePathFromRequest: null
[swa]  - shouldDisplayNotFoundPage: true
[swa] GET http://localhost:4280/404.html - 404
[swa]  - url: /404.html
[swa]  - target: /Users/wassimchegham/swa-cli-issue-test/
[swa] customUrl: false
[swa] is4xx: true
[swa] checking if file exists in user's output location
[swa]  - file: /Users/wassimchegham/swa-cli-issue-test/404.html
[swa]  - exists: false
[swa] checking if file exists in CLI public dir
[swa]  - file<EMAIL_ADDRESS>[swa]  - exists: true
[swa] serving static content
[swa]  - file<EMAIL_ADDRESS>[swa]  - url: /404.html
[swa]  - code: 404

I can take it 😊

https://github.com/Azure/static-web-apps-cli/pull/293 - Fixed with this


Upgrade Rust toolchain to 1.59.0

I'm looking to use destructuring assignment (which was stabilized in 1.59.0: https://github.com/rust-lang/rust/issues/71126) in a branch I'm working on.
Given that upstream reports this fixed by an XCode update from Apple, removing WIP and I'll get this ready to merge.


Partial update to 0.16.1

0.16.1 Changes
fast-underground 0.5>2s
express-underground 0.5>2s
locomotive 0.5>4
cargo-wagon 0.5>1
car 0.5>2
tank 0.5>5
roboport 10>5
nuclear-reactor 4>8
heat-exchanger 0.5>3
heat-pipe 0.5>1
oil-refinery 10>8
lab 3>2
explosives now craft x2
satellite 3>5
production science no longer needs assem 1
Still needed:
All the new items: Cliff Explosives, Nuclear Fuel, Artillery (wagon turret shell targeting-remote)

And now I see dump.lua /facepalm


[DO NOT MERGE][NEXT?] Unsubscribe source observable immediately after it completes or errors

Disclaimer:
For discussion.  ;)
Problem:
Currently many operators still subscribe for some time to source observables after they completed, and thus do not call teardown logic immediately after completion. These kind of marbles are in many tests:
---a-----b-----c-----|               <- source observable
^------------------------------!    <- subscription to that observable

It is a source of very subtle bugs, like: #2455
It also seems that this goes against tc39 proposal, which guarantees that source will be unsubscribed whenever it sends complete or error notification to an observer: https://tc39.github.io/proposal-observable/#subscription-observer-objects
There are operators that have to manually trigger their unsubscription to normalilze their behaviour: https://github.com/ReactiveX/rxjs/blob/master/src/operator/take.ts#L80

My personaly suggestion is starting for issue to discuss details first for these kind of cases, as it is not ceratin what desired behavior we want to have, this PR probably stall very long time and either be closed or required lot of updates in further.

(But not saying this should be closed now since it's already prepped)

Coverage increased (+0.004%) to 97.692% when pulling 270f2ada8bd2bc715cce77d49779ee1b104b7786 on Podlas29:subscriber-fix into 69d051bc56cb337148db30515029363c1e78b90d on ReactiveX:master.

@kwonoj Will have that in mind, but I wanted to see if my planned fix will work and won't break stuff it shouldn't.

@Podlas29 if you don't mind, would you create issue? It's better to make discussion continues on issue, not on PR.

@Podlas29 the spirit of the PR seems right, but allocating an extra Subscription per Subscriber shouldn't be necessary. We essentially hook up the "parent" subscription here, so I have a hunch we can update this PR to use that directly.

@kwonoj https://github.com/ReactiveX/rxjs/issues/2459

Coverage increased (+0.02%) to 97.707% when pulling 1c5282a1fabf413f86c475ffed600a3ded4a6a4f on Podlas29:subscriber-fix into 69d051bc56cb337148db30515029363c1e78b90d on ReactiveX:master.

Coverage increased (+0.02%) to 97.707% when pulling 1c5282a1fabf413f86c475ffed600a3ded4a6a4f on Podlas29:subscriber-fix into 69d051bc56cb337148db30515029363c1e78b90d on ReactiveX:master.

This PR will require a LOT of review. The behavior seems right-ish, on the surface, and the spirit of the PR seems solid. But there are a lot of questions and even more changes to review.
Most of our tests were modeled after the tests that existed in Rx4 and under. So many of these changes may incur breaking changes for the historical use of the library.
It also looks like (at a quick glance) that we're adding another Subscription reference per subscription, as well as a few new publicly available properties and methods.
Questions:
How does this affect performance?
How about memory pressure?
Who will this break?
Is this fixing a problem that many people are affected by? Is it really a problem?
There are just so many behavioral changes here, it's really going to be hard to tackle this and give it the proper attention it deserves in any reasonable amount of time.

As an aside: the format of your commit messages is incorrect.
Perhaps it would be better to file each bug you think is a bug separately with it's accompanying test from this PR, so each one could be discussed individually?

@benlesh - please take a look at #2459 In following days I will try to flesh out the problem more, provide more examples as well as alternative plans for fixing it in a more controlled/gradual way.

Okay, I think this PR should be redone and added as a bug fix during 6.0. It's not really a "breaking" change to anything but a broken behavior, IMO.

Closing in favor of #3963, which is an attempt to pull this in.


cy.msalLogin is not a function

I was getting the aforementioned error and I added import 'cypress-msal-login' to cypress/support/index.js. After adding that I then got this error: SyntaxError: 'import' and 'export' may appear only with 'sourceType: module'. Please kindly help me out.

Same here. Is there any missing configuration in the documentation? thanks

Tried adding import 'cypress-msal-login' to cypress/support/index.js file and it gives now a different error:
Command:  wrap
cypress_runner.js:194914 Error:    TypeError: Cannot read properties of null (reading 'get')

Probably related to this?
cy.wrap(null).then(function () { return __awaiter(_this, void 0, void 0, function () {
            return __generator(this, function (_a) {
...

Not sure exactly what cy.wrap(null) is used for

Same. Can not use this

@andreaslarssen @soajetunmobi @ivantrave I updated the library to version 1.1.0 and removed the cy.warp(null). It's tested with latest Cypress version 12.8.1.
Please let me know if you still face the issue.


UPDATE - Header in articles.show and users.profile view to be boxed

Issue https://github.com/laravelio/laravel.io/issues/1122

This looks really good already. I'm wondering if we can pin the "back to articles" more to the top so the left and top padding is the same. And also pin the rest to the bottom of the box so the left and bottom padding is the same. For the article header that is.

@tauseefsshah looks perfect really.
Actually I don't remember. I think we can re-add it now. Can you do that? Then this PR is gtg 🚀

Amazing, thanks for your help! 👏


Removed any extra miqSparkle On/Off calls from jquery requests.

Removed any extra miqSparkle On/Off calls from jquery requests that were added in this https://github.com/ManageIQ/manageiq/pull/1225 PR.
#1150
@martinpovolny please review

Taking this one.

@h-kataria : in function miqObserveCheckboxes() { the sparkleOn/sparkleOff logic is gone. Is that ok?

Other than the above question I did not find any inconsistencies.

@martinpovolny added code back to miqObserveCheckboxes to turn spinner on/off when it is set in check_box_tag properties.

@martinpovolny i am working on testing/creating a PR to remove miqObserveCheckboxes method(this method was a workaround method to send up checkbox transactions to server as UJS binding didn't work with dhtmlxtabs) and get UJS binding to work for checkboxes.

@h-kataria : ok, so we can merge this?

@martinpovolny yes if everything else looks good to you.

yes, all else looks good, so waiting for travis...


Add remote peer info

Motivation:
The core package added remote peer info to the server context. We should provide that info from connected channels.
Modification:

Set the remote peer info
Add tests

Result:
More insight into remote peer

API breakage check fails to build as the API in the core package changed so the current version of main in this package won't compile (as it depends on main in the core package).


Build and push official docker image

📁 Related issues

#1111
#1053
#728

✍ Description
This PR adds a docker job to the ci.yml workflow.
It should only trigger if the following jobs have been executed:
needs:
      - lint
      - test-unit
      - test-integration
      - test-integration-windows
      - validate-components

and if the ref starts with refs/tags/v.
For now, this should enable building and pushing of tagged versions following semver style.
Example:
Github tag: v1.2.3
Docker tags:

1
1.2
1.2.3
latest (overwrites previous latest tag)

The necessary secrets should have been added by @RubenVerborgh.
This temporarily adds some manual versions to immediatly build and push the current image
✅ PR check list
Before this pull request can be merged, a core maintainer will check whether

[ ] this PR is labeled with the correct semver label

semver.patch: Backwards compatible bug fixes.
semver.minor: Backwards compatible feature additions.
semver.major: Breaking changes. This includes changing interfaces or configuration behaviour.

[ ] the correct branch is targeted. Patch updates can target main, other changes should target the latest versions/* branch.
[ ] the RELEASE_NOTES.md document in case of relevant feature or config changes.

Super! Let's also add a link in README and a CHANGELOG entry.

So first manual build is online at https://hub.docker.com/r/solidproject/community-server
I have updated the readme on docker hub to just contain some generic information and the docker section, for more details it refers to the main repo.

@Falx let me know if you would already merge this PR now or if there are still things that you want to add.

I would be ok with merging this now, than add some finegrained adjustments on the main branch directly (without PR) for the other tags (dev, nightly, next). If that is ok for you?

I would be ok with merging this now, than add some finegrained adjustments on the main branch directly (without PR) for the other tags (dev, nightly, next). If that is ok for you?

Sounds good. Would still go through a PR if you're not sure about a certain choice.
Can you also add a line to the RELEASE_NOTES.md on the 3.0.0 branch then?


fix issue with company renaming

fix the bug in "her" gem by continuing the patching.
The error is the original gem is due to the next call that does not returns the hash
that hash is then given in the inject method as first argument, and since it is nil, cause an error

Good catch.
It'd be nice to make more obvious what was patched as it's hard to see without looking at the source. It's the usual problem with monkey patching as we often copy/paste the original implementation.
Maybe something like:
        # @private
        # TODO: Handle has_one
        def embeded_params(attributes)
          associations[:has_many].select { |a| attributes.include?(a[:data_key])}.compact.inject({}) do |hash, association|
            params = attributes[association[:data_key]].map(&:to_params)
            # PATCH - Return hash
            next hash if params.empty?
            # EO PATCH
            if association[:class_name].constantize.include_root_in_json?
              root = association[:class_name].constantize.root_element
              hash[association[:data_key]] = params.map { |n| n[root] }
            else
              hash[association[:data_key]] = params
            end
            hash
          end

What do you think?

I agree,  I'll update.


Add stats subcommands with 'demux'.

Description
Add subcommands stats and sub subcommand demux. I think we will need some very simple statistics on split size or something like that, hence the creation of the stats subcommand.
quickwit stats demux --index-id hdfs_logs

Note: I did not find a maintained simple crate for descriptive statistics, if you have any names, I would welcome it.
How was this PR tested?
Locally only. No unit test added. Waiting first for feedback before adding some simple unit tests.
Screenshot of output

I decided to use the colored crate everywhere as:

the API is really nice for setting colors
by default, it detects automatically if the output is a terminal. The user can also disable color by setting the env variable NO_COLOR.
the crate follow nice standards: https://no-color.org/ and https://bixense.com/clicolors/

@evanxg852000 can you make the last check on this small PR? thanks!


rememberViewModel throw NotFoundException when you pass Kotlin function as an args

says we have a ViewModel and a function as the argument.
class SomeViewModel(private val fooArg: (Foo) -> Bar) : ViewModel()

val SomeViewModelModule: DI.Module
    get() = DI.Module(name = "module name") {
        bindFactory { fooArg: (Foo) -> Bar ->
            SomeViewModel(fooArg = fooArg)
        }
    }

when you try to retrieve the ViewModel via Kodein rememberViewModel it will throw NotFoundException due to it expects the args passed as a Function1<*, *>
org.kodein.di.DI$NotFoundException: No binding found for SomeViewModelModule, with argument Function1<*, *>
    at org.kodein.di.internal.DIContainerImpl.factory(DIContainerImpl.kt:208)
    at org.kodein.di.DIContainer$DefaultImpls.factory$default(DIContainer.kt:32)
    at org.kodein.di.internal.DirectDIBaseImpl.Factory(DirectDIImpl.kt:18)
    at org.kodein.di.compose.android.KodeinViewModelScopedFactory.create(KodeinViewModelScopedFactory.kt:18)
    at androidx.lifecycle.ViewModelProvider.get(ViewModelProvider.kt:187)
    at androidx.lifecycle.ViewModelProvider.get(ViewModelProvider.kt:153)
    at androidx.lifecycle.ViewModelLazy.getValue(ViewModelLazy.kt:53)
    at androidx.lifecycle.ViewModelLazy.getValue(ViewModelLazy.kt:35)
...

we previously used our own simple wrapper to retrieve the ViewModel
remember {
    ViewModelLazy(
        viewModelClass = VM::class,
        storeProducer = { viewModelStoreOwner.viewModelStore },
        factoryProducer = {
            object : ViewModelProvider.Factory {
                override fun <T : ViewModel> create(modelClass: Class<T>): T {
                    @Suppress("UNCHECKED_CAST")
                    return direct.instance<A, VM>(tag = tag, arg = arg) as T
                }
            }
        },
    )
}

....and apparently, it doesn't work well with Pair and Triple either. am I missing something here?

I found the issue. It was a problem of generics erased while retrieving dependencies.
Version 7.20.2 should fix it.

thank you, tested on 7.20.2 and it does fix the issue.


Icon Duplication in Task Bar

Microsoft PowerToys version
0.57.2
Running as admin

[ ] Yes

Area(s) with issue?
General
Steps to reproduce
Place Power Tools Icon in Windows 10 Task Bar as normal
✔ Expected Behavior
Click Icon already in Task Bar to run Power Tools expect active underscore below icon
❌ Actual Behavior
A second Power Toys Icon appears in Task Bar with the active underscore under the second (new) icon.  I guess no one expected the icon to be pre-located in the Task Bar.
Other Software
No response

I can repro this

Sound like a duplicate of #8277, right?

Sound like a duplicate of #8277, right?

No it's not the same thing

Are you still experiencing this issue? /needinfo

Hello, I have changed computers since that time so I do not know.
Sorry!
Respectfully,
Floyd Beatty
@.***
207.837.5710
-----------------------------------------From: "Kayla Cinnamon"

To: "microsoft/PowerToys"
Cc: "Floyd-R-Turb0", "Author"
Sent: Wednesday October 23 2024 3:29:12PM
Subject: Re: [microsoft/PowerToys] Icon Duplication in Task Bar (Issue
#17770)
Are you still experiencing this issue? /needinfo

—

Reply to this email directly,  view it on GitHub [1], or  unsubscribe
[2].
You are receiving this because you authored the thread.Message ID:
Links:
[1]
https://github.com/microsoft/PowerToys/issues/17770#issuecomment-2433265406
[2]
https://github.com/notifications/unsubscribe-auth/AVLJTSJ3WLIMQTW7VM6P44DZ472IPAVCNFSM6AAAAABQPQ2W6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGI3DKNBQGY


update for Ott HOL4 backend changes for Inductive

See https://github.com/ott-lang/ott/pull/114

Changes have now been merged to Ott master, but a release of Ott could take a while. If you want to use the version in the GitHub repository, you can do:
opam repo add coq-extra-dev https://coq.inria.fr/opam/extra-dev
opam pin add ott dev -k version

This will install an opam package based on the GitHub repository version of Ott.
In the meantime, I bundle here a generated p4Script.sml, which I think is the best approach (no need for people who clone the repo to install Ott). There could be a CI job that checks that p4Script.sml is up-to-date.

We also need to fix install_ott.sh and INSTALL.md to take into account these changes.


[cpp-httplib] CMAKE configure failure at httplibConfig.cmake

Operating system
Windows x64
Compiler
MSVC
Steps to reproduce the behavior
1. `vcpkg install cpp-httplib`
2. `vcpkg add port cpp-httplib` at work dir
3. Add `find_package(httplib REQUIRED)` to CMakeLists.txt
4. Run CMAKE configure
5. CMAKE returned error
> `CMake Error at build/vcpkg_installed/x64-windows-static/share/httplib/httplibConfig.cmake:12 (message)`

Failure logs
[cmake] -- Running vcpkg install
[cmake] Loading dependency information for 152 packages...
[cmake] Detecting compiler hash for triplet x64-windows-static...
[cmake] Compiler found: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.37.32822/bin/Hostx64/x64/cl.exe
[cmake] Detecting compiler hash for triplet x64-windows...
[cmake] Compiler found: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.37.32822/bin/Hostx64/x64/cl.exe
[cmake] All requested packages are currently installed.
[cmake] Total install time: 500 ns
[cmake] cpp-httplib provides CMake targets:
[cmake]
[cmake]     find_package(httplib CONFIG REQUIRED)
[cmake]     target_link_libraries(main PRIVATE httplib::httplib)
[cmake]
[cmake] -- Running vcpkg install - done
[cmake] -- Selecting Windows SDK version 10.0.22621.0 to target Windows 10.0.22631.
[cmake] CMake Error at build/vcpkg_installed/x64-windows-static/share/httplib/httplibConfig.cmake:12 (message):
[cmake]   File or directory <some_path>/build/vcpkg_installed/include referenced by variable
[cmake]   HTTPLIB_INCLUDE_DIR does not exist !
[cmake] Call Stack (most recent call first):
[cmake]   build/vcpkg_installed/x64-windows-static/share/httplib/httplibConfig.cmake:64 (set_and_check)
[cmake]   C:/dev/vcpkg/scripts/buildsystems/vcpkg.cmake:859 (_find_package)
[cmake]   CMakeLists.txt:13 (find_package)
[cmake]
[cmake]
[cmake] -- Configuring incomplete, errors occurred!
Additional context
I believe the reason is as follows.

In line 12 of the httplibConfig.cmake, a marco named set_and_check is defined.
In line 64 of the httplibConfig.cmake, the marco is called, but the triplet information is missing in the second input "${PACKAGE_${CMAKE_FIND_PACKAGE_NAME}_COUNTER_1}/include", which is a path to the header file.
The problem seems to be solved by changing "${PACKAGE_${CMAKE_FIND_PACKAGE_NAME}COUNTER_1}/include"toset_and_check(HTTPLIB_INCLUDE_DIR "${PACKAGE${CMAKE_FIND_PACKAGE_NAME}_COUNTER_1}/${VCPKG_TARGET_TRIPLET}/include")`.
The same goes for line 67.

https://github.com/microsoft/vcpkg/issues/37968
Get of CMake 3.29.1, and reinstall all ports that were built with that version.

Duplicate of #37968

Much appreciated


First branch

Hello Microverse Project
Add new branch practice
Set-up Linters

Sure I will Work on it. Thank you for your review..


Current site should redirect to 2024.paravotar.org

2020.paravotar.org should continue to be accessible for the foreseeable future
Related to https://github.com/Para-Votar/paravotar/issues/9
Written by: @rnegron (lol)

Done. The domain now points to paravotar.org.


add a command to set replication factor

@nelgau

Overall, this looks good to me. Thanks, Alexis. :+1:

:+1:


gin => echo

This is technically ready to merge in, although there's still some weirdness with the transition to echo.
Namely, echo seems to have some actual error handling, since every HandlerFunc needs to return error, which makes me assume echo has a way to do something with that error.  This would be a huge improvement on our panic(err)'s everywhere.
Echo also has built-in Logger middleware we might use instead of rolling our own like we do now.

Yeah, actually, hold off on merging this for a bit.

K, we're using echo's logger now and we no longer have panics everywhere, we good to merge.

Merging this in #YOLO


[BUG] tspcontrol restart with tables plugin - error

Bug description:
running the following command 👍
tspcontrol -t <IP_ADDRESS>:9000 restart  --same  1
the main tsp process crash with the following error:

* Error: tables: line 1: trailing Declaration, invalid XML document, need one single root element
* Error: tables: Main model for TSDuck XML files not found: tsduck.tables.model.xml

seems same bug as #856
Environment:

OS: Windows
OS version: 10
TSDuck 3.31-2737
Installation type: official

Please provide the complete tsp command. Does the plugin tables use JSON output? The bug in #856 related to plugin psi.

yes, use json output:
tsp --control-port 9000 -I ip <IP_ADDRESS>:5001 -l <IP_ADDRESS> -P tables --log-json-line --invalid-sections  -O drop

@lelegard  any news on this bug? it is still present in verison 3.36-3353. When i run
tsp -v -d1 --control-port 3001 -I ip <IP_ADDRESS>:5555 -l <IP_ADDRESS> -P bitrate_monitor -p 1 -t 1 -P tables --log-json-line --psi-si --pid 18 --pid 20 --invalid-versions  -O drop
and restart the table plugin
tspcontrol -t localhost:3001 restart -s 2
i get:
`Debug: control commands: setting socket receive timeout to 5,000 ms

control commands: received from <IP_ADDRESS>:55051: restart -s 2
tables: restarting due to remote tspcontrol
Debug: tables: loading XML file C:\Program Files (x86)\TSDuck\bin\tsduck.tables.model.xml
Error: tables: line 1: trailing Declaration, invalid XML document, need one single root element
Error: tables: Main model for TSDuck XML files not found: tsduck.tables.model.xml
Debug: tables: debug level set to 1
Debug: tables: restarted plugin tables, status: false
Debug: control commands: debug level set to 1
Debug: control commands: closing socket writer
Debug: tables: packet processing thread aborted after 0 packets, 125,815 passed, 0 dropped, 0 nullified
Debug: tables: stopping the plugin
Debug: drop: stopping the output plugin
Debug: bitrate_monitor: packet processing thread aborted after 125,822 packets, 125,822 passed, 0 dropped, 0 nullified
Debug: drop: output thread terminated after 125,815 packets (125,815 output)
Debug: bitrate_monitor: stopping the plugin
Debug: ip: stopping the input plugin
ip: leaving multicast group <IP_ADDRESS> from local address <IP_ADDRESS>
Debug: ip: input thread aborted after 125,836 packets
Debug: control commands: control command thread completed
Debug: ip: aborting IP input
Debug: Report logging thread terminated`

thanks!

Hi @enricovittorini
Very sorry for having let this one under the radar for so long. It unfortunately slipped into oblivion...
Too bad, since the fix is a simple one-liner 😞
Better late than never, it is fixed now.

Thanks @lelegard !


Replace ProjectPathResolver with simpler structure

ProjectPathResolver should be replaced entirely with a simpler structure like python dict or other.
Validation done by resolver should be moved elswhere.

@cptartur Is this one still relevant?

I think so, yes.


Function type alias (typedef) is not a constant

language/first_class_types_constants_test

12.1 Constants
 * An identier expression that denotes a constant variable or a class.

15.3.1 Typedef
The eect of a type alias of the form typedef T id(T1 p1; : : : ; Tn pn;[Tn+1
pn+1; : : : ; Tn+k pn+k]) declared in a library L is is to introduce the name id into
the scope of L, bound to the function type (T1; : : : ; Tn;[Tn+1 pn+1; : : : ; Tn+k
pn+k]) ! T. The eect of a type alias of the form typedef T id(T1 p1; : : : ; Tn
pn;fTn+1 pn+1; : : : ; Tn+k pn+kg) declared in a library L is is to introduce the
name id into the scope of L, bound to the function type (T1; : : : ; Tn;fTn+1
pn+1; : : : ; Tn+k pn+kg) ! T.

As I can see, typedef (form for type alias) does not declare class.
So, it is not a constant.

What is the expected action here?

The spec has been updated since then to include type aliases into constants.

So, the test is OK now.


Update pull.go

fix the typo

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

License
Apache 2.0

What's in this PR?
fix the typo in cmdline utility

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


renderToString memory leak

Vue version
3.2.41
Link to minimal reproduction
none
Steps to reproduce
Every 10ms uses a different data request renderToString, for a total of 12000 requests.
What is expected?
The use of heap memory is expected to remain around 100MB.

The picture above is the state without renderToString
OR
The memory used by renderToString can be freed by GC.
What is actually happening?
renderToString used 800MB memory and cannot be freed.

System Info
Node v16.15.0

Any additional comments?
No response

Closing as reproduction is missing.


[New Server Request]: Integrate solc 0.8.11's LSP server

Which server would you like to request to be added?

Server name: solc LSP
Server homepage (e.g., GitHub page): https://github.com/ethereum/solidity/releases/tag/v0.8.11

Which languages does this server target?

Languages: solidity

How is this server distributed?

Is the server distributed through a standardized channel (such as release files, npm, pip, etc.)? (leave empty if unknown): there are js bindings but that introduces overhead. If possible it'd be preferable to pull a system-dependent release from their github releases channel.
Is this server added in lspconfig?: Not for solc

Hello! Cool! I remember reading about this being in the works. Happy to receive PR (also, lspconfig needs to be covered first), otherwise I'll look into it eventually

Issue opened with lspconfig, I have not written a line of Lua / explored neovim scripting so I'll probably have to tap out of this one. I consumer LunarVim repo so am pushing these issues through as I think there is definitely some desire for strong solidity support in neovim

https://github.com/neovim/nvim-lspconfig/pull/1623
https://github.com/williamboman/nvim-lsp-installer/pull/371

I have not written a line of Lua / explored neovim scripting

Understandable! Lua is probably the simplest language there ever was (in a positive way), so don't be afraid of it. Neovim is also very friendly to get into as there's a tremendous amount of docs (:help is your friend).


perf: disabled yfmLang temporarily for the test

fixed here https://discuss.codemirror.net/t/the-editor-freezes-when-using-markdown-and-large-text/8752


issue when we click on search icon in blisss theme in mobile view

Bagisto Version

[ ] master branch

Steps To Reproduce

click on seardch icon in home page of blisss

Actual Result

please check there is alligment issue
https://prnt.sc/26ixjao


python38Packages.aioesphomeapi: 9.0.0 -> 9.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.aioesphomeapi is: "Python Client for ESPHome native API"
meta.homepage for python38Packages.aioesphomeapi is: "https://github.com/esphome/aioesphomeapi"
meta.changelog for python38Packages.aioesphomeapi is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 9.1.0 with grep in /nix/store/fa0zdkz7piwsk5iaw1wdq7mjwq2lwjpm-python3.8-aioesphomeapi-9.1.0

found 9.1.0 in filename of file in /nix/store/fa0zdkz7piwsk5iaw1wdq7mjwq2lwjpm-python3.8-aioesphomeapi-9.1.0

directory tree listing: https://gist.github.com/55fb3f93ca63c1c97f3f98baba578dfc

du listing: https://gist.github.com/900ca01e0747a0de91aceb7a8144d6cf

Rebuild report (if merged into master) (click to expand)

11 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
3 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
3 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
home-assistant
python38Packages.aioesphomeapi
python39Packages.aioesphomeapi

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fa0zdkz7piwsk5iaw1wdq7mjwq2lwjpm-python3.8-aioesphomeapi-9.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.aioesphomeapi https://github.com/r-ryantm/nixpkgs/archive/c29840afad537bf8e46aa59e3f063733bc51ea3e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fa0zdkz7piwsk5iaw1wdq7mjwq2lwjpm-python3.8-aioesphomeapi-9.1.0
ls -la /nix/store/fa0zdkz7piwsk5iaw1wdq7mjwq2lwjpm-python3.8-aioesphomeapi-9.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @fabaff @mweinelt for testing.

Result of nixpkgs-review pr 137943 at c29840af run on aarch64-linux 1

1 package failed to build:

home-assistant

2 packages built successfully:

python38Packages.aioesphomeapi
python39Packages.aioesphomeapi

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

Duplicate https://github.com/NixOS/nixpkgs/pull/137860.


publishing pipeline upgrades

The publishing pipeline for digital.ca.gov should include:
The existing dev/test/live WP instances should hook up to:

Separate test/staging/prod WP-to-github instances
Each of these instances publish to the appropriate branches on digital.ca.gov
All branches of digital.ca.gov deploy to corresponding S3 instances + CloudFront with expected test, staging, live domains
End to end tests are integrated into the digital.ca.gov frontend build workflows

New development instances of publishing FaaS service have been deployed based on development and staging branches of cannabis and digital.ca.gov


backport: Initialise plans using sync.Once

Backport of 7f030ea5b94b7ca756c19c24c58dd2a8b6b275d2. Fixes data race.

gopkg.in/gorp.v1 is pointing to https://github.com/go-gorp/gorp/tree/v1.7.1 where this issue is not fixed.
@neelance how about releasing v1.7.2?

Ah, right.  Tagging.  Let me go do that...


Q42.HueApi for Xamarin.iOS

I tried adding the Q42.HueApi to a Xamarin.iOS project and I got the following error:
Attempting to gather dependency information for package 'Q42.HueApi.3.2.8' with respect to project '', targeting 'Xamarin.iOS,Version=v1.0'
Gathering dependency information took 9.66 sec
Attempting to resolve dependencies for package 'Q42.HueApi.3.2.8' with DependencyBehavior 'Lowest'
Resolving dependency information took 0 ms
Resolving actions to install package 'Q42.HueApi.3.2.8'
Resolved actions to install package 'Q42.HueApi.3.2.8'
Found package 'Microsoft.Bcl.Build 1.0.14' in '\packages'.
Found package 'Microsoft.Bcl 1.1.10' in '\packages'.
Found package 'Microsoft.Net.Http 2.2.29' in '\packages'.
Found package 'Q42.HueApi.ColorConverters 3.0.0' in '\packages'.
Found package 'Q42.HueApi 3.2.8' in '\packages'.
Found package 'Newtonsoft.Json 9.0.1' in '\packages'.
Install failed. Rolling back...
Package 'Microsoft.Bcl.Build.1.0.14' does not exist in project ''
Removing package 'Microsoft.Bcl.Build.1.0.14' from folder '\packages'
Access to the path '\packages\Microsoft.Bcl.Build.1.0.14\tools\Microsoft.Bcl.Build.Tasks.dll' is denied.
Access to the path 'Microsoft.Bcl.Build.Tasks.dll' is denied.
Removed package 'Microsoft.Bcl.Build.1.0.14' from folder '\packages'
Executing nuget actions took 409.71 ms
Could not install package 'Microsoft.Bcl.Build 1.0.14'. You are trying to install this package into a project that targets 'Xamarin.iOS,Version=v1.0', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.
========== Finished ==========
Time Elapsed: 00:00:15.2143885

Same for installing on Android in Xamarin

Apparently the minimum version of Microsoft.Bcl.Build that is required for the app is not compatible with Droid or iOS.
Installing the latest Bcl.Build package from NuGet first, then install Q42HueApi and that solves the issue

I've added the latest Microsoft.Bcl.Build package to the Q42.HueApi nuspec file. Can you check if this fixes the issue? You can install Q42.HueApi 3.2.9


SpiffyTitles: add channel whitelist

Add option channels which is a whitelist of channels which the bot should respond to links in
If empty, respond to all channels
To turn off titles completely, unload the plugin

Accidentally closed this


[AN-258] Fix find workflow modal and update its layout

Jira Ticket: https://broadworkbench.atlassian.net/browse/AN-258

Summary of changes:

What

fixes broken Find workflow modal
updates the modal's UI

Why

bug fix

Testing strategy

manual testing
unit tests

Visual Aids

@sam-schu

Do you know why the Dockstore link is supposed to open in a separate tab, but the Broad Methods Repo link isn't?

This is in the mock design. I think it makes sense since Dockstore is a different website and so would make sense to open it in a new tab. The other Dockstore links in Curated workflows section also open in new tabs.

Was it intentional to keep the "Broad Methods Repository" naming for now?

Yes. The aim is to keep naming change in a single PR so that all the places that are affected are changed together.

@salonishah11 I was forgetting that the methods repo link will not lead to a different website for too much longer!


Create Blog “lets-measure”

Automatically generated by Netlify CMS

👷 Deploy Preview for endearing-sfogliatella-6c7dea processing.

Name
Link

🔨 Latest commit
59794e61f1ac1816209616a1954d7523e253890b

🔍 Latest deploy log
https://app.netlify.com/sites/endearing-sfogliatella-6c7dea/deploys/643a3b26f5b6a80008e688db


Tx summary new style

Can you please remove the last merge commit? @alexjavabraz

Can you please remove the last merge commit? @alexjavabraz

reverted, we will need to use rebase instead of merge commit.

Task have new branch


[Executors] Avoid using Job typealias and use the ExecutorJob explicitly

This is the same change as https://github.com/apple/swift/pull/65113 but in all remaining places.

@swift-ci smoke test and merge

@swift-ci smoke test linux

@swift-ci smoke test linux


Move TCL test unit/introspection to Go case

This closes #890.

@tisonkun  The test DEBUG will freeze server also failed casually in Go, because it may use a different connection between Debug Sleep 2.2 and Set a b command and Set a b won't wait for the debug command when it's served by the different worker.
Below is TCP packets, the debug sleep command is sent by connection(port 49225) and the set command is in connection(port 49221):
11:24:50.845605 IP localhost.49225 > localhost.16379: Flags [P.], seq 1:36, ack 1, win 6379, options [nop,nop,TS val<PHONE_NUMBER> ecr 643167600], length 35
....E..W..@.@............I?./..C.L.......K.....
G..*&U.p*3
$5
DEBUG
$5
sleep
$3
2.2

11:24:50.845681 IP localhost.49221 > localhost.16379: Flags [P.], seq 97:124, ack 301, win 6375, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 27
....E..O..@.@............E?. .F..........C.....
.q.i....*3
$3
set
$1
a
$1
b

May be we can limit the pool size to 1 to workaround this issue.

@tisonkun The test DEBUG will freeze server also failed casually in Go, because it may use a different connection between Debug Sleep 2.2 and Set a b command and Set a b won't wait for the debug command when it's served by the different worker.
Below is TCP packets, the debug sleep command is sent by connection(port 49225) and the set command is in connection(port 49221):
11:24:50.845605 IP localhost.49225 > localhost.16379: Flags [P.], seq 1:36, ack 1, win 6379, options [nop,nop,TS val<PHONE_NUMBER> ecr 643167600], length 35
....E..W..@.@............I?./..C.L.......K.....
G..*&U.p*3
$5
DEBUG
$5
sleep
$3
2.2

11:24:50.845681 IP localhost.49221 > localhost.16379: Flags [P.], seq 97:124, ack 301, win 6375, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 27
....E..O..@.@............E?. .F..........C.....
.q.i....*3
$3
set
$1
a
$1
b

Maybe we can limit the pool size to 1 to work around this issue.

@tisonkun this conclusion is incorrect. The debug command is an exclusive command, so it will behave like the Redis(single thread on the server side). The root cause is the set command can be executed before the debug command since they are in a different connection.

@git-hulk Thank you! Updated.


Skip validate if the input is null

In order to avoid no method error when the input is null, each validators excluded Present Validator and Required Validator return null if the input value is null.

Required Validator should be invalid if the input is null, so fix the commitment and comment of this P-R.

All the validators should be return error if the input value is null, so I fixed code.


Visually represent DISK IMAGE SNAPSHOT dependency for Ceph datastores which "ALLOW_ORPHANS"

Bug Report
Version of OpenNebula

[X] 5.4.11

Component

[x] Command Line Interface (CLI)
[x] Storage & Images
[x] Sunstone

Description
With storage systems that allow "ALLOW_ORPHANS=YES" like Ceph there is no indication for the user that there are dependencies between snapshots, like with "qcow2" for example. When you never "revert" to a VM DISK IMAGE snapshot there is indeed no dependency. But, if at some point in time you revert to a snapshot, and later on make snapshots based on that, there are dependencies.
Expected Behavior
Visually represent snapshot dependencies when they occur, "tree" wise representation like with qcow2 snapshots.
Actual Behavior
Snapshots seem to be independent, but in fact are not.
How to reproduce
ONE CLI POINT OF VIEW
VM DISKS
ID DATASTORE  TARGET IMAGE                               SIZE      TYPE SAVE
0 BITED-1526 sda    stefan-snapshot-issue-test01        -/10G     rbd   YES
1 -          hda    CONTEXT                             -/-       -       -
CEPH  "rbd ls" POINT OF VIEW
one-596        10240M                                        2      excl

take snapshot "snap1"

ONE CLI POINT OF VIEW
VM DISKS
ID DATASTORE  TARGET IMAGE                               SIZE      TYPE SAVE
0 BITED-1526 sda    stefan-snapshot-issue-test01        -/10G     rbd   YES
1 -          hda    CONTEXT                             -/-       -       -
VM DISK SNAPSHOTS
AC  ID DISK PARENT            DATE SIZE         NAME
=>   0    0     -1  05/01 16:13:24 -/10G        snap1
CEPH  "rbd ls" POINT OF VIEW
one-596        10240M                                        2      excl
one-596@0      10240M                                        2 yes

take snapshot "snap2"

ONE CLI POINT OF VIEW
VM DISKS
ID DATASTORE  TARGET IMAGE                               SIZE      TYPE SAVE
0 BITED-1526 sda    stefan-snapshot-issue-test01        -/10G     rbd   YES
1 -          hda    CONTEXT                             -/-       -       -
VM DISK SNAPSHOTS
AC  ID DISK PARENT            DATE SIZE         NAME
0    0     -1  05/01 16:13:24 -/10G        snap1
=>   1    0     -1  05/01 16:14:53 -/10G        snap2
CEPH  "rbd ls" POINT OF VIEW
one-596        10240M                                        2      excl
one-596@0      10240M                                        2 yes
one-596@1      10240M                                        2 yes

revert to snapshot "snap1"

ONE CLI POINT OF VIEW
VM DISKS
ID DATASTORE  TARGET IMAGE                               SIZE      TYPE SAVE
0 BITED-1526 sda    stefan-snapshot-issue-test01        -/10G     rbd   YES
1 -          hda    CONTEXT                             -/-       -       -
VM DISK SNAPSHOTS
AC  ID DISK PARENT            DATE SIZE         NAME
=>   0    0     -1  05/01 16:13:24 -/10G        snap1
1    0     -1  05/01 16:14:53 -/10G        snap2
CEPH  "rbd ls" POINT OF VIEW
one-596        10240M BITED-152641-TESTCLOUD/one-596-0:1@0   2
one-596-0:1    10240M                                        2
one-596-0:1@0  10240M                                        2 yes
one-596-0:1@1  10240M                                        2 yes

delete snapshot "snap2"

ONE CLI POINT OF VIEW
VM DISKS
ID DATASTORE  TARGET IMAGE                               SIZE      TYPE SAVE
0 BITED-1526 sda    stefan-snapshot-issue-test01        -/10G     rbd   YES
1 -          hda    CONTEXT                             -/-       -       -
VM DISK SNAPSHOTS
AC  ID DISK PARENT            DATE SIZE         NAME
=>   0    0     -1  05/01 16:13:24 -/10G        snap1
CEPH  "rbd ls" POINT OF VIEW
one-596        10240M BITED-152641-TESTCLOUD/one-596-0@0     2      excl
one-596-0      10240M                                        2
one-596-0@0    10240M                                        2 yes

create snapshot "snap3" (based on reverted snapshot "snap1")

ONE CLI POINT OF VIEW
VM DISKS
ID DATASTORE  TARGET IMAGE                               SIZE      TYPE SAVE
0 BITED-1526 sda    stefan-snapshot-issue-test01        -/10G     rbd   YES
1 -          hda    CONTEXT                             -/-       -       -
VM DISK SNAPSHOTS
AC  ID DISK PARENT            DATE SIZE         NAME
0    0     -1  05/01 16:13:24 -/10G        snap1
=>   1    0     -1  05/01 16:20:02 -/10G        snap3
CEPH  "rbd ls" POINT OF VIEW
one-596        10240M BITED-152641-TESTCLOUD/one-596-0@0     2      excl
one-596@1      10240M BITED-152641-TESTCLOUD/one-596-0@0     2 yes
one-596-0      10240M                                        2
one-596-0@0    10240M                                        2 yes

Try to delete "snap1": Error message: rbd: snapshot '0' is protected from removal.

So at this point there is a "dependency" of the snapshots which is not visible in cli/sunstone. No error is logged when trying to delete the snapshot (other than "Error" in VM template).

+1
IMO it should be a configuration variable(probably in TM_MAD_CONF?) to trigger the tree-view visualization

I can see a problem. It looks like that having tree and flat (list) only ONE snapshots inventory doesn't match how CEPH snapshots are managed. They are kind of both, e.g. if I have 5 snapshots, revert to snapshot ID=2, and do another 5 snapshots, the hierarchy might look like:
 +
 |- 0
 |- 1
 |- 2 -+- 5
 |     |- 6
 |     |- 7
 |     |- 8
 |     \- 9
 |- 3
 \- 4

Rescheduling the issue.

Please also note that this means that if you ever revert to a earlier snapshot it becomes unremovable.
So, even with a single snapshot, you wouldn't be able to remove it if you ever reverted to it.


Is npm mediapipe package open source?

I am learning about the use of javascrpt in mediapipe, but I found that the npm package drawing_utils.js introduced in the case is through script cdn. I can’t find the corresponding source code description for drawing_utils.js  in the project. Is this part open source? Where can I find the corresponding description?
https://google.github.io/mediapipe/getting_started/javascript.html

I would highly welcome to open the source of these NPM packages! I am trying to run MediaPipe/FaceMesh within an Electron/Vite environment which might require certain adaptation in fetching any dependencies.

This is an issue.  It seems like in order to achieve just basic customization of the drawing_utils.js, we will be required to reproduce everything from scratch.


fix: UAT3 Testing Feedback

Fixes

Repo Folder structure
Readmes to fix linking on folder names
gitignore files
API Add to GKE Project - serviceusage.googleapis.com
Bastion don't create fw flag
Bastion SA to get Source Writer in app-cicd proj
Bin Auth add to whitelist
Enforce Bin Auth off on dev and non prod - default tfvar change
FW Rule Allow egress all ips all protocol 2022 port
Node Tag egress-internet

@bharathkkb should be ready for review

New Fix after first review:
Ingress FW Rules for cross node ingress used hardcoded dev cidrs changed to vars coming from specific envs


Squashed commit of the following:

commit<PHONE_NUMBER>537aa8a3cfad33f85a83a8d2bb12af
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Fri Apr 12 21:45:54 2024 +0200
Added documentation

1. Added some text to the data-project, and the related README.
2. Added dependencies to the requirements.txt

NOTE: We have not tested the instruction but assume it works.

commit 764d62c4312689815d4bdce6fb8856b933a161f6
Merge: f8a2cc6 df22a58
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Thu Apr 11 18:11:11 2024 +0200
Merge remote-tracking branch 'origin/development' into development

commit f8a2cc61a363c1c9eb417fade2e9554d30dea1c4
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Thu Apr 11 18:07:48 2024 +0200
Initial Dataproject

The project needs more detailed explanation. It is probably too simple for the course; but thats a discussion for another time.

commit df22a583ae9056a86e79007e40ec446220eb93c1
Merge: 2e8bf15 87e2bfd
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Sat Mar 23 00:34:14 2024 +0100
Merge branch 'NumEconCopenhagen:development' into development

commit 87e2bfd05573653ae4281cfbc990623ddd4ab022
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Sat Mar 23 00:32:43 2024 +0100
Finished Inaugrual Project (#1)

* Preliminairy Work

In this commit Question 1 part 1 has been answered. The edgeworth box needs to be optimized.

In the next commit it will be made as a function.

* Updated Jupyter-Notebook (Errors, New Answers, etc)

Minor Errors Found in Notebook:

1. The good have been changed from N = 76, to N = 75.
2. The initial endowments are the reciprocal of each other, so these
are to represent 1 point on the Edgewort box - not 2. Awkward.
3. Added Solution to Question 5 a + b

The Notebook needs to be heavily optimized and cleaned up. It is insanely messy at the current stage.

* # Commit from Pycharm

This commit is a forced commit from pyCharm - without it doesnt allow to switch branches.

Completed Question 6.a. Next commit 6b, 7 and 8.

* # Commit from Pycharm

This commit is a forced commit from pyCharm - without it doesnt allow to switch branches.

Completed Question 6.a. Next commit 6b, 7 and 8.

Question 6b, 7 and 8 is answered.

NOTE: I am not sure how pycharms git extension works.

* # Added Requirements.txt

This file should make it easier to identfy all dependencies.

commit 2e8bf1572127e75ad4a0de8c80d47d5ca684bc0a
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Sat Mar 23 00:29:52 2024 +0100
# Added Requirements.txt

This file should make it easier to identfy all dependencies.

commit 71f5d91bd2dd80a97c1e76307b5ca51b80dade72
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Sat Mar 23 00:27:20 2024 +0100
# Commit from Pycharm

This commit is a forced commit from pyCharm - without it doesnt allow to switch branches.

Completed Question 6.a. Next commit 6b, 7 and 8.

Question 6b, 7 and 8 is answered.

NOTE: I am not sure how pycharms git extension works.

commit d8b722362039523f28117693280bfdf73a25fa41
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Fri Mar 22 23:26:00 2024 +0100
# Commit from Pycharm

This commit is a forced commit from pyCharm - without it doesnt allow to switch branches.

Completed Question 6.a. Next commit 6b, 7 and 8.

commit d4d737f7362e3c5980d233aafb5d94a16d008aed
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Thu Mar 21 15:47:22 2024 +0100
Updated Jupyter-Notebook (Errors, New Answers, etc)

Minor Errors Found in Notebook:

1. The good have been changed from N = 76, to N = 75.
2. The initial endowments are the reciprocal of each other, so these
are to represent 1 point on the Edgewort box - not 2. Awkward.
3. Added Solution to Question 5 a + b

The Notebook needs to be heavily optimized and cleaned up. It is insanely messy at the current stage.

commit a65f685c00a91cfb79bcbdcc6fb5619a8beb78e0
Author: Serkan Korkmaz<EMAIL_ADDRESS>Date:   Mon Mar 18 16:03:43 2024 +0100
Preliminairy Work

In this commit Question 1 part 1 has been answered. The edgeworth box needs to be optimized.

In the next commit it will be made as a function.

Good.


create new tuning exercise separate from nested resa

 create new tuning exercise material

Just 2 mini changes. But there are conflicts that prevent from merging right now. Please resolve those and upload the sheet to moodle (independently from resolving the conflicts..) thx @lisa-wm !

conflicts solved 🙏🏻


Terminal auto replies: Differing default values

Testing #141341
I tried several different configurations of the setting; I outlined 3 different ones here.

Config 1: Set:

"terminal.integrated.autoReplies": {
        "Terminate batch job (Y/N)": "Y\r"
      }

Run a script, ctrl+c
Batch job terminated without further user interaction, output is: Terminate batch job (Y/N)? Y and then returns
Config 2: Change setting to empty: "Terminate batch job (Y/N)": "" , run and terminate script again
Asked in terminal Terminate batch job (Y/N)?, need to manually type an answer
Config 3: Comment out setting line: run and terminate script again

"terminal.integrated.autoReplies": {
        //"Terminate batch job (Y/N)": ""
      }

❓: Batch job terminated without further interaction, behaving the same way as "Y\r"

I thought config 2 and config 3 would behave the same: either both defaulting to "Y\r" behavior or both defaulting to requiring the user to manually enter Y or N. Is it expected they'd behave differently?

I expected this too, the defaults and user settings are probably being mixed in which I don't think it desirable here.

The setting seems to have no effect for me.
I've tried this:
  "terminal.integrated.autoReplies": {
    "Terminate batch job (Y/N)": ""
  }

This
  "terminal.integrated.autoReplies": {
    "Terminate batch jobsss (Y/N)": ""
  }

and this
```json
  "terminal.integrated.autoReplies": {
    "Terminate batch job (Y/N)": "Y\r"
  }

They all behave the same. It ALWAYS enters the Y for me.

In general this works for me, but rarely it doesn't so there's probably some race condition when passing the updates over to the pty host (this may only happen when multiple windows are opened?). Pushing to next iteration as it's low priority and restarting VS Code should always fix it up.

I couldn't reproduce the issue again, I called out that you may need to restart VS Code to apply new values

/verified


Spyder not starting on python3.10

Problem Description
Spyder not start with error:
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3030, in _dep_map
    return self.__dep_map
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 2827, in __getattr__
    raise AttributeError(attr)
AttributeError: _DistInfoDistribution__dep_map

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/spyder", line 33, in <module>
    sys.exit(load_entry_point('spyder==5.2.2', 'gui_scripts', 'spyder')())
  File "/usr/bin/spyder", line 25, in importlib_load_entry_point
    return next(matches).load()
  File "/usr/lib/python3.10/importlib/metadata/__init__.py", line 171, in load
    module = import_module(match.group('module'))
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/usr/lib/python3.10/site-packages/spyder/app/start.py", line 47, in <module>
    from spyder.utils.external import lockfile
  File "/usr/lib/python3.10/site-packages/spyder/utils/external/lockfile.py", line 31, in <module>
    from spyder.utils.programs import is_spyder_process
  File "/usr/lib/python3.10/site-packages/spyder/utils/programs.py", line 28, in <module>
    import pkg_resources
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3266, in <module>
    def _initialize_master_working_set():
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3240, in _call_aside
    f(*args, **kwargs)
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3278, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 573, in _build_master
    ws.require(__requires__)
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 891, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 785, in resolve
    new_requirements = dist.requires(req.extras)[::-1]
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 2748, in requires
    dm = self._dep_map
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3032, in _dep_map
    self.__dep_map = self._compute_dependencies()
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3042, in _compute_dependencies
    reqs.extend(parse_requirements(req))
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3098, in parse_requirements
    yield Requirement(line)
  File "/usr/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3108, in __init__
    super(Requirement, self).__init__(requirement_string)
  File "/usr/lib/python3.10/site-packages/packaging/requirements.py", line 113, in __init__
    req = REQUIREMENT.parseString(requirement_string)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 1100, in parse_string
    loc, tokens = self._parse(instring, 0)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 790, in _parseNoCache
    loc, tokens = self.parseImpl(instring, pre_loc, doActions)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 3810, in parseImpl
    loc, exprtokens = e._parse(instring, loc, doActions)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 790, in _parseNoCache
    loc, tokens = self.parseImpl(instring, pre_loc, doActions)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 4024, in parseImpl
    return e._parse(
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 790, in _parseNoCache
    loc, tokens = self.parseImpl(instring, pre_loc, doActions)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 3810, in parseImpl
    loc, exprtokens = e._parse(instring, loc, doActions)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 790, in _parseNoCache
    loc, tokens = self.parseImpl(instring, pre_loc, doActions)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 4854, in parseImpl
    loc, tokens = self_expr._parse(instring, loc, doActions, callPreParse=False)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 790, in _parseNoCache
    loc, tokens = self.parseImpl(instring, pre_loc, doActions)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 3810, in parseImpl
    loc, exprtokens = e._parse(instring, loc, doActions)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 827, in _parseNoCache
    tokens = fn(instring, tokens_start, ret_tokens)
  File "/usr/lib/python3.10/site-packages/pyparsing/core.py", line 283, in wrapper
    ret = func(*args[limit:])
  File "/usr/lib/python3.10/site-packages/packaging/requirements.py", line 81, in <lambda>
    lambda s, l, t: Marker(s[t._original_start : t._original_end])
TypeError: slice indices must be integers or None or have an __index__ method

Versions

Spyder version: 5.2.2
Python version:  3.10.3
Qt version: 5.15.3+kde+r137
PyQt version: 5.15.6
Operating System name/version: Archlinux

Dependencies
Nope

The problem was related to one of the packages in the repository, the next day with the update everything was solved


modify CSL

current export
Bosch, F. D. K. 1922. “De oorkonde van Sendang Sedati.” Oudheidkundig verslag, no. Bijlage B: 22–27.
where "Bijlage B" is the contents of the "issue" field
should become
Bosch, F. D. K. 1922. “De oorkonde van Sendang Sedati.” Oudheidkundig verslag (Bijlage B): 22–27.

The CSL has been updated

is this to everyone's satisfaction?
@danbalogh @manufrancis @AnneSchmiedchen
if yes, we can close this "issue" issue.

I don't know the background to this but have no objection to displaying the contents of "issue" in parentheses, it looks fine. What I don't understand is why publications in this journal don't seem to have volume numbers, only issue numbers. By my logic, the publication year should be duplicated in the "volume" field if there are no volume numbers starting from 1. This seems to be so for some entries in the Journal of the Royal Asiatic Society, but many of those also don't have volume numbers.

No objection, but most of all, no time for tiny details of display of bibliography.


Czech characters not displayed correctly when printing or after download

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version: Firefox 102.0
Operating system and its version: macOS 12.0.1
PDF.js version: v2.14.305
Is a browser extension: No

Steps to reproduce the problem:

Fill in the linked pdf file with czech characters ěščřžýáíééňť in the Firefox pdf viewer
Try to print the file or download it and open it in Adobe Acrobat Reader
See that some of those characters - ě, č, ř, ň, ť are not displayed correctly

What is the expected behavior? Those newly added characters should be displayed correctly, just as they were written

What went wrong?

For the first screenshot, characters were added (and document was saved there) before opening it in the default pdf viewer in Firefox. For the second use case, characters were added directly in the Firefox pdf viewer.
Example of font issues.pdf
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Duplicate of #12264


ci: 🐝 add complete version number to codeql-action

Description
Add full version number of actions in CodeQL workflow.
Type of change

[X] CI

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (1e8a158) 20.90% compared to head (ea7d646) 20.90%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   20.90%   20.90%
=======================================
  Files           4        4
  Lines         177      177
  Branches       51       51
=======================================
  Hits           37       37
  Misses        140      140


Security fix - every field of a model is send - even password

Atm WebsocketBinding sends every field of a model, even the password of a user. Users of the class should have to think about which fields they want to send to the user. Also added a more intuitive option for sending all fields.

Agreed - this change presumably errors if fields is not provided?

Now it does check for empty fields

Thanks - I'll fix up the docs to match!


Applied fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Coverage decreased (-0.2%) to 84.114% when pulling dae28522fa3e9ee361e20e52d0d2249abd702d7a on analysis-z9bOd9 into 1a5be8767f55b26ec61cd2b50562640bb465a4a3 on master.


Improve collapsables style

@camillemonchicourt . What would you prefer for them ? make them higher or another color ? Or both .?

Then it's an issue with the transform css property on old Android phones. You can open another ticket for this if you want but I think it's not really important. (better for modern but not a problem for older versions)


Does not function as described.

Macos 13.5.2 (M2) doesn't throw an error or success dialogue box, I just hit return in alfred and nothing happens.

If Alfred's debugger (cmd+D) really does not display any errors or other information, I cannot help you with the information you have provided.

Apologies, I'm a total noob. I had  xcrun: error: invalid active developer path in my debug output. Installing the xcode command line tools (xcode-select --install in Terminal) fixed it. Thanks for pointing me toward the debugger.


Add support for binding docker ports to 'localhost'

The current setup automatically binds port 25 for postfix and 8000 for the webapp to the hosts external IP address.
However, in my scenario I want only port 25 to bind directly to the hosts IP whereas the webapp should be available only on localhost in order to expose it through an additional reverse proxy.
I achieved that by editing "docker-compse.yml":
   ports:       - "25:25"       - "<IP_ADDRESS>:8000:8000"
For some reason the "mode: host" option did not working to support the long variant of the port description you chose.
Anyway, there should be an config option available to set the binding ip address without the need to edit the compose yml.
Cheers.

@Flunkyball network_mode: host is not compatible with this image. It's also not recommended because that container’s network stack is not isolated from the Docker host (the container shares the host’s networking namespace), and the container does not get its own IP-address allocated. If you really want that you can fork this repo and make your own changes.

@Flunkyball network_mode: host is not compatible with this image. It's also not recommended because that container’s network stack is not isolated from the Docker host (the container shares the host’s networking namespace), and the container does not get its own IP-address allocated. If you really want that you can fork this repo and make your own changes.


Our Crew - text edit

"One of the advantages of partnering with SeeSaw Labs is immediate, cost-effective access to our many years of experience across myriad diciplines."
should be:
across a myriad of disciplines

@luigicr18  is this one ready? not seeing it.

@mjbarnes88 test it again.

sweet. thanks!


Removing items from tags input, not shown in autocomplete list if already opened

Hi,
If you set minLength to 0 and loadOnDownArray, loadOnEmpty and loadOnFocus on true for autocomplete, when removing one item from input and the autocomplete list is opened, the deleted item will not appear in autocomplete, till a search is made.
Plunker: http://plnkr.co/edit/zp5TMt?p=preview
Video: http://screencast.com/t/Kzx5xwCFYJ
Thanks!

I'm aware of that. I thought it wouldn't be an issue because the user wouldn't be interested in adding a tag she/he had just removed.
Do you expect the autocomplete list to be reloaded after a tag is removed?

Yes. Is there a way to reload the list after a tag is removed?

There's no way to do that right now. I'll add this to the backlog so I can fix it later.

You could always just bind to on-tag-removed and refresh it yourself when an element gets removed.

This issue was selected to be spring-cleaned by a script. More information can be found here.
If anyone thinks this was a mistake and this issue should be reopened, please leave a message below explaining why. Before doing so, please consider reading the CONTRIBUTING file.


lua function can not read from STDIN redirected.

.

I created issue : How to do io.input( <*os.File instance created by os.Pipe() or os.Open() on my application> ) · Issue #188 · yuin/gopher-lua on GopherLua for this issue.


PayPalButton is disappeared

Hi, I use the docker to run the source code, but there is no Paypal button on page <IP_ADDRESS>:3000, as shown in the tutorial. Is it a problem with react version?

Solved. It should go to set client id.


ODbL 1.0 formatting broken

Currently the formatting of the ODbL text would seem to be broken.
For example the formatted text for
`
4.0 Conditions of Use
4.1 The rights granted in Section 3 above are expressly made subject to
Your complying with the following conditions of use. These are important
conditions of this License, and if You fail to follow them, You will be
in material breach of its terms.
4.2 Notices. If You Publicly Convey this Database, any Derivative
Database, or the Database as part of a Collective Database, then You
must:
a. Do so only under the terms of this License or another license
permitted under Section 4.4;
b. Include a copy of this License (or, as applicable, a license
permitted under Section 4.4) or its Uniform Resource Identifier (URI)
with the Database or Derivative Database, including both in the
Database or Derivative Database and in any relevant documentation; and
c. Keep intact any copyright or Database Right notices and notices
that refer to this License.
d. If it is not possible to put the required notices in a particular
file due to its structure, then You must include the notices in a
location (such as a relevant directory) where users would be likely to
look for it.
`
looks like

Same result on both Firefox and Chrome.

Hi thanks for raising this. I've just pushed some formatting fixes to the licence texts which should resolve this issue


Allow setting review assignment limit

Part of #1753
With this patch we allow team members to set their own review capacity.
This will work by sending a message to the triagebot bot on Zulip. Usage:

work set #: will set the number of max PRs assigned to me

An update the documentation on the forge will follow up.
(I cannot test it, don't have yet a testing infrastructure with also a Zulip instance)
r? @jackh726
thank you

Has there been any update on this PR? Is there anything I can help with?

Has there been any update on this PR? Is there anything I can help with?

thanks @xFrednet it's on my radar. Was a bit busy on other fronts but I plan to get back to this soon

Ok, update: before proceeding with this patch, there seems to be an inconsistency in the triagebot DB migration table that should be fixed first. I am waiting for this issue to be resolved by t-infra (thanks!)

For some reason I "broke" this PR. Work continues on #1854


Opening local file with PWA causes the browser to crash

MacOS. Tested on Chrome / Edge. Both crashed after clicking Allow in the following dialog.

I could open local file with PWA before. Not sure what has changed, excalidraw or chromium.
Drag & Drop works though.

I can reproduce on MacOS 11, Chrome 97.0.4692.99, File Handling API flag enabled.
I Couldn't get Edge to register the filetype with the OS so couldn't test.
@tomayac is there a log or some way to figure out why the PWA is crashing?

Could you file a new.crbug.com and attach the crash report that you can obtain by enabling crash reporting in chrome://crashes/?

Could you file a new.crbug.com and attach the crash report that you can obtain by enabling crash reporting in chrome://crashes/?

I can't obtain the crash report. The only button is Upload. Anyway, I filed an issue there.
https://bugs.chromium.org/p/chromium/issues/detail?id=1290326

So it appears the bug is fixed Chrome 99 onwards. You can either wait ~33 days until be lands to stable channel, or use Chrome Dev/Canary channel today.


chore: bump flake nixpkgs

bump nixpkgs version
merge some inputs

Seems that build on Darwin is broken., I am not sure how to fix it
ld: warning: directory not found for option '-L/nix/store/w8s745hmd24c605d4fziy4fjakj06dn2-lean-bin-tools/lib/lean'
ld: warning: ignoring file /nix/store/hijjyl9fqynq0vis2bq7a6msyf4b0rb7-Init-lib/libInit.a, building for macOS-arm64 but attempting to link with file built for unknown-unsupported file format ( 0x21 0x3C 0x74 0x68 >
ld: warning: ignoring file /nix/store/2kink7vw6asw0ac64wvsil053frjm10q-Lean-lib/libLean.a, building for macOS-arm64 but attempting to link with file built for unknown-unsupported file format ( 0x21 0x3C 0x74 0x68 >
Undefined symbols for architecture arm64:
  "_initialize_Init", referenced from:

% tree /nix/store/w8s745hmd24c605d4fziy4fjakj06dn2-lean-bin-tools/
/nix/store/w8s745hmd24c605d4fziy4fjakj06dn2-lean-bin-tools/
├── bin
│   ├── leanc
│   └── leanmake
├── include
│   └── lean
│       ├── config.h
│       ├── lean.h
│       ├── lean_gmp.h
│       └── version.h
└── share
    └── lean
        └── lean.mk


Write a Perl program to convert minutes to hours

Description
Write a Perl program to convert minutes to hours
1 hour = 60 minutes
Input  : 300
Output : 5

How to contribute

Save the solution in program/convert_minutes_to_hours.pl file
Add convert_minutes_to_hours.pl file in convert-minutes-to-hours folder

!assign


Adding Auth Provider Templates to Chart

add azure workload identity and k8s secrets auth providers in config map
templatized so the provider used is determined by flags in values.yaml

@susanshi Appreciate a review when you get the chance


Added my site to Users List

Added site to users list.

Thanks for using Congo, @G-Snipes! I'll merge this change in and your site will appear on the public list when the next theme release is published.

No problem! It was my first time using Hugo and this theme was great!
On Wed, Aug 28, 2024 at 9:58 PM James Panther @.***>
wrote:

Merged #911 https://github.com/jpanther/congo/pull/911 into dev.
—
Reply to this email directly, view it on GitHub
https://github.com/jpanther/congo/pull/911#event-14054758580, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASMBVK2XWISXGKBOXXXNV3LZTZ555AVCNFSM6AAAAABK2MWTSOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGA2TINZVHA2TQMA
.
You are receiving this because you were mentioned.Message ID:
@.***>


[TASK] Update deployment pipeline for staging and main to reflect new monorepo

Staging and main setup using fleek.


Performance - Replace use of Activator in Is

Activator is notably slow.
Since you already have put a zero arg constructor constraint on T, you should be able to just do
return new T().FeatureEnabled;

If for some reason that doesn't work then you can look at a generic version of DynamicMethod, but I don't think you will need it.
https://stackoverflow.com/a/22651558

As far as I can see, then current implementation does what you suggest
https://github.com/jason-roberts/FeatureToggle/blob/master/src/FeatureToggle.Common/Is.cs

That's interesting.  When I decompile in Jetbrains Rider I get a different set of source.   I wonder if new T() actually compiles to using Activator.
// Decompiled with JetBrains decompiler
// Type: FeatureToggle.Is`1
// Assembly: FeatureToggle.Common, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
// MVID: 56948869-D82D-4DE1-93B1-08121C96F0D7
// Assembly location: /Users/richardcollette/.nuget/packages/featuretoggle.common/4.0.2/lib/netstandard1.4/FeatureToggle.Common.dll

using System;

namespace FeatureToggle
{
  public static class Is<T> where T : IFeatureToggle, new()
  {
    public static bool Enabled
    {
      get
      {
        return Activator.CreateInstance<T>().FeatureEnabled;
      }
    }

    public static bool Disabled
    {
      get
      {
        return !Activator.CreateInstance<T>().FeatureEnabled;
      }
    }
  }
}

Decompiling is not nearly as good as looking at actual source code. That is my experience when decompiling .NET framework BCL code and comparing with the open source code.

Is this something that needs to be looked at in v5 or can we close @rcollette @mikanyg  ?

I don’t know any other way to instantiate generic types than what is already implemented. So I would say close this issue.


Service account automountServiceAccountToken should be set to false

Is your feature request related to a problem? Please describe.
Disabling automatic token mounting with automountServiceAccountToken set to false enhances security and control in Kubernetes by reducing attack surface, providing explicit control over token usage, avoiding unnecessary access, and meeting compliance and security requirements.
Describe the solution you'd like
Set the automountServiceAccountToken flag to false.
Describe alternatives you've considered
Any explanation of why this change is not possible
Additional context
I'm trying to align the helm charts I pick from upstream to the Kubernetes best practices.

@alita1991 I'm not entirely sure what's the type of risk you're referring to. If you don't explicitly grant any permissions to the service account, I think there is no real attack surface / unnecessary access / and so on.
Nowadays, Grafana supports authentication using Workload Identity, and the common configuration approach is to set a custom annotation on a ServiceAccount to indicate that Pod's specs should be mutated to inject a env with a Client ID. So, if we change the defaults now, it'll break setups of many people when they upgrade the operator.
Even though the operator cannot accommodate all possible scenarios in its defaults, you're free to adjust any particular settings through Grafana CRD - it exposes all properties of Deployment / StatefulSet.

@alita1991 is this for the grafana service account that gets generated or the operator?
The operator needs token to be able to talk to the Kubernetes API (it's what it does).
But I agree that we should set this as a default value in the grafana service account that gets generated.
We are more then happy to take a PR around this area.


Update Composer dependencies (2024-01-18-00-11)

Visual regression test passed!
View the visual regression test report


Cannot use ServiceName as ServiceName because the name is already in use

This may not be an actual issue, but I keep getting the following error:
Cannot use ServiceName as ServiceName because the name is already in use
It seems to be because I'm using services in other services, so trying to load all the dependencies loads services multiple times. Maybe Laravel isn't supposed to load services into other services, or maybe it's a bug with this? Any thoughts?
Thanks!
Nathan

I got the same with a self defined ValidationException

Experiencing the exact same issue as nsyder outlined and unable to resolve without a major refactoring of existing code.  Any clarification on this issue would greatly be appreciated!  Thanks!

I managed to resolve my issues by including all of my facades in services in the following way:
use App\Facades\MyService as MyServiceFacade;
Hopefully that can help point others in the right direction. However, you will have to change any uses of those services in the file accordingly.

If this resolves the problem for others, I'd be happy to close it. My issue is resolved.

Thanks Nathan - appreciate you sharing!


[devops] Cross-compile to, and add aarch64 support in Docker images

In order to run the application in ARM 64-bit hardware, such as Raspberry Pi's or Mac M-series processors, it needs to be cross-compiled using GOARCH=arm64.
To do this, a go build needs to be ran for both x86_64 (default/current arch) and arm64.
Both binaries need to be kept with separate names, thus the x86_64 can remain as /app/instafix, and the arm binary as /app/instafix-arm.
Then, instead of running the application directly as an entrypoint, it will need to be ran from an entrypoint script that chooses between both available binaries depending on the running device's architecture.
Bonus points if the entrypoint shell script also includes a health check reporting back to the Docker engine.

I will be submitting a PR for this later on, as so far I have kept a separate fork that allows me to run ARM-only Docker images of instafix, and I would like to get rid of that since it's troublesome to keep up to date with (your) upstream.

Partly solved by @AstralJaeger in #112/#111. The only thing that's missing is building (at GitHub CI's build-time) all supported variants, save them into separate files, then adding an init script that gets called by the Dockerfile's entrypoint and checks the architecture and runs the corresponding binary. What do you think, @AstralJaeger, would that still work for your purpose?

Sorry, I was on vacation for the week,
I think it should already be done, no runtime check needed, that will be performed by the container runtime since the product of the build is a multi-architecture container which contains the variants for amd64 and arm64 (whoever decided to name 2 fundamentally different architectures this similarly deserves slow internet).
I just pulled the latest container and tried to run it on my M1 Mac and it worked out of the box. I encourage you to play around with it, as meantioned in #111 you can try it out on the Oracle Cloud Infrastructure Free Tier, they provide a 4 Core 24Gb ARM VPS (credit card required, I don't get sponsored or use OCI professionally either).
Any changes that still allow the image to run on ARM are fine by me and will still work for my purpose, worst case I'll rely on the current master sha. :D

Ahh, I see now. I missed reading the line with: platforms: linux/amd64, linux/arm64... in the push to the container registry step. Quite a nice setup! And indeed, with this one there should be no manual work needed at the runtime, should all be handled by the dockerd/containerd engine when they pull the correct arch's image. Will close this issue now, thanks!

I'm glad you liked the setup, the initial version I cooked up a year ago took me a while to figure out.


Feature suggestion/fork suggestion: Forge version?

Do you think this could be modified to set up a forge server? I like the idea for paper, I just want to be able to run mods instead of just being stuck with plugins.

I'm absolutely open to this.  It would most likely be a containerized version.  I believe people have actually modified this into Forge before but I'm not sure if anyone has shared the code publically.
I will definitely update this when something is out related to Forge!


Fix release changelog generation script

Description
I otherwise couldn't get this to build, see https://github.com/Gabriella439/turtle/pull/446#issuecomment-1747508735.
Also, add a fixed index-state to aid reproducibility, and some verbosity as it takes quite a while to actually build, and it is nice to get progressive feedback.
Checklist

[x] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated.  These may include:

golden tests
property tests
roundtrip tests
integration tests
See Runnings tests for more details

[ ] Any changes are noted in the CHANGELOG.md for affected package
[ ] The version bounds in .cabal files are updated
[ ] CI passes. See note on CI.  The following CI checks are required:

[ ] Code is linted with hlint.  See .github/workflows/check-hlint.yml to get the hlint version
[ ] Code is formatted with stylish-haskell.  See .github/workflows/stylish-haskell.yml to get the stylish-haskell version
[ ] Code builds on Linux, MacOS and Windows for ghc-8.10.7 and ghc-9.2.7

[x] Self-reviewed the diff

Note on CI
If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons.
You will need to get someone with write privileges.  Please contact IOG node developers to do this
for you.

@amesgen note that cabal run will still be influenced by cabal.project.
❯ cabal run -v scripts/generate-release-changelog-links.hs | head -5
this build was affected by the following (project) config files:
- /home/andrea/work/cardano-node/cabal.project
- /home/andrea/work/cardano-node/cabal.project.local
creating
/home/andrea/.cabal/script-builds/f2e0aa3350c22cdf879806b725cd2c1b2a13022e0ed6824d1a2ea0b2aedb6cd0


Select component icon not triggering select

When ever I try to click on the select icon that is present in the select compnent, It does not show the select options.
This video should explain better

@astraldev a fix has just been pushed. Please try installing latest version.

Thank you very much


Add all shims defined in iree to EmitC

Add shims in EmitC for all shims that are already defined in iree.

Should fix most of the issues reported in #7842.


Change pricing scheme to pricing plan

Change pricing scheme to pricing plan as my first doc update.

Hi @DaleKoetke, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Dale Koetke). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@alancameronwills do you approve doc changes for App Insights?  Not entirely sure how this works.

I will push this through on the -pr fork, with some other changes
#please-close

@DaleKoetke : Thanks for your contribution to the Azure documentation! The author, @alancameronwills, has been notified to review your proposed change.


Overlapping text in intellisense suggestions

VSCode Version: 1.7.0-insider 5bafcdb3d67b29b1f4667a3f497ac05a41dde667
OS Version: Mac OS Sierra

Steps to Reproduce:

Create a css file (or any other file)
start typing

#13813


Placeholder for missing jurisdictions

From the usability tesing https://docs.google.com/document/d/1YRlgHC-Venn5oaTY7jrk42OJfabM8171EykkcHC1EQE/edit
Users were confused that not all districts were visible.

Suggestions:

Provide placeholders or another method to clearly indicate that some districts are not up for election, and therefore have no campaign finance data.

@elinaru do you have any ideas here?


Possibility to reset remote connection

There would be need to do reset for remote connection at least in following conditions:

remote server is requested to do restart for testing purpose
remote server is requested to do something which is expected to broke remote connection for a while

Any comments or review for:
https://github.com/robotframework/robotframework/pull/2938

I guess this could be useful. There are some problems with the code in the PR that ought to be pretty easy to fix. More importantly it would need tests and documentation.

I guess this could be useful. There are some problems with the code in the PR that ought to be pretty easy to fix. More importantly it would need tests and documentation.

Can I do something? Is there documentation rule follow, test procedure to execute and descritption what should be fixed?

Short check with "Reload Library" shows it is working a bit differently, but may lead to same situation. Will need a bit more time that I can verify it fulfills my need

What's the situation with this issue/PR?


Kubernetes nodes scrape problems

Hi:
After working with a kubernetes cluster since 3 months with prometheus without problems, today scrape does not work in one node (master kubernetes node), is work for little metrics, an other metrics does not work.
Example:
For kube_node_labels metric the node appear to be scraped:

But for node_uname_info, node_boot_time, node_time cant see the IP of the node, in this case, should be <IP_ADDRESS> for ietsmartc09spre

I dont know what happend with this, all was working ok until today!
Thanks in advance!

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

I reply to myself, the issue was because I forget to taint the kubernetes master and the node-exporter pod was unable to run on a master node without taint!


CTC import compatibility with tensorflow 0.10

Try except clause to import ctc_loss in new path on tensorflow 0.10.

This is incomplete: you would also have to modify references to contrib.ctc in ctc_decode().

Sorry about that, I didn't realize it because there are no tests for ctc_decode(). After looking into it, I found there are some errors on ctc_decode() which I'm working on fixing. I'm also adding tests for greedy and beam search decoders based on the tensorflow tests. Will update soon.

LGTM, thanks!


chore: replace deprecated io/ioutil functions with modern equivalences.

As in Go 1.19, io/ioutil functions are marked as deprecated. This PR helps refactoring them and replace with modern equivalents, as suggested by Go's official docs

LGTM, thanks!


Enable nullable on the compiler, client, and shared

Closes #658

Notably this also fixes several uncaught exceptions that currently happen when one attempts to clone a window.