Terraform provider configuration improvement

Preflight Checklist

[X] I could not find a solution in the existing issues, docs, nor discussions
[X] I have joined the ZITADEL chat

Describe your problem
JWT session token has missed at Provider configuration.
Describe your ideal solution
At the moment Zitadel Terraform provider is used jwt_profile_file field for specifying full path to the private key of the service user.
How it works well described in documentation - https://zitadel.com/docs/guides/integrate/service-users/private-key-jwt
All works fine when we are staying within Zitadel TF provider scope.
If you want to do something outside of TF provider capabilities, you should use JWT token. For example, it may be some specific terracurl request to the Zitadel API.
Today I goes the following way:

Generate JWT token as described at the link above and save result to the environment variable such as TF_VAR_ZT_TOKEN.
Define TF_VAR_ZT_TOKEN inside of my TF files.
Run terraform apply which do all required things with Zitadel API using JWT token.

By my understanding, Zitadel TF provider already convert the jwt private key to the jwt token. Just only one thing is required - publish this value (session jwt token) via specfic Datasource. This datasource can be named as session-token or something like that.
Version
1.2.0
Additional Context
No response

May be a more straigthtrow idea is add the Terraform function key2jwt to Zitadel TF provider which will return a jwt token like as already implemented in Zitadel tools.

I'm not quite sure what the full idea is here, but just for my understanding, the issue is that the provider takes a MachineKey and once the 'terraform apply' is run, the provider takes this JWT and creates a token which is used once for the run?
And you want to provide the provider directly with a Token instead of the MachineKey and do this action outside of the provider?

Hi, Stefan!
You are right. My idea was - generate session token inside of TF provider (versus using an external jwt tools/code for this purpose) and use this session token for call Zitadel API directly (for change a human_user password, for example). If i right understand Zitadel position - team has no plans to add "operational" resources to the provider.
I have workaround as Python script for generate JWT token for API access. You can see source code here:
https://github.com/yandex-cloud-examples/yc-iam-federation-with-zitadel/blob/main/zitadel-config/ztgen.py
I think that will be great to hide this logic at provider inside - it is remove requirements with any external tools / language libraries.
WBR,
/Alexander


AMBARI-24868 JS error when changing service auto-start toggle

How was this patch tested?
21959 passing (31s)
48 pending

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4481/
Test PASSed.


dotnet restore --interactive broken

Issue moved from dotnet/sdk#16930

Please respond to @dustinmoris.

From @dustinmoris on Thursday, April 15, 2021 7:30:28 AM
How to test anger management of your developers:

Use Azure DevOps
Use NuGet feed from Azure DevPoops
Enable password expiry for users
Add NuGet package from private Azure DevPoops feed
User has to authenticate via dotnet restore --interactive
Works for a while until password expires
Now dotnet restore fails with a 401
User tries to run dotnet restore --interactive again but instead of initiating auth process keeps getting 401
User gets angry and punches a hole into the wall
User comes to GitHub and files issue with the hope that all of this bullshit will work for once in a while

Issue moved from dotnet/sdk#16930

Please respond to @dotnet-issue-labeler[bot].

From @dotnet-issue-labeler[bot] on Thursday, April 15, 2021 7:30:32 AM
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

We appear to be affected by this exact issue as well.  --interactive doesn't prompt at all.


bugfix for fprop_step definition in masked LSTM

@bartvm I think you'd be most qualified to review this?
On Mon, Jan 12, 2015, 7:31 AM mkraemer67<EMAIL_ADDRESS>wrote:

You can merge this Pull Request by running
git pull https://github.com/mkraemer67/pylearn2 patch-1
Or view, comment on, or merge it at:
https://github.com/lisa-lab/pylearn2/pull/1341
Commit Summary

bugfix for fprop_step definition in masked LSTM

File Changes

M pylearn2/sandbox/rnn/models/rnn.py
https://github.com/lisa-lab/pylearn2/pull/1341/files#diff-0 (2)

Patch Links:

https://github.com/lisa-lab/pylearn2/pull/1341.patch
https://github.com/lisa-lab/pylearn2/pull/1341.diff

—
Reply to this email directly or view it on GitHub
https://github.com/lisa-lab/pylearn2/pull/1341.

I can have a look, but the fprop_step is @jych's code which I'm less familiar with, so maybe @jych can have a quick look as well

Indeed looks like a bug to me, so happy to accept the bug fix. Will merge in a bit if I don't hear from @jych

I actually have a pull request to fix this issue.
This was what I've told three weeks ago.
https://github.com/lisa-lab/pylearn2/pull/1320

Oops, completely forgot about that PR over Christmas. Apologies! I've just merged it, which should make this PR redundant. Thanks either way @mkraemer67!

Cool. Thanks @mkraemer67 :)

Sure, glad to be of help. Thanks for the PR @jych.


Add javadoc

Codecov Report

Merging #229 (298a825) into master (822be20) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #229   +/-   ##
=========================================
  Coverage     71.21%   71.21%
  Complexity      794      794
=========================================
  Files           130      130
  Lines          2505     2505
  Branches        295      295
=========================================
  Hits           1784     1784
  Misses          578      578
  Partials        143      143

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/seedu/address/logic/LogicManager.java
68.18% <ø> (ø)
3.00 <0.00> (ø)

...eedu/address/logic/commands/AddStudentCommand.java
85.71% <ø> (ø)
8.00 <0.00> (ø)

...address/logic/commands/AttendanceBelowCommand.java
83.33% <ø> (ø)
6.00 <0.00> (ø)

...edu/address/logic/commands/ListStudentCommand.java
14.28% <ø> (ø)
1.00 <0.00> (ø)

...du/address/logic/commands/PreviousViewCommand.java
86.66% <ø> (ø)
5.00 <0.00> (ø)

.../address/logic/commands/ViewAttendanceCommand.java
100.00% <ø> (ø)
8.00 <0.00> (ø)

...edu/address/logic/commands/ViewStudentCommand.java
100.00% <ø> (ø)
8.00 <0.00> (ø)

...ess/logic/parser/AttendanceBelowCommandParser.java
100.00% <ø> (ø)
2.00 <0.00> (ø)

...dress/logic/parser/DeleteStudentCommandParser.java
100.00% <ø> (ø)
2.00 <0.00> (ø)

...s/logic/parser/EditParticipationCommandParser.java
53.84% <ø> (ø)
2.00 <0.00> (ø)

... and 9 more

Continue to review full report at Codecov.

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

LGTM!


Disable new rule C26814 as it's breaking builds

Wrote a follow up task #2941 to roll it out later.

Do you want to pile in the rest of the build break fixes to this PR too? Or do them one at a time?

Yeah probably just to make my life easier, I'll keep looking.

This merged with the build break fix.


External interface is not availiable in jupyter on Windows

Hi, I need to use external sound card on Windows, but I can't select it in jupyter notebook for some reason. When I'm running python -m sounddevice in the anaconda console on Windows 10 it gives me
   0 Microsoft Sound Mapper - Input, MME (2 in, 0 out)
>  1 MOTU S/PDIF 1-2 (MOTU Audio Wav, MME (2 in, 0 out)
   2 MOTU Reverb 1-2 (MOTU Audio Wav, MME (2 in, 0 out)
   3 MOTU Analog 1-2 (MOTU Audio Wav, MME (2 in, 0 out)
   4 Microphone (Realtek High Defini, MME (2 in, 0 out)
   5 MOTU Analog 7-8 (MOTU Audio Wav, MME (2 in, 0 out)
   6 MOTU Analog 3-4 (MOTU Audio Wav, MME (2 in, 0 out)
   7 MOTU Analog 5-6 (MOTU Audio Wav, MME (2 in, 0 out)
   8 MOTU Return 1-2 (MOTU Audio Wav, MME (2 in, 0 out)
   9 Microsoft Sound Mapper - Output, MME (0 in, 2 out)
< 10 MOTU Phones 1-2 (MOTU Audio Wav, MME (0 in, 2 out)
  11 P24h-10 (Intel(R) Display Audio, MME (0 in, 2 out)
  12 MOTU Analog 3-4 (MOTU Audio Wav, MME (0 in, 2 out)
  13 MOTU S/PDIF 1-2 (MOTU Audio Wav, MME (0 in, 2 out)
  14 MOTU Analog 5-6 (MOTU Audio Wav, MME (0 in, 2 out)
  15 MOTU Main Out 1-2 (MOTU Audio W, MME (0 in, 2 out)
  16 MOTU Analog 7-8 (MOTU Audio Wav, MME (0 in, 2 out)
  17 MOTU Analog 1-2 (MOTU Audio Wav, MME (0 in, 2 out)
  18 Speakers (Realtek High Definiti, MME (0 in, 2 out)
  19 MOTU Reverb 1-2 (MOTU Reverb 1-2), Windows WDM-KS (2 in, 0 out)
  20 MOTU S/PDIF 1-2 (MOTU S/PDIF 1-2), Windows WDM-KS (0 in, 2 out)
  21 MOTU Analog 3-4 (MOTU Analog 3-4), Windows WDM-KS (0 in, 2 out)
  22 MOTU Analog 5-6 (MOTU Analog 5-6), Windows WDM-KS (2 in, 0 out)
  23 MOTU S/PDIF 1-2 (MOTU S/PDIF 1-2), Windows WDM-KS (2 in, 0 out)
  24 MOTU Analog 7-8 (MOTU Analog 7-8), Windows WDM-KS (0 in, 2 out)
  25 MOTU Analog 7-8 (MOTU Analog 7-8), Windows WDM-KS (2 in, 0 out)
  26 MOTU Analog 1-2 (MOTU Analog 1-2), Windows WDM-KS (0 in, 2 out)
  27 MOTU Analog 1-2 (MOTU Analog 1-2), Windows WDM-KS (2 in, 0 out)
  28 MOTU Return 1-2 (MOTU Return 1-2), Windows WDM-KS (2 in, 0 out)
  29 MOTU Phones 1-2 (MOTU Phones 1-2), Windows WDM-KS (0 in, 2 out)
  30 MOTU Main Out 1-2 (MOTU Main Out 1-2), Windows WDM-KS (0 in, 2 out)
  31 MOTU Analog 5-6 (MOTU Analog 5-6), Windows WDM-KS (0 in, 2 out)
  32 MOTU Analog 3-4 (MOTU Analog 3-4), Windows WDM-KS (2 in, 0 out)
  33 Output (Intel(R) Display Audio Output 1), Windows WDM-KS (0 in, 2 out)
  34 Stereo Mix (Realtek HD Audio Stereo input), Windows WDM-KS (2 in, 0 out)
  35 Speakers (Realtek HD Audio output), Windows WDM-KS (0 in, 2 out)
  36 Microphone (Realtek HD Audio Mic input), Windows WDM-KS (2 in, 0 out)

But when sd.query_devices() in the jupyter notebook on the same machine gives me only
  0 Microsoft Sound Mapper - Input, MME (2 in, 0 out)
> 1 Microphone (Realtek High Defini, MME (2 in, 0 out)
  2 Microsoft Sound Mapper - Output, MME (0 in, 2 out)
< 3 Speakers (Realtek High Definiti, MME (0 in, 2 out)
  4 Stereo Mix (Realtek HD Audio Stereo input), Windows WDM-KS (2 in, 0 out)
  5 Speakers (Realtek HD Audio output), Windows WDM-KS (0 in, 2 out)
  6 Microphone (Realtek HD Audio Mic input), Windows WDM-KS (2 in, 0 out)

Running the same notebook in VS Code gives me full list of devices, so it looks like some Windows permissions issue and not the library, so I'll close it


Fix merchant model/texture

A bit of housekeeping with the buildscript update, but this fixes the model/texture not rendering.
The only implementation note is that the original 1.12 sources use a (not available in 1.7) layered rendering API to overlay a villagers head over... a full villager model. The only thing I can figure out is that this would allow items equipped in the merchant's helmet slot to render, but the merchant doesn't have a (usable) inventory...

Closing in favor of https://github.com/GTNewHorizons/FarmingForEngineers/pull/1.


Enhancing Code Quality

Describe the feature
In order to enhance the app's performance and ensure smoother functionality, we should focusing on improving its code quality.By doing so, we can reduce resource consumption, and ultimately deliver a more responsive and efficient user experience.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I am also interested in making changes on UI and improving the code quality.  I am a GSSOC's 24 Contributor I have an experience in flutter. @rohansnishad

Any Updates @harsh6045

Hello @andoriyaprashant Sir i have already completed the task of code cleaning. Sorry Sir for replying late due to some reason i haven't updated the code and creat an pull an Pull-request. Sir would you please re-assing me this issue so i can pull the pull request.

I would like to work on this issue , there are a lot of invalid use of classes where the key parameter is not passed as super and "const" is not added at many places reducing performance, there is also the issue of wrong declaration of statefulwidgets which happens due to using chatgpt and some of the packages like launch and canLaunch are depreciated and should be replaced with newer version.
I would like to improve the code wherever I can.
Please assign this to me @andoriyaprashant

Progress is good, there is a lot of work , btw I am also correcting the file names to the correct lower_case_with_underscores identifier.

hey @andoriyaprashant I am done with the issue, please review my PR.

@andoriyaprashant Please review my PR.


Update app.py: Trio example in docstring

It doesn't work without, as trio.run() says:
If you need to pass keyword arguments, then use functools.partial().

Maintainer merge checklist

[ ] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

ups

ups


503 Upstream connection failure issue with Envoy 1.13

We are observing intermittent Envoy 503 upstream connect failure issue when our client tries to stream data from Upstream server.
We are in the process of debugging this issue but so far we have 2 hypothesis:

Envoy race condition where it tries reusing connections that are closed while trying to communicate with upstream.
Client timeout is 5 seconds which is smaller than Envoy connection timeout (6 seconds). Client sends out RST after 5 seconds but Envoy never sees that (because of race condition) and still reuses same connection.

So far, I have not been able to reproduce this issue in a controlled environment. We also do not have debug logs to help us RCA this issue.
Any help / pointers from Envoy community to debug further ?

I would recommend looking at the access log output for the streams that are failing in this way: https://www.envoyproxy.io/docs/envoy/latest/configuration/observability/access_log/usage
To see the exact flags the access log reports.
You might be able to figure out from https://github.com/envoyproxy/envoy/blob/e2eaff90d31ead71af55f1535e819a6b382e1f68/source/common/router/router.cc#L1343C4-L1346 what error message is getting mapped to UpstreamConnectionFailure if that's the flag occuring in your case.
You might also want to look at your timers: https://www.envoyproxy.io/docs/envoy/latest/faq/configuration/timeouts.html in case the Envoy is timing out.
cc @zuercher

We had the same problem.Are there any new discoveries?

We enabled debug mode. After searching through the logs, we found this behavior to be more prominent for streaming workloads where a connection could be consumed for a long time (until the streaming completes). To tackle this, we had to do a couple of things:

Retry on 503.
Increase connection pool size in Envoy.

we also were using an older version of Envoy (1.13) which had a race condition bug. Although this did not seem to be the only root cause of the 503 error as other non-streaming workloads were running just fine.

来信已收到，祝好！


Improve git flow & git branching

http://scottchacon.com/2011/08/31/github-flow.html

As per discussion, @BogdanScherban will start to
use master branch as baseline,
use feature branches per issue
produce version branches, related to release versions
update repo with releases, linked to features & related commits
One PR per feature, try to keep code independent
WIll merge PRs soon as we can
Be prepared to revert to earlier commit if problems/mistakes made

Hi, @tony-shannon, @fzaninotto, @BogdanScherban, I have read the comments and looked through the suggested article. We have been using gitflow for PulseTile development for quite a while and it turned out to be working pretty well. The simplified approach that is described by Scott Chacon also does make sense, however it is more appropriate, in my opinion, for constant deployment (at least couple of times a day) and not one-two week releases that we are more used to. Moreover, Scott himself states that longer term releases (weekly or monthly wise) work better within the git-flow. Quoting him:
"For teams that have to do formal releases on a longer term interval (a few weeks to a few months between releases), and be able to do hot-fixes and maintenance branches and other things that arise from shipping so infrequently, git-flow makes sense and I would highly advocate it’s use."
I tend to think that initially chosen approach (git-flow) still looks more preferable for our context. What are your thoughts?

I think that having formal releases on a long term interval isn't really suited to PulseTile as it is now (if I understand correctly, in early stage, and not used by many real customers). So the extra protocol implied by gitflow is just time lost at this stage.
If you're more comfortable with git flow and you feel that you can develop faster with it, then I'm fine with your using it - as long as developers push their work on origin and not on master, and as long as they don't branch new features off other feature branches.

@kbeloborodko Are you ok with this?

@PhilBarrett sure, I believe we both agree with @fzaninotto that we can utilize the approach we are already following and switch to branching from master if we feel that we need to

yep, as long as you don't use forks ;)


Summarize Timestamp Type Spark

Branch : spark-branch
Spark TimestampType is currently not being summarized in spark
Feature:
TimestampType is currently not being summarized in spark. We want to generate summaries for them properly, including generating simple statistics (similar to pandas)

in progress : @laurabarrigon


FA icons appear unstyled for a short duration

The font awesome icons lack styling for a short duration, before the CSS gets applied during runtime.

Expected behaviour
The CSS rules should be present in the server-rendered content.
Proposed solution

Stop FA from applying the styles during runtime

// styles.js

fontawesome.config.autoAddCss = false;

Include them globally beforehand in layouts/index.js

// layouts/index.js

import Header from '../components/header';
import Footer from '../components/footer';

import 'prismjs/themes/prism-tomorrow.css';
// adding
import<EMAIL_ADDRESS>
Adding the styles in gatsby-browser.js should suffice but doesn't seem to be working in Gatsby v1.
Example

Elsewhere on GH
https://github.com/FortAwesome/react-fontawesome/issues/134
Specifications
whatsmybrowser.org/b/Q7HKKH7

Would love to submit a PR if needed.

Good spot! Thanks man :smile:
Now fixed, and I've helpfully moved to Gatsby v2 along the way as well, since I need it elsewhere. Thanks for pointing this out :+1:


add bundlesdf

Just found this very cool work at CVPR. Hope this helps.

Thanks a lot !

@ZhongqunZHANG feel free to let me know if there is any issue merging.


Update transmission generation code

Evalute tranmssion at energies not centers
Add energies to the returned tables
Update tests for theses changes
Add function to write transmission tables to csv

Codecov Report

Merging #79 (eab60a3) into master (9cf2732) will decrease coverage by 0.42%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##           master      #79      +/-   ##
==========================================
- Coverage   73.97%   73.54%   -0.43%
==========================================
  Files          43       43
  Lines        2628     2646      +18
==========================================
+ Hits         1944     1946       +2
- Misses        684      700      +16

Impacted Files
Coverage Δ

stixcore/calibration/transmission.py
65.26% <20.00%> (-12.66%)
:arrow_down:

Continue to review full report at Codecov.

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


Adjusting visibility of publicly exposed records to all pub properties

We need all of the types which are publicly exposed to have completely pub properties (not pub(crate)) because they will eventually be serialized over the wire and generally serializers will need to read the properties of these structs.

So this is kind of the reverse problem of the issue, but it's adjusting the visibility of various records to be public so they're externally serializable


Use aka.ms links

:warning: Validation status: warnings

File
Status
Preview URL
Details

sandbox/gamedev/monogame/azure-storage-monogame.md
:warning:Warning
View
Details

sandbox/gamedev/unity/azure-storage-unity.md
:warning:Warning
View
Details

sandbox/gamedev/monogame/azure-storage-monogame.md

Line 69: [Warning] Invalid file link:(~/gamedev/monogame/media/unity-select-dlls.png).
Line 73: [Warning] Invalid file link:(~/gamedev/monogame/media/unity-wsaplayer-include.png).
Line 77: [Warning] Invalid file link:(~/gamedev/monogame/media/unity-wsaplayer-exclude.png).
Line 97: [Warning] Invalid file link:(~/gamedev/monogame/media/unity-player-config.png).

sandbox/gamedev/unity/azure-storage-unity.md

Line 52: [Warning] Invalid file link:(~/gamedev/unity/media/monogame-storage-nuget.png).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Tracker version update

I am trying to build ilia on nixos, and found that the build dependency libtracker-sparql-2.0-dev is outdated, and has a new version 3.0, changed 4 years ago.
The nix package for tracker uses the new version. Got this info from this discussion
This might help in migration: https://gnome.pages.gitlab.gnome.org/tracker/migrating-2to3.html

Never mind, just found that the ubuntu/jammy branch uses the newer version :P

FYI, this directory represents a "build pipeline" for Regolith packages.  ordering: experimental -> unstable -> testing -> release_X
You can see what branches or refs are used to build a given stage of regolith, for example Debian Bookwork is building on the ubuntu/jammy branch in the unstable stage: https://github.com/regolith-linux/voulage/blob/main/stage/unstable/debian/bookworm/package-model.json#L5

Thanks, that helps 😃


[#302] Add support for authorization using client_credentials

A

Looks good. 👍
Could you put a note about this inside the CHANGELOG.md? That would be great.

Sure. I will update it right away.

Cool, thanks!


Add Giorgos Topalidis to Contributors list

gt stands for giorgos topalidis :octocat:

Hi  @TopFlankerKiller, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


new thing

what i learned

Runs the image but doesn’t build the docker file - will keep running it (ex in class we used ut to build an http file)


Update some error checks to use new log file structure

https://github.com/microsoft/InnerEye-Inference/blob/a90fb10b3d38818bcc88bf95f9a52e1be3fde229/app.py#L173
this downloads and checks 70_driver_log.txt. This is generated by the old AML runtime. The new runtime writes a file user_logs/std_log.txt that should be checked instead.

We have this problem also in hi-ml test runs. Provide a shared function that we can use across all repositories?


org rename in 2.2 branch

The related issue: https://github.com/stolostron/backlog/issues/18746
Signed-off-by: Yang Le<EMAIL_ADDRESS>

/lgtm

/approve


Download Schemas for 3.8.x

Automated changes by create-pull-request GitHub action

Closing since I ran this against kong-gateway instead of kong-gateway-dev


Bug: Can't capture the image node and data when dragging an image into the editor

function onDrop(event: DragEvent, editor: LexicalEditor): boolean {
  const node = getImageNodeInSelection();
 console.log("[onDrop] Image node : ", node); // node was not captured.
  if (!node) {
    return false;
  }
  const data = getDragImageData(event);
  console.log("onDrop data: ", data); // data was not captured.

  if (!data) {
    return false;
  }
  event.preventDefault();
  if (canDropImage(event)) {
    console.log("canDropImage event: ", event);
    const range = getDragSelection(event);
    node.remove();

    const rangeSelection = $createRangeSelection();
    if (range !== null && range !== undefined) {
      rangeSelection.applyDOMRange(range);
    }

    $setSelection(rangeSelection);
    editor.dispatchCommand(INSERT_IMAGE_COMMAND, data);
  }
  return true;
}

Lexical version: 0.12.4
The current behavior
Can't capture the dragged image node and data
The expected behavior
Image node and data get captured.

If you copy/pasted from the Playground, you might be missing the DragDropPastePlugin https://github.com/facebook/lexical/blob/main/packages/lexical-playground/src/plugins/DragDropPastePlugin/index.ts

I encountered a similar issue when a user was drag-dropping an image from a local server or an image from a website.
The image will display in the editor but is intercepted in CONTROLLED_TEXT_INSERTION_COMMAND as its type is  text/uri-list in event.dataTransfer. The image will be inserted in the editor with its original url and bypass the Drag And Drop Paste Plugin.
I added a method in DragDropPaste to catch these image URLs.
  useEffect(() => {
    return editor.registerCommand(
      CONTROLLED_TEXT_INSERTION_COMMAND,
      (event) => {
        if (typeof event === 'string') {
          return false;
        }

        const url = event.dataTransfer?.getData('text/uri-list');
        if (!url) {
          return false;
        }
         // Custom application logic here
         // handleSaveImageInDBAndInsert(url);
        return true;
      },
      COMMAND_PRIORITY_LOW,
    );
  }, [editor, handleSaveImageInDB]);


Hot update failed, I don't know if it's a problem with vite or vue

Vue version
3.2.45
Link to minimal reproduction
https://github.com/ketao4869/vite-bug
Steps to reproduce

pnpm install pnpm run dev

Go to src\Components\SearchForm\index.vue and modify this file

The browser finds that there is no hot update, but the new page in the network request has been requested

Go to src\Components\input\input.vue, modify this file,normal hot update

What is expected?
The Search subcomponent can be hot updated
What is actually happening?
no hot update
You can create a new TEST folder in the src directory, and it can be hot updated after registration.
Delete the SearchForm component under src\Components
Move the TEST folder to src\Components and rename it SearchForm
Now the SearchForm subcomponent can be hot updated. Everything is fine with the project
If you comment out the method of registering components on line 10 of src\index.ts, you should report an error that the component is not registered, but there is no error: Failed to resolve component: SearchForm

System Info
System:
    OS: Windows 10 10.0.22623
    CPU: (16) x64 AMD Ryzen 7 5800H with Radeon Graphics
    Memory: 5.12 GB / 13.86 GB
  Binaries:
    Node: 16.17.0 - C:\Program Files\nodejs\node.EXE
    npm: 8.15.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22621.1255.0), Chromium (110.0.1587.63)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    vue: ^3.2.45 => 3.2.47

Any additional comments?
No response

Is this the problem? second line：/node_modules/.vite/deps/vue.js?v=580ba9ff
import { createHotContext as __vite__createHotContext } from "/@vite/client";import.meta.hot = __vite__createHotContext("/src/components/SearchForm/index.vue");/* unplugin-vue-components disabled */const _sfc_main = { name: "SearchForm1" };
import { createElementVNode as _createElementVNode, createTextVNode as _createTextVNode, Fragment as _Fragment, openBlock as _openBlock, createElementBlock as _createElementBlock } from "/node_modules/.vite/deps/vue.js?v=580ba9ff";
const _hoisted_1 = /* @__PURE__ */ _createElementVNode(
  "div",
  null,
  "SearchForm",
  -1
  /* HOISTED */
);
function _sfc_render(_ctx, _cache, $props, $setup, $data, $options) {
  return _openBlock(), _createElementBlock(
    _Fragment,
    null,
    [
      _hoisted_1,
      _createTextVNode(" 123 ")
    ],
    64
    /* STABLE_FRAGMENT */
  );
}
_sfc_main.__hmrId = "1fe7aed7";
typeof __VUE_HMR_RUNTIME__ !== "undefined" && __VUE_HMR_RUNTIME__.createRecord(_sfc_main.__hmrId, _sfc_main);
import.meta.hot.accept((mod) => {
  if (!mod)
    return;
  const { default: updated, _rerender_only } = mod;
  if (_rerender_only) {
    __VUE_HMR_RUNTIME__.rerender(updated.__hmrId, updated.render);
  } else {
    __VUE_HMR_RUNTIME__.reload(updated.__hmrId, updated);
  }
});
import _export_sfc from "/@id/__x00__plugin-vue:export-helper";
export default /* @__PURE__ */ _export_sfc(_sfc_main, [["render", _sfc_render], ["__file", "D:/Code/NPM/CrudEl/CrudEl/src/components/SearchForm/index.vue"]]);

//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJtYXBwaW5ncyI6InNDQUNBOzs7QUFLSTtBQUFxQjtBQUFBO0FBQWhCO0FBQVU7QUFBQTs7Ozs7OztBQUFmO0FBQXFCIiwibmFtZXMiOltdLCJzb3VyY2VzIjpbIkQ6L0NvZGUvTlBNL0NydWRFbC9DcnVkRWwvc3JjL2NvbXBvbmVudHMvU2VhcmNoRm9ybS9pbmRleC52dWUiXSwiZmlsZSI6IkQ6L0NvZGUvTlBNL0NydWRFbC9DcnVkRWwvc3JjL2NvbXBvbmVudHMvU2VhcmNoRm9ybS9pbmRleC52dWUiLCJzb3VyY2VzQ29udGVudCI6WyI8c2NyaXB0IGxhbmc9XCJ0c1wiPlxyXG5leHBvcnQgZGVmYXVsdCB7IG5hbWU6ICdTZWFyY2hGb3JtMScgfVxyXG48L3NjcmlwdD5cclxuPHNjcmlwdCBzZXR1cCBsYW5nPVwidHNcIj48L3NjcmlwdD5cclxuXHJcbjx0ZW1wbGF0ZT5cclxuICAgIDxkaXY+U2VhcmNoRm9ybTwvZGl2PlxyXG4gICAgMTIzXHJcbjwvdGVtcGxhdGU+XHJcblxyXG48c3R5bGUgc2NvcGVkIGxhbmc9XCJzY3NzXCI+PC9zdHlsZT5cclxuIl19

That's likely because you use unplugin-vue-components, which will add import statements for any component it finds in src/components.
you end up with two paths to the component:

The direct import addded by the plugin
The indirect import from the barrel entry in /components/index, which you use for the component registration

It does not affect the input because you use a name for the component that is different from the filename, so unplugin-vue-components does not add a direct import for it.


Update package.json

as suggested ... but not tested, so pls. run a clean install/build/test.
thanx

All done, thanks!


Move docs for gh-pages branch into master

Github now supports docs from a specific directory in master, no need for the gh-pages.
It will make doc versioning easier.

@jeeyi Moving the docs in master. When this is landed, could you reconfigure the project to use the directory on master (Settings > GitHub Pages > Source : master branch/docs folder)? Thanks!


Implement ICU based platform agnostic interface on Linux builds

This change implements the UnicodeText Platform Agnostic interface on Linux builds.
With this change, Linux builds will temporarily have an ICU dependency
(we'll do another change later to make it optional but it shouldn't be too difficult).
Additionally, one new API is introduced to the UnicodeText interface- ChangeStringCaseInPlace.
This is a thin wrapper around CharUpperBuffer/CharLowerBuff on Win32.
On Linux it just falls back to the ICU based function ChangeStringLinguisticCase.
Note that while the ICU functions are generally analogous to their Windows counterparts, they
need not be identical in behavior. This can cause differences between Windows and Linux but we'll
deal with them as we encounter them.
Finally, one additional function was re-implemented for Linux. LogicalStringCompareImpl is a
reimplementation of CompareStringW(NORM_IGNORECASE | SORT_DIGITSASNUMBERS)

@jianchun Please review

Thanks for the review @Jianchun. Updated PR.

Updated the PR to rebase on top of the latest branch and squashed out the code review comment commits- merging now.


Can't get open circle %

Hi, I'm using vscode and here is my profiles.ini:
# name of profile (one or more sections, first should be name default)
[default]
# glyph=altglyph
percent=percent.alt
greater_equal.liga=greater_equal.2.liga
less_equal.liga=less_equal.2.liga

But I can't get open circle %. The % is still the close circle version.

Hm.. That's what I have. What about the >= and <= ... do they have the horizontal bar (alternate) or the slanted bar (default)? If you're not seeing those, then it looks like the profiles.ini file is not being processed. Make sure you put it in the ./original folder.

Hm.. That's what I have. What about the >= and <= ... do they have the horizontal bar (alternate) or the slanted bar (default)? If you're not seeing those, then it looks like the profiles.ini file is not being processed. Make sure you put it in the ./original folder.

I got the horizontal bar >= and <=. Only the % is not working.

The fonts I'm using are OperatorMono-Book and OperatorMono-BookItalic

Ok, I see. I just re-ran it on my end and I no longer have the percent.alt so apparently something must have broken since I last ran it. :(
Anyway, I'll take a look and let you know when it's fixed. I like the open-circle percent also.

Hmm.. regenerated my fonts to test the italics fix and now I see the percent.alt glyph.
Anyway, once I've merged in the italics fix, I'll try again to see if this is still working.

Let me know if this is still happening. We can reopen if necessary.


Exclude Doctrine Entities from the FinalRule

I would like to use the FinalRule and I've tried to write my own implementation before I stumbled over your package. Do you have any solution for the problem that doctrine entities must not be final because of their proxy classes? I understand that I can whitelist them one by one in your implementation, but that does not feel too clean as I have to adjust the config file of a CI tool every time I add something in my domain layer. I also don't put them into one huge Entity directory, so the location is no indication neither. Do you have any suggestions?

@dmecke
You might like #84!

Indeed! Thanks a lot!


Create feedback for research participants

What
We need to present findings and future plans to people with additional language needs who have participated in our research
Why
To show how participant's insights have contributed to our understand and work going forward in order to maintain good relationships with  communities and thank participants for their input
User needs

As a participant I want to see what findings the discovery has produced so that I know my community's needs are represented in the right way
As a participant I want to find out about future work so that I can see what impact my insight on this project has had

Must

[ ] get a BSL interpretation of our feedback
[ ] simplify our findings in to something that avoids jargon
[ ] have visual representations of our findings

@CatherineReader @amyj2110, has this been done yet?


replace Utils.getInterfaces with instanceof

Please see issue #97
Many thanks!

Coverage decreased (-1.1%) to 89.831% when pulling c5de76edd0a08043aecdbe46df1388ff31b51d80 on wg1j:master into eabb56042f2b90814c24a37a852accf27c385669 on j-easy:master.

Coverage decreased (-0.1%) to 90.857% when pulling d3c47a3a7a2f1c96147e3aa7a7666a03012946db on wg1j:master into eabb56042f2b90814c24a37a852accf27c385669 on j-easy:master.

Hello,
Great! less is more 😉
Thank you for the PR.
Kr
Mahmoud

My pleasure. 😀


corrected dependency coordinates in readme.md

Oh, yes, that's correct. Somehow edited the wrong line.


Fail all replicas if the node is down or the engine is dead during the restore

longhorn/longhorn#1270
longhorn/longhorn#1328
longhorn/longhorn#1336

I am not convinced this is the right fix.

Kubernetes node status are delayed.
Even if the auto-salvage or auto-attach is triggered, we shouldn't able to present a usable volume to the user, which is the case in https://github.com/longhorn/longhorn/issues/1336 . I don't know why it happened.


Add Migrate ABS 2 SPE to TOC

Category

[ ] Content fix
[ ] New article
[x] Add page to TOC

Related issues

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

No related issues

What's in this Pull Request?

Add new tutorial to Table of Content

Submission guidelines

In the future, please always complete the form & instructions in the opening template message for a new PR. This helps us identify what the PR is doing. Remove unnecessary things, like shown in #9867


Incorrect Behavior in MultiValuedValue.get_type()

>>> iv   # our variable
MultiValuedValue(vals=(SequenceIncompleteValue(typ=<class 'list'>, args=(UnresolvedValue(),), members=(UnresolvedValue(),)), KnownValue(val=[])))
>>> iv.get_type()
None
>>> iv.get_type_value()
MultiValuedValue(vals=(KnownValue(val=<class 'list'>), KnownValue(val=<class 'list'>)))
>>> iv.get_type_value().get_type()
None

I would expect get_type() to return list, since that is clearly the type of this node.

get_type() is basically deprecated, since it can't be usefully implemented for a lot of Value subclasses. I haven't removed it since there are still some internal use cases I need to replace.
What are you trying to use it for? Is there some other API we could provide that would cover what you need?

My needs are pretty straightforward. I need to know the type of expressions at various points in the code. For example, when there's a * b, or if a, etc., I want to know what's the type of a and b, and so on.
For now here's how my get_type function looks like:
def _get_inferred_type(v):
    if isinstance(v, pyanalyze.value.MultiValuedValue):
        types = {x.val for x in v.get_type_value() .vals if isinstance(x, pyanalyze.value.KnownValue)}
        if len(types) == 1:
            t ,= types
        elif len(types) > 1:
            without_none = types - {type(None)}
            if len(without_none) == 1:
                t ,= without_none
                return Optional[t]
            else:
                return make_union(types)
        else:
            return None
        return t

    return v.get_type()

If there was an interface function that took care of these details (or even some of them), with something like get_type(), it would make it much easier on me.

So basically you want to go back to the typing object corresponding to the Value object?
What do you need the types for? If it's just to display them, str() on the Value object should give a reasonable representation.

No, I need an actual object that I can test.
For example, for "a * b", I need different behavior for (list, int) than
(int, int). Same with +, it's different between (list, list) and (int,
int), etc.
Ideally the types would be either basic python types, or instances from the
"typing" library, since I already have code that can do issubclass on them,
like "issubclass(int, union [int, str]) returns true.

What do you need the types for? If it's just to display them, str() on
the Value object should give a reasonable representation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/quora/pyanalyze/issues/208#issuecomment-885758671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSSSFH3ZMY6R4VR7YPHVLTZGKTFANCNFSM5A3OOKLQ
.

Thanks for explaining!
I don't know your codebase, but I feel like the cleanest solution might be to rely on working with pyanalyze's Value objects directly, using methods like is_value_compatible() as the equivalent of isinstance. That's what pyanalyze's own type checking does.
Failing that, what you're really asking for is an API that given a Value object, returns the runtime typing.py equivalent. That's not really the kind of thing I'd like to maintain; there's all the code in annotations.py that does the reverse translation from runtime to Value object, and I'm not sure I'd want to have to deal with runtime typing across different Python versions any more. However, it shouldn't be hard to write something like this yourself by just casing on the different Value subclasses. They're documented at https://pyanalyze.readthedocs.io/en/latest/reference/value.html, the only part of pyanalyze that's fairly thoroughly documented so far :)

Thanks for the explanation. Is is_value_compatible() a theoretical function that I should write, or an actual one? Because I couldn't find it.
I'm okay with a different "comparison scheme" than what I described. What I really wanted to avoid is to have to write a huge function with a long list of isinstance calls, and a different behavior for each Value subclass and attribute configuration. Which I'll probably have to discover with trial-and-error as I go. It's not just a lot of work, which every user would have to replicate on their own, but I don't know if I could ever be satisfied that I covered every edge case.

Sorry, I meant can_assign (https://pyanalyze.readthedocs.io/en/latest/reference/value.html#pyanalyze.value.Value.can_assign). is_assignable is the easier-to-use version if you don't care about generics; can_assign returns a dictionary of TypeVar values.
I see where you're coming from with wanting to avoid lots of isinstance calls, so I'm happy to add more methods to Value in general; I just don't think "runtime type" is going to be a good concept to expose. There are already some useful helper functions like https://pyanalyze.readthedocs.io/en/latest/reference/value.html#pyanalyze.value.unite_values

Closing as we're instead moving to get rid of get_type().


ENH: STRtree optimize unprepared predicates

Several of the predicates provided by GEOS do not actually use prepared logic internally, e.g., touches.
From a bit of testing so far, it looks like running a pairwise touches on the results of query_bulk against the tree with predicate='intersets' is faster than using predicate='touches'.
We can pre-screen several of the predicates and first run a prepared intersects and then the intended predicate afterward.  For other predicates that have an existing prepared implementation in GEOS, we can use those directly (e.g., contains_properly).
Not sure if we'd want to do something similar for the prepared predicates in the ufuncs.  They would get the same benefit.
Long term, it would be best for the improvements to be made upstream in GEOS.  Will investigate raising there once we have some better test cases to demonstrate the issue.

I did some testing with the tree benchmark in #270, and added a screening check in C to test against the prepared intersects predicate before testing the other predicates.  Any benefits were marginal at best, even when there are many that overlap the bounding box but fail to intersect; removing these before testing the more complex predicates seems like it would have helped - but it didn't appear to, or was outweighed by the additional overhead of testing intersects.
Unfortunately, that means there isn't room to optimize the predicates further here; any real improvement needs to come from GEOS.


Support returning a CSV document instead of an HTML page

Add a new optional format query parameter to the API endpoint. Valid formats are html (the default) or csv.
Fixes #3

Thanks @julienrf!
@fee-mendes can you please review?

Thanks @julienrf! @fee-mendes can you please review?

No particular complaints other than few nitpicks with the particular purpose of this PR. Note I do have other reservations about it - this is likely something you'll want to discuss in-depth via other channels @guy9


Jenkins Collection updates

set CI integration test newman run iterations to 2
updated {{host}} param in collection to include the protocol http/https in itself as CI and CD work on different ones. Now can be set through the postman env

retest this please


Inconsistency between average validation loss presented in console and log

Describe the bug
The console message "Epoch Average Validation Loss" gives different result in the console and logs.
To Reproduce
Train a network for a single epoch with multiple batches
Expected behavior
There should be consistency
Screenshots
N.A.
GANDLF Version
0.0.9-dev
Desktop (please complete the following information):
N.A.
Additional context
Originally reported by Vinayak

Fixed.


Unexpected argument --cache-key with mise

Describe the bug
When launching a terminal that has oh-my-zsh and the mise plugin installed, I am getting the following error on MacOS:
error: unexpected argument '--cache-key' found

  tip: to pass '--cache-key' as a value, use '-- --cache-key'

Usage: usage generate completion <SHELL|BIN|--usage-cmd <USAGE_CMD>>

For more information, try '--help'.

To Reproduce
Have mise as a plugin installed. The issue started to appear with version 2024.11.2.
mise doctor output
version: 2024.11.2 macos-arm64 (2024-11-06)
activated: yes
shims_on_path: no

build_info:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS
  Built: Wed, 6 Nov 2024 00:49:45 +0000
  Rust Version: rustc 1.82.0 (f6e511eec 2024-10-15) (Homebrew)
  Profile: release

shell:
  /opt/homebrew/bin/zsh
  zsh 5.9 (arm-apple-darwin22.1.0)

dirs:
  data: ~/.local/share/mise
  config: ~/.config/mise
  cache: ~/Library/Caches/mise
  state: ~/.local/state/mise
  shims: ~/.local/share/mise/shims

config_files:
  ~/.config/mise/config.toml

backends:
  cargo
  core
  go
  npm
  pipx
  spm
  ubi
  vfox

plugins:
  flutter  https://github.com/oae/asdf-flutter.git#d7ae5ad
  poetry   https://github.com/mise-plugins/mise-poetry.git#431c335
  usage    https://github.com/jdx/mise-usage.git#fe3888a
  yarn     https://github.com/mise-plugins/asdf-yarn.git#74ea3b9

toolset:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
env_vars:
  MISE_SHELL=zsh

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf_compat = false
  cache_prune_age = "30d"
  ci = false
  color = true
  debug = false
  disable_backends = []
  disable_default_registry = false
  disable_hints = []
  disable_tools = []
  env_file = ".env"
  experimental = true
  fetch_remote_versions_cache = "1h"
  fetch_remote_versions_timeout = "10s"
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  http_timeout = "30s"
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  libgit2 = true
  lockfile = false
  log_level = "info"
  not_found_auto_install = true
  paranoid = false
  pin = false
  plugin_autoupdate_last_check_duration = "1 week"
  quiet = false
  raw = false
  task_output = "prefix"
  trace = false
  trusted_config_paths = []
  use_versions_host = true
  verbose = false
  yes = false

  [cargo]
  binstall = true

  [node]

  [pipx]
  uvx = false

  [python]
  default_packages_file = "~/.default-python-packages"
  pyenv_repo = "https://github.com/pyenv/pyenv.git"
  venv_auto_create = false
  venv_stdlib = false

  [ruby]
  default_packages_file = "~/.default-gems"
  ruby_build_repo = "https://github.com/rbenv/ruby-build.git"
  ruby_install = false
  ruby_install_repo = "https://github.com/postmodern/ruby-install.git"

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false
No warnings found
No problems found

Additional context
Seems to be related to https://github.com/jdx/mise/pull/2937.

you need to update usage

you need to update usage

I can confirm that updating usage the error was gone.
Thanks!

you need to update usage

Thank you. After mise upgrade (which updated usage) the error is gone.


Fixes #442: config_hook overwritten by config_updates

The updates contained in the return dictionary of a config_hook were overwritten if a corresponding
entry existed in config_updates as passed e.g. from the command line. This commit removes the line
that is responsible for this behavior, as well as adding a test case and a slightly modified documentation.

Coverage decreased (-0.004%) to 83.974% when pulling de8c904afd4b1a6edf27a3086032333f4405f686 on baldassarreFe:gh-issue-442 into 6fb05b1ee1b7706b44c3ebd852e1e234841ea2ce on IDSIA:master.

unstale

Hey @baldassarreFe! Thanks for reporting this and even providing a fix!
The overriding you describe in #442 was indeed the intended behavior. But to be honest I am not sure if it should be that way.
The downside I see with allowing config_hook to overwrite commandline updates is that it can become very confusing. I mostly use commandline updates when I want to make sure that a certain entry definitely has the given value. If there is a hook in place that modifies entries this would violate my expectations. And this might well happen by accident, because the configuration process is unfortunately complex.
On the other hand you might argue that you should only ever use config hooks in very special situations and when you know what you are doing. It is also a fair complaint that there is currently no way of overriding commandline updates.
A compromise could be to allow optionally overriding commandline updates in the config_hook.
I.e. if you return config, True from the hook, then it will overwrite even commandline updates, otherwise it sticks to the old behavior. This would also maintain compatibility. But the implementation of this might be
a bit tricky.
What are your thoughts on this?

Hi @Qwlouse thanks for the detailed answer!
I understand your point on the possible problems arising from command line updates and config hooks, there are pros and cons on both sides. On the one hand should be able to say "just run this" (command line update), on the other hand sometimes you want to specify a certain parameter and automatically derive other config parameters from it (e.g. I give the root folder as a parameter and a hook takes care of finding the data and log subfolders).
Overall, I have the feeling that the config mechanism is too restrictive and it is very hard to use in a workflow such as:

Define config (text files/command line params/native datatypes)
Preprocess config (solve paths/expand derived parameters)
Save config (in plain text)
Prepare the run (transform the parameters in actual objects/functions) and build a "running config" object that is a totally separate object from the original config
Run the run, having access to the "running config"

Basically, Sacred needs to distinguish a configuration that can be saved and written to disk from a configuration to use during the experiment. However, from what I understand, this would bring Sacred very far from how it currently is.
In the beginning I was very excited with Sacred and I invested a good month adapting my code to run with this library. Later on, I realized that I was spending more time in understanding how to make things work rather than focusing on my research and decided to abandon it. I rewrote most of the functionalities that I liked (config files, command line updates, environment snapshots, etc), left out those I would not use (e.g. there are logging libraries that are more specific to my needs) and renounced to dependency-injection because it was too heavy to live with. Now I'm pretty happy with my small library that does what I need and that does not require a lot of digging in the code to understand it.


Allow more time for http get

When specifying a URL for image src, we get a timeout already after 1 second. Some image providers (like security cameras) are taking more than 1 second. This simple change solves that.
(sorry for the 2 extra commits, mixed up branches)

Thanks. Please test the new build thoroughly and update docs where needed.


userTracker.resolve(event);

what is the userTracker ? it is undefined for me. Also do i need a v2 key  ? where would i use the secret key  ? does it work with enterprise key too ?

Hello, it was included in the readme as an example. It's not required to have a userTracker in your success event.

Also do i need a v2 key ? where would i use the secret key ? does it work with enterprise key too ?
For recaptcha, you need two keys, one for your front end and one for your backend. Please read the google documentation for implementation details.


about upgrade verdaccio

My reason:
Thank you. I want to ask questions. How to upgrade? Run commands npm update verdaccio ?
Steps to reproduce:
App Version:
3.8.5
Config file:
Additional information:
Additional verbose log:

Just install the package with the desired version (I suggest latest) .

so: 😃

pm2 stop verdaccio
npm update verdaccio  -g
pm2 restart verdaccio

Will it affect private packages stored in history?

No, package storage should not be affected. But, since my knowledge about your environment is limited I always suggest do a backup

👌 ok, thank you !

I wrote this recently, a migration guide
https://verdaccio.org/blog/2019/02/24/migrating-verdaccio


Making numpy object arrays from ragged sequences is deprecated

Describe the bug
Nothing is broken, currently, but this deprecation warning pops up at multiple places in the toolbox, and may cause errors in later versions of numpy:
/workspace/art/estimators/speech_recognition/pytorch_deep_speech.py:279: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray
  x_ = np.array([x_i for x_i in x] + [np.array([0.1]), np.array([0.1, 0.2])])[:-2]

Recommended Fix
A simple two-line change would fix this:
x_ = np.empty(len(x), dtype=object)
x_[:] = x

Other potential solutions considered
With the intention of producing a 1D object array, I can think of no single line way of doing this, as
x_ = np.array(x, dtype=object)

will not always produce a 1D object as can be seen here:
>>> x = [np.arange(2), np.arange(2)]
>>> x_ = np.array(x, dtype=object)
>>> x_
array([[0, 1],
       [0, 1]], dtype=object)
>>> x_.shape
(2, 2)

Hi @davidslater Thank you very much for proposing this update. We were also looking at this deprecation issue and will include a solution in the upcoming release.

@davidslater I tested the proposed solution above on branch development_issue_694 (https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/3ab399e77972b4d9517dfe22ac3c5f7b4c777fd3/art/estimators/speech_recognition/pytorch_deep_speech.py#L279) with our notebook asr_deepspeech_examples.ipynb and it seems to fail for prediction of a single array in cell 11:
pred1 = speech_recognizer.predict(np.array([x1]), transcription_output=True)

with error:
    278
    279         x_ = np.empty(len(x), dtype=object)
--> 280         x_[:] = x
    281
    282         # Put the model in the eval mode

ValueError: could not broadcast input array from shape (166960) into shape (1)

Have you seen something simila beforer?

Can you print out x, x.dtype, and x.shape?

Sorry, I think that line should be:
x_[:] = x[:]

No, I'm still getting errors for that as well, hold on.

x1:
array([0.0003357 , 0.00030519, 0.00036622, ..., 0.00210578, 0.00210578,
       0.00158696], dtype=float32)

x1.dtype:
dtype('float32')

x1.shape:
(166960,)

I think that this will work:
x_[:] = list(x)

Alternatively:
x_[:] = [x_i for x_i in x]

The original one that I sent only works if x is already a list, tuple, generator, iterator, or 1D numpy object array.

Great, thank you! I agree, the two solution seem to work for me too. I'll also update loss_gradient the line 364 in predict accordingly. Can I ask you for a review in a bit?

Sure


fix: add single quote string escape_sequence

Checklist

[xi] All tests pass in CI
[x] There are enough tests for the new fix/feature
[x] Grammar rules have not been renamed unless absolutely necessary (0 rules renamed)
[x] The conflicts section hasn't grown too much (0 new conflicts)
[x] The parser size hasn't grown too much (master: 2681, PR: 2682)
(check the value of STATE_COUNT in src/parser.c)

Closes #173

This is great work! Thank you for all your effort on improving the parser!


EX command obscured coming from visual mode

From Visual Mode with Code highlighted, Pressing ":" moves the cursor to the Command Line area but the word "VISUAL" obstructs the command-line (see screenshot).
Big Time XVim Fan.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29116986-ex-command-obscured-coming-from-visual-mode?utm_campaign=plugin&utm_content=tracker%2F252770&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F252770&utm_medium=issues&utm_source=github).

Thanks for reporting.
Duplicate of #893.


Implement BigQuery CAI GA Resources in Forseti Inventory

https://cloud.google.com/resource-manager/docs/cloud-asset-inventory/overview
bigquery.googleapis.com/Dataset
bigquery.googleapis.com/Table

Hi All,
How can I test out the bigquery Table component?
I have some CAI exports from prior to the 23 August, which have bigquery table asset types in it, and I was using them, and for some reason, on the 23rd they stopped being populated.
Bigquery Datasets are still there, but no tables anymore.
I installed Forseti using terraform, but it is in a dev environment and I am happy to "upgrade" to a RC release to get the bigquery table CAI items back again.

Hi @hurgh, do you also notice any errors in the log, around Aug 23 when this stopped working?  You can check the log in stackdriver, like this:

Hi @joecheuk
Thanks for the response.
I have checked and the bigquery API is enabled in the forseti project.
Just to clarify, I have a number of CAI dump files on that day, and the first few all had bigquery.googleapi.com/Table in them, then at a point in the day and from then on, there have been no Table asset types in the files. There does not seem to have been anything that changed that I can see.
With your description of how BQ resources are currently gathered (calls to GCP API directly), would the items end up in the CAI dump file?
In the forseti_conf_server.yaml file, none of the API's are enabled, as in, they are all commented out. Again, this has not changed from when it work to now.
@blueandgold
Thanks for the tip, I did look through the logs around the time of the last good scan and the next failed scan (when the CAI dump files changed) and i did not see any obvious errors that made me think they were related to BQ.
Could there have been a change to the GCP API that caused this to stop?
Thanks again for the pointers.

Hi @hurgh , thanks for the clarifications!
The data are stored in the forseti-server cloud sql instance, inside database forseti_security.
There are multiple ways to connect to the cloud sql instance, e.g. using cloudsql proxy
For the purpose of verifying the data exists, you can try the following approach:
ssh to the forseti server VM
Install mysqlclient
sudo apt-get install mysql-client
Connect to the cloudsql instance
mysql -u root --host <IP_ADDRESS> --database forseti_security
And query the gcp_inventory table
select * from gcp_inventory where category='resource' and resource_type= 'bigquery_table' and inventory_index_id=(select id from inventory_index order by completed_at_datetime desc limit 1);

@joecheuk
Thanks for that, I did check, and it appears that there is table data in the DB.
I added BQ Table Label checks to the "enforce_labels" rule for config validator, but i no longer get any findings in SCC for table label issues, only dataset ones.
Does config_validator rely on the output from CAI or should it work if there is data in the DB (as proved by the above query).
Thanks

Hi @hurgh ,
Yes, as for the config validator scanner we have in Forseti, we currently only scans on the data provided by CAI. The BQ table ingestion from CAI should be ready in the upcoming 1-2 releases and you will be able to scan that use case very soon.
Thanks,
Joe

Thanks all for the information and help.
I will probably just wait for the release and see how it goes then.
Regards

Hi @hurgh ,
Would you be ok to wait for another 2 weeks for the changes to go out or would you prefer having the feature asap?
Thanks,
Joe

Hi All,
Did this not make it into 2.22.0?
What is still outstanding for this to be integrated? I am happy to test this or something if required.
Thanks

Hi @hurgh, the bigquery table data from CAI did not get released yet, pending testing.  At this point, it is being target for v2.23 release, which will go out next week.  @hshin-g for FYI

Is this something I can help test?
Happy to do so if it means it will make it into 2.23 release.
Let me know what I need to do.

@hurgh Thanks for your interest to help this this.  The gotcha here is that this change is part of a much larger set of performance improvement changes, which needed further testing.  So, it is not something you can help, though we do appreciate it.  Will keep you in the loop on this.

No problem.
Thanks

Hi @hurgh, the 2.23 release still needs more time to pass quality assurance.  Aiming for either later this week or early next week.  Thanks for your patience on this.

@blueandgold Thanks for the update.

@hurgh Forseti v2.23.0 was release last Friday.  Can you upgrade to that, and see if the bigquery table are in inventory for you now?


Enhancement: Add hovering effect to images in 'Sponsors' and 'Our Community Partners'

The icons below could be made more cool if we apply some hovering effect when mouse points them

@Kumar-laxmi do it


Logs  healthcheck in docker swarm

We use this image in a docker swarm context and we have too many logs like :
2022-11-30T11:30:06.864636+00:00 INFO    postfix/smtpd[522]: lost connection after CONNECT from unknown[<IP_ADDRESS>]
2022-11-30T11:30:06.864661+00:00 INFO    postfix/smtpd[522]: disconnect from unknown[<IP_ADDRESS>] commands=0/0
2022-11-30T11:30:09.840613+00:00 INFO    postfix/smtpd[522]: connect from unknown[<IP_ADDRESS>]
2022-11-30T11:30:09.840769+00:00 INFO    postfix/smtpd[522]: lost connection after CONNECT from unknown[<IP_ADDRESS>]
2022-11-30T11:30:09.840792+00:00 INFO    postfix/smtpd[522]: disconnect from unknown[<IP_ADDRESS>] commands=0/0
2022-11-30T11:30:12.449844+00:00 INFO    postfix/smtpd[522]: connect from unknown[<IP_ADDRESS>]
2022-11-30T11:30:12.449987+00:00 INFO    postfix/smtpd[522]: lost connection after CONNECT from unknown[<IP_ADDRESS>]
2022-11-30T11:30:12.450011+00:00 INFO    postfix/smtpd[522]: disconnect from unknown[<IP_ADDRESS>] commands=0/0

Is it possible to add a filter like it was done for #89 ?

This is part of the image, not HELM deployment -- you shouldn't be getting these messages is you're using the latest image.


[Bug] {{接口响应错误}}

Please search before asking

[X] I searched in the issues and found nothing similar.

Please read README

[X] I have read the usage instructions section in README in detail.

Please check your network

[X] I am sure it is not a network issue.

Claude Translator Bob Plugin version
0.5.5
系统版本 | System version
MacOS 13.6.7
Bob version
1.9.2
复现步骤 | Reproduce step
填入APIkey后出现错误。在OPENCAT使用正常
你看到了什么错误？What errors do you see?
接口响应错误
你期望看到什么？What did you expect to see?
正常工作
还有其他的内容吗？Anything else?
No response
你是否愿意提交一份 PR 来修改这个错误？Are you willing to submit a PR?

[ ] 我愿意提供 PR! I'm willing to submit a PR!

找到问题了，打扰了

什么问题呢，我也遇到类似情况了


plugin-api: fix coding style errors

I've got the style checkers passing now, it's just that the tools are failing to build (for known reasons).


a problem with messages

the users are not getting dm for the mesages

Hey, sorry for the delayed response! If you're still experiencing this issue, please join the support server and we'll try to troubleshoot it there: https://discord.gg/vRuhG9R

Hey, sorry for the delayed response! If you're still experiencing this issue, please join the support server and we'll try to troubleshoot it there: https://discord.gg/vRuhG9R


All but Line chart disappear when responsive set to true

So I've been playing around with the library. Great so far, except for one detail : If any other graph type than "Line" is set to responsive, it simply won't show on the page, wether the value is passed as a global argument or directly when calling the function.
It works perfectly for Line-type charts, though.

Could a JSBIN test case please be created for this issue

Closing since there is no test case.


PDF viewer throws error on startup

PDF viewer throws error on startup, and closes the whole tab if you click on the "Close" button.

#8 closes


Upgrading org.bouncycastle:bcprov-jdk15to18 to 1.78

Is your feature request related to a problem? Please provide an existing Issue # , or describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you are proposing
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Check List

[ ] Backport Labels added.
[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Can we please merge this to unblock 2.16.0, there has been issue with whitesource recently, I also noticed other CI checks failing as well.
Thank you
@getsaurabh02

We can go ahead by merging https://github.com/opensearch-project/performance-analyzer/pull/656


unsupported protocol from cargo install rustfmt

$ cargo install rustfmt -v
    Updating registry `https://github.com/rust-lang/crates.io-index`
 Downloading rustfmt v0.5.0
error: failed to download package `rustfmt v0.5.0` from https://crates.io/api/v1/crates/rustfmt/0.5.0/download

Caused by:
  Unsupported protocol

I built rust and cargo from git a week or so ago. Updated cargo to HEAD and built again, still get this error.

Yeah this can happen if libgit2 wasn't configured correctly to have https support, what system did you build cargo on?

My Ubuntu xenial system.

libgit2 isn't installed via the package manager (and I haven't installed it by hand).
dpkg -l libgit*
dpkg-query: no packages found matching libgit*

I did:
sudo rm -rf target
make
sudo make install

and now cargo is happy. I'm not sure if you wish to consider this a valid bug or not. Perhaps a FAQ would be good though?

Ah yeah libgit2 is built from source. Did you perhaps install a library as part of the build when something else failed?

Nope, but I've had that build dir around for ages; I couldn't tell you how
deep the junk was.

It may have just been a stale build of libgit2 then or something like that? There's been a few fixes to the detection of system deps in libgit2, and cargo tests at least fail if https support isn't built in. I'm gonna close this for now as "probably fixed at some point" or "likely spurious", but thanks for the report!


meetup: RCN online meetup #1

This change contains an initial proposal of the first RCN online meetup
with a submission of talk about lockc.
Signed-off-by: Michal Rostecki<EMAIL_ADDRESS>

I think that is a good start.
Just a few remarks:

What about a template for the meeting doc?
Could we put a remark regarding the meeting on the very top of the page? Something like "We have a monthly meeting! Next one will be on yyyy.mm.dd, feel free to add a point to the agenda"

Sorry for being so slow with following up. I think I addressed everything except

What about a template for the meeting doc?

@FlrnFrmm what kind of template do you have in mind? In what kind of format? And where would you keep it? Every.md file from docs/ gets automatically deployed as a Github Page.
Something like this?
## Rust Cloud Native meetup #{{number}}

*{{ date }}*

This is the {{ number }} meetup of the Rust Cloud Native initiative!

Let's meet online to talk about:

* the present and the future of RCN
* projects and topics you're interested in
* casual stuff afterwards :)

Feel free to submit a pull request to the [rust-cloud-native.github.io repo](https://github.com/rust-cloud-native/rust-cloud-native.github.io)
if you want to propose any agenda items.

Rust Cloud Native meetups are going to be organized every month. Each month
the time slot is going to change between more AMER-friendly (like this one) and
more EMEA-friendly.

### Agenda

{{ agenda }}

@nickgerace Done. I also made a separation between upcoming and past events.

@FlrnFrmm I will try to figure out something with templating in a separate PR. i hope that's OK for you.

🎉


Updated Readme.md

Hi!
What's the goal of this PR ? Looks like a spam for hacktoberfest.
I signal it as SPAM.

I thought of adding this for the new conributors,sorry for that pull request.

My intention was not to spam .

Sorry but git clone <git repo> and cd filename is not helping anyone.


Add files_analyzed property for ImageLayer class

Fixes #444
Signed-off-by: Manaswini Das<EMAIL_ADDRESS>

@nishakm I'm aware that I forgot to add the body to the commit. But I want to commit it along with the changes I'll make to the pull request, I also wanted to ask whether force-pushing will bring all those commits here again. I don't want to mess up again.

@manaswinidas you did it!! I took a look at your branch on your fork and it looks like you are only one commit difference between master, so I don't think that amending your commit is going to pull any extra changes.
Let me do a review and post instructions here on how to update your commit with the required changes.

@manaswinidas Excellent work! There's a couple of changes I have asked to the code and, of course, a body for the commit message. Here are the steps to fix your commit:

Make the changes I had asked for in the code
Once you are satisfied with them, run git add tern/classes/image_layer.py tests/test_class_image_layer.py and git commit --amend. This will update your existing commit.
Then do git push -f origin patch4 to update your PR.

@nishakm I have made the changes. Please let me know if there are any other changes that I have to make.

@manaswinidas One last request: It would be really nice if you could describe what you are changing. You can refer to #444 for commit message body content. Thanks again for working on this!

@manaswinidas All good! I will merge this. Thanks again!


eos-s3: btn_counter: use F4PGA build flow by default

Signed-off-by: Paweł Czarnecki<EMAIL_ADDRESS>

Ref: #298.

Depends on https://github.com/chipsalliance/f4pga-examples/pull/347. Will rebase this once #347 is merged.

Rebased. Ready to merge if green.


Enable to use slack Events API

Hey, I am using this client in combination with the Events API.
However, in order to do so, the modeled events in Events.scala have to be wrapped by event type structure of slack (see https://api.slack.com/events-api#event_type_structure).
Furthermore, an server challenge is sent to the url that one provides when subscribing for slack events (see https://api.slack.com/events/url_verification). Therefore, this event has to be modeled.
This PR enables using the defined events of this repo with slack Events API.
I am already using this for few events.
Kind regards

The test failure is for unrelated thing (which I assume is because of this PR being based off pre-Travis master)

Thanks for the contribution. Merging it.


[WFRP4e System] App v2 Character Sheet - missing content from Expand Data

Describe the bug
When using App v2, a lot of data - including Tags and Buttons - is missing from Expand Data when trappings are expanded.
To Reproduce
Change Character Sheet to V2 and click on any trapping, like a Weapon.
Screenshots

I believe the same happens across all tabs and item types

I left these out to reevaluate what I actually want to keep.


Enable importing configuration

Resolves #25

Codecov Report

Merging #47 (0cea9fd) into main (54b7117) will increase coverage by 2.35%.
The diff coverage is 86.00%.

@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   75.00%   77.35%   +2.35%
==========================================
  Files           5        5
  Lines          80      106      +26
  Branches        4        4
==========================================
+ Hits           60       82      +22
- Misses         19       23       +4
  Partials        1        1

Impacted Files
Coverage Δ

src/Test/Tasty/AutoCollect/ModuleType.hs
55.55% <ø> (ø)

src/Test/Tasty/AutoCollect/Config.hs
87.83% <86.00%> (-1.75%)
:arrow_down:

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


Do references have to be under #/definitions/ ?

I added references to a custom location controllers but when I try to actually reference them from my tests I am getting this error: The fragment '/controllers' does not exist on schema c6ca8c54-47bf-5290-be91-fa4bbb3ed08f
All I am doing is: schema(:$ref => "#/controllers/some_controller")

I believe so but to clarify you should refer the Swagger 2.0 spec

https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md


CHECKOV Linter does not respect VALIDATE_ALL_CODEBASE

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Within GitHub Action workflow using VALIDATE_ALL_CODEBASE: false, the CHECKOV linter is scanning files outside of the files added/modified by the pull request.
Expected Behavior
CHECKOV linter should respect VALIDATE_ALL_CODEBASE as the other linters.
Super-Linter version
v6.1.1

Relevant log output
See https://github.com/mcanouil/super-linter-checkov/actions/runs/7961206354/job/21731881369.
Triggered by https://github.com/mcanouil/super-linter-checkov/pull/1 pull request which only adds a README.md thus no Dockerfile.
Steps To Reproduce
See https://github.com/mcanouil/super-linter-checkov.

Hi @mcanouil !
This is expected, in the current implementation because Checkov needs to lint entire directories, not just standalone files, so we would need to address those corner cases, which is work that we cannot easily automate or streamline.
What are you trying to achieve?

You can achieve what you need with the Checkov "baseline" feature. It's somewhat documented in the Checkov CLI command reference. By establishing a baseline (of known failing checks), you can introduce Checkov in a legacy codebase.
Example: https://github.com/super-linter/super-linter/blob/main/.github/linters/.checkov.yaml and https://github.com/super-linter/super-linter/blob/main/.github/linters/.checkov.baseline.
You can generate a baseline file by opening a shell in a super-linter container running locally, and then run:
checkov --create-baseline --directory /tmp/lint

Hopefully, this will point you in the right direction.

Thanks, not exactly the expected answer but this will do.
For the future, could some changes like this one be listed as breaking changes? If a feature makes the workflow behaves differently that one can expect when using VALIDATE_ALL_CODEBASE, it should really be stated.

Is there a reason why CHECKOV is enabled by default if it breaks assumption that with VALIDATE_ALL_CODEBASE: false will be checked only changed?

I saw this.
My question is why it's enabled by default?

Indeed, the explanation only state what CHECKOV does but does not explain why you are willing to break workflows by enabling it by default.

You can disable this linter (as any other), and that it also support configuring a "baseline" so you don't have to fix everything before enabling it. If there's anything that's preventing this from working, please file a new bug, we'll happily review it.
All linters are enabled by default, since the early super-linter days, and Checkov is not a special case in that regard. Imagine that a lInter adds a new check (this happens all the time) that might flag issues that it previously ignored. So workflows that run that linter (using super-linter or otherwise) will now report failures. I don't think that they are breaking workflows, but rather the opposite: they are doing their job, unless you explicitly disable them, or ignore specific checks.

PS: other linters that are designed to run against directories instead of individual files are implemented this way, such as ansible-lint

All linters are enabled by default, since the early super-linter days, and Checkov is not a special case in that regard.

That is the explanation.
When a directory level linter is added, could there be a section in the documentation to list the linter that does not follow VALIDATE_ALL_CODEBASE?

Ah! That's a nice proposal. I'll happily review a PR for that.

I would have done it, although I would have to look at all the linters which I don't know nearly anything about to make such list.


update pug version

pug3.0.x is still unsupported yet.
I'm wondering when this project can be up-to-date...

For those interested, I've published a (fork) package with updated dependencies:  @anduh/pug-cli
It contains the changes @borisovg did in #88, and I changed it's command to pug3, so you can have both pug-cli (using pug) & @anduh/pug-cli (using pug3) installed at the same time, without conflict.

I like using pug, but the fact that the pug-cli is outdated and has major security issues is a total detractor from using it. It doesn't lead to confidence that the project is going to be well maintained in the future. It's especially frustrating that there are people in the community who are submitting patches that update / fix it that are being ignored: to the point where they have to fork it like @Anduh did. Please at least let the community maintain it!

Yeah, let the community maintain it. It's already 6 months since I published the fork, and been happily using it instead.
Guess I should take a look at the open PRs and see what to merge, so there is improvement beyond just updated deps.

Yeah @Anduh agreed. Thanks for your work on it. I've been using your fork on my last project and have been enjoying it. Pug is a great templating language and a CLI compiler is essential IMHO for a templating language to thrive. Please @ForbesLindesay can you make the CLI a community project, or at least stay up to date accepting patches. Pretty pretty please!

July 2024
For those still following this issue, I can point to @tokilabs/pug3-cli as a frequently updated fork than my @anduh/pug-cli.
Beyond bumping the dependencies, @tokilabs/pug3-cli have also merged 4 other changes from the other PRs left hanging here.
While I plan to take a look at @anduh/pug-cli in the near future to include the other features & bump deps, it's not certain when I'll find the time. So, as of July 2024, @tokilabs/pug3-cli seems to be the best maintained fork of pug-cli.


video stops loading and freezes when using Dark Reader extension on youtube

Bug description
the video timestamp freezes which causes youtube to not buffer more of the video but video still plays
Steps to reproduce the behavior
have ambient light installed and enabled
have Dark Reader extension (version 4.9.80) installed and enabled on youtube
play a video, select 1440p60 quality, watch video
then timestamp freezes and video stops buffering new parts and buffer runs out
Browser
Firefox
Operating system
Linux
Extension version
2.38.1
The bug still happens in these conditions

[ ] The bug still persist when I disable all other extensions (ex. in incognito mode with only this extension enabled)
[x] When I logged out of my YouTube account (ex. in incognito mode)
[x] In incognito mode

Additional context and/or screenshots
this seems like it only or at least mostly happens on 1440p quality
interacting with the youtube player, such as pausing/playing or otherwise causes it to buffer a small bit of the video again but freezes again

I'm unable to reproduce this issue (on Windows, because I don't have Linux).
But Firefox versions 123 and 124 have a performance regression bug (#224) So, theoretically, I could imagine that it takes up more CPU/GPU power that your system also needs to download and decode the video.
Could you check if this issue also exists on Firefox 122? Here you can download the older version:
https://support.mozilla.org/en-US/kb/install-older-version-firefox#w_i-still-want-to-downgrade-where-can-i-get-the-previous-version

i did some more digging on Firefox 123 and it seems to only happen with many tabs/windows open (307 tabs over 14 windows if you were wondering, i know, i know, i should get rid of at least 90% of them)
so it seems to be performance related, it could be that Dark Reader combined with Ambient Light causes increased CPU/GPU power usage and combined with the regression in #224 causes it to not have enough CPU/GPU power left for video buffering
i do not believe i ever experienced this before Firefox 123, even with more tabs open than currently
i could still try checking if i am able to recreate this in Firefox 122 if you want

Ah, that's indeed a lot of tabs. Then I think we can be almost sure that it is performance related.
Then this should also be fixed once I have released version 2.38.2 which contains a workaround for Firefox 123 and 124. I'll probably do that in 1 or 3 days when I've found some spare time.

@Tracenji Firefox have fixed the performance regression in version 125. Could you check if you still have this problem in Firefox 125?

Strange, I improved nothing regarding the performance in the new extension version. Maybe they fixed something in Firefox 124 then.
Anyway, nice to hear that it has been fixed!


For Java 9: use -source/-target 1.6 as 1.5 is unsupported

Java 9 will be released in about 28 days. It no longer supports the -source 1.5, argument, the minimum is 1.6. After fixing this, and working around #89, the build succeeds on the current Java 9 builds.

Makes sense!


Add WordMatch + WordNoMatch descriptor

Implement WordMatch, WordNoMatch,  descriptor + feature and associated unit tests.
Resolves issue #1308
Resolves issue #1309

Hi @jon-bown ,
This one looks good as well!
Could you please double check your src/evidently/features/_registry.py?
It looks like paths to feature classes are invalid.

Hi @jon-bown , This one looks good as well!
Could you please double check your src/evidently/features/_registry.py? It looks like paths to feature classes are invalid.

Done! Also want to point out that the error you're getting with my ItemMatch PR would need to be resolved here also.

Done! Also want to point out that the error you're getting with my ItemMatch PR would need to be resolved here also.

Thank you for the update, fair enough! I suggest you change a list to a tuple here as well, similar to what we’ve discussed in the related issue. This should help prevent the error and maintain consistency across both implementations.
Thanks for your attention to detail! 🙏

Hi @jon-bown ,
It looks like there’s a small issue with the Python f-string here:
File "/home/runner/work/evidently/evidently/src/evidently/features/words_feature.py", line 198
    default_display_name=f"Text contains {self.mode.split("_")[1]} defined words"

I believe the string is being parsed incorrectly due to the nested double quotes inside the f-string. To resolve this, try changing the internal double quotes to single quotes ('_').
That should fix the issue!

Hi @emeli-dral I believe I've fixed all the issues with this PR. Can you let me know if anything else needs to change? Thanks!

Hi @jon-bown,
I had a chance to review your repo, and I really like how the descriptor is working—great job! However, I would like to request one more small change, apologies for the extra ask!
To keep things consistent and avoid potential confusion for users, I’d suggest making the WordMatch descriptor work directly with tuples, just like the ItemMatch descriptor. Having one descriptor use tuples and another use lists might be a bit unclear.
In the future, we plan to support lists as an input type in a more centralized way, and we’ll introduce list support across all relevant descriptors then.
Thanks again for your hard work!

Hi @emeli-dral, thanks for the update! As far as I can see, this implementation matches the ItemMatch/NoMatch implementation. Is there anything in particular you can see that needs to change? In the unit tests I convert the lists to tuples before they are fed to the feature, same as the other PR so it won't work with lists inside the data frame. I will also resolve the merge conflicts in the meantime.

Hi @jon-bown,
You’re absolutely right, that was my mistake — thank you for pointing that out! I've merged main into your branch, and I'll proceed with the merge as soon as the tests pass.
Thanks again for your patience and your thorough work on this!


Allow to disable assignment operator

In some cases for security reason expression can be useful to disable assignment operators or other operators that modify values.

Now you can write:
var target = new Interpreter()
	.EnableAssignment(AssignmentOperators.None);


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3084

Questions ?
Please refer to the Backport tool documentation


Update NEWS file for new alpha release

Updated the NEWS file with the most significant items from CHANGES
I think this should go in before the alpha1 release

In these days of public git logs, is it time to retire NEWS or CHANGES?

Absolutely not.  (Some) people who download our tarballs do read them, especially since we urge them to (for example for information on type becoming opaque).

In these days of public git logs, is it time to retire NEWS or CHANGES?

I think they do provide a useful source of documentation. Reading through the git log would be a very difficult way of figuring out the main changes. The CHANGES file is relatively easy to read and provides that summary view. Similarly the NEWS file is more of the "executive summary" ;-)

I am not sure we really need both files.  Certainly one of them.

BTW, I just realised that there doesn't seem to be any mention in CHANGES of the RAND/DRBG changes that were made. That seems fairly significant and worthy of an entry? @mspncp

This brings up a problem with the current system.  How do we know when to put something in CHANGES?  I an assume that NEWS turns each CHANGES entry into a one-liner, but the first issue?

I an assume that NEWS turns each CHANGES entry into a one-liner, but the first issue?

I haven't put everything from CHANGES into NEWS in this PR. As I said in the description of this PR I just read through it and pulled out the things that I thought were the most significant

NEWS has usually been the broad user-visible changes, such as added features, removed features or changes that affect users in a big way (structures becoming opaque).  CHANGES has been more details, bigger descriptions of changes and also described bigger internal changes that don't directly affect the user
So no, CHANGES and NEWS aren't meant to have some kind of 1:1 mapping, that's entirely false.  However, we've been reaaaaally sloppy with CHANGES lately.
There's still a difference between CHANGES and the git log.  We don't put in a note on every typo in CHANGES, nor do we note code that's been reverted, or every itty gritty bug correction.  And most of all, we can make "history changes" in CHANGES, while we cannot do so with the git log, so while CHANGES is supposed to be a log, it's still supposed to be a reader's digest compared to the git log.

BTW, I just realised that there doesn't seem to be any mention in CHANGES of the RAND/DRBG changes that were made. That seems fairly significant and worthy of an entry? @mspncp

I won't manage to do it in time for the freeze today, but I intend to add a lot of documention for the new RAND_DRBG api in the next weeks, hopefully until the next alpha. This will include a few lines to the changelog, of course.

I won't manage to do it in time for the freeze today, but I intend to add a lot of documention for the new RAND_DRBG api in the next weeks, hopefully until the next alpha. This will include a few lines to the changelog, of course.

Great (BTW we're already frozen, but I would have made an exception for a CHANGES update, since this is very low risk).

@mspncp, if you can get the updates done in the form of a PR to update CHANGES and NEWS then I can consider it before I do the release.

This PR is pushed. Closing.


Linux/WINE Compatability

Is anyone aware of a way to run/compile the GUI version of this on Linux/a fork which runs on modern Linux distros?
Alternatively any tips to set things up via WINE.
Thank you!

I did purposefully go down the NetCore5 route to facilitate this but as it stands it takes windows dependencies; and sorry I've never had the need/will to go down the Wine route; but I'd be happy to help with core changes if you were to give it a go.
Net6 I could probably achieve without too much effort if that would help.
Beyond that, Net.Maui has potential but I did this while on gardening leave; busy again now...

If switching to NET6 is the simplest cross-platform option then I’d definitely be interested in going that route.
I was able to compile/run the console version. It would accept files/run/generate a log file. However it would not generate any nested output files.
Are there any options to donate to this project?

The console's a hangover from DeepNestPort. I kept it compiling but didn't give it any more attention than that, tbh. I just wanted the UI. You might get more mileage from the console version in DeepNestPort.
Options to donate? You mean contributing code to the repo? Absolutely. Beyond that it wouldn't take very long to get to the point that buying a commercial package would be the most effective option...

I’ll definitely try the console for the time being - though I too would prefer to use the UI.
Sadly I can’t help much in that regard.
Just figured I’d ask - I like supporting FOSS projects!

Just bumped to Net6; dunno if that'll help much though as it still has the windows dependencies...

Sorry to necrobump, but I was able to get the latest DeepNestCore up and running on WINE after using winetricks to install the dotnet 6 runtime. Haven't tried with DeepNestSharp yet, but will soon. Were you aware of winetricks @gyundt?


cardano-client CI integration

https://github.com/input-output-hk/ouroboros-network/pull/2133

fixed in https://github.com/input-output-hk/ouroboros-network/pull/2155


Add CI

Copy-pasted from here: https://github.com/EnBWAG/netzlive.elsa/pull/2
I used a more recent versions for elixir and otp though.

@jtrees This repo is public. We shouldn't do project specific stuff here.

Makes sense. I think we should rename the repo then though.

@jtrees Rename or make it private?

@jtrees Rename or make it private?

I'm no longer blocked by this and have no preference.

Private: -> publish to our own hex registry
Rename & Public: -> publish to hex.pm
Would prefer Rename & Public, but this is more work to do.

Ich werde das Thema nicht weiter verfolgen.


test_all should check if the testing data exist or not

When no testing data were download before, test_all just fails and doesn't say why. It should check if the testing data folder exists and suggest to use flag -d 1 if not. (works properly with -d 1)
FAIL:

Testing folder was containing :

Using -d 1 :

Fixed in cf789565e037d190134829cfa4c87a2a7f1b646b


Add loading linter rules from .rules.verible_lint

This PR:

Moves conversions of LSP URI to paths and vice versa to a separate library
Enables checking .rules.verible_lint by default
Adds mechanism in parsed buffers to check for .rules.verible_lint up in directory hierarchy, starting from linted file, to load linter rules
Adds tests for verifying reactions of VerilogLanguageServer for .rules.verible_lint file

@hzeller - I am not sure about changing the default value of the --rules-config-search to true. This flag is available from the level of verible-verilog-ls - not sure whether I should change the flag value to true via SetFlag in a function delivered with VerilogLanguageServer, or change this default value directly in verilog_linter.

Codecov Report
Base: 93.00% // Head: 93.04% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (21aac1c) compared to base (28d898d).
Patch coverage: 99.43% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1730      +/-   ##
==========================================
+ Coverage   93.00%   93.04%   +0.03%
==========================================
  Files         352      353       +1
  Lines       25480    25503      +23
==========================================
+ Hits        23697    23728      +31
+ Misses       1783     1775       -8

Impacted Files
Coverage Δ

common/formatting/state_node.h
100.00% <ø> (ø)

verilog/analysis/verilog_linter.cc
92.97% <ø> (ø)

verilog/tools/ls/symbol-table-handler.cc
85.86% <ø> (-0.45%)
:arrow_down:

verilog/tools/ls/symbol-table-handler.h
100.00% <ø> (ø)

verilog/tools/ls/verilog-language-server.cc
94.59% <ø> (ø)

verilog/parser/verilog_lexical_context.cc
95.48% <93.33%> (ø)

common/formatting/state_node.cc
95.45% <100.00%> (ø)

common/text/concrete_syntax_tree.h
96.36% <100.00%> (ø)

common/text/text_structure.cc
97.20% <100.00%> (ø)

common/util/interval_set.h
99.60% <100.00%> (+<0.01%)
:arrow_up:

... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


change class_name to 8 classes

I have a problem, when I changed class_name to 8 classes and I set load_pretrained=False, freeze_body=False.

The class id in your annotation file should be 0 to 7. There is a 8, which leads to the error.

copy! Thank u very much.

your code is so good.Could you give me some advice for learning keras?

Not a piece of advice. I usually read Keras documentation or Keras python help, when I meet problems.


as8p194mb2000.eurp194.prod.outlook.com

https://as8p194mb2000.eurp194.prod.outlook.com/git.status

Thanks for taking the time to submit feedback about our content. We’ve implemented a new system to make it easier for you to provide feedback. As a result, we’re no longer accepting feedback in this repository via GitHub Issues so this issue is being closed. For more information about the new system, please visit https://learn.microsoft.com/office/new-feedback.


npm - No README

The readme doesn't seem to make it to npm, which is inconvenient. Would be nice to see the contents there rather than having to come through to github.

Saw the same thing just now.

Thanks for mentioning. That was quite weird. I tagged version 2.1.1 as latest and now the README shows again. The README also was part of 1.0.4, which was the previous latest, so maybe npm just needed some change to re-render the README.


Add proper docs for full usage

We need proper documentation for:

how to use the new CLI
how to use the programmatic APIs
migration guide from v3 to v4

calling node API no longer loads JSON,
no more bsconfig.json, only rokudeploy.json
name changes like .. also some have been removed and need to be called manually
talk about why changes were made in v4
new default file array
anything else that a dev might need to change when upgrading

#162

#166


Create frontend logic for abc quiz

Very unspecified issue. But I can do it anyways.

 Create frontend logic for abc quiz


feat(datepicker): add animation to calendar popup

Adds an animation when opening and closing the datepicker's calendar.
For reference:

This is a resubmit of #8542.

@crisbeto needs rebase

Rebased and sorted out some new test failures.


Add a contains method for postgres range types

Fixes #2210
(I've just needed this method, so I can simply put it into upstream
because it's probably useful for others too)

How to allow nullable Range?
I add
impl<ST> RangeHelper for Nullable<Range<ST>> {
    type Inner = ST;
}

But how impl<T, ST> PgRangeExpressionMethods for T for nullable range?

@a1ien It's already implemented for Nullable<Range<ST>>, but not released yet.


Add priyanka77492's file

This pull request adds priyanka77492's file to the _post folder.

if you dream of success make sure you are awake

yes


[C++][Parquet] Basic RowGroup filtering

See https://github.com/dask/fastparquet/blob/master/fastparquet/api.py#L296-L300
We should be able to translate this into C++ enums and apply in the Arrow read methods methods.
Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=uwe) / @xhochy
Related issues:

[Python] RowGroup filtering on file level (relates to)
[C++] Supply row group indices to parquet::arrow::FileReader::ReadTable (is related to)
[C++] Change parquet::arrow::FileReader::ReadRowGroups to read into contiguous arrays (is related to)

Note: This issue was originally created as PARQUET-1158. Please see the migration documentation for further details.

Keith Godwin Chapman:
@xhochy Do you have a timeline in mind for when this would be ready?

Joris Van den Bossche / @jorisvandenbossche:
Do we still have plans to tackle this in the parquet code itself? (now such filtering is handles by arrow::dataset)

Uwe Korn / @xhochy:
[~keithgchapman] No, I'm not actively working on this.


Email Infrastructure guide Link is broken.

Expected content
Email Infrastructure guide Link is broken.
Link to page:
https://sendgrid.com/docs/glossary/sender-id/

Hi @pradeep864 ,
We're sorry you struggled with this issue, and we appreciate the feedback! We've added this to our internal docs backlog and will resolve the issue as soon as possible.
Twilio SendGrid Docs Team