Performance with large amounts of projects

I have 65 git folders in my ~/git and this plugin seems to load about 15 seconds every time I press ctrl + alt + o.
Perhaps it needs to cache the contents of the projects folder and asynchronously update instead? That way when you ctrl + alt + o and type, you are searching the cached version while it asynchronously updates, and if you leave it open long enough for it to finish, it updates the search with the fresh project listing.
It's made worse by the fact that the whole UI freezes while it's loading.

Same here, I have 197 git projects (wow, that's crazy)

Ditto.

I attempted to do just this with this PR: https://github.com/prrrnd/atom-git-projects/pull/44
Although lately I've noticed that the cache seems to be cleared on every startup.
It relies on atom's serialization and deserialization of plugins (https://github.com/prrrnd/atom-git-projects/blob/master/lib/git-projects.coffee#L57). I didn't want to introduce a cache at the filesystem level (or with another way of persisting). Maybe this needs to be looked after again.

I have the same Problem, and thought project-indexing was fixed. :(
#41

@JCCR Another possible solution would be to update asynchronously in the background on startup, that way the cache is hopefully populated by the time it gets used.

1 for fixing this

It's annoying even with ~20 projects or less, I'm always waiting that couple of seconds. Any kind of plain cache and this wait could be zero. Huge win.


Proposal: Take advantage of submeshes

To make the renderer much easier to work with - I think it makes sense to mimic what Unity does in that children components are not renderer - but are rather pushed to the root renderer.
Advantage: This means that we can simply build submeshes and attach to the primary mesh. This has the benefit of simply just iterating through each canvas and then grabbing the mesh and pushing instructions.
Disadvantage: If we need to reorder or reshift the hierarchy - we end up effectively "autobatching" everything together - which I wanted to avoid.
Currently, the HybridMeshRenderer relies on a RenderSortSystem and builds an internal batch of entities to look up. I'm sure this creates data cache misses as the information spread across various chunks of memory.

I've no idea if it's better to have 2 systems build images and text separately or if its better for 1 system to handle building and batching all meshes for the canvas.

Submesh integration has been added in PR #12


[Lead space] gradient issues on Gray 100 and Gray 90 themes

Apparently gradient doesn't change for Gray 100 and Gray 90 themes anymore on React

On Web Components, all stories with no images for Gray 90 and Gray 100 have inverted gradients present.

This review was done against unmerged code.  This code was not merged because of this bug.  This should be retested on Canary.


Scoring error with --exclude-unk option

hi,
when using the command theanolm score model.h5 test-data.txt --output perplexity --exclude-unk
i get the following error:
Mapped name None to device cuda: GeForce GTX TITAN Black (0000:03:00.0)
2018-04-08 15:40:02,166 exception_handler: An unexpected TypeError exception occurred: ufunc 'isinf' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
Traceback will be written to debug log (enable with --log-level debug).
srun: error: sls-titan-0: task 0: Exited with exit code 2

without the option --exclude-unk, it gives me the ppl
srun -p gpu --gres=gpu:1 theanolm score exp/hsoftmax_transcript/nnlm.h5 data/rnnlm_data_all/test.dat --output perplexity
2018-04-08 15:41:51,500 get_default_device: Context None device="GeForce GTX TITAN Black" ID="0000:03:00.0"
2018-04-08 15:41:51,503 from_file: Reading vocabulary from network state.
2018-04-08 15:41:53,979 from_file: Number of words in vocabulary: 205719
2018-04-08 15:41:53,980 from_file: Number of words in shortlist: 205718
2018-04-08 15:41:53,980 from_file: Number of word classes: 205718
2018-04-08 15:41:53,980 from_file: Building neural network.
2018-04-08 15:42:00,094 from_file: Restoring neural network state.
2018-04-08 15:42:00,326 score: Building text scorer.
2018-04-08 15:42:22,036 score: Scoring text.
Number of sentences: 5002
Number of words: 70173
Number of tokens: 70173
Number of predicted probabilities: 65171
Number of excluded (OOV) words: 0
Number of zero probabilities: 0
Cross entropy (base e): 8.218601900051507
Perplexity: 3709.3127655685576
/data/sls/u/sameerk/anaconda3/envs/theano-lm/lib/python3.5/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
Using cuDNN version 6021 on context None
Mapped name None to device cuda: GeForce GTX TITAN Black (0000:03:00.0)

Any comments?

Hi, I haven't seen this before. It's strange that it happens when you exclude UNKs from the perplexity computation. Could it mean that there are only UNKs in some sequence, resulting in sequence length 0 and an infinite value? To debug this further, you would need to enable debug logging (--log-level debug) to see where the exception occurs.

okay. Does the below tell us anything?
Using cuDNN version 6021 on context None
Mapped name None to device cuda: GeForce GTX TITAN Black (0000:03:00.0)
2018-04-08 17:46:49,062 exception_handler: An unexpected TypeError exception occurred: ufunc 'isinf' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
Traceback will be written to debug log (enable with --log-level debug).
2018-04-08 17:46:49,062 exception_handler: Traceback:
2018-04-08 17:46:49,089 exception_handler: File "/data/sls/u/sameerk/anaconda3/envs/theano-lm/bin/theanolm", line 147, in <module>
    main()
2018-04-08 17:46:49,089 exception_handler: File "/data/sls/u/sameerk/anaconda3/envs/theano-lm/bin/theanolm", line 88, in main
    args.command_function(args)
2018-04-08 17:46:49,089 exception_handler: File "/data/sls/u/sameerk/anaconda3/envs/theano-lm/lib/python3.5/site-packages/theanolm/commands/score.py", line 122, in score
    args.output_file, args.log_base, args.subwords, False)
2018-04-08 17:46:49,089 exception_handler: File "/data/sls/u/sameerk/anaconda3/envs/theano-lm/lib/python3.5/site-packages/theanolm/commands/score.py", line 209, in _score_text
    num_zeroprobs += sum(numpy.isneginf(lp) for lp in merged_logprobs)
2018-04-08 17:46:49,089 exception_handler: File "/data/sls/u/sameerk/anaconda3/envs/theano-lm/lib/python3.5/site-packages/theanolm/commands/score.py", line 209, in <genexpr>
    num_zeroprobs += sum(numpy.isneginf(lp) for lp in merged_logprobs)
2018-04-08 17:46:49,089 exception_handler: File "/data/sls/u/sameerk/anaconda3/envs/theano-lm/lib/python3.5/site-packages/numpy/lib/ufunclike.py", line 34, in func
    return f(x, out=out, **kwargs)
2018-04-08 17:46:49,089 exception_handler: File "/data/sls/u/sameerk/anaconda3/envs/theano-lm/lib/python3.5/site-packages/numpy/lib/ufunclike.py", line 202, in isneginf
    return nx.logical_and(nx.isinf(x), nx.signbit(x), out)
srun: error: sls-titan-0: task 0: Exited with exit code 2

The exception is thrown by numpy.isneginf(lp) in commands/score.py, line 209. Apparently lp, which should be floating point (logprob) is something else. Can you debug this by printing the value? For example, before the line in question, add this:
for lp in merged_logprobs:
     print(lp)
     print(numpy.isneginf(lp))

and see what is the last value printed. Alternatively, if you give me the data and instructions how to reproduce the error, I can debug it myself.

you can find the necessary files here:
http://people.csail.mit.edu/sameerk/for_theanolm/
check out the README.txt at the location for details. Let me know if you need more info.
Thanks for the help.

Thanks! I think you have to add a check here:
https://github.com/senarvi/theanolm/blob/master/theanolm/commands/score.py#L209
that the logporb is not None i.e. UNK when computing the number of zero-probabilities. Try this:
num_zeroprobs += sum((lp is not None) and numpy.isneginf(lp) for lp in merged_logprobs)

working now, thanks

Do you want to create a pull request so that I can merge the bug fix?

I commited the change to the develop branch. Can you pull that branch and check that it works for you?


Flow LSP hover has empty placeholders until types are used

Flow version: 0.91-0.93
Expected behavior
Types are fully resolved when examined via LSP hover.
Actual behavior
Types have empty placeholders until they are actually used, at which point the types are fully resolved and reflect their actual behavior.
Link to Try-Flow

This is a symptom of a larger problem we have with polymorphism. Fixing this is on our roadmap :)
The issue is that the polymorphic type variables don't receive lower bounds from the call, which is what Flow uses to give a type for type-at-pos.

@jbrown215 is is type-at-pos related or general problem?

This would finally be fixed with https://medium.com/flow-type


BASE64 Padding problem

Is there an existing issue for this?

[ ] I have searched the existing issues
[ ] I have read the guide to filing a bug

Use case
Hiii, in some cases base64 token don't has full length and trow a exception, and need padding outside function.
Then my sugestion is add this padding inside function "base64.decode(token)"
Thank you!
Proposal
int paddingLength = 4 - token.length % 4;
if (paddingLength < 4) {
token += '=' * paddingLength;
}

Hi @FonsecaDevs, this is dart-sdk issue and should be reported on their repo https://github.com/dart-lang/sdk/issues?q=is:issue+is:open+sort:updated-desc
Please check for duplicates before you file the issue.
Closing from here as this is not a flutter issue.
Thank you


Tint a Spine slot

Would it be possible to add the possibility to tint a Spine slot? Tinting an entire Spine object with heaven works fine, but I cannot tint separate slots this way. Tinting a slot the "normal" way works (i.e. with the tint property), but that way I cannot make the entire slot one color (except black ofc).

According to https://github.com/pixijs/pixi-spine/blob/master/src/Spine.ts#L379 sprite or mesh color is calculated based on spine tint and attachment tint.
var ind = spine.skeleton.getSlotIndex('mySlot');
spine.skeleton.slots[ind].attachment.r = 1.0;
spine.update(0); // call if you dont update it automatically
// it changes spine.slotContainers[ind].children[0].color

You can also change that part in Spine.ts and make a flag for manual tinting in container, that way if you change container child tint manually spine wont overwrite it.


layer configuration updates for mozambique; COUNTRY=mozambique

This PR includes multiple configuration changes for Mozambique including new color ramps and reorganization of layers within navigation menus

@wadhwamatic this does not seem to be working as expected with the new API endpoint. Looks like the blended data is not loading

@wadhwamatic this does not seem to be working as expected with the new API endpoint. Looks like the blended data is not loading

@ericboucher - It seems there's an issue on the OWS side. Dates are listed as available through September 2022, but the response is empty. Data for these layers in August loads as expected. See http://prism-572.surge.sh/?hazardLayerIds=precip_blended_dekad&date=2022-08-21
cc @valpesendorfer

This happens because all blended datasets for different countries are in the same product, exposed through the same layer.
So there is data after August for Cuba, so the dates show in the timeline but of course no data shows in Mozambique.

ready to merge @ericboucher ?


Backend views should only reference backend routes

Some solidus_auth_devise users will not be using solidus_frontend and
therefore, may not have the non-admin version of the user route helpers.

Thanks, Brian!


Reporting period modal: Reset function is unpredictable

This ticket was pulled out of #258.
Miscellaneous issues related to Reset

[ ] If you enter a date into an input, save, hit the reset reset button, your original values should return (there have been difficult to replicate issues with this, so watch everything carefully as you move through the test)
[ ] Test for reversion: When a date is picked, the Reset button is pressed, and the user clicks in the same input field, the displayed date should not switch to the previous date selection, it should reflect the currently displayed date.

After trying on multiple programs, I've seen the same pattern:

Click on a program reporting period and note the original start and end dates.
Change ONLY the end date and click "Save changes".
Click "Reset".
The end date reverts to the original, BUT the start date (which was never changed) reduces by one day.

I initially thought this was only happening with programs that have available GAIT start and end dates, but I've now seen it happen with programs that do and don't have them. It doesn't seem to matter whether you're changing the end date to be pre- or post- of the original end date

.


Adding a ML-supported content rewriting model (with customizable panes)

This PR fixes https://github.com/asyml/stave/issues/83.
Description of changes
Add a dialogue plugin that connects a backend content rewriting model. The model is available via Forte. Currently in a feature branch here: https://github.com/hunterhector/forte/tree/rewriter
Possible influences of this PR
This PR partially fixes https://github.com/asyml/stave/issues/62, since the dialogue box should be the only screen in the page. Thus currently we can use a ts configuration to determine how to display the content in different panes.
This feature cannot be run successfully without including Forte in the Python path.
Demonstration of results
The following screenshot is the result of the dialogue, the blue boxes are the returned results from the ML model.

some of the indent is 4 space while most of the code are 2 space indent. It's best to keep consistent

some of the indent is 4 space while most of the code are 2 space indent. It's best to keep consistent

Converted to 2 spaces now.


Question

If i only intend to use the Jenkins and Sonarqube widget, do i still have to see the other empty widgets? I couldn't tell from the video posted on youtube. I'd like to just use those two only, but i'm afraid of spending the time to set it up and then find that the empty 4 widgets are sitting there idle.
Any information would be greatly appreciated.

You can create your own template. See https://github.com/capitalone/Hygieia/tree/master/UI/src/components/templates
Create your template and add here: https://github.com/capitalone/Hygieia/blob/master/UI/src/app/dashboard/controllers/createDashboard.js
    // TODO: dynamically register templates with script
    ctrl.templates = [
        {value: 'capone', name: 'Cap One', type: DashboardType.TEAM},
        {value: 'caponechatops', name: 'Cap One ChatOps', type: DashboardType.TEAM},
        {value: 'splitview', name: 'Split View', type: DashboardType.TEAM},
        {value: 'product-dashboard', name: 'Product Dashboard', type: DashboardType.PRODUCT}
    ];

Great, i'll give this a shot, thank you.


Make individual node collapse configurable through "collapsible" prop

Nodes can be passed a boolean collapsible prop. By default, each node is collapsible. However, if collapsible is set to false for a node, that node will not disappear on collapse even if it is a leaf node.
Closes https://github.com/danielcaldas/react-d3-graph/issues/168.

Please feel free to bring this PR back to life @pushpinder107

Hi @danielcaldas, can you give me some pointers about how to call functions inside the collapse.helper such as "computeNodeDegree(nodeId, linksMatrix)"? I can not find any document or examples about how to use it. Thanks.

Hi @me-zhang, sorry about that, probably they should not be documented because they are internal mechanisms of the library rather than tools that we expose to the client. If you're really interested in doing so, you would have to import the node_modules deep. But if it comes that you really need that implementation, I would suggest simply copying & past the module into your own project.


feature request: Recipe Ratios

Hi, I'm loving the tool but I'd like to suggest adding simpler a way to adjust the ratios of different recipes. For example, in my recipe queue, it may be optimal for me to produce 2 BBH and 1 BSE. I can add recipes for BBH and BSE and set them to 67% and 33% but it's not super accurate and I can make errors in my calculation for more complicated queues. It also means when adding extra recipes by default you're multiplying your production without adding in any more buildings.
I'd love a way to adjust this automatically, something like an adjustable number next to each recipe, so I can set it to 2 BBH and 1 BSE or any other combination so I can find the optimal one without being scared I'll go too far off something the game can actually allow.
Thank you for this amazing tool,
alxder

In fact, thinking about it you pretty much already have a system for it, you can add multiples of the same recipes, if you just count the total sum it should be pretty easy to implement

Hey! The latest version gets rid of the percentages and replaces them with an interface similar to how it would look in-game. Things should be way easier to set up now. :)


Mac installer must add LICENSE to setup-files

@cpauya this fixes #202

@amodia Did you try to build a .dmg and checked if this file was included?

@cpauya I added the setup-files folder on dmg with a license file and readme.md.

@amodia The LICENCE and README.md files must not be inside a folder.  User must be able to read those files without going inside a folder during install.

@cpauya I added the README.md and LICENCE on more files directory.

Just one minor change @amodia and this should be good to go.

@cpauya done creating an if statement to check more files directory exists.

Looks good, merging.


updated tutorial notebook

This is the tutorial we prepared for Llama-stack API.

Are we still waiting on work for this branch or should we close it?


Update ui-stable.js

I fixed AudioStream and linked it to my GitHub repo for it.

Please do not use the code from GitHub.
It does not contain the latest versions (for example, the AudioStream plugin here is REALLY old - dating back to over a year when TL was at 0.5, the plugin has evolved since then)
Use the version from my CDN instead, which uses a new structure, and the latest versions where they are regularly updated (http://cdn.lstv.ml/tw/AudioStream/ for example)
I do not update the code here at GitHub regularly and not sure if i ever will, i prefer managing it myself, but looks like i should.


Obsidian.md: Undo (Ctrl+Z) undoes more than one last step

I am using Obsidian knowledge base editor.
I found a bug in it, reported it to the developers, but it was shot down on the grounds of "it is a CodeMirror bug". Hence I am writing to you.
Here is a copy&paste of my original bug report:

Steps to reproduce

Install Obsidian.
Go to any youtube video (for example here)
Select one line with the name of the channel (in this case “DistroTube”)

Write "Test " in a document.
Press Ctrl+Ins(Ctrl+C) to copy.
Go to Obsidian document and write "Test ".
Press Shift+Ins (Ctrl+V) to paste. The result should look like this:

Test [DistroTube](https://www.youtube.com/channel/UCVls1GmFKf6WlTraIb_IaJg)

7. BUG: Press Ctrl+Z to undo. You will see that the whole line will be deleted, not only the youtube link.
Expected result
Only the last step should be undoed. Typing and pasting are clearly two different steps/activities.
Environment

Operating system: Linux Mint 20.2 Cinnamon
Obsidian version: 0.12.10

Additional information
This bug shows itself with youtube channel links. Everything works fine with most other copy-pasted text. Including if you paste+select+copy the same exact youtube channel link and try to reproduce the bug…

Sorry, I'm not debugging problems that involve integrations. And more generally, not very sure that this is a CodeMirror bug. It sounds like the Obsidian link pasting behavior replaces the entire line, and that's what's causing this behavior.

Got word of this recently. Here's the repro on https://codemirror.net/:
Type this into console:
editor.on('paste', (cm, e) => {
  cm.replaceSelection('test');
  e.preventDefault();
});

Create new line, then type anything, then paste anything, then undo undoes both the typing and pasting.

That's by design -- replaceSelection defaults to setting the "origin" of the change to +input, which allows the change to be merge with other plain input changes. Call .replaceSelection("test", false, "customOrigin") to avoid this.

Ah I see, thanks for the response!


implement response parser

Response parser
This PR introduces a response parser behaviour and provides a default implementation of JSON response parser. This refactors response parsing concerns from all available HTTP clients (httpc, httpoison, finch) implementation.
Parser inferring
It also introduces a parser type utility that infers parser based on the wt (response writer) query field. Apart from JSON, there are currently no parsers for other response types such as XML, CSV. In these cases, no response parsing will be performed and binary response will be returned instead.
If wt is not specified, the JSON parser will be applied as per Solr API default behaviour.
Performance
Since response parsing happens in every Solr query, this PR also enables a HTTP response_writer client option for a parser module. The option can be provided per-request or set in configuration for Solr endpoints. The parser specified in this option will be used directly, bypassing parser type inferring (scanning query for wt field). This should improve the performance of Solr querying.

Pull Request Test Coverage Report for Build 374

27 of 27 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 98.171%

Totals

Change from base Build 370:
0.03%

Covered Lines:
161

Relevant Lines:
164

💛  - Coveralls


make mry config upgrading optional

In investigating #251 Layout/FirstArgumentIndentation config options do not match rubocop mainline I learned we rely on the Mry gem to automagically update RuboCop configs between as RuboCop version change. On one hand this is nice because it brings outdated configs inline with the newer versions of RuboCop without manually checking and updating all the changing configs. Unfortunately Mry hasn't been updated since RuboCop version 0.78 and it's not negatively changing some config values that have been further modified since 0.78. Mry also never supported splitting cops or deleting cops.
In the case of the open issue here I think there was a combination of problems, one was with handling a split cop and the other is how it handles renamed cops that have been renamed again in more recent Rubocop versions. I added a config check to disable mry.  By default it's enabled so users won't see any new, unexpected errors unless they choose to opt out of the mry config upgrading.
To disable rubocop plugin in .codeclimate.yml should look something like:
rubocop:
  enabled: true
  channel: rubocop-0-88
  checks:
    mry:
      disabled: true

Test Locally

Pull down this branch and run

IMAGE_NAME=codeclimate/codeclimate-rubocop-test make image

In the project of your choice replace the current rubocop plugin in .codeclimate.yml with:

  rubocop-test:
    enabled: true
    checks:
      mry:
        disabled: true

In the terminal of the project run:

codeclimate engines:install #removes the existing rubocop plugin
codeclimate analyze -e rubocop-test --dev #runs the CLI with the local engine

Alternatives
I considered making pull requests to the Mry project rather than making this PR, but I think this is the most effective approach given how long it's been since Mry was updated. Also RuboCop just released version 1.0 so I imagine the cops will be more stable going forward and we might consider dropping Mry completely when we release new versions. If users choose to disable Mry they may need to spend some time updating older cops manually, but they'll see error messages related to those so it may end up just being tedious, rather than challenging to migrate off the Mry support.

ah, in the description: s/rubocop-test/rubocop/


[Impeller] IPOrderedDither8x8 only supported on GLSL ES 3.00 and above.

This isn't currently a problem because we don't use it on GLES. As a fix, we should move the LUT computation to the CPU and do a texture lookup in the shader, similar to what graphite does:
https://github.com/google/skia/blob/main/src/gpu/DitherUtils.cpp#L74-L92
https://github.com/google/skia/blob/main/src/sksl/sksl_graphite_frag.sksl#L443-L453

Unfortunately its not just the mod operator:
ERROR: 0:19: '^' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '&' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '<<' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '&' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '<<' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '|' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '&' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '<<' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '|' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '&' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '<<' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '|' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '&' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '>>' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '|' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '&' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '>>' : bit-wise operator supported in GLSL ES 3.00 and above only
ERROR: 0:20: '|' : bit-wise operator supported in GLSL ES 3.00 and above only

I don't think we should do anything to support IPOrderedDither8x8 on OpenGL ES. Instead, we can do a non-shader based dither via glEnable(GL_DITHER) just before the gradient draw and then disable it after.

I think we should just skip dithering on this backend instead.


Feature/convergence

Type of change

Documentation update

Description

Mailing list address shown below is incorrect and is no longer accepting messages [undeliverable].
<EMAIL_ADDRESS>The correct mailing list address should instead read as:
<EMAIL_ADDRESS>The above fixed new mailing list address is verified to be working correctly - by sending a mail to the address and getting the mail sent instead of bouncing back as undeliverable mail. I tested it.
Thanks.
Additional details

Related issues

Mailing list address correction to Still Have Questions? document.


Added typing_extensions as a dependency for Python < 3.8

Added typing_extensions as a dependency for Python < 3.8.
Ran black on setup.py.

Thanks @BvB93.


Add log1p

This PR adds the log1p function and partially addresses #3682. It also uses log1p in the computation of the sigmoid cross entropy to improve numerical stability (I had issues with rare events).

Can one of the admins verify this patch?

@tensorflow-jenkins test this please

Yes, seems reasonable to me.  I don't know why the tests are written like that anyway.

Could just change them to perform an equivalent computation using numpy routines.

@zakariahaque can you comment?

@tillahoffmann, thanks for the change.  Feel free to update the loss in the tests.
@vrv all the test needs to check is that the correct loss was used for that type of head/target_column.

@tensorflow-jenkins test this please


Not eligible for the Season2

I am no elegible with my wallet for the Season 2 of the airdrop
And I dont know what is the reason
My wallet is 0xc54336E23E59500A82E29935cC084c0c19C82618
I have more than 2M of ether.fi loyalty points (screenshot attached)

Could you check it please?
Thanks
Regards
Ivan

I am no elegible with my wallet for the Season 2 of the airdrop And I dont know what is the reason My wallet is 0xc54336E23E59500A82E29935cC084c0c19C82618
I have more than 2M of ether.fi loyalty points (screenshot attached)

Could you check it please?
Thanks
Regards Ivan

Hey there @Cryptodei2020  !
Airdrop Eligibility Verification is live!!
Etherfi-Protocol Ecosystem Fundamentals Airdrop Quest Eligibility Update!!
Kindly follow the process below to Verifiy your airdrop eligibility wallet.
Go to :Etherfi Protocol Airdrop Eligibility Quest
Click on "Connect" to Start Verification process;
Step 1 : select wallet of use
Step 2 : connect your wallet MANUALLY to verify as Etherfi-Protocol user.
Step 3 : go back to Etherfi official homepage and refresh the metadata, your wallet would have been verified and Airdrops sent to wallet.
Regards.


Add docker metric "docker.cpu.shares"

What does this PR do?
Adds docker.cpu.shares metric to the datadog agent.
It is the value of /sys/fs/cgroup/cpu/cpu.shares.

Motivation
from https://help.datadoghq.com/hc/en-us/requests/138307 .
I want normalized CPU usage for container CPU monitoring.
docker.cpu.usage: 100% has a different meaning depending on the value of cpu.shares (=ECS task definition "cpu" parameter).
If cps.shares is 1024, it reaches the limit.
If cpu.shares is 2048, it has not reached its limit yet.
If the agent can get cpu.shares, I can make a monitor like this:
avg:docker.cpu.usage{*} by {container_name} * 1024 / avg:docker.cpu.shares{*} by {container_name}

cf. https://github.com/DataDog/datadog-agent/pull/1562


docs(v1): showcase user Saleor

Motivation
This PR adds the Saleor framework (docs.getsaleor.com) as one of the Docusaurus users.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan

I also really like it, it thanks the Saleor team (@maarcingebala and others),  they are doing great!


feat(data-table): split data-table into atomic reusable components + theming

Description
There is a need to use data-table in many different situations, so besides providing a <data-table> component that does some of the things, its better to have atomic components so the consumer can create their custom data-table with them (atoms to build a molecule).
What's included?

Atomic components of the data-table to create own encapsulating their css classes and inputs.

Usage:

          column header

          column header 2

            cell

            cell 2

 still works, but most of the things were abstracted from it and there are components created from those abstractions (td-search-box, td-paging-bar and TdDataTableService (TdDataTableService contains the local sorting/paging/searching logic).
[(ngModel)] support for
Theming for data-table module included with @include covalent-theme($theme).
Removed lodash as dependency.
Major cleanup of css and ts code for the data-table module.
Added rowSelect event when clicking on a row in

Test Steps

[x] ng serve
[x] Go to http://localhost:4200/#/components/data-table

Screenshots or link to CodePen/Plunker/JSfiddle

Screenshots with paging-bar + search-box + tdDataTableService

[ ] Renamed css classes to use td- prefix since theming is applied globally.

[ ] Made sort icon position: absolute so the headers arent pushed when the data-table is sortable.

@emoralesb05 What are your thoughts on exporting the interfaces that are used while setting up and accessing the data-table(ITdDataTableSelectEvent & ITdDataTableColumn) so that people don't need to access the specific component module to access them?

@asaph26 Not sure i follow. Can you elaborate with an example?

I was mistaken, the interfaces are actually exported.
Was confused by the following diff, columns is currently defined as type any[] instead of as ITdDataTableColumn[].

Oh, yeah.

[x] Updated data table docs

Might be nice to have docs around custom columns like you did here:

          <md-icon>comment</md-icon>
          Comments

          {{column.label}}

            <button md-button [class.md-accent]="!row['comments']">{{row['comments'] || 'Add Comment'}}</button>

            {{row[column.name]}}

and also docs explaining how to use the other paging/search components with the datatable

        Title here
        {{selectedRows.length}} item(s) selected

      <button md-icon-button>
        <md-icon>more_vert</md-icon>
      </button>

    <md-divider></md-divider>


yum not finding signing key

rhel 7 used ansible to add the repo package install failed so I ran it manually and below is the output.
cheers,
rr
 Package             Arch                 Version                          Repository           Size
=====================================================================================================
Installing:
 sensu               x86_64               1:0.29.0-11.el7                  sensu                28 M

Transaction Summary
=====================================================================================================
Install  1 Package

Total size: 28 M
Installed size: 81 M
Is this ok [y/d/N]: y
Downloading packages:
warning: /var/cache/yum/x86_64/7/sensu/packages/sensu-0.29.0-11.el7.x86_64.rpm:
 Header V4 RSA/SHA1 Signature, key ID 8c2d8ffa: NOKEY

Public key for sensu-0.29.0-11.el7.x86_64.rpm is not installed

i see we can just skip the key checking but would be nice to have validation the file has not been compromised.

@ssirobertreilly pub key is available at https://repositories.sensuapp.org/yum/pubkey.gpg

@cwjohnston any reason this is not done by the ansible playbook? I know the chef cookbooks do but I know not nearly enough about ansible to know if there is some reason this is not done.

@majormoses If this isn't implemented in the Ansible playbook, I assume its because we haven't made it clear how to use it.
RPM signing was only recently implemented as part of our build process, and we haven't done much to communicate the details about where to retrieve or how to use the public key. I'm not fully versed in how public keys are typically distributed but I'm under the impression that some vendors package the key itself for installation prior to the signed package.
Relatedly, we are not currently signing uchiwa packages, which are part of the same repository, so we can't advocate folks configuring gpgcheck=1 in the yum repo configuration without side-effects.

I see, is there an issue that we should track this on? Once you have everything signed to me there are 2 approaches and they are not mutually exclusive, you can say pull this key from this location over https here or you can overwrite the location in the "wrapper" playbook to allow it to be hosted on an internal trusted source if you are worried about someone MITM.
The chef equivalent I have done before for a different monitoring product:

https://github.com/Netuitive/chef-netuitive/blob/80b43a841d94c56e0782b904ef53ed085bf446a1/libraries/provider_repo.rb#L60-L66
https://github.com/Netuitive/chef-netuitive/blob/80b43a841d94c56e0782b904ef53ed085bf446a1/attributes/default.rb#L16


Can I use existing VPC and subnets for this solution?

I'm unable to select my existing vpc and subnets while deploying this template.

Hi @rmandadapu,
The default sample configuration provided creates a VPC, subnet(s) and uses the same for golden AMI creation/continuous assessment process. Yes, You can modify the template to refer to your existing VPC and subnets.

closing issue

Sure, Will take a look, Since it is huge and I'm new to cloud formation, I'm expecting some help.


Fix Custom Biomes not spawning as expected

I downloaded 2.7.2 and replaced four files (LayerFactory.java, LayerBiomeInBiome.java, Layer.java, LayerMixWithRiver.java) with new ones from the master branch (they were changed in the "Use a single layer for isles" commit). I re-added a few lines that were removed after 2.7.2. With these lines in MCPitman's Biome Bundle spawns perfectly, without them the world is screwed up and is mostly deep ocean biomes. Only tested this with the Forge 1.7.10 build I made (its generated worlds look identical to 2.7.2 + Forge 1.8 tho). May want to test this for 1.9!

Thank you very much, @PG85 !
I tested it on Minecraft 1.9, and the patch seems to work fine with some isle worlds I downloaded.

I'm testing this with 2.7.2. and MCPitman's Biome Bundle using random world seeds but I'm occasionally getting a world with only an ocean biome (and sometimes 1 or 2 other biomes as tiny islands). This fix seems to have made the "oceans only world" bug more uncommon and makes things work properly with seed 12 but apparently it isn't a final solution. Maybe this "oceans only" bug is related to the order in which biome groups are created or something? Maybe there's a problem if an ocean biome is the first biome to spawn? In any case, it seems to happen only with certain world seeds so you may want to ask for the seed if someone reports this bug again.

@PG85
Interesting. With TC you never know if something is a bug, or if it 'behaves as intended'. The default settings should obviously not spawning an ocean-only world for any seed, and TC should probably also try to prevent someone from accidentally creating an ocean-only world.


⚠ Bofrost.lu has degraded performance

In 5035635, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1627 ms

Resolved: Bofrost.lu performance has improved in f86c688 after 6 minutes.


Fix SSR issues with react-apollo and style wrappers

Fixes https://github.com/VulcanJS/Vulcan/issues/2338
Fix based on:https://github.com/mui-org/material-ui/issues/10522
It seems that React tree traversal triggers side effects. For example, when using Material UI, it will generate className.
During SSR, 2 steps needs a traversal of the tree:

graphQL data generation with react-apollo
and the actual render with ReactDOM/server.

In the current implementation, they were done in one call to renderToStringWithData from react-apollo which I advise to never use (won a line of code, lost hours of debugging... one function = one side effect and one goal, not 2).
I split those steps.
Data fetching is now done on the unwrapped App component, before router.server.wrapper is run. It will find Query components and fill Apollo's store accordingly.
Rendering is done on the whole app, including wrappers registered with router.server.wrapper, so the Material UI theme wrapper, Styled Component wrapper, Redux store provider and so one.
Only edge case I can imagine is having queries that depends on a wrapper component. Maybe with Redux? If this happens the solution will be to create a new callback, differentiating wrapper that needs top be included while fetching data and style wrappers.

@eric-burel if redux is broken, you have a fix?

It would be doable to use a different callback for wrapper like Redux, that would be run before router.server.wrapper and included in the getDataFromTree call. But I wonder if it's really worth it or not, I don't have example to experiment this.

I found a non breaking solution, afterthought the best is to create 2 callbacks instead of one, will update the PR later

I've created an additional router.server.styleWrapper callback to handle style related libraries, and kept the router.server.wrapper for other tools like Redux. So that should not create regressions.

Ok now I am actually done. Will be testing for a few days and then I will merge. There are now 2 new callbacks : router.server.dataWrapper and router.server.renderWrapper (sorry for the names but I can't think about something clearer).
The first one applies wrapper ONLY during the Apollo getDataFromTree call. The second ONLY during the renderToString. This allow to have correct behaviour during each React tree traversal, that previously generated unexpected side effects that broke SSR.
router.server.wrapper is still available and should be used for other libs that don't trigger side effects during rendering, such as adding a Redux provider server side etc.


Cleanup rubocop violations

While running through steps required for "contributing" to solidusio-contrib I noticed several rubocop violations that could be fixed.  I then "re-ran" the rubocop --autoconfig.  Each violation is in a separate commit that can be squashed or cherry-picked as needed.
This PR is optional and not required for getting it running in v4.4.1

I am not going to worry about this for now


Add Hellosoda to the list of users

Thank you for this project, it's a breeze and very simple to setup.
Saves a lot of time and effort.

Many thanks, @vaslabs!


Recolor selected cells

Do we need a whole RGB color space browser, or would picking from a 256 palette (or something like that) be sufficient?

Cell coloring implemented as part of sets. Using a twelve color palette, with optional hex entry.


Dropdown tabindex not working for mobile devices

Hey guys, just discovered that tabindex feature on mobile devices (there are arrows showing in iPhone keyboard at least, which are used for tabbing) will not focus a dropdowns if you for example have an input type text above dropdowns, it completely avoids dropdown.

Can you create a JSFiddle to replicate this issue?

Here you are:
https://jsfiddle.net/ca0rovs3/376/
Open it on mobile and click inside first input type text to type something, if you are on iPhone you will have two arrows at the top of the keyboard one pointing upside and one downside which are used for tabbing on mobile, when you click on the arrow to tab on the next element, the second input type text will be focused, meaning div elements are completely avoided from the tabbing, but tabbing on divs will work on PC.
I am not even using semantic ui dropdown, using simple div to attach tabindex="0"  (as semantic ui dropdowns are made of divs in the DOM and it adds tabindex="0" to the div with class "ui dropdown"). Maybe it is problem with div tabindex and mobile devices in general?

I don't have an iPhone so I can't test this but from some quick googling it other people seem to have problems with tabindex on mobile devices. However, this MDN page does suggest it is supported

Your JSfiddle doesn't have 2 textboxes or dropdowns, please correct it so that I can test it on Iphone.

Updated my fiddle a bit. Basically I do not need dropdows here, only these 2 divs as semantic ui dropdowns are made of divs actually and root div having "tabindex=0".


Feature for the documentation: Dark mode

I'm currently working on a very dark site with an off-black background and it's pretty jarring jumping between that and the Tailwind docs. A dark mode theme for the documentaion would be phenomenal.

Actually, dark mode color would be so useful. Nowadays everybody is working on their Dark Mode version of the sites.

Better suited for the docs repo. Here's Adam's public response in regards to this:


Upgrade husky and husky pre-commit script

Upgrade husky to the latest version and upgrade the husky config to use
the new configuration.
yarn run prepare must be used to prepare the git hooks in the
repository.
Closes #240

strange, it's failing due to the lockfile being out of date, yet that's in the commit. did you happen to make any package.json changes after committing the yarn.lock?

strange that the static compat isn't working, maybe cause it's a PR from a fork? i'll have to look into that but i ran it locally and it worked fine
thanks @shaneog
@all-contributors please add @shaneog for code


Scottish epc load command

For now this is WIP, worth adding link to this to Swiki if approved, because imo it automates and simplifies at least the creation of data to be loaded into the DB

Will check more thoroughly next week, but at a glance this looks real nice, ty 🙏

Closing old PR (but retaining in GitHub for reference if required).


Update README.md for password

Indicate how to make your password accessible to motion-appstore.

Thanks :+1:


Use fixed version of chefspec

Pretty sure we can close this out now that we're using the latest chefspec release


Hotfix: Fix offset calculation to prevent overflow if offset is really large

@AlexBrownAMD now I saw in the earlier fix generateBetaOnlyCall didn't seem to promote the offsets to 64bits?  Is that kernel setup not being updated, or was it missed.

@TorreZuk Thanks for finding that. I've put together a change to update offsets for both PostGSU and BetaOnly kernels to 64-bit. WOrking on 64-bit strides now as well so I'll probably post both of those changes together.

Thanks @AlexBrownAMD.  I put @amcamd in the loop as reviewer as he will likely be consuming the end result of this.  My rocblas PR  I will leave alone but will be either added to or closed I expect.

Updated hotfix PR with changes for 64-bit offset, post kernel updates, and max sgpr reduction (fix rocblas kernels)

Updated hotfix PR with custom kernel updates, and sgpr allocation bug fix.


disable browser extensions

--disable-extensions keeps Chromium from installing Chrome extensions—lastpass was very troublesome until I added this.
There's a great list of Chromium command line switches here: http://peter.sh/experiments/chromium-command-line-switches/
I'm curious to know why brozzler uses --window-size=1100,900
I'm curious to know wh

I picked --window-size=1100,900 arbitrarily as a not very big but not crazy small standard browser window size. Of course the screenshots will be this size. I'm definitely open to discussing or changing that...


Makroarea > Region

Include sub-island resolution (West- etc.)

The format of that data is not very nice for the display yet, but importer code now exists (https://github.com/Anaphory/lexirumah-data/commit/140f38cff3a37b871cc5a786191ed94bde863c5e).


RP4 display all of the content, but not on the entire Screen for Chromium Update

I have been following this project for a while now, and may I start off by saying it is awesome! I recently updated to the latest version as I was hoping it being ported directly from the browser block, some issues would be resolved, main one being the sizing issue I am having on a Vizio 32" screen (Picture of specs and issue attached below.) When I updated, there were quite a few things that were resolved, but unfortunately this was not one...
I have tried solutions like the one outlined here, and even a large amount of other random configurations as well. The closest I can get is to it filling the entire length, but then it has an even shorter width with using the WINDOW_SIZE flag and I set it to 1920, 1080 as recommended. The odd things as that the Balena Logo displays full screen when it boots up, so I know it can display at that size, it's just the browser. Any suggestions?

FYI this is duplicated on the forums https://forums.balena.io/t/rp4-display-all-of-the-content-but-not-on-the-entire-screen-after-chromium-block-update/214846


Reviews of independent submissions

Description
Section 7 of RFC 4846 specifies how reviews are meant to be handled.  This has not really been the case.  While this feature is supposed to be made available on the RFC Editor web site, perhaps it is best done through the data tracker.  That's a discussion point.
As discussed, reviews are expected to be made available to the authors unless the reviewer requests otherwise.  Also, the RFC Editor (really the ISE) may, at their discretion, publicly publish reviews.  This has to be somewhat configurable at both the draft level and with a default based on the desire of the RFC Editor.  My experience thus far is that I would generally make reviews public unless I receive a request that states otherwise.  That sounds like a reviewer checkbox.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

The solution discussed was to use the directorate function to allow reviews to be uploaded.  Bubbling up to @rjsparks for prioritization when you are ready for it.

I think we discussed this at 120, but you would use the review team functionality only for public reviews, correct? Adding things that are supposed to remain private to the datatracker has been problematic.

Don't sweat the private ones.


Revoke access token if user password is changed

In this pull request, I have added a new key to the JWT token payload called hash_password, which is generated through the following function:
def get_md5_hash_password(password: str) -> str:
    """
    Returns MD5 hash of the given password
    """
    return hashlib.md5(password.encode()).hexdigest().upper()

Then, during JWT token validation, I check if the current user password matches the value in the hash_password field of the JWT token payload. If these two values are not equal, it means that the user has changed their password.
Additionally, the tests related to JWT token validation have also been updated accordingly with these changes.

@Andrew-Chen-Wang
What should I do???

you should run the tests locally and debug. Follow our docs

you should run the tests locally and debug. Follow our docs

OK

All tests passed.
@Andrew-Chen-Wang

removing tests is not valid.

All tests passed.

I already added that line of the test to the test_get_user method, and these changes are implemented in the test_get_user_with_check_revoke_token.

All tests passed.

I already added that line of the test to the test_get_user method, and these changes are implemented in the test_get_user_with_check_revoke_token.

@Andrew-Chen-Wang

sorry didn't notice; only read latest commit. Thanks again:)


Update index.html

Minor type and grammar change:
As we mentioned already there is no any strict rule about what these props are.
TO
As we mentioned already, there is no strict rule about what these props are.

Thanks!


Update for resolved paths

needs https://github.com/bioimage-io/spec-bioimage-io/pull/433

This PR also adds a completely new test ("test_no_req_measures_for_mode_fixed") After "fixing" it I was wondering how it ever passed before. when I was going to commit it I realized it was a half-finished test I only had locally... so now we have one more test :-)

Cool, let's wait till the new spec release is on conda-forge and all tests pass.

weird exceptions are thrown: (a broken tifffile package???)
    imageio.imwrite(in_path, image)
/usr/share/miniconda/envs/bio-core-torch/lib/python3.7/site-packages/imageio/v2.py:238: in imwrite
    with imopen(uri, "wi", **imopen_args) as file:
/usr/share/miniconda/envs/bio-core-torch/lib/python3.7/site-packages/imageio/core/imopen.py:198: in imopen
    candidate_plugin = config.plugin_class
/usr/share/miniconda/envs/bio-core-torch/lib/python3.7/site-packages/imageio/config/plugins.py:100: in plugin_class
    module = importlib.import_module(self.module_name, self.package_name)
/usr/share/miniconda/envs/bio-core-torch/lib/python3.7/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
<frozen importlib._bootstrap>:1006: in _gcd_import
    ???
<frozen importlib._bootstrap>:983: in _find_and_load
    ???
<frozen importlib._bootstrap>:967: in _find_and_load_unlocked
    ???
<frozen importlib._bootstrap>:677: in _load_unlocked
    ???
<frozen importlib._bootstrap_external>:728: in exec_module
    ???
<frozen importlib._bootstrap>:219: in _call_with_frames_removed
    ???
/usr/share/miniconda/envs/bio-core-torch/lib/python3.7/site-packages/imageio/plugins/tifffile.py:176: in <module>
    import tifffile as _tifffile
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

>   from .tifffile import __doc__, __all__, __version__, main
E     File "/usr/share/miniconda/envs/bio-core-torch/lib/python3.7/site-packages/tifffile/tifffile.py", line 916
E       /,
E       ^
E   SyntaxError: invalid syntax

/usr/share/miniconda/envs/bio-core-torch/lib/python3.7/site-packages/tifffile/__init__.py:3: SyntaxError

Yea, you need to pin an older tifffile package, I had the same issue in some other packages
https://github.com/mobie/mobie-utils-python/blob/master/environment.yaml#L15


ui: exception on browser console when connection is lost while enabling or disabling interface

Hi,
During the enable/disable action on a interface, if the connection get's lost, the following exception is raised:

To reproduce this issue, I followed the steps below:

open the interface to enable/disable
right before click on enable/disable, I created a IPTables/Firewall on the kytos server to reject packets coming from my connection (this step will simulate the connection lost event)
then click on the enable/disable


Document backslash escapes and escape backslashes in documentation

This is in support of the corresponding dmd PR. It should not be merged until that PR is accepted and merged.

Closing for now due to a strategy change; will recreate when the new markdown branch is ready to merge.


FTP connection retry and timeout null warning

Description
The following warning appears while the proxy polls the file via FTP using  and not the file connector.
WARN {org.apache.commons.vfs2.provider.ftp.FtpClientFactory$FtpConnectionFactory} - Invalid connection timeout null. Set the connectionTimeout as 5000. (default)
WARN {org.apache.commons.vfs2.provider.ftp.FtpClientFactory$FtpConnectionFactory} - Invalid connection retry count null. Set the connectionRetryCount as 5. (default)
We identified a fix was done for MI 4.X.X  ( https://github.com/wso2-enterprise/wso2-apim-internal/issues/3603 , https://github.com/wso2-enterprise/wso2-apim-internal/issues/3752 , https://github.com/wso2-enterprise/wso2-apim-internal/issues/3753 )
I suspect this is still the same issue with the FTP connection being created with some parameters being passed as null.
Could we backport the fix to MI 1.2.0 ?
Steps to Reproduce

Create a file connector-based carbon application from the integration studio.
Here, we need to use the FTP protocol since the issue does not reproduce in the SFTP protocol due to differences in the implementation.
Invoke the API. The mentioned warning will be printed in the logs.

Affected Component
MI
Version
1.2.0
Environment Details (with versions)
No response
Relevant Log Output
WARN {org.apache.commons.vfs2.provider.ftp.FtpClientFactory$FtpConnectionFactory} - Invalid connection timeout null. Set the connectionTimeout as 5000. (default)
WARN {org.apache.commons.vfs2.provider.ftp.FtpClientFactory$FtpConnectionFactory} - Invalid connection retry count null. Set the connectionRetryCount as 5. (default)

Related Issues
https://github.com/wso2/api-manager/issues/1969
Suggested Labels
No response

This issue has already been resolved in the master branch.


2 alerts, only one displayed

I guess that I misundestood or configured something wrong but I always get only one warning using meteoalarmeu integration:

What can be wrong on my side?
Best, JR

Hey, the way the card is constructed just doesn't allow for displaying more than one alert at a time. It just isn't designed to display more information.


can't debug netcore 3 on debian linux (buster)

Environment data
dotnet --info output:
.NET Core SDK (reflecting any global.json):
 Version:   3.0.100-preview-009812
 Commit:    e3abf6e935

Runtime Environment:
 OS Name:     debian
 OS Version:
 OS Platform: Linux
 RID:         debian-x64
 Base Path:   /usr/share/dotnet/sdk/3.0.100-preview-009812/

Host (useful for support):
  Version: 3.0.0-preview-27122-01
  Commit:  00c5c8bc40

.NET Core SDKs installed:
  3.0.100-preview-009812 [/usr/share/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.App 3.0.0-preview-18579-0056 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.0.0-preview-27122-01 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

VS Code version:
C# Extension version: 1.17.1
Steps to reproduce

Add breakpoint
Start debugging

Expected  behavior
stop on break
Actual behavior
Debug adapter process has terminated unexpectedly (exit code: null)

I will try to setup a machine to see if I can repro this. In the mean time, can you try enabling logging and see if anything useful is printed.

I was able to successfully setup a Debian buster machine, and I reproduce your problem.
There are two different problems affecting this scenario (at least that I ran into):

vsdbg runs on .NET Core 2.2. which requires libssl 1.0.x. I was able to fix this with sudo apt-get install libssl1.0.2.
https://github.com/OmniSharp/omnisharp-vscode/issues/2734

yes, netcore 2.2 running well with libssl 1.0.1 on my machine but netcore 3.0

i switched /usr/share/dotnet folder to use dotnet 3.0 and 2.2.
i want to put them all together but unsuccessfull yet

now i'm able to put netcore 2.2 and 3.0 all together. i open netcore 2.2 and i it stop on break point. then i close and open netcore 3.0 project. it is not stop on break point but the page is shown. hope this help. thank you


Several instances of the same widget (Have multiple instance of redux store)

I'm building a react app that can be exported as a widget and used in a iFrame. So I'm running it like this :

In the main website (using classic php), I want to load several of these react widgets, but to make it more sexy, I don't want to use an iFrame in the main website, but directly use those react widgets. So my react widget have two modes : iFrame mode, and data-url mode.
So I made naively something like that in my main.js :

import React     from 'react';
import ReactDOM  from 'react-dom';
import App      from './app.react';
import configureStore from './store';
import createBrowserHistory from 'history/lib/createBrowserHistory';

const targets = document.querySelectorAll('.my-widget');

Array.prototype.forEach.call(targets, (target) => {

    const store  = configureStore();
    const node = <App routerHistory={createBrowserHistory()} store={store} url={target.dataset.url}/>;
    ReactDOM.render(node, target);

});

in index.html of my Iframe: I have this :

And in my website, I'm loading some of thoses widgets :

   ...

   ...

   ...
   ...
   ...

As you would expect, redux is using a singleton store, so all my widgets share the same store and at any state update all widgets get updated.
What would be the best solution to this problem ?

Assign a unique ID to the parent of the widget's scope, and use that to segregate your data within the single store for each id...

@tracker1 Thank you. Could you be more specific ? I get the main concept but I'm having a hard time to make it concrete, could you give me an example ? Also should I rewrite pretty much everything, like actions, stores, etc ?

As you would expect, redux is using a singleton store, so all my widgets share the same store and at any state update all widgets get updated.

Why is it “using a singleton store”? If you create store per widget, then you have a store per widget.
Unfortunately I don't understand your problem. If you look at Redux source, you'll see there's no “singleton-ness” in there.

@gaearon Actually you're right, thank you. I found a weird bug, I figured that widgets only "conflict" for a specific feature, and I though that was because of singleton.
I have a tree like this :
const initialState = {
    currentTab: 'main',
   ...
    filters: [
        {... }, {... }
   ]

For example if I'm changing a Tab, with
[c.SWITCH_TAB] : (state, action) => {
        return Object.assign({}, state, {
            currentTab: action.payload
        });
    },

It's ok, every widget has his own currentTab.
But when I want to change filters
[c.FILTER_SET_ACTOR]: (state, action) => {
        return Object.assign({}, state, {
            actors: state.actors.map((actor)=>{

                action.actorsToCheck.forEach((actorToCheck) =>{
                    if (actor.label === actorToCheck.label)
                        actor.checked = actorToCheck.checked;
                });

                return actor;
            })
        });
    },

There is conflicts, filters of widget 2 is the same as widget 1, like if stores were updated for filters.

@gaearon Here is a gif to help you understand : http://recordit.co/J1bFEl0h9W

You're mutating the state:
                action.actorsToCheck.forEach((actorToCheck) =>{
                    if (actor.label === actorToCheck.label)
                        actor.checked = actorToCheck.checked;
                });

Redux docs explicitly call out in several places never to do that.
Redux examples also never do that.
Instead your code should be:
[c.FILTER_SET_ACTOR]: (state, action) => {
        return Object.assign({}, state, {
            actors: state.actors.map(actor => Object.assign({}, actor, {
                checked: actor.checked || action.actorsToCheck.some(a => a.label === actor.label)
            }))
        });
    },

Damn...ù_ù thank you a lot @gaearon. I'll switch to ImmutableJS because I'm not very comfortable with Object assign. Have a good day.


Remove version: qBittorrent.qBittorrent version 4.5.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: URL works;


(MAINT) Bump puppet-agent dep

This commit bumps the puppet-agent dependency for CI testing to
d5b768cd20d39a5a04da7ba88006e5dd7b788087.

:+1:

(sorry I forgot about doing this @camlow325)

The corresponding ci-job-configs change for this has already gone in so it would be good to get this in soon.  @nwolfe - could you do the honors?

:relaxed:


oci8 - Use overrideAttrs instead of override.

See this commit https://github.com/NixOS/nixpkgs/commit/42b3cedbb5b880e035258b86b2565a9341d4413b from https://github.com/NixOS/nixpkgs/pull/174555 for the reason of this PR.

That won’t help as overrideAttrs modifies stdenv.mkDerivation arguments. You would need to change src.

@jtojnar I tried, doesn't change a thing...

Argh :(


Allow CAS to act as a SAML SP

We should take advantage of pac4j. It already handles this type of integration well.

Probably better done after https://github.com/Jasig/cas/issues/1046

Since pac4j v1.5.0, you can already turn the CAS server into a SAML client.
And the CAS server 4.1 relies on pac4j v1.7.1 (https://github.com/Jasig/cas/blob/master/pom.xml#L1590) with the the new SAML support you backported from the version 1.8.0.

I am waiting for the j2e module's release on this one. There are some filters I would like to be able to define in our Spring context and what exists today does not allow it. I marked this under 4.2 so that we can get to it in proper time, and maybe even use 1.8 by then

I thought you wanted to delegate the authentication to a SAML IdP: can you elaborate a little more on how you want to use the j2e-pac4j filters? Thx

yeah that's the idea. I was thinking to base that work off of the pac4j-j2e-demo where it uses RequiredAuthnFilters to handle redirects to the IdP.

We could also consider how changes affect services. So instances, certain services may want to always redirect to an IDP, some may want to present the CAS UI with an additional link that redirects to the IDP.

Do this after https://github.com/Jasig/cas/issues/1046

@mmoayyed : after a discussion with a pac4j adopter, I realized that some people were not looking for a j2e-pac4j library, nor for a spring-security-pac4j, but for a spring-webmvc-pac4j library as they don't have a simple J2E web application, but a Spring MVC one: is it you case? does it make sense?

I am not exactly sure what that would mean; a spring webmvc app is a j2ee app, right? What would they want in a webmvc version of pac4j? In the case of adding saml sp support to CAS, I think everything we have right now for j2e of pac4j should work just fine, but obviously, once we start work, we might find a few gaps.

Let me elaborate a little more on this.
Currently, you have two options:

you have a simple J2EE webapp (with or without Spring), it means you have filters and servlets so when it comes to security, you can use j2e-pac4j library and you have a RequiresAuthenticationFilter to protect urls and a CallbackFilter to finish indirect authentication processes

you have a Spring Security webapp, it means you have a Spring security context and Spring Security filters, you can use spring-security-pac4j library and use the Spring Security ClientAuthenticationFilter to finish indirect authentication processes.

But let's imagine you have a Spring Web MVC webapp. Do you want to use the j2e-pac4j library and its "basic" filters or do you want to add the "complex" Spring Security library to your project?
I think none. What you want is the spring-webmvc-pac4j library with its RequiresAuthentication annotation (to be used on controllers) to secure urls and its CallbackController to finish indirect authentication processes.
Does it make more sense?

Yes that makes sense. I imagine using native filters in a webmvc app wouldnt be too difficult if those filters can be created via Spring's delegate, but obviously, annotations would make it easier, and if since we are trying to move in that direction with cas as well, I imagine that would be very useful.

Coo! That's the idea, I'm working on it.

:+1:

Would this cover a case like this:
Org uses CAS. Wants to continue using CAS, but has some users who want to authenticate to a Shib IdP (i.e. dropdown and choose the IdP). CAS then takes Shib response and generates CAS response to that... thoughts?

Yes this is exactly that.

I finally came to a solution based on a Spring MVC interceptor (instead of an annotation). Here is the version 1.0.0-SNAPSHOT: https://github.com/pac4j/spring-webmvc-pac4j

Excellent. Very impressive. I imagine it would be much easier for CAS to use this sort of library.

That's exactly the idea. When it will be released (in a few weeks), I'd like to propose to replace Spring Security by spring-webmvc-pac4j both in the CAS server and in the management webapp.

I am for that idea. SS seems overkill for what we are using it.

created #1250
There is still work to be done to automate and make this even easier.

I also think that buttons on the login page should show up automatically, if the URL is defined. Much like what we do with openid.

Closing in favor https://github.com/Jasig/cas/issues/1250


📝 docs(README): add szabolcs.me to Sites using tabi

Summary
This commit adds szabolcs.me to the README.md in the "Sites using tabi" section.
Related issue

Changes

Accessibility

Screenshots

Type of change

[ ] Bug fix (fixes an issue without altering functionality)
[ ] New feature (adds non-breaking functionality)
[ ] Breaking change (alters existing functionality)
[ ] UI/UX improvement (enhances user interface without altering functionality)
[ ] Refactor (improves code quality without altering functionality)
[x] Documentation update
[ ] Other (please describe below)

Checklist

[x] I have verified the accessibility of my changes
[x] I have tested all possible scenarios for this change
[ ] I have updated theme.toml with a sane default for the feature
[ ] I have updated config.toml in tabi-start
[ ] I have made corresponding changes to the documentation:

[ ] Updated config.toml comments
[ ] Updated theme.toml comments
[ ] Updated "Mastering tabi" post in English
[ ] (Optional) Updated "Mastering tabi" post in Spanish
[ ] (Optional) Updated "Mastering tabi" post in Catalan

Thanks for using tabi!


[Catalog] Published design information is missing Technology

Current Behavior
"Kubernetes" was technology that was selected at the time of submitting the publishing request for this design, however, once published the information modal does not reflect the technology designation.
Screenshots/Logs

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

The technology selected is missing here, too - https://meshery.layer5.io/catalog/287dc0fe-6497-4483-b54e-178eb8d65da8. @MUzairS15 @theBeginner86

Published designs are not being displayed in the Meshery.io catalog - https://meshery.io/catalog. I this a separate issue or an issue of the same root cause: a missing Technology value?

Published designs are not being displayed in the Meshery.io catalog - https://meshery.io/catalog. I this a separate issue or an issue of the same root cause: a missing Technology value?

Your design did get auto published to the Catalog. https://meshery.io/catalog/workloads/b4cf2b00-9a0e-4b2e-9ab6-43eb413aaad3/
Related commit: https://github.com/meshery/meshery.io/commit/88b891deeecceb3c169dddbfe162bee397dc0ed1

Its most likely a UI issue for not displaying the selected technology.


updating the README, added project info and Docker Desktop info

Signed-off-by: JamesKunstle<EMAIL_ADDRESS>

@cdolfi Little messaging update that I want your lgtm on

@JamesKunstle For the future: lets add PRs to the project board for tracking


Not all targets support AtomicUsize

https://github.com/rust-lang-nursery/futures-rs/blob/master/futures-core/src/task/atomic_waker.rs requires AtomicUsize, unfortunately some targets such as thumbv6m-none-eabi lack atomic primitives and so currently don't build futures 0.2.

One solution is to use the #[cfg(target_has_atomic = "ptr")] check to avoid compiling the futures_core::task::atomic_waker module when not supported, that currently requires using nightly for the unstable cfg_target_has_atomic feature:
diff --git futures-core/src/lib.rs futures-core/src/lib.rs
index edef417d..a3335849 100644
--- futures-core/src/lib.rs
+++ futures-core/src/lib.rs
@@ -4,6 +4,8 @@
 #![deny(missing_docs, missing_debug_implementations, warnings)]
 #![doc(html_root_url = "https://docs.rs/futures-core/0.2")]

+#![feature(cfg_target_has_atomic)]
+
 #[macro_use]
 #[cfg(feature = "std")]
 extern crate std;
diff --git futures-core/src/task/mod.rs futures-core/src/task/mod.rs
index 8a6055af..bca90953 100644
--- futures-core/src/task/mod.rs
+++ futures-core/src/task/mod.rs
@@ -13,7 +13,9 @@ if_not_std! {
     type Exec<'a> = ();
 }

+#[cfg(target_has_atomic = "ptr")]
 mod atomic_waker;
+#[cfg(target_has_atomic = "ptr")]
 pub use self::atomic_waker::AtomicWaker;

 /// Information about the currently-running task.
diff --git futures/src/lib.rs futures/src/lib.rs
index 3096ffe0..409bd32a 100644
--- futures/src/lib.rs
+++ futures/src/lib.rs
@@ -22,6 +22,8 @@
 #![no_std]
 #![doc(html_root_url = "https://docs.rs/futures/0.2")]

+#![feature(cfg_target_has_atomic)]
+
 extern crate futures_core;
 extern crate futures_channel;
 extern crate futures_executor;
@@ -354,9 +356,12 @@ pub mod task {
     //! executors or dealing with synchronization issues around task wakeup.

     pub use futures_core::task::{
-        AtomicWaker, Context, LocalMap, Waker, UnsafeWake,
+        Context, LocalMap, Waker, UnsafeWake,
     };

+    #[cfg(target_has_atomic = "ptr")]
+    pub use futures_core::task::AtomicWaker;
+
     #[cfg(feature = "std")]
     pub use futures_core::task::{LocalKey, Wake};

An alternative solution would be to add an extra atomic feature to futures and futures-core to control when this is built, but that doesn't seem very nice to me.
The last solution I thought of is to just tie futures_core::task::atomic_waker to the std feature. The only current usage of it within the futures facade is itself only compiled with std.

Addressed by https://github.com/rust-lang-nursery/futures-rs/pull/873


Add ecosia search engine and make description more generic

Adds support for the ecosia.org search engine, and correspondingly changed "DuckDuckGo and Google" in the blocklist description to the more generic "search engines".

Thanks for your contribution!
I would like to rewrite filters without CSS :has() when possible for performance issues. Doesn't seam to be possible with Ecosia.


Add Stackdriver Logging to the list of transports

Add a link to Stackdriver Logging transport.

ping @indexzero

Thanks for your contribution! 2463f83 was cherry-picked onto 3.x<EMAIL_ADDRESS>should hit the wild in about a week.
winston@3 is a pretty significant rewrite to the guts of transports, but there is a compatibility layer so your Stackdriver transport should work with no modifications. That said, @MylesBorins mentioned you were interested in being a contributor. If that's still something you'd be up for let me know.


Missing features in DB2 z/OS support

Hello
I see from issue #86  that there is code in the branch dev_zos_support that supports SQLAlchemy connections to z/OS. Thank you for this feature.
I installed the package from this branch and was trying to run a few tests. When I try to list all the schemas on my z/OS instance, I get an empty array as response. I know for a fact there are multiple schemas on this instance.
from sqlalchemy import create_engine
from sqlalchemy.engine.url import URL
from sqlalchemy import inspect

# uri contains connection string to connect to db2 z/OS
engine = create_engine(uri, case_sensitive=False)
connection = engine.connect()

inspector = inspect(engine)
schemas = inspector.get_schema_names()
print(schemas)

However, when I am able to list tables within a schema without any issue.
from sqlalchemy import create_engine
from sqlalchemy.engine.url import URL
from sqlalchemy import inspect

# uri contains connection string to connect to db2 z/OS
engine = create_engine(uri, case_sensitive=False)
connection = engine.connect()

inspector = inspect(engine)
tables = inspector.get_table_names("my_test_schema")
print(tables)

Is there a reason one call would fail while the other succeeds?
Please help with this.
Also, is there a timeline for the release supporting db2 z/OS?
Thanks

HI,
Thank you for pointing out the issue.
I did see this and will fix it along with the actual release.
We will be releasing the new version in this week itself.
Thanks.

@amukherjee28 Has this been released yet?

Hi @shreyas-shankar
I have release the new version 0.3.6.
Please update to the latest version for ZOS support.
Thanks.

Thank you. I will try the new release.


Migrate from isomorphic-fetch to fetch-everywhere (React Native support)

This pull request replaces isomorphic-fetch with fetch-everywhere to support React Native. React Native support has been requested in these issues: #119 #111 #106

Coverage decreased (-0.7%) to 99.273% when pulling ec5d1697b901569c730a2a0dd557b8131408a4e9 on makeshopio:master into e72ac27801cab4f5c5cedfaa8d1787e0ffce6f5b on agraboso:master.

I think an import of fetch should be removed. According to the specs, fetch is supposed to be global, and therefore my vote goes out to do it that way as well. Let fetch be global, and let redux-middleware-api use the global. That way anyone can choose for his own environment what polyfill to use.

The next branch (which will lead to version 2) removes isomorphic-fetch in favor of the global fetch.


docs: homepage, concepts cleanup

@fforootd @livio-a could you take a look at the b2b, b2c texts if this makes sense. If so, could you extend Access Control and maybe what we do for attack prevention (in B2C) 😀

When @fgerschwiler has no objections is would recommend merging this 😁

With Node.js version >v17 we will starting see an error
  opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}

This relates to https://github.com/facebook/docusaurus/pull/5814
A simple upgrade should do the trick without using the legacy flags

@peintnermax do you have any ideas around this error?

the new docusaurus version had a problem with our login-users guide, so i cleaned it up. The commented architecture scenarios are not yet reviewed so I would have merged the new homepage and concepts instead :)

The commented architecture scenarios are not yet reviewed so I would have merged the new homepage and concepts in the meantime :)

@mffap @fgerschwiler any comments from your sides for the guides?

The commented architecture scenarios are not yet reviewed so I would have merged the new homepage and concepts in the meantime :)

@mffap @fgerschwiler any comments from your sides for the guides?

CF built failed, so I can't preview. Did something material change?

CF built failed, so I can't preview

Yeah the CF build job used node 12 per default. I changed this now to use 17.3.0

The preview is now working again check the latest here

@livio-a can you take a look at the implicit flow guide? https://github.com/caos/zitadel/blob/b2bguide/docs/docs/guides/authentication/authmethods/implicit.mdx

@peintnermax @livio-a
You broke some links can you have a look into this? 🙇
The test can be run with yarn build
[ERROR] Unable to build website for locale en.
[ERROR] Error: Docusaurus found broken links!

Please check the pages of your site in the list below, and make sure you don't reference any path that does not exist.
Note: it's possible to ignore broken links with the 'onBrokenLinks' Docusaurus configuration, and let the build pass.

It looks like some of the broken links we found appear in many pages of your site.
Maybe those broken links appear on all pages through your site layout?
We recommend that you check your theme configuration for such links (particularly, theme navbar and footer).
Frequent broken links are linking to:
- ../../apis/openidoauth/endpoints#authorization_endpoint
- ../../apis/openidoauth/endpoints#error-response

Exhaustive list of all broken links found:

- On source page path = /docs/guides/authentication/authmethods/basic:
   -> linking to ../../apis/openidoauth/endpoints#authorization_endpoint (resolved as: /docs/guides/apis/openidoauth/endpoints)
   -> linking to ../../apis/openidoauth/endpoints#error-response (resolved as: /docs/guides/apis/openidoauth/endpoints)
   -> linking to ../../apis/openidoauth/authn-methods#client-secret-basic (resolved as: /docs/guides/apis/openidoauth/authn-methods)

- On source page path = /docs/guides/authentication/authmethods/implicit:
   -> linking to ../../apis/openidoauth/endpoints#authorization_endpoint (resolved as: /docs/guides/apis/openidoauth/endpoints)
   -> linking to ../../apis/openidoauth/endpoints#error-response (resolved as: /docs/guides/apis/openidoauth/endpoints)

- On source page path = /docs/guides/authentication/authmethods/jwtpk:
   -> linking to ../../apis/openidoauth/endpoints#authorization_endpoint (resolved as: /docs/guides/apis/openidoauth/endpoints)
   -> linking to ../../apis/openidoauth/endpoints#error-response (resolved as: /docs/guides/apis/openidoauth/endpoints)

- On source page path = /docs/guides/authentication/authmethods/pkce:
   -> linking to ../../apis/openidoauth/endpoints#authorization_endpoint (resolved as: /docs/guides/apis/openidoauth/endpoints)
   -> linking to ../../apis/openidoauth/endpoints#error-response (resolved as: /docs/guides/apis/openidoauth/endpoints)

- On source page path = /docs/guides/authentication/authmethods/pkcenative:
   -> linking to ../../apis/openidoauth/endpoints#authorization_endpoint (resolved as: /docs/guides/apis/openidoauth/endpoints)
   -> linking to ../../apis/openidoauth/endpoints#error-response (resolved as: /docs/guides/apis/openidoauth/endpoints)

    at reportMessage (/home/ffo/workdir/zitadel/docs/node_modules/@docusaurus/utils/lib/index.js:306:19)
    at handleBrokenLinks (/home/ffo/workdir/zitadel/docs/node_modules/@docusaurus/core/lib/server/brokenLinks.js:138:35)
    at async buildLocale (/home/ffo/workdir/zitadel/docs/node_modules/@docusaurus/core/lib/commands/build.js:155:5)
    at async tryToBuildLocale (/home/ffo/workdir/zitadel/docs/node_modules/@docusaurus/core/lib/commands/build.js:33:20)
    at async mapAsyncSequencial (/home/ffo/workdir/zitadel/docs/node_modules/@docusaurus/utils/lib/index.js:262:24)
    at async build (/home/ffo/workdir/zitadel/docs/node_modules/@docusaurus/core/lib/commands/build.js:68:25)
error Command failed with exit code 1.


fix(docs): Update the Security Hall of Fame with an Absolute URL

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #53476

@dyx9
It looks like one of the automated tests is failing.
I would suggest updating your branch with the latest changes from main to fix that.

@dyx9
It looks like one of the automated tests is failing. I would suggest updating your branch with the latest changes from main to fix that.

I've updated the branch, it now passes all tests


bash helpers for android (dotnet is not available)

On the Android there is a terminal emulator application, named https://termux.com/
Using this application I can install git, opensll, openssh, etc.
For this environment three helpers are needed:
list.sh
lists the files the current folder. Sample output:
  gmail.com
  github.com

decrypt.sh file
decrypts file infile with openssl aes-256-cbc -d -salt -pbkdf2 and prints its content
encrypt.sh outfile
encrypts the stdin into outfile with openssl aes-256-cbc -e -salt -pbkdf2.
if the environment variable PWDPWD is not set, the first script that runs asks for the password and sets it. Other invocations of the script reads encode and decode using this environment variable.

implemented


Chat notification popup is redirecting to Apps Page

Hi..
I have deployed communication app and noticed on Desktop client chat notifications are redirecting to Apps Page instead of directly opening the chat. This is stopping us for deploying the app for all users.

Hi..Do we have any update for the query?

Are you saying that users get a desktop client notification when any message is sent to them using the Author tab, but when users click on that notification, they get redirected to "Apps" screen instead of the message?

Yes, I am facing the same issue. Even tried updating the apps by changing the version in manifest files... still the issue persists...

Hello @pabbinedi,
We are not able to repro this issue. Which version of Company communicator you are using? Please try to update it to latest.
Let us know if issue still persists
Thanks.


⚠ Bofrost.es has degraded performance

In 6a270d0, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1994 ms

Resolved: Bofrost.es performance has improved in 7f32294 after 6 minutes.


No 'Access-Control-Allow-Origin"

Hi again,
I get this error on the browser console.
XMLHttpRequest cannot load https://s3-us-west-1.amazonaws.com/mybucket-dc-ab. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3050' is therefore not allowed access.
As the manual said I put s3.upload() on the client side, so why is says it says "No 'Access-Control-Allow-Origin"?
this is event handler (test1.js on the client folder)
Template.test1.events({
"click button.upload": function () {
Meteor.call('configAWSS3');
       var files = $("input.file_bag")[0].files;

       console.log(files);
       S3.upload({
            files: files,
            path: "subfolder"
        },
        function (e, r) {
            if (e) {
                alert('error: ' + e);
            }
            else {
                console.log(r);
                alert('success');
            }
        });
}

})
and my CORS config on s3 bucket is exactly as manual says.
Thanks

I found the solution, so i leave it here for pother people:
my s3 is on us-west1, but when I put it=> region: "us-west-1"  it does not work and gives the above error. I changed it to "us-east-1"  and it works! I really do not know why but now it works!

Lol, that's great to hear.


model parameters contains a lot of 'nan' at certain stage of training using AdamOptimizer

I am training a deep neural network using a simple and well defined case. The training goes very well and reach a high accuracy (~95%), and then, suddenly, just one step iteration, the accuracy is drop to almost zero. I then printed out all the model parameters, and there are lot of 'nan', something like this
....0.076455489, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, 0.08646702....
Here is the information about my system.

Installation. I installed tensor flow from source code on AWS ubuntu g2 instance (GPU instance). During the installation, I use customized configuration so that the tensor flow can run on g2 instance with computing capacity 3.0.

DNN structure: I have a 30 hidden fully connected layers. Each layer has 32 nodes. To make the DNN more stable, I have added short-cuts for every three layers (similar to a recent publication from Microsoft Research "Deep Residual Learning for Image Recognition", by Kaiming He et al.)

Optimizer: AdamOptimizer.
I first found the problem with AdamOptimzer. However, I just realized that the same problem also happens to GradientDescentOptimizer.

Procedure to produce the bug.

I have a saved model, which has good accuracy, and I restart the training by reading the saved model, and the problem shows up for the first iteration.
I wonder if this is a known issue or not. I would be happy to provide my code and input data if this is necessary. I need to create a simple case with simplified code as I am working on a production code that I cannot post it here.

I'm not hugely experienced with either neural networks or Tensor Flow, but it sounds like it could be the "vanishing gradients" problem. When you have a very deep network (and30 layers is very deep), the gradients in the loss function become very small, and I have seen elsewhere that this can cause NaN values. It sounds like in the single step you mention, the gradients reduce and cross a precision threshold, and therefore become undefined.

As for a solution, you should just try decreasing the number of layers in your network. (Possibly) increasing the precision of your data might help, e.g. to ft.float64.

This type of question is better handled at StackOverflow, please ask it there. We're trying to keep github issues for bug reports. Thanks.

I report it as a bug. I will post a simple code for reproducing the bug.

Yes, please post some code to reproduce. Getting NaNs during training is quite common and not usually a sign of a bug. A small code snippets that shows an actual bug will help a lot. I'll reopen this once you post code to reproduce.

Hi Martin,
Here is the code for reproducing the bug. I have made the code much simpler, and only have two hidden layers, a very moderate NN.
To run the code, type
python TFDebug.py
After about 5000 iterations, the bad model parameters will be printed out.
I have tested it on all platforms that I have (ubuntu CPU, ubuntu GPU and Mac OS).
All show the same type of error.
Many thanks.
BWT, since I have the source code, if you can point out which codes are related to the problem, I can also try to debug it.
Jiabo
TFDebug.txt

Please use the check_numerics op to find out where the NaNs are first introduced. In particular, split the .optimize() call on the optizer into compute_gradients and apply_gradients, so you can look at the computed gradients. If you see any variable get really large or very small, that's a sign of trouble.
The distance function you wrote may be the cause, you could try making sure the value inside the square root is greater then 0 before you take the square root.

I think it's a bug in your code rather than in TensorFlow. Your confidences converge to 0's and 1's, then you take a log of 0 which gives -Inf, and multiply it by 0, which gives NaN

Thanks! I got too hypnotized by the sqrt to see the log. :)

Thanks, I will take a closer look at the code.
Yaroslavb, could you explain your idea in a little more detail? I am not sure what you mean "confidence converge to 0's and 1's.

You are taking a tf.log(y), and you get a 0 as an element of y as you train

Good point. Thanks.  However, y comes from a softmax function, which should always return a positive value, right?  I will check if some elements in y is actually rounded to zeros. Any simple trick to avoid log(0)? Add a very small positive number?
I also check the distance function, and I don't think the problem comes from it. To make sure that is not a problem, I even put a check before taking sqrt.
I will dig a little more deeper into the code, following Martin's suggestion.

Mathematically, softmax can never be zero, but because we use floating point numbers, a number close enough to zero is represented as zero. Anyway, this is a common issue with neural networks, search for NaN and softmax

Many thanks. I appreciate very much your comments!

I have verified that the problem is indeed from log(y). I added a small constant to y, and the problem is solved. Thanks to all.


Directives link API documentation did not describe or mention the controller param.

The 4th param for link() was not defined in the api doc, but used in the examples. This change adds the param and describes it.

It is not that simple :smile:
Basically, there is a fifth parameter (transcludeFn) and the controller(s) paremeter value depends on several factors. I suggest you briefly mention the 5 parameters and link to $compile API docs for more info.
@hahla, could you make that change ?

Yes sounds good, will update
On Tue, Oct 6, 2015 at 11:43 PM Georgios Kalpakas<EMAIL_ADDRESS>wrote:

It is not that simple [image: :smile:]
Basically, there is a fifth parameter (transcludeFn) and the controller(s)
paremeter value depends on several factors. I suggest you briefly mention
the 5 parameters and link to $compile API docs
https://docs.angularjs.org/api/ng/service/%24compile#-link- for more
info.
@hahla https://github.com/hahla, could you make that change ?
—
Reply to this email directly or view it on GitHub
https://github.com/angular/angular.js/pull/13028#issuecomment-146091481.

--
Sent from my iPhone

Fantastic !
Feel free to ping when you're done, so I can take another look.

@gkalpak updated and ready for your review

@hahla, thx ! I left a couple of comments.
I wonder if can link directly to https://docs.angularjs.org/api/ng/service/$compile#-link-.
LGTM with the comments addressed.

@gkalpak few comments:

I tried to link (no pun intended ;)) to the $compile.link function, but was unable to. AFAICT it is because of the way compile.js is written and link is not a public function.  Would love to link directly to the $compile#methods_link so if you have a suggestion there let me know

I did another round of edits to improve readability in the context of the article example ("build a directive to display the current time"), while giving the key information on the link function. I decided to not copy the full description of the controller parameter because it was going to be a bit too detailed at this point in the docs and would be distracting to have sub-bullets.

added an alert-info to the page which gives a clear link for more detail on the $compile API.

Anyway let me know what you think.

@gkalpak Looks like there was a CI error with the protractor execution, but I don't see any code errors on my side? How do we get Travis CI to re-run the tests, or should I send in a no-op commit to re-trigger the CI test?

LGTM. I restarted the job  (it was indeed a flake).
I'll merge if Travis is happy.
Thx @hahla for working on this :+1:

Backported to v1.4.x as 8a944b0.


Code comments to resolve Javadoc errors

This PR is related to discussion item https://github.com/synthetichealth/synthea/discussions/1345.  Need to evaluate if this level of code commenting is appropriate, or whether we need to relax linting rules.

My vote is that comments like these where it's just a rehash of the code (or in this case, the code itself verbatim) are not useful, and in fact have negative value because now you have neither a meaningful comment nor a warning that says "you need to have comments".
Ideally we'd have real comments describing the what and/or why of everything but I realize that's a nontrivial effort. Maybe there's a flag we can use to disable that linter on a given field such as these BB enums? Loosening the restrictions on comments globally sounds like admitting defeat


New python samples

@nedrebo Why shouldn't this pass the CI? The CI barely does anything, no samples are run. If the samples were actually run, then it should fail.

@nedrebo Why shouldn't this pass the CI?

Because you try to import a non-existing module.


Matter.js tutorial not working

I have exactly this
// module aliases
var Engine = Matter.Engine,
    Render = Matter.Render,
    Runner = Matter.Runner,
    Bodies = Matter.Bodies,
    Composite = Matter.Composite;

// create an engine
var engine = Engine.create();

// create a renderer
var render = Render.create({
    element: document.body,
    engine: engine
});

// create two boxes and a ground
var boxA = Bodies.rectangle(400, 200, 80, 80);
var boxB = Bodies.rectangle(450, 50, 80, 80);
var ground = Bodies.rectangle(400, 610, 810, 60, { isStatic: true });

// add all of the bodies to the world
Composite.add(engine.world, [boxA, boxB, ground]);

// run the renderer
Render.run(render);

// create runner
var runner = Runner.create();

// run the engine
Runner.run(runner, engine);

I am getting an error
matter-js: Render.create: options element was undefined, render.canvas was created but not appended warn

How can I fix it?
Also here is the HTML code
'''

'''

Hey, Hopefully this can help out:
Try moving your script tag for your script.js into the body itself:
<!DOCTYPE html>
<html>
  <head>
    <title>Matter.js</title>
  </head>
  <script  type="text/javascript" src="/matter.js"></script>

  <body>
    <script  type="text/javascript" src="/script.js"></script>
  </body>
</html>

If you're script.js is not being loaded within the body itself, it doesn't have any context as to what the "body" is that the Render.create() method is trying to use.
Give it a test by (before making a change) try console.log(document) and see what props it has.

As @CarterCodesRSA says, I think you need to make sure your HTML page is complete and make sure the script is loaded last so that the body element is available when it runs.


Missing links: AF Portal & myfitness

Additional missing links:
dps

DEERS: https://milconnect.dmdc.osd.mil/milconnect/

ID Card Office Online: https://idco.dmdc.osd.mil/idco/


Optimize NN::Cpid less than and greater than operators

Compare bytes directly instead of ToString(). Shaved nearly 100 ms off the testnewsb call on my (slow) computer... 🙂

I agree. We don't need the ToString, because the nature of the ordering is irrelevant. It is simply to ensure consistent serialization/deserialization.


Make packages, prepare, before, & after standard

By moving these 'hooks' to hookit, it alleviates the stress on engine developers to support them, and gives users a standard customization offering across all engines.

Did you mean to exit in the build script if the before/after hooks aren't present? They should be optional.

oops, nope, that was leftover from when i had them in their own scripts, I'll fix that just now

Closes #34


Support prod deployments via Pyvespa

This PR adds rudimentary support for performing deployments to the prod environment using Pyvespa.
The process is as follows:

Define an application package with a ContentCluster and ContainerCluster that fulfil the minimum resource requirements (at least 2 nodes per cluster, min-redundancy>=2)
Add a DeploymentConfig to the application package (used to generate deployment.xml)
Call the VespaCloud.deploy_to_prod() method

Here's a minimal example:
from vespa.package import DeploymentConfiguration

deploy_config = DeploymentConfiguration(
    environment="prod",
    regions=["aws-us-east-1c", "aws-us-west-2a"],
)

package = ApplicationPackage(
    name=application,
    deployment_config=deploy_config,
    clusters=[
        ContentCluster(
            id="deploytest_content",
            nodes=Nodes(count="2"),
            document_name="deploytest",
            min_redundancy="2"
        ),
        ContainerCluster(
            id="deploytest_container",
            nodes=Nodes(count="2"),
        )
    ],
)

app = VespaCloud(
    tenant=os.environ["TENANT_NAME"],
    application=application,
    key_content=None,
    key_location=api_key_path,
    application_package=package,
)

app.deploy_to_prod()

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Please add an integration test for this functionality, then we should also consider changing all examples to use the new cluster abstractions?

Will add integration test in new PR.


Minisign signatures done with 0.10 are not valid

This appears to be due to a change in the format adding "hashed" into the comment line
https://github.com/jedisct1/minisign/releases/tag/0.10

Fixed in https://github.com/infraweavers/monitoring-agent/releases/tag/0.1.741


Move W3SchoolsClientTest

Currently at src/test/java/com/google/step/snippet/W3SchoolsClientTest.java
Should be at src/test/java/com/google/step/snippet/external/W3SchoolsClientTest.java

Addressed in #50