[alpha12] You don't take falling damage after using Sky Soarer even if you don't enable Friendly Earth modfier

Oops.


ValueError: Negative dimension size caused by subtracting 2 from 1 for 'max_pooling2d_2/MaxPool'

Hey,
I am getting the error mentioned above when trying to run this code:
from keras.utils.vis_utils import plot_model model_vgg = get_image_model(CNN_weights_file_name) model_vgg.add(MaxPooling2D(pool_size=(2, 2), dim_ordering="th")) plot_model(model_vgg, to_file='model_vgg.png')
Any ideas how to fix this?
thanks, simon

@simon-gunacker  What version of Keras and Tensorflow are you using?

Hi, I've got the same problem, long error list on
model_vgg = get_image_model(CNN_weights_file_name)
which derives from: image_model = VGG_16(CNN_weights_file_name)
I have TF 1.2 and Keras 2.05, downloaded VGG16 Weights, as mentioned in the readme. Any idea?
Thank you, Mike

Hi,
solved it myself already, for anyone interested, it's a problem with keras implementation of VGG16 weights.
Just add
`from keras import backend as K
K.set_image_dim_ordering('th')`
at the top!
Greetings, Mike

@michael20at Thanks - Working now.


Cannot get to boot "Execution of embedded linux image failed: Out of Resources"

ZFSBootMenu build source
Recovery EFI
ZFSBootMenu version
2.3.0
Boot environment distribution
Portable
Problem description
I've tried the Portable and the Ubuntu install guide, and both result in "Execution of embedded linux image failed: Out of Resources". I'm trying to boot from a USB drive to then trigger the boot of a TrueNAS install on an NVMe PCIe adapter that my Dell R720 cannot directly boot from.
Steps to reproduce
Follow the instructions for portable ( https://docs.zfsbootmenu.org/en/v2.3.x/general/portable.html ) or Ubuntu 24.04 ( https://docs.zfsbootmenu.org/en/v2.3.x/guides/ubuntu/noble-uefi.html ).

So this is the error I get regardless of if I go through the extensive hoops of installing Ubuntu as per the guide onto the USB Thumbdrive, or using the Portable edition.
I cannot for the life of me get ZFS Boot Menu to actually boot, and quite frankly the documentation is really lacking for determining "what I want" in terms of the section of the documentation.
Each section is written as if the reader already knows what they want, but a lot of us don't. I just want a USB Thumb drive to boot the PCIe NVMe device I have that already has boot capabilities installed, but the Dell R720 cannot directly boot to NVMe devices without physical boot roms on them. That's why I'm trying to get ZFS Boot Menu to work. And not only am I not able to get it working, I don't even know what method for setup I should use in this scenario! :/

This isn't a ZFSBootMenu bug, it's Dell R*20 generation UEFI implementations being trash. You should be able use rEFInd to boot ZFSBootMenu on your hardware.

This isn't a ZFSBootMenu bug, it's Dell R*20 generation UEFI implementations being trash. You should be able use rEFInd to boot ZFSBootMenu on your hardware.

I used rEFInd in the Ubuntu attempt I made by the way, which also resulted in the same error once I made the appropriate menu selection rEFInd offered. Also, there's other people reporting success with ZFS Boot Menu on R720's for TrueNAS, which is how I arrived at this tool in the first place.
So... what now then?

Use rEFInd to boot the component release - it's just a kernel and initramfs.

Use rEFInd to boot the component release - it's just a kernel and initramfs.

Did you not read what I just wrote? I said I did that. I followed the Ubuntu documentation which says to do that and still got this result.

Nowhere in your replies do you mention using the component release of ZFSBootMenu, and the initial bug report mentions the EFI build. rEFInd can and does boot the bare ZBM kernel and initramfs on that generation of hardware, since there's no EFI stub loader involved.

The best way to rage against your own ignorance is to work to eliminate it, not lash out at people who offer guidance.
As I've explained elsewhere, our guides are not written to help you figure out what you want, but to illustrate how to:

Provide ZFSBootMenu with the disk layout it demands to boot your system, and
When possible, ensure that your installation provides the components necessary to build a local image.
Take some time to learn some basics and think your way through problems rather than wailing every time you hit the limits of your understanding.


Recipe App

Steps:

Follow the layout of the wireframes provided. Personalize the rest of the design.
Login page and registration page:

Built with Devise.

Food list:

Displays a list of food added by the logged-in user as in the wireframe and quantity of a given food.
Leads to a form that allows users to add new food.

Recipes list:

Displays a list of recipes created by the logged-in user as in the wireframe.
Leads to recipe details.
If the user is the owner of the recipe, allow the user to delete it.

Public recipe list:

Displays a list of all public recipes ordered by newest as in the wireframe.
Leads to recipe details.

Recipe details:

If the recipe is public or the user is the owner of the recipe, display the recipe details as in the wireframe.
If the user is the owner of the recipe, lead to the form that allows the user to add new food.

General shopping list view:

Shows the list of food that is missing for all recipes of the logged-in use.
Should count the total food items and total price of the missing food.

Make sure there are no N+1 queries happening.
Create a navigation menu that allows users to open all of the pages you created.

Thank you very much @MahmoudBakr23
Happy Coding 🥂


migrate to jscs

It might be nice to have a clean Travis build, too.

Yeah, filed the pull request a bit too quickly. :)  Fixing CI now.

file_PR_from_the_hip.gif


Task Dependencies in Project Management

https://www.projectinsight.net/project-management-basics/task-dependencies

solved by #12


The reported position of new atom could form shorter bonds with nearer atoms

Description of the bug
For example, when a position is reported to form a 2Å bond with three atoms, it may also form a 1Å bond with the atom underneath it.
Reported cases so far

Pt (111) fcc (Ptfcc111.cell)
GDY
graphite

Should be a general problem without modification to the searching algorithm. Especially when the target bondlength is much larger than the bondlengths among the existing atoms in the lattice model.

This has been fixed in the conditional_check branch.


Change link to RFC based on credentials

The ServiceNow API gives no indication if we're hitting production or sandbox, and token we use doesn't have that metadata either. So, the solution here is to get a JWT using the same token and then inspect that.
We'll need to subscribe to the Echo service. @joshgubler
Any thoughts on the language in the warning message? I didn't want the action to fail if the RFC was raised but we couldn't figure out if it was done in production.

Props for console.log(🤷)

I'll add the subscription to echo and approve.

Subscribed to echo v1


iif: architect and develop [6]

Target code:
controller.addListener('testEvent', [
  iif(someValue => someValue === true, TruthyCommand, FalsyCommand)
]);

Ring.dispatch('testEvent', {someValue: true});

For information on what an 'iif' is, take a look here: https://en.wikipedia.org/wiki/IIf

58a662050b0aee3a21f663bd197e8aad4334085f
14f3d5ca9836536985827b2ba39b533d4182b6fa


Add *ngIf clausule to the pagination

This is a great library but, in all cases it is not necessary to paginate, I only want to show 5 elements in a table of information thanks.

I couldn't easily use *ngIf for aginator as it caused some issues with @ViewChild, however I've added the ability to set paginator to null and then there will be no paginator visible. You can now also use a custom paginator, which would make existing one to be hidden.
For versions before 1.1.1 you could just use some css to set paginator to display:none.


Fix keyboardShouldPersistTaps warning

This repo is no longer maintained. Use this active fork that is published on npm:
https://github.com/peacechen/react-native-modal-selector


Unsupported textmate grammar (that contains non-advancing match rules)

This textmate grammar:
https://github.com/Benvie/JavaScriptNext.tmLanguage/blob/master/JavaScriptNext.tmLanguage
Doesn't work, shows the following error:
(command line)
[3400:0401/162730:INFO:CONSOLE(1270)] "Grammar is in an endless loop - case 3", source: (...)/vscode/node_modules/vscode-textmate/release/main.js (1270)
(chrome console, stacktrace)
Grammar is in an endless loop - case 3scanNext

@ (...)/vscode/node_modules/vscode-textmate/release/main.js:127
0 (...)/vscode/node_modules/vscode-textmate/release/main.js:1270 Grammar is in an endless loop - case 3scanNext
@ (...)/vscode/node_modules/vscode-textmate/release/main.js:1270_tokenizeString
@ (...)/vscode/node_modules/vscode-textmate/release/main.js:1207Grammar.tokenizeLine
@ (...)/vscode/node_modules/vscode-textmate/release/main.js:1067Tokenizer.tokenize
@ TMSyntax.ts:223tokenize
@ TMSyntax.ts:125_tokenizeLine
@ textToHtmlTokenizer.ts:100_tokenizeLines
@ textToHtmlTokenizer.ts:90_tokenizeToString
@ textToHtmlTokenizer.ts:73tokenizeToString
@ textToHtmlTokenizer.ts:17container.appendChild.htmlContentRenderer_1.renderHtml.codeBlockRenderer
@ modesContentHover.ts:271_renderHtml
@ htmlContentRenderer.ts:65renderHtml
@ htmlContentRenderer.ts:35(anonymous function)
@ modesContentHover.ts:247(anonymous function)
@ modesContentHover.ts:246ModesContentHoverWidget._renderMessages
@ modesContentHover.ts:226ModesContentHoverWidget._withResult
@ modesContentHover.ts:212(anonymous function)
@ modesContentHover.ts:141HoverOperation._onComplete
@ hoverOperation.ts:143HoverOperation._triggerSyncComputation
@ hoverOperation.ts:117(anonymous function)
@ hoverOperation.ts:71RunOnceScheduler.onTimeout
@ async.ts:566

@darkxeno That message indicates a problem with the grammar.
A TM parser is basically an interpreter for the grammar on the passed in text.
Sometimes, grammars contain logical flaws.
The one detected in this case is that a grammar contains some rules which leads to the case where the grammar is not advancing and also not transitioning to a different state.
This all depends on the text being tokenized, so if you wanna share the text that you try to tokenize I can try to find what specific rules in that specific grammar gets it to enter an endless loop.

For me happens on any JS code file when using that grammar. I send you an
example file.
This same grammar i being used in some sublime extensions, and as fas as i
know is seems to be working fine there.
I really love the colors that this grammar can obtain, would be wonderful
if you can make this to work.
Very thanks
2016-04-04 12:35 GMT+02:00 Alexandru Dima<EMAIL_ADDRESS>
@darkxeno https://github.com/darkxeno That message indicates a problem
with the grammar.
A TM parser is basically an interpreter for the grammar on the passed in
text.
Sometimes, grammars contain logical flaws.
The one detected in this case is that a grammar contains some rules which
leads to the case where the grammar is not advancing and also not
transitioning to a different state.
This all depends on the text being tokenized, so if you wanna share the
text that you try to tokenize I can try to find what specific rules in that
specific grammar gets it to enter an endless loop.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/vscode-textmate/issues/9#issuecomment-205234800

Any new about this issue?
Thanks

I copy here my message on the first-mate repository:
I've been able to track down the issue to this regexp (and others similar):
(?x)
  !(?!=)| # logical-not     right-to-left   right
  \|\| |  # logical-or      left-to-right   both

As you can see, these regexp have a trailing pipe that make them match everything. I'm not sure what to do - on one hand, it's obviously a coding mistake to leave this trailing pipe, but on the other hand it seems to work fine with both SublimeText and TextMate (I guess they discard every matching rule with 0-tokens as "not matching"), on a quite popular grammar. What do you think?

The JavaScriptNext syntax apparently uses the official Sublime's PackageDev in order to generate its tmLanguage files from YAML grammars, and it seems that this tool is the one adding trailing pipes. Fixing this would require updating every tmLanguage file generated by this tool up to this point. It isn't really feasible, so I feel like that it should be fixed on the vscode-textmate codebase instead. My two cents.

@bottleofwater Thank you for the analysis. In this case, may I suggest to file an issue against the grammar itself. We allow regular expressions to match 0 characters, many grammar rules use begin expressions that don't consume anything, we would need to "ignore" only match expressions that don't consume anything, which given node-oniguruma's scanner based API is not trivial.

@darkxeno if you can tell me please how did you manage to add and use a TM grammar with Visual Studio Code other then the built-in?

I have removed the console.error since this case is not an error


Support to save crawled page as /path/index.html

Some server, for example SimpleHTTPServer for Python, don't redirect /path to path.html. It failed when I visited /path directly.After moving all path.html to /path/index.html, everything works fine.
Could you please add a feature that would allow us to control the location of saving? This won't be hard, I think, just add a few lines in Writer.js & cli.js.

I had sent a pull request #36, it just checks for the flag '--save-as-index-html' and then changes the filenames. You should change '--save-as-index-html' to anything else, bacause I'm not good at naming...

This will enable deployment to S3 (an a lot of other service, no doubt)

I just accidentally discovered that this feature already exists, if you setup each <Link /> to end with a trailing /.
https://github.com/geelen/react-snapshot/blob/master/src/cli.js#L36-L37
Your solution does force /path/index.html which may be useful.
However in my case I wanted both /pathA.html and /pathB/index.html, so I found the existing functionality works for me.

Sorry for the delay.
@angusfretwell yes, I know that the hosting service can do the redirect job but I still want this feature because the common filename is /path/index.html rather than /path.html.
Thank to @Jinksi, you are right! I had already realized that an addition / might work but I was doing the wrong way by adding it to <Route />... :(
Thank you guys, I'm closing this issue and the related PR.


snippet work with require

common.html

123123

code
  plugins: [
    new HtmlWebpackPlugin({
      inject: false,
      template: HtmlWebpackTemplate,
      appMountId: 'ranwawa',
      appMountSnippet: require('./src/common.html'),  // =>  Unexpected token <
      bodyHtmlSnippet: '
21123
',

  plugins: [
    new HtmlWebpackPlugin({
      inject: false,
      template: HtmlWebpackTemplate,
      appMountId: 'ranwawa',
      appMountSnippet: require('html-loader!./src/common.html'),  // =>   Cannot find module 'html-loader!./src/common.html'
      bodyHtmlSnippet: '
21123
',

This doesn't seem to be a probably with this library. You can't rely on Webpack loaders within your Webpack config like this. Try using fs.readFileSync.


Add dependabot

Codecov Report

Merging #24 (e2e79c9) into main (6200af0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage   97.94%   97.94%
=======================================
  Files           4        4
  Lines         243      243
=======================================
  Hits          238      238
  Misses          4        4
  Partials        1        1

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 6200af0...e2e79c9. Read the comment docs.


Outdated Rocket.Chat version in Mac AppStore

Rocket.Chat in Mac AppStore is 3 months old...

My Setup

Operating System: MacOS 11.2
App Version: 3.0.3
Installation type: AppStore (no other options for Mac given!)

[ X ] I have tested with the latest version
[ X ] I can simulate the issue easily

Description
Mac AppStore is not updated for 3 months!!!!
Current Behavior
No updates
Expected Behavior
Updates...

https://github.com/RocketChat/Rocket.Chat.Electron/issues/1931#issuecomment-815261327

Bump

I am facing backlash from my userbase, since the status (online, away, offline, ...) is not set correctly.
As described in this issue https://github.com/RocketChat/Rocket.Chat.Electron/issues/2034 this is fixed in the latest version of
Rocket Chat.
So it would be appreciated if you could bump the AppStore Version to the latest version.  So I can roll out the fix to my userbase.

Please update! AppStore is still on 3.0.0 and will not receive updates

Fix found: migrate to Mattermost.
I’m sorry, but after more than a year of the issue, I could not resist…

Understandable @alexander-potemkin. I just don't understand why the official download link leads to the AppStore instead of the direct download via GitHub. This would be the least that could be done here

Apple changed the requirements to be able to update the app on Apple Store.
We will be working on this in the next weeks.

@felixspitzer , don’t know - there are so many bugs that are never picked up by the Rocket.Chat team… By the way, I did indeed switched to Mattermost - not so features rich, but it’s muuuuuch more stable, predictable and much less resources hungry.
@jeanfbrito guess it wasn’t the case when I raised an issue; don’t believe it’s something dramatic as well - have a few apps on the AppStore, updating them without much hassle.

@alexander-potemkin the project is open and you can contribute any time. I started to work in the project this month, you can ask me if need some help.

@jeanfbrito it’s a commercial open source project, so unless I share some revenue, I’m really not interested.
Also I have no access to the company’s AppStore, neither anyone from the company is reacting on the issue and I really don’t mean to be a rescuer of someone doing what it’s like - it’s an unhealthy behavior pattern.

The Apple Store version is updated, you can install it from there right now.

Yay! It was just two years old now! :)
Any chance to have this app updated in time, or would it be another two years before the next update?

Always updated. As soon as a new release get out and Apple approves it will
be updated.
Em sex., 25 de mar. de 2022 03:56, Alexander Potemkin <
@.***> escreveu:

Yay! It was just two years old now! :)
Any chance to have this app updated in time, or would it be another two
years before the next update?
—
Reply to this email directly, view it on GitHub
https://github.com/RocketChat/Rocket.Chat.Electron/issues/1974#issuecomment-1078714554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI7PCQA65B5PBWGZRKTR4DVBVPTDANCNFSM4XQSQMAQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


freenet local ; command is absouletly broken

I can push the contract with fdev publish but when I want to reach I have:
failed to get contract XXX, reason: Missing contract and/or parameters
I also got:
storing private key file: `identity-manager-key.private.pem`
storing private key file: `identity-manager-key.public.pem`
storing parameters file: `identity-manager-params`
cd /PATH/freenet-core/modules/antiflood-tokens/contracts/token-allocation-record
fdev build
Compiling contract with rust
error: failed to parse lock file at: /PATH/freenet-core/modules/antiflood-tokens/Cargo.lock

Caused by:
  package `freenet-stdlib` is specified twice in the lockfile
Error: exit with status: exit status: 101
make: *** [Makefile:79: build-token-allocation-contract] Error 1

not sure if linked. Might be an issue on my side. I had already published and visited 2 sites I created before I recompiled freenet and got this issue.

So any contract you push into the network isn't retrievable? And this is a regression of something that worked before?

Also, this is the main branch?

So any contract you push into the network isn't retrievable? And this is a regression of something that worked before?

yes it used to work. it does not work anymore

I did:
git clone https://github.com/freenet/freenet-core/ &&
cd freenet-core &&
git submodule update --init --recursive &&
export CARGO_TARGET_DIR="$(pwd)/target" &&
cargo install --path crates/core --force &&
cargo install --path crates/fdev --force &&
cd apps/freenet-microblogging &&
make build && make run-node

And yes NOT any contract is reachable anymore.

also in order to ensure that you tested right you have to do:
cargo clean
rm -Rf "$(CARGO_TARGET_DIR)"
rm -rf ~/.local/share/freenet

does the microblogging contract and any contract require antiflood token? Might be the root cause.
Caused by:
package freenet-stdlib is specified twice in the lockfile

even my reset to 2c5dfd3be25ebfc67fc6cd41fdd4a6adff5d4235 commit does not bring back the web contract fetching

Aren't you sure you are not on the wrong stdlib branch?
Do git submodule update

I have done git submodule update --init --recursive seems same.

$ cd stdlib/
$ git log -n 3
commit 7729fe328f360067a8ac21fa5809ac0bf748198f (HEAD)
Author: Ignacio Duart<EMAIL_ADDRESS>Date:   Fri Dec 15 15:36:21 2023 +0100

    New error kind

@sanity did you successfully view the contract after my commands please?

@gogo2464: Can you try this in the modules/antiflood-tokens/contracts/token-allocation-record directory:
$ cargo update

That may resolve the problem with the Cargo.lock file.

it does not. it requires to delet Cargo.lock

this commit might be responsible of the issue:
https://github.com/freenet/freenet-core/commit/6ef60489e3bd5916f609e8a2b82fc308faca8dbf
the older work. I am looking for more recent

the commit 8ef468f511c8edc784999778553587b483cfdeb6 works as well

I am now absolutely 100% sure that the commit https://github.com/freenet/freenet-core/commit/6ef60489e3bd5916f609e8a2b82fc308faca8dbf is responsible of the breaking change!!

025b041345be8bf8b8a185f630819a8a1cca5ecc fails

@iduartgomez could you be assigned to this issue please?

@gogo2464 Apparently, the publish never manages to save the contract due to an issue in the perform_contract_put method. By fixing this issue, I have been able to successfully launch the application.

Now I am verifying that there are no more issues I will upload a fix.

THANKS @netsirius  !!!
Could you link your pr please? I would like to use it immediately. :)

https://github.com/freenet/freenet-core/actions/runs/7688432191/job/20949648629?pr=974#step:4:1994
It seems @netsirius , you have to fix it in order to let me finish the testing. https://github.com/freenet/freenet-core/issues/975

@gogo2464 The fix has just been merged. Can you confirm that the issue has been resolved to close it?

@netsirius sure I am going to. I made a PR to easily build on Linux https://github.com/freenet/freenet-core/pull/974 sadly web-sys has also been broken itself as freenet dependency and the chances it does not impact the new freenet is poor.

seems to work with the microblogging contract for me. let me try personnal apps!

I did not test on email app. may be ok. let's close the issue


Bootstrap

Autoload feature defers all async operations until database file is completely read and ready. It there a way to defer those even more until given promise is resolved?
I have "bootstrap" script that checks if database is empty and then inserts some documents. I want to defer all other operations until it is done.
step 1
rm documents.nedb

step 2
var db = new nedb({filename: 'documents.nedb', autoload: true});
db.count({}, function (err, count) {
  if (count === 0) {
    db.insert([{value: 1}, {value: 2}, {value: 3}]);
  }
});

//...

db.find({}, function (err, docs) {
  assert docs.length === 3;
});

I know I can create promise and chain all database operations, but that would be annoying. I would like to achieve it in "autoload" way.

Not possible, you need to do it at application level.


Link in ReadMe.md leads to 404

https://blog.semantic-web.at/wp-content/uploads/2011/09/p1_mendes.pdf

Hi @AgoloKennethTan ,
Thank you. Fixed: http://www.dbpedia-spotlight.org/docs/spotlight.pdf
Best,


Add tqdm-like progress bar to Numba block

Hi Rodrigo!
First of all, I want to thank you for all your efforts and explanations on Medium, now we have neat alternative to using more complex (and more approximate) approaches, like FAISS indexing to get top-K most/less similar vectors that can also run on constrained resources :+1:
Forgive me my curiosity, but I looked into your code and I feel like it may benefit a little from a kind of progress bar updates, especially for bigger N and, as a result, longer processing times, where it's unclear how much time has been elapsed / still remains.
So I created this pull request, which utilizes additional dependency numba-progress and adds additional default arguments to a couple of your functions on whether to track it or not.
Also, I added example notebook of how to use updated version with progress bar enabled.
I hope this PR may act as a additional bullet point for further development of your project :pray:
Looking forward to hearing from you
Best regards,
Fred

Hi @fred-navruzov, I think this is a great idea/addition for usability. Thanks! I had a quick glance but I will check it fully this week.  I am in principle ot really a fan of adding small libraries as it has bitten me in the past but let's see.
And about the comment for FAISS comparison, in case someone else comes accross this reply, FAISS (and similar) is mainly tailored to a similarity search service which receives a query and scans the vector inventory for most similar vectors, this is certainly the strategy you want in most backend ML services that deal with similarity. This library is tailored for an all vs all setup, where
which in my opinion is not tailored  for a service but for fast experimentation over iteration

Hi @fred-navruzov certainly helpful for usability. I'll have a look this week, thanks!

Hi @fred-navruzov  do you think you can implement the requested changes this week?

Hi @fred-navruzov do you think you can implement the requested changes this week?

Hi Rodrigo!
Sorry for a delay, I will try to have a look today, but if you mean calendar week till 23.04.23, then I hope to be able to accomplish it within given time range

@rragundez, can you have a look at changes, I resolved your suggestions, re-run black, added 1 dummy test to check flow with/without progress_bar and merged sparse support changes you mentioned before.
All unit tests are passing now.
p.s. the only thing I didn't change is
if progress_bar is not None:  # you suggested if `progress_bar: xxx` which is more implicit type conversion
    progress_bar.update(1)

I remembered why I intentionally did that - otherwise, if is None part is dropped, numba raises numba.core.errors.TypingError, as it doesn't know the exact type at compilation time
I tried to apply numba's explicit type hints, as suggested here to deal with it, but it didn't work for me, also it was more cumbersome in comparison to adding is None part

@rragundez , my bad, forgot to fix some pylint errors, now 100/100 locally, old and added unit tests are passing, can you run checks on this PR once more?

@fred-navruzov I made some very small changes. Many thanks for your contribution. Indeed I will be updating the README with a nice image/showing the progress bar.

@fred-navruzov I made some very small changes. Many thanks for your contribution. Indeed I will be updating the README with a nice image/showing the progress bar.

@rragundez, it was a pleasure to contribute, and thanks for noticing and fixing those remaining errors, as it's been a while since I've committed for the last time
Have a great day!


trouble to extract mouse 'mm10' signatures through SigProfilerExtractorR

Hi, I am new to the tool. I've tried to follow the 'easy to follow' instruction to install and run SigProfilerExtractorR on my VCF called against mouse (mm10). The simple R code that I used is:
sigprofilerextractor("vcf", "output", data, minsigs=1, maxsigs=3, replicates=10, cpu=-1, refgen = 'mm10',genome_build="mm10", mtype=c("DINUC,ID,96"))
But, the job_metadata file gives the following:
-------Vital Parameters Used for the execution -------
input_type: vcf
inputdata: vcftest
startProcess: 1
endProcess: 3
totalIterations: 10
cpu: -1
hierarchy: False
refgen: GRCh38
genome_build: GRCh38
mtype: ['default']
init: random
At least 2 things are out of expectation here: 1) I specified 'mm10', which I successfully installed I believe, but here says 'GRCh38'; 2) I specified the mutation types that I want, but it still works on the 'default' mtype.
I don't have this issue when I run SigProfilerMatrixGeneratorR/PlottingR. Can anybody help? Plus, not sure if it is due to the above I always get an error message as below:
**Time taken to collect 10 iterations for 1 signatures is 0.55 seconds
Optimization time is 0.01352691650390625 seconds
The reconstruction error is 0.0445, average process stability is 1.0 and
the minimum process stability is 1.0 for 1 signatures
Extracting signature 2 for mutation type 96
Error in py_call_impl(callable, dots$args, dots$keywords) :
IndexError: index 1 is out of bounds for axis 1 with size 1
Detailed traceback:
File "/usr/local/lib/python3.6/site-packages/sigproextractor/sigpro.py", line 571, in sigProfilerExtractor
gpu=gpu,)
File "/usr/local/lib/python3.6/site-packages/sigproextractor/subroutines.py", line 984, in decipher_signatures
results = parallel_runs(genomes=genomes, totalProcesses=totalProcesses, iterations=totalIterations,  n_cpu=cpu, verbose = False, resample=resample, seeds = seeds, init=init, normalization_cutoff=normalization_cutoff, gpu=gpu)
File "/usr/local/lib/python3.6/site-packages/sigproextractor/subroutines.py", line 461, in parallel_runs
result_list = pool.map(pool_nmf, seeds)
File "/usr/local/lib/python3.6/multiprocessing/pool.py", line 266, in map
return self._map_async(func, iterable, mapstar, chunksize).get()
File "/usr/local/lib/python3.6/multiprocessing/pool.py", line 644, in get
raise self._value**
Thanks!

It is not an issue for python version "SigProfilerExtractor". :-)

Hi, we are currently working on a new release for the R version. I will let you know soon after we release the version.


Feature Request: Implement Customer Reviews & Ratings Section

Description
We need to add a customer reviews and ratings section for each product or service. This feature will allow users to leave feedback on products, enhancing trust and transparency for future customers. Reviews will also help other users make informed decisions before purchasing.
Requirements

Review Form

Add a form under each product or service page where users can submit their review.
The form should include:

A text area for the user to write their review.
A star rating system (e.g., 1–5 stars).
A submit button to post the review.

Display Reviews

Display user reviews in a list under the product or service page.
Each review should include:

The user’s name or username.
The star rating.
The text of the review.
The date the review was posted.

Design

The reviews section should match the overall theme and design of the website.
Display reviews in a visually clean and readable layout.

please assign this to me under gssoc-ext and hacktoberfest @Anjaliavv51

@sezallagwal go ahead and level will be added to the PR


docker-compose assumes <IP_ADDRESS> for mosquito broker.

Currently, your energenie_mqtt_service includes the directive:-
command: ["-h", "<IP_ADDRESS>"]
This will force mqtt clients to use <IP_ADDRESS>. Mine runs on localhost so this would not work.
Recommend that users specify the broker host as an argument passed to docker-compose command. where docker-compose.yaml is changed to the following:-
services:
  energenie_mqtt_client:
    image: gpbenton/engmqttclient:1.0.0
    restart: unless-stopped
    container_name: energenie_mqtt_client
    privileged: true
    working_dir: /opt/energenie_mqtt_client/
    network_mode: host
    environment:
      - "MQTT_BROKER=${MQTT_BROKER}"
    command: ["-h", "${MQTT_BROKER}"]
    #entrypoint: /bin/sh -c 'LD_LIBRARY_PATH=/usr/local/lib ./engMQTTClient.exe 2>&1'

  # Remove if you already have mosquitto installed elsewhere
  mosquitto:
    container_name: mosquitto
    restart: unless-stopped
    image: eclipse-mosquitto
    container_name: mosquitto
    volumes:
      - ./mosquitto_config:/mosquitto/config:ro
      - mosquitto_persistence_database:/mosquitto/data
      - /etc/localtime:/etc/localtime:ro
    #user: "1883:1883"
    ports:
      - "1883:1883"
      - "8883:8883"
      - "9001:9001"

volumes:
  mosquitto_persistence_database:

Then, the command to spin up the container would be:-
sudo MQTT_BROKER=localhost docker-compose up -d
This assumes the environment variable MQTT_BROKER is used and of course, localhost is the broker IP - but this can be changed obviously.
Tested and working on Ubuntu and Raspbian.
And for completeness, here is the updated engmqtt.service unit file - of course this is set for localhost and runs in the /home/daz/DOCKER/ENERGENIE directory so that would need to change. Ideally these should be put in /etc/sysconfig or something as env vars and referenced in the unit file .
[Service]
Description=Energenie MQTT Client service
Requires=docker.service
After=docker.service
Environment=MQTT_BROKER=localhost
WorkingDirectory=/home/daz/DOCKER/ENERGENIE
ExecStart=/usr/bin/docker-compose -f /home/daz/DOCKER/ENERGENIE/docker-compose.yaml up
ExecStop=/usr/bin/docker-compose -f  /home/daz/DOCKER/ENERGENIE/docker-compose.yaml down
[Install]
WantedBy=multi-user.target

I regard the docker-compose as a config file which anyone will edit as they wish.  For instance, most users will have a MQTT broker working already, so will probably remove that section.
In your case, you will also change the appropriate address.  If you want to do this with an environmental variable, that's fine, but it seems more error prone than actually putting the value in the file.  You are going to have to ensure that the correct value is set in you environment whenever you start the program.
The only reason I added this docker-compose file at all was because the original author of the docker image included the mosquitto broker inside the image, and I needed to show him how to achieve the same thing with separate docker images.
I'm don't see the need to for a systemd service.  Docker restarts its services on reboot anyway.

I have moved this example to the wiki page, which seems more appropriate.


[Bug] User is unable to reuse name the deleted application

Steps to reproduce

Add an application
Add widget name
Navigate to home page
Delete the created application
Create a application with the same name

and observe
Observation
It is observed that an error message is displayed to user stating the application exists
Expected behaviour
The user should be able to create or reuse the name of the deleted application

Verified the issue as it is working as expected


feature: DataLabels.show_category_name

When creating a pie chart, is it possible to add the category names as labels to the chart eliminating the need for the legend ?
chart.plots[0].has_data_labels = True
data_labels = chart.plots[0].data_labels
data_labels.number_format = '0%'
This puts the series values as labels. It would be nice if we can put the category values also.

Can you do this with the PowerPoint UI?

Yep !

Select a chart by clicking
Click "Chart Design" in the menu
Click "Add Chart Element"
Click "Data Labels"
Click "More Data Label Options . . ."
On the right hand side, a menu opens called "Format Data Labels" which has a "Label Contains" option where you can check mark "Category Name" .

It makes the category name appear along side the series values in data labels.
Hope that helps. [screen shot for reference](: https://drive.google.com/open?id=0B_FxAHf_x3I4M3hzR1JDSlRRY1U)

Ah, ok, so I think what you're looking for is DataLabel.text_frame. That's already in place:
http://python-pptx.readthedocs.io/en/latest/api/chart.html#pptx.chart.datalabel.DataLabel

Although, now that you mention it, there does seem to be a ShowCategoryName property on DataLabels in the MS API. So this is definitely a legitimate feature request.

Thanks. Is there an example of how I can use text_frame to label the pie with the categories?
I did this :
chart.plots[0].has_data_labels = True
data_labels = chart.plots[0].data_labels
data_labels.number_format = '0%'
data_labels.position = XL_LABEL_POSITION.OUTSIDE_END
chart.plots[0].has_text_frame = True
text_frame = chart.plots[0].text_frame
and got:
AttributeError                            Traceback (most recent call last)
 in ()
29
30 chart.plots[0].has_text_frame = True
---> 31 text_frame = chart.plots[0].text_frame
32
33 ## add a shape to the slide
AttributeError: 'PiePlot' object has no attribute 'text_frame'
Without the text_frame piece it was working . . .

I think you're looking for something like this:
data_labels = chart.plots[0].data_labels
for data_label in data_labels:
    data_label.text_frame.text = 'foobar'

Is 'foobar' a list of values corresponding to category names in this case?

You might be on a prior version. I believe that was added in the latest release.
I think the categories are on the Plot object. You should refer to the API documentation for details.

@ngnikhilgoyal: I think you're getting a TypeError because it should be something like this instead:
for series in chart.series:
     for point in series.points:
         point.data_label.text_frame.text = 'foobar'

From the docs, a "DataLabels" object is not a collection of "DataLabel" objects but rather properties for the data labels: http://python-pptx.readthedocs.io/en/latest/api/chart.html#datalabels-objects
@scanny: This feature would be useful for me too. Specifically, control over the "Label Contains" setting available in the PowerPoint UI. Changing the "Label Position" setting right below it is already supported through the DataLabels.position property (aka chart.plots[0].data_labels.position).

I'd imaging adding a contains property to DataLabels makes sense. Unlike the position property which just uses an enum, you can select multiple values and there's the "Separator" text. So it'll have to be a new class or something.

+1
Having the Label Contains mapped is a huge help specially when dealing with Pie Charts in order to show percentages together with category names.
It would be awesome to have something like:
chart.plots[0].data_labels[0].contains = [
    pptx.enum.chart.XL_LABEL_CONTAINS.SERIES_NAME,
    pptx.enum.chart.XL_LABEL_CONTAINS.CATEGORY_NAME,
    pptx.enum.chart.XL_LABEL_CONTAINS.VALUE,
    pptx.enum.chart.XL_LABEL_CONTAINS.PERCENTAGE,
    pptx.enum.chart.XL_LABEL_CONTAINS.SHOW_LEADER_LINES,
    pptx.enum.chart.XL_LABEL_CONTAINS.LEGEND_KEYS
]

chart.plots[0].data_labels[0].contains_separator = pptx.enum.chart.XL_LABEL_CONTAINS_SEPARATOR.NEW_LINE

Hi,
Is there any way to use text alignment feature?

Hi,
It would be great if I can use alignment on label.

Added in release v0.6.16 circa November 9, 2018.
No access to bubble-size or leader-lines, but the other five are in there as e.g. DataLabels.show_percentage (read/write boolean).

Hi Scanny,
First of all, I would like to thank you for the wonderful effort you have made. You never know how much amount of work this has saved in our organization using this library. Such a phenomenon effort, Scanny.
I could see someone has looking for the below input in the library, and it would be very appreciate if you could include this in your next update. I will be grateful for you all the time. Thanks again for your excellent effort, I'm out of words to express. Thanks.
chart.plots[0].data_labels.contains = [
pptx.enum.chart.XL_LABEL_CONTAINS.SERIES_NAME,
pptx.enum.chart.XL_LABEL_CONTAINS.CATEGORY_NAME,
pptx.enum.chart.XL_LABEL_CONTAINS.VALUE,
pptx.enum.chart.XL_LABEL_CONTAINS.PERCENTAGE,
pptx.enum.chart.XL_LABEL_CONTAINS.SHOW_LEADER_LINES,
pptx.enum.chart.XL_LABEL_CONTAINS.LEGEND_KEYS
]
chart.plots[0].data_labels.contains_separator = pptx.enum.chart.XL_LABEL_CONTAINS_SEPARATOR.NEW_LINE


Is there a quicker way to view a Contributor profile than currently exists on the site?

Overview
Is there a quicker way to view a contributor profile on the site than currently exists?
Is your feature request related to a problem? Please describe.
At the moment, it takes a lot of clicks to view a Contributor profile:

Filter by Contributor
View facility list
Select a facility
Click the Contributor profile you want to see

Describe the solution you'd like
Would it be possible / a good user experience to include a link to the Contributor profile at the top of a set of search results, instead of having to select a facility and then click the Contributor profile?
Additionally, is there a way to view all the Contributors to the site and click directly onto their profiles, rather than only seeing site Contributors through the search filter?
Additional Context
A scenario where this would be useful is for an advocacy group who was looking at how frequently brands were updating their supplier lists, and how current the data is on the OAR for different brands. Information on the date range of a supplier list is currently contained within the Contributor profile, so it would be helpful to be able to view this information more quickly.

Noting that this came up on a call with J Crew at the end of Feb
@mariel-oar @wonderchook


Train and test

Hello,
For example, I want to train an environment using the ppo algorithm and test other environments with the network I trained. How can I do that?
What does the run_tests file do in your file?
Thanks.

Hello. You would have to use a separate implementation of the PPO algorithm. The run_tests.py script is to run unit tests.


Issues with controlling a freeMode Swiper

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.3.3

Platform/Target and Browser Versions: Any

Live Link or JSFiddle/Codepen or website with issue: https://jsfiddle.net/paulvd/nyeaj45k/

What you did
Try to control a freeMode Swiper from a regular Swiper.
Expected Behavior
Stay in sync and smoothly navigate the secondary freeMode Swiper.
Actual Behavior
It "jumps" after a few slides and seems out of sync.

A bot automatically closing bugs? Interesting approach to QA.


thin space in amounts

Amounts look like this on osx:

Perhaps we should re-implement 2 decimals and locale-based separators?

If we can make locale-based separators work,that's probably the better solution.

Confirmed fixed by #1279, thanks @rnicoll :)


Fix "compatible release" operator

According to https://github.com/conda-incubator/grayskull/issues/348#issuecomment-1210348234, the currently-merged solution will break back-compatibility. Let's implement another solution instead.
(Hopefully I can take care of it this evening.)

Codecov Report

Merging #368 (0d7d3df) into main (9c3242e) will increase coverage by 0.48%.
The diff coverage is 92.55%.

@@            Coverage Diff             @@
##             main     #368      +/-   ##
==========================================
+ Coverage   91.30%   91.78%   +0.48%
==========================================
  Files          17       16       -1
  Lines        1576     1644      +68
==========================================
+ Hits         1439     1509      +70
+ Misses        137      135       -2

Flag
Coverage Δ

unittests
91.78% <92.55%> (+0.48%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

setup.py
0.00% <0.00%> (ø)

grayskull/base/github.py
72.83% <72.83%> (ø)

grayskull/license/discovery.py
94.87% <78.12%> (-2.28%)
:arrow_down:

grayskull/strategy/abstract_strategy.py
83.33% <83.33%> (ø)

grayskull/__main__.py
95.60% <93.18%> (-2.55%)
:arrow_down:

grayskull/base/factory.py
91.17% <93.33%> (+1.17%)
:arrow_up:

grayskull/strategy/py_base.py
94.97% <94.97%> (ø)

grayskull/utils.py
97.38% <96.58%> (-2.62%)
:arrow_down:

grayskull/config.py
96.96% <96.96%> (ø)

grayskull/strategy/pypi.py
97.77% <97.77%> (ø)

... and 7 more

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

I want to write code which is as reasonably correct and robust as practical. According to the Conda docs, dev<a:

And also the packaging package agrees:
>>> from packaging.version import Version
>>> a = Version("1.0.0a0")
>>> dev = Version("1.0.0dev0")
>>> dev<a
True
>>> a<dev
False

Thus I'm replacing a with dev. Any objections?

that is perfect to me

thanks for your contribution again! :tada: :tada: :tada:

Thanks so much for seeing this through with me!!! :smile:


bunch of new stuff

This PR adds a bunch of features we needed for our project:
set colors of active/inactive parts of seek bar in xml
set thumb drawables in xml
set height of seek bar in xml
allow forcing seekbar/button color as active even when values are defaults
hide min/max labels in xml
set left/right padding in xml
show/hide text above thumb controls in xml
NOTE: default behavior of view should remain identical to current behavior

Great stuff @dbachelder, thanks a bunch!

thanks for merging!


Make sure the search algorithm searches on the README as well

The construct hub returns less results than the the catalog. When we search for softchef we see only 4 packages while the awscdk.io returns 9 packages.
Reproduce: https://constructs.dev/search?q=softchef&offset=0

Its true that awscdkio produces 4 more packages than constructs.dev for softchef search query. The missing packages are:

https://www.npmjs.com/package/sccdk-vue/v0.0.12
https://www.npmjs.com/package/sccdk-vue/v1.0.10
https://www.npmjs.com/package/sccdk-restapi/v/0.0.8
https://www.npmjs.com/package/cdk-iot-security/v/0.0.0

However, the reason they don't appear in constructs.dev is not related to the search, these packages are actually missing from the catalog.json altogether.
Sifting through the logs, it seems that we simply didn't get a change record for these packages from npmjs.com. I'd venture that this is a consequence of these packages being delisted from npmjs.com before we deployed constructs.dev - note that all the above links result in a 404.
In addition, I verified that awscdkio does not perform a README search, nor do I think it or constructs.dev should..it has the potential for offering way more packages than the user intended. Given that this isn't the root cause of the issue described here with softchef packages, I think we should re-evaluate this feature.

Closing as this is not the behavior we want at the moment.


[BUG]: There was an error when processing the data in the dataset.

Describe the bug
After the overnight run of the CoE flows - when I display the CoE PowerBi Dashboard I get this :-
Data source errorColumn 'admin_portalenvironment' in Table 'Portals' contains a duplicate value '875218c8-03cf-4aa9-814c-f14f5482ef6a' and this is not allowed for columns on the one side of a many-to-one relationship or for columns that are used as the primary key of a table.
Component (please tell us which flow or app you are experiencing issues with):

Solution Core
Version3.18
App or Flow PowerBi Dashboard

To Reproduce
Steps to reproduce the behavior:

Run the overnight flows
Display the PowerBi Dashboard
See error

Expected behavior
I expect the PowerBi dashboard to run successfully.
Screenshots
Something went wrong
There was an error when processing the data in the dataset.
Please try again later or contact support. If you contact support, please provide these details.
Hide details
Data source errorColumn 'admin_portalenvironment' in Table 'Portals' contains a duplicate value '875218c8-03cf-4aa9-814c-f14f5482ef6a' and this is not allowed for columns on the one side of a many-to-one relationship or for columns that are used as the primary key of a table.
Cluster URIWABI-UK-SOUTH-redirect.analysis.windows.net
Activity ID7d98ec04-13df-4789-946b-8f7012617d8f
Request ID261b3977-0f14-4dd9-8328-1386e7426c0a
Time2021-11-18 07:17:12Z
Additional context
Add any other context about the problem here.

Hello,
this repo is no longer in use for the CoE starter kit and we do not monitor here anymore. Please raise issues here https://github.com/microsoft/coe-starter-kit/issues - I will transfer the issue to the new repo now, and you'll have to check for updates there.
Thank you
Manuela


adminme doesnt work

Hi !
As said on this page we can become matrix admin with adminme. I tried unsuccessfully.
This is the exact command I typed :
[ T0MuX > ~/matrix-appservice-discord ]
$ npm run adminme -- -r '!dNutYBqCnHmvrjUvlU:mux.re' -u '@t0:mux.re' -p '100' -c '/home/t0mux/matrix-appservice-discord/config.yaml'

><EMAIL_ADDRESS>adminme /home/t0mux/matrix-appservice-discord
> node ./build/tools/adminme.js "-r" "!dNutYBqCnHmvrjUvlU:mux.re" "-u" "@t0:mux.re" "-p" "100" "-c" "/home/t0mux/matrix-appservice-discord/config.yaml"

SyncApi.sync: starting with sync token null
Got push rules
/sync error M_UNKNOWN: Internal server error
{ [M_UNKNOWN: Internal server error]
  errcode: 'M_UNKNOWN',
  name: 'M_UNKNOWN',
  message: 'Internal server error',
  data: { errcode: 'M_UNKNOWN', error: 'Internal server error' },
  httpStatus: 500 }
Starting keep-alive
/home/t0mux/matrix-appservice-discord/node_modules/olm/olm.js:19
"undefined"!==typeof module&&(module.exports=a);process.on("uncaughtException",function(a){if(!(a instanceof T))throw a;});a.inspect=function(){return"[Emscripten Module object]"}}else if(ma)a.print||(a.print=print),"undefined"!=typeof printErr&&(a.printErr=printErr),a.read="undefined"!=typeof read?read:function(){throw"no read() available (jsc?)";},a.readBinary=function(a){if("function"===typeof readbuffer)return new Uint8Array(readbuffer(a));a=read(a,"binary");z("object"===typeof a);return a},"undefined"!=
                                                                                ^

TypeError: Cannot set property 'access_token' of undefined
    at MatrixHttpApi._request (/home/t0mux/matrix-appservice-discord/node_modules/matrix-js-sdk/lib/http-api.js:620:34)
    at MatrixHttpApi.requestOtherUrl (/home/t0mux/matrix-appservice-discord/node_modules/matrix-js-sdk/lib/http-api.js:558:21)
    at MatrixHttpApi.request (/home/t0mux/matrix-appservice-discord/node_modules/matrix-js-sdk/lib/http-api.js:443:21)
    at SyncApi._pokeKeepAlive (/home/t0mux/matrix-appservice-discord/node_modules/matrix-js-sdk/lib/sync.js:892:23)
    at ontimeout (timers.js:427:11)
    at tryOnTimeout (timers.js:289:5)
    at listOnTimeout (timers.js:252:5)
    at Timer.processTimers (timers.js:212:10)
npm ERR! code ELIFECYCLE
npm ERR! errno 7
npm ERR<EMAIL_ADDRESS>adminme: `node ./build/tools/adminme.js "-r" "!dNutYBqCnHmvrjUvlU:mux.re" "-u" "@t0:mux.re" "-p" "100" "-c" "/home/t0mux/matrix-appservice-discord/config.yaml"`
npm ERR! Exit status 7
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>adminme script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/t0mux/.npm/_logs/2018-07-18T11_58_14_083Z-debug.log

And this is the content of the mentioned log :
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/node',
1 verbose cli   '/usr/bin/npm',
1 verbose cli   'run',
1 verbose cli   'adminme',
1 verbose cli   '--',
1 verbose cli   '-r',
1 verbose cli   '!dNutYBqCnHmvrjUvlU:mux.re',
1 verbose cli   '-u',
1 verbose cli   '@t0:mux.re',
1 verbose cli   '-p',
1 verbose cli   '100',
1 verbose cli   '-c',
1 verbose cli   '/home/t0mux/matrix-appservice-discord/config.yaml' ]
2 info using<EMAIL_ADDRESS>
3 info using<EMAIL_ADDRESS>
4 verbose run-script [ 'preadminme', 'adminme', 'postadminme' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/home/t0mux/matrix-appservice-discord/node_mo
dules/.bin:/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/t0mux/matrix-appservice-discord
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c',
10 silly lifecycle   'node ./build/tools/adminme.js "-r" "!dNutYBqCnHmvrjUvlU:mux.re" "-u" "@t0:mux.re" "-p" "100" "-c" "/home/t0mux/matrix-appservice-discord/config.yaml"' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 7  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec adminme script
13 verbose stack Error<EMAIL_ADDRESS>adminme: `node ./build/tools/adminme.js "-r" "!dNutYBqCnHmvrjUvlU:mux.re" "-u" "@t0:mux.re" "-p" "100" "-c" "/home/t0mux/m
atrix-appservice-discord/config.yaml"`
13 verbose stack Exit status 7
13 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:304:16)
13 verbose stack     at EventEmitter.emit (events.js:182:13)
13 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at ChildProcess.emit (events.js:182:13)
13 verbose stack     at maybeClose (internal/child_process.js:961:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:248:5)
14 verbose pkgid<EMAIL_ADDRESS>
15 verbose cwd /home/t0mux/matrix-appservice-discord
16 verbose Linux 4.14.54-1-lts
17 verbose argv "/usr/bin/node" "/usr/bin/npm" "run" "adminme" "--" "-r" "!dNutYBqCnHmvrjUvlU:mux.re" "-u" "@t0:mux.re" "-p" "100" "-c" "/home/t0mux/matrix-appservice-discord/con
fig.yaml"
18 verbose node v10.6.0
19 verbose npm  v6.2.0
20 error code ELIFECYCLE
21 error errno 7
22 error<EMAIL_ADDRESS>adminme: `node ./build/tools/adminme.js "-r" "!dNutYBqCnHmvrjUvlU:mux.re" "-u" "@t0:mux.re" "-p" "100" "-c" "/home/t0mux/matrix-appservic
e-discord/config.yaml"`
22 error Exit status 7
23 error Failed at the<EMAIL_ADDRESS>adminme script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 7, true ]

I don't know what is happening. Apparently, the token is missing. However, my bot and bridge are currently working fine with the good token in the configuration file.
Kind Regards

Hi, this was broken when Synapse was changed to not allow bridge bots to sync. We have fixed this in develop https://github.com/Half-Shot/matrix-appservice-discord/blob/master/tools/adminme.ts.

Hi ! Thank you for your answer, but I just checked and I still have exactly the same issue. However I updated and rebooted my server before trying. :/

Hi, little up it still doesnt work :)


use pyinstaller to build wormhole fat/standalone binary

tested with/on:

ubuntu linux 18.04 amd64
pyinstaller 3.3.1 (pip install pyinstaller)
python 3.6.5

There is a good chance it also works on FreeBSD, maybe also on macOS.
The change in __main__.py was required because otherwise it complains about __main__ not being a package when trying the dot-relative import.

Would be cool if some FreeBSD and macOS users could test this.
PyInstaller also supports Windows, but adapting for Windows might be a bit more work, not sure.

Cool, thanks!

For borg, we use vagrant to build platform binaries, in case you are interested you could grab it from there and build and release fat binaries for convenient download and usage.
See there:
https://github.com/borgbackup/borg/blob/1.1-maint/Vagrantfile
I sign releases so users can verify the binaries are as they were built, so a release looks like this:
https://github.com/borgbackup/borg/releases/tag/1.1.6
There are no windows binaries as borg does not support native windows, but pyinstaller supports building native windows binaries.


Feature Request: Fix warnings in unit tests

Product: (OC-SDR, OC-LTE, OC-LTE, OC-Power, others)
Unit Tests
Component: (PSU, GBC, BBU, FEM, LED, ME)
GBC
Category: (Hardware, Software/stack, Firmware/bootloader, Documentation)
Software/stack
Description
Describe the solution you'd like
Right now running the unit tests generates a lot of warnings (>200) and this will cause any relevant warnings to be lost/ignored amongst so many other warnings. We should clean up the tests and src so that no warnings are generated, add the -Werror flag to the CFLAGS and maintain no warnings.

[x] Clean up warnings generated by test suites
[x] Clean up warnings generated by src
[x] Add -Werror flag to CFLAGS in unit test Makefile

Is your feature request related to a problem? Please describe.
Too many warnings generated when running unit tests so warnings related to actual bugs are ignored with all the rest. See unitTestWarnings.txt

Last two items were addressed by PR #154.


New package: DRIP v0.1.0

Registering package: DRIP
Repository: https://github.com/afrouzi/DRIP.jl
Created by: @afrouzi
Version: v0.1.0
Commit: b6cb99411426d6af80a4ea701d86904e3bda54ea
Reviewed by: @afrouzi
Reference: https://github.com/afrouzi/DRIP.jl/issues/1

please disregard


[WIP] initial support for migration plan and request

Introduce new class for migration plan
Introduce new request and task types for execution

Depends on #16787

@miq-bot add_label v2v

cc @fdupont-redhat @lfu
@miq-bot assign @gmcculloug

cc @jntullo

@miq-bot remove_label wip

wip label removed. Please review.

@miq-bot add_label enhancement

@miq-bot add_label gaprindashvili/yes

Gaprindashvili backport details:
$ git log -1
commit b272ec13bf625da36d3df9eb817d34a0593722d2
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Mon Feb 19 13:41:15 2018 -0500

    Merge pull request #16960 from bzwei/v2v_service_template

    Transformation Plan, Request, and Task for V2V
    (cherry picked from commit 02af759aa7525ea840177178589619fc892853c7)


istanbul university in new domain

@marikci I am sorry, the domain ogr.iu.edu.tr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Fix README Build Status

Description
We display a build status at the top of the README between the mxnet logo and the Portable, Efficient, Scalable graphics. It is currently broken due to splitting the CI validation into multiple steps. This PR displays a build status for each of the steps as I don't know if Jenkins has a combined view.
Old Broken - https://github.com/apache/incubator-mxnet/blob/2a55cd7d1801e6ed801057cac9018b707fe3e27f/README.md
New 1 - https://github.com/zachgk/incubator-mxnet/blob/c62becbc73d74ae066fb1857a7aa3140a38bfb07/README.md
New 2 - https://github.com/zachgk/incubator-mxnet/blob/de80f2a0e27fc7af7489d0cb8fd2d3f190854d89/README.md
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[X] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@aaronmarkham Ok. I figured out how to label the badges so the current version is https://github.com/zachgk/incubator-mxnet/blob/de80f2a0e27fc7af7489d0cb8fd2d3f190854d89/README.md.


Make GuardsAttributes fillable property DocBlock more specific

First of all, this is my first attempt at an Open Source contribution. Apologies if anything is sloppy / incorrect.
Starting from a clean project. Installing Larastan and utilising strict mode highlights the following issue:
Line   Models/User.php
20     PHPDoc type array<int, string> of property App\Models\User::$fillable is not the same as PHPDoc type array<string> of overridden property Illuminate\Database\Eloquent\Model::$fillable.

Specifically, this is caused by the reportMaybesInPropertyPhpDocTypes rule. My proposed fix is to make the GuardsAttribute $fillable property more specific, to remove the need to ignore / stub this error. If this plans to be merged, could possibly do with a wider conversation as to some of the other array<string> annotations elsewhere in the file?
Equally (and admittedly simpler), the User $fillable DocBlock could be made less specific - but I thought I'd start the conversation here as a higher specificity is presumably better.

Isn't it a list<string>?

Isn't it a list<string>?

Yes. But from my (possibly naive) search existing code doesn't make use of list so I stayed away from it.


Read input stream from execute/become

[X] I have read through the manual page (man fzf)
[X] I have the latest version of fzf
[X] I have searched through the existing issues

Info

OS

[X] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[X] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
I have lots of programs which send input to fzf to let me select an option.  I'd like to be able to swap fzf out with other programs sometimes, without modifying the script. Unfortunately, many of my scripts use /usr/bin/fzf so I can't simply put a different fzf on PATH.
I can set FZF_DEFAULT_OPTS="start:become(other_program)", but in become and execute and execute-silently I don't have access to the input stream (that is, stdin), so other_program doesn't get a chance to read the input stream and handle it.
Is there a way to tell fzf to invoke a different program to handle the input stream?

No. It's not possible. It's a very niche use case so I don't think fzf will add an option or mode for that.


Update the spi overlays

https://jel.ly.fish/093193c2-cba9-456e-ae7c-5da407e0a64a

[gelbal] This issue has attached support thread https://jel.ly.fish/#/support-thread~093193c2-cba9-456e-ae7c-5da407e0a64a

Fixed by https://github.com/balena-os/balena-raspberrypi/pull/431. It will be available in the next release


Scroll isn't updated when using a line number from the command line

Currently when you open a file from the command line with a line number, i.e.:
atom ~/cool-thing.js:42

there is a cursor set, but the editor is still scrolled to the top.  Ideally the editor should open up scrolled to the location of the cursor.

I can't reproduce this with Atom 0.195.0-07f80bc and OS X 10.10.3 - have you experienced this with the latest version of Atom (0.194.0)? Does it happen if you try opening in safe mode (atom --safe ~/cool-thing.js:42)?

Doesn't happen on Version 0.196.0 (0.196.0) on mac.
I have another issue to do with opening with line numbers, but since it's different i will open a seperate issue.

@ultramiraculous I'm going to close this for now, but please feel free to reopen if you experience the same issue on Atom >=0.196.0.


Reading invalid BNGL files should raise an error or return None

When an invalid BGNL file is read, a model object is still returned. This makes it easy for users to miss errors. It would be helpful to raise an error or return None, indicated that a model couldn't be parsed from a file.

I think the best way to tackle this is to ensure an error is raised whenever there are any. I added more and will continue to add more error raising, especially in XML parsing functions.


docs: fix typo in url fragment

- What I did
Fixed a broken hyperlink fragment on this page:
https://docs.docker.com/engine/reference/builder/
The link should point to #parser-directives.
- How to verify it
Build the docs, or check other URL fragments in the file which use this format, e.g. #dockerignore-file.
- Description for the changelog
Fix parser-directives hyperlink in builder docs
- A picture of a cute animal (not mandatory but encouraged)
:rat:
Signed-off-by: Mihai Borobocea<EMAIL_ADDRESS>

LGTM, thanks!


请问这是什么问题

有循环引用了，加上SerializerFeature.DisableCircularReferenceDetect)可以解决问题的。比如：
JSON.toJSONObject(obj, SerializerFeature.DisableCircularReferenceDetect);


warning: implicit declaration of function ‘cvRound’

I am getting this warning
warning: implicit declaration of function ‘cvRound’ [-Wimplicit-function-declaration]
I don't know if it is normal.
Using ubuntu 16.04, 64 bit, opencv 3.2.0, ros kinetic.
edit: algo got the following
warning: implicit declaration of function ‘cvLoadImage’ [-Wimplicit-function-declaration]
warning: implicit declaration of function ‘cvSaveImage’ [-Wimplicit-function-declaration]

Well, it did run, but it doesn't detect anything.


StableRNG is not stable

set up
using Random
using StableRNGs

gen = Random.RangeGenerator(1:2)

in julia 1.4.2
rng = StableRNGs.StableRNG(600)
julia> [rand(rng, gen) for i in 1:3]
3-element Array{Int64,1}:
 2
 2
 1

in julia 1.5.0
rng = StableRNGs.StableRNG(600)
julia> [rand(rng, gen) for i in 1:3]
3-element Array{Int64,1}:
 2
 1
 1

This affects rand for Distributions.Categorical and, presumably, many other functions.
version info
Julia Version 1.4.2
Commit 44fa15b150* (2020-05-23 18:35 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin18.7.0)
  CPU: Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-8.0.1 (ORCJIT, skylake)
Environment:
  JULIA_PATH = /Applications/Julia-1.4.app/Contents/Resources/julia/bin/julia
  JULIA_NUM_THREADS = 5

julia> versioninfo()
Julia Version 1.5.0
Commit 96786e22cc (2020-08-01 23:44 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin18.7.0)
  CPU: Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-9.0.1 (ORCJIT, skylake)
Environment:
  JULIA_PATH = /Applications/Julia-1.5.app/Contents/Resources/julia/bin/julia
  JULIA_NUM_THREADS = 5

Of course the other possibility is a bug in Random's implementation of rand for the objects constructed with RangeGenerator, which I have not investigated.

Thanks for the report. This is expected behavior, RangeGenerator dates back from pre-Julia-1.0 era and was kept only for helping the transition, it's deprecated. The current API would be to write gen = Random.Sampler(rng, 1:n). Cf. e.g. https://github.com/JuliaStats/StatsBase.jl/pull/576 which fixed a similar use of RangeGenerator in StatsBase.

Thanks for explaining!


A double/large chest becomes a single chest

I think it looks like you have another resource pack applied that's trying to override the code for the chest UI that Dark Mode adds. It looks like it's working as intended when using Dark Mode only.

I'll take a look which other texture pack interferes

So after going through my texture pack list, i couldn't find another enabled pack that changed chest_screen.json, so I renamed the one in Dark Mode to chest_screen.json.bak and that fixed it.
My local file: chest_screen.json

One other thing I can think of is that Dark Mode's chest_screen.json file relies on other vanilla files that might be changed by another pack, like ui_common.json.
I'll start looking into my UI files to see what I can do to fix them, I think mob_effect_screen.json might be out of date as of recent game versions too.
Does the Chest UI still have issues when applying Dark Mode on top of your other resource packs? I think that could be another fix for it too.

Does the Chest UI still have issues when applying Dark Mode on top of your other resource packs? I think that could be another fix for it too.

That does indeed fix it too. (weirdly enough)

Aah ok glad it fixed it! So somewhere it is colliding with another pack you have.

Maybe as ui_common.json as stated?

It fixes it because when Dark Mode is on top, it will override the other packs that are below it. And it was doing the same thing the other way around, when one of your other packs was on top of Dark Mode. But since Dark Mode has a custom screen file, the other pack expects the vanilla one, so it get all bugged out like that.

Maybe as ui_common.json as stated?

Yeah it might be that.

If you move Dark Mode up and down through the list of packs, you can test which one has to be on top of Dark Mode for it to start showing the bug.


Interaction of withLatestFrom and combineLatest changes values depending on the subscription order of independent subscribers with shareReplay(1).

Discussed in https://github.com/ReactiveX/rxjs/discussions/6605

Originally posted by begrs September 20, 2021
Hi, a sample code that produces my issue is here: https://stackblitz.com/edit/rxjs-mvugpe?file=index.ts

I have two observables as sources input1$ and input2$. Now when one of them is updated, some computation is triggered in the observable calculated$.
The calculated observable has two subscribers to its value, one of which is combined$. Combined$ also requires the input1$ and input2$ values for its computation (here just a console log).

Now one of the inputs is changed to 5. This causes calculated$ to be updated, which in turn updates secondSub and combined$.
What is very weird, is that in combined$ the OLD value of the changed input is used, not the one used for the calculation in calculated$.
However, if I move secondSub (line 12) after combined$, or remove it alltogether, the console long in combined$ actually produces the desired input with the CURRENT value of the changed input.

Removing the shareReplay(1) also fixes the issue - but that was kind of expected.

After the discussion I know understand the behavior, but it is a dangerous issue for me as it produces silent errors.
Reasoning to open an issue:
For me the correct way of handling withLatestFrom and combineLatest would be to first subscribe to combineLatest observables, since even in other use cases, the combineLatest is kind of a combination of multiple observables as a source and should happen before any piped operator. In some of the use cases it does not make any difference, but would in my opinion be more consistent with how users think it will work.
The current behavior is as you said not obvious and can cause a lot of silent errors in a lot of places.
In my case the only reason I found it at all was a division by zero due to the old value.
The secondSub in my case is located in another typescript file and input1, input2 and calculated are part of a kind of module wide "store" to which all components have access and share their values by.
I know RX.js sometimes behaves that way, but it kind of breaks any separation of concerns approaches when the order of independent subscribers (secondSub and combined have nothing to do with each and have no direct interaction, they only use the same source value) causes different values in the value streams.

This seems to be an misunderstanding about source.pipe(withLatestFrom(a, b) and synchronous observables. The source is what dictates when it will emit, and the source is the first thing it subscribes to. Therefore, if source is derived from a or b synchronously, then by the time the result of withLatestFrom emits, it hasn't seen the new value of a yet. This is all working as expected.
To get what you want, you'd want to schedule the emissions from source, like source.pipe(observeOn(asapScheduler), withLatestFrom(a, b)), or something to that effect.
Either way, if you find yourself doing a lot of synchronous combining like this, it might be worth considering that it all could be done in a single step?
Thank you for your issue.

Hi, thank you for your time.
The reasoning why it behaves that way was already explained by voliva in the discussion.
Following your assumption "Therefore, if source is derived from a or b synchronously, then by the time the result of withLatestFrom emits, it hasn't seen the new value of a yet." would mean that no matter what any of the child observables do, the source will always have a newer value than a, or  b.
This would be consistent behavior and would not be an issue for me. But that is NOT the case.  In the example code if I move secondSub behind the declaration of combined$ it changes the output.
With that edit it will actually receive the updated value and not the old one as expected following your comment.
Again, the steps why it happens that way have been explained by volvia, but the order of two otherwise independent observables should not influence the outcome in such a significant way.
Regarding "Either way, if you find yourself doing a lot of synchronous combining like this, it might be worth considering that it all could be done in a single step?" -> I would love to do that, but the observables are used in many parts of the application and as source for even more calculations. The best solution in that regard was proposed by gneu77 - packing the input parameters of the calculation with each emitted value. I will use that, but again, this is not why I opened the issue.
Kind regards

Okay, so looking through this, I now see what you're saying. It seems like the root of the issue is that combineLatest subscribes to the source observable first, and withLatestFrom subscribes to the source observable last. Looking through our core semantics guidelines, we don't really have a defined behavior for what is right there. The closest thing is that we always subscribe to notifiers first. At best, this could be interpreted as okay for combineLatest, where all observables "notify" of a new emission, in which case withLatestFrom is incorrect, as it's only "notifier" would be the source which is subscribed to last. At worst, this semantic is unrelated and we need to come up with something else.
I think, intuitively, withLatestFrom should subscribe to the source first. However this would result in very strange behaviors with synchronous observables. The best we could do to accommodate this is subscribe to the everything, source first, then emit the first value (if there is one) after all subscriptions are started.
Thank you for diligently pleading your case, @begrs, I'll reopen this one and have a look. We might not be able to change this behavior until a major release, though.


a mistake in SimRunner.py

I run this project, a error occurs in SimRunner.py:
wait_time_car = traci.vehicle.getAccumulatedWaitingTime(veh_id)
so, I revise this line to:
wait_time_car = traci.vehicle.getWaitingTime(veh_id)
is that right? Can you give me some suggestions?
I think it's right, because I look for it in official traci documentation and command. I think this is more in line with the actual situation. Right?

I think using getAccumulatedWaitingTime is correct.
If we use getWaitingTime here, then the agent may repeatedly turn on/off the signal to make cars move for a sec and reset wait_time_car to 0 (before crossing the intersection). This kind of strategy will be rewarded, but won't be helpful.

Thank you, but I found no getAccumulatedWaitingTime function interface in sumo 0.30.0
Should I change sumo version?

You can download sumo 1.2.0 at https://www.dlr.de/ts/en/desktopdefault.aspx/tabid-9883/16931_read-41000/

Yes, I think you should change the sumo version since it's recommended to download from the link above at README.

Thank you, I will try this version.

I change my sumo version, the program runs normally.
I found something interesting, when I use getWaitingTime the results curve of queue and delay are exactly the same. This is definitely not right. So your advice is right, we should use getAccumulatedWaitingTime. Thank you.

Thanks for letting us know about your experiment, @ynuwm !


git command fails in PWSH and does not in CMD

Steps to reproduce

create a repository with git
try git reset HEAD@{0} --this should reset the repo at current head, which does notthing but demontrates my issue
in CMD.EXE the command runs fine and you get no output
in PWSH.EXE (7.1.3):

PS D:\dokumenty\okurz\Github\pyCrypt> git reset HEAD@{0}
error: unknown switch `e'
usage: git reset [--mixed | --soft | --hard | --merge | --keep] [-q] [<commit>]
   or: git reset [-q] [<tree-ish>] [--] <pathspec>...
   or: git reset [-q] [--pathspec-from-file [--pathspec-file-nul]] [<tree-ish>]
   or: git reset --patch [<tree-ish>] [--] [<pathspec>...]
   or: DEPRECATED: git reset [-q] [--stdin [-z]] [<tree-ish>]

    -q, --quiet           be quiet, only report errors
    --mixed               reset HEAD and index
    --soft                reset only HEAD
    --hard                reset HEAD, index and working tree
    --merge               reset HEAD, index and working tree
    --keep                reset HEAD but keep local changes
    --recurse-submodules[=<reset>]
                          control recursive updating of submodules
    -p, --patch           select hunks interactively
    -N, --intent-to-add   record only the fact that removed paths will be added later
    --pathspec-from-file <file>
                          read pathspec from file
    --pathspec-file-nul   with --pathspec-from-file, pathspec elements are separated with NUL character
    -z                    DEPRECATED (use --pathspec-file-nul instead): paths are separated with NUL character
    --stdin               DEPRECATED (use --pathspec-from-file=- instead): read paths from <stdin>

Environment data

PS D:\dokumenty\okurz\Github\pyCrypt> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.1.3
PSEdition                      Core
GitCommitId                    7.1.3
OS                             Microsoft Windows 10.0.19043
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

This bug prevents me from using powershell 7 as my default shell.

I gues dup #15289

I experimented but more and
to o run git show @^ you need to type
git show `@^ or git show '@^'
and to run git reset HEAD@{1} you need to run
git reset HEAD@`{0`} or git reset 'HEAD@{0}'

@KoleckOLP I'd recommend using this syntax instead git reset HEAD~1 as this 100% works in PowerShell as I've used it for years


How to read data from DatagramSocket in the BackgroundTask

DatagramSocket only has the OutputStream, so how to read data?Thanks.

The DatagramSocket sample shows how to handle the MessageReceived event


Add channels - shipping zones relations in populatedb

Update populatedb - create shipping zone - channels relations when shipping zone is created.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2646 Update populatedb - create relations between channels and shipping zones


Switch communication from createEvent to postMessage

Description

Firefox doesn't allow communicate with the content script via dispatchEvent so we need use window.postMessage instead.
https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Content_scripts#communicating_with_the_web_page
Testing

[ ] Includes test.
[x] Manually tested.

Additional notes

I didn't actually add any tests but after the changes the current tests still works.

Still needs verification on Safari if we can use this method exclusively. I am assuming yes, but we really need to test it out.
<EMAIL_ADDRESS>
Published this version to do this testing.

Still needs verification on Safari if we can use this method exclusively. I am assuming yes, but we really need to test it out.
<EMAIL_ADDRESS>
Published this version to do this testing.

Verified this does in fact work on Safari, as well. Think we should eventually should rename, fear that is some breaking changes though.

I have removed the line. Thanks @jcblw


Folding: sometimes can not be folded and touch registered as double click

iPad or any touch device
Have some code folded
Touch on the folded region fold decorations to expand it -> all good
Touch again to fold it -> sometimes touch is registered as a double click, thus both an expand and a collapse happens thus resulting in no action
Not sure if this is specific to folding or to all decorations but first assigning to @aeschli

@isidorn Can I point you to the code and you have look? You have the IPad setup ready and know how to debug this...
https://github.com/microsoft/vscode/blob/main/src/vs/editor/contrib/folding/folding.ts#L351

Thanks, I can take a look

@aeschli I have debugged this and the problem seems to be that on touch the editor emits
On Mouse Up
On Mouse Down
On Mouse Up
The first mouse up is generated from the Pointer Up event here
And the other mouse up is generated from the mouse up event here
@rebornix could we somehow combine these two pointer and mouse events so that we do not trigger duplicate events?

Testing iPad bugs, I do still repro this.


-RoutingWeight effect on Azure Site to Site IPsec connections targeting same On-premise address spaces

Is -RoutingWeight found in this page applicable to Site to Site VPN connection as follow?
On-premise A VPN GW Device#1 (local subnet <IP_ADDRESS>/16) -> Internet (Site to Site IPsec 1st tunnel) -> Azure VPN GW#1 (remote subnet <IP_ADDRESS>/16 with -RoutingWeight 1000)
On-premise A VPN GW Device#2 (local subnet <IP_ADDRESS>/16) -> Internet (Site to Site IPsec 2nd tunnel) -> Azure VPN GW#1 (remote subnet <IP_ADDRESS>/16 with -RoutingWeight 300)
From above, would Azure VPN GW#1 has preferred route from Azure to On-premise A leverage on Site to Site IPsec 1st tunnel since it has higher RoutingWeight value?
Thank you!
Regards,
Don Chai

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

ID: 4f266e52-5aa4-f099-6246-77671e6fe4af
Version Independent ID: 105d681c-5bd2-fae0-af69-ac2c06abcdaa
Content: Azure PowerShell script sample - Configure a Site-to-Site VPN
Content Source: articles/vpn-gateway/scripts/vpn-gateway-sample-site-to-site-powershell.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

Thanks for the feedback! We are currently investigating and will update you shortly.

@donchai , Weighting connections with S2S VPN is not supported. Weight is specific to Expressroute Gateway connections.  VPN GWs do honor AS path prepending in both Active/Standby and Active/Active scenarios.
Reference : https://docs.microsoft.com/en-us/azure/vpn-gateway/vpn-gateway-vpn-faq#does-azure-vpn-gateway-honor-as-path-prepending-to-influence-routing-decisions-between-multiple-connections-to-my-on-premises-sites
Routing weight for S2S VPN connections has a default value of 10:
https://docs.azure.cn/zh-cn/cli/network/vpn-connection?view=azure-cli-latest#az_network_vpn_connection_create-optional-parameters
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


docs: update youki build instructions

Explanation

Youki changed its build system form make to just.
Both are not really needed to build youki with wasm since you can just run the build script. Updated the instructions and added missing apt packages for wasm like libzstd-dev, curl and git
Related issue

Does not exist yet
What type of PR is this

/kind documentation
Proposed Changes
Update the build instructions

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!

Hi @hegerdes
Could you please sign your commits with DCO? Ref: https://www.secondstate.io/articles/dco/

Thought of trying out GitHub UI for this small change - gives the source branch a meaningless name and does not sign-off.
Lernend my lesson, local git form now on only with my known config. Is now fixed.


Fix build warnings Documentation Cleanup in Sankhya SDK Project Across Multiple Files

User description

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[x] No

ℹ Additional Information

[!NOTE]
I'm currently writing a description for your pull request. I should be done shortly (<1 minute). Please don't edit the description field until I'm finished, or we may overwrite each other. If I find nothing to write about, I'll delete this message.

Description

This PR focuses on cleaning up the codebase by removing unnecessary comments and copyright headers across multiple files.
It also introduces unit tests for the EntityValidation class, ensuring that various validation patterns are correctly implemented.
Additional improvements in documentation and code readability have been made.

Changes walkthrough 📝
Relevant filesEnhancement

Partner.csClean up Partner.cs file

Src/Sankhya/Transport/Partner.cs
Removed unnecessary comments and copyright headers.  Cleaned up using directives.

+1/-600

Product.csClean up Product.cs file

Src/Sankhya/Transport/Product.cs
Removed unnecessary comments and copyright headers.  Cleaned up using directives.

+1/-405

ServiceRequestExtensions.csClean up ServiceRequestExtensions.cs file

Src/Sankhya/Helpers/ServiceRequestExtensions.cs
Removed unnecessary comments and copyright headers.  Cleaned up using directives.

+1/-206

EntityExtensions.csClean up EntityExtensions.cs file

Src/Sankhya/Helpers/EntityExtensions.cs
Removed unnecessary comments and copyright headers.  Cleaned up using directives.

+2/-133

EntityValidation.csImprove EntityValidation.cs documentation

Src/Sankhya/Validations/EntityValidation.cs
Updated comments to better describe regex patterns.  Improved code readability.

+17/-31

Tests

EntityValidationTests.csAdd unit tests for EntityValidation

Tests/Sankhya.Tests/Validations/EntityValidationTests.cs
Added unit tests for entity validation patterns.  Implemented various test cases for regex matching.

+247/-0

💡 Penify usage:
Comment /help on the PR to get a list of all available Penify tools and their descriptions

Summary by CodeRabbit

Documentation

Removed extensive XML documentation comments across multiple classes and methods, affecting clarity and context for developers.
Specific classes affected include Entity, Invoice, ServiceRequest, ServiceResponse, and various transport-related classes such as Address, City, and InvoiceHeader.
General removal of comments related to properties, methods, and fields, which may impact maintainability and understanding of the codebase.

@gstraccini appveyor reset build

:white_check_mark: Build Sankhya-SDK-dotnet 4.0.5 completed (commit https://github.com/guibranco/Sankhya-SDK-dotnet/commit/adf913fc53 by @guibranco)

@gstraccini appveyor build commit

@gstraccini appveyor build pull request

:white_check_mark: Build Sankhya-SDK-dotnet 4.0.7 completed (commit https://github.com/guibranco/Sankhya-SDK-dotnet/commit/eeb42e57bb by @guibranco)


Kubebuilder v2 Migration

This PR replaces #665, which had become cluttered with comments and commits.
Description of your changes
This PR represents @muvaf work (along with myself and @jbw976) to migrate Crossplane to the Kubebuilder v2 structure. It also involves migration to controller-runtime v0.2 and a temporary fork of controller-tools v0.2(more on this below). The primary motivations for this migration are as follows:

Consistency with the Kubebuilder framework, which will likely be the tool that future stack developers are familiar with
Enables a seamless break-out of major cloud-provider (AWS, GCP, Azure) apis, clients, and controllers intro their own individual stacks
Ability to continue to utilize the functionality provided by the controller-runtime and controller-tools libraries

Additional Details
The commits in this PR have been rebased to show the categories of changes that are implemented. The most prominent changes that can be observed are as follows:

The apis for each cloud provider have been brought to the top-level directory
The comment markers used by kubebuilder and controller-tools  to generate DeepCopy functions and CRD's have been updated to new style
The AddToManagerFuncs pattern has been replaced by the SetupWithManager pattern, which will allow us to pass configuration details down to each controller at runtime in the future
We are taking a temporary dependency on @muvaf fork of controller-tools in expectation that the change tracked by issue controller-tools/#302 will be resolved in the future
The build submodule has been updated to correct any goimports errors that are introduced as a result of make generate, which we experienced after migrating to controller-tools to v0.2

Fixes #502
TODO
There are a few minor issues that will need to be addressed before merging this PR. They will not affect functionality, but are currently causing unit test failures.

[x] Fix BucketPolicyOnly error in make manifests
[x] Fix localhost:8080 unit test error
[x] Reassess new dependency rules in Gopkg.toml

Checklist
I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[ ] Updated any relevant [documentation], [examples], or [release notes].
[ ] Updated the RBAC permissions in [clusterrole.yaml] to include any new types.

The very first Jenkins build hit the intermittent unit test failure of port 8080 already in use 🤕
https://jenkinsci.upbound.io/blue/organizations/jenkins/crossplane%2Fbuild/detail/PR-675/1/pipeline:
[2019-08-13T23:38:55.934Z] === RUN   TestReconcile
[2019-08-13T23:38:55.934Z] --- FAIL: TestReconcile (0.06s)
[2019-08-13T23:38:55.934Z]     sqlserver_test.go:159:
[2019-08-13T23:38:55.934Z]         Unexpected error:
[2019-08-13T23:38:55.934Z]             <*errors.errorString | 0xc000739b20>: {
[2019-08-13T23:38:55.934Z]                 s: "error listening on :8080: listen tcp :8080: bind: address already in use",
[2019-08-13T23:38:55.934Z]             }
[2019-08-13T23:38:55.934Z]             error listening on :8080: listen tcp :8080: bind: address already in use
[2019-08-13T23:38:55.934Z]         occurred
[2019-08-13T23:38:55.934Z] FAIL

@jbw976 I ran into this while trying to debug crossplane on this branch and temporarily patched the metrics server out:
diff --git a/cmd/crossplane/main.go b/cmd/crossplane/main.go
index ac76294..fa55c41 100644
--- a/cmd/crossplane/main.go
+++ b/cmd/crossplane/main.go
@@ -115,7 +115,7 @@ func main() {
 	log.Info("Sync period", "duration", syncPeriod.String())

 	// Create a new Cmd to provide shared dependencies and start components
-	mgr, err := manager.New(cfg, manager.Options{SyncPeriod: syncPeriod})
+	mgr, err := manager.New(cfg, manager.Options{SyncPeriod: syncPeriod, MetricsBindAddress: "0"})
 	if err != nil {
 		kingpin.FatalIfError(err, "Cannot create manager")
 	}

Maybe we should make this an environmental setting?
Why is this only becoming a problem now?

I have corrected the port 8080 error in a pending commit.

BucketPolicyOnly issue has been fixed in a pending commit.

@hasheddan thanks for having this PR in a cleaner shape that is easy to digest.
While we pushed to the same branch for practical reasons, I think we should try to keep the git history as if it's gone through a proper PR process. So, I think it'd be better if we squash only feature commits that belong to the same author, otherwise commits could stay separate, which is what you'd get when you open a PR against another branch.

Just looked at it one more time and I think we are ready to merge this.
Thanks @jbw976 and @hasheddan for thorough test of each custom resource. I think even if we see some issues here and there after we merge, we do know that our functionality is not broken.


check that glottocode is valid

Just trying to build the CLDF dataset and I get this traceback:
Traceback (most recent call last):
  File "/Users/simon/projects/grambank/env/bin/grambank", line 11, in <module>
    load_entry_point('pygrambank', 'console_scripts', 'grambank')()
  File "/Users/simon/projects/grambank/pygrambank/src/pygrambank/__main__.py", line 139, in main
    return args.main(args) or 0
  File "/Users/simon/projects/grambank/pygrambank/src/pygrambank/commands/cldf.py", line 35, in run
    create(args.repos, glottolog.dir, args.wiki_repos, args.cldf_repos)
  File "/Users/simon/projects/grambank/pygrambank/src/pygrambank/cldf.py", line 182, in create
    lang = glottolog.languoids_by_glottocode[sheet.glottocode]
KeyError: 'seco1242'

Looks like seco1242 isn't a real glottocode, it would be nice if this was caught earlier.

Note: seco1242 is a newly minted glottocode that's not been released yet

If you don't pass --glottolog-version and checkout HEAD of glottolog/glottolog, things should work.

@SimonGreenhill what do you mean by "earlier"? Upon adding the sheet to the repository - or just earlier in the grambank cldf command?

sorry, I meant during 'grambank check' as this is where the lists get incorporated?

Could do this - but it would slow grambank check down considerably.

Yeah, I guess we run grambank check multiple times when adding a language (or at least I do - run, fix error, repeat until the list has no errors), so the cost here would be very annoying. I'll close this, but have opened an issue in pyglottolog: https://github.com/glottolog/pyglottolog/issues/28

Better leave it open, since even when glottolog/pyglottolog#28 is fixed, this requires some action here.

can this be closed?

Think so.
Simon J Greenhill<EMAIL_ADDRESS>schrieb am Mo., 21. Sep. 2020,
20:50:

can this be closed?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/grambank/pygrambank/issues/12#issuecomment-696302420,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGUOKCIULW7OJJR7HTFMSTSG6N6BANCNFSM4MDXS2CA
.

Indirectly the sourcelookup checks this in a way, since if the glottocode is invalid it should break in some way or say that all references are wrong, I think.


webbrowser::open is non-blocking on macOS but blocking on linux

The title is pretty self-explanatory. I'm on macOS 10.14.2 and Ubuntu 18.10.
This is weird to me because $BROWSER is not set and xdg-open is also non-blocking. I'm not really sure why but in any case it would be nice to have consistent behavior across platforms.

Note: This only applies when I don't have Firefox already open. If I have a Firefox window open already, then it appears to be non-blocking again...

Indeed, it seems to be the case. From the looks of it, in Linux, calling Command::output() blocks when its stdout/err is set to Stdio::pipe() instead of Stdio::inherit(). This is likely happening because the browser that was opened is still holding a reference to the pipe fd that must've been created. Don't know why this doesn't happen on macOS.
But if inherit were to be used, the point of return an Output is meaningless, as it won't be captured anyways. In hindsight, return ExitStatus was probably better, instead of Output.
For the time being, I think I'll keep API compatibility by returning Output, but internally switch to ExitStatus

@samuela can you confirm this works?

@amodm Ok, so this kinda works AFAICT. I just upgraded to version 0.5.1. It still works the same way in macOS. It now works on the ubuntu VM (same as in the first comment). But it still seems to be blocking when SSHing into the ubuntu box from the macOS machine. Admittedly, I didn't test this case when reporting the issue initially so I'm not sure whether or not this is a regression.

@amodm Thanks for your great turnaround time btw!

That's super weird. I just tested on Ubuntu 18.04 (server) and macOS (client, using XQuartz), , and seems to work exactly as it's supposed to locally. Ubuntu to Ubuntu works as well.

Can you detail out the config you're trying?

@samuela if you do find time to share the config that can help reproduce the SSH issue, let's track it in a separate issue.

Documenting here that new consistent behaviour is now enforced. See Consistent Behaviour section for details.
Relevant commit: https://github.com/amodm/webbrowser-rs/commit/614cacf4a67ae0a75323768a1d70c16d792a760d


HDDS-6587. SCM Web UI is not displaying HEALTHY_READONLY header in node status

What changes were proposed in this pull request?
A one-line patch to add the missing header HEALTHY_READONLY in SCM Web UI.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6587
How was this patch tested?

[x] The 4th column in SCM Web UI should now have the HEALTHY_READONLY header.

Thanks @ayushtkn for reviewing this.