Add an OpentelemetryTracer in LangChain

Feature request
There's already many tracers in LangChain (https://github.com/langchain-ai/langchain/tree/master/libs/langchain/langchain/callbacks/tracers).
I would like to ask for adding an OpentelemetryTracer.
Motivation
Community tracing
Your contribution
Would like to contribute

Hey, @npuichigo! Just found a solution to this in another related post: https://github.com/langchain-ai/langchain/issues/14594#issuecomment-1851951445 . Also noticed it doesn't use LangSmith, is this sufficient or, if not, could you point to some resources/repo code incase improvements are required?

@debdhriti I'm not sure the robot answer actually works. My point of view is that since we support wandb in addition to LangSmith, opentelemetry as a widely used community tracing component should be supported as well. So it's convenient for LangChain user to integrate with ZipKin/Jaeger/Grafana for telemetry.

I'm also interested in an opentelemetry tracing adapter for langchain. Happy to take a swing at the PR, likely will create another file in langchain_community/callbacks/tracers/opentelemetry.py

At openinference they have such OTEL (+ llm extension) instrumentation.. Works pretty neat..

fyi there's a discussion I think if moved forward could ease some trouble-spots of tracing langchain. If any contributors have time to pipe in, much obliged! https://github.com/langchain-ai/langchain/discussions/27954


Build failure with /healthz 'context deadline exceeded'

A build in a PR, https://github.com/cilium/cilium/pull/2947, which only fixed typos in comments, failed due to a timeout in waiting for endpoints to be ready.
Build link: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/868/
Logs are attached. Specifically, check the logs in RuntimeValidatedPolicyImportTestscheckspolicytraceoutput for the context deadline exceeded messages
test_results_Cilium-PR-Ginkgo-Tests-Validated_868.tar.zip

Closing, duplicates #2949


Inconsistent timeouts

Hello,
I've been getting some strange timeouts, and I wanted to double check if the timeout checking code is correct. In the below code, is the last line in the function supposed to return true? If I am reading the logic correctly, shouldn't it report "no timeout" if it reaches the last line, i.e. return false instead?
https://github.com/nhnent/toast-haste.sdk.dotnet/blob/f2d0d5a15e2244eb00f0eb7bf288676486ea03d0/HasteClientLib/Network/UDP/OutgoingCommand.cs#L195-L206

@ghroot Thanks for feedback!!
Yes, this is so strange. I'll check this out.
If you can, would you write pull request for changing this?


Slam wont play sounds anymore

Problem started when I was using slam earlier today, whenever I try selecting another audio file in console, it played the same sound I had previously. (Even though it said the new sound was selected)
Now it has gotten even worse, now when I try to play sounds, not a single sound plays. Whenever I hit the keybind to play the sounds, my microphone just echoes the people talking in the game. I have no clue how to fix this, any ideas?
Error log
--------------------7/31/2016 1:15:43 AM--------------------
System.IO.IOException: The process cannot access the file 'D:\Steam\steamapps\common\Counter-Strike Global Offensive\voice_input.wav' because it is being used by another process.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
at NAudio.Wave.WaveFileWriter..ctor(String filename, WaveFormat format)
at NAudio.Wave.WaveFileWriter.CreateWaveFile(String filename, IWaveProvider sourceProvider)
at SLAM.Form1.LoadTrack(SourceGame Game, Int32 index, String SteamappsPath)
--------------------7/31/2016 2:02:01 AM--------------------
System.NullReferenceException: Object reference not set to an instance of an object.
at SLAM.Form1.GetFilepath(String ProcessName)
at SLAM.Form1.PollRelayWorker_DoWork(Object sender, DoWorkEventArgs e)

I see getting help on this forum is useless considering no one has been helped since early april.


[9.x] Report file system exceptions

@taylorotwell
This is an attempt to fix issue https://github.com/laravel/framework/issues/41269
Suppressing all those file system exceptions was a bad decision made in PR https://github.com/laravel/framework/pull/33612

There is no breaking change, all methods still return boolean
We are just reporting the exception to respective logger

Imaging your server's hard disk runs out of space or AWS S3 service went down, you will never gets notified about it and your customers will keep complaining about their missing uploaded files. I have spent hours already to figure out why my S3 uploads are not working.
This PR reports those errors so at-least the developer can look into logs and see what went wrong.
Willing to see feedback form the community on this solution. :)

We cannot use a global helper in this class. You'll need to set the logger onto the class with a setter method and add a new config option that by default disables it. Then people can opt-into this. Please resubmit with these changes and make sure tests are passing. Thanks

driesvints
Thanks, i will re-attempt it soon.


Add heated floor area to Evaluation

This PR refactors heatedFloorArea. The new heatedFloorArea is just a Float value in the evaluation.

Does the float represent surface area?

Okay, if it's surface area in meters squared, make sure that is written into the field description/documentation.


Implement the todo-priority feature.

Add a time-stamp field to each todo as and when the todos are created so that according to the current time the todo's are sorted based on priorities such as high, medium and low. The users can then filter out the todos based on preference and complete those first.

thanks for the interest @RuthChisom ! assigned the issue to you

@RuthChisom what's the status?

@RuthChisom status?


.Net: Fix for #2820 Avoid unnecessary conversion of strings to Unicode encoding

Fixes #2820
Motivation and Context
This PR resolves an issue where search results containing UTF-8 characters were automatically converted to Unicode, adversely affecting readability and resulting in token wastage.

Description
 This is the second submission addressing feedback from the initial review. Changes have been made to explicitly use UnsafeRelaxedJsonEscaping for all serialization cases, as suggested by @stephentoub. Please review the modifications and provide further feedback. Thank you!

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

@yuezhishun , please follow the license agreement change and we can merge it.

@microsoft-github-policy-service agree

@yuezhishun请阅读以下贡献者许可协议（CLA）。如果您同意 CLA，请回复以下信息。
@microsoft-github-policy-service agree [company="{your company}"]

选项：

（默认 - 未指定公司）我对我提交的内容拥有知识产权的唯一所有权，并且我不会在为我的雇主工作的过程中提交内容。

@microsoft-github-policy-service agree

（当公司提供时）我在为我的雇主工作的过程中提交意见（或者我的雇主根据合同或适用法律对我提交的意见拥有知识产权）。我已获得雇主的许可，可以代表我的雇主提交意见并签订本协议。通过在下面签名，定义的术语“您”包括我和我的雇主。

@microsoft-github-policy-service agree company="Microsoft"

贡献者许可协议

@microsoft-github-policy-service agree


Particles not showing after update to new Combo Mode Version

I recently updated the package to the new combo mode version 1.1.0 and although I see combo mode and the numbers increasing animation, the particles no longer appear. I checked the setting and the option is checked. I reinstalled and restarted my Mac and the issue persists.

You need to reach 50 (default) in the combo mode to activate the other effects.
You can disable combo mode or set a lower activation threshold.

Ah I see, this should be emboldened in the README.

Thanks though, I should have caught that!

Yes, indeed, I will update the readme to make it more clear.


Added 'starting out with programm..." to book list

Added "Starting Out With Programming Logic and Design" by Tony Gaddis to book recommendation list, with Amazon link and ISBN #.

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

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

Thank you for opening this pull request.
Book recommendations are beyond the scope of the freeCodeCamp Guide, so we've decided to go ahead and remove these book recommendation articles.
I encourage you to review this book in the ratings category of the freeCodeCamp forum. Your recommendation will reach way more people there.
Thanks again recommending this book, and for contributing to the freeCodeCamp Guide.


bizongo seller app has been added in showcase

I have added bizongo seller app in react native showcase section. Bizongo is one of the leading B2B ecommerce marketplace of India.
Know more about bizongo here -: https://yourstory.com/2016/11/bizongo-series-a-idg-ventures/

Thanks for sending this pull request. According to the guidelines in that file, showcase apps should be added to the featured array.
Also, "infoLink" must be one of:
1/ An English-language news article discussing the app, built either by a funded startup or for a public company
2/ An English-language technical post on a funded startup or public company blog discussing React Native
I am going to close this, but if you have one of those two things, and address the incorrect placing, feel free to reopen.


[FIX] transaction submission regression

With recent refactoring and an updated of ripple-lib, some transaction submissions started waiting for the transaction to be finalized instead of replying immediately with a status url. The only time we should wait for a transaction to be finalized before returning is when the validated flag is set to true, otherwise a status url should be returned. This PR fixes the recent regression

LGTM

Coverage increased (+0.05%) when pulling 741fd5ae94c82fce49644d102e5eaa59fca9683c on fix/transaction_regression into 87ca02cdb9258a50e178fbc0f5266232c6ac886c on develop.


Open button for each pdf generated

Maybe something like this?

Maybe a preview window would be great

https://github.com/AngelGallegosE/Osiris/pull/56


Tax form queries are not considering currency

This expense triggered a tax form, even though  ₹4,130.00 ~= $50 which is under the $500 threshold normally used to trigger tax form.
This is happening because the collective currency is different than the host currency.

This can lead to compliance issues; I'd love to see this issue prioritized in the next sprint.


docs: Unbreak readthedocs build

I apparently broke, see:
https://readthedocs.org/projects/luigi/builds/3418481/

Seems to have unbroken it: https://readthedocs.org/projects/luigi/builds/3424040/
But seems like the parameters still are not looking as I intended them http://luigi.readthedocs.org/en/latest/api/luigi.tools.range.html?highlight=rangehourly#luigi.tools.range.RangeBase.of, relating to #1326.

Ok. I tried to fix this but give up.
I tried enabling the checkbox in readthedocs saying "Install your project inside a virtualenv using setup.py install". But then I got error logs like 1, and other people have had exact same issue 2.

:D :D :D


[Bug]: iOS16 / v2.2.0 b5 - Doorbell Motion Toggle doesn't retain state but still works

What happened?
I'm not sure whether this was affected by iOS16 or moving to v2.2.0b5, as I did both on the same day. But the motion toggle switch no longer retains its status. It still works, but visually flips back to the last status.
So if it's off, I toggle it on, this is reflected in Eufy, and the switch flips back to off.
From there, I have to quickly toggle it on and then off again to turn it off.
Running HomeBridge 1.5.0
Device Type
Doorbell (Supported)
Plugin Version
beta (Best Effort)
HomeBridge Version
other (UnSupported)
NodeJS Version
v16 (Supported)
Operating System
Docker (UnSupported)
Relevant log output
I can't paste any log here, as it disappears. I mean I see it appear as "set motion true / false" etc, but then it deletes when the switch reverts back.  I do, however, get this error on restarting the bridge.

[9/16/2022, 11:50:38 AM] [EufySecurity-2.2.0-beta.5] ERROR: Error cleaning config: Error: EACCES: permission denied, open '/homebridge/config.json'
    at Object.openSync (node:fs:585:3)
    at Object.writeFileSync (node:fs:2170:35)
    at EufySecurityPlatform.clean_config (/homebridge/node_modules/homebridge-eufy-security/src/plugin/platform.ts:577:10)
    at new EufySecurityPlatform (/homebridge/node_modules/homebridge-eufy-security/src/plugin/platform.ts:146:10)
    at ChildBridgeFork.startBridge (/usr/local/lib/node_modules/homebridge/src/childBridgeFork.ts:150:42) {
  errno: -13,
  syscall: 'open',
  code: 'EACCES',
  path: '/homebridge/config.json'
}

config
{
            "username": "XXXX",
            "password": "XXXX",
            "country": "AU",
            "ignoreStations": [
                "XXXX"
            ],
            "cameras": [
                {
                    "serialNumber": "XXXX",
                    "enableCamera": true,
                    "useCachedLocalLivestream": true,
                    "videoConfig": {
                        "audio": true,
                        "vcodec": "copy",
                        "useSeparateProcesses": true
                    },
                    "enableButton": false,
                    "motionButton": true,
                    "refreshSnapshotIntervalMinutes": 360,
                    "unbridge": true,
                    "rtsp": false,
                    "talkback": true,
                    "immediateRingNotificationWithoutSnapshot": false,
                    "snapshotHandlingMethod": 3,
                    "hsv": true
                }
            ],
            "platform": "EufySecurity",
            "_bridge": {
                "username": "XXXX",
                "port": XXXX
            },
            "ignoreDevices": [],
            "cleanCache": true,
            "enableDetailedLogging": true,
            "experimentalMode": true
        }

Hey @DaveM8686
Can you try if this also happens with the latest 2.1.0 version? Thx!

Hey @thieren, unfortunately not right now. My NAS died this morning, which is what I was running Homebridge on. I've ordered a new one. If it's still happening once that arrives and I set everything up, then I'll let you know.

Hey @thieren, managed to get every thing back up and running today. Can confirm it's the latest beta. If I drop back to 2.1 stable it doesn't do it, but then go back to 2.2b5 and it does again.

Hey @thieren, figured this out. It was experimental mode causing it. I turned that off and it's all good again. I have another issue now that is definitely since beta 5, and really weird, but I'll raise another bug about that. Not sure if you want to keep this one open or closed.

Thx for the feedback.
I'd let it open for now. When I have time, I'll look into this, but since this seems to be a really isolated issue it might take some time.
Did you just enable experimantal mode, or did you actually use an experimental feature, like e.g. enabled RTSP on a doorbell?

No problem.
I just had it enabled to see what other features it provided.  Hadn't actually used any.


fix(Badge): imported type

LiveURL: https://orbit-components-mainframe-missingType.surge.sh

Just noticed that it does not match other imported same types, did not find this type anyway In declarations of sc

It's basically any. Can you change it for React.ComponentType? 🙏 I'm fixing the flow declarations in #1709


PowEmailConfirmation extension can easily be hacked

Hi, I noticed a person can easily ignore e-mail confirmation restrictions in Pow-using app and change e-mail to any e-mail. I uploaded an example git repository for you to easily reproduce this.
Steps to reproduce:
git clone https://github.com/intpl/pow_email_confirmation_issue && cd pow_email_confirmation_issue
vim config/dev.exs
mix deps.get && mix ecto.setup
mix phx.server

create an account using http://localhost:4000/registration/new
copy and open confirmation link from the terminal
sign in
navigate to http://localhost:4000/registration/edit
fill in current password, change email to "hacked@hacked" and submit
it does not change, awaits confirmation, right?
...right?
don't refresh. repeat step 5
email has been changed to "hacked@hacked"

Same issue occurs in my other app with custom controllers. I guess issue is with Pow.Plug.update_user or the extension. What may be helpful is that I could not easily reproduce the issue in tests. On the other hand, I didn't try for long.

Thanks @intpl! I'm able to replicate the issue. Going through Pow now to see how this is possible.

Found the PR culprit: https://github.com/danschultzer/pow/pull/259
Embarrassing bug. I've set up a PR to resolve it in #515. I can confirm that I can no longer reproduce the bug in the demo repo using that PR. I'll release a new version of Pow today or tomorrow. Thanks a lot @intpl!

No wories! Thank you for addressing it on such short notice!!!


SPARKC-114: Ability to publish artifacts to the repository specified in system properties

retest this please

Jacek... testing PR builder... nothing for you to do on my account :-)

retest this please

retest this please

retest this please


"Roseți" is listed in the drop-down menu

Description:
"Roseți" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Roseți" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Roseți, CĂLĂRAȘI (Roseți)".


Issue with de-activation of actor

I'm playing with this very cool implementation.
Not sure if I am doing anything wrong, but it seems that the list of observers (StateManager 'topic') does 'survive' if the the observable actor is being de-activated and then later re-activated (in my case the observable actor should be able to time out and be deactivated, and when activated continue to broadcast).
I get an exception when trying to access:
this.StateManager.TryGetStateAsync<Dictionary<Uri, ObserverInfo>>(topic)
System.AggregateException:
Expecting element 'ArrayOfKeyValueOfanyURIObserverInfo7r8AFj9z' from namespace 'http://schemas.microsoft.com/2003/10/Serialization/Arrays'.. Encountered 'Element'  with name 'ArrayOfKeyValueOfanyURIEntityIdbTQh_Sm3C', namespace 'http://schemas.microsoft.com/2003/10/Serialization/Arrays'.
(ie. in NotifyObserversAsync, when trying to send a message after the observable actor has been re-activated)
Is it by any chance because ObserverInfo cannot correctly be serialized and de-serialized?
Any thoughts on how to solve this?
Best regards,
Thomas

Hi Thomas
Sorry for the delayed answer but it was a crazy time. Yes, you are right, the list of observers survives when an observable is deactivated. This is not an error, it's by design. If you want to change the default behavior, you may add a property to the observable interface that says whever you want or don't want the list of observers to survice to the deactivation of the observable object and then implement the proper logic in the base classes. My 2 cents :)
Mmmm....the ObserverInfo is serializable, otherwise when an observer tries to subscribe to an observable, you would get an error. Could you set a breakpoint on the line of code that throws the exception and see the content of the observer dictionary?
Ciao
Paolo


CB-10698 Missing the env and resource crn from the restarted flows. I…

… have added to all restarted flows.
See detailed description in the commit message.

@schfeca75 could you check from flow perspective?


support for windows

I just thought that it wouldn't hurt to add support for windows systems.

I rebased this in https://github.com/alexcrichton/dlmalloc-rs/pull/35 and handled some comments of mine, thanks for the PR!


[Storage] [STG 96] Added BlobAccessTierNotSupportedForAccountType to BlobErrorCode

API change check
API changes are not detected in this pull request.


Expose panning sensibility and other properties for arc rotate camera

For larger or smaller scenes panning sensibility might not be optimal for users, thus expose the sensibility

This was done via exposure of complete BABYLONJS engine. It is now possible to create scenes, cameras and everything as in BABYLONJS playground.
Refer to this project
https://bitbybit.dev/projects/public/zsBjqCqhygY3EkkqkbaB/project-babylonjs-demo-by-author-matas


Pedestrians see train track junctions as valid crossing points

Extension, but separate issue, to issue https://github.com/VictorPhilipp/Cities-Skylines-Traffic-Manager-President-Edition/issues/144 ...
As far as I can tell, this only affects train tracks (including station tracks) from the workshop. I don't recall pedestrians ever crossing vanilla train tracks (whether in a station or not).
The only time pedestrians should be allowed to cross train tracks is at a level crossing.
I suspect the game code which 'links' pedestrian routes in and around buildings is getting confused by tracks from the workshop. Alternatively, the game might be silently adding junctions to train track nodes but only for tracks in the workshop.
Also, the issue possibly only relates to workshop tracks deployed as mods (eg. Extra Train Station Tracks) - it's possible that tracks deployed as "road" (network) assets don't exhibit the problematic behaviour.
Station which exhibits the problem is linked in https://github.com/VictorPhilipp/Cities-Skylines-Traffic-Manager-President-Edition/issues/144
https://github.com/VictorPhilipp/Cities-Skylines-Traffic-Manager-President-Edition/issues/182

Shouldn't that be fixed by those who created the train tracks in the first place? If vanilla train tracks are not affected then I assume that it should be possible for the creators to prevent cims from crossing train tracks at arbitrary locations.

It's been a very long time since I used a vanilla train station so it's possible the issue happens there too - I just don't recall it, but that was like 5 years ago and a lot has changed since. I've not recently tested my assumption that it's only workshop assets that exhibit the problematic behaviour.
I'll try and get some vanilla stations set up in my current city and report back (might be a few days as the map is riddled with rivers and mountains so finding a nice space for a station and the tracks to reach it is somewhat challenging).

Tagging #787


Update docs

Codecov Report

Merging #59 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   95.86%   95.86%
=======================================
  Files           7        7
  Lines         701      701
=======================================
  Hits          672      672
  Misses         29       29

Continue to review full report at Codecov.

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


Introduction story (scrolling text) is off-screen on mobile devices

The scrolling text needs to be optimized for mobile devices.

I've been tinkering with the CSS, trying to get the heights right... but no luck. I'm testing on an iPhone 6 and an old iPad mini. Weird how the mobile browser totally interprets the height % selector different from the desktop browser.


AD: PowerLink is finally blocked, but there is a slight problem.

URL
https://namu.wiki/w/비트소닉/제품 목록/사운드 어댑터/토요타
FILTERS
https://github.com/List-KR/NamuLink/issues/78
Same as the list of the corresponding thread.
Browser
(Other)
Operating System
Android
Ad-Blocker
AdGuard for Android
Userscript Manager
Tampermonkey
USERSCRIPT
https://github.com/List-KR/NamuLink/issues/78
Same as the list of the corresponding thread.
Comment

When you first access Namuwiki, a blank space appears in the power link area at the top and the power link at the bottom is displayed as is, but when you refresh, both are removed.  It was tested on Kiwi Browser and SM-G955N, and is presumed to be the same for other browsers.

@Githubmaniak
Please update to 3.11.0 and try again.


Write fixed versions

Currently we are generating the version, that ends quite often in confusion so we should skip that and write the versions statically.

Git hash is very useful to find the problem. Fixed version cannot help us on that.

agree with @lunny.

We should just take the tag name + the last commit. This is something set by the CI. That's not so confusing like the crappy versioning that is currently used. I just say 1.0.0+388-g60db7ed5 instead of a proper version like 1.1.0+dev or 1.1.0+dev-60db7ed5 or 1.1.0+60db7ed5.

Isn't the currently generated version already using tag name + last commit ? I see 1.1.0+10-ge2b2fd6e on try.gitea.io, why do you think it's bad ?

As a user, when I download and install a tag (release) version, I want to see that tag when I start the application.
A tag version does not change. So, if I file a bug against a tag version, you have all the necessary information.
When I look at my logs, I want to see the version I am running, not its hash tag. I don't want to come here trying to find the exact commit.
I am running a version, not a commit...
Thank you very much.

I'm the package maintainer for the FreeBSD package of Gitea. The FreeBSD port downloads the respective release as a tar from Github; git never enters the picture. Right now, building the release 1.1.2 makes gitea info gitea show a version 1.0.0+dev (also in the web pages), which is confusing to users of the package.
My Go knowledge is not sufficient to suggest an implementation, but releases should integrate the actual release version instead of a fallback.
See also https://github.com/stblassitude/gitea-port/issues/1

I think it should be just tag when release version and version+hash when built from master


Toast notifications are not displayed when passing a react node as briefMessage

Describe the bug
When passing a ReactNode as briefMessage to display a toast notification, the toast notification is not displayed.
To Reproduce
Steps to reproduce the behavior:

Create a ReactnotifyMessageDetails with a reactNode as parameter for the briefMessage:
ex:
const msg = new ReactNotifyMessageDetails(OutputMessagePriority.Info, { reactNode });
Set display time and make sure the message type is toast:
ex:
msg.displayTime = BeDuration.fromSeconds(7.0);
msg.msgType = OutputMessageType.Toast;
Output the message:
ex:
MessageManager.outputMessage(msg);

Expected behavior
The toast notification should be displayed
Desktop (please complete the applicable information):

OS: Windows
Browser Chrome
Version 109.0.5414.120
iTwin.js Version 3.6.0

Additional context
Same problem if I try using MessageManager.displayMessage instead.
This behavior worked on previous versions

The fix will be available in<EMAIL_ADDRESS>

<EMAIL_ADDRESS>was released on Feb 24th.


A small step towards removing C-style loops

This is a small step towards removing C-style loops in favor of Swift for/in loops as presented in this accepted proposal:https://github.com/apple/swift-evolution/pull/23/commits . Nothing big here, just getting my feet wet and trying help improve code quality :)

I'll run tests and merge.

@dabrahams Just noticed your comment.  I'll take care of that.

@gribozavr I don't mean to sound cheesy, but this is the first commit I've ever made to a programming language ever. Thank you :smile:

@hamin You're welcome! :100:


Delete stale placeholder policies

There are a few placeholder policies which the policy-publisher
doesn't know about.
This is because those policies were renamed at some point before
the migration to publishing-api was completed.
We should delete these manually.
We already introduced another PR to deal with publishing-api.
See
https://trello.com/c/xARBi6HF
and
https://github.com/alphagov/publishing-api/pull/356

Closed in favour of doing a unpublish API call to the publishing-api.


Added Gavilan College domain

As per the instructions on the JetBrains website ( https://www.jetbrains.com/shop/eform/students/contributeUniversity?email=ann.orman%40my.gavilan.edu ) added the EDU domain name for gavilan.edu
Our college website is https://www.gavilan.edu/

@AnnOrman-edu I am sorry, the domain gavilan.edu is blacklisted because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).

This is not possible, to create an email account we need to work with our
MIS department or get approval from the Dean.
The procecess for it is here -
https://www.gavilan.edu/staff/info/communication.php
On Mon, Feb 10, 2020 at 4:02 AM Philip Torchinsky<EMAIL_ADDRESS>wrote:

@AnnOrman-edu https://github.com/AnnOrman-edu I am sorry, the domain
gavilan.edu is blacklisted because the school allows to register an email
address there even if you are not a student of the school. If you are the
student of this school, please apply with official document (OFFICIAL
DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/7711?email_source=notifications&email_token=AOQJUDJO5WQRUF6J2WYRFEDRCDGVVA5CNFSM4KSEGEGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELHE5PQ#issuecomment-583945918,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOQJUDJJFQ2WNKBHGAJ7JDTRCDGVVANCNFSM4KSEGEGA
.


Crash when evaluating a particular recording

I have incorporated "@spotify/basic-pitch": "1.0.1" into my website.
I'm getting good results from most of my test files, but this one particular file crashes: https://production--endearing-hamster-0de147.netlify.app/audio/paul_001.mp3 (Note that this URL is good for a while but will eventually go away.)
In the console I see:
webgl_util.ts:116 Couldn't parse line number in error: ERROR: 0:140: 'getT1' : no matching overloaded function found
#version 300 es
    precision highp float;
    precision highp int;
    precision highp sampler2D;
    in vec2 resultUV;
    out vec4 outputColor;
    const vec2 halfCR = vec2(0.5, 0.5);
etc...

Then it throws the following error:
Error: Failed to compile fragment shader.
    at createFragmentShader (webgl_util.ts:106:11)
    at compileProgram (gpgpu_math.ts:113:26)
    at backend_webgl.ts:942:25
    at _MathBackendWebGL.getAndSaveBinary (backend_webgl.ts:997:31)
    at _MathBackendWebGL.runWebGLProgram (backend_webgl.ts:941:25)
    at concatImpl2 (Concat_impl.ts:123:26)
    at concatImpl2 (Concat_impl.ts:103:26)
    at Object.concat3 [as kernelFunc] (Concat.ts:49:10)
    at kernelFunc (engine.ts:644:22)
    at engine.ts:710:23

Thanks for this amazing library!

It does the same thing when I drop that file onto https://basicpitch.spotify.com/

Hi @paulrosen , can you provide the audio that you're using? In the future, please use basic-pitch-ts for issues related to the typescript library.

The audio is here: https://production--endearing-hamster-0de147.netlify.app/audio/paul_001.mp3
(With some experimenting, I found that if I shorten the mp3 just slightly it works fine.)

Well, it would be nice if the original audio worked. I will be submitting random audio so it will fail periodically. What my workaround looks like is:

catch the error
shorten the audio by 0.3 seconds
retry
repeat until it works

The drawback of that is:

there is a huge amount of info dumped to the console, which I'd rather not have,
redoing the audio will take a little time, so the user will have to wait,
there is probably a little bit of extra space at the end of the file, but chopping it off might get rid of part of the last note.

Also, I've tried about 30 recordings so far and 3 of them failed like this, so this isn't a particularly rare case.

If you change the encoding or resample the file does it still fail? Also has it always been files of this length?

I just tried resampling the file to 22050Hz and 44100Hz and converting the file to ogg. I'll take a deeper look.

All the files tend to be different lengths. I didn't see any pattern having to do with file length.
Adding zeros is a better idea for a workaround, thanks!

Here's another example: https://production--endearing-hamster-0de147.netlify.app/audio/mBpJ9URFpMuTXg1HuOaG.mp3

Is there any progress on this? Is there anything I can do to help? I don't know anything about tensorflow so I'm not sure if I could get up to speed.

No progress. I've been a bit busy on #100 and admittedly forgot about this issue. Hopefully, I can get to this issue in a week or two. I do wonder if tfjs has been updated with a fix since you posted this issue. I can check sometime next week.

This sort of issue tends to be an edge case for whenever an audio file is chunked up and processed that way. Most likely, some chunk (probably a trailing chunk) is too small for some part of the code. This is why it doesn't have to do with any specific file length, but instead has to do with multiples of some chunk length with a tail chunk that's too small.
If I catch it I'll see what I can do as well.


Scripts to make a new vm container host do not work.

When I try to run the quick start code located here. The script errors out trying to validate versions of tp5. Sorry but I dont know where else to post this issue.
https://github.com/Microsoft/Virtualization-Documentation/blob/live/virtualization/windowscontainers/quick_start/quick_start_configure_host.md#scripted---new-vm---1--

@neilpeterson  ?  any update on this? is the package in the powershell gallery wrong?

I ran this today, and it worked for me:
wget -uri https://aka.ms/tp5/New-ContainerHost -OutFile c:\New-ContainerHost.ps1
powershell.exe -NoProfile -ExecutionPolicy Bypass c:\New-ContainerHost.ps1 -VMName TP5Host_0505 -WindowsImage ServerDatacenterCore -Hyperv

My VM host is also running Windows Server 2016 TP5. Is your host running TP5 or another version?

Please make sure you have the latest ContainerImage package provider

I am running from a win10 host. it looks like I have ContainerProvider install, the containerImage package will not import, it needs powershell 5.1. I will just call it a day and create a azure vm from the image.

We no longer support this script as the method to set up containers on your host. Please see this quick start guide for the new method of doing this.


Fixing Alarm TimePicker

The Alarm TimePicker text field was only 256px wide, i don't know about
other languages, but in Russian caption just didn't fit. Stretched it to
the window width, now it's looking nice.

Screen shot before/after?

Before:


Electron version used is vulnerable - severity critical

The version of electron use actually (1.7.9) is affected by a critical vulnerability (CVE-2018-1000006)
https://electronjs.org/blog/protocol-handler-fix
Please upgrade it

Hey,
thanks for the notice, 0.13.0 is out in the wild with updated Electron.


Missing translation for 'Incorrect Password'

forms.py line 234

Thank you for reporting this issue. This will be fixed in v0.6.9.

Released in v0.6.9.
Still need proper translation string for each translated language.


fix(transformer): fix TS annotation transform scopes

#3817
#3816  👈
main

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

Join @overlookmotel and the rest of your teammates on  Graphite


Moving resources into idle

(I'm not sure if this is the right place for feature requests that are not formal RFCs, or if I should rather open an issue in the RFC repo. Please let me know, if you want me to move this.)
RTIC provides resources that a context has exclusive access to as mutable references. This makes sense for interrupt handlers, but (unless I misunderstand something) it is not the only option for the idle function: In principle, resources that idle has exclusive access to could be moved there outright.
Usually this doesn't matter, but there are APIs that require you to move values into and out of them. One such case I encountered is a DMA API that requires me to move a Channel struct into the Transfer struct, while the transfer is going on. Unless I'm missing something (I couldn't find anything related in the documentation), I can't use this API in my idle function (without further complications), because I only have a mutable reference to the Channel.
For now I'll just use Option::take to work around this problem, so this isn't very high-priority to me. Still, it would be nice to have this.

Now that I think about it, it should also be useful and possible to move resources into the interrupt context, as long as they're moved back out.
Something like this:
#[task(binds = DMA0, resources = [transfer])]
fn dma0(cx: dma_int::Context) -> dma_int::Context {
    // Context contains a transfer that is in progress, which we're handling
    // the interrupt for. `wait` transforms the type state, thus requires us to
    // move out of the context.
    let transfer = cx.resources.transfer.wait();

    // TODO: Process received data here.

    // Prime the transfer again. `start` transforms the type state back to what
    // it originally was.
    cx.resources.transfer = transfer.start();

    cx
}

Again, nothing that can't be emulated with Option::take, and I'm not sure if adding this as an option would be worth the complexity. I just want to leave it here as further food for though.

Hi,
When it comes to type states they don't really play nice with RTIC as of today (as you have seen).
What I usually do is wrap the type-state in an enum and use the following code to achieve what you are looking to do (with the replace_with crate):
pub fn change<F>(&mut self, mut closure: F)
where
    F: FnMut(Self) -> Self,
{
    unsafe {
        replace_with::replace_with_or_abort_unchecked(self, |val| closure(val));
    }
}

This allows the following usage, lets say for an RF chip:
/// Wrapping enum for type-states
pub enum Wrapper<...> {
    Uninitialized(Chip<...>),
    Sleeping(Chip<...>),
    Idle(Chip<...>),
    Sending(Chip<...>),
    Receiving(Chip<...>),
}

// ... usage bellow

self.change(|wrapper| match wrapper {
    Wrapper::Receiving(driver) => driver.finish_receiving().into(),
    Wrapper::Sending(driver) => driver.finish_sending().into(),
    _ => wrapper,
});

I have somewhat similar problem. Prior to 0.6 it was possible to implement DMA transfers like this:
    #[task(binds=EXTI0, resources = [led, tele, extih])]
    fn handle_mpu(ctx: handle_mpu::Context) {
        let led = ctx.resources.led;
        let _ = led.set_low();
        let maybe_tele = ctx.resources.tele.take();
        if let Some(tele) = maybe_tele {
            let new_tele = tele.send(|b| fill_with_str(b, "interrupt!\n"));
            *ctx.resources.tele = Some(new_tele);
        }
        ctx.resources.extih.unpend();
    }

with "symmetric UI" I'm forced to use locks, which don't move my resources:
    #[task(binds=EXTI0, resources = [led, tele, extih])]
    fn handle_mpu(mut ctx: handle_mpu::Context) {
        ctx.resources.led.lock(|led| led.set_low());
        ctx.resources.tele.lock(|maybe_tele| {
            if let Some(tele) = *maybe_tele {
                let new_tele = tele.send(|b| fill_with_str(b, "interrupt!\n"));
                *maybe_tele = Some(new_tele);
            }
        });
        ctx.resources.extih.lock(|extih| extih.unpend());
    }

^ this fails with
error[E0507]: cannot move out of `maybe_tele.0` which is behind a mutable reference
   --> dma_int/main.rs:139:33
    |
139 |             if let Some(tele) = *maybe_tele {
    |                         ----    ^^^^^^^^^^^ help: consider borrowing here: `&*maybe_tele`
    |                         |
    |                         data moved here
    |                         move occurs because `tele` has type `DmaTelemetry`, which does not implement the `Copy` trait

error: aborting due to previous error

Does this mean that using unsafe and 3rd party crate (replace_with) is mandatory, when using rtic?

I have somewhat similar problem. Prior to 0.6 it was possible to implement DMA transfers like this:
    #[task(binds=EXTI0, resources = [led, tele, extih])]
    fn handle_mpu(ctx: handle_mpu::Context) {
        let led = ctx.resources.led;
        let _ = led.set_low();
        let maybe_tele = ctx.resources.tele.take();
        if let Some(tele) = maybe_tele {
            let new_tele = tele.send(|b| fill_with_str(b, "interrupt!\n"));
            *ctx.resources.tele = Some(new_tele);
        }
        ctx.resources.extih.unpend();
    }

with "symmetric UI" I'm forced to use locks, which don't move my resources:
    #[task(binds=EXTI0, resources = [led, tele, extih])]
    fn handle_mpu(mut ctx: handle_mpu::Context) {
        ctx.resources.led.lock(|led| led.set_low());
        ctx.resources.tele.lock(|maybe_tele| {
            if let Some(tele) = *maybe_tele {
                let new_tele = tele.send(|b| fill_with_str(b, "interrupt!\n"));
                *maybe_tele = Some(new_tele);
            }
        });
        ctx.resources.extih.lock(|extih| extih.unpend());
    }

^ this fails with
error[E0507]: cannot move out of `maybe_tele.0` which is behind a mutable reference
   --> dma_int/main.rs:139:33
    |
139 |             if let Some(tele) = *maybe_tele {
    |                         ----    ^^^^^^^^^^^ help: consider borrowing here: `&*maybe_tele`
    |                         |
    |                         data moved here
    |                         move occurs because `tele` has type `DmaTelemetry`, which does not implement the `Copy` trait

error: aborting due to previous error

Does this mean that using unsafe and 3rd party crate (replace_with) is mandatory, when using rtic?

I just came across this issue while randomly browsing. On Matrix, @jamesmunns recently suggested moving resources into idle. And I had a similar revelation, that resources could be moved into tasks, as long as they get moved back out.
It's not necessary, but I think it would be a nice add-on feature for RTIC to support this. It would give RTIC more expressive power, allowing it to match the semantics of the corresponding resource. Otherwise, you have to use the shoe-horn approach of replace_with.

This sounds like a reasonable extension to RTIC.

To my understanding a "move" resource cannot be shared between tasks. With the replace_with approach that is possible.

We need some syntax to define a resource as being "moved", perhaps mark it by #[move] in the declaration.

For idle moving in would amount to the field in the context actually containing the initial state of the moved resource. I think it may be possible to do this totally on the stack, so no backing heap representation of the moved resource.

For tasks, it is similar BUT the moved resource(s) would need to be returned in their initial state(s). Not sure if it is that useful at the moment. However, if we implement async/await for tasks, this might be more useful. It might be possible to capture the type state in the context of the async/await "state machine" (without backing heap storage), but it depends on how we choose to implement async/await. In general its still a no-go until we can allocate storage for async/await context statically (not yet stabilized as far as I know).

With async 0 priority tasks this is now supported.


Fuzzyness value per search

Hi,
Is your feature request related to a problem? Please describe.
Would be nice to have fuzzyness value modifable per search, I have found better results in some searches when changing this value or making a custom rule
Describe the solution you'd like
A new search param for indicate fuzzyness value.
Maybe it worth to disable fuzzyness by param, so if someone wants to do a search with exact match, it could get it, something like searching for an id, product code, etc.
Additional context
ES example reference: https://www.elastic.co/guide/en/elasticsearch/reference/current/common-options.html#fuzziness

Hello! Sorry for the delay in answering your questions. It's not possible to customize the number of typos allowed ATM. Don't hesitate to create an issue if you think it should be an option :)
Originally posted by @CaroFG in https://github.com/meilisearch/MeiliSearch/discussions/1096#discussioncomment-259870

Hey @sonic182,
This is indeed a feature that we want to implement at some time, I am not sure if it has already been requested but you can use the roadmap.meilisearch.com website to request it by yourself.
Thank you very much, have a great day!

is not on the readmap anymore ... but i found it in https://github.com/orgs/meilisearch/discussions/377 ...

is not on the roadmap anymore ... but i found it in https://github.com/orgs/meilisearch/discussions/377 ...

Re-directed to https://github.com/orgs/meilisearch/discussions/727


Authorization Header is not beared.

  [Header("Authorization")]
        AuthenticationHeaderValue Authorization { get; set; }

            this.BasicApi = RestClient.For<IAcesoBasicApi>(baseUrl, async (request, cancellationToken) =>
            {
                if (this.Token == null)
                {
                    return;
                }
                this.BasicApi.Authorization = new System.Net.Http.Headers.AuthenticationHeaderValue("Token", Sign());
            });

BasicApi is an api interface.
After requesting remote, api server response: Authentication header missing.

That code is being called after the HttpRequestMessage has been created. Changing properties on IAcesoBasicApi will affect the next request, but the current request has already been created.
You probably want to be assigning to request.Headers.Authorization. That will change the current request.

Thank you, solved. My first mistake is what you mentioned, and another mistake is that Apache route rewriting do not redirect headers.
This .htaccess file may be helpful for someone facing same problem:
RewriteEngine On
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ /index.php?/$1 [L]
SetEnvIf Authorization .+ HTTP_AUTHORIZATION=$0 # this line is required


Error with resmico evaluate:   OSError: [Errno 9] Bad file descriptor

Hi,
I got a new error with resmico evaluate.   OSError: [Errno 9] Bad file descriptor    Please see the following screenshot.

I am sorry for so many issues.
Thanks.

Hi @Yunliang81,
this error should not affect the predictions. Did you try to access predictions/default-model.csv?

resmico evluate just identifies misassembled contigs. resmico filter filters the contigs.

@nick-youngblut
Thanks, but I am still confused.
If resmico evalutae doesn't work properly, there should be no misassembled contigs identified, right?   Why there some contigs were removed by resmico filter  without resmico evaluate step?
Thanks,

Can you please provide the full resmico filter command that you ran?


⚠ 2 Minutes de génie (gitlab page) has degraded performance

In 4c9adb3, 2 Minutes de génie (gitlab page) (https://2mdg.gitlab.io/) experienced degraded performance:

HTTP code: 502
Response time: 3162 ms

Resolved: 2 Minutes de génie (gitlab page) performance has improved in b13c089.


Anchor link "to" prop should be of LocationDescriptor | string Type

it will add functionality of adding state in location while routing from Anchorlink

This seems legit. attaching type for reference.
Is the pr created. If not I will create one
export type LocationDescriptorObject = {
    pathname?: Pathname;
    search?: Search;
    query?: Query;
    state?: LocationState;
};


Ported to 1.18.2 and fixed few bugs

Thanks for your excellent mod!
This is a 1.18.2 port. Mainly focused on updating dependencies and migrating Minecraft source mappings from 1.16.5 to 1.18.2.
Changes

Removed FlexCrateRegistry as AdjustableCrateBlock is removed in after Create v0.4.0.
Added MountedStorageManagerMixin to access mounted storage info. MountedStorageManager is used in Create 1.18.2 to manage mounted inventories.
Temporarily removed Storage Overhaul support () as they do not support 1.18.2 for now.
Replaced OverWrites with Injects for better compatibility and removed OverWrite of serialize().

There are some details that I'm not sure about:

Is Overwriting serialize() necessary? It seems to work just fine without it.
In Create 0.5.0c there is no specific code for Double Chest anymore (here), is DoubleChestItemStackHandler still necessary? It seems little broken according to my tests, leading to ArrayIndexOutOfBoundsException.

Besides the port, I'm trying to fix some minor bugs and make mixins more portable with newer versions of Create, but I'm not sure whether they are the best practice or not. It would be good if you would take some time to review them, and please let me know if any improvement is needed!
Fixes

Fixed Trash Can's nbt not synced to client when readded to world after being disassembled.
Fixed Storage Drawer's upgrades not synced and not recognized by temp handlers.
Fixed Industrial Foregoing's Black Hole Unit handler holding only 64 items every time assembled into contraptions.

Wow, thanks! I will take a closer look at this later today.
Regarding your two points, I don't remember exactly, so I will answer after I take a look at code again

1.18 create doesn't care about order of first/second chest, so I think we shouldn't either. Right now, priority 1 is used by enderchest-like storages, so setting one of the chests to it will mess up insertion ordering. Tho, I think I should move endercehsts into their own priority at like 5, and then double chest insertion priority can be kept

Oh, I see your point. I agree with you that the behavior of double chest should stay the same with Create, because double chest insertion priority seems to add unexpected complexity to this mod. The commit about this should be reverted later.
I am thinking that should there be some extra logic like Create here? According to my understanding, the logic here is to mount the whole double chest as long as any one of the two chests is glued onto the contraption.
Vanilla double chest with only right part glued onto contraption.

Expanded Storage's double chest with only right part glued onto contraption.

Wow, this is very useful, thanks! I have some troubles setting up my IDE for minecraft modding, it just broke at one point, I will try to merge as soon as I fix my IDE

I understand you. Java dev environment is just always annoying. Thanks for your attention for this! I didn't mean to take up too much of your time, so no need to rush😉.

I'm really sorry that it's taking so long, I wasn't able to fix my IDE quickly when I was free, amd rn I'm very busy at work, so have no mental power to work on hobby projects. I will really try to look into it more this week

hey, that's just fine. I had a busy month too and left some remaining part unfinished here. Probably finish it later when I'm free.
After all, it's just a hobby to me as well. I'm not in a hurry. It would be good if you care about the comments, but you don't have to. It's okay to just leave it there when not interested.

For anyone who would like to test it out, this is a snapshot build for mc 1.18.2 + create 0.5.0c.
https://jitpack.io/com/github/yanang007/MoreMountedStorages/c77e13ce/MoreMountedStorages-c77e13ce.jar
As for now, versions for mc 1.18.2 + create 0.4 or for mc 1.18.1 is skipped. They may be considered if they are really needed.

For anyone who would like to test it out, this is a snapshot build for mc 1.18.2 + create 0.5.0c.
https://jitpack.io/com/github/yanang007/MoreMountedStorages/c77e13ce/MoreMountedStorages-c77e13ce.jar
As for now, versions for mc 1.18.2 + create 0.4 or for mc 1.18.1 is skipped. They may be considered if they are really needed.

For anyone who would like to test it out, this is a snapshot build for mc 1.18.2 + create 0.5.0c. jitpack.io/com/github/yanang007/MoreMountedStorages/c77e13ce/MoreMountedStorages-c77e13ce.jar
As for now, versions for mc 1.18.2 + create 0.4 or for mc 1.18.1 is skipped. They may be considered if they are really needed.

does this work for 0.5d?

Seems like I won't be able to get any free time soon, so @yanang007 if you wish to, you can release MMS port for 1.18.2 as a separate curseforge project, I allow use of any assets in this repository and on old curseforge page.
If you won't, I will still try to get to this as soon as possible, but no idea when that "soon" will come

@genesy I've made a new version for v0.5.0.d, please refer to my original comment for the download url for the snapshot build.
@juh9870 I will look into it at my free time, but it would still be good if all the versions are held in one project.

If you won't, I will still try to get to this as soon as possible, but no idea when that "soon" will come
Seems like official port is not planned
@yanang007 do you plan to release your port? (As separate CurseForge project, as I understood @juh9870 allowed that)
If you don't plan to keep it updated, I can look into becoming your port's maintainer

If you won't, I will still try to get to this as soon as possible, but no idea when that "soon" will come

Seems like official port is not planned.
@yanang007 do you plan to release your port? (As separate CurseForge project, as I understood @juh9870 allowed that) If you don't plan to keep it updated, I can look into becoming your port's maintainer

That would be great! Feel free to create a fork. I'm totally fine with it as long as @juh9870 agrees.

Hello, yes, you can release it for 1.18 and other versions. I allow reusing icon and other stuff in this repository

Hello, yes, you can release it for 1.18 and other versions. I allow reusing icon and other stuff in this repository

Okay, I'll be checking the fork and releasing it soon if it's stable
@yanang007 can you give your Curseforge username so I can list you as a contributor?

I appreciate that, but I actually don't have a Curseforge account.

Will there be a port to 1.19.2?

I'll look into that on this weekend, but I'll publish 1.18.2 version first. I want to test it and make sure everything is all right before releasing it.

CF project got approved, here's the link: https://www.curseforge.com/minecraft/mc-mods/create-mounted-storages
Would appreciate the link in this mod page if that's possible

I've added a link to my project description. Also note, that alpha releases can't be added to modpacks

Thanks! I've marked it as alpha for now before I test it thoroughly but it looks stable enough so I will change it to beta


Fix missing parenthesis in docker-deployment.md

Fixes a small typo and a missing parenthesis on the Docker deployment page.

https://deploy-preview-3664--opentelemetry.netlify.app/docs/demo/docker-deployment/#bring-your-own-backend

thank you @heitortsergent


Fix missing version flag on update do subcommand.

Still does not work though. :(

Rebased this commit and tried again. For some reason it now seems to work:
❯ cloudctl update do --version v0.12.16
checksum:b3cb5325c3abd1a473079c533008ded8
54.76 MiB / 54.76 MiB [------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00% 6.80 MiB p/s 8.3s
❯ cloudctl version
client: v0.12.16 (1b002a4d), tags/v0.12.16-0-g1b002a4, 2024-02-02T05:53:02+00:00, go1.21.6
server:
    builddate: "2024-04-25T09:37:41Z"
    gitsha1: 649541f7
    minclientversion: v0.12.18
    name: cloud-api
    revision: pipelines/1243265-0-g649541f
    version: 649541f


Auto register for custom social login provider

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
I did setup a custom SocialLogin Provider using the Logical Theme System. This works as expected with existing users.
Now, I want to enable "auto register" for this provider (just like e.g. TWITCH_AUTO_REGISTER=true in .env file).
Is there any way to add some configuration to enable auto register for custom providers, without editing source files?
Exact BookStack Version
v22.02.3
Log Content
No response
PHP Version
8.0
Hosting Environment
Ubuntu 20.04 VPS

No bother at all! Happy to hear it works!


Cutting a new release

Hello :)
I am wondering when you plan on cutting a new release for tetragon? The last one was in May, and the code has both been improved and updated since then.

That's a good point! We do not have release management scheme yet, but we plan to do a minor release with all the updates since May soon.

This has been done :) thanks


default to unknowns

Codecov Report

Merging #535 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #535      +/-   ##
==========================================
- Coverage   92.29%   92.25%   -0.04%
==========================================
  Files          37       37
  Lines        2348     2349       +1
  Branches      202      202
==========================================
  Hits         2167     2167
- Misses        136      137       +1
  Partials       45       45

Impacted Files
Coverage Δ

api/app/data.py
41.23% <0%> (-0.43%)
:arrow_down:

Continue to review full report at Codecov.

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


ES-2475: Wrap create RBAC role request in retry

Created branch in e2e-test repo: https://github.com/corda/corda-e2e-tests/pull/664
Tests running: https://ci02.dev.r3.com/job/Corda5/job/corda5-e2e-tests/job/PR-664/

Alpha artifacts tested successfully on CI05
https://ci05.dev.r3.com/job/QA/job/CRAFT5/job/Craft_Tests/job/C5/job/Playgrounds/job/Ledger_Playground/job/main/93/
https://ci05.dev.r3.com/job/QA/job/CRAFT5/job/Craft_Tests/job/C5/job/Playgrounds/job/Ledger_Playground/job/main/94/


Fix markdown rendering

Fix rendering bug caused by PR #186.
Simplify nested "if".

Maybe you should wait for #1527 before touching this file. \cc @lunny

Have you give an example of this bug on https://try.gitea.io?

@strk I don't think so.

Should fix #1273, #1265.

@lunny https://try.gitea.io/morli/test
After fix:

Yeah, looks like an issue I've missed. Thanks, @Morlinest!
It would also be great to have tests written for this case.

LGTM

LGTM

LGTM


iterate plain object's prototype key

the code in iterate.js
// The target is a plain object
for (k in target)
  callback(target[k], k);

should add a test of target.hasOwnProperty(k)

You are right indeed @chaosforfun. Do you want to open a PR to submit your fix?

ok


Generate a test image that corresponds to that part of the code.

Hi,
Thanks for sharing the code!!!
Why did I test with MSMT17 and only get map data, but not the generated images.
Generate a test image that corresponds to that part of the code.

你好，
感谢分享代码！！！
为什么我用msmt17进行测试最后只得到了map数据，而没有得到生成的图片。
还有就是生成测试后的图片，对应着那部分的代码。 @ @CHENGY12

Thanks for your interest in our paper.
APNet is a discriminative model for the person ReID task. Given a benchmark (including MSMT), APNet extracts the features of both query and gallery images and matches them for evaluation. This evaluation process is not related to image generation.   The details can be found in the paper.  Hope you find it helpful.

不好意思我可能能没表达清楚，我想要的是数据可视化的代码。

具有不同金字塔级别的注意力图的可视化。我们采用 Grad-CAM 来可视化我们的注意力金字塔的学习注意力图。

@CHENGY12  Can you share the code for your visualization part? I really won't, thanks

@CHENGY12 Can you share the code for your visualization part? I really won't, thanks

Hello,
As mentioned in our paper, we used Grad-CAM to visualize the attention map. Please consider using the open-source implementation of Grad-CAM such as https://github.com/jacobgil/pytorch-grad-cam.

@Gutianpei thank you


[HOW-TO] record YUYV video from USB

Please only ask one question per issue!
How to record YUYV video from USB camera?
Describe what it is that you want to accomplish
Add control properties to enable recording YUYV video from USB camera.
Describe alternatives you've considered
First I changed encoder.py format() to enable YUYV by adding:
       elif value == "YUYV":
            self._format = V4L2_PIX_FMT_YUYV

But this fails with
     min_frame_duration = self.camera_ctrl_info["FrameDurationLimits"][1].min
 KeyError: 'FrameDurationLimits'

If I set the required properties (per section 4.2.3 of https://datasheets.raspberrypi.com/camera/picamera2-manual.pdf)
self.video_config = self.picam2.create_video_configuration(
     main = {'format': format},
             display = 'main',
             controls = {
                'NoiseReductionMode': libcamera.controls.draft.NoiseReductionModeEnum.Fast,
                'FrameDurationLimits': (33333, 33333)})

I fails with
RuntimeError: Control NoiseReductionMode is not advertised by libcamera

The error originates at self.controls.set_controls(self.camera_config['controls']) at the end of configure() in picamera2.py
It seems like I can't record video, because I can't set an encode frame rate, because libcamera didn't supply (or maybe picamera2 doesn't retrieve) the original info.
Not supplying the info is understandable, as it may not exist.
But It seems I should be able to set encoding frame rate regardless of camera source.
How do I set this (synthetic) frame rate?
Additional context
Add any other context or screenshots about the request here.

Hi, I'm afraid this isn't supported. Even if all these issues are fixed, recording H264 is only known to work for sensors that output dmabufs and may well fail in other cases.  Alternatively the JPEG encoder won't accept YUYV either, you'd have to convert the buffers to some form of RGB. We have no plans at this time to extend USB camera support.
Sorry not to be more helpful. It might be worth looking at gstreamer for this kind of use case?

I realize this isn't a direct answer to the question of how to record to disk, but this is an answer of how to stream YUYV video from a USB camera.  As @davidplowman said, the YUYV format isn't supported in picamera2's H.264 encoder, but it is supported in gstreamer's v4l2h264enc when described as YUY2.
So this is a gstreamer pipeline that can stream video:
libcamerify gst-launch-1.0 v4l2src device=/dev/video2 ! video/x-raw,format=YUY2,width=256,height=192  ! videoscale   ! v4l2convert ! v4l2h264enc extra-controls='controls,repeat_sequence_header=1' ! 'video/x-h264,level=(string)4,video_bitrate={selected_bitrate}' ! tcpserversink host=<IP_ADDRESS> port=8000

I don't think Picamera2 supports YUYV at the moment, but I don't think there's any reason why it couldn't - someone would just have to do the plumbing!
The h.264 hardware encoder probably won't play nicely with a USB cam, but the latest software has a "libav_h264_encoder" which probably would, so you could try that.
Generally, I don't mind extending the USB camera support within Picamera2, but it's not really what we're targeting so are unlikely to do extra work on it ourselves.


Fix camera movement key from D to Z

D is duplicated in the hotkeys section. The correct keybinding is Z.

Thank you @rcjsuen!

Thank you @rcjsuen!


[TR] PIM-4914: Made raw configuration accessible from configurable step elements

Q
A

Specs & Static
Y

Changelog updated
Y

Related PRs
https://github.com/akeneo/pim-community-dev/pull/3342

We already have access to configuration item directly from step element, so it's maybe not necessary to provide the full configuration.
I agree the current system has limitations (I would even say it's kinda broken), but in the same time I would prefer to find a better overhaul of the solution than breaking the current encapsulation.
A new solution to manage configuration would need some more design ;)

As discussed, I will write a custom writer in the PIM, I'm closing this PR


Update the build number

Rebuilding solely in the hopes of getting gmsh and pysparse
to build with the same version of mkl

Checklist

[X] Used a fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

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

@conda-forge-admin, please rerender


Add UI for Http Request node headers

fixes #3428
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes
Add UI for headers...

NOTES:

If header "FOO": "BAR" is sent in msg.headers and is NOT present in the UI, then "FOO" is added to req.headers
If header "FOO": "BAR" is sent in msg.headers and is "FOO": "BAZ" in the UI, then the "FOO" header is overwritten with "BAZ" in req.headers
If header "FOO": "BAR" is sent in msg.headers and is "FOO": "" (i.e. no value) in the UI, then "FOO" is deleted from req.headers
Header names are Camel Case by default
Header names are set in the case specified by the user
If user wants to send content-type instead of Content-Type then can either add it to msg.headers or add a UI entry set to "other" and name it Content-Type

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.07%) to 67.39% when pulling e57774e1215600b669ab126187767a347fca149e on Steve-Mcl:httpreq-headers-ui into 6a5c50ff774f174f1788e12ad9fd17557f1eaa81 on node-red:dev.


[TASK-ORDER] Enhanced Item Module

Background
Need to enhance item management with several configuration to support management operational
Item configuration

Item can be configured as available for sell or not
Item stock status can be watch to enhance stock availability
If item cant be sell, it's price must can be configured following several payment methods
Item storing location label information for each stock point
Master item import data from csv. Consider for accounting numbering
Specification management (Consider as List of Values)

Objective

✅ Item can be configured as available for sell or not
Item stock status can be watch to enhance stock availability
If item cant be sell, it's price must can be configured following several payment methods
✅ Item storing location label information for each stock point
Master item import data from csv. Consider for accounting numbering
✅ Specification management (Consider as List of Values)


fix: use boolean fn for option keys in template-data

Fixes #106
Tested against #105 because I introduced #104 in master (sorry again) as follows:

Before updating the code, get a baseline of new projects for each option:

PROJDIR=`pwd`
DIFFDIR=`mktemp -d`

# Create a directory in DIFFDIR for each option
mkdir $DIFFDIR/0
mkdir $DIFFDIR/10x
mkdir $DIFFDIR/breaking-point
mkdir $DIFFDIR/cider
mkdir $DIFFDIR/garden
mkdir $DIFFDIR/handler
mkdir $DIFFDIR/less
mkdir $DIFFDIR/re-com
mkdir $DIFFDIR/re-frisk
mkdir $DIFFDIR/re-pressed
mkdir $DIFFDIR/routes
mkdir $DIFFDIR/test

# Create a project for each option, move to DIFFDIR
lein install
lein new re-frame aa                 && mv aa $DIFFDIR/0/project
lein new re-frame aa +10x            && mv aa $DIFFDIR/10x/project
lein new re-frame aa +breaking-point && mv aa $DIFFDIR/breaking-point/project
lein new re-frame aa +cider          && mv aa $DIFFDIR/cider/project
lein new re-frame aa +garden         && mv aa $DIFFDIR/garden/project
lein new re-frame aa +handler        && mv aa $DIFFDIR/handler/project
lein new re-frame aa +less           && mv aa $DIFFDIR/less/project
lein new re-frame aa +re-com         && mv aa $DIFFDIR/re-com/project
lein new re-frame aa +re-frisk       && mv aa $DIFFDIR/re-frisk/project
lein new re-frame aa +re-pressed     && mv aa $DIFFDIR/re-pressed/project
lein new re-frame aa +routes         && mv aa $DIFFDIR/routes/project
lein new re-frame aa +test           && mv aa $DIFFDIR/test/project

# Init a git repo for each project, commit, then delete the project
for D in $DIFFDIR/*; do cd $D; git init && git add . && git commit -m "init" && rm -R project; cd $PROJDIR; done

# Verify the projects were committed and deleted
for D in $DIFFDIR/*; do cd $D; git status; cd $PROJDIR; done

Update the code by replacing invoke-option with option? in re_frame.cljs.

Create new projects for each option using the updated code and diff:

# Verify only re_frame.cljs was updated
git diff

# Create a project for each option after the source update, move to DIFFDIR
lein install
lein new re-frame aa                 && mv aa $DIFFDIR/0/project
lein new re-frame aa +10x            && mv aa $DIFFDIR/10x/project
lein new re-frame aa +breaking-point && mv aa $DIFFDIR/breaking-point/project
lein new re-frame aa +cider          && mv aa $DIFFDIR/cider/project
lein new re-frame aa +garden         && mv aa $DIFFDIR/garden/project
lein new re-frame aa +handler        && mv aa $DIFFDIR/handler/project
lein new re-frame aa +less           && mv aa $DIFFDIR/less/project
lein new re-frame aa +re-com         && mv aa $DIFFDIR/re-com/project
lein new re-frame aa +re-frisk       && mv aa $DIFFDIR/re-frisk/project
lein new re-frame aa +re-pressed     && mv aa $DIFFDIR/re-pressed/project
lein new re-frame aa +routes         && mv aa $DIFFDIR/routes/project
lein new re-frame aa +test           && mv aa $DIFFDIR/test/project

# Check if there are any changes between the original and post-update projects
for D in $DIFFDIR/*; do cd $D; git status; cd $PROJDIR; done

# Diff the projects to see any differences
for D in $DIFFDIR/*; do cd $D; git diff; cd $PROJDIR; done

Observe no differences between the projects before and after the code update.

Clean up

rm -rf $DIFFDIR

All good. I didn't release so it was only on master (i.e.  via lein new re-frame.. this wouldn't impact general users as it only gets released jars from clojars). Thanks for the fix :+1:


Update to use new MDC select

https://github.com/material-components/material-components-web/issues/2399
They have now reverted to standard select element, so as soon as this is out we should update MDC and make needed changes.

Can't upgrade since the don't allow custom styles any more.


commands and events queues must be in separate tables

@vladyslav-fenchak this is for the Handler, not for the Broker.
my mistake

I included a reference to the line in which the minos.common.MinosConfig is used by the minos.networks.MinosQueueDispatcher:
https://github.com/Clariteia/minos_microservice_networks/blob/433f158207bb8ce4e5c697a492eaf82d0ee2e7ad/minos/networks/broker/dispatchers.py#L43-L56

As you said, I'll proceed to close it because the question is resolved. Thanks!


Simpler fake-addon installation in Blender

Hi!
first of all I would like to send you lots of :heart::heart::heart::heart: for sharing this. It was crucial to setup pytest on my own add-on.
while doing it I discovered a little hidden utility function that allows you to effectively change  script_directory after setting it with python, without closing and opening Blender.
By using it you can enable the add-on for development purposes without zipping or copying any files. The load_pytests.py ends up like this:
import pytest

import bpy

import sys
from os.path import join, abspath, dirname
dev_script_folder = join(dirname(abspath(__file__)), "../scripts")

addon = "fake_addon"

class SetupPlugin(object):
    def __init__(self, addon):
        self.addon = addon

    def pytest_configure(self, config):
        bpy.context.preferences.filepaths.script_directory = dev_script_folder
        bpy.utils.refresh_script_paths()
        bpy.ops.preferences.addon_enable(module=addon)

        config.cache.set("bpy_module", self.addon)

    def pytest_unconfigure(self):
        bpy.ops.preferences.addon_disable(module=self.addon)
        print("*** test run reporting finished ***")

try:
    exit_val = pytest.main(["test_cases"], plugins=[SetupPlugin(addon)])
except Exception as e:
    print(e)
    exit_val = 1
sys.exit(exit_val)

I'm pointing the script_directory to "../scripts" which should replicate the Blender's script folder structure, as stated here:
scripts/addons/fake-addon/<__init__.py and addon-code here>

Hope this information helps improving your fake add-on!

Interesting.  I think I see what you are suggesting, instead of copying the addon into a known/used script area (and deleting it at the end), you just get the scripts directory to point to a different directory. Is that correct? (all of it)
I am a bit mixed about this, as in my business we try and replicate the real world example as close as possible, because even though the two should be the same doesn't mean that they are. Windows is notorious for been flaky here in that regard.
What issues are you seeing with it the way it is currently implemented?
I'll see about providing it as a option as a first step, what do you think?

Looking at your code, it ignores some of the zipping  I am doing under the hood.  If you have a single file addon, it will probably be fine.
While my example is a single file, it is written to service multiple file, so even in the case of a single file addon, I install it as from a zip file.
Also in my example scripts has already been used up as a directory for my ... scripts.
I'll look in to this a bit more, but it is not a simple fold in, in its current form.

Hi!
You are right about the practice of replicating the real world example as close as possible. I don't see any issue with the way it is implemented right now.
My suggestion was just targeted to avoid duplicating files, and polluting the real Blender addons folder in the unlikely case of an interrupted execution of the test suit.

While my example is a single file, it is written to service multiple file, so even in the case of a single file addon, I install it as from a zip file.

This strategy works with a python package as well as a single file

Yes I agree with and have seen the interrupted execution problem, more when I was developing this tool.
I think I am zeroing in on a solution that takes care of the zip problem too.

OK i have updated a fix to master where a local_directory is created and installed into and then deleted at the end.
It was probably not of interest to you, but the code required to run on <=2.79 was slightly different but I got that to work to.

Great!
thanks for considering my suggestion!


Reorder ssclj classpath

resourcesfolder must have precedence over ssclj.jar
Connected to #741

What is (are) the conflicting file(s) when the classpath is in the wrong order?

There seem to be a large number of files that probably should not be in the ssclj service jar file:
auth_privkey.pem
auth_pubkey.pem
config-hsqldb-mem.edn
config-hsqldb.edn
es-build.properties
example-event.json
project.clj
sqlite.edn

And these file should probably be moved to a specific namespace or incorporated directly into the files that read them:
filter-grammar.txt
mapping-not-analyzed.json

Any comments on removing/including the above files?

auth_pubkey.pem

These 2 files are actually used by auth service (served by ssclj server)
(can be overriden after installation: http://ssdocs.sixsq.com/en/latest/administrator_guide/maintenance.html?highlight=pem#authentication-keys)
project.clj

For people who want to use leiningen, this is usefull.
Could not find this file in project:
es-build.properties

to be removed:
config-hsqldb-mem.edn
config-hsqldb.edn
example-event.json
sqlite.edn

I've removed unused files and dependencies from the repository and/or jar file.  Certificates were removed from the jar and moved to the RPM as configuration files.  @st review the changes to make sure nothing has been broken, if not, the PR can be merged.


ReferenceError:Buffer is not defined & Frame: function:'require',file

I tried to use sqlite-cipher plugin with my nativescript angular app. But that time i got issue, please refer below screenshot.

After that i have searched and followed the nativescript-nodeify plugin.
"https://www.nativescript.org/blog/how-to-use-any-npm-module-with-nativescript"
After implemented the nativescript-nodeify, then tried to build and run the project. But i got again errors like,

How to resolve this issue.
Thanks.

Here i have attached my app's log file.
nodeify_logcat.txt

5f

Happens to me as well , @EddyVerbruggen, could you help please?

I encountered the same issue.
It seems that something has changed in the npm module loading.
When I manually add
var Buffer = require("buffer").Buffer;
at the start of the make.hash.js File in the node_modules directory, this error disappears.
This is no solution but perhaps a hint how this issue can be resolved.
I'm a nativescript noob, so I have no idea, where to start. :-(

Before diving into this I have to ask: did you guys include require("nativescript-nodeify"); before running the problematic code runs? For instance @kumaran-dena's make-hash.js on line 3 seems to try and load Buffer which should be patched in case that aforementioned require is added.

This solves it for me, thanks.
I assumed that this would be pulled in when using a shimed module.


All Measures are missing 'Maintain Layer Order' setting

I'm working on the code samples for this.


Support RedHat 7

I only see EL 6, looks dated.

https://github.com/dockpack/base_redis is a drop-in replacement role that supports Centos/RHEL7 redis sentinel.


Throw error when user closes the popup window

I think that when we prompt a user with the popup window, we should throw error in case the user closes the window without entering credentials.

This will also make the "takeWhile" redundant.
I'll look more into it this weekend.

Yea i saw that after i commited to pull request.
There is also a similar in the eventListening method that i had no idea who use that :/

Actually I think I missed something. when testing this I saw that there are two ways to return from that modal: One by clicking the back button and the other by a close icon inside the modal (At least on an android device.) The error is not invoked when using the close icon

Actually I think I missed something. when testing this I saw that there are two ways to return from that modal: One by clicking the back button and the other by a close icon inside the modal (At least on an android device.) The error is not invoked when using the close icon

Can you try 6.1.0, I've thrown a new Error('Authentication Canceled'); when the window closes.
all you need to do when authenticating is
.subscribe(
    next => <do something>,
    error => <do something and stop spinner>,
    (/* complete */) => <stop spinner>
)


Ensure that all cyclonedx components have bom-refs

Signed-off-by: Sambhav Kothari<EMAIL_ADDRESS>BOM-Refs are important for VEX to refer to components uniquely. This allows us to create an independent vex document and refer to components within the SBOM through a BOM Link (see https://cyclonedx.org/capabilities/bomlink/)
The current logic for setting the bom ref is -

PURL + syft id as a qualifier if PURL exists
syft artifact id otherwise.

cc: @stevespringett @coderpatros
This is needed for independent VEX documents in grype.

@samj1912 what if we always include the ID, something like bom-ref = fmt.Sprintf("%s-%s", p.PURL, p.ID())? This way we can have a more readable bom-ref and guarantee uniqueness?

Thanks!


MetaCoq is incompatible with QuickChick (universe inconsistency)

With the following:
From QuickChick Require Import Sets.
From MetaCoq.Template Require Import TermEquality.

I get the error:
Universe inconsistency. Cannot enforce
Coq.Relations.Relation_Definitions.1 <=
All_Forall.All2_All_mix_left.u1 because All_Forall.All2_All_mix_left.u1
<= All_Forall.All2.u2 <= All_Forall.All2_map.u5
< Coq.Relations.Relation_Definitions.1.

Strangely, pasting the line From QuickChick Require Import Sets. on top of file https://github.com/MetaCoq/metacoq/blob/coq-8.13/template-coq/theories/TermEquality.v raises the error:
No 1st non dependent hypothesis in current goal even after
head-reduction.

Tested with Coq 8.13.2, QuickChick 1.5.0 and MetaCoq 1.0~beta2+8.13 from opam
This is related to https://github.com/smtcoq/sniper/issues/4 and https://github.com/QuickChick/QuickChick/issues/232.

You can probably minimize this with the bug minimizer

This kind of universe inconsistency is usually due to a template-polymorphic inductive type being partially applied somewhere. Those are not easy to find though.

Does coqbot watch this repo?
@coqbot minimize
opam install coq-metacoq coq-quickchick
echo > bug.v <<EOF
From QuickChick Require Import Sets.
From MetaCoq.Template Require Import TermEquality.
EOF
coqc -q bug.v

I guess not.  Trying at https://github.com/coq-community/run-coq-bug-minimizer/issues/2#issuecomment-982842826

We're working on it

This is also the incompatibility with Iris that we run into at Bedrock with @gmalecha and others; we thought it might relate to https://gitlab.mpi-sws.org/iris/stdpp/-/issues/80 but I don't have strong evidence it's related.
In our case, we just found out that we can import skip MetaCoq.Template.Typing instead of using MetaCoq.Template.All, and quoting and MetaCoq Run still works fine.
Minimized example against coq-iris.dev.2021-01-12.0.bbaf3eaf from git+https://gitlab.mpi-sws.org/iris/opam.git:
From iris.algebra Require Import ofe cmra frac.
(* Imported by MetaCoq.Template.All but unneeded: *)
Fail From MetaCoq.Template Require Typing.
(* The problem is in fact introduced by: *)
Fail From MetaCoq.Template Require TermEquality.

Some partial minimizations appear in https://gist.github.com/Blaisorblade/aae837cb9b39e7693d3f2037960b0f80, but dunno if it's helpful.

On a recent coq-8.14 version (71210b48eae67bc70f4810161765d0d1294c8b1f), we get somewhat similar errors, and avoiding Typing seems to still avoid those.
From iris.algebra Require Import frac.
From MetaCoq.Template Require Import Typing.
(*
Universe inconsistency. Cannot enforce Coq.Relations.Relation_Definitions.1
<= All_Forall.All.u1 because All_Forall.All.u1
< Coq.Relations.Relation_Definitions.1.
*)

From MetaCoq.Template Require Import Typing.
From iris.algebra Require Import frac.
(*
Universe inconsistency. Cannot enforce base.equivL.u0 = eq.u0 because eq.u0
<= stdpp.base.47 <= cstr_respects_variance.u0 < eq.u0 = base.equivL.u0.
*)

@gmalecha suggested dropping Typing from MetaCoq.Template.All since it's not needed for many metaprogramming usecases.
Other users can try replacing requires of MetaCoq.Template.All with the following, and report whether it is sufficient for their uses.
From MetaCoq.Template Require Import
     utils.MCUtils (* Utility functions *)
     monad_utils   (* Monadic notations *)
     uGraph        (* The graph of universes *)
     BasicAst      (* The basic AST structures *)
     Ast           (* The term AST *)
     AstUtils      (* Utilities on the AST *)
     Induction     (* Induction *)
     LiftSubst     (* Lifting and substitution for terms *)
     UnivSubst     (* Substitution of universe instances *)
     (* Typing *)        (* Typing judgment *)
     TemplateMonad (* The TemplateMonad *)
     Loader        (* The plugin *).

I assume the partial application in QuickChick is from coq-ext-lib.
Candidates include:

https://github.com/coq-community/coq-ext-lib/blob/73fa2d83a075adedc65c1f5888d7f688f9e31f0c/theories/Data/Monads/ListMonad.v#L7
https://github.com/coq-community/coq-ext-lib/blob/73fa2d83a075adedc65c1f5888d7f688f9e31f0c/theories/Data/List.v#L123

There might be more list monads in there.

If indeed this is the issue, the solution is to eta-expand list everywhere it's used as a function, as I say at https://github.com/coq-community/coq-ext-lib/pull/123#discussion_r783590882


Vsco ripper broken?

Ripme version: 2.1.2-3-ea90b172
Java version: java version "18.0.2" 2022-07-19
Java(TM) SE Runtime Environment (build 18.0.2+9-61)
Java HotSpot(TM) 64-Bit Server VM (build 18.0.2+9-61, mixed mode, sharing)
Operating system: Microsoft Windows [Version 10.0.19044.1826]
Exact URL you were trying to rip when the problem occurred: https://vsco.co/[ANY_USER]/gallery
Expected Behavior
Rip user images.
Actual Behavior
Error: Cookie value must not be null
Log - Cookie value must not be null

Same issue

Notable that ripping a single image still works correctly, which means vscoImageToURL should still be working correctly.

https://vsco.co//media/

Full exception details:
java.lang.IllegalArgumentException: Cookie value must not be null
        at org.jsoup.helper.Validate.notNull(Validate.java:28) ~[jsoup-1.14.3.jar:?]
        at org.jsoup.helper.HttpConnection$Base.cookie(HttpConnection.java:609) ~[jsoup-1.14.3.jar:?]
        at org.jsoup.helper.HttpConnection$Request.cookie(HttpConnection.java:630) ~[jsoup-1.14.3.jar:?]
        at org.jsoup.helper.HttpConnection.cookies(HttpConnection.java:329) ~[jsoup-1.14.3.jar:?]
        at com.rarchives.ripme.utils.Http.cookies(Http.java:145) ~[main/:?]
        at com.rarchives.ripme.ripper.rippers.VscoRipper.getSiteID(VscoRipper.java:148) ~[main/:?]
        at com.rarchives.ripme.ripper.rippers.VscoRipper.getURLsFromPage(VscoRipper.java:85) ~[main/:?]
        at com.rarchives.ripme.ripper.AbstractHTMLRipper.rip(AbstractHTMLRipper.java:127) ~[main/:?]
        at com.rarchives.ripme.App.rip(App.java:100) ~[main/:?]
        at com.rarchives.ripme.App.ripURL(App.java:293) [main/:?]
        at com.rarchives.ripme.App.handleArguments(App.java:276) [main/:?]
        at com.rarchives.ripme.App.main(App.java:75) [main/:?]```

tkn is null at line 149
That means getUserTkn() is returning a null value there is a catch block for IOExceptions during the web request, so vsco probably changed how this is being returned


Error building Example code in Readme

if I create a new project with
cargo new bincode_test;
cargo install cargo-edit;
cargo add bincode serde_derive;
cd bincode_test;

And then replace the contents of bincode_test/src/main.rs with the Example code from the README.md.
Then when I try to build the code with:
cargo build

It gives error:
error[E0463]: can't find crate for `_serde`
 --> src/main.rs:7:21
  |
7 | #[derive(Serialize, Deserialize, PartialEq, Debug)]
  |                     ^^^^^^^^^^^ can't find crate

System:
cargo 1.29.0 (524a578d7 2018-08-05)
rustc 1.29.1 (b801ae664 2018-09-20)

cargo add serde

Same problem here. I could fix it by adding
#[macro_use]
extern crate serde_derive;

To the example

I had more or less the same issue. I solved it with adding in my Cargo.toml
[dependencies]
bincode = "1.2"
serde =  { version = "1.0", features = ["derive"] }


Back button missing on second step of registration form

Preflight checklist

[X] I could not find a solution in the existing issues, docs, nor discussions.
[X] I agree to follow this project's Code of Conduct.
[X] I have read and am following this repository's Contribution Guidelines.
[ ] I have joined the Ory Community Slack.
[ ] I am signed up to the Ory Security Patch Newsletter.

Ory Network Project
No response
Describe the bug
The back/current identifier button is missing on the second step of the registration form.
Reproducing the bug

Enable two-step registration
Go to sign up page
Enter email address
See the method chooser, but without a way to navigate back to the initial screen (e.g. if you want to re-type your email address)

Relevant log output
No response
Relevant configuration
No response
Version
v1.0.0-next.12
On which operating system are you observing this issue?
None
In which environment are you deploying?
None
Additional Context
No response

Fixed via https://github.com/ory/elements/pull/281


Copy magic item

Detailed Description: Hold a magic item, press F, and then press F without releasing, you can copy the item.
Video: https://youtu.be/XP2uR2R528M
Magic Version: 7.9.6
Server Version: Paper-354 (MC: 1.15.2)

Is there any more to it than that? Do you double-press F and hold it? Does it only work with specific items or does it have to be a magic shield
I've tried this myself every which way and have not been able to get it to happen. This has been reported before.
Are you able to test on vanilla Spigot?

Ah, ok I was able to reproduce it on a remote server, so it must be ping-related. Thanks, I'll see what I can do!

I think this is fixed in the latest dev build.

I think this is fixed in the latest dev build.

I slept last night so I did not reply in time, it has indeed been repaired now, thank you for your excellent work!


Update to use golang v1.21 (#211)

Signed-off-by: Tesshu Flower<EMAIL_ADDRESS>(cherry picked from commit d3b9f0532ef25bad380c671e3d894c8b59d313ec)
Cherry-picking PR https://github.com/stolostron/volsync-addon-controller/pull/211 into release-2.7
For: https://issues.redhat.com/browse/ACM-10522

/override "ci/prow/sonar-pre-submit"


Commission Rate might not exists in the validator object

https://github.com/forbole/big-dipper/blob/599fe72898e1d9d18e88d8302fa72b029bf22985/imports/ui/validators/List.jsx#L19
props.validator.commission.commission_rates might not exists. Should add checking before display.

Fixed.


[Blameable] Added new parameter to allow definition of the user entity

Closes #486
I can't find a way to get the user in the security provider. In order to make Blameable works well, a added the parameter doctrine_behaviors_blameable_user_entity which allows the user to define the entity to use

Test is needed, to make sure this won't happen again.
Ideally failing CI first, than fix with CI passing.

Ok I will add it as soon as I have some time

I tested this PR and it solves the problem with the migrations well for my app.
For the record: Another symptom of the underlying problem is also solved by this PR: The error message Error: Property "createdBy" in "App\Entity\MyEntity was already declared, but it must be declared only once vanishes where MyEntity itself has no Blameable, but it is one of many extending MyBaseEntity using single table inheritance which uses the BlameableTrait and implements the BlameableInterface.

@fsoedjede Thank you for your work and adding a test case

@spackmat Thanks for feedback, that's great to hear. I've merged it now

@TomasVotruba I already had written a new issue for that problem (driving me nuts the whole day) but then realized that it is maybe related to the migrations/cli problem I also had and volià it is. One less issue. :) I don't have a clear concept of why this error vanishes, but I am quite happy it does so. So thanks to you both!
(Btw: v2.0.0 worked not at all for me creating segmentation faults on cache clear, memory exhaustion based 500s at the webinterface and freezing my whole VM on doctrine:migrations:diff)

The segfaults was probably caused by https://github.com/KnpLabs/DoctrineBehaviors/pull/493


module not resolved in JS file using var x = require('...') syntax

From @timty on October 19, 2016 17:18
Intellisense works great for my postgress Pool type when explicitly declared:

However when passed as a parameter to a function and described through a JSDoc parameter intellisense fails:

VSCode Version: Code 1.6.1 (9e4e44c19e393803e2b05fe2323cf4ed7e36880e, 2016-10-13T15:27:10.246Z)
OS Version: Darwin x64 15.6.0
Extensions: Unity.unity-debug-1.0.1, benjaminromano.typings-installer-0.2.0, dbaeumer.vscode-eslint-1.0.7, dkundel.vscode-npm-source-0.0.3, joelday.docthis-0.3.4, mohsen1.prettify-json-0.0.3, ms-vscode.omnisharp-0.3.3, rafaelmaiolla.diff-0.0.1, stevencl.addDocComments-0.0.8, xabikos.JavaScriptSnippets-1.1.0, xabikos.ReactSnippets-1.1.1

Copied from original issue: Microsoft/vscode#14008

@timty thank you for opening this issue. I have confirmed this with the latest bits and no extensions. Migrating to JavaScript language service.
Code to copy and paste.
class Foo {

    bar() {}
}

var f = new Foo();
// IntelliSense works correctly
// f.

/**
 * @param {Foo} x
 */
function zoo(x) {
    // IntelliSense is not being given
    x.
}

@waderyan your code doesn't repro for me on master. Intellisense works fine when I paste the code in a javascript file.
Intellisense does not work for me in a TypeScript file, but Typescript doesn't rely on jsdoc to give types to parameters. It only uses the documentation.

When I paste the code into a typescript file that has --noImplicitAny turned on, I get an error at zoo(x).
If you change the jsdoc line to read @param {Foo} x documentation, you will see "documentation" in the quick info hover for x, indicating that Intellisense is actually working in typescript too.

@timty Are you using typescript or javascript? In a typescript file you will not get intellisense for the above reason. In a javascript file the code you gave will not work either, because jsdoc requires the parameter name to match:
/**
 * @param {Pool} pool2 <-- Names must match
 */
function myFunction(pool2) {
  pool2./*intellisense should work now*/
}

If you're using typescript, then you have to use the postfix type syntax: function myFunction(pool2: Pool) {

@sandersn @waderyan
Thank you both for your responses. I am using javascript (nodejs). Sorry about the typo when recreating the issue. This does not work with correctly matching names either.
var pg = require('pg');

var pool = new pg.Pool("");
//intellisense works...
//pool.

/**
 * @type {Pool}
 */
var pool1;
//no intellisense here...
//pool1.

/**
 *
 * @param {Pool} pool2
 */
function myFunction(pool2) {
//no intellisense here...
//pool2.

}

Intellisense here:

No Intellisense with JSDoc type:

No Intellisense with JSDoc param:

The type Pool was not imported in the module, it is visible as pg.Pool

@alexrock nice catch.

Typescript doesn't rely on jsdoc to give types to parameters. It only uses the documentation.

@sandersn really glad things are staying that way. jsdoc comments are often out of sync with the actual code.  I have a colleague who uses JetBrain's Webstorm and it auto emits all these meaningless jsdoc tags into a pure typescript project... sorry for the rant.

@alexrock @aluanhaddad
It was imported via nodes require statement on the top line. Correct? I do get intelligence in this file for the pg.Pool type just not for my var and function parameter.

@timty you imported the module with the binding pg.
That means that you must qualify all references to exports of the module with pg as in pg.Pool. The issue is that Pool is not in scope.
some options:
A
import pg = require('pg');
/**
 * @type {pg.Pool}
 */
var pool1;

B
import pg = require('pg');
import { Pool } from 'pg';
/**
 * @type {Pool}
 */
var pool1;

@aluanhaddad Thanks for the feedback. Got it working with Option B...
var pg = require('pg');
import { Pool } from 'pg';

/**
 * @type {Pool}
 */
var pool1;

However for the sake of completeness this does not seem to work...
var pg = require('pg');
/**
 * @type {pg.Pool}
 */
var pool1;

I don't think that the language service has special knowledge of the var x = require('pg') pattern, so it doesn't try to resolve 'pg' as a module. @vladima or @andy-ms, can you confirm?
It does understand ES module syntax and the typescript-only import x = require('...') that @aluanhaddad mentioned.
@timty it's good that you can use ES module syntax to work around this. Does Option B need both lines to work?
I'll leave this bug open until we decide whether the language service should also support var x = require('...') in Javascript files.

@andy-ms poked around the code and there is code that looks for require in Javascript: collectRequireCalls. So it must be failing to work as intended.

@sandersn @andy-ms
Your suspicions are correct. No intellisense with the require pattern at least in this case.
No Intellisense here:
var pg = require('pg');

/**
 * @type {Pool}
 */
var pool1;
//no intellisense
pool1.

Intellisense works here:
import { Pool } from 'pg';

/**
 * @type {Pool}
 */
var pool1;
//intellisense works
pool1.

The issue here is that there is no Pool in scope. moreover, pg is a value and not a namespace, so there is no type called pg.Pool.
in the case of the ES6 import, there is a type called Pool and then things work.
if we want to make this to work, we need to make sure that pg is a namespace as well as a value, and to use it you will need to fully qualify it as pg.Pool.

@mhegazy

we need to make sure that pg is a namespace as well as a value, and to use it you will need to fully qualify it as pg.Pool.

Yes! In TS 2.3, checkJs was introduced. So this feature is more important now.
var pg = require('pg');

/** @type {pg.Pool} */
var pool1;

It should work for intellisense and checkJs.
https://github.com/Microsoft/TypeScript/issues/14377 is another solution for this issue.

@vladima you wrote the code for collectRequireCalls. Maybe you can shed some light into this issue? Thanks!

I do not think we should treat values as types or namespaces; this will just lead to confusion in the future. https://github.com/Microsoft/TypeScript/issues/14377 would be the way we would proceed with this request.

@mhegazy I agree.
Treating values as types requires const type = require('only-for-typing') in JS code.
#14377 is better solution.

Seems like another variant of https://github.com/Microsoft/TypeScript/issues/16489


Revert "Add retries in update_go_mod"

This reverts commit ea8ea4ec51425a41ffe2c92ef8814fccaf37d23b.

Please add an explanation in the commit message to clarify why we’re doing this.

Sure

It’s also worth noting that this might cause problems with mikefarah/yaml again :-(.

It shouldn't, I checked the fix locally a while ago it should still be valid.


Using the screen texture in a visual shader and trying to preview it crashes the engine

Godot version:

Godot 3.1 alpha 3
OS/device including version:

Windows 10 Nvidia 1070
Issue description:

When trying to preview the texture of a Screen texture or any node connected to its output makes the editor crash.
Steps to reproduce:

Add a ColorRect
Add a visual shader to that node with CanvasItem mode
Add a constant texture node and set it to Screen
Try to preview the rgb data of that texture node

A backtrace might be useful.