You're using an expired Personal Access Token, please get a new PAT. More info: https://aka.ms/vscodepat

You're using an expired Personal Access Token, please get a new PAT.
More info: https://aka.ms/vscodepat
Looks like my token expired since I last published. Time to fix.

Fixed by following https://code.visualstudio.com/api/working-with-extensions/publishing-extension#get-a-personal-access-token


Final Changes

All the changes and form validation has been done.

Added Form validation
Changes in UI of MainForm.tsx (..../generate-profile)

Looks good to me! Thank you so much =D

@khairalanam Can you pls do something?

It is showing not accepted.


Chart drawing problem with Angular 1.3

After updating my angular to 1.3, I noticed some troubles with the first draw of my charts.
Indeed, I'm showing my chart and adding data to it at runtime and it seems to be a problem with the new version of Angular.
The legend is broken and the chart is generally weirdly drawn.
See these example:

With latest Angular 1.2 (1.2.27): http://jsfiddle.net/5kazz1c9/ (Working)
With Angular 1.3.0: http://jsfiddle.net/5kazz1c9/1/ (Broken)

A simpler example (also broken), that simply adds some series before showing the chart:
http://jsfiddle.net/5kazz1c9/2/
So far, I have tried to place the initChart() function of highcharts-ng in a $timeout call to be sure it is run after the DOM is initialized. It works for the drawing problem but there is then some troubles with the chart not filling its parent Div.

Hi,
I'm no expert but removing ng-show (or changing the default value to true) fixes the issue with the legend. In general, I've had trouble with Highcharts and CSS's "display: none !important" selectors on parent elements, used by the ngHide directive (https://docs.angularjs.org/api/ng/directive/ngShow).
Perhaps you could try overriding the default ngHide with the alternative method and test it.
I had a similar problem working with Angular-UI's tabs, which relied on the display hidden property to work.
Good luck! Hope it helps

ng-show is a native feature of Angular and the use-case presented in the fiddle worked in 1.2 so it should work also in 1.3 without needing to remove ng-show...

@ValentinH  That fiddle seems to work ok for me. Are you still having the problem

Indeed it works correctly now :) A recent commit has fixed it even if I don't know which one.

Actually it is pretty weird because the fiddle is fixed with the current version of highcharts-ng but the charts in my app are still broken if I don't add the series in a timeout... I'll try to reproduce it on a jsfiddle.


fix[testing-code]: fix workflow goodbye_message in

testing-code practice

What was changed
workflow.py in testing-code practive
Why?
when doing "Test a Workflow Definition" exercise, the tests were not successful
- The goodbye_message field in the result is Au revoir, Pierre

however the workflow returns Au revoir Pierre not Au revoir, Pierre
Checklist

python -m pytest

Thanks for the PR. However, this mistake is intentional. Part of the exercise is to detect and correct it.


ST7789 Parallel: Work with SYS clock faster than default

When I tried to overclock my Tufty the display went odd (it actually split in two taking up the top half of the screen - seems the display hardware copes OK with the clock being two fast - it just misses every other byte, or something like that).
This PR makes PIO setup take account of the current system clock when specifying the divider.  Using a maximum of 32MHz is a fairly conservative guess at where the safe limit is - my Tufty works OK at 40MHz.  The old default was 31.25MHz if you kept a 125MHz system clock.

Thank you. A default overclock is something I'm really keen to add (upstream support to MicroPython at least), specially for our first-party boards like Tufty where we're sticking an RP2040 onboard. Worked out pretty fine for PicoSystem


Stresses read by pyansys do not match ANSYS at every node

Hi, attached a zipped debug_pyansys.rst,
import pyansys
result = pyansys.ResultReader("debug_pyansys.rst")
result.edge_node_num[0:10]
result.NodalStress(0)[0:3]

I get from pyansys
>>> result.edge_node_num[0:10]
array([ 1,  2,  4,  6,  8, 10, 12, 15, 34, 35])
>>> result.NodalStress(0)[0:3]
array([[-1.2874937e-06,  1.2874934e-06,  5.6843419e-14,  0.0000000e+00, 8.1756007e-06, -8.1756007e-06],
       [-1.1674185e-04, -1.1674478e-04, -3.0856981e-04, -1.7892545e-06, -2.5823609e-05,  2.5835518e-05],
       [-5.7354209e-05, -5.5398770e-05, -1.4944717e-04, -1.0580692e-06, -1.7659733e-05, -3.5462126e-06]], dtype=float32)

From ANSYS with
set,1,1
*get,foo,node,1,s,x
*get,foo,node,1,s,y
*get,foo,node,1,s,z
...

The mesh is done using solid186 (20 node) element, but the shape of the elements in this mesh is prism.  When ANSYS calculates the stress averages, at each location each element is counted only once (and not twice when the nodes of the element are coincident in case of prisms).
debug_pyansys.zip

Hi Alex,
I am looking for a tutorial where starting from creating python then how it need to feed to ansys workbench.
If you have any video or flow chart.
Kindly share me.
It will be helpful
Burhan Udden M I

Thanks for identifying that.  Looks like I'll have to add in some logic for mixed element types when averaging results at nodes.  I'll have an update for 0.7.0

Alex,
If I understand correctly, this is what I was referring to. Ansys gives
these options for stress averaging while printing nodal stresses:

Average across different real constants but not materials
No average across real constants and material
Averaging across all (i.e. both material and real constant)

Apart from this, we have the option for how principal stresses are
calculated:

Principal stress calculated from nodal average stress (average of
contribution from each neighbourhood element)
Principal stress calculated at Gauss points then extrapolated to the
given node from each neighbourhood element before averaging

Shells get extra option to get stress at:

Mid plane
Top plane
Bottom plane

Regards,
Abhi
On 9 Mar. 2018 1:19 am, "Alex Kaszynski"<EMAIL_ADDRESS>wrote:

Thanks for identifying that. Looks like I'll have to add in some logic for
mixed element types when averaging results at nodes. I'll have an update
for 0.7.0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/akaszynski/pyansys/issues/17#issuecomment-371518394,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXa6V4oHMajOFPswg4VeRM4Jb273wwEgks5tcUvygaJpZM4SYF2u
.

I've corrected the node averaging for mixed element types, but not for shell elements.  Will be released in v0.26
Abhi, please open a separate issue with exactly what you'd like to see for SURF154 elements (and ideally an example as well)


Pass auth URL correctly to browser

Fixes #1051

Oops, I should have turned of jenkins-uwp before creating this PR. It's expected to fail.


Can't find the ArduinoTapTempo library

Hi,
I tried to find the library in Arduino IDE library manager, but I can't find it. Could I get a .zip file or do you have some instructions on how to find it in the library?

Sorry. read the instructions again and found that the src folder has them.


fixed

fixed collision issue, issue #3

some small issue at the top and bottom borders other than that its great!!! :) please try to fix this small bug

okay, working on it

I think I got it, can u check?

added 2nd commit by accident, pls ignore that

!bounty 25


Remove needless MRB_API

ref #3215
If a function (such as mrb_read_irep_file()) is declared without MRB_API
in header file (such as include/mruby/dump.h), implementation of the
function in source file (such as src/load.c) should also defined without
MRB_API.
If MRB_API is mismatch, Visual C++ reports link error with C2375 error
code: https://msdn.microsoft.com/en-us/library/5k6kw95a.aspx

merged. But mrb_str_strlen() should be MRB_API. I will fix.

Thanks.
BTW, how about removing all MRB_API from **/*.c? They are optional. If we have MRB_API in **/*.h, we don't need to put MRB_API into **/*.c. If we only have MRB_API in **/*.h, we don't need to care about this problem.

:+1: We shouldn't need extern only for declaration, not definition. But, I wonder if changing them all at once will have side-effects :thought_balloon:


Support 接尾辞-動詞的 and 接尾辞-形容詞的

Tentative support for "強がっているが怖がりなのだ", please see detail in my diary written in Japanese.

Thank you for reporting chiTra.
This conjugation normalization is not what we expect and we would like to fix it.
However, the tokenizer used for the pre-trained model must be retained.
Therefore, it needs to be rebuilt with the new options.

I'm vague what kinds of "new options" are suitable for BertSudachipyTokenizer...

Sorry for the lack of explanation.
To summarize the current issue, "NORMALIZED_AND_SURFACE" using "conjugation_matcher" is set to be used in the currently released pre-trained model, so we would like to avoid changing this behavior.
The solution is to add a new Enum to WordFormTypes and add a separate process to word_formatters.
Also, the pos_matcherh called at that must be generated as a separate item from the conjugation_matcher.

This seems same to the word_form_type implemented in #50.
Check:
import sudachipy
dic = sudachipy.Dictionary()
pm1 = dic.pos_matcher(lambda p: p[5] == "*")
pm2 = dic.pos_matcher(lambda p: p[0] in chitra.word_formatter.CONJUGATIVE_POS or p[0] == '接尾辞' and p[1] in {'動詞的', '形容詞的'})
set(pm1) == set(~pm2)  # returns True


Players Not Being Able To Use Warps?

Hi. This Is A Very Strange Issue. Whenever I de-op Myself Or Bring My Friends On My Server, it’s impossible to use /warp OR /warps. I Want To Fix This To Specific Warps. EX:

/warp playerlounge (able to do ✔)
/warp adminlounge (unable to do ❌)

Please Help Me.
PS: here are the nodes I used: - essentials.warp.crates
essentials.warp.spawn (etc)
PSS: I’m typing this on my phone and not my pc. Therefor I don’t know what exact nodes I used.
If Anyone Requests the PEX permissions.yml file I will show it.
] Edit by Stormbow: Corrected typo. :-)  [

“I want to Fox this to specific Warps”
Fix* not fox lol

Ok, show me the permissions.yml file please.

Is this another ticket where I don't hear from you for 3-5 days at a time?  LOL

Srry :( didn’t know I troubled u that much ;(

No trouble.  I'm just teasing you.  :-)
I'm here whenever ya need me. :-)

groups:
default:
options:
default: true
inheritance: []
permissions:
- essentials.warp.*
- essentials.warp.list
- essentials.kit
- essentials.kits.*
- essentials.msg
- essentials.ping
- essentials.warp.*
- essentials.warp.cratekeys
- essentials.warp.crates
- essentials.warp.diamonds
- essentials.warp.duel
- essentials.warp.help
- essentials.warp.house
- essentials.warp.minerals
- essentials.warp.portals
- essentials.warp.spawn
- essentials.warp.sspawn
VIP:
options:
default: false
inheritance:
- default
permissions:
- essentials.enchant.*
- essentials.ptime
- essentials.repair
- essentials.me
- essentials.nick.*
- essentials.warp.*
- essentials.warp.*
- essentials.warp.cratekeys
- essentials.warp.crates
- essentials.warp.diamonds
- essentials.warp.duel
- essentials.warp.help
- essentials.warp.house
- essentials.warp.minerals
- essentials.warp.portals
- essentials.warp.spawn
- essentials.warp.sspawn
MVP:
options:
default: false
inheritance:
- VIP
permissions:
- essentials.msg.color
- essentials.seen.*
- essentials.clearinventory
- essentials.enderchest.*
- essentials.getpos
- essentials.warp.*
- essentials.warp.*
- essentials.warp.cratekeys
- essentials.warp.crates
- essentials.warp.diamonds
- essentials.warp.duel
- essentials.warp.help
- essentials.warp.house
- essentials.warp.minerals
- essentials.warp.portals
- essentials.warp.spawn
- essentials.warp.sspawn
Admin:
options:
default: false
inheritance:
- MVP
permissions:
- essentials.ban
- essentials.ban.exempt
- essentials.banip
- essentials.burn
- essentials.clearinventory
- essentials.clearinventory.others
- essentials.ext
- essentials.gc
- essentials.kick
- essentials.kick.exempt
- essentials.kick.exempt.notify
- essentials.ping
- essentials.mute
- essentials.mute.exempt
- essentials.mute.notify
- essentials.tempban
- essentials.ban.notify
- essentials.vanish
- essentials.vanish.effect
- essentials.vanish.interact
- essentials.vanish.see
- essentials.vanish.pvp
- essentials.warp.*
- essentials.warp.*
- essentials.warp.cratekeys
- essentials.warp.crates
- essentials.warp.diamonds
- essentials.warp.duel
- essentials.warp.help
- essentials.warp.house
- essentials.warp.minerals
- essentials.warp.portals
- essentials.warp.spawn
- essentials.warp.sspawn
schema-version: 1
users:

oops
srry ment to paste github link

OK, that's an easy fix.  You just got he permission nodes wrong.
essentials.warp gives you access to the /warp slash command.
essentials.warp.* doesn't give you anything.

soo, ho do i fix this?

Delete everything in the file that says essentials.warp.* and make it essentials.warp.

Do I ad that extra period?
essentials.warp.
(After warp there is a period)

The end of a sentence always has a period.  :P
The permission node essentials.warp inside the gray box does not have a period at the end of it.

Look at the Essentials Permissions List.  Do you see essentials.warp.* on there?  No.  Remove it from your file entirely, everywhere.
The correct node is essentials.warp to give players access to the /warp command.

You know the correct node.  You know how to put permissions in your file.
This ticket is resolved.

sigh it’s not working

Ok. Give me about 30 minutes.  I'll rewrite your file for you.  :-)

OK, use this as your permissions.yml file.

Thx

But there is some warps on there I don’t want players to use. How to fix this?

Delete those permissions from the group(s).


API-37717 IDP Metadata loads from Config as Fallback

Description
To make the retrieving and loading of IDP metadata more stable prior application start up, saml-proxy will first try to make an http request to retrieve IDP metadata before application start up.  If that fails, then saml-proxy will pull the IDP metadata from configs.
Related Issue
https://jira.devops.va.gov/browse/API-37717

Bug ticket to address logger.error abruptly stopping saml-proxy prior to application start up - https://jira.devops.va.gov/browse/API-38222


add mocks

it's hard to test the behavior of ConnectionManager, so I added useful mock class to write the tests

This change is

this change is not so big. I'll close this so check #264


search according to post type

according to the storyboard,
the users shall be able to specify which type of posts they want to search for. i.e. offer, demand, do together, critique.
this functionality is missing on the server side.

Wasn't search implemented as a temporary need? (-> filtering could be instigated by the client via picking the fitting counter-type)


How to turn the OBF bat file to EXE Without detection

Hi, thanks for the code,
How can i turn the BAT file into EXE file without getting flagged as Virus?
almost every way makes the outcome exe file as virus so eventually it's pointless to even obf the bat file since it gets detected as virus anyway at the end

Hi, thanks for the code, How can i turn the BAT file into EXE file without getting flagged as Virus? almost every way makes the outcome exe file as virus so eventually it's pointless to even obf the bat file since it gets detected as virus anyway at the end

bat2exe 0 detections
https://github.com/Pirate-Devs/Pirate-Dropper


Add IBM Cloud to LF-Edge lanscape

Add logo and metadata for IBM Cloud

https://landscape.cncf.io/logos/ibm-cloud-private.svg
Full entry to leverage for PR (website, Crunchbase) = https://landscape.cncf.io/format=card-mode&grouping=no&selected=ibm-cloud-private


[HybridApp] Improve API DevX in HybridApp

Problem
By default when running on an emulator:

OldApp will use the dev API expensify.com.dev
NewApp will use the staging API staging.expensify.com

This causes an issue when switching between the two apps because dev and staging do not share the same database, making authentication fail while switching between Old and New apps.
Solution
I would suggest we copy how NewApp works today for OldApp:

Use dotenv to be able to set a .env file locally with the API constants set in the file
If there is no .env file (as is default since it is gitignored), then we should hit staging.

This means that when you clone the repo, by default we will use staging

If there is an .env file, we should use those values
We should make sure when building for staging/production, to use those values. On NewDot we have a .env.production, .env.staging, etc. I don't know if we need all of those, but we can add them if necessary.

Other issues

https://github.com/Expensify/App/issues/49844
https://github.com/Expensify/App/issues/49845
Work with Jules to create some sort of diagram for documentation

cc @mateuuszzzzz - Please comment here!

With small delay but commenting here 😅

I think @mateuuszzzzz is working on this!

Overall PR seems to be ready. I would like to address one comment here

Labelling this Internal as they're /mobile PRs.

Initial environment config PR is in review

I linked to a couple of seperate DevX issues that @staszekscp has proposed to the description.

Responded to review comment responses. Looking pretty good but I am yet to test, will do that once comments are fully resolved.

The PR is looking good. Just noting that gotcha's like this have been confusing me and I want to ensure we document these well.

As discussed in last week's thread, I'm removing this from #wave-collect. I've put it in the #newdot-quality project under debugability.

No further progress yet


Fixing News Component dark theme bug and sidebar bug

Fixing two bugs ->

News component theme is not changing from preferred theme to dark or light when we changed from toggle theme button
When we open sidebar by hovering and place our cursor down to change the theme of the app , sidebar closes automatically and reopen and closes until we open sidebar again .

Before ->
screen-capture (3).webm
After ->
screen-capture (7).webm

rajdeep7Singh

Ok Sir let me check


Performance telemetry around Routerlicious Driver Network Calls

Work Item
Describe the outcome you expect
Performance telemetry logging for Routerlicious driver internals.
Approach

Add PerformanceEvent.timedExecAsync wrappers to network calls in routerlicious-driver
Add performance calculations and logging to other useful locations

Open questions

What are all the useful locations for logging performance telemetry?
Acceptance criteria

We can confidently report on the performance of the Routerlicious Driver and accurately assess how changes to the fluid service affect perceived performance.

Completed in #5605


[TFLITE]Quantize & Dequantize op

@anijain2305 @FrozenGene  Added Quantize & Dequantize op to tflite frontend.
Please help to review.
I havent added any testcase beacause Im not able to simulate this from testcases as there are no such ops in tf. This tflite ops will be added to network when quantizing models with and inputs are float. And couldnt find any publically available model.
Error:

Model eg:

LGTM. Is there any way to add a test?
@inadob Do you have any suggestions here?

@siju-samuel

Have you tried to recreate the TFL operations using tf.quantization.quantize() https://www.tensorflow.org/versions/r1.15/api_docs/python/tf/quantization/quantize.
Another way to produce quantize/dequantize in the graph is to use converter.optimizations = [tf.lite.Optimize.DEFAULT]. To avoid messing up with the method we already use to test quantized ops, I suggest creating a really simple fp32 TF graph, convert it to TFL as above and then use only that chunk containing "quantize" for testing.

@inadob Thanks. I will give a try.

@inadob Thanks.

I have a more fundamental question - I don't expect Quantize and Dequantize to show up in Tflite models for inference as IIUC these are operators that will appear in the training loop.
Are we expecting users, using tvm as part of a training loop to be using the tflite frontend or would they have to integrate with their own python scripts directly into Relay ?
Ramana

@masahi @anijain2305 @inadob Please help to review and merge this PR. TIA

@FrozenGene @masahi @anijain2305 Please help to review and merge this PR. TIA.

LGTM. I will leave the PR to FrozenGene to resolve his comment.

THANKS！Merged!


Tooltip not showing with large z-index

Utilizing tootip-component in my React component. When placing my component in a parent component with large z-index - the tooltip disappear.
My parent component has z-index: 1050
Temporary I've solved the issue by replacing the Tooltip from @equinor/eds-core-react with Tooltip form @material-ui/core.

Hi!
Did you try setting the z-index on the tooltip higher than the parent component?
We can ofcourse set it astronomically high but I bet someone will always find a use-case for going higher 🤪

Should it have an automatic z-index find algorithm? To prevent hardcoding the z-index. As of now it works with the Tooltip-component in Material-ui.
The issue is for the VectorCalculator in equinor/webviz-subsurface-components. The component is used in a Dash-plugin for equinor/webviz-subsurface. So I do not know which z-index the parent has. I'm using it in a Dash-plugin.

The mui is hardcoded to be 1500 (overridable by its theming), which is why it probably works for you in this use-case. We have "only" 350 for our Tooltip, albeit maybe a bit low on our side, we can ofcourse set the default to be higher :)
I have never heard of algorithms for automaic z-indexs, did you have a specific example in mind or?

My colleague has made an implementation of such a utility previously: https://github.com/equinor/webviz-core-components/blob/master/react/src/lib/utils/dom.ts.
Not sure if it is applicable. As of now I'll stick with mui version due to the default - 350 is a bit low. My hope was to not think of the parents z-index. Everything else in my component works fine - only the Tooltip is hidden due to the 350 z-index

We can increase our Tooltip z-index to 1500 for you (along with our other components) so the chances for it hiding a smaller :)
The algorithm is not applicable for us as the performance hit for traversing the dom-graph in a Tooltip would be to big (even when memonized) and it doesn't really find the highest z-index as it only traverses the specifics graph for which the element resides in.

This has been resolved in our latest release<EMAIL_ADDRESS>


Add OVA page link to /download template and 0-9-1.yml

May need help fixing up ampersands...
What this PR does / why we need it
This pull request adds a link to the Customer Connect page for the OVAs for this release.
Details for the Release Notes (PLEASE PROVIDE)
NONE

Which issue(s) this PR fixes

Fixes: #3114
Describe testing done for PR

Check page on Staging site
Special notes for your reviewer

@pnbrown can this be merged to staging but not upgraded till it actually works?

@pnbrown can this be merged to staging but not upgraded till it actually works?

@qnetter There's a staging that renders in the checks called deploy preview. When the updates are able to build, you'll see it there. It is currently failing.

@stmcginnis I'm having difficulty finding the remaining lint issues - please help.

The jobs give the specific errors. For yamllint:

32:13     error    trailing spaces  (trailing-spaces)

So basically, you just added extra spaces to the end of the old last line when you added the new lines.
For the urllint failure, it looks like that project must have updated their docs and (re)moved a page we were linking to. That is outside this change, so I think we can just ignore that for now. We will fix that separately.

Not sure how this got closed by pushing an update, but now I am unable to reopen and getting permission denied to try to push anything more.
Will open a new one and tag you.


EpochMonitor callback for monitoring progress within epochs

When using a large training set, it can take a long time for the first epoch to complete and display the first monitor update. I wanted to see that progress was being made during the epoch, so I wrote an SGD callback to do this using the method suggested by Ian here https://groups.google.com/forum/#!topic/pylearn-users/Mz1_1yqSYeA
This can be used to print simple one-line updates giving the current epoch and batch at a specified rate, and/or to print monitor updates within epochs at a specified rate.
I also made the file consistent with PEP8.

Could you rebase the commits you've added against current master? That would make it easier to review this. But it sounds really useful, so thank you in advance.

Sure, I suppose it's about time I figure out how rebasing works anyway

I think I did it right, if not let me know.

It looks good, thanks!
Now that you have fixed all sort of pep8 violations in that file, could you take it off the pep8 whitelist in https://github.com/lisa-lab/pylearn2/blob/master/pylearn2/devtools/tests/test_format.py#L145 ?
Also, don't worry about it now, but for the future, if you can separate commits that do only reformatting from commits that change the code itself, it can make things easier to review.
@dwf, do you want to take another look?

Sure, that sounds like a good idea. I was originally just intending to add the EpochMonitor code, and I fixed the PEP8 without really thinking about it in order to pass the automated tests.

Hmm.. Somehow the build is now failing, with PEP8 problems that apparently aren't recognized by my check PEP8 script. I can fix them, but I wonder why it passed the tests before.

Sorry for the reviewing delay.
Do you think you could add a small test, based on your YAML example, so that this feature does not break in the future? In particular, you could check that the right number of values are in the monitoring channels.
You can use the classes (datasets, and so on) defined in pylearn2/testing for dummy objects.
@dwf did you have other comments?

Sure, I'll give it a try.

It seems I'm failing my own test in the build check, but I'm not seeing why. If I run the test myself, there's no assertion error. Maybe I'm missing something about how the testing is supposed to work?

The buildbot launches the tests in different settings, including with "floatX=float32" in the theano flags, and using python 3.
My guess is that n_batches/monitor_rate is interpreted as floating point division on python 3, and integer division in python 2, so they could give different results. n_batches // monitor_rate should be integer division for both.

Ah yes, that would explain the discrepancy. I didn't see 'from future import division', so I thought I was safe.

Test passing now.


Yarn is required, but only notifies after project creation failure

Fresh install using nvm:  npm install -g @anansi/cli
   create src/pages/Home/price.fixture.ts
   create src/pages/Home/price.stories.tsx
   create src/pages/Home/index.scss
Could not finish installation.
Please install yarn with npm install -g yarn and try again.
If yarn is already installed, try running the following command manually: yarn install

Project setup complete! Opening editor now...
Command failed with exit code 127: "${VISUAL:-code}" ./gms gms/README.md
/bin/sh: 1: code: not found
user@box2:~/zzz$ npm install -g yarn

><EMAIL_ADDRESS>preinstall /home/user/.nvm/versions/node/v14.16.1/lib/node_modules/yarn
> :; (node ./preinstall.js > /dev/null 2>&1 || true)

/home/user/.nvm/versions/node/v14.16.1/bin/yarn -> /home/user/.nvm/versions/node/v14.16.1/lib/node_modules/yarn/bin/yarn.js
/home/user/.nvm/versions/node/v14.16.1/bin/yarnpkg -> /home/user/.nvm/versions/node/v14.16.1/lib/node_modules/yarn/bin/yarn.js
+<EMAIL_ADDRESS>added 1 package in 0.492s
user@xxx:~/zzz$ anansi hatch gms

Added yarn install to docs for now. In future yeoman will hopefully use npx to automatically fallback in these situations


tensorflow 1.4 tf.keras gives different result compared with using keras directly

I have tensorflow 1.4, when running the following code, the accuracy is different (78% vs. 34.90%) when I import Sequential, Dense, and model_from_json directly from keras (uncomment first 3 lines) compared with import from tensorflow.python.keras. Why is the big discrepancy?
(the data pima-indians-diabetes.csv is available at http://archive.ics.uci.edu/ml/machine-learning-databases/pima-indians-diabetes/pima-indians-diabetes.data)
#from keras.models import Sequential
#from keras.layers import Dense
#from keras.models import model_from_json
from tensorflow.python.keras.layers import Dense
from tensorflow.python.keras.models import Sequential, model_from_json
import numpy
import os
fix random seed for reproducibility
numpy.random.seed(7)
load pima indians dataset
dataset = numpy.loadtxt("pima-indians-diabetes.csv", delimiter=",")
split into input (X) and output (Y) variables
X = dataset[:,0:8]
Y = dataset[:,8]
create model
model = Sequential()
model.add(Dense(12, input_dim=8, kernel_initializer='uniform', activation='relu'))
model.add(Dense(8, kernel_initializer='uniform', activation='relu'))
model.add(Dense(1, kernel_initializer='uniform', activation='sigmoid'))
Compile model
model.compile(loss='binary_crossentropy', optimizer='adam', metrics=['accuracy'])
Fit the model
model.fit(X, Y, epochs=150, batch_size=10, verbose=0)
evaluate the model
scores = model.evaluate(X, Y, verbose=0)
print("%s: %.2f%%" % (model.metrics_names[1], scores[1]*100))
serialize model to JSON
model_json = model.to_json()
with open("model.json", "w") as json_file:
json_file.write(model_json)
serialize weights to HDF5
model.save_weights("model.h5")
print("Saved model to disk")

I have no idea, but note that tf.keras is a tensorflow-specific re-implementation of the Keras API. AFAIK the two implementations are very different.

@bchu The re-implementation is expected to achieve the same functionality as keras, though the implementation is different. I actually tried on multiple datasets, the resulting accuracies of the two are very different  (78% vs. 34.90%; 98% vs. 50%).

seems critical if it's true. cc @fchollet

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.

Dear @fchollet , I got a similar issue by using your method(tensorFlow 1.14.0, keras 2.2.4).  Here is my code.
Source code(pure keras import)：
import keras from keras.datasets import mnist from keras.models import Sequential from keras.layers import Dense, Dropout, Flatten from keras.layers import Conv2D, MaxPooling2D from keras import backend as K
Source code(tf.keras import):
from tensorflow import keras from tensorflow.keras.datasets import mnist from tensorflow.keras.models import Sequential from tensorflow.keras.layers import Dense, Dropout, Flatten from tensorflow.keras.layers import Conv2D, MaxPooling2D from tensorflow.keras import backend as K
commom parts:
`import numpy as np
import random as rn
import tensorflow as tf
np.random.seed(2019)
rn.seed(2019)
tf.compat.v1.set_random_seed(2019)
batch_size = 128
num_classes = 10
epochs = 12
input image dimensions
img_rows, img_cols = 28, 28
the data, split between train and test sets
(x_train, y_train), (x_test, y_test) = mnist.load_data()
if K.image_data_format() == 'channels_first':
x_train = x_train.reshape(x_train.shape[0], 1, img_rows, img_cols)
x_test = x_test.reshape(x_test.shape[0], 1, img_rows, img_cols)
input_shape = (1, img_rows, img_cols)
else:
x_train = x_train.reshape(x_train.shape[0], img_rows, img_cols, 1)
x_test = x_test.reshape(x_test.shape[0], img_rows, img_cols, 1)
input_shape = (img_rows, img_cols, 1)
x_train = x_train.astype('float32')
x_test = x_test.astype('float32')
x_train /= 255
x_test /= 255
print('x_train shape:', x_train.shape)
print(x_train.shape[0], 'train samples')
print(x_test.shape[0], 'test samples')
convert class vectors to binary class matrices
y_train = keras.utils.to_categorical(y_train, num_classes)
y_test = keras.utils.to_categorical(y_test, num_classes)
model = Sequential()
model.add(Conv2D(32, kernel_size=(3, 3),
activation='relu',
strides=(1, 1),
padding='valid',
dilation_rate=(1, 1),
use_bias=True,
kernel_initializer='glorot_uniform',
bias_initializer='zeros',
input_shape=input_shape))
model.add(Conv2D(64, (3, 3),
strides=(1, 1),
padding='valid',
dilation_rate=(1, 1),
use_bias=True,
kernel_initializer='glorot_uniform',
bias_initializer='zeros',
activation='relu'))
model.add(MaxPooling2D(pool_size=(2, 2)))
model.add(Dropout(0.25))
model.add(Flatten())
model.add(Dense(128, use_bias=True, kernel_initializer='glorot_uniform', bias_initializer='zeros', activation='relu'))
model.add(Dropout(0.5))
model.add(Dense(num_classes, use_bias=True, kernel_initializer='glorot_uniform', bias_initializer='zeros', activation='softmax'))
model.compile(loss=keras.losses.categorical_crossentropy,
optimizer=keras.optimizers.Adadelta(),
metrics=['accuracy'])
model.fit(x_train, y_train,
batch_size=batch_size,
epochs=epochs,
verbose=1,
validation_data=(x_test, y_test))
score = model.evaluate(x_test, y_test, verbose=0)
print('Test loss:', score[0])
print('Test accuracy:', score[1])`
Result(pure keras)：
Epoch 1/12 60000/60000 [==============================] - 10s 161us/step - loss: 0.2684 - acc: 0.9177 - val_loss: 0.0599 - val_acc: 0.9812 Epoch 2/12 60000/60000 [==============================] - 6s 95us/step - loss: 0.0893 - acc: 0.9731 - val_loss: 0.0408 - val_acc: 0.9859 Epoch 3/12 60000/60000 [==============================] - 6s 92us/step - loss: 0.0658 - acc: 0.9797 - val_loss: 0.0344 - val_acc: 0.9884 Epoch 4/12 60000/60000 [==============================] - 6s 93us/step - loss: 0.0534 - acc: 0.9843 - val_loss: 0.0315 - val_acc: 0.9888 Epoch 5/12 60000/60000 [==============================] - 6s 93us/step - loss: 0.0471 - acc: 0.9859 - val_loss: 0.0297 - val_acc: 0.9903 Epoch 6/12 60000/60000 [==============================] - 6s 93us/step - loss: 0.0431 - acc: 0.9869 - val_loss: 0.0273 - val_acc: 0.9910 Epoch 7/12 60000/60000 [==============================] - 6s 94us/step - loss: 0.0379 - acc: 0.9881 - val_loss: 0.0258 - val_acc: 0.9911 Epoch 8/12 60000/60000 [==============================] - 6s 93us/step - loss: 0.0338 - acc: 0.9896 - val_loss: 0.0253 - val_acc: 0.9917 Epoch 9/12 60000/60000 [==============================] - 6s 93us/step - loss: 0.0315 - acc: 0.9902 - val_loss: 0.0282 - val_acc: 0.9911 Epoch 10/12 60000/60000 [==============================] - 6s 94us/step - loss: 0.0277 - acc: 0.9908 - val_loss: 0.0262 - val_acc: 0.9920 Epoch 11/12 60000/60000 [==============================] - 6s 93us/step - loss: 0.0275 - acc: 0.9910 - val_loss: 0.0253 - val_acc: 0.9918 Epoch 12/12 60000/60000 [==============================] - 6s 92us/step - loss: 0.0252 - acc: 0.9921 - val_loss: 0.0273 - val_acc: 0.9916 Test loss: 0.02725972963081831 Test accuracy: 0.9916
Result(tf.keras):
Epoch 1/12 60000/60000 [==============================] - 7s 118us/sample - loss: 2.2976 - acc: 0.1186 - val_loss: 2.2629 - val_acc: 0.3077 Epoch 2/12 60000/60000 [==============================] - 4s 70us/sample - loss: 2.2391 - acc: 0.2488 - val_loss: 2.1936 - val_acc: 0.5507 Epoch 3/12 60000/60000 [==============================] - 4s 70us/sample - loss: 2.1677 - acc: 0.3645 - val_loss: 2.0994 - val_acc: 0.6332 Epoch 4/12 60000/60000 [==============================] - 4s 70us/sample - loss: 2.0664 - acc: 0.4451 - val_loss: 1.9656 - val_acc: 0.6864 Epoch 5/12 60000/60000 [==============================] - 4s 70us/sample - loss: 1.9266 - acc: 0.5110 - val_loss: 1.7900 - val_acc: 0.7180 Epoch 6/12 60000/60000 [==============================] - 4s 70us/sample - loss: 1.7591 - acc: 0.5528 - val_loss: 1.5817 - val_acc: 0.7464 Epoch 7/12 60000/60000 [==============================] - 4s 70us/sample - loss: 1.5767 - acc: 0.5961 - val_loss: 1.3670 - val_acc: 0.7686 Epoch 8/12 60000/60000 [==============================] - 4s 70us/sample - loss: 1.4031 - acc: 0.6257 - val_loss: 1.1722 - val_acc: 0.7870 Epoch 9/12 60000/60000 [==============================] - 4s 70us/sample - loss: 1.2534 - acc: 0.6560 - val_loss: 1.0119 - val_acc: 0.8047 Epoch 10/12 60000/60000 [==============================] - 4s 70us/sample - loss: 1.1304 - acc: 0.6804 - val_loss: 0.8855 - val_acc: 0.8194 Epoch 11/12 60000/60000 [==============================] - 4s 71us/sample - loss: 1.0355 - acc: 0.6964 - val_loss: 0.7891 - val_acc: 0.8317 Epoch 12/12 60000/60000 [==============================] - 4s 70us/sample - loss: 0.9585 - acc: 0.7164 - val_loss: 0.7156 - val_acc: 0.8390 Test loss: 0.7156027168273926 Test accuracy: 0.839
Any resolution？

@wangyexiang The gap of performance between tf.keras and external keras in your example is because of the default learning rate of Adadelta. tf.keras uses 1e-3 while keras uses 1.0. Once you use 1.0 in tf.keras, you can still achieve the same performance obtained by external keras.
ref1: https://www.tensorflow.org/api_docs/python/tf/keras/optimizers/Adadelta
ref2: https://keras.io/optimizers/
Notebook: https://colab.research.google.com/drive/1UI-QlfR6_sCmJu5XY8yH3397OM7EGXon

Once you use 1.0 in tf.keras, you can still achieve the same performance obtained by external keras.
I see. Thank you very much.  @WindQAQ

Hi, I get difference as well with the same model, but one done with Keras and the other with TensorFlow.
I check the default parameter (for the loss), but until now, I do not have found an explanation.
Maybe using "tf.compat.v1.RMSPropOptimizer"  is not the same than using optimizer="rmsprop" within model.compile (for keras) ?


Error: mkdir(): Permission denied

I have my storage folder above the craft folder like this:

config
craft
public
plugins
storage
templates

After installing the plugin and enabling it in the template files the plugin gives me this error: mkdir(): Permission denied.
I tried creating a nocache folder inside de runtime folder with the right permissions. But still same error.
PHP warning

mkdir(): Permission denied

/Volumes/HDD/Sites/myredlight/craft/app/helpers/IOHelper.php(734)

722     {
723         if ($permissions == null)
724         {
725             $permissions = craft()->config->get('defaultFolderPermissions');
726         }
727
728         $path = static::normalizePathSeparators($path);
729
730         if (!static::folderExists($path, false, $suppressErrors))
731         {
732             $oldumask = $suppressErrors ? @umask(0) : umask(0);
733
734             if ($suppressErrors ? !@mkdir($path, $permissions, true) : !mkdir($path, $permissions, true))
735             {
736                 Craft::log('Tried to create a folder at '.$path.', but could not.', LogLevel::Error);
737                 return false;
738             }
739
740             // Because setting permission with mkdir is a crapshoot.
741             $suppressErrors ? @chmod($path, $permissions) : chmod($path, $permissions);
742             $suppressErrors ? @umask($oldumask) : umask($oldumask);
743             return new Folder($path);
744         }
745
746         Craft::log('Tried to create a folder at '.$path.', but the folder already exists.', LogLevel::Error);

Stack Trace
#0
–  /Volumes/HDD/Sites/myredlight/craft/app/etc/web/WebApp.php(688): CApplication->handleError(2, "mkdir(): Permission denied", "/Volumes/HDD/Sites/myredlight/craft/app/helpers/IOHelper.php", 734)
683         if (version_compare(PHP_VERSION, '7', '>=') && $code === 2 && strpos($message, 'should be compatible with') !== false)
684         {
685             return;
686         }
687
688         parent::handleError($code, $message, $file, $line);
689     }
690
691     // Private Methods
692     // =========================================================================
693
#1
 unknown(0): Craft\WebApp->handleError(2, "mkdir(): Permission denied", "/Volumes/HDD/Sites/myredlight/craft/app/helpers/IOHelper.php", 734, ...)
#2
–  /Volumes/HDD/Sites/myredlight/craft/app/helpers/IOHelper.php(734): mkdir("../../storage/runtime/", 509, true)
729
730         if (!static::folderExists($path, false, $suppressErrors))
731         {
732             $oldumask = $suppressErrors ? @umask(0) : umask(0);
733
734             if ($suppressErrors ? !@mkdir($path, $permissions, true) : !mkdir($path, $permissions, true))
735             {
736                 Craft::log('Tried to create a folder at '.$path.', but could not.', LogLevel::Error);
737                 return false;
738             }
739
#3
–  /Volumes/HDD/Sites/myredlight/craft/app/helpers/IOHelper.php(1502): Craft\IOHelper::createFolder("../../storage/runtime/", 509, false)
1497      */
1498     public static function ensureFolderExists($folderPath, $suppressErrors = false)
1499     {
1500         if (!IOHelper::folderExists($folderPath, false, $suppressErrors))
1501         {
1502             IOHelper::createFolder($folderPath, craft()->config->get('defaultFolderPermissions'), $suppressErrors);
1503         }
1504     }
1505
1506     /**
1507      * Cleans a filename.
#4
–  /Volumes/HDD/Sites/myredlight/craft/app/services/PathService.php(81): Craft\IOHelper::ensureFolderExists("../../storage/runtime/")
76      * @return string The path to the craft/storage/runtime/ folder.
77      */
78     public function getRuntimePath()
79     {
80         $path = $this->getStoragePath().'runtime/';
81         IOHelper::ensureFolderExists($path);
82
83         if (!IOHelper::fileExists($path.'.gitignore'))
84         {
85             IOHelper::writeToFile($path.'.gitignore', "*\n!.gitignore\n\n", true);
86         }
#5
+  /Volumes/HDD/Sites/myredlight/craft/app/services/PathService.php(354): Craft\PathService->getRuntimePath()
#6
+  /Volumes/HDD/Sites/myredlight/plugins/nocache/services/NoCacheService.php(20): Craft\PathService->getCompiledTemplatesPath()
#7
+  /Volumes/HDD/Sites/myredlight/plugins/nocache/services/NoCacheService.php(90): Craft\NoCacheService->getCompilePath("o7Yyxd8CkaPY72PV4Fnu6LhS")
#8
+  /Volumes/HDD/Sites/myredlight/plugins/nocache/NoCachePlugin.php(107): Craft\NoCacheService->render("o7Yyxd8CkaPY72PV4Fnu6LhS", "mwm9BRFP")
#9
 unknown(0): Craft\NoCachePlugin->Craft\{closure}(array("<!--nocache-o7Yyxd8CkaPY72PV4Fnu6LhS-mwm9BRFP-->", "o7Yyxd8CkaPY72PV4Fnu6LhS", "mwm9BRFP"))
#10
+  /Volumes/HDD/Sites/myredlight/plugins/nocache/NoCachePlugin.php(109): preg_replace_callback("/<!--nocache-([a-z0-9]+)-([a-z0-9]+)-->/i", Closure, " <!DOCTYPE html> <!--[if IE 9]><html class="lt-ie10" lang="en" >...")
#11
 unknown(0): Craft\NoCachePlugin->Craft\{closure}()

To be honest, I'm really not sure why this is happening. Your folder structure is a little unorthodox, but I don't think that should be an issue. All I'm using is Craft's built in file helpers, so I'm tempted to say this isn't a problem with NoCache.
If you're comfortable fiddling around, try changing this line to another directory, and see if that gets it working.

I'm seeing this, too. The error is actually thrown by the call to IOHelper::ensureFolderExists() in craft()->path->getCompiledTemplatesPath().
Strangely, calling craft()->path->getCompiledTemplatesPath() from NoCachePlugin::init() doesn't throw an error. Calling it from NoCacheService does :|

Setting your CRAFT_STORAGE_PATH constant to an absolute filepath seems to resolve the issue – the problem is that for some hosts, register_shutdown_function will actually change the working directory, which means relative filepaths like '../storage' won't work. You may be able to use something like define('CRAFT_STORAGE_PATH', realpath('../storage'); to make it work (it does, on my end).
However, it'd be nice if No-Cache actually worked with relative filepaths for the storage directory out of the box – so I just submitted a PR that I believe should fix the problem. The fix itself was shamelessly lifted from http://stackoverflow.com/a/10861674/4663184 :)

@mmikkel I've implemented your PR – I assume this fixes this problem? I'll reopen if needed

@benjamminf Fixes the issue on my end! Thanks :)


Login Two Factor

hi
how login two factor ?
please help me

There is no way to login two factor with this library yet.

@cellojaan how can i add two factor to library ?
you can help me ?

@drsoosmar Sorry but there's no way to integration two factor authority with private api. You must use official API.


Client secret key is not used in get_user_profile?

Hi!
I was playing around with the get_user_profile example and realized that the client secret key is not used in the authentication process. I thought this was necessary for authentication. Does that mean it's not the case?
Thanks for the help in advance!
Amin

The get_user_profile example uses the "authorization code with PKCE" method, which you can read about here. It is meant for client side authorization, in situations where the client secret cannot be stored securely. It uses a verification method instead of the client secret


Gems and rings

Description
Pretty much the whole game
Test Instructions

Play a game against someone else
Unlock new markers
Select different markers
Read the instructions
Check your record

Ticket Links
#233
Readiness Checklist

[x] I have run make format to format the changes
[x] I have compiled the firmware and the changes have no warnings
[x] I have compiled the emulator and the changes have no warnings
[x] I have run make cppcheck and checked that cppcheck_result.txt has no warnings for the changes
[x] I have added doxygen comments to any code used by more than one Swadge mode. This includes /*! \file comments with Design Philosophy, Usage, and Example sections for new headers.
[x] I have run make docs and checked that doxy_warnings.txt has no warnings for the new code

I'm out of town without my laptop right now but I should be able to take a proper look at this on Thursday

I had a neighbor demo it and almost all the feedback went to the instructions, so that's getting overhauled

Oh I love the new tutorial!

Multiplayer died for that tutorial


better compat with sensu-puppet for subdue param

Our internal sensu-puppet doesn't have subdue param on sensu::check yet because this change was merged fairly recently.
This change introduces a simple compat mechanism to allow us and anybody else who's not running fairly recent sensu-puppet, to continue using monitoring_check.

Hmm. I was actually going to just bump our version of the puppet module, but I understand what you are getting at.
Instead of this though, what if subdue defaulted to undef and we used the https://github.com/puppetlabs/puppetlabs-stdlib#delete_undef_values function to remove it if it was unset?
That way only the people who want to use this feature will be required to provide a sensu module new enough to use it?

I thought about it but I think messing with default values for sensu::check params is not ideal.
I am concerned that by setting default to undef we will be introducing more problems down the road, primarily because sensu-puppet includes a lot of low-level ruby. For example, I'd be concerned about something like https://github.com/sensu/sensu-puppet/pull/379 and https://github.com/sensu/sensu-puppet/issues/374

But we are not messing with the default values of sensu::check, we are using them.
For use, I'm suggesting that subdue be undef, but most of the other values can stay as they are.

I see what you mean. Pushed new diff. I agree that it looks much nicer :)

Ship!


Footer restructure

Check if links and all are accurate, based on the new navbar of the homepage

See comments, URL links, colors, dimensions, CSS references directly on the Artboard

Note: we can't use that logo because we're not an OSI affiliate.

I've created a new onboarding for anyone coming for the first time on the website and willing to chat, whether gardeners and/ or OS contributors: https://goo.gl/forms/ATCZcU6Ohsy57wuI3
So below my updates suggestion for the footer:
*[ ] Replace Logo by the updated one: https://drive.google.com/open?id=0B4a1xw3OY2hBVWplSHFtNjA3ZGM
*[ ] Replace Openfarm/ Contact by FAQ
*[ ] Replace Contribute/ Slack group by Join Slack + link to https://goo.gl/forms/ATCZcU6Ohsy57wuI3
*[ ] Replace Openfarmers/ FAQ by How it works
*[ ] Replace Openfarmers/ How it works by Ask questions
*[ ] Remove Openfarmers/ Help
*[ ] Add Stay in touch/ Slack icon (+ link to https://goo.gl/forms/ATCZcU6Ohsy57wuI3 )
*[ ] Replace Stay in touch/ envelop icon url by mailto<EMAIL_ADDRESS>*[ ] Create a contact form and/ or<EMAIL_ADDRESS>to be redirected to the Slack room?
*[ ] Add a space between & and Privacy

Hi @sophiakc @simonv3, I am working on this issue..

@roselynemakena did great work on this, which is now on the live site.
I want to move a conversation we've had in e-mail to here though, so that it's a bit more public and other people can pitch in if they have strong opinions. CC @roryaronson @sophiakc
This conversation is about whether the Google Form should be by-passable. I think on every step of the way it should be. I also think it's a good opportunity to gather more information about interested users, and I think that's true, but I currently think it's a bit too long.
Rory also said that he didn't think the Slack group was for every day users of OpenFarm, and I wanted to point out that I had told Sophia pretty much the opposite in a conversation a while ago. I don't want to discourage anyone - no matter how technically knowledgeable they are - from joining Slack and asking questions on it about farming. I just don't think  that we'll be able to provide the answers to a lot of them there.
But from research that Sophia has done those kind of Q&A is actually what most people are looking for when it comes to gardening.
SO I guess what I'm saying is that I'm not sure! But the Google Form should probably reflect that.

Thanks @sophiakc @simonv3 , I am super glad to be working with the team :-)
I tend to agree on the by-assability of the form. An option could be have a direct link that allows the user to skip the form and get the invite straight away as @simonv3 pointed out this way we wouldn't discourage a user who may not have all the answers at the time but wants to contribute thoughts/ideas.
About the slack group, I think all users should feel welcome to join a channel of their choice, with channels more suited to best answer their questions/interests. That way in the far future, OpenFarm can have a large community of supporters who can give answers among themselves without necessarily involving the core contributors. The group could also be a useful source of feedback for OpenFarm...

Hi @simonv3 @roselynemakena @roryaronson
My take on that is that the question isn't really should the form be by-passable or not. The main question is more, how to get knowledge about our users, both gardeners and contributors: https://github.com/openfarmcc/OpenFarm/issues/939

Are there issues not addressed in the code done by @roselynemakena that should be added?

The code is great, I don't think our form has been made by-passable though yet. cc @sophiakc ?


Runthrough highlighting some hardcodes

In running setup_terraform.py noticed that some hardcoding was present thanks to reliance on terraform/backends.tf for defaults on the storage account.
Updating and running through README instructions.

Updated for invocation of the initial terraform state via:
./setup_terraform.py -v \
  -i [[redacted]] \
  -s [[redacted]] \
  -g [[redacted]] \
  -rg [[redacted]] \
  -sa [[accountname]] \
  -sc [[containername]]

terraform init -backend-config=backend.secrets \
  -backend-config='storage_account_name=[[accountname]]' \
  -backend-config='container_name=[[containername]]'

It doesn't really simplify things, but means that out of the box runs will work. I'll add this to the readme.


Export ThemeProviderProps in @material-ui/core/styles

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡
ThemeProvider is currently being exported in @material-ui/core/styles  however ThemeProviderProps is NOT also exported here.
If you need to import ThemeProviderProps into your project then you have to import it from @material-ui/styles  instead.
This means having to add @material-ui/styles as a dependency in your project (otherwise you will have extraneous dependencies, which is not a good practice) just to access ThemeProviderProps, which wouldn't be necessary if it were simply exported in @material-ui/core/styles.
It also has the potential to cause issues if ThemeProvider and ThemeProviderProps are modified in the future and a user upgrades @material-ui/core to the new version but doesn't also update @material-ui/styles.
Examples 🌈
Here's an example where the user is creating an AppProvider component to wrap the entire app. It includes react-redux's state provider and material-ui's theme provider.
Current implementation
import React from "react";
import { ThemeProvider } from "@material-ui/core/styles";
import { ThemeProviderProps } from "@material-ui/styles"; // Requires adding @material-ui/styles as a dependency in package.json
import { Provider as StateProvider, ProviderProps as StateProviderProps } from "react-redux";

interface AppProviderProps {
  store: StateProviderProps["store"];
  theme: ThemeProviderProps["theme"];
}

export default function AppProvider(props: AppProviderProps): React.ReactElement<AppProviderProps> {
  const { children, store, theme } = props;
  return (
    <StateProvider store={store}>
      <ThemeProvider theme={theme}>
        {children}
      </ThemeProvider>
    </StateProvider>
  );
};

Better implementation
import React from "react";
import { ThemeProvider, ThemeProviderProps } from "@material-ui/core/styles";
import { Provider as StateProvider, ProviderProps as StateProviderProps } from "react-redux";

interface AppProviderProps {
  store: StateProviderProps["store"];
  theme: ThemeProviderProps["theme"];
}

export default function AppProvider(props: AppProviderProps): React.ReactElement<AppProviderProps> {
  const { children, store, theme } = props;
  return (
    <StateProvider store={store}>
      <ThemeProvider theme={theme}>
        {children}
      </ThemeProvider>
    </StateProvider>
  );
};

Motivation 🔦

Eliminate the need to install unnecessary extra dependencies
Maintain best practices by not importing extraneous dependencies
Ensure no compatibility issues between ThemeProviderProps and ThemeProvider

A PR that adds ThemeProviderProps from /core would be more than welcome.

If you don't mind I would love to take care of that ;)

https://github.com/mui-org/material-ui/pull/20390


Add support for disabling the video call button

We got a request to disable Zulip's in-app video call button, which I think is pretty reasonable.
This can be implemented in the models.py layer by extending VIDEO_CHAT_PROVIDERS with an option for no button (value=0 makes sense for this to me).  And then that value should cause the video call button to be hidden in the frontend templates for both message editing and compose.
Tagging as a priority since it should be simple and valuable.
@sahil839 maybe this is a good issue for you?

@zulipbot claim


IDVA5-448-What is your name test branch

gov.uk links and scripts added for CSS
new design implement
validation separate from common
new test case added and updated

CI: No security warnings found

CI: No security warnings found

CI: No security warnings found


Missing last entry's unit in pm25 data scraper output

@lstillwe - Looking at the pm25.csv sample there are a couple oddities.

The last row is missing the unit information
The unit text is fugly for loading into PostgreSQL

Wondering if we can validate the final unit information being added, and while at it replace the units with standard text, i.e. ugm3 instead of the superscripted bits.
I've done this manually on the sample found in pm25_raw

Huh - thats weird. I’ll check it out.
Lisa
On Mar 20, 2017, at 4:03 PM, Michael J. Stealey <notifications<EMAIL_ADDRESS>wrote:
@lstillwehttps://github.com/lstillwe - Looking at the pm25.csv sample there are a couple oddities.

The last row is missing the unit information
The unit text is fugly for loading into PostgreSQL

[screen shot 2017-03-20 at 3 56 13 pm]https://cloud.githubusercontent.com/assets/5332509/24119003/2740e540-0d86-11e7-9d8a-36cf5cdbe87a.png
Wondering if we can validate the final unit information being added, and while at it replace the units with standard text, i.e. ugm3 instead of the superscripted bits.
I've done this manually on the sample found in pm25_rawhttps://github.com/mjstealey/datatranslator/tree/master/sample-data/pm25_raw
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mjstealey/datatranslator/issues/7, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABtgRGsITpsN6-GhKhzdeSf3KDZss7u2ks5rntt8gaJpZM4Mi8T5.

Michael,
The unit characters (mu, etc) look okay in the raw text file. I wonder if this is an issue in Excel? I wonder if this necessarily translates to a problem importing into the DB?
It looks like now there is an issue retrieving the data for Wilmington - getting a http 400 error message. So this explains why the data didn’t finish writing.
I’ll add a flush of the writer stream just before exiting with the 400 error, but this does not fix the problem. I wish I could find out why certain cities are causing this error. Maybe I should ask James about it.
Lisa
On Mar 20, 2017, at 4:31 PM, Lisa Stillwell <lisa<EMAIL_ADDRESS>wrote:
Huh - thats weird. I’ll check it out.
Lisa
On Mar 20, 2017, at 4:03 PM, Michael J. Stealey <notifications<EMAIL_ADDRESS>wrote:
@lstillwehttps://github.com/lstillwe - Looking at the pm25.csv sample there are a couple oddities.

The last row is missing the unit information
The unit text is fugly for loading into PostgreSQL

[screen shot 2017-03-20 at 3 56 13 pm]https://cloud.githubusercontent.com/assets/5332509/24119003/2740e540-0d86-11e7-9d8a-36cf5cdbe87a.png
Wondering if we can validate the final unit information being added, and while at it replace the units with standard text, i.e. ugm3 instead of the superscripted bits.
I've done this manually on the sample found in pm25_rawhttps://github.com/mjstealey/datatranslator/tree/master/sample-data/pm25_raw
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mjstealey/datatranslator/issues/7, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABtgRGsITpsN6-GhKhzdeSf3KDZss7u2ks5rntt8gaJpZM4Mi8T5.

The unit characters (mu, etc) look okay in the raw text file. I wonder if this is an issue in Excel?

@lstillwe - That could very well be the case. I did open the file in excel and it may have saved unwanted bits into the source file after adding the last empty unit to the file. Will have to redo without excel in the loop and see if that makes a difference.

Looking at this some more,I think the http 400 errors actually are caused by spaces in some of the city and location values. I will encode with "%20" and test.

@mjstealey - I believe this is fixed now - can I close?

closing


cargo-deny ding on an OSI license

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Related issues
A list of issues either fixed, containing architectural discussions, otherwise relevant
for this Pull Request.
Additional Notes
Anything else we should know when reviewing?

#1124
#1120
#1125
#1126  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Auto Crop image

Hai friends......
We take the photo (with A4 Sheet and some background) using camera....... We need to crop the A4 Sheet only (Without background) is possible...? please advice........

It's not possible with Tesseract, as far as I know.
I'm closing an issue.
Feel free to reopen it, if you think it's related with Tesseract-OCR-iOS wrapper.


feat: Add support for aws_bedrock embedder

Description
This PR is to add integration for AWS Bedrock embedder
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] Documentation update

How Has This Been Tested?

[x] Unit Test
[x] Test Script (please provide): Notebook for aws_bedrock

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Maintainer Checklist

[ ] closes #xxxx (Replace xxxx with the GitHub issue number)
[ ] Made sure Checks passed

Hi @hung-phan, Can you please resolve the conflicts so that we can review and test the PR.

@Dev-Khant will do

Updated @Dev-Khant

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

@hung-phan Can you please resolve the conflicts?


add content from 'fun...' and 'standard...'

Content from  'fun with matrices' and the 'standard gate set' that is required for 'proving universality' is added in to an extended version of 'proving universality' .
Fixes #443.

Thanks! done

your welcome!


Update Makefile and add install script target

The install script lets a user build and distribute an easy-to-use self-extracting package that installs the A314 applications on a Linux system.

Improvement idea: add latest release-tag and arch (32/64 bits) to installer filename.

Updated version on merge-branch


Editor: Icon render

Description
Icon renderer is drawing meshes outside of icon in case of mesh contain a lot of surfaces.
Steps to Reproduce
1. Launch WorldEditor
2. Import mesh with several surfaces with center out of (0.0, 0.0, 0.0)
3. Check the icon

Expected behavior:
Icon should contain all mesh
Actual behavior:
Icon contain only part of mesh or nothing.
Reproducibility:
5 of 5
Versions
release-2.0
Additional Information
N/A

Root cause:

Surface position was ignored in bounding box calculations


[R-package] [ci] added more linters

This PR adds a few more R linters:

absolute_path_linter: prevent paths like ~/repos/
nonportable_path_linter: prevent paths like C:\Users\ which only work on certain operating systems
undesirable_function_linter: prevent the use of some functions which could cause issues, specifically:

dyn.load() / dyn.unload(): direct calls to use symbols in .dll / .so files. My expectation is that someone using this in a PR either doesn't understand how to write R packages that call C++ code or is introducing something that will be very difficult to maintain.
help(): call up help documentation. This should only be used interactively, not in code checked into this repository.
install.packages(): install new libraries, e.g. from CRAN. This should only be used interactively, not in code checked into this repository.
ifelse(): This is a convenient but dangerous function, because its output type depends on whether or not the input is all-TRUE, all-FALSE, or a mix.

rbind(): stack two data frames on top of each other (similar to pd.concat() in pandas). data.table::rbindlist() is faster and safer in the case where the data frames have different columns
is.list(): check if an object is a list. I found that in all uses in lightgbm, we are checking if something is exactly a list, not "list-like". is.list() will be TRUE on a data.table object, so using identical(class(x), "list") is safer.
require(): load a package if it hasn't been loaded yet and raise a warning if it is not available. library() is preferred, since hitting a loud error right away that says "this package does not exist" makes debugging easier than getting some of error like "function 'something' not found" further down in the code

undesirable_operator_linter: prevent the use of certain operators in R code, specifically:

%>%, %.%, %..%: variations of "pipe" operators, commonly used in some R packages. The lightgbm package doesn't currently use this programming model, and preventing it with this linter is about preventing a mix of different programming styles in the codebase.
?: call up help documentation. This should only be used interactively, not in code checked into this repository.

This PR also re-ordered the list of linters in .ci/lint_r_code.R to alphabetize them for readability.

@Laurae2 can you take a look when you have a chance? I'm making the R code more opinionated here, would like to be sure you agree with it before merging.

gently ping @Laurae2 for a review

@jameslamb I looked thoroughly and I agree with the changes.
Do you think we should also add cbind() to undesirable_function_linter?

@jameslamb I looked thoroughly and I agree with the changes.
Do you think we should also add cbind() to undesirable_function_linter?

Yep I agree! I'll do that

@jameslamb I looked thoroughly and I agree with the changes.
Do you think we should also add cbind() to undesirable_function_linter?

Yep I agree! I'll do that

@Laurae2 I had to change one of the demos where we were using cbind(). Could you take a look and let me know if it's ok? https://github.com/microsoft/LightGBM/pull/2803/commits/b60e94b2545b8b817919783d7099b58840512559

I just rebased to master to catch up with recent changes.

nit.
Dots for the consistency with other descriptions.

no problem, thanks!
by the way I just pulled this and rebased it to current master. None of the changes on master fail these new linting rules, so we can merge this as-is (without force pushing the rebased version of this branch I have locally) once it builds

@jameslamb Did you see this warning:
* checking R code for possible problems ... NOTE
multiple.tree.plot.interpretation: no visible binding for global
  variable ‘bar_color’
Undefined global functions or variables:
  bar_color

I guess it's from this PR.

@jameslamb Did you see this warning:
* checking R code for possible problems ... NOTE
multiple.tree.plot.interpretation: no visible binding for global
  variable ‘bar_color’
Undefined global functions or variables:
  bar_color

I guess it's from this PR.

@StrikerRUS I did see that a bit after we merged, working on #2901  I was trying to avoid any more R PRs since we have so many open right now, but I think this one would be easy to fix. This comes from the fact that data.table uses non-standard evaluation. so  tree_interpretation[, bar_color] (where tree_interpretation is a data.table) will work correctly, but static analysis of the code makes it look like a variable that doesn't exist is being referenced.
I'll submit a PR right now


chore: add support for path in config traversal

Fixes: https://github.com/flanksource/duty/issues/646

@moshloop The logic can be improved I guess but let me know if this is the direction we want to go for


ci: migrate to github workflow ci

What type of PR is this?

[x] Refactor
[ ] Feature
[ ] Bug Fix
[ ] New Query Runner (Data Source)
[ ] New Alert Destination
[ ] Other

Description

How is this tested?

[x] Unit tests (pytest, jest)
[x] E2E Tests (Cypress)
[ ] Manually
[ ] N/A

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

I've just force merged this.  Hopefully that doesn't screw anything up. :grin:

Thanks heaps for being on top of this @junnplus. :smile:


[Event Request] CU 225 trigger OnRun - new event raise and publisher OnExtendAccTypeCase()

Hello,
at the moment it is not possible to apply account types other than Customer, Vendor or Employee.
For applying e.g. the type "G/L Account" there is no possibility to extend it.
May I suggest a solution approach?

Thank you in advance!
Thanks in advance,
best regards,
G.
Side note: TextConstant Text005 must be different then, or at least include the type "Employee" ("The %1 or %2 must be Customer or Vendor.").

We are going to provide apply functionality for G/L Accounts for Fall 2019 release.


Lag between actions

UI Version: 0.X.X
Description:
I experienced some lag between actions, while this is OK on Desktop app, it can be less accepted by a web user. We can add some spinner just to know that something is happening, either on the whole application or contextualised (probably the last PR from @faboweb already started in this direction)

definitely! we are moving towards a more optimistic ui: actions should be instant and should not block on pre-loading content. there are a number of issues open to address these kinds of things.
#1284
#1596
#865
#984
(there may be others)

Thanks! will use those as reference


kprove: Full variable substitution and function evaluation of specification rules

at initialization phase. Required for logging, performance and predictability.

jenkins: test this please

@denis-bogdanas looks good to me. But could you please add some tests, as I did in #168? Others can understand better what is the specific problem, and also prevent from the haskell backend team to make the same mistake.

@daejunpark This PR doesn't change the functionality. It only facilitates logging/debugging. So there's no need for tests.


Comment Block Disruption Not Marked

How are you using the lua-language-server?
Visual Studio Code Extension (sumneko.lua)
Which OS are you using?
Windows
What is the issue affecting?
Diagnostics/Syntax Checking
Expected Behaviour
The debugger should mark the nested comment block as erroneous, because Lua will still interpret the contents within a comment block for other comment triggers including the close block.
Actual Behaviour
The debugger doesn't mark the comment block as erroneous, instead allowing it to seem fine to have two comment blocks within another.
Reproduction steps

Put a regular full comment block, ie. "--[[" and "--]]" around any block of code.
Put another "--[[" before the closing "--]]".
Try to run that code; should spit out a compilation error.

Additional Notes
No response
Log File
No response

Only affect Lua 5.1 (exclude LuaJIT)


Fixes #31: Changes TimeZoneData Load() to use JsonSerializer.Create()…

… instead of JsonSerializer.CreateDefault() to avoid serializer using custom user settings.

Yep.  Makes sense! Will merge and release soon.  Thanks.

Ping :)

Sorry for the delay.  I'll push a release today, with this change included.  Thanks.

Released in version 3.2.0.  Again, sorry for the delays.


Create a simple music player using basic HTML,CSS and JavaScript

Proposed title of article
[Languages] Create a simple music player using basic HTML,CSS and JavaScript
Proposed article introduction
In this digital era, media(be it audio or video) has become an integral part of our life. Nearly all platforms for education or entertainment integrate either video or audio.Thus everyday at some point we navigate through a player of some sort  be it on your phone, TV or desktop.
The design of the player influences our experience with the media to a very great extent. In some extreme cases a poor design of the player has led to people disliking the media content all together. Therefore, a player should be designed to have minimum "friction" during usage. It should be easy to use,appealing to the eye and most important of all provide the needed functionality. In this tutorial we shall thus tap into the best design of a music player using HTML,CSS and Javascript
Key takeaways

The reader will learn how to positon elements in desired positions using CSS
How to implement a visual design in code
How to add functionality in HTML using Javascript functions

Article quality
The article is unique in the sense that it displays results at the end of each section. The user can thus relate to whether they are achieving the same results. I have linked all the materials I have used in a github repository which they can use to obtain similar results.
The article is simple to follow through and I give some of my best cosing practices to ensure a smooth flow in implementing the music player.
References
Youtube Video: https://www.youtube.com/watch?v=Hd-0vZ4edYM&t=1248s

@mercymeave noted


[Solved] How to enable Error Handling function support?

Update
I've created a PR with the fix as https://github.com/michal-choluj/express-promisify-router/pull/3 and I've also published my own version with the fix, and updated docs:

npm install --save @root/async-router
@root/async-router

Original
I've got a scenario where all four possible params can be passed in (which should only be called where there is an error) but instead it's only getting called every time, and with only 3 arguments (req, res, next).
var app = require('express-promisify-router').Router();
app.use('/', function (err, req, res, next) {
    // should only get called when there is an error
});

I'm trying to figure out what to change for this to work as expected.
So far I've been able to make it skip, but I don't think this actually works as expected.
function wrapFunction(fn, that) {
        // ...
        if (4 == len && 4 !== args.length) {
                // the function is intended to handle errors,
                // but no error occurred in the previous iteration
                next();
                return;
        }
        // ...
}

My current workaround is to move the errorhandler middleware up to the top level as a true express app:
var bigApp = express();
var app = Router();

bigApp.use('/', errorhandler());
bigApp.use('/', app)

app.use('/', ...)

Thank you for your patience. I'm glad to inform you that I've finally resolved the issue and updated the library accordingly.


add StructTypes.jl support

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.20% :tada:

Comparison is base (2223432) 95.09% compared to head (c90e886) 95.29%.
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   95.09%   95.29%   +0.20%
==========================================
  Files           1        1
  Lines         163      170       +7
==========================================
+ Hits          155      162       +7
  Misses          8        8

Files Changed
Coverage Δ

src/StructHelpers.jl
95.29% <100.00%> (+0.20%)
:arrow_up:

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


Let asset pipeline manage the favicon

Description
Use asset pipeline to manage the favicon so that if it needs to be changed, it will auto expire the etag.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thanks!


chore(just): Improve wording in install-resolve prompts

oh if italic is needed it might be worth PRing it into https://github.com/ublue-os/config/blob/main/build/ublue-os-just/lib-ujust/libformatting.sh for the future.


⚠ Thumbnails API Endpoint has degraded performance

In 306d568, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1247 ms

Resolved: Thumbnails API Endpoint performance has improved in 0c198a8.


Fix launch toggle for Native Image Java applications

Summary
This PR will disable toggle and its launch configuration for Java Native Image configuration.
It will also update the README and the Buildpack descriptor accordingly.
Use Cases
When building a native java application container, it will refuse to start assuming you inserted the (Java) toggle layer on a NodeJS application as JAVA_TOOL_OPTIONS is not used for Java native application (the agent being used at build time, not run/launch time).

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Thanks for this PR. I'm not totally sure I follow the problem that it is trying to address though. Can you expand on the issue you're seeing and include steps to reproduce the problem? I'll need to know how I can replicate the issue to do validation test. Thanks

Hi @dmikusa
Sure, I will provide you a way to reproduce and understand the issue.
Reproducing

First, let's get a Spring Boot application with Native Image support from start.spring.io.

Once downloaded, edit the build.gradle file to enable the Databdog buildpack by adding at the end of the file:

tasks.named('bootBuildImage') {
	environment['BP_DATADOG_ENABLED'] = 'true'
}

Once the configuration done, build the project using: ./gradlew bootBuildImage
Finally, run the application using: docker run --rm -p 8080:8080 docker.io/library/demo:0.0.1-SNAPSHOT

You should have the following output:
disabling Datadog at launch time is unsupported for Node
ERROR: failed to launch: exec.d: failed to execute exec.d file at path '/layers/paketo-buildpacks_datadog/helper/exec.d/toggle': exit status 1

Investigating
disabling Datadog at launch time is unsupported for Node means the toggle app believe it is running a NodeJS application.
Why? Because it did not find the JAVA_TOOL_OPTIONS environment variable (code).
This behavior is bogus as JAVA_TOOL_OPTIONS is not expected to be present when using Native Image build.
So my pull request is disabling the addition of the toggle layer when Native Image is used. I removed it as it makes no sense to try to disable the Datadog tracing library once it is baked into the application at build time.
Testing
From here? How do you think we can test it and ensure the toggle feature won't break native image builds again?
Best,
Bruce

This behavior is bogus as JAVA_TOOL_OPTIONS is not expected to be present when using Native Image build.
So my pull request is disabling the addition of the toggle layer when Native Image is used. I removed it as it makes no sense to try to disable the Datadog tracing library once it is baked into the application at build time.

I'm not familiar with the native-image support for agents like Datadog, but if they cannot be toggled without rebuilding the binary then what you're saying makes perfect sense.

From here? How do you think we can test it and ensure the toggle feature won't break native image builds again?

For testing, I think we just want to make sure that the toggle helper isn't added given the circumstances you mentioned.
If you start with a copy of this test that should get you most of the way there. Just wrap your new test in a Context and set up the test conditions in the Before method.  See here for an example of test setup.
It seems like one test with JAVA_TOOL_OPTIONS set to ensure the helper is added, and then one test with it unset to ensure the helper is not added would be sufficient. You can use t.Setenv to set an env safely, it'll automatically be unset after the test. If you just want to ensure the env variable isn't set then, you can os.Unsetenv in the Before block.

Thanks for the clear directions to help me to write tests!
I went with a test that ensures the toggle layer is not added when Native Image build are enabled.
I would rather not relying on JAVA_TOOL_OPTIONS as is will be deprecated in a near future and is related to the toggle logic, not the build logic for whose the built_tests is. Is it okay with you?

Yes, that sounds good. Everything looks good. The branch is just out of date. Normally, GH lets me update it but I can't on this PR. Can you update the branch? Then I can get this merged. Thanks!

Can you update the branch?
Sure! Just did 👌

To follow up. This has been released in the 4.6.0 version of the Datadog buildpack.

Great! Thanks for your help getting this fixed, merged and published 🎉


feat: generic data primitives block builder test framework

Closes #13239

hi @emhane please help me review this PR


More accurate indication of optional/required props

Because the number of required props are much less than the optionals let's indicate the required props by a star.
I experimented a bit with the chart and added a * for the properties that if were missing broke the chart.

Coverage remained the same at 57.391% when pulling ebccc5e404dd09063507b56baea80010eda8456b on aminmarashi:docs into e36ca498e39a3d98d407289e0a08b7de836d4a8f on binary-com:master.

Also Flow is good indicator :)


unexpected formatting when replacing some range selections

Steps to reproduce

place cursor just outside of italic in "hello ┘italic and bold" (where "┘" represents cursor state)
ctrl+→ twice to select "italic and"
start typing

Expected

hello darkness bold

(similar to selection that includes   before italic)
Got

hello darkness bold

Similarly,

hello italic┘ and bold

Ctrl+→ to select " and", type "ized", expected

hello italicized bold

got

hello italicized bold


Configuration Library

Configuration Library.
The goal is to have a configuration infrastructure with the following features:

[x]  The program can set a configuration default internally
[x]  Configuration can be loaded from a config file
[x]  Configuration can be loaded from environment variables
[ ]  Configuration can be loaded from the CLI arguments
[ ]  Configuration is merged/augmented in this order: Default -> File -> Env -> CLI
[x]  The program display the configuration through the program config command
[ ]  The configuration values are available globally and safely in the program

Loading configuration values from CLI: This will probably have to be postponed until support we get support from clap.rs. It's a low priority/need task that will take lots of work that clap.rs might implement or setup the framework to implement in the future.

Configuration is merged/augmented in this order: Default -> File -> Env -> CLI

This might not be possible without moving from config-rs to a better library. The current one can't convert a Struct to the config type for merging another configuration.


[TB] Fix CVE Scan errors from dependency scans

Problem/Concern
Looks like sometime last week our dependency scans started to fail and are still failing.  One example: https://github.com/finos/a11y-theme-builder/actions/runs/7388955840/job/20100832907.  Need to look at whether updating dependencies will address this.
Proposed Solution

fixed in my repo, waiting for PR to be reviewed before merging.

Merged, scans successful.  Closing.


codon build prime.py fails with: error: process for 'gcc' exited with status 255

Running on Linux Debian Bullseye. Installed the pre-built binary (v0.15.5) as per the github page.
I can successfully run the example program for primes (I called it prime.py) with the command:
$ codon run prime.py 99
25

However, I can not build an executable with the command:
$ codon build   prime.py
error: process for 'gcc' exited with status 255

Any insight on how to dig out more info on the failure?

Hi @cvmiller -- which version of gcc is on your system (you can check with gcc --version)? I'm wondering if this is due to an older gcc version.

Well, this is a little embarrassing. gcc was not installed on the machine. So I have installed it, and now I am getting this error:
$ codon build   prime.py
/usr/bin/ld: cannot find -lz
collect2: error: ld returned 1 exit status
error: process for 'gcc' exited with status 1

Is there something else I need to install?
thanks,

This works for me but I'm on macos

For the record, I am using gcc on Linux (Debian Bullseye), and my (installed today gcc)
$ gcc --version
gcc (Debian 10.2.1-6) 10.2.1 20210110
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

@cvmiller Can you try installing zlib, i.e. apt install zlib1g-dev?

Thanks! That did the trick. Now I can build an image (and run it). Now I can see how much faster it is. Thanks again.


Added 6 faq for service-reliability-management-faqs.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[X ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

@deepanshuHarness Can you review and approve this

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Update css-focus-visible.json

Remove extra bugzilla link that only pertains to the adding of documentation -- not to the implementation of the feature in Gecko.

Thanks!


Unable to get a specific number of messages from history

Hi,
I am trying to save the chat history into a file. However, when I use the execute_function function, I always get 40 results (the default number)
n_messages = 0
for message in sender.execute_function('history', 'my_contact', '10'):
    n_messages += 1
# It is always 40
print(n_messages)

What is the way for specify the size of the history?
Thanks!

I can reproduce that bug. I'll look into it.
from pytg.sender import Sender ; s = Sender("localhost", 4458)
n_messages = len(s.history('my_contact', '10'))
print("Length is {length}".format(length=n_messages))
#  40

I located the bug. The problem is that the '10' is a string, so the optional parameter limit which requires an int is skipped.
import logging ; logging.basicConfig(level=logging.DEBUG)
s.history('my_contact', '10')
# log shows:
DEBUG:pytg.sender:Parsing history: Argument user_name - <user> (needed)
DEBUG:pytg.sender:Parsing history: Argument 10 - [limit] (needed)
pytg.exceptions.ArgumentParseError: Not a int/long
DEBUG:pytg.sender:Skipping unfitting optional parameter #1 [limit] (type PositiveNumber) in function history.
DEBUG:pytg.sender:Parsing history: Argument 10 - [offset] (needed)
pytg.exceptions.ArgumentParseError: Not a int/long
DEBUG:pytg.sender:Skipping unfitting optional parameter #1 [offset] (type NonNegativeNumber) in function history.
DEBUG:pytg.sender:Sending command > [disable_preview] history user_name

So that works like intended, the actual problem is that the int as argument does not work.

Fixed.


Checking for distributions (Arch in this case) should be done in a more reliable way

Hi!
In setup.py, on line 249:
    if 'arch' not in platform.platform():
This does not work on many Arch installations. The kernel version reported by platform.platform() is really independent of the distribution.
On my system:
>>> platform.platform()
'Linux-6.6.9-zen1-1-zen-x86_64-with-glibc2.38'

Also see discussion in AUR.
Cheers!

I suggest using platform.freedesktop_os_release(). I haven't red the freedesktop standard (just found it), but something like: platform.freedesktop_os_release()['ID']) == 'arch' could work.
See:

https://docs.python.org/3/library/platform.html
https://www.freedesktop.org/software/systemd/man/latest/os-release.html

The project provides pre-compiled wheels for manylinux_aarch64 on PyPI. AArch64 binaries are also available on conda-forge. If you must build imagecodecs form source and the default configuration does not work, provide a customize_build function in an importable imagecodecs_distributor_setup module. That allows to define which extensions are built and to modify library names, include paths, compiler options, etc.


[chiplevel/padring] Correct mio_in_raw connection width for FPGA targets

Fix #6774
This did not affect the ASIC target as the MIO array has no stubbed pads.
Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

LGTM, let's ignore the test "CI (Execute tests on the Verilated system (excl. slow tests)) ", it's timing out and will be fixed in #6649

OK, will merge this in then.
Yeah, template magic :)...


SSL Verify - verify_certificate fails to apply to page_load tests

Hello,
I've been setting the value of verify_certificate to 0 to disable SSL verification on the test. We are monitoring some back end systems which all use self-signed certificates. I can see that this field is in both this provider and the go-thousandeyes, this provider also see's that the value is different during a plan. However, it does not apply the value correctly.
terraform {
  required_providers {
    thousandeyes = {
      source  = "william20111/thousandeyes"
      version = "~> 0.5.2"
    }
  }
  required_version = ">= 1.0"
}

resource "thousandeyes_page_load" "test" {
  test_name             = "Test SSL Ignore Cert"
  url                   = "https://www.thousandeyes.com"
  verify_certificate    = 0 # Ignored
  interval              = 120
  http_interval         = 120
  content_regex         = "" # Required to avoid "NONE" entry
  agents {
    agent_id = data.thousandeyes_agent.internal-mgmt01.agent_id
  }
  lifecycle {
    ignore_changes = [
      api_links,
      custom_headers,
      include_headers,
      bgp_measurements,
      use_public_bgp
    ]
  }
}

For the plan the difference is detected
  # thousandeyes_page_load.test will be updated in-place
  ~ resource "thousandeyes_page_load" "test" {
      + custom_headers         = {}
        id                     = "2154391"
      - page_load_target_time  = 6 -> null
      ~ verify_certificate     = 1 -> 0
        # (32 unchanged attributes hidden)

        # (1 unchanged block hidden)
    }

During the apply this comes up during the log
2021-07-05T15:03:10.286+0100 [WARN]  Provider "provider[\"registry.terraform.io/william20111/thousandeyes\"]" produced an unexpected new value for thousandeyes_page_load.test, but we are tolerating it because it is using the legacy plugin SDK.
    The following problems may be the cause of any confusing errors from downstream operations:
      - .page_load_target_time: was null, but now cty.NumberIntVal(6)
      - .modified_date: was cty.StringVal(""), but now cty.StringVal("2021-07-05 14:03:06")
      - .verify_certificate: was cty.NumberIntVal(0), but now cty.NumberIntVal(1)
      - .modified_by: was cty.StringVal(""), but now cty.StringVal("Philip Whiteside (<redacted>)")
      - .bgp_measurements: was cty.NumberIntVal(1), but now cty.NumberIntVal(0)

The agent level has a disable SSL verify, but this only applies to page load (broswer bot), however this applies to the http server test. The test then does not have a valid "Availability" despite having a completed page load.

My workaround for anyone else who hits this in the mean time. This will run every apply for every resource. I did not see a way of having a resource provisioned run when only modified, only create/delete.
resource "null_resource" "no-ssl-verify" {
  for_each   = local.tests
  depends_on = [thousandeyes_page_load.default]
  provisioner "local-exec" {
    command = "curl --connect-timeout 30 --max-time 30 --retry 5 --retry-delay 30 --retry-max-time 300 --location --request POST 'https://api.thousandeyes.com/v6/tests/page-load/${thousandeyes_page_load.default[each.key].id}/update.json' --header 'Content-Type: application/json' --header 'Authorization: Bearer ${var.TE_TOKEN}' --data-raw '{\"verifyCertificate\": 0}'"
  }
  triggers = {
    always_run = timestamp()
  }
}

FYI - I just created #59 which is a different issue but I suspect the underlying cause I outlined there probably applies to this issue as well.


Quadratic complexity in search_dates

Using current master (289fcc4ca936069b2c75bb5378b4136633b3edf9) we see quadratic complexity with respect to the number of date-like numbers in the input:
%time _ = search_dates(' '.join(str(2000 + i) for i in range(20)))
Wall time: 907 ms
%time _ = search_dates(' '.join(str(2000 + i) for i in range(40)))
Wall time: 3.4 s
%time _ = search_dates(' '.join(str(2000 + i) for i in range(80)))
Wall time: 14.4 s

Timings with  languages=['en']) are the same.
https://github.com/scrapinghub/dateparser/issues/439 might be related

Hi @lopuhin, I'm trying to reproduce this with the last version (dateparser==0.7.4) and I can't (furthermore, it takes less time to finish than older versions). It seems that it was fixed when we added some performance improvements in the last version.
Could you confirm that it's also fixed for you?
Thanks in advance :slightly_smiling_face:

Thanks @noviluni I can confirm that growth is linear now and constant is much smaller:
In [1]: from dateparser.search import search_dates

In [2]: %time _ = search_dates(' '.join(str(2000 + i) for i in range(20)))
CPU times: user 55.7 ms, sys: 3.99 ms, total: 59.7 ms
Wall time: 59.4 ms

In [3]: %time _ = search_dates(' '.join(str(2000 + i) for i in range(20)))
CPU times: user 31.1 ms, sys: 3.94 ms, total: 35 ms
Wall time: 34.6 ms

In [4]: %time _ = search_dates(' '.join(str(2000 + i) for i in range(40)))
CPU times: user 67.1 ms, sys: 122 µs, total: 67.2 ms
Wall time: 66.8 ms

In [5]: %time _ = search_dates(' '.join(str(2000 + i) for i in range(80)))
CPU times: user 135 ms, sys: 18 µs, total: 135 ms
Wall time: 135 ms

In [6]: %time _ = search_dates(' '.join(str(2000 + i) for i in range(160)))
CPU times: user 246 ms, sys: 16.1 ms, total: 262 ms
Wall time: 262 ms

Thank you! Closing


Bug Report: play here missing

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
Windows (x86_64)
What version of the game are you using? Look in the bottom left corner of the main menu.
Kade 1.6 / 1.6.1
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
just play 1.6, look at the chart editor, the play 1.6.1, and look at the chart editor.
Please describe your issue. Provide extensive detail and images if possible.
play here was removed, care to explain?

stickbug sound plays as PlayHere mysteriously disappears

who removed my baby

https://github.com/KadeDev/Kade-Engine/commit/fa77122877ad915df2d3bbee997ac8251734104a


(ADP-319) Add golden test vectors for Shelley Transactions

Context
ADP-319
We want to add Shelley transaction support.
There is a cardano-cli which should now support creating txs.
3rd part folks also want test vectors.
Decision

Add a yaml file with transaction test vectors

Acceptance Criteria

There must exist a yaml file with transaction test vectors
The wallet must pass these tests

Development

QA

closing, this was done: https://gist.github.com/KtorZ/5a2089df0915f21aca368d12545ab230


[in_app_purchase] Problem with IAPConnection.instance.queryProductDetails

Hi, in my flutter app, IAPConnection.instance.queryProductDetails is returning In-App Purchases but not subscriptions. I have an active subscription in google play console with one active base plan. Please help.

Hi @sgondhi
Which in_app_purchase plugin version are you using?
Please try running plugin sample code to see if it works without this issue. If the issue doesn't happen, please provide a completed and minimal reproducible code sample so that we may verify this. Thank you!


just today on reddit

v.0.11.0—2 of us posted that yesterday’s Daily Note had disappeared…

We investigated the reports of data loss and found the offending plugins. They have since been updated. Periodic Notes wasn't responsible for the issues reported. I'm going to close this issue.

Thanks—I had no idea what the real source was, so I just notified all the most relevant authors I could think of…

On Mar 13, 2021, at 5:03 PM, Liam Cain @.***> wrote:
We investigated the reports of data loss and found the offending plugins. They have since been updated. Periodic Notes wasn't responsible for the issues reported. I'm going to close this issue.


Changing conda selectors syntax

Changing the syntax for the conda [unix] selector (for MPI to install only on linux and osx), because sel(unix) was not widely supported and sometimes led to install problems

Just wondering about the motivation for 9bd78f0. From my understanding the mamba project is a much more performant version of the conda project, and operates as a drop in replacement.

Just wondering about the motivation for 9bd78f0. From my understanding the mamba project is a much more performant version of the conda project, and operates as a drop in replacement.

I was trying to get the original sel(unix) commands to work on conda too (because I hoped that would work with the new libmamba solver). It's needed to make the the instructions for our local system at UU work, as well as make the installation instructions for developers simpler.
As I couldn't get that sel(unix) to work, I tried to revert back to the # [unix] preprocessing selectors (see here and therefore (temporarily) reverted back to conda instead of mamba. Nevertheless, that didn't seem to work in environment files either?
Anyways, I agree this PR is a bit of a mess, but trying to get installation instructions as simple as possible for conda. Any ideas how?

Looks like selectors aren't supported in environment.yml files (only meta.yml) https://github.com/conda/conda/issues/8089. Not sure what the best solution to this is unfortunately. I guess we migrated away from maintaining 3 different environment.yml files?


AHOYAPPS-468: Add hourly CI status check

https://issues.corp.twilio.com/browse/AHOYAPPS-468

Added join room UI test to be used by status check.
Converted all schemes to use test plans to enable multiple test configurations per scheme.
Added Unit, UI, and StatusCheck test plans to Video-Twilio scheme. StatusCheck is configured to not automatically add new tests.
Removed Video-TwilioUITests scheme.
Configured CI to run StatusCheck test plan hourly.
Added SignedInTestCase to make it easier to add UI tests that require signed in user.
Updated fastlane version to support test plan option.
Send Slack message to #eng-sdk-ahoyapps when the status check fails. Message has link to failed build. We are using Android webhook for now until we receive a new one for this.

This is what the Slack message looks like:

Ok this is ready for another review. Sorry for the messy commits to test CI. I think these are the files that had important changes:
.circleci/config.yml
fastlane/Fastfile
VideoApp/Video-TwilioUITests/Activities/AuthActivities.swift
I have not seen CI schedule the hourly build but thinking maybe that will happen after this is merged to master.