Steady state solution vs transient simulation

This got me curious and I wondered if it's possible to converge to a steady state solution rather than a time-dependent solution.
In theory for monochromatic light it should be possible possible to represent oscillation at every point of the scene with just its complex amplitude (so amplitude + phase) and nothing else.
I wrote a quick test to see what the steady state solution looks like:

My approach is not grid-based, so I'm not sure if it's relevant to discuss it in a context of cellular automata, but I still think it is certainly relevant to try and find a steady state solution for your approach too because the process of waves actually propagating is not really needed for the final image.

I think the point of this simulation is to demonstrate how very simple interactions give birth to complex wave phenomena. But your solution is very interesting, what's the basic idea? I struggle to think of a way that is not grid-based.

I think the point of this simulation is to demonstrate how very simple interactions give birth to complex wave phenomena. But your solution is very interesting, what's the basic idea? I struggle to think of a way that is not grid-based.

Instead of simulating a grid, I simulate surface dipoles. Light reflection and absorption can both be simulated by induced surface emitters. These emitters react to incident waves and re-emit them under a specific rule. Then, the final field is simply superposition of all emitters (both initial ones as well as induced ones): instead of a grid, one can calculate interference of surface point emitters.


Auto fetch startBlock

Auto fetch start block

Use etherscan API to fetch contract deploy transaction
Get the blockNumber using eth_getTransactionByHash by calling a public RPC endpoint

This is great! Only left some small comments. What happens if these requests time-out?

Thanks @incrypto32! and sorry for delays reviews


[BUG] Schema object cannot have additionalProperty: Status

Describe the bug
Can't have Status field in methodResponse model
To Reproduce
If a Status field is present in the response model, I get this error:
Error:
AssertionError: Schema object cannot have additionalProperty: Status

SuccessResponse.json

{
  "Status": 200,
  "EmbedUrl": "url",
  "RequestId": "b35fc1ff-578b-4bb4-a975-1abe5574da1b"
}

Here is where it's used in my serverless.yml file:
functions:
  getQuickSightDashboardUrl:
    handler: handler.getQuickSightDashboardUrl
    events:
      - http:
          path: aws/quicksight/dashboardUrl/{id}
          method: get
          cors: ${self:custom.cors-config}
          documentation:
            summary: 'Get a QuickSight dasbhoard URL'
            description: 'Used to get a QuickSight dashboard URL based on its id'
            tags:
              - aws
              - quicksights
            pathParams:
              - name: 'id'
                description: 'The QuickSight dashboard id'
                schema:
                  type: 'string'
                  pattern: '^[-a-z0-9_]+$'
            methodResponses:
              - statusCode: 200
                responseBody:
                  description: 'Success'
                responseModels:
                  application/json: 'SuccessResponse'

From my point of view, you should not check if a field in the response model contains duplicate fields?

That worked. Thanks!


Build basic Users, Roles and Auth

Consider UCD, clean usernames, email and password validations/restrictions.

Add locale as well.

Auth pages

[ ] Sign In
[ ] Sign Up
[ ] Sign Out
[ ] Forgot Password
[ ] Forgot Password Done
[ ] Forgot Password Email
[ ] Password Reset
[ ] Password Reset Done

Breaking this out into separate issues in #6 #7 #8 & #9


doc: usb: add initial USB device configuraiton howto

Add initial documentation how to configure and enable new USB device
support. Use literalinclude to pull code snippets from the samples.

Is device_next no longer experimental? The doc changes suggest so.

There is still a note that it is experimental. I just removed "experimental" from documentation title.

actually, unblocking so that the new sample can get in before feature freeze. @jfischer-no be great if you can address the cosmetic updates in a follow-up PR, thanks!

Thanks, I am on it.


How can I check a successful upload and match?

I would like to be sure that my application has a necessary proGuard mapping files. Is there a way to control uploaded data?
I can see that necessary files are uploaded but could not be sure that they are totally correct files.
 Building 64% > :app:shrinkInternalReleaseResources5% > :app:uploadBugsnagInternalReleaseMapping:app:uploadBugsnagInternalReleaseMapping

Any idea?

Hey @cemo we automatically upload proguard files as part of each build, so they should always be correct. Are you seeing a particular problem?

Closing for lack of response, but can reopen with any new information.


Can not use "root" compartment in "Node Template"

Hello, I'm not able to add "root" compartment for Nodes/VCN/Subnet, because the validation throws an error exception "Specifying a valid oci node compartment is required" or "Specifying a valid oci VCN compartment is required" even thou the OCID is valid (example of beginning of the root compartment OCID: ocid1.tenancy.xxxxxxxxxxxx....). Can you check the validation rules for compartments? Thank you.
Best regards
Peter

The fix for this issue has been merged and back-ported.

I'm still facing this using in both rancher-stable and rancher-latest (Both point to v2.4.8)

Hi @Pelrauns. The issue you are describing originally affected both the OKE cluster driver and the OCI node driver.
The  fix I put in was only on the OKE cluster side, which is why you are still seeing it in the OCI node driver.
I'll submit a fix and ask that it also be back-ported. Sorry for the confusion.

PR:
https://github.com/rancher/ui/pull/4178

See also PR to bump the available versions of OL in the node template:
https://github.com/rancher/ui/pull/4179


Failed to parse cast expression with non-standard types

void test(void) {
   int a = test2((uint64_t) b);
}

That's correct, it shouldn't parse. Types must be defined before use.


ml-webload front end: pick a different color for gpu memory

By default the graphing package uses orangeish for GPU core and yellowish for GPU memory, the difference is not great and hard to remember too.  Better to use eg green for one of them or -- if green creates accessibility problems, I don't know -- at least something other than shades of yellow, red, or blue.

This is bling.  Folded into #236.


Add support for running E2E tests on KIND

Description
Add support for running E2E tests on KIND.
Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist

[X] Unit tests and e2e tests updated
[X] Documentation updated

@Smiley73 can you please rebase your branch with the latest changes from master? This will make the changes easier to review. Thanks!

@seanmalloy synced with master.


feat: split clean_log from rotate by file

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message follows commit guidelines

Affected core subsystem(s)

rotate_by_file
Description of change

split clean_log from rotate by file

Current coverage is 94.40% (diff: 91.07%)

Merging #6 into master will decrease coverage by 3.91%

@@             master         #6   diff @@
==========================================
  Files             5          6     +1
  Lines           119        125     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            117        118     +1
- Misses            2          7     +5
  Partials          0          0

Powered by Codecov. Last update 371c27f...d792aa3

为啥要把这块逻辑抽出来

因为是两个功能

补全用例不容易

可以 review 了

我正在改成面向对象的。

提交一部分先看看

改好了

@fengmk2 @jtyjty99999

我修一下

改了 c69651f

为啥 review 不会折叠，你们选了什么？

继续 review 吧

@jtyjty99999 没问题我就合并了。

2.2.0

@fengmk2 我不小心点退订了，没看到消息。。。辛苦了。。

@popomore 这个有同步到内网版本了么?


Test Randomly Failing: componentBrowser.spec.ts:240:1 › Visualization preview: user visualization selection

See the following failed test run:
https://github.com/enso-org/enso/actions/runs/8232620344/job/22520547106?pr=9342#step:8:707
 1) componentBrowser.spec.ts:240:1 › Visualization preview: user visualization selection ──────────

   Test timeout of 60000ms exceeded.

   Error: locator.click: Test timeout of 60000ms exceeded.
   Call log:
     - waiting for getByRole('button', { name: 'Table' })

     249 |   await expect(locate.jsonVisualization(page)).toExist()
     250 |   await locate.showVisualizationSelectorButton(page).click()
   > 251 |   await page.getByRole('button', { name: 'Table' }).click()
         |                                                     ^
     252 |   // The table visualization is not currently working with `executeExpression` (#9194), but we can test that the JSON
     253 |   // visualization is no longer selected.
     254 |   await expect(locate.jsonVisualization(page)).not.toBeVisible()

       at /runner/_work/enso/enso/app/gui2/e2e/componentBrowser.spec.ts:251:53

Another occurrence: https://github.com/enso-org/enso/actions/runs/8232224820/job/22509187926?pr=9323#step:8:707

Not seen since quite a long time, I think it was fixed by one of the test-fixing PRs.


CoreSpotlight

I got the issue reported as fixed in the last release while updating the pod this morning (to 0.11.4)

removes the need to include CoreSpotlight
I am compiling from Xcode 6.4

Is the patch available in this cocoapod release?

@pgherveou yes it is. Please let us know if you're still running into trouble


[HPRO-415] Add link to open embedded PDFs

[HPRO-415]
Users who's browsers are configured to not allow embedded PDFs previously had no way of downloading or opening the PDFs in a new window/tab.  This adds a link that opens the PDF in a new window.  If a browser is not configured to view PDFs at all (embedded or not), the PDF will be downloaded (and, at least in Chrome, the new tab will be automatically closed).
A direct link to PDFs has been added for biobank order labels, biobank order requisitions, and SOP documents.
Chrome setting that breaks embedded PDFs

New open/download link

We are also embedding pdf in Technical FAQs page so I think we should also add this option over there as well.


Search results not searching through file contents (as the default windows explorer does)

As described in the title, it doesn't seem as if the Files App searches through file contents when searching within a folder (regardles whether the folder is indexed or not). I don't particularly mind, as i can just momentarily use the Windows default explorer, but as a student with lots of scattered note-taking documents on his computer, it would be rather nice to be able to search for a phrase, and get all documents matching said phrase. Oh, and... Otherwise the app is absolutely amazing :)

Mergin with #8815. Please use the template if you open any more issues.


Default HTTP/2 max_header_list_size to 16kb

The RFC doesn't specify a default. h2 has an "emergency" default that's quite high. It makes sense for hyper to pick a safer default to protect users who don't think about it. When checking other libraries, many set a default of either 8kb or 16kb.

FYI when logged into Google, console.cloud.google.com sends headers bigger than this default size, thus that seems too low to me.

Do you have a suggested default instead?
Part of the choice here was noting that most other libraries either picked 8kb or 16kb, and rely on users to opt-in to more. I suppose one possibility we could consider is different defaults for server vs client.

The previous value was just fine in 2024 IMO.


update: update OOSFilterPlugin to support intent_threshold_maps

I have updated the OOSFilterPlugin to support intent_threshold_maps.

Codecov Report

Merging #196 (db59de5) into master (17c49ed) will decrease coverage by 0.07%.
The diff coverage is 88.88%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #196      +/-   ##
==========================================
- Coverage   99.10%   99.03%   -0.07%
==========================================
  Files          60       60
  Lines        2788     2803      +15
==========================================
+ Hits         2763     2776      +13
- Misses         25       27       +2

Flag
Coverage Δ

unittests
99.03% <88.88%> (-0.07%)
:arrow_down:

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

Impacted Files
Coverage Δ

dialogy/plugins/text/oos_filter/__init__.py
94.28% <88.88%> (-5.72%)
:arrow_down:


Style export not working from browser

I've managed to create xlsx files through node without any problems, however doing the same through the browser creates files that don't have any styling.
I used the code from example.html for my tests except I replaced calls to JSDateToExcelDate() with simple values as the JSDateToExcelDate() function wasn't included.

Thank you for reporting that and sorry for the speedbump.  First diagnosis question, are you using https://github.com/protobi/js-xlsx#beta?

node:  npm install xlsx-style
bower:  bower install js-xlsx-style#beta

workbook is just a wrapper for js-xlsx

If you install SheetJS/js-xlsx (npm install xlsx) styles are not used
If you install protobi/js-xlsx (npm install xlsx-style) styles are used

My first hypothesis is the browser is using the main branch SheetJS/js-xlsx because the docs are not yet clear.  I've delayed updating the README page thinking this branch might be merged in.  Will change gears and plan to make this project a separate maintained fork.
Or it might be an issue with use in the browser, and shd be reported here:  https://github.com/protobi/js-xlsx/issues

Thanks for your quick response. I've looked into it further and found out that I was in-fact using the master branch in the browser rather than the beta branch of https://github.com/protobi/js-xlsx
Thanks again.


filter workflows using criteria builder on TRS endpoint

#3838
This only filters workflows for now. I'll work on the tools part while waiting for feedback, but since I'm off tomorrow I just wanted to make sure at least this got in before the release.
Behavior is slightly different from before. The source control filter used to check if the column contained the term, but now it needs to be equal.

This only filters workflows for now. I'll work on the tools part while waiting for feedback, but since I'm off tomorrow I just wanted to make sure at least this got in before the release.

Not sure what state Galaxy tools are in, I know they're interested though

What sort of performance difference did you see for Galaxy?
Before locally:

time curl -X GET "http://localhost:4200/api/ga4gh/trs/v2/tools?descriptorType=GALAXY" -H "accept: application/json"
real	0m3.482s
user	0m0.007s
sys	0m0.008s

After:
time curl -X GET "http://localhost:4200/api/ga4gh/trs/v2/tools?descriptorType=GALAXY" -H "accept: application/json"
real	0m0.625s
user	0m0.007s
sys	0m0.006s


request to add DepthToSpace Spilt and conv2d_transpose operation in post training quantization(full integer quantization))

System information

Linux Ubuntu 16.04:
TensorFlow installed from binary:
TensorFlow version 1.15.0:

Provide the text output from tflite_convert
RuntimeError: Quantization not yet supported for op: SPLIT

@wujiexiaowugui, Could you share the standalone code to reproduce the reported issue. Thanks!

@wujiexiaowugui, Could you share the standalone code to reproduce the reported issue. Thanks!

import tensorflow.compat.v1 as tf
#Eager Mode is essential!
tf.enable_eager_execution()
import sys
import glob
if sys.version_info.major >= 3:
import pathlib
else:
import pathlib2 as pathlib
import random
import cv2
import numpy as np
tf.logging.set_verbosity(tf.logging.DEBUG)
pb_file='model.pb'
input_arrays=['net_input']
output_arrays=['net_output']
input_shapes=[1, 128, 128, 3]
sample_img_dir='/tmp'
converter =  tf.lite.TFLiteConverter.from_frozen_graph(graph_def_file=pb_file,
input_arrays=input_arrays,
output_arrays=output_arrays,
input_shapes={input_arrays[0]:input_shapes})
converter.allow_custom_ops=True
tflite_model = converter.convert()
tflite_models_dir = pathlib.Path("./tmp/tflite_models/")
tflite_models_dir.mkdir(exist_ok=True, parents=True)
tflite_model_file = tflite_models_dir/"model.tflite"
tflite_model_file.write_bytes(tflite_model)
tf.logging.set_verbosity(tf.logging.DEBUG)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
def preprocess_img(img):
# preprocess
img = cv2.resize(img, (128, 128))
# FLOAT32
img = img.astype(np.float32)
# img=(img/128.)-1.
img = img - 128.
return img
def create_datastream_from_imgs(img_dir):
img_path_list=glob.glob(img_dir+'/*.jpg')
random.shuffle(img_path_list)
img_path_list=img_path_list[:200]
imgs_list=[]
for path in img_path_list:
    img=cv2.imread(path)
    img=preprocess_img(img)
    imgs_list.append(img)
imgs=np.stack(imgs_list,axis=0)
return tf.data.Dataset.from_tensor_slices((imgs)).batch(1)

ds=create_datastream_from_imgs(sample_img_dir)
def representative_data_gen():
for input_value in ds.take(20):
yield [input_value]
converter.representative_dataset = representative_data_gen
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.inference_input_type = tf.uint8
converter.inference_output_type = tf.uint8
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
tflite_quant_model = converter.convert()
tflite_model_quant_file = tflite_models_dir/"model_quant.tflite"
tflite_model_quant_file.write_bytes(tflite_quant_model)

@wujiexiaowugui, I tried reproducing the error but looks like some files are missing.
Please find the gist here and provide the more information to reproduce the issue. Thanks

@wujiexiaowugui, I tried reproducing the error but looks like some files are missing.
Please find the gist here and provide the more information to reproduce the issue. Thanks

You can download the model in under link to replace model.pb.
http://mcml.yonsei.ac.kr/files/srzoo/eusr_x2.pb
The error is :
RuntimeError: Quantization not yet supported for op: DepthToSpace

@wujiexiaowugui, Please take a look at gist and confirm the issue. Thanks

@wujiexiaowugui, Please take a look at gist and confirm the issue. Thanks

if you use the "eusr_x2.pb" , the input_arrays=['sr_input']  and  output_arrays=['sr_output']

@wujiexiaowugui,
as per above update, i ran the code shared, please find the gist here

@wujiexiaowugui,
Could you please add the below two lines to your code before converting the model.
converter.experimental_new_converter = True
converter.target_spec.supported_ops =[tf.lite.OpsSet.TFLITE_BUILTINS, tf.lite.OpsSet.SELECT_TF_OPS]

I was able to convert the model without any issues after adding these lines. Please find the gist of it here. Thanks!

@wujiexiaowugui,
Any updates regarding this issue? Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

@wujiexiaowugui,
Any updates regarding this issue? Thanks!

When I added two lines to my code , there is another error occurred:
RuntimeError: Unsupported output type UINT8 for output tensor 90 of type FLOAT32.

@wujiexiaowugui,
Could you please add the below two lines to your code before converting the model.
converter.experimental_new_converter = True
converter.target_spec.supported_ops =[tf.lite.OpsSet.TFLITE_BUILTINS, tf.lite.OpsSet.SELECT_TF_OPS]

I was able to convert the model without any issues after adding these lines. Please find the gist of it here. Thanks!

Any updates regarding this issue? Thanks!

When I added two lines to my code , there is another error occurred:
RuntimeError: Unsupported output type UINT8 for output tensor 90 of type FLOAT32.
You can set "output_arrays=['DepthToSpace']". Then you will get the same error.
I think it is because that "DepthToSpace" layer still can't be quantized to int8.

You can set "output_arrays=['DepthToSpace']". Then you will get the same error.
I think it is because that "DepthToSpace" layer still can't be quantized to int8.

Was able to reproduce the issue with TF v1.5, TF v2.2 and TF-nightly. Please find the attached gist. Thanks!

You can set "output_arrays=['DepthToSpace']". Then you will get the same error.
I think it is because that "DepthToSpace" layer still can't be quantized to int8.

Was able to reproduce the issue with TF v1.5, TF v2.2 and TF-nightly. Please find the attached gist. Thanks!

Yeah, I met the same issues when using TFv1.5, TFv2.2 and TF-nightly.
So, will you support this "DepthToSpace" op in later tensorflow version?

@wujiexiaowugui One of the attribute(block_size) of DepthToSpace has dtype of int64 which is not supported. Can you please change it to int32 and create the .pb? Thanks!

@wujiexiaowugui One of the attribute(block_size) of DepthToSpace has dtype of int64 which is not supported. Can you please change it to int32 and create the .pb? here is a gist for our reference. Thanks!

@jvishnuvardhan  I changed my code in upsampling section:
scale = 2 scale = np.asarray(scale, dtype=np.int32) out = tf.depth_to_space(input, scale)
But, I still met the error as before.

@wujiexiaowugui

One of the attribute(block_size) of DepthToSpace has dtype of int64 which is not supported. Can you please change it to int32 and create the .pb?
did you check block_size? Can you do tf.cast to cast it to int32? Please let us know if you have any doubts?  Can you share the updated *.pb after tf.cast of block_size to tf.int32. Thanks!

@wujiexiaowugui

One of the attribute(block_size) of DepthToSpace has dtype of int64 which is not supported. Can you please change it to int32 and create the .pb?
did you check block_size? Can you do tf.cast to cast it to int32? Please let us know if you have any doubts?  Can you share the updated *.pb after tf.cast of block_size to tf.int32. Thanks!

@jvishnuvardhan  When I set:
scale = tf.cast(scale, tf.int32)
There is an error occurred:
TypeError: Expected int for argument 'block_size' not <tf.Tensor 'generator/WDSR/Cast/x:0' shape=() dtype=int32>.

@wujiexiaowugui

One of the attribute(block_size) of DepthToSpace has dtype of int64 which is not supported. Can you please change it to int32 and create the .pb?
did you check block_size? Can you do tf.cast to cast it to int32? Please let us know if you have any doubts?  Can you share the updated *.pb after tf.cast of block_size to tf.int32. Thanks!

@jvishnuvardhan When I set:
scale = tf.cast(scale, tf.int32)
There is an error occurred:
TypeError: Expected int for argument 'block_size' not <tf.Tensor 'generator/WDSR/Cast/x:0' shape=() dtype=int32>.

@jvishnuvardhan
block_size = 2 block_size = tf.cast(block_size, tf.int32) out = tf.depth_to_space(input, block_size)
An error occurred:
TypeError: Expected int for argument 'block_size' not <tf.Tensor 'generator/WDSR/Cast/x:0' shape=() dtype=int32>.

Hey! Is the DepthToSpace support added now?

Hey! Is the DepthToSpace support added now?

I think DepthToSpace is still not suppored.

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


Testing against lightweight network

Hey @mitschabaude.
Couple of questions about this one.

Are we interested in networks with proving enabled or the lightweight one you mentioned will be enough?
Are we talking about enabling the "Berkeley Live" simple zkApp execution against real network or maybe do we want to exercise something more complex like Voting (instead of default Hello World)?
Do we want to exercise zkApps flow (deployment + interaction) in-browser (and against real networks) as well?
Do we want to run tests for all major branches with corresponding mapping to Mina's branches (and available networks) or we're interested in some particular branch only?

Hey @shimkiv, good questions!

lightweight is enough
Yes, the old "berkeley live" test = hello world example is enough for an initial version IMO. So I think the first PR should just do that. We can then still decide if we want to move to a more complex example, that could be voting or DEX-with-actions
No

Re 4) (branch mapping) -- currently we run CI on the 3 major branches main, berkeley and develop. Mapping those to rampup, berkeley and develop and running the test on those branches would be a great start.
The only problem I'm seeing is that currently, main is not compatible with rampup due to some complications. The current Mina branch for snarkyjs/main is this one: https://github.com/MinaProtocol/mina/pull/13602
I think we should make the effort to bring rampup back up to date with main, and use the branch mapping

Thank you @mitschabaude, got it and will land the corresponding PR.

The only problem I'm seeing is that currently, main is not compatible with rampup due to some complications. The current Mina branch for snarkyjs/main is this one: https://github.com/MinaProtocol/mina/pull/13602

Well, that is why o1labs/mina-local-network:rampup-latest-lightnet is actually based on Mina's rampup-before-accidental-merge for now 😄


Support for 'YouTube auto generated playlist'

Is there a way to download 'YouTube auto generated playlist'?
Example: https://www.youtube.com/watch?v=ASj81daun5Q&list=RDASj81daun5Q

You just need to run youtube-dl 'https://www.youtube.com/watch?v=ASj81daun5Q&list=RDASj81daun5Q'.


How to provide custom shape to the text drawable?

I want to provide a hexagon shape to the text drawable. Is there any way to achieve this?

Still looking for a solution to this, thanks. It would be helpful it we could do slanted borders with dp values like "15/45dp"


Show workspace number as an alternative to circles/lines icons

Request

Show workspace number as an alternative to circles/lines icons
New entries for the style options in the settings:

Circle and Numbers
Numbers

Examples
○ ○ ○ ❹ ○
❍ ❍ ❍ ❹ ❍
① ② ③ ❹ ⑤
⚫ ⚫ ⚫ ④ ⚫
⬤ ⬤ ⬤ ④ ⬤
❶ ❷ ❸ ④ ❺
Sample Code
function getWorkspaceChar(active, index) {
  ISACTIVE = [ '⓿', '❶', '❷', '❸', '❹', '❺', '❻', '❼', '❽', '❾', '❿', '⓫', '⓬', '⓭', '⓮', '⓯', '⓰', '⓱', '⓲', '⓲', '⓴'];
  INACTIVE = [ '⓪', '①', '②', '③', '④', '⑤', '⑥', '⑦', '⑧', '⑨', '⑩', '⑪', '⑫', '⑬', '⑭', '⑮', '⑯', '⑰', '⑱', '⑲', '⑳' ];
  windex = index < 1 || index > 20 ? NUMS[0] : NUMS[index];
  return active ? ISACTIVE[index] : INACTIVE[index];
}

Hi, @juarezr
Thanks for the contribution.
The idea looks obvious but this indicator appeared as an alternative to Indicator from Florian Mullner with absolutely different approach.
For some reason I don't really like the way to show numbers in indicator. In my opinion it looks alien in gnome's icons design. And since 2021 (this year I started to use my indicator) I had time to make sure that numbers are redundant.
My suggestion to you is to try indicator from Florian Mullner.
But  if you still wan't mixture of his indicator and mine just fork this project. It has MIT license, so this code is as yours as mine.
Thanks.


Add OCSP_RESPID_set_by_key_ex() and OCSP_RESPID_match_ex()

OCSP_RESPID_set_by_key() calculates a SHA1 hash of the supplied
certificate. We need to be able to specify which libctx and property
query string is used to fetch that algorithm so we introduce
OCSP_RESPID_set_by_key_ex() which does the same thing but enables you to
speicfy the library context and propery query string explicitly.
OCSP_RESPID_match() matches with certificates based on the SHA1 hash.
Therefore for the same reason we introduce OCSP_RESPID_match_ex().
This will be needed in a later PR where we will make ssapitest run with a non-default libctx.

Needs a rebase of the generated file

I rebased, and I'm assuming that your approval still holds.

Yes. rebasing a generated file should be fine.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Pushed. Thanks.


DatabaseException: Unable to connect to the database

Hi everyone,
I am running the dependencyCheck task in debug mode with Gradle 3.5 and
and I am getting the following error:
08:55:07.227 [DEBUG] [org.owasp.dependencycheck.data.nvdcve.ConnectionFactory]
org.h2.jdbc.JdbcSQLException: Feature not supported: "MV_STORE combined with FILE_LOCK=SERIALIZED" [50100-193]
at org.h2.message.DbException.getJdbcSQLException(DbException.java:345)
at org.h2.message.DbException.get(DbException.java:179)
at org.h2.message.DbException.get(DbException.java:155)
at org.h2.message.DbException.getUnsupportedException(DbException.java:216)
at org.h2.engine.Database.(Database.java:236)
at org.h2.engine.Engine.openSession(Engine.java:64)
at org.h2.engine.Engine.openSession(Engine.java:176)
at org.h2.engine.Engine.createSessionAndValidate(Engine.java:154)
at org.h2.engine.Engine.createSession(Engine.java:137)
at org.h2.engine.Engine.createSession(Engine.java:27)
at org.h2.engine.SessionRemote.connectEmbeddedOrServer(SessionRemote.java:349)
at org.h2.jdbc.JdbcConnection.(JdbcConnection.java:115)
at org.h2.jdbc.JdbcConnection.(JdbcConnection.java:99)
at org.h2.Driver.connect(Driver.java:69)
at java.sql.DriverManager.getConnection(DriverManager.java:664)
at java.sql.DriverManager.getConnection(DriverManager.java:247)
at org.owasp.dependencycheck.data.nvdcve.ConnectionFactory.getConnection(ConnectionFactory.java:228)
at org.owasp.dependencycheck.data.nvdcve.CveDB.open(CveDB.java:124)
at org.owasp.dependencycheck.data.nvdcve.CveDB.(CveDB.java:87)
at org.owasp.dependencycheck.Engine.ensureDataExists(Engine.java:752)
at org.owasp.dependencycheck.Engine.analyzeDependencies(Engine.java:501)
at org.owasp.dependencycheck.Engine$analyzeDependencies$0.call(Unknown Source)
at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:48)
at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:113)
at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:117)
at org.owasp.dependencycheck.gradle.tasks.Check.check(Check.groovy:80)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:73)
It used to work just fine with previous dependency-check plugin versions. Can you advise?
Thank you,
Alin

Hi,
By configuring the plugin 'data' configuration it seems to work:
data {
connectionString = 'jdbc:h2:file:%s;AUTOCOMMIT=ON;''
}
Is this introducing any regression? :-)
Thanks!
Kind regards,
Alin

The only possible issue with that configuration, using 1.4.5 or earlier, is
if you run multiple builds/dependency-check scans at the same time. The
issue arises when one of the builds is executing an update while the other
is analyzing - we've seen some random false negatives occur.
I'm hoping to release 2.0.0 this weekend (might be slightly delayed because
I have to update the gradle plugin to match the changes in the Maven
plugin). With 2.0.0 the issue described above has been fixed.
On Thu, Jun 22, 2017 at 4:13 AM, alinbutura<EMAIL_ADDRESS>wrote:

Hi,
By configuring the plugin 'data' configuration it seems to work:
data {
connectionString = 'jdbc:h2:file:%s;AUTOCOMMIT=ON;''
}
Is this introducing any regression? :-)
Thanks!
Kind regards,
Alin
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jeremylong/dependency-check-gradle/issues/46#issuecomment-310309010,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0qwgF9IoQY36mI1FeJaO4KdKZ_cfEfks5sGiItgaJpZM4N_Kbi
.

I am using version 2.0.1 of the plugin with Gradle 4.0.1 on CentOS 7 and am getting the same error and stack trace as @alinbutura.

You should be able to get around the bug (for now) by specifying:
data {
connectionString = 'jdbc:h2:file:%s;AUTOCOMMIT=ON;''
}

I'll do some more testing to try and get a solution working for both older and newer instances of gradle.

I stumbled across the same error today when having added the current version of h2 as a buildscript dependency like this:
buildscript {
    repositories {
        mavenCentral()
    }
    dependencies {
        classpath('org.owasp:dependency-check-gradle:2.0.1')
        classpath 'com.h2database:h2:1.4.196'
    }
}

Due to this DependencyCheck will actually use the version 1.4.196, which seems not to work anymore with the pre-configured connection string.

@stefanneuhaus correct, I actually need to spend some time and see what version of H2 we can upgrade the core to (must be compiled against Java 7) and see if we can find a default connection string that will work with multiple JVMs connecting to it.

... and for what it is worth, dropping down to only the AUTOCOMMIT flag worked for me in Gradle 4.0.1. Thanks Jeremy for working through this!

The next release of dependency-check will update the H2 database driver version. There are changes to the connections string - so if you have modified your connection string you will likely need to remove the modification for things to work smoothly when you upgrade. I'm hoping to push a release today/tomorrow.

Hi Jeremy,
Just to let you know: tried this morning the 2.0.0 version and it still exhibits the same behavior if I do not change the jdbc connection string. Could you let us know when the new version becomes available? Thanks a lot!

2.1.1 was released, see the announcement here. The fix for this issue was included.
Remember - if you changed your connection string and are using the embedded H2 database you should likely remove the change.

Hello,
when I add

jdbc:h2:file:%s;FILE_LOCK=FS;AUTOCOMMIT=ON;
in the configuration of dependency check in pom.xml , the error  database connection is resolved
Can I unterstand the line added !!

I have been getting this since March 30.
gradle dependencyCheckUpdate --refresh-dependencies
Download https://plugins.gradle.org/m2/org/eclipse/jgit/org.eclipse.jgit/maven-metadata.xml
Download https://plugins.gradle.org/m2/org/eclipse/jgit/org.eclipse.jgit.ui/maven-metadata.xml
Download https://plugins.gradle.org/m2/com/jcraft/jsch/maven-metadata.xml
Download https://plugins.gradle.org/m2/org/slf4j/slf4j-api/maven-metadata.xml
IO Exception: HEAD request returned a non-200 status code
IO Exception: HEAD request returned a non-200 status code
IO Exception: HEAD request returned a non-200 status code
IO Exception: HEAD request returned a non-200 status code

> Task :dependencyCheckUpdate FAILED
Unable to download the NVD CVE data; the results may not include the most recent CPE/CVEs from the NVD.

IO Exception: HEAD request returned a non-200 status code
IO Exception: HEAD request returned a non-200 status code

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':dependencyCheckUpdate'.
> Unable to connect to the dependency-check database

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 25s
1 actionable task: 1 executed

I am facing the same issue. Is there a solution to this ?

Is there any way to detect if the dependency-check database is up and running ?

How do you upgrade the version?


Printing current price

Hi @gokulgovind this project is so amazing. Thumbs up. But I have a question. How can I make it so that the current package/price is printed when the boost button is pressed. And if no package is selected then the print statement would be "no package"

Hai jackpaster,
When user selects a package, that package info will be returned through their completion handler. From where we can maintain a reference of selected package's price and other details.

@gokulgovind can you show me how I could handle the reference of the selected package on completion

@jackpaster i have update the code, now boost button completion handler will return 3 parameters.

button title -> String
isOptionSelected -> Bool
selectedOption -> GLInAppAction

@jackpaster have you checked it?

It's not updated. I checked it's the same code. Please show me where you
changed it. BecAuse I'm very confused
On Tuesday, April 18, 2017, Gokul<EMAIL_ADDRESS>wrote:

@jackpaster https://github.com/jackpaster have you checked it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gokulgovind/GLInAppPurchase/issues/1#issuecomment-294673331,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARCMmRgM_wdGlogvwMrK4ZlTPCKmR9Aeks5rxDu6gaJpZM4M_Fug
.

Sry,  Clone or download the code and check it now. pls.

@jackpaster Have you checked the update..?
Shall we close this issue.


repo_tree

Hi I am trying to use repo_tree however, the entire repo is still being copied. Is this supported?

set sub_directory is the workaround.

:+1:


Depend on regex-syntax 0.7 or 0.8

Depend on regex-syntax 0.7 or 0.8.

If you get a chance, can you rebase and update your branch so we can get CI running? I tried to do it but i can't push to your branch

and fixes #379

this actually got addressed by our recent merge of dependabot:
https://github.com/proptest-rs/proptest/pull/386

If you get a chance, can you rebase and update your branch so we can get CI running? I tried to do it but i can't push to your branch

This PR is different than @dependabot's PR #386 in that this PR permits either regex-syntax version 0.7 or 0.8, whereas @dependabot's PR permits only version 0.8. I've rebased this PR onto origin/master. Would you please reopen this PR and merge it?

https://github.com/proptest-rs/proptest/pull/386

Is there a use case for still requiring 0.7? I expected a dependency on 0.8 to suffice.

A more permissive dependency allows a project to use either regex-syntax version 0.7 or 0.8, so that if that project hasn't yet updated their direct dependency on regex-syntax from version 0.7 to 0.8, they don't need to install two versions of regex-syntax, one for the direct dependency of their project, and one for the indirect dependency through proptest.
Dependabot supports updating version = ">=0.7, <0.9" to `version = ">=0.7, <0.10" in a Dependabot PR.


Moving some things around in Cedar/Thm/Partial

Moving some things around in preparation for the next major lemmas towards PE soundness.

move Partial/Evaluation/WellFormed.lean to Partial/WellFormed.lean because the WellFormed definitions are used throughout the Partial folder, not just in Partial/Evaluation.  (And they will soon be used in even more places outside Partial/Evaluation/)
Move theorems about Partial.evaluateBinaryApp from Partial/Evaluation/BinaryApp.lean to a new dedicated file Partial/EvaluateBinaryApp.lean.  The former file now only contains theorems about Partial.evaluate applied to Expr.binaryApp.  This will allow us to reuse theorems about Partial.evaluateBinaryApp in upcoming theorems about Partial.evaluateValue without a circular import.
Similarly for Partial.evaluateUnaryApp, Partial.evaluateGetAttr, Partial.evaluateHasAttr, and Partial.evaluateCall
This PR also includes small tweaks to the proofs being moved around

I think I'd expect a structure like Evaluation/Evaluate & Evaluate/EvaluateValue, or just a flat structure under Evaluate. You've spent more time working with the formalization than me though, so take my suggestions with a grain of salt...

Is this the proposal?
Partial/
  Authorization/
  Evaluation/
    Evaluate.lean
    Evaluate/ -- files that are today flat under Evaluation
    EvaluateValue.lean
    EvaluateValue/ -- analogues of Evaluate/ files, as needed
    Lemmas/ -- stuff that needs to be shared by `Evaluate` and `EvaluateValue`
      EvaluateBinaryApp.lean
      EvaluateGetAttr.lean
      ...

(Or no Lemmas/ and have the lemmas files be flat in Evaluation

I was thinking the lemmas would just be flat in Evaluation. But otherwise yes, that's what I was imagining.

Reorg'd to that structure.


[Docs] Set a falsy host so we can access the docs from a remote ip

That's useful when using a mobile phone.

Actually, this change is breaking the hot reloading 😞.


[FEATURE REQ] complete native support for remaining azure-resourcemanager libs

Is your feature request related to a problem? Please describe.
relate to https://github.com/Azure/azure-sdk-for-java/issues/36755
For packages under https://github.com/Azure/azure-sdk-for-java/tree/main/sdk/resourcemanager

do nothing, if it already contains src/main/resources/META-INF/native-image/, e.g. https://github.com/Azure/azure-sdk-for-java/tree/main/sdk/resourcemanager/azure-resourcemanager-appservice/src/main
regen it with current codegen, e.g. https://github.com/Azure/azure-sdk-for-java/tree/main/sdk/resourcemanager/azure-resourcemanager-appplatform/src/main/java

if there is already a PR for it (appplatform is this case), just finish that PR, as it would already include the src/main/resources/META-INF/native-image/

1 PR for each package, if you regen it.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[ ] Expected solution specified

A sample graalvm project https://github.com/weidongxu-microsoft/az-java-sdk-graalvm
You only need install graalvm when you test it (regen code does not need that).
https://www.graalvm.org/latest/docs/getting-started/

https://github.com/Azure/azure-sdk-for-java/pull/38558


Welcome bot

The Welcome bot doesn't work on this repo.
It does work on this repo:
https://github.com/data-umbrella/event-transcripts/blob/main/.github/config.yml
I wonder if hosting it on Netlify somehow interferes with its ability to work.

I needed to install the bot on the repo.
Let's see if it will work, moving forward.
https://github.com/behaviorbot/welcome


Java 7 test build to avoid issues such as #144

:+1:

@gianm looks like jdk7 builds are timing out

@xvrl did you see one time out? From what I can see they have not yet started

Although that's weird since they've been in the queue for a long time…

I'm seeing "Hang tight, the log cannot be shown until the build has started."
given it's been like that for a while, I'm just gonna bounce and see what that does.

@gianm I had rebased on master in case the build got triggered before #144 got merged, so there were multiple builds in the queue. Not sure which one is which though, so we'll just have to wait.

@xvrl ah okay, let's just wait then.

@xvrl could you try changing this?
diff --git a/build.sbt b/build.sbt
index 1d66b95..fae2cdd 100644
--- a/build.sbt
+++ b/build.sbt
@@ -162,7 +162,7 @@ val kafkaTestDependencies = Seq(
 lazy val commonSettings = Seq(
   organization := "io.druid",

-  javaOptions := Seq("-Xms512m", "-Xmx512m"),
+  javaOptions := Seq("-Xms512m", "-Xmx512m", "-XX:MaxPermSize=256M"),

   // Target Java 7
   scalacOptions += "-target:jvm-1.7",

@gianm just pushed the change, let's see if it helps

:+1:


fix: rename chrome to chromium

[x] remove google products from our software

Merged in 0e8c7e7de7008bbb5e5ac893ba5d040cb6088bf2.
Thanks for your contribution @albjeremias!


Latitude / Longitude data in returned object?

Hi there,
Love the component; it meets all but one of my requirements, as I will explain.  I've been looking for something to replace React GeoSuggest.  Feature wise React GeoSuggest is perfect, but its just not MUI friendly.
Latitude /  Longitude?
It's just come to my attention that mui-places-autocomplete does not include Lat / Lng data for the selected suggestion.   This is a deal-breaker for me; Lat Lng is really the only data I actually need.
React GeoSuggest provides this data, which is super useful. I wonder if this explains why their component assumes the user's API Key also has access to the Google Maps Geocoding API?
Would you consider adding such a feature?
Love your work.
Thanks

Hi @charliedotau,
Thanks for the positive feedback! :smile: I would be more than happy to add geocoding features into <MUIPlacesAutocomplete> for you. 😄
Honestly when I started <MUIPlacesAutocomplete> all I needed as well was the lat/long of places that people selected/searched. I have since implemented that functionality into my project that uses <MUIPlacesAutocomplete> but haven't had time to add it to <MUIPlacesAutocomplete> so that it is natively supported.
Can I gather any additional requirements from you, if you have any, around lat/long? I will see if they are reasonable to include with the requirements of my current implementation of how I get lat/long from results returned by <MUIPlacesAutocomplete>.
It may take me a week plus to get this implemented in <MUIPlacesAutocomplete> as I have been swamped with work and two little kids lately. 😄 But in the mean time here is my implementation of how to get geocoding working with <MUIPlacesAutocomplete>. Maybe it can be used as a stopgap until the functionality is implemented natively in <MUIPlacesAutocomplete> (or you can just use it if you like and don't want to wait 😄).
export class CreateListingFormLocation extends React.Component {
  // Helper method to parse the results of a 'geocode()' request so that an address for a listing
  // can be extracted. We use the results of the 'geocode()' request vs. the selected suggestion as
  // the selected suggestion can't reliably guarantee that all of the components to compose an
  // address will be present whereas the geocode result ought to (hopefully).
  //
  // Returns a non-null object with the parsed 'geocode()' request results if an address can be
  // composed from it. If an address can't be composed then 'null' is returned.
  static parseGeocodeResults(results) {
    // Reduce the geocode results to a single set of details about the listing
    return results.reduce(
      // The Google Maps JavaScript API returns objects with property identifiers that are snake
      // cased. We will disable ESLint errors for this and in other places in the function to avoid
      // it yelling at us.
      // eslint-disable-next-line camelcase
      (listingDetails, { address_components, geometry }) => {
        // In the event that there could be multiple results that would allow us to compose our
        // listings address we will just use the first result. Although I'm unsure of if and when
        // this could actually happen...
        if (listingDetails) {
          return listingDetails
        }

        const reducedDetails = address_components.reduce((
          { streetNumber, route, city, state, zip, country },
          // eslint-disable-next-line camelcase
          { long_name, types },
        ) => {
          const accumulatedDetails = {
            streetNumber,
            route,
            city,
            state,
            zip,
            country,
          }

          // Just make an new var that is camelCased so we don't have to deal with ESLint errors
          // when assigning 'long_name' to the listing detail property
          // eslint-disable-next-line camelcase
          const longName = long_name

          // The main address is composed of a route for sure but possibly a street # as well
          if (!streetNumber && types.includes('street_number')) {
            accumulatedDetails.streetNumber = longName
          } else if (!route && types.includes('route')) {
            accumulatedDetails.route = longName
          } else if (
            !city &&
            types.includes('locality') &&
            types.includes('political')
          ) {
            accumulatedDetails.city = longName
          } else if (
            !state &&
            types.includes('administrative_area_level_1') &&
            types.includes('political')
          ) {
            accumulatedDetails.state = longName
          } else if (!zip && types.includes('postal_code')) {
            accumulatedDetails.zip = longName
          } else if (
            !country &&
            types.includes('country') &&
            types.includes('political')
          ) {
            accumulatedDetails.country = longName
          }

          return accumulatedDetails
        }, {})

        // Check that we have all the details necessary to construct an address for our listing. We
        // don't need a street number specificially for the listing but if there is one be sure to
        // use it.
        const {
          streetNumber,
          route,
          city,
          state,
          zip,
          country,
        } = reducedDetails

        if (route && city && state && zip && country) {
          return {
            addressMain: streetNumber ? `${streetNumber} ${route}` : route,
            city,
            state,
            zip,
            country,
            cords: {
              lat: geometry.location.lat(),
              long: geometry.location.lng(),
            },
          }
        }

        // If we got to this point we haven't yet found a geocode result that provides all the
        // necessary details for a listing. Return 'null' which is the inital value of our accumulator
        // to ensure that our conditional logic at the beginning of the reducing function works (i.e.
        // parses the next geocode result).
        return null
      },
      null,
    )
  }

  constructor() {
    super()

    this.onSuggestionSelected = this.onSuggestionSelected.bind(this)
  }

  // Disable the ESLint rule camelcase on the next line as the suggestions returned from the Google
  // Maps service have properties that are snake cased
  // eslint-disable-next-line camelcase
  onSuggestionSelected({ place_id }) {
    const { change, geocoder } = this.props

    return new Promise((resolve, reject) => {
      geocoder.geocode({ placeId: place_id }, (results, status) => {
        if (status !== window.google.maps.GeocoderStatus.OK) {
          reject(
            new Error(
              `Unable to query for listing details - response status: ${status}`,
            ),
          )

          return
        }

        const listingDetails = CreateListingFormLocation.parseGeocodeResults(
          results,
        )

        if (listingDetails === null) {
          reject(new Error(`Couldn't compose details of the listing`))
          return
        }

        resolve(listingDetails)
      })
    }).then(({ addressMain, city, state, zip, country, cords }) => {
      // charliedotaus business logic here :)
    })
  }

  render() {
    const { handleSubmit } = this.props

    return (
      <CreateListingFormLocationView
        handleSubmit={handleSubmit}
        onSuggestionSelected={this.onSuggestionSelected}
      />
    )
  }
}

Two things to note about the above code snippet. The Google Geocoding API is passed as a prop to this component. I used the pattern so I didn't render <MUIPlacesAutocomplete>/the lat and long logic before the API was downloaded and available for use. If you want you can use this logic to set it as state directly in the component (logic might have to change for React 16.3):
  componentDidMount() {
    // After the component is mounted it is safe to create a new instance of the Geocoder client.
    // That's because at this point the Google Maps JavaScript API has been loaded. Also if we do it
    // before the component is mounted (i.e. in 'componentWillMount()') we won't be safe to render
    // on the server (SSR) as the 'window' object isn't available.
    const geocoder = new window.google.maps.Geocoder()

    // We set the state in the 'componentDidMount()' lifecycle method with care as this pattern can
    // lead to performance issues. We do it here though so that we can provide logic to not render
    // our form which depends on the Geocoder client until it is actually available to use.
    //
    // We disable the ESLint rule 'react/no-did-mount-set-state' as we are aware of the performance
    // issues that setting the state can lead too but the React docs state that any initialization
    // that requires DOM nodes ought to go here:
    // https://reactjs.org/docs/react-component.html#componentdidmount
    // eslint-disable-next-line react/no-did-mount-set-state
    this.setState({ geocoder })
  }

Also I expose the <MUIPlacesAutocomplete> component in a view component (<CreateListingFormLocationView>) where the callback is just passed along to it.

thanks @Giners.  Wonderful! Thank you.
My requirement is simple: i use the autocompletion component to enable people to search for a location.  On selection my map component jumps to to that location on the map.   To do that I only need the Lat and Lng, that I then pass to my map component.
Happy to wait.
Thanks again.
Charlie

Hey @charliedotau,
I found some unexpected time and got a head start on adding the geocoding functionality. If you want you can checkout the diff on commit 2e7e134886ea0c0edab050c92e1f75df5bd02710 made to the feature/geocode-suggestion branch or you can wait for the PR/merge to master.
I plan on providing geocoding utility functions to enable consumers of <MUIPlacesAutocomplete> to geocode a selected suggestion at their discretion.

Hey @charliedotau,
PR #36 brought in the geocoding utility functions. Here is demo code showing how to use them: https://github.com/Giners/mui-places-autocomplete/blob/master/demo/DemoGeocodeLatLong.jsx
You also need to enable the Google Maps Geocoding API to make use of them. This is all documented in the README.md. Let me know what you think/if you have any questions.

wonderful, thanks @Giners.   I hope to give this a try in the next day or so.
Thanks so much, for a great component.

just confirming @Giners this is working really nicely.  Thanks again.

@charliedotau Glad to hear! :)
Ill look into #35 in the coming week if I have time.


Draw some diagrams to help explain possible ways to map orgs

I have an SVG with layers. I've demonstrated this to Daryl, and I think we've agreed that using the layers as an impromptu animation as we talk is probably better than a document.
We were thinking of presenting this as a talk in January some time.

[

](url)


Replace the 'Open in Histomics UI' menu item with a button

Makes it more convenient to view a selected image in the Histomics UI by moving the option from the 'Action' menu to it's own separate button.
Addresses #146

@eagw Looks good.  If the codecov test prevents you from merging, let me know and I'll merge it.


fix(38073): Extract to function refactoring generates invalid code for arrow function that references this

Fixes #38073

@minestarks I can't remember if you wanted all refactor PRs delegated immediately to @jessetrinity so I assigned it to you for now. (But @jessetrinity I requested your review already.)

Thanks for checking, yes @jessetrinity is the designated refactor PR magnet, at least until he resigns or riots

Looks good. Thanks!


Draft: Fixed parsing of utf8 identifier with associated value

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

  Generated by :no_entry_sign: Danger


Update Windows Jenkins Jobs to Remove Docker Testbed

For #5641

Remove all windows docker jobs.
Rename the trigger phases of all Windows containerd jobs to normal Windows trigger phases.
Remove windows proxyall job as we have set this feature as the default value in agent.

/test-windows-all

Let's discuss https://github.com/antrea-io/antrea/issues/5630 at the next community meeting

@XinShuYang , this PR would be merged after #5603 ( as still 5603 uses containerd in trigger phase ), we are currently planning to prioritise #5603 @jainpulkit22 , can mark it as dependent.

LGTM One question, any plan to clean up the docker related in the CI shell scripts?

Yes, I will refactor the script in the next release.

LGTM Is there really a dependency on #5603 ?

@antoninbas This PR is only for removing existing docker jobs. We can merge it first, then I will resolve conflicts in #5603.


Upgraded project to latest flutter, flame, bonsoir, & all deps incl code changes required.

Major versions:

Flutter to 3.7.4
Flame too 1.6.0
Bonsoir to 2.0.0

Had to change a bunch of code as library has changed.

closes #9


Problem in sendSuccessAnswer when inserting in a table where primary key is auto increment key.

Suppose we have some table where Primary Key is in Auto Increment form. Then the current version of code will throw error while inserting.
I am proposing a fix at the time of calling the function sendSuccessAnswer in "insert" method.
var field = findPrim(columns, req.query[settings.paramPrefix + 'field']);
                           console.log(field);
                           console.log(insertJson[field]);
                           if(insertJson[field]==='undefined'){}
                               else { sendSuccessAnswer(req.params.table , res,insertJson[field], field);}

If primary key is provided in input then it will show the inserted row as result. And if primary key is not present it will just show the success message.

Hello @sidharthaa ,
thanks for opening this issue. I am having troubles to understand your problem. Do you mean it throws an error if we supply an primary key when there is an auto increment primary key?


Network down flag with splash screen

Scenario: network is down/not working properly but looks like it is up.
This to provide a manual way to force the app to show a message instead of the main app.
This is a manual intervention.
Use flag to render network down splash screen.
Effects all pages on token frontend

Shows loading spinner turns into an explanation.

To test

In an environment, set the flag to show this message (will require netlify access). flag string:REACT_APP_NETWORK_DOWN. if tue this behaviour will be active (the app will show splash not normal app)

When the flag REACT_APP_NETWORK_DOWN is set to 1 on netlify and site is republished, accessing any of the below links on TFE on Staging throws the network down message correctly.
https://staging.token.vega.xyz
https://staging.token.vega.xyz/rewards
https://staging.token.vega.xyz/governance


Updated NETMF Gadgeteer Device to include Cellular connections to Azure

I modified the NETMF Gadgeteer device ( connectthedots/Devices/DirectlyConnectedDevices/NETMF ) to include options for either Ethernet, WiFi or Cellular network connections to Azure and updated the documentation as well.  You can switch which network connection type you are using by adding the desired network module into the designer, connecting it and then using the #defines in the Program.cs (ETHERNET, WIFI, CELLULAR) class to enable the code needed for that type of network.
Documentation for NETMF Gadgeteer devices has also been updated.

Hi @ZingPow, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@ZingPow, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;


Properly set decoder when using DefaultAppConfig

This pull request resolves issue #274.
When using the DefaultAppConfig with a custom decoder or custom string interpolator, only some of the configurations would receive the decoder or interpolator.

archaius-pull-requests #201 SUCCESS
This pull request looks good

NetflixOSS » archaius » archaius-pull-requests #32 SUCCESS
This pull request looks good

archaius-pull-requests #202 SUCCESS
This pull request looks good

NetflixOSS » archaius » archaius-pull-requests #33 SUCCESS
This pull request looks good

archaius-pull-requests #203 SUCCESS
This pull request looks good

NetflixOSS » archaius » archaius-pull-requests #34 SUCCESS
This pull request looks good


publish an ipc message when a mob is seen

not sure if this will be the final implementation of this, but i wanted to get a PR made in order to start discussing the idea.
it would be nice to essentially be alerted whenever a mark is seen. scout helper would like to use that to automatically update tracker links as scouters do their scouting. so this change introduces a publish channel in the hunt helper ipc system that sends a message every time a mob is seen. scout helper can then subscribe to that in order to receive updates.
i will want to test this thoroughly with scout helper before merging this PR, but wanted to get your feedback on the idea while i implement it in scout helper.

Sounds good.
Seems to only be called when the train is recording + updated so maybe mobseen/markfound could be renamed?

oh whoops, yeah i forgot add is only called when recording. i'll try to see if i can make it publish regardless of if it's recording or not. it seemed to me that it would be better to just always publish that info, but maybe i can include in the published data a flag about if the train recorded that mob.

okay, i changed it a bit so the language is consistent and it always publishes a message when the mark is first detected, regardless of train recording. i intentionally did it so that if you walk away and come back to the mark's location, it will publish another message. i figured that is useful information for any listeners.
i also tested the integration with scout helper and it works fantastically! :D
let me know your thoughts :3

i got the scout helper update into D17. just let me know if you wanted any other changes to this pr ^w^.

LGTM, i'll merge if it's done?

yep! done and works in my tests 👍


[Joy UI] Add FormControl component

Add FormControl similar to Material UI. I improve the DX a little bit when composing the FormLabel with other components.
It can be composed with:

Input
Textarea
Select
Checkbox
RadioGroup

It links the FormLabel and FormHelperText with the associated form element.

[x] I have followed (at least) the PR section of the contributing guide.

Looking good! I wonder though if we should have a specific FormControl component case? I see how it could be redundant given we have the text field already.

I have an RFC about removing the TextField and leaning on FormControl instead. Let's discuss it in the RFC but this PR is not related to the discussion because FormControl is independent.


Created .delete() from delitem in node

Re-open https://github.com/bluesky/tiled/pull/245

I am 👎🏻 on this change.
The Node class is derived from Mapping so we should stick with the dict-like API of (e.g. n[k] (__getitem__), n[k] = v (__setitem__), and del d[k] (__delitem__)) without providing extra aliases for them.
If there were additionally a delete method I would wonder if it was different that __delitem__ and if so why?


Scale factor update

Updated scale factor in gen_lef.json

@arvuce, please check out the proposal in #677 comments.

Does that mean all units are nm now?

Does that mean all units are nm now?

No. The conversion of a value x in layers.json to nm is as follows: x/ScaleFactor
This is a floating point division as the numbers could require sub-nm precision.
Any number in any json should not involve any scaling.
When consuming LEF, apply any conversions needed following the UNITS directive. Does this make sense?

@arvuce22 , @854768750 , the test cases I am running are getting killed during GcellGlobalRouter without any error messages. I think there is a memory issue, using top, I see memory consumption going up to 30GB. It does not look right. Example:
schematic2layout.py $ALIGN_HOME/align/pdk/finfet/examples/common_source -p $ALIGN_HOME/align/pdk/finfet -c

Yes. Because lef unit is changed to nm now.

Another question, it there any possible circumstance that the number in nm is not an integer? For example, in layers.json, width=321 and scalefactor=10

Another question, it there any possible circumstance that the number in nm is not an integer? For example, in layers.json, width=321 and scalefactor=10.
Then after conversion, width becomes 32.1nm

Yes.

@Yaldiz, Soner @.***> @854768750
https://github.com/854768750 The memory issue should not be due to
lef file. It may be due to .gds.json or gds file.
On Tue, May 4, 2021 at 11:59 AM soneryaldiz @.***>
wrote:

Another question, it there any possible circumstance that the number in nm
is not an integer? For example, in layers.json, width=321 and
scalefactor=10.
Then after conversion, width becomes 32.1nm
Yes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ALIGN-analoglayout/ALIGN-public/pull/685#issuecomment-832095594,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTUJYJPTN2HEYEMJJ5XRNTTMARY5ANCNFSM4356MWVA
.

@Yaldiz, Soner @.***> Is the generated LEF and json files
using #685 correct?
On Tue, May 4, 2021 at 12:09 PM Arvind Sharma @.***> wrote:

@Yaldiz, Soner @.***> @854768750
https://github.com/854768750 The memory issue should not be due to
lef file. It may be due to .gds.json or gds file.
On Tue, May 4, 2021 at 11:59 AM soneryaldiz @.***>
wrote:

Another question, it there any possible circumstance that the number in
nm is not an integer? For example, in layers.json, width=321 and
scalefactor=10.
Then after conversion, width becomes 32.1nm
Yes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ALIGN-analoglayout/ALIGN-public/pull/685#issuecomment-832095594,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTUJYJPTN2HEYEMJJ5XRNTTMARY5ANCNFSM4356MWVA
.

Another question, it there any possible circumstance that the number in nm is not an integer? For example, in layers.json, width=321 and scalefactor=10.
Then after conversion, width becomes 32.1nm

Yes.

@Lastdayends It may need a lot of changes.

@yaldiz, Soner @.> Is the generated LEF and json files using #685 correct?
…
On Tue, May 4, 2021 at 12:09 PM Arvind Sharma @.> wrote: @yaldiz, Soner @.> @854768750 https://github.com/854768750 The memory issue should not be due to lef file. It may be due to .gds.json or gds file. On Tue, May 4, 2021 at 11:59 AM soneryaldiz @.> wrote: > Another question, it there any possible circumstance that the number in > nm is not an integer? For example, in layers.json, width=321 and > scalefactor=10. > > Then after conversion, width becomes 32.1nm > > Yes. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#685 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AKTUJYJPTN2HEYEMJJ5XRNTTMARY5ANCNFSM4356MWVA > . >

@arvuce22 , with a minor fix, the units in JSON and LEF look good now. Thank you.

As the checker reports some errors and gds cannot be generated automatically, I manually run json2gds to see the layout. The layout is correct. Waiting for some changes to the checker.

gds.json can be generated. Then the checker reports errors because the unit is changed. So gds cannot be generated automatically.
But it can be be generated using json2gds.py circuit.gds.json circuit.gds manually.

@soneryaldiz @854768750 @Lastdayends Based on our conversation today, I think the main change that needs to be made is to the parameters to the calls to rational_scaling in align/pnr/checkers.py
There are three conversions;

From primitive layout .json files (has been 0.1nm) to PnRDB units (0.5 nm), this means divide by 5.
From the PnRDB units (0.5nm) to the layers.json units (was 1nm, is now 0.1nm). This was divide by 2. Now it should be multiply by 5.
Convert from layers.json units (was 1nm, now 0.1 nm) to layout .json file (has been 0.1nm). This was multiply by 10, now it should be do nothing.

@stevenmburns , that is correct. I have already done this on my local clone. I will commit the changes shortly. I also started working on moving the off grid check to align/cell_fabric/drc.py. This is taking longer than I anticipated as tests have dependencies. I will make another commit later Today.

@soneryaldiz @854768750 @Lastdayends Based on our conversation today, I think the main change that needs to be made is to the parameters to the calls to rational_scaling in align/pnr/checkers.py
There are three conversions;

From primitive layout .json files (has been 0.1nm) to PnRDB units (0.5 nm), this means divide by 5.
From the PnRDB units (0.5nm) to the layers.json units (was 1nm, is now 0.1nm). This was divide by 2. Now it should be multiply by 5.
Convert from layers.json units (was 1nm, now 0.1 nm) to layout .json file (has been 0.1nm). This was multiply by 10, now it should be do nothing.

@soneryaldiz In another PR we could probably streamline this to just one call to rational scaling. Instead of moving from layout .json coordinates to PnRDB coordinates and then back to layout .json coordinatate later, we could just transform the rectangles imported from PnRDB to layout .json coordinates as they are being read in. But that is an optimization and refactoring for another day.

@stevenmburns , good point. I removed off-grid check from align/cell_fabric/drc.py as capacitors use a custom grid (defining metal layers in a hacky way). This could be improved in a future PR.
However, there is still a scaling issue capacitors which leads to large number of errors with adder. I have not figured out why and help is welcome.

@soneryaldiz Seem like in adder there are rectangles with two different units being commingled together.
align.cell_fabric.remove_duplicates WARNING : DIFFERENT WIDTH('Rectangles on layer V1 with the same 2x centerline 72000 but different '
 'widths {320, 32}:',

@stevenmburns , yes, the objective is to keep all JSON files consistent and use the units that is specified by the ScaleFactor in layers.json. The expectation is to eliminate scaling during conversions as much as possible. Yet this goal has not been achieved yet as PnR is still using 2nm internally.

@stevenmburns , yes, the objective is to keep all JSON files consistent and use the units that is specified by the ScaleFactor in layers.json. The expectation is to eliminate scaling during conversions as much as possible. Yet this goal has not been achieved yet as PnR is still using 2nm internally.

But even the primitives layout .json files don't have the right units.

@stevenmburns , yes, the objective is to keep all JSON files consistent and use the units that is specified by the ScaleFactor in layers.json. The expectation is to eliminate scaling during conversions as much as possible. Yet this goal has not been achieved yet as PnR is still using 2nm internally.

(That's 0.5nm internally) But even the primitives layout .json files don't have the right units.The PnR C++ code doesn't look at the layout .json files, so we should be able to make them be in angstroms. And that was what I was assuming when I make the scaling assumption above.

Can we chat about this when you have few minutes?

Sure, But I think it is because you haven't set the scale_factor to 10 in the layers.json file. Ok.

@854768750  and @Lastdayends , We believe that there is another hidden scaling issue in the capacitor place and route. Could you please look into it? Example:  schematic2layout.py $ALIGN_HOME/examples/adder. Please see the snapshot below:

@stevenmburns @soneryaldiz
I can look into it.


CSV Support

Hi,
can i get sample how to display csv like data from string please ? Im transfering csv from api, it is a string with comma delimeter..
Regards

BTW it would be nice to add this to nuget :)


Recycler optimise

Recyclers only check for intersecting entities that have been triggered via collision instead of actively querying it every tick. This means they pretty much don't show up on the profiler when idle.
Requires space-wizards/RobustToolbox#1254

I originally did it like this because otherwise items that passed by the recycler without being thrown at it wouldn't be detected, once the engine PR is merged though this will be a much better way to do it.


Minor fix to MKL allocator to address a build regression

Uses char* instead of char[] to deal with build issues related to allocator strings.

Can one of the admins verify this patch?

@rmlarsen had to go back to using char* instead of char[] since we are running into some linker issues around undefined references.

@tensorflow-jenkins test this please

@jbobba thanks for the quick fix.

confirm the mklallocator problem during the compile is solved on #13731
but meet another problem:
ERROR: /root/tensorflow/tensorflow/core/kernels/BUILD:819:1: C++ compilation of rule '//tensorflow/core/kernels:transpose_op' failed (Exit 1): gcc failed: error executing command
tensorflow/core/kernels/mkl_transpose_op.cc:53:10: error: template-id 'MKLTranspose2D' for 'tensorflow::Status tensorflow::MKLTranspose2D(char, const tensorflow::Tensor&, tensorflow::Tensor*)' does not match any template declaration
Status MKLTranspose2D(const char trans, const Tensor& in, Tensor* out) {
^
tensorflow/core/kernels/mkl_transpose_op.cc:60:3: note: in expansion of macro 'INSTANTIATE'
INSTANTIATE(float, s)
^
tensorflow/core/kernels/mkl_transpose_op.cc:122:1: error: expected '}' at end of input
}  // namespace tensorflow
^
OS: redhat 7.4
bazel: 0.6.1
build command:
bazel build --config=mkl --copt="-g" --copt="-DEIGEN_USE_VML" --copt="-mavx2" --copt="-mfma" --copt="-O3" --verbose_failures --copt="-L/opt/intel/gcc/lib64" -s -c opt //tensorflow/tools/pip_package:build_pip_package
Any ideas?


hget & hexists

Why hget & hexists are command in mode "Write"?

oh, It's a bug, welcome submit PR to fix it


Kronecker matrix utilities

From @bob-carpenter on January 3, 2015 2:22
Need utilities for Kroneckers for Gaussian processes.  I don't know much about it but want to get it down as an issue and point to Ben's R example:
https://groups.google.com/forum/#!msg/stan-users/iddShgJVwTQ/QgcDEP3h9JQJ
Talk to Ben to figure out how this should look in Stan.
Copied from original issue: stan-dev/stan#1196

From @bgoodri on January 3, 2015 2:47
I think the most immediate need is to have a user-facing function called something like quad_form_kronecker that takes 3 arguments (matrix X, matrix Y, and vector or matrix Z) and outputs the equivalent of Z' * kronecker_product(X, Y) * Z. Algebraiclly, that is a lot of subvector * scalar * matrix * subvector but with a smaller number of unique matrix * subvector expressions that could be performed once each internally.
It is often the case that either X or Y is diagonal or otherwise sparse, in which case there could be some more optimizations. I'm not sure we need any more than that because it is rare in statistics to see a Kronecker product that is not embedded in a quadratic form.

From @flaxter on February 24, 2015 0:28
Here's a summary of the main points from the Stan dev meeting on this topic (17 February 2015):
Eigen now has (experimental) support for Kronecker products: http://eigen.tuxfamily.org/dox-devel/unsupported/group__KroneckerProduct__Module.html and Eigen also has a fair bit of supported sparse types (some experimental) e.g.: http://eigen.tuxfamily.org/bz/show_bug.cgi?id=275 http://eigen.tuxfamily.org/dox/group__SparseQuickRefPage.html
http://eigen.tuxfamily.org/dox/group__Sparse__chapter.html
http://eigen.tuxfamily.org/dox-devel/group__SparseCholesky__Module.html
We discussed ways of implementing, a type kronecker_matrix (which would also be useful for new types tridiagonal_matrix, sparse_matrix and toeplitz_matrix and symmetric_matrix) which would be similar to cholesky_factor_cov.
Important design decision
If K = K1 ⊗ K2 ⊗ K3  and K1, K2, and K3 are different sizes (let's say n x n, m x m, and o x o) we need a new data structure to be able to handle this. There was a discussion of using view or map in C++/Eigen, with a vector of matrices all sized N x N where N = max(n,m,o). @bob-carpenter needs to weigh in on this, and others should make their cases.
Once we get that worked out, the main wish list for Kronecker matrix functions is simply everything in section 34.7 of the manual, Linear Algebra Functions and Solvers, e.g.:
row_vector operator/(row_vector b, kronecker_matrix A)
row_vector mdivide_right_kronecker(row_vector B, kronecker_matrix A)
vector eigenvalues_sym(kronecker_matrix A)
We also need matrix-vector multiplication (not sure the naming convention here):
row_vector matrix_multiply_vector(kronecker_matrix A, vector B)
row_vector vector_multiply_matrix(row_vector B, kronecker_matrix A)
I guess we need a function (although like inverse it shouldn't be the first choice for a user):
matrix kronecker_product(kronecker_matrix A)
Also very useful will be:
y ~ multi_normal_kronecker(vector mu, kronecker_matrix A)
Relatedly, the matrix normal distribution is already implemented, but not exposed. It should be exposed. (@mbrubake may have thoughts.)

From @bgoodri on February 24, 2015 5:9
As a start, we might think about exposing a Triplet
http://eigen.tuxfamily.org/dox/classEigen_1_1Triplet.html
That would make it possible for a C++ function to take a Triplet and do
sparse stuff internally. In addition, that would be useful for
(non-)missing data.

@syclik @flaxter is there a branch open for this yet?

I don't have one.

I'm going open a new issue to add
vector kronecker_matrix_vector(matrix K1, matrix K2, vector V) to the library that references this issue rather than using this issue. Better to keep issues manageable.


Drop .position-sticky utility class

.position-sticky doesn't work out of the box without top value. I think we better drop it in v5. .sticky-top fully covers the behaviour people will need.

Nevermind, we have .sticky-top and dropping this won't be that much of a difference.


Multistage yaml pipeline

It would be helpful if the Azure Pipeline would be a multistage (yaml) pipeline instead of the classic.
Regards
/Dan

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

ID: 1e22e4d6-366f-fbf8-e141-8905ca7ab7ae
Version Independent ID: 5be2acf2-66e7-cadf-6666-5a85a65b9771
Content: Continuous integration and delivery in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/continuous-integration-deployment.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

@dansydner - Thanks for posting your valuable feedback. Can you please elaborate on the purpose of having multistage pipeline example in this document context as we have Azure DevOps document covering that feature.

@HarithaMaddi-MSFT  It is our understanding that the classic pipelines are being phased out. Because of that we are fully focused on yaml pipelines. I am trying to wrap my head around CI/CD for Data Factory and how to fit it in with our existing pipelines. Having to convert the pipelines from classic to yaml takes away my attention from learning what's important and specific for Data Factory deployment.

@djpmsft - Can you please take a look and share your thoughts.

Hey @dansydner ,
I am a bit confused on your question, so I am looking to clarify a bit. ADF CI/CD deployment uses Azure DevOps release pipelines and runs the generic PowerShell and ARM template deployment tasks for each stage. Each task is YAML script under the hood and can be written as such. For example, here is what the PowerShell script looks like in YAML.

Does this help? Our documentation uses the Azure DevOps UX as thats what the majority of our customers use, but writing your own script is a very valid approach
Thanks,
Daniel

Does this help? Our documentation uses the Azure DevOps UX as thats what the majority of our customers use, but writing your own script is a very valid approach

Thanks for taking the time @djpmsft your answer was indeed very helpful.
I've haven't used the classic UX very much so I wasn't aware how close the functionality mirrors that of YAML.
And after realizing that most of your customers use the classic UX I will stop bothering you with how useful it would be to ME :)


Standard to create documentation and comment every script

I was wonder if we could elaborate a good example of how to comment a script and elaborate documentation for it. @shanshanhsieh maybe you can give it a try to create a template. @daren-thomas could we also look for automated ways to do it?. I head from Sergio that java has something called java docs. it basically reads the code and extracts all the comments into an organized and structured way.
@shanshanhsieh for the time being we might start with a description of what the script does, what it is based on/a reference at the beginning, and include comments in the code.

got it, I will make documentation in my current contribution (i056), and derived a template from it.

Here is a link to the documentation guide for python: https://docs.python.org/devguide/documenting.html
This is for the core python and standard libraries. Following these rules will make it easy to use Sphynx (like javadoc) to generate documentation automatically.
Also check the PyCharm Documenting Source Code page: https://www.jetbrains.com/help/pycharm/2016.1/documenting-source-code-in-pycharm.html
Since we are using PyCharm, it makes sense to use some of it's cool stuff. Adding type information to the doc string of methods for instance let's PyCharm assist in code-completion/IntelliSense and makes life easier for everyone.
A very, very important step to documenting code that is often overlooked is naming. You need to name your variables, functions, classes etc. properly. This will make the code much clearer than documenting messy code. This is also very important as the code tends to change faster than the documentation and keeping those two in sync has a long history of not happening...

hey @daren-thomas, maybe you could give me a hand on this issue.
as I understand, currently, we are sort of using ReStructured Text format in our codes, which has the possibility to generate documentation in HTML/PDF via Sphinx tool integrated in Pycharm.
This leads me to two questions:

Do we want replace gitbook with documentation generated from this workflow (via Sphinx tool)? Seems like there is no direct way to read markdown docstring in Pycharm like Sphinx, tell me if I am wrong.
I couldn't quite get the "Running Sphinx Task" step to generate documentation. If you know how it works, could you please take storagetank_mixed.py as an example and show us how the documentation will look like?
https://www.jetbrains.com/help/pycharm/2016.1/run-debug-configuration-sphinx-task.html

Thanks!

hey @shanshanhsieh, to answer your questions:

No. This should not replace the documentation, but rather augment it: The generated documentation is more of an API documentation. We can maybe add it as a chapter to the gitbook documentation. On gitbook, we are also documenting team stuff and other things that don't fit into file headers / function docstrings.

I will look into the "Running Sphinx Task" as soon as I can.

any update?

@daren-thomas just checking out the issues we need to finish before M1. i don't think we are going to use the documentation for the SBE workshop, shall we move this to M2?

I think we need to document the code as much as possible for M1. No need to generate the sphynx api docs, but the code should be in the best shape we can get it by then as it goes public on M1.

As discussed, here is a general in-function documentation format we could adapt.
@JIMENOFONSECA @daren-thomas @gabriel-happle @martin-mosteiro @amraladdin Please feel free to comment on what else we could include, I will make a standard format and write it in the Gitbook.
"""
    Description
    ---------------
This algorithm calculates the heat flows within a fully mixed water storage tank.
Heat flows include sensible heat loss to the environment (ql), heat charged into the tank (qc), and heat discharged from the tank (qd).

    Assumptions
    -----------------

    Parameters
    ---------------
    :param Tww_st_0: Initial tank temperature in C
    :param Utank: dhw tank insulation heat transfer coefficient in W/m2-K, value taken from [1]
    :param AR: # tank height aspect ratio, H=(4*V*AR^2/pi)^(1/3), value taken from commercial tank geometry[2]. global variable.

    Returns
    ----------
    :return ql: storage sensible heat loss in W.
    :return qd: heat discharged from the tank in W.
    :return qc: heat charged into the tank in W.

    References
    ---------------
    [1] SIA 385
    [2] Jenni.ch
"""

This is the link to the source code of sandia pv library, more examples could be found there:
http://pvlib-python.readthedocs.io/en/latest/_modules/pvlib/pvsystem.html#LocalizedPVSystem

@shanshanhsieh was this added to the gitbook? could be close this issue?

The documentation comment template is added to gitbook in the "Style" section. This issue can be closed now.


Grammar ambiguities

Type of issue
Other (describe below)
Description
In the Grammar ambiguities section mentioned below,

If a sequence of tokens can be parsed (in context) as a simple_name, member_access, or pointer_member_access ending with a type_argument_list, the token immediately following the closing > token is examined, to see if it is:
...
One of the relational operators < > <= >= is as;
...
If the following token is among this list, or an identifier in such a context, then the type_argument_list is retained as part of the simple_name, member_access or pointer_member-access and any other possible parse of the sequence of tokens is discarded. Otherwise, the type_argument_list is not considered to be part of the simple_name, member_access or pointer_member_access, even if there is no other possible parse of the sequence of tokens.

The statements
F(G<A, B>7);
F(G<A, B>>7);

will each be interpreted as a call to F with two arguments.

it is mentioned that if the token following the > token in a simple name / member access / pointer member access is one of the relational operators (<  >  <=  >=  is as), the simple name / member access / pointer member access will be interpreted with type argument list.
But in the mentioned example, F(G<A, B>>7), the token > is followed by the token >. Need help to understand why it is not being interpreted with type argument list.
Page URL
https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/language-specification/lexical-structure
Content source URL
https://github.com/dotnet/csharpstandard/blob/draft-v8/standard/lexical-structure.md

@logeshkumars0604 – Thanks for spotting this. The token > should not be in that list and needs to be removed. If you wish to submit a PR that removes it please do so, you can put me down as a reviewer.
Note there is a more complex disambiguation rule possible where > might be used, it is likely this error stems from that – but as the rest of that rule is not mentioned simply removing the errant > is a valid fix until, and if, that more complex rule is added to the Standard.


Add collector example

This PR adds OpenTelemetry Collector based example for metrics export.
Also fixes misconfigured metrics-dashboard json

Only files in the example directory are changed. Coverage calculation should not include these files.


Packetizer should optionally add abs-send-time extension to packet-sets

Summary
libwebrtc on android really wants abs-send-time to be enabled
https://webrtc.org/experiments/rtp-hdrext/abs-send-time/
Motivation
I believe that libwebrtc.aar isn't interpreting stream timestamps without this extension, and choppy video is the result. Even if i'm wrong about this, getting closer to feature-parity with webrtc.org isn't a bad thing.

We have abs support https://github.com/pion/rtp/commit/6b0d25a61c0874b5703cb26f38c6c17d8ca26796! Not closing because it still needs to be added to the packetizer itself.

This should be an interceptor, and as close to the end as possible


Make releases for WoWUp

Hello,
Could you make releases so that WoWUp and others addons managers can update this addon ?
Thanks !

I only intend to publish on CurseForge, thanks.


admin Panel for deployment

Currently, the admin panel "/admin" somehow is not available when deployed through containers.

This is due to container volume overwrites the content copied into container image.


FEATURE : Scroll to Top Button component

Description:
The WordWise website currently lacks a "Scroll to Top" button, which is a key feature for enhancing user experience, especially on pages with extensive content. Without this button, users have to manually scroll back to the top of the page, which can be inconvenient and time-consuming.
Implementing a "Scroll to Top" button would:

Improve the website's accessibility and usability.
Provide a smoother navigation experience, particularly on long articles or blog posts.
Align the user interface with modern design practices.

I recommend adding a floating "Scroll to Top" button that becomes visible as users scroll down the page.
Please @ANSHIKA-26  assign me this task with labels "GSSOC-Ext", "hacktoberfest-accepted" and "level2".

#213 already exists


fix issue with crio rpm

probably should open a BZ but this is easier. We need to fix this:
[vagrant@f27vanilla ~]$ sudo dnf install crio
Fedora 27 - x86_64 - Test Updates                                                                                23 MB/s |  20 MB     00:00
Fedora 27 - x86_64 - Updates                                                                                2.1 kB/s | 373  B     00:00
Fedora 27 - x86_64                                                                                12 MB/s |  58 MB     00:04
Last metadata expiration check: 0:00:00 ago on Wed 18 Oct 2017 06:41:22 PM UTC.
Error:
 Problem: conflicting requests
  - nothing provides runc >= 2:1.0.0-10 needed by cri-o-1.0.0-5.gita636972.fc27.x86_64
  - nothing provides runc >= 2:1.0.0-10 needed by cri-o-1.0.0.rc1-2.gitbb1da97.fc27.x86_64

Is it in updates-testing?

dnf info runc
Last metadata expiration check: 2:05:01 ago on Wed 18 Oct 2017 01:47:52 PM EDT.
Installed Packages
Name         : runc
Epoch        : 2
Version      : 1.0.0
Release      : 10.rc4.gitaea4f21.fc26
Arch         : x86_64
Size         : 7.1 M
Source       : runc-1.0.0-10.rc4.gitaea4f21.fc26.src.rpm
Repo         : @System
From repo    : updates
Summary      : CLI for running Open Containers
URL          : https://github.com/opencontainers/runc
License      : ASL 2.0
Description  : The runc command can be used to start containers which are packaged
: in accordance with the Open Container Initiative's specifications,
: and to manage containers running under runc.

this is an issue in f27, not f26.

I just pushed runc-1.0.0-11.rc4.gitaea4f21.fc2* into updates testing.  not sure why 10 never made it there.  Please check and update karma.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-994680b67b


chore: prep the repo for automated releasing

Awesome!
Just need to update this header: https://github.com/sendgrid/sendgrid-php/blob/master/CHANGELOG.md#2019-12-11-version-740

Awesome!
Just need to update this header: https://github.com/sendgrid/sendgrid-php/blob/master/CHANGELOG.md#2019-12-11-version-740

@thinkingserious You already did.

Hello @childish-sambino,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Break down with multiple labels

@tormorOoO hello IQLabel is really cool. but it has a bug. For example ,if you create two labels,when you create the second one,it become the first responder,then you delete the second one and directly click on the first one(not click on the view and then click on the first one),it will break down at the line "[_textView resignFirstResponder]; it will helpful if you fix it."
Apparently you are using an older version. I do not like to say it, but I could not reproduce this bug. Try the latest version and let me know if this happens again.

i am sure that i downloaded the latest version and use the latest ios SDK 8.1.
however  it happened again.

@tormorOoO it's in - (void)hideEditingHandles method?  I can't reproduce this :unamused:  Can you send stack trace?

i am so sorry wasting your time . but it is strange  that  i  firstly opened  the control without  the car there  is the bug i have ever mentioned ,however tonight when i opened  the project,the controller was  just like the one in your gif.
thank you all the same.
By the way,if i do not want the label to expand and contract,how can i do that ?

@tormorOoO just set enableRotate property to NO state


[build-script] Use a toolchain file and always cross compile cmark.

NOTE: We still build for the host platform always, but even in that case within
cmark itself, we are cross compiling it.

@swift-ci smoke test

preset=buildbot_iphoneos_arm64_crosscompiler
@swift-ci Please test with preset macOS

I want to prove that this still works on the bots for cross compiling cmark appropriately.

@swift-ci test

preset=buildbot_iphoneos_arm64_crosscompiler
@swift-ci Please test with preset macOS

@swift-ci test windows platform

preset=buildbot_iphoneos_arm64_crosscompiler
@swift-ci Please test with preset macOS

@swift-ci test

preset=buildbot_iphoneos_arm64_crosscompiler
@swift-ci Please test with preset macOS

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - a2264f1658d103c04bd1802e2b8c837dd405c3df

The cross compilation for arm64 worked. There is some other bug in build-script around cross compiling but I don't believe it is caused by this PR (we are trying to build swift-test-stdlib-macosx-x86_64 which isn't generated for some reason).

:doh: I forgot to make the cmark test only for macOS since I haven't fixed it yet on Linux.

Once the macOS test finished, I am going to push that fix and then do a smoke test.

Build failed
Swift Test OS X Platform
Git Sha - a2264f1658d103c04bd1802e2b8c837dd405c3df

Timedout in LLDB

@swift-ci smoke test

@swift-ci smoke test OS X platform


Remove once_cell::sync::Lazy dependency for std::sync::OnceLock

Description
Since std::sync::OnceLock is now stable in 1.70, once_lock can now be safely be removed. Also, this requires a version update by bumping the minimum version so this should be considered after a few more versions.

I am going to close this PR.
Because @sunli829 has updated this in 9c076fc to improve use, now rust-i18n users don't need to care about the once_cell now.
Thanks you.


Module 3 exercises [KESEGO MOTSWASELE]

Please "drag & drop" exercise 1 pdf file below.
LAND CONDITION IN PANDA EX 1 MODULE 3.pdf

Please "drag & drop" exercise 2
.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 3 and make sure to take
the post-training survey. Participating in surveys is required for receiving
the "GALUP Certificate" by the University of Florida.

Well done, Kesego. You clearly grasped the material outlined in Module 3 and successfully applied it to these exercises. Thank you for your participation!


Added new features to README.md and fixed crash when player had no armor in party manager

Approved?


Add github app details in the app settings

Tasks:

Add app description
Add home page url
Add OAuth(optional)
Add post installation destination
Add logo
List the app on the marketplace

added all the details in the GitHub app settings


https://clap.rs - certificate expired

Dupe of #887 I should have it up soon, sorry for the delay!


Is this still working like image donwloading ??

hi ,
How can i get accessToken your Link seem not working ok if I get accessToken is it possible to donwload images and videos pls reply anybody get any success here

Same question!..Did you resolve it?

yes but i didnt use this method because these are blocked by instagram because of change in API's

i made a demo for you check
it
https://github.com/sunil-singh-chaudhary/InstagramPic-VideoDownloader


Fix JS vulnerabilities

Summary
Updated JS dependencies for security fixes
Pull Request checklist

[x] ~Add/update test to cover these changes~
[x] ~Update documentation~
[x] ~Update CHANGELOG file~

This change is

This PR only affects dev dependencies!
We may ignore it for now.

Why close this?

Just to prevent distraction for now, as it is not needed for the upcoming release.
For that, I commented: "We may ignore it for now".

changelog?


Add group attribute flow to enable Audiences Member Of

The Active Directory Management Agent must have the sAMAccountName -> accountName Import flow on the ADMA in order for Audiences to resolve Active Directory groups when using a Member Of rule. This is currently missing in the PnP-Tools sample.

We stumbled upon this issue recently. AFAIK we didn't manually delete the sAMAccountName -> accountName Import flow in the ADMA, so propably this issue still isn't fixed in th PnP-Tools sample. Are you aware of that @tseward?


VEGA-1252 Fix unlink person pact tests

Uncommented unlink person pact test.
Issue not discovered yet, will see if failure on build pipeline provides more insight
VEGA-1252 #patch

Codecov Report

Merging #77 (dcebeb2) into main (52f0d45) will increase coverage by 1.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
+ Coverage   76.73%   78.02%   +1.29%
==========================================
  Files          41       41
  Lines         851      851
==========================================
+ Hits          653      664      +11
+ Misses        131      116      -15
- Partials       67       71       +4

Impacted Files
Coverage Δ

internal/sirius/unlink_person.go
50.00% <0.00%> (+50.00%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52f0d45...dcebeb2. Read the comment docs.