MacVim font editing?

Any thoughts on how to make this compatible with MacVim? It kinda seems to work, but it seems to be missing the header and footer fonts, eg:

Hello,
The problem is probably that by default, my config files use Nerd Font symbols,
see the following section of the README: https://github.com/SebastianMuskalla/ModelessVim#nerd-font
You have two options

Install a Nerd Font and use it in your terminal
Edit modeless.vimrc  so that the _ascii versions of the config files are used. If you do that, it will look not quite as nice, but it will work without changing the font.

If this doesn't help, feel free to open another issue.

Actually option 2 works like a charm. You could see #4 (last comment) for a reference.
By the way this is how it looks like in macOS: Beautiful!

Fwiw, the bottom status line issue is resolved in MacVim, but the top line is missing. Not a big deal as I don't plan to really use MacVim much - more of a curiosity for me. It would be interesting to see how this might be resolved to make MacVim play nice together with your config.


feat(footer): Updated with new design specs

Related Ticket(s)
#1668
Description
I've updated the React Footer so it matches the designs specs provided.
OLD:

NEW:

The logic seems off, is the first row supposed to be just one item?

And just a thought, why not use columns? http://w3schools-fa.ir/cssref/css3_pr_columns.html

@jeffchew
The first column is supposed to have only one item, as done in the latest design specs.

@jeffchew
The first column is supposed to have only one item, as done in the latest design specs.

hmm, not sure if this is the best idea. If users have their adblocker turned on, then the cookie preferences link isn't there and the way the links will look would be weird. @wonilsuhibm @larahanlon2 @oliviaflory would it be ok to have this treatment be natural column sorting?
cc: @RaphaelAmadeu

@jeffchew
The first column is supposed to have only one item, as done in the latest design specs.

hmm, not sure if this is the best idea. If users have their adblocker turned on, then the cookie preferences link isn't there and the way the links will look would be weird. @wonilsuhibm @larahanlon2 @oliviaflory @jacobottesen-dgc would it be ok to have this treatment be natural column sorting?
cc: @RaphaelAmadeu

My suggestion is that the items should be split as evenly as possible, starting on the far-left/first column. In that case, I would position 2 items in the first col, 2 in the second, 1 in the third:
1: Contact IBM, Terms of Use
2: Accessibility, Privacy
3: Cookie Preferences
If there is an adblocker, then cookie preferences would not appear in the third col, it would just be empty. The cols are always balanced. My two cents.

Agreed with @jeffchew and @larahanlon2 suggestion of natural sorting and starting at the far left.

Thank you @oliviaflory and @larahanlon2 !
@RaphaelAmadeu I think this would simplify the logic, right?

Sounds good to me too. One minor suggestion: I would group them slightly differently.
1: Contact IBM, Accessibility
2: Terms of use, Privacy
3: Cookie Preferences

@RaphaelAmadeu I noticed the interactive states within the Sketch file and PDF are not following the color tokens or color values for the Gray 90 theme and also have outdated HEX values listed. Some of the incorrect specs will not pass accessibility, the design team needs to look into updating the spec file, can you revert any interactive style updates you made?
Please keep the layout updates to the legal links though!
cc @larahanlon2 @wonilsuhibm @jacobottesen-dgc @jeffchew


Replaced Light Sprites With Non-Linear Falloff Variants

These new sprites have a more perceptually consistent falloff, as opposed to simple linear gradients like in the old ones, which look less smooth.
I also made a new version of the example torch sprite

Could you recreate this PR using the latest version of the dev branch please.


paste XH.sorted.txt with unsorted Xmap.txt results in error results

Hi, thanks for the wonderful tool seGMM. But I may have found a serious bug when I use it in the WES mode.
In my data, I have a sample 1-XJ-XY20220727002 with Xmap value 0.0599118 in the Xmap.txt file, but in the feature.txt and seGMM_result.txt, the Xmap value become 0.031172, which is totally different. And the final gender is wrong.
And I read the source code, I found that  the XH.sorted.txt，Xmap.txt, Ymap.txt are directly pasted together. But the XH.sorted.txt has already sorted with a different sample order, the sample order may be totally different with Xmap.txt and Ymap.txt etc. Thus, the feature is wrong, which results in error gender calculation.
my data snapshots:

please check that, thanks!

Thanks for your reminder and we apologize for this bug. Please update seGMM to the newest version which we have fixed this bug.

Thanks for your reminder and we apologize for this bug. Please update seGMM to the newest version which we have fixed this bug.

Thanks！


Unable to Call Custom Package in /packages

I am using nixdir = ./.; and creating a custom package in the packages folder.
I cannot directly use that package with pkgs.{package_name}.
Evidence from my code:
https://gitlab.com/jongjapra/nix-jongjapra-config/-/blob/main/home-modules/theming/gtk.nix?ref_type=heads

Hi!
The pkgs arg of a home-manager module comes from home-manager; in this case the NixOS module configures it to use the NixOS pkgs set. This is created by lib.nixosSystem and is different from the flakelight pkgs set.
Flakelight provides config.propagationModule to apply flakelight package set customizations to NixOS package sets and home-manager package sets.
Theres two ways you can use it by modifying your system configuration.
The first option is not calling lib.nixosSystem, in which case flakelight can handle this for you. That would look like this:
 {
   inputs,
   src,
   ...
 }: let
   vars = import ./variables.nix;
 in
-  inputs.nixpkgs.lib.nixosSystem {
+  {
     system = vars.ARCHITECTURE;
     specialArgs = {inherit inputs src vars;};
     modules = with inputs; [
       ./hardware-configuration.nix
       disko.nixosModules.disko
       ./disko-configuration.nix
       ./configuration.nix
       self.nixosModules.hardware
       self.nixosModules.environment
       self.nixosModules.service
       self.nixosModules.wayland
       ./home-manager.nix
     ];
   }

The second option is still calling lib.nixosSystem yourself, and adding config.propagationModule to modules:
 {
   inputs,
   src,
+  config,
   ...
 }: let
   vars = import ./variables.nix;
 in
   inputs.nixpkgs.lib.nixosSystem {
     system = vars.ARCHITECTURE;
     specialArgs = {inherit inputs src vars;};
     modules = with inputs; [
+     config.propagationModule
       ./hardware-configuration.nix
       disko.nixosModules.disko
       ./disko-configuration.nix
       ./configuration.nix
       self.nixosModules.hardware
       self.nixosModules.environment
       self.nixosModules.service
       self.nixosModules.wayland
       ./home-manager.nix
     ];
   }

After applying one of the above two, you will be able to use pkgs.everforest-gtk in your nixos and home modules.

Thank you very much, your answer worked well. I hope that in the future, more people will use this great framework. 👍 👍 👍


Roll to engine b28a35dde310690860d45f36757c61584372b325.


docs: add verification steps for gene-normalizer, uta, + seqrepo

close #87

@bwalsh could you re-review? I updated the gene instructions to make it specific to dynamodb and state that postgresql instance is not supported


Positioning of children not working as expected

Query:
Positioning of children beneath the parent entity does not work as expected. (For example a parent entity with position="0 30 -60" doesn't seem to move the children instanced mesh entities by the same amount as I would expect normally.) Is this the same issue as "Diverse Frames of Reference" mentioned in the readme? Regarding the performance consideration, maybe it'd be possible to allow initial setting of the position but then not after that for children unless explicitly requested?

The documentation on "frames of reference", the documentation isn't very clear - sorry!
Basically the code just naively takes the localMatrix (i.e. position, orientation, scale) for the member's object3D and adds an entry for this position matrix in the instanced mesh array of positions, without any adjustments.
That means that

If the instanced mesh and the child objects have the same parents in the scene graph, everything should work just fine.
If the instanced mesh and the member object are at different positions in the scene graph, all bets are off, since the localMatrix of the member won't have the expected meaning in the context of the instanced mesh.

I'd be up for fixing this.  It's pretty straightforward to transform a localMatrix into a different frame of reference.  The concern is what happens when the scene graph above the members, or above the instanced mesh, gets modified subsequently.
I think it would be OK to start off with a naive implementation where things only "catch up" on explicit updates to the member.  Sounds like that would meet your needs, and it's a net improvement on what we have today.  I think I'll do that for now, and then if it proves that more flexibility is needed, I can go ahead and do that.

I've made some changes which should support this:
https://github.com/diarmidmackenzie/instanced-mesh/commit/ec92cc6b3137f086bd2207349f4410bd510877f0
Tested with some static examples here: https://github.com/diarmidmackenzie/instanced-mesh/blob/main/tests/positioning.html
Waiting for some positive feedback on this before cutting a new release.

Just to be clear, you need to set "positioning: world" on the instanced mesh to enable the new function.

Yes the new version works as expected when "positioning: world" is used: https://aframe-instanced-mesh-building-test.glitch.me/

I've now published v0.4.0 with this fix included.


pangenie-index failed: std::runtime_error, "Hash full"

Hi Jana,
I was in the process of indexing a cactus-pangenome based variant callset (seven diploids). The variants were filtered with vcfbub. I tested pangenie by running it with data on a single chromosome and it worked OK.
When I tried it on the full dataset the indexing step failed with the following lines:
pangenie-index -r $infa -v $invcf -o idx -e 100000

GraphBuilder: skip variant at sscro11_Y:42522731 since alleles contain undefined nucleotides: T,TCTCTCTCTCTCNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNCTCTCTCT
GraphBuilder: skip variant at sscro11_Y:43174887 since alleles contain undefined nucleotides: CAACACAAAACCAGGTTCAAGGAATATTGAAAGGGCTTCTCTAAACCAAAAAGTAAGGAAGGAAAGGAAGAAAAAGAAAAGAAAAAAAAAAGAAGAAGAAGAAGAGGAAGAACTAGGACTGAGGAAACCGCAATCAGAGAGCAGTCACTC,CNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNNN
Identified 21347831 variants in total from VCF-file.
Write path segments to file ...
Found 347 chromosome(s) in the VCF.
Count kmers in graph ...
terminate called after throwing an instance of 'std::runtime_error'
what():  Hash full

Can you please tell me what does the error "Hash full" means and how can I overcome this problem?
Thank you,
Lel

Hi Lel,
I'd assume this is either because not enough memory is available on the machine or due to the -e 100000 parameter which sets the hash size. For a whole genome dataset, I would not recommend using this parameter, this is mainly meant to be used for demo purposes (see README).
Best,
Jana

This should then be the -e parameter, as only half of the allocated memory was used by pangenie-index.
Thank you,
Lel


Unable to publish due to baseline difference in .d.ts emit

We seem to be hitting some sort of issue with parenthesization on .d.ts files. This has been blocking nightly publishes, and will block any sort of beta publish next week.
https://typescript.visualstudio.com/TypeScript/_build/results?buildId=76918&view=logs&j=fd490c07-0b22-5182-fac9-6d67fe1e939b&t=00933dce-c782-5c03-4a85-76379ccfa50a&l=139

@rbuckton is this an intended outcome of the node factory work? Should we LKG and accept the difference?

It could be. Odd though that this didn't show up in the API tests before now.

Iirc, the API tests are consistently captured with the lib compiler, so it doesn't appear until you do an LKG (as is done for nightlies and publishes).

I'm not entirely certain why this changed. factory.createUnionTypeNode calls parenthesizer.parenthesizeConstituentTypesOfUnionOrIntersectionType(types) which should do the same thing it did prior to moving the API: when unioning A & B with C it should be parenthesizing it as (A & B) | C since our parenthesizing logic for types is fairly aggressive and doesn't take precedence into account.

It's very likely the parenthesizer was never being invoked before, as those return type nodes are normally just copied from the source (since it's an interface)

@weswigham if it was copied from source then it should be copied from source now as well?

It should be! The parenthesizer in the factory may have had other ideas when the updateUnionType method was fed back in the  input nodes, though!

Ah, I discovered the difference, and accepting the change should be fine. In 3.9 we called parenthesizeTypeParameters in declarations.ts when emitting a type reference:
https://github.com/microsoft/TypeScript/blob/986e9dd252b24d61772d3ddfb1487c43d25025b8/src/compiler/transformers/declarations.ts#L864-L868
And parenthesizeTypeParameters would unconditionally create a new NodeArray even if there were no changes:
https://github.com/microsoft/TypeScript/blob/986e9dd252b24d61772d3ddfb1487c43d25025b8/src/compiler/factory.ts#L1274-L1286
NOTE: It's not the call to createNodeArray but rather the initialization of params to a new array here:
https://github.com/microsoft/TypeScript/blob/986e9dd252b24d61772d3ddfb1487c43d25025b8/src/compiler/factory.ts#L1276
By returning a new NodeArray here, we end up creating new nodes starting at NodeArray<TypeNode> in the union SignatureDeclaration & {typeArguments?: NodeArray<any>} | undefined, which triggers us to hit our over-aggressive parenthesization logic.
One of the node factory changes was to move the parenthesizeTypeParameters logic into the factory, as we do with all other parenthesis rules:
https://github.com/microsoft/TypeScript/blob/fe33e618237d774af4c2b0416b24182b8977ac78/src/compiler/factory/nodeFactory.ts#L1626-L1632
In the updated parenthesizer rules, we use sameMap instead of allocating a new array:
https://github.com/microsoft/TypeScript/blob/fe33e618237d774af4c2b0416b24182b8977ac78/src/compiler/factory/parenthesizerRules.ts#L398-L402
This results in us just reusing the original node since there are no changes in the type reference.

I just pushed up #39173 which runs LKG and accepts the new API baselines.

Publish is working again:


D3 Visualization: Add Classes 'Content Security Policy' error shows.

First, thanks for setting up this really great site...
Describe your problem and how to reproduce it:

The task is to use attr to create a class of 'bar' on a set of divs created by d3.

I did that and confirmed that the dom inspector shows those divs as having a class of 'bar'.

It keeps saying that the divs do not have the class.

In the dev tools console, there is this error, which may or may not be the issues
Content Security Policy: The page’s settings blocked the loading of a resource at inline (“script-src”).

There is also an UnspecifiedAssertionError in the console coming from frame_runner 84:16

This error is in FF.  In Chrome, the same exact code passes.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/data-visualization/data-visualization-with-d3/add-classes-with-d3
Tell us about your browser and operating system:

Browser Name: Firefox
Browser Version:  81.0.1
Operating System: Mac OS 10.15.7

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
First image shows errors in the console.  Second image shows dom inspector validating that the divs have class of bar.

  Could you try again with them off?  Just opening a private mode window might be enough.

Hey @ojeytonwilliams
Thanks for the quick reply.  I had one addon running and turned it off and the page worked.  Also it worked when I turned the addon back on.
My computer did a spontaneous restart since i posted this, perhaps there was something else going on?
Anyway, it seems not to be a problem now.
thanks again.
Robert

Hi Robert, that's a bit strange, but I'm glad it's resolved itself.


Image cropper or gravatar

Idea:
Adding next type of user's avatars:

use gravatar
use basic file upload widget
use croppic widget

Why?
I make more than 10 projects on yii2 with this module, and I always make functional wiht uploading images.

говорили, что в первой версии будет аватар
осталось узнать, когда будет первая версия

если есть наработки, попробуйте добавить pull request, многие думаю оценят.

Support for uploading avatars will be in stable version 1.0.0. However, I'm sorry, but I don't know when it will be released :( I'm really busy right now, and I don't think that I start work more on Yii2-user before summer. But you're very welcome to send pull-requests.


MacOS

How can I download Stremio Enhanced on MacOS (M-series)

Download the ARM64 version?
https://github.com/REVENGE977/stremio-enhanced-community/releases/download/v0.6/stremio-enhanced-darwin-arm64.zip

Download the ARM64 version?
https://github.com/REVENGE977/stremio-enhanced-community/releases/download/v0.6/stremio-enhanced-darwin-arm64.zip

Downloaded but cant run on macOS 15

Yep, does not run on MacOS 15 (24A335). Gives error The application “stremio-enhanced" can’t be opened. when trying to open the downloaded application.

Bumping this. "This app can't be opened" message. M3, Sequoia

Download the ARM64 version?
https://github.com/REVENGE977/stremio-enhanced-community/releases/download/v0.6/stremio-enhanced-darwin-arm64.zip

Downloaded but cant run on macOS 15

Actually, I figured out why, might work for you.
I was downloading from safari, the file would auto decompress the .zip file which messed it up. Ended up downloading it on free download manager, chrome would have been fine too. Then using unarchiver to extract the files.
Next issue now is when I open I get an error Stemio service not found please download, and I have it download. haha. All well, step closer.

@Vegals i'm getting the same "Stremio service not found" error on mac

@Vegals i'm getting the same "Stremio service not found" error on mac

Ah sorry to hear. I ended up trying to go in and fix it for Mac but wasn't able to suss it. Not smart enough.


[iOS] Fixes the exported synchronous method not being called on the method queue when it's the main queue

Summary:
If the module sets the method queue to the main queue, we should call it on the main queue if it contains some UI operations, otherwise it may lead to some undefined behavior.
Changelog:
[IOS] [FIXED] - Fixes the exported synchronous method not being called on the method queue when it's the main queue
Test Plan:
The sync method should be called on the main queue if the module's method queue is main queue.

I don't think this is desirable to do automatically.
Any method marked as sync should be invoked synchronously from the JS thread. If a module requires this to block the UI thread and avoid race conditions, that dispatch_sync should be written explicitly inside the native module method.

I don't think this is desirable to do automatically.
Any method marked as sync should be invoked synchronously from the JS thread. If a module requires this to block the UI thread and avoid race conditions, that dispatch_sync should be written explicitly inside the native module method.

@javache Yes, module can call dispatch_sync in its own module, but actually, we said we would run all module's methods on the specific queue. And in some modules like Appearance, it just thinks it should be called on the main queue and access some UIKit methods, maybe user's custom modules also have this potential issue.
https://github.com/facebook/react-native/blob/b37101486bd4dc26f6ec4e646e12d1484ec9479f/packages/react-native/React/Base/RCTBridgeModule.h#L160-L163

@javache Hi, as you said, we need to fix the issue in the modules like Appearance right? If so, I can change my PR to fix those modules issues.

Yes, let's do that. If they have sync methods called on the wrong thread, that's the right place for a fix.

@javache Done. Please review again.


PyPI link update

Thanks to your PR, we have a lot of links about PYPI that need to be updated. Can you continue to help?


Fix Score Postprocessing in QA Task for English and French Versions

Fixed scores postprocessing for the QA task specifically when using torch.triu, which sometimes gives the same start and end indices if the diagonal parameter is not set to 1, and similarly the np.triu with the k parameter, which may give same start and end indices if k is not set to 0.

Both English and French version are fixed in this Pull Request.
Both Torch and Tensorflow code is fixed in this Pull Request.

It is quite weird why the Build didn't succeed as I didn't change the code of the Spanish version of the course, where it seems to be a build error 🤷

Sorry for the issue, the error is unrelated to your PR!

Yeah I thought so !


[Bug] 设置了tunnels和dialer-proxy 均不生效

Verify steps

[X] I have read the documentation and understand the meaning of all the configuration items I have written, rather than just piling up seemingly useful options or default values.
[X] I have carefully reviewed the documentation and have not resolved the issue.
[X] I have searched the Issue Tracker for the issue I want to raise and did not find it.
[ ] I am a non-Chinese user.
[X] I have tested with the latest Alpha branch version, and the issue still persists.
[X] I have provided the server and client configuration files and processes that can reproduce the issue locally, rather than a sanitized complex client configuration file.
[X] I provided the simplest configuration that can be used to reproduce the errors in my report, rather than relying on remote servers or piling on a lot of unnecessary configurations for reproduction.
[X] I have provided complete logs, rather than just the parts I think are useful out of confidence in my own intelligence.
[X] I have directly reproduced the error using the Mihomo command-line program, rather than using other tools or scripts.

Operating System
Windows
System Version
windows
Mihomo Version
1.18.9
Configuration File
mode: rule
mixed-port: 7897
allow-lan: false
log-level: silent
ipv6: true
external-controller: <IP_ADDRESS>:9097
secret: ""
hosts:
  time.facebook.com: <IP_ADDRESS>
  time.android.com: <IP_ADDRESS>
dns:
  enable: true
  use-hosts: true
  nameserver:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    - tcp://<IP_ADDRESS>
    - tcp://<IP_ADDRESS>
    - tls://dns.google:853
    - tls://<IP_ADDRESS>:853
    - tls://<IP_ADDRESS>:853
    - tls://dns.alidns.com
    - tls://<IP_ADDRESS>
    - tls://<IP_ADDRESS>
    - tls://dot.pub
    - tls://<IP_ADDRESS>
    - tls://<IP_ADDRESS>
    - https://dns.google/dns-query
    - https://<IP_ADDRESS>/dns-query
    - https://<IP_ADDRESS>/dns-query
    - https://dns.alidns.com/dns-query
    - https://<IP_ADDRESS>/dns-query
    - https://<IP_ADDRESS>/dns-query
    - https://doh.pub/dns-query
    - https://<IP_ADDRESS>/dns-query
    - https://<IP_ADDRESS>/dns-query
  default-nameserver:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    - tcp://<IP_ADDRESS>
    - tcp://<IP_ADDRESS>
    - tcp://<IP_ADDRESS>
  use-system-hosts: true
  ipv6: true
  enhanced-mode: fake-ip
  fake-ip-range: <IP_ADDRESS>/24
  fake-ip-filter:
    - "*"
    - +.lan
  fallback:
    - https://<IP_ADDRESS>/dns-query
  nameserver-policy:
    geosite:cn,private:
      - https://doh.pub/dns-query
      - https://dns.alidns.com/dns-query
profile:
  store-selected: true
tun:
  auto-detect-interface: true
  auto-route: true
  device: Mihomo
  dns-hijack:
    - any:53
  mtu: 1500
  stack: system
  strict-route: true
  enable: true

external-controller-cors:
  allow-private-network: true
  allow-origins:
    - "*"
# tunnels:
# - network: [tcp, udp]
#   address: <IP_ADDRESS>:19275 //这里换成 <IP_ADDRESS>:7897 也不行
#   target: <IP_ADDRESS>:465
#   proxy: 日本A01
rules:
  match,<IP_ADDRESS>:19275 // 这里换成 tunnels或者其他地址 都是优先匹配match后面的ip 也就是没响应tunnels配置

Description
问题1: tunnles 配置如上  tunnels流量没被监听到. address 替换成<IP_ADDRESS>:7897 也不行. 都是直接走了 rules match的代理
问题2: dialer-proxy 没生效,
proxies:

name: 日本SS
server: ****
port: 476
type: ss
dialer-proxy: 链式代理2
cipher: aes-256-gcm
password: ****
udp: true

proxy-groups:

name: 链式代理2
type: select
proxies:

🔰 选择节点

rules:
MATCH,链式代理2
其他配置如上, 也是直接走选择节点没有进入日本SS代理
Reproduction Steps
导入配置启动
Logs
No response

为什么意思哦  那一部分没有  following template

@hanyaxxx 中文用户使用英文模板不是不遵守模板是什么?


updated type sigs for function parameters

I've updated the type signatures so as not to imply that certain functions take curried functions when they don't. I've also tried to normalize the spacing and paren placement on functions with 2+ arity functions parameters.

Very nice!


Add Support for GitLab CI Code Quality Report Format

closes https://github.com/charliermarsh/ruff/issues/1298

You rock, thank you for this. Are you able to test the output format against Gitlab in practice? (I don't use Gitlab so a little difficult for me to test.)

@charliermarsh
Workflow:
include:
  - template: Code-Quality.gitlab-ci.yml

code_quality:
  image: python:3.10-slim
  allow_failure: false
  script:
    - pip install ruff
    - ruff . --format gitlab > gl-code-quality-report.json
  artifacts:
    reports:
      codequality: gl-code-quality-report.json

Output:

Awesome, thank you!


Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt

Environment :
tensorflow 1.0
cuda 8.0+cudnn 5.1
GPU: new titan x *2
python ./faster_rcnn/demo.py  --model /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcublas.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcudnn.so.5 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcufft.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcurand.so.8.0 locally
./faster_rcnn
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE3 instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX2 instructions, but these are available on your machine and could speed up CPU computations.
W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations.
I tensorflow/core/common_runtime/gpu/gpu_device.cc:885] Found device 0 with properties:
name: TITAN X (Pascal)
major: 6 minor: 1 memoryClockRate (GHz) 1.531
pciBusID 0000:02:00.0
Total memory: 11.90GiB
Free memory: 11.35GiB
W tensorflow/stream_executor/cuda/cuda_driver.cc:590] creating context when one is currently active; existing: 0x33471a0
I tensorflow/core/common_runtime/gpu/gpu_device.cc:885] Found device 1 with properties:
name: TITAN X (Pascal)
major: 6 minor: 1 memoryClockRate (GHz) 1.531
pciBusID 0000:82:00.0
Total memory: 11.90GiB
Free memory: 11.75GiB
I tensorflow/core/common_runtime/gpu/gpu_device.cc:777] Peer access not supported between device ordinals 0 and 1
I tensorflow/core/common_runtime/gpu/gpu_device.cc:777] Peer access not supported between device ordinals 1 and 0
I tensorflow/core/common_runtime/gpu/gpu_device.cc:906] DMA: 0 1
I tensorflow/core/common_runtime/gpu/gpu_device.cc:916] 0:   Y N
I tensorflow/core/common_runtime/gpu/gpu_device.cc:916] 1:   N Y
I tensorflow/core/common_runtime/gpu/gpu_device.cc:975] Creating TensorFlow device (/gpu:0) -> (device: 0, name: TITAN X (Pascal), pci bus id: 0000:02:00.0)
I tensorflow/core/common_runtime/gpu/gpu_device.cc:975] Creating TensorFlow device (/gpu:1) -> (device: 1, name: TITAN X (Pascal), pci bus id: 0000:82:00.0)
Tensor("Placeholder:0", shape=(?, ?, ?, 3), dtype=float32)
Tensor("conv5_3/conv5_3:0", shape=(?, ?, ?, 512), dtype=float32)
Tensor("rpn_conv/3x3/rpn_conv/3x3:0", shape=(?, ?, ?, 512), dtype=float32)
Tensor("rpn_cls_score/rpn_cls_score:0", shape=(?, ?, ?, 18), dtype=float32)
Tensor("rpn_cls_prob:0", shape=(?, ?, ?, ?), dtype=float32)
Tensor("Reshape_2:0", shape=(?, ?, ?, 18), dtype=float32)
Tensor("rpn_bbox_pred/rpn_bbox_pred:0", shape=(?, ?, ?, 36), dtype=float32)
Tensor("Placeholder_1:0", shape=(?, 3), dtype=float32)
Tensor("conv5_3/conv5_3:0", shape=(?, ?, ?, 512), dtype=float32)
Tensor("rois:0", shape=(?, 5), dtype=float32)
[<tf.Tensor 'conv5_3/conv5_3:0' shape=(?, ?, ?, 512) dtype=float32>, <tf.Tensor 'rois:0' shape=(?, 5) dtype=float32>]
Tensor("fc7/fc7:0", shape=(?, 4096), dtype=float32)
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]
Loading network VGGnet_test...
Traceback (most recent call last):
  File "./faster_rcnn/demo.py", line 124, in <module>
    saver.restore(sess, args.model)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1439, in restore
    {self.saver_def.filename_tensor_name: save_path})
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 767, in run
    run_metadata_ptr)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 965, in _run
    feed_dict_string, options, run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1015, in _do_run
    target_list, options, run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1035, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.NotFoundError: Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]

Caused by op u'save/RestoreV2_20', defined at:
  File "./faster_rcnn/demo.py", line 123, in <module>
    saver = tf.train.Saver()
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1051, in __init__
    self.build()
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1081, in build
    restore_sequentially=self._restore_sequentially)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 675, in build
    restore_sequentially, reshape)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 402, in _AddRestoreOps
    tensors = self.restore_op(filename_tensor, saveable, preferred_shard)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 242, in restore_op
    [spec.tensor.dtype])[0])
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_io_ops.py", line 668, in restore_v2
    dtypes=dtypes, name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 763, in apply_op
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2395, in create_op
    original_op=self._default_original_op, op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1264, in __init__
    self._traceback = _extract_stack()

NotFoundError (see above for traceback): Tensor name "conv4_2/biases" not found in checkpoint files /home/user/workspace/liuhao/5_TFFRCNN/TFFRCNN/Resnet_iter_200000.ckpt
	 [[Node: save/RestoreV2_20 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_20/tensor_names, save/RestoreV2_20/shape_and_slices)]]

Hi, I think you should can use tf_upgrade to update the code, or try tensorflow 0.11
https://www.tensorflow.org/install/migration


Install libnghttp2 on all FreeBSD jobs

@bors r+

:pushpin: Commit 32715d27062586ff5f653a4b05ab0864353c5f95 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 32715d27062586ff5f653a4b05ab0864353c5f95 with merge 2d2d74ce7059725f5c07aa0bf292cceef42de307...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 2d2d74ce7059725f5c07aa0bf292cceef42de307 to master...


Colored Glowstone Dust doesn't pick up correctly.

A method wasn't overrode properly.  If you pick up your first glowstone dust, it will behave appropriatly.  You'll pick up a green one or whatever.  But the SECOND dust you pick up, will always stack with the first color you picked up, even if it has a different metadata.
We'll need to find out what we missed there.

This was fixed a while ago for 1.3.6


The service should list all available node url

In GitLab by @wknet123 on Aug 21, 2018, 12:55

In GitLab by @yhua123 on Nov 8, 2019, 07:58
step1: George ping ip before response URL
Step2: Zhanghu get Node list from Kubernets Api

In GitLab by @wknet123 on Nov 8, 2019, 07:58
Fixed with PR #1703

In GitLab by @tmon546596046 on Nov 8, 2019, 07:58
now the node list has already retrived from kubernetes.

In GitLab by @Robin-Mao on Nov 8, 2019, 07:58
fixed

In GitLab by @sokril on Nov 8, 2019, 08:26
closed

In GitLab by @sokril on Nov 8, 2019, 08:26
assigned to @Robin-Mao


Security issues of malicious URLs

Currently mpv-handler passes the decoded URL directly to mpv, which may have security issues.
See https://github.com/mpv-player/mpv/issues/7985#issuecomment-670681943
If the URL is encoded by attacker:
av://lavfi:ladspa=file=/some/evil/so/here.so
=>
mpv://play/YXY6Ly9sYXZmaTpsYWRzcGE9ZmlsZT0vc29tZS9ldmlsL3NvL2hlcmUuc28=

Then the user clicks on the URL, it may cause security issues:
$ mpv-handler 'mpv://play/YXY6Ly9sYXZmaTpsYWRzcGE9ZmlsZT0vc29tZS9ldmlsL3NvL2hlcmUuc28='

Playing: av://lavfi:ladspa=file=/some/evil/so/here.so
[ffmpeg] Parsed_ladspa_0: Failed to load '/some/evil/so/here.so'
[ffmpeg/demuxer] lavfi: Error initializing filter 'ladspa' with args 'file=/some/evil/so/here.so'
[lavf] avformat_open_input() failed
Failed to recognize file format.

Exiting... (Errors when loading file)
Error: Downloader or player exited with error or termination signal

Like ytdl_hook.lua does, only some safe protocols are allowed:
local safe_protos = Set {
    "http", "https", "ftp", "ftps",
    "rtmp", "rtmps", "rtmpe", "rtmpt", "rtmpts", "rtmpte",
    "data"
}

local function url_is_safe(url)
    local proto = type(url) == "string" and url:match("^(%a[%w+.-]*):") or nil
    local safe = proto and safe_protos[proto]
    if not safe then
        msg.error(("Ignoring potentially unsafe url: '%s'"):format(url))
    end
    return safe
end

https://github.com/mpv-player/mpv/blob/master/player/lua/ytdl_hook.lua#L52-L56
https://github.com/mpv-player/mpv/blob/master/player/lua/ytdl_hook.lua#L155-L162


added stash_remover

If a check is coming back to OK, and there is a stash on this check,
stach_remover deletes this. So new error on this service won't be
ignored silently.

@chrigl
Can you take a look at the Travis output please.
Thanks!


Insert records into 'new' table then rename tables

Switchboard users are complaining that sometimes when they search
for a valid barcode they get no results and then later they do. Looking
at how sqlite-utils populates the table, we see the potential for this
happening: the delete and the insert are not in the same transaction,
and with a growing number of records to insert, inserting all records is
taking longer. This means that there is a moment when the table is empty
and many moments before all the records are inserted into the table.
This commit causes sqlite-utils to insert the records into a temporary
table (not a temp table in the DB sense, but a table with 'new' appended
to its name). A sqlite3 command drops the existing table and renames
the new table to the real table name in a transaction.

 Investigate scan-switchboard issue that Brian reported: switchboard often showing no results for HCT barcodes


CLI docs for avn service keypair command

What should we add to our content collections?
Documenting the avn service keypair that Aiven provides

Re-opening because we don't have information about what this command does, or an example of how to use it (also I just tried and it isn't obvious to me either how to use it or what it would do if I did!) https://developer.aiven.io/docs/tools/cli/service.html#avn-service-keypair


[Errno 2] No such file or directory: '/home/pi/output.png'

[Errno 2] No such file or directory: '/home/pi/output.png'
I get this error when I run the command python3 ~/zero-btc-screen/main.py
Please advise

Could you check if the "picture" screen is disabled in the configuration?

Same here.
Yes, disabling "picture" in configuration file solves this. Then I wonder what is purpose of this function in the first place?


Develop/Code Freeze for 0.8.0 release

This branch is used to aggregate all features and improvements (hot fixes should go to main branch and rebased here)

CC @secondsun


Feat/#521 enbale http2 consul config

Related issue: https://github.com/networknt/light-4j/issues/521
Default value of enableHttp2 in consul.yml is false.
If no such entry in consul.yml, the value also be false.

@jiachen1120 Could you please update the light-doc for this new config field?
https://doc.networknt.com/concern/consul/


Integrate with SauceLabs for cross-browser real-world testing

Running CI at Sauce in addition to travis would help minimize browser-specific bugs that aren't easy to test for in a headless environment like we use at Travis (phantom).

closed for #326


Jupyter notebook for syserol COVID data

working on markup draft, aim to finish in a week

Finished fixing markup.
Unsure if I can assume readers will have all dependencies installed already or if I need to mention which ones to install.


CI: rename installer deploy job from *-u7 to *-i7

What does this PR do?
This renames the installer deploy job, replacing u7 with i7

Motivation
Being consistant with the renamed product

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable DEPLOY_AGENT=true --variable RUN_UNIT_TESTS=on

This will require https://github.com/DataDog/test-infra-definitions/pull/748 in order for the new-e2e-updater tests to pass

/merge


Support AQL Sort and Limit

Added support for AQL sort and limit, for example:

{
  "files": [
    {
      "aql": {
        "items.find": {
          "repo": "libs-release-local"
        },
        "sort" : {"$desc" : ["name"]},
        "limit" : 5
      },
      "target": "files"
    }
  ]
}

This looks great; any chance of seeing it merged into a release soon? We're anxious to try it out.
For reference, this fixes #56.

Hi @kpersohn. I agree. We do want to merge this pull request.
We'll soon finish testing this and merge the code.

Here's an update:
We're considering an alternative for this implementation, to make the AQL you can define in the File Spec less restrictive. We'll update with more information as our research progresses.

Any timeline for this?

Sure @jgeorgeson,
We plan to commit the code to the dev branch next week, so that it can be added to the upcoming release next month.
We'll update this issue with some more details after the commit.

awesome.

My hope for this feature is the ability to have an AQL spec file that lets me delete all the the N youngest builds in a repo whose packaging/layout doesn't support automatic integration build cleanup. Is that something you expect to be able to support?

Yes @jgeorgeson,
The File Spec schema will be extended and include three additional properties - sort-by, which accepts any of item AQL domain fields, sort-order and limit.
These properties will also be exposed as command options.

@jgeorgeson,
Most of the sort and limit code has been pushed by @DimaNevelev into the dev branch.

@jgeorgeson and @kpersohn,
Version 1.12.0 adds sort and limit support. We'd appreciate your feedback for it. Since we can use https://github.com/JFrogDev/jfrog-cli-go/issues/56, I think we can closing this pull request.


Support serving static files

Adds support for serving static files with the via-serve-static crate by making the following changes:

Adds a Body enum to support streaming responses.
Adds a param method to Location to enable middleware authors to use dynamic path parameters without having use macros.
Implements Respond for tokio::fs::File. A similar approach can be used  in the future if we need to implement respond for other types that implement AsyncRead.
Creates a via-serve-static crate with the actual implementation of the static file server.

Closing this PR in favor of #3.


Cannot import load_dataset on Colab

Describe the bug
Got tqdm concurrent module not found error during importing load_dataset from datasets.
Steps to reproduce the bug
Here colab notebook to reproduce the error
On colab:
!pip install datasets
from datasets import load_dataset

Expected results
Works without error
Actual results
Specify the actual results or traceback.
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-2-8cc7de4c69eb> in <module>()
----> 1 from datasets import load_dataset, load_metric, Metric, MetricInfo, Features, Value
      2 from sklearn.metrics import mean_squared_error

/usr/local/lib/python3.7/dist-packages/datasets/__init__.py in <module>()
     31     )
     32
---> 33 from .arrow_dataset import Dataset, concatenate_datasets
     34 from .arrow_reader import ArrowReader, ReadInstruction
     35 from .arrow_writer import ArrowWriter

/usr/local/lib/python3.7/dist-packages/datasets/arrow_dataset.py in <module>()
     40 from tqdm.auto import tqdm
     41
---> 42 from datasets.tasks.text_classification import TextClassification
     43
     44 from . import config, utils

/usr/local/lib/python3.7/dist-packages/datasets/tasks/__init__.py in <module>()
      1 from typing import Optional
      2
----> 3 from ..utils.logging import get_logger
      4 from .automatic_speech_recognition import AutomaticSpeechRecognition
      5 from .base import TaskTemplate

/usr/local/lib/python3.7/dist-packages/datasets/utils/__init__.py in <module>()
     19
     20 from . import logging
---> 21 from .download_manager import DownloadManager, GenerateMode
     22 from .file_utils import DownloadConfig, cached_path, hf_bucket_url, is_remote_url, temp_seed
     23 from .mock_download_manager import MockDownloadManager

/usr/local/lib/python3.7/dist-packages/datasets/utils/download_manager.py in <module>()
     24
     25 from .. import config
---> 26 from .file_utils import (
     27     DownloadConfig,
     28     cached_path,

/usr/local/lib/python3.7/dist-packages/datasets/utils/file_utils.py in <module>()
     25 import posixpath
     26 import requests
---> 27 from tqdm.contrib.concurrent import thread_map
     28
     29 from .. import __version__, config, utils

ModuleNotFoundError: No module named 'tqdm.contrib.concurrent'

Environment info

datasets version: 1.10.0
Platform: Colab
Python version: 3.7.11
PyArrow version: 3.0.0

I'm facing the same issue too on Colab today.

@phosseini
I think it is related to 1.10.0 release done 3 hours ago. (cc: @lhoestq )
For now I just downgraded to 1.9.0 and it is working fine.

1.9.0

Same here, downgraded to 1.9.0 for now and works fine.

@phosseini
I think it is related to 1.10.0 release done 3 hours ago. (cc: @lhoestq )
For now I just downgraded to 1.9.0 and it is working fine.

Same here, downgraded to 1.9.0 for now and works fine.

Hi,
updating tqdm to the newest version resolves the issue for me. You can do this as follows in Colab:
!pip install tqdm --upgrade

Hi @bayartsogt-ya and @phosseini, thanks for reporting.
We are fixing this critical issue and making an urgent patch release of the datasets library today.


Replace funky imports with standard ones.

Lots of Guava imports were not coming from the actual Guava
junit.framework.Assert should be org.junit.Assert

:+1:

:+1: after travis


Support Poll Delay

Allow users to specify a polling delay (defaults to 100ms)

Updated based on your feedback.

Updated.

This looks good, I am going to re-enable the travis test runs to ensure none of the test break before merging it though.

Looks like it succeeds on Linux but fails on OSX?
In my environment I'm developing on Windows and running on Linux.
Look at historic builds, it looks like the OSX unit tests just fail from time to time?


[DUOS-293] Automation/Gatling Tests

WIP - do not merge
Addresses
https://broadinstitute.atlassian.net/browse/DUOS-293
Changes

automation directory with gatling test framework

Usage
Subject to change. With this branch, navigate to automation directory and run sbt gatling:test. Current tests will all be run and a report generated.
TODO

Sort out the ENV configurations
Run via docker image
Jenkins job to run this
See script necessities in DUOS-293

Coverage remained the same at 73.826% when pulling 6c8612b2bd4ec99b1e07fbc275df17b6b902c23d on DUOS-293 into 888477fdb5e27ae8441907926f938d03f4246ef3 on develop.

@dags FYI, this is not too far off from where I want it to be. Still digging into making this work inside the FC jenkins world (short story, it's not right now), but from a code and test organization perspective, I'd like your opinion on where we're going here. Thanks!

Nice implementation to learn from !.


Change API entry point or lib name.

The names of the library and API entry point should correlate. Either change the name of the lib to Plumbing, or change the name of the API entry point to Plumb.

We should change the name of the library to get away from the "Plumb" metaphor. I'm not sure if we want an "Rx" prefix.
Nouns that come to mind when thinking of this library and what it should be used for:

Fit (like fitting pipes?)
Cycle (The goal is to create cycles of streams)

I'm open to suggestions.

CycleRx, RxCycle

One thing that is steering me away from "cycle" is that we don't require you to create a cycle. You might have inputs or outputs without the other. In that sense, the "fit" verb makes more sense.

Going along with the stream/fit analogy. We're providing pipes and couplings. ¯_(ツ)_/¯

In an offline discussion, we settled on RxJoin.


Bug fixes for object IDs and key errors

Closes #101
The notebook now runs end-to-end for me with no errors (nice work everyone 🎉 ). The plots look like reasonable light curves to me, but others should definitely inspect them.
Included changes
Attach a persistent objectid to each SkyCoord object. The objectid is the index in the original, de-duplicated list. I put this change at the end of the remove_duplicate_coords function, and renamed that function to clean_sample. coords_list is now a list of tuples containing (objectid, coord). The archive calls are no longer responsible for constructing the objectid.
Code blocks which previously started like this:
for ccount, coord in enumerate(coords_list):
now start like this:
for objectid, coord in coords_list:
In the rest of the code block, a direct ccount -> objectid substitution gives the desired, bug-free behavior.
The objectids in df_lc now match the original index (i.e., they are no longer incremented by 1). If there was a particular need for the 1-based indexing, can someone let me know what that was? I changed it because Python is 0-based, so (a) I can't imagine a function/method having trouble with it, and (b) switching to 1-based makes it very easy for off-by-one indexing errors to creep in to the code.
Bug-fix the KeyError in the plotting cell. This was caused by syntax errors when accessing the multi-index in .loc. Note that if you're asking for a single value (e.g., a single objectid) that index level is completely removed from the dataframe that is returned.
Bug-fix the KeyError in the ZTF archive call. I'm fairly certain this was caused by a floating point comparison issue with the time. The fix was to not use the time here, but instead to drop everything with the same objectid, label, and band.
Potential problems not addressed by this PR
The two lists labels_list and object_names are still implicitly assumed to have ordered, one-to-one relationships with the original coords_list that is returned by sample_selection.clean_sample. This does not currently cause problems (that I am aware of), and it allows this PR to fix the current bugs without changing a lot of extra code. However this could easily cause bugs in the future because the relationships are not robustly enforced, thus anyone changing code without fully understanding the implicit relationships risks getting unexpected results.
It would be better to tie all of the input arguments for one astronomical/literature object together into a single python object. Implementing this would require significant changes to the code, so I'm guessing that we don't want to do it for this notebook. However, similar issues related to the organization of input arguments also came up in the forced photometry notebook. So I'll open an issue that describes my suggestions and can be a place for general discussion/ideas for future notebooks.
Finally, the changes here will not affect this issue, which will need to be addressed separately:

I have also run into issues with the object id later on in the ML work.  When I remove some of the sample from the df_lc, then I get holes in the object ids, which causes problems.
Originally posted by @jkrick in https://github.com/fornax-navo/fornax-demo-notebooks/issues/98#issuecomment-1528108764

@jkrick @afaisst @xoubish Will you all take a look at this, at your convenience? I think it touches stuff that everyone worked on.

Letting all of these changes settle into my brain.....
What/which change(s) do you think is/are responsible for the multiprocessing chunking now working properly?  ie., the plots look good to me now with chunking.

What/which change(s) do you think is/are responsible for the multiprocessing chunking now working properly? ie., the plots look good to me now with chunking.

@jkrick Attaching a persistent objectid to each SkyCoord object up front so that the archive calls are no longer responsible for constructing it. (It used to be that each archive call would assign an objectid based on the object's index in the coords_list that get's passed in as an argument. So when the list got sliced into chunks for parallel processing, multiple different objects would end up with the same objectid, since the indexes for each chunk start over at 0.)

I'm going to add the 'vectorize archive calls' as an issue outside of this PR.  It's a good comment that we can ask the archives to do this for the benefit of all....

I think this looks great, and am very excited to see this notebook be fast and functional. Merge when you are ready!

This looks good!


Add information for usage custom registries on e2e testing #4488

Hi, this PR provide more information about how to modify the custom registry configuration to running E2E test.
Refers to: #4488

This should probably be in the root-level CONTRIBUTING.md, it has no bearing to react-scripts.

Agree, if the text is ok for you. I'd suggest the following structure.

Nice work @juanpicado 👏

🥁

❤ thanks


Assigning to a block variable changes the actual block (thus affecting block_given? and yield)

def foo1(&block)
  block = proc {}
  p block_given?
end
def foo2(&block)
  block = nil
  p block_given?
end
foo1
foo2 {}

Ruby prints false true, while mruby prints true false

In CRuby, block is a copy of a given block reference. In mruby, block is a reference itself. This is primarily to reduce memory consumption. So please don't assign to the block arguments.
We will fix this someday. But the priority is low.

Interesting. Indeed it changes the block itself, so something like block = nil; yield would fail. I'll rephrase the issue title.
I don't mind this behavior if it's intentional, with a note added to limitations.md - or perhaps it would even be possible to issue a warning on such an assignment?
This alias/reference semantics is unique to block variables? Would it not be a potentially interesting new language feature, e.g.
foo = 1
bar = ref(foo) # via some imaginary ref()
bar = 2 # changes foo

To answer the question, it won't happen for other arguments, as far as I intended.

This bug is fixed by 9c5812a4633e65b4c10cd212a3ae3f87a249a61c


Error of fa calculation

The falarm value is wrong when the falarm occurs at the begining of the rttm file. The following is an example:
"""
(enh)# cat ref.rttm
SPEAKER bkwns 1 10.00000 35.320000   spk01
SPEAKER bkwns 1 48.160000 0.800000   spk00
(enh)# cat sys.rttm
SPEAKER bkwns 1 0.00 45.12   spk01
SPEAKER bkwns 1 45.12 0.4   spk00
(enh)]# ./md-eval-22.pl -r ref.rttm -s sys.rttm
command line (run on 2022 Oct 21 at 17:17:48) Version: 22  ./md-eval-22.pl -r ref.rttm -s sys.rttm
Time-based metadata alignment
Metadata evaluation parameters:
time-optimized metadata mapping
max gap between matching metadata events = 1 sec
max extent to match for SU's = 0.5 sec
Speaker Diarization evaluation parameters:
The max time to extend no-score zones for NON-LEX exclusions is 0.5 sec
The no-score collar at SPEAKER boundaries is 0 sec
Exclusion zones for evaluation and scoring are:
-----MetaData-----        -----SpkrData-----
exclusion set name:     DEFAULT    DEFAULT        DEFAULT    DEFAULT
token type/subtype      no-eval   no-score        no-eval   no-score
(UEM)              X                         X
LEXEME/un-lex                    X
NON-LEX/breath                                              X
NON-LEX/cough                                               X
NON-LEX/laugh                                               X
NON-LEX/lipsmack                                            X
NON-LEX/other                                               X
NON-LEX/sneeze                                              X
NOSCORE/            X         X               X         X
NO_RT_METADATA/            X
SU/unannotated               X
*** Performance analysis for Speaker Diarization for ALL ***
SCORED SPEAKER TIME =36.120000 secs
MISSED SPEAKER TIME =0.800000 secs
FALARM SPEAKER TIME =0.200000 secs
SPEAKER ERROR TIME =0.200000 secs
OVERALL SPEAKER DIARIZATION ERROR = 3.32 percent of scored speaker time  `(ALL)
---------------------------------------------
Speaker type confusion matrix -- speaker weighted
REF\SYS (count)      unknown               MISS
unknown                   1 /  50.0%          1 /  50.0%
FALSE ALARM             1 /  50.0%
---------------------------------------------
Speaker type confusion matrix -- time weighted
REF\SYS (seconds)    unknown               MISS
unknown               35.32 /  97.8%       0.80 /   2.2%
FALSE ALARM          0.20 /   0.6%
---------------------------------------------
"""
The falarm time is 0.2s . However sys.rttm contains spk01 from 0-10s, which is not included in the falarm time. The correct falarm time should be 10.2s.

This is an issue in the underlying implementation by NIST's tool. For consistency with how NIST has scored things, I won't touch this. However, if you want a more correct implementation, I'd recommend using pyannote's implementation.

But it seems like dscore implements it correctly here, or am I missing something?

Looks like I replied in haste originally. There is not in fact a bug in that antiquated Perl script. Since no UEM was supplied, the tool defaulted to [min_ref_onset, max_ref_offset] (as pointed on by @desh2608 in the linked code snippet). In this case [10, 48.96]. Which is why the initial 10 seconds of speech from the system RTTM is not scored. To get the desired behavior, supply a UEM that marks the entire recording for scoring.

I think the antiquated Perl script is indeed wrong. It gets UEM only from the reference turns, which would underestimate false alarms as pointed out by the OP. Your tool, however, would compute the DER correctly since it estimates UEM from both the ref and hyp turns.


Build and host our own Chat Server, get off Discord/Slack

ao_chat! Let's make it happen!
Discord is ridiculous for too many reasons to list.
Someone please save us!
We will provide cloud infra (how's AWS?) and cover all other costs.
Possible candidates:

https://once.com/campfire
https://mattermost.com/
? any suggestions? Pure IRC and see what happens? A massive whatsapp group (please shoot me)?

Requirements:

owning our own history
doesn't make us broke
keeps our chat less noisy than the wild and places like discord
makes it so that more serious people join, not just discord randos

The other options I'm aware of are rocket.chat and matrix
I'm pretty sure they both can be self-hosted.
Matrix looks like a good option in terms of control and cost, but I doubt it'll be as smooth as Slack or Discord.
Campfire also looks good in those regards, but with more up-front cost since there's a one-time purchase.
Part of this decision should be figuring out how big the server will get, though.


error in response to Subscription::create

I'm getting a 'error converting between wire format and Rust types: invalid type: null, expected a string at line 34 column 26'. This was working previously. Stripe succeeds at creating the subscription and Stripe's body looks ok.

Sorry. The bug was on my side.

Ah sorry I haven't responded to this early.
Have you checked whether the null field is null-able according to the Stripe API?
This type of error often occurs when I forgot to make a field Option<_> in the implementation.


Document PreserveTaxonomyNames

Closes #1648

Thanks!
Merged as 280b2f2 (master) and f47fd0c (v0.15.docs).


Damageable item handling confuses RS

Issue description:
Damageable items seem to have inconsistent behavior in modded minecraft and this can confuse RS's damaged item heuristics and leads to crafting malfunctions.
What happens:
In my test world, the cutting tool is ID 6680 with no ID metadata.  Damage is stored in NBT. Most items seem to be like this including Shears, Immersive engineering wire cutters, hammer, etc.
Other mods (eg: IC2) tend to store the damage value in ID metadata.  eg: forge hammer might be 1123/0 for undamaged, 1123/1 for 1% damaged etc.
When RS does crafting with an ingredient that is marked as damageable, it has a heuristic to ignore both NBT and ID/metadata.
This is fine until you try and craft with items like the Thermal Expansion Flux Capacitors.  In my case, the tiers are: basic flux capacitor is 4429/0, hardened is 4429/1, reinforced 4429/2 etc.
Crafting each tier requires the previous one.  However, they are chargeable items and RS applies the damage rules - ignore NBT and ignore ID metadata.  When doing this RS can't tell the difference between the unique items 4429/1 and 4429/2.
Example:  with this in the system: https://i.imgur.com/wZjpoSW.png
this malfunction happens: https://i.imgur.com/MQQXSst.png
What you expected to happen:
I expected items with unique ID/metadata to be treated as unique unless I enabled substitutions, even though that would break mods like IC2 with its damageable items.
I would suggest making the damage equivalency more selective.  Specifically, Damageable items should continue to ignore NBT but should NOT ignore player-visible ID/metadata differences.  Also, so that IC2-style damageable items can be reused, either:

change "OreDict" in patterns to "Allow substitutions" (which would allow both ID/metadata substitutions and oredict variations
or add an additional pattern checkbox: "Ignore Metadata"

In the 1) case, it would be "[X] Processing   [X] Allow substitions", while 2) would be "[X] Processing   [X] Ignore Metadata   [X] Oredict".
If you wanted to treat the ignore-damage as a special case for the cutting tool, then there would need to be an additional pattern property, eg: '[X] Ignore NBT'.  If there's no user-accessible toggle for this in the pattern interface then there should be a configurable whitelist for users and pack makers list item-id's that should be damage-ignored.  "Ignore NBT" and "Ignore Metadata" are not equivalent in the TE Flux Capacitor case.
Version

Minecraft: 1.12.2
Forge: <IP_ADDRESS>59
Refined Storage: 1.6.6-289 (Jenkins dev build)

Was first encountered a few versions ago. I suspect (but have not verified) that it applies to the entire 1.6 line.
Does this issue occur on a server? Yes (and single player)

I think there might have been an oversight in the heuristic.  The ICompare.COMPARE_NBT / COMPARE_DAMAGE checks didn't have code for handling subtypes.
I've seen sample code for comparing damage equivalence that also checks getHasSubtypes() to determine whether getMetadata() should be a factor.

Hijacking this for some brainstorming/ideas on the issue. I can think of 3 issues that the Oredict/metadata/NBT thing comes with.

The magic button. This is the biggest one IMO. Most players have no or only a very rough idea what Oredict/metadata/NBT actually means. That means you either need to show exactly what the option does or make it reasonable to figure out. If there is a solution with just one option to test that would maybe work. Adding a button for each just results in players enabling them on everything.

All or nothing. In some rare cases, it might be desirable to enable Oredict on only a part of the recipe. For example, a recipe that uses copper and thermal expansion hardened glass. You want all types of copper but not all types of hardened glass.
Similarly, wanting only hardened glass and copper hardened glass to be used in crafting is currently not possible.

Calculation time. Mostly in correlation to issue 1 when players just enable the Oredictionary on all their recipes. No idea about fixing this one as I do not know the code (Caching?).

My still very unrefined solution would be a type of filter interface.  But each showing a preview of the Items that match that filter.
Not sure if possible but maybe an option that ignores specific NBT tags? For example, when right-clicking a chargeable item it has an option for ignore Energy.
A config for relevant NBT tags would then be needed I believe. Custom filters seem easy to add then too.
Still don't like how bulky this makes the whole process though. Maybe someone else has better ideas.


Create b1-log-file-system-rep

baturc
Id: 55f4d51459189501
Version 0.3.2

rejected:B1
It looks like you are trying to submit B1, however it seems like the following submission criteria are problematic:

Logs are missing
We’ve thus marked B1 as rejected, however if this is a mistake, please upload the log summary and we can re-review.


Only filter on plugins if a plugin getter is provided.

If there is no plugin getter this assumes that all drivers etc are built-in and
this always available, and when they are not proper errors will be generated
and propagated.
This avoids the need in https://github.com/docker/swarmkit/pull/1965 to
populate a spurious api.NodeDescription.Engine field (spurious because there
is no engine in this case).
Signed-off-by: Ian Campbell<EMAIL_ADDRESS>The api.NodeDescription.Engine field is https://github.com/ijc25/swarmkit/blob/42f853a3c262666e29d4c0bd3ea301d5e736bc82/agent/exec/containerd/executor.go#L93...L101 in the current PR (but I expect that link to break when I next force push there) it is the executor.Describe method and is just:
		// Possibly this should have a Containerd entry
		// instead. There is some code which
		// `desc.Engine.Plugins` to be non-nil see
		// `PluginFilter.Check` in
		// `manager/scheduler/filter.go`
		Engine: &api.EngineDescription{
			Labels:  map[string]string{},
			Plugins: []api.PluginDescription{},
		},

As an alternative solution, I would suggest changing (*PluginFilter).Check to return true (bypassing the filter) if n.Description.Engine == nil. This would avoid manipulating the list of filters, and also avoid making the decision based on whether a PlugginGetter is available on the leader node. Would this work for you?

@aaronlehmann That sounds much better and I think ought to work just fine, I'll make the change.

Force pushed (a much smaller change now!) and update title + head comment to reflect new commit message.

Is it correct that this is temporary behavior to support #1965? I imagine that eventually we will want to validate the drivers for volumes and networks against some variant of Description which doesn't exist yet? If this is correct, it might be good to add a comment explaining what will change to make the plugin filter operate correctly once the information is available.

@aaronlehmann that's what I was trying to get at with "built-in and this always available, and when they are not proper errors will be generated and propagated." (oops, s/this/thus/) in the commit message.
Right now in my (unpublished) CNI POC based on the #1965 work the network plugin here is "cni" (at @al's suggestion), so there isn't much to check. The actual cni backend/plugin name is in the CNI JSON config blob, which for the most part of opaque to us, although the relevant field type is one of the ones I think it would valid for us to inspect. If we did that then we would be able to provide the listing of /opt/cni/bin on each node in the corresponding info and check against that (that wouldn't catch e.g. using an ipam plugin as a network plugin or vice versa, but would be better than nothing).
I haven't thought much about storage yet, but if we guess we might go the csi route I would expect something similar there.
I had vaguely considered adding a method to the executor interface (or a new interface which the executor could implement) and plumbing the executor through to here so it could be asked, but I guess that falls afoul of using the leader's executor to answer node specific questions.

LGTM

LGTM


Revamp the Falco.org landing page

/area documentation
What would you like to be added:
As a user, I would like to get the most needed information about Falco as soon as I land on Falco.org.

As a prospective user or a curious explorer, I would like to have instant answers on Falco's capabilities, architecture and use cases.
As a prospective user, I want to be able to try Falco as quickly as possible
As a prospective user, I want to understand whether Falco is well supported and has a clear roadmap
As a prospective user, I want to find whether others in my industry are using it, and whether they validate my interest in Falco
As an experienced user, I would like to be able to quickly navigate to the documentation that I need

Why is this needed:
A beautiful, modern, and easy-to-navigate landing page is table stakes for the adoption of any open source project. It's important for our users to find the information they need quickly, and understand Falco's architecture quickly and easily.

Adding the proposed design of the new falco.org landing page. Main features include:
Section 1:

Additional item on the top level menu called "About"
Falco description + high level diagram
Clear CTA called "Try Falco"

Section 2:

Falco use cases
What makes Falco different?
Testimonials (it will be hidden until we have a real content to put there)

Section 3:

Featured videos, news and events
Easy links to join the community
Highlight that we are a CNCF project

Section 4:

Hopefully wall of logos with 100s of users :)

The PDF has notes/annotations with extra explanation for clarity.  We are working on the implementation and the PR will be submitted shortly. Please, share your feedback soon.
Falco_Landing_page (1).pdf


appservice: Fix createDeployWizard and update utils

Fixes https://github.com/microsoft/vscode-azurefunctions/issues/3951
Fixes https://github.com/microsoft/vscode-azurefunctions/issues/3952

Sorry, trying to get a build out for CTI. Someone please review later 🙏


Appveyor MinGW test results need improvement

Currently, when a test fails, the build fails, without any clue which test failed and why. This is probably due to the -junit switch being used.
We either need to use -v -junit, or use gtest the way it now works on Travis.

I'm puzzled about the "or use gtest" comment? What do you mean by that?

I looked at the yml, and it may not be the way to go here -- but remember that @mmidor added the -lg option? He also added to the travis script the following lines:
    ctest -V || exit 1

as well as code in the CMakeList.txt to run the tests group by group. The output of this can be seen on the Travis CI cmake build.

In the case of Appveyor, it has its own way of running the tests, based on xml junit output, but the script is malfunctioning / incomplete in that failures are not printed to the console. Maybe just a line is missing.

Correction: I think ctest -V || exit 1 has always been there, but the code in CMakeLists.txt has been improved.

@basvodde - you were so fast doing 3.7.1 :P
In my thinking, the MinGW / Appveyor stuff is still in the middle of being put together...

Fast is different for a change. I'll go back to being slow soon again :)
The Appveyor things aren't needed for the release. The MinGW is now compiling, and that was the main reason for putting out 3.7.1. Now I can update the homebrew scripts :)

I didn't realize that information about (failing) tests is displayed on a separate "Test" tab in AppVeyor. Very nice, actually.

@arstrube I did not modified Travis script. I added more code to tests/CMakeLists.txt to run each group of test separately and to get more verbose output from CTest.

@mmidor Yeah, I realized that when I looked more closely. Either way, the output is better than before. Could you pls have a look at issue #638? It is really a problem in the TestRegistry; just happened to show up in ctest unexpectedly...

@basvodde oh, I only noticed now that I seem to have a thing mixing up gtest and ctest. I meant ctest -- but actually, the Appveyor script does it much better.


bootBASIC doesn't boot

The [https://github.com/nanochess/bootBASIC bootBASIC] image doesn't boot, whether I use Standard PC or ACPI. As a floppy disk, it's stuck at Downloading images [2/3] 100%, but the serial console appears. As a CD, I get code 0003. As a hard disk, it simply says it couldn't be read.

Seems to be working fine: https://copy.sh/v86/?profile=bootbasic
It needs to have the correct size to detect the floppy type. There's an error in the console. I extended it as follows:
dd if=/dev/zero of=bootbasic.img ibs=1k count=1440
dd if=basic.img of=bootbasic.img conv=notrunc


Delayed appearance of packages on the Archive

In the context of archlinux/infrastructure#531 we are currently thinking about whether packages appearing on the Archive with some delay could cause issues for the repro infrastructure 🤔
As far as we understand it right now the packages are exclusively synced from the archive:
https://github.com/archlinux/archlinux-repro/blob/6e8cee92270c127a67ef27414116b6c45deffc76/repro.in#L24
https://github.com/archlinux/archlinux-repro/blob/6e8cee92270c127a67ef27414116b6c45deffc76/buildinfo#L72
In the new setup that could cause some issues if the rebuilder picks up the package faster than it is synced to the archive. So we wanted to ask about input to this, would it be good just to add a fallback to the T0 or geo mirrors? Or some kind of retry mechanism..? 🤔

I think we could query both URLs. We already have code for this as we need to check for xz and zst extensions to files.
So if devops needs to delay sync, we can implement this :) No worries.

I also think it'd be fine, the scheduler part of the repro infra has a retry mechanism, so if the first attempt fails it's going to try again after some time.
This was also common back when we had to wait until the PKGBUILD became available in the svntogit repo. It causes a "Packages which have become not reproducible" email notification though.


localhost mapping

Hello,
I have finally tried aspire, was hard to setup in my case because I am developing on an Apple Silicon (M1) and using Parallels Windows ARM for Visual Studio and Linux (Fedora) for Podman.
And The problem I have is I have setup on mac in etc/host the DNS for Fedora Linux so I can access in the Windows VM the Podman containers via DNS. On command line everything is fine, and the environment works fine.
MAC etc/hosts file:
<IP_ADDRESS>     fedora.podman

I have not found any possibility to map in Aspire localhost to fedora.podman for resource like Postgres (which is running as a container in fedora Linux as a Podman container) or any other resource.
What i try to do is something like this:
var builder = DistributedApplication.CreateBuilder(args);

// Secrets
var postgresPassword = builder.Configuration["postgres.password"];

// Resources
var postgres = builder.AddPostgres("postgres", password: postgresPassword)
    .WithImageTag("16.2");

var dbA = postgres.AddDatabase("db-a", "db.a");
var dbB = postgres.AddDatabase("Db-b", "db.b");

// Apps
builder.AddProject<Projects.SOME_PROJECT>("some-project")
    .WithReference(dbA)
    .WithReference(dbB);

builder.Build().Run();

this results in that the connection string is like this:
Server=localhost;Port=5432;Database=db.a;User ID=XXX;Password='XXX'
I need to override for this resource to not use localhost but instead fedora.podman.
Is this missing or not supported?

This isn't possible today. The only feature we have today around localhost mapping is when the containers need to access resources on the host, we have a setting that replaces localhost with host.docker.internal (which is configurable).
This is similar but because your container host is running remotely. We'd need another setting for host -> container host.

Tangentially related to https://github.com/dotnet/aspire/issues/2639

Would it make sense then to introduce a broader concept here, which can be also leveraged by other future tools.
The concept would be to put resource into groups or so called "containers" (not docker containers or K8s). and a group or "container" can have associations to Docker, Podman K8s or things like that. In future may be web assembly containers. this could also help to generate out manifest to support multi cloud deployments or not?

Sounds great for the future, the only problem is that it would be in the far future 😅 as this is not the 90% case (AFAIK). Having given this more thought, by default when you add endpoints to containers thy go through our proxy. Even if the container is on a remote host the proxy is local.
cc @karolz-ms for ideas on short term and long term solutions

@davidfowl I have already tried those things:
appconfig:
"AppHost": {   "ContainerHostName": "bits.fedora" }
using this package: <PackageReference Include="Aspire.Hosting" Version="8.0.0-preview.5.24161.15" />

@biqas can you tell me what is shown in the ports section of an inspected container when you run podman container inspect from your Windows VM on some container with mapped ports?
I am thinking this may be as simple as a bug in the Aspire orchestrator in the sense that it assumes containers will always be exposed via one of the loopback interfaces when configuring proxies.


Types Generated using the LibraryTypesMacro throws an error when instantiating

I've been experimenting with the new feature Macros and I'm trying to use it to introduce a new class/Type in a library. The editor successfully shows me the library augment but I cannot instantiate this new class.
This was tested using the following:

macros: ^0.1.0-main.5 using Dart SDK version: 3.5.0-180.3.beta (beta) (Wed Jun 5 15:06:15 2024 +0000) on "linux_x64"
macros: ^0.1.2-main.4 using Dart SDK version: 3.5.0-307.0.dev (dev) (None) on "linux_x64"

Minimally reproducible example:

Create a new package  dart create -t package macro_library_bug
Edit the analysis_options.yaml file to enable macros

... redacted contents of analysis_options.yaml
analyzer:
  enable-experiment:
    - macros

Edit the pubspec.yaml file and add a dependency on macros

... redacted contents of pubspec.yaml
dependencies:
  macros: ^0.1.2-main.4

Run dart pub get
Edit the lib/src/macro_library_bug_base.dart file and create a new LibraryTypeMacro

import 'dart:async';
import 'package:macros/macros.dart';

macro class LibraryAnnotator implements LibraryTypesMacro{
  const LibraryAnnotator();

  @override
  FutureOr<void> buildTypesForLibrary(Library library, TypeBuilder builder) {
    final randomType = "MyLibType";
    builder.declareType(randomType, DeclarationCode.fromParts([
      "class $randomType {",
      "\n\tconst $randomType();",
      "\n}"
    ]));
  }
}

Edit the lib/macro_library_bug.dart file and annotate the library with @LibraryAnnotator()

@LibraryAnnotator()
library macro_library_bug;

import 'package:macro_library_bug/src/macro_library_bug_base.dart';
export 'src/macro_library_bug_base.dart';

Edit the example/macro_library_bug_example.dart file

import 'package:macro_library_bug/macro_library_bug.dart';

void main() {
  final newLibType = MyLibType();
  print(newLibType);
}

Run the example file with the command dart --enable-experiment=macros example/macro_library_bug_example.dart

Expected Result
The console should show Instance of MyLibType
** Actual Result **
example/macro_library_bug_example.dart:4:9: Error: 'MyLibType' isn't a type.
  final MyLibType newLibType = MyLibType();
        ^^^^^^^^^
example/macro_library_bug_example.dart:4:32: Error: Method not found: 'MyLibType'.
  final MyLibType newLibType = MyLibType();

Looks like this is just not implemented in the front end, but is implemented in the analyzer (which is why you can see the class, but it isn't actually there when you run the app).
It is expected that you will run into unimplemented stuff at this time, which is why I marked the priority as P2 for now, but this definitely should get implemented prior to launch.


New folder structure breaks theme.link and theme.path

With the new folder structure, theme.link and theme.path point to the /theme folder and not to the theme root folder, and that feels wrong.
Previously:

theme.link would return http://example.org/wp-content/themes/my-timber-theme
theme.path would return /wp-content/themes/my-timber-theme

And now:

theme.link returns http://example.org/wp-content/themes/my-timber-theme/theme
theme.path returns /wp-content/themes/my-timber-theme/theme

I'm using Timber for the first time in the website I'm working on, so if I misunderstood something I'd love to get some feedback on this.
Also, as I like this idea of having the theme files in the /theme folder, I'm bypassing the issue I pointed by setting a context variable in my functions.php and using its value instead of calling theme.link:
$context['theme_uri'] = dirname( get_template_directory_uri() );

Good point.
I suggest you not to use an intermediate variable but explicit ones for e.g the static/ directory.

Thank you, Viktor.
I've set a global context variable static_dir pointing to static/ for the cases in which I need to link files that live in that folder.
And how would you fix the link below? It's currently broken in the starter-theme, right?
https://github.com/timber/starter-theme/blob/37fe91997483e338f6e405a732fc340a41302a8c/views/html-header.twig#L12
I had to keep a context variable theme_dir for this specific case.

There are a plethora of ways we may go!

Move humans.txt file to /theme
Patch stuff in html-header.twig
Make Timber provide a root path variable ✅
Make the starter-theme do the same

@szepeviktor is there an upside to the new structure? i spent my afternoon setting this up to play along with my gulp config and i cannot get it to work the way i want. i can't go up from the theme folder and it looks like i will have to compile into the theme/dist folder to get it to work.

AFAIK If you are compiling thing like SASS into CSS - you should put all of that in the root, above theme/.

Thanks for the heads-up on this @fturcheti. The issue is definitely an unintended side effect of what we saw as a way to clean things up and improve the cleanliness of the organization.
What I'm thinking is that we version the prior structure as the 1.x version of the starter theme and this new org marks the beginning of the 2.x branch. This also will allow us to lean into the new features of 2.x w/o worrying that we're messing up the workflow of existing patterns and development expectations.

This is now set. Fans of the new organization can find it here: https://github.com/timber/starter-theme/tree/2.x
... this will become the default once 2.x is released. But in the meantime (esp. for WordPress.org deployments) the old familiar file stricture scheme reigns supreme!

@fturcheti My approach to this is probably very bad… what I do is override link() by subclassing Timber\Theme inside of the Timber\Site subclass in my custom theme. I'm not really fluent with PHP and not really knowledgable.
class Website extends Timber\Site
{
    public function __construct()
    {
        add_action('after_setup_theme', array($this, 'theme_supports'));
        add_filter('timber/context', array($this, 'add_to_context'));
        add_filter('timber/twig', array($this, 'add_to_twig'));
        add_action('init', array($this, 'register_post_types'));
        add_action('init', array($this, 'register_taxonomies'));
        parent::__construct();

        $theme = new class() extends Timber\Theme
        {
            public function link()
            {
                return dirname(get_stylesheet_directory_uri());
            }
        };
        $this->theme = $theme;
    }
…

then I override the global context:
…
    public function add_to_context($context)
    {
        $context['theme'] = $this->theme;
        return $context;
    }
…

What don't you introduce a new theme.something that point to the directory of interest?
theme.assets 👉🏻 my-timber-theme/theme/assets

@szepeviktor I guess your approach is way more sane and it makes sense.
I just wanted to try and see if I could make it work and also because in my mind {{ theme.link }}, {{ site.theme.link }}, {{ theme.path }} and {{ site.theme.path }} should logically return the link or the path of the theme's root folder and not wherever index.php and style.css are.
I guess the thing about putting those in a /theme/ subdirectory is fundamentally flawed and not the WordPress way but I like it this way…


[@firebase/admob] Angular implementation of Native Ads

To me, the Angular implementation of Native Ads doesn't look straight-forward. Would it be possible to add an explanation on that for newbies?
Awesome work with the plugins, btw!

Sorry about that after importing the module you can use them in your view as follow
<BannerAd
  height="100"
  width="100"
  [unitId]="bannerAdUnit"
  (layoutChanged)="bannerLoaded($event)”
></BannerAd>

Added in https://github.com/NativeScript/firebase/commit/85ca9dbf80eb3fb19d7c802e4bd296a9b3003626


Supported monitor missing ("duplicate" serial numbers?)

This widget is working great, and is a nice improvement over the built-in control.
Issue: it appears to be detecting/showing controls for only two of my three monitors. The third monitor (Display 3 in below output) works with the built-in plasma display brightness control but is missing from this widget.
output of ddcutil detect:
Display 1
   I2C bus:  /dev/i2c-4
   DRM connector:           card0-DP-2
   EDID synopsis:
      Mfg id:               GSM - Goldstar Company Ltd (LG)
      Model:                E2442
      Product code:         XXXXX  (0xXXXX)
      Serial number:        XXXXXXXXXXXX
      Binary serial number: XXXXXX (0xXXXXXXXX)
      Manufacture year:     2013,  Week: 1
   VCP version:         2.1

Display 2
   I2C bus:  /dev/i2c-6
   DRM connector:           card0-HDMI-A-1
   EDID synopsis:
      Mfg id:               GSM - Goldstar Company Ltd (LG)
      Model:                E2342
      Product code:         XXXXX  (0xXXXX)
      Serial number:
      Binary serial number: XXXXXXXX (0xXXXXXXXX)
      Manufacture year:     2011,  Week: 1
   VCP version:         2.1

Display 3
   I2C bus:  /dev/i2c-8
   DRM connector:           card0-DP-1
   EDID synopsis:
      Mfg id:               PHL - Philips Consumer Electronics Company
      Model:                PHL 243V7
      Product code:         XXXXX  (0xXXXX)
      Serial number:
      Binary serial number: XXXXX (0xXXXXXXXX)
      Manufacture year:     2020,  Week: 30
   VCP version:         2.2

Looking at this code snippet:
https://github.com/davidhi7/ddcci-plasmoid/blob/3c002d9822ce4534bff2ed2e14b4f8ed8da5eef8/backend/ddcci_plasmoid_backend/ddcci.py#L50-L56
I suspect this may have something to do with two of the monitors having no Serial number? (though they do both have a Binary serial number).

I just released version 0.1.3 which should fix this bug. To apply the fix, update the backend with the command pip install -U ddcci_plasmoid_backend. Thank you for your report.

Can confirm it is fixed. Thanks!


chore: prettier everywhere once and for all

somehow my editor keeps on creating prettier changes which is quite annoying and noisy in PRs.
E.g. https://github.com/opencollective/opencollective-frontend/pull/1098/commits/037718be9ea691722b07e1f0546d00b2d04cbecb
Can we update frontend and api with the prettier rules (max col width: 120) and make sure we enforce this in all PR?
Also, let's keep in mind that we cannot expect external contributors to change their editor, so let's make sure that we take that into account.

Looks like this changes might be due to an update in Prettier. We should used a fixed version of Prettier in the future.
We used to have quite big PRs in the pipeline in the last days and this is why we were waiting for the big change.

Ok cool, the sooner the better obviously.

Done.
https://github.com/opencollective/opencollective-api/pull/1541
https://github.com/opencollective/opencollective-frontend/pull/1025


buildWordFrequencies count frequencies wrongly.

            for (final String word : words) {
                final String normalized = normalize(word);
                if (!wordFrequencies.containsKey(normalized)) {
                    wordFrequencies.put(normalized, 1);
                }
                wordFrequencies.put(normalized, wordFrequencies.get(normalized) + 1);

Since there is no else, each frequency is counted as frequency + 1.

The most elegant solution is to just initialize the map with 0 instead of 1 =)

Good catch. :)
I replaced it with:
                wordFrequencies.put(normalized,
                                    wordFrequencies.getOrDefault(normalized, 0) + 1);

I did not do a version bump/release since this doesn't really affect anything functionally.


Export VueDraggableDirective as separate name

Is your feature request related to a problem? Please describe.
I would like to use this directive in specific components, but not globally.
Describe the solution you'd like
if the VueDraggableDirective from index.js could be exported, I could then add it to a specific component within its directive section
Describe alternatives you've considered
I've considered copying the directive definition into my own project and using it as needed, but figured iId see if this was something y'all might add first.

Hi @alex-nextraq ,
Thanks for the feedback.
VueDraggableDirective is exported and section about local directive usage is added in readme.


Support assert_equal for CTEs

Problem
Currently, the library only allows to check for equality on the final query result. In case a query has a lot of CTEs, the "unit tests" turn into more end-to-end like tests in those cases.
We want to be able to also test parts of the query logic independently and check whether certain CTEs act like they should.

This will be supported with the upcoming v0.3.0 version


Option to indent empty lines when using block indent. [with usable fork]

disclaimer: This was posted here earlier, but it was suggested I post it here (IIUC, VS Code repo).
Block indent (i.e. pressing tab with multiple lines selected) currently only indents non-empty lines. I'd like an option allowing empty lines to also be indented. There's a branch below that implements this, which could be merged if anyone here decides it looks reasonable.
Details below:

Say you have selected code looking like:
foo

zug

and you press indent. Some people (and/or coding styles) prefer that there are no lines with only whitespace, even when in an intended region of code, so they would prefer the result to be method A:
[INDENT]foo

[INDENT]zug

Others prefer indents to be consistent (between brackets etc) regardless of whether lines are empty or not, so they would prefer method B:
[INDENT]foo
[INDENT]
[INDENT]zug

Method B is also simpler and, IMHO, more intuitive, as there's no "except for" - block indent just indents all selected lines.
However, the current behavior is to use method A only.
I created a branch that adds an option that allows you to select method A or B (default = A). The option is Always Indent Empty Lines. It's a very small change and I endeavored to add it cleanly and properly, and I added tests.
I feel this would be a good option to add to the codebase, it's a tiny change and provides the preferred functionality for many people and coding styles. I added this as an issue a while ago, but I think it flew under the radar as I never discussed it elsewhere.

There is another option (which would require a bit more coding) where you could have 3 options.
method C would be to indent empty lines if and only if they were between non-empty lines in the selection. So indenting the selection:

foo

bar

would result in:

[INDENT]foo
[INDENT]
[INDENT]bar

and not
[INDENT]
[INDENT]foo
[INDENT]
[INDENT]bar
[INDENT]

That should be fairly easy to add, but I'm not sure if it's surplus to requirements.
--
Does this seem like a sensible option for VSCode to have?
Does this patch look reasonable?

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Implement Overview page and sidebar component

Features

Removed HomeScreen
Implemented Icons, IconButton, Divider helper components
Implemented NavigationMenu and NavigationMenuItems
Created WithSideBar wrapper to handle pages with sidebar
Implemented OverviewScreen and integrate WithSideBar

Screenshots

Desktop
Mobile Menu Close
Mobile Menu Opened

refs #33

plz align mobile margin with design

@hochiw ui nitpick on mobile:

best to have bottom padding for the button list / menu in case scroll is needed:

on click current tab, should dismiss the menu on mobile?

@chihimng updated :adore2x:


TSC Meeting #25 2020-06-03 20:00 UTC

Agenda

Discussion about Google Search's update to add page experience criteria as a ranking factor
Meet Kasiana
[@brianwarner, @rginn] Present and discuss the plan for handling AMP's CLA
Review https://github.com/ampproject/meta/pull/56
Review https://github.com/ampproject/meta/pull/55
add topics here
Review TSC issue backlog
Review TSC-tagged issues (toggle to "open" after checking for recently "closed" items)

@ampproject/tsc
@tobie Can you specify AC attendees?

We traditionally cancel TSC meetings that conflict with the AMP design review, which this one does, so I'll push this one out to our next slot 2 weeks later.

I'd like to propose an agenda topic for our upcoming meeting: "Discussion about Google Search's update to add page experience criteria as a ranking factor"
Note that I've also proposed this as a topic in the AC meeting on 1 June, and since TSC members are now invited to that, we may cover most of the discussion there. Still, I wanted to ensure we had the opportunity to discuss within the TSC in case some of you can't make it.

I'd like to invite @brianwarner and @rginn to present and discuss the plan for handling AMP's CLA: AMP would move move to a CLA-assistant based solution for the short to mid-term. During that time, there would be collaboration to develop a frictionless and contributor-friendly  solution that the whole foundation would then adopt.
Given the schedule to complete onboarding, this is rather time-sensitive.

I've added

Review https://github.com/ampproject/meta/pull/56
Review https://github.com/ampproject/meta/pull/55
because I saw requests come in for the TSC to review these.

Meet Kasiana

We did intros of everyone in the meeting
Kasiana spoke about role at Axios
Next step: TSC needs to determine new member addition process

Discussion about Google Search's update to add page experience criteria as a ranking factor
Review ampproject/meta#56 and ampproject/meta#55

TSC members should approve on pull request

[@brianwarner, @rginn] Present and discuss the plan for handling AMP's CLA

Projects will need to switch to new CLA anyway, so changes about process or text are coming at a good time

Review TSC issue backlog

Not covered explicitly

Review TSC-tagged issues (toggle to "open" after checking for recently "closed" items)

Not covered explicitly


Do not use CacheBackedStorageManager

Local Caching can lead to inconsistent results in HA mode. We should not use CacheBackedStorageManager in HA mode. It may be ok to just turn it off in the non-HA mode as well since the DB operations are not performance critical and the database query cache would already be caching the query results.

Closed via a175fb3


https://github.com/googleapis/google-cloud-java/blame/8f37b9a318820e544bcaae25337e828ae9192ecb/.github/CODEOWNERS#L5

https://github.com/googleapis/google-cloud-java/blame/8f37b9a318820e544bcaae25337e828ae9192ecb/.github/CODEOWNERS#L5

Did you find anything interesting at the line 5?
(I'll close this if this was created accidentally)