Uncaught Error: ENOENT: no such file or directory, open '~/.atom/packages/Ensime/ensime...

[Enter steps to reproduce below:]

...
...

Atom Version: 1.4.2-fb0cb45
System: Ubuntu 14.04.3
Thrown From: Ensime package, v0.32.2
Stack Trace
Uncaught Error: ENOENT: no such file or directory, open '/home/felix/.atom/packages/Ensime/ensime_update_/project/build.properties'
At fs.js:549

Error: ENOENT: no such file or directory, open '/home/felix/.atom/packages/Ensime/ensime_update_/project/build.properties'
    at Error (native)
    at Object.fs.openSync (fs.js:549:18)
    at Object.module.(anonymous function) [as openSync] (ATOM_SHELL_ASAR.js:137:20)
    at Object.fs.writeFileSync (fs.js:1171:15)
    at updateEnsimeServer (/home/felix/.atom/packages/Ensime/lib/ensime-server-update.coffee:69:6)
    at /home/felix/.atom/packages/Ensime/lib/ensime-startup.coffee:103:7
    at withSbt (/home/felix/.atom/packages/Ensime/lib/utils.coffee:58:5)
    at startEnsimeServer (/home/felix/.atom/packages/Ensime/lib/ensime-startup.coffee:102:5)
    at startClient (/home/felix/.atom/packages/Ensime/lib/ensime-startup.coffee:89:5)
    at Object.module.exports.Ensime.startInstance (/home/felix/.atom/packages/Ensime/lib/ensime.coffee:240:5)
    at /home/felix/.atom/packages/Ensime/lib/ensime.coffee:305:46
    at /home/felix/.atom/packages/Ensime/lib/ensime.coffee:301:9
    at HTMLDivElement.commands.atom.commands.add.core:confirm (/home/felix/.atom/packages/Ensime/lib/views/select-dot-ensime-view.coffee:41:13)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/usr/local/share/atom/resources/app.asar/src/command-registry.js:260:29)
    at /usr/local/share/atom/resources/app.asar/src/command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/usr/local/share/atom/resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:536:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/usr/local/share/atom/resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:359:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeydown (/usr/local/share/atom/resources/app.asar/src/window-event-handler.js:97:36)
    at HTMLDocument.<anonymous> (/usr/local/share/atom/resources/app.asar/src/window-event-handler.js:3:61)

Commands
     -3:53.9.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -3:52.3.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -3:52.3.0 ensime:start (atom-text-editor.editor.hyperclick)
     -3:51.1.0 core:confirm (div#select-file.select-list.fuzzy-finder)
  6x -3:41.7.0 core:cancel (div.ensime-log-view.scroll-view.native-key-bindings)
     -2:58.9.0 command-palette:toggle (atom-text-editor.editor.is-focused)
  8x -2:55 core:move-down (atom-text-editor.editor.mini.is-focused)
     -2:50.4.0 core:move-up (atom-text-editor.editor.mini.is-focused)
     -2:49.8.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -2:49.8.0 settings-view:view-installed-packages (atom-text-editor.editor)
     -0:07.9.0 command-palette:toggle (atom-text-editor.editor.is-focused.hyperclick)
     -0:05.8.0 core:move-up (atom-text-editor.editor.mini.is-focused)
     -0:03.3.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -0:02.8.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:02.8.0 ensime:start (atom-text-editor.editor.hyperclick)
     -0:01.8.0 core:confirm (div#select-file.select-list.fuzzy-finder)

Config
{
  "core": {
    "disabledPackages": [
      "markdown-preview",
      "go-playground",
      "go-playground",
      "ide-haskell-cabal",
      "language-scala"
    ]
  },
  "Ensime": {
    "markImplicitsAutomatically": false,
    "sbtExec": "/usr/bin/sbt"
  }
}

Installed Packages
# User
Ensime, v0.32.2
editorconfig, v1.2.2
go-plus, v3.5.2
go-rename, v0.3.0
haskell-ghc-mod, v1.1.3
haskell-pointfree, v0.1.2
hyperclick, v0.0.35
jsonlint, v1.1.2
language-babel, v2.5.0
language-haskell, v1.5.0
language-lua, v0.9.4
language-nginx, v0.6.1
last-cursor-position, v0.9.0
linter, v1.11.2
linter-eslint, v5.2.0
linter-ruby, v1.2.0
markdown-preview-plus, v2.2.2
markdown-scroll-sync, v0.4.2
mathjax-wrapper, v0.1.0
raml, v0.2.0
scala-format, v0.1.8

# Dev
No dev packages

Weird. This is the relevant code block:
tempdir =  packageDir() + path.sep + "ensime_update_"

  @serverUpdateLog = new EnsimeServerUpdateLogView()

  pane = atom.workspace.getActivePane()
  pane.addItem @serverUpdateLog
  pane.activateItem @serverUpdateLog

  if not fs.existsSync(tempdir)
    fs.mkdirSync(tempdir)
    fs.mkdirSync(tempdir + path.sep + 'project')

  # write out a build.sbt in this dir
  fs.writeFileSync(tempdir + path.sep + 'build.sbt', createSbtClasspathBuild(scalaVersion, ensimeServerVersion,
    mkClasspathFileName(scalaVersion, ensimeServerVersion)))

  fs.writeFileSync(tempdir + path.sep + 'project' + path.sep + 'build.properties', 'sbt.version=0.13.9\n')

from here https://github.com/hedefalk/ensime-atom/blob/master/lib/ensime-server-update.coffee#L69
I can't seem to understand how you can get a "no such file or directory" on that line… Hm. Can you check permissions on that folder? But it should fail earlier…

Hi,
I checked the folder "project" and there wasn't there. So, I created it and started ensime again, now it works...
But, It should be necessary the manual folder creation?

No, it shouldn't so this is definitely a bug. I guess there's something weird with permissions. Can you check owner + permissions on:
'/home/felix/.atom/packages/Ensime
'/home/felix/.atom/packages/Ensime/ensime_update_
'/home/felix/.atom/packages/Ensime/ensime_update_/project
Do you run atom with sudo?
Do I understand correct that "ensime_update_" folder was there but not "project" inside? Really weird since they are created the same way pretty much simultaneously.

What I mean is it's really weird if only project dir was missing since they are created in one run:
if not fs.existsSync(tempdir)
    fs.mkdirSync(tempdir)
    fs.mkdirSync(tempdir + path.sep + 'project')

drwxrwxr-x 11 felix felix 4096 Fev  3 10:37 Ensime/
drwxrwxr-x  4 felix felix  4096 Fev  3 10:36 Ensime/ensime_update_/
drwxrwxr-x  3 felix felix 4096 Fev  3 10:37 Ensime/ensime_update_/project/

and No, i do not run Atom with sudo.
and Yes, "ensime_update_" was there but not  "project".
and Yes: really weird!

SUPER WEIRD. I just understand how this can be. It basically could boil down to this simple spec:
fs = require 'fs'
path = require 'path'

describe 'mkdirSync should really be sync?', ->

  tempdir =  "/tmp/blahablaha"

  it "should not blow with ENOENT: no such file or directory", ->
    if not fs.existsSync(tempdir)
      fs.mkdirSync(tempdir)
      fs.mkdirSync(tempdir + path.sep + 'inner')

    fs.writeFileSync(tempdir + path.sep + 'foo', 'foo content')
    fs.writeFileSync(tempdir + path.sep + 'inner' + path.sep + 'bar', 'bar content')

For me, this of course works. Would be super cool if you could run this in the context of Atom…  I'm gonna add this as a test case and If I get a bit of time I'll try to run it from an Ubuntu docker image.

I've just got this error too.

The offending file in my case is http (added recently for the ScalaDoc lookup).  The file did exist when I went to check it from the sheel.  Could need extra watch on the http file too?
I reloaded and second time, no problem.
Other info: I always start atom from the shell (e.g., $ atom .) in that that matters
--
[Enter steps to reproduce below:]

...
...

Atom Version: 1.4.3
System: Mac OS X 10.11.3
Thrown From: Ensime package, v0.33.0
Stack Trace
Uncaught Error: ENOENT: no such file or directory, open '/Users/richard/Desktop/QCon London 2016/code/.ensime_cache/http'
At fs.js:549

Error: ENOENT: no such file or directory, open '/Users/richard/Desktop/QCon London 2016/code/.ensime_cache/http'
    at Error (native)
    at Object.fs.openSync (fs.js:549:18)
    at Object.module.(anonymous function) [as openSync] (ATOM_SHELL_ASAR.js:137:20)
    at Object.fs.readFileSync (fs.js:397:15)
    at Object.fs.readFileSync (ATOM_SHELL_ASAR.js:385:29)
    at FSWatcher.<anonymous> (/Users/richard/Developer/third-party/atom-ensime/lib/ensime-startup.coffee:87:43)
    at emitTwo (events.js:87:13)
    at FSWatcher.emit (events.js:172:7)
    at FSWatcher.<anonymous> (/Users/richard/Developer/third-party/atom-ensime/node_modules/chokidar/index.js:171:15)
    at FSWatcher._emit (/Users/richard/Developer/third-party/atom-ensime/node_modules/chokidar/index.js:212:5)
    at FSWatcher.NodeFsHandler._handleFile (/Users/richard/Developer/third-party/atom-ensime/node_modules/chokidar/lib/nodefs-handler.js:275:10)
    at FSWatcher.<anonymous> (/Users/richard/Developer/third-party/atom-ensime/node_modules/chokidar/lib/nodefs-handler.js:473:21)
    at FSReqWrap.oncomplete (fs.js:82:15)

Commands
     -0:05.8.0 command-palette:toggle (atom-text-editor.editor.is-focused.hyperclick)
     -0:03.9.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:03.9.0 ensime:start (atom-text-editor.editor.hyperclick)
     -0:03.5.0 core:confirm (div#select-file.select-list.fuzzy-finder)
     -0:01.3.0 tree-view:toggle (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "audioBeep": false,
    "destroyEmptyPanes": false,
    "disabledPackages": [
      "linter-scalac",
      "archive-view",
      "language-c",
      "language-clojure",
      "language-csharp",
      "language-go",
      "language-make",
      "language-objective-c",
      "language-perl",
      "language-php",
      "language-python",
      "language-ruby",
      "language-ruby-on-rails",
      "language-toml",
      "metrics",
      "atom-ctags",
      "autocomplete-haskell",
      "haskell-ghc-mod",
      "ide-haskell",
      "language-haskell",
      "language-purescript",
      "language-mustache",
      "language-sass",
      "tabs-to-spaces",
      "autocomplete-ctags",
      "autocomplete-paths",
      ".bin",
      "editorconfig",
      "raml",
      "symbol-gen",
      "vim-mode"
    ],
    "excludeVcsIgnoredPaths": false,
    "followSymlinks": false,
    "ignoredNames": [
      ".git",
      ".svn",
      ".DS_Store",
      "target",
      ".settings",
      ".ensime"
    ],
    "themes": [
      "native-ui",
      "d6y-syntax"
    ]
  },
  "Ensime": {
    "devMode": true,
    "markImplicitsAutomatically": false,
    "markWarningsInline": false,
    "modifierKeyForJumpToDefinition": "alt",
    "noOfAutocompleteSuggestions": 8,
    "sbtExec": "/usr/local/bin/sbt"
  }
}

Installed Packages
# User
Ensime, v0.33.0
browser-plus, v0.0.54
coffee-compile, v0.21.1
color-picker, v2.1.1
d6y-syntax, v1.1.0
hex, v0.6.2
hyperclick, v0.0.35
language-idris, v0.4.0
language-scala, v1.1.6
last-cursor-position, v0.9.0
latex, v0.29.0
linter, v1.11.3
linter-coffeescript, v1.0.0
native-ui, v0.16.1
pretty-json, v0.4.1
project-manager, v2.7.6
webbox-color, v0.5.9

# Dev
No dev packages

Same top of the stack error message, but I think the problem @d6y has is pretty is separate to the original and has been fixed by his PR.
The original one is for creating a file under a folder that should have been created just before, sync. It says "open" but it's from within "writeFileSync" and I started reading the node code for it and I think it's the file descriptor that's "opened". The " no such file or directory, open " message must mean the folder.

Ok, so:
if not fs.existsSync(tempdir)
    fs.mkdirSync(tempdir)
    fs.mkdirSync(tempdir + path.sep + 'project')

For some reason this must have happened that tempdir was already there. Or that it was created and stuff broke the first time so the second mkdir was never run. Next time it wasn't created since the condition is false.
I added appveyor for win build and just saw this: https://ci.appveyor.com/project/hedefalk/ensime-atom/build/5/job/dadoiwdqaxsv7vym
which was a test I wrote to try to provoke it that never actually failed for me, but that was because I had hardcoded /tmp somthing and not using a real temp folder lib thingy. Anyways, easy fix!


Cant find OnProgressClickListener written in Java

Hi, is there an OnProgressClickListener written in Java ?(not the kotlin one)
If not, do you think you can add one so we will be able to do this:
`
progressView.setOnProgressClickListener(new OnProgressClickListener()
{
        @Override
        public void onClickProgress(boolean b)
        {
            Toast.makeText(MainActivity.this, progressView.getProgress() + "% selected: " + b, Toast.LENGTH_SHORT).show();
        }
    })

`
(Im coding in Java)
so im getting the error:

class file for kotlin.jvm.functions.Function1 not found

Thanks a lot.

@Dor-Sloim
Hi, it seems not has any problem setting onClickProgress a listener in Java like this.
progressView.setOnProgressClickListener(new OnProgressClickListener() {
      @Override
      public void onClickProgress(boolean b) {

      }
 });

Thanks for answering.
I copied your code but still getting the same error:

error: cannot access Function1
class file for kotlin.jvm.functions.Function1 not found

Do i have to enable kotlin support for the project for this to work?

@Dor-Sloim
Hi, it is released a new version 1.0.5 now.
Thank you for your issue!

Updated to 1.0.5 but still got the same error.

Also, i'm getting the same error when using setOnProgressChangeListener.
The error:

error: cannot access Function1

class file for kotlin.jvm.functions.Function1 not found

The only working listener for me is the setOnClickListener(new View.OnClickListener().
Do you think you can add java support?
I really love this library.

I got this to work by configuring Kotlin for project unfortunately


Export benefit.css

It should basically be<EMAIL_ADDRESS>

The problem with doing this is that styleWith + emotion normalizes class names (via css), which means I can't readily make the generated name match the original:

https://emotion.sh/docs/babel-plugin-emotion#labelformat

Perhaps the best thing is to just generate a blob for all utilities via:
https://github.com/cdonohue/benefit/blob/c5f01b11e4181382dc454d6bf3bf63a9da4f7b8b/src/createUtilitiesFromConfig.js#L9

@cdonohue This will help with #8, but there are a couple things that are different for Tailwind in particular:

https://unpkg.com/tailwindcss/dist/

tailwind.css includes normalize.css & a custom preflight.css:
https://github.com/tailwindcss/tailwindcss/blob/7e7b6d9166459978f5b5390fc5c14177fc7ab484/src/plugins/preflight.js#L4-L10

Tailwind exports utilities.css.

With benefit, the reset happens as part of the generated style (for isolation).
Given #8, I'm curious in your thoughts for how compatibility should work, and to what degree.  The same question goes for resetting, as well.
With the targeted nature of utility libraries intentionally avoiding cascades, I'm rethinking the practicality of resetting for every application.

@cdonohue This should be in a good spot to continue on #8's compatibility testing.
I've included an example from https://codepen.io/steveschoger/pen/YbQXGq to show that benefit.css & tailwind.css behave the same:

Clearly there are some subtle differences, but those will be revealed in #8.
What we have here, at least, is a fantastic baseline to ensure parity with.


fix: raise failed request to not run mark_webhook_sent

error is not raised, even if failed webhook would be marked as sent every time

In dispatch_webhook the status code is set to -1 in case of failure. We should be consistent in handling the error.
        try:
            r = await client.post(payment.webhook, json=data, timeout=40)
            await mark_webhook_sent(payment, r.status_code)
        except (httpx.ConnectError, httpx.RequestError):
            await mark_webhook_sent(payment, -1)

There are 3 places where this pattern ☝ is repeated.
A common function should be extracted: call_webhook(payment, data, timeout)


strip local_ from operator capability names

Also removing the separate unban capability (which is a slight compatibility break, albeit one I'm confident no one will notice)

Also removing the separate unban capability (which is a slight compatibility break, albeit one I'm confident no one will notice)


Add Lorem ipsum extension to String!

Add loremIpsum(ofLength length: Int)

Example:

label.text = String.loremIpsum(ofLength: 39) -> "Lorem ipsum dolor sit amet, consectetur"

https://www.lipsum.com/

An initializer is also a good addition
let text = String(loremIpsumOfLength 39)
print(text) // prints "Lorem ipsum dolor sit amet, consectetur"


Consider switching to a CC license

Creative Commons are specifically tailored for written works (as opposed to other open source licenses that deal more with source code).  CC-BY (Attribution 4.0 international) is the most comparable to the Apache license currently listed.  More details: https://creativecommons.org/choose/

Thanks @mekkim for the suggestion. CC0 is the only option I see in the template. Please let me know if this looks ok: https://github.com/dewan-ahmed/recruiter-response-template/pull/3

Fixed via https://github.com/dewan-ahmed/recruiter-response-template/pull/3.


SetRecFilter() documentation should be clearer that it might use a secondary key

Right now it just says

Sets the values in the current key of the current record as a record filter.

Which I had always assumed meant the current primary key fields, but after some days of confusion... I realise it will use any key that is the current key, including secondaries.
This code reproduces what I mean:
pageextension 50100 CustomerListExt extends "Customer List"
{
    trigger OnOpenPage();
    var
        Customer: Record Customer;
    begin
        Customer.FindLast();
        Customer.SetCurrentKey(Name);
        Customer.SetRecFilter();
        Message(Customer.GetFilters());
    end;
}

The resulting message reads:

No.: CASH, Name: Cash Customer hey fun, Address: '', City: ''

So, the SetCurrentKey(Name) really got mapped to a larger key containing the Name field, and all fields in that key are filtered to their current values.
Perhaps the doc could instead read something like:

Sets the values in the current key of the current record as a record filter. This is not necessarily the primary key, as the table view might be set to use a secondary key. All fields in the key as defined on the table are filtered (even if the table view only requests a subset of those fields).

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

ID: 9812c244-cb0d-a55b-4193-91382d4d4f4b
Version Independent ID: cddb6d21-00cf-2ac1-4a84-fab5f61bcf07
Content: Record.SetRecFilter Method - Business Central
Content Source: dev-itpro/developer/methods-auto/record/record-setrecfilter-method.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for the valuable feedback. The key selection is explained a bit in the SetCurrentKey method article, but it could be more prominent here as well. We've added some content based on your feedback, which hopefully helps. The update will be published soon. Thanks!


Why does ShopifyAPI::Variant#to_hash exclude inventory_quantity ?

Issue summary
I try to get variant's inventory_quantity via ShopiryAPI::Product#to_hash.
But inventory_quantity did not exist in Hash object.
> ShopifyAPI::Auth::Session.temp(shop: 'test.myshopify.com', access_token: 'token') do |session|
  variant = ShopifyAPI::Variant.create_instance(session: session, data: { inventory_item_id: 1234, inventory_quantity: 10 })
  pp variant.inventory_quantity
  pp variant.to_hash
end

10
{"inventory_item_id"=>1234}
=> {"inventory_item_id"=>1234}

Expected behavior
I hope ShopifyAPI::Variant#to_hash include all API properties.
Actual behavior
Some properties exclude?
Specifications

shopify_api version: 10.0.2
Shopify API version used 2022-01

Hey @koshigoe, thanks for reporting this! The inventory_quantity field can't be sent when saving, so we started filtering it out when we call to_hash (because that is the method we use to serialize the object for saving). In your case, we shouldn't be ignoring that field.
We'll add a parameter to to_hash to make that behaviour optional. That way, when you call the method you'll get all the data.


Steamvr and Godot Editor causes freeze (only while both are running, may be radeon specific)

Tested versions
tested: godot 4.2.1; 4.1 had freezes too, but unknown if same.
System information
Windows 10 - Godot 4.2.1 - Radeon RX 7900XTX - Vulkan - Forward+ - AMD Threadripper 1920x (in case it's more specific)
Issue description
Using the editor while steamvr is open completely freezes PC. (may be radeon specific)
Steps to reproduce
•Have the Godot Editor running while steamVR is running (tested with a brand new 3d scene.)
•Add, for example, a mesh node. Might already crash, but potentially not
•Try to set the mesh type (I never got farther, freezes pretty quick.)
Video for reference. still crashes whether or not Discord is running.
https://youtu.be/5bXcfyy6CbU
Minimal reproduction project (MRP)
N/A

Can you reproduce this after enabling Single Window Mode in the Editor Settings then restarting the editor?

After a couple minutes of testing, it does seem more stable. Seems a good interim solution. It might be an RDNA3 bug, if so, probably needs a solution other than single window, or at least tracking down the root cause.
Going to go ahead and switch my main project to single window, and come back if it's still unstable.

update: after enabling "single window render" I get this new render bug that looks like some form of z-fighting. it's not, cuz there's nothing to fight with but it makes those artifacts on distant geometry...


Data-Target doesn't match modal id in Modal Documentation.

There is an error in the documentation of the modal. The data-target of the button does not match the id of the modal
.
https://v4-alpha.getbootstrap.com/components/modal/#examples

<button type="button" class="btn btn-primary" data-toggle="modal" data-target="#exampleModal">
  Launch demo modal
</button>

Already fixed by : #21671 and will be available on our next release, thanks 👍


run_afl.sh file empty

The run_afl.sh file is empty. Is this bash file no longer needed?
~/tests/libjpeg@master✗✗✗✗✗✗ #! cat /vagrant/tools/run_afl.sh
#!/bin/bash
Thank you!

Hi, the file is touched by kleefl_init and will be setup by kleefl_prepare_afl after KLEE is done/canceled once. Only then the run_afl.sh can be executed to start the fuzzers :)

Makes sense. After running KLEE to generate the .bc files I'm having issues getting the kleefl_prepare_afl to work. Please see output below:
~/test/project_1/tiff-3.8.2 #! ls fuzz
in  run_afl.sh
~/test/project_1/tiff-3.8.2 #! python2 /vagrant/tools/kleefl_prepare_afl
Traceback (most recent call last):
  File "/vagrant/tools/kleefl_prepare_afl", line 472, in <module>
    main(sys.argv)
  File "/vagrant/tools/kleefl_prepare_afl", line 364, in main
    copy(afl_dir+"app", "app")
  File "/usr/lib/python2.7/shutil.py", line 119, in copy
    copyfile(src, dst)
  File "/usr/lib/python2.7/shutil.py", line 82, in copyfile
    with open(src, 'rb') as fsrc:
IOError: [Errno 2] No such file or directory: 'fuzz/app'

Is app supposed to be a subdirectory with the binary name? I have changed the script to create a subfolder under the fuzz directory which passes this issue, however the script fails later on. I'll see if I can get  the script to work as intended.

Argh! I usually took the kleefl_pick utility to choose a binary for evaluation.
This will copy a target $app like /klee_build/$app.bc -> klee/app.bc and /afl_build/$app -> fuzz/app
This is missing in the README & has to be done before executing KLEE!
(What also explains why your klee-last folder doesn't have proper data & structure)
Thanks for sharing! 👍

Awesome, I will add the documentation to my fork and send a merge request. (assuming I get it to work correctly the first time :) )
Thanks again!!


Issue with multiple children

I ran into the following problem where not all nested children would be visited:
This works
const A = ({ children }) => children
const B = ({ children }) =>
{children}

const tree = (

)

reactTreeWalker(tree, visitor, context)

So does this
const A = ({ children }) =>
{children}

const B = ({ children }) => children

const tree = (

)

reactTreeWalker(tree, visitor, context)

This does not work
Because there are no basic HTML elements wrapping the children of A, those individual children are not visited
const A = ({ children }) => children
const B = ({ children }) => children

const tree = (

)

reactTreeWalker(tree, visitor, context)

A PR will follow shortly and should provide a fix for this issue.

Hey @grxy
Awesome, thanks so much for picking this up.  I'll try review and merge within the next 24 hours. 👍

@ctrlplusb No prob, dude. I actually noticed the issue when writing unit tests for my code while using this.
P.S. Great work breaking it out from react-apollo!

Hi @grxy 👋
Apologies for the late one on this.
I have struggled to recreate your initial issue.  I have added another branch containing a test that I was initially expecting to fail as it doesn't have your fix applied. The test passes though.
Do you mind having a look to see if I have configured the test correctly?
Thanks!

@ctrlplusb The issue happens when you have multiple react components that are siblings, so if you updated your tree to be:
const tree = (
      <OnlyChildren something={1}>
        <OnlyChildren something={2}>

Hello world!

        </OnlyChildren>
        <OnlyChildren something={3}>

Hello world!

        </OnlyChildren>
      </OnlyChildren>
    )

you would expect actual to be [1, 2, 3], when in fact the children are never visited, resulting in actual being only [1]. See screenshot:

Perfect thanks!

@ctrlplusb Any chance you'll be able to merge my PR and publish 2.1.2 soon?

All done and published. Thanks for being persistent with me!

@ctrlplusb Great! Thank you!


update dav-network.io

additional wallet connected to the phishing site.

included in https://github.com/MrLuit/EtherScamDB/pull/739


Backport: Subctl fallback devel 0.10

During releases from branches, the e2e jobs look for a subctl matching
the target release; but that doesn't exist yet, so the download fails.
To avoid that, if installing the target release version fails, fall
back to devel subctl (which should provide the behaviour we need for a
release, and is more appropriate than the default, which is the latest
release).
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>(cherry picked from commit d8ec5cea2f34cc654459615aaa04d3fb2e9d7475)

PR targets release-0.9

This will never pass, I guess we need a CI fix for release-0.10 branch.


'RobustTerminaton' object has no attribute 'n_max_gen'

Hi when attempting to follow along the example to introduce objective space tolerance termination i.e.
termination = RobustTermination(         MultiObjectiveSpaceTermination(tol=0.005, n_skip=5), period=20)
I am getting the following error:
if algorithm.termination.n_max_gen == algorithm.n_gen:
AttributeError: 'RobustTermination' object has no attribute 'n_max_gen'

I attempted to set this when initialising both MultiObjectiveSpaceTermination and RobustTermination as well as inspecting both classes (and DeltaToleranceTermination) but it doesn't appear as if there is any class variable for n_max_gen.
Any guidance here would be appreciated!
Cheers

Can you provide an example where this code fails?
from pymoo.algorithms.moo.nsga2 import NSGA2
from pymoo.optimize import minimize
from pymoo.problems import get_problem
from pymoo.termination.ftol import MultiObjectiveSpaceTermination
from pymoo.termination.robust import RobustTermination
from pymoo.visualization.scatter import Scatter

problem = get_problem("zdt1")

algorithm = NSGA2(pop_size=100)

termination = RobustTermination(MultiObjectiveSpaceTermination(tol=0.005, n_skip=5), period=20)

res = minimize(problem,
               algorithm,
               termination=termination,
               seed=1,
               verbose=False)

plot = Scatter()
plot.add(problem.pareto_front(), plot_type="line", color="black", alpha=0.7)
plot.add(res.F, facecolor="none", edgecolor="red")
plot.show()

Seems to run without any issue for me.

@KyleYoung1997 I also came across this same issue. As the discussion here highlights, the documentation for this Termination appears to be out of date. However, what you can do to get a max generations termination in there as well, is to use a TerminateIfAny to combine the two termination conditions:
from pymoo.core.termination import TerminateIfAny
from pymoo.termination import get_termination
from pymoo.termination.ftol import MultiObjectiveSpaceTermination
from pymoo.termination.robust import RobustTermination

termination = TerminateIfAny(
    RobustTermination(
        MultiObjectiveSpaceTermination(tol=0.005, n_skip=5),
        period=20
    ),
    get_termination("n_gen", 200)
)

Ahh yes now I understand what is going wrong even without a code sample. Yes the code snipped above is the correct one.
@oli-hall can you send me the link where the documentation needs to be updated?

Sure thing @blankjul – I got the tip about the documentation being out of date from this discussion – the documentation page itself is here – the main page that looks at Termination, in particular the section called 'Objective Space Tolerance'. Thanks!

Apologies for the delayed response @blankjul. The fix from @oli-hall works perfectly. Thank you for the assistance.


Fix event expression Function scope when called

A la https://github.com/ractivejs/ractive/issues/2019
Just adds a simple check to see if the item is a function and if so, binds it to the instance.

Not to be nitpicky but... ===?

@fskreuz not needed with typeof. Always returns a string and when comparing with another string no conversion will occur

Nearby code a few lines down uses === for typeof so... uniformity :D

Thanks for the PR @JonDum!
IMO == means "something special happening here" (like foo == null ), so agree with @fskreuz that if vanilla string compare, use ===.
Also, need to add a test in events

IIRC, one should use tabs instead of spaces for indents.  It looked right in Vim, but not in GH I guess. Should be tab characters now.

@JonDum Thanks for this PR. Tabs vs spaces aside (and GitHub really should get their act together on that... probably my most commented issue ever), this fix will unfortunately be swept aside by #2022, so it's probably best we don't merge it, so that it stays open and we don't forget about it.

I'm not sure where/how to put a test case for this though.

The same applies to any tests as to the PR itself. This'll do for now - we'll just have to ensure that it goes into the test suite before #2022 is merged. I'm tied up with a gazillion other things at the moment but if anyone wants to submit an equivalent PR against the arboretum branch it would be gladly received!

We stumbled upon a better way of approaching this problem – binding functions on init/set (#2115) – so I'll close this PR. The test from the fiddle above is included via #2111


polling: Total participation needed does not match quorum defined in PollCreator

Describe the bug (required)
A clear and concise description of what the bug is.
The parameter values for the PollCreator on Rinkeby can be found here. The total participation needed shown in the poll detail view is 33% which is different from the quorum value in PollCreator.
Side note: The quorum value in the PollCreator on Rinkeby is actually wrong right now, so we'll need to deploy an updated version with the right value as well.
Expected behavior (required)
A clear and concise description of what you expected to happen.
The total participation needed displayed in the poll detail view to be the same as the quorum value defined in the deployed PollCreator.
Device (required):

Device: Macbook Pro
OS: OSX
Browser: Chrome
Version: https://polling.livepeerorg.now.sh/voting/0xbf790e51fa21e1515cece96975b3505350b20083

Screenshots (optional)
If applicable, add screenshots to help explain your problem.

Fixed. Now pulls from quorum field subgraph.


Dummy PR for running swift-integration-tests

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

Please test with following pull request:
https://github.com/apple/swift-integration-tests/pull/64
@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 1092cde5b9ad25c81a8ecee8d945aaee47f362a2

Build failed
Swift Test OS X Platform
Git Sha - 1092cde5b9ad25c81a8ecee8d945aaee47f362a2

Can one of the admins verify this patch?

Figure we probably don't need this anymore.


Update Excel (React) quickstart with guidance about using HTTPS.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/includes/file-get-started-excel-react.md
:white_check_mark:Succeeded

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


Bushmaster ACR with better texture and APC9 Test

🤔 What type of PR is this? (check all applicable)

[X] 🍕 Addition
[ ] ⌨ Productivity
[ ] 🐛 Bug Fix
[X] 🔥 Optimization
[X] ⚙ Configuration
[ ] 🌟 Quality Of Life
[X] ✨ Enhancement
[ ] 📝 Documentation

📝 Description
It is time to give texture to a mythical and old weapon in the mod, on the other hand it is the first test to see how the new weapons work, I doubt that it will break something, I have already tried it many times
🖼 Screenshots/Recordings

📖 Added to documentation?

[ ] 📜 README.md
[ ] 📓 Javadoc
[ ] 🍕 Comments
[X] 🙅 no documentation needed

Could you please provide the Befores?

Could you please provide the Befores?

I didn't understand how before?

Was there no textures before? What exactly did you change?
Or did you update the guns to the new Gun System?

Was there no textures before? What exactly did you change?
Or did you update the guns to the new Gun System?

I updated the texture in the acr and in the apc9 the system was updated

Could you attach screenshots of what these guns looked like before your changes are applied?

Could you attach screenshots of what these guns looked like before your changes are applied?

Yes ready

By the way, next pull request it would be a really good idea to delete you entire fork and contact me on Discord to be added to the repository so you can use branches and stay up to date with master it would make everything must cleaner

By the way, next pull request it would be a really good idea to delete you entire fork and contact me on Discord to be added to the repository so you can use branches and stay up to date with master it would make everything must cleaner

sorry that's why I'm using the outdated mod files

Missing changelog

ready

By the way about the APC, is it working fine are the attachment good how many customization options are there, and also when you are supposed to revamp the weapon for the new system it seems to be essentially the same

By the way about the APC, is it working fine are the attachment good how many customization options are there, and also when you are supposed to revamp the weapon for the new system it seems to be essentially the same

what are you talking about?

By the way about the APC, is it working fine are the attachment good how many customization options are there, and also when you are supposed to revamp the weapon for the new system it seems to be essentially the same

what are you talking about? It does work well as a current renewed weapon, with respect to its customization it is related to the model and the APC9 model does not have any accessory that modifies the weapon and regarding whether to pass the weapon from system 1 to system 2 is different in a certain sense because the old code has more modification like the position of the weapon during proning, the recoil and movements of the old animations and that covers many lines

Okay but it would need to be revamped to bring it to the standard of quality, like it needs to have a lot of possible attachements etc...
Look at all the other weapons that use the new system versus the old to see the quality difference

By the way about the APC, is it working fine are the attachment good how many customization options are there, and also when you are supposed to revamp the weapon for the new system it seems to be essentially the same

what are you talking about? It does work well as a current renewed weapon, with respect to its customization it is related to the model and the APC9 model does not have any accessory that modifies the weapon and regarding whether to pass the weapon from system 1 to system 2 is different in a certain sense because the old code has more modification like the position of the weapon during proning, the recoil and movements of the old animations and that covers many lines

Okay but it would need to be revamped to bring it to the standard of quality, like it needs to have a lot of possible attachements etc... Look at all the other weapons that use the new system versus the old to see the quality difference

add more content to the weapon?

By the way about the APC, is it working fine are the attachment good how many customization options are there, and also when you are supposed to revamp the weapon for the new system it seems to be essentially the same

what are you talking about? It does work well as a current renewed weapon, with respect to its customization it is related to the model and the APC9 model does not have any accessory that modifies the weapon and regarding whether to pass the weapon from system 1 to system 2 is different in a certain sense because the old code has more modification like the position of the weapon during proning, the recoil and movements of the old animations and that covers many lines

Okay but it would need to be revamped to bring it to the standard of quality, like it needs to have a lot of possible attachements etc... Look at all the other weapons that use the new system versus the old to see the quality difference

add more content to the weapon? Or do you mean the texture?

Both

Both

Honestly, I would like to add more weapon accessories, but there is no time, as far as the texturing is concerned, it is fine because it is similar to the current style of the mod.

The APC9 seems to be way closer to the camera is that intended


add buy-plank menu entry swap

Added the left-click buy-plank option for the lumber yard. Addresses  #1930

This is a rejected feature:

Menu reorder in shop/artisan skill screens: Other clients don't implement these swaps, and would be too strong for numerous activities (examples include "Buy-50" swaps, "Make All" swaps for smithing/crafting, etc.)

This was added in menu swapper extended in the plugin hub


tinymce not visible in demo

tinymce is not visible in demos.
Example: https://grijafoja.ep.io/admin/cms/page/2/

Confirming (at least in that demo), Chrome says:
Failed to load resource: the server responded with a status of 404 (Not Found)
https://grijafoja.ep.io/media/tiny_mce/tiny_mce.js

Uncaught ReferenceError: tinymce is not defined


Mangled white jewelry tab

I've got a tab called "1x1" where I dump jewelry items for crafting. When loaded in Procurement, it looks like a hot mess:

Is this a quad tab by any chance?

It is not! Premium tab yes, but not a quad.

This may be related to PR #718

Aha, quite likely. "Loot" before that tab is a quad.


Setup blacken docs.

https://github.com/asottile/blacken-docs

Blocked by: asottile/blacken-docs#8


make option specs more readable and immediate

@ronuchit good point, I changed it to print in both cases only if option learning


Measure test coverage with GitHub Actions

Migrate the coverage step from CircleCI to GitHub Actions. It's pretty easy: compile Themis with gcov instrumentation, run the tests, use lcov to extract coverage data, post results using official Action.
Compared to CircleCI, the difference is in how we exclude the tests directory. For some reason the version of lcov available on GitHub's runners handles relative paths slightly differently.
(Also, if I understand the docs correctly, Coveralls can (will?) post coverage results in pull requests, thanks to that action.)
On CircleCI retirement
Now that GitHub Actions have stolen the last job CircleCI had, it's no longer necessary on the master branch. However, CircleCI is still running nightly builds on the stable branch so we can't turn it off just like that.
Due to the way CircleCI works, it needs to see the .circleci/config.yml file on the default branch of the repo (i.e., master). However, we don't need it to do anything. So put there the minimal valid config and let CircleCI rest. It had worked hard for us all these days, after all.
Once Themis 0.14 is out, GitHub Actions configuration will be applied to the stable branch, and only then CircleCI can be deconfigured and finally retired.
On GoThemis coverage
Also note that these is no replacement for goveralls execution that is removed in this commit. It does not seem to be working right now, and no one knows when exactly it stopped working. Well, we can't replace it easily right now so GoThemis is in the same boat with other wrappers: that is, test coverage is collected only for Themis Core. Maybe future work on Themis 0.14 will add more coverage coverage before the release.
Checklist

[X] Change is covered by automated tests
[X] The coding guidelines are followed
[X] Public API has proper documentation
[X] Changelog is updated
[ ] x86_64 job is marked optional in master branch settings

(Also, if I understand the docs correctly, Coveralls can (will?) post coverage results in pull requests, thanks to that action.)

Yes, it can, but it needs to be enabled explicitly. I'm not a great fan of bots spamming comments into PRs, so I'd leave that disabled.
However, I have enabled status API usage there. Starting with the next build, Coveralls should be phoning GitHub with coverage analysis results, failing the build on any of the following events:

coverage falls below 70% (currently it's 84%)
coverage drops by more than 10% with a single change

coverage drops by more than 10% with a single change

What exactly is a single change? Single commit in master after squashing+merging everything from some PR?
And the coverage: XX% badge on top of README.md will now use data from GitHub Actions?

@iamnotacake,

What exactly is a single change?

I'm not sure, let's find out! 🤠
On a serious note, Coveralls docs don't seem to go into much detail here. I'd assume this is measured as a difference between the baseline for the branch and the next submission for. So its granularity is a push. Submit a PR with 10% loss over the target branch – denied. Submit an update within a PR which crosses the limit – denied. Merge it regardless – still red.

And the coverage: XX% badge on top of README.md will now use data from GitHub Actions?

Right. It sources the data from Coveralls, so it's not going to change. It does not care which CI submits results to Coveralls.

Yup, here are the status checks:


Customize delete button for editable cards

Reference: #42
Hi - Thanks for your work! Just wondering if it is possible to add custom component to the delete button? It fits well with standard setup but my styling is using material-ui components and I would like to change the delete button (or actually remove it - so I could have a popup show info on click)

Any idea??
Cheers,
M

v1.27.1 should fix this.
Introduced a new prop called hideCardDeleteIcon which will hide the delete icon on cards when the board is editable. Refer to story for illustration:
https://rcdexta.github.io/react-trello/?selectedKind=Editable Board&selectedStory=Custom Buttons&full=0&down=0&left=1&panelRight=0


Hitting issue with user running Norton AV

We have client side uploaidng directly to s3. all is good for all users with the exception of a couple which we have traced down to running Norton AV on the machine. For these users they are getting this in the console
Failed to load resource: the server responded with a status of 403 (Forbidden)
custom.fineuploader-5.0.9.min.js:25 [Fine Uploader 5.0.9] POST request for 0 has failed - response code 403
custom.fineuploader-5.0.9.min.js:25 [Fine Uploader 5.0.9] Specific problem detected initiating multipart upload request for 0: 'The difference between the request time and the current time is too large.'.
custom.fineuploader-5.0.9.min.js:25 [Fine Uploader 5.0.9] Chunked upload request failed for 0, chunk 0
Any idea of a workaround?

It looks like you posted the same question on Stack Overflow. I'll paste my answer from SO here:
If an external program or service is preventing you from uploading files directly to S3, you'll need to do one of the following:

Disable the external program or service (Norton in this case)
Re-configure the service to allow requests to S3
Use Fine Uploader's traditional endpoint uploader to send files to a server you control, which then relays them to S3.

It's not clear from your question what Norton is flagging. If it is objecting to the content of the requests, then simply ensure you are using HTTPS to contact S3.


Issues in forwarding ASM logs to Splunk

Environment

Telemetry Streaming Version: 1.8
BIG-IP Version:15.0.1

Summary
A clear and concise description of what the bug is.
Please also include information about the reproducibility and the severity/impact of the issue.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
   "class": "Telemetry",
   "controls": {
     "class": "Controls",
     "logLevel": "debug"
   },
   "My_Poller": {
     "class": "Telemetry_System_Poller",
     "interval": 60,
     "enable": "false"
   },
   "My_Listener": {
     "class": "Telemetry_Listener",
     "port": 6514
   },
    "My_Consumer1": {
        "class": "Telemetry_Consumer",
        "allowSelfSignedCert":true,
        "type": "Splunk",
        "host": "
        "protocol": "https",
        "port": 8088,
        "passphrase": {
            "cipherText":
        }

Observe the following error response:

{
    "message": "success",
    "declaration": {
        "class": "Telemetry",
        "controls": {
            "class": "Controls",
            "logLevel": "debug",
            "debug": false
        },

Expected Behavior
According to the TS declaration I am expecting the ASM logs to be sent to the Splunk via TS
Actual Behavior
I dont see any ASM logs in the Splunk but when I generate the AFM, LTM , I am able to see them in the Splunk.
Your help would be greatly appreciated.
Please include any applicable error output.

After reviewing and testing in lab using AS3 for configuration, ASM logs were received into splunk. Provided declarations and it is working for @anithamareedu as well.


Maintainers wanted for kube-prod-runtime fork

I was consider to start using kube-prod-runtime when find out that deprecation notice looks to be added to here recently https://github.com/bitnami/kube-prod-runtime/commit/a79bcda68a3423de01280c198f68ac05a1fa077f so open this issue to discuss if we can find enough persons who are interested to keep fork of this project alive?
I think that in minimum we need find 5 persons who can spent 1 hour in month to review pull requests and issues.

@ppbaena any possibility to get deprecation notice also to https://bitnami.com/kubernetes/bkpr ?

Hi @olljanat,
Thanks for your interest to look for maintainers and fork kube-prod-runtime to keep it alive. With regards to your request, we're updating the deprecation notice also in the website.


Migrates to Swift 4.2

HI @mtnbarreto  could you approve this pr?

if you can add s.swift_version = "4.2" when the next breaking change happens, it will keep the pod on 4.2 and won't break anyone. I think that should be part of this PR

Can we get this PR approved? Thanks!

looks like this changes have been made in master? the commits reference #616 but it doesn't look like the pull request was merge through github? this pr can probably be closed.

@mcmurrym when I do a pod update I get build errors for the changes listed in this PR. Would you happen to know why?

@pccole Even though the changes are in master, the owners have not published a new version to the cocoapods podspec repo, it is still pointing to the March release I believe. Maybe we can ask  @mtnbarreto or @m-revetria to publish an updated pod soon, alternatively you can point your pod directly at the repo instead: i.e.: pod 'XLPagerTabStrip', :git => 'git@github.com:xmartlabs/XLPagerTabStrip.git' but I can't recommend this particular solution because I'm not sure on the state of master on this repo.


load extension account after user selects extension option (kusama)

The changes includes:

moved load extension accounts to a separate module (extension.js), and set it to load the extension after user has selected the polkadot extension option. (closes UX - asking for permission to read accounts is premature #41 , Don't ask to enable Extension until you need to #73)
fixed the case where work-0 would be selected in verify seed phrase (fixes Seed phrase confirmation screen asked for "Word 0" #82 )

closing this as it is included in #90


ArgumentExceptionAnalyzer ignores several code constructs

Currently ArgumentExceptionAnalyzer assumes the exception is thrown within a method. Several other code constructs have parameters:

Lambda expressions (both parenthesized and simple)
Indexer properties
Certain accessors (set, add, and remove) have an implicit value parameter

Lambda expressions were solved on a367d9de. I am fine without analyzing indexer properties or event accessors. @code-cracker/owners If you disagree please reopen the issue.


Hackily include a different, timezone safe pyrolite artifact

We need to get a new version of pyrolite (the Python unpickler that SparkSQL uses) to support datetimes with timezones. This does that by adding a maven local repository with a manually installed JAR that I built from a local clone.
I did this:
oration ~/C/Pyrolite (more_timezone_unpickles) ➜  mvn deploy:deploy-file -Durl=file:///Users/hornairs/Code/spark/shopify_repo/ -Dfile=build/pyrolite.jar -DgroupId=org.airhorns -DartifactId=pyrolite -Dpackaging=jar -Dversion=9.9.0
[INFO] Scanning for projects...
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building Maven Stub Project (No POM) 1
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-deploy-plugin:2.7:deploy-file (default-cli) @ standalone-pom ---
Uploading: file:///Users/hornairs/Code/spark/shopify_repo/org/airhorns/pyrolite/9.9.0/pyrolite-9.9.0.jar
Uploaded: file:///Users/hornairs/Code/spark/shopify_repo/org/airhorns/pyrolite/9.9.0/pyrolite-9.9.0.jar (86 KB at 5680.5 KB/sec)
Uploading: file:///Users/hornairs/Code/spark/shopify_repo/org/airhorns/pyrolite/9.9.0/pyrolite-9.9.0.pom
Uploaded: file:///Users/hornairs/Code/spark/shopify_repo/org/airhorns/pyrolite/9.9.0/pyrolite-9.9.0.pom (393 B at 383.8 KB/sec)
Downloading: file:///Users/hornairs/Code/spark/shopify_repo/org/airhorns/pyrolite/maven-metadata.xml
Uploading: file:///Users/hornairs/Code/spark/shopify_repo/org/airhorns/pyrolite/maven-metadata.xml
Uploaded: file:///Users/hornairs/Code/spark/shopify_repo/org/airhorns/pyrolite/maven-metadata.xml (300 B at 146.5 KB/sec)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 0.366s
[INFO] Finished at: Sat Apr 11 16:39:02 EDT 2015
[INFO] Final Memory: 3M/81M
[INFO] -----------------------------------------------------------------------

and then changed the org ID and version in the core/pom.xml
@drdee @yagnik can you tell me if this is an ok way of doing this? I will undo it when a new version of pyrolite is published, and presumably Spark core will bump to the new version as well.

Oh also it builds cleanly on Packserv http://pack.chi.shopify.com/packages/Shopify/spark/f2848e3665f49dadb8a5814665eba25f0ab12cdf.log

ping @drdee @yagnik

If the jar is on packserv, why not add packserv as a repository?

It isn't on packserv (though I could get it there), and I'd also like people to be able to build without needing access to packserv (like on a plane or whatever).

2 minor comments, otherwise :+1:

I agree with @drdee on both points but apart from that :ship:
Please do create an issue to fix this when things are updated.

Closing cause upstream spark has already done this


[Fleet] Add source information to configuration settings

Previous PR before packages got moved around : https://github.com/DataDog/datadog-agent/pull/19832
TL;DR
Enable knowing which source is setting which setting in the Agent configuration
Future consequences
When using the .Set() method to change a configuration setting in the agent, you must now add a model.Source parameter, keeping in mind the following hierarchy between them:
CLI > Remote Config > Agent Runtime > Env Var > File > Unknown > Default
You could also use the .SetWithoutSource() method, it will then assume SourceUnknown
If you want to override a setting (e.g. for a unit test), you can now call .Set() specifying a source higher in the hierarchy, then use the .UnsetForSource() method ; the applied setting will fallback to its previous value
What do the sources correspond to?

Default is the lowest in the hierarchy, also Go fallback state (empty string). Also used when calling .SetDefault()
Unknown is used by the .SetWithoutSource() method, you should avoid using it on purpose
File corresponds to the datadog.yaml configuration file
Env Var corresponds to the environment variables
Agent Runtime is used when the agent itself changes values for its configuration, not to be confused with runtime settings
Remote Config is used to remotely change the behaviour of the agent
CLI is used when a user manually enters commands

What does this PR do?

Move the Source type from the settings package to model (Viper wrapper)
Add one Viper instance per source, and edit the .Set() method to include a source parameter
Add a .SetWithoutSource() as a legacy way to change a setting without specifying a source
Add the different sources to the inventories payload

Motivation
For Fleet Automation, we want to display the configuration of the agent and the source of every setting.
Use case: a customer doesn't understand why when he changed a setting in the datadog.yaml file it didn't have any effect on the agent (because he had forgotten to unset an ENV VAR).

Additional Notes
A lot of the changes are moving .Set() calls to .SetWithoutSource(), the more impactful changes are in the following files:

pkg/config/model/viper.go
Adding the sources to the Agent inventories payload
comp/remote-config/rcclient/rcclient.go

More detailed explanations
Because Viper doesn't support multiple sources of setting, the workaround is to spawn one instance of Viper per configuration source. Everytime a setting is changed, we check the value of every instance and the value of the highest source is applied to the "main" Viper instance.
Only the Set core concept is changed, the Get behavior do not change

Possible Drawbacks / Trade-offs

The new Viper instances will require more computing for the Agent
The inventories payload will be heavier, REDAPL agreed for the storage aspect

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Following @BaptisteFoy review: the .SetWithoutSource() method is only used in tests and mocks


Copy some files to Japanese content directory

This is a...

[x]  Feature Request
[ ]  Bug Report

Problem:
hugo build does not succeed because there aren't several required files.
Proposed Solution:
Copy the following files to Japanese content directory.

[x] content/ja/_common-resources/index.md
[x] content/ja/case-studies/_index.html
[x] content/ja/examples/minikube/Dockerfile
[x] content/ja/examples/minikube/server.js
[x] content/ja/docs/concepts/overview/what-is-kubernetes.md

#11571
/close


[BrokenLinksH2] Fix path in link

Fixing external broken link

Docs Build status updates of commit 3057ff8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav-app/LocalFunctionality/Spain/report-347.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Subcharts with the same dependency but different versions override each other's templates functions

Helm version: version.BuildInfo{Version:"v3.11.2", GitCommit:"912ebc1cd10d38d340f048efaf0abda047c3468e", GitTreeState:"clean", GoVersion:"go1.18.10"}
I noticed some odd behavior of helm when a chart has 2 different sub-charts, each of them with a dependency on the same chart. I have created a minimal example to reproduce the issue.
Steps to reproduce
Clone the repo: https://github.com/yardenshoham/helm-bug-report.git and run helm template reproduce .. You should see the following output:
coalesce.go:223: warning: destination for postgresql.ldap.tls is a table. Ignoring non-table value ()
Error: template: reproduce/charts/keycloak/charts/postgresql/templates/primary/statefulset.yaml:433:20: executing "reproduce/charts/keycloak/charts/postgresql/templates/primary/statefulset.yaml" at <include "postgresql.readinessProbeCommand" .>: error calling include: template: reproduce/charts/airflow/charts/postgresql/templates/_helpers.tpl:120:15: executing "postgresql.port" at <.Values.service.port>: nil pointer evaluating interface {}.port

Use --debug flag to render out invalid YAML

You can also run helm create reproduce and add the following to reproduce/Chart.yaml:
dependencies:
  - name: airflow
    version: 10.3.1
    repository: https://github.com/bitnami/charts/raw/archive-full-index/bitnami
  - name: keycloak
    version: 13.3.0
    repository: https://github.com/bitnami/charts/raw/archive-full-index/bitnami

Then run helm dependency build reproduce and helm template reproduce reproduce and you should see the same error.
What I think is happening
Both the airflow chart and the keycloak chart have a dependency on the postgresql chart. Airflow has a dependency on the postgresql chart version 10.9.3 and keycloak has a dependency on the postgresql chart version 12.2.1. When helm is trying to render the keycloak chart, it is using the postgresql chart version 10.9.3 _helpers.tpl instead of the version 12.2.1. This is because the postgresql chart version 10.9.3 is the first postgresql chart that is found in the charts directory of the keycloak chart. So there is a bug in the way helm is looking for the postgresql chart version 12.2.1.
Because both charts use functions with the same name they override each other.
Expected behavior
There should be isolation between the charts. Each chart should use its own dependencies and not use the dependencies of other charts. I see no reason for the current behavior. I should be able to use any version of a chart as a dependency without worrying about other charts using the same chart.

mostly the same behavior here:
in Chart.yaml
dependencies:
  - name: elasticsearch
    alias: es-7
    version: ~7.17.3
    repository: https://helm.elastic.co
    condition: es-7.enabled
  - name: elasticsearch
    alias: es-6
    version: ~6.8.13
    repository: https://helm.elastic.co
    condition: es-6.enabled

in values.yaml:
es-7:
  enabled: true
es-6:
  enabled: true

when I install my umbrella chart without packaging/pushing/pulling from registry, everything is fine.
If I do so, only one of ne charts is installed.
More info:
$> helm dep list
NAME         	VERSION	REPOSITORY             	STATUS
elasticsearch	~7.17.3	https://helm.elastic.co	too many matches
elasticsearch	~6.8.13	https://helm.elastic.co	too many matches

and
$> helm package .
Successfully packaged chart and saved it to: /home/user/tmp/helm/bug/bug-0.1.0.tgz
$> tar xzf bug-0.1.0.tgz
$> more bug/charts/elasticsearch/Chart.yaml
(...)
version: 7.17.3

i also met this issue. when i update the most inner sub chart for some minor change, i'll need update all of the subcharts referecing it. otherwise i'll get rendered output of the old version.

This is working as designed.

An important detail to keep in mind when naming templates: template names are global. If you declare two templates with the same name, whichever one is loaded last will be the one used. Because templates in subcharts are compiled together with top-level templates, you should be careful to name your templates with chart-specific names.

https://helm.sh/docs/chart_template_guide/named_templates/

At first look this is fairly reasonable, noting the chart-specific names. This issue is about versions, I think Helm should recommend chart-specific and version-specific names for named templates to avoid what happened here.

Good idea. Please open an issue in the docs repo. :smiley:

Duplicate of https://github.com/helm/helm/issues/11561


No service of type Factory available in ProjectScopeServices.

I have cloned the repository and have installed Maven but it show me this error while building project

Unless you need Android support, you can try a more actively maintained fork.


Fix Problem with redirect_uri with dots too

@pandafox The commit in this PR (fc140b4) is already being used in accounts.freedom.tm for 10 months now. I'll merge this.
https://github.com/anyTV/accounts.freedom.tm/blob/9d2d05be3c900eddd5c344916852cf4fd0f9aa7b/backend/package.json#L28


Build a PoC to take an arbitrary JSON and convert it into the equivalent Avro

It must respect the rules of converting JSON Schema -> Avro schemas too (https://github.com/fge/json-schema-avro).

Can probably just use this: http://avro.apache.org/docs/1.6.2/api/java/org/apache/avro/data/Json.html

Closing as Avro support will need its own RFC.


WINTER watchdog: always look at the past 2 nights.

For winter since the topics are expressed in local time we already did UTC date - 1 to get the topic we want to look at.
Another issue is that at 0 UTC is when we move to new topics (when we also restart the machine every day btw), so if the WINTER alerts (that are processed and sent to IPAC manually every day, after the night of observation) are sent at or after that time, upon reboot we already are listening to the next topic and won't read the data from the previous night.
So in this PR, we just make the obsdate parameter a list (obsdates), which defaults to not just UTC -1, but UTC -1 AND UTC -2.
We could consider looking back a third day just to be safe, might want to just make that configurable at some point.

FYI WINTER alerts are not sent manually to IPAC each day. They are sent to IPAC as the final step in our automated DRP, similar to ZTF, but our processing time for a given night can vary substantially and we currently process the entire night sequentially rather than image-by-image. The upshot is that if we observe a lot of the galactic plane, processing can last beyond the cutoff time for Kowalski.

FYI WINTER alerts are not sent manually to IPAC each day. They are sent to IPAC as the final step in our automated DRP, similar to ZTF, but our processing time for a given night can vary substantially and we currently process the entire night sequentially rather than image-by-image. The upshot is that if we observe a lot of the galactic plane, processing can last beyond the cutoff time for Kowalski.

Oh cool! I remembered that this was not fully automated earlier on, is that correct?

FYI WINTER alerts are not sent manually to IPAC each day. They are sent to IPAC as the final step in our automated DRP, similar to ZTF, but our processing time for a given night can vary substantially and we currently process the entire night sequentially rather than image-by-image. The upshot is that if we observe a lot of the galactic plane, processing can last beyond the cutoff time for Kowalski.

Oh cool! I remembered that this was not fully automated earlier on, is that correct?

Or rather, that you guys had to wait before running the fully automated pipeline

It starts at a fixed time every day, rather than processing data in real time.
We do also have the option to push candidates in manually via the API, maybe that's the source of confusion? But that only gets used if we need to reprocess data, it's a few times a year that it gets used.


can't add docker secret in all namespaces

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

choose a project
add a docker secret and select all namespaces
finish writing and save

Result:
It can't be saved and the console will print 'TypeError: this.namespace is null'
Other details that may be helpful:
When you choose single namesapce and then choose back to all namespaces, the save will work
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.4.4
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): K3s
Machine type (cloud/VM/metal) and specifications (CPU/memory): cloud
Kubernetes version (use kubectl version):

version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.3+k3s1", GitCommit:"96653e8d86b74a1f73eadf09d4edc44778c2f8dd", GitTreeState:"clean", BuildDate:"2020-06-01T16:47:11Z", GoVersion:"go1.13.11", Compiler:"gc", Platform:"linux/amd64"}

containerd version (use docker version):

 1.3.3-k3s2

I can confirm I am having the exact same issue - I can't add a private repo.   Rancher v2.4.4 with Rancher UI v2.4.22.

I can confirm I'm also having this issue trying to add the github registry (docker.pkg.github.com) in rancher v2.4.4. It however works if I specify a namespace.

I think I have the same issue with Rancher 2.4.3
I think the issue is coming from here
assets/ui/components/cru-registry/component.js (line 78)
    willSave: function willSave() {
      var nsId = this.namespace.id,
          pr = this.primaryResource;
      Ember.set(pr, 'namespaceId', nsId ? nsId : TEMP_NAMESPACE_ID);
      var ok = this.validate();
      return ok;
    },

as this.namespace is null, we can create the registry.
I temporary resolve the issue by opening chrome debugger, added a breakpoint and execute this javascript command.
this.namespace={}

In fact, I see it has been already fixed in the ticket #26981

Issue is fixed in this this - https://github.com/rancher/rancher/issues/26981
Closing as its a duplicate.


Common criteria

@gzaronikas As discussed, I've refactor the Common Criteria to use community.general.xml instead of the xpath CLI and added it again to the collection.

@rpelisse +1, xpath was a nightmare!


Deferred loading does not defer @HEAD

We should get this fixed before the first alpha release.

This is fixed internally.


Using a custom xib file for the TableViewCell

Is it supported? I'm having trouble with it...

hello,
yes, using custom xib files for the tableViewCell(s) is possibile.
Even if the demo won't build, you should take a look at the following MLPAutoCompleteTextField's methods:

(void)registerAutoCompleteCellClass:(Class)cellClass forCellReuseIdentifier:(NSString *)reuseIdentifier
(void)registerAutoCompleteCellNib:(UINib *)nib forCellReuseIdentifier:(NSString *)reuseIdentifier

in the MLPAutoCompleteTextField/MLPAutoCompleteTextField.m file
Also, take a look at the DEMOViewController and DEMOCustomAutoCompleteCell classes.

This one I did find, and here's exactly where I got complicated.
Usually when I have a UITableView the cellForRowAtIndexPath has something like:
MyCell *cell = (SnackCell *)[tableView dequeueReusableCellWithIdentifier:@"myCellID"];
    if (cell == nil)
    {
        NSArray *nib = [[NSBundle mainBundle] loadNibNamed:@"myCellID" owner:self options:nil];
        cell = [nib objectAtIndex:0];
    }

How do I incorporate this concept into registerAutoCompleteCellNib (it constantly kept crashing)
Thanks for the help - love the package

I'll elaborate.
I have a text and an image.
The image is downloaded async via SDWebImage
It's size is unknown - and I want to "resize" it after download.
Couldn't do it so far with the default UIImageView.
A possible solution is a custom cell (As discussed in https://github.com/rs/SDWebImage/issues/30)
But I got in trouble when trying to add my xib (with it's "attached" class)
A point to the direction (or a demo with xib) will be appreciated

If you need to use a custom UITableViewCell with the MLPAutoCompleteTextField, you just need to define which Cell Class and Cell xib you want to use.
In your ViewController (where you're using the MLPAutoCompleteTextField instance), usually (but not necessarily) inside the - (void)viewDidLoad method, you just need to call the methods previously mentioned:
(void)registerAutoCompleteCellClass:(Class)cellClass forCellReuseIdentifier:(NSString *)reuseIdentifier
and
(void)registerAutoCompleteCellNib:(UINib *)nib forCellReuseIdentifier:(NSString *)reuseIdentifier
Example:
for example's sake, let's suppose that:

your custom UITableViewCell sub class is called MyCustomCell
your Cell view is called MyCustomCell.xib
your view controller's MLPAutoCompleteTextField property is defined as:

@property (weak, nonatomic) IBOutlet MLPAutoCompleteTextField *searchField;

in your view controller's viewDidLoad method you'll have:
- (void)viewDidLoad
{
    [super viewDidLoad];
    [_searchField registerClass:[MyCustomCell class]
           forCellReuseIdentifier:@"MyCustomCell"];

    [_searchField registerNib:[UINib nibWithNibName:@"MyCustomCell"
                                               bundle:nil]
         forCellReuseIdentifier:@"PersonAutocompleteCell"];
}


Insert Routing Trigger (with stub functions)

After discussions with Felix, we have decided to break #14 into more reviewable tickets and close that one.
In this one we will have a simple routing trigger which has stub functions for other related tasks such as table creation, cache plan management etc.  This one will be simple and is not expected to solve anything other than:

Route the inserts into the correct table
Detect if the table doesn't exist and then direct to a stub function

Plan caching, table creation, and bad data handling will all come later.

So looking at this, what I think needs to be done is:

Create a state memory context and a cache memory context if not exist
Query and store the dimension info in a single linked list if not exist
Generate the name of the table from the dimensions and JSON
Check if we have a cached plan and use it if we have.
If we don't hjave a cached plan, write one, and then use it.

It seems to me we can probably use Postgres datetime logic internally.
How does this sound?

We will need to store the following in the state context:  generation information and dimension information

Current WIP handles points 1 and 2.


Add functionality

Adding in the ability to replace land/surface from an existing start dump file, and correcting paths used when using the BARRA-R2 archive.

Thank you for the review comments @CodeGat.
I marked this PR as draft for now, because its head branch add-functionality has a few missing commits.
The new branch add-functionality_corrected has been created and will likely be used instead.
@engelca please consider adding Tommy's comments in the add-functionality_corrected branch instead of this one.
Thank you.

Closed as substituted by an updated PR


New pkl files in alpha and updated alphamodel.py

Good afternoon, Professor! First, I am so sorry for the occasional errors that occurred during the modeling process. After a careful examination, it is due to our misunderstanding with some of the data and thus we updated our pred_prep_dic dataset in the folder alpha, and use the function get_embedding from the ANN instead of creating a new PKL file. Do you mind checking this out? In addition, thank you so much for solving the issue of reading PKL this morning!

columbia-e4579-api-1  |   File "/usr/src/app/src/recommendation_system/recommendation_flow/model_prediction/AlphaModel.py", line 90
columbia-e4579-api-1  |     except:
columbia-e4579-api-1  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
columbia-e4579-api-1  | SyntaxError: default 'except:' must be last
This broke production, reverting


SAK-33788: Added DejaVuSans font for UTF8 characters as it is in Calendar. (Based on the SAK-21922 patch)

SAK-33788: Added DejaVuSans font for UTF8 characters as it is in Calendar. (Based on the SAK-21922 patch)

I agree, these really shouldn't be duplicated. Possibly could have even went with the webjar.
https://mvnrepository.com/artifact/net.sourceforge.jeuclid/dejavu-fonts/2.29
Even though last released in 2009 still looks "newer" than what was used here.


deps: exclude nock >= 12x

Like a lot of our deps, the latest major release of Nock v12.x requires Node 10.x or higher, so adding this to our dependabot exclusion list (I also reordered the list to be alphabetical for some reason 🤷♂)
Refs #4674

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @calebcartwright!

  Generated by :no_entry_sign: dangerJS against 111b5d988d9387c64797e7ed3215a0451c5ca2e0


OWNERS: make the LLVM team own all of LLVM

Things done
Comes from @infinisil's work on the Nix owners bot. We've had issues where not all members of @NixOS/llvm are pinged by Ofborg when they probably should be so let's just make everyone own it.

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Idk, I think someone who has the right perms just needs to edit the team and add the nixpkgs repo to it? Probably @infinisil knows best for this.

Looking into it, I thought this should work :thinking:

Ah, turns out it's not at all possible to request a review from a team if it's not added as a repository collaborator. Write access is not needed, just read access is enough. @Lassulus can do that (go here and add Nixpkgs with read access)

Yeah, I figured that one out when making the PR, hence the pings.

I'm requesting Alyssa on this because I am unsure if we're wanting redundancy in the owners file.


⚠ Nextcloud has degraded performance

In 16e6f65, Nextcloud ($URL_NEXTCLOUD) experienced degraded performance:

HTTP code: 200
Response time: 5787 ms

Resolved: Nextcloud performance has improved in b7e7d9c.


Feature request: Queue should have #addAllFirst

This issue was originally filed by<EMAIL_ADDRESS>
Per the discussion on https://code.google.com/p/dart/issues/detail?id=10891, I propose adding Queue#addAllFirst to add items in front of the queue.

I agree with order-preserving insert (i.e. queue.AddAllFirst(iterable) would be the queue equivalent of list = [...iterable, ...list], without the copying overhead)
It felt really weird to me that the Queue class has all the features to be used as a double ended queue, but the AddAll method only exists for insertion at the end. This addition would tidy it up.

I agree with order-preserving insert (i.e. queue.AddAllFirst(iterable) would be the queue equivalent of list = [...iterable, ...list], without the copying overhead)
It felt really weird to me that the Queue class has all the features to be used as a double ended queue, but the AddAll method only exists for insertion at the end. This addition would tidy it up.

Also, I'd prefer order-preserving insert, because if I wanted the order to be reversed, iterable.forEach(queue.addFirst) is clean enough as it is. There isn't a clean way for doing the opposite.

Also, I'd prefer order-preserving insert, because if I wanted the order to be reversed, iterable.forEach(queue.addFirst) is clean enough as it is. There isn't a clean way for doing the opposite.

This should behave the same as JS' unshift() method, which preserves the iteration order.
final fruits = Queue<String>.of(["Apple", "Banana", "Peach"]);
fruits.addAllFirst(["Pear", "Pineapple"]); // ["Pear", "Pineapple", "Apple", "Banana", "Peach"]


Document Management and Tooling Q3

Main Branch

https://github.com/SingularityNET-Archive/snet-doc-manager/commits/main/?since=2024-07-01&until=2024-09-30

10 July 2024

Added headers to Archival copies
Created notifications when we cant access docs or they are deleted

24 July 2024

Added testing component to test existing and future functions

Right now I have to change the API keys every week. I will look into finding a way to generate a more permanent key.

6 September 2024

Refactored code to use different google auth tokens (Fixed the problem of having to replace API keys every week)

Next steps - work on documentation on how to use the tool and continue testing

Complete for Q3.
Ready to send for payment in Oct 2nd fund request


Plugin user should have the option to scale the mobile service to paid scale mode .

Plugin Version - 0.1.226
In the current plugin , the mobile services created are by default of free scale mode and there is no option with in the plugin to scale up the mobile service to paid scale mode .
This could cause limitations to the user such as inability to create more than one scheduled jobs .

mobile services support in IntelliJ toolkit has been deprecated.


Instructions do not connect with reality

Please check the instructions as they describe options that aren't there.  User Configuration does not have an option to enable windows hello for business, it only has PIN complexity.  Computer configuration has the option to enable windows hello for business.

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

ID: 0ec90f64-9a12-fdf9-25db-0e83cf77b7a2
Version Independent ID: fb68756c-2d0c-c28c-4bda-ef6a876576e7
Content: Configure Windows Hello for Business Policy settings in an on-premises certificate trust
Content Source: windows/security/identity-protection/hello-for-business/hello-cert-trust-policy-settings.md
Product: windows-client
Technology: itpro-security
GitHub Login: @paolomatarazzo
Microsoft Alias: paoloma

Thank you for the feedback, @ptpro-douglashahn . I'll verify the instructions and update the article as needed.

I checked and the policy is documented correctly:


[HOLD for payment 2023-07-05] [HOLD] [$1000] Message deletion is delayed when clicking on the 'Reply Thread' button and attempting to delete the message

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

Action Performed:

Open any chat.
Write a message and click on the "Reply Thread" button.
From within the reply thread, try to delete the message that started the thread. The message is not deleted immediately.

Note from Slack: I have tested the issue and found that the problem only occurs when attempting to delete the initial message within the thread. Deleting messages outside of the thread works successfully
Expected Result:
The delete button should function properly and the message should be deleted immediately.
Actual Result:
The delete button is not working properly and the message is not deleted immediately.
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
screen-capture (71) (1).webm
https://github.com/Expensify/App/assets/93399543/11f0532a-65e9-408a-8c6c-a726d6c31dbf
Expensify/Expensify Issue URL:
Issue reported by: @tewodrosGirmaA
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686248573425969
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e29b3bd427d9027b
Upwork Job ID:<PHONE_NUMBER>226010112
Last Price Increase: 2023-06-21

Asked a quick clarifying question about the behavior in the Slack thread above.

Ok! I've clarified the OP (and reprodiced the behavior myself)! Let's get some eyes on this.

Thanks @mananjadhav for surfacing that! I'll go ahead and put this one on hold.

@mananjadhav can you complete the above BZ checklist when you get a moment?

I don't think the BZ Checklist is necessary here. It looks like the linked PR above is associated with a different Issue, and the BZ Checklist will be completed there.

Thanks, @aimane-chnaif! It looks like that issue has been closed and paid out. I think that means we're good to close this one as well.

Hi @johncschuster  , I am not get paid for this issue

@aimane-chnaif , I am not eligible for payment, i am confused by this https://github.com/Expensify/App/issues/20907#issuecomment-1611452797

It's automatic post and that's because PR linked this issue

Thank you brother


fix broken footer links

We don't make changes like that to gh-pages. Everything goes into v4-dev and committed to gh-pages when a release is made.


custom group naming convention for metabase provider

for this PR: https://github.com/odpf/guardian/pull/162
Describe the bug
custom groups created by guardian in metabase are added as resources
Context

existing metabase provider supports collection and database resource access
(in metabase) database, table, and collection access are granted to groups, and a group has many users
while in guardian, a user access is pointing to a resource directly.

so to accommodate that, we make the metabase provider create a group that is specific for accessing a resource with a specific role, in other words, this group will always have access to one resource with one role.
see below diagram for illustration

we're currently introducing 2 new resources in guardian which are table (child resource under database) and group
table resource is fine, but for group, we don't want to include those custom groups as resources in guardian

Expected behavior
custom groups shouldn't be added as resources in guardian
Proposed Solution

have a naming convention for the group name e.g. add prefix _guardian_<group_name>
when fetching group resources from metabase, exclude any groups that have prefix _guardian_

https://github.com/odpf/guardian/pull/162/commits/5d01b7c9ae0ef8ddc64a5745f81a208b0b4be9cf
@rahmatrhd added _guardian_ as a prefix to groups created by Guardian, also check on _guardian_ while adding group as resources into Guardian.


Release?

So I was running some calculations on a 100MB BAM file and freebayes ate >200 GB of RAM.  It appears this issue has been resolved (#225) in the latest master, but not in any of the releases.
The bioconda binary versions are currently at 1.0.1.  If you made a tiny release, someone there could increment the version and rebuilt a binary that doesn't have the memory issues.
https://github.com/bioconda/bioconda-recipes/blob/master/recipes/freebayes/meta.yaml

Agree, It would be great if you could release a new version.

@ekg To clarify, once you post a Github release I will happily take care of the bioconda update.

Do you need the build as well, or just the tag?
On Sun, Jan 10, 2016 at 12:05 AM, Kyle Beauchamp<EMAIL_ADDRESS>wrote:

@ekg https://github.com/ekg To clarify, once you post a Github release
I will happily take care of the bioconda update.
—
Reply to this email directly or view it on GitHub
https://github.com/ekg/freebayes/issues/233#issuecomment-170295475.

All I need is the version number and ill just download using the release
link tarball.
On Jan 11, 2016 5:30 AM, "Erik Garrison"<EMAIL_ADDRESS>wrote:

Do you need the build as well, or just the tag?
On Sun, Jan 10, 2016 at 12:05 AM, Kyle Beauchamp<EMAIL_ADDRESS>
wrote:

@ekg https://github.com/ekg To clarify, once you post a Github release
I will happily take care of the bioconda update.
—
Reply to this email directly or view it on GitHub
https://github.com/ekg/freebayes/issues/233#issuecomment-170295475.

—
Reply to this email directly or view it on GitHub
https://github.com/ekg/freebayes/issues/233#issuecomment-170549351.

@kyleabeauchamp  https://github.com/ekg/freebayes/releases/tag/v1.0.2

Thank you for the help :)

Thanks @ekg
The new package is now online: https://anaconda.org/bioconda/freebayes/files

Awesome, thank you again!
On Mon, Jan 11, 2016 at 5:37 PM, Kyle Beauchamp<EMAIL_ADDRESS>wrote:

Thanks @ekg https://github.com/ekg
The new package is now online:
https://anaconda.org/bioconda/freebayes/files
—
Reply to this email directly or view it on GitHub
https://github.com/ekg/freebayes/issues/233#issuecomment-170628415.

Thanks!


Execute It on Server

Hi, I have generated the exe successfully but is It possible run It from a server (PHP)?
Thanks

Yes. You should be able to, just like you execute any other shell command from php.
http://php.net/manual/en/function.shell-exec.php


[BUG] Too many keys specified; max 64 keys allowed

regression test failed with tidb master branch
https://internal.pingcap.net/idc-jenkins/blue/organizations/jenkins/tispark_regression_test_daily/detail/tispark_regression_test_daily/1890/pipeline
[2020-11-02T20:48:51.206Z] org.apache.spark.sql.TiDBMapDatabaseSuite *** ABORTED ***

[2020-11-02T20:48:51.206Z]   Exception encountered when invoking run on a nested suite - Failed to initialize SQLContext:Too many keys specified; max 64 keys allowed, please check your TiDB cluster and Spark configuration (SharedSQLContext.scala:97)

should revert this PR https://github.com/pingcap/tispark/pull/1760 when this issue fix

https://github.com/pingcap/tidb/pull/20353
TODO: add a configuration to tidb

https://github.com/pingcap/tidb/pull/21192


Fix for...of bug in /learn command

This loop previously ignored the first item in the array (which is the name of the Pokemon), but this was lost in the for...of refactor. I think this is better readability-wise than any shorter or hackier solution.
On the other hand, I'm really behind on the latest standards, so this may not be the optimal way to fix this.

Good catch, thanks!

Travis tests have failed
Hey Marty-D,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
Node.js: 7.7

npm test

><EMAIL_ADDRESS>test /home/travis/build/Zarel/Pokemon-Showdown
> eslint --rulesdir=dev-tools/eslint --cache . && tsc && mocha

/home/travis/build/Zarel/Pokemon-Showdown/chat-plugins/datasearch.js
  1531:191  error  Irregular whitespace not allowed  no-irregular-whitespace

✖ 1 problem (1 error, 0 warnings)

npm ERR! Test failed.  See above for more details.


[Bug] KPSB: FAIL###KPSB-Unexpected on moving directory

What happened?
I changed the directory of BB and changed the config and now i am getting KPSB: FAIL###KPSB-Unexpected
BoxBilling Version
v4.21
What version of PHP are you running?
PHP 8.0
Additional Notes
No response

This was a server error, it fixed itself after about 6 hours


Git revert action

It would be nice to have an action to create a revert from a pull request.

Hello @jd! is this issue still open for contribution?

Any chance?

Hi @mkurz !
Everything's possible. What'd be your use case?

Everything's possible. What'd be your use case?

Quickly reverting a pull request.

Like pull requests that got merged accidentally.

Knowing that GitHub has a revert button, what'd be the difference in your regard?

Wait... wat? GitHub has... a... revert button? omg... yes you are right I just found it :stuck_out_tongue_winking_eye: Sorry for the noise (I usually never want to revert PRs just today and I thought I tell mergify, because we use it a lot and overlooked the button, thanks!)


SDF-based mechanism for making letters bold

Pull Request Description
Fixes 181641027. There are new methods of text::Area for making some letters bolder, without moving them nor any other letters in the line.
Tested on DejaVuSans text:

On MacOS:

Important Notes

Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the Scala, Java, and Rust style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH ./run dist and ./run watch.

Acceptance/QA passed.
Note: For QA, I extended the debug scene to make it easier to visually inspect that the width and positions are maintained. Some complex unicode is not rendered correctly by text::Area, but fixing that is not in the scope of this task. The SDF mechanism appears to be working correctly even in complex cases and on characters composed of disjoint areas.


[Snyk] Upgrade com.google.errorprone:error_prone_annotations from 2.17.0 to 2.18.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade com.google.errorprone:error_prone_annotations from 2.17.0 to 2.18.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-01-09.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Closing as we have a manual PR


Should probably document Azure Government's altered devicelogin URL

Have hit a few edge cases where logging into CLI to Gov still shows the Commercial devicelogin url:
https://microsoft.com/deviceloginus

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

ID: ca77e611-bec8-f429-e880-44c1aa433baa
Version Independent ID: 46432101-58dc-8a29-100e-91cd4ac90b66
Content: Connect to Azure Government with Azure CLI
Content Source: articles/azure-government/documentation-government-get-started-connect-with-cli.md
Service: azure-government
GitHub Login: @Juliako
Microsoft Alias: juliako

Thanks for your comment. The feedback has been shared with the content owner for further review. Thanks for your patience.

This is an old issue.  I followed up separately but could not establish a current repro.

#please-close


Updated internal naming conventions to follow Bayesian syntax

Pull Request Test Coverage Report for Build 398

26 of 26 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 98.011%

Totals

Change from base Build 396:
0.01%

Covered Lines:
2267

Relevant Lines:
2313

💛  - Coveralls


fix issues #223

https://github.com/gocolly/colly/issues/223
Use body response to validate it is html instead of use response header Content-type

Detecting the content type this way leads to errors too if the content doesn't start with '<!doctype html>`.
E.g. https://www.imdb.com has an extra whitespace before the docstring declaration.
The proper solution would be a validator public member function which could handle non-standard edge cases too. Something like Collector.IsHTML func(Response) bool. What do you think?

Collector. IsHTML func (Response) bool```
Agree, can you give me some suggestion?
Can I getting first 200 character of the body, then confirm it contain doctype html

I think checking the doctype tag is the right approach, but perhaps the body should be passed through strings.TrimSpace() first.  I think any characters other than whitespace would break html parsing any ways.
Or perhaps use regex but I'm not sure which would be more efficient.

These are just heuristics, so I'd like to be able to change it, that's why I suggested a public attribute for this function. I can have a textfile which starts with <!doctype, this case it would be good to overwrite the default heuristics function (which checks the content-type and/or the first n bytes of the response).

Closing due to inactivity.


Initialize chains without having the full parent chains.

Especially for clients but also for validators it would be useful to be able to initialize and execute a child chain, and propose and validate blocks for it, without fully downloading the parent chain. The one certificate containing the OpenChain message should suffice.

I'd like to proceed in two steps here:

not blocking on a missing OpenChain message when executing certificates (should make the grand parent chain unnecessary)
not blocking on a missing OpenChain message when creating blocks (should entail from

Closing because
(i) was done addressed by #1220
(ii) is redundant with #413


How to npm install to include to libvips mozjpeg support

Hi!
trellisQuantisation() / trellisQuantization() && overshootDeringing() requires libvips to have been compiled with mozjpeg support.
How to I "npm install" sharp.js to include mozjpeg?
Thank you.

Hello, sharp searches for and supports a globally-installed libvips at npm install time.
http://sharp.dimens.io/en/stable/install/#linux
You'll need to install libvips from source, with its --with-jpeg-includes and --with-jpeg-libraries flags being appropriate here.
https://github.com/jcupitt/libvips#optional-dependencies
Good luck!

Thnak you for prompt reply.
Doest it mean that if there is no libvips on my machine yet, then during "npm install" sharp.js package will install basic version of libvips to my machine ? But if I want to use extra libraries than I have to install libvips from source myself with flags mentioned in your reply?

http://sharp.dimens.io/en/stable/install/#linux

libvips and its dependencies are fetched and stored within node_modules/sharp/lib during npm install. This involves an automated HTTPS download of approximately 6.5MB.

To use your own version of libvips instead of the provided binaries, make sure it is at least the version listed under config.libvips in the package.json file and that it can be located using pkg-config --modversion vips-cpp.


Weird Translation of Data Units under Chinese

I found that, under Chinese translation, data units such as gigabyte is translated to "吉字节（G字节（B））".
This is no doubt too long and confusing to read. 吉字节 is the correct translation to gigabyte, but we are still more accustomed to GB. The default name of gigabyte under Chinese in Windows is GB, too.

Other units also have such issue. Please fix them, thanks.

请问你觉得是否需要将“字节”保留呢？还是改为“B”？其他的都转换为英文字符吗？比如: 兆字节--->MB

其它都转换成英文字符。“字节”的话，我个人没有意见，也许也可以为了统一而写成“B”吧…

其它都转换成英文字符。“字节”的话，我个人没有意见，也许也可以为了统一而写成“B”吧…

谢谢，我查一下台港澳的繁体中文是否也是这样显示，然后再看看要怎么改。

This issue has been fixed with #1610.


Fixed style problems caused on certain pages

As seen on https://www.html5rocks.com/en/tutorials/security/content-security-policy/
Close #119

ooo nice fix. Thank you!