using nuller and joiner

Seen here, but if I do something like... _ -> [\s]:* {% nuller %} it doesn't work - it says nuller is not defined. I see that you spit out id into the generated code, but not joiner, nuller, or arrconcat. How do you use them?

Ah, interesting, someone must have slipped those guys in with the coffeescript integration. You should just include the definitions in {% %}s and move on with writing your grammar. When I have time, I'll figure out what those weird definitions are doing. :-)

Hehe. Yeah, so all I did was::
@{% function nuller() { return null; } %}
@{% function joiner(d) { return d.join(''); } %}

Works like a charm for now. Thanks.

Glad to hear it!

The docs mention these as "builtins" but not how to include them. I can't seem to use them in my own grammar without defining my own.

They don't exist, is why. You have to add them yourself.

Ok, so either the docs need to be fixed or the nuller needs to be added. Happy to send a PR for either just let me know which one would be preferred.

So we have the following "builtins":

id, which is emitted by generate.js for js/coffeescript/typescript:
output += "function id(x) { return x[0]; }\n";

joiner, arrconcat, arrpush, and nuller, which are used internally (e.g. by the EBNF modifiers), by passing e.g. postprocessor: {builtin: 'nuller'} to a rule.
generate.javascript.builtinPostprocessors = {
    "joiner": "function joiner(d) {return d.join('');}",
    "arrconcat": "function arrconcat(d) {return [d[0]].concat(d[1]);}",
    "arrpush": "function arrpush(d) {return d[0].concat([d[1]]);}",
    "nuller": "function(d) {return null;}",
    "id": "id"

I don't think any of the latter are accessible from within a nearley
grammar. You could perhaps write {% {builtin: 'nuller'} %}, but I don't think
that counts as working as intended!
@Hardmath123 What would you like to do here? Amend the docs; include all of the
built-in postprocessors in every generated grammar; or come up with a
scheme for detecting which postprocessors are used?
I should note that as discussed elsewhere, we want to discourage use of
arrconcat and arrpush (because of icky complexity reasons).

nb. I added a couple of builtin tests in 4826851.

We should amend the docs to remove mention of everything except id. Then we should go and make builtins/postprocessors.ne not suck, and finally document that.

@kach Well, this issue is still not being resolved :-(
I just read the generated JS, found out nuller isn't defined, added one, but... the docs really should be fixed.


Check if playerNode is already connected to mixerNode

Addresses #2541. Also added tests for this.

Cool.


Internal server error while //set dirt

Hello, when I select a zone and then do anything on it, server kicks me, the area get edited but it throw an error. I'm using latest version of PocketMine (that one exited yestersay), but i don't know which version of this plugin.-.
[18:08:49] [Server thread/CRITICAL]: Error: "Undefined property: pocketmine\network\mcpe\protocol\LevelChunkPacket::$chunkX" (EXCEPTION) in "src/pocketmine/network/mcpe/protocol/DataPacket" at line 160

[18:08:49] [Server thread/DEBUG]: #0 plugins/ItemCase.phar/src/aliuly/itemcasepe/Main(281): pocketmine\network\mcpe\protocol\DataPacket->__get(string[6] chunkX)
[18:08:49] [Server thread/DEBUG]: #1 src/pocketmine/plugin/MethodEventExecutor(38): aliuly\itemcasepe\Main->onSendPacket(object pocketmine\event\server\DataPacketSendEvent)
[18:08:49] [Server thread/DEBUG]: #2 src/pocketmine/plugin/RegisteredListener(98): pocketmine\plugin\MethodEventExecutor->execute(object aliuly\itemcasepe\Main, object pocketmine\event\server\DataPacketSendEvent)
[18:08:49] [Server thread/DEBUG]: #3 src/pocketmine/event/Event(95): pocketmine\plugin\RegisteredListener->callEvent(object pocketmine\event\server\DataPacketSendEvent)
[18:08:49] [Server thread/DEBUG]: #4 src/pocketmine/Player(3209): pocketmine\event\Event->call()
[18:08:49] [Server thread/DEBUG]: #5 src/pocketmine/Player(3234): pocketmine\Player->sendDataPacket(object pocketmine\network\mcpe\protocol\LevelChunkPacket, boolean , boolean )
[18:08:49] [Server thread/DEBUG]: #6 plugins/BuilderTools.phar/src/czechpmdevs/buildertools/editors/Filler(143): pocketmine\Player->dataPacket(object pocketmine\network\mcpe\protocol\LevelChunkPacket)
[18:08:49] [Server thread/DEBUG]: #7 plugins/BuilderTools.phar/src/czechpmdevs/buildertools/editors/Filler(169): czechpmdevs\buildertools\editors\Filler->czechpmdevs\buildertools\editors{closure}(object pocketmine\level\Level, integer 322, integer 238, integer 324, integer 239)
[18:08:49] [Server thread/DEBUG]: #8 plugins/BuilderTools.phar/src/czechpmdevs/buildertools/commands/FillCommand(84): czechpmdevs\buildertools\editors\Filler->fill(object pocketmine\Player, object czechpmdevs\buildertools\editors\object\BlockList)
[18:08:49] [Server thread/DEBUG]: #9 src/pocketmine/command/SimpleCommandMap(270): czechpmdevs\buildertools\commands\FillCommand->execute(object pocketmine\Player, string[4] /set, array[1])
[18:08:49] [Server thread/DEBUG]: #10 src/pocketmine/Server(2026): pocketmine\command\SimpleCommandMap->dispatch(object pocketmine\Player, string[9] /set dirt)
[18:08:49] [Server thread/DEBUG]: #11 src/pocketmine/Player(2257): pocketmine\Server->dispatchCommand(object pocketmine\Player, string[9] /set dirt)
[18:08:49] [Server thread/DEBUG]: #12 src/pocketmine/network/mcpe/PlayerNetworkSessionAdapter(245): pocketmine\Player->chat(string[10] //set dirt)
[18:08:49] [Server thread/DEBUG]: #13 src/pocketmine/network/mcpe/protocol/CommandRequestPacket(54): pocketmine\network\mcpe\PlayerNetworkSessionAdapter->handleCommandRequest(object pocketmine\network\mcpe\protocol\CommandRequestPacket)
[18:08:49] [Server thread/DEBUG]: #14 src/pocketmine/network/mcpe/PlayerNetworkSessionAdapter(111): pocketmine\network\mcpe\protocol\CommandRequestPacket->handle(object pocketmine\network\mcpe\PlayerNetworkSessionAdapter)
[18:08:49] [Server thread/DEBUG]: #15 src/pocketmine/network/mcpe/protocol/BatchPacket(123): pocketmine\network\mcpe\PlayerNetworkSessionAdapter->handleDataPacket(object pocketmine\network\mcpe\protocol\CommandRequestPacket)
[18:08:49] [Server thread/DEBUG]: #16 src/pocketmine/network/mcpe/PlayerNetworkSessionAdapter(111): pocketmine\network\mcpe\protocol\BatchPacket->handle(object pocketmine\network\mcpe\PlayerNetworkSessionAdapter)
[18:08:49] [Server thread/DEBUG]: #17 src/pocketmine/Player(3158): pocketmine\network\mcpe\PlayerNetworkSessionAdapter->handleDataPacket(object pocketmine\network\mcpe\protocol\BatchPacket)
[18:08:49] [Server thread/DEBUG]: #18 src/pocketmine/network/mcpe/RakLibInterface(169): pocketmine\Player->handleDataPacket(object pocketmine\network\mcpe\protocol\BatchPacket)
[18:08:49] [Server thread/DEBUG]: #19 vendor/pocketmine/raklib/src/server/ServerHandler(99): pocketmine\network\mcpe\RakLibInterface->handleEncapsulated(string[18] <IP_ADDRESS> 51115, object raklib\protocol\EncapsulatedPacket, integer 0)
[18:08:49] [Server thread/DEBUG]: #20 src/pocketmine/network/mcpe/RakLibInterface(109): raklib\server\ServerHandler->handlePacket()
[18:08:49] [Server thread/DEBUG]: #21 src/pocketmine/network/mcpe/RakLibInterface(99): pocketmine\network\mcpe\RakLibInterface->process()
[18:08:49] [Server thread/DEBUG]: #22 vendor/pocketmine/snooze/src/SleeperHandler(123): pocketmine\network\mcpe\RakLibInterface->pocketmine\network\mcpe{closure}()
[18:08:49] [Server thread/DEBUG]: #23 vendor/pocketmine/snooze/src/SleeperHandler(85): pocketmine\snooze\SleeperHandler->processNotifications()
[18:08:49] [Server thread/DEBUG]: #24 src/pocketmine/Server(2349): pocketmine\snooze\SleeperHandler->sleepUntil(double<PHONE_NUMBER>.051)
[18:08:49] [Server thread/DEBUG]: #25 src/pocketmine/Server(2203): pocketmine\Server->tickProcessor()
[18:08:49] [Server thread/DEBUG]: #26 src/pocketmine/Server(1785): pocketmine\Server->start()
[18:08:49] [Server thread/DEBUG]: #27 src/pocketmine/PocketMine(272): pocketmine\Server->__construct(object BaseClassLoader, object pocketmine\utils\MainLogger, string[37] C:\Users\mrcso\Desktop\PocketMine-MP, string[45] C:\Users\mrcso\Desktop\PocketMine-MP\plugins)
[18:08:49] [Server thread/DEBUG]: #28 src/pocketmine/PocketMine(295): pocketmine\server()
[18:08:49] [Server thread/DEBUG]: #29 (1): require(string[92] phar://C:/Users/mrcso/Desktop/PocketMine-MP/PocketMine-MP.phar/src/pocketmine/Po)
[18:08:49] [Server thread/DEBUG]: [PurePerms] Unregistering player Invy55...
[18:08:49] [Server thread/INFO]: Invy55 left the game

That's caused by ItemCase plugin

Is there any way to keep both the plugins?

I can try debug it, however the best way is to submit this issue to the ItemCase plugin

I can try debug it, however the best way is to submit this issue to the ItemCase plugin

Yeah, but I removed itemCase and the chunks still doesn't render and stop doesn't stop

SOLVED: all the problem were created from a uge schematic in the schematic folder


docs: rewrite the resource deps guide

Hello @lizzthabet, @landism,
Please review the following commits I made in branch nicks/issue4994:
59b5592faab5d1f0d944022707dc16c0e93618c6 (2021-10-19 16:30:33 -0400)
docs: rewrite the resource deps guide
Fixes https://github.com/tilt-dev/tilt/issues/4994
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

trying to re-orient more docs away from "here's a tilt feature and here's how you use it", and more towards "here's a problem you might need to solve and here are all the related tools tilt gives you to solve it"


Update SampleAsyncActionFilter.cs

The existing code sample implies that the user can access the result of the action method in context.Result, but that is not the case. User needs to capture the result of the next delegate in order to access this info.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@ardalis Please review

@irontoby Congratulations on your 1st contribution to the docs, and thank you for fixing this. Your change will go live within the next 24 hours.


*(ticdc):  remove cdc context

What problem does this PR solve?

Issue Number: close #xxx
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

Please also change the package name from context to another more proper name.

@sdojjy Please describe in the PR what problem you are trying to solve, and how do you solve it.

/retest-required

/retest-required

/retest-required

/retest

/retest

/retest-required


Mention that GITHUB_* and RUNNER_* can't be set using GITHUB_ENV (but that CI can be)

Why:
Closes #19626

What's being changed (if available, include any code snippets, screenshots, or gifs):
Adding note about default vars immutability.
https://docs-20069-a70e64.preview.ghdocs.com/en/actions/learn-github-actions/environment-variables#default-environment-variables

https://docs-20069-a70e64.preview.ghdocs.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-environment-variable

Check off the following:

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

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

[ ] This pull request impacts the contribution experience

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

👋 @jsoref Thanks so much for opening a PR! I'll get this triaged for review :zap:

Unfortunately, due to us taking so long to review this, there are now conflicts in these files which need resolved before we can merge this:

content/actions/learn-github-actions/environment-variables.md
content/actions/using-workflows/workflow-commands-for-github-actions.md

@hubwriter: can you deal w/ them? I'm unlikely to know the context of those changes.
If things are simple enough, I can do a rebase and reapply the spirit of the current head of this branch...

@jsoref - I don't have write access to your fork of the docs repo, so I can't fix up the conflicts. I'll close this PR and make the same changes on a new PR.

Changes now published:

https://docs.github.com/en/actions/learn-github-actions/variables#default-environment-variables
https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-environment-variable


shell: Don't check permissions on window

Topic: win2
Reviewers: brian-k

#
head
base
diff
date
summary

0
dbabf973
661636d5
diff
Oct 11 0:49 AM
1 file changed, 1 insertion(+), 1 deletion(-)


Rewrite and reformulate Draft

Adjust draft with respect to the new core message, incorporate a Section of Need and avoid scientific jargon within the introductory section.

Opened branch "paper_draft", modified library and draft in folder "paper". Will promote the drafting to (internal) editorial stage with new issue.


IUploader is not included in heron-spi.jar

The heron spi jar that is built and distributed to Maven Central does not include the com.twitter.heron.spi.uploader package and the IUploader file. These are needed to extend Heron by adding a uploader.

@chris-pardy @kramasamy Thanks for heads up, the 'heron-spi.jar' artifact uploaded to Maven Central is generated by the Heron release script.  We will need to check the release process to include the IUploader file.  I'm discussing with Karthik to resolve, thanks.

I just looked into the spi-jar genrule, only files in utils and common are included in the spi jar.
java_binary(
    name = "spi-unshaded",
    srcs = glob([
        "**/spi/utils/**/*.java",
        "**/spi/common/**/*.java",
    ]),
    deps = utils_deps_files,
)

genrule(
    name = "heron-spi-jar",
    srcs = [":spi-unshaded_deploy.jar"],
    outs = ["heron-spi.jar"],
    cmd  = "cp $< $@",
)

@chris-pardy - this is fixed in the PR https://github.com/twitter/heron/pull/1133 and it will be available in the next release 0.14.2


Report a warning when using props data as items.

Code:
Component.js:
import {SortableContainer, SortableElement} from 'react-sortable-hoc';

let introModuleList = this.props.introModuleList;

const SortableItem = SortableElement(({value}) => {
 return

     <Button onClick={() => this.deleteModule(value.moduleOrder)}>删除</Button>

         {this.getCurrentModule(value)}

;
});

const SortableList = SortableContainer(({items}) => {
 return (

         {items.map((item, index) =>
             <SortableItem key={`item-${index}`} index={index} value={item} />
         )}

 );
});

return <SortableList items={introModuleList} onSortEnd={this.onSortEnd} />;

onSortEnd = ({oldIndex, newIndex}) => {
   this.props.changeModuleOrder(oldIndex, newIndex);
};

Reducer:
import {arrayMove} from 'react-sortable-hoc';

arrayMove(initialState.introModuleList, oldIndex, newIndex)

Result:
warning.js:45 Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op. Please check the code for the SortableList component.

https://github.com/insekkei/react-sortable-hoc/commit/5f3fb69a33760c47ef8f059ec650cab1b789b4af
After remove this.setState({...}) the warning disappeared and everything goes well...

@insekkei would you be able to provide a more complete code sample or share a gist / jsfiddle? It's very hard for me to help you out based only on what you've provided so far, I'd need to see more.

@clauderic The problem here is that when using redux the component may get re-rendered within the onSortEnd Callback. This will unmount the SortableList Component.
Since the Callback is fired before setState is called, the component is already unmounted and the Error  Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op. Please check the code for the SortableList component. will come up.
Is there any reason the provided onSortEnd Callback from props is fired before the setState is used? See here: https://github.com/clauderic/react-sortable-hoc/blob/master/src/SortableContainer/index.js#L290
I think we should simply put that block at the end of the handleSortEnd-Function, just like here (my fork) -> https://github.com/ekrokowski/react-sortable-hoc/blob/51f2c4d3ac763639e66fbd6659cdd8c5fd568cbe/src/SortableContainer/index.js#L302

i'm having the same issue :/ anyone found the problem or the fix to this?

@sag1v Read my comment..
In the handleSortEnd function ->  move this:
if (typeof onSortEnd === 'function') {
	onSortEnd({
		oldIndex: this.index,
		newIndex: this.newIndex,
		collection
	}, e);
}

after the setState (or just to the end of the function)
this.setState({
	sorting: false,
	sortingIndex: null
});

Just like in my fork: https://github.com/ekrokowski/react-sortable-hoc/blob/51f2c4d3ac763639e66fbd6659cdd8c5fd568cbe/src/SortableContainer/index.js#L302

Hmm thanks man, but changing it feels awkward. Why isnt the aithor approve this change? Does it have any side effects?

@ekrokowski
I had the same issue. Thanks for your fix! I tested it and t works for me.
I think you should submit a pull request to the author.

Hey @ekrokowski, thanks for pointing that out. Just shipped 0.4.12 with that change


Additional template list

@matt-bullock need additional graphics for boxes
SEE Grouping in Design chat

All icon SVGs can be found here under 'Crypto'
https://www.figma.com/file/bZh7tOiLm6Ls9Hl4RL6qMC5X/Augur-Redesign?node-id=15700%3A50


Fix AWS named profiles

Description
Refactor the AWS section to use the correct environment variable that the (new) AWS cli uses. Its using named profiles.
It fixes the following issue: #259

Since the AWS_DEFAULT_PROFILE var is completely removed from the awscli i don't expect that to happen. The link in the aws section which links to the documentation of the cli explains this new AWS_PROFILE var.

Maybe @cdimitroulas or @robertomoreno can help us out here. ping 😉

I've been using spaceship with the exact same code change as this PR for a long time and it works great (I know I know... super guilty that I didn't make the PR myself!)
From my perspective this is good to be merged

Merged, Thank you @RolfKoenders @cdimitroulas :clap:


Cannot find module './yarn.lock.cached'

Trying to install via npx create-react-wptheme my_theme but... get getting this:
Cannot find module './yarn.lock.cached'

Hi Benjamin, apologies for the delay.
I just published a new version of create-react-wptheme to NPM. Version 2.0.5.
When you run "npx create-react-wptheme my_theme" you should see it say:
2.1.1-wptheme-20.5
The 2.1.1 is the current version of Facebook's create-react-app, which wptheme uses to scaffold the theme.
If you continue to see that error, it might be that you have create-react-app installed locally on your machine...? And if so, maybe it's an older version?
I wasn't able to actually reproduce your error, so I just removed all the Yarn stuff from create-react-wptheme, as it isn't necessary. So the only thing that is using Yarn now would be create-react-app proper.

This should be fixed. I've basically eradicated yarn from the entire process. NPM only from here on out.


trash-cli: <IP_ADDRESS> -> <IP_ADDRESS>

Result of nixpkgs-review pr 108265 1

2 packages built:

rmtrash
trash-cli

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108265 run on x86_64-linux 1

2 packages built:

rmtrash
trash-cli

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108265 run on x86_64-linux 1

2 packages built:

rmtrash
trash-cli


Allow for optional startup code, such as sending data to workers

There is the occasional need to do special housekeeping before processing, such as sending data to dask workers or setting up dask plugins.   We should allow for code to be set in the config file that can be executed after the dask client is established, but before processing begins.

And now this has become pressing as dask-ssh isn't going to work on multiple machines without this issue being resolved.  That is, I have local "remote" modules that need to use client.upload() to distribute to workers, and there is no convenient way currently to do this.  So I need to add support for the Gremlin config files for optionally specifying "things to do with an active Dask Client before we start the EA."

This is implemented and has been successfully tested.


No DB migration

I installed v3.0 and run yarn install and yarn build, then yarn start.
But DB tables aren't created.
Is this bug?

try running yarn create-all-migrations

I got an error:
yarn run v1.22.5
warning package.json: No license field
$ node scripts/create-migrations
node:internal/fs/utils:343
    throw err;
    ^

Error: ENOENT: no such file or directory, scandir './prisma/migrations'
    at readdirSync (node:fs:1380:3)
    at getFirstDir (/srv/zipline/zipline/scripts/create-migrations.js:13:17)
    at createPSQLMigrations (/srv/zipline/zipline/scripts/create-migrations.js:21:15)
    at Object.<anonymous> (/srv/zipline/zipline/scripts/create-migrations.js:40:1)
    at Module._compile (node:internal/modules/cjs/loader:1095:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1124:10)
    at Module.load (node:internal/modules/cjs/loader:975:32)
    at Function.Module._load (node:internal/modules/cjs/loader:816:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:79:12)
    at node:internal/main/run_main_module:17:47 {
  errno: -2,
  syscall: 'scandir',
  code: 'ENOENT',
  path: './prisma/migrations'
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

the create-all-migrations command is supposed to be run in development environments. Could you show your config/env variables for the first issue?

My config.toml is:
[core]
secure = true
secret = 'secret
host = '<IP_ADDRESS>'
port = 8000

[database]
# postgresql
type = 'psql'
url = 'postgres://zipline:password@<IP_ADDRESS>:5432/zipline'

# mysql
# type = 'mysql'
# url = 'mysql://postgres:postgres@mysql/mysql'

# sqlite
# type = 'sqlite'
# url = 'file:sqlite.db'

[uploader]
route = '/u'
embed_route = '/a'
length = 6
directory = '/srv/zipline/files'

Hi, I have started  building 3.0 from git clone .... Then I can run it successfully.
Thanks!
p.s.
I put directory = $HOME/files in config.toml but Zipline tried to scan $HOME/zipline/uploads. Why?

That's because it only reads from its directory. I wouldn't recommend having it outside of Zipline's directory as Zipline is meant to be run in containers, which wouldn't have this sort of functionality.


the difference of predict result between using xgboost-predictor and using python

Hi,
i train xgboost model with python and save it as 001.model,for test, i use this 001.model to predict one sample in python, get 0.223，but get 0.0604 with using xgboost-predictor，same model,same sample,but different predicted result,and i test another sample, still get different result,only the same thing is using python to predict can get larger score. i have no idea to deal with this problem.note that,my input is a hashmap,like this:{33 : 1.0,34 : 1.0,125 : 0.04261,185 : 0.01504}

i try once more,seems the problem is gone,i just change double format to float format in the input hashmap

@fenxouxiaoquan
Hello, I've actually found the same with standard xgboost4j.
However, when I tryied this library, I got practically the same result as in python (I inserted float[] array)
In xgboost4j I was inserting DMatrix(float[] mydata) as well
How can this be possible?

I would recommend to always be using features as floats. XGBoost is explicit that it treats things as 32 bit due to performance optimizations (one example dmlc/xgboost#1410). If a model has been trained using xgboost its split values will be stored as floats and so giving it doubles may cause inaccurate predictions if hit just the right values.


added octave cheatsheet

What does this PR do?
This PR adds a cheat sheet for Octave language
For resources
Description
Why is this valuable (or not)?
this is valuable for math-cs graduates that work on octave for mathematical plotting and solving math problems
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Loading .p12 certificate: ans1: indefinite length found (not DER)

Hello,
I am trying to use the apns2 module but when I try to load the p12 file I get the following error
ans1: indefinite length found (not DER)

I was able to convert the file to .pem using the command line openssl, and it worked but this is not an ideal solution, I need to be able to use the p12 file directly or convert it to pem at runtime. Sadly the openssl bindings do not offer this functionality as of now. Any suggestions?

Hello,
I'm using the following link as example for the .p12 and .pem files https://www.raywenderlich.com/123862/push-notifications-tutorial I have no idea about the error you getting except the fact that belongs to this https://golang.org/src/encoding/asn1/asn1.go


Warn on unsupported Ruby/Rubygems versions

From: https://github.com/bundler/bundler-features/issues/36

We now do this on 2.0


control which images are used for cellpose

expose a command line arg (and app panel param) that lets users choose which images are used for cellpose anatomical mode.
existing anatomical_only arg does this, so all we need to do is add correlation as an option, and add an app panel to our production capsule(s).

Currently the command line arg --anatomical_only supports  mean and max, but neither correlation nor multi-select

great, updated the issue


Download pictures only from post (Instaloader Module)

WHAT I'M DOING:
So I'm trying to go through through a profile post by post while only downloading the pictures.
MY PROBLEM:
I don't know how to ONLY download pictures from a post. Currently my code downloads pictures, the caption, and a json file. Plus, if my script runs into a Reel, it downloads the first frame for some reason. I'd like to only download pictures.
Any help would be greatly appreciated, since I was having trouble finding how to do this via the module (documentation showed a way to do it via command-line).
MY CODE:
#Main variables
USER = 'instagram_account'
PASSWORD = open(PASSWORD_TEXTFILE, "r").read()
VALID_FILE_TYPES = ['.jpg', '.jpeg', '.png']
PROFILE = 'girls'

#Login to Instagram
L = Instaloader()
L.login(USER, PASSWORD)

#Set profile and attributes
profile = Profile.from_username(L.context, PROFILE).get_posts()
L.download_pictures = True
L.download_comments = False
L.download_geotags = False
L.download_videos = False

chdir(r'C:\Users\music\Downloads\instagram')
counter = 0
for post in profile:
    if counter < 10:
        sleep((rand(10,15)/rand(2,4))*(rand(80,120)/100))
        L.download_post(post, PROFILE)
        counter = counter + 1
    else:
        break

Use L.save_metadata = False to turn off metadata downloading.
Use L.download_video_thumbnails = Flase to turn off video thumbnail downloading.

Try this:
post_url = "https://www.instagram.com/p/"+post.shortcode
L.download_pic(f'static/{user}/{post.date_utc}', post.url, post.date_utc)
this will download single post specific picture


Create and manipulate comments on post data

As a user, I can leave a comment on another user's post

As a user, I can see comments other users leave on my posts

As a user, I can delete a comment on a post

Integrated under: commit 7215d86fa841c20eb5403213ee935c2cd7c75fed .


Is it possible to disable the drag cursor?

This library is awesome, so thank you for your work! It's incredibly fun and easy to use.
I created a grid with two items on each row, and inside of each items i have some forms and a chart. I was wondering if it's possible to disable dragging only in some parts of the item or, if it isn't possible, if i can disable the drag cursor and use the normal one instead.
The problem is that sometimes, while i use the form,  the item is moved, and i would like to avoid that, if possible.

https://jbaysolutions.github.io/vue-grid-layout/guide/properties.html#dragignorefrom
https://jbaysolutions.github.io/vue-grid-layout/guide/04-allow-ignore.html


Refactor grid

This is a major refactor of how the grid module works.
When it is complete, we will be able to support an arbitrary number of axes with different boundary conditions.

Codecov Report

Merging #61 into master will increase coverage by 1.65%.
The diff coverage is 80.24%.

@@            Coverage Diff            @@
##           master     #61      +/-   ##
=========================================
+ Coverage   78.04%   79.7%   +1.65%
=========================================
  Files           2       3       +1
  Lines         123     202      +79
  Branches       28      40      +12
=========================================
+ Hits           96     161      +65
- Misses         16      28      +12
- Partials       11      13       +2

Impacted Files
Coverage Δ

xgcm/duck_array_ops.py
54.54% <54.54%> (ø)

xgcm/comodo.py
64.28% <83.33%> (+5.19%)
:arrow_up:

xgcm/grid.py
86.18% <84.32%> (+4%)
: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 f3022a3...909c980. Read the comment docs.


I can't figure out how to use this.

I copied keyboard.c and .h and draw.c and .h into my project, included keyboard.h, added the char returnedstring[256] and launchkeyboard(&returnedstring) into my code, ran it, nothing happened.Also tried putting all the stuff in your main.c (initOSFunctionPointers, etc) into mine, didn't work, and memoryinitialize () gave me an error, I replaced it with memory_init().

Nevermind, completely unrelated, disregard.


User profile is undefined

I'm following the auth example, but the user profile comes as undefined, although the auth object is fine:
@firebase()
@connect(
  	(state, props) => ({
		user: pathToJS(state.firebase, 'auth'),
		profile: pathToJS(state.firebase, 'profile')
  	})
)

const createStoreWithFirebase = compose(
    reduxReactFirebase(firebaseConfig, {userProfile: 'users'}),
	window.devToolsExtension ? window.devToolsExtension() : f => f
)(createStore);

I've also tried without the devTools stuff, but no luck.
I do have the 'users' node in my Firebase database, although no ID is being created in the 'users' node when using firebase.createUser(credentials, { name.value }), which might be the real cause of the issue...
BTW, in the docs says: "profile if initialized with userProfile support then profile will be saved into ${userProfile}/${auth.uid}", so that means it should create the new key in the 'users' node.
I've also tried to rename userProfile to profile in the store, but it doesn't make any effect:
reduxReactFirebase(firebaseConfig, {profile: 'users'}),

@gazpachu I was having this same problem, looking at the source for reduxReactFirebase it only accepts one object. If you merge the firebase config with { userProfile: 'users' } then it works.


Broken support for special prefixes

https://ruby-doc.org/stdlib-2.5.1/libdoc/readline/rdoc/Readline.html#method-c-special_prefixes
I think this snippet gets us someway there but @rl_special_prefixes seems to be broken.
module Readline
  def self.special_prefixes()
    @rl_special_prefixes
  end

  def self.special_prefixes=(prefixes)
    @rl_special_prefixes = prefixes
  end
end

Hello and thanks for reporting this problem!

@rl_special_prefixes seems to be broken.

I'm afraid I'm not sure what you mean by this. What are you expecting and what result do you get?


The sign up link is not visible with the current background

Very minor, but not to be forgotten.
The current colour of the Sign up link blends too much in the current background.

Yes, this only happens with the wallpaper-02.jpg file though. To avoid future problems I suggest doing what Twitter does:

That is, setting a white background to the div containing the form. This way the link will look just fine regardless of the background image being shown. Note that this also applies to the Sign up page.
What do you think?

Nice idea. However the whole theme is just temporary. @eotchi is going to talk with one of our designers tomorrow. Let's see what is going to happen.

Indeed. I should have some more insights by next week. Let's put this on hold for now.

this will be fixed with the merge of the branch https://github.com/SUSE/Portus/tree/front-end-fixes

This is gorgeous :)

fixed with merge of front-end-fixes


Publish for Scala 2.13.0-M4?

M4 JARs are on Maven Central now

https://github.com/non/kind-projector/pull/71

Yes please :-)

Published version 0.9.7 for Scala 2.13.0-M4.


html fragments -> #marker mount points

This represents a breaking change.
It changes the embed method to support PL outputs without hardcoding the JS version for the quiz viewer in PL.
Instead of using HTML Fragments to embed, it uses named anchor links as mount points.

Closed in favour of #82


PP-4002 Make gocardless_mandates.gocardless_creditor_id not null

WHAT YOU DID

Initialise gocardless_mandates.gocardless_creditor_id with default value LEGACY_CREDITOR_ID to make new mandates correct until all changes complete.
Update gocardless_mandates.gocardless_creditor_id with value LEGACY_CREDITOR_ID to make existing mandates correct until all changes complete.
Add NOT NULL constraint

with @danworth

Yes, we are going to drop DEFAULT in https://github.com/alphagov/pay-direct-debit-connector/pull/248


[HPRO-802] Format imported PatientStatus comments

Q
A

Target Release?
Next available

Bug fix?
✅

New feature?
❌

Database migration?
❌

New configuration?
❌

Composer updates?
❌

NPM updates?
❌

Jira Ticket(s)
HPRO-802

Had considered that as well, but looked to how we were handling it Work Queue as a guide.
I'm not sure there's a glob setting in DataTables we can enable. May result in needing to make a lot of other code changes to ensure it doesn't break anything we've been doing server side previously in other sections of the platform.


Add a test case to investigate #2947

Just a quick PR to see if I can reproduce the issue using the CI.

Seems to be a libpq issue


DO NOT MERGE - REVIEW ONLY [NOD-773] Switch MYSQL to Postgres

Rebuild after update in master

Rebuild after update in master

Rebuild after update in master

Rebuild after update in master


Subscription For all countries Public holidays.

Hi Team,
Currently I am using simple php CalDAV-Client library. I am looking for a country wise public holidays' list. I would like to know, is there any provision to get or subscribe the public holidays in sabre account?
Waiting for your response.
Thanks,
Sagar

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/37378074-subscription-for-all-countries-public-holidays?utm_campaign=plugin&utm_content=tracker%2F294006&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F294006&utm_medium=issues&utm_source=github).

Hi Sagar,
There's no default functionality for this. Would be interesting to offer this one day though!
Mozilla provides calendar data for this:
https://www.mozilla.org/en-US/projects/calendar/holidays/


Add env for workflow and step name

closes #1681

Codecov Report
Patch coverage: 73.33% and no project coverage change.

Comparison is base (6af9371) 39.14% compared to head (d38a8c1) 39.15%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1693   +/-   ##
=======================================
  Coverage   39.14%   39.15%
=======================================
  Files         167      167
  Lines       10760    10763    +3
=======================================
+ Hits         4212     4214    +2
- Misses       6280     6281    +1
  Partials      268      268

Impacted Files
Coverage Δ

pipeline/frontend/yaml/compiler/convert.go
0.00% <0.00%> (ø)

server/pipeline/queue.go
0.00% <0.00%> (ø)

pipeline/stepBuilder.go
81.91% <91.30%> (+0.12%)
:arrow_up:

pipeline/frontend/yaml/constraint/constraint.go
84.36% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Make using v8flags package optional

The v8flags package requires a file to be opened in read/write mode,
we want to be able to run knex cli migrate inside a contains with a read-only file system

@GabrielCastro Crash on node 8 is probably random, rerunning tests should resolve it.
Is it possible to add tests for the change?

I can't seem to find existing tests for the cli, could you point me in the right direction.

@GabrielCastro Doesn't look like any exist at this point; you don't have to cover all of the functionality with tests, just the new stuff should be ok :).

@GabrielCastro You can take a look at https://github.com/tgriesser/knex/pull/2884 for CLI tests, we finally have an example of how to do it!

@GabrielCastro Status update :)?

I'm also interested in this fix.  The not-so-great workaround was to define the environment variable TMPDIR=/dev/shm

@markmsmith Would you consider rebasing the PR?

I'm currently talking with my company's legal to department to get approval to contribute (if the original author doesn't come back).  Do you have any kind of CLA that needs signed?

Nope, we don't have anything like that.

@GabrielCastro This still needs to be documented. Could you please add documentation PR to https://github.com/knex/documentation?

@markmsmith Ping?..

v8flags was removed, so this is no longer relevant


Reinstate google analytics

Fixes #718

coverage: 32.901%. remained the same
when pulling dda17f03246b3fdb793556e7b099245d3950a59b on josh/reinstate_google_analytics
into 9b9a31d46befbd11f53d270f420b164897e779d4 on dev.


[Direct Feeds] [Backend] Update the policy violation for broken card connection to include data properties to identify the correct violation

This is part of the Third party workspace feeds V2 (direct connections) project.
Implement this section of the design doc.

PR merged!


Support for private repos

This feature will greatly benefit us. I would like to help with this.
I've created a fork and deployed to heroku but getting an error:
org.apache.jasper.JasperException: Unable to compile class for JSP:
An error occurred at line: 1 in the generated java file
The type java.util.Map$Entry cannot be resolved. It is indirectly referenced from required .class files
I guess this have some dependencies I'm missing can you please help me with that ?

Fantastic, thank you so much for offering to help! :+1:

So does this work now? After taking a quick look at the pull request, it seems like @motiko got it working for private Github repos?
Side thought: I would love to get this working with bitbucket. @motiko or @afawcett , any chance you could point me in the right direction for adding this support? I would love to contribute to get bitbucket supported with this tool.
Having the ability to just one-off deploy a package to an org is awesome; only catch is that github private repos are too expensive (aka not free like bitbucket)

It will once merged, still working on tweaking a few implementation details, but its looking really good!
Yeah the BitBucket API's look good, great idea.
I think we would want to build in some abstraction layer for authenticating and getting at a source control repo though, as the main class would start to get to complex i'm sure. So perhaps start with agreeing what such an interface might look like, then move out the GitHub code, implement the interface and write a small factory. We could maybe host it under another URL (e.g. bitbucketsfdeploy) with a Heroku environment var determine which source control factory to use. Thoughts gents?

Ok, after a bit of playing around this morning i got this deployed. Not sure if its the best strategy retrospectively, but i have two Heroku apps one for production and one for sandbox to support this tool. As the libraries i was using didn't allow dynamic switching of the SF end points. So since the GitHub oAuth callback is now being used, i had to create a new Connected App in my GitHub config. Anyway.... It all seems to be working now... Thanks so much to @motiko !


Authentication-results message header: dmarc bestguesspass - explanation incomplete

hello,
under the section "Authentication-results message header fields" under the field "dmarc" the different results of DMARC checks are explained.
For "bestguesspass" it says the following:

Indicates that no DMARC TXT record for the domain exists, but if one had existed, the DMARC check for the message would have passed.

This is true.
Then it continues with:

This is because the domain in the 5321.MailFrom address (also known as the MAIL FROM address, P1 sender, or envelope sender) matches the domain in the 5322.From address (also known as the From address or P2 sender).

This is only true in case the DMARC check has passed with SPF-alignment only.
However an email can also pass DMARC validation with DKIM alignment alone or SPF and DKIM alignment combined.  These cases are not covered in this explanation.
Therefore the current explanation suggests that only SPF-alignment is accepted as "bestguesspass" which is definitely not true.  Adding DKIM alignment to the paragraph would make the total explanation even longer, therefore I would suggest to simply remove the second part of the description.

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

ID: e47e11dd-53ab-5f62-ac6c-59dbc00eed0d
Version Independent ID: 362ad038-3b64-f342-2f8e-111635c4337d
Content: Anti-spam message headers - Office 365
Content Source: microsoft-365/security/office-365-security/anti-spam-message-headers.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@rawIce, you state the paragraph that you want added, and I'll add it.
Or, you can add it directly as described in https://github.com/MicrosoftDocs/microsoft-365-docs/blob/public/README.md

I have created this PR #5741

PR has already been merged. Issue closed.


refactor(api): Allow aspirate, dispense and blow-out in place in PE and PAPI

Overview
closes https://opentrons.atlassian.net/browse/RCORE-540.
Allow PE and PAPI via engine core to preform aspirate, dispense and blow-out in place.
Test Plan
Test on an OT-2 with core ff on or on an OT-3.
upload the following protocol:
from opentrons import protocol_api, types

metadata = {
    "apiLevel": "2.12",
}

def run(ctx: protocol_api.ProtocolContext) -> None:
    tip_rack = ctx.load_labware("opentrons_96_tiprack_300ul", 6)
    labware = ctx.load_labware("nest_96_wellplate_100ul_pcr_full_skirt", 3)
    pipette = ctx.load_instrument("p20_single_GEN2", types.Mount.RIGHT, [tip_rack])
    well = tip_rack.wells()[0]

    # offset the well to something obvious
    # to observe that the offset is not respected
    well_offset = types.Point(x=9, y=-9, z=0)

    pipette.pick_up_tip(well.top().move(well_offset))
    location_no_well = types.Location(point=labware["A1"].top().point, labware=None)
    pipette.move_to(location_no_well)
    pipette.aspirate()
    pipette.dispense(volume=20.0)
    pipette.blow_out()
    pipette.drop_tip()

make sure the pipette does an in-place command without a move to.

Changelog

Add PE aspirate and blow-out in place commands.
Added sync_client methods to aspirate, dispense and blow-out in place.
Wired up engine/core aspirate, dispense and blow-out to use in-place commands when well_core is None.
refactored instrumnt_context and added a method to return wether we are passing a Well or a Location to the core's.

Review requests
Does the changes make sense?
Did I forget anything for the PE commands?
Risk assessment
Low. Should act the same as the legacy papi. Should test with core ff on and off to make sure it behaves the same

Tested the aspirate-in-place and dispense-in-place on an OT2 with core ff on. will test blow-out-in-place

Tested blow-out in place on an OT2 with ff on. tested dispense in place on dev server. need to test on a robot before merging

Test on OT2 with apiLevel 2.14 and 2.12. works as expected!


RC8 release execute error on Windows

Error:Execution failed for task ':app:preDexDebug'.

com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command 'C:\Program Files\Java\jdk1.8.0_40\bin\java.exe'' finished with non-zero exit value 1

...while parsing com/github/pires/obd/commands/control/DistanceTraveledSinceCodesClearedCommand.class
A think this error is occuring on windows because of the huge-length name of this DistanceTraveledSinceCodesClearedCommand.class.
The Windows has a 260 character path length limit.
Please refactor it's name to a shorter one. e.g: DistTravSinCCCmd.class
Hope it helps!

Yes, this has been identified by other users and I'm very glad to review a PR that fixes it.

Ok, I will be wainting for the release.
Thank you!

Pull request made.


CNDB-11508 main-5.0 branch

What is the issue
We cannot:

compile and run main-5.0 on JDK22
run tests on JDK22 with the current config

What does this PR fix and why was it fixed
...

Adding test fixes based on testing we already did in a POC branch; enables us to build and run Cassandra on JDK22 plus run all test suites on JDK22 from the main-5.0 branch. We also added numerous fixes that get CI to a very good state on JDK22. CI runs posted on the GH issue.
apply a few fixes for JDK17 to bring CI to a good shape too. CI runs posted on the GH issue.

Checklist before you submit for review

[ ] Make sure there is a PR in the CNDB project updating the Converged Cassandra version
[ ] Use NoSpamLogger for log lines that may appear frequently in the logs
[ ] Verify test results on Butler
[ ] Test coverage for new/modified code is > 80%
[ ] Proper code formatting
[ ] Proper title for each commit staring with the project-issue number, like CNDB-1234
[ ] Each commit has a meaningful description
[ ] Each commit is not very long and contains related changes
[ ] Renames, moves and reformatting are in distinct commits

CI is in a bad shape today going up and down and I am not being able to really check test results.
I am pushing some review feedback updates to docs and JDK17 test fixes and I will move the PRs to draft so they do not trigger useless CI triggers. I will move to another issue I am working on and check back later.

looks good. we should remove all the byte buddy -D, they should not be needed. possibly upgrade to the latest byte buddy to prevent needing to touch it again later.

Please check below comment:
https://github.com/datastax/cassandra/pull/1408#discussion_r1840883671

Feedback addressed. @jacek-lewandowski , I believe the main question here is:
let me know what you want me to do with this one - https://github.com/datastax/cassandra/pull/1398#discussion_r1850776852
Option 1: remove it from both main and main-5.0 and diverge from Apache (I am against that);
Option 2: remove the deletions in main and open follow up ticket to improve that target for all branches? (I prefer this one)

Thank you for the reviews! There are a few conversations about how we can improve/change certain things like generate-idea-files ant target or test JDK properties - we agreed in Slack we can tackle any of that in follow-up tickets if we want to improve things further, but the current work is already good enough to be committed as-is. I just pushed the rebased branch. To save resources, I will wait to confirm the JDK 11 PR gate looks good, and then I will push the JDK17 and JDK22 last runs if there are no new issues. As this is primarily a build and testing patch, it should not affect CNDB, and I do not plan to run a test on CNDB before merging.

JDK11 CI finished successfully.
There are a few test failures marked new. They all do not seem related, and all but two I cannot reproduce, seems like flakies.
The two I can reproduce, I can reproduce also on main-5.0 . They are part of - ClientRequestMetricsLatenciesTest
Moving forward to the additional testing I will have to push manually. I publish links on the main issue.


be more consistent with GameStatus

checkmate, stalemate, etc.. should take a Color

Add NotStarted as a GameStatus


Need your confirmation about availability of "entitlement management" in Azure China portal

Hi writer team,
In the article I can see that in "License requirements" segment, it says "Specialized clouds, such as Azure Germany, and Azure China 21Vianet, aren't currently available for use".  Does it mean that "entitlement management" is unsupported in Azure China?  Our customers are confused about the description as they can see and click "Identity Governance" blade in "Azure Active directory" service in Azure China portal. Can you help to confirm the availability of "entitlement management" in Azure China portal? Thanks.
Regards
Alex

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

ID: 06201faf-0037-3647-4e91-bb0af80d4812
Version Independent ID: 1b397d76-ecfe-046e-a4c4-0fe5293bceb3
Content: What is entitlement management? - Azure AD - Microsoft Entra
Content Source: articles/active-directory/governance/entitlement-management-overview.md
Service: active-directory
Sub-service: compliance
GitHub Login: @OWinfreyATL
Microsoft Alias: owinfrey

@alexchen2016
Thanks for your feedback! We will investigate and update as appropriate.

Any update from your side?

Any update from your side?

Hello Alex,
I am looking into this issue now. I have noticed that sentence is very old, and includes a service(Azure Germany) which is no longer offered, and I see that Azure AD is fully supported on our matrix page for China here.  I will be back to you with a definitive answer by the end of the week :)

Hello Alex,
I have confirmed this statement is no longer applicable in regards to china cloud and am removing that statement from the document.
thanks!

Thanks for your reply

@OWinfreyATL
can you confirm if this issue is resolved. I will go ahead to close it.

#please-close


Not able to install sonarwhal version 0.26.0

Hi Team,
I am getting an issue in installing the latest version of the sonarhwal.
/ # sonarwhal -v
v0.26.0
/ # npm -v
5.5.1
/ # node -v
v8.9.3
This was working perfectly fine with 0.25.0 version . But now it is not working .
Error
/ # sonarwhal --init
Welcome to sonarwhal configuration generator
? What connector do you want to use? jsdom
? What formatter do you want to use? stylish
? Do you want to use the recommended rules configuration? Yes
Using recommended rules
? What browsers are you targeting? Default (last 2 versions of each browser, and browsers with globaly usage over 1%, plus Firefox ESR)
Error: npm ERR! code ENOLOCAL
npm ERR! Could not install from "" as it does not contain a package.json file.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2018-02-21T17_16_32_031Z-debug.log
at installRules (/node_modules/sonarwhal/dist/src/lib/cli/init.js:31:19)
at exports.initSonarwhalrc (/node_modules/sonarwhal/dist/src/lib/cli/init.js:130:11)
at <anonymous>

Something went wrong installing the packages, please run:
sudo npm install  -g
to install all the rules.
/ # sonarwhal http://www.google.com
Error: Resource apple-touch-icons not found
at Object.exports.loadResource (/node_modules/sonarwhal/dist/src/lib/utils/resource-loader.js:117:15)
at rulesIds.reduce (/node_modules/sonarwhal/dist/src/lib/utils/resource-loader.js:140:30)
at Array.reduce ()
at Object.exports.loadRules (/node_modules/sonarwhal/dist/src/lib/utils/resource-loader.js:139:28)
at Sonarwhal.initRules (/node_modules/sonarwhal/dist/src/lib/sonarwhal.js:127:38)
at new Sonarwhal (/node_modules/sonarwhal/dist/src/lib/sonarwhal.js:79:14)
at exports.analyze (/node_modules/sonarwhal/dist/src/lib/cli/analyze.js:93:25)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
at Function.Module.runMain (module.js:678:11)
at startup (bootstrap_node.js:187:16)
at bootstrap_node.js:608:3
Can you please help in resolving this issue.
or is their any way i can rollback to older version (0.25.0) for now and continue my testing?
Thanks,
Hitesh

@hgakhr Until we fix that, you need to choose No for the option Do you want to use the recommended rules configuration.

You can install it via npm install -g<EMAIL_ADDRESS>

Closing as doing npm install -g sonarwhal should now point to v0.25.0`, which works.


feat: allow for custom action mapping function

Provides the ability to provide a custom mapping function for actions when the default behavior is not ideal. This provides a route that fixes #327 by allowing a custom mapping function to define naming criteria.

This also appears on the surface that it would solve #306

Thanks for your contribution! ❤
However, I found this approach a bit overkill in terms of fixing the no plural form and fallback naming issue.
I would like to not introduce yet another way to provide customization by config/configuration like custom functions in custom files.
I prefer to be consistent and stick with the comment "directives" in the prisma schema file to provide custom plural form of the model name for some conflicting words.
/// @@TypeGraphQL.plural(form: "equipments")
model Equipment {
  id String @id @default(cuid())
}

I have done it on my local branch, will try to add tests and docs and publish new release today 😃

Directives are fine to me, yes, my solution was overkill was looking for a way to provide the ability to overwrite any name that was generated which was why I went the custom mapping route, but this solves my direct problem, look forward to seeing it in the codebase!


Fid DB Search Results columns are not resizeable

While the FID Db Search window is open, I can't resize any columns in the Fid DB Search Results Table. Perhaps due to modality?
Can this be changed, and/or is it by design?

Mine seem to be resizable when the FID Db Search window is open.  Sometimes columns are resizable but the cursor doesn't change when you hover over the right spot.  Could that be the issue?

The cursor is changing but nothing happens when I click and try to drag it either direction.

After closing and re-opening Ghidra I can't reproduce it either. I had had some errors logged during that session that might have interfered with the GUI.
However, the minimum width for many of the columns is overly wide. It's hard to imagine numbers of 18 digits or so (a rough estimate of what would fit currently) in the Code Unit Size or Spec. +Size columns.
I can easily imagine the Name column needing to be wide, but currently its minimum width is excessive.

And while we're at it (but probably material for another issue), only being able, in the Search Results window, to Edit only one function at a time is really limiting if you have a few dozen likely matching functions. (That shouldn't be the case, I suppose, but I'm still working at getting these function IDs straightened out.)

I've requested an improvement to column widths in issue 2989


:package: feat(monorepo): Add new config package for evmts.config.ts (#91)

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #93   👈

PR #94

This comment was auto-generated by Graphite.


translate all examples in 08_Math

Addresses #[646]
Changes:
Translated all examples in 08_Math, src/data/examples/zh-Hans/08_Math/*.
Problem:
10_Interpolate.js, not sure how to translate the last sentence in description.

Looks great! Thanks for all your effort.


Fix the helloworld example by importing the network.js file (PR 8617 follow-up)

Fixes https://stackoverflow.com/q/45393309/5317012

Thank you for fixing this!


om bosh-env / get-credentials / etc. feedback

related: #290
As admins, in order to more-efficiently manage credentials and bosh targets in multiple environments, we are using eval $(om bosh-env) more. We also are finding a lot of value in om --env my-env.ml credentials <xyz> Here is some feedback.
I can break these out into separate issues if preferred?
ability to logout (unset) env vars
bosh-env exports helpful env vars. But without unset'ing them, you may inadvertently target the wrong foundation or credhub. Unsetting the env vars require clearing the shell with the following:
unset BOSH_CLIENT
unset BOSH_CLIENT_SECRET
unset CREDHUB_CLIENT
unset CREDHUB_SECRET
unset CREDHUB_SERVER
unset CREDHUB_CA_CERT
unset BOSH_CA_CERT
unset BOSH_ENVIRONMENT
unset BOSH_ALL_PROXY
unset CREDHUB_PROXY

It would be valuable for om to have a logout analogy which unsets any of the env vars it leverages, so that eval $(om bosh-env-logout) (or some equivalent) could give reassurance that the env is clean.
Or perhaps simply om logout, which unsets any/all env vars, not just the CREDHUB_* and BOSH_* vars? Or it could be made more specific with om logout --bosh --credhub, etc.?
selectively export BOSH_* keys, not CREDHUB_*
Some of our automation tasks use om bosh-env to get the director credentials, but in the same script will use a call to a different credhub than the bosh director's credhub... But om bosh-env will have the environment vars targeting the director's credhub.
It would be valuable to be able to specify "only export BOSH_*" params, so that the existing CREDHUB_* params aren't clobbered. E.g.,
om bosh-env -e some-auth-file.yml \
  --ssh-private-key=some-private-key-used-for-socks-proxy.pem \
  --bosh-only

# or --credhub-only?

... or something to that effect.
login flow
One of the only downsides of using om for interfacing with om is having to have either basic auth user credentials or client credentials on my machine in plaintext for specifying the --env auth rather than being able to leverage an --sso workflow equivalent.
feedback, general
I really like the om bosh-env command as the preferred way to target environments. It's also very valuable for extracting credentials such as a uaa admin client via om -e myenv.yml credentials --product-name=cf --credential-reference=.uaa.admin-client (or something like that)
Thanks for implementing it, and I hope it continues to get love as the way to get credentials from opsmgr, both for automation && for operators.
Thanks for your time and consideration.

Awesome! Thanks for the great feedback and suggestions:

Would the proposed logout behavior completely clear the environment, or would you expect this to act like a login / logout stack? (i.e. if there was a previous environment set before using a om bosh-env command, this env would be restored after logout?) Or would we expect the user to be required to always run eval $(om bosh-env) (or some equivalent) right afterwards to set a new env?

We have recognized that there are some gaps in using credhub with platform automation. This appears to be another one. It makes sense that you wouldn't want the pre-exisiting CREDHUB target to be swallowed by the new eval.

I'm not sure how an SSO workflow would happen in om. Would this be key based? Can you clarify or show a workflow?

Thanks again for the awesome feedback. We will definitely continue to optimize the om bosh-env. This feedback is probably good in here for now; we will discuss this and we will get back to you if we need these in separate issues for some reason!

@aegershman apologies for the late reply on this one! could you expand some more on some of the questions above?

@kcboyle no that's my bad for missing this, I'll write up a response in the next day or two 👍 standby

@jpmcb sorry for delay in writing this up my bad

I don't think I'd expect the previous session to be restored, because the way eval $(om print-env) is used already assumes you're passing in the om creds each time: there isn't really a standalone om login which sets a "session". If a user runs eval $(om bosh-env) it exports the values into their current shell env, but it's almost always performed as eval $(om --env sandbox-aws.yml bosh-env...) or eval $(om --client-id foo --client-secret bar bosh-env...), etc.

I don't think there is any other way to get the env vals into the current shell from a subshell except for eval... Including unsetting env vars. So doing om logout (or whatever it's named) would have to be ran as source $(om logout), where logout would allow for the env values to be cleared.
TL;DR, if eval $(om bosh-env...) is used, I'd expect source $(om unset-bosh-env) (or `$(om logout) or whatever would be most appropriate?)  to clear env vars and have a clean shell slate.
The only other way I could (maybe?) see this working would be is if om were used as a "prefix" to issue certain commands, like om --env sandbox-aws.yml bosh -d service-instance foo xyz, e.g. having om act as the auth pass-through to issue bosh or credhub params but only for the context of that request. That way when you do om <auth method like --client-id> bosh <bosh-commands>, the lifespan of acquiring the bosh creds would exist only in that one om call, and the user's environment would remain completely clear.
This is kinda-sorta how the hammer cli works, where you can use hammer bosh to show bosh creds (and thus could do eval $(hammer -t sandbox bosh) to export the values into the user shell) or pass-through the command with hammer -t sandbox bosh <bosh commands>, and the user's env would remain clear. The awesome thing about that is then you can have multiple targets in automation or on the operator's machine, and don't have to worry (as much) about the env; you can namespace each bosh env with the --target or --env flag, similar to fly -t or bosh -e, etc.
It's not a massive priority though, I'm just speculating.

Again, not a huge deal, I like the fact that it gets access to the bosh director's credhub at all since otherwise it'd be kind of a pain. It's okay that it clobbers the current CREDHUB_* env, it'd just be nice to have a convenience method to unset those env vars so I could log back in again. No reason to get fancy with restoring previous env values, etc. This could also benefit from using om as a "passthrough", e.g. om --env sandblorp.yml credhub find -n /foo, because then I could use the "outer" env to get a credhub value from , and set it to the director credhub via om without worrying about clobbering the "outer" env, and without worrying about the env being "tainted" with the credhub targeted with om. Once the om command completes, that's it.

It'd have to use https://pcf.example.io/uaa/login to get an sso token from the currently authenticated user to allow an om request to go through. E.g., I'm logged in to the opsmgr console, and when I do eval $(om --env sandbox.yml bosh-env) or eval $(om --target https://pcf.example.io --sso bosh-env) or something similar, it'd halt and request a token obtained from https://pcf.example.io/uaa/login before giving me the commands. That way as a human user (not automation) I don't need to store client creds in a local env.yml, I can just rely on however I'm auth'd against the opsmgr (like SAML/LDAP/etc.)... Someone smarter than me should evaluate this workflow, though. Again, not essential, just an idea stolen from uaa/cf/fly CLIs.

thoughts? no rush to respond 👍

Thanks again for the suggestions, @aegershman.
Let's go ahead and break this out into separate issues. We're reviewing om issues and as it stands, there's too much going on in this one for us to get any traction. If you wanna break it out into the three elements and link back to this one, we'll probably be able to get somewhere on at least some of them.
We'll probably break this up ourselves, at some point, but at this very moment we're headed into a meeting, so - it's a race! And we're deploying the "slow and steady" strategy.

split these out into three issues. The first two seem reasonable nice-to-haves. The last I'm a little more iffy about, so I threw a "discussion" label on it. Let's talk about the technical implications on that one, and whether it is feasible there.
Closing this issue. Thanks for the discussion @aegershman ! and apologies for the delay in responses!


pretty yaml serialisation preserving comments

As noted on readme and lib.rs, catalog add uses prettier for formatting of the yaml, and isnt comment preserving.
https://github.com/xbcsmith/ymlfxr/issues/2 was my last foray into comment preserving yaml in Rust.
yaml-rust2 writes pretty yaml, but isnt comment preserving.
https://crates.io/crates/tree-sitter-yaml might be comment preserving.
c.f. https://github.com/tree-sitter-grammars/tree-sitter-yaml/blob/master/src/node-types.json#L483
But that adds a dependency on a C compiler, and is a lot more effort to read and write.
https://github.com/pantoniou/libfyaml is likely the best route, and rust wrappers are improving https://crates.io/search?q=libfyaml

Oh boy. My earlier attempt at this got me sniped pretty hard!
I actually did get started with https://lib.rs/crates/yaml_parser which creates rowan concrete syntax trees, but editing those was incomprehensible to me due both to confusing rowan docs and the bizarre structure that it parses. Maybe there's more juice down that path, but the squeeze was no fun.
Inevitably, I thought "surely it's not that hard to write a yaml parser," and my fate was sealed. That said, I think I got it up to recognizing a decent chunk of the test suite with code gen from the grammar repo, but ran into similar lack of docs and it's annoyingly not in sync with the actual spec text, so then i saw it was generated from the spec markdown...
Ugh.


Fix docs

Coverage remained the same at 40.892% when pulling 59f91abea0cb04d1c395d03218bd4e2355005a5d on der-lyse:fix-docs into c89659a5a7b8e601896da90f3cc64f22129a411d on newsboat:master.

Thank you!
I merged without waiting for CI to pass because docs aren't tested by CI anyway, and I don't think you could break any syntax by moving things around. (AppVeyor CI failed because I removed the project from there just now.)


Print functionality layout and quirks

This small pull request slightly changes the print layout by placing the copyright on the bottom with a small © symbol.

A new function also tries to intercept the print event while viewing a cocktail, redirecting the user to the print page.

Nice, lgtm.
Thanks!


Android account hold and restore features

I've come across this worrying message in the Play Store:

Reminder: You must support account hold and restore in your app by November 1, 2020. Subscription pause and resubscribe will also be enabled by default unless you explicitly opt out.

What does that mean for react-native-iap? Are these features supported by the latest version?
These are my current subscription settings, i'm not sure if this is compliant with this warning:

Could someone help me out understanding this better ?

Go check : https://developer.android.com/google/play/billing/subs#lifecycle and https://developer.android.com/google/play/billing/subs#account-hold
I tried to work of that this morning. Here's what I did :
Back-end

Modified back-end subscription check function to compute the "On Hold" status and return failure + message to the user (according to the documentation)

Mobile app

On subscription check from the mobile, display alert message when the new back-end response occurs and lock all content (== disable subscription)

Play Console

Enable "Account hold" for all users

I'm definitively not sure that this is the solutionn but this is what I'm going to do. I think the warning message is about enabling the "Account hold" into the play console settings. Just be sure that your app can handle properly an account with a subscription that would be on hold (message + blocking subscription features).

Go check : https://developer.android.com/google/play/billing/subs#lifecycle and https://developer.android.com/google/play/billing/subs#account-hold
I tried to work of that this morning. Here's what I did :
Back-end

Modified back-end subscription check function to compute the "On Hold" status and return failure + message to the user (according to the documentation)

Mobile app

On subscription check from the mobile, display alert message when the new back-end response occurs and lock all content (== disable subscription)

Play Console

Enable "Account hold" for all users

I'm definitively not sure that this is the solution but this is what I'm going to do. I think the warning message is about enabling the "Account hold" into the play console settings. Just be sure that your app can handle properly an account with a subscription that would be on hold (message + blocking subscription features).

That's not right. It means we have to incorporate account hold and restore functionality in the app as well. I have the same feature in some of the apps lately.

Go check : https://developer.android.com/google/play/billing/subs#lifecycle and https://developer.android.com/google/play/billing/subs#account-hold
I tried to work of that this morning. Here's what I did :
Back-end

Modified back-end subscription check function to compute the "On Hold" status and return failure + message to the user (according to the documentation)

Mobile app

On subscription check from the mobile, display alert message when the new back-end response occurs and lock all content (== disable subscription)

Play Console

Enable "Account hold" for all users

I'm definitively not sure that this is the solution but this is what I'm going to do. I think the warning message is about enabling the "Account hold" into the play console settings. Just be sure that your app can handle properly an account with a subscription that would be on hold (message + blocking subscription features).

That's not right. It means we have to incorporate account hold and restore functionality from the app as well. I have seen the same functionality in some of the apps recently, not sure how to to do this with RNIAP.

Sorry I did not add enough details : my app already have a restore function so I forgot to mention it and I only talked about "account on hold". I've just edited my previous post to correct it.
What to you mean by incorporating account hold ? Do you have the name of the apps you are mentioning ?
Restore : Restoring purchases is a pretty common IAP process and I think RNIAP supports restoring since the very beginning. There's a ton of documentation everywhere.
Account hold : from my understanding, account holding is just a way to keep user on the hook of the subscription, it's past grace period, supposed to notify the user, so I guess that functionality just supposed to offer a new chance to the customer to update its payment method without having to subscribe again. The account is "hold" during 30 days where the customer is prompt to update its payment method. My guess is that Google now makes account holding mandatory just to keep more customers using IAP transactions because it's obvious that this will improve sales.
Please, correct me if I'm wrong.

Hey @looggi thank you for your answer, i'm looking into in right now. Do you know if there is a way to fake a payment failure? I want to see what the API returns in that case.

what happens if you just set it to "Enabled" but you don't handle the "On hold" response in your app? That is of course not my goal but i'm wondering if that would affect the app's behaviour at all...

@clems36 I'm just guessing but I would not be surprised if the required part will concerns just the Play Console settings. I don't think they will review the app to simulate an account on hold situation and see if you've implemented some specific response.
Guys, again, please correct me if I'm wrong. I'm in the same situation I need my app and subscriptions to work after November 1st
I didn't manage to simulate account hold so I blindly followed the specifications here : https://developer.android.com/google/play/billing/subs#lifecycle

We're in the same boat. I agree with you, i don't think they will review the app neither, just check if the setting is enabled.
I can't find how to simulate an Account Hold situation.
The linked you've shared points to native response codes but (correct me if i'm wrong) we don't know how react-native-iap handles these.
Could you show an example of how you handled the response in your js code?

I use react-native-iap but I do back-end checks to validate the subscriptions while calling direclty Apple and Playstore services server to server so technically react-native-iap doesn't have a role here.
Account hold seems to be a status returned when you're going to check if your customer is a subscriber (== check subscription)

Ok thanks for your help. Would love to know exactly what is returned by RNIP in an Account hold case. I'll keep the issue open until somebody from RNIAP sees this.

Thanks for the important discussion. I've tried exposing more variables from the new android SDK which is v3, and exposed obfuscatedAccountIdAndroid and obfuscatedProfileIdAndroid in #1171 but nothing more than that. Would this be needed? I currently need to investigate this but I also think there isn't anything that could be provided in react-native-iap but rather it looks like the work is needed in the backend.
Hope I can have better clarification to help you around.

More information here:
https://stackoverflow.com/questions/64303962/you-must-have-account-hold-and-restore-implemented-in-your-app-by-november-1-2/64318101#:~:text=Play Console saying%3A-,You must have account hold and restore implemented in your,I am confused about this.
and here: https://android-developers.googleblog.com/2020/06/new-features-to-acquire-and-retain-subscribers.html
There doesn't seem like we need to do anything else from the rn-iap side, you have to check your subscription and see if it's still valid on your end. Closing as stale, if more discussion needed, please reopen or even better, open a discussion instead of an issue.


[Test] Rename substitution so it isn't incorrectly substituted

%swiftc_driver is a substitution already, which means %swiftc_driver_stdlib_target was actually being substituted as %swiftc_driver first with -stdlib-target tacked on the end.
Resolves rdar://106170241 and rdar://106170343.

@swift-ci please test


add Do func to execute after selection , with example.

Give package user an ability to add Do func() in Select{} object to execute after selection.

you can find simple example how to use, the example about get selected next weekday date.
please feel free to ask for more update.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

What is the use case for this over having a function defined outside the struct and using Run()?

@tarekbadrshalaan, I'm wondering if you are able to update this PR so we can merge it? Thanks!

thanks @enmand for your reply, I will try to work on it tomorrow.


TypeScript typings?

I'd love to use react-socks, but I'm working on a TS project, and this library doesn't have typings. Any chance of adding them? dts-gen may be able to automatically create during build-time.
Update: managed to adopt react-socks by adding generating react-socks.d.ts using dts-gen -m react-socks, then adding the output to my typings folder (wrapped as a module declaration: declare module "react-socks"{output}), but it would be nice if this was included.
Update 2: this just didn't work in my app for some reason, probably related to general strangeness of my project. Resorted to window.outerWidth >=, which will do fine :)

@alirobe – I created a temporary shim until the maintainers build typings.
Replace the breakpoint names with your own:
types/react-socks/index.d.ts
declare module 'react-socks' {
  import React from 'react';

  export function setDefaultBreakpoints(breakpoints: any[]);

  export const BreakpointProvider: React.StatelessComponent<{}>;

  interface Props {
    handset?: boolean;
    tablet?: boolean;
    desktop?: boolean;
    widescreen?: boolean;
    superwidescreen?: boolean;
    up?: boolean;
    down?: boolean;
    only?: boolean;
  }

  export const Breakpoint: React.StatelessComponent<Props>;
}

@alirobe @drewlustro TypeScript is a bit out of my depth. Are you interested in figuring this out? I'm open to reviewing the PR with some help.

@alirobe @drewlustro TypeScript is a bit out of my depth. Are you interested in figuring this out? I'm open to reviewing the PR with some help.

I'm down, but I'm not 100% familiar how to create typings with dynamic props. This might be a fun PR though. I'll be sure to let you know if I find time in the next week.


Release commit for Fabric CA v1.4.8

Add release notes v1.4.8.
Signed-off-by: David Enyeart<EMAIL_ADDRESS>

+1 Looks good to me! :)


v1.22.0 ambient inject iptables rules in pod failed for CentOS 7

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
When adding a pod into ambient mesh, istio-cni-node log the follwing err:
24-05-14T09:44:22.086970Z     info    ambient in pod mode - adding pod ambient-demo/sleep-75bbc86479-vc6ss to ztunnel
2024-05-14T09:44:22.087458Z     info    iptables        Running iptables-legacy-restore with the following input:
\* mangle
-N ISTIO_PRERT
-N ISTIO_OUTPUT
-A PREROUTING -j ISTIO_PRERT
-A OUTPUT -j ISTIO_OUTPUT
-A ISTIO_PRERT -m mark --mark 0x539/0xfff -j CONNMARK --set-xmark 0x111/0xfff
-A ISTIO_PRERT -s <IP_ADDRESS> -p tcp -m tcp -j ACCEPT
-A ISTIO_PRERT ! -d <IP_ADDRESS>/32 -p tcp -i lo -j ACCEPT
-A ISTIO_PRERT -p tcp -m tcp --dport 15008 -m mark ! --mark 0x539/0xfff -j TPROXY --on-port 15008 --tproxy-mark 0x111/0xfff
-A ISTIO_PRERT -p tcp -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT
-A ISTIO_PRERT ! -d <IP_ADDRESS>/32 -p tcp -m mark ! --mark 0x539/0xfff -j TPROXY --on-port 15006 --tproxy-mark 0x111/0xfff
-A ISTIO_OUTPUT -m connmark --mark 0x111/0xfff -j CONNMARK --restore-mark --nfmask 0xffffffff --ctmask 0xffffffff
COMMIT
\* nat
-N ISTIO_OUTPUT
-A OUTPUT -j ISTIO_OUTPUT
-A ISTIO_OUTPUT -d <IP_ADDRESS> -p tcp -m tcp -j ACCEPT
-A ISTIO_OUTPUT -p tcp -m mark --mark 0x111/0xfff -j ACCEPT
-A ISTIO_OUTPUT ! -d <IP_ADDRESS>/32 -o lo -j ACCEPT
-A ISTIO_OUTPUT ! -d <IP_ADDRESS>/32 -p tcp -m mark ! --mark 0x539/0xfff -j REDIRECT --to-ports 15001
COMMIT
2024-05-14T09:44:22.087510Z     info    Running command (with wait lock): iptables-legacy-restore --noflush -v --wait=30
2024-05-14T09:44:22.090028Z     error   Command error output: Warning: Extension CONNMARK is not supported, missing kernel module?

node os version:
CentOS Linux 7 (Core)   3.10.0-1160.el7.x86_64
node iptables version:
iptables --version
iptables v1.4.21

connmark module is loaded
$ lsmod | grep connmark
xt_connmark            12755  103
nf_conntrack          139264  12 ip_vs,nf_nat,xt_state,nf_nat_ipv4,nf_nat_ipv6,xt_conntrack,nf_nat_masquerade_ipv4,nf_nat_masquerade_ipv6,nf_conntrack_netlink,xt_connmark,nf_conntrack_ipv4,nf_conntrack_ipv6

Version
$ istioctl version
client version: 1.22.0
control plane version: 1.22.0
data plane version: 1.22.0 (5 proxies)

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.0", GitCommit:"ab69524f795c42094a6630298ff53f3c3ebab7f4", GitTreeState:"clean", BuildDate:"2021-12-07T18:16:20Z", GoVersion:"go1.17.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.0", GitCommit:"ab69524f795c42094a6630298ff53f3c3ebab7f4", GitTreeState:"clean", BuildDate:"2021-12-07T18:09:57Z", GoVersion:"go1.17.3", Compiler:"gc", Platform:"linux/amd64"}

Additional Information
No response

I think you need xt_connmark https://istio.io/latest/docs/setup/platform-setup/prerequisites/#kernel-module-requirements-on-cluster-nodes. Note that doc will need to be updated to say it's needed for ambient as well

oops, missed where you said you have that

cc @bleggett

This is an extremely old version of linux/iptables, and centos7 is ~1 month away from EOL.
It might be because the container iptables binary and the host kernel are just too far out of skew.
I'll see if I can repro locally with Centos7

Hi @escoffier - given centOS 7 is going to be end of life soon, could you reproduce it on a newer centOS version?  Also, do you have this prob for sidecars?

I can't repro this in a centos7 Docker image, with iptables 1.4.21:
[root@d48fd0199421 /]# iptables -t nat -N ISTIO_OUTPUT
[root@d48fd0199421 /]# iptables -t nat -A ISTIO_OUTPUT -p tcp -m mark --mark 0x111/0xfff -j ACCEPT
[root@d48fd0199421 /]# iptables -t nat -A ISTIO_OUTPUT ! -d <IP_ADDRESS>/32 -p tcp -m mark ! --mark 0x539/0xfff -j REDIRECT --to-ports 15001
[root@d48fd0199421 /]# iptables --version
iptables v1.4.21

Naturally, since it's a docker image, the underlying kernel is much newer (6.8) - so I do think this is likely a kernel problem, not an iptables problem, or an istio problem.
Since 3.10 has been EOL'd for years, and CentOS has been keeping it alive in CentOS 7 by backporting fixes, it's entirely possible they didn't backport something we need, it's a very nonstandard kernel by this point.
@escoffier unless you can repro this with a kernel/OS that's not on the verge of being EOL'd, we probably aren't going to fix this.

I can't repro this in a centos7 Docker image, with iptables 1.4.21:
[root@d48fd0199421 /]# iptables -t nat -N ISTIO_OUTPUT
[root@d48fd0199421 /]# iptables -t nat -A ISTIO_OUTPUT -p tcp -m mark --mark 0x111/0xfff -j ACCEPT
[root@d48fd0199421 /]# iptables -t nat -A ISTIO_OUTPUT ! -d <IP_ADDRESS>/32 -p tcp -m mark ! --mark 0x539/0xfff -j REDIRECT --to-ports 15001
[root@d48fd0199421 /]# iptables --version
iptables v1.4.21

Naturally, since it's a docker image, the underlying kernel is much newer (6.8) - so I do think this is likely a kernel problem, not an iptables problem, or an istio problem.
Since 3.10 has been EOL'd for years (~2017), and CentOS has been keeping it alive in CentOS 7 by backporting fixes, it's entirely possible they didn't backport something we need, it's a very nonstandard kernel by this point.
@escoffier unless you can repro this with a kernel/OS that's not on the verge of being EOL'd, we probably aren't going to fix this.

thanks， i‘am trying to upgrade kernel.

Hi @escoffier - given centOS 7 is going to be end of life soon, could you reproduce it on a newer centOS version? Also, do you have this prob for sidecars?

Thanks for responding. it works fine on newer centOS version and for sidecar.

Thanks @escoffier!  Seems we should document this as a limitation for centOS 7 under https://istio.io/latest/docs/ambient/install/platform-prerequisites/?  PR would be welcome!

Thanks @escoffier! Seems we should document this as a limitation for centOS 7 under https://istio.io/latest/docs/ambient/install/platform-prerequisites/? PR would be welcome!

No, we need to update https://istio.io/latest/docs/setup/platform-setup/prerequisites/#kernel-module-requirements-on-cluster-nodes as @howardjohn mentioned.
Also, this isn't ambient specific, really.

Doc PR: https://github.com/istio/istio.io/pull/15121

Closing this as WONTFIX for centos7 specifically - doc PR is merged.


[MRG] Improve gamma example by adding new viz methods

This PR

improves the narrative of the gamma example
adds viz.plot_tfr_morlet: utility for plotting TFR of single-trial dpl time course (uses MNE and SciPy)
adds viz.plot_spectrogram: utility for plotting Welch periodogram of single-trial dpl time course (uses SciPy)

Visualisation functions are poorly-tested, and input validation is largely missing. However, the readability of the example has improved dramatically, IMHO.

@jasmainak I don't get why the new plot_tfr_morlet and plot_spectrogram don't appear as hyperlinks in the examples? Some init file I need to include them in? They're in api.rst
@ntolley take a look at @2218c9c : you could use the new function in your alpha/beta demo?
For a 0.1-release, perhaps the missing parameter checks are an acceptable omission...?

@jasmainak I don't get why the new plot_tfr_morlet and plot_spectrogram don't appear as hyperlinks in the examples? Some init file I need to include them in? They're in api.rst
@ntolley take a look at @2218c9c : you could use the new function in your alpha/beta demo?
For a 0.1-release, perhaps the missing parameter checks are an acceptable omission...?

Thanks @jasmainak ! Any idea how I can get plot_spectogram and plot_tfr_morlet to be clickable in the example?

Thanks @jasmainak ! Any idea how I can get plot_spectogram and plot_tfr_morlet to be clickable in the example?

Any idea how I can get plot_spectogram and plot_tfr_morlet to be clickable in the example

I think it will be fixed once we merge this PR :-)

Any idea how I can get plot_spectogram and plot_tfr_morlet to be clickable in the example

I think it will be fixed once we merge this PR :-)

/Users/mainak/anaconda3/lib/python3.7/site-packages/scipy/signal/spectral.py:1969: UserWarning: nperseg = 8192 is greater than input length  = 8000, using nperseg = 8000
.format(nperseg, input_length))

I get this warning when running the example. Ideally, examples shouldn't throw warnings so user is not distracted.

/Users/mainak/anaconda3/lib/python3.7/site-packages/scipy/signal/spectral.py:1969: UserWarning: nperseg = 8192 is greater than input length  = 8000, using nperseg = 8000
.format(nperseg, input_length))

I get this warning when running the example. Ideally, examples shouldn't throw warnings so user is not distracted.

Here's another warning I get:
/Users/mainak/Documents/github_repos/hnn-core/hnn_core/viz.py:378: MatplotlibDeprecationWarning: Adding an axes using the same arguments as a previous axes currently reuses the earlier instance.  In a future version, a new instance will always be created and returned.  Meanwhile, this warning can be suppressed, and the future behavior ensured, by passing a unique label to each axes instance.
  cbar_ax = fig.add_axes([l + w + 0.05, cb_b, 0.03, cb_h])

Here's another warning I get:
/Users/mainak/Documents/github_repos/hnn-core/hnn_core/viz.py:378: MatplotlibDeprecationWarning: Adding an axes using the same arguments as a previous axes currently reuses the earlier instance.  In a future version, a new instance will always be created and returned.  Meanwhile, this warning can be suppressed, and the future behavior ensured, by passing a unique label to each axes instance.
  cbar_ax = fig.add_axes([l + w + 0.05, cb_b, 0.03, cb_h])

I'll fix the MPL deprecation warning tomorrow and figure out a solution to the nperseg warning too (my version didn't throw that one...). As for mne-dependency... I agree we should probably remove deep ones like that, most likely copying relevant parts into hnn_core, as you mentioned. But perhaps acceptable for now (0.1)?

I'll fix the MPL deprecation warning tomorrow and figure out a solution to the nperseg warning too (my version didn't throw that one...). As for mne-dependency... I agree we should probably remove deep ones like that, most likely copying relevant parts into hnn_core, as you mentioned. But perhaps acceptable for now (0.1)?

yes, we can do it after 0.1. My comment was mostly to say that let's add a line in the tutorial asking them to install mne for the particular example.

yes, we can do it after 0.1. My comment was mostly to say that let's add a line in the tutorial asking them to install mne for the particular example.

happy to merge once the warnings are fixed!

happy to merge once the warnings are fixed!

Alright, the last three commits should reduce the warnings and includes mention of mne. Please check that this is the case.
Note that the new plt_show method copied from mne should suppress warnings in all examples!
I'm done here!

Alright, the last three commits should reduce the warnings and includes mention of mne. Please check that this is the case.
Note that the new plt_show method copied from mne should suppress warnings in all examples!
I'm done here!

Mine ran with a reasonable execution time regardless of decimation as well, ~39 ms with decimation vs. ~295 ms without. For simplicity's sake, I'm not convinced we need an extra line defining the decim variable (just like we don't define a separate var for n_cycles) but that's stylistic.

Ah, just noticed this. Maybe my macOS renderer is just inefficient. Feel free to remove the decim's from the example before merging!

Mine ran with a reasonable execution time regardless of decimation as well, ~39 ms with decimation vs. ~295 ms without. For simplicity's sake, I'm not convinced we need an extra line defining the decim variable (just like we don't define a separate var for n_cycles) but that's stylistic.

Ah, just noticed this. Maybe my macOS renderer is just inefficient. Feel free to remove the decim's from the example before merging!

Thanks a ton @cjayb for these improvements!

Thanks a ton @cjayb for these improvements!


changes required for Starling's render texture and stencil masking

Moving merge request into develop branch

this is missing the other changes you made, look at the files changed.

my mistake


Specifying GNUPLOTTERMINAL="x11" in important_settings gives terminal qt in gnuplot files

When I specify
# Which gnuplot terminal to use by default.
GNUPLOTTERMINAL="x11" # other options are "aqua" and "x11"

in the file important_settings, the first line in .gnuplot files produced by TDEP reads
set terminal qt  size 500,350 enhanced font "CMU Serif,10"

So it does not respect my setting.

Yes, the mechanism actually works a bit different through precompiler flags: https://github.com/tdep-developers/tdep/blob/ac38d93c3e5e468d3160508264ff2ae04e219345/src/libolle/konstanter.f90#L142-L155
I fear there is no direct way to specify the terminal.
I recommend to run phonon_dispersion_relations -p which creates a outfile.group_velocities.gnuplot_pdf from which you can create a outfile.dispersion_relations.pdf and view it (that's what I do all the time anyways)


Support for Django 1.11.x

Installing on new django 2.1.1 will downgrade it.
Can this widget still work if compiled?

What do you mean with "compiled" ? Javascript asset minification ?

compiled as in call python setyp.py with modified code...
Currently the project is aimed at Django 1.X which is OLD.. But I guess it may simply not work until the project is verified to work with Django 2.x.

You could try support with the tests, just changing the tox config in a development environment: https://djangocodemirror.readthedocs.io/en/latest/development.html
But i'm pretty sure it won't work, since Django 1.11 there was some incompatible changes that breaks the widget i should have fixed before even thinking to Django 2.x

are these fixes planned for some point in time? or is the project considered abandoned?

It's on the way, i got some tests to fix with Django 1.11, Django 2.0 should not take more time. This should be released to a new major version at the end of the week.

Who hoo?!
Thanks for the amazing update :-)
Going to try it now!


GH-3277 : Fix coroutine detection login on MessagingMessageListenerAdapter

Motivation:

https://github.com/spring-projects/spring-kafka/issues/3277
isCoroutines is a method unit, but on the determineInferredType method it determine by each parameter type. So I think that determine by parameter type is more nice way to improve it

Modifications:

Change variable naming isCoroutines to isKotlinContinuation variable
Change the coroutine detection logic

DIscussion

Feel free to discuss about the test code improvement way

I add my name on @author to affected class. Thanks


Fixing react-magma-dom version

Codecov Report

Merging #653 (f9a06c5) into main (1fc0bbc) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #653      +/-   ##
==========================================
- Coverage   89.20%   89.19%   -0.02%
==========================================
  Files         174      174
  Lines        4226     4222       -4
  Branches     1406     1403       -3
==========================================
- Hits         3770     3766       -4
  Misses        232      232
  Partials      224      224

Impacted Files
Coverage Δ

...t-magma-dom/src/components/Accordion/Accordion.tsx
100.00% <ø> (ø)

...a-dom/src/components/Accordion/AccordionButton.tsx
100.00% <ø> (ø)

...ma-dom/src/components/Accordion/AccordionPanel.tsx
100.00% <ø> (ø)

patterns/header/src/components/header/Header.tsx
87.50% <ø> (-0.74%)
:arrow_down:

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 26cd441...f9a06c5. Read the comment docs.


Start reorganisation of ChIP-seq

Hi,
I started the reorganization of ChIP-seq directory.
To do:

[ ] Move the current pdf presentations to reveal.js
[ ] Add images
[ ] Build the Docker image
[ ] Test correctly the tutorial
[ ] Add the input datasets on Zenodo
[ ] Add a Galaxy tour

Bérénice

I think the templates are ready for tomorrow hackathon :)

We are getting closer! Thanks a lot!