[Docs] Separating documentation in multiple files, and make main README with less information

This is a draft, it should not be merged yet.
Hello @jonataslaw !
I am thinking about this for a few days, but i don't even know if that is a good idea. Since README is getting gigantic and it will be even bigger when everything gets documented, the main readme file is getting bloated with information. It's a lot for one place.
My proposal is to split documentation in different files, make the main README have just a resumed version of all GetX can do, and link for the more extended documentation.
Advantages:

The first contact of new users with get can be better, improving the amount of people that want to try it out, and making easier for them to do this
It is easier to maintain, because, if you make an update in just one thing, you can change just one file
more organization in docs

Disavantages:

Since it will not have all features on one file, the user can't simply use CTRL + F and search the term he/she wants to learn more
We will have to focus on two documentations: the resumed version and the extended version

I think it worth it.
If you like the idea, can you give me some advice of what to put in the main README? I'm with little creativity hahah

I like the idea. I hope this doesn't cause any more issues because the person hasn't "seen" the other readmes.
This is also a concern of mine, the readme has become one of the largest library readmes within git, it is gigantic, maybe the readme is just for initial instructions and I am using it as "documentation".
I think the essentials are the navigation methods, the examples of the state managers, and the examples of the instance manager, but even so, I think that perhaps the readme is still at a limit, everything that goes beyond what is written is exaggeration , and translations, GetUtils and Queue are still missing.

Can't you use the WIKI portion of github here to split the files?

i'll put the link both in beginning and end of the topic. Maybe it helps.
I've put the three main parts of GetX under the name "The three pillars". I could not think of a better name on that time, do you have a better idea?
I didn't understand what you meant with this part: but even so, I think that perhaps the readme is still at a limit, everything that goes beyond what is written is exaggeration. Could you rephrase it?
i've noticed you put a bunch of methods on GetUtils and some translation stuff and  didn't said a word about it hahahaha, you must've be very busy. Now, i have no ideia what is this Queue you are talking about.
tomorrow i'll work more on this PR, now i will sleep

Can't you use the WIKI portion of github here to split the files?

@stefandevo actually, that is a good idea! But i don't know how to use this wiki... Do we even have permission to edit? or should we send all of our changes to @jonataslaw to only them he puts there?
well, if anyone can edit, then it will be a lot better to use wiki.

@Nipodemos some info:
https://help.github.com/en/github/building-a-strong-community/changing-access-permissions-for-wikis
So you give acces to contributors for example to edit the wiki. But of course @jonataslaw needs to set it up (and trust the contributors :-) )

@stefandevo i like the idea, if @jonataslaw likes then we can do this
for now i'll update this PR, because when we migrate to wiki, it will be easy as copy & paste 😄
@jonataslaw i put a why i made this package topic in documentation, putting your words of an issue i read. Hope you like it 😄
I think it is ready for review, since this PR is not to put new documentation, it is just to organize

i tried to mess with github wiki a bit, but i've noticed a problem:
it is harder for people to contribute
When a wiki is made, only people that have permission to edit can edit. That is necessary for security/safety reasons, but a person who wants to improve or to make a new page (like for example, that guy that translated the whole readme to spanish) can't do that. They can't make a pull request on wiki either. Here are the options:

He opens a issue asking for permission on the repository, but jonatas can't give permission to anyone, since you can do anything, even push code directly.
Put his changes on a gist or something, create an issue warning @jonataslaw about his desire to improve docs, then jonatas would get his gist and make manually a new commit.
Send the file via email to jonatas and ask him to make a new commit on wiki

It Is just getting harder and harder to help!
For that reason, i think we should give up Github Wiki, and think on other thing. One that supports multi language, and docs versioning

I think it is only possible to disable the Wiki now, not to delete it.
Maybe doing something external like Kauê is doing here https://kauemurakami.github.io/getx_pattern/ (Kauê is the one who created the snippet for vscode) would be a good thing.
Difficult is that generally doing this type of thing would involve working with html, something that not everyone is used to.
Yes, this is a situation that we will have to analyze how to resolve.

I think you can use github pages. It is just a repo and then people can contribute.

I saw now that github generated a page from the Readme's markdown, and updates it whenever a change is made.
There is the possibility to insert another branch, and select for it, in addition to the possibility of creating a new repository just for that.
It might be a good idea to do this.
https://jonataslaw.github.io/getx/

There is an option on the wiki to open for everyone to edit, I have temporarily enabled it now for testing, but I don't know if I need to authorize it, like a merge, or something.

i liked the idea of github pages, now we just need to find a option to have a sidebar with every file of documentation, i wonder if they have that already set up

yeah, github pages doesn't offer the option of different languages, but that's what we get for now
i'll update the other languages to use the same layout as the english version

I have a domain purchased from getx.site that I used to validate the package on pub.dev, and some servers stopped (I pay some servers annually to do tests) I am thinking of doing the Get documentation, using Get with Flutter web, and put on that site. Perhaps it is the best option of all. More laborious, to be sure, but certainly the best option of all.

@jonataslaw i totally agree, and i think that would be an awesome experience. But recently i am with little free time, studying a lot. For now i can change these files, and in a short future, the flutter web site can be made

We can use Get's internationalization to change the language, and dynamic routes to access content directly via the link.
The language can be saved using GetStorage so that if the person visits it again, they can read everything in their preferred language. A little ambitious, but I think the right thing to do.
I would like to have a better communication channel too, currently there are groups spread across the telegram, discord, gitter, Facebook, but nothing officially supported.

@jonataslaw i totally agree, and i think that would be an awesome experience. But recently i am with little free time, studying a lot. For now i can change these files, and in a short future, the flutter web site can be made

I know that, we all have time limitations. So I think about making the "skeleton", and the text part can be inserted lazily.
This done in a repository will probably attract more collaborators, since I see that there are a lot of people who are afraid to offer PRs and something is wrong. Those people who want to contribute, but do not feel at the level of cooperating with something more complex like route / state management, could easily help with the part of the website, which will basically be NavigatorRails + Column> Text.

We can use Get's internationalization to change the language, and dynamic routes to access content directly via the link.
The language can be saved using GetStorage so that if the person visits it again, they can read everything in their preferred language. A little ambitious, but I think the right thing to do.
I would like to have a better communication channel too, currently there are groups spread across the telegram, discord, gitter, Facebook, but nothing officially supported.
that will be so wonderful 🤩 🤩 🤩 , can't wait to see it

For now, the changes made in documentation are ready, could you review and merge them?

I am reviewing them right now, thank you very much for this contribution.

I have a domain purchased from getx.site that I used to validate the package on pub.dev, and some servers stopped (I pay some servers annually to do tests) I am thinking of doing the Get documentation, using Get with Flutter web, and put on that site. Perhaps it is the best option of all. More laborious, to be sure, but certainly the best option of all.

I agreed with this, so the docs itself can be used as a working example of GetX :+1:

See https://antora.org, and to use asciidoc

@maginkgo by using this site wouldn't we have to convert markdown do ascii? that doesn't sound nice

Do not confuse ascii with asciidoc.  It is very similar to markdown but more powerful, and still Github compatible. https://asciidoctor.org/docs/asciidoc-vs-markdown/

@maginkgo i looked in that asciidoc that you've mentioned, and i am amazed 😲
absolutely loved it, and i want to convert all docs to asciidoc hahaha
that's only one problem... there is not flutter package to convert asciidoc to flutter 😢 😞
so in order for we to continue on the idea of creating a flutter website of the docs, we need to keep markdown... which is a shame
if you find some pratical way of show asciidoc on flutter, let me know! i haven't found anything

@jonataslaw did something happened while reviewing this PR? if you want some changes, just tell me 👍

that's only one problem... there is not flutter package to convert asciidoc to flutter 😢 😞 so in order for we to continue on the idea of creating a flutter website of the docs, we need to keep markdown... which is a shame

Why be ashamed, usability is the first priority.  Lacking a directory that is easy to navigate like Bloc, and no recommended best practices. Even so, if you still think about using flutter to create a document yourself, you are simply focusing on one thing and losing the other. Is GetStorge really comparable to commonly used databases? Where is the documentation for GetRouterOutlet? Trying to do everything will only result in doing nothing well.


Source maps

Yo, any chance that you could add sourcemaps support to this?
I'd like to use it with wallaby.js, but wallaby needs a source map to operate. Ref: https://github.com/wallabyjs/public/issues/156

I agree that sourcemaps would make this package a lot more usable, but I am currently not planning to invest in this.


Native ES Classes doesn't work

I use minimum babel transformers to work with chrome.
My trying code.
/* flow */
import "skatejs-web-components";
import {Component, h} from "skatejs";

class Hello extends Component {
  static get props () {
    return {
      name: { attribute: true }
    };
  }
  renderCallback () {
    return h("div", `Hello, ${this.name}`);
  }
}
customElements.define("x-hello", Hello);

It works with this .babelrc and webpack2.
{
  "plugins": [
    "transform-es2015-classes"
  ]
}

Without it, I encounter this error
Uncaught TypeError: Class constructor Hello cannot be invoked without 'new'
    at HTMLElement.StandInElement (eval at <anonymous> (main.js:17), <anonymous>:130:24)
    at CustomElementRegistry.window.customElements.define (eval at <anonymous> (main.js:17), <anonymous>:144:18)
    at Object.<anonymous> (main.js:17)
    at __webpack_require__ (main.js:17)
    at main.js:17
    at main.js:17

hi!
same issue, by reading the docs I was thinking that the line
import "skatejs-web-components";

was optional if your browser supports natively the v1 specs (latest Chrome) or if you plan to load the polyfills "globally" (by script tag).
So this is error (Uncaught TypeError: Class constructor Hello cannot be invoked without 'new') is a bug or I haven't understand correctly?
tnx

@mizchi the original issue you posted here is because you're consuming the transpiled source which is pointed to in the package.json's main field. You can consume src/ directly, but that's not covered by our API versioning, so use at your own risk. More info on the issues around transpiled classes are explained at https://github.com/webcomponents/custom-elements#known-issues. I've updated the docs to clarify this.

I created #992 so that we can offer a native class-based version to get around the native issues while still not making you worry about what other ES features we're using that you may have to transpile.

@treshugart thanks


Upgrade to postcss 5.x

Most plugins are ready but there is still some work to prepare.

Are we just waiting for some plugins to upgrade? If so, can we get a checklist of the ones remaining?

Autoprefixer, for one - waiting on that to be v6

cssnano 3 will support PostCSS 5. Coming shortly after autoprefixer 6.

It looks like Autoprefixer 6.0.1 and cssnano 3.0 are now released. Is there anything else holding us back now?

[x] "autoprefixer": "^6.0.0",
[x] "cssnano": "^3.0.0",
[x] "pixrem": "^2.0.0",
[x] "postcss-calc": "^5.0.0",
[x] "postcss-color-function": "^2.0.0",
[ ] "postcss-color-gray": "^?",
[ ] "postcss-color-hex-alpha": "^?",
[ ] "postcss-color-hwb": "^?",
[ ] "postcss-color-rebeccapurple": "^?",
[x] "postcss-color-rgba-fallback": "^2.0.0",
[x] "postcss-custom-media": "^5.0.0",
[x] "postcss-custom-properties": "^5.0.0",
[x] "postcss-custom-selectors": "^3.0.0",
[ ] "postcss-font-variant": "^?",
[x] "postcss-import": "^7.0.0",
[x] "postcss-media-minmax": "^2.0.0",
[x] "postcss-pseudo-class-any-link": "^1.0.0",
[ ] "postcss-pseudoelements": "^?",
[x] "postcss-reporter": "^1.0.0",
[x] "postcss-selector-matches": "^2.0.0",
[x] "postcss-selector-not": "^2.0.0",
[x] "postcss-url": "^5.0.0",

I can handle all updates except postcss-pseudoelements.

@MoOx you can cross postcss-pseudoelements off the list now. See https://github.com/axa-ch/postcss-pseudoelements/releases/tag/3.0.0

@MoOx see https://github.com/postcss/postcss-color-gray/pull/5

@MoOx see https://github.com/postcss/postcss-color-hex-alpha/pull/5

@MoOx see https://github.com/postcss/postcss-color-hwb/pull/6

@MoOx see https://github.com/postcss/postcss-color-rebeccapurple/pull/6

@MoOx see https://github.com/postcss/postcss-font-variant/pull/7

@jedmao nice work !
@MoOx can you merge all these in ? cssnext is the last package in my pipeline on postcss4 and I would love to upgrade it.

It's not that easy. I think I can't just "merge" yet, because cssnext runners are sync, and plugins like cssnano are now async only.
I will need to release this in a major release, and so, I will probably need to deprecate all runners in favors of postcss ones. But I need to do this the right way, by warning everyone & with a proper changelog on how to upgrade.
That said, I will merge & publish all pr tomorrow so this branch can work if used just as a plugin.
I think I will also release "postcss-cssnext" or something like that, but I didn't get any feedback on #208 and https://github.com/postcss/postcss/issues/477 to take a decision...

@MoOx I only care about the plugin use-case myself. Is there anything else I can do to help?

What do you think about creating a postcss-cssnext that will "just" handle all features plugins but no options like import, url, cssnano ? I can create a repo here so we can extract the main code.
I think not including plugins like import, url & cssnano (that are considered cssnext options, not "features") to make classic postcss usage more intuitive. I don't really know if this postcss-cssnext should use autoprefixer or not. You tell me guys.
poke @ai

I like the idea of postcss-cssnext that includes "just" features plugins. The bonus plugins are clearly out of project scope anyway, as far as I'm concerned.
If I understand correctly, the project goals of cssnext are to write future CSS syntax, today. Seeing as autoprefixer enables you to do that w/o worrying about vendor prefixes, I definitely feel that it belongs in the cssnext project goals; thus, the postcss-cssnext plugin.
That's my $0.02.

It seems to me that import, url, cssnano, autoprefixer are falling out of the scope of this plugin's objective.

I think the best thing is to limit scope where possible. For example, https://github.com/jakubpawlowicz/clean-css has a processImport option which does the same things as postcss-import, but I feel is outside the scope of a minifier and so I chose not to bundle similar functionality with cssnano. Indeed, cssnano's usage of autoprefixer is uncommon because it just removes old prefixes, and even I wasn't sold on the idea until @ai convinced me to include it.
Options like import, url and compress don't contribute to cssnext's scope, I feel. They are a nice convenience if you want to get up and running quickly, but they are not cssnext's modus operandi, which is to polyfill future capabilities of the CSS specification. But I agree with @jedmao, the inclusion of autoprefixer should still make the cut because it ensures that the correct vendor prefixes are applied.

cssnext was first designed when was not widely used, and so when no ones was still using postcss directly via a runner. That's why there is "so many" features, like imports, url,...
It's clear that today it does not clearly make sense, but I also know people that are still happy to "just us it" as it is.
That why I think it's time for a postcss-cssnext that just handle css features. cssnext like it is (with it's own interface) will still be keep for legacy, or for people that like to use a simple (and stupid) tool.
$ cssnext input.css > output.css

That's something no other tool simply bring as I am writing.

All of that makes sense to me. :+1:

Yep, options like import, url, cssnano should be excluded from cssnext.

postcss-cssnext is almost ready. There is still a minor warning, but we will find the responsible.
You can thanks @jedmao for this.
For difference, checkout the README and don't hesitate to provide feedback on the explanation I added.
cssnext itself will need a major breaking change in the API so we can deliver an upgrade, because cssnano is an async plugin, and cssnext api is currently sync only.
cssnext 2.0 will use postcss-cssnext under the hood.

I've been struggling with this half day
How do I install postcss-cssnext to be able to use postcss?
npm install postcss-cssnext --save-dev

didn't work so I cloned the repo and built it using npm install
This didn't work either.
https://github.com/cssnext/postcss-cssnext

On repo page it doesn't say that it's not ready or not working
I'm using this template
https://github.com/kriasoft/react-starter-kit

In the config I changed this
postcss: [
   require('postcss-nested')(),
   require('cssnext')(),
   require('autoprefixer')(AUTOPREFIXER_BROWSERS)
 ]

to this:
postcss: [
   require('postcss-nested')(),
   require('postcss-cssnext')(),
   require('autoprefixer')(AUTOPREFIXER_BROWSERS)
 ]

and it looks like it's still not working
c:\Users\PW\desktop\repos\react\tools\config.js:95
postcss: [require('postcss-nested')(), require('postcss-cssnext')(), require('
autoprefixer')(AUTOPREFIXER_BROWSERS)]
^
TypeError: require(...) is not a function
at Object. (c:/Users/PW/desktop/repos/react/tools/config.js:93:31
)
at Module._compile (module.js:434:26)
at normalLoader (c:\Users\PW\Desktop\repos\REACT\node_modules\babel\node_mod
ules\babel-core\lib\api\register\node.js:199:5)
at Object.require.extensions.(anonymous function) [as .js] (c:\Users\PW\Desk
top\repos\REACT\node_modules\babel\node_modules\babel-core\lib\api\register\node
.js:216:7)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at Object. (c:/Users/PW/desktop/repos/react/tools/start.js:16:16)
at Module._compile (module.js:434:26)

npm ERR! Windows_NT 6.1.7601
npm ERR! argv "c:\Program Files\nodejs\node.exe" "c:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "start"
npm ERR! node v4.0.0
npm ERR! npm  v2.14.2
npm ERR! code ELIFECYCLE
npm ERR! @ start: babel-node --eval "require('./tools/start')().catch(err => co nsole.error(err.stack))"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ start script 'babel-node --eval "require('./tools/start
')().catch(err => console.error(err.stack))"'.
npm ERR! This is most likely a problem with the  package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     babel-node --eval "require('./tools/start')().catch(err => console.
error(err.stack))"
npm ERR! You can get their info via:
npm ERR!     npm owner ls
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     c:\Users\PW\desktop\repos\react\npm-debug.log

@DVLP you need to npm install cssnext, not postcss-cssnext.

@jedmao That was my starting point. Then I was getting this error https://github.com/postcss/postcss/issues/531
So it looks like cssnext is outdated and it will not work with postcss 5. That's why I was hoping to get postcss-cssnext running. Currently I develop with most bootstrap styles missing and hope to fix it tomorrow.

@DVLP yeah. cssnext doesn't support PostCSS 5.0 yet, so you have to use 4.x plugins until it does. postcss-cssnext is almost ready though. Just waiting for the pleeease-filters plugin to upgrade to PostCSS 5.0 and we can pull the switch.

Cool, I can't wait for that. I'm just working on a test project which needs to be ready for tomorrow to get a React contract so probably will have to use the latest 4.x version for now.

We are waiting for @iamvdo to merge & release this PR https://github.com/iamvdo/pleeease-filters/pull/12

postcss-cssnext is out https://github.com/cssnext/postcss-cssnext/releases/tag/2.0.0
Check out the difference between cssnext and postcss-cssnext
Next version of cssnext will use this plugin as a legacy interface.

@MoOx Great work. :+1:

You can thanks @jedmao for this.

@jedmao Thanks! Great work. :+1:

https://github.com/postcss/gulp-postcss/issues/55

Is there any problems with this issue?

My problem is that a day only contains 24h and I have to keep some to sleep :)

(No problem, I am on it since yesterday)

Are PR welcome?

Sadly, it's not that simple and there is a lot of work to do to offer a good upgrade experience.
See #208. I will take care on it, I started to work on it yesterday.

I am currently making a monorepo with postcss-cssnext and legacy/cli cssnext package. Sorry for taking so long. Life can be complicated sometimes.

I changed my mind and think it's better to keep things simple. So I will close this issue in favor of the convergence #208


[Feature Request]: add lyrics translation support

Choose the extension for feature request.
Full Screen Mode
📝 Provide a description of the new feature
Allow language conversion extensions to work on your full app display mod
examples: Japanese Romaji Lyrics, Korean Romaji Lyrics
➕ Additional Information
I don't have a proposed solution as it uses lyrics+ app to function and the extensions only affect the normal lyrics menu, maybe an option to pull lyrics off that

Hey @niruvibes, isn't this added in lyrics-plus currently, so I think I can close this issue.
Feel free to reopen if you mean something else.

Hey @daksh2k this is on lyrics + but the translations dont seem to sync over to this, i assumed it was the way the lyrics were being fetched on this app so i opened an issue. Not a big deal, feel free to not bother as people probably dont care about this

It's working fine for me, maybe you are using an older lyrics-plus? @niruvibes
This functionality is now integrated with lyrics-plus so you don't need a separate extension for that.


Receiving "An exception occured: Cannot send a content-body with this verb-type" running cmdlets

When running the cmdlets, I'm receiving the following:
An exception occured: Cannot send a content-body with this verb-type.

Details:
Position: At C:\Program Files\WindowsPowerShell\Modules\EasyPIM\1.1.0\EasyPIM.psm1:391 char:21
+ ... $response = Invoke-RestMethod -Uri $restUri -Method $method -Headers  ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Error, script did not terminate normaly

Example command:
Export-PIMAzureResourcePolicy -TenantID <redacted> -SubscriptionId <redacted> -rolename "webmaster","contributor" -exportFilename "C:\temp\roles.csv"

Hi @bryanmillerhk tahnk you for reporting this. I think I found the issue, should be fixed in the next version (maybe next week)


Edit review of the node components section

Re-ordered the sections alphabetically.
Edited for clarity and consistency.
Added consistent punctuation.

/lgtm


[Feature] Add support for sub-tasks

This feature is mainly targeted at the JIRA integration, but may be applicable to other integrations as well.
Suppose the following scenario, based on the default configuration files:

FortifyBugTrackerUtility submits 3 XSS vulnerabilities in somePage.jsp to JIRA ticket TST-001
Some developer introduces 2 more XSS vulnerabilities in somePage.jsp
FortifyBugTrackerUtility submits a new JIRA ticket TST-002 for the 2 new XSS vulnerabilities in somePage.jsp

From a process-perspective this makes sense; someone may have already started working on TST-001, so you don't want to add any new vulnerabilities to this existing ticket (this may cause confusion to the developer working on TST-001, or the developer may not even notice the new vulnerabilities  being added to the existing ticket so they remain unfixed).
Instead of the current per-run grouping functionality of FortifyBugTrackerUtility, it would make sense to support the following process for bug trackers that support parent/child relationships:

In the FortifyBugTrackerUtility configuration file, allow for defining parent/child relationships on the target bug tracker configuration
When submitting a new issue (XSS vulnerability 1 in somePage.jsp), check whether parent ticket 'Fix XSS in somePage.jsp' already exists in the target bug tracker
If parent ticket 'Fix XSS in somePage.jsp' does not yet exist, create parent ticket
Add 'XSS vulnerability 1 in somePage.jsp' as sub-task to the 'Fix XSS in somePage.jsp' parent ticket

Basically, instead of grouping vulnerabilities within FortifyBugTrackerUtility based on some configurable grouping criteria, this functionality would allow FortifyBugTrackerUtility to submit each vulnerability as an individual bug tracker ticket, but 'bind' that ticket to some parent ticket that dynamically groups related tickets together.

Jira parent issues are supported in latest version of utility


Added css hover effect on buttons

hope you like it

Change the orange color and set  #6610f2
Also lesser the hover effect

hope you like it

Change the orange color and set #6610f2 Also lesser the hover effect

done

Good work!


Flickr support

woo! done w/first review pass. thanks again!

oh, btw, do Flickr access tokens expire? and i assume users can revoke our access. does this translate those into 401s so bridgy knows to disable the account cleanly?

closing -- superseded by #34


python3Packages.stevedore: 3.2.2 -> 3.3.0

Motivation for this change
Update to latest upstream release 3.3.0
Change log: https://docs.openstack.org/stevedore/latest/user/history.html
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 120274 run on x86_64-linux 1

2 packages marked as broken and skipped:

python38Packages.optuna
python39Packages.optuna

22 packages built:

checkov
jenkins-job-builder (python38Packages.jenkins-job-builder)
nox
papis (python38Packages.papis)
python38Packages.bandit
python38Packages.bugwarrior
python38Packages.cliff
python38Packages.doc8
python38Packages.dogpile_cache
python38Packages.stevedore
python38Packages.subliminal
python38Packages.virtualenvwrapper
python39Packages.bandit
python39Packages.bugwarrior
python39Packages.cliff
python39Packages.doc8
python39Packages.dogpile_cache
python39Packages.jenkins-job-builder
python39Packages.papis
python39Packages.stevedore
python39Packages.subliminal
python39Packages.virtualenvwrapper

Result of nixpkgs-review pr 120274 at 19e052d0 run on x86_64-linux 1

2 packages marked as broken and skipped:

python38Packages.optuna
python39Packages.optuna

22 packages built successfully:

checkov
jenkins-job-builder (python38Packages.jenkins-job-builder)
nox
papis (python38Packages.papis)
python38Packages.bandit
python38Packages.bugwarrior
python38Packages.cliff
python38Packages.doc8
python38Packages.dogpile_cache
python38Packages.stevedore
python38Packages.subliminal
python38Packages.virtualenvwrapper
python39Packages.bandit
python39Packages.bugwarrior
python39Packages.cliff
python39Packages.doc8
python39Packages.dogpile_cache
python39Packages.jenkins-job-builder
python39Packages.papis
python39Packages.stevedore
python39Packages.subliminal
python39Packages.virtualenvwrapper

1 suggestion:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/python-modules/stevedore/default.nix:32:3:
   |
32 |   meta = with lib; {
   |   ^


Expose Dictionary to reader

Many Parquet query engines have optimizations that rely on Dictionary encoded columns, e.g. for selections with filter.
The Rust implementation of the Parquet reader makes it difficult for a reader to read dictionary encoded values because it doesn't expose the RLE decoder to the reader code, so a reader that wishes to work with dictionary values has to re-implement an RLE decoder to read values from dictionary encoded data pages.
This can be easily addressed by making the RLE code public outside the crate.

FYI I think @tustvold  has some plans to contribute functionality that may be similar to the parqet crate directly in https://github.com/apache/arrow-rs/issues/1191

I'd be very interested in any details you can share about your particular use-case, in particular if there is anyway we might be able to combine efforts in this space. The proposal in #1191 is just that, and any input you'd be willing to provide would be most appreciated :+1:
I'd also potentially draw your attention to #1180 which if you're using arrow provides a mechanism to extract data preserving the dictionary encoding present in the parquet file, and is slated for inclusion in arrow 9.

@alamb @tustvold My use case is a proprietary analytical DB engine, it has its' own proprietary storage format but also allows running queries against external formats like Parquet.
As it already has a highly optimized scan capability of dictionary encoded data, all I want is for it to have access to the raw Parquet dictionary.
I don't want to take a dependency on Arrow array for that as I'm not using Arrow at all, I don't deserialize Parquet into Arrow since the engine I'm working on has its' own in-memory representation (I don't even build Arrow with Parquet)

Thank you for taking the time to respond, I figured that might be the case, but thought it couldn't hurt to check.
As a heads up if you're reading the data directly, the RLE encoding is not length preserving (#1111), and there are some other shenanigans concerning dictionary spilling.
FWIW there were some generics added in #1041 and evolved since to aid decoding columns to custom in-memory representations. They're currently crate-local, but that could be changed if you wished to use them. Just let me know 😀

@tustvold My engine assumes a column is either fully dictionary encoded or not, so for my use case I first have to scan the headers of all pages in the a column chunk to assert they're all dictionary encoded, if any of them are not (other than the dictionary page itself of course), I treat the column as not-dictionary encoded, meaning I'll read with a ColumnReader instead of a PageReader and let the library handle the variously encoded pages.


Remove Panopticon importer.

Now that this has taken on the arbitration role, there's no longer a need
to import the data from Panopticon.

Always nice to see a diff that’s pure red :grinning:


Allow explicit Dataset and Fields to be passed as Study inputs

Allow explicit datasets and fields to be passed to Study inputs (in ExplicitDatasets|ExplicitFields objects). Iterables of Dataset|Field objects should be able to be passed as an input when using the dictionary inputs form.
Will need to implement the match(subject_id=None, visit_id=None) method, drawing the appropriate subject and visit ids from the provided datasets (although will typically be of 'per_project' frequency).
This will enable templates (e.g. atlases) to be passed to studies as inputs. A class attribute default_inputs could also be used to specify default templates to use for particular pipelines.

This should be implemented by #53, but hasn't been tested

This is implemented in develop


Unable to run the experiment by slurm

Hi,
I've this issue while running the launcher.py by slurm:
"GLFWError: (65544) b'X11: The DISPLAY environment variable is missing'
warnings.warn(message, GLFWError)"
Can you help me regarding the problem?

Hi,
did you maybe have rendering enabled?

closing this due to inactivity. Feel free to reopen!


Add donkeykong game

Fix issues and add PLE related functions

Also added a LICENSE and removed the menu.

Some of the assets can be removed too, I think the one file with Donkey Kong on it is from the menu? Correct me if im wrong

Fixed most of the problems, could you review again. The game_over function does not seem to be called by PLE, is there something that I am not doing to get it to work?

Yea so PLE exposes a method called game_over(), this is called by an agent or whatever wants to use it. We only provide information and possible actions to the agent

@erilyth It is almost there, I think those changes will get it 99% of the way there. I became busy and didnt get a chance to completely finish. I will test the changes and leave more comments and reviews of it if I see anything else. Thank you for taking the time to do this, its a bit involved :)

@erilyth It is almost there, I think those changes will get it 99% of the way there. I became busy and didnt get a chance to completely finish. I will test the changes and leave more comments and reviews of it if I see anything else. This process also really helps me, I need to update the docs alot more to help people adding games. Thank you for taking the time to do this, its a bit involved :)

I used self.rng.rand() instead of random.random() everywhere and I fixed the issues you mentioned with the images. Review please.

So there are a few bugs, one or two changes needed for PLE, and some style nitpicks. Otherwise the majority is related to performance fixes. Once you adjust it I will start start testing and let you know if anything else is needed!

@ntasfi I made most of the changes you mentioned. Is image.convert() faster than just using scale? Why do we need it?

Scale and image.convert() are two different things. Image convert makes sure the display and image pixel format match. It helps speed up bliting to surfaces. The conversion itself is slow and causes performance issues if its done on the fly, so it is done as you preload the images. I suggest you call convert() or convert_alpha() after loading any image.

So besides one very small non-critical fix, I think its basically there. I will test it a bit, suggest further fixes (if there are any) and we can then merge it!

@ntasfi I added the convert_alphas() and used the dir for the path as mentioned. Review please.

Updated and sent path while creating objects.

Ty @erilyth. So I managed to get the game to run, it looks good! Its pretty difficult haha. It feels a little laggy so I am going to do some profiling and see what tweaks we can do to speed things up. Also the game crashes/closes once the player loses all their lives, can you look into that?

Once the player loses all their lives, I quit the game. Should I restart it?

The game should just keep running. PLE will see via the game_over function the game has ended and will issue a called to init of the game wrapper which will reset the game to a clean state

@ntasfi Made the required changes

The player sprite had convert used on it, the background is showing up as black for me (ignoring alpha). Can you make it use convert_alpha?

No do not create a local copy, .. base is working for me.

Removed the alpha for the background, rest of the images require alpha.

@ntasfi I removed the labels.

Awesome! Testing it with the RandomAgent right now with PLE and it seems to be working after I made those changes I suggest above!
I will merge it after you do those changes. I will most likely do some performance tweaking on it (image loading in different parts etc.) and I also might rename the game and alter the images used as I am not sure what the deal is with some of those images with respect to 'fair use'. So just to be safe!
Good work @erilyth, ty for sticking through the process :+1:

@ntasfi This was initially supposed to be a project for my university (so didn't really look into the 'fair use' aspects of the images). If you do manage to change the images, do let me know, I will update them in my repository as well. Thanks!


fix crash for invokation without args

hi @marcosnils, I just noticed that my patch is making bin crash when invoked without any arguments (should print help instead):
➜  bin git:(hotfix) bin
panic: runtime error: index out of range [0] with length 0

goroutine 1 [running]:
github.com/marcosnils/bin/cmd.defaultCommand(0xc000004900, {0xc000022050?, 0x0, 0xc000012700?})
	/home/marcos/Projects/bin/cmd/root.go:103 +0x1f2
github.com/marcosnils/bin/cmd.(*rootCmd).Execute(0xc0000112c0, {0xc000022050, 0x0, 0xb4f060?})
	/home/marcos/Projects/bin/cmd/root.go:31 +0x71
github.com/marcosnils/bin/cmd.Execute({0xc00007a230, 0x6b}, 0xdf2220?, {0xc000022050, 0x0, 0x0})
	/home/marcos/Projects/bin/cmd/root.go:25 +0xbb
main.main()
	/home/marcos/Projects/bin/main.go:21 +0x97
➜  bin git:(hotfix)

@marcosnils can you also cut a release including this fix please?

done

Cheers :+1: :rocket:


support secondary key in get table without the need of modifying abi file

#3695
New parameter:
--seckeytype: secondary key type valued from i64/i128/i256/float64/float128, this is added in such a way that users don't need to modify their abi files.
--secindexposition: position of the secondary index, default is 0 (first secondary index):
Command example:
Get all the bids sorted by highbid:
./cleos get table eosio eosio namebids -t i64
{
  "rows": [{
      "newname": "zoo",
      "high_bidder": "a123",
      "high_bid": 9990000,
      "last_bid_time": "1528797387500000"
    },{
      "newname": "ddd",
      "high_bidder": "a123",
      "high_bid": 120000,
      "last_bid_time": "1528797387500000"
    },{
      "newname": "abc",
      "high_bidder": "a123",
      "high_bid": 110000,
      "last_bid_time": "1528797387500000"
    },{
      "newname": "com",
      "high_bidder": "a123",
      "high_bid": 100000,
      "last_bid_time": "1528797387000000"
    }
  ],
  "more": false
}

Get all the producers which have votes equal to zero:
./cleos get table eosio eosio producers -t float64 -L "0" -U "0.00001"
{
  "rows": [{
      "owner": "a123",
      "total_votes": "0.00000000000000000",
      "producer_key": "EOS8kE63z4NcZatvVWY4jxYdtLg6UEA123raMGwS6QDKwpQ69eGcP",
      "is_active": 1,
      "url": "",
      "unpaid_blocks": 0,
      "last_claim_time": 0,
      "location": 0
    }
  ],
  "more": false
}

This is direly needed

This feels like a workaround for a mistake: the ABI serializer cleanly distinguishes among types, except for the indexes. Maybe we should revisit that.
"senary"? "denary"? Are these Portuguese words? Italian? Could we abandon using words as numbers?

Might have gotten a little carried away. I'm ok with removing the word support or limiting it to primary, secondary, third.

@tbfleming I agree however, there will be some non-zero amount of time between pushing a new version of the ABI format which includes typed indices and when existing ABIs are migrated to it.  Even if this gets deprecated in the next release, we will have coverage for our users until then.   I do not think this is the last word on the usability of these indices.

not support search by string？

you can use  --key-type name if you want to search by name in string

this is my struct , if i want search by businessName，how to get it

this is my struct , if i want search by name or phone，how to get it
struct chaninfo {
uint64_t channelId;
string name;
string phone;
   }
   uint64_t primary_key() const { return channelId; }

  EOSLIB_SERIALIZE(chaninfo, (channelId)(name)(phone); };

We don't support std::string as key of any indexes. However you can find a way to encode it to 64/128/256 bit integers.

i want to encode string into 64 bit integers by boost，but i get the error when i cleos set contract，so i change the contract，use stringstream and sprintf/sscanf ，also have the same problem，why?

Does the abi have to be updated as well in the key_names and key_types fields?


fix: incorrect return

:tada: This PR is included in version 1.1.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Future.wait track progress of resolved futures

I need to track progress of resolved futures, is there a way to do it? if not, please add such feature.

That's a fairly specialied requirement.
You have a set of futures that you are waiting on, and you want a to somehow get notified when any of the futures complete. I'm guessing for a progress bar?
I wouldn't add such functionality to the platform libraries. It might fit in package:async, say, as a specialized version of FutureGroup which reports on the status of the futures.
Until then, it's very easy to write yourself, and that also allows you to get exactly the API you need for your use-case:
Future<List<T>> progressWait<T>(List<Future<T>> future, void progress(int completed, int total)) {
   int total = futures.length;
   int completed = 0;
   void complete() {
     completed++;
     callback(completed, total);
   }
   return Future.wait<T>([for (var future in futures) future.whenComplete(complete)]);
}


Prevent opening bottom panel when no content

Lock or hide the SlidingUpPanel when it has no content.

Implemented with #44


kth-smallest-number-in-multiplication-table

Problem Link
https://leetcode.com/problems/kth-smallest-number-in-multiplication-table/
Coding platform
Leetcode
Programming language

[ ] Java
[ ] Python

Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
I want to work in Java and python

Hey @Nivedita967 I would like to do it in python. Can you please assign it to me! :)

I want to solve it in C++

Hi @Nivedita967
I would like to solve it in C. Please assign it to me.

If it is open can do it in C++ or python

@Amit366 Go for Java only.

If it is open can do it in C++ or python

Not open.

@Nivedita967 Please check my pull request


OpenAPI spec has minor validation errors

I'm trying to make a Python module off of the XPipe OpenAPI spec using https://github.com/openapi-generators/openapi-python-client, and it's complaining that the keys need to be strings instead of numeric (e.g. this line).  You can see that in the official OpenAPI examples, numeric return codes are quoted so the YAML parser doesn't interpret them as numbers.

I guess I can fix that but I checked with multiple openapi editors and they didn't complain about that. Is there anything else it complains about?

So I fixed that even though it's a little bit weird that it complained in the first place


The launcher/CLI script should be included in the main directory and symlinked to from /usr/bin

See https://github.com/Microsoft/vscode/issues/4398, the main program directory should be complete.

Let's see how it goes with the packages before doing something like this. This is only really an issue if the zip archive is used on Linux.

https://github.com/Microsoft/vscode/commit/cd12a7e389bff8e7ef96a01735e287f536c7f07a


Ensure the Scoper is serializable

Box leverages amphp/parallel which requires the a closure. This closure is gonna be serialized, hence the closure and its content needs to be serializable

Fixed


Looking forward to see the tensorflow version

Hi author,
Are you still planning to release the tensorflow version?
I'm looking forward to see it.
Thank you!

Hi henryace,
yes, we plan to either release a tensorflow or PyTorch version of Deep SVDD in the future.
Thanks!
Lukas

Hi, lukasruff,
Is there a tensorflow version now ? I am also looking forward to it.
Thanks!


Export submodules to ease usage of weaviate.outputs

Also required were a number of seemingly irrelevant pyright fixes due to change in version number on main

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.55%. Comparing base (6516ca4) to head (5fd6293).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1098      +/-   ##
==========================================
- Coverage   94.62%   94.55%   -0.07%
==========================================
  Files         182      190       +8
  Lines       18870    18890      +20
==========================================
+ Hits        17855    17861       +6
- Misses       1015     1029      +14

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Better signature for optional function parameters

Hello, I'm just trying rot.js, and I noticed that several functions have optional parameters that are typed as SomeType | null (like here).
The issue is that TypeScript does not consider those parameters as optional: you either give a value, or an explicit null:

A more correct function signature would be
draw(x: number, y: number, ch?: string | string[], fg?: string, bg?: string) {
/** **/
}

I guess it should be an easy fix (look for all | null in the code), but as I'm literally discovering the lib, I'm not too confident on making a PR for now.

Passing null to an optional argument would work fine though, provided we test those values with simple if (!fg) ....

Sure, and you'll have to do the nullish check anyway because the default value for optional arguments is undefined. Calling draw(5, 4, "@") would be the same as calling draw(5, 4, "@", undefined, undefined).


Multiple templates for one extension?

I am working on a project and write with PHP code different extensions/plugins for it. Thus, each plugin should have a different header comment. Please add an option for template name, so we can have a number of templates for the same file extension and choose which one to insert.

I don't want to require users to select different templates when they want to save a file (which would also have to affect every save where parts of the header can be modified on each save). However, it might be possible to solve this a different way - say through the presence of a specific file in the project directory (maybe even part of package.json) that includes the comment (or indeed any project specific property value) to be inserted for all files within that project. There is already a precedent for this in the extension for getting the project name from package.json.
I will take a look at what is feasible.

In my case, I am talking about a project inside a project. My workspace needs to be in the general folder of the code, while the projects I am working on, and that needs a different header, are in subfolders. Yes, I can create a VSCode workspace in the subfolder for each project(plugin), but in this case, I am missing autocomplete and check for class existence that resides in the main project. Plus, inteliphence also complains about undefined functions etc. However, any option to solve this and have multiple templates will be a good addon. There are other plugins, that have the ability to set template name and choose upon insert, but they lack the option for "last modified time" which is important for some projects.

I am re-opening this to keep track of adding the suggested enhancement from my previous message.
Your comment about lack of last modified in other options is the key to why this extension works differently and decided by file type. To use different templates for the same file type would require something to be embedded into the header to identify which template to use for updates - and that is not desirable.
The bit that may not have been explained very well in my last message is that the suggested new config file(s) can live anywhere and the extension will use the file closest to the source file - so each time a save happens the extension will search for a file in the same directory and work its way up the file hierarchy until it finds one. So you could have a config file within each sub-project branch with its own values which would achieve what you originally asked for.

That will be perfect. And very neat and simple solution BTW.


Update to allow react 18

Hi, seems the repo is dead.
I created a fork and applied your fix here.
It's published to npm under @cyberblast/react-wordcloud. So you could use it with npm add @cyberblast/react-wordcloud.


Check failed: error == cudaSuccess (8 vs. 0)  invalid device function

一经搜索好像是显卡或cuda问题？(gtx1660)
看到类似的解决https://github.com/rbgirshick/py-faster-rcnn/issues/2
望解决~

According to a similar issue under Caffe repo, it is mainly the issue of GPU setting. You can find some solutions here.


Check for the presence of sourceEntryId in case upsert API method

(Check failure is a codecov upload flake.)


Adjust index generation to be 1 based.

Adjust how indicies are generated for fields and enumeration values. The indicies now start with 1 rather than 0. This prevents typical default initializers in C from accidentally being a valid case.
This PR also changes how fingerprints are generated by adding content to each hashed string consisting of the major and minor cauterize versions. This helps prevent cases where specifications generated with two different Cauterize versions could mistakenly resolve to the same hash.

After talking this over with Jay, Job, and Marc, I'm merging this.


Prevent cocktail repeats

When fetching API via the EC2 server, check that the incoming cocktail has not been picked in the past 365 days

Fixed on server side, cocktails can repeat after 6 months

This fix led to incorrect timing of cocktail posting to mongodb, fixed since then. Issue was that I accidentally called the getDate() function twice.


sel

This is an attempt to implement a unit-aware Dataset.pint.sel (no loc for now).
Unlike the implementation in MetPy, the current implementation converts the indexes to the units of the indexers. This is usually slower but closer to the implementation of Dataset.sel, where the indexers become the new indexes. Not sure if speed is more important, though.
cc @jthielen

more complex of an implementation than that in MetPy

actually, I think we can remove most of this once we merge #43. Then, I would expect this to become something like
def sel(...):
    indexer_units = {name: conversion.extract_indexer_units(indexer) for name, indexer in indexers.items()}
    # TODO: handle "tolerance"
    converted = conversion.convert_units(self.ds, indexer_units)
    stripped_indexers = {name: conversion.strip_indexer_units(indexer) for name, indexer in indexers.items()}
    return converted.sel(stripped_indexers, method=method, tolerance=tolerance, drop=drop)

thanks for the review, @jthielen


Create vitbstudent

VIT University, Bangalore
VIT Bangalore
.group
The official website of VIT Bangalore: https://vitbangalore.in/

@itzshivam1999 lib/domains/in/vitbstudent: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@itzshivam1999 Please provide us with a proof, which shows that the school recognizes the domain vitbstudent.in that you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

proof attached in this email
On Mon, Nov 11, 2024 at 12:04 AM Philip Torchinsky @.***>
wrote:

@itzshivam1999 https://github.com/itzshivam1999 Please provide us with
a proof, which shows that the school recognizes the domain vitbstudent.in
http://vitbstudent.in that you are submitting as an official email
domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you
by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that
students use this email domain
A reference to a recognized journal where an article was published
by someone from the school, with explicit affiliation to the school, and
e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that
the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be
verified independently.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/24938#issuecomment-2466839371,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6WQL25MJXU2EJXPEGKJRY3Z76RM3AVCNFSM6AAAAABRGUGAYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWHAZTSMZXGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

proof here ignore previous email
On Mon, Nov 11, 2024 at 7:34 AM Shivam Shivam @.***>
wrote:

proof attached in this email
On Mon, Nov 11, 2024 at 12:04 AM Philip Torchinsky <
@.***> wrote:

@itzshivam1999 https://github.com/itzshivam1999 Please provide us with
a proof, which shows that the school recognizes the domain vitbstudent.in
http://vitbstudent.in that you are submitting as an official email
domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you
by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that
students use this email domain
A reference to a recognized journal where an article was published
by someone from the school, with explicit affiliation to the school, and
e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying
that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be
verified independently.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/24938#issuecomment-2466839371,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6WQL25MJXU2EJXPEGKJRY3Z76RM3AVCNFSM6AAAAABRGUGAYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWHAZTSMZXGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Gmail - VITB Official Email ID and Password.pdf

@itzshivam1999 Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


next.js macro error

Environment
System:

OS: macOS 11.4
CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
Memory: 3.39 GB / 16.00 GB
Shell: 5.8 - /bin/zsh

Binaries:

Node: 14.16.1 - /usr/local/bin/node
Yarn: 1.22.10 - /usr/local/bin/yarn
npm: 6.14.12 - /usr/local/bin/npm

npmPackages:

babel-plugin-styled-components: ^1.13.2 => 1.13.2
styled-components: ^5.3.0 => 5.3.0

Steps to reproduce
The project uses next.js. Introducing styled-components/macro into pages/index.tsx
Expected Behavior
import styled from 'styled-components'
import 'styled-components/macro'

const Title = styled.h1`
  font-size: 50px;
  color: ${({ theme }) => theme.primaryColor};
`

export default function Home() {
  return (
    <Title>
      My page
      <div
        css={`
          color: red;
        `}
      >
        2312
      </div>
    </Title>
  )
}

Actual Behavior

Have you actually set up babel-plugin-macros? This kind of looks to me like the package itself is being bundled and imported, which happens when the macro isn't being executed in Babel


Create a tools/release-all directory to contain release-all.sh and su…

…pporting README.md

Update this part of .gitignore:
# For the release-all.sh tool.
tools/workingspace/
workingspace/

Add tools/release-all/workingspace.
Leave the others in place, because I don't tend to 'cd' down into the tool directory like you all do.


Add left most non-repeating char TesseractCoding#868

Have you read the Contributing Guidelines on Pull Requests?
Yes.
Description
C program solution for finding left most non-repeating character in string.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
issue: #868

@plazzy99 @siddharth25pandey @Kajol-Kumari @iamrajiv please review this.

@all-contributors please add @rajanshoo25 for code


Add support for page link by UID

Currently lepiter supports two formats of page links:

double-bracket, e.g. [[Page Name}}
gtPage annotation, e.g. {{gtPage:Page Name|db=<db uid>}}.

Extend the gtPage annotation to allow the page UID to be specified in the page name.
This will allow non-named pages to be referenced.

Cool - can't wait to try this :)

Works! One small issue... the markup doesn't disappear when it loses focus. I'll create a new issue...


Add pagination to site-local.yaml and site-publish.yaml

Previous | Next
requires
page-pagination: ''
in the asciidoc:
attributes:

Closed with https://github.com/stargate/docs/pull/102


feat(recommend): Increased Performance via Batching

Please Review:
@seenusuvarna
@J-Fid
@skhtet

:+1:


how to access speech data on DFStreamingDetectIntentResponse?

Regarding DFStreamingDetectIntentResponse - does anyone know how to access 'output_audio' (such as with StreamingDetectIntentResponse) or 'speech' property in the response?  I'm seeing what appears to be .wav data as part of the response in the console, but it is in response.unknownFields.  Does anyone know how to utilize this speech response audio in response.unknownFields?

@sjmn Please refer this repository. This sample explains how to use the beta features of dialogflow. Speech data or audio output on DFStreamingDetectIntentResponse is one of the beta features.


Reproducing the result on scannet200 of MSC

Thank you for your excellent work!
It seems that the README section for MSC only includes the guide to reproduce results on ScanNet20:
https://github.com/Pointcept/Pointcept/blob/c4363c86e1e77325645d44bc8052b338031f1b30/README.md?plain=1#L835-L848
How can I reproduce the results on ScanNet200 according to the paper?
It appears that there isn't a config file under configs/scannet200 that starts with "pretrain". Should I directly use the pretrained weights from ScanNet20 and fine-tune it on ScanNet200? If so, it seems that the input_channel of SPUNet under configs/scannet200 is set to 9, while it is set to 6 under configs/scannet, which prevents direct use of a model pre-trained on ScanNet20 for fine-tuning on ScanNet200.
https://github.com/Pointcept/Pointcept/blob/c4363c86e1e77325645d44bc8052b338031f1b30/configs/scannet200/semseg-spunet-v1m1-0-base.py#L18-L19
https://github.com/Pointcept/Pointcept/blob/c4363c86e1e77325645d44bc8052b338031f1b30/configs/scannet/semseg-spunet-v1m1-4-ft.py#L14-L15
Alternatively, should I modify the pretrain config of scannet20 (configs/scannet/pretrain-msc-v1m1-0-spunet-base.py) to support ScanNet200 pretraining?
Please let me know if I am mistaken or how to resolve this issue. Thank you again!

@Gofinge If you can see this, thank you very much!

Hi, sorry for the late response, I just recovered from the busy schedule for the CVPR project.
Yes, I usually copy the config for ScanNet20 to ScanNet200 and then compare the copied config with the ScanNet200 default config to adapt it to ScanNet200. I think only need to change the num_classes to 200 and the dataset type.

Thank you for your response, It is of great help to me! I think this means that for the ScanNet20 and ScanNet200 datasets, I need to pretrain and finetune separately to replicate the results of MSC, rather than using the same pretrained weights from ScanNet20, is that correct?
I am a bit confused about this because I read another CVPR24 paper you were involved in, which is a follow-up to MSC, titled "GroupContrast: Semantic-aware Self-supervised Representation Learning for 3D Understanding". In its experimental section, it references MSC's data and mentions that “For in-domain transfer learning where pre-training and finetuning are both conducted on the ScanNet dataset”. Additionally, the S3DIS experiments were also pretrained on ScanNet and finetuned on S3DIS.
This makes me a bit confused about whether these settings should involve separate pretraining or if a same scannet20 pretrained model should be used.
I’m not sure if there is a misunderstanding on my part. I understand that these works might be from some time ago, I would greatly appreciate it if you could help me out with these relevant details. Please correct me if my understanding is incorrect. Thank you very much! @Gofinge

rather than using the same pretrained weights from ScanNet20, is that correct?

Same pre-training weight for both ScanNet20 and ScanNet200 as well as any other benchmark.
PS: we will soon have a super powerful self-supervised learned PTv3 with ScanNet, ScanNet++, S3DIS, Structured3D, HM3D, ArkitScene etc. Wait for one month, I will make the code and paper available. I think it would be better to follow that one.

That's great, thank you very much!

rather than using the same pretrained weights from ScanNet20, is that correct?

Same pre-training weight for both ScanNet20 and ScanNet200 as well as any other benchmark.
PS: we will soon have a super powerful self-supervised learned PTv3 with ScanNet, ScanNet++, S3DIS, Structured3D, HM3D, ArkitScene etc. Wait for one month, I will make the code and paper available. I think it would be better to follow that one.

Hi, Xiaoyang.
Very interested in 3D self-supervised representation learning and very curious about your novel method. Does this powerful method use the same architecture as PPT and scale to more datasets, or is it a totally different self-supervised method?


Multiple files upload is broken on rack-test

Hi, I was trying to test multiple files upload and noticed it doesn't work well on rack-test. Everything is OK on javascript capybara drivers though.
Here's a repro: https://github.com/wojtekmach/refile/commit/90380c6fb8724c04cd9a2b939d81c6178026a7e5
I guess there're 2 ways to address this:

change rack-test: I guess it would be OK and the behavior would be similar to other drivers like poltergeist
change refile: instead of assuming that the first object in the array (e.g.: {document_files: ["{}", "#<Rack::Test::UploadedFile>", ...]}) is JSON string for cache - make it as a separate parameter, e.g. {document_files: [#<Rack::Test::UploadedFile>, ...], document_files_cache: "{}"}. Seems more elegant to me.

What do you think?

The problem is that RackTest doesn't handle this very well. Nor does Selenium. This is why we use Poltergeist for Refile's own test suite, since it's the only Capybara driver with decent multiple-file support. In fact there are several other issues with RackTest's file implementation which don't work well with Refile. I don't think it's a good idea to work around those in a worse way in Refile itself. Fixing RackTest, or rather Capybara's RackTest driver, is the better solution.

Closing this as there's nothing really to be done within Refile.


T32705 autoprovision

https://phabricator.endlessm.com/T32705

I have a few comments, and some things to fix. Also, may you please run reorder-python-imports src/**/*.py and black on these commits? (Sorry, keep forgetting to add that pre-commit hook here).

Yeah. I would prefer to add the pre-commit now, and format the files with it. I will append that to this PR.

I'm happy with how this looks now, @manuq. Can you let me know if it's ready to merge? I see there's some fixup commits and I can squash those, but I'm wondering if they're there because you're still working on something :)

I'm happy with how this looks now, @manuq. Can you let me know if it's ready to merge? I see there's some fixup commits and I can squash those, but I'm wondering if they're there because you're still working on something :)

Yep it is ready to merge. I only left them as fixups so you don't have to review the original commits again :)
I just appended yet another fixup to make the device name translatable, in the same way as the facility name.

@dylanmccall I just rebased on my end. All the fixes were for the  "kolibri-gnome: Automatically provision the device" commit.


Issues reported by awesome-lint

The awesome-lint program has run for the first time; I wasn't aware of those issues. It seems the link checker was aborting the action.
https://github.com/ohenley/awesome-ada/actions/runs/4267389929/jobs/7428942700
There are some legitimate and important issues to be fixed:

Double links in the Contents (only first one will work).
Double links due to incorrect copy-paste for a book.

Other puristic ones, but easy to solve.
A few looks like there's some bug in awesome-lint.
And finally, other ones that I'm not sure if they should be solved: either because they would affect presentation or because we would lose something, like a second link to the same URL. It doesn't like either the out-of-place links that I precisely included to centralize the URL, so I don't see a solution for those.
I'll try to clean the important and easy ones, and we should discuss the remaining: whether there's a way to fix them, the rule can be disabled or the lint has to be removed altogether.

I directly fixed the non-controversial issues and have made pull requests with the ones requiring review.
These are the only remaining issues if everything is merged:
README.md:1:1:   The repository should have "awesome-list" as a GitHub topic  remark-lint:awesome-github

This can be solved adding the corresponding tag, that can only be done by the owner.
README.md:194:3:   https://matrix.to/#/#ada-lang_Lobby:gitter.im                remark-lint:double-link
README.md:202:112: https://matrix.to/#/#ada-lang:matrix.org                     remark-lint:double-link

These seem like a bug in how the linter interprets URLs. See https://github.com/Scrum/remark-lint-double-link/pull/10

There's still a pending issue reported by the linter:
README.md:1:1
✖  1:1  The repository should have "awesome-list" as a GitHub topic  remark-lint:awesome-github
@ohenley I think this can only be fixed by you.

Yep missed it. Did add "awesome-list" as a GitHub topic.

Great! Everything is passing now, except a timeout in https://www.ada-europe.org/ which I hope is a transitory problem. Closing this, thanks.


Observe is now a level 13 action

I'm using the solver at https://ffxiv-beta.lokyst.net on my low level Weaver.
I noticed that the action Observe is assumed to be a level 1 action, when it is a level 13 action.
diff --git a/app/js/actions.js b/app/js/actions.js
index f062518..0c21929 100644
--- a/app/js/actions.js
+++ b/app/js/actions.js
@@ -27,7 +27,7 @@ function Action(shortName, name, durabilityCost, cpCost, successProbability, qua
 //parameters: shortName,  name, durabilityCost, cpCost, successProbability, qualityIncreaseMultiplier, progressIncreaseMultiplier, aType, activeTurns, cls, level,onGood, onExcl, onPoor
 var AllActions = {
     //                              shortName,              fullName,              dur,     cp, Prob, QIM, PIM, Type,          t,  cls,           lvl,  onGood,     onExcl,     onPoor
-    observe: new Action(            'observe',              'Observe',               0,      7,  1.0, 0.0, 0.0, 'immediate',   1,  'All',           1),
+    observe: new Action(            'observe',              'Observe',               0,      7,  1.0, 0.0, 0.0, 'immediate',   1,  'All',          13),

     basicSynth: new Action(         'basicSynth',           'Basic Synthesis',      10,      0,  0.9, 0.0, 1.0, 'immediate',   1,  'All',           1),
     standardSynthesis: new Action(  'standardSynthesis',    'Basic Synthesis',      10,      0,  1.0, 0.0, 1.2, 'immediate',   1,  'All',          31),

Thanks! Deployed to beta site.


Fix missing dependency in DFP Grafana example

Description

Dependencies for DFP container were recently moved from examples/digital_fingerprinting/production/conda_env.yaml  to conda/environments/examples_cuda-125_arch-x86_64.yaml but python-logging-loki was not included.
Add python-logging-loki to example-dfp-prod in dependencies.yaml.
Remove conda_env.yaml since it's no longer being used anywhere.

Closes #1917
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

/merge


collection.find is very slow and get_by_uuid is deprecated

This code is very slow when used in a big pool +150 servers
connection.servers.find { |server| server.uuid == vm_uuid }

This method is a lot faster, but it's deprecated?
connection.servers.get_by_uuid(vm_uuid)

The shown message is pretty confusing:
[fog][DEPRECATION] This method is deprecated. Use #get_by_uuid instead.

Are you sure are you using fog-xenserver and not fog itself?
If you take a look at this line you will see the methods does not raise any warning and the request implementation here does not also.
In the main Fog gem this method is defined here and it calls the method get_vm_by_uuid that is raising the exception because it is deprecated.
Just use the fog-xenserver and you are good to go. LMK if it does not work for you. Thank!

OMG!!! silly me, xenserver module inside fog is not fog-xenserver... I have update my gemfile to include this, and the deprecation warning is gone.
Thanks!!

 That was kind of a lack of documentation issue.
We are going to remove the support in fog and add fog-xenserver as dependency soon. Until that try to use fog-xenserver instead. =)


Flake8-annotations: fix errors in component

Preparations to add Flake8-annotations plugin.
This fix is only for errors in component folder.
It is assumed that the next flake8-annotations errors will be ignored:

ANN002 - Missing type annotation for *args
ANN003 - Missing type annotation for **kwargs
ANN101 - Missing type annotation for self in method
ANN102 - Missing type annotation for cls in classmethod

Jenkins please retry a build


Huge memory consumption due to default setup of tsconfig.json

TypeScript Version:  any version
Expected behavior: consume normal memory amounts
Actual behavior: consume huge memory while building angular 7 app
Details:
I'm having huge memory issues, trying to publish an angular project with Visual Studio 2019.
Paging file size=3712MB and RAM=9GB. Running Windows 10 over Parallels (Mac)

After checking VS child processes, I noticed that “node server” is the primary consumer of memory (about 4.5 GB alone), due to typescript compilation.
Solution
After modifying tsconfig.json and excluding "node_modules" and "dist", the memory consumption of "Node.js server-side javascript process" went down from 4.5GB to 2GB (about 100%)
...
"exclude": [
        "node_modules",
        "dist"
        ]

Perhaps, tsconfig.json should have those excludes by default

Steps to reproduce:

Use visual studio to create an angular project and publish it.
Check memory consumption during the process
Now change the tsconfig.json and exclude node_modules
publish again and check memory

Of course, the larger the angular project (more dependencies), the clearer the problem appears.
Full details of the source of this issue: https://twitter.com/SeaRyanC/status/1131419372507975680

We'd need some actual project (zip or repo) in order to investigate. In general the file exclusion heuristics should prevent this from happening in VS Scenarios

Perhaps, tsconfig.json should have those exclusions by default or providing the developer some hints when building the project

node_modules is excluded by default. Not sure what you were encountering here.


Importing a CSV without file extension

Turi Create returns an error when importing a CSV that doesn't have a file extension. The below example worked fine after renaming "data" to "data.csv"
RuntimeError: Unable to parse frame index file /Users/xxx/code/Modeling/temp/data
Can we automatically recognize it's a CSV, or at least return a more descriptive error message?

Is it even intentionally the design that we infer file type from extension in the SFrame constructor, despite having static methods for parsing CSV from arbitrarily named files?
Maybe we should take a consistency pass over the API surface

I am not able to reproduce this issue. Calling tc.SFrame.read_csv(...), with a CSV with no file extension, works.

how to solve this issue

@sachika11 - SFrames are saved as folders with several files inside of them. The file you are trying to open looks like one of those files. You should also be using tc.load_sframe(...) rather than tc.SFrame(...). So call tc.load_sframe(...) and pass it the path to the folder with that file.

 how to solve this issue

Use the file with .frame_idx extension and upload all of the files from zipped file.


Cascading visibility control?

Is your feature request related to a problem? Please describe.
I'm fairly new to this lib, so please correct me if I'm missing how to get this functionality already, but I can't find it in the documentation or the code. I'd like to be able to control visibility of whole groups based on containment, and I find the current behavior confusing.
Based on my read of your examples, I add elements to a UIContainer by first creating the container, then create the member elements while setting container=<parent> in their init call.
Then, I set the container's visible attribute to 0 prior to running the main loop, and all those elements are not visible, as expected.
But when I later set the container's visible to 1, nothing appears (the container itself doesn't have graphics), and I can confirm the member elements still have their visible value of 0.
Describe the solution you'd like
I'd like either a change to visible on a parent container to automatically cascade to its members, or for there to be a different method to force that to happen if it's not desirable for that to be the default. If there's a better way to be doing this with the existing code, I'd love to see a whole section added to the docs about organizing containers and controlling their visibility dynamically at run time.
Describe alternatives you've considered
Right now, I have to manually organize which things belong to what and set visibility individually using a function.
Additional context
I do see a add_element method which is not used in the examples, nor is it documented.
I'd be willing to make a PR for the new functionality if it's desirable.

You want to use the show() and hide() methods of the container.
The API documentation is currently not being generated for an unknown
reason but there is underlying documentation for these functions.
Thanks for the report! I think there are areas of the library where it
could make more use of properties as public attributes instead of methods.
My background was more old school C/Java before moving to python so a fair
bit of the library doesn't use python exclusive features like this from my
coding style influence.
Though in this case I didn't write the hide() and show() functionality.
On Thu, 9 Feb 2023, 15:33 Robert Clewley, @.***> wrote:

Is your feature request related to a problem? Please describe.
I'm fairly new to this lib, so please correct me if I'm missing how to get
this functionality already, but I can't find it in the documentation or the
code. I'd like to be able to control visibility of whole groups based on
containment, and I find the current behavior confusing.
Based on my read of your examples, I add elements to a UIContainer by
first creating the container, then create the member elements while setting
container= in their init call.
Then, I set the container's visible attribute to 0 prior to running the
main loop, and all those elements are not visible, as expected.
But when I later set the container's visible to 1, nothing appears (the
container itself doesn't have graphics), and I can confirm the member
elements still have their visible value of 0.
Describe the solution you'd like
I'd like either a change to visible on a parent container to
automatically cascade to its members, or for there to be a different method
to force that to happen if it's not desirable for that to be the default.
If there's a better way to be doing this with the existing code, I'd love
to see a whole section added to the docs about organizing containers and
controlling their visibility dynamically at run time.
Describe alternatives you've considered
Right now, I have to manually organize which things belong to what and set
visibility individually using a function.
Additional context
I do see a add_element method which is not used in the examples, nor is
it documented.
I'd be willing to make a PR for the new functionality if it's desirable.
—
Reply to this email directly, view it on GitHub
https://github.com/MyreMylar/pygame_gui/issues/402, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGDGGUGTL6SJU75IU6ZSKDWWUE5TANCNFSM6AAAAAAUWWMQDQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Add Vowpal Wabbit to benchmarks

I haven't added Vowpal Wabbit to the benchmarks because I am incapable of making it do what I want. I tried to perform a simple logistic regression trained with plain SGD, but to no avail:
from creme import datasets
from creme import metrics
from creme import preprocessing
from vowpalwabbit import pyvw

scaler = preprocessing.StandardScaler()

metric = metrics.LogLoss()

vw = pyvw.vw(quiet=True, sgd=True, learning_rate=.005)

for x, y in datasets.fetch_electricity():
    x = scaler.fit_one(x).transform_one(x)
    y = int(y)
    ex_str = f'{y} | {" ".join(map(str, x.values()))}'
    ex = vw.example(ex_str)
    y_pred = vw.predict(ex)
    y_pred = {True: y_pred, False: 1. - y_pred}
    vw.learn(ex)
    metric.update(y, y_pred)

metric

This should print output a score of 0.413533, as per these benchmarks. Instead if returns a score of 0.641063. I've tried other parameters, such as loss_function='logistic' and link='logistic', but nothing worked.
If someone could get this to work it would be really great.

We could also add a comparison with jax.

Hey @MaxHalford,
I got to 0.427246 using pyvw.vw but unfortunately couldn't get down to 0.413533.
Here is the code (had to change datasets.fetch_electricity() unto datasets.elec2.Elec2(), I am using creme version 0.6.1) :
from creme import datasets
from creme import metrics
from creme import preprocessing
from vowpalwabbit import pyvw

scaler = preprocessing.StandardScaler()

metric = metrics.LogLoss()

vw = pyvw.vw(sgd=True, learning_rate=0.005, loss_function="logistic", link="logistic")

for x, y in datasets.elec2.Elec2():
    x = scaler.fit_one(x).transform_one(x)
    y = int(y)
    y_vw = 2 * y - 1
    ex_str = f'{y_vw} | {" ".join(map(lambda x: ":" + str(x), x.values()))}'
    ex = vw.example(ex_str)
    y_pred = vw.predict(ex)
    y_pred = {True: y_pred, False: 1. - y_pred}
    vw.learn(ex)
    metric.update(y, y_pred)

metric

The differences with your code is the following:

You need to use the options loss_function='logistic' (for logistic regression) and link='logistic' (to output probabilities in [0, 1]).
When using these options the labels must be {-1, 1} and not {0, 1}. If you don't do it you'll see on the console the message "You are using label 0 not -1 or 1 as loss function expects!".
For numerical feature you need to prepend it with {name}: or just : otherwise it will be considered as categorical. For example in your case  vw.get_weight(vw.hash_feature("0.0", 0))  will return non-null float meaning that VW learnt some weight for the categorical "0.0" (which is not the desired behavior).

By the way congrats :tada: for your library, it's quite fresh and easy to use ! Keep up the good work :+1:

Hello @etiennekintzler! Thank you so much for those much-sought clarifications. It's nice to see that the performances are the same. I'll 1) add VW to the benchmarks 2) consider implementing that learning rate schedule. Thanks for the kind words, they always keep me motivated.

Ok I added VW to the benchmarks. I didn't rerun the benchmarks because I'm having some issues with installing VW, but they should work. We can close this for now.

Did you try to pip install it or you used another way ? What error did you get ?
Btw I just check the Linear regression benchmark and the MSE is 3 times higher for VW, which seems quite unlikely. If you want, I can investigate to see if I get the same thing.

I actually installed via conda, and got a segmentation fault.
If you have time to check it out that would be wonderful :). By the way, can you check if setting the adaptive, normalized, invariant parameters to False with logistic regression produces the exact score of 0.413904?
I'll actually leave this issue open until we're 100% happy with the benchmarks.

By the way, can you check if setting the adaptive, normalized, invariant parameters to False with logistic regression produces the exact score of 0.413904?

Yes they do produce the same exact score, as expected since these parameters are not used for SGD (see https://github.com/VowpalWabbit/vowpal_wabbit/wiki/Command-line-arguments#update-rule-options):
--sgd                        use regular/classic/simple stochastic  descent update, i.e., non adaptive,  normalized, non invariant                              (this is no longer the default since it is  sub-optimal)
So other parameters might be at play but I don't know which one.

Regarding the linear regression notebook I got a MSE of 22.8221 with VW (and 23.0371 with creme).
Below is a reproducible example:
from creme import (stats,
                   compose,
                   feature_extraction,
                   datasets,
                   metrics,
                   preprocessing,
                   linear_model,
                   optim)

from vowpalwabbit import pyvw

# Defining pipeline
def add_hour(x):
    x['hour'] = x['moment'].hour
    return x

pipe_prep = compose.Select('clouds', 'humidity', 'pressure', 'temperature', 'wind') + \
        (
            add_hour | \
            feature_extraction.TargetAgg(by=['station', 'hour'], how=stats.Mean())
        )\
| preprocessing.StandardScaler()

# Defining models
lr = 0.005
linmod = linear_model.LinearRegression(
            optimizer=optim.SGD(lr),
            l2=0.,
            intercept_lr=lr
        )
vw = pyvw.vw(sgd=True, learning_rate=lr, power_t=0)

# Defining metrics
metric_vw = metrics.MSE()
metric_creme = metrics.MSE()

# Streaming through examples
for x, y in datasets.bikes.Bikes():

    # Preprocessing
    pipe_prep.fit_one(x=x, y=y)
    x = pipe_prep.transform_one(x)

    # VW
    ex_str = f'{y} | {" ".join(map(lambda x: ":" + str(x), x.values()))}'
    ex = vw.example(ex_str)
    y_vw = vw.predict(ex)
    vw.learn(ex)

    # Creme
    y_creme = linmod.predict_one(x)
    linmod.fit_one(x=x, y=y)

    # Computing metrics
    metric_vw.update(y, y_vw)
    metric_creme.update(y, y_creme)

print("Creme: ", metric_creme,
      "\nVW: ", metric_vw)

Thanks, got it. Good work on the comparison. I do wonder where the discrepancy comes from, but it's so slight that it might just be numeric instability.
Anyway now we have all the steps required to add Vowpal Wabbit to the benchmarks.

Thanks, got it. Good work on the comparison. I do wonder where the discrepancy comes from, but it's so slight that it might just be numeric instability.

Yep I do wonder too haha. I check extensively the command line and did not find what could cause the variation. Also the weight initialization is 0 by default so it's not the cause either (if random it could have been).

Anyway now we have all the steps required to add Vowpal Wabbit to the benchmarks.

Ok great !

So I tried to install Vowpal Wabbit again today and I got some cmake issues that got on my nerves. I'll set up a cloud machine and try it out.

You might want to raise issue on vowpal wabbit project.
Otherwise the cloud machine is a good idea. FYI it works well on my cheap (5$) Digital Ocean VM ; CPU is Intel Xeon E5-2650L v3 @ 1.80GHz, OS is Ubuntu 18.04, and python is version 3.6.9. I installed following instructions from http://vowpalwabbit.org. I can lend you the ressource if necessary.


Release/0.0.x

Codecov Report

Merging #5 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   98.49%   98.55%   +0.05%
==========================================
  Files           9        9
  Lines         199      207       +8
  Branches        8        8
==========================================
+ Hits          196      204       +8
  Misses          3        3

Impacted Files
Coverage Δ

...ain/kotlin/ar/com/agomez/choco/ScalarExtensions.kt
100% <100%> (ø)
:arrow_up:

...main/kotlin/ar/com/agomez/choco/ModelExtensions.kt
100% <100%> (ø)
: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 000e342...d007dce. Read the comment docs.


[doc feedback]A Single VM could be added to ip based backend pool for different SKU ILB at the same time

[Enter feedback here]
Hi Team,
from the doc, you mentioned that the VM in same availability set could be added to different SKU ILB ip based backend pool. However, during testing, even a single VM could be added to different SKU ILB ip based backend pool.
Could you please update the Q&A and add the scenario of single VM to avoid cause customer confusion?
Thanks!

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

ID: 26f2ee4c-633b-8579-47e1-1543db5acaeb
Version Independent ID: 0baa18b9-14ca-1703-ba64-f036ec39b639
Content: Frequently asked questions - Azure Load Balancer
Content Source: articles/load-balancer/load-balancer-faqs.yml
Service: load-balancer
GitHub Login: @mbender-ms
Microsoft Alias: mbender

I got the same question.
Can this be updated in https://learn.microsoft.com/en-us/azure/load-balancer/backend-pool-management as well?

@MaxlerWu , thank you for your feedback. We'll review this and get back to you shortly!

@MaxlerWu can you please elaborate on your verbatim.
Which exact FAQ Question are you referring to here?
I can see the document says that a VM from a single availability set to different backend pools of a load balancer and no mention of different SKU

@MaxlerWu since there is no active discussion going on, I shall close this thread now. Please tag me, should you require further information, we shall reopen the issue and work with you

hi @KapilAnanth-MSFT ,
the doc only mentioned that the VM in one av set could be added behind different SKU ILB. However, during our testing, even single VM that could add to backend pool of both basic&standard ILB at same time. Please update the doc for this scenario since the current description confuse the users.

@MaxlerWu I am afraid I am still not following your comment here.
Please mention the question you are referring to.

This is the only question I see with Availability set and it does not mention any "SKU"

hi @KapilAnanth-MSFT ,
Sorry to make it confuse on this one.
We have worked with Azure PG and PM and got confirmation as below:
1, VM in same availability set could be added to basic ILB and standard ILB ip based backend pool at the same time.
2, single VM could also be added to basic ILB and standard ILB ip based backend pool at the same time.
The above scenario was not supported before, and currently it's supported but no doc mentioned.
As my understanding, it's better to add one more Q&A item for this scenario. Thanks!


Add expiration_date to CreateKey

Proposed changes

Context

Possible Implementation

Other information
https://developers.deepgram.com/reference/create-key

PR posted:
https://github.com/deepgram/deepgram-go-sdk/pull/206

Dup: https://github.com/deepgram/deepgram-go-sdk/issues/205


Grype vulnerability scanner

Repository URL
https://github.com/dtbaum/grypescanner-plugin
New Repository Name
grypescanner-plugin
Description
Grype is a vulnerability scanner for container images and filesystems. This grypescanner-plugin scans a given target and saves a report as job artifact.
There is no such grype plugin yet.
GitHub users to have commit permission
@dtbaum
Jenkins project users to have release permission
tbaum
Issue tracker
GitHub issues

cc @Wadeck

Superficial audit:

GrypeScannerStep.java#L57 and GrypeScanner.java#L21 I am not a big fan of such approach. It means, that once you install this plugin, anyone with write access to https://github.com/anchore/grype will be able to execute code in the build. I imagine that since you install this plugin, we can assume that the admin is trusting the grype project in general and thus this is ok.
💡 GrypeScannerStep.java#L142 seems not what you expect. This method is checking if the build step fits the project type, not the agent where it will be executed on. If you have a Windows controller and a Unix agent, it will wrongly return false. If you have a Unix controller with Windows agent, it will wrongly return true.

So fine to move forward :-)

Thanks, I have addressed both points.
I added an 'autoInstall' checkbox in the UI - the user can choose to use his 'grype' from the PATH or download and install it each time. In my own use case the second variant is an appropriate one, the explanation would be too long :-)
I added some more improvements, looking forward to the feedback!

@dtbaum Thank you very much for the changes. Having the install + existing location is perfectly fine 👍
Also great job on the system detection 👍
@timja Audit complete with success :)

Non blocking comments:

Jelly files are formatted all over the place, would be good to run a formatter over them
Java files do not follow Java formatting standard, we're starting to adopt the spotless-maven-plugin with google format in a few places, something to consider.
Markdown broken in https://github.com/dtbaum/grypescanner-plugin#usage-in-a-pipeline, ensure you wrap code blocks in triple backticks to make it render better
There's TODO's in the issue tracker and contributing section of the README

/hosting host


Update README.md

Indicate that the issue tracker for this repo has moved.

Coverage remained the same at 81.25% when pulling 18c36cd1ea20ef48fb55ebc61123e59064947180 on jcollins-g-patch-1 into 296cd5dadec5008d77205c9a16627bdd808773af on master.


small mistake in the size of the models in train_hello_world_model.ipynb notebook

In the file tensorflow/lite/micro/examples/hello_world/train/train_hello_world_model.ipynb
the tensorflow model size is determined by this code
size_tf = os.path.getsize(MODEL_TF)

while it works for the tfilte files it does not work for MODEL_TF because it is a folder. As you can see in the related picture this function returns 4096 (the size of metadata used by a directory in linux)

I will submit a pull request to fix this.
Thank you

You may find the following useful in creating your PR: https://github.com/tensorflow/tflite-micro/pull/1385/files
This change was made to the automatable training script.


gRPC 0.14.0 migration

I've moved gRPC dependency to root build.gradle file. However, I did not apply gRPC plugin to all subprojects to leave us an ability to enable\disable gRPC code generation. It looks like grpc {} block in protobuf task plugins sections in server and client modules.
Please tell me if we want to move these blocks to root build.gradle too.

[Current coverage][cc-pull] is 87.57%

Merging [#141][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #141   diff @@
==========================================
  Files            99         99
  Lines          2929       2929
  Methods           0          0
  Messages          0          0
  Branches        341        341
==========================================
  Hits           2565       2565
  Misses          288        288
  Partials         76         76

Powered by Codecov. Last updated by [5767d32...d3f9782][cc-compare]
[cc-base-branch]: https://codecov.io/gh/SpineEventEngine/core-java/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/SpineEventEngine/core-java/compare/5767d32ac35c1dec4ac76c672f9064ce9e87fed7...d3f9782726bf41ce905a21f4b26d11d1586ac87b
[cc-pull]: https://codecov.io/gh/SpineEventEngine/core-java/pull/141?src=pr

Is there a way to re-run checks? I can't understand why one of them failed

Yeah, the fourth check always fail. But now there are only three of them and it is OK.

LGTM


Pause game for n seconds

This is another animation option. Though it might look similiar to sleep/wait, it's not. This new option pauses the game (like enemies' movement, mario, his motions and other stuff like the time though maybe not circuits) and the animation can do something when the game is paused. This is useful for dynamic boss fights (for example, having a pillar fall down when mario approach the boss, pausing the game activites is used to make it dramatic. So mario would be stuck until the pillar fully falls down (when the n amount of time runs out, preferably after the pillar fully falls down) and he continues moving in the direction before the pause.)

Moved to #59


Do not pass nil to the assignment mixin

BZ
https://bugzilla.redhat.com/show_bug.cgi?id=1440917
Addressing
 [NoMethodError]: undefined method `base_model' for NilClass:Class  Method:[rescue in _async_generate_table]
app/models/mixins/assignment_mixin.rb:146:in `block in get_assigned_for_target'
app/models/mixins/assignment_mixin.rb:146:in `collect'
app/models/mixins/assignment_mixin.rb:146:in `get_assigned_for_target'
app/models/chargeback/rates_cache.rb:13:in `rates'
app/models/chargeback/rates_cache.rb:7:in `get'
app/models/chargeback.rb:9:in `block in build_results_for_report_chargeback'
app/models/chargeback/consumption_history.rb:31:in `block (2 levels) in for_report'

Introduced in 31fac86a962f16c080f453a44dcd20345b8fd24e
Lesson learned: when you see two lists with [].compact above your edit, it is usually good idea to consider using .compact too.
If we merged https://github.com/ManageIQ/manageiq/pull/14090 we wouldn't see this at customer. 😑 ❗

Euwe backport details:
$ git log -1
commit 7b145d073db2b540dbece9b4064323f6e8e1ad11
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Mon Apr 10 15:47:21 2017 -0400

    Merge pull request #14713 from isimluk/bz-tbd

    Do not pass nil to the assignment mixin
    (cherry picked from commit 17e9829704c2d19dec2787f5c7d11128bfea40b8)

    https://bugzilla.redhat.com/show_bug.cgi?id=1441199

Fine backport details:
$ git log -1
commit bdeaaf101a321e0cee174262b886d08494e79db4
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Mon Apr 10 15:47:21 2017 -0400

    Merge pull request #14713 from isimluk/bz-tbd

    Do not pass nil to the assignment mixin
    (cherry picked from commit 17e9829704c2d19dec2787f5c7d11128bfea40b8)

    https://bugzilla.redhat.com/show_bug.cgi?id=1441198


Update Descriptive ReadMe

Added a descriptive readme file for Blogify.

Repo is being cleaned . No PR's are approved for the time being


Issue running the docker image

issues running and/or building the docker image for multiple architectures and installing chrome and chromedriver (necessary for Selenium).

Build succeeds again, but on Mac systems, a chrome installation is still not found during run.

Previous changes resolved the build failing, but still failed to include a working chrome installation. Issue might be related to path variables not properly being set. Included those. Additionally, switched build to a matrix strategy to better leverage docker's multi-platform support. Bonus upside is that this creates 2 simultaneous build jobs, so should speed up build as well. Issue seems resolved on windows systems. Closing for now.


No text on the About page

About page has no descriptor of the app.

As you can see, your layout appears much different then the current version layout. Maybe you have an older version?

Closing as @ResaWildermuth was testing on the July public site. As mentioned above, the iconography of the ants is explained here.