Basic grapheme offset function for visual caret positioning

Related to https://github.com/iTwin/itwinjs-backlog/issues/1152.
This PR adds a new GraphemeOffset interface exposed via a RunLayoutResult. This allows the caret/cursor to know its visual location in a text block.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Alfonso-Martello:x: claudiaareneeeYou have signed the CLA already but the status is still pending? Let us recheck it.

I have only skimmed the changes. A couple of things to consider:

A given TextBlock will contain many graphemes. The JSON representation of a grapheme is very verbose. Can you reduce it?
Not every caller of layoutTextBlock cares about individual graphemes. They shouldn't have to pay the price (in terms of processing and extra data). Can you make it opt-in?

A given TextBlock will contain many graphemes. The JSON representation of a grapheme is very verbose. Can you reduce it?

By reduce, I assume you mean reduce the number of properties? All of the properties are being used. I may be able to remove the charCount property. I could combine the leadingGraphemeOffset and trailingGraphemeOffset properties into just a range.

Not every caller of layoutTextBlock cares about individual graphemes. They shouldn't have to pay the price (in terms of processing and extra data). Can you make it opt-in - or, make it a separate function?

The call to compute grapheme offsets should only occur when the user calls the separate computeLayoutTextBlockResult function, so only when they are asking for the layout information, not when a caller is just calling layoutTextBlock. I can make it even more opt-in by adding a boolean arg required to be true to opt-in. Or are you suggesting a separate function from the "toResult" workflow that a user can call if they explicitly want grapheme information (such that they pass a RunLayoutResult and get grapheme info for that RunLayoutResult)?
Based on the above questions, I'm starting to like the idea of having a separate function that I can call to get the grapheme information for a RunLayoutResult. This would allow me to do the logic for determining what constitutes a grapheme on my end, and only ask core for the range of the characters. This would allow me to remove the GraphemeOffset interface and decouple grapheme offsets form the layout result.
If you agree I can modify this PR to introduce that function instead.

By reduce, I assume you mean reduce the number of properties? All of the properties are being used. I may be able to remove the charCount property. I could combine the leadingGraphemeOffset and trailingGraphemeOffset properties into just a range.

I mean that this:
graphemeOffsets: [{
  charCount: 1,
  charOffset: 2,
  leadingGraphemeOffset: 3,
  trailingGraphemeOffset: 4,
}, {
  // ...repeat for each grapheme...

is very verbose. There are 4 characters that contain actual data, and a ton of characters that are just repeated property names. The same information could be transmitted much more compactly.

I'm starting to like the idea of having a separate function that I can call to get the grapheme information for a RunLayoutResult.

Me too.

This PR is now introducing a new computeGraphemeOffsets function to allow computing the range of a grapheme. Really, it is just computing the trailing edge (since each range is from the start of the RunLayoutResult to the trailing edge of the grapheme).


Possible to use with multiple cursor?

Thanks for creating this great tool.
I notice that the multiple cursor is till not supported, any plan to add it in near future?

@ktutnik cool idea. i'll work on it

need to do some more validation tomorrow to make sure it won't break anything, but it's looking pretty good. should be released soon. @ktutnik

That's fast!
Thank you @meganrogge for making this feature available quickly

I’ll push the updated version tomorrow. Thanks for the idea!

done, should be live in a few minutes


⚠ Samsung has degraded performance

In e4362a9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2326 ms

Resolved: Samsung performance has improved in f116439.


Why is the multiple regions not applicable to ql_memory_bank-flatten protocol?

According to OpenFPGA Configuration Protocol, multiple configuration regions is not applicable to ql_memory_bank configuration protocol when BL/WL protocol flatten is selected.
I have tried flatten protocol with different "num_regions" values, OpenFPGA did not generate any error and generated FPGA works well. Is there any bug here? Should I use different protocol if I need multiple regions?
I am aiming big FPGA (like 32x32). I think I can not program all FPGA fabric, it is problematic.  So I need multiple regions to program a small piece of fabric at each program cycle. May you suggest physical design, power friendly way to do that?
Best regards.

@mustafaarslan0 In the flatten protocol, all the BLs and WLs are exposed to you. It gives you the maximum freedom to manipulate the BLs and WLs. It is up to you to separate BLs and WLs into different regions.
Advanced settings for regions can be found in the fabric key file

If you can select to enable more than 1 WLs during each programming cycle.
If you can split WLs in one row of BLs through fabric key, you can enable partial configuration during each programming cycle.

This is what I need. So there isn't any concern to use QL-MemoryBank-Flatten with multiple regions, documentation need to be updated.
Thanks.


Use middleware in routes

same as canActivate in Angular

Hi @MichalPorag! I would like to the assignee for this issue


feat(cicd): add code scanning

Current dependencies on/for this PR:

main

PR #39

PR #40

PR #41   👈

This comment was auto-generated by Graphite.


Unsupported DNS provider

Hello ,
I am using rancher v1.1.3  and docker v1.8 .
I tried to configure your producat to work with Namecheap and get the error:
level=info msg="Starting Let's Encrypt Certificate Manager v0.3.0 f8965f6"
level=info msg="Generating private key (2048) for<EMAIL_ADDRESS>level=info msg="Creating Let's Encrypt account for<EMAIL_ADDRESS>level=fatal msg="LetsEncrypt client: Could not set DNS provider: Unsupported DNS provider: Namecheap"
Best Regards

still no answer to this.

@RotemEmergi Namecheap support has been dropped. The new template soon to be released in the catalog should reflect this. Thanks!


vnext branch Path issue + System.IO.Packaging.dll

Hi,
it is the first time I tried to work with the Open-XML-Powertools.
I cloned the vnext branch and followed the "new" youtube video for the VS2071 build process.
The build process (using OpenXMLPowertools -> targeting .net 4.5)  runs fine with 2 successfull builds and no errors.
While importing the module I run into the following error multiple times:
Add-Type : Cannot bind parameter 'Path' to the target. Exception setting "Path": "Cannot find path '<path>\WindowsPowerShell\Modules\Open-Xml-Sdk\DocumentFormat.OpenXml\bin\debug\DocumentFormat.OpenXml.dll' because it
does not exist."

The "Open-Xml-Sdk" directory does not exist. I assume that this was the path when the Open-XML-SDK was build from source.
As the vnext branch uses nuget the path to the DocumentFormat.OpenXml.dll (to my understanding) should now be:
....\packages\DocumentFormat.OpenXml.2.7.2-vnext0027\lib\net45
Once I changed that in AddTypes.ps1 I get the following while trying to import the module:
Add-Type : Cannot bind parameter 'Path' to the target. Exception setting "Path": "Cannot find path ```
'<path>\Documents\WindowsPowerShell\Modules\Open-Xml-Sdk\DocumentFormat.OpenXml\bin\debug\System.IO.Packaging.dll' because it
does not exist."

The System.IO.Packaging.dll is not included in the open-xml-powertools directory before or after the build.
I tried to install System.IO.Packaging.dll via nuget but it failed due to the targeting  .NET framework (4.5).
I could not find System.IO.Packaging.dll anywhere on my system.
I am working on a windows server 2012 R2 with Powershell 5.
.NET 3.5 and 4.5 are installed.
Can anyone help me out on this?
Thanks
Sven

It seems that the reference is not necessary, and should be removed (at least in .net 4.5). Perhaps the System.IO.Packaging class is made its own DLL in newer versions of .net? The System.IO.Packaging namespace is actually contained within "WindowsBase", which is already references. While building it, I removed all of the references to System.IO.Packaging, and it built cleanly.

Thanks pigrew.
Just to be clear, I am getting a clean build with System.IO.Packaging in the source files,
The errors I mentioned in my first post occurred while trying to import the Open-XML-Powertools module in a Powershell "session".
I now got a clean clone of Open-XML-Powertools and replaced all occurrences (5) of "using System.IO.Packaging;" with "//using System.IO.Packaging;".
But during the build I now get 7 Errors all related to missing types or namespaces:
They all look like this:
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0246	The type or namespace name 'PackagePart' could not be found (are you missing a using directive or an assembly reference?)	OpenXmlPowerTools	C:\Users\HadarS\Documents\Open-XML-Powertools\OpenXmlPowerTools\WmlComparer.cs	4451	Active

only the missing type of namespace varies:
WmlDocument, PackagePart, Package
I may be missing something and would be very grateful is someone can point me in the right direction.
Thanks
Sven

i faced the same issue in my project.
i am using .Net framework 3.5
even i got the error, after included like this
using System.IO.Packaging;
Solution which helped for me is
i have added WindowsBase as reference in project which already exist in .Net framework.
Thanks
Maran

Closing all issues as this repo is being archived and will no longer be maintained by Microsoft. The project is licensed for continued use and development by forking to your own repo.


BrowserOnly hydration issuse

🐛 Bug Report
Prerequisites

[x] I'm using the latest version of Docusaurus.
[x] I have tried the npm run clear or yarn clear command.
[x] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.

Description
BrowserOnly seems to have a hydration issue. It

removes the first parent node of the would-be content
and then picks the next sibling of the page and ... wrap the rest that is left in it

Say you have:
<BrowserOnly>
  () => ..............

hello

</BrowserOnly>

...

In production (not in dev) Docusaurus will transform it to:

hello

...

Here is a real world example:

Docusaurus will start to mix that code in very strange ways:

And this is how it looks in dev:

A workaround is to wrap BrowserOnly into the the first element of the would-be content. Why this works is even stranger, i can't see a pattern in that.

Thanks, I think we should hydrate with the exact same markup and use useLayoutEffect to render the client JSX a bit later.
This is probably related to https://www.joshwcomeau.com/react/the-perils-of-rehydration/

Oops, sorry, it is my mistake, will be fixed in #4935.

@drcmda it should be fixed now.
Please try to use the @canary dist tag on your repro case and let us know if it is fixed (2.0.0-beta.d81d43c24 for example)

@slorber @lex111 for some reason the canary gives me this now:
index.js?1ef8:1 Warning: React has detected a change in the order of Hooks called by BrowserOnly. This will lead to bugs and errors if not fixed. For more information, read the Rules of Hooks: https://reactjs.org/link/rules-of-hooks

   Previous render            Next render
   ------------------------------------------------------
1. useContext                 useContext
2. undefined                  useRef
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

    at BrowserOnly (webpack-internal:///./node_modules/@docusaurus/core/lib/client/exports/BrowserOnly.js:12:26)
    at div
    at div
    at nav
    at Navbar (webpack-internal:///./src/theme/Navbar.js:28:423)
    at DocsPreferredVersionContextProviderUnsafe (webpack-internal://

there seems to be nothing unusual in the file that im loading inside browseronly.

@drcmda sorry, will be fixed in #4960.

@drcmda surprised because it looks fine to me.
Are you calling hooks in the children of BrowserOnly?
The children prop is executed as a plain function, not a React component so the following wouldn't work:
<BrowserOnly>
  {() => {
    const value = useContextValue();
    return

  }}

Is this a dev/browser or prod/nodejs warning?
Are you adding a browser only component to the navbar?
Can you try yarn docusaurus clear and run again?

Navbar.js

          <BrowserOnly fallback={null}>
            {() => {
              const MenuItems = require('../components/MenuItems').default
              return <MenuItems />
            }}
          </BrowserOnly>

MenuItems.js
export default function MenuItems({ targetItem = 'navbar__item', targetLink = 'navbar__link' }) {
  const [visibleSignIn, setVisibleSignIn] = useState(false)
  const [visibleSignUp, setVisibleSignUp] = useState(false)
  const isSignedIn = useBackend((b) => b.user.isSignedIn)

  return (
    <>
      {isSignedIn && (
        <Link to={routes.profile} className={`${targetItem} ${targetLink}`}>
          <UserOutlined />
        </Link>
      )}
      {!isSignedIn && (
        <Popover style={{ background: 'white' }}>
          <Popover.Button as="div">
            <a href="#" className={`${targetItem} ${targetLink}`}>
              Sign in

          </Popover.Button>
          <Popover.Panel className="popover-panel">
            <SignIn modal />
          </Popover.Panel>
        </Popover>
      )}

The menu items are tied to firebase and auth, signin, signup, they do use hooks. without browseronly it wouldn't work because it starts to complain. the context hook doesn't seem to come from us, i think that's from browseronly internals.

   Previous render            Next render
   ------------------------------------------------------
1. useContext                 useContext
2. undefined                  useRef
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

@drcmda I double-checked and I don't see any place in our codebase where we would use useContext + useRef and where the ref is called conditionnally (we don't use a lot of refs so it's fast to check)
I suspect this is one of your integration that is calling the useRef hook inside BrowserOnly callback. As far as I understand you have multiple BrowserOnly so it may be worth to check them all, and try to replace them one by one with a placeholder until the problem is resolved?
If you still have the issue after removing all your BrowserOnly usage, that would be surprising because we don't have this issue on our own website, deployed with fwk code from master (and we do use BrowserOnly for the json debug view here: https://docusaurus.io/__docusaurus/debug)


Make buying in the shop an atomic operation

Hey,
While playing this fun game I found out that giving multiple items to the buy subcommand was just buying the items one after the other. I thought that it would be better for the player to be informed when the buy command can't buy all of the items instead. What do you think?
I wanted to implement it by cloning the Game struct and trying to buy all of the items with this "temporary game", outputting a nice message saying that the given shopping list can't be bought but the Game struct can't be cloned and that was tedious to make it so. I decided to implement it in another way.

In fact, it is not the right design to support buying multiple consumables at the same time i.e. buy potion potion potion.
It would have been better to just clone the Game as I said before.

Is the intended effect that you either buy all or none of the given arguments? Before discussing the approach for the implementation, can you elaborate on why do you think this improves the user experience?
What I've perceived in my play test sessions is that my flow is usually: I get home, I print the status to know if I have, let's say, enough potions to go back fighting. If not I run something along the lines of rpg b p p p. I really don't care if I get two or three, and I don't want to print the shop list and calculate how many I can afford; rather my intent is "buy as much potions as I can with my current gold and move on". In this context it seems that the current behavior with be more convenient most of the time (although certainly the output could be improved so you know how many you did get to buy, e.g. +potionx2 instead of not enough gold).
But I'm curious about your usage patterns (and that of other players).

Yeah, I agree with you I like the rpg buy p p p pattern and this is why I am not pleased with my implementation. The pattern I was doing is a little less common I think, I was just trying to buy the best shield and sword I could every time I get home by doing rpg buy sword shield but one time I tried and I didn't have enough gold to buy both, unfortunately, the game bought me the sword but I would have preferred to buy the shield. The solution to my problem would have been to just reorder my arguments :)
Anyway, cool game and cool concepts. We gain just a little too much gold in late games, but this is another issue :)

Thanks for the feedback. I will close this one then, and make a note to improve the buy item output.


Updating repo reference from balenalabs to balena-labs-projects

Signed-off-by: Flynn Joffray<EMAIL_ADDRESS>Change-type: patch

@nucleardreamer build failed.. did you authorise service accounts to have access? Compare with previous org/team/permissions.

@balena-ci retest

@balena-ci I self certify!


fix: use central as default mirror

fixes https://github.com/jenkins-x/jx3-versions/issues/2768

Hi @babadofar. Thanks for your PR.
I'm waiting for a jenkins-x or todo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign abayer
You can assign the PR to them by writing /assign @abayer in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rawlingsj: Overrode contexts on behalf of rawlingsj: Semantic Pull Request

In response to this:

/override Semantic Pull Request

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

/ok-to-test
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rawlingsj
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [rawlingsj]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


List and switch models during chat session

Add /models /model /m commands to list available models and switch in same chat session
based off #401 from @fire17

Awesome to see progress being made!
Does this include model autocomplete?
Waiting to see this pushed, thanks and all the best!
Ps. Is there an aider discord or collaborator meetings? I'd love to participate

@joshuavial happy you liked the alias idea :)
Thanks for the improvements, Hoping to see this merged
All the best!

I'm still intending to get to this at some stage @paul-gauthier but struggling to prioritise it.

I thought you made your own version already....
Also isn't this a classical task for Aider itself.. don't understand why it should take long, and hotswapping models during a run, is so important and useful
@joshuavial @paul-gauthier


Does not download The Simpsons

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.11.15. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ x] I've verified and I assure that I'm running youtube-dl 2017.11.15

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

Trying to fetch The Simpsons does not work.
Sample single-episode URL: http://www.simpsonsworld.com/video/1078509635693
Pull request 12000 is merged in for Dish support.
Browser playback works, and displays the Dish logo (seems to remember me, didn't ask for a re-login.)
keybounceMBP:Fox michael$ youtube-dl --verbose --get-filename http://www.simpsonsworld.com/video/1078509635693

[debug] System config: []
[debug] User config: ['-k', '--hls-prefer-native', '--abort-on-unavailable-fragment', '-o', '%(series)s/s%(season_number)02d-e%(episode_number)02d-%(title)s.%(ext)s', '-f', '\nbest[ext=mp4][height>431][height<=576]/\nbestvideo[ext=mp4][height=480]+bestaudio[ext=m4a]/\nbest[ext=mp4][height>340][height<=431]/\nbestvideo[ext=mp4][height>360][height<=576]+bestaudio/\nbest[height>340][height<=576]/\nbestvideo[height>360][height<=576]+bestaudio/\nbestvideo[height=360]+bestaudio/\nbest[ext=mp4][height>=280][height<=360]/\nbest[height<=576]/\nworst', '--ap-mso', 'Dish', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', '--write-sub', '--write-auto-sub', '--sub-lang', 'en,enUS,en-us', '--sub-format', 'ass/srt/best', '--convert-subs', 'ass', '--embed-subs', '--mark-watched', '--download-archive', 'downloaded-videos.txt']
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '--get-filename', 'http://www.simpsonsworld.com/video/1078509635693']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.11.15
[debug] Python version 3.6.3 - Darwin-13.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3.4, ffprobe 3.3.4, rtmpdump 2.4
[debug] Proxy map: {}
ERROR: Unable to download webpage: HTTP Error 401: Unauthorized (caused by <HTTPError 401: 'Unauthorized'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/Users/michael/bin/youtube-dl/youtube_dl/extractor/common.py", line 506, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/Users/michael/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2196, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 532, in open
    response = meth(req, response)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 642, in http_response
    'http', request, response, code, msg, hdrs)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 570, in error
    return self._call_chain(*args)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 504, in _call_chain
    result = func(*args)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 650, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)

keybounceMBP:Fox michael$

Dish is not a supported TV Provider.


Make nsieve benchmark fair

Some of the implementations differ enough to make the comparison worthless. The one standing out the most does seem to be Julia's implementation, which stores an array of booleans in order to speed up the computation. After applying Julia's code logic C++ code gets over twice as fast. All tested with Julia 1.9.2 and g++-13
Here are my results (user time only, Ryzen 4600h);
nsieve 10
C++:  121 ms
Julia:  293 ms
C++ with Julia's code structure: 48 ms
nsieve 12
C++:  681 ms
Julia: 466 ms
C++ with Julia's code structure: 243 ms
I have attached my file with the faster C++ code I've used for comparison, however, it looks like many more languages do use a slow implementation instead of a fast one.
3.cpp.txt

Nice catch.
But yeah unfortunately not all implementations are using the same algorythms or approaches (with/without SIMD).
Some langs for example have internal bidings to GMP.. which is of course available to the lang developer, but still is external library.
So the main point is in that all values of the benchmarks should be considered as an example and need some detailed comparison, because not all implementations are the same.


small formatting fix

I'm an SAP employee so I don't think there is any problem with my contribution.
My company email<EMAIL_ADDRESS>Btw I love how detailed and explanatory is your contributing doc.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Thanks 👍


support empty line in

I find a way to support empty line in <Code>, and do not effect the copy result in IDE. #32

Ahh got it! So the issue was not having an empty line just above the code start and hence leaving empty lines was giving Cannot close document, a token (mdxJsxFlowTag, 84:1-84:7) is still open error.
 Thanks for the contribution!


Fix with gem name

The gem name has been changed to be included automatically by Bundler

@adrianotadao great job


Are there plans to be able to record performance logs from Chrome

I've used ChromeDriver with Selenium before and it allows the recording of performance timeline logs.
From the look of it, that's the only feature that's preventing our team from giving this serious consideration.

@andyjessop Do you mean being able to capture a HAR file as mentioned in #119?

No I don't think so. I mean the trace data mentioned at the bottom here: https://sites.google.com/a/chromium.org/chromedriver/logging/performance-log
If you to to chrome://tracing you'll see the data there. It's basically data that you get from the "performance" tab.


Update overview.md

updated to include a mention of the ordinals explorer.
Description

Breaking change?

Example

Checklist

[ ] All tests pass
[ ] Tests added in this PR (if applicable)

Closing due to age & not needed anymore.


Fixes for Trivia / Highlights / Roulette Issues

Removes old highlights from the website
Fixes #249
Trivia: Reture if there is a link in the question
Fixed #235
New Roulette settings

Fixed #168
Boolean if you allow roulettes in whispers.
Fixed #211
Options for result output

Chat
Whisper
Chat if X points else whisper

And you can now change the message

Yeah it's ugly but if you use options you need the whole text or? :octocat:

I meant this line:
if points <= -self.settings['min_show_points']:
Why not just
if points  > self.settings['min_show_points']:
For  the options you're right, it has to look like that for now. It's pretty ugly. Also not good if you ever want to change the message a bit, fix a spelling mistake or something.

Looks great though, I love the simple fixes for the highlight/trivia fixes 😊

Not sure if this would work without changing the code more in a different way,
since this are only the negative points there.

Points are always negative
self.settings['min_show_points'] are always positive

it would never get the if setting, it would be always the else setting.
This would work if the code would be structured for every option,
but i structured it after win and lose

Ah yes, you're right of course 😊
On Mar 29, 2016 7:52 PM, "Nacht123"<EMAIL_ADDRESS>wrote:

Not sure if this would work without changing the code more in a different
way,
since this are only the negative points there.

Points are always negative
self.settings['min_show_points'] are always positive

it would never get the if setting, it would be always the else
setting.
This would work if the code would be structured for every option,
but i structured it after win and lose
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/pajlada/pajbot/pull/270#issuecomment-203025261


Automated binary building with merge

Closed on #184. Actually constantly updates a draft release when there is a PR into main as long as the version number has been bumped in package.json.


New project structure with integration directories.

Adding integration tests to the framework...

Coverage remained the same at 68.636% when pulling 08bd65541b2fb735c45fc404fde1a7d5c0e9a075 on add-integration-tests into 47240fe8d758bcecd9b5c106b32e342934573547 on develop.

Coverage remained the same at 68.636% when pulling 016466666f845b68deba64c69928aa8cb3491495 on add-integration-tests into 47240fe8d758bcecd9b5c106b32e342934573547 on develop.

Coverage increased (+7.3%) to 75.909% when pulling a9078860dad7c4947119a2aa393e4cb6225927f2 on add-integration-tests into 47240fe8d758bcecd9b5c106b32e342934573547 on develop.


rankerType options left

Versions
What package version of the SDK are you using. - JS botbuilder-ai 4.8.0
What nodejs version are you using - v10.16.3
Describe the bug
Currently in QnAMaker class the constructor doesn't initialize the rankerType option.
https://github.com/microsoft/botbuilder-js/blob/9befacb893820b2b7863f03e72617cfb057e618c/libraries/botbuilder-ai/src/qnaMaker.ts#L84
 const {
            scoreThreshold = 0.3,
            top = 1,
            strictFilters = [] as QnAMakerMetadata[],
            metadataBoost = [] as QnAMakerMetadata[],
            timeout = 100000
        } = options;

        this._options = {
            scoreThreshold,
            top,
            strictFilters,
            metadataBoost,
            timeout
        } as QnAMakerOptions;

The only file this is referenced is in  qnamakerOptions.ts.
 /**
     * Id of the current question asked.
     */
    qnaId?: number;

    /**
     * A value indicating whether to call test or prod environment of knowledgebase.
     */
    isTest?: boolean;

    /**
     * Ranker types.
     */
    rankerType?: string;

There is nothing else in the class that would assign rankerType tor QnAMaker options to actual values of argument of options were passed in.
To Reproduce
Steps to reproduce the behavior:
Create instance of QnAMaker, passing in an option such as rankerType: "QuestionOnly".
We can use postman to see the spected behaviour: https://docs.microsoft.com/azure/cognitive-services/qnamaker/concepts/best-practices#choosing-ranker-type
Expected behavior
QnAMaker class initializes values of options if passed in and only returns possible answers found only in questions.

@trojenguri can you please take a look at this?  thanks.

There is an active PR in review.


Update gradient_descent.py

Python 3 allows this kind of syntax. As of now, the old syntax is used, where you specify the child's name. Should supercharge be referred to in this way, or does this change break functionality somehow? I can update statements in other files as well, if this change is accepted and meaningful. My apologies for this silly commit!

I signed it!

I'd love it if we could do this but we still have to support python 2 :-/

I see, I thought this might be the case after I'd submitted the pull. Thank for the reply anyway!


Revise and expand artifact types to include more frameworks, remove torch.compile

:rocket: Feature Request
Currently we only suggest Pytorch artifact types.
And torch.compile is not an artifact type. From the pytorch docs

torch.compile() is a JIT compiler whereas which is not intended to be used to produce compiled artifacts outside of deployment.

https://pytorch.org/docs/stable/export.html#existing-frameworks
So I think we should remove torch.compile from the list since no one should be using this to specify a model artifact type.
Here's an initial list that includes more frameworks
Scikit-learn (Python)

Pickle (.pkl)
Joblib (.joblib)

TensorFlow (Python)

SavedModel
HDF5 (.h5)
Checkpoint

ONNX (Open Neural Network Exchange) (Language-agnostic)
.onnx
PyTorch (Python)

TorchScript Tracing (.pt)
Torchscript Scripting (.pt)
torch.export (.pt2)
torch.save Pickle (.pt)

Other Frameworks
XGBoost (framework specific binary format)
LightGBM (framework specific binary format)
PMML (Predictive Model Markup Language - XML)
R
.rds, .rda
???
Julia
JLD, JLD2
BSON
???
the above list was partially llm generated so take it with some salt, I can look into and confirm use if we decide to move forward wtih this and provide a more exhaustive set of options.
:sound: Motivation
We'd like users outside of the Pytorch ecosystem to understand how to describe their model artifacts so that it is easier to know what part of a framework should be used to load the model and the different runtime dependencies involved since different artifact types have different runtime dependencies.
Francis raised that the list is currently overly specific to Pytorch and I agree: https://github.com/crim-ca/dlm-extension/pull/2#discussion_r1560193533
:satellite: Alternatives
Keep this part of the spec focused on Pytorch and loose with no more recommendations.
:paperclip: Additional context
I'm down to expand this suggestion. I think in terms of validation, we can be lax on the artifact value type that is passed here.

Although torch.compile is not recommended, I wouldn't remove it completely, since some existing artifacts might exist and still depend on it (e.g.: older code that doesn't work with newer methods). Instead, I think we should add a (*) + "WARNING" block under the https://github.com/stac-extensions/mlm?tab=readme-ov-file#artifact-type-enum to "STRONGLY" advise migrating to other more appropriate artifact types. Basically, removing it might cause more problems by obfuscation rather than explicitly mentioning why it is bad practice.
As for the other proposed types, I agree. We need more non-pytorch examples.
The examples don't have to be an exhaustive list. The mlm:artifact_type field is not even explicitly defined in the JSON schema at the moment. We should define it as a lax string, similar to other definitions:
https://github.com/stac-extensions/mlm/blob/25bef80362629188c78938277643a2c53659163b/json-schema/schema.json#L325-L366

Although torch.compile is not recommended, I wouldn't remove it completely, since some existing artifacts might exist and still depend on it (e.g.: older code that doesn't work with newer methods).

I'm not saying that torch.compile is deprecated or anything, I'm saying it was never used for saving model artifacts. so it shouldn't be listed as an artifact type.

I disagree.
torch.compile can use the same .pt2 as torch.export.
The only issue (which torch.export is trying to address) is that torch.compile can break when a certain reference cannot be resolved and trying to fall back to the Python implemention, which might not exist if the package defining it is missing.
Because torch.export is still in prototype feature in development, we should still give the option to use torch.compile. Furthermore, there are still official documentation of pytorch using this exact strategy: https://github.com/idiap/pytorch-serve/blob/master/examples/pt2/README.md#torchcompile

Here's the example you linked from torchserve, which is discussing packaging both model code (that is optimized with torch.compile) and a state_dict (.pt), which is the model artifact file.
# 1. Convert a regular module to an optimized module
opt_mod = torch.compile(mod)
# 2. Train the optimized module
# ....
# 3. Save the opt module state dict
torch.save(opt_model.state_dict(), "model.pt")

# 4. Reload the model
mod = torch.load(model)

# 5. Compile the module and then run inferences with it
opt_mod = torch.compile(mod)

In this example, torch.compile needs to be called twice, at train time and inference time. this is because the artifact (model.pt) that is saved does not bake in the optimizations from torch.compile.
therefore, torch.compile is not used to produce model artifacts. the model can be deployed without torch.compile. torch.compile is a detail about the runtime dependencies for modls that depend on eager Pytorch code.

torch.compile can break when a certain reference cannot be resolved and trying to fall back to the Python implemention

this is true but it only happens in eager mode because there's no way to save artifacts that bake in these torch.compile optimizations with graph breaks.

to summarize the above, torch.compile optimizes a model's graph, but doesn't handle saving the model graph + state_dict weights. I think it'd be great to have a place in the mlm extension for calling out what hardware specific optimizations have been applied to an inference pipeline that depends on nn.Module or similar ml framework constructs. But I don't think the model artifact section is the right place to do that.

My intuition would be that mlm:accelerator, mlm:accelerator_constrained, etc. would be used in combination with the corresponding state dict generated for a given hardware optimization. Beyond that, I don't think it is the role of MLM anymore to be explicit about each parameter, and something like torch.export that addresses those issues should be used instead if that really matters.
The idea of specifying mlm:artifact_type = torch.compile instead of directly using mlm:artifact_type = torch.save is simply to hint, as you mentioned and as in the example code, that this step was performed before torch.save to obtain the .pt2, and that reloading the model should ideally do torch.load + torch.compile rather than only torch.load. Since the compile step could modify which inference operations actually happen and which results are obtained, such as by affecting digit precisions for example, replicable results need to consider this hint that affects the obtained state dict. In other words, mlm:artifact_type doesn't only indicate the "content-type" of the artifact, but how it is intended to be used as well.
Now, I'm not saying torch.compile is necessarily the "right way", but it is "a way", and I prefer MLM indicating it to guide best practices.

torch.compile doesn't affect the state_dict at all though. it has no bearing on the contents of the file from torch.save. I don't think we should guide users to call torch.compile an artifact type because it has no bearing on the contents of artifacts saved with torch.save.
also it is not the convention to save state_dict weights as ".pt2". of course it is possible but the torch docs state it is convention to only use .pt2 from torch.export. I could call a torch.export file .pt just like I could call a .tiff file a .txt but that's be misleading and go against convention.

The same docs also says:

torch.compile() also utilizes the same PT2 stack as torch.export

and

This feature is a prototype under active development and there WILL BE BREAKING CHANGES in the future.

Therefore, the convention is not really well-established. I cannot consciously force users toward torch.export until the feature is stable, and the workaround is currently to use torch.compile that has been available for a long time.
As mentioned previously,

mlm:artifact_type doesn't only indicate the "content-type" of the artifact, but how it is intended to be used as well

Therefore, even if the state_dict would be identical with/without torch.compile, this is a hint telling you explicitly that the reproducible runtime using the provided .pt[h][2] file expects the user to do torch.compile before running model inference, and if not done, results might differ (either in terms of prediction values, runtime speed, etc.). If we could only rely on the file type to infer how to employ the model, we wouldn't need the mlm:artifact_type. This is exactly what https://github.com/stac-extensions/mlm#artifact-type-enum is already mentioning.
Again, I have no issue about adding more references to explanations/recommendations for using a preferred approach over another in the best-practices for better reproducibility, but I do not think this is a sufficient reason to remove torch.compile entirely. There are some scripts out there that still use torch.compile, and they must be able to indicate it somehow. I would be happy to link to any official pytorch guidance in the best-practices document about which approach is better and why to use it, but this explanation should most definitely live in pytorch docs, not in MLM, since the extension intends to be framework-agnostic.

mlm:artifact_type doesn't only indicate the "content-type" of the artifact, but how it is intended to be used as well

maybe this is the core issue. are we overloading artifact_type with these two concepts? I think so. And I think it will confuse users looking at MLM metadata.

Therefore, even if the state_dict would be identical with/without torch.compile, this is a hint telling you explicitly that the reproducible runtime using the provided .pt[h][2] file expects the user to do torch.compile before running model inference, and if not done, results might differ (either in terms of prediction values, runtime speed, etc.).

this seems like a lot to infer. I'd rather have a separate field that marks what inference optimizations are made in the inference pipeline that don't affect the content type. then it would be clear that the content-type field indicates how to load the model and the other, optimization focused field dictates the suggested inference code path to use at inference time.
I also have no issue with adding more recommendations and I don't want to remove mention of torch.compile. I'm fine even with removing torch.export until the API is solidified and stable. I also don't think torch.compile is out of date or deprecated or it's an either or between torch.compile and torch.export, they serve different purposes. I just don't want users to think that we are hinting that it defines a unique content type.

My motivation for being more explicit here wrt content-type vs. properties of the inference pipeline is that earlier this year I was very confused going through the Pytorch documentation on how to produce a compiled model artifact that had no runtime dependencies other than Pytorch. I think it is unfortunate that the docs do not make it clear that torch.compile and torch.export are not used for the same purposes even though they share the underlying compile workflow.
I spent a good amount of time thinking torch.compile would produce the model artifact I needed when it wasn't the right tool. I don't want to mislead users into doing the same.

overloading artifact_type with these two concepts: "content-type" and "intended use"

Somewhat (and maybe even more than 2 concepts).
The "content-type" should really be indicated by the type, as detailed in https://github.com/stac-extensions/mlm#model-artifact-media-type. However, because .pt (and its corresponding content media-type), could refer to different (and overlapping) encoding/frameworks/etc., the type alone is not enough. This is where the mlm:artifact_type comes in to add more context on top to disambiguate similar uses of the same type. The type and mlm:artifact_type should somewhat align naturally.
The reason a single mlm:artifact_type property is used is that, in pretty much any other case, we would inject framework-dependant concepts. For example, if we defined some kind of mlm:artifact_compile: true|false property to indicate whether torch.compile is used or not, what happens for frameworks that have no concept "compile"? What if they do, but that refers to something else entirely? Then, MLM definitions are not meaningful anymore in terms of searching/filtering STAC items.  I want to avoid having any sort of listing of specific properties from all possible frameworks, because it becomes a maintenance burden and goes against the unifying intention of MLM.
If usage is not obvious from the type and mlm:artifact_type alone, then documentation should be provided, either as a description property, or strait-up an accompanying reference code/script using roles: [mlm:source_code] that makes usage obvious.
Personally, I think this is such a specific edge-case, that we would be better off by just adding a "⚠ (see best-practice)" next to torch.compile in the table, and provide some explanations of the intended use. Then, users are free to follow that recommendation or not.

Personally, I think this is such a specific edge-case

Pytorch, Tensorflow, and JAX all provide mechanisms for JIT and AOT compiled models. JIT and AOT models have very different deployment environments and level of effort from those looking to run the models. I think it is important that the MLM capture this variation, it's probably something users even want to search on because "level of effort to try the model out" is often a first order concern.
Examples of AOT and JIT options besides Pytorch
https://www.tensorflow.org/tutorials/keras/save_and_load#savedmodel_format
https://jax.readthedocs.io/en/latest/export/export.html
I think if we try to describe the artifact_types for JAX and Tensorflow models this would come up again. Therefore I'd like to be more explicit that the artifact_type is the method of how a model was saved. We're already doing this to some extent, with the exceptions being

Old Artifact Type
New Artifact Type

torch.jit.script
torch.jit.save

torch.export
torch.export.save

torch.compile
remove in favor of a separate, framework agnostic jit_compiled flag on the model source code asset

I think this gets around the problem of a .pt file not being very informative. This would also contain information on if a model was AOT compiled or not. for those who find it useful, users can look up the framework specific methods listed here and learn more about them when choosing how to save and describe their models.

we would be better off by just adding a "⚠ (see best-practice)" next to torch.compile in the table, and provide some explanations of the intended use. Then, users are free to follow that recommendation or not.

I think the MLM README is already too length and difficult to navigate, and I've had this reaction from some folks who have gone through it at Wherobots. IMO the spec is too open to interpretation and we are trying to fill in those gaps with recommendations. Rather than you and I providing more paragraph recommendations to plug gaps, I think we could make the spec more helpful by providing the options to accurately describe a model without ambiguity.
I'm down to do the work here wrt to describing the artifact_type options. And how to represent if the model source code asset has a JIT compile step, this could be a an additional boolean field that indicates the presence of JIT compilation somewhere.

The "content-type" should really be indicated by the type, as detailed in https://github.com/stac-extensions/mlm#model-artifact-media-type. However, because .pt (and its corresponding content media-type), could refer to different (and overlapping) encoding/frameworks/etc., the type alone is not enough.

Ideally I would want there to be clear IANNA types but since there are not I don't see authors of MLM metadata using this if there are no conventional options. And as you noted, it wouldn't be informative enough on it's own to understand how to load the model because of .pt ambiguity.

belatedly coming back to your earlier comment

The examples don't have to be an exhaustive list. The mlm:artifact_type field is not even explicitly defined in the JSON schema at the moment. We should define it as a lax string, similar to other definitions:

agree, I can do this

I think the MLM README is already too length and difficult to navigate

I agree. This is why I proposed adding this details in the best-practices. The full definition and table in the current https://github.com/stac-extensions/mlm#artifact-type-enum could actually be entirely in a specific best-practices section. From the point of view of the https://github.com/stac-extensions/mlm#model-asset, the mlm:artifact_type is just "some string". The fact that framework function names are use are just to facilitate reuse/comprehension, but are not required.
Framework-specific details shouldn't be explicit at the README level.

framework agnostic jit_compiled

I think we need to refine this definition. Rather than mlm:jit_compiled and start having many flags for every combination, I think it is better to have a mlm:compiled_method with a set of values like "aot", "jit", null (default/none applied).
There is also a mlm:compiled role under https://github.com/stac-extensions/mlm#model-asset that should be better described (or removed since redundant from mlm:compiled_method?) in the best-practices.

after chatting we will add a mlm:compiled_method field and remove the torch.compile artifact_type. Good suggestion!


Unresolved external symbol

Hello, i've generated SDK for RocketLeague with your tool.
But when i use this code
UGameEngine* pGameEngine = UObject::FindObject<UGameEngine>("GameEngine_TA Transient.GameEngine_TA_1");
i get this error
Error	LNK2019	unresolved external symbol "public: class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > __thiscall SDK::UObject::GetFullName(void)const " (?GetFullName@UObject@SDK@@QBE?AV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@XZ) referenced in function "public: static class SDK::UGameEngine * __cdecl SDK::UObject::FindObject<class SDK::UGameEngine>(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &)" (??$FindObject@VUGameEngine@SDK@@@UObject@SDK@@SAPAVUGameEngine@1@ABV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@Z)	RocketLeague	C:\RocketLeague\test.obj	1
I didnt get any error when generating the SDK, is there something i have done wrong?

Maybe you haven't added the needed .cpp files to your project?
See https://github.com/KN4CK3R/UnrealEngineSDKGenerator#and-now

Yes, now i have generated new SDK after recent update, and i get alot of
Error	C2079	'SDK::FVOscillator::X' uses undefined struct 'SDK::FFOscillator'

In the new update some of the generated struct are out of order, make sure to sort them out

Yes, you need to move the structs FNotificationInfo and FFOscillator above the structs which are using them. Currently I don't know if I can fix this problem.


create linode Unknown field(s): metadata

version：webinarium/linode-api 1.0
If adding the metadata key ,When creating an instance, Unknown field (s): metadata will be returned

fix it:
src/Entity/Linode.php
// Extra field for create/update operations.
public const FIELD_METADATA         = 'metadata';

src/Internal/LinodeRepository.php
protected function getSupportedFields(): array
{
    return [
        xxxx,
        Linode::FIELD_METADATA,
    ]
}

https://www.linode.com/docs/api/linode-instances/#linode-create__request-body-schema
metadata
An object containing user-defined data relevant to the creation of Linodes.

I have removed the fields checking as of the version 3.0.0.
Please try again.


Document symbol

closes https://github.com/gluon-lang/languageserver-types/issues/72

Fixed and verified that it actually works with VSCode, unless you use None for details

I would appreciate a release with this change btw :)

Released as 0.49


All commands are guild only

None of the commands are working when not ran in a guild.

I think this is related to Common.Context

2fd93ff fixes this.


What is updated if calling sudo gem update fastlane

I just want to know if all components (snapshot, deliver, etc.) will be updated or just fastlane. I hacked snapshot to fit my needs and I don't want to overwrite it.
Also I didn't understand, will the old UIAutomization using .js files still work in newer versions of snapshot?

The dependencies are all defined here: https://github.com/fastlane/fastlane/blob/master/fastlane.gemspec

Going to close this issue for now as it seems like the question was answered. @bb-git if you still have any questions please reopen the issue and let us know!


Question : individual column searching (select input)

I'd like to use your package with  serverside data for doing something like this :
http://www.datatables.net/examples/api/multi_filter_select.html
I'm new to Datatables (and JS !!!). I've tested the example, but it works "client side".
Do you thinks it's possible (and easy) to transform it in serverside ?
Does somebody code things like this ?
Regards,
Paguemaou

I just tried this and the package works with column search. However, the search result is not 100% accurate especially when the value have special chars. I also made some adjustment on the demo js for it to work.
initComplete: function () {
            this.api().columns().every( function () {
                var column = this;
                var select = $('<select><option value=""></option></select>')
                    .appendTo( $(column.footer()).empty() )
                    .on( 'change', function () {
                        var val = $.fn.dataTable.util.escapeRegex(
                            $(this).val()
                        );

                        column
                            .search( val ? val : '', true, false )
                            .draw();
                    } );

                column.data().unique().sort().each( function ( d, j ) {
                    select.append( '<option value="'+d+'">'+d+'</option>' )
                } );
            } );

Will tag this as for enhancement. And will fine tune the search result next week.


Add some checkboxes to the "Delete Project" modal

With any luck, this will slow down users who usually blow by the warning text here and then ask us to un-delete.

Seven checkboxes feels right:


Add basic sample to show normals

Basic sample to show usage of Normals without any external dependencies.

@torbsorb I added fixup commits for my own comments.


categorize model

Dear developer.
This model convolutional neural network or how to properly categorize this model?
Thanks.

Sorry, are you asking what type of model SparseTSF should be categorized as? If that's the question, SparseTSF should be classified as a linear-based model.

Thanks
I was put in doubt by nn.Conv1d.


Anime4K isn't compatible with --vo=libmpv in memento

Describe the bug
When adding anime4K shaders to Memento
you can't use presets 1 and 2 and clear the shaders doesn't work
To Reproduce
Steps to reproduce the behavior:

Go to Memento and clone the repo
Compile from source and install memento
download anime4K shaders and install in the memento config folder
See error

Expected behavior
Anime4K should work flawalessly
~/D/J/A/[Judas] Kimetsu no Yaiba - Mugen Ressha-hen [1080p][HEVC x265 10bit][Multi-Subs]> memento \[Judas\]\ Kimetsu\ no\ Yaiba\ -\ Mugen\ Ressha-hen\ -\ 05.mkv
AO: [pipewire] 44100Hz stereo 2ch floatp
VO: [libmpv] 1920x1080 yuv420p10
[libmpv_render] after creating texture: OpenGL error INVALID_ENUM.
[mpv_thumbnail_script_client_osc] The 'tick' event is deprecated and will be removed.
[libmpv_render] before loading program: OpenGL error INVALID_ENUM.

Desktop (please complete the following information):

OS: Manajro Linux
Version: I guess anime4K version? the latest
GPU/Version: GTX 1650 Nvdia drivers version 550.107.02
Media Player and Version: Memento 1.4.1 and main

you can't use presets 1 and 2 and clear the shaders doesn't work

This sounds like you may not have installed input.conf or mpv.conf in the correct location.

I've tested on both MacOS and Windows 11 (have not tested Linux), and Anime4K is working correctly with vo=libmpv.

you can't use presets 1 and 2 and clear the shaders doesn't work

This sounds like you may not have installed input.conf or mpv.conf in the correct location. ) I've tested on both MacOS and Windows 11 (have not tested Linux), and Anime4K is working correctly with vo=libmpv.

side note: in the image you show mode A+A which I said "works" (as in the text appears) mode A or mode B are the "problematic" ones
I'm experiencing an issue on linux, telling me it works on windows/macOS does not help, and I tested it on more than one machine so it's not a me-issue
Also I did install the conf files in the correct location, I'm not that clueless

as per memento's instructions:

( the symbol ~ is equal to the home folder)

side note: in the image you show mode A+A which I said "works" (as in the text appears) mode A or mode B are the "problematic" ones

I specifically chose Mode A+A so that you can clearly see the lines being sharper in the screenshots, thus implying that the shaders worked correctly. Mode A and B also worked in my testing.

I'm experiencing an issue on linux, telling me it works on windows/macOS does not help, and I tested it on more than one machine so it's not a me-issue

The only difference I had from your setup was that I put the option as a config in my mpv.conf file, and I was using a different operating system. And the result is that Anime4K is working as intended on both operating systems. Indicating that --vo=libmpv may just not be compatible with your specific Linux kernel. There’s nothing we can do to fix compatibility issues with libmpv itself as Anime4K is just a set of upscale shaders. https://github.com/mpv-player/mpv/issues/15019


refactor: Serialize transaction to use Buffers instead of borsh

This PR removes the dependency of borsh and the custom/hacky schemas to native transaction serialization.
Create Account:
https://explorer.solana.com/tx/4Cb1H5epxB7dsXTK2iYhnpNEBSCXV1B88mrMzmbS6UijLtgmhTErq8TmB9dMBUPpJik4aPhSaaqg1Vb9YwfkisU3?cluster=devnet
Make Transfer:
https://explorer.solana.com/tx/64nS2zeYmSepcQa2hGGD4MP11JBWaYyPhd3vb4NC9mKCYQTQgM11Vu3Py5o5LFHCcsv9VAfdUwQzY5tws2cFFeCa?cluster=devnet

@RamirezAlex looks and works great on my end!
I moved the main parse-and-sign logic out into the Solana library (see libs/solana/src/lib/helpers/parse-and-sign-token-transfer.ts and libs/solana/src/lib/helpers/parse-and-sign-transaction.ts ) and made a few small tweaks (closes #160).
Feel free to squash and merge.


Authentication Problem when collecting token

Hey guys,
I'm trying to connect to the basic profile api through jupyter. When I click on the link to get my token and authorize my application I get this error on the link:
http://localhost:8000/callback/?error=invalid_scope&error_description=Your+application+has+not+been+authorized+for+the+scope+"r_fullprofile"#!
I read that now you have to apply through email to get full profile access but for me only basic profile is enough as I don't have the time to wait for linked in to approve this. Is there a way to get around this problem? Maybe i can choose parameters for linkedin.PERMISSIONS.enums.values()?
I'm a bit new to python and have to do this application for a school project. Some help would be greatly appreciate and thanks for making the updates for python 3!!

Maybe i can choose parameters for linkedin.PERMISSIONS.enums.values()

You answered your own question, look what are the values returned by linkedin.PERMISSIONS.enums.values() and replaced this by the one you want.


prop of component, in conditionally rendered slot, stops being reactive

Version
2.5.14+
Reproduction link
https://jsfiddle.net/cLdbL2t2/22/
Steps to reproduce
Clicking "update message" button adds dots after message.
Once "alter" button is clicked, the message is no longer updated.
What is expected?
Updating message continues working even after slot is altered.
What is actually happening?
Prop value stops being reactive, without error nor warning.

This behavior takes place after v2.5.14  including v2.5.17-beta.0.
Any version 2.5.13 and lower works as expected.
Unit test can be found in my github https://github.com/honeroku/vue-slot-investigate
I am understanding this is edge case and refactoring structure to avoid placing identical slot in conditional positions resolves my problem.
However, this behavior was not occurring in older versions and suddenly changed without error or warning,
so I am considering this is a bug and reporting to let you know.

I have tried to find the difference between v2.5.13 and v2.5.14, finally find these commits  956756b may cause this bug.

I think the problem here is that you have the same slot twice in your template. I modified your jsfiddle to show that conditional rendering of slots works as expected.
https://jsfiddle.net/5kcdcpr6/2/
Note: only the last slot is working

As a workaround, you can attach the slot attribute to a template element instead of div.


offline signing with typescript

hi,
i want to sign my transaction offline, i don't want to use iota sdk's sign functions, i'm using elliptic's signing function(ed25519) to sign my transaction. however, it does not match with the signature generated by your sdk.
could you provide an example for offline signing with javascript.
thanks, dante

Hello, we have an example for offline signing here https://github.com/iotaledger/iota-sdk/tree/develop/bindings/nodejs/examples/client/offline_signing
With the wallet module one can also use the prepare methods and sign them offline the same way

this is still using your signing algorithm, i want to use elliptic or tweetnacl, but im not getting same result, could you please provide the steps i need to do, or share a working example with me.
thnaks, dante

Please provide a code example that shows what you're doing and the example with which you compare the result

this is what i am currently doing
const preparedTransaction = await this.client.prepareTransaction(
      undefined,
      {
        inputs,
        outputs,
        coinType: CoinType.IOTA,
      },
    );

    const signedTransaction = await this.offlineClient.signTransaction(
      secretManager,
      preparedTransaction,
    );

now i want to take the hash as follows, and sign it outside of my project
const hash = Utils.hashTransactionEssence(preparedTransaction.essence);

when i want to sign this hash using elliptic js library, it does not match with the first one!
if you want, you can verify it using this website (gives the same result as elliptic library)
https://cyphr.me/ed25519_tool/ed.html
thanks, dante

    const signedTransaction = await this.offlineClient.signTransaction(
      secretManager,
      preparedTransaction,
    );

What exactly is the secretManager here? (ofc remove/replace any secret with 0x000 or something)

const secretManager = { mnemonic: bip39.entropyToMnemonic(prop.privateKey) };

And prop.privateKey you also use in https://cyphr.me/ed25519_tool/ed.html?

yes it's exactly the same

it worked! thanks @Thoralf-M
const customSecManager = new SecretManager({
      privateKey: '0x' + prop.privateKey,
});```

thanks, dante


Wip.email multi event

Thanks for the feedback @gulfaraz! This was really what I was hoping to get out of this review

Thanks for the review! Great that you summarized the changes!

Note for future refactors: Let's keep the new features in a separate PR from the refactor. If there are features added or removed, it's impossible to track if we've lost some functionality.

I get this, and agree that it is sub-optimal

Would it be possible to document the notifications logic in a flow chart? Feel free to commit the flow chart image in the notifications folder. So it can be used as documentation coupled with the code.

Sound like a good idea. I am creating an extra task for this on the task board AB#28230.

Could we use an email HTML framework like mjml?

I also created a separate task for this AB#28231. That probably would have been a better approach from the start. However considering the limit time to complete this work I'd rather first finish the API tests before implementing this.


(Suggestion) Totem popping in void

Normally having a totem while falling in the void just kills you instead of pop the totem.
Maybe an option to make totems pop in the void as well (this would be compatible with inventory totems)

and how would that help?
Would only make sense if it teleported you to the next safe spot, which is not necessarily close (if we talk about the end)

and how would that help?
Would only make sense if it teleported you to the next safe spot, which is not necessarily close (if we talk about the end)

gives you time to do /home or anything idk

Well assuming the command is activated on the server, yeah, but a more elaborated and universal solution would be what I suggested then

Yeah maybe, but I don't think such an option will harm anyone. It could be used by the few servers out there that use /home or anything related. And if this would be added a tp option could also help, something that basically tps you to the last block you stood on also known as antivoid

gives you time to do /home or anything idk

If you have time to type the command then you haven't died yet, so the totem wasn't used. In fact, having a totem doubles your available time to type the command, which would save the rest of your inventory from certain destruction otherwise ;)


Disable features after 5 invalid credentials check

For now, all feature are disabled as soon as the credential check fails.
There might be a temporary issue (network by exemple) and credentials are still valid.
So all user's enabled feature should be disabled after 5 invalid credentials check.

Merged in #10.


Do not validate Content-Type if it is missing

This is the behaviour flex had prior to d91debaa, in that commit the
Content-Type validation logic was changed to assume the Content-Type
of a request was application/octet-stream if it was not otherwise
given. This is not wrong per se, but as the Content-Type header is
not given with requests with no body (e.g. a GET request) this meant
that these requests were considered to have the an invalid
Content-Type when they really, correctly had provided None. This
restores the previous behaviour where, in all cases, Content-Type
was would be ignored if not provided.

@pipermerriam sorry about this.  That's because GET requests (which do not provide a Content-Type) were now considered invalid as the validator would default the content type to application/octect-stream.
As stated above this restores the previous behaviour.
I guess the completely correct behaviour would be something like this:

No Content-Type and no Body: fine, don't validate
Content-Type and no Body: not 100% sure what to do in this case, I guess validate the Content-Type as usual (even though no data has been sent?)
Content-Type and Body: validate content type as normal
No Content-Type and Body: validate with default content type

Merged and released.  Thanks again for the contribution and for catching and fixing this.


Updated Korean version to 1.1.0

Finally!

YAY! :)

Hopefully I didn't miss anything.

The title “Strings as CSS values” is not translated. Is it intended?

Of course I missed something... I'll fix it now.

Done.

Thanks! Actually you forgot to replace code snippets with file includes but I did it. ;)
See: https://github.com/HugoGiraudel/sass-guidelines/commit/280407c1a858b38ef0c766a1aab462ec5823a433.
I'm glad to have the korean version in 1.1.0! \o/

Aw I totally missed it. Sorry about that, and thanks for the work!

Thank you. :)

My pleasure. :)

Stay tuned for 1.2.0! :P


model_id=None on alexa and other devices

System Health details
System Information

version
core-2024.11.3

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.12.4

os_name
Linux

os_version
6.6.54-haos

arch
x86_64

timezone
America/Sao_Paulo

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

HACS Data
ok

GitHub API Calls Remaining
5000

Installed Version
2.0.1

Stage
running

Available Repositories
1462

Downloaded Repositories
42

AccuWeather

can_reach_server
ok

remaining_requests
31

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 13.2

update_channel
stable

supervisor_version
supervisor-2024.11.4

agent_version
1.6.0

docker_version
27.2.0

disk_total
116.7 GB

disk_used
57.2 GB

healthy
true

supported
true

host_connectivity
true

supervisor_connectivity
true

ntp_synchronized
true

virtualization

board
generic-x86-64

supervisor_api
ok

version_api
ok

installed_addons
Duck DNS (1.18.0), ESPHome Device Compiler (2024.11.1), Home Assistant Google Drive Backup (0.112.1), Mosquitto broker (6.4.1), NGINX Home Assistant SSL proxy (3.11.1), Node-RED (18.1.1), Advanced SSH & Web Terminal (19.0.0), Samba NAS (12.3.2-nas), Studio Code Server (5.17.3), UniFi Network Application (3.5.0), Grafana (10.2.2), Log Viewer (0.17.1), VLC (0.3.0), Uptime Kuma (0.12.4), InfluxDB (5.0.1), Jellyfin NAS (10.10.3), Portainer (2.24.0), Filebrowser (2.23.0_14), Fusion (2024.10.1), qBittorrent (5.0.2), OpenSpeedTest (v2.0.5), Homarr (0.91), Scrutiny (Full Access) (v0.8.1-9)

Dashboards

dashboards
5

resources
21

views
4

mode
storage

Recorder

oldest_recorder_run
20 de novembro de 2024 às 00:23

current_recorder_run
25 de novembro de 2024 às 21:33

estimated_db_size
878.88 MiB

database_engine
sqlite

database_version
3.45.3

Sonoff

version
3.8.1 (ffa7e22)

cloud_online
3 / 3

local_online
2 / 2

Spotify

api_endpoint_reachable
ok

Checklist

[X] I have read the FAQ to see if there is a known solution to my problem.
[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
Hi, i'm new on powercalc, I read about it on some site and I'm trying to add it, on my Ha, but it doesn't find anything on autodiscovery.
I followed the documentation, included the powercalc: at the end of config restated it and it not showing anything. I have a lot of supported devices like alexa echo dot 3rd and 4 gen...
after looking for some informations about the echos, I notice that amazon alexa media player integration does not get the device_id alog a lot of devices that I have.
my tuya outlets have the abilty to count energy used, but is not showing too
sorry for the bad english
Reproduction steps

install integration via hacs
add powercalc on configuration.yaml
restart HA
no devices discovered
...

Debug logs
2024-11-25 22:31:07.177 DEBUG (MainThread) [custom_components.powercalc.discovery] media_player.echo_sala: Model not found in library, skipping discovery
2024-11-25 22:31:07.177 DEBUG (MainThread) [custom_components.powercalc.discovery] media_player.echo_sala: Auto discovered model (manufacturer=Amazon, model=Echo Dot Clock (Gen4), model_id=None)

Diagnostics dump or YAML config
No response

Also share the model id's how they are discovered for the other devices you expect to be discovered, than we can also add the correct aliases for these in one go.

It is! But the device information doesn't give The "model_id" and I don't
know why
Em qui., 28 de nov. de 2024, 10:29, Bram Gerritsen @.***>
escreveu:

Also share the model id's how they are discovered for the other devices
you expect to be discovered, than we can also add the correct aliases for
these in one go.
—
Reply to this email directly, view it on GitHub
https://github.com/bramstroker/homeassistant-powercalc/issues/2719#issuecomment-2506132346,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFIQOOJMGDC3ZSUSWLE54N32C4LDBAVCNFSM6AAAAABSPJX2G6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBWGEZTEMZUGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

It is! But the device information doesn't give The "model_id" and I don't know why Em qui., 28 de nov. de 2024, 10:29, Bram Gerritsen @.> escreveu:
…
Also share the model id's how they are discovered for the other devices you expect to be discovered, than we can also add the correct aliases for these in one go. — Reply to this email directly, view it on GitHub <#2719 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFIQOOJMGDC3ZSUSWLE54N32C4LDBAVCNFSM6AAAAABSPJX2G6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBWGEZTEMZUGY . You are receiving this because you were mentioned.Message ID: @.>

That's normal and correct.
model_id is a rather new field in HA which has been added 2 months ago. Not many integrations expose model data there yet.
Powercalc looks at both model and model_id and tries to match that agains Powercalc library.
So in your case it's just a matter of getting the models as alias added in Powercalc library.
That's Echo Dot Clock (Gen4) for the one you shared. You are probably using another integration for the Amazon echo's as others, causing the models to be different on your HA installation than others.
Please provide full list of the models how they are identified in your system, and which corresponding model it should be in Powercalc library.
Than I will add the aliases accordingly which should solve your issue and get the models correctly discovered.


devtools: license header check fails on .hcl files

The new license header checker seems to not work on hcl files, as seen here: https://github.com/terraform-google-modules/terraform-google-kubernetes-engine/pull/980

I believe upstream addlicense has support for HCL and we just missed headers on that. Also okay to remove that file since we no longer need that for caching.

Oops, I guess I should have checked the file first. Thanks. :)


feat: launching with unlock_time set to 0 burns the token

#186

Thanks, Doing the requested changes

@enitrat What am I missing here?

@enitrat Changes done. Kindly review

@cpp-phoenix needs to be formatted


[Kitty] Error: open /dev/tty: no such device or address

running it directly result in nothing, so i check the code and get the command:
:!kitten @ launch --type=window kitten icat --hold '/home/ogios/.config/eww/ava.jpg'
Error: open /dev/tty: no such device or address
shell returned 1

i run it in kitty terminal bash shell, it worked, but run it in nvim doesn't
what should i do?

you need to enable "remote control" (the error message for some reason doesn't show up on neovim)
you can achieve this by either editing kitty.conf or adding kitty -o allow_remote_control=true to your command
for example, the command you sent, becomes: kitty -o allow_remote_control=true kitten @ launch --type=window kitten icat --hold '/home/ogios/.config/eww/ava.jpg'
now, you have three options:

wait for the devs fix this
make your own build
adding this custom command to your file manager (if you use nvim-tree i can help you with that)

i've set allow_remote_control yes in kitty.conf
but can not open image in current kitty window using !kitten @ launch --type=window kitten icat --hold '/home/ogios/.config/eww/ava.jpg', i have to open another kitty window !kitty kitten @ launch --type=window kitten icat --hold '/home/ogios/.config/eww/ava.jpg'

did you try adding kitty -o allow_remote_control=true?

did you try adding kitty -o allow_remote_control=true on vim?

I encountered the same problem that ogios mentioned. It worked after I added kitty -o allow_remote_control=true.Thanks a lot!


Make bot @mention user on user's first time joining

How would I make a bot do something (like send a message) when a user first joins the a discord server?
I've been looking all over for a hint as to how to do that...

c.on('serverNewMember',(x,y)=>{
	if(x === c.servers.get('id',"SERVERID"))
		c.sendMessage(x.channels.get('name','general'),"Welcome to my Server "+y.mention()+"!")
})

pls
**https://eslachance.gitbooks.io/discord-js-bot-guide/content/samples/welcome_message_every_x_users.html
**


motion detection example

Hello,
Could someone provide a example for motion detection?

Did you find any resources ? please link if any.
Thanks in advance. !


Pre-receive hook to block commits by size under 50MB

Pre-receive hook to block commits smaller than repo Maximum Object Size setting.
We had a requirement from a user to block any file and any commit that was over 100000 bytes in size.
Obviously the Maximum Object Size setting would be useful for this:

Set the maximum size of Git objects that can be pushed to this repository.
Allowing large objects to be pushed into Git can degrade performance, consider other options
(e.g. git-lfs) before increasing this value.

However, as the minimum value that can be set is 50 MB this was not the way forward.
So knocked up this pre-receive hook. Not sure that it is the most efficient way of doing things, but it seems to work OK.

Cool addition, @mzzmjd.
Currently it will not work for newly created branches, when e.g. doing a git push -u origin HEAD.
Feel free to take a look at one of the other scripts on how to also capture new branches, for example
https://github.com/github/platform-samples/blob/fd0818b76362aa5f46436a14763c14f8cc9f4fd7/pre-receive-hooks/block_file_extensions.sh#L11-L36

@mzzmjd: GitHub.com and GitHub Enterprise Server (since version 2.20) now have an ability to restrict the commit size per repository to values smaller than 50 MB,, e.g. 1 MB:
https://help.github.com/en/enterprise/2.20/admin/installation/setting-git-push-limits
I will close this pull request because of this. Feel free to reopen if you believe there is still a reason for a pre-receive hook.


added kaspresso integration

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

Will check ASAP

Could you squash the commits at first

pull request with squashed commits https://github.com/agoda-com/Kakao/pull/143


Option to Disable Logging

Is your feature request related to a problem? Please describe.
Disable logging for when running the test suite etc.
Describe the solution you'd like
  Langchain.logger.level = :warn could disable farday logging.
Describe alternatives you've considered
Additional context

@bendangelo What're your thoughts of dropping:
Langchain.logger.level = :warn

somewhere in the spec_helper.rb?

I should have mentioned the logs are coming from faraday. Changing that log level doesn't stop faraday from logging things.

I'm still waiting on a response.

For lack of a better solution, I just commented out the offending line in the installed gem...

I'm looking into this right now...

I'm rolling out a fix but I'd like to overhaul the whole logger solution over the long term.
If you set:
Langchain.logger.level = :error`

you should not be seeing anything from Ollama unless it errors.


Add support for frameworks 4.6.2, 4.7 and 4.7.1

Fix #201
This also adds support for using VSSetup module to find location of v15.0 build tools.

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

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

Looks good @rkeithhill. Thanks!

there must be some type of way to make this work without constantly updating psake as new frameworks are released. An issue for another time

Agreed!  Thought the same thing as I was tweaking the script.


Improve the code to print the secrets using printTable

Improve the code to print the secrets using the cli-runtime line and printTable.
Refactor the code to use the print functions from the util package
See: #463

To be investigated as test is failing.
=== RUN   TestOutput
    secrets_test.go:222:
        	Error Trace:	/Users/cmoullia/code/cnoe/fork-idpbuilder/pkg/cmd/get/secrets_test.go:222
        	Error:      	Expected nil, but got: &json.SyntaxError{msg:"unexpected end of JSON input", Offset:0}
        	Test:       	TestOutput
    secrets_test.go:223:
        	Error Trace:	/Users/cmoullia/code/cnoe/fork-idpbuilder/pkg/cmd/get/secrets_test.go:223
        	Error:      	Not equal:
        	            	expected: 4
        	            	actual  : 0
        	Test:       	TestOutput
    secrets_test.go:242:
        	Error Trace:	/Users/cmoullia/code/cnoe/fork-idpbuilder/pkg/cmd/get/secrets_test.go:242
        	Error:      	Not equal:
        	            	expected: 0
        	            	actual  : 2
        	Test:       	TestOutput
    secrets_test.go:243:
        	Error Trace:	/Users/cmoullia/code/cnoe/fork-idpbuilder/pkg/cmd/get/secrets_test.go:243
        	Error:      	Not equal:
        	            	expected: 0
        	            	actual  : 2
        	Test:       	TestOutput

Question: Should we for the core package secrets display the information as such ?
[
  {
    "name": "argocd-initial-admin-secret",
    "namespace": "argocd",
    "data": {
      "password": "cP0HzmKo8vbzpa5L",
      "username": "admin"
    }
  }
]

OR
[
  {
    "name": "argocd-initial-admin-secret",
    "namespace": "argocd",
    "username": "admin",
    "password": "cP0HzmKo8vbzpa5L",
    }
  }
]

To be investigated as test is failing.

Fixed

Question: Should we for the core package secrets display the information as such ?
[
  {
    "name": "argocd-initial-admin-secret",
    "namespace": "argocd",
    "data": {
      "password": "cP0HzmKo8vbzpa5L",
      "username": "admin"
    }
  }
]

OR
[
  {
    "name": "argocd-initial-admin-secret",
    "namespace": "argocd",
    "username": "admin",
    "password": "cP0HzmKo8vbzpa5L",
    }
  }
]

I vote for 2nd option.

I vote for 1st option. This way we can have flexibility for complex fields.

This is what I did

I suggest that we create a separate Printer package

I suggest that we create a separate Printer package

Great idea. Fixed: 771fd89

and Printer Interface.

Why ? Can you elaborate ?

Question: Should we for the core package secrets display the information as such ?
[
  {
    "name": "argocd-initial-admin-secret",
    "namespace": "argocd",
    "data": {
      "password": "cP0HzmKo8vbzpa5L",
      "username": "admin"
    }
  }
]

OR
[
  {
    "name": "argocd-initial-admin-secret",
    "namespace": "argocd",
    "username": "admin",
    "password": "cP0HzmKo8vbzpa5L",
    }
  }
]

I prefer the second format but like you said, we can discuss that in another PR / issue.

and Printer Interface.

Why ? Can you elaborate ?

We can address this later. As of now, we can merge this PR

Should we wait an approval from @nabuskey @blakeromano ? @punkwalker


SameSite for Session cannot be set

What steps will reproduce the problem?
The Yii Guide describes that yii\web\Session supports security flags: https://www.yiiframework.com/doc/guide/2.0/en/runtime-sessions-cookies#security-settings
What is the expected result?
That I can set in config those flags for the session cookie.
What do you get instead?
The Session object does not allow for those flags, nor does it allow to define the cookie..
Additional info
Flags: httpOnly, secure, sameSite.
For the identyCookie it can be set through the user compoenent, but it seems not possible todo same for session.
'user' => [
         'identityCookie' => [
            'name' => '_identity',
            'httpOnly' => true,
            'sameSite' => \yii\web\Cookie::SAME_SITE_STRICT,
            'secure' => true,
         ],
]

Q
A

Yii version
2.0.36

PHP version
7.4

Operating system
Win10

I probably am missunderstanding something, as session code does somehow create the cookie based on other cookie infos...
https://github.com/yiisoft/yii2/blob/master/framework/web/Session.php#L394
On the other hand, the comment here seems to be wrong too:
https://github.com/yiisoft/yii2/blob/master/framework/web/Session.php#L371

For securtiy, an exception will be thrown if sameSite is set while using an unsupported version of PHP.

Yet the code later runs different code based on the PHP version, but well as I did not test, it might throw an error as commented.
https://github.com/yiisoft/yii2/blob/master/framework/web/Session.php#L398

So for the user it can be set as shown above, and for Session I overlooked the option cookieParams; so it work actually.


Add gruntwork badge to readme

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes

0 out of 2 committers have signed the CLA.

[ ] Etiene
[ ] yorinasub17

Have you signed the CLA already but the status is still pending? Recheck it.


Removed "merge marks" from pupilometry and media.

I removed "<<<<<<HEAD" "======" and other bits of code used created by git during
a merge. These were causing the program to crash when run. No testing on actual
data done, but at least it doesn't immediately cause python to crash with
a syntax error :).

Thanks again Dan! And there I was thinking I'd caught all the merge markup :) Let me know how the testing goes with your own data.


Limit syslog.go build to unix-like OSes.

Otherwise building / cross-building for Windows fails.

Thank you!


Unable to.compile V.0.191.0

After update from.V.0.170.2 to V.0.191.0 I'm unable to compile the project.
System.IO.DirectoryNotFoundException: the working directory "bin\Debug\net5.0-windows.playwright\win-x64\native" not exist

@MaxBex I see that the output is .NET5.0-windows... I can infer something from there. Could you tell us a little bit more about your projects? type of project, target framework. If you can share some relevant parts of your csproj that would be awesome.

I'm getting the same:
Error	MSB6003	The specified task executable "cmd.exe" could not be run. System.IO.DirectoryNotFoundException: The working directory "bin\Debug\net5.0\.playwright\win-x64\native" does not exist.
   at Microsoft.Build.Tasks.Exec.GetWorkingDirectory()
   at Microsoft.Build.Utilities.ToolTask.GetProcessStartInfo(String pathToTool, String commandLineCommands, String responseFileSwitch)
   at Microsoft.Build.Utilities.ToolTask.ExecuteTool(String pathToTool, String responseFileCommands, String commandLineCommands)
   at Microsoft.Build.Tasks.Exec.ExecuteTool(String pathToTool, String responseFileCommands, String commandLineCommands)
   at Microsoft.Build.Utilities.ToolTask.Execute()	TestEngine	C:\Users\...\.nuget\packages\playwrightsharp\0.191.0\build\netstandard2.0\PlaywrightSharp.targets

Potentially relevant project details:

<Project Sdk="Microsoft.NET.Sdk">

	<PropertyGroup>
		<TargetFramework>NET5.0</TargetFramework>
	</PropertyGroup>

	<ItemGroup>
	  <PackageReference Include="PlaywrightSharp" Version="0.191.0" />
	</ItemGroup>

</Project>

<Project Sdk="Microsoft.NET.Sdk">

	<PropertyGroup>
		<TargetFramework>NET5.0</TargetFramework>
	</PropertyGroup>

	<ItemGroup>
		<PackageReference Include="ExcelDataReader" Version="3.7.0-develop00310" />
		<PackageReference Include="ExcelDataReader.DataSet" Version="3.7.0-develop00310" />
		<PackageReference Include="ExtentReports" Version="4.1.0" />
		<PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="6.0.0-preview.1.21102.2" />
		<PackageReference Include="Microsoft.Extensions.Configuration" Version="6.0.0-preview.1.21102.12" />
		<PackageReference Include="Microsoft.Extensions.Configuration.Binder" Version="6.0.0-preview.1.21102.12" />
		<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="6.0.0-preview.1.21102.12" />
		<PackageReference Include="PlaywrightSharp" Version="0.191.0" />
		<PackageReference Include="SpecFlow" Version="3.7.13" />
	</ItemGroup>

</Project>

<Project Sdk="Microsoft.NET.Sdk">

	<PropertyGroup>
		<TargetFramework>NET5.0</TargetFramework>
	</PropertyGroup>

	<ItemGroup>
		<PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="6.0.0-preview.1.21102.2" />
		<PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.10.0-preview-20210219-03" />
		<PackageReference Include="NUnit" Version="3.13.1" />
		<PackageReference Include="NUnit3TestAdapter" Version="4.0.0-beta.1" />
		<PackageReference Include="PlaywrightSharp" Version="0.191.0" />
		<PackageReference Include="SpecFlow" Version="3.7.13" />
		<PackageReference Include="SpecFlow.NUnit" Version="3.7.13" />
		<PackageReference Include="SpecFlow.Tools.MsBuild.Generation" Version="3.7.13" />
	</ItemGroup>

</Project>

I was able to reproduce it

v0.191.1 published. Sorry for the hiccup folks.


Use the public ip from host labels if provided in metadata

Use the public ip from host labels if provided in metadata use the host agent's ip
https://github.com/rancher/rancher/issues/3201

@alena1108 please review

@ibuildthecloud @vincent99 need label name suggestion for defining ip address used by external DNS programming. We've put "io.rancher.public.ip" in this PR just to test things out, but the name is open for discussion.

Tested manually

LGTM, but won't merge till we decide on the naming and apply changes.

@ibuildthecloud and I vote io.rancher.host.external_dns_ip

@vincent99 @ibuildthecloud awesome, @prachidamle could you update the PR?

@alena1108 updated


Proxy support for Dropwizard HTTP client

I often encounter with a situation when in closed corporate environments direct access to the Internet is blocked by security reasons. So all connections to external resources happen through a proxy server (usually with BASIC authentication).
Deploying applications in such environments becomes a huge pain, because usually you don't know in advance that an application will need proxy connection support and sometimes you just don't have possibility to change code to add this facility.
I believe, it would be quite useful to have this facility in the Dropwizard config. It frees an application developer from thoughts about environments where an application will be used and shifts this responsibility to operations guys. And they can configure proxy client support just in the application config without using external tools.
PS I've added the docs to proxy configuration. But I'm not a skilled english speaker, so, please, fix them if I messed up something.

I see #681 has been merged.
Is there any work needed for this PR? If Java properties configuration is sufficient then this PR could be closed or moved to backlog from 0.8 release.

@arteam I'll move this PR out of the 0.8.0 milestone for now but I won't close it for now.
Maybe someone else wants to chime in whether the current state of affairs is sufficient or if this PR should be updated and merged.

@arteam did you want to update this for 0.9.0 or should we close this out?

I've rebased this PR against master.

I've got a few projects that are restricted to operating inside of an AWS VPC and need to communicate via a proxy, so I think this would be a great addition.

Thanks a lot for this contribution!


Changes for dataservice rework

changed to handle dataservice property changes.
includes dataservice deployment (uses the "test" button on the summary page).  Still needs work

I amended the commit to make the extension changes in fileImportControl.js.  I think we want to disable the other tabs for now, as discussed in team meeting yesterday.


🐛 Handle non-existing user IDs in read_user_by_id

Fix an issue where read_user_by_id would fail to return if the requested user ID did not exist.

Return 404 - Not Found when ID does not exist.
Request without sufficient permission will always result in 403 - Unauthorized.
Add tests to test requesting non-existing user IDs as superuser and normal user.

Hello. I'd like to know why this pull request has not been approved. It is valid.

Yeah pretty minor upgrade but it makes sens to merge IMO.


[CdapIO] Fix SparkReceiverIOIT

Resolves #24257

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Looks like we need to bump the timeout. Test still failing: https://ci-beam.apache.org/view/PerformanceTests/job/beam_PerformanceTests_SparkReceiver_IO/95/console
21:48:15     [Test ****] WARN org.apache.beam.runners.dataflow.DataflowPipelineJob - No terminal state was returned within allotted timeout. State value RUNNING
21:50:35
21:50:35 org.apache.beam.sdk.io.sparkreceiver.SparkReceiverIOIT > testSparkReceiverIOReadsInStreamingWithOffset FAILED
21:50:35     java.lang.AssertionError: expected:<5000000> but was:<3023859>
21:50:35         at org.junit.Assert.fail(Assert.java:89)
21:50:35         at org.junit.Assert.failNotEquals(Assert.java:835)
21:50:35         at org.junit.Assert.assertEquals(Assert.java:647)
21:50:35         at org.junit.Assert.assertEquals(Assert.java:633)
21:50:35         at org.apache.beam.sdk.io.sparkreceiver.SparkReceiverIOIT.testSparkReceiverIOReadsInStreamingWithOffset(SparkReceiverIOIT.java:337)
21:50:35
21:51:08 Gradle Test Executor 6 finished executing tests.

Looks like we need to bump the timeout. Test still failing: https://ci-beam.apache.org/view/PerformanceTests/job/beam_PerformanceTests_SparkReceiver_IO/95/console
21:48:15     [Test ****] WARN org.apache.beam.runners.dataflow.DataflowPipelineJob - No terminal state was returned within allotted timeout. State value RUNNING
21:50:35
21:50:35 org.apache.beam.sdk.io.sparkreceiver.SparkReceiverIOIT > testSparkReceiverIOReadsInStreamingWithOffset FAILED
21:50:35     java.lang.AssertionError: expected:<5000000> but was:<3023859>
21:50:35         at org.junit.Assert.fail(Assert.java:89)
21:50:35         at org.junit.Assert.failNotEquals(Assert.java:835)
21:50:35         at org.junit.Assert.assertEquals(Assert.java:647)
21:50:35         at org.junit.Assert.assertEquals(Assert.java:633)
21:50:35         at org.apache.beam.sdk.io.sparkreceiver.SparkReceiverIOIT.testSparkReceiverIOReadsInStreamingWithOffset(SparkReceiverIOIT.java:337)
21:50:35
21:51:08 Gradle Test Executor 6 finished executing tests.

Opened new PR: https://github.com/apache/beam/pull/24428


Timing updates/tracking

Q
A

Issue Type
Feature Request

Description
It would be nice to show some more information by default, eg. the estimated time for tasks (based on previous deployments), the task/total duration and perhaps and updating time how long it's busy. For some longer tasks (Magento2 compilation/assets etc) it can take multiple minutes (depending on server and configuration), so some feedback would be nice :)

It will be cool to have. I think for something like this for next major version.

I also would like to see this. Using npm install/build can take some time as well and it would be helpful to see a running time.

@barryvdh does https://github.com/deployphp/deployer/pull/1564 help you?

No, I meant the time it takes to process task on the server.

Closing this issue as inactive. If you still have this problem, please open a new issue.


Use the recorded nAGQ in the deviance method if not given

Previously if you fit a GLMM with nAGQ > 1 and later invoked the deviance method the value returned was that from the Laplace approximation (i.e. nAGQ = 1) not the actual deviance from the fit.  Add a method to provide for the correct result in these circumstances.

Codecov Report

Merging #190 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #190      +/-   ##
=========================================
+ Coverage    94.3%   94.3%   +<.01%
=========================================
  Files          18      18
  Lines        1141    1142       +1
=========================================
+ Hits         1076    1077       +1
  Misses         65      65

Impacted Files
Coverage Δ

src/generalizedlinearmixedmodel.jl
91.62% <100%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


Login to Hypothesis not intuitive

System and Software

Operating System:
Web Browser: Firefox

Page or Route

Link:

Description
When first showing the annotated article, annotations may not show if user isn't logged into Hypothesis
Notes
Any other comments you think may be important to reproducing the issue

@adam3smith @nniiicc @dkgeorgetown Would this inline notification message in combination with the Annotation sidebar (Hypothesis client) initially being expanded/shown be enough to inform users that in order to see their annotations they would need to log in to Hypothesis using the Annotation sidebar? The place of this inline notification message is immediately above the grey toolbar (which contains the "Show data sources" icon and "Delete manuscript" icon".

Yes - this is a good solution and we can use in UX tests