mpi: Do not send padding region when doing pythonland halo exchange

This change is both a tiny optimisation and also a fix for a potential correctness issue in the case where the application provides a customised padding, since the provided padding is not required to be the same on all ranks.
There are three commits here which hopefully make sense.

add a timeout to MPI tests
add a test which fails with the two ranks having different padding (it hangs without the first commit)
a fix

The timeout one can be a separate PR if you like, or we can drop it altogether. It's needed for the tests to behave as expected (i.e. fail) with the second commit, but after the third commit they pass again (and obviously don't hang) so I don't mind either way.
I'm also not sure if 30s is an appropriate timeout, or if we should make it e.g. an environment variable to be set in the test environment script?

@tjb900 the only thing maybe is that the timeout is a bit too short as IIRC there are a few tests that will take much longer
@mloubout thoughts?

I'll double check the logs to see if there is some test that are borderline but seems to all pass so timeout must be long enough.
Thanks @tjb900

Everything looks legit in the logs. And I don't think the timeout can actually cause issue, since it only concerns non-responding mpi not long running script with working comm (in my understanding)

Thank you again @tjb900. Merging.


[ALBEF] vqa finetune

Summary:

ALBEF VQA finetuning logic draft (all configs are hard-coded for now)
examples/albef/model.py: has the training model for vqa task
examples/albef/finetune.py: has the methods for training and validation
Test plan:

Ran the training validation loops (saw decreasing loss and improved outputs)

Codecov Report

Merging #174 (34a96fd) into main (81116d9) will decrease coverage by 0.43%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #174      +/-   ##
==========================================
- Coverage   92.97%   92.54%   -0.44%
==========================================
  Files          47       47
  Lines        2805     2802       -3
==========================================
- Hits         2608     2593      -15
- Misses        197      209      +12

Impacted Files
Coverage Δ

torchmultimodal/models/albef.py
90.08% <ø> (ø)

torchmultimodal/utils/attention.py
47.82% <31.25%> (-41.07%)
:arrow_down:

...modal/modules/encoders/albef_multimodal_encoder.py
96.87% <75.00%> (-3.13%)
:arrow_down:

...hmultimodal/modules/encoders/albef_text_encoder.py
98.93% <100.00%> (ø)

torchmultimodal/modules/losses/albef.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Trim all whitespace in elements written to hugo_stats.json

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.78.2/extended darwin/amd64 BuildDate: unknown

Does this issue reproduce with the latest release?
Yes

I have a markdown content file with a form element declared on several lines :
<form
  action="[URL]"
  method="POST"
>

I run hugo server -e production
EXPECTED: a "form" tag is added in hugo_stats.json tags list
ACTUAL: a "form\n" tag is added in hugo_stats.json tags list
Note: The issue does not happen if the form is declared in one line :
<form action="[URL]" method="POST">

I'm looking to contribute to Hugo. This will be my first issue in a big project. Let me know if this is simple enough for me to make as a PR. Also, please guide me with more context if any

Edit2: Would using json.Compact

I don't know what that does, but I was thinking more inline of doing strings.TrimSpae on all the Classes, IDs etc. in
https://github.com/gohugoio/hugo/blob/master/publisher/htmlElementsCollector.go#L233

My skills in Go language are pretty poor at the moment, I let you fix that one 😅
Happy to see the fix for a bit of learning.
My 2 cents : trimming the spaces while parsing HTML elements, as suggested by @bep, is probably more efficient than calling a function like json.Compact after building an entire json.

@bep
https://github.com/gohugoio/hugo/blob/master/publisher/htmlElementsCollector.go#L234 has the strings.TrimSpace already.  Am I missing something then?


Pressing the counties list modal "X" sometimes doesn't work on Android

Seen in build 46

Fixed in https://github.com/project-vagabond/covid-green-app/pull/516 will be in build 47


[Improvement][SQL] Query return number should be configurable

Describe the question
The SQL query result can only return 10,000 records, which are currently hard-coded
Which version of DolphinScheduler:
-[1.3.6-Release]
Describe alternatives you've considered
I think it should be configurable.

@syb853553110 Welcome to contribute code


Support Excluding Agent Server Properties during Migration

Chose plural because it's all and not just some, ever

Works as expected 👍


False "Cannot reassign to a variable declared with const" error with destructured function arguments in Typescript

Describe the bug
The code below will admittedly seem a bit contrived but it is a boiled down version of something we have in our codebase.
What seems to trip the compiler is the const fn3 = ... assignment.
Looks like it mistakens the ({x,y} ... => string) type for a destructured variable declaration.
The code parses and compiles correctly with tsc, ts-node and esbuild.
Input code
function fn1({x,y}: {x: string, y: string}): string {
    return x + y
}

function fn2({x,y}: {x: string, y: string}): string {
   const fn3: ({x,y}: {x: string, y: string}) => string = fn1
   return fn3({x,y})
}

Config
{
  "jsc": {
    "parser": {
      "syntax": "typescript",
      "tsx": false
    },
    "target": "es5",
    "loose": false,
    "minify": {
      "compress": false,
      "mangle": false
    }
  },
  "module": {
    "type": "es6"
  },
  "minify": false,
  "isModule": true
}

Playground link
https://play.swc.rs/?version=1.2.151&code=H4sIAAAAAAAAA0srzUsuyczPU0jLM9SortCprLVSqK6wUiguKcrMS9dRqIQxazVhLIVqLgUgKEotKS3KU6hQ0Fao5Krl4kpDmGREkknJ%2BXnFJUBdxlYKBPQp2NrBdNqCHMyFcAZQO0SzJsgtAFg82JHWAAAA&config=H4sIAAAAAAAAA0WNSwrDMAxE76J1ts0id%2BghjKsEF%2F%2FQKBBjfPfawSU7aebpqdIXlrZK2QhYxoQS1Vy0kZbMsOKy0kKKHu3Gg1tfjBysHWG8eudTAs92oeCi28sw2RSyMPBUJh7%2BT7YuCulzjqDez27hSu1xzDuH9wRVTm4%2FOkwQUrUAAAA%3D
Expected behavior
The code should compile to:
function fn1({ x, y }) {
  return x + y;
}
function fn2({ x, y }) {
  const fn3 = fn1;
  return fn3({ x, y });
}

Actual behavior
error: Cannot reassign to a variable declared with `const`
 --> input.ts:5:15
  |
5 | function fn2({x,y}: {x: string, y: string}): string {
  |               ^
  |
note: x was declared here
 --> input.ts:6:17
  |
6 |    const fn3: ({x,y}: {x: string, y: string}) => string = fn1
  |                 ^

error: Cannot reassign to a variable declared with `const`
 --> input.ts:5:17
  |
5 | function fn2({x,y}: {x: string, y: string}): string {
  |                 ^
  |
note: y was declared here
 --> input.ts:6:19
  |
6 |    const fn3: ({x,y}: {x: string, y: string}) => string = fn1
  |                   ^

Version
1.2.151
Additional context
No response

I tried fixing this... and seems like it's wasm-only bug.

@kdy1 I can still reproduce this issue in 1.2.152:
https://play.swc.rs/?version=1.2.152&code=H4sIAAAAAAAAA42OsQqAIBRF9%2FcVd0xqqTahfkYyXJ5gBor47ykhjnWnu5zD0TcrbyxD8zykMMUskYLE5Z3hc0JsN4v2kAhl7vC3YwSMiJSJdDct1YTfKmX58gVbJT4SsO2N3Gox9Y6Cv7Cg%2FADW9e751QAAAA%3D%3D&config=H4sIAAAAAAAAA0WNSwrDMAxE76J1ts0id%2BghjKsEF%2F%2FQKBBjfPfawSU7aebpqdIXlrZK2QhYxoQS1Vy0kZbMsOKy0kKKHu3Gg1tfjBysHWG8eudTAs92oeCi28sw2RSyMPBUJh7%2BT7YuCulzjqDez27hSu1xzDuH9wRVTm4%2FOkwQUrUAAAA%3D


Any interest in removing activesupport?

Do you think it's feasible?  It looks like all you're using it for is the IndifferentAccess base class.  If that's it, I'd like the option to drop it and inherit from Hash.

Hi Rick,
I have just pushed some commits in the edge branch in which DirtyHashy and DirtyIndifferentHashy are inherited from Hash and HashWithIndifferentAccess respectively.
The definitions of both is child's play:
class DirtyHashy < Hash
  include Dirty::Hash
end

class DirtyIndifferentHashy < HashWithIndifferentAccess
  include Dirty::Hash
end

You can look at the new DirtyHashy tests to see how DirtyHashy handles stringified and symbolized keys.
Is this what you were asking for? If so, then I will release DirtyHashy 0.2.0 ^^
Greets,
Paul.

Well, it is released! Spread the word if you like DirtyHashy, ok? :)


Scaling question

[Can you be more specific about the scaling part mentioned in this tutorial? In the compute target creation part, the maximum nodes was set to 4. Is the scaling able to exceed that number? ]

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

ID: 4bee4b9a-ce0f-f98b-3160-d6c4a40ef3ac
Version Independent ID: a0475a9e-47f6-2803-8707-fac54ba898dd
Content: Image classification tutorial: Train models - Azure Machine Learning
Content Source: articles/machine-learning/tutorial-train-models-with-aml.md
Service: machine-learning
Sub-service: core
GitHub Login: @sdgilley
Microsoft Alias: sgilley

@stevenluda Yes, you should be able to use a different max node number and if required the scaling should automatically add nodes upto your max nodes to run the experiment.

#assign:@sdgilley

@stevenluda, thanks for your report. The author has updated the article to be more clear. You should see the changes live in a few hours. I'll proceed to #please-close this issue. If you feel that your documentation issue hasn't been resolved, please open a new issue with additional information.


⚠ NCAA has degraded performance

In 1edabd1, NCAA (https://www.ncaa.com) experienced degraded performance:

HTTP code: 200
Response time: 739 ms

Resolved: NCAA performance has improved in 20c5c4a.


bacon: 2.18.0 -> 2.18.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Canop/bacon/releases.
meta.description for bacon is: Background rust code checker
meta.homepage for bacon is: https://github.com/Canop/bacon
meta.changelog for bacon is: https://github.com/Canop/bacon/blob/v2.18.1/CHANGELOG.md
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.18.1 with grep in /nix/store/2iplx344szmrz8v9ccl77v0ixdaavqfx-bacon-2.18.1
found 2.18.1 in filename of file in /nix/store/2iplx344szmrz8v9ccl77v0ixdaavqfx-bacon-2.18.1

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

bacon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2iplx344szmrz8v9ccl77v0ixdaavqfx-bacon-2.18.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/7369c9f860078c185a951db3eda33487995d1228#bacon

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2iplx344szmrz8v9ccl77v0ixdaavqfx-bacon-2.18.1
ls -la /nix/store/2iplx344szmrz8v9ccl77v0ixdaavqfx-bacon-2.18.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

bacon

Maintainer pings
cc @FlorianFranzen for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Canop/bacon/releases.
meta.description for bacon is: Background rust code checker
meta.homepage for bacon is: https://github.com/Canop/bacon
meta.changelog for bacon is: https://github.com/Canop/bacon/blob/v2.18.2/CHANGELOG.md
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.18.2 with grep in /nix/store/yqw5aabfw5s6vbgjlb5z83rs058h7xz2-bacon-2.18.2
found 2.18.2 in filename of file in /nix/store/yqw5aabfw5s6vbgjlb5z83rs058h7xz2-bacon-2.18.2

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

bacon

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/6e96c2dfb4236f8eed9494eb5156f8c0ebe16694#bacon

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yqw5aabfw5s6vbgjlb5z83rs058h7xz2-bacon-2.18.2
ls -la /nix/store/yqw5aabfw5s6vbgjlb5z83rs058h7xz2-bacon-2.18.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

bacon

Maintainer pings
cc @FlorianFranzen for testing.


added two repo links

Fixed #81

Hi @nishantdub37, these repositories are not valid. Kindly raise your PR with new repo links


Error: Unexpected intrinsicName Error, [object Object]

I'm using this extension in a Deno project, not a Node project. Maybe that's relevant.
The error only appears when I position the cursor on anything I've imported from an external library.
All my imports of external libraries are done the Deno way, so they look like:
import { xml2js as xml2pojo } from<EMAIL_ADDRESS>
And they are cached locally by Deno.
When I hover over the line where I call the function, VSCode is able to show my the type definition of the function, so VSCode itself is not having any troubles:

I am just unsatisfied that I can't see what the type of the 2nd function argument (userOptions: Xml2JsOptions) expands into without cmd + click on the function name to navigate to the source file. I was hoping this extension could fix that.
But when I position my cursor on the function name xml2pojo the extension shows nothing and the logs say:
Error: Unexpected intrinsicName Error, [object Object]

Edit
I'm doing a little debugging and I think I have found some useful information. When I cmd + click to go to the source file of the imported xml2js library then this extension doesn't work in that file at all. It raises the error:
SourceFile not found<EMAIL_ADDRESS>
So I'm just guessing this extension can't work with Deno because:

It doesn't know where Deno caches its imported libraries locally.
And it can't open up a typescript file from a web URL either.

Is that a fair assessment? Please I would super appreciate it if you could fix this.

Thank you for reporting this issue and providing detailed information about the problem you encountered while using this extension in a Deno project.
You are correct that this extension was primarily designed with Node.js in mind, and currently, it does not support other platforms. While it might work seamlessly with bun, Deno has a different module resolution mechanism, which is causing the extension to fail in your project.
Supporting Deno would require collaboration with the Deno extension for VSCode, and given my limited experience with Deno's inner workings, it might be challenging for me to implement a solution quickly. However, based on the logs you provided, it seems that with a few modifications, it might be possible to add support for Deno.
I believe that supporting Deno would be a great addition to this extension, and I appreciate you bringing this to my attention. When time permits, I will investigate the issue further and attempt to implement the necessary changes.
Thank you once again for your valuable feedback and for taking the time to report this issue. Your contribution is much appreciated, and I will keep you updated on the progress of adding Deno support to the extension.

@Evertt
I took a look at the implementation of vscode_deno.
As an overview, it seems that by disabling most of the functionality of VSCode's standard tsserver and building a Language Server for deno, it supports deno's URI-based imports and performs type checking.
This extension depends on that tsserver is built and resolving types. This is because it receives type information from tsserver using tsserver's Language Server Plugin(with CompilerAPI). Since the deno Language Server only has the correct type information, it might be possible to support it if the Language Service Plugin is supported by deno. However, the following document states that it is not supported.
https://docs.deno.com/runtime/manual/advanced/typescript/faqs#why-dont-you-support-language-service-plugins-or-transformer-plugins
I apologize for piquing your interest, but it appears that supporting deno is difficult.

Ah, that's too bad, but I understand.


Daemon set merge

cherry-picks @DanLavine commit from #55 and updates them to work with current master.

@dgrove-oss can you squash your two commits to one so I can credit both committers when merging?

squashed my two into 1.


Clarify imagePullPolicy for empty value

imagePullPolicy when defined without a value defaults to Always not to IfNotPresent
Fixes issue #91944

/assign @savitharaghunathan

/sig node

Validated.
/lgtm
/approve


⚠ Bofrost.lu has degraded performance

In 4ab672b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 888 ms

Resolved: Bofrost.lu performance has improved in 243e281 after 10 minutes.


"Not a parameterized type!" when indexing a record with a compact constructor and an annotated parameterized types

Hello,
Considering this java record with a compact constructor:
public record MyRecord(List<@NotEmpty String> list) {
    public MyRecord {
        list = null;
    }
}

When indexing the record
Indexer indexer = new Indexer();
indexer.indexClass(MyRecord.class);

Jandex throws a java.lang.IllegalArgumentException: Not a parameterized type!

The error occurs when parsing a type annotation on the record constructor. If the constructor is removed, the class file is parsed just fine. This is how javap -v -p shows the constructor:
  public test.MyRecord(java.util.List<java.lang.String>);
    descriptor: (Ljava/util/List;)V
    flags: (0x0001) ACC_PUBLIC
    Code:
      stack=2, locals=2, args_size=2
         0: aload_0
         1: invokespecial #1                  // Method java/lang/Record."<init>":()V
         4: aload_0
         5: aload_1
         6: putfield      #7                  // Field list:Ljava/util/List;
         9: return
      LineNumberTable:
        line 6: 0
        line 7: 9
      LocalVariableTable:
        Start  Length  Slot  Name   Signature
            0      10     0  this   Ltest/MyRecord;
            0      10     1  list   Ljava/util/List;
      LocalVariableTypeTable:
        Start  Length  Slot  Name   Signature
            0      10     1  list   Ljava/util/List<Ljava/lang/String;>;
    MethodParameters:
      Name                           Flags
      list
    Signature: #37                          // (Ljava/util/List<Ljava/lang/String;>;)V
    RuntimeVisibleTypeAnnotations:
      0: #28(): FIELD, location=[TYPE_ARGUMENT(0)]
        test.NotEmpty

It seems to me that the target_type value is wrong -- it's FIELD (0x13), but it should be METHOD_PARAMETER (0x16). Congrats, you found a bug in javac :-)
Jandex should be able to ignore that. I'll try to figure something out.

Also, it seems the problem only appears with a compact constructor. As a workaround, you can use a normal constructor instead of a compact one:
public record MyRecord(List<@NotEmpty String> list) {
    public MyRecord(List<@NotEmpty String> list) {
        this.list = list;
    }
}

It seems to me that the target_type value of the type annotation is wrong -- it's FIELD (0x13), but it should be METHOD_PARAMETER (0x16). Congrats, you found a bug in javac :-)

The merit goes to you :)
Are you sure it should be METHOD_PARAMETER? AFAIU compact constructor is operating on the fields of the class

Actually a compact constructor has implicitly declared parameters, per the JLS (<IP_ADDRESS>. Compact Canonical Constructors). In this case, there's just one parameter, and its type contains a type annotation.
The record also implicitly declares a component field (8.10.3. Record Members). That field's type also contains a type annotation, and it seems to me that javac simply takes the type of the field it generated and pushes it into the constructor. Which is fine, until there are type annotations :-)

In any case, I have a workaround for Jandex that will just drop the invalid type annotation.

Done in #219. Thanks for the report!


🐛 CSS: media query range formatting

Environment information
CLI:
  Version:                      1.9.2
  Color support:                true

Platform:
  CPU Architecture:             x86_64
  OS:                           windows

Environment:
  BIOME_LOG_PATH:               unset
  BIOME_LOG_PREFIX_NAME:        unset
  BIOME_CONFIG_PATH:            unset
  NO_COLOR:                     unset
  TERM:                         unset
  JS_RUNTIME_VERSION:           "v22.9.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "npm/10.8.3"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    true
  VCS disabled:                 true

Workspace:
  Open Documents:               0

What happened?
https://biomejs.dev/playground/?lintRules=all&files.main.css=LgBjAGwAYQBzAHMAIAB7AAoACQBAAG0AZQBkAGkAYQAgACgAMQAwADIANABwAHgAIAA8AD0AIAB3AGkAZAB0AGgAIAA8AD0AIAAxADIAOAAwAHAAeAApACAAewAKAAkACQBjAG8AbABvAHIAOgAgAHIAZQBkADsACgAJAH0ACgB9AA%3D%3D
Expected result
Formatting should match prettier... is what I'd like to say, but prettier doesn't seem to format this well: if I remove any spaces in the range, prettier will not add spaces.
Biome instead adds spaces except after the last <=.
I'd expect 1024px <= width <= 1280px to not change formatting.
I'd expect 1024px<=width<=1280px to be formatted as 1024px <= width <= 1280px.
https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_media_queries/Using_media_queries
Code of Conduct

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

Hi @nstepien
I would like to resolve this issue. Could please assign it to me?
Thanks

Closed by https://github.com/biomejs/biome/pull/4083


fix CI cleanup

currently if one of the steps in CI fails, resources won't be clean up. need to figure out a way to know how which step failed and execute the appropriate cleanup

@scrungus is this one fixed now?

yep, closing


Don't preserve main audio(?)

While implementing audio timings, I'm starting to feel the whole AudioSource logic a little bit twisted up and I'm losing my mind on it.
I'm also improving error messages and discovering new issues.
Instead of keeping the source and trying to restart it, it's probably better to just clear it when end is forced. Maybe even without forcing it (since I would like to keep the logic simple and clean, and also try to implement audio timings with media fragments).

[ ] Don't keep audio src when ended (needs better decision)
[ ] Change my mind and use media fragments for audio timings
...more? will see

Something to note:

https://developer.mozilla.org/en-US/docs/Web/API/HTMLAudioElement/Audio#determining_when_playback_can_begin

OK this is scary... https://stackoverflow.com/a/39529112/16804863
Apparently, you can remove the src attribute to clear it...?
Normally, if you try to set src to an empty string, it throws something like this:
'MEDIA_ELEMENT_ERROR: Empty src attribute'
Error code: 4 (MEDIA_ERR_SRC_NOT_SUPPORTED)

or, if you set it to null, it just tries to find a null file in the path of the page HTML:
'file:///C:/Users/Gabriel/[...]/Soundboard/app/windows/mainWindow/null'

Hmm, trying to resume it after a load() seems to just hang and invoke a waiting event. And trying to pause it throws an AbortError: The play() request was interrupted by a call to pause(). https://goo.gl/LdLk22 (in promise)

The custom betterSrc should help guard this, and I think it's the best solution (removing the src with some weird DOM manipulation even if we're not in the DOM, as per previous comments)


cpu hog fixed

ScheduledExecutorService created via Executors.newScheduledThreadPool caused extensive cpu hog (>100%).
ScheduledExecutorFactoryBean solved the problem.

Great work! Thank you for solving this problem!


Windows PATH environment variable format

Using the following FFMPEG_PATH variable I received a ENOENT error:
C:\ffmpeg\bin\ffmpeg.exe
Escaping the '' slashed fixed this.
C:\ffmpeg\bin\ffmpeg.exe

Hello,
This is not related to fluent-ffmpeg. You must escape backslashes in JS strings.

Im using the Windows environment variable. Its not common to escape the backslashes in Windows.
I'm suggesting something like the following in lib/capabilities.js proto._getFfmpegPath or an example in the Readme?
var os = require('os'); if (os.platform() === 'win32') {   var path = process.env.FFMPEG_PATH;   path = path.replace('\', '\\'); }

This will not work (you don't have any '' chars in your variable, you have '\f' and '\b' characters instead).
All paths must have escaped backslashes on windows. Or you can use path.join. This is not fluent-ffmpeg responsibility :)


[BUG] Read streams do not buffer, resulting in multiple redundant HTTP requests.

Library name and version
Azure.Storage.Files.Shares 12.8.0
Describe the bug
When opening a stream and feeding it into an API that will perform a few different reads of the stream, a new HTTP GET request is made for each read of the stream. Providing an explicit bufferSize parameter to OpenReadAsync has no effect.
Expected behavior
I'd expect that only a single GET is performed. This was the case for the Microsoft.WindowsAzure.Storage SDK.
Actual behavior
For this specific repro, 5 separate GET requests, each for the entire contents of the file, are made.
Reproduction Steps
Run the following, putting a URI for an image in the appropriate spot.
Observe the console output. See there's one HEAD request (expected), followed by multiple duplicate GET requests (expected only one). All GET requests are made with the same x-ms-range header representing the full byte range of the file.
<PackageReference Include="Azure.Storage.Files.Shares" Version="12.8.0" />
<PackageReference Include="System.Drawing.Common" Version="6.0.0" />

using System.Collections.Concurrent;
using System.Diagnostics;
using Azure.Storage.Files.Shares;

var httpListener = new HttpMonitoringListener();
DiagnosticListener.AllListeners.Subscribe(httpListener);

var file = new ShareFileClient(new Uri("<SAS uri of an image - the one I tested was a 44.8KB PNG file>"));
var stream = await file.OpenReadAsync();

var gdiImage = System.Drawing.Image.FromStream(stream);

public class HttpMonitoringListener : IObserver<DiagnosticListener>, IObserver<KeyValuePair<string, object>>
{
    public void OnCompleted() { }
    public void OnError(Exception error) { }
    public void OnNext(DiagnosticListener listener) { if (listener.Name == "HttpHandlerDiagnosticListener") listener.Subscribe(this); }

    int i = 1;
    public void OnNext(KeyValuePair<string, object> value)
    {
        if (value.Key == "System.Net.Http.HttpRequestOut.Start")
        {
            var request = ((HttpRequestMessage)value.Value.GetType().GetProperty("Request").GetValue(value.Value));

            Console.WriteLine($"======= Request #{i++} =======");
            Console.Write(request.Method + " ");
            Console.WriteLine(request.RequestUri);
            Console.WriteLine(request.Headers);
        }
    }
}

Output from Repro
Output from a 44.8 KB PNG file:
======= Request #1 =======
GET https://<url removed>
x-ms-version: 2020-10-02
Accept: application/xml
x-ms-client-request-id: 74421d50-d353-4ba8-ad38-f37c41b311f6
x-ms-return-client-request-id: true
User-Agent: azsdk-net-Storage.Files.Shares/12.8.0 (.NET 6.0.1; Microsoft Windows 10.0.19043)
traceparent: 00-c3fb475d27f3cc346bdb8fee324df91d-54895f9a961eb8a2-00

======= Request #2 =======
GET https://<url removed>
x-ms-version: 2020-10-02
x-ms-range: bytes=0-4194303
x-ms-range-get-content-md5: false
Accept: application/xml
x-ms-client-request-id: 4fa4ffac-7e76-47df-a637-bafbe715ad15
x-ms-return-client-request-id: true
User-Agent: azsdk-net-Storage.Files.Shares/12.8.0 (.NET 6.0.1; Microsoft Windows 10.0.19043)
traceparent: 00-7236457e0a74370761f4278105f84729-e598de71a40b9258-00

======= Request #3 =======
GET https://<url removed>
x-ms-version: 2020-10-02
x-ms-range: bytes=0-4194303
x-ms-range-get-content-md5: false
Accept: application/xml
x-ms-client-request-id: 393710ce-07e2-469b-ad4d-c491b661ec0f
x-ms-return-client-request-id: true
User-Agent: azsdk-net-Storage.Files.Shares/12.8.0 (.NET 6.0.1; Microsoft Windows 10.0.19043)
traceparent: 00-14b312a86fb8ff5fa38d7a90bdb42fde-10814ba33d601a57-00

======= Request #4 =======
GET https://<url removed>
x-ms-version: 2020-10-02
x-ms-range: bytes=0-4194303
x-ms-range-get-content-md5: false
Accept: application/xml
x-ms-client-request-id: 7ffe8dd8-6bcb-40e5-a20f-f0f59056ad2f
x-ms-return-client-request-id: true
User-Agent: azsdk-net-Storage.Files.Shares/12.8.0 (.NET 6.0.1; Microsoft Windows 10.0.19043)
traceparent: 00-072de5ec78830ebfe1e68da00643687e-af5018c0c605ebf8-00

======= Request #5 =======
GET https://<url removed>
x-ms-version: 2020-10-02
x-ms-range: bytes=0-4194303
x-ms-range-get-content-md5: false
Accept: application/xml
x-ms-client-request-id: c163bda7-1ce0-4df5-b0f5-7ef6d30d7353
x-ms-return-client-request-id: true
User-Agent: azsdk-net-Storage.Files.Shares/12.8.0 (.NET 6.0.1; Microsoft Windows 10.0.19043)
traceparent: 00-21e86590502777720398f421d58e7b23-4298db6c9005f0c2-00

======= Request #6 =======
GET https://<url removed>
x-ms-version: 2020-10-02
x-ms-range: bytes=0-4194303
x-ms-range-get-content-md5: false
Accept: application/xml
x-ms-client-request-id: f1f6f888-6821-4c60-8fe4-d557328a1e4e
x-ms-return-client-request-id: true
User-Agent: azsdk-net-Storage.Files.Shares/12.8.0 (.NET 6.0.1; Microsoft Windows 10.0.19043)
traceparent: 00-38caf4775b4590a38fb5cce4eb43ae44-37bcee204c0c0bc3-00

Environment
.NET 6.0.1, Windows 10

@jaschrep-msft can you please help here.
Hi @ascott18 Andrew,
The earlier versions of storage sdk were stateful and would cache a lot of state and data. The current one is not, this enables a easier programming model and better concurrency but will not cache the data for you. Having said that, once James get a chance to look at it, we will let you know if you are missing a knob (different api) or there is a genuine bug we need to fix.
Thanks.

It looks to me like the logic in LazyLoadingReadOnlyStream.Seek is incorrect.
The predicate in this conditional should be newPosition < _position && newPosition >= beginningOfBuffer.
Right now, if seeking to the start of the stream when the whole file is buffered, newPosition == beginningOfBuffer == 0. The condition for "newPosition is less than _position, but within _buffer" fails on this, despite position 0 being within the beginning of the buffer.

I was going to PR this, but there may be something else as well. Debugging through the Seek method and tweaking some local variables to get past the condition didn't make the redundant HTTP requests go away.
I suspect there's a similar off-by-1 error in some other code.

I can investigate the seek logic. Renaming this issue to reflect that this method does buffer and appears to incorrectly determine when it needs to refetch data for that buffer.


Error propagation from tritonserver for nvtabular workflow

This PR introduces error propagation from tritonserver. Allowing failures in Tritonserver to properly display for users. Giving them more valuable information about what is happening under the hood. This pattern can be implemented for all triton operators. This PR focuses on nvtabular.

Looks like the test for this uses something that isn't available from the installed version of the package:
Failed to initialize Python stub: ModuleNotFoundError: No module named 'tests.unit.systems'


Add L3 API Feature Files and Support for C++ Test Agent

@agosh01 is implementing the L3 APIs for up-cpp.


Adding 153 sketch rnn

Is this ready for review @kfahn22 ?

Yes.  Please let me know if I need to change anything.


Doesn't work properly in standard AWS EC2 Ubuntu bash

In a vanilla AWS Ubuntu bash session, while jpterm installs nicely with pip install jpterm and runs seemingly fine.
However after creating a notebook, things go wrong quite quickly.  It's not possible to execute a cell - ctrl-E and ctrl-R don't work and cause editing to stop properly functioning.

Maybe these key bindings are intercepted by the shell? Do you know if other Textual applications work fine in this environment, for instance toolong?

Tried toolong: works very well - ctrl-t, ctrl-f, ctrl-g all fine.  I hit ctrl-e and ctrl-r (no function in toolong) and these don't do anything - but don't mess anything up as far as I can tell.
I tested AWS since it seems a great use-case.  There's plenty of people who would love to use Jupyter but won't want to open browsers or, e.g. VSCode nor use Jupyter remotely while administering AWS.
I'm ssh'ing into AWS from an Ubuntu box using GNOME Terminal Version 3.49.92.
BTW jpterm seems a great project.

Thanks for the kind words!
I guess I should try myself with an AWS free tier.

I tried with a local SSH server, and it seems to work fine. What version of jpterm do you have installed?

I did pip install ... jpterm-0.2.8

Tried with a fresh (local) Ubuntu 23.10 install and ssh to (the same) AWS box:
In a .ipynb file

ctrl-r doesn't execute the cell nor launch a subsequent cell
after ctrl-r can't backspace
ctrl-e doesn't execute the cell (seems to do nothing)

I can reproduce this in fedora (WSL2) as well.  I have tried multiple different terminal interfaces, but it doesn't work at all.  It looks great, and I am excited to work with it, but right now, it's non functional.  I can load notebooks, I can create notebooks, but no executions are possible.

 daemoen  systeminfo | findstr /B /C:"OS Name" /B /C:"OS Version"
OS Name:                   Microsoft Windows 11 Pro
OS Version:                10.0.22631 N/A Build 22631

 daemoen  scoop list wez*
Installed apps matching 'wez*':

Name    Version                  Source Updated             Info
----    -------                  ------ -------             ----
wezterm 20240203-110809-5046fc22 extras 2024-02-07 21:22:11

❯ pcat /etc/os-release
NAME="Fedora Remix for WSL"
VERSION="39"
ID=fedoraremixforwsl
ID_LIKE=fedora
VERSION_ID=39
PLATFORM_ID="platform:f39"
PRETTY_NAME="Fedora Remix for WSL"
ANSI_COLOR="0;34"
CPE_NAME="cpe:/o:fedoraproject:fedora:39"
HOME_URL="https://github.com/WhitewaterFoundry/Fedora-Remix-for-WSL"
SUPPORT_URL="https://github.com/WhitewaterFoundry/Fedora-Remix-for-WSL"
BUG_REPORT_URL="https://github.com/WhitewaterFoundry/Fedora-Remix-for-WSL/issues"
PRIVACY_POLICY_URL="https://github.com/WhitewaterFoundry/Fedora-Remix-for-WSL/blob/master/PRIVACY.md"
FEDORA_REMIX_VERSION=39.0.1

❯ zsh --version
zsh 5.9 (x86_64-pc-linux-gnu)

❯ brew list | ag python
<EMAIL_ADDRESS><EMAIL_ADDRESS>
❯ pipx list
venvs are in /home/daemoen/.local/share/pipx/venvs
apps are exposed on your $PATH at /home/daemoen/.local/bin
manual pages are exposed at /home/daemoen/.local/share/man
   package jpterm 0.2.11, installed using Python 3.12.2
    - jpterm

That's pretty much everything I could think of to include.

Ok, just realized I'm stupid, apparently.  Need to connect to a server.  I had figured that jpterm would launch a kernel.  It doesn't appear to.  @rthickling try starting your notebook/server manually, and then pointing jpterm at it via jpterm --server http://blahblah:blah.  This worked for me.


Take out ming/cygwin stuff in autotools build

Before @WardF came along and wrote the CMake build system, I had attempted to build the netcdf dll with Cygwin and also with ming32.
I had various degrees of success, but it was a real pain, and now I am happy that the CMake build system handles Windows.
However, my attempts left a lot of cruft in the build files. I will remove it as part of my make -j check changes.

But I see some work since then on at least the cygwin build.
Let me get my old windows machine out of the closet and I will try a Cygwin build...

@DennisHeimbigner still uses cygwin and I believe autotools therein.

I routinely use cygwin, but not mingw. I also use cmake with visual studio.


DB, ticket router update

Brief Description
Added instructions to connect PGAdmin to the docker DB
Updated the DB to mark only instances from the current season as active
Fixed the SQL query that returns tickets to active instances
-for rows returned by this query, fixed the date parsing to account for the new DB
References Issue
Partial resolution of Issue #115
Type of change

[x] Bug fix
[ ] Feature
[ ] Major change (potentially breaking, list anything that is broken below)
[ ] Documentation

Breaking Changes
None
Testing done

[x] Docker was used.
[ ] Docker was not used and I have listed my configuration below.

System Specifications:

Operating System:
Node Version:
NPM Version:

Checklist:

[x] My code follows the styling guidelines.
[ ] I have added unit tests and verified that they pass.
[x] I have used the linter and fixed any linting issues.
[x] I have commented the code.
[ ] I have adjusted the documentation to match the changed code.
[x] Prior to submitting the PR, I made sure the latest changes were merged with my branch.
[ ] Existing tests pass locally with changes.

@ben16williams pinging you for a review/approval

This is totally optional but you could also modify setup.py so that the default database name is wondertix

Can I get these changes merged for the moment? I'm updating the field names in all tables to explicitly denote a field as a foreign keys (eg: eventid_fk) which will require explicit joins instead of natural joins. I will also set the default DB name as wondertix

Can I get these changes merged for the moment? I'm updating the field names in all tables to explicitly denote a field as a foreign keys (eg: eventid_fk) which will require explicit joins instead of natural joins. I will also set the default DB name as wondertix

Sure, just make sure to put it in a ticket


[Feature-Request] Stereo inference

Start with the why:
To run inference on both mono frames.
Move to the what:
Ability to run a neural network on both left&right mono frames instead of on the color frame (by default). This was supported in gen1
Move to the how:
Create 2 NN nodes and stream mono frames to them

Thanks.  This will be great to get these Gen1 supported things into the Gen2 demo.  Thanks for coming up with these and making issues.  Excited to get these out!  They'll be great for quickly trying out various capabilities.


Update to use Variadic Tuple Types

:wave: Hey thanks for publishing this tiny library!
Now that TS 4 is out with variadic tuple types, would you consider an Nary release?

Hey @bennypowers, do you have a particular encoding in mind? I have a "functioning" one, but it unfortunately still has the "excessively deep" type warnings. I'm thinking 4.1's recursive types might help here, but I've yet to try.

lol I was really hoping you were going to pop in here and be like "yeah I solved that one months ago"
yeah I don't have a working solution, but maybe I could hack on one for #hacktoberfest


:sparkles: Add command for fetching permission claims

Summary
This PR adds command to get permission claims related to an APIBinding.
Example:
➜  kcp git:(cmd/permission-claim-get) ✗ kubectl-kcp claims get apibinding cert-manager
NAME                       BINDING            STATUS
group1-resource1    cert-manager   Rejected
group2-resource2   cert-manager   Rejected

➜  kcp git:(cmd/permission-claim-get) ✗ kubectl-kcp claims get apibinding
NAME                       BINDING            STATUS
group1-resource1    cert-manager   Rejected
group2-resource2   cert-manager   Rejected
group3-resource3   solo                   Accepted

Related design doc: https://docs.google.com/document/d/1J31wXY1-2aCyyGFjUlusKoHDzV-zktAmRdIMjMf4OR0/edit

@stevekuznetsov a super basic sample output is in the PR description (github is screwing up the formatting).  It is basically the output of whatever I am specifying in apibinding.spec.permisisonclaims
NAME                        BINDING             STATUS
corev1-configmap    cert-manager    Rejected
corev1-secret           cert-manager     Rejected

@stevekuznetsov a super basic sample output is in the PR description (github is screwing up the formatting). It is basically the output of whatever I am specifying in apibinding.spec.permisisonclaims
NAME                        BINDING             STATUS
corev1-configmap    cert-manager    Rejected
corev1-secret           cert-manager     Rejected

Hi @varshaprasad96, how will it look like when name/namespace or selectors have been defined? Will they get added to the first column? What about verbs and subresources that may come later on? Will the verbs be in an additional column? It should not prevent this PR to get progressed but it may be interesting to have in mind what will come next.
What about naming the first column RESOURCES or CLAIMS if verbs were to be added to it later on? My feeling is that NAME does not carry much information.

Yep - would love to have the leading column be the APIBinding name, and then have a column for group/version, and another for resource(s)

@fgiloux @stevekuznetsov Have modified the first column to be the APIBINDING name and the second to be RESOURCE GROUP-VERSION. We haven't decided on what the UI would like when additional fields like verbs/subresources/namespaces are added. We could either have them here by adding another column in future, or probably have just the sub resources included here and verbs in a metadata/additional-details column.

/lgtm
/approve


gNMI-1.27-updating deviation,isis_level_enabled

Added deviation,isis_level_enabled to metadata.textproto

Pull Request Functional Test Report for #2815 / 3250fb92883ac93268fac57e0a6e6776bf3da88a
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

gNMI-1.27: gNMI Sample Mode Test

Cisco 8000E

gNMI-1.27: gNMI Sample Mode Test

Cisco XRd

gNMI-1.27: gNMI Sample Mode Test

Juniper ncPTX

gNMI-1.27: gNMI Sample Mode Test

Nokia SR Linux

gNMI-1.27: gNMI Sample Mode Test

Openconfig Lemming

gNMI-1.27: gNMI Sample Mode Test

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

gNMI-1.27: gNMI Sample Mode Test

Cisco 8808

gNMI-1.27: gNMI Sample Mode Test

Juniper PTX10008

gNMI-1.27: gNMI Sample Mode Test

Nokia 7250 IXR-10e

gNMI-1.27: gNMI Sample Mode Test

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 54.484%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1847

Relevant Lines:
3390

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 55.268%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
1983

Relevant Lines:
3588

💛  - Coveralls


SONRMSBRU-79: added global property file class...

Added processors to handle obtaining properties from the command line properties and files
Added a property aggregator to handle resolving properties from multiple providers
Added unit tests
To simplify the review process I'm trying to break the argument parsing changes into chunks, of which this is the first. The changes in this commit are mainly additive: they don't change the existing code use the new providers.
The next step will be to start refactoring the existing code to use the new properties providers instead of from the sonar-runner.properties file.

LGTM

Thanks @dbolkensteyn.


[FEATURE] Introduce QC report generation

For the commands sct_propseg, sct_label_vertebrae and
sct_register_to_template, an extra parameter is added. The parameter takes a
output path and generates png images of the input and segmentaion output as well
as log of output values.
Partial implements #1247

is it ready to be tested?

@jcohenadad yes it is.

warning after propseg, then error after sct_qc:

Done! To view results, type:
fslview t2.nii.gz t2_seg.nii.gz -l Red -b 0,1 -t 0.7 &

/Users/julien/code/sct/python/lib/python2.7/site-packages/scipy/ndimage/measurements.py:1301: RuntimeWarning: invalid value encountered in double_scalars
  for dir in range(input.ndim)]
WARNING:/Users/julien/code/sct/spinalcordtoolbox/reports/qc.py:Can not get GIT hash value
Sucessfully generate the QC results in ~/qc_data/qc_results.json
Use "sct_qc -folder ~/qc_data" to see the results on a browser
julien-macbook:~/data/temp/sct_015/t2 $ sct_qc -folder ~/qc_data

--
Spinal Cord Toolbox (qc-generator/af8edf141656c541c5d75d7bad47387ae46d690b)
Running /Users/julien/code/sct/scripts/sct_qc.py -folder /Users/julien/qc_data
Check folder existence...
Can not start the quality control viewer. This is not a proper QC folder

/Users/julien/code/sct/scripts/sct_qc.pyNone

@jcohenadad  can you list me the contents of the ~/qc_data folder?

it is empty

I do this and nothing happens:
sct_qc -folder ~/qc_data/

--
Spinal Cord Toolbox (qc-generator/dcb7cbff9c58599894aa0ba145c1d0fa8ef9c5dc)
Running /Users/julien/code/sct/scripts/sct_qc.py -folder /Users/julien/qc_data/
Check folder existence...
QC viewer started on http://<IP_ADDRESS>:8888
Press "Ctrl" + "C" to stop sct_qc

can you please make the segmentation blink twice as fast?

I've added another command for the same subject (label_vertebrae), but when generating the report i only get the view for propseg.

[ ] error with label_vertebrae: /Users/julien/code/sct/python/lib/python2.7/site-packages/scipy/ndimage/measurements.py:1301: RuntimeWarning: invalid value encountered in double_scalars
for dir in range(input.ndim)]
[ ] get rid of the warning
[ ] I've added another command for the same subject (label_vertebrae), but when generating the report i only get the view for propseg.
[ ] can you please make the segmentation blink twice as fast?
[ ] Sucessfully generate --> Sucessfully generated
[ ] put in next line: sct_qc -folder /Users/julien/qc_data, and in green ('info' in printv)
[ ] change sct_qc message for: "Please open a browser and copy/paste this link: [next line] sct_qc -folder /Users/julien/qc_data ('info' in printv)
[ ] add small space between picture and table

i unchecked some boxes after testing it:


Allow HTML answer-blocks

Currently, you can have either text or image-based answers. This is usually more than enough. Sometimes, however, I'd like to have some more control over the answer blocks using HTML and CSS.
Having an option to show the answer blocks, similar to the image blocks, but containing DIVs with custom styling instead of images would be perfect:

@egyptik, I understand. I'll think of some options that we could include to make this easier.

I second this request, right now I am hacking my own question options into the formless version but having an official supported way to include some custom styling would be welcome.


Near CALL target memory block/segment varies unexpectedly in overlay

Describe the bug
When disassembling code in an overlay, sometimes the segment/memory block of the target of a near CALL varies unexpectedly.
Example 1
Code is enclosed in ASSUME CS = 38e4.
The memory map is shown in a section below.
First CALLs uses memory block 38e4.1 as expected.  The second CALL uses segment 3000, even though there is a location in memory block 38e4.1 that it should target and CS was not modified.
     38e4.1::03ff61 0e                PUSH         CS
     38e4.1::03ff62 e8 db 92          CALL         38e4.1::039240
     38e4.1::03ff65 83 c4 06          ADD          SP,0x6
     38e4.1::03ff68 0b c0             OR           AX,AX
     38e4.1::03ff6a 7d 0f             JGE          LAB_38e4.1__03ff7b
     38e4.1::03ff6c 6a 01             PUSH         0x1
     38e4.1::03ff6e 8a 46 06          MOV          AL,byte ptr [BP + 0x6]
     38e4.1::03ff71 50                PUSH         AX
     38e4.1::03ff72 0e                PUSH         CS
     38e4.1::03ff73 e8 a3 01          CALL         3000:0119

Example 2
Code is enclosed in ASSUME CS = 38e4.
The memory map is shown in a section below.
The first CALL uses segment 4000, even though there is a location in memory block 38e4.1 that it should target.  The second CALL uses 38e4.1 as expected even though CS was not modified.
     38e4.1::040283 0e                PUSH         CS
     38e4.1::040284 e8 78 e7          CALL         4000:e9ff
     38e4.1::040287 5b                POP          BX
     38e4.1::040288 8a 46 08          MOV          AL,byte ptr [BP + 0x8]
     38e4.1::04028b 2a e4             SUB          AH,AH
     38e4.1::04028d 50                PUSH         AX
     38e4.1::04028e 6a 01             PUSH         0x1
     38e4.1::040290 8a 46 06          MOV          AL,byte ptr [BP + 0x6]
     38e4.1::040293 50                PUSH         AX
     38e4.1::040294 0e                PUSH         CS
     38e4.1::040295 e8 26 86          CALL         38e4.1::0488be

Memory Map

Name
Start
End
Length
Type

1000
1000:0000
1000:fd0f
0xfd10
Default

1fd1
1fd1:0000
1fd1:fc7f
0xfc80
Default

2f99
2f99:0000
2f99:94af
0x94b0
Default

38e4.0
38e4:0000
38e4:fcbf
0xfcc0
Default

48b0.0
48b0:0000
48b0:126f
0x1270
Default

49d7
49d7:0000
49d7:02ff
0x300
Default

4a07
4a07:0000
4a07:02ff
0x300
Default

4a37
4a37:0000
4a37:016f
0x170
Default

4a4e
4a4e:0000
4a4e:08ff
0x900
Default

4ade
4ade:0000
4ade:d97f
0xd980
Default

38e4.1
3000:8e40
4000:8aff
0xfcc0
Overlay

38e4.2
3000:8e40
4000:7c11
0xedd2
Overlay

48b0.1
4000:8b00
4000:9d6f
0x1270
Overlay

Expected behavior
Near CALL is targeted at function in same overlay if possible.
Environment:

OS: Windows 10
Java Version: 12.0.2
Ghidra Version: 9.0.4

This occurs with jumps as well:
     38e4.1::03fef0 75 03             JNZ          38e4.1::03fef5
     38e4.1::03fef2 e9 96 00          JMP          38e4.1::03ff8b
     38e4.1::03fef5 80 7e a0 00       CMP          byte ptr [BP + -0x60],0x0
     38e4.1::03fef9 75 03             JNZ          38e4.1::03fefe
     38e4.1::03fefb e9 15 02          JMP          3000:0113


Remove TypeScript dependency

https://github.com/tamagui/tamagui/blob/a3562a570b0bb291ba5487dbd27e950730b3fa4c/packages/build/package.json#L21
This should either get removed or be a dev dependency.

fixed


Save fails when using update_or_create

if you use update_or_create to, update or create, a model the model saves to airtable, but does not save to the Django instance.

This might also be a repeat of #2


Pass options to the childApps definition function

childApps is called prior to initialize.  It should have access to the same options as initialize

@nerdpruitt ping


Updated avail documents

Hi
I have made changes to avail documentation to

Align with Testnet
Removal of docker examples and focus more on binary installation
Included instructions for full node, rpc, archive
Updated validator documentation, more screenshots. Removed manual way of creating keys, rotate_keys does all
Updated with backup and uprading of validators
Chill & slash info

If such contributions are helpful can add more
Thanks

This is fantastic, thank you!
Unfortunately, main has had a few changes and it doesn't apply cleanly anymore. Any chance you might merge and update?
Also, not required, but we're in the middle of a big docs refresh in the dm/docs-refresh branch. If you could open a new PR against that branch that would be extra helpful for us. We plan to release it in the next week or so. If not, no worries, I can merge your PR and then merge our changes on top.
Thanks again!

Cherry-picked commit into dm/docs-refresh branch, closing this PR. Thanks again for the contribution!


net: LookupCNAME only works for hostnames with CNAME records

What version of Go are you using (go version)?
go version go1.7.3 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH=""
GORACE=""
GOROOT="/usr/lib/google-golang"
GOTOOLDIR="/usr/lib/google-golang/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build668040631=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"

What did you do?
package main

import (
	"fmt"
	"net"
)

func main() {
	if cname, err := net.LookupCNAME("www.google.com"); err != nil {
		fmt.Printf("lookup failed: %q\n", err)
	} else {
		fmt.Printf("lookup success: %s\n", cname)
	}
}

What did you expect to see?
The documentation says "LookupCNAME returns the canonical DNS host for the given name."  To me, this means that it returns the canonical name as provided by the POSIX getaddrinfo() function when given the AI_CANONNAME flag.  So I expected the following:
lookup success: www.google.com
What did you see instead?
lookup failed: "lookup www.google.com on <IP_ADDRESS>:53: no such host"
It turns out that LookupCNAME simply resolves CNAME records, it does NOT return the canonical DNS host for the given name as documented.
Proposed change
Ideally LookupCNAME would return the canonical hostname as provided by the POSIX getaddrinfo() function.
If the intention of the LookupCNAME function is to only resolve CNAME records, then replace the first sentence of the documentation with:

LookupCNAME resolves the CNAME DNS record for the given name.  It is an error if the given hostname does not have a CNAME record.

Agreed. We need to update the current documentation for avoiding unnecessary confusion the same as other documentation issues in the net package. Also it's recommend to use the correct terminology defined in https://tools.ietf.org/html/rfc7719.
Like other Lookup functions, I guess that the original author's intention is that the function simply takes an alias and returns a canonical name when the CNAME RR for the alias exists.
Note that the handling for the corrupted or nested CNAME RRs depends on each implementation and varies. #8516 is just for the issue.

Yeah, the docs tried to help by spelling out the full name for CNAME, but
it turns to be misleading...

@mdempsky, in our 1.8 release meeting the other day we decided to make sure that the Go DNS resolver path acted the same as the cgo path for LookupCNAME (does it? don't think so) and then document whatever that behavior is.
You have any interest in taking this on? It might be nice even for Go 1.8, if it's trivial enough.

Indeed. From the code at top:
$ GODEBUG=netdns=go go run cname.go
lookup failed: "lookup www.google.com on <IP_ADDRESS>:53: no such host"

$ GODEBUG=netdns=cgo go run cname.go
lookup success: www.google.com.

The behavior on GODEBUG=netdns=go looks reasonable to me. Otherwise, we'll lose the way to test whether the full resolvers (or recursive servers) keep the canonical name for the alias. Moreover, we may be confused about the difference between Lookup{IP,Host} and LookupCNAME.
Isn't it better to drop the use of cgoLookupCNAME for LookupCNAME?

@mikioh, well, the documentation has always says:

LookupCNAME returns the canonical DNS host for the given name

And until Go 1.5 we always used this cgo LookupCNAME behavior (https://golang.org/doc/go1.5#net), so it seems that the cgo behavior is what people (e.g. @rhansen) expect.
It seems more conservative to figure out what cgo did, make the pure Go version do that, and document it.
But arguably it's been broken for 3 releases and nobody's noticed, so probably few people care, so your proposal might also work.
Is it weird to use a mix of Go's DNS and libc DNS, though? Would that even work on Macs with the restrictive firewall settings?
/cc @ianlancetaylor @rsc

Please match the C library. Sometimes that is all we have. We can't write APIs that duplicate C library functionality but can't be implemented using it.
I looked into netdns=go on Mac a few months ago and I believe I found that it still didn't work under the most restrictive firewall settings. I don't remember the issue number though.


OSX and portable build failures

OSX - problems finding ASP.NET targeting pack archive
Portable - overwriting some portable bits

Fixed by #1741.


yubikey-neo-manager is deprecated

The guide makes mention of the YubiKey NEO Manager:

Windows Use the YubiKey NEO Manager to enable CCID functionality.

Linking to this invalid URL: https://www.yubico.com/products/services-software/download/yubikey-neo-manager/
The correct URL is: https://developers.yubico.com/yubikey-neo-manager/
However at the top of that website it states:

This project is deprecated and is no longer being maintained. Use YubiKey Manager (GUI, CLI) to configure a YubiKey device.

#129 will fix this.


fix jsx text foreground color in tomorrow-night theme

@aeschli
This PR fixes #147146

You can simplify "meta.jsx.children.js.jsx", "meta.jsx.children.tsx" with meta.jsx.children

Closed in favour of https://github.com/microsoft/vscode/pull/151478


Build Failed : SWIFT_VERSION

Error output from Xcode build:
↳
** BUILD FAILED **
Xcode's output:
↳
The “Swift Language Version” (SWIFT_VERSION) build setting must be set to a supported value for targets which use Swift. This setting can be set in the build settings editor.
=== BUILD TARGET Pods-Runner OF PROJECT Pods WITH CONFIGURATION Debug ===
fatal error: module map file '/Users/user/Documents/app/build/ios/Debug-iphonesimulator/vibrate/vibrate.modulemap' not found
1 error generated.

Hello, I think you should re-create your project to include Swift support
see https://github.com/clovisnicolas/flutter_vibrate/issues/7


RPC error using frcli, loop, pool

having some issues trying to interact with bundled daemons using frcli, loop, and pool.
they all give me the same rpc unknown service error and I'm unsure how to troubleshoot further.
for example:
$ pool --rpcserver localhost:10009 --macaroonpath ~/.lnd/data/chain/bitcoin/mainnet/admin.macaroon --tlscertpath ~/.lnd/tls.cert listauth
[pool] rpc error: code = Unimplemented desc = unknown service poolrpc.Trader

or:
$ frcli <same options> insights
[frcli] rpc error: code = Unimplemented desc = unknown service frdrpc.FaradayServer

verbose output for the same:
INFO: 2020/11/08 03:44:23 parsed scheme: ""
INFO: 2020/11/08 03:44:23 scheme "" not registered, fallback to default scheme
INFO: 2020/11/08 03:44:23 ccResolverWrapper: sending update to cc: {[{localhost:10009  <nil> 0 <nil>}] <nil> <nil>}
INFO: 2020/11/08 03:44:23 ClientConn switching balancer to "pick_first"
INFO: 2020/11/08 03:44:23 Channel switches to new LB policy "pick_first"
INFO: 2020/11/08 03:44:23 Subchannel Connectivity change to CONNECTING
INFO: 2020/11/08 03:44:23 blockingPicker: the picked transport is not ready, loop back to repick
INFO: 2020/11/08 03:44:23 pickfirstBalancer: HandleSubConnStateChange: 0x2c7a670, {CONNECTING <nil>}
INFO: 2020/11/08 03:44:23 Channel Connectivity change to CONNECTING
INFO: 2020/11/08 03:44:23 Subchannel picks a new address "localhost:10009" to connect
INFO: 2020/11/08 03:44:23 Subchannel Connectivity change to READY
INFO: 2020/11/08 03:44:23 pickfirstBalancer: HandleSubConnStateChange: 0x2c7a670, {READY <nil>}
INFO: 2020/11/08 03:44:23 Channel Connectivity change to READY
INFO: 2020/11/08 03:44:23 Channel Connectivity change to SHUTDOWN
INFO: 2020/11/08 03:44:23 Subchannel Connectivity change to SHUTDOWN
[frcli] rpc error: code = Unimplemented desc = unknown service frdrpc.FaradayServer

I'm running lnd 0.11.1-beta and lit 0.3.0-alpha. loop operations seem to work fine with REST with the web UI.
# lnd.conf

...

maxpendingchannels=5
listen=<IP_ADDRESS>:9735
restlisten=localhost:8080
rpclisten=<IP_ADDRESS>:10009
accept-keysend=true

bitcoin.active=1
bitcoin.mainnet=1
bitcoin.node=bitcoind

tor.active=true
tor.v3=true
tor.streamisolation=true

Since you are running litd in remote mode, you need to specify the RPC port set by the httpslisten flag (default: 8443). So you should use pool --rpcserver localhost:8443 ...
Your lnd node is listening on port 10009 which is why it does not understand the pool and frcli requests.
See the config-lnd-remote doc for more details.

Awesome, thank you! Didn't realize I was running lit in remote mode since I'm running lnd locally. Makes sense now.


My system keeps crashing

Issue Type: Bug
I downloaded a zip folder and then unzipped it. My VSCode keeps crashing i cant even open any folder, help pls.
The error message is the wondow has crashed (reason: 'oom', code: '-5368....
pls help i need to finish some work ;-;
VS Code version: Code 1.69.2 (3b889b090b5ad5793f524b5d1d39fda662b96a2a, 2022-07-18T16:12:52.460Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
AMD Ryzen 5 3500U with Radeon Vega Mobile Gfx   (8 x 2096)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
5.88GB (3.13GB free)

Process Argv
-n --crash-reporter-id 601a9c63-9584-4c8b-b17b-202e32f7d758

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

prettier-vscode
esb
9.5.0

copilot
Git
1.36.6340

gc-excelviewer
Gra
4.2.55

vscode-edit-csv
jan
0.7.1

python
ms-
2022.10.1

vscode-pylance
ms-
2022.7.40

jupyter
ms-
2022.6.1201981810

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.9

material-icon-theme
PKi
4.19.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vscscmwlcmt:30465135
cppdebug:30492333
pylanb8912:30529769
vsclangdf:30486550
c4g48928:30535728

Do you have a crash log for this? Sorry about missing this issue earlier!

Is this issue still occurring for you in 1.71.0?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Loader for purge expire jobs should consider "dateModified"

Currently, the loader for the purge expire jobs action checks against the "datePublishEnd" or - if not available - "dateCreated".
It certainly makes more sense to check against the "dateModified".

@kilip dateCreated has to be changed to dateModified. Maybe Tests have to be adjusted.
see:

YAWIK : LoadExpiredJobsToPurge.php #34-43
SimpleImport : JobProcessor.php #188


Add a Gitter chat badge to README.md

LimeStreem/My_gruntTest now has a Chat Room on Gitter
@LimeStreem has just created a chat room. You can visit it here: https://gitter.im/LimeStreem/My_gruntTest.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

みす


Support for Contrastive Search

Does vllm supports contrastive search? If not, would be great to add that support as soon as possible? Research shows that this improves model quality significantly and it is currently supported in transformers HF library. Would be great to get parity for this in vllm.

Related to #535

so would the implementation of that ticket enable the end user to be able to implement contrastive search on their side? or do you mean that the contrastive search support work needs that work to happen first?

I may be mistaken, but implementing contrastive search would require more than just enabling logit processing.
Contrastive search, from my understanding, requires access to the last hidden state of each previous token. From the paper linked above, we can see

where h is the last hidden state.
Looking at Huggingface's implementation of Contrastive Search, particularly the _ranking_fast function, we can see that you need the hidden states to calculate the output.
Is access to the last hidden states of each previous token even possible using paged attention? If not, I would guess that even having access to sum of the hidden states would still result in generation similar to contrastive search (this is just a hypothesis though).
Anyway, thanks for the great package :)

Any update on adding this feature?👀


flutter logs --clear should show logs in addition to clearing them

Looks like it works this way already


TTY looks blocked on OSX

From Node.js 0.11.12, REPL sometimes blocked. It does not proceed until I press any key.
Please see joyent/node#9310 for more detail.
When I go deeper, it seems that the commit https://github.com/libuv/libuv/commit/b197515367d1a996dca9009483d202b306f9474e occurs this problem.
I don't understand what the commit exactly means, but I found some odd code.
This is the uv__dup2_cloexec's definition.
int uv__dup2_cloexec(int oldfd, int newfd);

In tty.c
newfd = uv__open_cloexec("/dev/tty", O_RDWR);
if (newfd == -1)
  return -errno;
r = uv__dup2_cloexec(newfd, fd);
if (r < 0 && r != -EINVAL) {
 ...
}

When I changed the code to 'uv__dup2_cloexec(fd, newfd)', the problem was gone.

I'm not sure, but I guess #217 is the same problem.

When I changed the code to 'uv__dup2_cloexec(fd, newfd)', the problem was gone.

That's incorrect. The intention is to replace th given fd with a fd which points to /dev/tty. In order to do  that we open /dev/tty as newfd, and duplicate it with 'fd'. As a result, fd will be closed and reopened with newfd, just what we wanted.
I don't know what the bug is (I haven't dived into it) but the call to uv__dup2_cloexec is correct.

Any progress here?

AFAIK nobody has investigated this deeply yet.
On May 4, 2015 23:46, "Dan Kaplun"<EMAIL_ADDRESS>wrote:

Any progress here?
—
Reply to this email directly or view it on GitHub
https://github.com/libuv/libuv/issues/304#issuecomment-98857111.

I did some quick digging with the following test case which seems to exhibit the issue, albeit not always:
> setInterval(function () {console.log('log')}, 100);

Attaching with GDB shows the following backtrace:
(gdb) bt
#0  0x00007fff8bc9a9f0 in read ()
#1  0x00000001006250c6 in uv__stream_io ()
#2  0x0000000100625a89 in uv__stream_osx_select_cb ()
#3  0x000000010061e68d in uv__async_event ()
#4  0x000000010061e810 in uv__async_io ()
#5  0x000000010062c42e in uv__io_poll ()
#6  0x000000010061ec8b in uv_run ()
#7  0x0000000100516aa6 in node::Start ()
#8  0x0000000100000cb4 in start ()

Here is my understanding of what's going on:

Since this is OSX and we are using a TTY, i/o is performed in the auxiliary thread and select is used
Input is detected on the TTY
The select thread calls uv_async_send to wakeup the loop and "fake" i/o on the TTY
uv__stream_io is called with UV__POLLIN
read() blocks

And here is where I get lost. I don't see how that read could block, AFAIK we do set it to be non-blocking... maybe the cake is a lie?
/cc @indutny any ideas?

@saghul mm... test case? ;)

@indutny, I think @saghul mentioned a test case above:

I did some quick digging with the following test case which seems to exhibit the issue, albeit not always:
> setInterval(function () {console.log('log')}, 100);

Bear in mind that this bug is not consistent, so it takes a few tries to reproduce.

:+1:

Any progress?

Nope, sorry.
On May 22, 2015 21:01, "Dan Kaplun"<EMAIL_ADDRESS>wrote:

Any progress?
—
Reply to this email directly or view it on GitHub
https://github.com/libuv/libuv/issues/304#issuecomment-104744309.

FWIW, I can confirm the issue but that's where I stopped.

Looking.

OK, this is what seems to be happening:
Both uv__read() and select() call are running in parallel for some reason... Guess you know what happens next, select() emits new uv__read() right when the first one returns.
Looking more deeply into it.

I have a fix. Will push it in a moment.

Should be fixed by https://github.com/libuv/libuv/pull/365, sorry for not handling it earlier!

:+1: @indutny :metal:

Fix landed in 7005d29. Let's release it, @saghul !

I don't see why we need to rush a release now, there are other things I'd like to see for 1.6.0.

I don't see why we should delay a release. This is a very important fix that should have been released yesterday.

For me, this is very annoying bug.
I made a nodejs console application to manage database (http://croquiscom.github.io/cormo/guides/CLI.html), and it runs many asynchronous functions by nature.
Without this patch, I should be pressing a key until the execution is done.
I hope this patch is released asap. (Currently I use a custom nodejs build)

See #383


pip not running on ios simulator

i've been try to run on android real device, at the end its work normaly, but not with ios simulator with error message above :
[VERBOSE-2:ui_dart_state.cc(209)] Unhandled Exception: PlatformException(VideoError, Failed to load video: The operation could not be completed, null, null)
flutter: Picture in picture is not supported in this device. If you're using Android, please check if you're using activity v2 embedding.

and this is my flutter doctor :
[✓] Flutter (Channel stable, 2.10.4, on macOS 12.3.1 21E258 darwin-arm, locale en-ID)
[!] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.3)

Hello, @roiskhoiron
I know it's been long time, but i would like to know did you get it work for ios by any chance?
I am facing the same problem while compiling for ios. any response will be appreciated.


Upgrade org.postgresql:postgresql 42.2.20 -> 42.2.21

This PR contains the following updates:

Package
Update
Change

org.postgresql:postgresql (source)
patch
42.2.20 -> 42.2.21

[ ] If you want to rebase/retry this PR, check this box.

Suggested commit message:
Upgrade PostgreSQL JDBC 42.2.20 -> 42.2.21

See:
- https://github.com/pgjdbc/pgjdbc/blob/master/CHANGELOG.md
- https://github.com/pgjdbc/pgjdbc/compare/REL42.2.20...REL42.2.21


VS2017 15.4.0 P2 + current nightlies chokes on compiler-specific syntax for ILASM like (#...#)

I don't know if I'm the only one seeing this, but since Visual Studio 15.4.0 P2 came out, and (at least) since nightly build <IP_ADDRESS>91801, possibly earlier, constructs with compiler-only syntax, like (#...#) for ILASM generates errors in the editor.
These errors didn't use to be there (unless you were trying to use these outside this particular project).
I started seeing them after making a few local adjustments (in the tests, so not related, I assume). Building the whole project, also from within VS, is fine, it's just that the editor seems confused.
I.e., in prim-types.fsi:

or prim-types-prelude.fsi, this error is usually only shown if you use this construct in a normal project:

Just wondering if this is possibly a local issue and whether I should delve deeper or whether other devs out there have the same and have some thoughts.
Rebuidling everything from scratch, clean checkout doesn't seem to have a discernible effect.

@dsyme, @KevinRansom , did you commit anything to fix this? Saturday this was still a problem. With the master of this morning (or at least, the current one), this seems to be fixed. It looks "normal" again:

maybe error highlighting is broken?

@abelbraaksma I don't think we fixed this over the weekend or today.  If you restart VS, does it still result in no errors?

@cartermp, I also installed VS 2017 15.4.0 P3. But several merges were done of fixes that date back further than this weekend. Another thing that I noticed on my own system is that, while I did run the VSIXInstaller, I did not to a update-vsintegration.cmd. While I believe this only updates the local SDK installation, perhaps this is of influence somehow?
Did anybody else notice the original error, or has it only been seen as in the last screenshot (where it seems fixed)?
 But since it's not happening anymore (and it's useless to check historical commits), it's either fixed by accident, or whatever else. Either way, closing is a good thing ;)


Fix burn and renounced filters

Fixing the burn filter by removing the check for error -32602. Otherwise this filter catches non-burned token.
Fixing the freeze filter by checking if the freezeAuthority address is either null or set to to a null address. Otherwise this filter catches freezable tokens.
Also fixes an issue where setting renounce to true and freezable to false would reject every token.

@TheoBrigitte I tried the the fix but it looks like it is not working.
8pUmSNVM37u1nncneC3MWsfge3edqZc9JJ1zJ4TuvE6d
This mint has freeze disabled but the bot said that Creator can freeze tokens and skipped it.
RenouncedFreeze -> Creator can freeze tokens
Is there a possibility that the initially the token did not have freeze disabled or there is something wrong in the logic?


Build failure while building the 'bpf_conformance' package.

Describe the bug
The ebpf-for-windows\external\bpf_conformance package fails to build when using the VS2022 build enviroment.
OS information
Edition: Windows 11 Enterprise
Version: 22H2
OS build:	22623.1037
Experience: Windows Feature Experience Pack 1000.22638.1000.0
Steps taken to reproduce bug
(Assumes installation of VS2022 build environment, per the 'GettingStarted.md' document).

cd to ebpf-for-windows\external\bpf_conformance directory.
Per instructions in the README.md at this location, run the commands cmake -S . -B build followed by cmake --build build to build the required bpf_conformance_runner.exe binary.

Expected behavior
The build should succeed, and the bpf_conformance_runner.exe binary should be present in the ebpf-for-windows\x64\debug directory.
Actual outcome
The cmake --build build command terminates with this linker error:
Generating Code...
  bpf_conformance.vcxproj -> D:\wrk\ebpf-for-windows\external\bpf_conformance\build\lib\Debug\bpf_conformance.lib
  Building Custom Rule D:/wrk/ebpf-for-windows/external/bpf_conformance/src/CMakeLists.txt
  runner.cc
LINK : fatal error LNK1104: cannot open file 'libboost_program_options-vc143-mt-gd-x64-1_80.lib' [D:\wrk\ebpf-for-windows\external\bpf_conformance\build\src\bpf_conformance_runner.vcxproj]

Additional details
As a work-around, the bpf_conformance_runner.exe executable can be downloaded from this location.

Fixed:
https://github.com/Alan-Jowett/bpf_conformance/actions/runs/3833008149/jobs/6523985313


authenticateSession does not authenticate a User in tests

I'm using Implicit Grant authentication and redirect a User to the external corporate login page if he is not authenticated in index route:
import Route from '@ember/routing/route';
import { inject as service } from '@ember/service';
import config from '../config/environment';
import isFastBoot from 'ember-simple-auth/utils/is-fastboot';
import UnauthenticatedRouteMixin from 'ember-simple-auth/mixins/unauthenticated-route-mixin';
import window from 'ember-window-mock';

export default Route.extend(UnauthenticatedRouteMixin, {
  session:      service('session'),
   _isFastBoot: isFastBoot(),

  beforeModel: function() {
    if (this.get('session.isAuthenticated')) {
      this.transitionTo('dashboard');
    } else {
      if (!this.get('_isFastBoot')) {
        let oauthUrl = config.oauthUrl;
        let clientId = config.clientID;
        let redirectURI = `${window.location.origin}/callback`;
        let responseType = `token`;
        let scope = `profile%20openid`;
        window.location.replace(oauthUrl
                              + `?client_id=${clientId}`
                              + `&redirect_uri=${redirectURI}`
                              + `&response_type=${responseType}`
                              + `&scope=${scope}`
        );
      }
    }
  }
});

In an acceptance test:
#tests/acceptance/dashboard-test.js

import { module, test } from 'qunit';
import { visit, currentURL } from '@ember/test-helpers';
import { setupApplicationTest } from 'ember-qunit';
import { authenticateSession } from 'ember-simple-auth/test-support';
import setupMirageTest from 'ember-cli-mirage/test-support/setup-mirage';

module('Acceptance | Dashboard', function(hooks) {
  setupApplicationTest(hooks);
  setupMirageTest(hooks);

  test('Authenticated users can visit /dashboard', async function(assert) {
    let shop = this.server.create('shop');
    this.server.create('user', { shop });

    await authenticateSession({
      token: 'abcdDEF',
      token_type: 'Bearer'
    });
    await visit('/dashboard');

    assert.equal(currentURL(), '/dashboard', 'user is on dashboard page');
  });
});

When putting a break point in beforeModel hook of  index route, I always pass in to the else block, i.e. not authenticated.
What am I missing? Thank you.
Used versions:
ember-simple-auth: 1.8.1
ember: 3.7.3
node: v10.12.0
npm: 6.7.0

Even after changing the value for access_token:
await authenticateSession({
      token: 'abcdDEF',
      token_type: 'Bearer'
    });

has't solved the problem and this.get('session.isAuthenticated') still returns false in index route beforeModel hook.

You should be calling this._super() inside beforeModel() as documented here to allow Ember Simple Auth to do its work.

@pichfl Nice catch, thank you ! But, unfortunately, even after adding:
import UnauthenticatedRouteMixin from 'ember-simple-auth/mixins/unauthenticated-route-mixin';
import window from 'ember-window-mock';

export default Route.extend(UnauthenticatedRouteMixin, {
  session:      service('session'),
   _isFastBoot: isFastBoot(),

  beforeModel: function() {
    this._super(...arguments);
    console.log('+++++++++++ session.isAuthenticated: ' + this.get('session.isAuthenticated'));
...

it still returns false for this.get('session.isAuthenticated').

@belgoros
I am on<EMAIL_ADDRESS>and mocking authenticated session just works fine for me.
One thing that might be doing something wrong with, is the sessionData that you pass as argument to authenticateSession, for mocking the authenticated session.
You might wanna double check that( authenticateSession(sessionData);)

@shankarsridhar Thank you for your response. First, I don't know what you mean under POJO, - it is not Java world here, but JS universe 😄 .
As for what I pass in to the  authenticateSession in my test, it was clearly indicated in my previous comment

@shankarsridhar Thank you for your response. First, I don't know what you mean under POJO, - it is not Java world here, but JS universe 😄 .

@belgoros I understand your confusion. :)
By POJO, I was referring to Plain Old Javascript Object, and it is a common term used in the JS Universe, to refer to javascript objects.

@shankarsridhar it"s the first time I hear about it, glad to know that.

So first of all when you're using the UnauthenticatedRouteMixin (with calling this._super(...arguments)) in your own beforeModel (as @pichfl pointed out), that would actually prevent anyone from entering the route already. As you're not calling this._super(...arguments) you might as well remove the mixin though as it doesn't to anything besides implementing beforeModel.
The exact session data you're passing in shouldn't actually be relevant in order to get the session mocked correctly as authenticateSession will always use a test authenticator, thus the data you pass in to authenticateSession will never be validated by the OAuth2ImplicitGrant anyway in tests.
Can you check what the session contents are in the route's beforeModel though? I wonder whether the session data is ever set up correctly at all. You might also want to check whether the session is maybe invalidated automatically due to an error before you reach the beforeModel - check the log or maybe put a breakpoint here.

@marcoow Thank you very much for your response.

As for removing/keeping this._super(...arguments);. If I remove this._super(...arguments); as follows:

import Route from '@ember/routing/route';
import { inject as service } from '@ember/service';
import config from '../config/environment';
import UnauthenticatedRouteMixin from 'ember-simple-auth/mixins/unauthenticated-route-mixin';
import window from 'ember-window-mock';

export default Route.extend(UnauthenticatedRouteMixin, {
  session:      service('session'),
  routeIfAlreadyAuthenticated: 'dashboard',

  beforeModel: function() {
    if (!this.get('session.isAuthenticated')) {
      let oauthUrl = config.oauthUrl;
      let clientId = config.clientID;
      let redirectURI = `${window.location.origin}/callback`;
      let responseType = `token`;
      let scope = `profile%20openid`;
      window.location.replace(oauthUrl
                            + `?client_id=${clientId}`
                            + `&redirect_uri=${redirectURI}`
                            + `&response_type=${responseType}`
                            + `&scope=${scope}`
      );
    }
  }
});

then the content of dashobaord template is not displayed, it is just emty and there are no errors in the browser console.

You say:

when you're using the UnauthenticatedRouteMixin (with calling this._super(...arguments)) in your own beforeModel (as @pichfl pointed out), that would actually prevent anyone from entering the route already

should may be:

from entering the route if already authenticated ?

That is the actual behaviour, - a user should be redirected to the corporate login page (see window.location.replace statement on the last line) if he/she is not authenticated.
I followed the same way as you used in your dummy app with the only exception that I'm using current-user.js service as it is explained in your docs:

once getting the token, I hit the backend to fetch the User data and set it on current-user service:

#services/curretn-user.js
....
async loadCurrentUser() {
    this.get('flashMessages').clearMessages();
    if (this.get('session.isAuthenticated')) {
      let user = await this.get('store').queryRecord('user', { me: true });
      if (user) {
        this.get('flashMessages').success(this.get('intl').t('flash.signed_in'));
        this.get('currentShop').setShop(user.get('shop'));
        this.set('user', user);
        this.get('shopService').loadShops();
        return user;
      } else {
        this._respondWithError();
      }

    } else {
      this.get('flashMessages').info(this.get('intl').t('flash.signed_off'));
      return RSVP.resolve();
    }
  },

  _respondWithError() {
    this.get('flashMessages').danger(this.get('intl').t('flash.authentication.error'));
    return RSVP.reject();
  }
...

I tried once again just to display the vaue of session.isAuthenticated in beforeModel hook in index.js route. When executing the **dashboard-test** it stilll displays false.
#routes/index.js
...
  beforeModel: function() {
    this._super(...arguments);
    console.log('+++++++++++ session.isAuthenticated: ' + this.get('session.isAuthenticated'));
    if (!this.get('session.isAuthenticated')) {
      let oauthUrl = config.oauthUrl;
      ...

The authentication works correctly when running the app, the only trouble I have is when running this acceptance test:
#tests/acceptance/dasnboard-test.js

import { module, test } from 'qunit';
import { visit, currentURL } from '@ember/test-helpers';
import { setupApplicationTest } from 'ember-qunit';
import { authenticateSession } from 'ember-simple-auth/test-support';
import setupMirageTest from 'ember-cli-mirage/test-support/setup-mirage';
import { setupWindowMock } from 'ember-window-mock';

module('Acceptance | Dashboard', function(hooks) {
  setupWindowMock(hooks);
  setupApplicationTest(hooks);
  setupMirageTest(hooks);

  test('Authenticated users can visit /dashboard', async function(assert) {
    let shop = this.server.create('shop');
    this.server.create('user', { shop });

    await authenticateSession({
      access_token: 'abcdDEF',
      token_type: 'Bearer'
    });

    await visit('/dashboard');

    assert.equal(currentURL(), '/dashboard', 'user is on dashboard page');
  });
});

What am I missing ? Thank you

Can you try and set up a minimal reproduction app? I think we need an app that exhibits the problem that we can debug.

@marcoow OK, I'll create a simple app without pointing to the corporate login page and come back to you once it is done. Thank you.

@marcoow Marco, I created the app here. Thank you for once again for your help.

Your session gets authenticated just fine but it gets immediately invalidated afterwards as loading the current user fails:
_loadCurrentUser() {
  return this.get('currentUser').loadCurrentUser().catch(() => this.get('session').invalidate());
},


Start testing 3.12 pre-releases

This PR will start running our test suite against Python 3.12 beta builds to hopefully catch any issues early before the final release later this year.

Codecov Report
Patch coverage has no change and project coverage change: -0.03 :warning:

Comparison is base (0c833f2) 93.29% compared to head (1ae49e3) 93.26%.

:exclamation: Current head 1ae49e3 differs from pull request most recent head 8f73689. Consider uploading reports for the commit 8f73689 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2967      +/-   ##
===========================================
- Coverage    93.29%   93.26%   -0.03%
===========================================
  Files           64       64
  Lines        13592    13592
===========================================
- Hits         12680    12677       -3
- Misses         912      915       +3

see 2 files with indirect coverage changes

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


/subsys/storage/flash_map/flash_map_default.c missing declarations.

Hi,
Testing settings storage subsystem.
Got missed declarations for:
zephyr-current/subsys/storage/flash_map/flash_map_default.c:49:13: error: ‘FLASH_AREA_STORAGE_OFFSET’ undeclared here (not in a function)
   .fa_off = FLASH_AREA_STORAGE_OFFSET,
             ^~~~~~~~~~~~~~~~~~~~~~~~~
zephyr-current/subsys/storage/flash_map/flash_map_default.c:50:14: error: ‘FLASH_AREA_STORAGE_SIZE’ undeclared here (not in a function)
   .fa_size = FLASH_AREA_STORAGE_SIZE,
              ^~~~~~~~~~~~~~~~~~~~~~~

I have defined CONFIG_FS_FLASH_MAP_STORAGE
and CONFIG_SETTINGS_FCB_FLASH_AREA=4
target SOC is CONFIG_SOC_SERIES_NRF52X
should I use CONFIG_SETTINGS_FCB_FLASH_AREA=3 for now and disable CONFIG_FS_FLASH_MAP_STORAGE ?
Thanks.

Storage definition should be added to DTS for every board.
Made a temporary fix for nrf52_blenano2
in boards/arm/nrf52_blenano2/nrf52_blenano2.dts
                storage_partition: partition@7d000 {
                        label = "storage";
                        reg = <0x0007d000 0x4000>;
                };

Now the missed definition is present.
I am not familiar with the DTS tree system, is this change should be added to every board ? Or there is some kind of template for these configs ?

This is not a bug - just as @leodesigner found, declaration of storage space is undefined for this board. For instance it is possible to add such by DT overlay file. Anyway I open PR for improve this.

Thanks @nvlsianpu, I was hoping it could be done for every board all at once.
But anyway, closing the issue for now.


ISSUE-247:  Added startsWith param to getFindAllHref

Added startsWith option to request.models.ts and data.service.ts
Added basic tests for getFindAllHref in DataService
This PR closes #247

Thanks @mohideen !


Use Chatbot for stench levels

Thanks to Icyman for the chatbot script. Expanded it to be used by the main portion of cognac.
Known issues:

Chatbot script doesn't get copied to output folder when building - Fixed
Needs more testing. Confirmed chatbot script is being ran, but haven't done much more than that
Consider removing updates made to whiteboard. Believe it makes this change incompatible with the old system. Easier to test if we don't change stench level on whiteboard - Not removing as don't believe it will actually be incompatible with old system

testing in stony is going mostly well. Only issue is chat goes into away mode, which is kinda a big deal for this. Anyone know how to prevent this? Suppose I could have the chatbot script put something in /hobo periodically

Updated to report current stench. Folks starting will use this to set initial stench level. This chatter also prevents chat from going into away mode
Believe this is good to go now

oops don't know how I accidentally closed it, but I shouldn't have


Could not interpret optimizer identifier: <keras.src.optimizers.adam.Adam object

Issue type
Documentation Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.16.1
Custom code
No
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Some days ago, I tried to run codes by using Google Colab from a tutorial of transfer learning with movinet without any error. Today, I tried to run codes by using Google Colab from the same tutorial and it produced error when I tried to compile the model.
`num_epochs = 2
loss_obj = tf.keras.losses.SparseCategoricalCrossentropy(from_logits=True)
optimizer = tf.keras.optimizers.Adam(learning_rate = 0.001)
model.compile(loss=loss_obj, optimizer=optimizer, metrics=['accuracy'])`
Standalone code to reproduce the issue
https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/tutorials/video/transfer_learning_with_movinet.ipynb#scrollTo=dVqBLrn1tBsd

Relevant log output
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-12-0c295092a661> in <cell line: 7>()
      5 optimizer = tf.keras.optimizers.Adam(learning_rate = 0.001)
      6
----> 7 model.compile(loss=loss_obj, optimizer=optimizer, metrics=['accuracy'])

/usr/local/lib/python3.10/dist-packages/tf_keras/src/utils/traceback_utils.py in error_handler(*args, **kwargs)
     68             # To get the full stack trace, call:
     69             # `tf.debugging.disable_traceback_filtering()`
---> 70             raise e.with_traceback(filtered_tb) from None
     71         finally:
     72             del filtered_tb

/usr/local/lib/python3.10/dist-packages/tf_keras/src/optimizers/__init__.py in get(identifier, **kwargs)
    333         )
    334     else:
--> 335         raise ValueError(
    336             f"Could not interpret optimizer identifier: {identifier}"
    337         )

ValueError: Could not interpret optimizer identifier: <keras.src.optimizers.adam.Adam object at 0x7af7a01536d0>

It works. Thanks for your help.

@hilmyas,
Glad the issue is resolved for you, Could you please feel free to move this issue to closed status. Thank you!


Add support for multiple hosts per multiple web apps, document script parameters

Introduces a syntax for specifying multiple hosts for multiple web apps

This incorrect PR was created by mistake.


Article from wp

Waiting for merge on #26.
Does a basic JSON fetch from the Wellcome Library blog and shoves it into a template.
We can work on the model as we go on.

@wellcomecollection.org

@wellcometrust/wellcomecollection-org

@alfraser86 to see what I am up to.


Request: Animation system

See: https://github.com/surface-ui/surface/issues/218
In #218, it was determined that LV itself had plans to implement an animations system like the one in Svelte, for convenient transitions when elements leave/enter the page. It's been a while since #218 and to my knowledge that system has yet to materialize in LV. Would it be possible to reopen the idea here?
There is a library which has made some cool progress on this concept here: https://github.com/benvp/live_motion

@sevensidedmarble what Surface could bring to the table in terms of API that could improve DX when compared to just use LiveMotion directly?


[Bug]: Error: Docker build failed

Description
Not sure what happened but I'm no longer able to deploy new versions of my app and I can't seem to figure out why. Hoping someone could see & tell me what I'm missing.
Minimal Reproduction (if possible, example repository)

Please look at the error logs below.

 Please update your lock file with npm install before continuing.
#11 4.432 npm ERR!
#11 4.432 npm ERR! Invalid: lock file's<EMAIL_ADDRESS>does not satisfy<EMAIL_ADDRESS>#11 4.432 npm ERR!
#11 4.433 npm ERR! Clean install a project
#11 4.433 npm ERR!
#11 4.433 npm ERR! Usage:
#11 4.433 npm ERR! npm ci
#11 4.433 npm ERR!
#11 4.433 npm ERR! Options:
#11 4.433 npm ERR! [--install-strategy <hoisted|nested|shallow|linked>] [--legacy-bundling]
#11 4.433 npm ERR! [--global-style] [--omit <dev|optional|peer> [--omit <dev|optional|peer> ...]]
#11 4.433 npm ERR! [--include <prod|dev|optional|peer> [--include <prod|dev|optional|peer> ...]]
#11 4.433 npm ERR! [--strict-peer-deps] [--foreground-scripts] [--ignore-scripts] [--no-audit]
#11 4.433 npm ERR! [--no-bin-links] [--no-fund] [--dry-run]
#11 4.433 npm ERR! [-w|--workspace  [-w|--workspace  ...]]
#11 4.433 npm ERR! [-ws|--workspaces] [--include-workspace-root] [--install-links]
#11 4.433 npm ERR!
#11 4.433 npm ERR! aliases: clean-install, ic, install-clean, isntall-clean
#11 4.433 npm ERR!
#11 4.433 npm ERR! Run "npm help ci" for more info
#11 4.435
#11 4.435 npm ERR!  Run "npm help ci" for more info
4.435
4.435 npm ERR! A complete log of this run can be found in: /root/.npm/_logs/2024-06-24T04_33_41_874Z-debug-0.log

Dockerfile:20
18 | ENV NIXPACKS_PATH /app/node_modules/.bin:$NIXPACKS_PATH
19 | COPY . /app/.
20 | >>> RUN --mount=type=cache,id=hkswcwk-/root/npm,target=/root/.npm npm ci
21 |
22 | # build phase
ERROR: failed to solve: process "/bin/bash -ol pipefail -c npm ci" did not complete successfully: exit code: 1
Error: Docker build failed
[2024-Jun-24 04:33:45.261899] Deployment failed. Removing the new version of your application.
Version
v4.0.0-beta.297

how did you fixed it ?

I encountered a similar issue. I ran npm install and redeployed the app using the updated package-lock.json


Faulty json string causes memory leaks

In case key in map exists (caused by faulty json string) key's first value (Value-ptr) becomes overwritten by new Value *v causing memory leaks since original Value-ptr will never be deleted.

Fixed in a7884426af7c343b45e258fdb741731a517ad7fc.


Add additional Motherlode Mine GroupId and WidgetInfo values

These will be used in a future PR, however, it's most likely best to add such values separately as they modify core classes of the main RL branch, hence this PR.

As per Discord discussion, it has been clarified to best include such changes alongside the relevant plugin:

Closing this PR.


Fix bug of detecting exist process

Description
There might be some mistake in detecting exist docker-gc process.
First of all, if you use "pidof -s program", you will get just one pid, eg:

root@hzwangxing01:~/go/src/docker-gc# sleep 100 &
[1] 12945
root@hzwangxing01:~/go/src/docker-gc# sleep 100 &
[2] 12946
root@hzwangxing01:~/go/src/docker-gc# sleep 100 &
[3] 12947
root@hzwangxing01:~/go/src/docker-gc# pidof -s sleep
12947
root@hzwangxing01:~/go/src/docker-gc# pidof sleep
12947 12946 12945

Secondly, pidof won't take effect if program is started by bash, eg:

root@hzwangxing01:~/go/src/docker-gc# ps -ef | grep docker-gc | grep -v grep
root     13090 26069  0 11:47 pts/1    00:00:00 /bin/bash ./docker-gc
root@hzwangxing01:~/go/src/docker-gc# pidof docker-gc
root@hzwangxing01:~/go/src/docker-gc# pidof bash
26069 13090 11706 10666

So I use pgrep instead.

I don't think the first part of using pidof -s is a concern, as the script really just cares if any docker-gc process is running (one or more, although the guard here prevents more than one from running).
But the bash point is good - thanks for the PR!

pgrep is also likely more portable for things like OS X