Better support for different pixel densities

Right now the method is to scale the canvas by a certain amount before painting and round paint coordinates for rects to the nearest integer. But that only produces sharp edges when the zoom is an integer.
It's not hard to support any arbitrary zoom level (like 1.3333). Don't scale the canvas at all and instead, the painting code receives the zoom level as an input, multiplies the layout coordinates by that, and then rounds. This is particularly useful for browser zoom, which makes the window.devicePixelRatio non-even. Things should line up correctly.
Browsers use a much more complicated method of snapping the layouts to pixels. I don't totally know why that is; maybe to produce more desirable results at the extremes.

See https://github.com/DioxusLabs/taffy/issues/369 for some investigation into how browsers to rounding. Conclusions:

For layout: use fractional values (don't round)
For painting: round positions (not lengths). So for each box, round left and right positions separately and then compute width from the new left and right positions.
I believe there is special handling for border widths which get separately rounded to integer values earlier in the process. Not quite sure on the details of this, but browsers will still happily render a 0.5px border with a single pixel in contexts with a pixel ratio of 2.

Thanks for that! Your comment here is interesting:

Rounding of fractional pixels does not affect layout at all. As can be clearly be seen in the test 2 screenshot, once the whole-pixel rounding has occured there is actually room to render both grandchildren nodes on the same line for the 1st and 3rd children. But they wrap anyway because the layout is computed using the unrounded float values

I made a similar example here a while back (zoom in and out a lot - their relative sizes change even though their defined pixels sum to the same) that shows that browsers do round the layout boxes. I agree they don't totally re-layout after that. Sounds like that's what Taffy does now (rounds the layout boxes before render)?
I can see this getting difficult. If an inline table changed total size, the line it's on has to change size too. It's like you have to do layout again, but only changing sizes and positions rather than what flex item or text run goes on what line. Did you come across anything like that with flexboxes-in-flexboxes?

For painting: round positions (not lengths). So for each box, round left and right positions separately and then compute width from the new left and right positions.

In the issue you posted, I can see why this would fix it. According to this old WebKit Wiki, Safari rounded lengths sometimes, and edges sometimes. They don't say when they use which methods though.

made a similar example here a while back (zoom in and out a lot - their relative sizes change even though their defined pixels sum to the same) that shows that browsers do round the layout boxes.

Hmm... I'm not quite sure what I'm supposed to be looking at here. Is the ratio between the first and second columns that you're comparing? How are you measuring the widths?
What I found (mostly testing Chrome, although FF seemed similar) was that:

getBoundingClientRect() will give you fractional values for box size and position.
Older JS APIs like offsetWidth, clientWidth, and scrollWidth will give you those same values naively rounded (literally the fractional values with Math.round` called on them.
That rounding is not what is used for painting which seems to use an algorithm similar to the one that Taffy and Yoga use where it is the fractional x/y values relative to the viewport that is rounded (and widths/heights then recomputed from those rounded positions).
This rounding appears not to affect layout in my observations (they're not available to JS and it seems that when implementing layout algorithms they can be ignored (and just rounded at the end)). I haven't checked table layout though and it's possible that things are different there.

If an inline table changed total size, the line it's on has to change size to

My mental model is that it doesn't ever change size. It ends up with a fractional size. Which will be rounded up or down  by (0 < x < 1) pixels for paint but that won't cause anything else to move.

According to this old WebKit Wiki, Safari rounded lengths sometimes, and edges sometimes. They don't say when they use which methods though.

Yeah, I think I've read that before. I think borders thicknesses are one of those cases but I'm not sure on all of them. I suspect different browsers may do this slightly differently in some cases.

You should see the table and the float (which is defined to be the same height as the table when you sum its row sizes) have different heights:

You can use getBoundingClientRect() and you'll get (very!) different values for the table's height at different zoom levels.
If you put something beneath those, it gets shifted too. So the browser actually does a whole second pass on the layout tree before painting. Not only do the boxes change size, but the boxes that depend on the positions of other boxes also get moved.
Some of the other tests I've done sound more like what you're saying, which could be because it doesn't always manifest obviously. I'll see if I can find some browser code that tells us more.

Ah I see. This does actually seem to be something to do with the borders. Remove the borders between cells and the table is always exactly 400px high (and always matches the float if you change it to have 400px height)

Huh, interesting. With the borders, though, I can't explain it unless pixel snapping is layout-aware. If it's only reproducible with borders it means something, but I don't know what.

My understanding is that there is an additional pixel rounding/snapping step specifically for borders (rounding to integer physical pixels perhaps?) that takes place either prior to layout or as part of layout. I can't remember where I've seen this, but I've definitely seen this referenced in material about one of the major browsers somewhere.

That would explain it! Kinda complicates zooming/pixel density though. Let me know if you remember where you read that.
I feel relieved. I can't find much in Firefox that would indicate pixel snapping in/after layout, and I've read some things saying it happens in WebRender (lots of pixel snapping here). So I think you're right, it can mostly be done in painting.

This isn't what I read before, but this issue (https://github.com/w3c/csswg-drafts/issues/3720) has good information. In particular https://github.com/w3c/csswg-drafts/issues/3720#issuecomment-471779711:

Gecko's border rounding behavior (round down in device pixels, except round values between 0 and 1 device pixels up to 1)

This test is also interesting:
https://codepen.io/SelenIT/pen/LbbGZL

This test is also interesting:
https://codepen.io/SelenIT/pen/LbbGZL

So getBoundingClientRect pretty much proves that borders are rounded before layout. Because width and height remain fractional from CSS to getBoundingClientRect, but borders don't. Huh.
I also found this comment which says it definitively:

I think all browsers snap borders to whole-number display-pixel-values (at least, they do for borders less than 1px) -- so e.g. 0.4px will just round up to one display-pixel (and so will 0.01px), at computed-value-time (i.e. and it actually reserves & occupies that much layout space, rather than the specified tiny-fraction-of-a-pixel).

Thanks for pointing me in the right direction. I guess it's because the edge-snapping behavior could snap borders to zero-sized? Rounding during computed style calc is a little better than snapping to at least 1px during paint since it won't go over content, but the whole thing strikes me as too odd to be better. Did Taffy adopt this behavior/are you planning on it?

https://github.com/w3c/csswg-drafts/issues/5210#issuecomment-1292794138 also seems good:

To snap a length as a border width means to convert that length (presumed to already be nonnegative) as follows:
if the length is greater than 0 and less than 1 device pixel, it should be rounded up to 1 device pixel;
if the length is greater than 1 device pixel, it should be rounded down to the nearest integer number of device pixels.

So it seems like border values in device pixels  are rounded to integer values (again device pixels) using this floor if > 1, ceil if < 1 formula
Where device pixel is defined as:

A device pixel is the smallest unit of area on the device output capable of displaying its full range of colors. For typical color screens, it’s a square or somewhat rectangular region containing a red, green, and blue subpixel. Many non-traditional outputs exist that can blur this definition, such as by displaying some colors at higher resolutions. Such devices still expose some equivalent notion of "device pixel", however.

Taffy doesn't currently implement this, but probably will, perhaps as an option (although I think it can currently be implemented outside of Taffy (and for Blitz/Stylo-Dioxus, I wonder whether stylo will actually already be doing this for us).
Currently Taffy has no concept of scale factor: you get ought the units you use as input. But Taffy's existing rounding/pixel-snapping behaviour effectively assumes physical pixels (it doesn't make much sense to snap to logical pixels) and we may well want to make it a setting (where you can the scale to 1.0 to get the existing behaviour).
At a meta-level, I'd definitely like Taffy to serve a role in documenting this kind of "kinda-in-the-spec-but-kinda-not" stuff.
Also:

https://chromestatus.com/feature/4651561863610368


simple POST addition, can be added to others easily

It appears that POST is also predefined, and we can just route it to the logic in the GET method.
It also shows up in Swagger.

I can do that, but I need to check how Flask requires POST parameters to be set. (Probably according to standard, but one never knows...)

I'm thinking that there is no real reason for adding POST, as it is often added for when a request might reach the typical URL length limit of 2048 characters. Unless an extremely precise set of coordinates is supplied (≈ 1000 digits), we will probably never need this.
Closing my request.


ORC-1515: Skip publishing orc-example module

What changes were proposed in this pull request?
This PR aims to skip publishing orc-example module to Maven Central from Apache ORC 2.0.0.
Why are the changes needed?

Although ORC example code is meaningful for developers as a source code, we don't need it at Maven Central.
We can reduce ORC distribution footprint from 131MB to 74M.

How was this patch tested?
Manual test.
BEFORE
$ du -h ~/.m2/repository/org/apache/orc
 57M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-examples/2.0.0-SNAPSHOT
 57M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-examples
120K	/Users/dongjoon/.m2/repository/org/apache/orc/orc-shims/2.0.0-SNAPSHOT
124K	/Users/dongjoon/.m2/repository/org/apache/orc/orc-shims
3.9M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-mapreduce/2.0.0-SNAPSHOT
3.9M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-mapreduce
 48K	/Users/dongjoon/.m2/repository/org/apache/orc/orc/2.0.0-SNAPSHOT
 52K	/Users/dongjoon/.m2/repository/org/apache/orc/orc
 62M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-tools/2.0.0-SNAPSHOT
 62M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-tools
8.1M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-core/2.0.0-SNAPSHOT
8.1M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-core
131M	/Users/dongjoon/.m2/repository/org/apache/orc

AFTER
$ du -h ~/.m2/repository/org/apache/orc
120K	/Users/dongjoon/.m2/repository/org/apache/orc/orc-shims/2.0.0-SNAPSHOT
124K	/Users/dongjoon/.m2/repository/org/apache/orc/orc-shims
3.9M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-mapreduce/2.0.0-SNAPSHOT
3.9M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-mapreduce
 48K	/Users/dongjoon/.m2/repository/org/apache/orc/orc/2.0.0-SNAPSHOT
 52K	/Users/dongjoon/.m2/repository/org/apache/orc/orc
 62M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-tools/2.0.0-SNAPSHOT
 62M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-tools
8.1M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-core/2.0.0-SNAPSHOT
8.1M	/Users/dongjoon/.m2/repository/org/apache/orc/orc-core
 74M	/Users/dongjoon/.m2/repository/org/apache/orc

Thank you, @mystic-lama .

Merged to main for Apache ORC 2.0.0.


[FEATURE REQUEST] harmonized tonal palettes

It is easy to just import the material library and do getColorTonesMap(Color(harmonize(Color.Blue.toArgb(), MaterialTheme.colorScheme.primary.toArgb()))), however it would be really nice to have this specific thing built into the library and mentioned somewhere in the readme.
Nobody at all is talking about harmonized color palettes and ive spent hours trying to find a solution until I've stumbled on your library by looking up something unrelated to color harmonization

Hi. A little bit late, may i ask what's exactly the feature requested? You want it to be on README if i understand it correctly? There is nothing wrong with the feature?

Hi. A little bit late, may i ask what's exactly the feature requested? You want it to be on README if i understand it correctly? There is nothing wrong with the feature?

hi. there is nothing wrong with the feature. what i mean is it would be nice to have an implementation of harmonized tonal palettes inside the library itself (including a variant with adjusted saturation which i have done here if you're curious).
right now, to have a harmonized tonal palette, you have to import a whole library, which is inconvenient and inefficient

@SmartToolFactory I was having the same problem. I created a library to make using custom harmonized colors easier with a composition local.
compose-custom-colors
It builds on top of this library, so the colors are slightly inaccurate. I wasn't able to find the exact formula online.


Remove gas tank throwing damage

:cl:

remove: Removed gas tank throwing damage.

Thank you!


Avoid Ext calls from outside of runtime

Alternative fix of this panic:
thread '<unnamed>' panicked at 'Externalities not allowed to fail within runtime: "Trie lookup error: Invalid state root: 0x609a�~@�3f5a"', libcore/result.rs:945:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
stack backtrace:
   0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
             at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::print
             at libstd/sys_common/backtrace.rs:71
             at libstd/sys_common/backtrace.rs:59
   2: std::panicking::default_hook::{{closure}}
             at libstd/panicking.rs:211
   3: std::panicking::default_hook
             at libstd/panicking.rs:227
   4: std::panicking::rust_panic_with_hook
             at libstd/panicking.rs:467
   5: std::panicking::begin_panic_fmt
             at libstd/panicking.rs:350
   6: rust_begin_unwind
             at libstd/panicking.rs:328
   7: core::panicking::panic_fmt
             at libcore/panicking.rs:71
   8: core::result::unwrap_failed
   9: <substrate_state_machine::ext::Ext<'a, B> as substrate_state_machine::Externalities>::storage
  10: substrate_state_machine::prove_execution
  11: <substrate_client::call_executor::LocalCallExecutor<B, E> as substrate_client::call_executor::CallExecutor<Block>>::prove_at_state
  12: <substrate_client::client::Client<B, E, Block> as substrate_network::chain::Client<Block>>::execution_proof
  13: <substrate_network::protocol::Protocol<B, S>>::handle_packet
  14: <substrate_network::service::ProtocolHandler<B, S> as substrate_network_libp2p::traits::NetworkProtocolHandler>::read

It should be fixed as @arkpar suggested - by checking if the state is not pruned (he's preparing PR). However, calling 'unsafe' Ext methods (storage && exists_storage) from outside of runtime should be avoided (since Ext expect-s backend calls to succeed) => this PR removes temporary Ext creation for those calls.

I'm not feeling good implementing the option#1 - it is just the ignoring of an error. Like if it was implemented in that way, we would never see today's panic => there would never be a #648 . Everyone ignore logs until something bad happens && in this case nothing bad will happen (except for that guy who has tried to run a light client).
Looks like option#2 is available for WasmExecutor (at least we're able to return Result from env functions). At the same time the only way (at least I know about - maybe there are other ways to bail out?) to skip frames when we're executing native code is to panic. And panic in this case returns us to original implementation of this PR.
So I would prefer to close this PR if it is a mustntgrumble


Added Docker Image for Postges + PostGIS

Hi,
I added an image with a Postgres DB with PostGIS support. As our target data warehouse has big similarities to PostGIS the transition should be easier than using MariaDB for spatial data. Furhthermore PostGIS offers many more features for spatial data.
Additionally I changed the Kafka download link, as the current one was not working for me.
Best Regards,
Karsten

Hi @KarstenSchnitter,
sry for my late answer but I was on vacation. Thanks for your work on the PostGIS docker image. The initial idea behind using MariaDB was mainly to have a simple datastore where @vsimko will save some meta information (tile-indices, etc.) on the raster pipeline use case (Milestone M3). Thus, we would not need advanced spatial features of PostGIS. Since @vsimko is writing the application code for M3 (raster pipeline), I would suggest that he should decide wether he wants to use MariaDB or PostGIS for this demo purpose. However, we want to integrate the Exasolution Docker image in the pipeline in the long run.
Additionally, Kafka's mirrorlist changes rapidly so it can happen that the provided download url is not valid from time to time.
So, I leave this PR open till @vsimko replies.

I just needed the mysql db for the demo pipeline. In future, we should switch to exasol/postgis solution.
I would suggest to make this as an option.

I close this PR due to refactoring the project structure.


New 'wallet load create' page

Abstract
I've changed the design of the 'wallet load create' page to make it more responsive and have a cleaner UI.
What does this PR address?
MPW was not really responsive for other devices and lacks a good design (IMO). My intent is to redesign MPW for a better look and also make it as responsive as possible for every device.
What features or improvements were added?

New wallet load create page

Added Advance mode
Added animations
Redesigned layout
Responsive for mobile

How does this benefit users?
A better design will lure more users in using MPW. This also applies for making the site more responsive.
PIVX Address
DDS5v6mPqUN3rdzkp3268LV7TKjHCZqy8c

Closing as stale, feel free to reopen it or make a new one


node_redis: WARNING: You passed "rediss" as protocol instead of the "redis" protocol!

Hello this is to reopen : https://github.com/NodeRedis/node_redis/issues/1331
This issue has still not beed resolved and its been well over a year. Can someone please tell me when the update is coming out. Its really urgent. Thanks

Closing as #1331 is reopened


[FEATURE] Raw/direct api, without PUG

Is your feature request related to a problem? Please describe.
You've got a great library here with a ton of functionality all in one package. I can't speak for everyone, but I was interested in the ability to conjugate words, adjectives and verbs. I was off-put by the pug part of the equation. I have my own text rendering engine, and having to compile each individual sentence in a separate pug file (never mind packaging it for the browser like i need to do) didn't seem very practical.
Describe the solution you'd like
With a bit of hacking, I was able to use the library in a bit of a "hardcore" mode. What i'd like is to show what i've done, and ideally get this a bit cleaned up, and maybe officially supported.
Just to re-iterate, my ultimate goal is to run multi-lingual text generation, on the fly, in the browser.
I have a script that runs in node to extract a subset of the resources.
const fs = require('fs');
const path = require('path')
const languageCodeGenHelper = require('rosaenlg-pug-code-gen/dist/languageCodeGenHelper')

export function extractVerbs(lang: string) {
    let verbMap = {}
    // my own code here that finds the verbs my app uses.
    return Object.keys(verbMap)
}

const argv = require('yargs').argv
const outPath = argv.out || "./assets/nlg/"
const language = argv.lang ? 'es_ES'
const verbsList = extractVerbs('es')
const wordsList = [] // similarly here,  i would populate my own word and adjectives list using my own app's data sources.
const adjectivesList = []

var codeGenHelper = languageCodeGenHelper.buildLanguageCodeGen(languageCodeGenHelper.getIso2fromLocale(language));

const verbs = codeGenHelper.getVerbsInfo(verbsList)
const words = codeGenHelper.getWordsInfo(wordsList)
const adjectives = codeGenHelper.getAdjectivesInfo(adjectivesList)

const embeddedLinguisticResources = {
    verbs,
    words,
    adjectives
};

let outFile = path.join(outPath, `resources_${languageCodeGenHelper.getIso2fromLocale(language)}.json`)
fs.writeFileSync(outFile, JSON.stringify(embeddedLinguisticResources, null, 4))
console.log("Written to " + outFile)

then, in the browser, you can use as:

const embeddedLinguisticResources = {... (load from generated file above )... }

// define a few helpers (this is lifted out of the generated packages, would be available in lib already in some way)
...
function getAnonymous(gender, number) {
    return util.genderNumberManager.getAnonymous(gender, number);
}
function getAnonMS() {
    return util.genderNumberManager.getAnonMS();
}
function getAnonMP() {
    return util.genderNumberManager.getAnonMP();
}
function getAnonFS() {
    return util.genderNumberManager.getAnonFS();
}
function getAnonFP() {
    return util.genderNumberManager.getAnonFP();
}
....

let pug_html= ""
let spy = {
    getPugHtml: function () {
        return pug_html;
    },
    getPugMixins: function () {
        return pug_mixins;
    },
    setPugHtml: function (new_pug_html) {
        pug_html = new_pug_html;
    },
    appendPugHtml: function (append) {
        pug_html = pug_html + append;
    },
    appendDoubleSpace: function () {
        pug_html = pug_html + "  ";
    },
    getEmbeddedLinguisticResources: function () {
        return embeddedLinguisticResources;
    },

    // we should avoid this one as util. is already available
    isEvaluatingEmpty: function () {
        return util.saveRollbackManager.isEvaluatingEmpty;
    },

    isEvaluatingChoosebest: function () {
        return util.saveRollbackManager.isEvaluatingChoosebest;
    }
}

const util = new NlgLib({language})
util.setSpy(spy)

// once the boilerplate is out of the way, API usage isn't so bad like so:
// except for the fact that "r" doesn't actually contain the result.
const r = util.sentenceManager.verb(getAnonMP(), { verb: 'hablar', tense: 'INDICATIVE_IMPERFECT' }, params)

console.log(util.filterAll("el " + pug_html))

Code similar to this looks like it is running for me;
It looks like a few changes +  tests + documentation would enable another usecase.
Is there interest in this?

Thanks for the ticket and the idea.
RosaeNLG is made of 70+ packages: https://github.com/RosaeNLG/rosaenlg/tree/master/packages, which can be used independently.
Each package brings simple features (e.g. english-a-an-list is a list of words that take "an" vs "a" determiner, english-a-an is a helper on this list, etc.), while some packages like rosaenlg-pug-* bundle everything for Pug templates usage. rosaenlg run the compiled templates and makes the link with all the linguistic resources, but has no link with the Pug syntax itself.
Each package is meant to be reused independently of RosaeNLG as a whole - anybody who don't like Pug syntax but want to write a natural language generator can reuse the linguistic resources to speed up his/her development.
If you just want to conjugate verbs or agree words, you may use directly the simple packages provided, e.g. english-plurals and english-plurals-list for English words.
An issue when rendering in working in browser is that most linguistic resources are just too big. You must only bundle the verbs words adjectives you really want to render and send them to the browser, and not the whole resource. This is the job made by rosaenlg-pug-code-gen: it parses the Pug templates and takes only the required resources.
What I understand is that:

you know the verbs or words to render in advance
the multilingual part is important: using the wrapper functions of rosaenlg package (like sentenceManager.verb) is handy
you want a programmatic API (no templates)

I would say that your approach is the good one (I can't think of a better one).
I am definitely interested in moving forward. Let's think of a proper packaging.
What about a rosaenlg-api package that would do the proper packaging?

Always great to see movement in this repo, this is still the most practically impressive NLG template in open source.

Ludan. You are correct about your understanding of my request (referring to the list of 3 things you outlined).
I've gained a bit more information since the last message. I'd like to add more color to the discussion. You are right that rosangl is a collection of packages. What is nice is that they have a very similar API, which is good.
I was able to make a simple unified API for verb conjugations with just spanish and english with just a few lines.
a little bit like this:
en/index.ts:
const EnglishVerbs = require('english-verbs-helper');
const Irregular = require('english-verbs-irregular/dist/verbs.json');
const Gerunds = require('english-verbs-gerunds/dist/gerunds.json');

const VerbsData = EnglishVerbs.mergeVerbsData(Irregular, Gerunds);

export function getConjugation(verb, tense: string, person: number) {
      return EnglishVerbs(VerbsData, verb, tense, person <= 2 ? 'S' : 'P')
}

es/index.ts:
export { getConjugation } from 'spanish-verbs';

Note that this works only for verb conjugation, and only because Spanish and English are mostly regular.
This break down when trying to scale this up for nouns and adjectives. This is where being able to resolve the used resources comes in handy.
rosaenlg-api:
I understand you would be proposing this module as a way to expose the right functionality? The devil is in the details here, as there is an api for node to generate the resources, then an api for the runtime. It's not clear if a whole new module is necessary here. I could think of a few ways of simply exposing a few new functions in the right modules to achieve the goal.
Other interesting issues:
For solving the multilingual use-case, we'd also have to unify the tenses a bit.  It seems like there should be some "least common denominator" tenses between a few languages. There is no intersection between Spanish tenses and English tenses, it seems.
export type Tense =
    // english
    'SIMPLE_PAST' | 'PAST' |
    'SIMPLE_PRESENT' | 'PRESENT' |
    'SIMPLE_FUTURE' | 'FUTURE' |
    'PROGRESSIVE_PAST' |
    'PROGRESSIVE_PRESENT' |
    'PROGRESSIVE_FUTURE' |
    'PERFECT_PAST' |
    'PERFECT_PRESENT' |
    'PERFECT_FUTURE' |
    'PERFECT_PROGRESSIVE_PAST' |
    'PERFECT_PROGRESSIVE_PRESENT' |
    'PERFECT_PROGRESSIVE_FUTURE' |
    // spanish
    'INDICATIVE_PRESENT' |
    'INDICATIVE_IMPERFECT' |
    'INDICATIVE_PRETERITE' |
    'INDICATIVE_PERFECT' |
    'INDICATIVE_PLUPERFECT' |
    'INDICATIVE_FUTURE_PERFECT' |
    'INDICATIVE_PRETERITE_PERFECT' |
    'SUBJUNCTIVE_PRESENT' |
    'SUBJUNCTIVE_IMPERFECT_RA' |
    'SUBJUNCTIVE_IMPERFECT_SE' |
    'SUBJUNCTIVE_FUTURE' |
    'SUBJUNCTIVE_PERFECT' |
    'SUBJUNCTIVE_PLUPERFECT' |
    'SUBJUNCTIVE_FUTURE_PERFECT' |
    'CONDITIONAL_PRESENT' |
    'CONDITIONAL_PERFECT';

But this is probably an issue for another day.

I'm happy that you set high expectations :joy:
Well, in the branch I got completely rid of the spy stuff.
As a user it works like that:

create the NlgLib object, indicating merely the language
optionally indicate where linguistic resources are stored, if working browser side
code the texts to generate - it can be as simple as generating one verb with nlgLib.sentenceManager.verb as well as containing thousands of instructions
get the final texts using nlgLib.getFiltered()

A sample in French:
const nlgLib = new NlgLib({ language: 'fr_FR' });
nlgLib.sentenceManager.verb(nlgLib.genderNumberManager.getAnonMS(), { verb: 'devenir', tense: 'FUTUR' });
const rendered = nlgLib.getFiltered();
assert.strictEqual(rendered, 'Deviendra');

noPugMode.js is updated.
Things can be improved in the future:

test extensively which RosaeNLG features really work in "no pug" mode
improve the syntax (the code was meant to be generated, not to be written)

I'd be happy to have some help to build an example for server packaging and browser rendering (without using Pug templates).
PS: the #[ ] is the way (for Pug) to call a template from within a template (and I don't like this syntax).

is available in version 3.0.0

Amazing! I'll try it out as soon as I can.
Then we can talk about packaging.
On Thu, May 6, 2021 at 3:13 AM Ludan Stoecklé @.***>
wrote:

is available in version 3.0.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RosaeNLG/rosaenlg/issues/72#issuecomment-833407739,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKFINQWP7BILEAG6N7CYF3TMJTTJANCNFSM432KE6MA
.


[SDK] Add pagination to ERC721's getAllOwners method

Problem solved
Currently this method has no limit on how many items one can query, which create excessive RPC calls. Adding this option should provide users some control over their RPC usage
Changes made

[x] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[x] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

/release-pr

/release-pr

Just wanna credit a community member for pointing out what I couldn't see in my code 💀


Mount /var/log as readOnly

The agent DaemonSet needs to mount /var/log to parse node logs. However, on GKE Autopilot clusters, hostPath mounts are required to be in readOnly mode, making the crowdsec chart unusable on the platform:
admission webhook "warden-validating.common-webhooks.networking.gke.io" denied the request: GKE Warden rejected the request because it violates one or more constraints.
Violations details: {"[denied by autogke-no-write-mode-hostpath]":["hostPath volume varlog in container crowdsec-agent is accessed in write mode; disallowed in Autopilot."]}

Mounting /var/log as readOnly would fix the issue and doesn't seem to affect how Crowdsec works.

Thanks for the contribution :+1:


Use of scipy.interpolate.interp1d function

In various places, the scipy.interpolate.interp1d function is used. Since this function is written in pure python, it is much slower than scipy.interpolate.InterpolatedUnivariateSpline which is a wrapper around Fitpack.
A benchmark of the functions is discussed here: http://www.shocksolution.com/2009/04/lookup-tables-and-spline-fitting-in-python/

Addressed long, long ago. See the custom_spline implementation in empirical_models/model_helpers.py


Clicking on settings gives ArgumentNullException

What You Are Seeing?

System.ArgumentNullException: Value cannot be null.
Parameter name: name
   at System.Resources.ResourceManager.GetString(String name, CultureInfo culture)
   at ChocolateyGui.Common.Windows.ViewModels.SettingsViewModel.<OnActivated>d__100.MoveNext() in C:\buildAgent\work\774e9aed9020e028\Source\ChocolateyGui.Common.Windows\ViewModels\SettingsViewModel.cs:line 615
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
   at System.Windows.Threading.DispatcherOperation.InvokeImpl()
   at MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(Object obj)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at MS.Internal.CulturePreservingExecutionContext.Run(CulturePreservingExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Windows.Threading.DispatcherOperation.Invoke()
   at System.Windows.Threading.Dispatcher.ProcessQueue()
   at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
   at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Int32 numArgs)
   at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)
   at MS.Win32.UnsafeNativeMethods.DispatchMessage(MSG& msg)
   at System.Windows.Threading.Dispatcher.PushFrameImpl(DispatcherFrame frame)
   at System.Windows.Application.RunDispatcher(Object ignore)
   at System.Windows.Application.RunInternal(Window window)
   at ChocolateyGui.App.Main(String[] args) in C:\buildAgent\work\774e9aed9020e028\Source\ChocolateyGui\App.xaml.cs:line 122

What is Expected?
For settings to load.
How Did You Get This To Happen? (Steps to Reproduce)

Open Chocolatey GUI
Click settings on the top right

System Details

OS Build (In PowerShell run [System.Environment]::OSVersion.version.tostring()): 10.0.19044.0
Windows PowerShell version (Run: $PSVersionTable): 5.1.19041.1620
Chocolatey version (Run choco --version): 1.1.0

Output Log

Full Log Output

2022-03-30 11:39:24.633 -05:00 [Information] chocolatey - https://chocolatey.org/api/v2/ | Priority 0|Bypass Proxy - False|Self-Service - False|Admin Only - False.
2022-03-30 11:39:27.354 -05:00 [Fatal] Unhandled Exception

I can provide a crash dump if necessary.

@ImportTaste can you please provide some additional information here...
Is this a fresh installation of Chocolatey GUI, or have you upgraded from a previous version?  If so, what version?
Is the system that you are running on using a locale of something other than English?  If so, what locale is it configured to use?

And just to confirm, what version of Chocolatey GUI are you installing?  The latest? Version 1.0.0.

And just to confirm, what version of Chocolatey GUI are you installing? The latest? Version 1.0.0.

1.0.0

Are you a Chocolatey Business user, or are you using only the OSS packages?
Could you provide the output from the following command:
choco list -lo

Are you a Chocolatey Business user, or are you using only the OSS packages?
Could you provide the output from the following command:
choco list -lo

I'm an OSS user.
Chocolatey v1.1.0
3proxy 0.9.3
autohotkey <IP_ADDRESS>
autohotkey.install <IP_ADDRESS>
avisynthplus 3.7.2
bulk-crap-uninstaller 5.2
choco-cleaner <IP_ADDRESS>
chocolatey 1.1.0
chocolatey-core.extension <IP_ADDRESS>
chocolatey-dotnetfx.extension 1.0.1
chocolatey-windowsupdate.extension 1.0.4
chocolateygui 1.0.0
curl 7.82.0
disable-nvidia-telemetry <IP_ADDRESS>90306
dngrep <IP_ADDRESS>
dotnet 6.0.3
dotnet-5.0-runtime 5.0.15
dotnet-6.0-desktopruntime 6.0.3
dotnet-6.0-runtime 6.0.3
dotnet-desktopruntime 6.0.3
dotnet-runtime 6.0.3
DotNet3.5 3.5.20160716
DotNet4.5.2 <IP_ADDRESS>40902
DotNet4.6 4.6.00081.20150925
dotnetcore-3.1-desktopruntime 3.1.23
dotnetcore-desktopruntime 3.1.23
dotnetfx <IP_ADDRESS>90930
etcher 1.7.8
exiftool 12.40
fastcopy.portable 4.1.3
FirefoxESR 91.7.1
fzf 0.29.0
git <IP_ADDRESS>
git.install <IP_ADDRESS>
golang 1.18
gsudo 1.2.0
honeyview.install 5.46
hugo 0.96.0
hwinfo 7.22
hwinfo.install 7.22
ilspy 7.2
jre8 8.0.321
KB2533623 2.0.0
KB2919355 1.0.20160915
KB2919442 1.0.20160915
KB2999226 1.0.20181019
KB3033929 1.0.5
KB3035131 1.0.3
KB3063858 1.0.0
keepass 2.50
keepass.install 2.50
keepass.portable 2.50
kitty <IP_ADDRESS>
LinkShellExtension <IP_ADDRESS>
llvm 14.0.0
Ln <IP_ADDRESS>
netch 2020.3.5-pre
netfx-4.6.2 <IP_ADDRESS>10905
netfx-4.7.2 <IP_ADDRESS>
nirlauncher 1.23.56
nodejs-lts 16.14.2
nsis 3.08
nsis.install 3.08
nvidia-display-driver 512.15
nvidia-inspector <IP_ADDRESS>00706
nvidia-profile-inspector <IP_ADDRESS>
pandoc <IP_ADDRESS>
pet 0.4.0
pping 3.0.0
python 3.10.4
python3 3.10.4
qbittorrent 4.4.2
ruby <IP_ADDRESS>
ruby.install <IP_ADDRESS>
rufus 3.18
snoop 4.0.1
streamlink 3.2.0
SwissFileKnife <IP_ADDRESS>
sysinternals 2022.2.16
tortoisegit <IP_ADDRESS>
tortoisesvn <IP_ADDRESS>370
trid 2.24.20220327
vcredist140 14.31.31103
vcredist2010 10.0.40219.32503
vcredist2015 14.0.24215.20170201
vcredist2017 14.16.27033
vidcutter <IP_ADDRESS>
vscodium.install 1.65.2
windowsdriverkit10 10.0.19041.685
winrar 6.11
wkhtmltopdf <IP_ADDRESS>10114
xnviewmp.install 0.99.7
yarn 1.22.15
yt-dlp 2<IP_ADDRESS>

I second this, seeing the same as a C4B customer. Fresh install of Chocolaty GUI with chocolateygui 1.1.0
chocolateygui.extension 1.0.0.
What is Expected?
For settings to load.
How Did You Get This To Happen? (Steps to Reproduce)
Open Chocolatey GUI
Click settings on the top right
System Details
OS Build (In PowerShell run [System.Environment]::OSVersion.version.tostring()): 10.0.19044.0
Windows PowerShell version (Run: $PSVersionTable): 5.1.19041.1682
Chocolatey version (Run choco --version): 1.1.0
Output Log

Full Log Output

2022-09-12 09:49:20.082 -04:00 [Information] No branding files exist, nothing else to be done
2022-09-12 09:49:25.733 -04:00 [Information] chocolatey.licensed - https://licensedpackages.chocolatey.org/api/v2/ (Authenticated)| Priority 10|Bypass Proxy - False|Self-Service - False|Admin Only - False.
2022-09-12 09:49:26.056 -04:00 [Information] Interal-repo - https://someInternalRepository (Authenticated)| Priority 1|Bypass Proxy - False|Self-Service - True|Admin Only - False.
2022-09-12 09:49:33.701 -04:00 [Information] 7-zip|21.7|21.7|false
2022-09-12 09:49:34.960 -04:00 [Information] dell-controlvault-host-components|<IP_ADDRESS>|<IP_ADDRESS>|false
2022-09-12 09:49:35.664 -04:00 [Information] openssl|<IP_ADDRESS>0|<IP_ADDRESS>0|false
2022-09-12 09:49:38.505 -04:00 [Fatal] Unhandled Exception

Hi @ImportTaste and @rsolo93,
Could you try running chocolateyguicli from an administrative terminal and report back if there is any output? If there is, perhaps try running chocolateyguicli feature and chocolateyguicli config and report back the outputs?

Sure thing,
chocolateyguicli feature
Chocolatey GUI v<IP_ADDRESS> Business
[ ] AllowNonAdminAccessToSettings - Controls whether or not a non-administrator user can access the Settings Screen.  NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[x] DefaultToDarkMode - Default to dark mode. It is still possible to switch during use.
[x] DefaultToTileViewForLocalSource - Default to tile instead of list view. It is still possible to switch during use.
[x] DefaultToTileViewForRemoteSource - Default to tile instead of list view. It is still possible to switch during use.
[ ] ExcludeInstalledPackages - Do not show packages that are already installed when viewing sources.
[ ] HideAllRemoteChocolateySources - Controls whether or not remote sources (for example the Chocolatey Community Repository) are displayed in Chocolatey GUI.  Enabling this feature makes Chocolatey GUI a read-only application, that will only show the currently installed packages.  NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] HidePackageDownloadCount - Allows control over whether package download count is displayed on remote source views.
[x] HideThisPCSource - Controls whether or not the This PC source is displayed in Chocolatey GUI.  Enabling this feature means that Chocolatey GUI will no longer show a list of all the currently installed packages on the machine.  NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] PreventAllPackageIconDownloads - Prevents the download of the icon for all package. NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] PreventAutomatedOutdatedPackagesCheck - Prevents automated check for outdated packages on local and remote source views.
[ ] PreventPreload - Prevents preloading results with a blank search when opening the remote source view.
[ ] PreventUsageOfUpdateAllButton - Prevents the ability for user to use the Update All button. NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] ShowAdditionalPackageInformation - Show additional package information on Local and Remote views.
[ ] ShowAggregatedSourceView - Show additional source combining all sources in one place.
[ ] ShowConsoleOutput - Shows output from the commands being executed when a job is running.
[ ] SkipModalDialogConfirmation - Skip modal dialog confirmations when performing potentially destructive actions.
[ ] UseDelayedSearch - Enables live search, which returns results after a short delay without clicking the search button.
[x] UseKeyboardBindings - Allows keyboard bindings to be used to interact with different areas of the Chocolatey GUI User Interface.
chocolateyguicli config
Chocolatey GUI v<IP_ADDRESS> Business
DefaultSourceName =  - The name of the source which should be shown by default when opening application. NOTE: This configuration setting will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
OutdatedPackagesCacheDurationInMinutes = 60 - Default timeout for outdated packages to be cached.  Default is 60 minutes.
UseLanguage = en-US - The preferred language for Chocolatey GUI. Defaults to Windows System Locale.

What is the status of this issue? It seems to be unresolved still. I am an OSS user and I get the same exact error.
It points to this function, but I can't figure out which line. Line 627 can't be it, because GetString() is not called there. I can't find any reference to GetString() so it must be called indirectly...
My config:
OS: Win 10
System locale: de
❯ chocolateyguicli feature
Chocolatey GUI v<IP_ADDRESS>
[ ] AllowNonAdminAccessToSettings - Controls whether or not a non-administrator user can access the Settings Screen.  NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] DefaultToDarkMode - Default to dark mode. It is still possible to switch during use.
[x] DefaultToTileViewForLocalSource - Default to tile instead of list view. It is still possible to switch during use.
[x] DefaultToTileViewForRemoteSource - Default to tile instead of list view. It is still possible to switch during use.
[ ] ExcludeInstalledPackages - Do not show packages that are already installed when viewing sources.
[ ] HideAllRemoteChocolateySources - Controls whether or not remote sources (for example the Chocolatey Community Repository) are displayed in Chocolatey GUI.  Enabling this feature makes Chocolatey GUI a read-only application, that will only show the currently installed packages.  NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] HidePackageDownloadCount - Allows control over whether package download count is displayed on remote source views.
[ ] HideThisPCSource - Controls whether or not the This PC source is displayed in Chocolatey GUI.  Enabling this feature means that Chocolatey GUI will no longer show a list of all the currently installed packages on the machine.  NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] PreventAllPackageIconDownloads - Prevents the download of the icon for all package. NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] PreventAutomatedOutdatedPackagesCheck - Prevents automated check for outdated packages on local and remote source views.
[ ] PreventPreload - Prevents preloading results with a blank search when opening the remote source view.
[ ] PreventUsageOfUpdateAllButton - Prevents the ability for user to use the Update All button. NOTE: This feature will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
[ ] ShowAdditionalPackageInformation - Show additional package information on Local and Remote views.
[ ] ShowAggregatedSourceView - Show additional source combining all sources in one place.
[ ] ShowConsoleOutput - Shows output from the commands being executed when a job is running.
[ ] SkipModalDialogConfirmation - Skip modal dialog confirmations when performing potentially destructive actions.
[ ] UseDelayedSearch - Enables live search, which returns results after a short delay without clicking the search button.
[x] UseKeyboardBindings - Allows keyboard bindings to be used to interact with different areas of the Chocolatey GUI User Interface.

❯ chocolateyguicli config
Chocolatey GUI v<IP_ADDRESS>
DefaultSourceName =  - The name of the source which should be shown by default when opening application. NOTE: This configuration setting will only work with Chocolatey for Business and the Chocolatey GUI licensed extension installed.
OutdatedPackagesCacheDurationInMinutes = 60 - Default timeout for outdated packages to be cached.  Default is 60 minutes.
UseLanguage = en-US - The preferred language for Chocolatey GUI. Defaults to Windows System Locale.

My error message:

This has been resolved. The issue stemmed from setting a chocolatey
licensed only option in the config before adding the chocolatey licensed
extension and license
On Thu, Jun 8, 2023 at 11:14 AM Sympatron GmbH @.***>
wrote:

—
Reply to this email directly, view it on GitHub
https://github.com/chocolatey/ChocolateyGUI/issues/931#issuecomment-1582785922,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCHABIA7BTPFKT5ACDJ6ETXKHT3TANCNFSM5SCQZGRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Do you mean PR #1010? When will this be available via chocolatey?

A new version of Chocolatey GUI (v2.1.0-alpha-20230608) has been released which aims to address this issue.  You can find the release notes here:
https://docs.chocolatey.org/en-us/chocolatey-gui/release-notes#alpha-20230608-june-8-2023
And the package here:
https://community.chocolatey.org/packages/chocolateygui/2.1.0-alpha-20230608
You should be able to upgrade to this package version using:
choco upgrade chocolateygui --pre

If you could take this for a spin, and verify that the issue is now resolved, that would be very helpful.


Add swagger convert command

For converting swagger spec from older versions using swagger-converter

Fixed with #264


0 bytes grommet-hpinc.min.css with bower

When I include grommet in an application and use bower install it downloads grommet which includes minified versions of the css files. The grommet-hpinc.min.css file is 0 bytes which isn't usable.

Fixed


Added admin_form_fields to Image model

In Django 1.7, creating forms from a model without specifying a list of fields is deprecated.
This commit makes the get_image_form function look for a field called admin_form_fields on the Image model (which is set to a tuple of field names) and uses it for building the form.

Setting the default admin_form_fields list on AbstractImage means that users with custom image models will have their extra fields silently dropped from the forms, if they don't add admin_form_fields to their image model right away when they upgrade. Effectively, that means we're not giving our users the deprecation period that Django is giving us, and I think we can handle it more gracefully than that:

set admin_form_fields on Image, and leave it unspecified on AbstractImage
have get_image_form check for the presence of an admin_form_fields property, and fall back on the old behaviour if it's not present. (IIRC, you can pass fields='all' to modelform_factory to simplify the 'do I pass fields or not' logic, and suppress Django's own deprecation warning)
In the latter case, throw a PendingDeprecationWarning to tell users to add admin_form_fields to their image models.

Fixed, thanks!


prometheus: limit scrapping to currently used metrics only

Prometheus is still (by default) scrapping a large number of metrics we
are currently not using on our deployments.
This does not mean to say they are usless (which they are not), but by
default adding all metrics is causing serious memory bottlnecks when
increasing scale even at lower numbers.
A previous commit hacked the bucket timeseries out of the equation, as they
amounted for roughly 75% of memory usage on prometheus (from 20Mb per pod to 6Mb)
https://github.com/openservicemesh/osm/pull/2117
This patch pinpoints the exact metrics we use on our deployments, and limits
ingestion to only metrics we we are currently using in our current.
This brings down 6Mb to 2Mb per pod (additional 66%), further allowing operation
of prometheus under larger setups.
Fixes: #2122
Affected area:

Performance            [X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #2276 (885ea8e) into main (a48de43) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2276      +/-   ##
==========================================
+ Coverage   56.89%   56.90%   +0.01%
==========================================
  Files         147      147
  Lines        6560     6560
==========================================
+ Hits         3732     3733       +1
+ Misses       2823     2822       -1
  Partials        5        5

Impacted Files
Coverage Δ

pkg/envoy/route/config.go
96.03% <0.00%> (+0.79%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #2276 (885ea8e) into main (a48de43) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2276      +/-   ##
==========================================
+ Coverage   56.89%   56.90%   +0.01%
==========================================
  Files         147      147
  Lines        6560     6560
==========================================
+ Hits         3732     3733       +1
+ Misses       2823     2822       -1
  Partials        5        5

Impacted Files
Coverage Δ

pkg/envoy/route/config.go
96.03% <0.00%> (+0.79%)
:arrow_up:

Continue to review full report at Codecov.

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


Swift 4 version of the plugin?

Hi, swift 3.x is depricated. Is there a swift 4 version of this plugin available?

That would be great! I love that plugin but I can't use it in iOS... Please?

already swift version 5 since yesterday...

The plugin does not compile anymore on iOS and XCode 10.2. Swift 3 is deprecated and changing to Swift 4 gets many errors. Is there any plan to upgrade?

I went to spin off a new build today as well and after updating to XCode 10.2. I was forced to use at least Swift 4, when choosing Swift 4, 4.2, and 5 I got errors exclusively from this plugin that weren't there prior to switching Swift build version.

So, is it my understanding that, as of right now, that plugin is not going to work for iOS using Phonegap Build?

It won't work if you use the newly-released Xcode 10.2. As a workaround, you can download an older version of Xcode here (search for "Xcode 10.1"). To use it for command-line builds, run the following command:
xcode-select --switch <path to the old Xcode.app>/Contents/Developer

Note that this is only buying you more time; eventually a new version of Xcode will be required for publishing anything to the App Store.

Thanks for your answer, @mernen but I'm not using Xcode. I was referring to Phonegap Build (PGB). Either way, I guess we'll have to wait...

@Razorholt Sorry, I'm indeed not familiar with PhoneGap Build; I thought it was similar to Cordova, in that it ran on your machine? In that case, xcode-select should do the trick; if it's some kind of external build service, I guess you'll be entirely dependent on what options they offer you.

@mernen Thanks for the temporary solution, hopefully that works for me for my testing phase and the plugin is fixed before my next release date.

I fixed the code to make it compatible with Swift 5, but the camera can't detect a QR code. Maybe I'm forgetting something before captureSession!.startRunning(). Of course, suggestions are welcome.
https://github.com/bitpay/cordova-plugin-qrscanner/pull/236

@cmgustavo I built using your code and I'm at the same state. My project successfully built (using Swift 4.2) but the QR scanner isn't able to successfully scan a code.

Hi everyone.
captureOutput() has been removed from Swift, use metadataOutput() instead
Working example https://github.com/SilentKernel/cordova-plugin-qrscanner/blob/master/src/ios/QRScanner.swift

Hi everyone.
captureOutput() has been removed from Swift, use metadataOutput() instead
Working example https://github.com/SilentKernel/cordova-plugin-qrscanner/blob/master/src/ios/QRScanner.swift

Just testing using this code on XCode 10.2 with Swift 4.2 and it's working just like it used to. Thank you so much.

Thanks @SilentKernel !! That worked for me too.

That's great news! Thank you guys for contributing to that project. Please let me know when I can try the new version on Phonegap Build.
 didn't work...

Hi @Razorholt , try to modify directly the file in Xcode (src/ios/QRScanner.swift) and build again. Maybe you have to do cordova rm cordova-plugin-qrscanner and install again.
Or wait a bit until they upgrade the package in npmjs.com https://www.npmjs.com/package/cordova-plugin-qrscanner

I guess I'll wait till npmjs updates the package. Thanks @cmgustavo for all you do!

@Razorholt it should work now you can add it with repo url

The new version has been published: https://www.npmjs.com/package/cordova-plugin-qrscanner
I think that it 's not compatible with XCode 10.1. I'll take a look and let you know.

PhoneGap Build error:
Check dependencies
The “Swift Language Version” (SWIFT_VERSION) build setting must be set to a supported value for targets which use Swift. Supported values are: 3.0, 4.0, 4.2. This setting can be set in the build settings editor.

Is there a Swift4 version available by any chance?

Just updated plugin to v3.0.0 (not available yet on npm but master). It forces to use Swift 5 with:
´´
Also you can add it in your config.xml and re-build. But in v3 it won't be necessary.

Sorry, it didn't work. Same error msg: '... Supported values are: 3.0, 4.0, 4.2...'

Lastest version (3.0.1) is available in https://www.npmjs.com/package/cordova-plugin-qrscanner
Since the plugin depends of cordova-plugin-add-swift-support and it has Swift 3 as default. You should add this to your config.xml
<preference name="UseSwiftLanguageVersion" value="5" />

For old Xcode (<= 10.1). Use cordova-plugin-qrscanner 2.6.2.
Thanks!

Version 3.0.1 works perfectly fine. Thank you!
Will updating cordova-plugin-add-swift-support allow us to set this?
<preference name="UseSwiftLanguageVersion" value="5" />

agree with @baotpham
Running with swift v 5 stop works with add-swift-support.
Without swift-support none of my plugins works. This solution was not great. We really need support to 4.2 for now, while the swift-support is on 4.2.0...

I really can't build when using<EMAIL_ADDRESS>I'm using add-swift-support version 5, and x-code version 10.2.1 (in x-code, swift 5 is selected).
When I try build I get the follow errors:
The following build commands failed:
     PrecompileSwiftBridgingHeader normal armv7
     CompileSwiftSources normal armv7 com.apple.xcode.tools.swift.compiler
     PrecompileSwiftBridgingHeader normal arm64

If I remove qrscanner plugin works perfectly (using swift 5 too).
Please help?

This Example working for me.

Hi everyone.
captureOutput() has been removed from Swift, use metadataOutput() instead
Working example https://github.com/SilentKernel/cordova-plugin-qrscanner/blob/master/src/ios/QRScanner.swift

@SilentKernel Just tested using this code on XCode 10.2 with Swift 4.2 and it's working just like it used to. Thank you so much.

This Example working for me.   But need  change "string: UIApplication.openSettingsURLString" to "string: UIApplicationOpenSettingsURLString"  in openSettings function. line 470


Bump patch level

To include https://github.com/sass/sassc-ruby/pull/192 into the release

@bolandrm can we have this merged and released as a new patch version? thank you!

I'm wondering if sassc is still alive somehow, last commit on master was 17 months ago, which is kind of concerning.
Note: since this aimed for #192, there's a workaround here.


Support of .net core

Are there any plans to introduce support of .net core in nlog?

I think you mean #641 ?
Yes, but we could use some help on that - we have not experience with CoreCLR.

Closing this as this is a dup and has no activity


KubernetesSeedProvider: endpoints null for first node

https://github.com/kubernetes/examples/blob/8774c894fa623dcc7a43bdfdc4fbf5668074fadf/cassandra/java/src/main/java/io/k8s/cassandra/KubernetesSeedProvider.java#L134-L137
Regarding this comment, are there any other scenarios in which endpoints can be null? If not, possibly the local ip could be placed in the seed list instead of using the default seeds list.

@chrislovecnm maybe you can provide some insight?

Have not used this code in a long time ;(

Is there a newer version of the seed provider I could reference?


Remove Success Page from Crypto Contribution Flow

The Contribution flow for crypto has an usability issue such that before the user clicks on Finish button at the end the thank you email is sent. This is initially reported at, https://github.com/opencollective/opencollective/issues/4935#issuecomment-965100288. As a solution we could remove the success page for the Crypto flow and modify the the QR code page so that the user knows that the contribution is pending. The design for this is proposed at, https://github.com/opencollective/opencollective/issues/4935#issuecomment-970936925

Figma Design: https://www.figma.com/file/ZQBMWhnGGtRWeIZknFW1eP/[OC-Design]-Production-Ready-✅?node-id=15015%3A205718

Just to note that the success page has been added back as per new design the contribution pending message has been added to the QR code page and the finish button has been changed to reflect that user that they need to pay using their crypto app.


Packaging kompiled definitions with Python

Right now semantics projects like evm-semantics rely on the user executing custom build commands before running the tools. Instead we could kompile and distribute definitions together with the Python code written to interact with it.
There are two approaches that differ in when kompilation occurs.
Kompile during build time
So for example
pip install evm-semantics

or
poetry install

would kompile all the K targets and install the definitions in site-packages along with kevm-pyk.
How to do that requires investigation, but in general, one either uses a hook for an existing build backend (e.g. tool.poetry.build.script for Poetry), or wraps a build backend with a custom, task-specific backend implementation. Kompilation itself can rely on custom Python build code or kbuild (which enables caching).
One obvious problem here is that a kompiled definition is tied to a K version, so changing the K version breaks the Python package. A solution would be to provide a Python package for K Framework itself so that it can be marked a dependency with a strict version constraint, but this is an issue of its own. However, doing this would make packaging semantics projects very simple.
Kompile during run time
I.e. kompile the definition dynamically when the tool is first run, or when some build dependency (like K version) changes. This is a very flexible approach, however it requires each semantics project to implement the logic, though kbuild might provide support for that.

Since pyk is carrying around it's own K version, can we just rely on pyk version identifier? Or do you mean specifically that we need a pip install kframework package which installs the frontend?
I would argue for something more specific then, like pip install kframework-kompile and pip install kframework-kast, the two binaries that we actually need from the frontend, because pyk becomes the de-factor "frontend" for K in this scenario, and uses these two tools of the Java codebase.


Optionally forgo the setting of extended metadata

I'm mainly using Skicka as a faster way to update my stuff on Drive. The metadata cache make this far faster than Rclone which seems to insist on re-hashing all my local data every time I sync.
However I don't really need the additional metadata Skicka provides like UNIX style file permissions and with the amount of data I have (~50000 files), those extra requests can significantly slow things, particularly with Drive API rate limiting.
It'd be great if we had a switch to disable the extra metadata stuff.

I don't think that the metadata is causing any extra HTTP requests (but am open to being convinced that I'm wrong). To confirm, I looked at some upload traces using -debug -dump-http.  The results were:

When the local metadata cache is updated, it does ask for the extra properties that encode the permissions metadata, but that's one of multiple fields asked for, so it costs some bandwidth and such, but I assume that is fairly limited.
When doing an upload, skicka currently does 3 HTTP transactions for a new file (and 2 for an already-existing one). For a new file, it does a POST to create the file; it includes the metadata during this one. It then does a PUT to fill in its contents. Only after a successful upload does it then do a PATCH to update the modification time. (It is careful to leave the modification time in the year 1970 until a successful upload is done, just so that if there's an upload error, there's no confusion that a partial file is actually legitimate.)  For an existing file, it does a PUT and a PATCH.

(It'd be better to create new files with just a PUT and a PATCH and skip creating the zero-length file, but that's a separate issue.)
So, I don't think there are any separate HTTP transactions related to the permissions; that's just all wrapped up in the existing calls (as far as I can see).

Ah, sorry, you might be right. I just saw the logging messages all over the place and assumed that must be what was happening.
I ended up 'fixing' the issue by using my own API key and raising the limits anyway so it's all fine on my end :)


Feature_Request : Interactive Drifting Bubbles

Is your feature request related to a problem? Please describe.
The bubbles in the screen are somewhat obstructive towards the content a user is interested in or the target button the user is going for. The user might find this unsettling.
Describe the solution you'd like
Interactive Drifting Bubbles. The bubbles on the screen will drift away when the cursor is brought close to them. Upon clicking, the bubbles will move farther away from the point of the click in a faster way.
Purpose:
•	Enhances User Engagement: Captures attention and encourages exploration.
•	Improves User Experience: Dynamic, playful, and intuitive design.
•	Visual Appeal: Makes the website more attractive and memorable.
•	Guides Navigation: Subtly directs users to key content.
•	Differentiates Website: Unique feature that stands out from competitors.
•	Showcases Technical Skills: Demonstrates our proficiency and creativity.
•	Modern Design: Aligns with current web design trends, reinforcing brand identity.
Describe alternatives you've considered
Demo Video/Screenshot (Mandatory )
https://github.com/apu52/Travel_Website/assets/130382676/e56a5ef5-fa0f-4f69-88a3-ba00e903bff4
Bubbles will be moving in the actual implementation this is just a demo on how the bubbles will drift away upon interaction with the cursor.
Are you contributing under any open-source program ?
I agree to follow this project's Code of Conduct
I'm a GSSOC'24 contributor
I want to work on this issue
@viki2211 Is also contributing to this issue so please assign her as well.

@itsekta Can you please tag the level of the mentioned issue?

It seems our bubbles are having a bit of a party—unfortunately, they’re crashing the content and target buttons that users are trying to focus on! These lively bubbles might be causing some unintentional distractions and making the experience less enjoyable.
I’m eager to step in and adjust these bubbles so they add to the user experience without overshadowing the important elements. Could you please assign me this task for GSoC '24? I’m ready to help refine our design and ensure our content gets the spotlight it deserves!
Thanks for considering my request!

It seems our bubbles are having a bit of a party—unfortunately, they’re crashing the content and target buttons that users are trying to focus on! These lively bubbles might be causing some unintentional distractions and making the experience less enjoyable.
I’m eager to step in and adjust these bubbles so they add to the user experience without overshadowing the important elements. Could you please assign me this task for GSoC '24? I’m ready to help refine our design and ensure our content gets the spotlight it deserves!
Thanks for considering my request!

Actually the website does not include the bubble effect now , so it's not necessary now.


Add browser tests for CiviForm admins creating/editing/deleting statuses

Description
Adds the necessary browser test infrastructure and associated tests for validating the CiviForm admin experience for working with program statuses. Still to come are server-side and browser-test-side tests for managing translations.
Release notes:
Add browser tests for program statuses.
Checklist

[X] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[X] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
#1572

Codecov Report

Merging #3015 (f207730) into main (00f487f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3015   +/-   ##
=========================================
  Coverage     77.88%   77.88%
  Complexity     2151     2151
=========================================
  Files           274      274
  Lines          8575     8575
  Branches        582      582
=========================================
  Hits           6679     6679
  Misses         1678     1678
  Partials        218      218

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


⚠ Bofrost.es has degraded performance

In 2f9b74e, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1485 ms

Resolved: Bofrost.es performance has improved in 2ba3eb1 after 6 minutes.


Add new kill-others-if-one-fails option (#55)

I opened up a PR that resolves the conflicts in this PR: #93


How to use relative paths in configuration?

What I am trying to do...
I try to reference a pyproject.toml file containing pyling settings.
I would like to reference it relative to the notebook directory, for example:
"pylsp.plugins.pylint.args": ["--rcfile $notebook_directory$/pyproject.toml"],
That would allow me to move my portable python IDE to a different folder and still keep the reference working.
How I would like to learn how to do it...
Could you please include a comment about relative paths in the documentation of the configuration
and clarify if some variables exist that could be used as a reference for relative paths.

Is $notebook_directory = folder where you start JupyterLab?
If so we should implement pyproject.toml detection in python-lsp-server and then it would work without any configuration. We do that for pycodestyle and flake8 so why not for pylint. Please open an issue and PR over on https://github.com/python-lsp/python-lsp-server

a) Exactly, I refer to the --notebook-dir argument https://jupyterlab.readthedocs.io/en/stable/getting_started/starting.html.
b) If I understand you correctly, currently no path variables are supported in the configuration.
c) Automatic detection also would be fine.

Great! I will close this issue then because this needs to be resolved on the language server side.
(b) we do not touch the configuration at all in jupyterlab-lsp, just forward it to the language server as-is. python-lsp-server just forwards them to pyliny as-is. pylint might have some path variable expansion but you would need to check.
(c) then please follow up in the python-lsp-server repository :)


update index.js

I actually completely brainfarted for 6 months myself, and forgot that you can just use exports directly instead of reassigning module.exports...
90% of the time, I think it's actually nicer to just assign to module.exports instead of reassigning module.exports :)
for example, this:
exports.foo = 'bar'

is better than
module.exports = {
   foo: 'bar'
}

if nothing else, 1 fewer JS object is created in memory :)

:+1: Nice! definetly better.


Include repository size limits for GHAE

After discussions in GHAE channel with @pevd950 I believe we should update the documentation to clarify our guidance for GHAE matches that for GHEC

Why:
Closes #20810
What's being changed (if available, include any code snippets, screenshots, or gifs):
Include text currently showing as below to the documentation for AE for clarity

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

preview located at https://docs-20811-05b97c.preview.ghdocs.com/en/github-ae@latest/repositories/working-with-files/managing-large-files/about-large-files-on-github#repository-size-limits

👋 @gitstua - Thank you for opening a PR and linking it to your issue! ✨
I'll get this triaged for review ⚡


CB-12469 (ios) Appium tests can now run on iOS 10

Platforms affected
iOS
What does this PR do?
https://issues.apache.org/jira/browse/CB-12469
Updates the Appium tests to be able to run not only on iOS 9, but on iOS 10 too.
What testing has been done on this change?
Tested on SauceLabs' iOS 9 and iOS 10 emulators. Did not test on local emulators!
Checklist

[x] Reported an issue in the JIRA database
[x] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
[x] Added automated test coverage as appropriate for this change.

Here's a sample build on both iOS 9 and 10:
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-TEST/37/

Cordova CI Build has completed successfully.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 10  Store
Link
Link
Link

iOS 9.3
Link
Link
Link

iOS 10.0
Link
Link
Link

Android
Link
Link
Link

@filmaj take a look if you'd like 🙂


Publish and link to builtins and tutorial on website

What it looks like here: https://kframework.github.io/k/

Some of the links from https://kframework.github.io/k/tutorial/README.html don't work.
I don't think it's the scope of this PR but we should at least track them with an issue.

Addressed the text of the builtin readme, but fixing all the links/text of the tutorial is another task, I agree. This is to just to get them in place on the website so we can begin iterating on them.


Wrong SHAs on windows-latest machines

I run a E2E matrix on Github Actions. The workflow: https://github.com/Phillip9587/nx-stylelint/blob/main/.github/workflows/ci.yml
Example Run with the problem: https://github.com/Phillip9587/nx-stylelint/actions/runs/1757001450
The jobs which run on a windows-latest machine print the following error and set the wrong SHAS:
Run nrwl/nx-set-shas@v2
  with:
    main-branch-name: main
    set-environment-variables-for-job: true
    error-on-no-successful-workflow: false
    last-successful-event: push
Run node $GITHUB_ACTION_PATH/dist/index.js *** main false push
  node $GITHUB_ACTION_PATH/dist/index.js *** main false push
  shell: C:\Program Files\Git\bin\bash.EXE --noprofile --norc -e -o pipefail {0}
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.
The system cannot find the path specified.

WARNING: Unable to find a successful workflow run on 'origin/main'
We are therefore defaulting to use HEAD~1 on 'origin/main'

NOTE: You can instead make this a hard error by settting 'error-on-no-successful-workflow' on the action in your workflow.

Run echo "Base SHA"
Base SHA
68f16e8ea90c8cb774235fcb085ce20b6cc28b5b

Head SHA
55ff800f33821008db940385b9c299d3429dda6e

Output from a ubuntu-latest machine:
Run nrwl/nx-set-shas@v2
Run node $GITHUB_ACTION_PATH/dist/index.js *** main false push

Found the last successful workflow run on 'origin/main'
Commit: 9d26243d7918a14af4214b11b29568fb63c20d64

Run echo "Base SHA"
Base SHA
9d26243d7918a14af4214b11b29568fb63c20d64

Head SHA
55ff800f33821008db940385b9c299d3429dda6e

Thank you @Phillip9587, I will look into it as soon as possible.


:sparkles: Add rudimentary way to skip failing tasks until other tasks have been completed

@shawn-hurley I cleared the output from the notebook. You can add a git commit hook to do the same by adding
#!/bin/bash
for f in $(git diff --name-only --cached); do
    if [[ $f == *.ipynb ]]; then
        jupyter nbconvert --clear-output --inplace $f
        git add $f
    fi
done

if git diff --name-only --cached --exit-code
then
    echo "No changes detected after removing notebook output"
    exit 1
fi

to .git/hooks/pre-commit. This way we'll never commit the output. We could add a test to check this as well if we wanted to.


Suggestion: CI for the templates

Expected Behaviour

The templates should validate via a travis build. Whenever someone raises a PR, all templates should be used to create the default sample, build it and run it checking the result
Current Behaviour

Manually verified.
Possible Solution

A shell script should be able to do this. Functions can be run using docker run rather than using full OpenFaaS.
Automation would be ideal or some kind of config file. This should be minimal as to not bloat the repository.

For reference the faas-cli does something similar for its samples.

@burtonr thanks for taking a look at this.


ibmi: Fix isatty EBADF handling and refactor

In e14c56b, support was added to implement true isatty() support when
running in the IBM i PASE environment, but it did not handle EBADF
properly. This commit fixes the EBADF handling, but because the
handling was a bit more complicated than previously, it was moved to a
separate isatty function to keep the mainline code simpler.
This also documents why we can't just use isatty on PASE a little more
completely.

@kadler , seems compiler error happened?
src/unix/tty.c:53:12: error: static declaration of 'isatty' follows non-static declaration
 static int isatty(int file) {
            ^~~~~~
In file included from /QOpenSys/pkgs/lib/gcc/powerpc-ibm-aix<IP_ADDRESS>/6.3.0/include-fixed-7.1/fcntl.h:234:0,
                 from ./include/uv/unix.h:27,
                 from ./include/uv.h:66,
                 from src/unix/tty.c:22:
/QOpenSys/pkgs/lib/gcc/powerpc-ibm-aix<IP_ADDRESS>/6.3.0/include-fixed-7.1/unistd.h:203:12: note: previous declaration of 'isatty' was here
 extern int isatty(int);
            ^~~~~~

Is that in Node? I did not get an error when building just libuv. Well, we could work around that by renaming the function and making isatty a macro

I can build libuv with the second commit. I use below configuration for libuv build & test --
export OBJECT_MODE=64
./autogen.sh
./configure LDFLAGS="-Wl,-blibpath:/QOpenSys/pkgs/lib:/QOpenSys/usr/lib" \
  --with-aix-soname=svr4 \
  --disable-static \
  --disable-silent-rules \
  --prefix=/QOpenSys/pkgs
make -j4 && make check

I will test node.js build now, which may take some more time.

Tested with Node.js, the 55555 test passed! LGTM.

Need to revert the commit of PR https://github.com/nodejs/node/pull/32338 once this PR landed into Node.js

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/1831/

Landed in 0ed9692a80b2f310ad52060a8cb400f2a963d09c...4878e8284f5f983968de188d6d740bd3f6ac0e2f.


When importing a dump. Forward the date to milli.

When importing a dump we recreate all the milli indexes.
Since milli v0.35 it’s possible to specify a creation and last update date when opening a milli index.

TODO

[ ] Implement changes in Milli
[ ] Release a Milli version containing these changes
[ ] Bump this new Milli version in Meilisearch and merge it into main

I would like to get assigned on this issue

/assign

Hello @waveywaves,
Thanks for your interest in this project 🔥 You are definitely more than welcome to open a PR for this!
For your information, we prefer not assigning people to our issues because sometimes people ask to be assigned and never come back, which discourages the volunteer contributors from opening a PR to fix this issue.
We will accept and merge the first PR that fixes correctly and well implements the issue following our contributing guidelines.

@Kerollmops I understand :) Can you point me to documentation which would be helpful for me to get started on this issue ?

Hey @waveywaves sorry for the delay I missed your message.
You can start looking at this code: https://github.com/meilisearch/meilisearch/blob/main/meilisearch-http/src/lib.rs#L267
The metadata contains the date. Thus you’ll need to update the create_raw_index method to add the possibility of specifying the date-time.
And then call the new_with_datetime or something function from milli instead of the simple Index::new.

Hey @amab8901, since you've almost finished your PR on the geosearch, I was thinking that maybe you would be looking for something else to do.
This issue should be a good starter for a new contributor as well if you're looking for something else to do.
Then I'm afraid I don't have anything else fun to do :thinking:
But if you want, I could ask the team on Monday :+1:

Hey @amab8901, since you've almost finished your PR on the geosearch, I was thinking that maybe you would be looking for something else to do.
This issue should be a good starter for a new contributor as well if you're looking for something else to do. Then I'm afraid I don't have anything else fun to do thinking But if you want, I could ask the team on Monday +1

thanks! I'll have a look at it after I finished my first PR on geosearch

new_with_datetime - is this an existing function, or do you want me to create a new function?

Hey, this is an existing function, but it's on milli's side.
You can find it here; https://github.com/meilisearch/milli/blob/main/milli/src/index.rs#L143
And I was wrong about the name; it's called new_with_creation_dates instead of new_with_datetime.

So here's what I need to do:

Look at this code (https://github.com/meilisearch/meilisearch/blob/main/meilisearch-http/src/lib.rs#L267)
Identify the date in the metadata
Update create_raw_index method to add the possibility to specify date-time
Call the new_with_creation_dates method (https://github.com/meilisearch/milli/blob/main/milli/src/index.rs#L143) instead of the simple Index::new

Yes exactly!

When you say that Index::new should be replaced, are you referring to this one? https://github.com/meilisearch/meilisearch/blob/2ed601760359abfac1f280de86a615ad572917c1/index-scheduler/src/lib.rs#L353-L407

oh! So this is the one you're referring to as Index::new

ok, so something is calling this, but it's supposed to call this one instead. That's how I understood it.

This one calls create_raw_index.

Hey, create_raw_index currently calls create_index, which then calls create_or_open_index, which finally calls Index::new().

for some reason, all the links we mentioned above lead to 404 "No page found" :thinking: Why is that the case?

Just taking notes here:
create_index(...) calls create_or_open_index(...)

ok I made some solid progress and it compiles. I look forward to your feedback before I do anything further.


[CT-3064] [Feature] Allow comments in jinja directives

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
I find myself writing code like this:
  , {{
    dbt_utils.star(
      from=ref('fees'),
      except=[
        'immutable_fee_id',
        'date_local',
        'deliveries'
      ],
      relation_alias='f',
      quote_identifiers=False
    )
  }}

and find myself wanting to document why certain columns were excluded there. Alas, it does not appear possible to add comments inside jinja2 directives currently.
jinja2 apparently allows for the presence of comments in directives if one declares what such comments should look like in the jinja2 Environment. Would it be possible to e.g. add the standard Python comment string # to the environment dbt uses so we can use comments in directives?
Describe alternatives you've considered
No response
Who will this benefit?
No response
Are you interested in contributing this feature?
No response
Anything else?
No response

@RoyalTS This is a really cool idea!!
https://jinja.palletsprojects.com/en/3.1.x/api/#jinja2.Environment
The place to turn this on inside dbt-core would be here.
However, I haven't quite been able to get this working in local testing, even just using Jinja (not dbt):
>>> import jinja2
>>> env = jinja2.Environment()
>>> env.parse("string")
Template(body=[Output(nodes=[TemplateData(data='string')])])
>>> env.parse("{% set some_value = 123 %} # comment")
Template(body=[Assign(target=Name(name='some_value', ctx='store'), node=Const(value=123)), Output(nodes=[TemplateData(data=' # comment')])])
>>> env.line_comment_prefix = "#"
>>> env.line_comment_prefix
'#'
>>> env.parse("{% set some_value = 123 %} # comment")
Template(body=[Assign(target=Name(name='some_value', ctx='store'), node=Const(value=123))])
>>> env.parse("{% set some_value = # comment \n 123 %}")
...
jinja2.exceptions.TemplateSyntaxError: unexpected char '#' at 20

Basically, while line_comment_prefix works for ignoring comments after Jinja statements:
{% set some_value = 123 %} # this is a comment, Jinja will ignore it
{{ some_macro(...) }}.  # another comment

Jinja doesn't seem to support nesting comments within statements:
{% set some_value = [
    1, # comment
    2, # comment
    3 # comment
] %}
{{ some_macro(
    arg1 = 'blue', #comment
    ...
) }}

So this is (I think) really just syntactic sugar on top of {# #}, rather than the more-compelling capability I was hoping it might be.

Ah, such a shame! Thanks for looking into this though!

Going to close - happy to reopen if someone finds that Jinja can support the thing we really want :)


script runs with no error but no files get modified

Hey there.
Thanks for this script.
It runs pretty quickly despite a few hundred notes. However, I don't get any errors and my .md files don't seem to get modified.
Any ideas?
My usage...
note-link-janitor /Users/michaelmachine/Dropbox/BearNotes
Thanks!!

ahh just realised it doesn't traverse the tree.


AttributeError: 'NoneType' object has no attribute 'rpn'

Traceback (most recent call last):
File "/data/xyz/anaconda3/envs/swin/lib/python3.8/site-packages/mmcv/utils/registry.py", line 52, in build_from_cfg
return obj_cls(**args)
File "/data/xyz/code/Swin-Transformer-Object-Detection-master/mmdet/models/detectors/mask_rcnn.py", line 17, in init
super(MaskRCNN, self).init(
File "/data/xyz/code/Swin-Transformer-Object-Detection-master/mmdet/models/detectors/two_stage.py", line 34, in init
rpn_head_.update(train_cfg=rpn_train_cfg, test_cfg=test_cfg.rpn)
AttributeError: 'NoneType' object has no attribute 'rpn'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "tools/train.py", line 187, in
main()
File "tools/train.py", line 158, in main
model = build_detector(
File "/data/xyz/code/Swin-Transformer-Object-Detection-master/mmdet/models/builder.py", line 77, in build_detector
return build(cfg, DETECTORS, dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "/data/xyz/code/Swin-Transformer-Object-Detection-master/mmdet/models/builder.py", line 34, in build
return build_from_cfg(cfg, registry, default_args)
File "/data/xyz/anaconda3/envs/swin/lib/python3.8/site-packages/mmcv/utils/registry.py", line 55, in build_from_cfg
raise type(e)(f'{obj_cls.name}: {e}')
AttributeError: MaskRCNN: 'NoneType' object has no attribute 'rpn'

Excuse me, have you solved this problem?


extend sync submodules to update docs

The umbrella repository contains all RSTUF components as submodules.
The umbrella repository is the Read The Docs source code for https://repository-service-tuf.readthedocs.org, and it builds some guides and development documentation from the RSTUF components (submodules).
rstuf has a daily sync for the submodules and this commit extends to build the documentation from the submodules.
Closes #187
Signed-off-by: Kairo de Araujo<EMAIL_ADDRESS>

An example of a PR
https://github.com/vmware/repository-service-tuf/pull/205


Tried to work out the pipelines, but fixed the class

network admin, i swear this is good

good merge


x/website:

What is the URL of the page with the issue?
https://go.dev/dl/
What is your user agent?

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Screenshot

N/A
What did you do?

Visit:  https://go.dev/dl/ Look for info about https://go.dev/dl/?mode=json :-)
I was looking for a way to get a (more :-) machine readable version of https://go.dev/dl/ and resorted to tracking down the source to the site and found https://go.googlesource.com/website/+/refs/heads/master/internal/dl/dl.go#18 and that documents: https://go.dev/dl/?mode=json
However, it'd be really nice if there was text on the HTML version of the page which led a user/developer to that URL for the machine-readable (JSON) version of the page.
What did you expect to see?
A link to the machine readable version of the page.
What did you see instead?
No information about how to find the machine readable version of the page.

Duplicate of #34864


Feature/update cappuccino

Sorry I'm late.
Cappuccino uses Asio standalone.

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 4
- Added 20

See the complete overview on Codacy

I'll resolve conflcts

Now it can be merged.


Issues with Setup-Host

Hi,
I've been running into the issues (not sure if they're related or not) seen in the first attached screenshot, when I run Setup-Host, by this point; I've followed the instructions on the Detailed Setup Instructions page, have got Hyper-V installed and running, have completed the pre-checks and run the installation commands with no issues.
To troubleshoot; I've tried deleting all of the AutoLab files and starting again 3 or 4 times over to no avail. I tried to proceed regardless of the issues, I get the error message seen in the second screenshot. I've checked and the vmms.exe does exist in that file directory. I also noticed that hyper-v doesn't have a value against it. I've ensured that I've got vituralisation enabled in BIOS.
For what it's worth, I've got 160+gb of free space on the C:\ SSD, 32gb RAM and a Ryzen R5 3600 CPU, on Win10 Professional
Any help would be greatly appreciated.
Thanks!

I'm assuming you installed from the PowerShell Gallery?

Hi Jeff, that's right, I used the Install-Module PSAutolab -force -skipPublishercheck command as listed in the instructions.

I am getting this now as well. Looking into it.

That's great, thanks for looking into it.

I've been trying to better handle module dependencies and broke it in 4.13.0. Update to 4.14.0 and try again. Although I'm concerned you have another potential issue.  Instead of screenshots do this:
start-transcript .\trans.txt
Setup-Host -verbose
stop-transcript

If there is an error or failure, post the transcript file. You can edit out computer or user name if you need to.

Hi Jeff, I've done as you advised and here's the trascript text file. Thanks again.
transcript.txt

Well, that was a surprise. Apparently Enable-PSRemoting had a bug with -Verbose. Verbose messages almost never show internal code.  You also needed to update the PSAutolab module before running Setup-Host. That would get rid of the missing version message. Despite all that did you try and see if it works? Do you have files under C:\Autolab? Can you change to C:\Autolab\Configurations\SingleServer and run Unattend-Lab?
Regardless, at some point you need to run:
Update-Lab PSAutolab -force
Refresh-Host

It also wouldn't hurt to update to a newer version of Windows 10 if you can.

Hello,
I have got VM setup perfectly fine, but dont know the creds to login the VM server 2019?
Can anyone help me to find the VM creds? Thank you

Passwords for all labs should be P@ssw0rd. Look at the instructions.md file for each lab.
Sent from BlueMail
On Aug 10, 2020, 12:13 AM, at 12:13 AM, norman-hag12<EMAIL_ADDRESS>wrote:

Hello,
I have got VM setup perfectly fine, but dont know the creds to login
the VM server 2019?
Can anyone help me to find the VM creds? Thank you
--
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub:
https://github.com/pluralsight/PS-AutoLab-Env/issues/228#issuecomment-671154057

Closing due to inactivity. If this is still a problem, open a new issue.


Allow BCCSP config to be set using environment variables (release-1.4)

Type of change

Bug fix
Test update

Description
First three commits:

backport PKCS11 test to release-1.4

Fourth commit:

backport BCCSP changes to enable environment variable override of PKCS11 configuration

Related issues
FAB-17969

I genuinely can't decide if this should be merged. The parent of this cherry-pick was merged into master, but it pushed pkcs11 packages out of the pkcs11 build tag. I know this is a cherry-pick of a merged changed, but I don't think it should have been merged. I think the root of the problem with why you can't set environment variables is because you don't have access to the pkcs11 code without the pkcs11 build tag.

I genuinely can't decide if this should be merged. The parent of this cherry-pick was merged into master, but it pushed pkcs11 packages out of the pkcs11 build tag. I know this is a cherry-pick of a merged changed, but I don't think it should have been merged. I think the root of the problem with why you can't set environment variables is because you don't have access to the pkcs11 code without the pkcs11 build tag.

I think we could actually fix things on release-1.4 by moving the PKCS11Opts out of the pkcs11 package to avoid pulling in the rest of the package. I tried this quickly on master but ran into issues due to some newly added private fields of the PKCS11Opts struct that don't exist in this release.

Closing this. It needs to be revisited in master as well, I am reverting the change in master. We need to approach it differently to prevent cross-compilation from breaking due to the PKCS11 code being pushed out of the build tag.


Windows path issue when downloading models

Stumbled upon this error on a freshly installed comfyui_controlnet_aux node. On first launch, it tries to download external models from huggingface and save to a temp folder, but failed because of a hundred-year-old bug in Windows concerning folder and file naming (see: https://learn.microsoft.com/en-us/windows/win32/fileio/naming-a-file?redirectedfrom=MSDN)
Anyways, here is an error log output:
Error occurred when executing MeshGraphormer-DepthMapPreprocessor:

[WinError 267] The directory name is invalid: 'C:\\Users\\user\\AppData\\Local\\Temp\\aux'

  File "D:\Stable Diffusion\ComfyUI_windows_portable\ComfyUI\execution.py", line 152, in recursive_execute
    output_data, output_ui = get_output_data(obj, input_data_all)
  File "D:\Stable Diffusion\ComfyUI_windows_portable\ComfyUI\execution.py", line 82, in get_output_data
    return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
  File "D:\Stable Diffusion\ComfyUI_windows_portable\ComfyUI\execution.py", line 75, in map_node_over_list
    results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
  File "D:\Stable Diffusion\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\node_wrappers\mesh_graphormer.py", line 63, in execute
    from controlnet_aux.mesh_graphormer import MeshGraphormerDetector
  File "D:\Stable Diffusion\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\mesh_graphormer\__init__.py", line 5, in
    from controlnet_aux.mesh_graphormer.pipeline import MeshGraphormerMediapipe, args
  File "D:\Stable Diffusion\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\mesh_graphormer\pipeline.py", line 54, in
    hrnet_checkpoint=custom_hf_download("hr16/ControlNet-HandRefiner-pruned", 'hrnetv2_w64_imagenet_pretrained.pth')
  File "D:\Stable Diffusion\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\util.py", line 316, in custom_hf_download
    model_path = hf_hub_download(repo_id=pretrained_model_or_path,
  File "D:\Stable Diffusion\ComfyUI_windows_portable\python_embeded\lib\site-packages\huggingface_hub\utils\_validators.py", line 118, in _inner_fn
    return fn(*args, **kwargs)
  File "D:\Stable Diffusion\ComfyUI_windows_portable\python_embeded\lib\site-packages\huggingface_hub\file_download.py", line 1159, in hf_hub_download
    os.makedirs(storage_folder, exist_ok=True)
  File "os.py", line 215, in makedirs
  File "os.py", line 215, in makedirs
  File "os.py", line 215, in makedirs
  File "os.py", line 225, in makedirs

Got it, I'll fix this soon.
BTW,  win11 dosen't have this bug.


The locally ran gpt-neo-2.7B is using CPU instead of GPU

I was able to get the model at https://huggingface.co/EleutherAI/gpt-neo-2.7B to work on my computer locally. After following the steps and using the "generator" command, I see the CPU usage go up a lot but the GPU remains idle. Is there a way to configure it to use GPU? I have CUDA and CuDNN installed.

I got it working following the guide at https://www.youtube.com/watch?v=MTrO5_ooUCw&lc=UgzXx8OGCvkqkCAo_Qp4AaABAg with the note book https://github.com/mallorbc/GPTNeo_notebook. Using pip instead of conda, I used the command python -m pip install torch==1.9.0+cu111 torchvision==0.10.0+cu111 -f https://download.pytorch.org/whl/torch_stable.html to install torch's cuda


copernicus data provider

[ ] I added an entry to CHANGELOG.md if knowledge of this change could be valuable to users.

Here is a brief summary of what I did:

 Daten vom Copernicus Data Space Ecosystem STAC Server beschaffen (Prototyp)


Add hard coded Fluentd config

Need to decide what to do with this..
Do we have a dedicated Monasca branch?

Let's keep this as a dedicated branch