Uncaught Error: ENOENT, no such file or directory '/Users/alexhawkins/.atom/packages/linter-jsxhint/node_modules/.bin'

[Enter steps to reproduce below:]

...
...

Atom Version: 0.187.0
System: Mac OS X 10.10.2
Thrown From: linter package, v0.12.0
Stack Trace
Uncaught Error: ENOENT, no such file or directory '/Users/alexhawkins/.atom/packages/linter-jsxhint/node_modules/.bin'
At fs.js:807

Error: ENOENT, no such file or directory '/Users/alexhawkins/.atom/packages/linter-jsxhint/node_modules/.bin'
  at Error (native)
  at fs.statSync (fs.js:807:18)
  at Object.fs.statSync (ATOM_SHELL_ASAR.js:206:16)
  at LinterJsxhint.Linter._cachedStatSync (/Users/alexhawkins/.atom/packages/linter/lib/linter.coffee:69:17)
  at LinterJsxhint.Linter.getCmdAndArgs (/Users/alexhawkins/.atom/packages/linter/lib/linter.coffee:86:16)
  at LinterJsxhint.Linter.lintFile (/Users/alexhawkins/.atom/packages/linter/lib/linter.coffee:127:24)
  at /Users/alexhawkins/.atom/packages/linter/lib/linter-view.coffee:158:18
  at Array.forEach (native)
  at /Users/alexhawkins/.atom/packages/linter/lib/linter-view.coffee:157:18
  at FSReqWrap.oncomplete (fs.js:100:15)

Commands
     -0:57.7 core:save (atom-text-editor.editor.is-focused)
     -0:52.6 grammar-selector:show (atom-text-editor.editor.is-focused)
     -0:49.1 core:move-down (atom-text-editor.editor.mini)
     -0:47.8 core:confirm (atom-text-editor.editor.mini)
     -0:44.2 core:save (atom-text-editor.editor.is-focused)
     -0:34.4 core:undo (atom-text-editor.editor.is-focused)
     -0:29.1 editor:newline (atom-text-editor.editor.is-focused)
  9x -0:28.1 core:backspace (atom-text-editor.editor.is-focused)
     -0:24.9 core:move-right (atom-text-editor.editor.is-focused)
     -0:24.6 core:backspace (atom-text-editor.editor.is-focused)
     -0:24.4 editor:newline (atom-text-editor.editor.is-focused)
  5x -0:22.9 core:backspace (atom-text-editor.editor.is-focused)
  2x -0:19.1 grammar-selector:show (atom-text-editor.editor.is-focused)
  6x -0:14.8 core:move-down (atom-text-editor.editor.mini)
     -0:13.1 core:confirm (atom-text-editor.editor.mini)
     -0:01.2 editor:newline (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "monokai"
    ],
    "disabledPackages": [
      "autocomplete"
    ]
  },
  "linter": {}
}

Installed Packages
# User
angularjs, v0.2.0
angularjs-styleguide-snippets, v0.5.0
atom-beautify, v0.21.4
atom-jsx-transform, v0.1.0
atom-ternjs, v0.5.4
autoclose-html, v0.15.0
autocomplete-css, v0.4.0
autocomplete-html, v0.3.0
autocomplete-plus, v2.4.2
autocomplete-snippets, v1.0.1
color-picker, v1.4.4
html-id-class-snippets, v1.4.1
html2haml, v0.6.0
jshint, v1.3.0
linter, v0.12.0
minimap, v4.5.0
monokai, v0.12.0
react, v0.9.11
script, v2.18.0

# Dev
No dev packages

Try:

$ cd /Users/alexhawkins/.atom/packages/linter-jsxhint/
$ apm install


"CommandLine Error: Option '' registered more than once!" with DYLIB linking enabled

Hi,
while working on our downstream fork we noticed an issue resulting in the error mentioned in the title, when LLVM_LINK_LLVM_DYLIB, MLIR_LINK_MLIR_DYLIB and LLVM_BUILD_LLVM_DYLIB settings are turned on in LLVM. We were able to reproduce the issue using an unmodified version of onnx-mlir and llvm-project, using the onnxmlirczar/onnx-mlir-dev docker container.
We were using commit 476e2b8f436feb12592759c0a0deb4987baa1c42 to reproduce this. We haven't checked whether this still happens with the latest version onnx-mlir.
The exact steps to reproduce are:

Use the onnxmlirczar/onnx-mlir-dev docker container
Checkout commit 476e2b8f436feb12592759c0a0deb4987baa1c42 in onnx-mlir
Checkout the corresponding LLVM commit (438e59182b0c2e44c263f5bacc1add0e514354f8)
Configure LLVM with LLVM_BUILD_LLVM_DYLIB=ON, LLVM_LINK_LLVM_DYLIB=ON and MLIR_LINK_MLIR_DYLIB=ON
For all other settings, follow the build guide described in BuildOnLinuxOSX.md
Build llvm-project and onnx-mlir
Run ninja check-onnx-lit

The tests should fail with the aforementioned error.
Is this a bug or are dynamic libraries unsupported?

@tungld - can you suggest any contributors who might be knowledgeable on this topic?

onnxmlirczar/onnx-mlir-dev builds everything static. If you want to enable shared libraries, you need to rebuild llvm-project with -DBUILD_SHARED_LIBS=ON. You also need to rebuild onnx-mlir which will observe llvm-project and also build as shared.

Our user image onnxmlirczar/onnx-mlir is built with -DBUILD_SHARED_LIBS=ON so we know that works. In fact, you can see in docker/Dockerfile.onnx-mlir that we run
make -j${NPROC} LIT_OPTS=-v check-onnx-lit

after building onnx-mlir shared.
I think the problem is that onnx-mlir currently only checks whether llvm-project was built with -DBUILD_SHARED_LIBS=ON. If you use the other DYLIB options, onnx-mlir will actually build itself static.

Another problem is that with the DYLIB options, even though it builds a couple of libraries shared, such as lib/libLLVM-16git.so, etc., all the rest libraries are still built static. But when you build onnx-mlir shared, it sort of expects everything from llvm-project was built shared. So when llvm-project has a mix of shared and static libraries, onnx-mlir might not work.


Added missing replicator configs

Added heartbeat, maxRetries, and maxRetryWaitTime to CBLReplicatorConfiguration.
Added CBLReplicatorConfiguration_Default() to return a default config.
NOTE: CBLReplicatorConfiguration_Default() is inline with CBLDatabaseConfiguration_Default().

The test failure is not related to this PR and will be fixed seperately.

After thought, I'm converting this to draft until the name of the maxRetries is sorted out.


Supported or not supported?

Here it states "For example, restoring a database from SQL Server 2014 to SQL Server 2016 isn't supported" However here https://docs.microsoft.com/en-us/sccm/core/plan-design/configs/support-for-sql-server-versions#upgrade-options-for-sql-server you will find "Upgrade options for SQL Server: Use backup and recovery". Is it supported or not?

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

ID: 9d2bf93f-aa79-ef8a-ee16-c1ed27eb8e13
Version Independent ID: d64f0463-a1e0-b80b-f16b-39f54dd87154
Content: Site recovery - Configuration Manager
Content Source: sccm/core/servers/manage/recover-sites.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @Zebbelin for calling out that discrepancy. I want to confirm what I think is the right answer, stay tuned.

Aczechowski - Did you ever confirm if this is supported?

Thanks @JOBTheITGuy for the reminder. There's not a definitively clear response on this, as these statements have actually flipped a few times. Microsoft Support generally asks customers to use the same version to avoid any SQL problems. After discussion and testing, we changed the statement from "same version" to "any version". So I think the safest route is same version, but across version should work (and we've seen it work in some testing).

I revised this article, the changes will publish live on Monday 8/26 around 11am Pacific time.


Remote Sampling part 1 (Fetching Sampling Rules)

This PR is a part of the remote sampling support. This PR is adding support for fetching sampling rules from X-Ray service via GetSamplingRules API using aws proxy extension provided by collector. The PR has added necessary functions and interface to achieve this functionality and the scope of this PR is limited to fetching sampling rules. I have added couple ToDos which I will address in next consecutive PRs. I will just give a highlight of next follow ups PR after this.

Fetching Sampling Target via GetSamplingTargets API
Business logic for remote sampling
Instantiate fixed ration based sampler to address fixed rate requirements
Integration testing

Codecov Report

Merging #987 (741a285) into main (2e0f29a) will increase coverage by 2.4%.
The diff coverage is 50.0%.

@@           Coverage Diff           @@
##            main    #987     +/-   ##
=======================================
+ Coverage   69.3%   71.7%   +2.4%
=======================================
  Files        127      16    -111
  Lines       5496     585   -4911
=======================================
- Hits        3810     420   -3390
+ Misses      1538     147   -1391
+ Partials     148      18    -130

Impacted Files
Coverage Δ

...z/gomemcache/memcache/otelmemcache/test/version.go
0.0% <0.0%> (ø)

...n/github.com/gocql/gocql/otelgocql/test/version.go
0.0% <0.0%> (ø)

...b.org/mongo-driver/mongo/otelmongo/test/version.go
0.0% <0.0%> (ø)

...adfitz/gomemcache/memcache/otelmemcache/version.go
100.0% <100.0%> (ø)

...tation/github.com/gocql/gocql/otelgocql/version.go
100.0% <100.0%> (+100.0%)
:arrow_up:

...ongodb.org/mongo-driver/mongo/otelmongo/version.go
100.0% <100.0%> (+100.0%)
:arrow_up:

detectors/aws/ec2/version.go

detectors/aws/ecs/version.go

detectors/aws/eks/version.go

detectors/gcp/version.go

... and 127 more


Add FreeBSD instructions to Installation page

Mostly for visibility. Manual build only works for Meson and is what downstream package uses. CMake build needs more fixes due to manually written GNU Makefile.

thanks!


Unable to cancel file upload from controller

Hey everyone,
I need to display error message in case the file we are trying to deploy is with invalid extension.
here is my code:
UI:
Javascript Controller:
$scope.pngFileAdded = function(file, event) {
// Check file extension and initialize UI error message if it is not png
event.preventDefault(); //in debug this event is 'change'
}
After this the deploy of the file is still happening in the UI...
Am I doing it wrong? Maybe there is another way to display error message if the file is with incorrect extension?

btw 'return false;' in $scope.pngFileAdded did the trick.

You can do anything in here, so pass the error to the scope and display it.
$scope.pngFileAdded = function(file, event) {
// Check file extension and initialize UI error message if it is not png
event.preventDefault(); //in debug this event is 'change'
}

The thing is that even though I wrote event.preventDefault(); , the file didn't stop uploading. But after I added 'return false;' to this method, the file wasn't uploaded.

Same here! Thank you

i confirm that this bug exists @AidasK goin to work on this.

related: https://github.com/flowjs/ng-flow/issues/131

@AidasK i've got it and the issue is not easy to be solved i thing;
The event that we get inside a onFileAdded is not cancelable (e.cancelable is set to true)  and any change to the event is not appliced to the object.
    console.log(e.cancelable) # -> false
    e['defaultPrevented'] = true;
    console.log(e.defaultPrevented); # -> false
    e.preventDefault();
    console.log(e); # -> false

do you have any hint?

https://developer.mozilla.org/en-US/docs/Web/Events/change
the event in fact is an event of type 'change' and this events are not cancelable.

We have to return false here somehow. Strange I hadn't issues with this before
https://github.com/flowjs/ng-flow/blob/master/src/directives/init.js#L20

got it the issue is in suggesting users to do:  flow-file-added="onFileAdded($file, $event, $flow)"
obviously this makes no sense as the event is the browser event change that is not cancellabel and not the fileAdded event generated internally by flow

i think that we should be simple more clear that doing flow-file-added="onFileAdded($file, $vent, $flow)" in HTML is totally different from doing $scope.onFileAdded = function($file, $event, $flow) { i was also fooled by this.

this thicket could be closed so as there reason is clear and is simply a user bug;
@beastie1891 if you have any doubt on this let us know.


Can't import Bestiary 3 - Fossil Golem

Can't import statblock:
Fossil Golem      CR 12
XP 19,200
Male fossil golem (Pathfinder RPG Bestiary 3 136)
N Huge construct
Init +1; Senses darkvision 60 ft., low-light vision; Perception +0
Defense
AC 26, touch 9, flat-footed 25 (+1 Dex, +17 natural, -2 size)
hp 122 (15d10+40)
Fort +5, Ref +6, Will +5
DR 10/adamantine, 10/bludgeoning; Immune construct traits, magic
Offense
Speed 30 ft.
Melee 2 bites +20 (6d6+7 plus petrification)
Space 15 ft.; Reach 15 ft.
Statistics
Str 25, Dex 13, Con —, Int —, Wis 10, Cha 1
Base Atk +15; CMB +24; CMD 35
Feats Combat Reflexes[B]
Special Abilities
Combat Reflexes (2 AoO/round) Can make extra attacks of opportunity/rd, and even when flat-footed.
Construct Traits (+40 HP) Constructs have many immunities.
Damage Reduction (10/adamantine) You have Damage Reduction against all except Adamantine attacks.
Damage Reduction (10/bludgeoning) You have Damage Reduction against all except Bludgeoning attacks.
Darkvision (60 feet) You can see in the dark (black and white only).
Immunity to Ability Damage Immunity to ability damage
Immunity to Ability Drain Immunity to ability drain
Immunity to Bleed You are immune to bleed.

Immunity to Disease You are immune to diseases.
Immunity to Energy Drain Immune to energy drain
Immunity to Exhausted You are immune to the exhausted condition.
Immunity to Fatigue You are immune to the fatigued condition.
Immunity to Magic (Ex) Immune except vs. transmute rock to mud, or stone to flesh.
Immunity to Mind-Affecting effects You are immune to Mind-Affecting effects.

Immunity to Poison You are immune to poison.

Immunity to Stunning You are immune to being stunned.
Low-Light Vision See twice as far as a human in dim light, distinguishing color and detail.
Petrification (DC 19) (Su) Foes slowly turn to stone (1d6 Dex drain, Fort neg), turn to stone when DEX is drained to 0.

Used your latest module that was available as of this afternoon at about 2:30pm.  I used Hero Lab to produce this statblock. For greater accuracy, you might want to use one of the online resources. Hero Lab can truncate some of the text. All the other statblocks from Bestiary 3 imported -- a total of more than 86% imported!


Unable to power on AVR-1912

Hello Team,
I need your help on an issue that I have with a Denon AVR-1912.
All is working wonderfully except the power_on. I thought it was an issue from the HA interface but browsing the GIT I kinda understood how to interact with it directly from python. See for yourself :
``# python
Python 3.7.3 (default, May  8 2019, 05:28:42)
[GCC 6.3.0 20170516] on linux
Type "help", "copyright", "credits" or "license" for more information.

import denonavr
d = denovar.DenonAVR("<IP_ADDRESS>")
Traceback (most recent call last):
File "", line 1, in
NameError: name 'denovar' is not defined
d = denonavr.DenonAVR("<IP_ADDRESS>")
Receiver name could not be determined. Using standard name: Denon AVR.
d.power
'ON'
d.power_off()
True
d.power
'STANDBY'
d.power_on()
Connection error: power on command not sent.
False

``
Can you help me debug this see if I can get it to work?
Thanks
Chris

Oh by the way. I tried using "denon" lib to no avail either. Maybe an option on my AVR that I have to enable? (I do have "Network Standby" setting to "ON")

On my denon i have something like IP Control in my web setup menu.
If you have something like that turn it on


TypeError: moments() got an unexpected keyword argument 'shift'

Hey there. When trying to import InceptionV3, ResNet50, or Xception, using 1.2.1 I got this error:
>>> from keras.applications.xception import Xception
>>> m = Xception()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/mnt/hd/envs/esteban/miniconda3/envs/caffe/lib/python2.7/site-packages/keras/applications/xception.py", line 114, in Xception
    x = BatchNormalization(name='block1_conv1_bn')(x)
  File "/mnt/hd/envs/esteban/miniconda3/envs/caffe/lib/python2.7/site-packages/keras/engine/topology.py", line 572, in __call__
    self.add_inbound_node(inbound_layers, node_indices, tensor_indices)
  File "/mnt/hd/envs/esteban/miniconda3/envs/caffe/lib/python2.7/site-packages/keras/engine/topology.py", line 635, in add_inbound_node
    Node.create_node(self, inbound_layers, node_indices, tensor_indices)
  File "/mnt/hd/envs/esteban/miniconda3/envs/caffe/lib/python2.7/site-packages/keras/engine/topology.py", line 166, in create_node
    output_tensors = to_list(outbound_layer.call(input_tensors[0], mask=input_masks[0]))
  File "/mnt/hd/envs/esteban/miniconda3/envs/caffe/lib/python2.7/site-packages/keras/layers/normalization.py", line 120, in call
    epsilon=self.epsilon)
  File "/mnt/hd/envs/esteban/miniconda3/envs/caffe/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 1375, in normalize_batch_in_training
    shift=None, name=None, keep_dims=False)
TypeError: moments() got an unexpected keyword argument 'shift'

VGG16 runs just fine, tough.
Anything on my end that I could do? Thanks!

Are you on latest tensorflow and keras? What is your setup?
Works fine for me on bleeding-edge keras/tf/ubuntu.
Cheers

Tensorflow, theano and keras change so quickly that I would recommend using the bleeding-edge at all times if possible.

What version of TF are you using? shift has been supported since v.10 (or May 2016)

The fix is to upgrade your TF version to 0.12.


small fix, check that dataUrls are valid

Fix because every paste in textarea will show error message and replace image canvas with empty image
From https://github.com/ntsd/screenshot-to-code/pull/1
feat: https://github.com/abi/screenshot-to-code/pull/19

Thanks!


core-*-api: backport support for external route ref default targets + config route binding

Hey, I just made a Pull Request!
The new frontend system supports route binding through static configuration, for example:
app:
  routes:
    bindings:
      catalog.createComponent: catalog-import.importPage

It also lets you define default targets for external route references, for example:
export const registerComponentRouteRef = createExternalRouteRef({
  optional: true,
  defaultTarget: 'catalog-import.importPage',
});

This adds support for both of these features in the existing frontend system.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Merged by mistake, reverted and replaced by #24777


[FR] Store reply in folder of original message rather than "Sent"

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.714
Problem you are trying to solve
Every email written as a response is stored in the "Sent" folder (or whichever folder the user specifies).
This causes a hige amount of messages in one folder that all need manual sorting.
Suggested solution
Add an option to store the response in the same folder as the initial message, i.e. if an email is replied to that is located in the subfolder "Family", the response email is also stored in "Family" rather than in "Sent".
Ideally this would also apply to the inbox.
Screenshots / Drawings / Technical details
No response

@AleXSR700 We're planing to add a "Conversation View", see #6197. That's maybe what would solve your issue.

It would help but it would still store the emails in the Sent folder.
The idea is to "physically" store the messages in the same folder.
I know this feature from work (Outlook) and it is very useful in helping to stay organized.
Especially when threads and conversations are not working (Threads e.g. seem to be broken in K-9 at the moment).

Yes please integrate this function in the app like in the TB desktop version!!!


Small choice dialog fixes

-ListView selector wasn't working; add a selector to the individual buttons instead
-Make target area larger (had to do this with padding because the single_dialog_choice_view wasn't respecting its layout_height parameter; still not sure why)

This is a followup to https://github.com/dimagi/commcare-odk/pull/749


When opening a ccp4 map from EDS the map isn't well aligned onto the structure

Hi,
when trying to visualize the structure 4o71 and others (3q0r etc) using an input pdb file and density (2Fo) from the pdbe we can observe that the map is not well centered on the structure itself. Also only a single cell is shown.

I noticed that the NGL code doesn't do symetry operations that can be specified in the ccp4 format, but this isn't the only issue here, as 3q0r also isn't displayed properly.
Any idea on what needs fixing here?
I investigated the symetry operations a bit and found source code in uglymol treating this, but didn't manage to make it work for now.

Hi @pschmidtke, it'd be nice to handle symmetry operations - I've never tried to implement this, but happy to look at some code and see if it's possible

Hi there,
I had a similar issue and solved it first by converting the experimental data to MTZ file format:
https://www.rcsb.org/docs/general-help/x-ray-electron-density-maps#converting-experimental-data-to-mtz-file-format
Then I used the command phenix.mtz2map <mtz_file> <pdb_file> to generate the ccp4 map. Importantly, specifying the pdb file gives it something around which to draw the map!
https://phenix-online.org/documentation/reference/mtz2map.html
Would you be able to share your code to generate the above output? I usually work with nglview in a python notebook, but have been trying to convert a system like this into html so I can embed it (unfortunately nglview .write_html is unsuccessful in doing so).


Chest and angry ghost bug report

Host sees all chests but connected clients cant seem to see any chests.
Angry ghost hurts all players except the player that took angry ghost.

Follow up to angry ghost: this means it can provide infinite healing bolt to everyone, but the negative of being hurt by it outweighs that benefit in my opinion.

i disabled it for the next release for now, i think the chest thing is just physics objects sync not being a thing, maybe will be fixed in next release


docs(keycloak): fixes in readme file

GitHub issue: https://github.com/janus-idp/backstage-plugins/issues/392
JIRA: https://issues.redhat.com/browse/RHIDP-127

@tumido Could you please also review the changes in this PR?


Incorrect Test Failure Count in Jenkins Console

Description:
Display incorrect value for the test failure count in the Jenkins console. Even though there are failures in the test suite, failure count displays as 0 failures in the console.
Eg:
There are test failures in "EI test Module-mediator-1" test suite. Also it shows a test suite failure in test summary section.

But in the section which shows the individual test execution summary gives the failure count as 0.

Suggested Labels:
Bug
Suggested Assignees:
Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

This is fixed.


Standalone Scripts Docs

So, this part of the docs is somewhat terse and obscure:
https://django-configurations.readthedocs.org/en/latest/patterns/#standalone-scripts
I'm not sure what it means or why I would want it. Could anyone clarify it for me? I'd be more than happy to update the docs with extra detail once I've understood...

before being able to use Django in a Script (which is not a managment-script) some initialization has to be done in a certain order, so you can use models etc.
Since the setup-format with django-comments is different a modified method has to be used for initializing django - without django-configurations you'd use this: https://docs.djangoproject.com/en/1.9/ref/applications/#django.setup
The documentation could be improved by adding a link do the documentation of django.setup().

I have tasks.py for uwsgi spooler, which is handled by uwsgi, when I setup it as:
import configurations
configurations.setup()

It works fine, but what if I'm developing reusable application? This app can be used with or without django-configurations, but I can't use:
from django.apps import apps

if apps.is_installed('configurations'):
    import configurations
    configurations.setup()

because django-configurations doesn't need to be listed in INSTALLED_APPS..
And I can't just check if environment variable for django-configurations exists, coz it can be just defined, but it doesn't means that project runs via django-configurations..
How to correctly detect if project runs via django-configurations or not?


R&D inlining, jitted and overloaded.

This patch contains some more concrete implementations for
supporting inlining of jitted functions and @overload'd functions

@ehsantn you may be interested in the contained code. @overload inlining is made a bit more robust. Here's a quick example:
from numba import njit, types, ir
from numba.extending import overload

#-------------------------------------------------------------------------------
# demonstrate @overload inliner

def bar(a):
    pass

def sentinel_17_cost_model(caller_inline_info, callee_inline_info):
    # sentinel cost model, this is a fake cost model that will inline if the
    # ir.Const(17) is found in the ir, it's for demonstration purposes
    for blk in callee_inline_info.func_ir.blocks.values():
        for stmt in blk.body:
            if isinstance(stmt, ir.Assign):
                if isinstance(stmt.value, ir.Const):
                    if stmt.value.value == 17:
                        return True
    return False

@overload(bar, inline=sentinel_17_cost_model)
def _bar(x):
    if isinstance(x, types.Integer):
        def impl(x):
            return x + 10
    else:
        def impl(x):
            return x / 17
    return impl

@njit
def foo(z):
    x = z + 1
    y = bar(x)
    return y + 3

print(foo(10.2)) # the second impl in _bar is used, and inlined as it contains 17
print(foo(10))   # the first impl in _bar is used, it is not inlined, no 17!

#-------------------------------------------------------------------------------
# demonstrate @njit inliner

def inline_the_world_cost_model(caller_ir, callee_pyfunc):
    return True # inline everything!

@njit(inline='always')
def foo(l, val):
    l.append(val)
    bar(l, val)

@njit(inline=inline_the_world_cost_model)
def bar(l, val):
    for i in range(val):
        l.append(i)

@njit
def func(val):
    l = []
    foo(l, val)
    return l

print(func(3))
#-------------------------------------------------------------------------------

TODO:

[ ] tests
[ ] docs
[ ] extend to other numba.extending API functions.

@shssf: This is the PR that we should compare to HPAT's inlining pass.

Thank you @stuartarchibald . This is very useful! A couple of comments:

Can this be extended to overload_method and overload_attribute?
Overloads are replaced after typing which makes sense, but can it be done during typing to avoid lowering the standalone implementation?
Will this work for functions accessed across modules using attribute, e.g. foo.bar.g()
I'm not sure if inline_closure_call can handle free vars properly. Any idea?

@ehsantn thanks for taking a look.

Can this be extended to overload_method and overload_attribute?

Almost certainly, once this bounded case is working and we're convinced of it.

Overloads are replaced after typing which makes sense, but can it be done during typing to avoid lowering the standalone implementation?

I think at present, inline='never' will trigger compilation, inline=<some function>, will trigger compilation, not much that can be done about that right now, but inline='always' should only run the front end to obtain typing information and not actually compile anything. This branch https://github.com/numba/numba/pull/4208/files#diff-d403faa8b837ae083d77e0cba8abd6c7R437 will run up to here https://github.com/numba/numba/pull/4208/files#diff-d403faa8b837ae083d77e0cba8abd6c7R454 in the 'always' case.

Will this work for functions accessed across modules using attribute, e.g. foo.bar.g()

Not sure, what is presented within targets the most simple case. I would expect that it's subject to the same resolution rules as current overloading.

I'm not sure if inline_closure_call can handle free vars properly. Any idea?

Could you clarify this question please? Perhaps with an example? There's a block in the inline_closure_call that does freevar replacement, are you wondering if this is sufficient?
https://github.com/numba/numba/blob/32cbb3211ee41adbb9c1a56032f25e67367db449/numba/inline_closurecall.py#L308-L324

@stuartarchibald
Thanks for the info. Sounds good.
follow up on 3 above: as I suspected by looking at the code, njit inlining doesn't work across modules. It can't resolve functions through getattr nodes.
contents of mymodule.py:
import numba

@numba.njit(inline='always')
def bar(a):
    return 4*a

contents of main file:
import numba
import mymodule

@numba.njit(parallel=True)
def g():
    return mymodule.bar(9)

g()

Regarding item 4 above, freevars don't seem to work either:
import numba

def bar():
    pass

@numba.extending.overload(bar, inline='always')
def bar_overload(a):
    val = 3
    return lambda a: 4* val

@numba.njit(parallel=True)
def g():
    return bar(9)

print(g())

The problem seems to be that get_ir_of_code (used for typing inlined overloads) doesn't handle freevars well. It uses __globals__ but not __closure__. It assigns freevars to None, expecting them to be handled later probably (i.e. inline_closureclall): https://github.com/numba/numba/blob/master/numba/ir_utils.py#L1557

@stuartarchibald
Thanks for the info. Sounds good.
follow up on 3 above: as I suspected by looking at the code, njit inlining doesn't work across modules. It can't resolve functions through getattr nodes.
contents of mymodule.py:
import numba

@numba.njit(inline='always')
def bar(a):
    return 4*a

contents of main file:
import numba
import mymodule

@numba.njit(parallel=True)
def g():
    return mymodule.bar(9)

g()

I think I've worked out how to do this now... loosely, the inliner looks at call site, gets a definition and checks the condition ir.Expr.op == 'getattr', if true it walks up the IR getattrs looking for a module and then replays the attributes with actual getattr on the module to find the actual function which can then just be pushed into the standard mechanism for deciding about inlining.

Regarding item 4 above, freevars don't seem to work either:
import numba

def bar():
    pass

@numba.extending.overload(bar, inline='always')
def bar_overload(a):
    val = 3
    return lambda a: 4* val

@numba.njit(parallel=True)
def g():
    return bar(9)

print(g())

I think I've got this working too, it seems to reduce down to the following: In   inline_closure_call if the callee presented is a function then don't strip out the code object and compile it in a new function object because it loses all the freevar info, instead just run the compiler frontend as that already handles this situation ok.
This means that:

If the callee is a function then it gets compile with its own freevars and globals and the resulting  IR is used as the inlinable IR.
If the callee is a code object then it gets synethesised as a function and the globals used in compiling it to IR are those supplied to the inline_closure_call function, the resulting IR is used as the inlinable IR.

I don't think this will solve the generic problem of "I have a code object and the freevars need somehow plumbing in", but for the situation above this seems to work as an approach.

sounds great @stuartarchibald. Looking forward to these.

@ehsantn I've pushed up some code that works for the above cases, if you have examples that are similar and don't work it would be great to see them, thanks.

It's a bit odd that the costmodel is receiving the callee has a python function whereas the caller is a func_ir.  The asymmetric is unexpected.  I'd think both should be a func_ir.  That way, the costmodel has still access the pyfunc via func_ir.func_id.func.

@sklam fair point, it is currently around the way it is to avoid building IR unless needed, am happy to change. Do you think the IR of the callee should be the directly interpreted IR or should it have closures inlined?

This fails if a function has both inlined and non-inlined overloads. See comment here: https://github.com/numba/numba/pull/4208/files#r316367561
import numba
from numba.extending import overload

@overload(len, inline='always')
def overload_len_foo(A):
    if False:
        return lambda A: 10

@numba.njit
def g():
  return len([2, 3, 4])

g()

File "/Users/ehsan/dev/numba/numba/compiler.py", line 573, in stage_inline_overloads_pass
inlineoverloads.run()
File "/Users/ehsan/dev/numba/numba/ir_utils.py", line 2124, in run
if guard(self._do_work, work_list, block, i, expr):
File "/Users/ehsan/dev/numba/numba/ir_utils.py", line 1379, in guard
return func(*args, **kwargs)
File "/Users/ehsan/dev/numba/numba/ir_utils.py", line 2195, in _do_work
inline_closure_call(self.func_ir, impl.globals,
AttributeError: Failed in nopython mode pipeline (step: inline overloads)
'NoneType' object has no attribute 'globals'

keyword arguments don't seem to work. The issue seems to be here: https://github.com/numba/numba/pull/4208/files#diff-d403faa8b837ae083d77e0cba8abd6c7R442
Does the main pipeline fold keyword arguments before calling type inference?
import numba
from numba.extending import overload

def foo(a, b):
    return a + b

@overload(foo, inline='always')
def overload_foo(a, b):
    return lambda a, b: a + b

@numba.njit
def g():
  return foo(3, b=4)

g()

Invalid use of Function(<function foo at 0x10cf5f268>) with argument(s) of type(s): (Literal[int](3), b=Literal[int](4))
 * parameterized
In definition 0:
    TypeError: Mismatch number of argument types
    raised from /Users/ehsan/dev/numba/numba/compiler.py:1140
In definition 1:
    TypeError: Mismatch number of argument types
    raised from /Users/ehsan/dev/numba/numba/compiler.py:1140
This error is usually caused by passing an argument of a type that is unsupported by the named function.
[1] During: resolving callee type: Function(<function foo at 0x10cf5f268>)
[2] During: typing of call at ../s.py (16)

File "../s.py", line 16:
def g():
  return foo(3, b=4)
  ^

@ehsantn thanks for taking a look at this. I think that there's a more fundamental problem with the inliner itself, this fails:
from numba import njit

@njit
def foo(a, b, c):
    def bar(x, y, z):
        return x + y, z
    return bar(a, b, z=c)

foo(10, 11, 12)

with
  File "<path>/numba/numba/inline_closurecall.py", line 436, in _replace_args_with
    assert(idx < len(args))
AssertionError: Failed in nopython mode pipeline (step: inline calls to locally defined closures)

as a result, even when the above is fixed such that the code makes it through typing, the inliner cannot seem to handle the result.

Does the main pipeline fold keyword arguments before calling type inference?

yes, kind of, IIRC it amalgamates args with kwargs and uses Omitted etc. If this is worked into the pass appropriately the IR becomes invalid because the inliner writes the literal values of the kwargs in as ir.Assign values. This is likely going to need more work.

@stuartarchibald I'm using a hacky piece of code to handle kws and defaults in inlining. You might be able to use it:
        # create explicit arg variables for defaults if func has any
        # XXX: inine_closure_call() can't handle defaults properly
        if pysig is not None:
            pre_nodes = [] if pre_nodes is None else pre_nodes
            scope = next(iter(self.func_ir.blocks.values())).scope
            loc = scope.loc
            def normal_handler(index, param, default):
                return default

            def default_handler(index, param, default):
                d_var = ir.Var(scope, mk_unique_var('defaults'), loc)
                self.typemap[d_var.name] = numba.typeof(default)
                node = ir.Assign(ir.Const(default, loc), d_var, loc)
                pre_nodes.append(node)
                return d_var

            # TODO: stararg needs special handling?
            args = numba.typing.fold_arguments(
                pysig, args, kws, normal_handler, default_handler,
                normal_handler)

Thanks @ehsantn, the standard methods do not work, as demonstrated above, they need defaults turning into actual values. I'll see if I can work something like this in/fix up the inliner to just sort it out. I am also of the view that the problem here should not be in scope of this PR, it's an issue independent of inlining.

@stuartarchibald I agree that we shouldn't increase the scope of this PR too much. It's a major improvement already - we can add more later.

@ehsantn, I presume you are referring to errors in some other project making use of the inline_closure_call routine as Numba CI is passing?
I think this change (run the front end if the callee is a FunctionType):
https://github.com/numba/numba/pull/4208/files#diff-69e01738577449bec5a17f54f5845f47R279-R283
or this (sort out kwarg defaults that are not ir.Var by virtue of the callee being a function not a closure):
https://github.com/numba/numba/pull/4208/files#diff-69e01738577449bec5a17f54f5845f47R325-R334
are potential causes.

Thanks for checking and for the review @ehsantn. Thanks for the review too @sklam. I think we can finally consider this for merge!?!


ISLE: add support for extended left-hand sides with given clauses.

This PR adds support for given clauses, as proposed in
rfcs#21. These clauses augment the left-hand side (pattern-matching
phase) of rules in the ISLE instruction-selection DSL with
sub-patterns matching on sub-expressions. The ability to list
additional match operations to perform, out-of-line from the original
toplevel pattern, greatly simplifies some idioms. See the RFC for more
details and examples of use.
Creating this as a draft since the RFC is pending; the intent for now
is to provide an implementation to play with, given initial positive feedback,
but this will not merge until after the RFC does.

cc @fitzgen @abrown

@fitzgen I've updated this implementation to correspond to the now-merged PR, and added language-reference updates as well. I think it's ready for review!
The one change relative to the RFC's language spec is that the pure designator comes on the decl instead of the extern constructor, which seemed more logically consistent to me once I realized that we'll want to do pure-expression-checking on internal constructor bodies invoked from if-lets as well.


A CSI client: gocsi

This patch introduces a CSI client written in Go: gocsi.

planning to re-submit in some other form?

Hi @jdef,
Yes, I will be resubmitting. I combined the example and library, and I'm waiting until it's a little closer to what I want to show before reopening the PR. By the way, Clint said we'd get along and work well together, so I do want to arrange a Hangout with you soon so we can chat :)


OOB write in zsl_mtx_eigenvectors

I'm not sure I'm understanding this block of code, but there's definitely an unchecked out-of-bounds write here which is crashing my code:
int
zsl_mtx_eigenvectors(struct zsl_mtx *m, struct zsl_mtx *mev, size_t iter,
		     bool orthonormal)
{
	<...>

	/* Since 'b' is the number of eigenvectors, reduce 'mev' (of size
	 * m->sz_rows times b) to erase columns of zeros. */
	mev->sz_cols = b;

	for (size_t s = 0; s < b; s++) {
		zsl_mtx_get_col(&mev2, s, f.data);
		zsl_mtx_set_col(mev, s, f.data);
	}

	<...>
}

I'm unclear why the number of eigenvectors would be greater than the number of columns. However, when it is, this code manipulates memory well outside the allocated space.
zsl_mtx_eigenvectors is used as I would expect by zsl_mtx_svd - allocating only enough space for eigenvectors matching the number of columns - but if that's not true, then there's an OOB write even within the same file.
int
zsl_mtx_svd(struct zsl_mtx *m, struct zsl_mtx *u, struct zsl_mtx *e,
	    struct zsl_mtx *v, size_t iter)
{
    ZSL_MATRIX_DEF(aat, m->sz_rows, m->sz_rows);
    ZSL_MATRIX_DEF(upri, m->sz_rows, m->sz_rows);

    <...>

    zsl_mtx_eigenvectors(&aat, &upri, iter, true);

Here, aat and upri are allocated the same size, but if it's true that there can be more eigenvectors than rows, then upri doesn't have enough memory allocated.
Most other math libraries I work with tend to return orthogonal eigenvectors, and therefore limit themselves to the number of columns.
What am I missing?

I think I may be running up against this as well. I have been trying to use zsl_mtx_eigenvectors on a symmetric 3x3 covariance matrix. Even reducing the number of iterations significantly still triggers a stack overflow unless an 80 kb stack size is used. This seems mighty excessive and certainly too much for my application.
However, I haven't got too far in understanding why. I ended up using the Jacobi approach instead which works fine for the square symmetric matrices I am using.


World map location markers (flashing arrow on minimap)

Description
I would like to propose a feature that would be amazing in your everyday scaping. I would love to have a feature like RS3 already has in it's world map:

An option to place a marker on the official OSRS world map.
Minimap arrow hinting to where you placed your marker (IIRC this is used for coordinate clues)

Use cases

Doing quests
Doing clue scrolls
Regular playing
...

Extra features

Clear marker option on minimap orb
Possibility to place multiple markers on the map (maybe make a chain of markers so when you reach point A you get a notification and the arrow now points to point B if that makes any sense)

Duplicate #4496 #8074


docs: add wallet-sdk-react example

Summary
This PR adds a new entry to our lineup of examples in the apps folder, and uses that example to add a docs page about the Wallet SDK's React hooks. The page is very basic and the purpose for now is to just have something that makes SDK users aware about the Wallet SDK, rather than having no mentions.
We can add more thorough examples once the wallet sdk <-> ts sdk migration thing becomes more stable. The Wallet SDK team is also migrating the React Hooks to a new package.
Closes #1569

Thanks for the explanation @Dhaiwat10 👍🏻


netavark: failed to delete if podman4: Received a netlink error message Operation not supported (os error 95)

While running recent CI VM F36 images through the podman integration tests, noticed a number of them are failing with this (and similar) errors (annotated logs)..  This is NOT happening on previous F36 CI VM images, suggesting a package update has caused a regression.  Examining the new images, I see netavark-1.0.3-3.fc36.x86_64 installed, so maybe simply a F37 package update is needed?

yes you have to update to 1.1

the error happens since kernel 5.19

Does Lokesh do that or is there a button a simpleton like me can press?

Crap so this is broken for everybody using Netavark on F36?  Ugh :vomiting_face:

Thanks for the quick attention @Luap99
@lsm5 Given the F37 timetable, I think I need to build new CI VM images that side-load this update in F36.  Please LMK as soon as a download URL is available somewhere.  TIA :smiley:

Oh, even better: I found a way to bypass the CI-problem in F36 images completely.
@lsm5 urgency--
Details: We're "suppose" to be testing CNI in Fedora N-1, but the number "35" was hard-coded in the build script.  I'm fixing that condition now, so CNI will be tested instead of Netavark.  That doesn't help users though, only our CI system.

yes broken for everyone @lsm5 Can we get https://bodhi.fedoraproject.org/updates/FEDORA-2022-29c1ba1101 into stable?

it should land in stable in 24 hrs or less.

Thanks @lsm5


[suggestion] Error Handling Event Strategy

In the provider.js file, I see a this.defaultNotifyTarget option, which I could use to disable notifications.
Would it be possible to take that further and create a strategy for handling validation errors when they submit a form via aa-submit-form? i.e. if I don't want to use your notifications or have a one-off case where I want to do something very specific.
The default strategy could be the notifications as it is now, so functionality isn't changed, but I could override that and say, do a native popup (on a mobile device) when they try and submit the error without their email.

Nice, that sounds like a good idea to allow for better customization. I'll see if I can do that sometime otherwise feel free to send a PR!


GenericContainer passes the wrong ItemStack to setInventorySlotContents when shift-clicking

When shift-click is used to remove items from a GenericContainer slot, setInventorySlotContents gets passed an empty ItemStack instead of the ItemStack that was removed.

Looks like we need to start listening to Slot#onSlotChanged.

This is no longer as critical, since I found a way to make shields work without needing the parameters to setInventorySlotContents to be correct, but it's still a bug.

Correction: it's normal to get passed an empty ItemStack; the bug is that the existing stack is empty too.


Support Quantized Model

Support Quantized Model.
For example:
https://huggingface.co/THUDM/chatglm2-6b-int4
https://huggingface.co/Qwen/Qwen1.5-72B-Chat-GPTQ-Int4

Inference's speed is slow


linebreak_optimized: Handle breakpoints separated only by spaces

This skips any predecessor checks when the line would start with a token that consists only of whitespace.
Fixes #1794

Confirmed, this fixes the issue I initially encountered in a production document.

I wonder why there are such breakpoints in the first place.

This only happens since we've switched to icu4x. I tested and xi-unicode does not report the problematic breakpoint. I opened an issue on the icu4x repository to get to the bottom of this.

Could you add a note to the added code linking to the icu4x issue? Since there seems to be little activity on that issue, I'm fine with merging this for now. However, if it is indeed an upstream issue, we can then easily later revert the change.

I think this can also happen without justification. There was a similar bug report on Discord without justification and it makes absolute sense. Moreover, this seems to have broken raw blocks (see #2019). Perhaps we should roll back this change and make a fix to the linebreak iterator to undo icu4x's weird behaviour.

The upstream bug is fixed.


[Carousel]: Update button style

User story
We need to update the carousel buttons to use the tertiary style instead of the secondary.
Carbon guidelines state that dual secondary styles should never be used together.
Additional information

Storybook
Carousel specs
Card section carousel specs

Acceptance criteria

[x] Review designs with Jeannie
[ ] Update or create design specs in Figma
[ ] Create development story
[ ] Update Figma component

Design crit 02/02/23
Moving forward with dual tertiary styling to bring in line with Carbon guidance and not introduce too many different styles.


Course enrollement

Worked On Issue Number: #662
Issue Type: Feature Added.
@heet-kumar Please check it.

@manisha-27

@manisha-27 it was stated that Contributorscan make pr upto 2nd June. @heet-kumar @abhijeet007rocks8


Avatar with Badge

The code does not match the displayed images. In GFAvatar there is not an option for badges.

I will take a look at it

@piyushbhatia891 any updates?

@rlee1990 Hey, using GFIconBadge widget you can create avatar with badge.
This is the sample code  :
const GFIconBadge(
child: GFAvatar(
backgroundColor: GFColors.SUCCESS,
),
counterChild: GFBadge(
text: '12',
shape: GFBadgeShape.circle,
),
),
We will update our docs soon. Thank you

@deepikahr thanks it worked.


New tag lists: EnglishDictionary.csv and Derpibooru.csv

I've put together two additional tag lists that should be formatted correctly for TAC to use. I made these for my tagging app and I figured you may be interested in them. I hope it's useful, but if you're not interested in incorporating them, no sweat!
I've had the English Dictionary tag list for awhile now, but recently created the Derpibooru list because of the popularity of Pony Diffusion V6 XL.
I also wanted to apologize because I've used the danbooru.csv and e621.csv tag lists in my own app without giving you credit, I wasn't thinking when I got the files from here. Sorry about that, I'll make sure to update the repo giving you credit.

EnglishDictionary.csv is in this format: <name>,<type>,<postCount>,"<aliases>"
The aliases for EnglishDictionary.csv are common typos and corrections. I wanted to classify words based on nouns, verbs, adjective, etc. but I couldn't find a good way to do this so the <type> column is always "0".
Because of the huge effort required, I tried to prioritize <aliases> with just the most common words. Also I was only able to retrieve useful <postCount> data for about half the total words, the rest are simply sorted alphabetically. This is less than ideal but I don't know how to improve it.

Derpibooru.csv is in this format: <name>,<type>,<postCount>, (no aliases)
The <type> key for Derpibooru is:
1 = content-official
2 = BLANK (general tag)
3 = species
4 = oc
5 = rating
6 = body-type
7 = character
8 = origin
9 = error
10 = spoiler
11 = content-fanmade

For many tags the "data-tag-category" was blank when scraping, it appears these are general tags.
I could probably create another scraper that could grab tag aliases, but I didn't have the time while setting this up originally.
I've tried to do my best to clean up the tag list without removing or altering useful tags, but there's always a chance of ruining something. Because of this I'm also including the file Derpibooru_original.csv. This is the direct result of scraping the tag list before any additional cleanup or changes. You may have a better idea of how to clean it up than I do.

CSV Files:
EnglishDictionary.csv
Derpibooru.csv
Derpibooru_original.csv

Much appreciated! Just a quick question, the derpibooru list still seems to contain quite a lot of tags with very low post counts from what I've seen at a glance. For danbooru I just cut it off at 100k (even then there are a lot of not-so-useful tags still included), but here it might be better to go directly by post count and e.g. drop all lines with less than 20 posts or some higher cutoff number. What do you think? For the english dictionary this of course isn't an issue.

I had the same thought, and I was going to cut off the tags at low post counts, but there are many tags that seem useful regardless of the low count. I really don't have much experience using these tags, so I don't know how useful they actually are with Pony V6.
I'm totally in favor of pruning the derpibooru tags. The list is fairly chaotic because there's so much OC, and the users seem to invent new tags all the time. I even setup my scraper to ignore "oc:" and "artist:" tags with a post count less than 35 because there were so many results.
A cutoff of less than 20 posts would be fine, but that's slightly more than half the tags gone. There's certainly a lot of junk tags, but I wouldn't exactly call a tag junk because it has a low count. It may be well represented in Stable Diffusion, just not on the imageboard.
I found some issues with sorting and some other small stuff, and I removed all tags with a post count less than 5. I still think it's a fine idea to cutoff more, but this is probably how I'll use it just for more completeness.
Derpibooru_EDIT.csv

Yes, it's always a balance problem, with the normal booru tags too. There are a lot of tags in the lower post counts that are perfectly understandable concepts for SD, just almost never used for tagging posts. Especially when it comes to landscape descriptions, lighting, styles etc. where the concept is common in natural language (and thus known by most SD models) but might get condensed to one tag like "outdoors" on the boorus, even if a more fine-grained distinction technically exists.
At the end it often comes down to how well the site enforces its tagging rules, and from what I've seen there at a glance, derpibooru tags are often more of a shitpost than in any way related to the images in the lower counts. Like, nobody can convince me "muffled_rap_music_playing_in_distance" or "less_than_five_seconds_in_mspaint" is a useful tag under any circumstances (and these two examples are at least understandable English).
Anyways, thanks for the edit.
There is one issue remaining that I encountered, which is the tag categories. Derpibooru uses pretty different categories and colors to normal danbooru. On the technical side, all is well, since TAC supports custom colors per file already. But the issue is that this is an option, so while I can update the default, existing users will not get a fitting color scheme unless they manually add it or delete that option from the webui's config file to reset it to default. That will result in a lot of tags being interpreted as -1 (unknown) since their number doesn't exist in the danbooru fallback color scheme, which marks them crimson red by default. Most noticeably every number > 5 and especially category 2, which is skipped in danbooru.
Since you mentioned the cat 2 was initially just blank, I have just replaced it all with 0 to match danbooru's general tag, which at least fixes a good amount. But that still leaves 6-11 unspecified.


undefined reference in libospray building any of the applications.

Hi,
I get the following error when building any of the example ospray applications:
../libospray.so.1.3.0: undefined reference to `ospray::PixelOp::Instance::~Instance()'
I am building version 1.2.1 from the source tarball ( git devel and master branch have same error). I am using ispc 1.9.1, embree 2.15, and tbb from tbb2017_20170226oss (the latest version from the website). I am using the intel15 compiler on a centos6 system. I have gcc/4.9.1 in my library path.
I hate to bug you guys but any idea where this error is coming from? I cant seem to find the missing bit in my configuration.
Thanks,
Dave Semeraro

I made the following change to PixelOp.h
//virtual ~Instance() = default;
virtual ~Instance(){};
This allowed the applications to compile. I have not had a chance to see if any of them run. It seems the default virtual destructor was the problem. I still think I have a configuration problem. The default destructor compiled but left the undefined reference. Perhaps order of linking? I dont know. ..

Hi Dave,
I can reproduce the problem: it is the old version 2015 of icc, version 2016 works. However, your workaround should do the trick as well (it works for me also with icc15).
Best regards,
Johannes

Thanks Johannes, compiler bugs are the worst.
Dave Semeraro
Visualization Group
Texas Advanced Computing Center
University of Texas Austin
From: Johannes Günther<EMAIL_ADDRESS>Sent: Friday, April 21, 2017 6:45 AM
To: ospray/ospray<EMAIL_ADDRESS>Cc: Dave Semeraro<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ospray/ospray] undefined reference in libospray building any of the applications. (#139)
Hi Dave,
I can reproduce the problem: it is the old version 2015 of icc, version 2016 works. However, your workaround should do the trick as well (it works for me also with icc15).
Best regards,
Johannes
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ospray/ospray/issues/139#issuecomment-296169718, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJR3p4F3naBPpwB2zkMihTzBU67wG2fzks5ryJbQgaJpZM4NDgLh.

FWIW, we don't get the traditional benefits of icc with OSPRay because most of our performance sensitive code is compiled by ispc. By the time ispc code makes it to the C++ compiler, it cannot be further optimized. Thus if it makes things easier for you, compiling with gcc or clang will essentially give you the same result as using icc.

Thanks Jeff, I usually default to the icc compiler because most of the stuff I deal with does benefit from it.  Looking forward to experimenting with the latest OSPRay.


Bugs

Error: Implicit conversion from float 593.6922 to int loses precision
File: pmsrc/src/world/World
Line: 1774
Type: ErrorException

Which version are you using?

2.0.2-alpha and 5.0 pmmp

Try latest 2.0.4-ALPHA


Offset fix

+qa

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable


Allowing PHP8

Hey @freekmurze
I've allowed PHP8 and upgraded one dev dependency to get it working. Tests run through on PHP8.0.0.
Cheers,
Peter

I don't see any jobs regarding PHP 8 here yet.

I don't see any jobs regarding PHP 8 here yet.

That's better, now get them all green 🙂

That's better, now get them all green 🙂

Little confused here: I've just tried to identify the issue by finding the place where this is called (line is 0 or "n/a" isn't overly helpful on the first look).
The method with parameters prepare(string $query, array $options = []) doesn't exist in the code or dependencies.
But it is described here: https://www.php.net/manual/en/pdo.prepare.php - notably PHP8 isn't mentioned in the versions on top. Wonder why? I guess like the missing second parameter is handled more strict now.
Not sure what to make from this really. I'll look at it with fresh eyes in the morning

Little confused here: I've just tried to identify the issue by finding the place where this is called (line is 0 or "n/a" isn't overly helpful on the first look).
The method with parameters prepare(string $query, array $options = []) doesn't exist in the code or dependencies.
But it is described here: https://www.php.net/manual/en/pdo.prepare.php - notably PHP8 isn't mentioned in the versions on top. Wonder why? I guess like the missing second parameter is handled more strict now.
Not sure what to make from this really. I'll look at it with fresh eyes in the morning

Looks like dbal is pretty low with v2.3. I could try to pull up it up to 3 or even the current dev-master by adding it to the dependencies. But they all have the same definition of the method in question as far as I remember from my quick look... Going to check it tomorrow unless you got an idea :+1:

Looks like dbal is pretty low with v2.3. I could try to pull up it up to 3 or even the current dev-master by adding it to the dependencies. But they all have the same definition of the method in question as far as I remember from my quick look... Going to check it tomorrow unless you got an idea :+1:

Hey @freekmurze
After noticing dbal v2.3 in the logs of the failed build I've enforced it to at least 3.0 - with this it's all green :+1: Not sure if there is anything else to look at?
Cheers,
Peter

Hey @freekmurze
After noticing dbal v2.3 in the logs of the failed build I've enforced it to at least 3.0 - with this it's all green :+1: Not sure if there is anything else to look at?
Cheers,
Peter

Looking good! Thanks!

Looking good! Thanks!

No worries, happy to help out!

No worries, happy to help out!


Clean up AWS testing

It is really confusing trying to start writing tests for chef-provisioning-aws.  We need to expose less boilerplate, remove as much of the metaprogramming as possible, clearly show the difference between the recipe section and the rspec section, etc.
We should add documentation about how to start writing tests for chef-provisioning-aws, and what is going on behind the scenes.
Finally we should add a dedicated AWS testing area that we can nuke regularly and not worry about interfering with others.  Then we should get these tests running through CI and CD.
\cc @randomcamel

Then we need a matrix of Chef versions to test against


memo: unignore test

test_memo_compute_limits() was ignored because the limits on bpf-tools v1.5 (solana master, solana v1.6) are significantly different than on previous bpf-tools versions (solana v1.5).
Once stable solana is on bpf-tools v1.5, unignore the test with new representative values, and update docs: https://github.com/CriesofCarrots/solana-program-library/tree/update-memo-test

Hi,
I would like to take it as my first contribution. Can you please help me and explain a little more about the issue or how should I go about it?
Thanks.


Spelling error in website boilerplate text regarding "You've got updates"

Here is a first shot at raising a Github issue:
The text in the "You've got updates" section on my iNat home page is
Stay current with updates on the following...
Your Content: Activity on your observations such as comments, identifications, or faves from the iNaturalist community. You will also receive updates when someone adds your observation to a project or annotated it with additional fields.
People: New observations or journal posts from the people you follow.
Organisms, Projects & Places: New observations from specific organisms or places you're subscribed to'. You can also join projects to be updated when they publish project journal posts.
Discussions you've engaged in: Commenting or adding identifications to someone >>>elses<<< content automatically subscribes you to any new activity on this content. Use the 'Manage your subscriptions' tool to stop >>>>recieving<<< updates.
"elses" should have an apostrophe, I think...
Also "receiving" is misspelt.
Regards
Geoff Shuetrim

I am going to fix the problems in the en.yml file (there are a couple of other recieve instances in the file) and will issue a pull request, just to make sure I have the workflow down...

Pull request works for us!

great talking you today Geoff!

Just to confirm, is this process the one that you prefer contributors to use?

Pretty much: fork, feature branch, PR. Though for minor stuff in repos that you're not doing a lot of active dev on, just using github's editing functionality is probably fine too (basically does the same thing minus the feature branch, I think).

This was fixed in https://github.com/inaturalist/inaturalist/pull/1145


Improve top level auto open detection.

As we discussed in https://github.com/safesparrow/fsharp/pull/24#discussion_r1035365291.
This makes the last resort link to all files scenario stricter and pulls up the content in a module with a namespace prefix scenario.

I added some comments, but I think there is a more general issue we're having, which is communication 😅
I'm realising that many of my comments are the result of different assumptions about what the current algorithm aims to do.
For example, I was always convinced that we would end up with a solution that avoids a link if the only reference is an open A.B statement and A.B only contains nested modules.
But, as we discussed earlier, this would require a separate piece of information to be tracked, which we would pass into the TcState for each file, so that type-checking doesn't fail on that open A.B statement.
I think we should have a chat about that and other things. It's probable that the above 'optimisation' would be negligible, because how often does someone reference a module but not use its contents? Maybe not too often.
That would somewhat justify different handling of namespaces and modules.
Overall, I just don't understand the dependency resolution algorithm TBH. Maybe that's fine, but it prevents me from reasoning about it.
I guess we just switched the roles 😅

Unrelated to this PR, but...
Given that our TrieNodeInfo allows only one file for a module, how do we handle a situation where the same module is (incorrectly) defined in two places? I guess we will somewhat randomly pick one of the files as a dependency when referencing it?
Or does the code throw when trying to merge two modules with the same name, from different files?

I think there is a more general issue we're having, which is communication

It appears so, although I think we can easily overcome that problem with a chat.

For example, I was always convinced that we would end up with a solution that avoids a link if the only reference is an open A.B statement and A.B only contains nested modules.

This is somewhat the case. Consider:
A.fs
namespace X.Y

module A =
   let a = 0

B.fs
namespace X.Y

type B = int

C.fs
module C

open X.Y

Only B will be a dependency of C. A is not exposing itself at the X or Y level of the trie.
However, if we don't have file B:
A.fs
namespace X.Y

module A =
   let a = 0

C.fs
module C

open X.Y

A will be a dependency of C because the open X.Y needs to lead to something in the trie.
This case is covered by the ghost dependency resolution. After all the content of C.fs was processed, we verify that the state does not have any lingering open statements. open X.Y will not lead to any files in the trie (as A doesn't expose anything at the Y level).
To overcome this we look in the child nodes of Y if we can find any index that is higher than the index of C and we pick this index as a dependency to satisfy the open X.Y statement in C.

Or does the code throw when trying to merge two modules with the same name, from different files?
No idea what happens in that scenario 🙈. It will probably throw.

I think this is good to go. Please take a peek at all this.
You would need to approve it before I can merge it.


cpu profiling delivers wrong result

I just profiled some Gradle builds using async-profiler and I noticed that the cpu profile has some really odd hot spots that definitely don't match reality:

You can see open() calls completely dominating the profile. I cross-checked this with the wall profiler (focussing only on the thread in question). The open calls are highlighted in purple. As you can see they really only make up ~1% of the runtime.

This is a single-threaded, CPU-bound program, so there should be very little difference between the cpu and wall results. I cross-checked with JFR, JProfiler and YourKit and all of them agree with the wall profile. I also did an experiment and completely removed this code path from the code base. It disappeared from the cpu profile, making it look more like the wall profile, however runtime was completely unaffected as I expected. This shows that there is in fact no significant time spent in this method.
You can reproduce these measurements with the following commands:

git clone git@github.com:gradle/gradle.git
./gradlew install -Pgradle_installPath=/usr/local/gradle-source-build largeJavaMultiProject (this will take some time)
cd subprojects/performance/build/largeJavaMultiProject
repeatedly run /usr/local/gradle-source-build/bin/gradle help while async-profiler is attached to the Gradle daemon process, then dump the results to SVG

I'm on macOS High Sierra and tried both Oracle Java 8 and OpenJDK 11 with the same results. Let me know if there's anything else I can do to get to the bottom of this.

Sounds strange.
Does the problem happen on Linux as well?
Is it possible to get some reduced test case?

I'll try it on Linux. Constructing a reduced case might be tough, as I have no clue what combination of calls is causing this pattern. I'll try, but the steps above should be relatively easy to follow, they just take a bit of time to execute since Gradle is a big project.

I get the correct results on Ubuntu 18.04, so this seems MacOS specific.

Thank you for the input. That's what I suspected.
macOS kernel is a damned blackbox, and I'm afraid I can't influence on how it counts CPU time spent inside kernel. Probably the result will be plausible if we count only user-space CPU time. Let me do some experiments to see if there is anything we can improve here.
Do I understand correctly that you have also tried -e itimer mode on Linux, and it also looks fine with no fileOpen bias?

Do I understand correctly that you have also tried -e itimer mode on Linux, and it also looks fine with no fileOpen bias?

No I tried it on MacOS and it had the same bias as -e cpu.

@oehme On macOS cpu and itimer modes are the same. itimer is basically the fallback mechanism on systems that have no perf_events support. Since you already have everything set up, can you please check if itimer behaves differently on Linux?

Related, I suppose :)
https://github.com/jvm-profiling-tools/async-profiler/pull/52#issuecomment-330086645

async-profiler (this PR) on Mac reports fewer total samples, and, of those, far too many seem to land in native IO methods.

@retronym thank you for testing!
Results seem legit to me, native stacks are not reported on mac os x and difference in samples count is acceptable: mac version samples random thread by timer interrupt and linux version samples all threads which are registered in perf_event_open with the same frequency


[BUG] Page header - doesn't support menu button

What package(s) are you using?

[X] Carbon for IBM Products (or Carbon for IBM Cloud & Cognitive) (Carbon 10)
[ ] Legacy/CDAI
[ ] Legacy/Security

Detailed description
Describe in detail the issue you're having.
The menu button from Carbon for IBM Products doesn't work in the page header. We assumed this is because the component is canary. However, my dev thinks that the page header restricts to either a menu button or regular buttons, and doesn't allow both to be used.

Desired design

What offering/product do you work on? Any pressing ship or release dates weshould be aware of?
App Connect SaaS. We are migrating to Carbon 11 in March, and as part of that work we are adopting the page header across the board. This is the one instance of the page header that isn't working.
Please create a reduced test case in CodeSandbox
Can ask my dev if needed.

@Laura-Marshall In v1, you would still need to enable the Canary component[^1].
Should be as simple as
pkg.component.ButtonMenu = true;

However, I do think your developer is probably right about us restricting the type of actions that can be passed. 😓
[^1]: Enabling canary components and flagged features

@elycheea Would we be able to request an enhancement removing some of those restrictions? In March we want to adopt the Carbon menu button + another in the page header (I assume as the menu button is new, it might not be supported in the v2 header?)

I think this is a reasonable request ... however, a design task for you if we need to support this ...

At smaller screen sizes, this collapses into a menu button. (Business logic we currently handle).

What’s the equivalent behavior for this if we allow support for a menu button?
Fwiw, we’ve been trying to reduce the amount of business logic we own in each component, but this one is probably hard to accomplish without refactoring a bit. 🤔
--
Notes for after the holidays so we don’t lose this train of thought ...
Breadcrumb might feel comparable with how they allow an OverflowMenu as a possible Breadcrumb Item. But they don’t handle the collapsing of items when space decreases — instead they wrap to the next line. (Compare against our BreadcrumbWithOverflow) when you change the containerWidth.
I definitely think the functionality we provide is useful, but probably need some smarter brains from our UXEs to figure out if it’s possible to provide this utility without owning more business logic for every possible outcome. 🤔

@asfordmatt Chatting with Laura about some tricky behaviors since there are a few breakpoints that would collapse from two buttons to a menu button and stumbled into “user defined actions” in the pageActions examples. Looks like it’s possible although as you might expect, the behavior would require a bit of work. (These are tested in v2 since I have it up currently, but as far as I’m aware, this shouldn’t have changed significantly from v1.)


Token names cannot have UTF8 characters

I have some funky token identifiers in my language. (e.g. the 'ł', '€' and 'ð' characters), but this results in syntax errors. I assume this is because token identifiers only support ASCII characters.

It looks like this functionality is determined by the isvarchar function in the shunting-yard.h file. I'll see if I can fix this myself.

This is not a perfect patch by any means, but this fixed my issues. I'll do a fork and pull-request with a prettier and more general patch (maybe tomorrow), so that variable names can be UTF8 characters.
static unsigned char unicodeCharCounter = 0;

...

  static inline unsigned char isvarUTF8char(const char c) {
      if(c & 0b10000000) {
          // This is the start of a unicode character!
          if(c & 0b01000000) unicodeCharCounter++;
          else return unicodeCharCounter+1;
          if(c & 0b00100000) unicodeCharCounter++;
          else return unicodeCharCounter+1;
          if(c & 0b00010000) unicodeCharCounter++;
          else return unicodeCharCounter+1;
      }
      if(unicodeCharCounter > 0) return --unicodeCharCounter;
      else return 0;
  }

  static inline std::string parseVar(const char* expr, const char** rest = 0) {
    std::stringstream ss;
    do {
        ss << *expr;
        ++expr;
    } while (isvarUTF8char(*expr) || rpnBuilder::isvarchar(*expr) || isdigit(*expr));
    if (rest) *rest = expr;
    return ss.str();
  }

That's cool, thanks @sillydan1. I am looking forward to seeing your PR.

Opened the pull request now. Closing this issue.


stable/airflow: add initcontainer initdb option

Add option to run airflow initdb as an initcontainer before git-clone.
Signed-off-by: Nathan J. Mehl<EMAIL_ADDRESS>Is this a new chart
No
What this PR does / why we need it:

Add an airflow.preinitdb option: if true, the initdb step
will be run as an initcontainer rather than inline in the
scheduler startup; this can work around dependencies in custom
db hooks that prevent initdb from running.

fix a bug wherein if the git-clone initcontainer was not activated,
but the git-sync sidecar container was, the web and scheduler
deployments would fail due to not correctly creating the
git-clone-secret volume, which is needed by both the initcontainer
and the sidecar.

Fix a similar bug for dag path mounting.

Fix another cosmetic data type mismatch bug in values.yaml:
airflow.pools should be a string containing a JSON
object, not a yaml map.

Which issue this PR fixes
n/a
Special notes for your reviewer:
@gsemet hi! :)
Checklist

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

@gsemet poke? :)

@gsemet hope you're surviving the pandemic well.  I'm hoping to resume forward progress on our airflow work; did you have an opinion about this change?

/ok-to-test

/retest
/lgtm

I am fine thanks ! I think this PR looks ok, let's try to merge it ! I am not sure when preinitdb script might be needed but if you need it, others might want it as well :)

/test pull-charts-e2e

Just curious, is there a reason this chart uses initdb everywhere instead of upgradedb? AFAICT the only difference is the recreation of all the default connections. I'll be happy to submit a pull request to allow users of this chart to have a choice between the two, I was just curious if there was something I was missing re upgradedb?

Hi. Actually I never used it, I don’t remember seeing this.
It should just work for new db and also for existing one, so if you have a PR I’ll welcome it !


Production bug captured with Sentry

Hi,
I've captured a bug in Sentry that I believe to have been thrown by redux-saga, by file redux-saga/es/internal/proc.js, at line 466:53. The version of redux-saga I'm using is 0.16.0. Here is a screenshot of the issue:

I've made the issue public, here is a link to it:
https://sentry.io/share/issue/0230d201a3404b01b6c443623a6ed664/
I will be glad to help you guys with any additional information you need

How many times this has happened to your app? Which useragents have been associated with this?

It has happened 1 time with 1 user. He were using Google Chrome v. 70.0.3538 on Mac OS X
v. 10.13.6

That's really strange - might be some obscure v8 engine bug :s If the source map is correct then it means that most likely it has thrown on "missing" isEnd which is statically available, its never redefined and its always a function.
I have a strong suspicion that you are using webpack older than v4 and if you are using webpack@3 then you are not using ModuleConcatenationPlugin in your production build. Could you confirm this guess?

Yes, I'm not using the ModuleConcatenationPlugin, but I'm indeed using WebPack 4 (v. 4.16.2)

Do you disable it somehow? (it should be used by default in production mode in webpack@4)

Oh, I've searched for "ModuleConcatenationPlugin" in my production webpack config file and didn't find anything. But if this is enabled by default, then I'm probably using it. I've uploaded my config file, let me know if it can help you:
https://gist.github.com/TheoneLucas/e24327741064d0408a419fc68691225c

I checked the documentation for onError function and I will send a bugfix implementing it on my application now, making it send the error to Sentry and as soon as an error is captured with it I will come back here with all the information it gathered.

I just noted that my application already have an onError handler on createSagaMiddleware function, and it was on production at the time this bug happened.

was the bug reported to sentry through that onError handler?

I don't think so, as the onError function handler don't appear on the error stack.

I would classify this as impossible then - we won't be able to act on it anyhow as it seems it just can't happen and that sentry might be reporting some false positives on this one.

Ok, if this error happen again I will make sure to get back on here with more details.

Thanks! For now I'm not going to close this issue, but let us know if you find more about it.


[bump] build-tools: 0.27.0 => 0.28.0 (minor)

Bumped build-tools from 0.27.0 to 0.28.0.

Thanks for the review, @alexvy86!


Limit size of backup directory

First of all, thanks for this useful program!
I often tag my photos just before importing them into LR and I like how the originals are backed up.
The thing is, that directory grows over time while tagging more and more photos and of course I can empty it manually each time (assuming that I remember to do so). I think however, that it would be nice if it could be pruned automatically based on file age and/or directory size.

Nice idea.   Thank you.  Let me think on how I might do that.  I'm inclined to not delete the backups, but instead put up a warning.  Or something like that.

I can understand why you're hesitant to delete backups, and it should definitely only happen in a way users expect.
Perhaps it is enough to disable such functionality by default so it only happens after enabling it in the preferences (this also ensures 'backwards compatible' behaviour).
Or, if you feel that is not safe enough, the operation could be put behind a dialog (or menu item, or something else), forcing the user to actively decide to clean up the backups each time.
For my workflow, auto-cleaning based on file age would probably work best, as I only use the backups to bridge the time between tagging and importing. As soon as I see them in LR I know everything is in order and I don't need those backups anymore.

I added a reminder to clean your backup/save folder.  The reminder is triggered if the backup/save folder contains more than 500 MB or if any file was added to the folder more than 7 days prior.  I know it's not what you really asked for, but it is what I'm comfortable doing right now.

I changed my mind.  I will warn if any files were placed in the the backup folder more than 7 days ago with an option to delete the older files.  Screen is similar to that shown above but contains two buttons, Close and Delete.

Nice, thank you!
I was already happy with the notification you implemented initially, but this addition makes it even more useful.


Join bug

Angus, when you look at the following picture, would you have an idea what might have went wrong during joining? The upper left join looks like 2 points off. Unrelated, I have a hunch line 853 should be _vertexList.Add(curr_v); and not _vertexList.Add(v0); ?

Hi Fribur.
Something certainly seems odd but I'll need some data to sort out what's happening.
You'll find a link to my email address here.

Attached the three integer polygons leading to the result in picture above. Interestingly this only occurs when I add them as PathD. Using Path64 leads to the expected result. Puzzling.
var jobClipper2 = new Clipper2Lib.ClipperD();
...
jobClipper2.Execute(ClipType.Union, FillRule.EvenOdd, result);

Outline_0.txt
Outline_1.txt
Outline_2.txt

OK, problem found.
private static double DistanceSqr(Point64 pt1, Point64 pt2)
{
  return (double) (pt1.X - pt2.X) * (double) (pt1.X - pt2.X) +
         (double) (pt1.Y - pt2.Y) * (double) (pt1.Y - pt2.Y);
}

The C# compiler indicated that my explicit casting to double in Clipper.Engine.DistanceSqr() was unnecessary (by fading out these casts). Unfortunately I removed them without testing the consequences and it turns out they are very much needed. Now I'm wondering what other casts I've removed that the compiler indicated were superfluous.

I assume the math runs on integer (with rounding errors) and converts to float at last step (the result is wrong at that point).

and converts to float at last step (the result is wrong at that point)

Yep. Anyhow, fixed now.


Toolbox scrollbar disappears on ChromeOS

Describe the bug
On ChromeOS, when the toolbox has more categories than can fit on the screen, the scrollbar doesn't show up so it looks like the toolbox can't scroll (so categories seem to be missing).
To Reproduce
I'm looking at the Planets card on the skill map where I have to use arrays, so I scrolled down, but later when I had to get at Sprites, I couldn't find it.
It shows up if you put the mouse over the scrollbar area, but it doesn't if you put it anywhere else in the toolbar

Desktop (please complete the following information):

OS: ChromeOS
Browser Chrome86

I believe this is just the browser styling.


Packer ProxMox looping

Please help.
For some reason looking at the console of Proxmox for this building VM; It keeps looping at subiquity/Network/_send_update: CHANGE ens18
Here is the output of the packer build command:
==> ubuntu-server-jammy.proxmox-iso.ubuntu-server-jammy: Waiting 5s for boot
==> ubuntu-server-jammy.proxmox-iso.ubuntu-server-jammy: Typing the boot command
==> ubuntu-server-jammy.proxmox-iso.ubuntu-server-jammy: Waiting for SSH to become available...
==> ubuntu-server-jammy.proxmox-iso.ubuntu-server-jammy: Timeout waiting for SSH.
==> ubuntu-server-jammy.proxmox-iso.ubuntu-server-jammy: Stopping VM
==> ubuntu-server-jammy.proxmox-iso.ubuntu-server-jammy: Deleting VM
Build 'ubuntu-server-jammy.proxmox-iso.ubuntu-server-jammy' errored after 20 minutes 22 seconds: Timeout waiting for SSH.
==> Wait completed after 20 minutes 22 seconds
==> Some builds didn't complete successfully and had errors:
--> ubuntu-server-jammy.proxmox-iso.ubuntu-server-jammy: Timeout waiting for SSH.
==> Builds finished but no artifacts were created.

This same problem is happening to me. Did you ever find a solution?

Hi Chrispro-21, Looks like ChristianLempa close it with not planned, which to me is no solution, so the code is junk now for me...
I use cloudInit now by following https://github.com/UntouchedWagons/Ubuntu-CloudInit-Docs

This is probably a bit late but for any1 getting these errors. it becausze I was attempting to use ssh username and password. Use ssh keys and it should work fine.


Licensing confusion

Hi Asciidoctor folk,
Looking at this file - https://github.com/asciidoctor/asciidoctor-pdf/blob/main/lib/asciidoctor/pdf/ext/prawn/coderay_encoder.rb - the licensing is GPL-3.0, which seems out of sync with the MIT licensing of Asciidoctor-pdf as a whole. (Earliest GitHub history here: https://github.com/asciidoctor/asciidoctor-pdf/commit/326e586c77c8c13b29df5339679133d819acd9cf#diff-4708f6bc776d6777ed49242a50fcf745256c2bc1c8757a49f5e62217d994deb2 )
It looks like it might have originally been this file https://github.com/prawnpdf/prawn-manual_builder/blob/1fdbb0beaa1b9d1ddf403640d70907d19c652d60/lib/prawn/manual_builder/syntax_highlight.rb - which seems to be under the Ruby tri-license of Matz's terms, GPL-2.0 OR GPL-3.0 ( https://github.com/prawnpdf/prawn-manual_builder/blob/master/COPYING ), as with the rest of the Prawn code included in the build.
Do you have any detail on the licensing situation here?

That file is licensed differently. That does not affect the licensing of Asciidoctor PDF itself, which is MIT.

This is covered in the NOTICE file. https://github.com/asciidoctor/asciidoctor-pdf/blob/main/NOTICE.adoc
If anything is missing from the NOTICE file, please point it out and I'll update it.

Thank you Dan; apologies for missing the mention in the NOTICE.adoc file; I was digging into the Maven jar and failed to subsequently notice it on the GitHub source.
Just to be transparent; I work in the AWS OSPO and the file auto-flagged as an oddity given the jar's overall licensing. It's the dependency of a dependency of a dependency of a ... (you get the picture; usual Maven story :) ). Coincidentally, I'm a big fan of asciidoctor on the personal side, using it to generate clean html and pdf documents for gaming rules in a community I'm a part of; so it's kinda fun to have it show up at work :)
Putting myself in the shoes of a project owner, the encoder file would concern me; or wearing my inactive hat as an Apache legal committee member, if that feels less corporately self-interested than a company OSPO :) It's all wrapped up in the jar and gem files, making the copylefting unclear, and it feels like an odd inclusion license-wise given it's tying together asciidoctor-pdf (MIT), coderay (MIT), and Prawn (Ruby-licensed).
[If relicensing was a topic of interest, the history here looks like it might be fairly simple even if Felipe is unavailable. The earliest version in GitHub is effectively the same as the current version in Prawn with the addition of some more colors (presumably the tokens are defined by asciidoctor). If no one but you has touched the file since, I think you could recreate the file with no licensing issue under Ruby-license.]
If this feels too pushy, let me know; I'm not intending to be a demanding pain in the derriere.

Thanks for providing more context. I understand it's a challenging situation to analyze.

I think you could recreate the file with no licensing issue under Ruby-license

Actually, I don't have to recreate the file. I just need to clarify that it's triple-licensed, since it always has been. The code came from Prawn, so it shares the Prawn license. I just didn't state that fully. I'll go ahead and do that.

Awesome - thank you :)

Thank you! It's really nice to have feedback from someone who is looking at things this closely.

FYI for any future watchers: Out of this chat this commit: https://github.com/asciidoctor/asciidoctor-pdf/commit/d3d96cf38c46db898cb7958001c393e42c5e11e1 seems to have been added.

Thanks for pointing that out. I forgot to mention that I had made the update. I'll also apply it to the v1.5.x branch for completeness.


Tag before switching to wagtail 2

Currently, longclaw works only with wagtail 1.x and commit 7458c39  pinned it to the requirements.
Before proceeding to update to wagtail 2,  current state needs tagging (and probably releasing as 0.3.0) to preserve all the 1.x functionality correctly

0.2.1 released


[Pumpkin brown] The name of the product is written in upper case

Steps to reproduce:

Go to Horondi Home Page https://horondi-front-staging.azurewebsites.net/
Go to 'Our Products' section
Click on 'Backpacks' link
Pay attention to 'PUMPKIN BROWN' model

Actual result: The name of the product is written in upper case (screenshot added).
Expected result: The product name is 'Pumpkin brown'.
Environment: Windows 10, 21H2 ; Google Chrome 100.0.4896.88 (64-bit).


Can TQVaultAE run on Windows 8?

I can't get TQVault to work because it asks me to install a .net framework 4.7.2 precisely.
and I haven't found a single way to install it on windows 8. is it even possible to run it on win8?

That's a good question, as we had quite a rework of the dependencies lately. @hguy might be better suited to answer this.
If this is a blocking issue for you, you could try using an older version of the tool in the meantime. There's a chance it might work.

Sorry, how does it work, will he probably come cause you tagged him or do I need to write him personally somehow? I don't use github frequently

it should be compatible.

https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/versions-and-dependencies#net-framework-472

Apply latest windows updates. Upgrade to Windows 8.1.
Try installing using Offline installer
https://dotnet.microsoft.com/download/dotnet-framework/thank-you/net472-offline-installer

Thank you hguy.
I see, so there's no way to do it without upgrading to 8.1
That's the point I chose not to update to 8.1 for years for several reasons and finally I find that one reason haha. Maybe I will, eh. It's a shame that I can't use TQ Vault on 8.0


[FEATURE REQUEST] Add Theme(light and dark) in the navbar

Description
Is your feature request related to a problem? Please describe.
The website currently lacks a theme toggle functionality, limiting user customization options and accessibility in different lighting environments.
Describe the solution you'd like
I would like to see a theme toggle added to the website, allowing users to switch between a light and dark theme according to their preference. This will provide a more personalized and comfortable browsing experience for users
Describe alternatives you've considered
As an alternative, I have considered using browser extensions or third-party tools to enable a theme toggle, but having this functionality built directly into the website would be more seamless and convenient for user
Additional Information
Context
Provide any additional context or information that helps explain the feature request.
Currently, the website only offers a light theme by default. However, many users prefer dark themes for improved readability in low-light conditions or personal preference. By adding a theme toggle in the navbar, the website can cater to a wider range of user preferences
Priority
Medium
Additional Notes
@Abhirup-02,please assign this issue to me under gssoc.

@Abhirup-02 ,please asssign this to me under gssoc.I will implement it in no time


Archive WG Budget - Quarter 2 2024

This Issue is for creating, agreeing and submitting the WG's budget for Q2 2024.
A final draft will be presented at the Archives meeting on Weds 17th April - dependent on discussions with Ubio about his Q2 work, and on any ideas from WG members about work they would like to do in Q2.

Proposed budget for Q2
https://docs.google.com/spreadsheets/d/1d9j-IPA0wary0EdR8YIoKl1BF0mAtkV_dgTs0hSnzAs/edit?usp=sharing

Q2 budget submitted here https://docs.google.com/spreadsheets/d/1BBogj9rAO52cpdGP3uvp8hAHNa4Qw66lz9JLjSC2yVs/edit?usp=sharing

Budget approved at last Treasury Guild meeting

We will keep this issue open until all the tasks in Q2 budget are closed

All Q2-related issues now closed, except this one https://github.com/SingularityNET-Archive/SingularityNET-Archive/issues/133 (tag taxonomy) where discussions are ongoing in the light of the summary tool redesign, so closing this issue


Travis build broken in MojoIntegrationEnvironmentsSpec

[main] ERROR org.apache.maven.cli.MavenCli - Failed to execute goal org.javalite:db-migrator-maven-plugin:2.3-SNAPSHOT:drop (default-cli) on project db-migrator-maven-plugin-test-environments: Execution default-cli of goal org.javalite:db-migrator-maven-plugin:2.3-SNAPSHOT:drop failed: Plugin org.javalite:db-migrator-maven-plugin:2.3-SNAPSHOT or one of its dependencies could not be resolved: Failed to collect dependencies at org.javalite:db-migrator-maven-plugin:jar:2.3-SNAPSHOT -> org.apache.maven:maven-plugin-api:jar:3.5.0: Failed to read artifact descriptor for org.apache.maven:maven-plugin-api:jar:3.5.0: Could not transfer artifact org.apache.maven:maven-parent:pom:27 from/to central (https://repo.maven.apache.org/maven2): No connector available to access repository central (https://repo.maven.apache.org/maven2) of type default using the available factories -> [Help 1]

@jfcabral  the build has been broken on Travis for some time, and since I cannot log into the box, it is hard to debug. Do you have any  idea?

we turned on debug logging https://travis-ci.org/javalite/activejdbc/jobs/541480875

Don't worry @ipolevoy, I'll check it out over the weekend!

@jfcabral thanks again!

Fixed in https://github.com/javalite/activejdbc/issues/863


How can I get the finetuned model?

Thank you for your inspiring work and open-source codes!
But I have a question that how can I get the finetuned model before unlearning? It seems that there is only scripts for forgetting in this repo.
Thank you for your time.

Hi, Tianle.
Thank you very much for your interest in our work!
Actually, we don't have any operations for the vanilla model (LLaMA and OLMo) before unlearning, as our paper describes. We do the unlearning on the vanilla models directly, trying to erase the knowledge that the models naturally possess after pretraining. We believe that the knowledge added through finetuning belongs to artificial knowledge (which should be more easier to unlearn), not the knowledge inherently possessed by the model itself.  This is a significant difference between our work and previous unlearning works.
We believe that the topic of LLM Unlearning should be more closely aligned with unlearning the knowledge inherently possessed by the pretrained model itself.
If you have any other questions, or if you encounter any issues while executing the code, please feel free to let me know : )

Thank you for your clear and in time explanation!


Insonsistency between README.md and actual implementation

Current README.md says like this.

main.go - defines main function. It includes application name, version, usage, author name and so on.

But actual implementation generates APPNAME.go.
Which is correct?
IMHO, I prefer main.go.

Oops.
I'll resolve this conflict and re-push.

Sorry for late, new version of cli-init uses main.go.
Thanks


Fix problem of status never set to failed

The issue is status is never set to failed after retries complete with failure.
https://github.com/utgarda/sidekiq-status/issues/125
Solved it by starting the try count from '1' not '0'
Notes:
The reason why CI failed for sidekiq 4 or lower version:
in or lower than sidekiq 4.xx: the retry_count in msg starts from 0 while retry_count starts from nil in sidekiq 5.xx, accordingly, try_count in sidekiq 4.xx starts from 1 while 0 in 5.xx
Added the version checking, CI still cannot pass for lower versions. (It passed on my local machine, not sure it failed on CI)

any progress on this? We would like to remove our monkey patch

I've taken another attempt at this, with a hopefully passing build, in #134. We'll be using this branch in production.


Implemented 'unstable options' command line option

Hello,
This is my first contribution to any open source project, so any advice would be awesome and thank you for being nice :)

[x] add the flag itself
[x] ensure the flag can be set programmatically when rustfmt is used as a library
[ ] decide what should be unstable (this can be ongoing, but needs to be done by the time we use Rustup)
[x] for each unstable feature, ensure that you need the --unstable-features flag. This includes any unstable configuration options (we would need some mechanism for marking options as stable or unstable)
[x] only allow the flag to be set if we are run with nightly *
I am checking CFG_RELEASE_CHANNEL environment option to see if we are running on nightly version, I am not confident about this solution. There's must be a better way that I've missed.

All the tests are passing, I've added 2 tests to verify if my changes are working, which are also passing.
Thank you

Great, thank you for the PR! Sorry it took me a while to get to it, I'm mostly on parental leave right now and only checking notifications intermittently.

It would be good if you could squash your commits (I find git rebase -i the easiest way to do this), thanks!

Hello, Sorry about the delayed reply, I was out of town, I've gone through your review and updating my changes accordingly.

Sorry, about the extra commits again.

Most of the changes look good, thanks, but you've removed the 'stable-ness' bool for each option, which I think was a good idea, sorry for the miscommunication there. We should also check when an option is set that if it is unstable, then the unstable features option is set.

No problems, I can understand the awesome feelings and the sleepless nights your going through as a parent :)
I think I got most of it correctly this time.

@tmahmood hi, sorry it's taken so long to get back to this. I'm back at work now and should be much quicker to respond. The changes to the unstable features flag looks good. However, it looks like you've pulled in some other commits here (changes to Cargo.*, some other changes to config.rs, etc.). I expect this was a rebasing issue. You can probably use git reflog to get back to a state before the rebase. If you need some help with rebasing or other Git stuff, let me know.

Good to hear from you.
I've fixed commit issues. Thank you for the help

Awesome, thanks!

@tmahmood if you'd like more rustfmt stuff to work on, then a good follow-up would be to extend the unstable-features flag so that it can apply to command line options and write modes. Each option and write mode could be marked stable or unstable and if unstable, then can only be used if the unstable features flag is set.

Awesome! I'll love to continue working. :)


center the dialog and add var to allow setting a top margin

i keep the section but change the header to div