Error occurred when executing KSampler:  local variable 'orig_memory_required' referenced before assignment

got prompt
[AnimateDiffEvo] - INFO - Ejecting motion module temporaldiff-v1-animatediff.ckpt version None.
ERROR:root:!!! Exception during processing !!!
ERROR:root:Traceback (most recent call last):
File "G:\AI\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-AnimateDiff-Evolved\animatediff_EVO\sampling.py", line 143, in animatediff_sample
orig_memory_required = model.model.memory_required # allows for "unlimited area hack" to prevent halving of conds/unconds
File "G:\AI\ComfyUI_windows_portable\python_embeded\lib\site-packages\torch\nn\modules\module.py", line 1614, in getattr
raise AttributeError("'{}' object has no attribute '{}'".format(
AttributeError: 'BaseModel' object has no attribute 'memory_required'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "G:\AI\ComfyUI_windows_portable\ComfyUI\execution.py", line 153, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "G:\AI\ComfyUI_windows_portable\ComfyUI\execution.py", line 83, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "G:\AI\ComfyUI_windows_portable\ComfyUI\execution.py", line 76, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "G:\AI\ComfyUI_windows_portable\ComfyUI\nodes.py", line 1237, in sample
return common_ksampler(model, seed, steps, cfg, sampler_name, scheduler, positive, negative, latent_image, denoise=denoise)
File "G:\AI\ComfyUI_windows_portable\ComfyUI\nodes.py", line 1207, in common_ksampler
samples = comfy.sample.sample(model, noise, steps, cfg, sampler_name, scheduler, positive, negative, latent_image,
File "G:\AI\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Impact-Pack\modules\impact\sample_error_enhancer.py", line 9, in informative_sample
return original_sample(*args, **kwargs)
File "G:\AI\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-AnimateDiff-Evolved\animatediff_EVO\sampling.py", line 220, in animatediff_sample
model.model.memory_required = orig_memory_required
UnboundLocalError: local variable 'orig_memory_required' referenced before assignment
I'm sorry to bother you. I have encountered the above problems. My comfyui version is 2023.11.10(002aefa).

https://github.com/Kosinkadink/ComfyUI-AnimateDiff-Evolved/issues/176
Update your ComfyUI.


CHANGELOG for upcoming v0.58.0 release

Thanks!
This whole "decimal system" thing evidently confuses me.  One day I shall learn...

Pulling once tests pass.


Episode detail back button does not return to episode list

Hi. Thanks for the superb project!
When trying to figure out which episode to watch you have to jump around a lot.
When pressing the back navigation button from an episode's detail page it goes back two screens, to the search result page, skipping "anime season details"/"episode list" page.

Hi, thanks for the feedback. I will take a look at this. I remember there is a reason why it was done this way or maybe just a bug.

Maybe I will look at this and make a pr if I have time ,and if thats okay. (I am quite a beginner in flutter, thought). I could finally start it up on my phone in debug mode and set up flutter enviroment properly on my machine.

That's fine. You can have a look. It is probably something with the Navigator but I need to recall why I did it like this. Potentially, I wanted to prevent infinite loop but I think it is really not a big problem.

@vkhobor I have found the issue. I think it is my design but clearly, it is very annoying. Please see this line. Go to branch update/1.3.1 and change Navigator.pushReplacement to Navigator.push should fix this issue.


(PUP-6445) Pin win32 gems to explicit versions

The Win32-Service gem was updated to 0.8.9 however the new gem pulled in
a dependent gem called ffi-win32-extensions which has a name collision
on the FFI:Pointer.read_wide_string method.  This causes Puppet to fail
in mutiple locations.  This commit pins the Win32-Service gem to a
prior version without the dependency.
Additionally, all other win32-xxx gems are pinned to explicit versions
so that Puppet may not experience a similar issue in the future.
This was performed in d4d6734e5f858b4e3f4db24c356ea7cb9db2d991 against
stable, but should have also targeted 3.x as well.

Good idea

Confirmed in local dev environment, expected gems are installed.

CLA signed by all contributors.


Incorrect information about ASPNETCORE_ENVIRONMENT

Description

Incorrect information about ASPNETCORE_ENVIRONMENT being used only WebApplication.CreateBuilder(). It is used by  Host.CreateDefaultBuilder() as well.

Page URL
https://learn.microsoft.com/en-us/aspnet/core/fundamentals/environments?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/environments.md
Document ID
1cb5eb30-dd84-e5e1-5826-d74243fabf04
Article author
@tdykstra

Per the doc, When using Host.CreateDefaultBuilder, the ASPNETCORE_ENVIRONMENT setting has higher precedence for environment configuration.


PC Login recipe

This has come up now and then. Also raised by USW yesterday.
It should be a straight-forward variation on the login/logout statement which specifies the values to use in the "object" for the PC scenario.

I'm closing this and will be opening a new ticket looking at a generic login recipe.


Cypress/E2E: Move cy.apiCreateBot and fix MM-T251 & MM-T252

Summary

Move cy.apiCreateBot
While testing change due to movement of cy.apiCreateBot, I came across to fix failing on MM-T252. I should make a separate PR for this change but those are connected in some ways.
add fix for MM-T251

Cherry pick is scheduled.


Add Mirantis.Lens 3.6.8

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run

/azp run


Expensify Card - There are two error messages when trying to proceed without a limit

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.6-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to workspace settings > Expensify Card (enable in More features)
Click Issue new card
Proceed to Step 4 - Set a limit
Click Next without entering anything

Expected Result:
There will be one error message when trying to proceed without a limit
Actual Result:
There are two error messages when trying to proceed without a limit
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/78cb5264-0717-4e01-9e1d-3376162a014f
View all open jobs on GitHub

@jliexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 2

Proposal
Please re-state the problem that we are trying to solve in this issue.
There are two error messages when trying to proceed without a limit
What is the root cause of that problem?
We don't pass shouldHideFixErrorsAlert into FormProvider to hide the fix error alert. Then we have two errors, one is the amount input error, and one is the form error.
https://github.com/Expensify/App/blob/105dce5151053bf6c81e60e68a610dd888c5da45/src/pages/workspace/card/issueNew/LimitStep.tsx#L80
What changes do you think we should make to solve the problem?
Pass shouldHideFixErrorsAlert as true into FormProvider as we do in other places that use AmountForm input
https://github.com/Expensify/App/blob/105dce5151053bf6c81e60e68a610dd888c5da45/src/pages/workspace/card/issueNew/LimitStep.tsx#L80
What alternative solutions did you explore? (Optional)

Not a blocker

@DylanDylann what do you think of the proposal?

@mountiny reviewing now

This is straightforward. @nkdengineer's proposal looks good to me
🎀👀🎀 C+ reviewed

Payment Summary

C: @nkdengineer $125
C+: @DylanDylann $125 - bump to complete checklist!

Upwork job

@DylanDylann will be paid separately for the entire project
Checklist is not required here as we are working on this new feature which caused this bug

Thanks Vit, fixed summary!

Paid and job closed!


cycle-detection finds cycles in commented-out code

To my surprise, if you create a file called asdf.sql, this will throw a compile-time error:
-- {{ ref('asdf') }}

SELECT 1;

My expectation is that commented-out code is commented out for the purposes of compilation. I can see why that may be not something you'd want to fix, technically, but at a minimum a smarter error message would be helpful.
Still on dbt 0.8.3...

@HarlanH the -- is a sql comment, but the jinja parser in dbt doesn't know anything about sql and does not treat that comment with any significance. You should instead use jinja comments to comment out jinja code:
{#
  {{ ref('asdf') }}      <------ this won't throw an error because it won't be parsed
#}

If you do need to refer to the currently-running model, you can use the {{ this }} variable which will not build a self-edge in the graph, avoiding the cycle, but will interpolate "schema"."table",
We've discussed making -- work as a "line comment" 1 2 in the past, but we haven't implemented that yet.
I think we need better docs around sql/jinja comments, as this is often tricky and unintuitive. Let me know if you think there's anything else for us to do here.
Thanks!

ah, gotcha. Yeah, not sure what stopgap would help. Will ponder.

@HarlanH I'm going to close this issue, but please re-open if you think there's something that needs to change in dbt here!

I ran into this problem today when commenting out code to remove a dependency... by the dependency was still there. It left me pulling out my hair for a bit. Glad I ran across this thread.

Ran across this as well - was about to file a bug report when I went deep into the issues list and found this thread. It's not just cycle detection, nonexistent nodes, etc. also run into this. Would a note about this be a worthwhile addition to the core documentation?


Change PersonSearchService to PersonMatchingService

The policy used by the Search method on PersonSearchService is specific to matching in AuthorizeAccess; this moves the service into the AuthorizeAccess project accordingly and renames it to PersonMatchingService. There will be additional methods added to it shortly for returning suggested matches for the support back end.
I've also amended the implementation to use raw SQL; the resulting query is simpler than the previous EF-generated query and is about 2x faster from an initial test locally. (I expect it to be even quicker when run in production with real data.)

 Resolve One Login 'not matched' support ticket


[Question] Accessing system env vars from inside Insomnia environments

Is it possible to have an environment variable in insomnia, that evaluates the value of a system env var? The goal is not to have hard-coded secrets inside Insomnia env variables, so when we export/import data, the output files would not have any sensitive information inside them.
Thanks.

Hi, I'm looking for the same functionality also.
Any plans for this?

Just saw that this is a duplicate of https://github.com/Kong/insomnia/issues/1883.

@etauker Totally, I haven't seen this one. Closing in favor of #1883


Include the following testcases

Test 1:
Add a small post as a demo (in test as a test case and destroy it after)
Wait and let it rebuild and then check to see if the default image is grabbed.
Test 2:
If you select the next_button you expect the image to change to a new one. Same for prev_button
Test 3:
That slides match the links in the recent posts section
Test 4:
When you click the links in the carousel they go to the corresponding webpages

Time any chance that you were able to add all these tests?

Time any chance that you were able to add all these tests?

Oops! I've added all except Test 1. I'll be on this right away. Thanks


Start a floating terminal

Hi there, thanks for the great WM. I was wondering if there's a possibility to configure alt + \ to start a floating terminal.

You can use a combination of chunkc commands to tell chunkwm to float the next window open. Something like:
alt + \                : chunkc set window_float_next 1; command_to_open_a_new_terminal_window

Did not know I can chain command using ;. Much thanks @crisidev, appreciate it.


bgp-control-plane ebgp-multihop config

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
how to set bgp-control-plane  ebgp config?
bgp-control-plane use gobgp, how to set more config
like gobgp this config:
[neighbors.ebgp-multihop.config]
enabled = true
multihop-ttl = 255
Cilium Version
1.12.0
Kernel Version
4.19
Kubernetes Version
1.25.0
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

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

Hi, I'm not sure I understand the question. Can you please elaborate on what you are trying to achieve?

Is bgp-control-plane supports  ebgp multihop？

Lets close this issue in favor of #21753


Addon not working on Ubuntu 20.04.4 with Godot 3.5

I followed the installation instructions, including enabling the addon and copying the udev rules.  I see the "Space" UI panel, but there is no response to Space Mouse input.  It works as expected in Blender.
In the Space UI, I tried camera and object mode.  I tried unplugging and re-plugging the Space Mouse receiver, and I tried running the udevadm commands you specify.

Which model Space Nav do you have? I just tested on Godot 3.5 and Ubuntu 22.04.1 and it still seems to be working. Are there any errors or warnings in the console? The plug-in also doesn't refresh when the editor is open, so if you replug it in, you have to restart the editor. I've noticed sometimes testing in Blender first, then opening Godot can help if it loses the connection.

Thank you for the timely response.  I have the 3dconnexion Space Mouse Wireless, which I purchased new about 3 or 4 years ago.
In the console (with non-system directory paths replaced by ellipses:
...$ OpenGL ES 2.0 Renderer: NVIDIA GeForce GTX 980/PCIe/SSE2
/lib/x86_64-linux-gnu/libasound.so.2: undefined symbol: snd_dlpath

ERROR: Can't open dynamic library: .../addons/spacemouse/bin/libspacemouse.so. Error: libhidapi-hidraw.so.0: cannot open shared object file: No such file or directory
   at: open_dynamic_library (drivers/unix/os_unix.cpp:412)
ERROR: No valid library handle, can't get symbol from GDNative object
   at: get_symbol (modules/gdnative/gdnative.cpp:510)
ERROR: No nativescript_init in "res://addons/spacemouse/bin/libspacemouse.so" found
   at: init_library (modules/gdnative/nativescript/nativescript.cpp:1503)
SCRIPT ERROR: Attempt to call function 'new' in base 'NativeScript' on a null instance.
          at: _ready (res://addons/spacemouse/SpaceMouse.gd:7)

I'm missing a dynamic link library.  I don't know if it matters, but I installed Godot by downloading the Linux binaries directly from the website and unzipping them.

It looks like you have to do this:
sudo apt install libhidapi-hidraw0
I thought that came with Ubuntu, but I guess not. If that works, I'll update the README.

You may also need to add this line to your .profile
export LD_LIBRARY_PATH=/usr/local/lib:$LD_LIBRARY_PATH

It works after installing libhidapi-hidraw0.  I didn't need the export line.  Thank you so much for this addon.  This functionality has been on my wish list since I started using Godot in 2019.


Error in DRIMSeq::dmDSdata(counts = counts, samples = samps)

Description of the bug
I'm getting an error with DRIMSeq when I try to run rnasplice. The pipeline works fine with other samples, so I'm not sure what's causing this error.
Command used and terminal output
Command Used:
nextflow run nf-core/rnasplice -profile docker -params-file /path/to/params.yaml

Error message:
ERROR ~ Error executing process > 'NFCORE_RNASPLICE:RNASPLICE:DRIMSEQ_DEXSEQ_DTU_SALMON:DRIMSEQ_FILTER (1)'

Caused by:
  Process `NFCORE_RNASPLICE:RNASPLICE:DRIMSEQ_DEXSEQ_DTU_SALMON:DRIMSEQ_FILTER (1)` terminated with an error exit status (1)

Command executed:

  run_drimseq_filter.R salmon.merged.txi.dtu.rds tximport.tx2gene.tsv Arc_RIPseq-samplesheet.csv \
      6 \
      3 \
      3 \
      10 \
      0.1 \
      10

  cat <<-END_VERSIONS > versions.yml
  "NFCORE_RNASPLICE:RNASPLICE:DRIMSEQ_DEXSEQ_DTU_SALMON:DRIMSEQ_FILTER":
      r-base: $(echo $(R --version 2>&1) | sed 's/^.*R version //; s/ .*$//')
      bioconductor-drimseq: $(Rscript -e "library(DRIMSeq); cat(as.character(packageVersion('DRIMSeq')))")
  END_VERSIONS

Command exit status:
  1

Command output:
  (empty)

Command error:

  Attaching package: 'DRIMSeq'

  The following object is masked from 'package:base':

      proportions

  Error in DRIMSeq::dmDSdata(counts = counts, samples = samps) :
    all(samples$sample_id %in% colnames(counts)) is not TRUE
  Calls: <Anonymous> -> stopifnot
  Execution halted

Relevant files
Samplesheet file:
sample,fastq_1,fastq_2,strandedness,condition
ID11-IP-Arc-R2,/path/to/rawdata/ID11-IP-Arc-R2_1.fastq.gz,/path/to/rawdata/ID11-IP-Arc-R2_2.fastq.gz,unstranded,treatment
IP-Arc-R1,/path/to/rawdata/IP-Arc-R1_1.fastq.gz,/path/to/rawdata/IP-Arc-R1_2.fastq.gz,unstranded,treatment
ID-2-Arc-IP-R3,/path/to/rawdata/ID-2-Arc-IP-R3_1.fastq.gz,/path/to/rawdata/ID-2-Arc-IP-R3_2.fastq.gz,unstranded,treatment
ID-5-Non-IP-R3,/path/to/rawdata/ID-5-Non-IP-R3_1.fastq.gz,/path/to/rawdata/ID-5-Non-IP-R3_2.fastq.gz,unstranded,Control
ID12-Non-Arc-R2,/path/to/rawdata/ID12-Non-Arc-R2_1.fastq.gz,/path/to/rawdata/ID12-Non-Arc-R2_2.fastq.gz,unstranded,Control
Non-Arc-R1,/path/to/rawdata/Non-Arc-R1_1.fastq.gz,/path/to/rawdata/Non-Arc-R1_2.fastq.gz,unstranded,Control
params.yaml file:
input: '/path/to/sequencing_data/Arc_RIPseq-samplesheet.csv'
contrasts: '/path/to/sequencing_data/Arc_RIPseq-contrastsheet.csv'
outdir: '/path/to/sequencing_data/'
fasta: '/home/gimena/reference_files/BDGP6.32_ensemble/Drosophila_melanogaster.BDGP6.32.dna.toplevel.fa'
gtf: '/home/gimena/reference_files/GTF_files/fromEnsembl/Drosophila_melanogaster.BDGP6.32.106.gtf'
source: 'fastq'
aligner: 'star_salmon'
min_samps_gene_expr: 6
min_gene_expr: 10
min_samps_feature_expr: 3
min_feature_expr: 10
min_samps_feature_prop: 3
min_feature_prop: 0.1
sashimi_plot: true
fig_height: 20
fig_width: 20
miso_genes: FBgn0265487
skip_bigwig: false
dexseq_dtu: true
suppa: false
edger_exon: false
dexseq_exon: false
rmats: false
System information
Nextflow Version: version 23.10.0
Linux OS
Version of nf-core/rnasplice 1.0.1

I'm also running into the same issue! I thought it was the minimum sample/feature tags, so I set it to zero:
min_samps_gene_expr: 0
min_gene_expr: 0
min_samps_feature_expr: 0
min_feature_expr: 0
min_samps_feature_prop: 0
min_feature_prop: 0
That wasn't the issue either, so I'm hoping someone knows how to solve this.

Hey @GimenaA
Thanks for reporting your issue. Can you please try with the latest release? I have a feeling you are using an older version where we didn't handle sample names with dashes in them correctly. See this PR for details.

Closing as completed. Please re-open if you are still facing the same issue.


Animation with ClipPath?

I'm trying to create an animation but nothing is happening when I put my AnimatedRect component inside of the ClipPath. The AnimatedRect does work if I move it outside of the ClipPath. Am I doing something wrong?
import React, { useEffect, useRef } from 'react';
import { View, Animated } from 'react-native';
import { Svg, Path, Rect, Defs, ClipPath } from 'react-native-svg';

const AnimatedRect = Animated.createAnimatedComponent(Rect);

function LiquidTank() {
  const liquidValue = 50;

  const factor = 1.54; // 154 / 100
  const height = factor * -liquidValue;

  const heightAnim = useRef(new Animated.Value(0)).current;

  useEffect(() => {
    heightAnim.setValue(0);
    const fillAnimation = Animated.loop(
      Animated.sequence([
        Animated.timing(heightAnim, {
          toValue: height,
          duration: 500,
          useNativeDriver: true,
        }),
        Animated.timing(heightAnim, {
          toValue: 0,
          duration: 500,
          useNativeDriver: true,
        }),
      ]),
    );
    fillAnimation.start();
  }, [height]);

  return (
    <View>
      <Svg width={250} height={100} viewBox="0 0 375 154">
        <Defs>
          <ClipPath id={`clip-${height}`}>
            <AnimatedRect x={0} y={154} width={400} height={heightAnim} />
          </ClipPath>
        </Defs>

        <Path
          fill="blue"
          clipPath={`url(#clip-${height})`}
          d="M375 77c0-26.515-13.353-50.48-33.922-67.641C335.034 4.316 327.339 2 318.606 2v150c8.733 0 17.362-2.311 23.376-7.377C362.353 127.462 375 103.506 375 77zm-56.394 75V2H56.394v150h262.212zM33.018 9.377C12.647 26.537 0 50.494 0 77c0 26.505 12.647 50.462 33.018 67.623C39.032 149.689 47.66 152 56.394 152V2C47.66 2 39.032 4.311 33.018 9.377z"
        />
        <Path
          fill="blue"
          clipPath={`url(#clip-${height})`}
          d="M86.333 0h17.334C104.955 0 105 .43 105 2.04v2.04H85V2.04C85 .43 85.045 0 86.333 0zM85 149.921v2.039c0 1.61.045 2.04 1.333 2.04h17.334c1.288 0 1.333-.43 1.333-2.04v-2.039H85z"
        />
        <Path
          fill="blue"
          clipPath={`url(#clip-${height})`}
          d="M85 4.08v145.84h20V4.079H85zM271.333 0h17.334C289.955 0 290 .43 290 2.04v2.04h-20V2.04c0-1.61.045-2.04 1.333-2.04zM270 149.921v2.039c0 1.61.045 2.04 1.333 2.04h17.334c1.288 0 1.333-.43 1.333-2.04v-2.039h-20z"
        />
        <Path fill="blue" clipPath={`url(#clip-${height})`} d="M270 4.08v145.84h20V4.079h-20z" />

        {/* It works if I move the AnimatedRect here */}
        {/* <AnimatedRect fill="#00AFEA" x={0} y={154} width={400} height={heightAnim} /> */}
      </Svg>
    </View>
  );
}

export default LiquidTank;

Which version are you using?
I didn't test the newer versions, but had similar problems on Android with v12.1.1.
Try downgrading to v12.1.0.
If this solves the issue, maybe I should reopen #1661.

Which react-native-svg version are you using?
I didn't test the newer versions, but had similar problems on Android with v12.1.1. Try downgrading to v12.1.0.
If this solves the issue, maybe I should reopen #1661.

I was using v12.1.1, but I tried downgrading to v12.1.0, and still having the same issue.
I noticed you are using reanimated in that post, but I'm using the Animated from RN itself, maybe that's the issue?
Let me try with reanimated instead.

I have the same issue!

For me, with the latest Version 14.1.0 it works fine on iOS but not on Android.


Translate the MenuItems and logging into English

I'd like to use this library (plus a few others you've made), but I don't speak Japanese. I also don't want to infringe upon your right to develop tools in your preferred language, so all Japanese text is still there. Because Unity doesn't have a good way to localize editor extensions, I had to get creative.

I wrapped all [MenuItem] attributes with a KOGANE_ENGLISH symbol. Users of your library can define this symbol in their project if they want [MenuItem]s to be in English. (Attributes in C# can only have constant values.)
All logging is done in either English or Japanese, depending on the value of Application.systemLanguage.
All text was translated with Google Translate, and I manually edited the results for grammar.

If I start to use some of your other libraries, I will contribute similar changes where appropriate.

Hi @baba-s, have you been able to review these proposed changes?

@JesseTG
Thank you for the message! And sorry for the late reply.
I have no plans to implement localization features.
But my library is basically Unlicense, so you have the following permissions:

Private use
Commercial use
Modification
Distribution

(License file is uncommitted but  this repository is "Unlicense".)
So I don't implement localization features in this repository, but you are free to modify it. :bow:

I have already done all the work for you; the only thing you have to do is approve and merge these changes, if you want them. The reason I submitted this pull request is to allow all users of this library to benefit from the changes without having to hunt for other (possibly outdated) forks.
However, if you sincerely don't want these changes then I will respect your wishes.


0.4.0: windows paths, remove form history, shell completion

still need to add windows paths per #42

breaking/important changes:

bugfix: stream didnt print path separator
removed --form-history


(Please share script) Auto quality 1080p or 2K in youtube.js

Can you share me the script setting auto quality 1080p or 2K to add into youtube.js?

I don't have time to implement this feature.
The code for Youtube has already been open-sourced, so if you can implement this feature, please feel free to submit a pull request.


ensure that there are no "nulls" in the signature calculation

Currently, when the Jupyter message is sent over the ZMQ socket, it's sent without any nulls:
https://github.com/dotnet/interactive/blob/474c320cb13c8bd9356713faad10a39782a3af79/src/Microsoft.DotNet.Interactive.Jupyter/ZMQ/MessageSender.cs#L37
But the message with nulls translates those fields <null> in signature generation. So the signature no longer matches what is actually sent over the socket.
https://github.com/dotnet/interactive/blob/474c320cb13c8bd9356713faad10a39782a3af79/src/Microsoft.DotNet.Interactive.Jupyter/ZMQ/SignatureValidator.cs#L44

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


Modal behind backdrop

bootstrap-less has the modal z-index equals to 1040. In new versions of bootstrap the z-index is 1050.
Adminlte is using two version of bootstrap, the less and the sass. The sass version has a z-index for the backdrop (1040). This causes an issue where the backdrop is above the modal.
If we remove the bootstrap import in the resources/assets/sass/app.scss the problem is solved.

i have a same problem

Could you please popose a PR?

No problem, give me some time ;)

Thanks!

Done https://github.com/acacha/adminlte-laravel/pull/140


Sandbox-exec

Hello,
After installing and configuring this library on a project when I try to run find the next issue:
'Processing node_modules failed. Error: Command sandbox-pod failed with exit code 71 Error output:
sandbox-exec<EMAIL_ADDRESS>Operation not permitted'
What I tried to solve this is:

Change sandbox's permission (Same issue)
I read about change Ruby version to 2.1.1 or bellow but Cocoapods require 2.2.2 or further

How can I solve this?

Did you try this http://stackoverflow.com/a/567105

Yes, doesn't work

If another have this error just disable the sand-box at the config file in your NativeScript Folder


downloadUrl in Task.Base.Linux.Commands entries requires a relative URL

This requirement is not spelled out anywhere and means that any use of things like {{ api.templates }} in one of these causes it to fail in a way that's highly non-obvious, being that the file you end up with contains:
Cannot GET <the absolute URL you specified>

On top of that, the on-http logs contain 404 errors with correct-looking URLs.
As a kicker, the entry is named downloadUrl where it probably should be downloadUri for consistency with everything else.

Take an example below, are you suggesting to rename "downloadUrl" to "downloadUri" ? Add @RackHD/corecommitters for attention.
module.exports = {
"friendlyName": "Arista Switch ZTP Discovery",
"injectableName": "Graph.Switch.Discovery.Arista.Ztp",
"tasks": [
{
"label": "catalog-switch",
"taskDefinition": {
"friendlyName": "Catalog Arista Switch",
"injectableName": "Task.Inline.Catalog.Switch.Arista",
"implementsTask": "Task.Base.Linux.Commands",
"options": {
"commands": [
{
"downloadUrl": "/api/1.1/templates/arista-catalog-version.py",
"catalog": { "format": "json", "source": "version" }
}
]
},
"properties": {}
}
}
]
};

@jeamland It looks like this is documented in the schema for the linux command task here: https://github.com/RackHD/on-tasks/blob/4cfd6a6efe141a83c27095a5eb6cadd7e5602c47/lib/task-data/schemas/linux-command.json#L44.  @leoyjchang Are we publishing task documentation anywhere yet.  The property name is definitely incorrect and misleading.  It should definitely be changed to downloadUri.  Do you have on-http logs available with the 404 errors you mentioned?

@brianparry No, current task document will only be generated by user in local environment. But we do have plan to provide an online/live version which is integrated with readthedoc and keep synced with RackHD latest repos. If the renaming does make sense, I will create a story in Maglev backlog. For 404 errors, @jeamland , do you happen to have on-http logs available?

@lanchongyizu is working on two PRs, that need to change the downloadUrl to fullpath (not relative path), so we can keep the name.

https://github.com/RackHD/on-skupack/pull/57
https://github.com/RackHD/on-taskgraph/pull/187


Adds Video Surfer to autogen_ext

This pull request introduces a new VideoSurferAgent and its associated tools to the autogen-ext package, while also making some adjustments to existing agents. The main changes include the addition of the VideoSurferAgent, updates to the MultimodalWebSurfer import paths, and the removal of the MultimodalWebSurfer from the __init__.py of agents.
New VideoSurferAgent and tools:

python/packages/autogen-ext/pyproject.toml: Added dependencies for the new VideoSurferAgent.
python/packages/autogen-ext/src/autogen_ext/agents/video_surfer/__init__.py: Introduced the VideoSurferAgent and its tools submodule.
python/packages/autogen-ext/src/autogen_ext/agents/video_surfer/_video_surfer.py: Implemented the VideoSurferAgent class, which can answer questions about local video files using various tools.
python/packages/autogen-ext/src/autogen_ext/agents/video_surfer/tools.py: Added tools for the VideoSurferAgent to extract audio, transcribe audio, get video length, save screenshots, and transcribe video screenshots using OpenAI API.

Adjustments to existing agents:

python/packages/autogen-ext/src/autogen_ext/agents/__init__.py: Removed the MultimodalWebSurfer import and adjusted the __all__ list accordingly.
python/packages/autogen-ext/src/autogen_ext/agents/web_surfer/__init__.py: Reintroduced the MultimodalWebSurfer in its own __init__.py.
python/packages/autogen-studio/autogenstudio/database/component_factory.py: Updated the import path for MultimodalWebSurfer.

@victordibia, FYI there is a small change in studio's component_factory.py in this PR, in particular the import.


Tracing inside actions

We want the ability to have insights. Some requirements:

Opentel compatible
Log arbitrary metadata
Recursive
Reactive on the hooks side
Sync + async friendly

API:
import asyncio
from typing import Tuple
import time

from burr.core import ApplicationBuilder, State, action, default
from burr.visibility import Tracer

@action(reads=["number"], writes=["number"])
async def test_async_action(state: State, __tracer: Tracer) -> Tuple[dict, State]:
    with __tracer("compute_number"):
        number = state["number"] + 1

    async with __tracer("sleep_async") as tracer:
        tracer.log_artifact("number", number)
        tracer.log_artifact("some_key", {"hello": "world"})
        await asyncio.sleep(1)

    with __tracer("sleep_sync"):
        with __tracer("nested_sleep"):
            await asyncio.sleep(1)
        await asyncio.sleep(1)

    return {"number": number}, state.update(number=number)

@action(reads=["number"], writes=["number"])
def test_action(state: State, __tracer: Tracer) -> Tuple[dict, State]:
    with __tracer("compute_number"):
        number = state["number"] + 1

    with __tracer("sleep"):
        time.sleep(1)
    return {"number": number}, state.update(number=number)

async def main():
    application = (
        ApplicationBuilder()
        .with_tracker("test:tracer").with_actions(
            test_action=test_action,
            test_async_action=test_async_action,
        )
        .with_transitions(
            ("test_action", "test_async_action", default),
            ("test_async_action", "test_action", default),
        ).with_state(number=0)
        .with_entrypoint("test_action")
        .build()
    )
    await application.arun(halt_after=["test_async_action"])

if __name__ == "__main__":
    asyncio.run(main())

This is implemented by #50


support unix domain socket of type SOCK_DGRAM (UDP) ?

Unix domain is provided through uv_pipe in libuv, and uv_pipe_t is a subclass of uv_stream_t.
Then how can I implement unix domain socket based on DGRAM(UDP) ?
I found some advise to create the socket manually and pass it to uv_pipe_open. However, when I try this way, uv_pipe_bind return EINVAL. If I bind the socket myself without uv_pipe_bind, then uv_listen return ENOTSUP.
Anyone could give an example ?

Is https://github.com/bnoordhuis/node-unix-dgram/blob/master/src/unix_dgram.cc the example you are looking for?

Thanks a lot.
The example is exactly what I need~
It comes from nodejs unix dgram module, what a good idea.


Create tac table migration

Description
Creates an empty TAC (transportation accounting codes) table for storing the TAC.
Reviewer Notes
Does the comment seem adequate for a description of tac field?  Do I also need a description of the table?  How are the naming of table and field?
I also assumed btree was the right type of index based on other index migrations- is that correct?
Setup
make  db_dev_migrate
view the table in a db viewer or find the table in local db from within scripts/psql-dev
Code Review Verification Steps

Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 09ff1fb9a037165522784e25a4b965573f325fb8


Preview does not retrieve new data

I have preview active with GatsbyCloud and WPEngine as backend host. Builds are fast (8-30 seconds) and previews builds are a bit slower (30 - 90 seconds). When I edit a post in the WP backend and hit "Preview Changes", a page appears but doesn't show an updated but the old page. The builds work fine, but after looking in the raw logs, there are some errors.
Questions:

Why differ the builds in duration so significantly?
In the demo I saw, that the preview window updated automatically after a few seconds – unfortunately, this doesn't work with my setup. Any ideas why?
Any ideas, why the new data isn't appearing in the preview? Is this related to the non-nunllable fields (see log below)?

What I tried and didn't work:

I waited until the preview build was done on GatsbyCloud, refreshed the preview page.
I cleared the cache (first regular build, then preview build) and reinitiated the preview build.
I tried different post-types, none worked.

gatsby-config:
{
      resolve: `gatsby-source-wordpress-experimental`,
      options: {
        url: `https://eham.wpengine.com/graphql`,
        verbose: true,
        schema: {
          queryDepth: 5,
          typePrefix: `Wp`,
          timeout: 30000,
        },
        develop: {
          nodeUpdateInterval: 3000,
          hardCacheMediaFiles: true,
        },
        debug: {
          graphql: {
            writeQueriesToDisk: true,
            showQueryOnError: true,
            showQueryVarsOnError: true,
            copyQueryOnError: true,
            panicOnError: false,
            onlyReportCriticalErrors: true,
          },
        },
        excludeFieldNames: [`blocksJSON`, `saveContent`],
        type: {
          Post: {
            limit: process.env.NODE_ENV === `development` ? 35 : null,
          },
          Project: {
            limit: process.env.NODE_ENV === `development` ? 35 : null,
          },
          Employee: {
            limit: process.env.NODE_ENV === `development` ? 35 : null,
          },
          CoreParagraphBlockAttributes: {
            exclude: true,
          },
          CoreParagraphBlockAttributesV2: {
            exclude: true,
          },
          CorePullquoteBlockAttributes: {
            exclude: true,
          },
          User: {
            exclude: true,
          },
          UserRole: {
            exclude: true,
          },
          Comment: {
            exclude: true,
          },
          PostFormat: {
            exclude: true,
          },
          Tag: {
            exclude: true,
          },
        },
      },
    },

Raw log of the latest preview build:
17:33:33 PM:
success  gatsby-source-wordpress  diff schemas - 1.126s

17:33:34 PM:
success  gatsby-source-wordpress  ingest WPGraphQL schema - 1.983s

17:33:35 PM:
success createSchemaCustomization - 2.423s

17:33:36 PM:

17:33:36 PM:
info  gatsby-source-wordpress  Preview for project 7240 was updated at /en/projects/alpine-style-meets-modern-clarity/.

17:33:36 PM:

17:33:37 PM:
success Checking for changed pages - 0.010s

17:33:37 PM:
success source and transform nodes - 2.007s

17:33:38 PM:
info Total nodes: 2467, SitePage nodes: 126 (use --verbose for breakdown)

17:33:38 PM:
success createPages - 1.250s

17:33:38 PM:
success Checking for changed pages - 0.145s

17:33:39 PM:
success update schema - 1.155s

17:33:39 PM:
success onPreExtractQueries - 0.021s

17:33:40 PM:
success extract queries from components - 1.067s

17:33:41 PM:
success write out requires - 0.008s

17:33:41 PM:
success run static queries - 0.863s - 1/1 1.16/s

17:33:51 PM:
ERROR There was an error in your GraphQL query:

Cannot return null for non-nullable field WpProject.uri.

The field "WpProject.uri." was explicitly defined as non-nullable via the schema customization API (by yourself or a plugin/theme). This means that this field is not optional and you have to define a value. If this is not your desired behavior and you defined the schema yourself, go to "createTypes" in gatsby-node.js. If you're using a plugin/theme, you can learn more here on how to fix field types:
https://www.gatsbyjs.org/docs/schema-customization/#fixing-field-types

  556 | }
  557 |
  558 | fragment PageFeaturedContentSectionFragment on WpPage_Flex_Content_FeaturedContentSection {
  559 |   sectionHeadline
  560 |   items {
  561 |     text
  562 |     internallLink {
  563 |       __typename
  564 |       ... on WpContentNode {
  565 |         __typename
> 566 |         uri
      |         ^
  567 |       }
  568 |     }
  569 |     linkTarget
  570 |     linkText
  571 |     image {
  572 |       sourceUrl
  573 |       mediaDetails {
  574 |         height
  575 |         width
  576 |       }

File path: /usr/src/app/www/src/templates/Page.jsx
Url path: /
Plugin: none
ERROR There was an error in your GraphQL query:

Cannot return null for non-nullable field WpPost.uri.

The field "WpPost.uri." was explicitly defined as non-nullable via the schema customization API (by yourself or a plugin/theme). This means that this field is not optional and you have to define a value. If this is not your desired behavior and you defined the schema yourself, go to "createTypes" in gatsby-node.js. If you're using a plugin/theme, you can learn more here on how to fix field types:
https://www.gatsbyjs.org/docs/schema-customization/#fixing-field-types

  556 | }
  557 |
  558 | fragment PageFeaturedContentSectionFragment on WpPage_Flex_Content_FeaturedContentSection {
  559 |   sectionHeadline
  560 |   items {
  561 |     text
  562 |     internallLink {
  563 |       __typename
  564 |       ... on WpContentNode {
  565 |         __typename
> 566 |         uri
      |         ^
  567 |       }
  568 |     }
  569 |     linkTarget
  570 |     linkText
  571 |     image {
  572 |       sourceUrl
  573 |       mediaDetails {
  574 |         height
  575 |         width
  576 |       }

File path: /usr/src/app/www/src/templates/Page.jsx
Url path: /
Plugin: none

17:33:52 PM:
ERROR There was an error in your GraphQL query:

Cannot return null for non-nullable field WpPost.uri.

The field "WpPost.uri." was explicitly defined as non-nullable via the schema customization API (by yourself or a plugin/theme). This means that this field is not optional and you have to define a value. If this is not your desired behavior and you defined the schema yourself, go to "createTypes" in gatsby-node.js. If you're using a plugin/theme, you can learn more here on how to fix field types:
https://www.gatsbyjs.org/docs/schema-customization/#fixing-field-types

  100 | fragment CompetenceServicesTeaserSectionFragment on WpCompetence_Flex_Content_ServicesTeaserSection {
  101 |   headline
  102 |   teaserHeadline
  103 |   teaserText
  104 |   teaserLinkText
  105 |   teaserLinkTarget
  106 |   teaserInternalLink {
  107 |     __typename
  108 |     ... on WpContentNode {
  109 |       __typename
> 110 |       uri
      |       ^
  111 |     }
  112 |   }
  113 |   fade
  114 |   images {
  115 |     imageCaption
  116 |     image {
  117 |       sourceUrl
  118 |       mediaDetails {
  119 |         height
  120 |         width

File path: /usr/src/app/www/src/templates/Competence.jsx
Url path: /kompetenzen/innenarchitektur
Plugin: none

17:33:52 PM:
ERROR There was an error in your GraphQL query:

Cannot return null for non-nullable field WpProject.uri.

The field "WpProject.uri." was explicitly defined as non-nullable via the schema customization API (by yourself or a plugin/theme). This means that this field is not optional and you have to define a value. If this is not your desired behavior and you defined the schema yourself, go to "createTypes" in gatsby-node.js. If you're using a plugin/theme, you can learn more here on how to fix field types:
https://www.gatsbyjs.org/docs/schema-customization/#fixing-field-types

  336 |
  337 | fragment CompetenceProgressSliderSectionFragement on WpCompetence_Flex_Content_ProgressSliderSection {
  338 |   headline
  339 |   teaserHeadline
  340 |   teaserText
  341 |   teaserLink
  342 |   teaserInternalLink {
  343 |     __typename
  344 |     ... on WpContentNode {
  345 |       __typename
> 346 |       uri
      |       ^
  347 |     }
  348 |   }
  349 |   teaserLinkText
  350 |   teaserBild {
  351 |     sourceUrl
  352 |     mediaDetails {
  353 |       height
  354 |       width
  355 |     }
  356 |     altText

File path: /usr/src/app/www/src/templates/Competence.jsx
Url path: /kompetenzen/innenarchitektur
Plugin: none
ERROR There was an error in your GraphQL query:

Cannot return null for non-nullable field WpProject.uri.

The field "WpProject.uri." was explicitly defined as non-nullable via the schema customization API (by yourself or a plugin/theme). This means that this field is not optional and you have to define a value. If this is not your desired behavior and you defined the schema yourself, go to "createTypes" in gatsby-node.js. If you're using a plugin/theme, you can learn more here on how to fix field types:
https://www.gatsbyjs.org/docs/schema-customization/#fixing-field-types

  336 |
  337 | fragment CompetenceProgressSliderSectionFragement on WpCompetence_Flex_Content_ProgressSliderSection {
  338 |   headline
  339 |   teaserHeadline
  340 |   teaserText
  341 |   teaserLink
  342 |   teaserInternalLink {
  343 |     __typename
  344 |     ... on WpContentNode {
  345 |       __typename
> 346 |       uri
      |       ^
  347 |     }
  348 |   }
  349 |   teaserLinkText
  350 |   teaserBild {
  351 |     sourceUrl
  352 |     mediaDetails {
  353 |       height
  354 |       width
  355 |     }
  356 |     altText

File path: /usr/src/app/www/src/templates/Competence.jsx
Url path: /kompetenzen/innenarchitektur
Plugin: none

Setup

WordPress hosted on WPEngine backend connected to GatsbyCloud
Custom Post Type UI 1.8.1 for the creation of the custom post types
WPGatsby v. 0.5.4
WPGraphQL v. 0.14.0
WPGraphQL for Advanced Custom Fields v. 0.3.5
WP GraphQL Polylang v. 0.5.0
gatsby-source-wordpress-experimental v. ^2.0.1

Hi @MobyDigg , the current preview experience is a little lacking. This is due to loading state and error handling across remote services not being fully handled. I have a new Preview PR here https://github.com/gatsbyjs/gatsby-source-wordpress-experimental/pull/252 which we're working on merging this week 🤞 Def follow that because it handles the remote state management which has been a big problem with the current preview implementation. In addition it fixes a bunch of cases where previews wouldn't be sourced into Gatsby properly.

Also, if you're interested you can test out the latest WP Preview by using<EMAIL_ADDRESS>and installing this branch of WPGatsby in your WP instance https://github.com/gatsbyjs/wp-gatsby/pull/50

Hi @MobyDigg , the current preview experience is a little lacking. This is due to loading state and error handling across remote services not being fully handled. I have a new Preview PR here #252 which we're working on merging this week 🤞 Def follow that because it handles the remote state management which has been a big problem with the current preview implementation. In addition it fixes a bunch of cases where previews wouldn't be sourced into Gatsby properly.

Hi @TylerBarnes, thanks! That sounds promising! I am definitely looking forward to see that, as it will be a HUGE addition to the usability of the stack.

Also, if you're interested you can test out the latest WP Preview by using<EMAIL_ADDRESS>and installing this branch of WPGatsby in your WP instance gatsbyjs/wp-gatsby#50

I will test that today! Thanks again.

Hi again @MobyDigg , I had a closer look and the errors here are related to the limit plugin type option. Follow https://github.com/gatsbyjs/gatsby-source-wordpress-experimental/issues/192 for news on when this will no longer be an issue.
Thanks!


token file does not exist of file

Hello! I am newbie to bingo, how to make it work? I am try to use it in vim 8 with "vim-go" and "LanguageClient-neovim" and most actions always get:
[LC] Error: Failure { jsonrpc: Some(V2), error: Error { code: InternalError, message: "token file does not exist of file:///home/ecx/go_test/hello-world.go", data: None }, id: Num(2) }
vim's embedded omni completion works good, but it is not a work of bingo. Can you explain, please, what token files it talking about?

@ecx
you should use go module project
Please set bingo's flag "--trace", "--logfile=/tmp/bingo.log", and show me the full /tmp/bingo.log from bingo startup.
thanks

langserver-go: reading on stdin, writing on stdout   --> request #1: initialize: {"capabilities":{"textDocument":{"colorProvider":null,"completion":{"completionItem":{"snippetSupport":false}},"signatur  eHelp":{"signatureInformation":{"parameterInformation":{"labelOffsetSupport":true}}}},"workspace":{"applyEdit":true,"didChangeWatchedFiles":{"dynami  cRegistration":true}}},"processId":27782,"rootPath":"/home/ecx/go_test","rootUri":"file:///home/ecx/go_test","trace":"off"}   Passing an initialize rootPath URI ("file:///home/ecx/go_test") is deprecated. Use rootUri instead.   <-- notif: window/logMessage: {"type":3,"message":"GOPATH: [/home/ecx/go], import path: "}   <-- notif: window/logMessage: {"type":3,"message":"GO111MODULE=auto, module mode"}   <-- notif: window/showMessage: {"type":3,"message":"load /home/ecx/go_test successfully! elapsed time: 0 seconds, cache: false, go module: false."}   <-- result #1: initialize: {"capabilities":{"textDocumentSync":2,"hoverProvider":true,"completionProvider":{"triggerCharacters":["."]},"signatureHel  pProvider":{"triggerCharacters":["(",","]},"definitionProvider":true,"typeDefinitionProvider":true,"referencesProvider":true,"documentSymbolProvider  ":true,"workspaceSymbolProvider":true,"implementationProvider":true,"documentFormattingProvider":true,"documentRangeFormattingProvider":true,"rename  Provider":true,"xworkspaceReferencesProvider":true,"xdefinitionProvider":true,"xworkspaceSymbolByProperties":true}}   --> notif: initialized: {}


Ability to reverse error - throw error if snapshots are the same and continue the test on different ones

In my app I actually need a functionality which is opposite to what cypress-image-diff does.
Instead of throwing error whenever snapshot is different from baseline snapshot I want to throw errors only when taken snapshot is THE SAME as baseline (with some testTreshold as well).
If baseline snapshot is different from comparison snapshot then test should continue normally.
How do I achieve that?

@Razthyae Hi, on top of my head there's no way around this at the moment unfortunately. Could you elaborate your case? Just to see if it's a solid use case for this feature


Ballutes

#rebuild

@HebaruSan Thanks for removing that extra vref.  But it didn't help.
I rebuilt the zip, reuploaded it, and it is still failing, says it can't find the path in the zip, but it is there, and there ARE files in the directory, so I'm stumped.  Any help would be appreciated

#rebuild

#release

#rebuild

Strange.  I changed the kref to use github and it worked.  Same file

It's down to find vs file in the install stanza. find matches subdirectories while file only matches full paths, so file would need to be set to GameData/KermangeddonIndustries to install this mod.
The commit history in this PR is confusing, though. It looks like my commit changed it back to file, but I definitely did not do that. Either way, the commits containing find have a green checkmark, and the ones with file have a red X.


Feat: Add helper functions for making syscalls

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind tests

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area commands

/area pkg

/area events

What this PR does / why we need it:
Which issue(s) this PR fixes:
Part of GSoC 2024 task

Fixes #
Special notes for your reviewer:
cc @alacuku @jasondellaluce

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: GLVSKiriti
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku, GLVSKiriti
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: 64793f26d127f3fd9021123085b92e010d29c686

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku, GLVSKiriti, LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku, GLVSKiriti, LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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


Docker.#Build fails when chaining actions in a plan

What is the issue?
I'm trying to build a plan like this
dagger.#Plan & {
  actions: {
    linux: {
      build: docker.#Build & {
     ....
   },
       updated: {
      build: docker.#Build & {
        steps: [
          linux.build,
     ...
   },
    box: {
      build: docker.#Build & {
        steps: [
          updated.build,
     ....
   }
}

and fails with this error
$ dagger do box build -p image_output.issue.cue
9:00AM FTL failed to load plan: actions.box.build._dag."0"._dag."0": structural cycle

It's look like related to the private field _dag, because I can work it around using the first step as follows:
....
    box: {
      build: docker.#Build & {
        steps: [
          {
            output: updated.build.output
          },
      ...
    }

Log output
9:00AM FTL failed to load plan: actions.box.build._dag."0"._dag."0": structural cycle
Steps to reproduce

git clone https://github.com/sebasjm/dagger-testing
cd dagger-testing
dagger project init
dagger do box build -p image_output.issue.cue

Dagger version
dagger devel (64cdadb8) linux/amd64
OS version
Ubuntu 18.04.5 LTS

Duplicate of #1466, which is about nesting instead of chaining but the underlying issue is the same (I think).
Please reopen if not the case.


pprof: 2017-08-28 -> 2018-08-15

Motivation for this change
Bump to a more recent version, also renaming it pprof-unstable as there is no tag.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build pprof

Success on aarch64-linux (full log)
Attempted: pprof
Partial log (click to expand)
shrinking /nix/store/7cld36lig8y7n4vas9pxfjl9v5g1ws58-pprof-unstable-2018-08-15-bin/bin/sample
cannot find section .dynamic
shrinking /nix/store/7cld36lig8y7n4vas9pxfjl9v5g1ws58-pprof-unstable-2018-08-15-bin/bin/pprof
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/7cld36lig8y7n4vas9pxfjl9v5g1ws58-pprof-unstable-2018-08-15-bin/bin
patching script interpreter paths in /nix/store/7cld36lig8y7n4vas9pxfjl9v5g1ws58-pprof-unstable-2018-08-15-bin
checking for references to /build in /nix/store/7cld36lig8y7n4vas9pxfjl9v5g1ws58-pprof-unstable-2018-08-15-bin...
cannot find section .dynamic
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
/nix/store/7cld36lig8y7n4vas9pxfjl9v5g1ws58-pprof-unstable-2018-08-15-bin

Success on x86_64-linux (full log)
Attempted: pprof
Partial log (click to expand)
shrinking /nix/store/fcac0qlgx4avql5dmjalzr1my05qn90p-pprof-unstable-2018-08-15-bin/bin/pprof
shrinking /nix/store/fcac0qlgx4avql5dmjalzr1my05qn90p-pprof-unstable-2018-08-15-bin/bin/sample
cannot find section .dynamic
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/fcac0qlgx4avql5dmjalzr1my05qn90p-pprof-unstable-2018-08-15-bin/bin
patching script interpreter paths in /nix/store/fcac0qlgx4avql5dmjalzr1my05qn90p-pprof-unstable-2018-08-15-bin
checking for references to /build in /nix/store/fcac0qlgx4avql5dmjalzr1my05qn90p-pprof-unstable-2018-08-15-bin...
cannot find section .dynamic
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
/nix/store/fcac0qlgx4avql5dmjalzr1my05qn90p-pprof-unstable-2018-08-15-bin

Success on x86_64-darwin (full log)
Attempted: pprof
Partial log (click to expand)
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
find: '/nix/store/rchh3fvxrdd7gzaq5bsif831ff09lzyp-pprof-unstable-2018-08-15': No such file or directory
/nix/store/g54j2d7fziv264sgyxshkr2mhm36q8r9-pprof-unstable-2018-08-15-bin


vizbuilder is operating over the paginated view and not the full query

also this: https://datasaudi-nightly.datawheel.us/en/data-explorer?cube=gastat_fdi_net_inflow_by_country&drilldowns[0]=Year&drilldowns[1]=Countries.Continent&locale=en&measures[0]=Million+SAR&panel=vizbuilder


Added Scaling Prometheus with Thanos Blog from CloudRaft

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Added Thanos Blog by CloudRaft https://www.cloudraft.io/blog/scaling-prometheus-with-thanos

Verification

Lgtm can you sign commit?

Done

Thanks @riteshsonawane1372


CodexLib Out of Date

I pulled down the most recent version of CodexLib.dll, and noticed a few problems after compiling and trying to run Wrath
I would get these errors:
[KineticistElementsExpanded] [Exception] MissingMethodException - Kingmaker.UnitLogic.Abilities.Components.AbilityShowIfCasterHasFact CodexLib.Helper.CreateAbilityShowIfCasterHasFact(CodexLib.AnyRef,bool)
System.MissingMethodException: Kingmaker.UnitLogic.Abilities.Components.AbilityShowIfCasterHasFact CodexLib.Helper.CreateAbilityShowIfCasterHasFact(CodexLib.AnyRef,bool)
  at KineticistElementsExpanded.ElementAether.Aether.CreateTelekineticBlast () [0x0001f] in C:\Users\Spenc\source\repos\Kineticist Element - Aether\ElementAether\Aether.cs:383
  at KineticistElementsExpanded.ElementAether.Aether.Configure () [0x0000b] in C:\Users\Spenc\source\repos\Kineticist Element - Aether\ElementAether\Aether.cs:56
  at KineticistElementsExpanded.StartGameLoader_LoadAllJson.LoadSafe (System.Action action) [0x00041] in C:\Users\Spenc\source\repos\Kineticist Element - Aether\Main.cs:171

(Received this error for CreateAbilityShowIfCasterHasFact and CreateAddClassSkill)
When I recreated the Helper.CreateAddClassSkill in my own project, and called it, I had no issues, and it went on to generate the same error at CreateAbilityShowIfCasterHasFact, so I believe the issue may be that CodexLib isn't up-to-date regarding the most recent game update, but I am uncertain
Any tips?

Just to note: https://github.com/SpencerMycek/KineticistExpandedElements
The "Wood" branch is up to date with my changes, and is the current commit where I am receiving these errors

You forgot to include the new CodexLib.dll in your mod folder.
Since your compiler had no error with this call, your reference is working. But obviously the mod will need the same DLL to run.
I recommend automating this with build events and macros. Otherwise you might forget to check.

I added automation to make sure CodexLib.dll will be put into the mod folder when I build, and the same error's persisted when trying to start Wrath

Well, you can only get that exception, if the lib you build against and the lib executed are different.
There could also be a version issue. E.g. the latest version does not have the highest number. Or the assembly load function loads the wrong version. I made sure to always increment my version. Can't tell why that would be an issue.
I will double check later. Maybe check the file properties.

Would a solution be to Build CodexLib on my own system? So it can reference the correct assemblies on my machine?

I believe I found/understand the issue, although not what's causing it.
Somehow, my project is loading the assembly "DarkCodex" as well as "CodexLib" (printed out all loaded assemblies in the AppDomain), where CodexLib was version <IP_ADDRESS>
I have the most recent release of DarkCodex, which has CodexLib of <IP_ADDRESS> instead of <IP_ADDRESS> (the most recent, and the file present in my mod folder)
My project has been loading the assembly in the DarkCodex mod folder somehow.
I fixed this temporarily by overwriting the CodexLib.dll in the DarkCodex mod folder, but there is still the bigger issue at play
Related logs showing "Fixed" loaded assemblies
[KineticistElementsExpanded] WOTRDamageBreakdown, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[KineticistElementsExpanded] **DarkCodex**, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[KineticistElementsExpanded] **CodexLib**, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[KineticistElementsExpanded] DataViewer, Version=<IP_ADDRESS>33, Culture=neutral, PublicKeyToken=null
[KineticistElementsExpanded] Microsoft.GeneratedCode, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[KineticistElementsExpanded] RespecWrath, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[KineticistElementsExpanded] KineticistElementsExpanded, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null

Sorry for the wait. Was updating my projects files to Sdk="Microsoft.NET.Sdk" and encountered some issues.
I still suspect that you are missing the updated CodexLib in your mod folder. If it is there and it still doesn't load it, then it's the fault of the method EnsureCodexLib(), which is already explained here https://github.com/Truinto/DarkCodex/tree/master/CodexLib
I have done some tests with an empty mod and it seemed to load the correct file for me, no matter how I arranged it. So I can't really figure out what's going on. Can you perhaps push your source online, so I can have a look?

Absolutely, https://github.com/SpencerMycek/KineticistExpandedElements, check the "wood" branch
I believe the issue is somehow related to my project loading "DarkCodex" as an assembly, which loads CodexLib at v<IP_ADDRESS>, and therefore it doesn't need to load the CodexLib in my mod folder

As far as I can tell do you not have references to DarkCodex. If you have, just don't.
Your csproj line 43 and line 114 have different macros defined. I think the path must be the same there.

So, I did a large cleanup of my csproj file
And I did some checking and realized: the AppDomain that is being checked is for the entirety of the unity project (named Unity Root Domain), so if DarkCodex loads before KineticistElementsExpanded (Which is does for me), and has a different version of CodexLib (latest release, last I checked, is <IP_ADDRESS>, KEE is using <IP_ADDRESS>), then the newer version is not loaded, which caused these issues, I believe

Okay, I get this bug too. But I haven't figured out yet why it happens. I am sure it has worked in the past.

Okay, apparently the parent directory of Mods\DarkCodex\ is Mods\DarkCodex.
Which means it only searches that specific mods folders. Which wasn't the intention.
Will fix.

You will need to update your copy of EnsureCodexLib() as well. Sorry for the inconveniences.


Import your SAP Build project into SAP Web IDE

Tutorial URL: https://developers.sap.com/tutorials/build-import-webide.html
In Step 8: Run your application:
Mock data is not deployed with demo application when following tutorial.
The documentation refers to testFLPService.html instead of testFLPServiceMockServer.html

The automated BUILD import to Web IDE is no longer supported.


Fivegr so driver fix parsing

Added Performance logs to track the actions performed.
Improved parsing to avoid exceptions when descriptors are sent with parameters that have not been taken into consideration by the parser.

Everything OK. I'm going to merge to development.
At the beginning of next week I'll release a v1.5.1 to include these fixes in master.

Great! Thank you Mateo


Javascript Minify : problem with comments

Hi ! I found an issue with the minify command when there is a comment at the bottom of js files.
/media/js/test1.js
//TEST1
alert('TEST 1');
//END TEST1

/media/js/test2.js
//TEST2
alert('TEST 2');
//END TEST2

Combine without Minify : OK
<script type="text/javascript" src="/munee.php?files=/media/js/test1.js,/media/js/test2.js" defer></script>

//TEST1
alert('TEST 1');
//END TEST1
//TEST2
alert('TEST 2');
//END TEST2

Combine & Minify : NOT OK
<script type="text/javascript" src="/munee.php?files=/media/js/test1.js,/media/js/test2.js&minify=true" defer></script>

alert('TEST 1');END TEST1

alert('TEST 2');END TEST2

I just tried this and had no issues.  I've recently updated the jsmin library to it's latest version. It could have been an issue in the previous version.  Thanks for the report!


add nextjs to project setup

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here

:star: PR built and available in a preview environment drw89-test-domi-pr-1 here


Modified mega.py to handle new file url.

Fixed Url key missing error when new(maybe) URL is used
example: https://mega.nz/file/DK5jWSAL#ZiN2zLKRrweGwo9pqXT1X4fje19HGYO0p9D8DfACtic.
Fixed issue in windows where shutil.move() was called before closing the file which causes permission error.

Fixed in https://github.com/odwyersoftware/mega.py/pull/8, thanks


[SPARK-12003] [SQL] remove the prefix for name after expanded star

Right now, the expended start will include the name of expression as prefix for column, that's not better than without expending, we should not have the prefix.

Test build #46725 has started for PR 9984 at commit c68c682.

Test build #46725 has finished for PR 9984 at commit c68c682.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46725/
Test PASSed.


Added The Little Ecto Cookbook

What does this PR do?
Add resource to The Little Ecto Cookbook
For resources
Description
This book is a curated collection of the Ecto guides on how to best use different features of Ecto 3.7 and later. This free ebook is made of 13 chapters.

Why is this valuable (or not)?
How do we know it's really free?
You should add your e-mail but you can skip mailing from Dashbit - PDF will be on your e-mail account as link to PDF.
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[X] Read our contributing guidelines
[X] Search for duplicates.
[X] Include author(s) and platform where appropriate.
[X] Put lists in alphabetical order, correct spacing.
[X] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Quick Accent missing hungarian accent

Description of the new feature / enhancement
I would suggest one more accent to be added, used in Hungarian language on characters O and U as follows:
Ű
Ő
Scenario when this would be used?
I'm working for a company in Hungary, but mostly in german language. Every now and then, especially when writing hungarian names, I would need to write the mentioned characters. Switching the installed keyboard isn't always convenient in that case, only for a single letter.
Supporting information
No response

can work on this

ping @damienleroy

Adding to #20798


Can't correctly mount a directory contains a file using non-ascii characters.

When we mount a directory to docker-machine, when that directory contains at least one file which has non-ascii characters in its name, we partially or entire files are not visible though the volume.
Tested on next environment.
$ sw_vers -productVersion
10.10.5
$ VBoxManage --version
5.0.0r101573
$ docker-machine --version
docker-machine version 0.4.1 (HEAD)

Repro step
On host OS X, run next command.
$ mkdir test
$ cd test
$ touch "🐱"
$ cat > Dockerfile
FROM busybox
VOLUME /test
ENTRYPOINT ["ls", "-al", "/test"]
$ docker build -t test .
$ docker run --rm -v `pwd`:/test test

Expected behavior
Shows Dockerfile and 🐱
Actual behavior
Only shows Dockerfile
total 8
drwxr-xr-x    1 default  staff          136 Aug 26  2015 .
drwxr-xr-x   24 root     root          4096 Aug 26 06:11 ..
-rw-r--r--    1 default  staff           60 Aug 26  2015 Dockerfile

Looks like we're not mounting directory with iocharset=utf8 which prevents us to see these files from the container.

Original conversation on Twitter.
https://twitter.com/upthecyberpunks/status/636468275308728320
cc: @nathanleclaire

Confirmed that next command solve the issue (Assuming that docker-machine name is dev)
$ docker-machine ssh dev
$ sudo umount /Users
$ sudo mount -t vboxsf -o uid=1000,gid=50,iocharset=utf8 Users /Users

Then,
$ docker run --rm -v `pwd`:/test test
total 8
drwxr-xr-x    1 default  staff          136 Aug 26 11:39 .
drwxr-xr-x   24 root     root          4096 Aug 26 11:39 ..
-rw-r--r--    1 default  staff           60 Aug 26 11:39 Dockerfile
-rw-r--r--    1 default  staff            0 Aug 26 10:09 ????

Note that "🐱" shows as "????" but it is because of busybox ls, it's correctly displayed as "🐱" when using FROM ubuntu.
So, possible fix might be updating next init script to give iocharset=utf8 by default.
https://github.com/boot2docker/boot2docker/blob/master/rootfs/rootfs/etc/rc.d/automount-shares#L11

Next pull request, which is not updated since May, because of unnecessary mount.vboxsf line change, can fix this issue.
boot2docker/boot2docker#697

Yeah looks like that PR is waiting on a change.  Perhaps we could carry that PR if the contributor doesnt' update soon.

I made a new pull request referenced from here, boot2docker/boot2docker#1044 which addressed the concerns on the original pull request but not updated.
Hopefully that will be merged soon.

boot2docker/boot2docker#697 is merged now, so probably this issue will be as well.

Nice work @niw !


Update data.json

resolve #492

Sure ! Is the issue resolved now
On Wed, May 12, 2021 at 1:37 AM Tom Schmelzer @.***>
wrote:

And could you add resolve #yourissuenumber to the description thus the
issue automatically gets closed on pr merge 🙈 this would be great
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/awesome-github-profiles/pull/492#issuecomment-839099804,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AML7AVIC3GQAWHYSCEMWUWLTNGE6XANCNFSM44VLBQYQ
.

--
R.M.K. Engineering College - Mail system

please also see my suggestion above (https://github.com/EddieHubCommunity/awesome-github-profiles/pull/492#discussion_r630135554)

Is it fine now
On Wed, May 12, 2021 at 2:10 PM Tom Schmelzer @.***>
wrote:

please also see my suggestion above (#492 (comment)
https://github.com/EddieHubCommunity/awesome-github-profiles/pull/492#discussion_r630135554
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/awesome-github-profiles/pull/492#issuecomment-839585247,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AML7AVMSW4J3IJJMATP4K7TTNI5IPANCNFSM44VLBQYQ
.

--
R.M.K. Engineering College - Mail system

Is it fine now

Unfortunately not take a look at line

Updated , Is it ok ?
On Wed, May 12, 2021 at 2:38 PM Tom Schmelzer @.***>
wrote:

Is it fine now
Unfortunately not take a look at line 872
[image: Screenshot_20210512_110730.jpg]
https://user-images.githubusercontent.com/30869493/117949346-53b3af00-b312-11eb-88dd-cd381ee84791.jpg
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/awesome-github-profiles/pull/492#issuecomment-839605026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AML7AVO2HWDKMNHRQSUXRY3TNJARZANCNFSM44VLBQYQ
.

--
R.M.K. Engineering College - Mail system

@suprajaarthi you have to commit your changes to patch-1 or you can directly accept the suggestion from me above

Updated , Is it ok ?
You can accept the changes or commit it

@suprajaarthi you have to commit your changes to patch-1 or you can directly accept the suggestion from me above

You're fast again xD

Committed :)
On Wed, May 12, 2021 at 4:13 PM Karuna Tata @.***>
wrote:

@suprajaarthi https://github.com/suprajaarthi you have to commit your
changes to patch-1 or you can directly accept the suggestion from me above
You're fast again xD
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/awesome-github-profiles/pull/492#issuecomment-839670159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AML7AVM6USCSFMRLUACOBBDTNJLWLANCNFSM44VLBQYQ
.

--
R.M.K. Engineering College - Mail system

Hey @suprajaarthi oh no now its little bit messed up. But no worries we will get to this 🚀
The Data.json should look like something like this. I've marked the lines green (with plus)
...
    {
    "name": "Priyanka Mishra",
    "githubUsername": "priyaankamishra",
    "imageUrl": "https://user-images.githubusercontent.com/71710239/117578918-55148a00-b10e-11eb-93f4-8d2bd01dcd8c.png",
    "issueNumber": 487
+  },
+  {
+    "name": "Supraja Arthi",
+    "githubUsername": "suprajaarthi",
+    "imageUrl": "https://github.com/suprajaarthi/awesome-github-profiles/blob/main/git.jpeg",
+    "issueNumber": 490
  }
]

the ] has to be the last line in the file.

You can also take a look at your Forked GitHub repository. I've created a PR (https://github.com/suprajaarthi/awesome-github-profiles/pull/1) here where you can see the changes needed. You can merge it into you patch-1 branch or can dismiss it and make the changes by your own.

Updated the mentioned changes . Thanks
On Wed, May 12, 2021 at 4:57 PM Tom Schmelzer @.***>
wrote:

You can also take a look at your Forked GitHub repository. I've created a
PR (suprajaarthi#1
https://github.com/suprajaarthi/awesome-github-profiles/pull/1) here
where you can see the changes needed. You can merge it into you patch-1
branch or can dismiss it and make the changes by your own.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/awesome-github-profiles/pull/492#issuecomment-839695449,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AML7AVKC7RDMEMNJABHYAP3TNJQ27ANCNFSM44VLBQYQ
.

--
R.M.K. Engineering College - Mail system

Done👍
On Wed, May 12, 2021 at 6:41 PM Tom Schmelzer @.***>
wrote:

@.**** approved this pull request.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/awesome-github-profiles/pull/492#pullrequestreview-657864826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AML7AVO6IZMHMZUTBFPUJ63TNJ465ANCNFSM44VLBQYQ
.

--
R.M.K. Engineering College - Mail system


ModalManager leaves behind padding styles on Body aria hidden

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

ModalManager should not leave behind DOM changes when it is closed
Current Behavior 😯

Leaves behind padding-right: 0px if this condition evaluates to true
https://github.com/mui-org/material-ui/blob/bba51682748343a7010e3bc8306d5ad59fa95969/packages/material-ui/src/Modal/ModalManager.js#L75
Context 🔦

This was reproduced in the context of a unit test. It looks like the computed style needs to considered when restoring the styles, not JSS.
Your Environment 🌎

Tech
Version

Material-UI
v4.2.0

React
v16.8.6

Browser

TypeScript
v3.5.1

etc.

@neeschit Does the same happen in the browser?

I didn't check to see if it does. I'm guessing it does, I just didn't bother to check to reproduce once I figured out the cause of the issue


az command for adding a new Private Link configuration and associate it with an existing Frontend IP

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
--- az network application-gateway private-link add --frontend-ip  --name  --subnet --gateway-name  --resource-group  ---
Describe the bug
--- For using existing subnet, If I put name of a existing subnet into '--subnet' argument, it won't start the operation, and return 'subnet duplicates' error msg, however, it will work, if I put subnet resource ID.
To Reproduce
--- create a App GW with 2 subnets, one for app GW, and one for Private Link, then run command ---
Expected behavior
--- Users would think subnet name and subnet resource ID, both should work, and expecting better explanation with error msg---
Environment summary
--- Cloud shell / az CLI  ---
Additional context

Please provide detailed info in the description

updated. thx

@elshat128, I have a concern here:
If a Subnet already exists in the VNet of this Application Gateway, why should we add it again? It doesn't seem like a valid usage.

@elshat128, I have a concern here: If a Subnet already exists in the VNet of this Application Gateway, why should we add it again? It doesn't seem like a valid usage.

so this command is to add new one or Use/specify existing one, As we know App GW subnet and Bastion subnet can't be used for Private link, so in the case a user have a subnet other than these two, and want to use it for private link, this is the use case.

@elshat128, for this case, we don't need to prepare a subnet in advance. We can create the subnet during az network application-gateway private-link add, i.e., it's a new --subnet at that moment.

@necusjz what if user just happened to have a subnet handy which wasn't prepared in advance for this private link, and now they want to use this subnet, in this case during the command, users are potentially forced to create a new subnet, aren't they?

@elshat128, so true, but such design also avoids the problem of subnet reuse.

@necusjz that's right, I understand the design. because there is user posted questions about this on the internet, maybe return error msg can be more clear, let user know, in order to use existing subnet, please enter subnet resources ID. thanks.


🐛 Fix indent-in-infix-expression with parentheses

Overview

Fix indent-in-infix-expression with parentheses.

    return value === null
      || typeof value !== 'object'
      || (
        value.constructor.name !== 'Array'
        && value.constructor.name !== 'Object'
      )

Since duplicated with #145, close this issue.


OpmlUrlReader: free XML property after using it

Fixes #624.
I'll merge this in three days unless someone stops me for a review.

Coverage increased (+0.07%) to 39.218% when pulling 0fa6248faa7d9fb4b71193e573c702d3cb7f3de0 on bugfix/624-opmlurlreader-memleak into f5225b8333185a11c6ef7d9d32475c0b66b104c4 on master.


Add more navigations Options

Hello @Prashant4900
Would you please tell where should we do more navigation in this project?
I'd like to do if you clarify your question.

Hello @Prashant4900
Would you please tell where should we do more navigation in this project?
I'd like to do if you clarify your question.

After the cloning the projects you find out all the navigation in route 📂. Our navigation are not for production at the level on now but it's really helpful for testing and development. So tried to make it's on that so it's taken less line and more easy.

Still I am not getting what are you saying?
Should I do code formatting of navigation or doing the another way?
What should we do in more navigations options? and if possible please say with example

Still I am not getting what are you saying?
Should I do code formatting of navigation or doing the another way? What should we do with more navigations options? and if possible please say with an example

if you currently see so you find there are only 3-4 navigations options are there with there are no animations, no arguments.
So it just tried to implement a passing argument, animations while moving other screens or back on screens also Adding other navigations property apart the push, pushReplace, remove


add elasticache

What does this PR do?
Adds elasticache ack-controller per this:
https://github.com/aws-ia/terraform-aws-eks-ack-addons/issues/48
Motivation

Resolves #48

Test Results
I deployed this myself to test:
kgp --all-namespaces
NAMESPACE         NAME                                            READY   STATUS    RESTARTS   AGE
ack-elasticache   ack-elasticache-7dd7ddb57-m5g6w                 1/1     Running   0          30s

Looking forward to having this merged. Thank you @sharkymcdongles

Hi @sharkymcdongles, are you able to make the requested changes?


Oscillator unison

Hello. Below is info on oscillator_multi, and how Cakewalk computes unison based on it.

oscillator_multi
Configure an region to use more than one oscillator.
range: 3-9
default: 1
If the value is 1, the region operates normally or in ring modulation.
If the value is 2, the region operates in frequency or phase modulation.
(see also oscillator_mode)
If between 3 and 9, this defines a unison, with the value being the number of oscillators.
In this case, oscillator_detune must also be set to indicate the spread between the oscillators.

Cakewalk unison

Let m be the number of oscillators defined by oscillator_multi, and d the detune value defined by oscillator_detune.
The array of m oscillators is tuned by multiplying d for each oscillator by its coefficient.

Oscillator
Coefficient

1
0

2
-1

3
1

4
-1/4

5
1/4

6
-1/2

7
1/2

8
-3/4

9
3/4

The oscillators sum into left and right channels with declining linear gain, opposite for each channel.
With i the number of the oscillator (starting at 1), the left gain is (i-1)/(m-1), and the right gain is 1-((i-1)/(m-1)).
Example
<region>
oscillator_detune=100
oscillator_multi=5

# Left:
[4]         25 cents,          0 dB
[3]        -25 cents,   -2.49878 dB
[2]        100 cents,    -6.0206 dB
[1]       -100 cents,   -12.0412 dB
[0]          0 cents,       -inf dB

# Right:
[0]          0 cents,          0 dB
[1]       -100 cents,   -2.49878 dB
[2]        100 cents,    -6.0206 dB
[3]        -25 cents,   -12.0412 dB
[4]         25 cents,       -inf dB

Thanks! Added in d55e069 with some minor changes, if result is OK.

Sorry, little mistake in the note above. I should have read the paragraph again.
Erratum.
If the value is 1, the region operates normally, in single oscillator mode.
If the value is 2, the region operates in ring modulation mode, if `oscillator_mode` is 0.
(see also `oscillator_mode`)

This is according to my understanding of book page 340, since oscillator_mode seems not present in any software. (and an attempt to use ring modulation will crash)


Add mixed precision inference incl loading

Add the option to load models in bfloat16 and float16. Esp important for large models like GPT-J and GPT-NeoX.
Ideally, load from HuggingFace in this low precision, do weight processing on the CPU, and then move the processed model weights to the GPU. Might be easiest to do the weight processing once and caching to HF (see #103 )

Maybe covered by #125

Solved with #298

Before I found this issue, I didn't @glerzing was working on #317 so I was planning to report separately.
Anyway, despite progress, I thought I'd share a demo where I get nans running in float16:
import torch
from transformer_lens import HookedTransformer

torch.set_grad_enabled(False)

# Issue #1: there's no way to use float16 on initialization so we're forced to
# convert to float16.
model32 = HookedTransformer.from_pretrained(f"EleutherAI/pythia-70m-deduped")
print(repr(model32.to_string(model32(" Unable")[0, -1].argmax())))

# Issue #2:
model16 = HookedTransformer.from_pretrained(f"EleutherAI/pythia-70m-deduped").to(
    torch.float16
)
print(model16(" Unable"))

Thanks for the progress on #317 !!! Psyched to see it merged.

I believe that Pythia 70m can have attention scores as low as -100,000,
which will get you nans in float16 because those can do max -65,536.
Honestly, my take is that this is not our problem, and you should use
bfloat16 instead, so long as HuggingFace *also *gives you nans. I have no
clue why Pythia is this high lol.
On Tue, 13 Jun 2023 at 01:55, Ben Thompson @.***> wrote:

Before I found this issue, I didn't @glerzing
https://github.com/glerzing was working on #317
https://github.com/neelnanda-io/TransformerLens/pull/317 so I was
planning to report separately.
Anyway, despite progress, I thought I'd share a demo where I get nans
running in float16:
import torch
from transformer_lens import HookedTransformer
torch.set_grad_enabled(False)
Issue #1: there's no way to use float16 on initialization so we're forced to
convert to float16.
model32 = HookedTransformer.from_pretrained(f"EleutherAI/pythia-70m-deduped")
print(repr(model32.to_string(model32(" Unable")[0, -1].argmax())))
Issue #2:
model16 = HookedTransformer.from_pretrained(f"EleutherAI/pythia-70m-deduped").to(
torch.float16
)
print(model16(" Unable"))
Thanks for the progress on #317
https://github.com/neelnanda-io/TransformerLens/pull/317 !!! Psyched to
see it merged.
—
Reply to this email directly, view it on GitHub
https://github.com/neelnanda-io/TransformerLens/issues/104#issuecomment-1588341142,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASRPNKIWFH77YISXJNTSBR3XK627RANCNFSM6AAAAAATDIORBA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Wow, that's fascinating about the giant attention scores!!
I'm seeing big differences in both bfloat16 and float16 between Huggingface and TL on Pythia 410M. I was suspicious that the TL processing (fold LN, center unembed, etc) was causing the differences so I tried from_pretrained_no_processing but the differences persist.
I'm gradually learning more about the internals of TL so if I have time soon, I'll dig in on this and try to figure out what's going on.

Source
import torch
from transformer_lens import HookedTransformer
import transformers

torch.set_grad_enabled(False)

model_name = f"EleutherAI/pythia-410m-deduped"
model32 = HookedTransformer.from_pretrained_no_processing(model_name)
logits32 = model32(" Unable", prepend_bos=False)[0, -1]
p32 = torch.softmax(logits32, dim=-1)
del model32

model16 = HookedTransformer.from_pretrained_no_processing(model_name).to(torch.float16)
logits16 = model16(" Unable", prepend_bos=False)[0, -1]
p16 = torch.softmax(logits16, dim=-1)
del model16

modelB16 = HookedTransformer.from_pretrained_no_processing(model_name).to(torch.bfloat16)
logitsB16 = modelB16(" Unable", prepend_bos=False)[0, -1]
pB16 = torch.softmax(logitsB16, dim=-1)
del modelB16

tokenizer = transformers.AutoTokenizer.from_pretrained("EleutherAI/gpt-neox-20b")

hf_model32 = transformers.GPTNeoXForCausalLM.from_pretrained(
    model_name, low_cpu_mem_usage=True, torch_dtype=torch.float32
).cuda()
hf_logits32 = hf_model32(tokenizer(" Unable", return_tensors="pt")["input_ids"].cuda()).logits[0, 0, :]
hf_p32 = torch.softmax(hf_logits32, dim=-1)
del hf_model32

hf_modelB16 = transformers.GPTNeoXForCausalLM.from_pretrained(
    model_name, low_cpu_mem_usage=True, torch_dtype=torch.bfloat16
).cuda()
hf_logitsB16 = hf_modelB16(tokenizer(" Unable", return_tensors="pt")["input_ids"].cuda()).logits[0, 0, :]
hf_pB16 = torch.softmax(hf_logitsB16, dim=-1)
del hf_modelB16

hf_model16 = transformers.GPTNeoXForCausalLM.from_pretrained(
    model_name, low_cpu_mem_usage=True, torch_dtype=torch.float16
).cuda()
hf_logits16 = hf_model16(tokenizer(" Unable", return_tensors="pt")["input_ids"].cuda()).logits[0, 0, :]
hf_p16 = torch.softmax(hf_logits16, dim=-1)
del hf_model16

print(f'TL, float32    top_token={repr(tokenizer.decode(logits32.argmax())):<16} p={p32.max().item():.3f}')
print(f'TL, bfloat16   top_token={repr(tokenizer.decode(logitsB16.argmax())):<16} p={pB16.max().item():.3f}')
print(f'TL, float16    top_token={repr(tokenizer.decode(logits16.argmax())):<16} p={p16.max().item():.3f}')
print(f'HF, float32    top_token={repr(tokenizer.decode(logits32.argmax())):<16} p={hf_p32.max().item():.3f}')
print(f'HF, bfloat16   top_token={repr(tokenizer.decode(logitsB16.argmax())):<16} p={hf_pB16.max().item():.3f}')
print(f'HF, float16    top_token={repr(tokenizer.decode(logits16.argmax())):<16} p={hf_p16.max().item():.3f}')

Output
TL, float32    top_token=' to'            p=0.745
TL, bfloat16   top_token=' to'            p=0.641
TL, float16    top_token='\n'             p=0.000
HF, float32    top_token=' to'            p=0.745
HF, bfloat16   top_token=' to'            p=0.746
HF, float16    top_token='\n'             p=0.750

I've also observed this - my weak guess is that it's due to implementation
details like the use of einsum and reshaping of attention matrices? I'm not
super sure otherwise what would be implemented differently.
It would be interesting to me to carefully go through each activation and
compare between TL and HF, and I'd be curious what you find!
On Tue, 13 Jun 2023 at 13:26, Ben Thompson @.***> wrote:

Wow, that's fascinating about the giant attention scores!!
I'm seeing big differences in both bfloat16 and float16 between
Huggingface and TL on Pythia 410M. I was suspicious that the TL processing
(fold LN, center unembed, etc) was causing the differences so I tried
from_pretrained_no_processing but the differences persist.
I'm gradually learning more about the internals of TL so if I have time
soon, I'll dig in on this and try to figure out what's going on.
Source
import torch
from transformer_lens import HookedTransformer
import transformers
torch.set_grad_enabled(False)
model_name = f"EleutherAI/pythia-410m-deduped"
model32 = HookedTransformer.from_pretrained_no_processing(model_name)
logits32 = model32(" Unable", prepend_bos=False)[0, -1]
p32 = torch.softmax(logits32, dim=-1)
del model32
model16 = HookedTransformer.from_pretrained_no_processing(model_name).to(torch.float16)
logits16 = model16(" Unable", prepend_bos=False)[0, -1]
p16 = torch.softmax(logits16, dim=-1)
del model16
modelB16 = HookedTransformer.from_pretrained_no_processing(model_name).to(torch.bfloat16)
logitsB16 = modelB16(" Unable", prepend_bos=False)[0, -1]
pB16 = torch.softmax(logitsB16, dim=-1)
del modelB16
tokenizer = transformers.AutoTokenizer.from_pretrained("EleutherAI/gpt-neox-20b")
hf_model32 = transformers.GPTNeoXForCausalLM.from_pretrained(
model_name, low_cpu_mem_usage=True, torch_dtype=torch.float32
).cuda()
hf_logits32 = hf_model32(tokenizer(" Unable", return_tensors="pt")["input_ids"].cuda()).logits[0, 0, :]
hf_p32 = torch.softmax(hf_logits32, dim=-1)
del hf_model32
hf_modelB16 = transformers.GPTNeoXForCausalLM.from_pretrained(
model_name, low_cpu_mem_usage=True, torch_dtype=torch.bfloat16
).cuda()
hf_logitsB16 = hf_modelB16(tokenizer(" Unable", return_tensors="pt")["input_ids"].cuda()).logits[0, 0, :]
hf_pB16 = torch.softmax(hf_logitsB16, dim=-1)
del hf_modelB16
hf_model16 = transformers.GPTNeoXForCausalLM.from_pretrained(
model_name, low_cpu_mem_usage=True, torch_dtype=torch.float16
).cuda()
hf_logits16 = hf_model16(tokenizer(" Unable", return_tensors="pt")["input_ids"].cuda()).logits[0, 0, :]
hf_p16 = torch.softmax(hf_logits16, dim=-1)
del hf_model16
print(f'TL, float32    top_token={repr(tokenizer.decode(logits32.argmax())):<16} p={p32.max().item():.3f}')
print(f'TL, bfloat16   top_token={repr(tokenizer.decode(logitsB16.argmax())):<16} p={pB16.max().item():.3f}')
print(f'TL, float16    top_token={repr(tokenizer.decode(logits16.argmax())):<16} p={p16.max().item():.3f}')
print(f'HF, float32    top_token={repr(tokenizer.decode(logits32.argmax())):<16} p={hf_p32.max().item():.3f}')
print(f'HF, bfloat16   top_token={repr(tokenizer.decode(logitsB16.argmax())):<16} p={hf_pB16.max().item():.3f}')
print(f'HF, float16    top_token={repr(tokenizer.decode(logits16.argmax())):<16} p={hf_p16.max().item():.3f}')
Output
TL, float32    top_token=' to'            p=0.745
TL, bfloat16   top_token=' to'            p=0.641
TL, float16    top_token='\n'             p=0.000
HF, float32    top_token=' to'            p=0.745
HF, bfloat16   top_token=' to'            p=0.746
HF, float16    top_token='\n'             p=0.750
—
Reply to this email directly, view it on GitHub
https://github.com/neelnanda-io/TransformerLens/issues/104#issuecomment-1589205548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASRPNKKGLYRIHCWJ27I63ODXLBMAZANCNFSM6AAAAAATDIORBA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I believe that Pythia 70m can have attention scores as low as -100,000, which will get you nans in float16 because those can do max -65,536. Honestly, my take is that this is not our problem, and you should use bfloat16 instead, so long as HuggingFace *also *gives you nans. I have no clue why Pythia is this high lol.

So I think Theo Horsley might have discovered why: Pythia models have large bias vectors on the K and Q values (He said the K bias vector for one head was like norm 300 which is especially silly given its just a constant offset). At least in normal supervised ML you don't apply L2 regularization to bias terms, so I assume similarly there is no weight decay on the attn biases and so they end up large and blow up the attn scores.

Interesting! Note that Pythia uses rotary attention, where b_K does matter
(the key gets rotated by the difference in positions, so it doesn't cancel
out between different source tokens)
On Thu, 27 Jul 2023, 6:31 pm Wes Gurnee, @.***> wrote:

I believe that Pythia 70m can have attention scores as low as -100,000,
which will get you nans in float16 because those can do max -65,536.
Honestly, my take is that this is not our problem, and you should use
bfloat16 instead, so long as HuggingFace *also *gives you nans. I have no
clue why Pythia is this high lol.
So I think Theo Horsley might have discovered why: Pythia models have
large bias vectors on the K and Q values (He said the K bias vector for
one head was like norm 300 which is especially silly given its just a
constant offset). At least in normal supervised ML you don't apply L2
regularization to bias terms, so I assume similarly there is no weight
decay on the attn biases and so they end up large and blow up the attn
scores.
—
Reply to this email directly, view it on GitHub
https://github.com/neelnanda-io/TransformerLens/issues/104#issuecomment-1654080123,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASRPNKJYMCQ2N3VWD64LKA3XSKQY7ANCNFSM6AAAAAATDIORBA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I just re-ran the test above with TL 1.5.0 and I'm getting much better results but there are still noticeable discrepancies from the HF implementation:
TL, float32    top_token=' to'            p=0.745
TL, bfloat16   top_token=' to'            p=0.730
TL, float16    top_token=' to'            p=0.737
HF, float32    top_token=' to'            p=0.745
HF, bfloat16   top_token=' to'            p=0.746
HF, float16    top_token=' to'            p=0.750

Since Pythia was trained in float16, we should probably ignore the bfloat16 comparison, but the discrepancy in float16 is still noticeable.
Thanks glerzing and slavachalnev for the improvements!

I think the main explanation for the differences is the fact that TransformerLens uses einsum instead of Linear layers or Conv1D.

Just a follow up this: I'm observing inconsistency when doing greedy decoding (using .generate) with a hooked transformer. One example is:

Input1: "List only the name of three songs performed by Eminem: 1."
Output1: ""Lose Yourself" 2. "The Real Slim Shady" 3. "Stan"."

Input2: "List only the name of three songs performed by Eminem: 1. "Lose Yourself" 2. "The Real Slim Shady" 3. ""
Output2: ""Without Me""

Note that input1 is a prefix of input 2, meaning if I do greedy decoding I should expect output1 to have the same three answers as output2. However, it is giving me different results. In fact, if I just use "List only the name of three songs performed by Eminem: 1. "" as the input (adding just one extra token " to input1), the third answer will be changed from Stan to Without Me.
The example above is when I'm loading Mistral with dtype=float16. However, this type of inconsistency is still happening with float32 for both Llama3 and Mistral. Any insights would be super helpful thank you! :)