satisfies("MIT", "MIT AND Apache-2.0) passes, but is expected to fail

Description
For function satisfies(first, second), my reading of the parameters is that the check is determining if the first expression satisfies the second.  If this is correct, then I expect...
satisfies("MIT", "MIT AND Apache-2.0")
    EXPECT: false
    ACTUAL: true

I expect it to return false since MIT AND Apache-2.0 requires both licenses to be present.
Similar results:
satisfies("MIT AND ICU", "(MIT AND GPL-2.0) OR (ISC AND (Apache-2.0 OR ICU))")
    EXPECT: false
    ACTUAL: true

I expect it to return false since MIT AND ICU does not satisfy any conditions in the second expression.
Analysis
function satisfies (first, second) {
  var one = expand(normalizeGPLIdentifiers(parse(first)))
  var two = flatten(normalizeGPLIdentifiers(parse(second)))
  return one.some(function (o) { return isANDCompatible(o, two) })
}

*** Entering satisfies(first:  MIT , second:  MIT AND Apache-2.0
firstExpanded:  [ [ { license: 'MIT' } ] ]
secondFlattened:  [ { license: 'Apache-2.0' }, { license: 'MIT' } ]
EXPECT:  false , ACTUAL: true

With the second expression flattened, it appears to act as Apache-2.0 OR MIT and MIT does satisfy that expression resulting in a return value of true.
Similar result:
*** Entering satisfies(first:  MIT AND ICU , second:  (MIT AND GPL-2.0) OR (ISC AND (Apache-2.0 OR ICU))
firstExpanded:  [ [ { license: 'ICU' }, { license: 'MIT' } ] ]
secondFlattened:  [
  { license: 'Apache-2.0' },
  { license: 'GPL-2.0' },
  { license: 'ICU' },
  { license: 'ISC' },
  { license: 'MIT' }
]
EXPECT:  false , ACTUAL: true

Since ICU and MIT are both in the flattened list, satisfies returns true.  But MIT AND ICU do not match any of the conditions in the second expression, so I would expect false.
Exploration
I tried flipping the processing such that the second expression is expanded and the first is flattened.  With that, satisfies("MIT", "MIT AND Apache-2.0") returns false as expected.  This leads to new failures in the set of assertions in the readme.
Example:
assert(!satisfies('(MIT AND GPL-2.0)', '(ISC OR GPL-2.0)'))

The assertion is written to expect satisfies to return false, but I would expect this to return true since GPL-2.0 in the first expression satisfies ISC OR GPL-2.0.
This leads me to wonder if I am misinterpreting the meaning of the two parameters in the context of which parameter is the test and which is the expression to be satisfied.

@kemitchell in the first example:
satisfies("MIT", "MIT AND Apache-2.0")
    EXPECT: false
    ACTUAL: true

I think it's confusing to use AND to express a list of acceptable policies instead of OR, given the definitions of those in the spec. satisfies("MIT", "MIT OR Apache-2.0") would seem to be a more natural way to express that.
At the very least, if the order of the arguments is relevant then the argument names themselves shouldn't be first and second. Maybe something more like artifact and acceptable or test and policy.

@kemitchell Would you be open to re-evaluating the behavior described in the title?
While trying to address this feature request we started using spdx-satisfies and quickly ran into the same issue (satisfies("MIT", "MIT AND Apache-2.0") returns true).
This behavior contradicts our expectations given the spec descriptions (linked by @dangoor above).

@kemitchell, I think the error might be in the flatten function. Could you please clarify what it is supposed to do?
For example, when for this check satisfies ("MIT" , "(MIT OR ISC) AND GPL-3.0"), flatten is called with the expression param set to:
{
  left: {
    left: { license: 'MIT' },
    conjunction: 'or',
    right: { license: 'ISC' }
  },
  conjunction: 'and',
  right: { license: 'GPL-3.0' }
}

expanded is evaluated to be
[
  { MIT: { license: 'MIT' }, 'GPL-3.0': { license: 'GPL-3.0' } },
  { ISC: { license: 'ISC' }, 'GPL-3.0': { license: 'GPL-3.0' } }
]

which to me seems to be making sense so I think the issue is with this reduce since the key GPL-3.0 is duplicated in both array elements, it is essentially overridden so the output is
{
  MIT: { license: 'MIT' },
  'GPL-3.0': { license: 'GPL-3.0' },
  ISC: { license: 'ISC' }
}

I would have expected it though to be something like
{
  'MIT and GPL-3.0': { license: 'MIT and GPL-3.0' },
  'GPL-3.0 and ISC': { license: 'GPL-3.0 and ISC' }
}

Could you please clarify if my understanding is correct? I can create a PR for this

However, I dont know what the use of isANDCompatible then will be

👋 Hi @kemitchell, just a bump on @cnagadya's question for clarification -- we're happy to submit a PR to resolve this but want to make sure we're doing it right 🙏

I'd be happy to see a PR, but my personal view on this package at the moment is that use of the SPDX expression syntax for the second argument was very likely a mistake.
A very small number of users likely want to allow specific licenses with exceptions, but mostly what people want is to call satisfies(expression, allowList), where allowList is a simple list (array) of license identifiers.

See #17.
The decision to take two SPDX expressions as arguments has caused far too much confusion.

See #17.
The decision to take two SPDX expressions as arguments has caused far too much confusion.

Thanks for your fast reply.
I have a PR ready in case you'd like to keep the argument comparison working with 2 strings, which fits my use case. Actually, I believe both approaches (second as a string with a SPDX expression or array of allowed licenses) may be compatible.

@kemitchell Please, see https://github.com/jslicense/spdx-satisfies.js/pull/18. Thanks in advance.

What's the use case for expression-versus-expression, specifically?

What's the use case for expression-versus-expression, specifically?

Sorry, but isn't that the original purpose of this package? I mean, judging by the examples on the README I would assume it allows comparing expressions. For instance:
satisfies('MIT AND ISC', '(MIT OR GPL-2.0) AND ISC')

In our case, we use it to compare the licenses of every package on the package.json versus any arbitrary expression the user provides as input. second as an array of simple licenses won't work. Switching arguments won't work either as there may be packages with license1 AND license2 for example.

I believe the original issue here has now been avoided. Per #17, we're no longer accepting satisfies(expression, expression) but instead satisfies(expression, array). For most users, trying to explain what AND and OR meant in the second argument caused endless, justified confusion.

Sorry again, but I don't understand why AND and OR would be a problem in this context. Aren't those operators supposed to work like any other logical operator in a logical expression? AND meaning both, and OR meaning one of.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_0fhzaiqd2m/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_0fhzaiqd2m/clone/recipe/meta.yaml:19: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_0fhzaiqd2m/clone/recipe/meta.yaml:26: python_build_tool_in_run: The python build tool setuptools is in run depends
ERROR: /tmp/abs_0fhzaiqd2m/clone/recipe/meta.yaml:50: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_0fhzaiqd2m/clone/recipe/meta.yaml:50: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_0fhzaiqd2m/clone/recipe/meta.yaml:50: missing_description: The recipe is missing a description
Errors were found


Add api documentation

This PR adds a separate API documentation site that should give an overview of the currently offered API calls. For easier testing and access an OpenAPI specification file (openapi.yml) is being provided that can be used with your favorite tools to call the API.
Fixes #40

That's great! Thank you, Andre.


Getting error code 090 on new microbit

I am using SPI functions for writing to SD card with new microbit and getting error code 090 on microbit screen every time after writing two lines to SD card as if its is periodic, writes two lines shows error code 090 and restarts again.

Hi @KalbeAbbas
An 090 panic code is a hardware configuration error. You can see these list of panic codes here:
https://github.com/lancaster-university/codal-core/blob/master/inc/core/ErrorNo.h#L101
Looking at the SPI driver, it looks like you are trying to create the SPI driver multiple times:
https://github.com/lancaster-university/codal-nrf52/blob/master/source/NRF52SPI.cpp#L55
Could you perhaps share some of your code so we can see how this might be happening?

Hi @finneyj, Thanks.
I first define function for allocating SPI pins:
static SPI* allocSPI() {
	SPI* spi = NULL;
        spi = new SPI(MOSI, MISO, SCK);
        return spi;
}

static SPI* p = NULL;

Then I allocate pins, set frequency and mode:
  CS_INIT();  // Chip Select Init
  CS_H();  // Chip Select High
  p = allocSPI();
  p->frequency(10000);
  p->format(8, 0);

and these are functions to write and read to/from SD card, they are called every time when writing:
static void xmit_mmc(
    const BYTE *buff,
    UINT bc)
{
  BYTE d;

  do
  {
    d = *buff++;
	p->write((int)d);
  } while (--bc);
}

static void rcvr_mmc(
    BYTE *buff,
    UINT bc)
{
  BYTE r;

  do
  {
    *buff++ = p->write(0xFF);
  } while (--bc);
}

These are low-level functions, all other function call these functions at lowest-level. Let me know if you want any other info

Forgot to mention, I call allocSPI() function to allocate SPI only once at the beginning.

That's curious.
I'm afraid I can't see any other way that panic code can be generated apart from calling the NRF52SPI constructor multiple times... All the other causes are raised from other modules:
libraries/codal-nrf52/source/NRF52I2C.cpp:        target_panic(DEVICE_HARDWARE_CONFIGURATION_ERROR);
libraries/codal-nrf52/source/NRF52Serial.cpp:        target_panic(DEVICE_HARDWARE_CONFIGURATION_ERROR);
libraries/codal-nrf52/source/NRF52SPI.cpp:        target_panic(DEVICE_HARDWARE_CONFIGURATION_ERROR);
libraries/codal-nrf52/source/peripheral_alloc.cpp:    target_panic(DEVICE_HARDWARE_CONFIGURATION_ERROR);
libraries/codal-nrf52/source/ZSingleWireSerial.cpp:        target_panic(DEVICE_HARDWARE_CONFIGURATION_ERROR);

Are you really, really sure there's no way the your initialization code is somehow being called twice? Maybe try putting some debug statement in there to be sure?
;)

Good idea. I'll try that :)

I guess you are right, its called each new cycle. Let me confirm if the same error code comes again.

@finneyj Thank you. That solved the issue :)

I am closing this issue now

Great! Thanks for reporting back @KalbeAbbas
If it make you feel better I'm teaching an Computer Science class as we speak.... and you fixed your code faster than my students did. ;)


app not launching on mac osx yosemite

kivy is not launching on my mac book air yosemite,
logs says:
Kivy[2267]: App did finish launching
but no gui,
and also python interpreter is opening fine on terminal.
Yes i am using the latest build, downloaded from the download page.
Thanks

Launching Kivy.app is supposed to do nothing.
To run your app you have to either run it from the terminal using kivy main.py or drop your main.py onto the kivy icon from /Applications.


Adding experimental command to list validators

Ref #188
Command:
near EXPERIMENTAL_validators

Current output:
.------------------------------------------------------------------------------------.
|     Validator Id      |  Stake  | # seats | % online | bls produced | bls expected |
|-----------------------|---------|---------|----------|--------------|--------------|
| node1                 | 254,026 | 7       |      100 |          198 |          198 |
| node3                 | 252,838 | 7       |      100 |          200 |          200 |
| node0                 | 250,148 | 7       |      100 |          201 |          201 |
| node2                 | 209,797 | 6       |      100 |          172 |          172 |
| hashquark             | 158,378 | 5       |       94 |          108 |          114 |
| nuc2.test             | 129,465 | 4       |       93 |           81 |           87 |
| unknown.test          | 125,853 | 4       |      100 |           85 |           85 |
| bowen.test            | 104,138 | 3       |      100 |           85 |           85 |
| dochpryof.test        | 103,993 | 3       |      100 |           85 |           85 |
| ractolechoc5.test     | 103,985 | 3       |       97 |           85 |           87 |
| olaiolsen.test        | 103,572 | 3       |      100 |           85 |           85 |
| melea.test            | 103,450 | 3       |      100 |           86 |           86 |
| staked.test           | 103,432 | 3       |      100 |           87 |           87 |
| janliamnilsson.test   | 103,405 | 3       |      100 |           85 |           85 |
| felixschulz.test      | 103,324 | 3       |       98 |           85 |           86 |
| launooskuarttu.test   | 103,318 | 3       |      100 |           56 |           56 |
| fredrikmalmqvist.test | 103,318 | 3       |      100 |           58 |           58 |
| tommywesley.test      | 103,317 | 3       |      100 |           58 |           58 |
| simonhugo.test        | 103,317 | 3       |      100 |           58 |           58 |
| villiamsivertsen.test | 103,315 | 3       |      100 |           57 |           57 |
| oligarr.test          | 102,866 | 3       |      100 |           57 |           57 |
| bitoven.test          | 102,538 | 3       |      100 |           58 |           58 |
| wetez11.test          | 102,502 | 3       |      100 |           58 |           58 |
| jaroslavrud.test      | 102,196 | 3       |       98 |           56 |           57 |
| anonstake.test        | 102,007 | 3       |      100 |           57 |           57 |
| jazza.test            | 101,998 | 3       |      100 |           57 |           57 |
| volvos60909.test      | 101,573 | 3       |      100 |           56 |           56 |
| buster.betanet        | 100,883 | 3       |      100 |           57 |           57 |
| harrypotter           | 100,392 | 3       |       96 |           56 |           58 |
| moonlet.test          | 100,146 | 3       |      100 |           56 |           56 |
| validatorsonline.test | 100,046 | 3       |      100 |           57 |           57 |
| nodeasy.test          | 100,000 | 3       |      100 |           57 |           57 |
| sparkpool.test        | 100,000 | 3       |        0 |            0 |           57 |
| nearkat               | 80,314  | 2       |      100 |           57 |           57 |
| awrelll.test          | 51,006  | 1       |      100 |           28 |           28 |
'------------------------------------------------------------------------------------'

Fishermen (total: 5):
.---------------------------.
|   Fisherman Id   | Stake  |
|------------------|--------|
| hashfield.test   | 400    |
| crypt            | 100    |
| cryptium.betanet | 20,000 |
| my_validator     | 100    |
| ubikcapital.test | 20,000 |
'---------------------------'

Next validators (total: 33, seat price: 34,312):
.-------------------------------------------------------------------.
|   Status   |       Validator       |       Stake        | # seats |
|------------|-----------------------|--------------------|---------|
| New        | stakingfund.test      | 100,000            | 3       |
| Rewarded   | node1                 | 254,026 -> 254,200 | 7       |
| Rewarded   | node3                 | 252,838 -> 253,011 | 7       |
| Rewarded   | node0                 | 250,148 -> 250,319 | 7       |
| Rewarded   | node2                 | 209,797 -> 209,940 | 6       |
| Rewarded   | hashquark             | 158,378 -> 158,486 | 5       |
| Rewarded   | nuc2.test             | 129,465 -> 129,553 | 4       |
| Rewarded   | unknown.test          | 125,853 -> 125,939 | 4       |
| Rewarded   | bowen.test            | 104,138 -> 104,210 | 3       |
| Rewarded   | dochpryof.test        | 103,993 -> 104,064 | 3       |
| Rewarded   | ractolechoc5.test     | 103,985 -> 104,057 | 3       |
| Rewarded   | olaiolsen.test        | 103,572 -> 103,643 | 3       |
| Rewarded   | melea.test            | 103,450 -> 103,521 | 3       |
| Rewarded   | staked.test           | 103,432 -> 103,503 | 3       |
| Rewarded   | janliamnilsson.test   | 103,405 -> 103,476 | 3       |
| Rewarded   | felixschulz.test      | 103,324 -> 103,395 | 3       |
| Rewarded   | launooskuarttu.test   | 103,318 -> 103,389 | 3       |
| Rewarded   | fredrikmalmqvist.test | 103,318 -> 103,389 | 3       |
| Rewarded   | villiamsivertsen.test | 103,315 -> 103,386 | 3       |
| Rewarded   | oligarr.test          | 102,866 -> 102,937 | 3       |
| Rewarded   | bitoven.test          | 102,538 -> 102,608 | 3       |
| Rewarded   | wetez11.test          | 102,502 -> 102,572 | 3       |
| Rewarded   | jaroslavrud.test      | 102,196 -> 102,266 | 3       |
| Rewarded   | anonstake.test        | 102,007 -> 102,077 | 3       |
| Rewarded   | jazza.test            | 101,998 -> 102,068 | 3       |
| Rewarded   | volvos60909.test      | 101,573 -> 101,643 | 3       |
| Rewarded   | buster.betanet        | 100,883 -> 100,952 | 3       |
| Rewarded   | harrypotter           | 100,392 -> 100,461 | 3       |
| Rewarded   | moonlet.test          | 100,146 -> 100,215 | 3       |
| Rewarded   | validatorsonline.test | 100,046 -> 100,115 | 3       |
| Rewarded   | nodeasy.test          | 100,000 -> 100,069 | 3       |
| Rewarded   | nearkat               | 80,314 -> 80,369   | 2       |
| Rewarded   | awrelll.test          | 51,006 -> 51,041   | 1       |
| Kicked out | tommywesley.test      | -                  | -       |
| Kicked out | simonhugo.test        | -                  | -       |
| Kicked out | sparkpool.test        | -                  | -       |
'-------------------------------------------------------------------'

There is still need to add another report for finish #188 that shows given current validators and proposals, what would epoch after next would possibly look like (e.g. run mock auction).
Leaving EXPERIMENTAL_ for now in the command to allow us to change command when Shell's NEP gets figured out.

WDYT about sorting the rows by total staked instead of by validator name?

@mfornet done.

Also @bowenwang1996 @frol interestingly, indeed  https://github.com/nearprotocol/nearcore/issues/2259 is not needed, but seat price is actually not everything. We also cut some validators from their seats, because usually with given threshold it will be over 100 seats filled. To reproduce this logic, we actually need to reproduce the random shuffle that our EpochManager does...

Super welcome feature! I feel limited by the structure of the CLI, mostly because it can only see "the now" and not in the past.
Example of needed features:

As a CLI user, I want to know the per-seat cost, such that I don't risk to stake a too small amount
As a CLI user, I want to know if my stake transaction was included in the proposals, such that I know that I will become validator
As a CLI user, I want to know if my "state" is [no stake, proposal, validator, future validator, future kicked out, past kicked out], such that I can plan my actions accordingly
As a CLI user, I want to know my "rewards" and performance from the last epoch, such that I can easily report my rewards

On the other hand, I'm not sure that displaying the whole Validator RPC endpoint would be helpful without some time-related queries (e.g., look at past blocks, to know my node performance over time). From this point of view, I'd rather build an application to capture the time series, trying to catch the anomalies.

To reproduce this logic, we actually need to reproduce the random shuffle that our EpochManager does... In this PR seat number is not always correct and doesn't add up to 100.

@ilblackdragon do we actually want to reproduce this logic? It seems very difficult. We not only need to implement rust's random number generator in javascript, but also find the random seed used there, which means we also need to reproduce the logic of epoch id computation. I suggest that we not do it.

@vgrichina Moving into https://github.com/near/near-api-js/pull/300
@bowenwang1996 Well, we need to make it reproducible for other clients to be able to reproduce... If we don't think our current random function is reproducible (and I looked into it - it's very complicated) - we should change it.

If we don't think our current random function is reproducible (and I looked into it - it's very complicated) - we should change it.

Agree. Probably some simple shuffling would suffice.


Update bamit to return vector of records.

@Irallia The goal is to have a function which returns a "seqan-like" vector of sam_record objects, so that I could do something like this:
auto records_vector = bamit::get_overlap_records(input, index, start, end)
uint32_t avg_read_depth = records_vector.size() / (end - start);

At the moment I was only returning the file position, so the user would have to parse the records themself. This just makes it a bit easier.


PDF Preview Environment

PDF Preview environment, addressing this jira card.
Will of course work with any template defined in the PDF-Generator itself.
Example with vulnerabilities:

Just realized there are some straggling files/changes from experimenting, going through and deleting them right now.


Remove unwanted closing tag

There's no opening "span" tag so we don't need the closing tag

Good spot and thanks for the contribution @nag!


Fix reloadOnChange for missing directory

Fixes https://github.com/aspnet/Configuration/issues/478
@divega @victorhurdugaci

3ea670863e44a8047b31f2552b4c4f9a6a2c236b


[POC] Use classy prisms for errors

This is a POC of using classy prisms to handle the embedding-errors problem.
Here's my assessment of it.
Pros:

No convertErrors weirdness.

Crucially, this means I'm pretty sure this will keep working as the projects grow.

Very explicit about what errors can be thrown.
Actual use sites are nice (look at Plugin.hs, which uses functions from both language-plutus-core and plutus-ir that throw different kinds of errors).

Cons:

Sometimes painfully explicit - we get quite big constraints for some of the language-plutus-core functions because they can throw lots of kinds of errors.
Error types are merely constrained, so if you use the error polymorphically you sometimes now need to specify what the error type is.

We need to do the constraint-based version rather than using concrete types because otherwise we lose all the benefit when we try to use the functions in other places.

Need to write a (trivial) instance for each kind of error you want to lift into your concrete error type.

I tried to write "chaining" instances like instance AsError e a => AsParseError e a, so that you would only have to say you can embed e.g. language-plutus-core's concrete error type and you would get the others for free, but this immediately gets stuck in a morass of overlapping and undecidable instances, and I gave up.

Some TH.

Neutral:

Lens dep. It seems like we were going to get this at some point anyway, and it lets us clear up some code that duplicates what it provides.
Still doesn't solve the problem of handling a specific type of error and having it fall out, but this seems to be even harder (nobody has a good solution to this AFAICT except maybe extensible-effects/freer-simple or something).

How about using ConstraintKinds to define synonyms for common combinations?

Yeah, I considered that. They're big, but don't appear that often, because there's only the one typechecking function etc. I think it's okay.

Or you could use class context wildcards, but I am not sure that I like them much

:see_no_evil:

I already did the thing where I hoist out subexpressions and give them type signatures, which isn't too much worse, so might as well leave it.

Okay, it seems like nobody hates this, so I propose we actually do it. Any more detailed critique welcome!

Here goes.


TS Control via Core

It would be beneficial to allow for the complete control of TS rooms via Core:

Addition of new permanent rooms
Modified of existing rooms
Addition of password to existing rooms
Creation of temporary (time limited?) rooms

This will mean that TeaMan always remains up to date.
Each room should have the following settings:

Boolean: Included in idle time checks
Boolean: Extended idle time permitted

This should, ideally, be a new module:

Also incorporate TeaMan into the module
Move the UI side into this module

Closed unimplemented


Spanish Scanner iPad App

ITS-Pro time category: https://its-pro.ucsd.edu/browse/MA-74
This is complete when:

[ ] The iPad scanner app supports the Spanish language

@lukedutta9 Can you please provide an ETA on the PR? We want to get this out to the Store this month.

@lukedutta9 Can you please provide an ETA on the PR? We want to get this out to the Store this month.

PR #41 requested for mobile qr scanner.

PR #41 requested for mobile qr scanner.


[vectors] update madd / maddN arg order

It occurred to me that the current arg order of the multiply-add functions is incompatible/flipped with the way that function exists in OpenCL. To avoid confusion, let's update the args and also all existing call sites (mainly in various geom packages)
For ref: https://man.opencl.org/mad.html

By extension, also update addm, addmN etc.


[FEATURE] #1217 Added feature to emulate gitlab hosted runner using the --container-emulate option with the name of one of the gitlab preset.

summary
Added an option to prefill memory, kernel-memory and  cpus docker options based on the configuration of Gitlab's hosted runner. This is useful when attempting to debug performance issue. By default, no limit is applied, the host resources are all available.
When passing the option --container-emulate= with the name of a gitlab hosted runner, the docker resources is limited to the resource available for this class of hosted runner. As of this PR, those are the values

name
memory
CPU

saas-linux-small-amd64
8 GB
2

saas-linux-medium-amd64
16 GB
4

saas-linux-large-amd64
32 GB
8

saas-linux-xlarge-amd64
64 GB
16

saas-linux-2xlarge-amd64
128 GB
32

see the gitlab documentation here
Of course, you are also limited to the host resources. You cannot have 128 GB of RAM on a host that only has 16 GB.
As of right now, I've only added support for the base linux machine. This does not include ARM based CPUs, or runner with a GPUs. This could be a nice amelioration in the futur.
Usage
You can pass an additional option --containter-emulate with one of the previously mentionned runner name. For example, using the name saas-linux-small-amd64 will limit the container resource to 8 GB or RAM and 2 CPUs.
gitlab-ci-local --cwd /path/to/.gitlab.yml --container-emulate=saas-linux-small-amd64

You can check the memory limit and CPU usage using the docker stats {container_id} command.

This PR Closes https://github.com/firecow/gitlab-ci-local/issues/1217

thanks for the contribution !

@nicDamours I don't see the purpose of the amd64 postfix.

@nicDamours I don't see the purpose of the amd64 postfix.

@firecow it's the name given by gitlab for those specific runner: https://docs.gitlab.com/ee/ci/runners/hosted_runners/linux.html#machine-types-available-for-linux---x86-64
It's not essential for this feature but it helps linking gitlab's runner to the preset defined here.

@nicDamours I don't see the purpose of the amd64 postfix.

@firecow it's the name given by gitlab for those specific runner: https://docs.gitlab.com/ee/ci/runners/hosted_runners/linux.html#machine-types-available-for-linux---x86-64
It's not essential for this feature but it helps linking gitlab's runner to the preset defined here.

I can imagine people with ARM processors will be very confused, when they realise that using such a preset, actually works :smile:

I can see how that would be confusing. I will remove the amd64 prefix :slightly_smiling_face:


Support multiple report/output files

It would be great if npm-groovy-lint could parse multiple -o parameters and produce multiple reports.
Rationale: for example, for us we need both XML and JSON, XML is for Jenkins CodeNarc plugin to actually pass it to Jenkins and JSON for our own custom parsing and better reporting.

Well, I do think the issue should be still open.

That would be hard to do with the current architecture, especially knowing that CodeNarc (which is called behind) does not manage multiple output files, and npm-groovy-lint does not generate XML output, it just transfers back codeNarc generated one
Meanwhile CodeNarc decide to manage multiple output (if they do), I don't have any other workaround than calling twice npm-groovy-lint with different output formats :/


Add compile instructions for node-6-rework

Would it be possible to add some instructions on how to compile and run the node-6-rework for testing?
So that people can start giving feedback without the risk of giving the wrong info because they didn't compile/run it correctly.
I also really want to try the latest version and see how it works out. :)

@LuaTenshi Works the same as master branch, other then you use node 6 instead.
nvm install 6
nvm use 6
npm i
npm run dev


Build failing?

Anyone know of an existing (open) issue that has details on the regression that broke IE10/Safari 6 (on OSX 10.8) ? I'm curious to help solve the failing build if possible but wanted to get some deep context before making any assumptions myself :)

I'm trying to gauge if this project is truly supported right now or if it's in maintenance mode for the most part. I do see a merge now for Aug but overall it seems like this project isn't being worked on. I've tried to contact @rtfeldman via email/twitter to confirm if he has plans to pass it on or if he is still maintaining it (no word yet)
Any feedback would be greatly appreciated -thank you in advance


Eliminate warnings in sphinx build

Lots of lines are spewed out in the doc build like /home/travis/build/dcolligan/schemas/doc/source/api/genotypephenotype.rst:340: WARNING: Title underline too short. ... it would be nice to get rid of these.

Fixed in #784


Support CheckMetadata signed extension

I have introduced a hacky way for it to hotfix our setup here: https://github.com/encointer/polkadart/tree/cl/add-checkmetadata-extension
Maybe I have some time soon to properly implement this and upstream, but I am not sure when.

Hello @clangenb , thank you for the heads up. May I ask is that a new Polkadot/Kusama signed extension? Or that's from another chain?

It is on Kusama since today, runtime releaae v1.2.5, and on all system chains. It will come to polkadot with the next runtime upgrade.


[PGE-178206904] Send Transaction ID to STX

source.tid is transaction id at the request level
bidReq.ortb2Imp may carry a transaction id at the impression level (imp.ext.tid)
While Prebid is still figuring out how they want to do it, this is following the same implementation as TheTradeDesk did in their own adapter so we know the transaction id is at least consistent with them (since they're the one asking for it in the first place). My understanding is that if ever the Prebid committee changes the spec, they will be the one updating all adapters like they did in the past for other changes.
Followed recommendations provided by contributors here: https://github.com/prebid/Prebid.js/issues/8573
PGE-178206904

Opened PR on official public repo: https://github.com/prebid/Prebid.js/pull/8988


Run cppcheck on a temporary file with a possibly unsaved document text

Here is the PR that hopefully fixes #14. Before the cppcheck is run, a temp file with the document contents is created, and the header search path with the original directory is prepended to mimic the normal behavior.
Unfortunately, I've had to move some things around to do it, so the diff is rather unnecessary noisy.

Wow, the plugin runs so much better with these changes!


Improve structured log exception experience

This fixes #1177

First, display preview of primary error when hovering over the error button as a tooltip
Second, change the error icon to a fluent button, as users would not know otherwise that this is an actionable element (see below for inline view with other log types)

Third, allows grid values to be specified as multiline. I'm assuming exception.stacktrace is a fairly safe bet to signify as multiline. We might also want to have a button to allow users to toggle between multiline and single-line view for properties.
Fourth, adds the log entry to the URI so that users can navigate directly to a structured log preview. I'm assuming TraceId + SpanId is a unique identifier for a log and SpanId is only unique within a trace, but would love for that to be confirmed.

Microsoft Reviewers: Open in CodeFlow

In that demo a first line of context.Message is shown in the tooltip. Is there a value in this when Message is already displayed right next to it? Or am I missing/misunderstanding something?
I think tooltip text can be either:

A combination of exception.type and exception.message
First line of exception.stacktrace

Per https://opentelemetry.io/docs/specs/semconv/exceptions/exceptions-spans/ those fields should be present in exception cases.

Error shouldn't be a button. It doesn't look good, and it is confusing about what the difference is between clicking on error and clicking on view details
Displaying the first line of Message isn't useful. It's already displayed in the grid. Hovering over the message column in the grid displays the complete message.
Associating the error status with there isn't an exception message is incorrect. Status and exception are independent of each other. A log entry with an error status doesn't necessarily have an exception: logger.LogError("I have no exception"). And a log entry with a lower status (trace/debug/etc) could have an exception: logger.LogInformation(ex, "I have an exception").
The better fix is to include exception information in the tooltip for the message column in the grid. It could be something like:var tooltip = context.Message;
tooltip + Environment.NewLine + $"{context.Properties["exception.type"]}: {context.Properties["exception.message"]}"
tooltip + Environment.NewLine + context.Properties["exception.stacktrace"]"

(need to take into account missing values, etc)
"I'm assuming TraceId + SpanId is a unique identifier for a log and SpanId is only unique within a trace, but would love for that to be confirmed." - this is incorrect. A span id identifies a span and a span can have multiple logs. There currently isn't a unique identifier for a log message. I don't think OTEL gives log entries unique IDs. However, we could generate a guid when one is received and use that value in the query string.
I like the idea of multi-line values in the details grid. However, the look at feel needs work. I think it should look just like other values except span multiple lines, and if it exceeds a maximum allowed height then a scroll bar is added. But I think we should look at what other apps do with this kind of data to get ideas.

Error shouldn't be a button. It doesn't look good, and it is confusing about what the difference is between clicking on error and clicking on view details

I strongly disagree with this. It is impossible to know whether an icon is clickable without directly hovering over it. Clickable icons should be in button form

unless we make the entire row clickable

👍


Empty list

When I open the website and select an airplane I can only see 1 item:

Although it works well when running the website on local.

things getting displayed 2x too

Hello, something on the automatic FTP upload when updating the github repository has removed all line breaks from the TXT files.
If you can test it again please, i just manually updated the files on the site and tested on 3 browsers on 2 devices and it seems ok now.
Thanks for reporting this.

Looks like it's working now, thanks.

Seeing this issue now, when using the upload feature. Can't pinpoint what exactly causes it; seems some checklists display fine, others don't.

Screenshots above are from desktop browser version, but same applies to Windows native 2.3. Following screenshots are from mobile, using the browser and Android 2.3 native. Hoping it's still helpful to mention the native versions, despite them being unsupported currently.


Conflict with the animate.css latest version!

Hi,
Unfortunatly I cannot install the lib via Bower, here is why:
$ bower install Morphext --save
bower Morphext#*                cached https://github.com/MrSaints/Morphext.git#2.4.4
bower Morphext#*              validate 2.4.4 against https://github.com/MrSaints/Morphext.git#*
bower animate.css#~3.2.5        cached https://github.com/daneden/animate.css.git#3.2.6
bower animate.css#~3.2.5      validate 3.2.6 against https://github.com/daneden/animate.css.git#~3.2.5
bower jquery#~2.1.4             cached https://github.com/jquery/jquery-dist.git#2.1.4
bower jquery#~2.1.4           validate 2.1.4 against https://github.com/jquery/jquery-dist.git#~2.1.4
bower jquery#>=1.5              cached https://github.com/jquery/jquery-dist.git#3.1.1
bower jquery#>=1.5            validate 3.1.1 against https://github.com/jquery/jquery-dist.git#>=1.5
bower jquery#>=1.7.2            cached https://github.com/jquery/jquery-dist.git#3.1.1
bower jquery#>=1.7.2          validate 3.1.1 against https://github.com/jquery/jquery-dist.git#>=1.7.2
bower jquery#>=1.4.4            cached https://github.com/jquery/jquery-dist.git#3.1.1
bower jquery#>=1.4.4          validate 3.1.1 against https://github.com/jquery/jquery-dist.git#>=1.4.4
bower jquery#>= 1.2.1           cached https://github.com/jquery/jquery-dist.git#3.1.1
bower jquery#>= 1.2.1         validate 3.1.1 against https://github.com/jquery/jquery-dist.git#>= 1.2.1
bower jquery#>= 1.9.0           cached https://github.com/jquery/jquery-dist.git#3.1.1
bower jquery#>= 1.9.0         validate 3.1.1 against https://github.com/jquery/jquery-dist.git#>= 1.9.0
bower                        ECONFLICT Unable to find suitable version for animate.css

I already have jQuery 3.1.1 and Animate.css 3.5.2 installed for my project... I don't know why I get this error! Do you have any idea?

Can you retry it? I can't seem to replicate it.

Yes, I have retried it several times... But I get the same error.

I'm closing this due to inactivity. Do let me know if the problem persists.


Item.addFurnaceFuel();

This would add a fuel(as coal) for your furnace the code would be like this: "Item.addFurnaceFuel(id,damage,burned);", the burned would be how many items a maximum fuel burn!!!

This is totally possible. I've done it before in an addon.

I would like this on ModPE


Using a Github App token with Steampipe doesn't work

Describe the bug
My company wants to setup a Steampipe instance and we want to avoid to use a Personal Access Token, but rather use an existing (private) Github App.
The token we used in Steampipe was generated following the GH docs here and here.
Steampipe version (steampipe -v)
Steampipe v0.21.1
Plugin version (steampipe plugin list)
+--------------------------------------------------+---------+-------------+
| Installed                                        | Version | Connections |
+--------------------------------------------------+---------+-------------+
| hub.steampipe.io/plugins/turbot/github@latest    | 0.38.0  | github      |
| hub.steampipe.io/plugins/turbot/steampipe@latest | 0.9.1   | steampipe   |
+--------------------------------------------------+---------+-------------+

To reproduce

Generate a token for a Github App, with ALL permissions set to read
GITHUB_TOKEN="ghs_xxxxxx" steampipe query "select * from github_organization_member  where organization='YOUR_ORG'".
It fails with the following error: Error: Resource not accessible by integration (SQLSTATE HV000)

Expected behavior
It should return the list of members of the organization that the app is installed in.
Additional context
As said above, in my attempts to make it work, I've maxed out the permissions of the application.
Here's the full list of permissions of the app and associated token:
{
  "token": "ghs_xxx",
  "expires_at": "2023-12-07T09:15:06Z",
  "permissions": {
    "members": "read",
    "organization_actions_variables": "read",
    "organization_administration": "read",
    "organization_announcement_banners": "read",
    "organization_codespaces": "read",
    "organization_codespaces_secrets": "read",
    "organization_codespaces_settings": "read",
    "organization_copilot_seat_management": "read",
    "organization_custom_org_roles": "read",
    "organization_custom_properties": "read",
    "organization_custom_roles": "read",
    "organization_dependabot_secrets": "read",
    "organization_events": "read",
    "organization_hooks": "read",
    "organization_personal_access_tokens": "read",
    "organization_personal_access_token_requests": "read",
    "organization_plan": "read",
    "organization_projects": "read",
    "organization_secrets": "read",
    "organization_self_hosted_runners": "read",
    "organization_user_blocking": "read",
    "team_discussions": "read",
    "actions": "read",
    "actions_variables": "read",
    "administration": "read",
    "checks": "read",
    "codespaces": "read",
    "codespaces_lifecycle_admin": "read",
    "codespaces_metadata": "read",
    "codespaces_secrets": "write",
    "contents": "read",
    "dependabot_secrets": "read",
    "deployments": "read",
    "discussions": "read",
    "environments": "read",
    "issues": "read",
    "merge_queues": "read",
    "metadata": "read",
    "packages": "read",
    "pages": "read",
    "pull_requests": "read",
    "repository_advisories": "read",
    "repository_hooks": "read",
    "repository_projects": "read",
    "secrets": "read",
    "secret_scanning_alerts": "read",
    "security_events": "read",
    "statuses": "read",
    "vulnerability_alerts": "read",
    "workflows": "write"
  },
  "repository_selection": "all"
}

I guess this overlaps #311 at least partly, looking at the discussion.

The SDK should support using an app, not sure what the config args look like though, https://github.com/google/go-github?tab=readme-ov-file#as-a-github-app.

I would also like to use this plugin for a Pipes installation at my company. A PAT is linked to my access, which is a problem if I don't work there anymore, or it expires, etc.
+1 from me to supporting app-based auth in this plugin, thanks!

Hello @xcambar, thank you for your patience.
I've made several updates in the issue-390 branch to resolve the issue, particularly to enable GitHub App authentication. The configuration file now allows for the inclusion of app_id, installation_id, and private_key. You can also set these values using the environment variables GITHUB_APP_ID, GITHUB_INSTALLATION_ID, and GITHUB_PRIVATE_KEY.
Here's a sample of what the ~/.steampipe/config/github.spc file should look like now:
connection "github" {
  plugin          = "github"
  app_id          = 803263
  installation_id = 29384756
  private_key     = "/Users/path-to-private-key/testappsetting.2024-01-18.private-key.pem"
}

The Steampipe GitHub plugin follows this order for authentication:

Token authentication using the GITHUB_TOKEN environment variable, if set.
Token authentication using the token configured in ~/.steampipe/config/github.spc, if GITHUB_TOKEN is not set.
GitHub App authentication using the relevant environment variables.
GitHub App authentication using credentials provided in the ~/.steampipe/config/github.spc file.

I'd greatly appreciate it if you would like to test these changes in the PR branch issue-390 and share your feedback.
Here are the steps to try it out:

Clone the repository: git clone https://github.com/turbot/steampipe-plugin-github.git
Change directory: cd steampipe-plugin-github
Switch to the branch: git checkout issue-390
Update the .spc file or set the environment variables as needed.
Run the command: make
Execute your queries

Thank you!

Hello @xcambar, I hope you are doing great. Have you had the opportunity to review the previous comment?

Hi. I quickly tested during the week, but since the results were inconclusive (read: the request failed), I need to investigate a bit more to find out if something wrong's on my side (configuration etc.) so I can come back with more useful data.

@ParthaI this approach is reasonable if you are able to access the GH app private key from where you define the connection config.
I have a requirement in Pipes to pass a GitHub App token to the plugin (like we already do with a PAT). It's not feasible/possible for me to expose the private key to the Pipes workspace so I will need to generate the app token centrally and securely distribute to the workspace as required.
So I'm not sure if this is a new issue for this requirement, or just that the solution for this issue requires both approaches:

Provide the plugin with the information it needs to generate an app token (app_id, installation_id, private_key)
Generate you own app token and pass to the plugin (token)

@johnsmyth FYI on this one further to our earlier discussion.


Added support for fortnight and century

Add support for other words ("decade", "fortnight", "century"...) #725

You seem to have accidentally deleted a file.

ohk I'm fixing them

@Gallaecio fixes the files , Now it seems like everything is Ok

@Gallaecio ok I'll them on tests , Thanks for your support and guidance

@Gallaecio - I have fixed all the changes that you were request 😊

@Gallaecio Fixed the changes you requested and also add some impovements 👍🏻


TypeError: Cannot read property 'onDidActiveEditorChange' of undefined

Running unit tests I see many printed of these:
[22429:0101/105912.154623:INFO:CONSOLE(996)] "Uncaught (in promise) TypeError: Cannot read property 'onDidActiveEditorChange' of undefined", source: file:///Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/contrib/terminal/browser/terminalService.js (996)

We need to stub IEditorService in terminalService.test.ts


To add working steps in perpendicular line checker

Is your feature request related to a problem? Please describe.
To add working steps in perpendicular line checker

@abhijeet141 This part already has steps, it was not visible due to bugs, which is now resolved. Please check this #2406

@abhijeet141 This part already has steps, it was not visible due to bugs, which is now resolved. Please check this #2406

Okay


oxker: 0.1.5 -> 0.1.6

Description of changes
https://github.com/mrjackwills/oxker/blob/v0.1.6/CHANGELOG.md

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 196443 run on aarch64-darwin 1

1 package built:

oxker


Add javascript support and examples

Need the ability to add topics, produce and consume form javascript.  The project as it stands requires typescript which is great for large projects, but not ideal for small scripts.
Requirements:

javascript support
examples

Typescript should be good enough for now. Reopen if there are customer requests.


Search bar is non functional

Description:
when we type the keyword to search, no result is displaying

@meowyx  can i work on this issue

Hey @subhamkumarr , for me search bar is working so not sure what you mean by non functional.
Do you want to explain the issue you're facing?

When clicking the search icon button/pressing the enter keyword in the search bar, the search is not working.
https://github.com/Consensys/polis/assets/99977240/feaa4063-9b2b-49af-9b02-2c40e3024311


minor-versioning issue?

Hi,
First, thanks for updating velocity-react to support React 15.
I wanted to inform that on my project, we had a dependency for:
"velocity-react": "^1.1.1"

In our package.json, while still using React 0.14.xx. The minor version update for this package caused 2 version of react to be loaded when deployed in production (0.14 and 15), which of course broke the build.
Maybe it will be wiser to give it a bigger version bump, since depending solely on React 15 now is a big change (or, alternatively, depending on react 0.14 and 15, I've seen other packages doing this)?
Thanks

Sorry to hear you had an issue. The 1.1.5 change does just add React 15 as an option along with 0.14 (the velocity-react code was already compatible with both) but NPM's interpretation of that seems to have done you wrong.
I suspect what happened was you're using a version of NPM that is installing peerDependencies automatically, and NPM pulled in React 15 off of velocity-addons-transition-group's peerDependencies (which it got by choosing to use v15 of that package, rather than 0.14, even though both are supported).
You way want to depend specifically on velocity-addons-transition-group at 0.14.x. velocity-react supports both.
I'm sorry this caused a production problem. Have you looked into npm shrinkwrap to try to keep deployed versions more nailed down?

@finneganh Thanks for the reply. I assumed without looking that the version bump on your part explicitly depended on React 15, but I see you're correct. I'll look into shrinkwrap and perhaps a newer version of NPM that doesn't automatically install peerDependencies.
Again, thanks for your help and contribution!

Good luck! Thanks for using velocity-react!
On Tue, Apr 12, 2016 at 4:50 PM, Lior Brauer<EMAIL_ADDRESS>wrote:

@finneganh https://github.com/finneganh Thanks for the reply. I assumed
without looking that the version bump on your part explicitly depended on
React 15, but I see you're correct. I'll look into shrinkwrap and perhaps a
newer version of NPM that doesn't automatically install peerDependencies.
Again, thanks for your help and contribution!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/twitter-fabric/velocity-react/issues/94#issuecomment-209096599


Adding EFI configuration to tmt-reboot

I performed a manual test of the tmt-reboot script on a Beaker machine which had been installed with UEFI.
tmt_1506_manual_test.txt
I wanted to include some automated tests, but I don't believe it is possible until the following functionality is delivered as that appears to be the only method that I can see through which tmt can specify that the provisioned machine use UEFI instead of legacy BIOS.
I intended to have tmt deploy a virtual machine with the following plan.fmf

summary: Basic smoke test on UEFI booted machine
discover:
how: fmf
provision:
how: virtual
hardware:
boot:
method: uefi
execute:
how: tmt

I will submit another PR with this plan and the related tests which can be merged after the functionality is delivered.

Manual test executed successfully against a beaker provisioned machine:
tmt_1506_manual_test_2.txt


When the mouse clicks on the imgui interface, an error message appears: Exception thrown: read access violation.window was nullptr.

When I clicked the imgui interface with the mouse, an error occurred; it prompted me Exception thrown: read access violation.windowwas nullptr.; I checked the error stack and found that the ImGuiButtonFlags_NoNavFocus macro definition was executed and ClearActiveID was executed. looking through the call stack it happens because the ClearActiveID() function is called，which inturn calls the SetActiveID() with 0 and NULL as arguments ,i.e. SetActiveID(0, NULL)，And this original error occured because in the function SetActiveID

can someone tell me is this the Hazel Source? or is it Hazel 2D only? I'm so confused!

When I clicked the imgui interface with the mouse, an error occurred; it prompted me Exception thrown: read access violation.windowwas nullptr.; I checked the error stack and found that the ImGuiButtonFlags_NoNavFocus macro definition was executed and ClearActiveID was executed. looking through the call stack it happens because the ClearActiveID() function is called，which inturn calls the SetActiveID() with 0 and NULL as arguments ,i.e. SetActiveID(0, NULL)，And this original error occured because in the function SetActiveID

I've had a similar issue where i had to remove the window-> part of those 2 window->DC.Layouts
Maybe either try that or update to a newer version of imgui?

When I clicked the imgui interface with the mouse, an error occurred; it prompted me Exception thrown: read access violation.windowwas nullptr.; I checked the error stack and found that the ImGuiButtonFlags_NoNavFocus macro definition was executed and ClearActiveID was executed. looking through the call stack it happens because the ClearActiveID() function is called，which inturn calls the SetActiveID() with 0 and NULL as arguments ,i.e. SetActiveID(0, NULL)，And this original error occured because in the function SetActiveID当我用鼠标点击imgui界面时，出现错误；它提示我抛出异常：读取访问冲突.windowwas nullptr.;我检查了错误堆栈，发现执行了ImGuiButtonFlags_NoNavFocus宏定义并执行了ClearActiveID。查看调用堆栈，这是因为调用了 ClearActiveID() 函数，该函数又调用了以 0 和 NULL 作为参数的 SetActiveID()，即 SetActiveID(0, NULL)，而出现这个原始错误是因为在函数 SetActiveID 中

I've had a similar issue where i had to remove the window-> part of those 2 window->DC.Layouts我遇到了类似的问题，我必须删除这 2 个window->DC.Layouts的window->部分 Maybe either try that or update to a newer version of imgui?也许尝试一下或更新到较新版本的 imgui？

Deleting Windows presents new problems and can only be attempted with a new version

I removed this loop and it works


Issue opening guides and sections

First off, thank you for the awesome application this is such a lifesaver. The problem I'm having is that I can download the docsets and they load up but whenever I try to load a page that is either a section or a guide I'm running into trouble.
E.g. I download the Django docs and search for migrations:
$ dasht 'migration' django
Near the bottom is an item named "Writing database migrations" and when I try to open it I get the following error:
Can't load file:///home/thorny/.local/share/dasht/docsets/Django.docset/Contents/Resources/Documents/<dash_entry_menuDescription=howto/writing-migrations><dash_entry_name=Writing%20database%20migrations>doc/howto/writing-migrations.

I'm sure it would go longer but I can't see the end. I can go to the docs directory and find the file at
/home/thorny/.local/share/dasht/docsets/Django.docset/Contents/Resources/Documents/doc/howto/writing-migrations.html

So I know it's there but it looks like an error in the link? It isn't limited to the django docs either I also get them in the python3 docs. It seems to be occuring for guides and sections.

The problem lies in the docset itself: the Django docset's internal database row for that topic is like this:
Writing database migrations|Guide|<dash_entry_menuDescription=howto/writing-migrations><dash_entry_name=Writing%20database%20migrations>doc/howto/writing-migrations.html

Please try the fix I pushed to the issue-28 branch, which strips out all embedded XML from URL fields.

Fix pushed to master branch now. 🤕 Thanks for reporting this issue. 👍 Closing.

Works well, Thanks so much!


what if just one argument is given

If just --curr_to is given or just --curr_from, then whole other currencies should be listed nicely. Suggested by Suyog.

Instead I should rather give an option to see all the exchange rates for one currency given as a separate option.


Fix incorrect selector layer when using a serial of sub selectors

This change is

Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on @ko1in1u)

snippet_uiautomator/byselector.py line 52 at r2 (raw file):

      if key in self.SUBSELECTOR:
        return self._find_bottom_selector(d[key])
    return d

Got the following error in g3:
  in _find_bottom_selector: bad return type [bad-return-type]
           Expected: Mapping[str, Union[bool, int, str]]
  Actually returned: Mapping[str, Mapping[str, Union[Mapping[str, Union[bool, int, str]], bool, int, str, Mapping]]]

Update the correct type

Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion

snippet_uiautomator/byselector.py line 45 at r3 (raw file):

    """Converts the keyword arguments to selector type."""
    self._selector = dict(kwargs)
    self._bottom = self._selector

Just curious, if we always copy with only SUBSELECTOR in self._bottom. Why not just filter SUBSELECTOR here:
self._bottom = self._find_bottom_selector(self._selector)

so we don't need to disable protected-access lint, WDYT?
Code quote:
self._bottom = self._selector

We only need to know the object of the innermost dictionary object when copying. Not sure if we need to check in the constructor but I'm okay to move if needed.


Discord Server

May I suggest we start a discord server for discussions and progresses?

sure, will update soon

updated now


feat(a11y): Make CdkFocusTrap and other a11y directives/components standalone

Feature Description
Currently the full A11yModule has to be loaded for simple CdKTrapFocus feature and others.
Is there any timeline for this kind of refactoring? Or is this not possible with the current implementation of the @angular/cdk package? I can see that there are two PRs untouched since December 2022 for migrating other directives to standalone directives.
Use Case
A standalone directive would allow attaching CdkTrapFocus to be attached to the hostDirectives feature introduced in Angular 15. This would reduce complexity as currently a dynamically attached component (like a context menu) needs an additional inner wrapper div to be able to attach the CdkTrapFocus directive to it.

Duplicate of #26036


Add Terraform SAM integration project

IHAC that would like to use both terraform and sam. This project demonstrates how to do this using SSM Parameter store.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Few notes:

terraform-sam-integration/sam/README.md - In SQS testing section include --region=us-west-1 to avoid error when user has different region set as default
/terraform-sam-integration/terraform/README.md - add note to the cleanup part noting that "sam delete" should be run first


fix(deps): update module github.com/aws/aws-sdk-go-v2/credentials to v1.13.38

This PR contains the following updates:

Package
Type
Update
Change

github.com/aws/aws-sdk-go-v2/credentials
require
patch
v1.13.35 -> v1.13.38

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (v1.13.38). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Using consistent kebab-cased type names [ENG-131]

Purpose

The JSON API v2 had inconsistent casing for types. Some were snake_case, and others were kebab-case. This pull request creates a new API (2.15) which makes all types kebab-cased. This API will accept both kebab, and snake, but will throw a deprecation warning when a snake cased type is submitted.
Changes

api/

addons/serializers.py, base/serializers.py comments/serializers.py, files/serializers.py, nodes/serializers.py, and users/serializers.py: Modified Meta class to check for version, and provide kebab-cased type.
base/renderers.py: Modified JSONAPIRenderer.render to add the deprecation warning to the response.
base/serializers.py: Modified class TypeField to accept both kebab and snake case for version 2.15, and only version 2.15.
-schemas/serializers.py: Modified Meta classes to check for version and provide kebab-cased type

api_tests/
-base/test_serializers.py: Added tests for the new API functionality including ensuring that all JSON Serializers use kebab case, and that version 2.15 accepts both kebab and snake case, as well as returning a deprecation message when snake case is used.
QA Notes

Test that 2.15 accepts both kebab and snake, but also test that no JSON Serializers are still using snake cases. See test_serailizers.py::TestSerializerMetaType::test_serailizers_types_are_kebab_case for more details. Test deprecation warning when snake case is used for version 2.15.
Documentation

Many sections of the documentation will need to be updated:
-Draft Registration type will need to be changed from draft_registrations to draft-registrations.
-External account type will need to be changed from external-account
-File Versions type will need to be changed from file_versions to file-versions
-Node Addons type will need to be changed from node_addons to node-addons
-Draft Registrations type will need to be changed from draft_registrations to draft-registrations
-User Addons type will need to be changed from user_addons to user-addons
-Registration Schemas type will need to be changed from registration_schemas to registration-schemas
-File Metadata Schema type will need to be changed from file_metadata_schemas to file-metadata-schemas
-New API Version will need to posted to changelog
https://github.com/CenterForOpenScience/developer.osf.io/pull/46
Side Effects

Ticket

https://openscience.atlassian.net/browse/ENG-131

Drive-by: If documentation is needed, you should submit a PR to https://github.com/CenterForOpenScience/developer.osf.io and link it in the documentation section.

🎉 LGTM!

merging into feature/staging-1 for testing


Update grpc

Fix https://github.com/pachyderm/pachyderm/issues/3265

This screen shows that the first travis build is in progress, but when I click through the link (to https://travis-ci.org/pachyderm/pachyderm/builds/466683207), Travis shows that the build is finished and succeeded. I suspect a message was dropped between Travis and GitHub, and as CI indicates that the build succeeded, I'm going to merge this PR


Not an issue. Question on api design

I looked into the my-books api and saw that you are breaking it down to purchasedBooks, freeBooks, otherBooks ... If there was another type of book, wouldn't it be better if you always returned one array and just have a type and filter it in react?
Also, shouldn't the filter have been done in the react side and not via api?

Click to see Hill for issue #209

Click to see Hill for all issues

created by Async

@denisedelbando Hi, this is a very good question.
A short answer is that you as a book owner or owner of a digital product should not return any purchased data to browser. Once you return purchased data to browser, a user who signed up and did not purchase a book/product will be able to access this data via, for ex, Chrome Dev Tools.
In general though, if data is public, you can indeed return an array from the server and filter it on the browser.
Does it answer your question?

if this was an internal api, you would do the business logic in the api instead. correct?

@denisedelbando Generally, if  you send data from the server to the browser and use data in multiple places inside your app - I would have one internal API that sends this data to the browser. Then I would add some methods to pages/ or /components or lib/store/ (if you use something like MobX) to filter data on the browser.
One of the exception would be paywalled/purchased data, you gotta check permission on server not on client for paywalled/purchased data. You can this check this conditional logic to the same internal API or create a second internal API.
In most of our products, to increase code's readability, we put most of the methods that manipulate data to /lib/store: https://github.com/async-labs/saas/tree/master/app/lib/store
If your web app is really simple and has few methods, like app in this repo or in the book, you can put methods that manipulate data to pages/ and components/.

Important note: we used Next.js in this open source repo and in the book, that means that whatever code you wrote to filter data (inside pages/ or /components or lib/store/) will run on the server for initial tab load and on the client for subsequent loads via internal links. So your front developers will write methods that will run on the server too.

thank you for  your insight @tima101. very much appreciated.


Cannot start two nodes in single machine.

I am trying to setup two nodes in the single machine. I have only 4 aws instances. But  i want to setup 6 nodes. I tried to setup two nodes in my local system first. First node started successfully. But second node failed saying "Unknown Error. Please check log. Program exiting".
First node rpc --> localhost:22002
Second node rpc --> localhost:33002 (This one fails to start).
I have checked the scripts. It is not able to fetch genesis data from node manager of the main node. I ran the post manually from the postman and it works. I think the problem is in pre_start_check.sh file.
Attaching screen shot -->

Yes  IP worked. Thanks.

Is it container issue?.

no. every machine has same loopback 127.* addresses. So is docker containers. If you ask container to connect to <IP_ADDRESS>, it will connect to itself.


Primer Component’s 'SelectMenu' should support 'onOpen' and 'onClose' callback functions

(moved over from the ds repo thanks @smockle 👋 !)
Problem
In another project at GitHub there is a need to perform actions when opening or closing SelectMenus, such as closing other menus or moving the focus to another part of the application besides the SelectMenu trigger
These actions can be performed by calling functions when a SelectMenu opens or closes, but SelectMenu does not currently accept callback functions via props such as onOpen or onClose.
Solution
SelectMenu should accept a callback function via an onOpen prop (to be called when the menu opens) and another callback function via an onClose prop (to be called when the menu closes).
Alternatives
Similar to the approach described in https://github.com/github/design-systems/issues/861 for open support, it is possible  to support onOpen/onClose callback functions by defining a custom SelectMenu component which renders a SelectMenuInternal component which consumes a useSelectMenuInternal hook
Because useSelectMenuInternal is called by SelectMenuInternal (a child of SelectMenu), it has access to both SelectMenu.Context and a set of propagated props (specifically, open, onOpen and onClose).
By using useEffect to respond to changes in SelectMenu.Context and props, useSelectMenuInternal is able to synchronize the SelectMenu.Context’s internal open value with the value of the open prop.
When these values indicate a SelectMenu is newly in an open/visible state, useSelectMenuInternal invokes the onOpen callback function. Conversely, when these values indicate a SelectMenu is newly in a closed/hidden state, useSelectMenuInternal invokes the onClose callback function.

We will soon be deprecating SelectMenu in favor of SelectPanel


fix(versions): update Activiti/activiti-cloud-audit versions into master

UpdateBot pushed helm dependency: common to: 1.1.11

UpdateBot commands:
updatebot push-version --kind helm common 1.1.11


Remake map using the Web Mercator projection

Right now the Web Mercator projection changes the map coordinates to a different system. Will need to figure out how make up for that.

Code committed.
With mapshaper, use these arguments to create a Web Mercator projected map with the correct coordinates:
-proj +proj=merc +a=55 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0 +k=1.0 +units=m +nadgrids=@null +wktext  +no_defs
Adjust 'a' parameter to your needs.


🚀 - Drawer should not define content styles

Description
When migrating from the old Sidebar to the new Drawer component I've noticed that the new component forces content styles on you.
Now in order to customize content CSS you have to use ::ng-deep on the .t-content element.
This is an issue not only if you want custom paddings, but also if you want custom background or custom content height (ie 100%).
I would suggest making .t-content into display: contents kind of element and let users supply their own content styles instead.

We want it to follow our design specs out of the box but be easily customizable. Maybe we can move padding to host element and then compensate it with negative margins on header and footer? I assume if you want custom padding, you are not using header + footer?

No, I am not using header and footer.
Negative margin workaround works for me.

Would you like to make a PR to speed it up?

Unfortunately, the simple trick with negative margins doesn't work because of the scrollbar:

Feel free to look into this when you have some time.

Ok, but that's not going to be soon because I'll be away for a few weeks

That's ok, it's not a major issue.


Scale canvas dimensions with rotated elements

Short description :
Rotated elements can exceed canvas dimensions.
Longer description :
Currently, the canvas dimensions are calculated, using the following values :

X of the leftmost element
Y of the uppermost element
X + element.width of the rightmost element
Y + element.height of the bottommost element

It is certainly not perfect but it renders center-cropped diagrams fine, but it has one caveat : not taking shape circumference and θ (theta) into account.
Here is a good illustration of the encountered problem.

Temporary workaround : By default, generated canvas have a 50px margin, this will be removed or settable in future.
A fix is under development, allowing the renderer to take shape circumference and θ (theta) into account when defining canvas dimensions.

UPDATE : I think I finally found how to do it right but I don't understand why my code works 🥳
Input data :
{
  "type": "excalidraw",
  "version": 2,
  "source": "https://excalidraw.com",
  "elements": [
    {
      "id": "kaZ2_VALK-oFWS7uP9IaU",
      "type": "rectangle",
      "x": 268,
      "y": 137,
      "width": 309,
      "height": 101,
      "angle": 5.6720575750337705,
      "strokeColor": "#c92a2a",
      "backgroundColor": "#fa5252",
      "fillStyle": "hachure",
      "strokeWidth": 1,
      "strokeStyle": "solid",
      "roughness": 1,
      "opacity": 100,
      "groupIds": [],
      "seed": 68483652,
      "version": 34,
      "versionNonce":<PHONE_NUMBER>,
      "isDeleted": false
    }
  ],
  "appState": {
    "viewBackgroundColor": "#ffffff",
    "gridSize": null
  }
}

Result :


FR: Show backlinks by default

Fantastic plugin - thank you.   Makes a huge difference to my workflow.
Any chance we could have an option to show backlinks by default inside the popovers.

Hmm, that should be doable. I'll take a look.

FYI, you can turn on the "Backlinks in document" setting in the "Backlinks" settings page, and it'll work for all panes, including popovers.


luci-app-nlbwmon other options for Commit interval

Hello,
It would be great if the package luci-app-nlbwmon will allow other options for the Commit interval into the Advanced Settings tab of the Configuration menu, like for example 30 minutes, 1 hour and so on.
In this way it is a better compromise between the 10m option and the 12h option.

It does allow other settings, just enter a custom value and press Enter.


[wip] fixed not closing page with no services configured

Motivation
Close page with corresponding services not configured
Description
When there are no services configures, the page should be closed after dismissing the alert. Due to current navigation implementation, the "home" page is displayed
Verification
remove a service configuration from mobile-services.json and go to the service's page and dismiss the alert
Progress

[x] Done Task

@pawelpaszki Cian Clark doesn't work for us anymore.
@secondsun @ziccardi  what is the behaviour on android and xamarin app when the service configuration is not found?


v9.1.7 and v9.1.8 import errors

Example Code
import {
  Collapse,
  NavbarToggler,
  // NavbarBrand,
  Nav,
  NavItem,
  NavLink,
  NavbarText,
} from 'reactstrap'

9.1.7 gives
Module not found: Can't resolve './Container'

9.1.8 gives
import _Container from './Container';
^^^^^^
SyntaxError: Cannot use import statement outside a module

reverting to 9.1.6 works fine

Additional informations:

nextjs v13.2.4
typescript v4.9.5
node v16

also getting this in ^9.1.4

seeing the same issue

I was JUST going to post an issue about this yesterday. I'm having the same issue.

I've made a PR, it would be great if someone can test this PR using yarn link?
To test using yarn link, you can follow this doc.

Any solution?

For now, you can import like this
let Row = dynamic(     () => import("reactstrap").then(d => d.Row), { ssr: false } );

I have changed it to version 9.1.6 It is working in Nextjs.
Please provide if anyone has any other solution or fix.

With reactstrap v9.1.8 I tried the following:
Change the extension from index.js to index.mjs in "node_modules/reactstrap/lib/index.js"
I think it is not the most appropriate but this solved the problem.
Dependencies in my case:
  "dependencies": {
    "next": "^13.2.4",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "reactstrap": "^9.1.8"
  },
  "devDependencies": {
    "@types/node": "18.15.11",
    "@types/react": "18.0.33",
    "typescript": "5.0.3"
  }

@rojaence how does this work for deployments?


Truncated note titles in backlinks

Is your feature request related to a problem? Please describe.
Note titles in backlinks are truncated, and I find myself wishing I had the option to always show the full note title in the backlinks list.
Describe the solution you'd like
I'd like to suggest this as a possible setting, but perhaps there's something I can do already to tweak the theme via CSS that would make it function as desired?
Thanks in advance!

Oh great, thanks! Hadn’t updated the settings, as I’d completely forgotten that was a separate plugin. Great that this is possible, but apologies for having raised this ticket unnecessarily!


Any way to disable the new loader?

Hi,
I have my own implementation for that, and would like to disable the integrated loader.
Any way to customize the loader or disable it?

@Aqua-Ye no not (yet). You can stay on the previous version which didn't have the loader, only new definitions were added but no bugfixes or so

👍

new release is coming soon


SQS trace header fails for unsampled traces

We're using the XRay SDK for tracing our Django apps, which use SQS for asynchronous work. For the most part, it's sending traces of Django requests successfully and we can see them in the console.
However, we had an unsampled request that happened to interact with SQS, and it failed with this error:
ClientError("An error occurred (InvalidParameterValue) when calling the GetQueueUrl operation: The request has a 'X-Amzn-Trace-Id' HTTP header which is reserved for AWS X-Ray trace header and has an invalid value 'Root=dummy;Parent={real_parent_id};Sampled=0'",)

It seems like unsampled requests get the Root of dummy, which seems to cause this validation exception in SQS. Ideally, it'd either have a real ID, or that header wouldn't be injected for unsampled requests.
We have:

python: 3.6.5
aws-xray-sdk: 2.4.2
boto3: 1.9.230
botocore: 1.12.253
django: 2.0.4

Here's our configuration in settings.py:
XRAY_RECORDER = {
    'AWS_XRAY_DAEMON_ADDRESS': f"{EC2_PRIVATE_IP}:{XRAY_PORT}",
    'AWS_XRAY_TRACING_NAME': xray_tracing_name,
    'AWS_XRAY_CONTEXT_MISSING': 'LOG_ERROR',
    'PATCH_MODULES': [
        'boto3',
        'requests'
    ],
    'PLUGINS': ('EC2Plugin', 'ECSPlugin')
}

And the SDK is in our INSTALLED_APPS and MIDDLEWARE.
Let me know if you need more information. Thanks for the work on this library, I think it'll help us a lot!

Hi @jimjshields
Thanks for bringing up this issue. If a trace is unsampled, the python SDK marks the traceid as dummy which fails a backend validation.
We have figured out the root cause of the issue and are working on rolling out a fix soon. I'd request you to please stay tuned for any updates.
Thanks.

That's great, thanks @srprash! Looking forward to a fix.

@srprash Do you have an ETA for the fix? It is affecting our production. Thanks.

Hi @myyee,
SQS is aware of this issue and rolling out a fix soon.

Hi @bhautikpip ,
The X-Ray Python SDK crashed due to this underlying SQS issue. I ended up with nothing in the X-Ray because it fails to send the traces over. This is a totally hidden issue until someone informed me that things are not working correctly. Would you make an enhancement to the SDK so that it fails gracefully and still be able to send out traces to the X-Ray? Thanks.

Hi @myyee,
Since this behavior is only visible in dummy segments. You might not have visibility of this issue on the console because XRay SDK never send out dummy segments to the daemon.

Hello,
Sorry for the inconvenience because of this issue. SQS has finished rollback so you might not be seeing this issue.

hi @jimjshields - you marked this issue as solved, this means that trace_id should be shared lambdas in composition lambda -> SQS -> lambda. Is it currently working for python SDK?

Hi @xrn
This issue was regarding a bug that was introduced as part of our integration with SQS, where if an instrumented SQS client was making a call and the trace is unsampled, the application breaks with the error mentioned in the original post. This was fixed on our end.
I believe the problem you're describing relates to this issue.

I can still see the issue , If all my requests are not sampled
An error occurred (InvalidParameterValue) when calling the DeleteMessage operation: The request has a 'X-Amzn-Trace-Id' HTTP header which is reserved for AWS X-Ray trace header and has an invalid value 'Root=dummy;Parent=39d23fb90647b57f;Sampled=1'

Hi @sachdevajanit123 ,
What's the version of the SDK you're using? This issue was fixed in the v2.6.0


< wxc-tab-page > not easy to use.

< wxc-tab-page > with slot without underlining, tab-titles must be set? How to set? The documentation is too simple, each component must try for a long time!

Look directly at the configuration code in demo may help a little more.
https://github.com/alibaba/weex-ui/blob/master/example/tab-page/config.js


AttributeError: module 'torch.library' has no attribute 'impl_abstract'

Note: I tested this for aqlm==1.0.3 and 1.1.0
I'm having issue trying to run inference with a HF model in Transformers
pip install transformers aqlm[gpu,cpu]

Pytorch 2.1.2 is installed
Then
from transformers import AutoTokenizer, AutoModelForCausalLM
quantized_model = AutoModelForCausalLM.from_pretrained("ISTA-DASLab/Llama-2-7b-AQLM-2Bit-1x16-hf", torch_dtype="auto", device_map="cuda").cuda()
tokenizer = AutoTokenizer.from_pretrained("mistralai/Mixtral-8x7B-v0.1")
output = quantized_model.generate(tokenizer("", return_tensors="pt")["input_ids"].cuda(), max_new_tokens=10)
print(output)

Output:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/transformers/generation/utils.py", line 1544, in generate
    return self.greedy_search(
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/transformers/generation/utils.py", line 2404, in greedy_search
    outputs = self(
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 1168, in forward
    outputs = self.model(
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 1008, in forward
    layer_outputs = decoder_layer(
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 734, in forward
    hidden_states, self_attn_weights, present_key_value = self.self_attn(
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 633, in forward
    query_states = self.q_proj(hidden_states)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/aqlm/inference.py", line 70, in forward
    self.prepare_matmul_op(input)
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/aqlm/inference.py", line 86, in prepare_matmul_op
    get_forward_pass_kernel(self.codebooks, False),
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/aqlm/inference_kernels/kernel_selector.py", line 35, in get_forward_pass_kernel
    from .cuda_kernel import CUDA_FOLDER
  File "/home/michael/venvs/nm-vllm/lib/python3.10/site-packages/aqlm/inference_kernels/cuda_kernel.py", line 20, in <module>
    @torch.library.impl_abstract("aqlm::code1x16_matmat")
AttributeError: module 'torch.library' has no attribute 'impl_abstract'

Well, it's definitely out there and everything was ever tested on torch==2.1.2 and worked fine.
Could you try and reinstall pytorch to make sure that your installation is valid?

@BlackSamorez thanks for the response - are you sure you've tested torch==2.1.2? This clearly hasn't worked for me and when I go to the pytorch 2.1 docs, impl_abstract doesn't seem to exist. I just found out that if I install torch==2.2.0 then it works fine, so maybe it's worth updating the minimum pytorch version?

Sorry, you're right, it doesn't work for pytorch==2.1.2 and lower. My pip+conda env has been hiding the real version from me.
Looks like we'll have to update the requirements after all.
Many thanks!


Added timeout option to add more control over async operations

Some hours ago I opened an issue, I decided to help with this.
The "test" function now optionally accept the timeout option to control async operations. The default value is 5000
Example:

Tests: npm scripts has been run without any error.
Hope this get merged.

Thanks for the PR and sorry it took so long to get it merged. I made a slight change so that it would default to no timeout specified rather than 5 seconds in order to ensure that there are no conflicts with timeouts set in jest configuration. This will be released before the end of the week.


Changing tabs sometimes causes main window to lose focus

Changing tabs sometimes causes main window to lose focus, requiring a click in the main window to restore main.rs InputManager functionality.

Fixed in #726294a

Fixed in commit 726294a


Update 03-BoardNetwork.md

增加自动获取WiFi的IP地址一节

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines.
[ ] Provide a description in this PR that addresses what the PR is solving. If this PR is going to solve an existing issue, please reference the issue (e.g. close #123).

What is the purpose of this pull request?

[ ] Bug fix
[ ] New feature
[ ] Documentation update
[ ] Other

Description

Screenshots

Before
After

非常感谢你提交的内容！


Using instance variables inside a model

I have a few model hooks that duplicate a couple of models to another datastore via API on create/update.
My problem is, I need to update a model at the end of the create hook, this leads to a never-ending loop.
In other frameworks/languages, I've solved this kind of problem with something like this;
this.addHook('afterUpdate', function * (next) {
  if (!this.disableHooks) {
    Kue.dispatch(UserToBapp.key, {type: 'update', id: this.id});
  }
})

and simply set model.disableHooks = true before calling model.save()
My problem is that with Adonis, any instance variables seem to be caught by the setter and then I get errors about them trying to be interested into the SQL database.
How can I avoid this?

Normally I would just set an instance variable on the model to do that, like the this.disableHooks mentioned above.
But because setting this.disableHooks tries to insert data into a field 'disableHooks' in the database upon save(), I cannot use this method because I get a SQL exception.

Hey @mikehhhhhhh !

My problem is, I need to update a model at the end of the create hook, this leads to a never-ending loop.
Why do you need to update a model directly after creating it?

Also, this shouldn't create a never-ending loop since you are doing create -> update -> afterUpdate.

Closing since not actionable.


Adding colors to Donut Chart

Adding donut-colors attribute to directive.

@andersonkoester thanks for the contribution


Find in buffer or Find in Project, removes indent when use copy and paste block code

Try to copy and paste the block code below and see the white-spaces to be removed.
[Original Block]
.vitr{
    &-blog,
    &-prod{margin-top:20px;}
}

[Try to select only]
  &-blog,
  &-prod{margin-top:20px;}

[Paste in Find field - see white spaces removed]
&-blog,
&-prod{margin-top:20px;}

[Atom do not Found]

Thanks for the report! Can you re-open this in https://github.com/atom/find-and-replace if there's no existing issue already so we have it in the right place?  And please fill out the issue template when creating a new issue, it's super helpful for us when triaging issues.


Add 'filled' as optional field in Event document.

If merged, I'll tag a release.

attn @arkilic

Fastest merge in the west

I was looking at it :)

🌵


Fix universal framework

When compiling Tesla-API with Carthage it will now produce framework
files that work on iOS, tvOS, watchOS and macOS.

@Boerworz Since we talked about it the other day, this is how you make a universal framework :)

Cool, thanks for the ping :D


Invalid memory access when trying to start a job

Here is the output of the command :
./bzk job start test dev
2015/04/18 08:07:20 Expected HTTP response code [202] when accessing URL(http://xxx.xxx.xxx.xxx:8000/api/project/test/job); got 500 instead with the following body:
{"error_code":500,"error_msg":"runtime error: invalid memory address or nil pointer dereference"}

You will find bellow the bazooka server log associated
retrieve users: []*bazooka.User{}
Key: &{7c0f3eed2b5557c81d3ccd3366403ea2 BpLnfgDsc2WD8F2qNfHK5a84jjJkwzDk 2084ec349991b5dd39bb9884d5af31cf}
found

My config file
auth: Og==
server_uri: http://xxx.xxx.xxx.xxx:8000/api
home: /root/bazooka
docker_sock: /var/run/docker.sock
scm_key: /root/.ssh/lucien-gitlab

This error is caused by an empty .bazooka.yml file.
The job should error with a more user-friendly message instead of the message you got.
Thanks for reporting this issue.

This issue was indirectly fixed by #114.
Instead of a panic, the logs now contains:

One of 'language' or 'image' needs to be set


rpi2: Include a first z-wave device

After https://github.com/130s/hut_10sqft/issues/338, I've started device inclusion process using Aeotec Smart Switch 6.

From web the inclusion seems tricky...

Hard to tell whether the server is in the state of inclusion. I ended up opening tail -f /var/log/z-way-server.log to see live update from log file.
Hard to tell the client/device is in inclusion mode (this is the device that is to be blamed though).

Interview has so far never completed. At best I saw it reached 80% once (after having forced re-interview many times).
I hit "force-interview" at the device top level, which seems to have cleared all interview info that has already obtained, incl. the device manufacturing info.

The communication with this device was good before I re-installed the server. So I'm not sure what's going on.

Communication of the Smart Switch 6 has been lost for days, even though the same device was communicating well before the last re-installation of z-way server software.
On z-way, exclusion at http://nooo-rpi2:8083/smarthome/#/zwave/exclude/2 "Reset and remove" ends up with An unexpected error occurred during loading data. Try to reload the page. error (server side log:
[2019-12-29 00:42:21.194] [E] [HTTP] Callback execution error: TypeError: undefined is not a function
    at WebServer.document_root (automation/Webserver.js:36:11)

Can't find hints online either. I might just re-re-install z-way server.

sudo apt-get purge zbw
sudo apt-get autoremove
sudo apt-get purge z-way-full
wget 'http://find.z-wave.me/zbw_new_user?box_type=razberry' -O /tmp/zbw_connect_setup.run && sudo bash /tmp/zbw_connect_setup.run -y && sudo /etc/init.d/zbw_connect start
sudo reboot
sudo apt-get install z-way-server

After this, I still see the devices although this time the "exclude" mode seems to be started as I see different message Now confirm the exclusion in the device to be removed. You may need to consult the device manual for more information about how to confirm the exclusion or removal of the device. This process will also reset the devcie back to factory default. and seems to be waiting to communicate with the device. However, the device has not been communicating over a week, which makes me think exclusion would still fail.
I'm now removing the server again, but this time erasing the config data as well.
After some struggle I think I was able to re-install the z-way software and removed configs.
$ sudo rm -fr /etc/zbw/
$ sudo rm -fr /etc/init.d/z-way-server
$ sudo rm -fr /opt/z-way-server/
$ sudo apt-get purge z-way* zbw && sudo apt-get autoremove
$ sudo apt-get install z-way-full
$ wget 'http://find.z-wave.me/zbw_new_user?box_type=razberry' -O /tmp/zbw_connect_setup.run && sudo bash /tmp/zbw_connect_setup.run -y && sudo /etc/init.d/zbw_connect start
$ sudo apt-get install zbw

Same device didn't get included for awhile, then I moved the device to a nearby power plug, and it finally got included...