Output of evaluate and fit are not consistent

I'm running the cifar_cnn example. I'm not using data_augmentation (changed the flag to False) and also I'm only training for 10 epochs.
After 10 epochs, I'm computing the loss on the train using the evaluate function of the method. The loss printed by the fit function at the end of the 10th epoch is much higher than the error on train when using evaluate.
Can someone please help me to understand what's wrong?
Thank you!

fit and evaluate are measuring completely different quantities (fit monitors the average of the loss during training while evaluate looks at the loss of a static network, after training).

Can you explain more on the measuring of fit? the print on each epoch is
the mean loss from the beginning of the training or from the beginning of
that epoch?
On Sat, Aug 8, 2015 at 12:49 PM, François Chollet<EMAIL_ADDRESS>wrote:

fit and evaluate are measuring completely different quantities (fit
monitors the average of the loss during training while evaluate looks at
the loss of a static network, after training).
—
Reply to this email directly or view it on GitHub
https://github.com/fchollet/keras/issues/503#issuecomment-128944936.

From the beginning of the epoch. It's the sigma of the loss of each batch times the number of samples in the batch divided by the total number of samples, for the batches in the epoch considered.

Please see the output below. At the end of 10th epoch I'm getting a train
error of 0.5850
Then using evaluate I'm checking the error on the train and I'm getting
0.371. While I understand that there should be a difference between the
loss at the end of 10th epoch (which is 0.5850) it's still very far from
the loss when calling evaluate (which is 0.371).
Epoch 0
50000/50000 [==============================] - 23s - loss: 1.6584
Epoch 1
50000/50000 [==============================] - 23s - loss: 1.2406
Epoch 2
50000/50000 [==============================] - 23s - loss: 1.0426
Epoch 3
50000/50000 [==============================] - 23s - loss: 0.9223
Epoch 4
50000/50000 [==============================] - 23s - loss: 0.8315
Epoch 5
50000/50000 [==============================] - 23s - loss: 0.7670
Epoch 6
50000/50000 [==============================] - 24s - loss: 0.7101
Epoch 7
50000/50000 [==============================] - 24s - loss: 0.6657
Epoch 8
50000/50000 [==============================] - 24s - loss: 0.6241
Epoch 9
50000/50000 [==============================] - 24s - loss: 0.5850
50000/50000 [==============================] - 6s
Train Score 0.371491642175
On Sat, Aug 8, 2015 at 2:14 PM, François Chollet<EMAIL_ADDRESS>wrote:

From the beginning of the epoch. It's the sigma of the loss of each batch
times the number of samples in the batch divided by the total number of
samples, for the batches in the epoch considered.
—
Reply to this email directly or view it on GitHub
https://github.com/fchollet/keras/issues/503#issuecomment-128966177.

One more thing: with train the network is in train mode (Dropout ON), with evaluate it is in test mode (Dropout OFF). Hence big differences.

Makes sense. Thank you.
On Mon, Aug 10, 2015 at 1:04 PM, François Chollet<EMAIL_ADDRESS>wrote:

One more thing: with train the network is in train mode (Dropout ON),
with evaluate it is in test mode (Dropout OFF). Hence big differences.
—
Reply to this email directly or view it on GitHub
https://github.com/fchollet/keras/issues/503#issuecomment-129393279.


Remove default length from segment output

Segment metadata file is now correct .tsv file

Codecov Report

Merging #56 (72e3a77) into master (05b2d49) will increase coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
+ Coverage   83.57%   83.68%   +0.10%
==========================================
  Files          24       24
  Lines         615      613       -2
==========================================
- Hits          514      513       -1
+ Misses        101      100       -1

Impacted Files
Coverage Δ

include/valik/split/reference_segments.hpp
80.00% <ø> (+0.96%)
: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 05b2d49...72e3a77. Read the comment docs.


Optimised plus_one() and minus_one() methods

It's possible to add the value 1 to a dilated int in a more optimal way than the regular Add implementation.
Adding 1 can be implemented as: (dilated - DILATED_MAX) & DILATED_MAX
Whereas adding an arbitrary "rhs" is implemented as: (dilated + !DILATED_MAX + rhs) & DILATED_MAX
It's only 1 operation difference, but it's worth adding since this object could be on the user's critical path.
Subtract 1 is no different in terms of number of operations, but we'll add it for the sake of user code hygiene.

Work completed in #49

Reopening as I think this needs to be implemented as a trait

And I think this needs to be a trait because otherwise function requires a propagation of the inner trait bounds to the users code, which would be ugly

Closed by #50


legend的宽度限制失效

Version
4.8.0
Reproduction link
https://codesandbox.io/s/elegant-dirac-kvkox?file=/src/options2.js
Steps to reproduce
打开链接可直接看到bug效果
What is expected?
legend的宽度限制 正常
What is actually happening?
legend的宽度限制失效


WIP: CI: Automated Linux builds

[x] add .travis.yml
[ ] adjust settings to match for BusterLabs/Partyshare at deploy key:

[ ] change branch to master
[ ] change repo to Busterlabs/Partyshare
[ ] review if tagged commits should trigger the pipeline tags: true
[ ] @bhstahl: generate travis api_key via travis utility by running in the repo folder: $ gem install travis; travis setup releases

snippet from .travis.yml for help
deploy:
  provider: releases
  api_key:
    secure: <your key>
  file: dist/Partyshare.deb
  skip_cleanup: true
  on:
    repo: hexagon6/Partyshare
    branch: ci/upstream-build
tags: true

@hexagon6, I'm excited to get started on a OSX build pipeline, so I just put together https://github.com/BusterLabs/Partyshare/pull/48, which added a travis configuration file for us to work concurrently on our respective targets

With #49, I setup an OSX build & release pipeline using travis, and used it to build & upload release v1.2.3.
When you're ready, lets get #42 merged and this target added to release v1.2.4 with a linux app! 🍾

@bhstahl could you trigger a draft build so we see the contents of dist/ after building on travis?

Hmm, it's not letting me trigger a build since the code is in your fork. Let's merge this and work off master builds & PR's. Thanks for your help!

Alright, looks like we didn't get the auto-update files, but we do have the deb and tar.gz in the draft.


Taking off Preview from and SharePoint since we are GA for ODB

Minor changes to take out the word Preview and SharePoint since we are GA for ODB and have not entered Preview yet for SPO
Category

[x] Content fix
[ ] New article
Related issues: fixes #X, partially #Y, mentioned in #Z

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
(DELETE THIS PARAGRAPH AFTER READING)

What's in this Pull Request?

Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
(DELETE THIS PARAGRAPH AFTER READING)

Guidance

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'master' branch. Released documents are in live branch.
(DELETE THIS PARAGRAPH AFTER READING)

Removed "Preview" and SPO since we are in ODB phase

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/solution-guidance/multigeo-introduction.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


TS-4649: Minor style improvement to the loggin system.

Make log field name arrays static.
Make LogField members const.
Remove unused LogFilter::reverse().
Make various other trivial member functions const.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/416/ for details.

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/310/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/417/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/311/ for details.


fix not working in windows because path (#30)

#30

It's valid to use basename here, but on the other hand, isn't this tackling the symptom rather than the problem? Why is Windows (or Git-for-Windows?) having a problem with an absolute path argument in the first place?


Option to change and prioritize IGDB search parameter

Feature description
Right now there seem to be two different behaviors while using the IGDB metadata addon.

While using the bulk editor "Library" > "Download Metadata" the application searches for the Source Name.
Using the "Edit" function directly for a game uses the Display Name of the game.

It would be great to have an option to change and prioritize the search parameter when downloading metadata through the IDGB addon. Both while editing a single game and bulk editing.
For example (sorted by my personal priority):

IGDB ID*
Original Name*
Name (Display Name)
Source Name

If a parameter is absent it would use the next one.
Parameters with * not available right now.
IGDB ID
https://github.com/JosefNemec/Playnite/issues/595
Additional name fields (Original Name and Source Name)
https://github.com/JosefNemec/Playnite/issues/2814
https://github.com/JosefNemec/Playnite/issues/2859
Screenshots
Screenshot
Bulk editing of metadata

Directly editing of metadata

This doesn't make sense because Playnite stores only one name, which is what IGDB then uses. Your issue is mostly caused by unreadable search on IGDB API's side (for example #119 is affected by the same thing). I am slowly rewriting IGDB search to custom search, but it's not finished yet.


Matter installation is failed when ./install.sh is ran (This has been working two weeks back)

Describe the bug
We have been esp-matter SDK commit "6aa7601258b10eaa0cb3845c39f41109ac0bfbaf" for a while in our products, but recently we are facing an issue when we run the ./install.sh command in esp-matter
Environment

ESP-Matter Commit Id: 6aa7601258b10eaa0cb3845c39f41109ac0bfbaf (main)
ESP-IDF Commit Id: 697e3e8eace4a7d71bdffb2f53b49a0cacdaee9b  (release/v5.1)
SoC (eg: ESP32 or ESP32-C3): ESP32 and ESP32C3
Device Logs (Please attach the log file):
Host Machine OS:
Distributor ID:	Ubuntu
Description:	Ubuntu 22.04.4 LTS
Release:	22.04
Codename:	jammy
Commissioner app and versions if present: NA
Commissioner's logs if present: NA

Any additional details
...
matter_install_fail.log

It's related to protobuf:
ERROR: Cannot install -r python/gen/matter_build_venv/generated_requirements.txt (line 83), protobuf and protobuf~=3.20.1 because these package versions have conflicting dependencies.

@lboue We have been using this commit ID for last 4 months.. We are facing build issue recently, two week back. Is any dependency packages have updated ?

protobuf

protobuf/3.20.1 is an old version released in Mar 25, 2022.
Maybe it should be upgraded to a newer.

protobuf/3.20.1 is an old version released in Mar 25, 2022. Maybe it should be upgraded to a newer.

Any recommendation on updating protobuff ? Do we need to change the constraint.txt file with updated version ?

It seems also related to googleapis-common-protos:
pip._vendor.resolvelib.resolvers.ResolutionImpossible: [RequirementInformation(requirement=SpecifierRequirement('protobuf'), parent=None),
RequirementInformation(requirement=SpecifierRequirement('protobuf~=3.20.1'), parent=None),
RequirementInformation(requirement=SpecifierRequirement('protobuf!=3.20.0,!=3.20.1,!=4.21.1,!=4.21.2,!=4.21.3,!=4.21.4,!=4.21.5,<5.0.0.dev0,>=3.19.5'), parent=LinkCandidate('https://files.pythonhosted.org/packages/dc/a6/12a0c976140511d8bc8a16ad15793b2aef29ac927baa0786ccb7ddbb6e1c/googleapis_common_protos-1.63.0-py2.py3-none-any.whl (from https://pypi.org/simple/googleapis-common-protos/) (requires-python:>=3.7)')),
RequirementInformation(requirement=SpecifierRequirement('protobuf'), parent=None),
RequirementInformation(requirement=SpecifierRequirement('protobuf~=3.20.1'), parent=None),
RequirementInformation(requirement=SpecifierRequirement('protobuf!=3.20.0,!=3.20.1,!=4.21.1,!=4.21.2,!=4.21.3,!=4.21.4,!=4.21.5,<5.0.0.dev0,>=3.19.5'), parent=LinkCandidate('https://files.pythonhosted.org/packages/f0/43/c9d8f75ddf08e2a0a27db243c13a700c3cc7ec615b545b697cf6f715ad92/googleapis_common_protos-1.62.0-py2.py3-none-any.whl (from https://pypi.org/simple/googleapis-common-protos/) (requires-python:>=3.7)')),

protobuf/3.20.1 is an old version released in Mar 25, 2022. Maybe it should be upgraded to a newer.

Any recommendation on updating protobuff ? Do we need to change the constraint.txt file with updated version ?

You can try it while waiting for an official answer from Espressif.

@lboue A Similar issue is logged in the connectedhomeip repo
https://github.com/project-chip/connectedhomeip/issues/32533

EspressIf team, Any update on this issue ?

Some python dependencies broke this setup. Can you try bumping the few python module versions and give it a try?
cd connectedhomeip/connectedhomeip
git apply | curl https://github.com/shubhamdp/connectedhomeip/commit/b636dc0d0f7d20e249e8b5a400fdfa668254d2a9.diff

NOTE: This patch was curated on esp-matter commit 6aa7601258b10eaa0cb3845c39f41109ac0bfbaf and may not work as is on others, so please check the diff and make changes accordingly for other revision.

@aggaddam if this is not an issue anymore and can be solved by above comment, can you close it.

Closing this now

@aggaddam if this is not an issue anymore and can be solved by above comment, can you close it.

I am still facing the issue even after applying this patch
cd connectedhomeip/connectedhomeip
git apply | curl https://github.com/shubhamdp/connectedhomeip/commit/b636dc0d0f7d20e249e8b5a400fdfa668254d2a9.diff

Here is the issue log
matter_install_fail.txt


Documentation and Summary page improvements

Added note that definition should include qualifiers (closes #168)
Documented the CSV export format (related to #257, but I won't close that until I like to it from somewhere in the interface) as a Markdown file.
Fixed a bug in which we were providing all the expected resolutions before all the actual resolution in the CSV file, despite the header saying that they should be interleaved (i.e. expected 1, actual 1, expected 2, actual 2).
The Open Tree of Life lookup code was previously recording the provenance of the phylogeny in the phylogeny description field, but since this field is no longer used, I've moved it into the phylogeny curator notes field (see example below).

This PR also closes #300 by modifying the summary table to make it clear that the final columns refer to phylogenies and replacing phylogeny descriptions (which are no longer displayed in Klados) with phylogeny curator notes.

[ ] No need to add "(including qualifiers)" -- free-form text already says I can write whatever I want.

Thanks for the feedback, @hlapp! I've rewritten the "expected" and "actual" column descriptions as following:

Phylogeny: [Phylogeny label] expected: The label of the node in the specified phylogeny to which the phyloreference was expected to resolve by the author.

Example: Alligatoridae

Phylogeny: [Phylogeny label] actual: The label of the node actually resolved in the specified phylogeny to which the phyloreference was expected to resolve by the author.

Example: Alligatoridae_ott195670, or (unlabelled) if the node has no label

Is this clear?
The N in these descriptions was a defunct description of what we do if a phylogeny is unlabeled; in this case, instead of labeling this column Phylogeny: [Phylogeny label] expected, we use Phylogeny [Phylogeny index] expected, e.g. Phylogeny 1 expected. Should we include this information in this documentation as well? We could also use Phylogeny: [Phylogeny index] expected so they're a bit more similar to each other. What do you think?

See inline note. Can you comment, fix, and/or clarify?

I'm having trouble parsing The label of the node actually resolved in the specified phylogeny to which the phyloreference was expected to resolve by the author in contrast to The label of the node in the specified phylogeny to which the phyloreference was expected to resolve by the author. In the first case, is the node to which the phyloreference is found to resolve different from what the author seems to have expected, or is it the node's label that different from what the author seemed to have expected?

I'm having trouble parsing The label of the node actually resolved in the specified phylogeny to which the phyloreference was expected to resolve by the author in contrast to The label of the node in the specified phylogeny to which the phyloreference was expected to resolve by the author. In the first case, is the node to which the phyloreference is found to resolve different from what the author seems to have expected, or is it the node's label that different from what the author seemed to have expected?

Ugh, yes, that's a bit of a mess. How about:

Phylogeny: [Phylogeny label] expected: The label of the node in the specified phylogeny to which the phyloreference was expected to resolve.
Phylogeny: [Phylogeny label] actual: The label of the node actually resolved for this phyloreference in the specified phylogeny.

I've implemented this change in 2003205.


Sync issue

Hi,
I'd found for example that post on Hubzilla (one day old). No comments/likes here:

But when I clicked into it I found many comments:

It looks like synchronization issue in federation servers.
BR,
Adam

Is the hubzilla post from the discovery tab or from your stream?

It is from "Commented Order" but that post is from Diaspora Protocol Settings -> Followed hashtags

Ah... That is probably just beeing imported...

It's a known issue with Diaspora Relay Servers, which are still somewhat experimental and not yet universally supported within Diaspora. We have followed the relay protocol precisely and have subscribed to tags successfully, but the tagged content is only intermittently delivered to hubzilla by the Diaspora relay servers. If they don't send us the comments or the tagged posts we have subscribed to, there isn't a lot we can do about it.


Added the ability to handle spaces in classpath.

This requires parameters passed to the scala process runner to be in Seq, so that space characters are properly escaped.

Thanks @edward720! Your changes will be included in the next release.


References and Occurrences Do Not work Inside of JSDoc

TypeScript Version:  2.2.1
Code
/**
 * @typedef Product
 * @property {string} title
 */

/**
 * @type {Product}
 */
const product = null

Trigger references and occurrences requests for Product in both @type {Product} and @typedef Product.
Expected behavior:
In both cases, references and occurrences return the results for Product
Actual behavior:
No results returned. Here's the trace for @type {Product}
[Trace - 12:50:19 PM] Sending request: references (113). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/sand/bar.js",
    "line": 7,
    "offset": 15
}
[Trace - 12:50:19 PM] Response received: references (113). Request took 2 ms. Success: false . Message: No content available.

[Trace - 12:50:50 PM] Sending request: occurrences (114). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/sand/bar.js",
    "line": 2,
    "offset": 15
}
[Trace - 12:50:50 PM] Response received: occurrences (114). Request took 2 ms. Success: false . Message: No content available.

And for @typedef Product
[Trace - 12:53:41 PM] Sending request: references (118). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/sand/bar.js",
    "line": 2,
    "offset": 19
}
[Trace - 12:53:41 PM] Response received: references (118). Request took 1 ms. Success: false . Message: No content available.

[Trace - 12:53:37 PM] Sending request: occurrences (117). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/sand/bar.js",
    "line": 2,
    "offset": 19
}
[Trace - 12:53:37 PM] Response received: occurrences (117). Request took 1 ms. Success: false . Message: No content available.

Fixed by  #15856


➕ add ML books and courses

What does this PR do?

Add resource(s)
Add info

For resources
Description

Add The Mechanics of Machine Learning book
Add Made with ML course
Add Mathematics for Machine Learning By Imperial College London on YouTube
Change URL for Introduction to Statistical Learning to a more direct URL
Change URL for Mining of Massive Datasets to a more direct URL

Why is this valuable (or not)?
The added resources help to learn Machine Learning, MLOps.
The changed URL makes it one click to access the pdf.
How do we know it's really free?
The resources do not ask for any email/credit card info to get access and the site is either a trusted site or an official site.
For book lists, is it a book? For course lists, is it a course? etc.

Books

The Mechanics of Machine Learning
Introduction to Statistical Learning
Mining of Massive Datasets

Courses

Mathematics for Machine Learning (Video)
Made With ML (misc)

Checklist:

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

Follow-up

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

ok, all checks passed now 😌

Removed links, and added author to MML course.

Thanks, and welcome to free-programming-books!


fix(quorum/api): use Web3 HTTP Provider by default

It is deprecated because it does not support
subscriptions, but we do not use that feature anyway
and just need the examples to work so it seems like
an okay trade-off to use HTTP instead of WS.
Signed-off-by: Peter Somogyvari<EMAIL_ADDRESS>

@denis-yu-glotov  FYI: CI won't pass on this until we merge the other fixes in.

ok, then you need to add this rpcPort to config/config, no?

Ooops, yes. :-)
https://github.com/hyperledger-labs/blockchain-integration-framework/pull/66/commits/fac4060712eaabd075be9bfe61fb3074baeac283


Install libncursesw5-dev for ncurses wide character support

See docker-library/python#235
A number of packages should be built against libncursesw5 instead of libncurses5, most notably, Python. The two libraries do no conflict.

Travis having some trouble today but it's clearing up. Here is a passing build.

:+1:


SyntaxError: Strict mode does not allow function declarations in a lexically nested statement.

Got following error:

node 7.5.0
android 6.0
Can be related to this: https://github.com/facebook/react-native/issues/11389

I upgraded to https://github.com/bartonhammond/snowflake/releases/tag/0.2.1 and my Android installation and my GenyMotion and I am not able to reproduce this.
Please try upgrading everything mentioned and try again.  If you can reproduce it, let me know and I'll open this back up.  But for now, I'm closing.
Many thanks for checking out Snowflake!

Just activate a JS Minify option in the Emulator Dev Settings
In Emulator: Menu > Dev Settings > JS Minify ☑(active)

Here it worked only with this, I hope it has helped;)

@jwerlley Its great, but I am curious about to know the reason behind this issue. Can you elaborate with detail?

+1


support top of file # encoding lines

While not specifically import sorting this is also a top-of-file issue.
https://www.python.org/dev/peps/pep-0263/
thoughts?

@timothycrosley
I could reproduce the error with the following code snippet
foobar.py file
# -*- coding: utf-8 -*-
from django.db import models
#import json as simplejson
from myproject.models import Servidor
import reversion

import logging

after execute isort foobar.py, results:

import logging

import reversion
from django.db import models

# -*- coding: utf-8 -*-
#import json as simplejson
from myproject.models import Servidor

I have verified this is an issue and Ill be working hard to fix it before the next release
Thanks!
~Timothy

Fixed in version 4.1.0
Thanks!
~Timothy

Thanks


Add New Networks to Overlays

Description
Added various new networks to the networks overlay, along with both color and white images.
Adds Sky Atlantic (IT) to Sky networks
Type of Change

[X] New feature (non-breaking change which adds functionality)

Checklist

[x] My code was submitted to the nightly branch of the repository.

Hi, some things to fix in the color logos :

TVNZ 2 > change the black text to white
U+ Mobile TV  > change the gray text to white
VRT Max > change the black text to white

Also can you give us the source logos, we also need to do the collection poster for the new networks

I updated all three logos.
All source logos have been updated here

@meisnate12 ready to merge


An example of writing a call instruction?

Sorry, I'm a beginner in cheating.
When I was studying a sample project, I discovered that it runs by allocating memory and then writing binary code to the allocated memory (using pymem). I want to achieve the same functionality in memflow, but as a DMA, memflow does not provide, and seemingly should not provide, a memory allocation function.
My current idea is to find an allocated but unused memory block with memflow, such as the (capacity - len) portion of a vector, and then write the binary code into that memory block to simulate the memory allocation action in the example. Is this approach feasible? How can I find such a vector? Or what should I do? I'm looking forward to receiving some advice or links to related resources. I realize that it may not be appropriate to ask this question here, and I apologize for not being able to find a better place to ask.

Off-topic


DocumentAssembler generates large file

The size of the Assembled Document is around the same size as the template (word doc) from which it was produced. In my current case, this is a problem, because my word doc template is 27 megs, but has numerous conditionals (especially around large images I need to hide/show based on data).
After the doc is assembled, I expect the assembled file to only be around 5 megs. Testing this assertion, I can open the 27 meg assembled document into MS Word, make a small change (add a space for example), save it, and now the saved file is only around 5 megs. This can make a huge difference in user experience, for example, when my users attempt to use my generated document as attachments in mail systems with attachment size limits.

Closing all issues as this repo is being archived and will no longer be maintained by Microsoft. The project is licensed for continued use and development by forking to your own repo.


failing test in transacting/transacting_test.go

Related to unupdated WebApi (hypothesis); went unnoticed before as transacting_test.go was transacting_tes.go; renaming required for reproducing
✘-1 ~/golang/src/github.com/eris-ltd/eris-db [failing_transaction L|●1…1⚑ 1]
11:36 $ go test ./test/transacting/
INFO[03-23|11:37:45] Server started.                          module=eris/server
INFO[03-23|11:37:45] File written.                            module=eris/serverserver name=/tmp/.edbservers/testnode29001/config.toml
INFO[03-23|11:37:45] File written.                            module=eris/serverserver name=/tmp/.edbservers/testnode29001/priv_validator.json
INFO[03-23|11:37:45] File written.                            module=eris/serverserver name=/tmp/.edbservers/testnode29001/genesis.json
Write and backup
INFO[03-23|11:37:45] File written.                            module=eris/serverserver name=/tmp/.edbservers/testnode29001/server_conf.toml
process started, waiting for token
INFO[03-23|11:37:46] [GIN] HTTP: <IP_ADDRESS>:50282              module=eris/server Code=200 Method=POST path=/server error=
Received Port: 29001
URL: http://localhost:29001
--- FAIL: Test_A0_Tx_Create (2.01s)
        Location:       transacting_test.go:130
	Error:		No error is expected but got Post http://localhost:29001/unsafe/txpool?hold=true: EOF

panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x4890ef]

@NodeGuy you might have an interest to look at this bug.

The similar API calls pass in the mock test, using gin, rather than erisdbss, specifically
https://github.com/eris-ltd/eris-db/blob/master/test/mock/mock_web_api_test.go#L313
this seems to be a pure question of clean-up

tests pass for /unsafe/txpool rather than /unsafe/txpool?hold=true; patching test and raising new issue for holding

Why would I be interested in this bug?


video recording with audio

here is a "dirty" commit just to get the job done asap.
do we need audio recording? I find it usefull, well at least for the particular use case of mine
https://github.com/atzorvas/headless/commit/f8cc361bfe6b52167302080ae82df0384057fac0

But this can be accomplished with the :extra recorder option, as in
Headless.new(video: { extra: %w(-acodec libmp3lame -ar 44100 -ab 128k -threads 0) }

Closing since it's been a year, please reopen if you have any questions.


[Bug]  explode_json_array_sting 无法解析复杂json

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.12
What's Wrong?
使用explode_json_array_string 函数解析 jsonObject 数组时解析为空
结果:

k1
e1

1
null

1
null

select k1, e1 from (select 1 as k1 ) t lateral view explode_json_array_string('[{"k1":"v1","k2":"v2"},{"k1":"v11","k2":"v22"}]') tmp1 as e1
What You Expected?

k1
e1

1
{"k1":"v1","k2":"v2"}

1
{"k1":"v11","k2":"v22"}

How to Reproduce?
select k1, e1 from (select 1 as k1 ) t  lateral view explode_json_array_string('[{"k1":"v1","k2":"v2"},{"k1":"v11","k2":"v22"}]') tmp1 as e1
Anything Else?
No response
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

when we use 'explode_json_array_string', we should give an array string which element is still string.
For example, your case is '[{"k1":"v1","k2":"v2"},{"k1":"v11","k2":"v22"}]', the first element of array is {"k1":"v1","k2":"v2"}, it is not a string because there is no quotation marks.
So, this string can meet your needs : '["{\'k1\':\'v1\',\'k2\':\'v2\'}","{\'k1\':\'v11\',\'k2\':\'v22\'}"]'

when we use 'explode_json_array_string', we should give an array string which element is still string. For example, your case is '[{"k1":"v1","k2":"v2"},{"k1":"v11","k2":"v22"}]', the first element of array is {"k1":"v1","k2":"v2"}, it is not a string because there is no quotation marks. So, this string can meet your needs : '["{'k1':'v1','k2':'v2'}","{'k1':'v11','k2':'v22'}"]'

怎么转变么。


Join 2 articles public form views into one for sake of DRY

Solves #336

In my opinion it is extremely helpful to have best practices in demo code. Is there a reason this hasn't been merged yet?

@ramusus Would you mind updating this PR against the latest master branch? I'd really like to see this change myself. Unless there's any objections of moving to a dry structure? @lirantal @ilanbiala @mleanos @trainerbill @rhutchison

@ramusus this looks fine to me, albeit some changes that need to be made such as removing data- and fixing the indentation.

@ramusus would you be able to tidy up this PR so it can be merged?

Sorry, unfortunatly I'm very busy now

@ilanbiala @codydaig
This looks like a good implementation. Since @ramusus doesn't have time for this, I'll take it over. This is related to #807 & #874 Once the latter is decided on, then I will work on this implementation.

@mleanos why do you need #874 to work on this?

@ilanbiala #874 is a refactor of the Article module. However, since that PR is set for 0.5.0 then this can probably move forward with consideration for both #807 and #874

@ilanbiala Can this be moved to 0.5.0?

mleanos do you need #874 to work on this?

@ilanbiala I'm not sure. I need to take another look at this implementation. I'd rather wait until the dust settles with #874 since it's a huge refactor, and this implementation might not be relevant once we implement John Papa's styleguide; I'm not exactly sure though.
Also, #807 will introduce a change in how we're allowing users to edit/create Articles, it may make sense to hold off on this; or just address these concerns in #807

@ramusus I agree with the direction, but not completely with the implementation.  The controller should call $save and the service should work out whether it's an insert or update.
@codydaig @mleanos

@rhutchison I agree. I would like to reintroduce this with #807. Also, would prefer to add the angular styleguide with it, since the article refactor PR has stalled for so long. We need to get moving on the styleguide, so we can have it out with the 0.5.0 release.


The last line of a returning function isn't executed if it isn't stored somewhere

Consider a very simple stack:
SimpleStack: class <T> {
	_data: T*
	_count: SizeT = 0
	_capacity: SizeT = 8
	init: func { this _data = calloc(this _capacity, T size) }
	push: func (element: T) {
		this _data[this _count] = element
		this _count += 1
	}
	pop_broken: func -> T {
		this _data[--this _count]
	}
	pop_works: func -> T {
		this _count -= 1
		this _data[this _count]
	}
}

Popping the stack using temp := myStack pop_broken() works, but if the line is simply myStack pop_broken(), then his _data[--this _count] isn't executed at all (and thus this _count is never decreased). The pop_works method always works because this _count is decreased, and then the last (returning) line is not executed, but that's fine, since we don't care about it.
A simple example demonstrating what works and what does not:
// This does not work
stack_broken := SimpleStack<Int> new()
stack_broken push(1) . push(2) . push(3)
a := stack_broken pop_broken()
stack_broken pop_broken() // The last line of pop is never run because the return value isn't stored
b := stack_broken pop_broken()

a toString() println() // Should be 3, is 3
b toString() println() // Should be 1, is 2 :(

// Here is a working way
stack_works := SimpleStack<Int> new()
stack_works push(1) . push(2) . push(3)
a = stack_works pop_works()
stack_works pop_works() // Here, the return value isn't stored, but it works anyway because _count is decreased before the last line of the method
b = stack_works pop_works()

a toString() println() // Should be 3, is 3
b toString() println() // Should be 1, is 1 :)

Will take a look at it today.
Sounds weird, probably some generics bug (what I'm thinking is happening is that the whole function becomes a single return statement and isn't executed when you don't store the result).

@shamanas I ran into something similar a few weeks ago, and I found that the generated C-code is checking if the receiver is a null pointer, and if it is, nothing is done. That is, instead of a pointer to an lvalue, Literal `NULL´ is passed to the generated C-function.

void stack_pop(stack_t* stack, int* returnValue) {
    if (returnValue != NULL) {
        // do stuff
    }

and the generated call without an lvalue would be: stack_pop(theStack, NULL)

Yes, generic return is implementing by taking a generic pointer for output, so if we discard the return value we pass NULL, than NULL checks are done around every return statement, so it's exactly like I suspected.
Now, the solution is obvious:
Instead of just generating an if statement, also generate an else branch that just executes he expression.

Actually, it looks like rock already tries to take this into account but hasSideEffects doesn't detect the side effects here :P

@marcusnaslund By the way, there is no decrement or increment operators in ooc (postfix or prefix), prefix operators just incidentally work because we have prefix + and - and it is apparently legal to have more than one.
Should probably fix that too.

There's no side effects in pop_broken - if you don't store the result, it's a no-op (it isn't currently due to OOC emitting "--" in C, but semantically should be)

Yeah, I agree, I am talking about the exact same code replaced with the correct way to do things this count -= 1.
hasSideEffects always returned false for array accesses.

Also, BinaryOp returned the opposite of what it should (it returned !isAssign(), while it should return isAssign())

Ah, makes sense


Fix flake8 files to run.

[X] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.
[X] I've been mindful about doing atomic commits, adding documentation to my changes, not refactoring too much.
[X] I've a descriptive title and added any useful information for the reviewer. Where appropriate, I've attached a screenshot and/or screencast (gif preferably).
[ ] I've written tests to cover the new code and functionality included in this PR.
[X] I've read, agree to, and signed the Contributor License Agreement (CLA).

PR Summary
Updating the flake8 command to adjust for the new structure.

Coverage remained the same at 29.268% when pulling 28735ade88698e90237cb65ac90176b2bde2e6ae on jammons:master into 92357ed343c954e2bee66de3d72581765af991b7 on slackhq:master.

Coverage remained the same at 29.268% when pulling 28735ade88698e90237cb65ac90176b2bde2e6ae on jammons:master into 92357ed343c954e2bee66de3d72581765af991b7 on slackhq:master.

Coverage remained the same at 29.268% when pulling 33d558083dac62e7c233a977d87d046a46ff9fcb on jammons:master into 92357ed343c954e2bee66de3d72581765af991b7 on slackhq:master.


page 584, eq 17.94 typo

on the RHS there is unnecessary left bracket before the sum encapsulating the kernel. This looks like the remnant of the copy paste of the expression in the middle of the equation chain.

fixed, thx


fix cuda half math function is undefined: hpow, htanh

I was trying to transfer BERT model to TVM, but encountered some problems with undefined function: hpow, htanh.

with refer to the cuda user manual, I just simulate with float32 function.
Thanks !

cc @yongfeng-nv @wpan11nv @yzhliu @vinx13


Please integrate option to change vrule color

As in https://github.com/foreachthing/latex-awesomebox/commit/91d7337e081e2d06e9bd97d6bf7956966d605133

Thanks!


xpath or accessibility features other than name does not work for some elements in consequence run

Hi,
I have a problem to access elements in a Pytest run from console. the details are below.
I have test cases in two modules and inside two classes. I have fixture to open the app by using driver and the scope of this fixture is "class". When I run the test from PyCharm individually, it find out every elements and run perfectly. But if I want to run both modules. then I go to terminal and run from there. Test cases in the first module (also class) run without issue. since the fixture is set as "class" scope, it tears down and reopen the app. When it open the app again. the driver is acting on some items but I have a list items set, it does not find xpath for that items. Only name features can be used (this is happening only from console (terminal) run, it is not an issue in individual run from PyCharm, as stated above). I cannot use the name property because I am actually testing if it's name is that or different. Moreover, the name property is the value in the list view item that should change in program by setting a new value, generally. xpath or automationid are more stable but they are not recognized for the second module (class also).
Sorry I have tried to describe the problem as much as I can.
Please help me with that issue
inoterpreter setting:
python : 3.10
pytest : 7.3.1
selenium: 3.141.0
appium: 1.3.0

I am unbale to understand the issue. can you share you code?


[5.2] Let specific middleware be skipped for testing

Let specific middlewares be skipped for testing

Replaces #14223

I think I will actually hold off on this. When testing, you would typically disable all middleware and test the controller in isolation. Then, test each middleware individually. I don't see the great value of being able to disable just 1 or 2 middleware in a stack. Sorry 😁

@taylorotwell when you say test the middleware individually, are you referring to unit tests? IMO the sort of logic in a middleware is the same as in controllers (returning responses and redirects etc) and that sort of stuff is a lot easier to test at the acceptance level (in the same way a lot of people, myself included, see no value in unit testing controllers).
Also, if we do as you say and 1. disable middleware and test the controller in isolation, and 2. Test middleware in isolation, then how do you suggest testing that they correct middleware are assigned to the correct routes?
For example, I want to be sure that you need to be logged in to view /admin. My AdminController is tested with middleware disabled, and my auth middleware is well tested, but if I forgot to assign the auth middleware to the the AuthController then my test suite would be passing but expected behaviour of having to be logged in to view the admin area actually didn't happen. How would you suggest I test this?


chore: use devcontainer

Read Developing inside a Container.
Using DevContainers allows us to share the development environment with all its dependencies.
Related to #6

It's ready


Allow implicit branch creation on file uploads / moves

Consider the following use case:
import pandas as pd

storage_options = {"client": client}
df = pd.read_parquet("lakefs://my-repo/main/lakes.parquet", storage_options=storage_options)

... # process the data, add columns, whatever

pd.to_parquet("lakefs://my-repo/new-branch/lakes.parquet", storage_options=storage_options)

Here, it might be beneficial to allow the creation of new-branch if it does not exist.
BUT: It might be surprising to silently create branches (think typos, etc.), so this should most likely be a toggle switch with a sensible default behavior.
The branch creation would only be relevant for additive operations, i.e. those that add or duplicate data (e.g. put, mv, cp).
Implementation

[x] Add a boolean switch (e.g. create_branch_ok) to the LakeFSFileSystem constructor, with documentation.
[x] Add the switch to the LakeFSFileSystem.scope() context manager, same as the other two options.
[x] Create the branch via client.branches_api.create_branch before an additive operation if create_branch_ok is True - either by creating if it does not exist (incurs a branch listing, which might be expensive for repos with many branches) OR by branch-ensuring via an unconditional create (needs client-side error handling).
[x] Create the branch in the same way for the LakeFSFile if it is opened in write mode, e.g. in __init__ if mode == "wb" OR in _upload_chunk directly before the put_object call).

Closed via #15.


Is it possibly to train models every N batchs in order to complete long trainings ? And should we set a test or validation data to these trainings even we know that we wont finish our first epoch ?

@Aminejr1
Moving this issue to closed status as there has been no activity, in case you still face the error please create a new issue.


remove requested mandatarissen

No deltas in migrations. Either run the migration connected to mu-auth or wait for the nightly healing


Updated assignment_1 answers in Assignment1 branch

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Added assignment_1 answers python file
What did you learn from the changes you have made?
I had the yahoo finance data available, the code worked but then I lost all the parquet files and got error messages throughout the code that I couldn't resolve despite lots of attempts
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Could not resolve errors
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
I had the yahoo finance data available, the code worked but then I lost all the parquet files and got error messages throughout the code that I couldn't resolve despite lots of attempts. Also had so so many problems completing this PR with problems syncing Github with my local machine because of a lot of conflicts.
How were these changes tested?
Kept running code so so so many times
A reference to a related issue in your repository (if applicable)
Checklist

[X] I can confirm that my changes are working as intended

Assignment 1 answers were redone. A new PR will be completed with a different file.


hiding html-reporter doesn't work

after meteor add velocity:html-reporter, all I see is the reporter html page with no way to 'see' my actual apps html anymore. Ctrl + v doesn't seem to work either? any ideas?
Screenshot so to show the problem I am facing. Probably super-ridiculous but still...  I am using webstorm on the right, chromium on the left, window manager is awesome on Debian.
After removing it again, apps html is back as before

+1, I had the same issue when upgrading to the 1.2 RC preview. Not sure if it's related. The first time I had checkmarks without text and the latest time no checkmarks and no text.

Fixed with 0.9.0


Update Scribe description and image location

I'm reorganizing the repo to put GH only images in .github. Realized that I should update the path here :)

Thank you! Hope you're well! 😊


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_7143c1c9-ddfb-40c6-80af-73c1c8730f8f/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_7143c1c9-ddfb-40c6-80af-73c1c8730f8f.git

Docs Build status updates of commit a157f7c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/opstest2/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit a157f7c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/opstest2/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Prevent crash on unset background color

HTMLColor2RGBA() tries to read its parameter as a string. If it receives undefined, it throws an uncaught TypeError. Filter input on the front end to prevent.

Thanks, looks good!


Simplify tautologies

Idea is to allow for tautologies in to_CNF conversions so that model counts aren't thrown off (variables mentioned in a tautological clause will be missing from the compilation).

Closes #30

Great idea! It caught a corner case where (x & ~x) | y was being simplified to just y. Can you have another look after the latest commit?

Ah! I hadn't spotted that.
My only concern now is that the name simplify_tautologies is inaccurate because it also covers contradictions.

Ah, right. Just simplify work?

Thanks, @blyxxyz ! Should be all set to go if you want to approve/merge.


progressbar should display after button click

Issue #627
@championswimmer Please Merge

Current coverage is 47.11% (diff: 100%)

Merging #626 into development will not change coverage

@@           development       #626   diff @@
=============================================
  Files                5          5
  Lines              554        554
  Methods             52         52
  Messages             0          0
  Branches            95         95
=============================================
  Hits               261        261
  Misses             293        293
  Partials             0          0

Powered by Codecov. Last update 4d793e8...1a09ae1


Clarify where the config is read from

Relevant when using --kube-subnet-mgr, e.g. when using the provided Kubernetes manifest.
Description

type of fix: documentation
why this PR should be merged: save others from wasting their time dealing with etcd needlessly

Release Note
None required

Thanks for the fix @ncabatoff - merging.


RegEx Pattern works on regex101.com but does not work with select-string -pattern

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
$vssadmin =  vssadmin list writers
($vssadmin | Select-String -Pattern "(?msi)^Writer name: (?'writer'.*)\r?\n Writer Id: (?'id'[^:])\r?\n Writer Instance Id:
(?'instance'[^:])\r?\n State: (?'state'[^:])\r?\n Last error: (?'err'[^:])\r?\n$" -AllMatches ).Foreach{$_}
Expected behavior
Named groups of matches. writer, is, instance, state and err.

Actual behavior
Empty string

Error details
No error

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

Visuals
(https://regex101.com/r/Itarw8/1)

I think your regex has multiple issues. I don't have time at the moment to really dig into it, but wanted to offer an alternative approach.
switch -regex ($vssadmin){
    "^\w.*'(?<Name>.*)'$" {$ht = @{Name = $matches.name}}
    "^\s+(?<Name>\w[\w\s]*): (?<Value>.*)$" {
        $ht.Add($matches.Name,$matches.Value)
        if($matches.Name -like '*err*'){
            [PSCustomObject]$ht
        }
    }
}

I suspect that $vssadmin is not a multiline (here) string but $vssadmin -is [array]
Possible solution:
$vssadmin = vssadmin list writers |Out-String

I think your regex has multiple issues. I don't have time at the moment to really dig into it, but wanted to offer an alternative approach.
switch -regex ($vssadmin){
    "^\w.*'(?<Name>.*)'$" {$ht = @{Name = $matches.name}}
    "^\s+(?<Name>\w[\w\s]*): (?<Value>.*)$" {
        $ht.Add($matches.Name,$matches.Value)
        if($matches.Name -like '*err*'){
            [PSCustomObject]$ht
        }
    }
}

I miss some output with this code

Multiple problems with the regex.  which I've fixed below, @iRon7  had the main problem though.
$Pattern = "(?msi)" +
           "^Writer name: (?'writer'[^\r\n]+)"           +  #  select after label to the end of line
           "\s*Writer Id: (?'id'[^\r\n]+)"               +  #  skip newline and spaces at the start,
           "\s*Writer Instance Id: (?'instance'[^\r\n]+)"+  #  and repeat
           "\s*State: (?'state'[^\r\n]+)"                +
           "\s*Last error: (?'err'[^\r\n]+)"
vssadmin list writers | Out-String |
        Select-String -Pattern $Pattern -AllMatches | ForEach-Object matches |ForEach-Object  {
            $_.groups | where-object name -ne 0  |
                foreach-object -begin {$ht=@{}} -process {$ht[$_.Name] = $_.value} -end {[pscustomObject]$ht}
        }

On a general note, @B-Art:

With respect to regexes, PowerShell is just the facilitator: the .NET regex engine does all the work.

Unless you have reason to believe that there is a problem with how PowerShell facilitates this access (which would require showing a PowerShell statement that behaves differently from the equivalent direct .NET API call), a potential bug with respect to regex processing should be reported at https://github.com/dotnet/runtime/issues (I don't think there is one here).

A notable difference between PowerShell-mediated regex functionality vs. direct .NET API use is that PowerShell uses case-insensitive regexes by default.

With respect to  using regex101.com  - a great tool, to be sure - for PowerShell, there are pitfalls:

It assumes different defaults (even if you choose the NET (C#) flavor): notably, the m (Multiline) option is on by default, and i (CaseInsensitive) is off by default; the g (global) option, which has no direct .NET equivalent is on by default, but it doesn't apply to match or switch -Regex, for instance.
It assumes (verbatim) C# string-literal syntax, requiring escaping of " chars. as ""
See https://github.com/firasdib/Regex101/issues/1838 for a request to support a PowerShell-specific flavor that would eliminate these problems.
However, two pitfalls will remain:

The site only supports a single, multiline input string to test against.

This notably contrasts with PowerShell often reporting lines as a stream of multiple, individual (newline-free) strings, as when you use Get-Content (without -Raw) and when you process output from external programs, such as vssadmin.exe in your case.
As @iRon7 has pointed out, piping to Out-String is the simplest solution for external programs; for Get-Content, it is -Raw.
Also note that Select-String, when given a file (path or file-info object) as input also searches line by line; searching across lines requires an idiom such as Get-Content -Raw ... | Select-String ...

Pasting multi-line strings as subject strings (field "TEST STRING") invariably converts CRLF (Windows-format) to LF (Unix-format) newlines.

@mklement0
Thank you for your excellent explanation.
I am aware that lost of functionality in PowerShell leans on .Net.
For instance -Replace is completely different to .Replace(,) I know.
-Replace can use Regex while .Replace(,) cannot.
I use PowerShell almost every day and I will keep on using it.
Again, thanks for your reply.


Custom sanitizing rules ignored

Bug Description
When I run vendor/bin/phpcs class-foo.php, I get this error:
------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------
 11 | ERROR | Detected usage of a non-sanitized input variable: $_POST['foo']
------------------------------------------------------------------------------

I would expect the error to be suppressed because sanitize_promos_array is whitelisted.
Minimal Code Snippet
<?php

class Foo {

	public static function sanitize_promos_array( $input ) {
		// sanitize me
		return $input;
	}

	public static function bar() {
		$taxonomies = isset( $_POST['foo'] ) ? self::sanitize_promos_array( wp_unslash( $_POST['foo'] ) ) : []; // phpcs:ignore WordPress.VIP.SuperGlobalInputUsage.AccessDetected,WordPress.Security.NonceVerification.Missing
	}

}

phpcs.xml:
<?xml version="1.0"?>
<ruleset name="MyRuleSet">
	<rule ref="WordPress-Extra" />
	<rule ref="WordPress.Security.ValidatedSanitizedInput">
		<properties>
			<property name="customSanitizingFunctions" type="array">
				<element value="sanitize_promos_array" />
				<element value="sanitize_announcements_array" />
			</property>
		</properties>
	</rule>
</ruleset>

Environment

Question
Answer

PHP version
7.3.3

PHP_CodeSniffer version
3.4.2

WPCS version
2.1.1

WPCS install type
Composer project local

Additional Context (optional)
This worked fine with WPCS 1.x. The rule was defined like so:
	<rule ref="WordPress.Security.ValidatedSanitizedInput">
		<properties>
			<property name="customSanitizingFunctions" value="sanitize_promos_array,sanitize_announcements_array" type="array" />
		</properties>
	</rule>

You seem to be sanitizing with self::sanitize_promos_array(), rather than the global sanitize_promos_array(). I'm not sure if there is a way to define non-global sanitizing functions (methods). @jrfnl?

Yes, we are using a class method. This worked with WPCS 1.x.
Also with WPCS 1, putting self::sanitize_promos_array() in the ruleset did not work.

@paulschreiber I'm not sure if my issue is related to this, I have the following error.
"Detected usage of a non-sanitized input variable" with $_POST['taxonomoy_ordering_data']
This is an array of numbers that are being sanitized using filter_var_array
filter_var_array( wp_unslash( $_POST['taxonomy_ordering_data'] ), FILTER_SANITIZE_NUMBER_INT )


JWT token performance issue when renew_token_without_revoking_existing is enabled

Describe the issue:

A delayed response time can be observed when the renew_token_without_revoking_existing config is set to true in IS v5.11.0 (shown below).
[oauth.jwt.renew_token_without_revoking_existing]
enable = true
allowed_grant_types = ["password", "client_credentials", "refresh_token"]

Furthermore, the response time continues to get worse when the number of tokens in the DB increases. This needs to be fixed.

Optional Fields
Related issues:

Suggested labels:

The issue has been resolved, and the fix has been merged into the master branch [1]. Additionally, the migration documentation has been updated. Therefore this issue is can be closed.
[1] https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/2630


sendData with array, delay, repeat, callback

Instead of bare string like now, I've tried to upgrade the sendData to support following schema in payload.
Repeat and array and waitAfter
[
 {"hex": "260058000001269412121237131114121236133615101413113613111536123613121411123712381236131115111410141113131311143513121435133712361336153513361213120005260001284912000c4f0001284715000d05", "waitAfter": 1000, "repeat": 2},
"260058000001269412121237131114121236133615101413113613111536123613121411123712381236131115111410141113131311143513121435133712361336153513361213120005260001284912000c4f0001284715000d05"
]

Bare string still supported
"260058000001269412121237131114121236133615101413113613111536123613121411123712381236131115111410141113131311143513121435133712361336153513361213120005260001284912000c4f0001284715000d05"

This could replace the switch-multi and switch-repeat (albeit the auto off after sending success)
Also, the sendData will now have callback, which will obviously help Homeapp UX
I'm sorry for undocumented code, but this works great with my projector (on: power toggle + switch input, off:  2x power toggle)

I can try to refractor switch-multi and switch-repeat to adapt new sendData API

Hey - I'm not sure this is the direction I want to take sendData. Other than switch multi and switch repeat what accessories do you envision using repeat and delay?

Also, there's no point in having callbacks for a function that doesn't do anything asynchronously.

Finally, I'd rather not allow the config to allow for different structures. I'm open to changing the config structure though.

Because I put the logic of delay and repeat inside the sendData, callback is required to notify that when it's all sent. The delay logic is asynchronous, therefore the need for callback. Otherwise, we would need to sleep everything for the delay logic.
Also switch-multi doesn't work for my case because it turns off automatically, I want to use Home app automation but it keeps turning off so it caused infinity loop in some cases.

@duyleekun You can use "disableAutomaticOff" as of 1.1.8 which will achieve what you need. sendData has been refactored to simplify it further. I don't see that we need to move support for delay/repeat in there unless other accessories need it too.


fix: update template .envrc LOG_LEVEL to LOGLEVEL

I found that when I used debug! to print logs, it always had no output. Eventually, I discovered that in the .envrc file it was LOG_LEVEL while in the code it was LOGLEVEL.

I would prefer the other way around and use LOG_LEVEL and update the code accordingly. I think it is more conventional, what do you think?

yes， should use LOG_LEVEL

want to make the change in the PR? 🙂

LOGLEVEL changed LOG_LEVEL.😀

Thanks!


training script

Dear Zhaoyang,
Thanks for your awesome work.
Could you please provide the training script? Did you use the fp16?

Thanks for your question!  We do not train DDPMs on ImageNet or LSUN datasets by ourselves. We use pretrained checkpoints from the repo https://github.com/openai/guided-diffusion as described in ReadMe.

Thanks for reply.


Added tests for "Value only" constraint

5 positive and 4 negative tests added

Ugh. Wrong merge target


fix pep8 error

pep8 run-test-pre: PYTHONHASHSEED='84345641'
pep8 run-test: commands[0] | flake8
pep8 run-test: commands[1] | flake8 /root/python-zvm-sdk/scripts/sdkserver
_______________________________________________________________ summary _______________________________________________________________
  pep8: commands succeeded
  congratulations :)

Ran 12 tests in 0.003s

OK
_______________________________________________________________ summary _______________________________________________________________
  py27: commands succeeded
  congratulations :)

minor change, no functional impact


Import data from Move.mil

Description
For the Rate Engine project, we need access to data that is currently stored in the "Tariff 400NG Baseline Rates" excel files. One way to get this data in the database would be to write a parser. To accelerate getting this data available, we got a database dump from Move.mil, which itself has an excel parser.
In this PR you'll find 3 major things:

The database dump from move.mil. There's not a lot to verify about this. It is what it is. This is captured in a migration file.
Another migration that translates the move.mil schema into something more like what we want. Feedback on this is welcome. The way this works is that first we import the (essentially) raw database from move.mil into a set of tables. Then, we select from those tables into new tables that we want.
Model files and tests for the tables that we imported in the above two steps.

Code Review Verification Steps

[x] All tests pass.
[x] Code follows the guidelines for Logging
[x] The requirements listed in
Querying the Database Safely
have been satisfied.
[x] Request review from a member of a different team.
[x] (TRIAL) Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

Re: your two concerns:

The millicents thing seems like an exception that is bound to be confusing to people, so I'm hoping to quarantine it to the one place it's used. I'd like to put good tests around it and hide the values behind accessors such that we don't have to think too much about it anymore.

For the ranges, yeah, I don't see how we would plug them into Pop. This is kind of related to the millicents thing as well (it'd be nice to use decimal types, but Go doesn't have them and Pop certainly doesn't know about them.) If we have a reasonable way of connecting custom types into Pop, maybe it would make these two things better.


authorisation control based on data in the tables!

Which Category is your question related to?
Usage
Amplify CLI Version
4.41.2
What AWS Services are you utilizing?
Cognito + Dynamodb
I am developing a B2B app. It means I have some customers (users) that are Admins and also some customers of the customer (normal users) and some employee of my customers.
Now assume I have the following models:
type User
@model
@key(name: "byEmail", fields: ["email"], queryField: "listUsersByEmail")
@auth(rules: [{allow:groups, groups:["Supers"], provider: userPools, operations:[create, update, delete, read]},
              {allow:groups, groups:["Admins"], provider: userPools, operations:[create, delete, read]},
              {allow: owner, ownerField: "id", operations: [update, read]}
             ])
{
  id: ID!
  email: String!
  name: String
  status: String
  createdAt: String
  updatedAt: String
}

type Institute
@model
@key(name: "sortByName", fields: ["self", "name"], queryField: "listInstitutesSortByName")
@auth(rules: [{allow:groups, groups:["Supers"], provider: userPools, operations:[create, update, delete, read]},
              {allow: private, provider: userPools, operations: [read]}
            ])
{
  id: ID!
  self: Int! @default(value: "1") @auto
  name: String!
  address: String
  tell: String
}

type Employee
@model
@key(name: "byInstitute", fields: ["instituteId"], queryField: "listEmployeesByInstitute")
@auth(rules: [{allow:groups, groups:["Supers"], provider: userPools, operations:[create, update, delete, read]},
              {allow:groups, groups:["Admins"], provider: userPools, operations:[create, update, delete, read]},
              {allow: owner, ownerField: "userId", operations: [read]}
             ])
{
  id: ID!
  instituteId: ID @assert(condition: ".length() > 1") @auth(rules: [{allow:groups, groups:["Supers"], provider: userPools, operations:[create, update, delete, read]},
              {allow:groups, groups:["Admins"], provider: userPools, operations:[create, update, delete, read]},
              {allow: owner, ownerField: "userId", operations: [read]}
             ])
  institute: Institute @connection(keyField: "instituteId")
  userId: ID @assert(condition: ".length() > 1") @auth(rules: [{allow:groups, groups:["Supers"], provider: userPools, operations:[create, update, delete, read]},
              {allow:groups, groups:["Admins"], provider: userPools, operations:[create, update, delete, read]},
              {allow: owner, ownerField: "userId", operations: [read]}
             ])
  user: User @connection(keyField: "userId")
  suspended: Int @default(value: "0") @auth(rules: [{allow:groups, groups:["Supers"], provider: userPools, operations:[create, update, delete, read]},
              {allow:groups, groups:["Admins"], provider: userPools, operations:[create, update, delete, read]},
              {allow: owner, ownerField: "userId", operations: [read]}
             ])
  accepted: Int @default(value: "0") @auth(rules: [{allow:groups, groups:["Supers"], provider: userPools, operations:[read]},
              {allow:groups, groups:["Admins"], provider: userPools, operations:[read]},
              {allow: owner, ownerField: "userId", operations: [update,read]}
             ])
}

I have a list of users and institutes. Each users can belong to 1 or more institutes via Employee model.
Now the question is:
I limited creation of employees to Admins. How can I allow admins of a specific institutes to modify all employees of that institutes?
On the other hand, all users (including admins) have at least an entry in my employee table.
While the users have different groups: Admins and Employees.
Now a user belong to an institute + he or she is an Admin -> then I want he or she could only manipulate those employees entries that are belong to the same institute.
Or how can I say, allow admins to manipulate a row in the employees table, if and only if, they have an entry in the employees table with the same InstituteId

@mjza you have multiple options here, depending on number of institutes you can have 1 group per institute and validate it with dynamic group but you can run into group count limitations if you have a lot of them. A more scalable way would be to have a Cognito pre-token generation trigger that adds the list of institutes to a custom claim that will be present in the JWT token issued by Cognito. Then in a custom resolver you can check if the passed in instituteId in the request can be found in the the custom claim.
Another solution could be to use a pipeline resolver and instead of having a claim in one of the steps in the pipeline resolver do the same check against DynamoDB there and then allow the execution to the next step in the pipeline resolver or fail with authorization error.
I'm closing the issue as answered your question, if something new comes up feel free to comment or open a new issue.

@mjza you have multiple options here, depending on number of institutes you can have 1 group per institute and validate it with dynamic group but you can run into group count limitations if you have a lot of them. A more scalable way would be to have a Cognito pre-token generation trigger that adds the list of institutes to a custom claim that will be present in the JWT token issued by Cognito. Then in a custom resolver you can check if the passed in instituteId in the request can be found in the the custom claim.
Another solution could be to use a pipeline resolver and instead of having a claim in one of the steps in the pipeline resolver do the same check against DynamoDB there and then allow the execution to the next step in the pipeline resolver or fail with authorization error.
I'm closing the issue as answered your question, if something new comes up feel free to comment or open a new issue.

@attilah Could you provide any documentation or link to a tutorial for the last 2 suggestions?


feat(vouching): vouching timeout for challenged vouchees

As discussed on the PoH governance forum, vouchers that had a vouchee challenged and removed in the last 60 days shouldn't be able to vouch. This will help mitigate vouch and challenge attacks.
For any enquiry on this modifications I can be reached on Telegram (https://t.me/nicobilinkis)

I don't think this is a good idea. I imagine that users who vouch-challenge know the system well and might have the technical skills to either interact with the smart contract directly or run the previous version of the UI locally. Vouching for someone can't be really prevented from the UI.
Best case scenario, I think this change would stop some vouch-challengers while the most experienced ones will continue to do it with less competitors around.

I disagree, I believe the vouch challengers are not so skilled or ate least good portion of them would not go through the hassle of working around the issue to vouch challenge.
Even if it there is just one unskilled vouch-challenger it is still worth it to raise the costs of the attack

I don't have a strong opinion about this.
For the record, for the vouch-challenge issue I think time would be better spent on solutions that tackle the root of the problem, such as:

Improving the onboarding. For example, I believe that adding a preview page with a bunch of checkboxes will help users a lot.
Add a fine to users who vouched for challenged profiles.

As I mentioned in Telegram:
There is zero risk in adding this and some marginal benefit. It could prevent lousy (not necessarily bad) actors.
I still do not understand who and how decides if these PRs are implemented or not.

Add a fine to users who vouched for challenged profiles.
@fnanni-0
How could this be implemented without a change on the contract?

Implementing it on the UI would be just a FE solution and still allow tech-saavy users to vouchallenge

I do think it's a bad idea since there are non-negligible chances that at least one vouch-challenger knows how to do it without the frontend (or running locally an older version of it). By removing their competition we are increasing the overall number of deposits that the tech-savvy vouch-challenger will get, thus increasing the incentive to be looking for these opportunities. It would be more effective to improve the onboarding process.
An alternative would be to publicly flag these profiles, with a warning sign and red text, so people can know who is vouch-challenging and there is some kind of public shame in doing so because you will be marking your identity.

Add a fine to users who vouched for challenged profiles.

I was thinking of a change to the policy. I'm not sure if I'm a fan of it, but it's possible. Here's an example of how it could work.

If you vouched for an Invalid Submission, pay the SUBMITTER an amount equal to the juror fees to be able to reapply your submission in the future. This way the voucher shares part of losses with the submitter, the vouch-challenge behavior will unlikely be profitable and honest vouchers are incentivized to be more careful.
If you vouched for a Deceased, Fake or Duplicate, pay the CHALLENGER an amount equal to the juror fees to be able to reapply your submission in the future. This way challengers have an extra incentive to find this hard type of frauds and the voucher gets extra penalized.

A simple contract similar to this one https://github.com/Proof-Of-Humanity/Proof-Of-Humanity/blob/master/contracts/Finer.sol could be used to track fine payments. Rejecting reapplications of users who haven't paid their fines would be enforced by specifying this in the policy.
Note that these vouchers can choose to ignore or delay the fine, but at the expense of being ostracized from POH after the submission has expired. This also increases the risk of selling/delegating your identity, as it could be used for vouch-challenging or for malicious vouching.

There is an unanswered question in this thread: who and how decides what PR
is to be approved?
On Wed., Dec. 15, 2021, 6:46 a.m. fnanni-0, @.***>
wrote:

Add a fine to users who vouched for challenged profiles.

I was thinking of a change to the policy. I'm not sure if I'm a fan of it,
but it's possible. Here's an example of how it could work.

If you vouched for an Invalid Submission, pay the SUBMITTER an
amount equal to the juror fees to be able to reapply your submission in the
future. This way the voucher shares part of losses with the submitter, the
vouch-challenge behavior will unlikely be profitable and honest vouchers
are incentivized to be more careful.
If you vouched for a Deceased, Fake or Duplicate, pay the CHALLENGER
an amount equal to the juror fees to be able to reapply your submission in
the future. This way challengers have an extra incentive to find this hard
type of frauds and the voucher gets extra penalized.

A simple contract similar to this one
https://github.com/Proof-Of-Humanity/Proof-Of-Humanity/blob/master/contracts/Finer.sol
could be used to track fine payments. Rejecting reapplications of users who
haven't paid their fines would be enforced by specifying this in the policy.
Note that these vouchers can choose to ignore or delay the fine, but at
the expense of being ostracized from POH after the submission has expired.
This also increases the risk of selling/delegating your identity, as it
could be used for vouch-challenging or for malicious vouching.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Proof-Of-Humanity/proof-of-humanity-web/pull/483#issuecomment-994758391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTKS62RAIBWDFTZJC5IOWDURCEZ7ANCNFSM5J2EE7LA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

An alternative would be to publicly flag these profiles, with a warning sign and red text, so people can know who is vouch-challenging and there is some kind of public shame in doing so because you will be marking your identity.
This was already included on this PR, but I decided to separate that feature onto another branch given that implementing that shouldn't require much discussion. The discussion on the timeout feature can continue here, and the red avatars can be found on a new Pull request #488.

There is an unanswered question in this thread: who and how decides what PR is to be approved?

Anyone in the Proof-Of-Humanity repo team can merge a PR. Usually we wait for at least someone in the team to review before merging. If there are any objections regarding a feature, we discuss them with the rest team.

Is the list of members of the repo team known? 2. How do they decide?

*Luis Andrés Gonzalez Gracia  *
luis.andres.gonzalez |  @gonzaluisandres
http://www.twitter.com/gonzaluisandres |  luisandresgonzalez
http://www.linkedin.com/in/luisandresgonzalez |

http://www.linkedin.com/in/luisandresgonzalez+1<PHONE_NUMBER>
https://api.whatsapp.com/send?phone=16399949701*

On Wed, Dec 15, 2021 at 11:04 AM fnanni-0 @.***> wrote:

There is an unanswered question in this thread: who and how decides what
PR is to be approved?
Anyone in the Proof-Of-Humanity repo team can merge a PR. Usually we wait
for at least someone in the team to review before merging. If there are any
objections regarding a feature, we discuss them with the rest team.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Proof-Of-Humanity/proof-of-humanity-web/pull/483#issuecomment-994986738,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTKS67V4SWB5CMCCPNKNA3URDDCRANCNFSM5J2EE7LA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Is the list of members of the repo team known?

@luiandresgonzalez Not publicly. This information is only available for those who have been added as members of the GitHub organization.
However, we could make the member list public by adding them to a CODEOWNERS file: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners

This issue is not possible at the frontend level.
It doesn't make sense to make this frontend worse for users on purpose (as someone can just fork it and remove this limitation), so I'll close the issue.


V1.2 misses 7th column on weapons scan in 1440p

In 2560x1440, it appears that the 7th column get missed completely. Artifact scanning appears to work fine.
It eventually goes into an infinite loop at the end because it's unable to count enough weapons.
It worked fine in 1080p but figured it was worth mentioning as people with 4K displays may not pick a windowed resolution as low as 1080p. Both 4K and 1440p are 16:9 resolutions so it should be working.

This is a known issue. I hope to get something out for 4k resolutions soon, or at the very least more helpful debugging outputs, but I do not have a 4k monitor myself to verify with.

Hi, it's not just 4k Resolutions. My Display does not run at 4k and my game was running at 1980x1020. (It didn't want what I normally use as 1600x1024.) For me, it would miss the 6th row.
(Unrelated it skipped some mats and when it got to traveler it stopped tracking characters)

Please check out the new v1.2.2 release. It improves on finding inventory items as well as an update for the game's new V2.5 inventory UI update.


Let the FAB compute the notch even when it doesn't overlap the BAB.

The Floating Action Button adds some margin around it for the notch.
So we need to compute a notch even when the FAB and the Bottom App Bar
doe not overlap.

This depends on #14851 which makes the FAB's notch computer work when the FAB and BAB does not overlap.
Tests are expected to fail until #14851 is merged.

LGTM


Migrate on this page component docs

Migrate on this page component docs. Split the component arguments out into two tables to better reflect the fact some are top-level the rest are for a slot.

Split the component arguments out into two tables to better reflect the fact some are top-level the rest are for a slot.

I quite like this approach. Reckon we should go back and do this for the other components that use slots. I'm happy to do it


Revert "Replace nanliu-staging by voxpupuli-staging"

As discussed in https://github.com/puppetlabs/puppetlabs-rabbitmq/pull/442
voxpupuli-staging is currently not published/hosted on Puppet Forge which is causing issues.
We should revert back to using nanliu-staging until voxpupuli-staging gets published

It looks as though the naming is puppet-staging, not voxpopuli-staging - if we update the metadata.json to show the correct name, this revert is possible not necessary.

nope, that's not it either .. my bad .. ignore that :)


Implements support for pick and omit

I've implemented support for pick and omit in NSDictionary.
pick returns a new dictionary that includes only the whitelisted keys, and omit is it's counterpart, returning a new dictionary that excludes the blacklisted keys.

@inf0rmer  Thanks!


[PL-43116]: Doc updates for audit trail and audit stream done as part of doc review

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

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

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

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

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

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


Prepare the 0.3.4 release

This is failing due to https://github.com/ocaml/ocaml-ci-scripts/pull/96:
Your request can't be satisfied:
  - Package base-threads is part of the base for this compiler and can't be changed

It should be fixed by the next release of ocaml-ci-scripts (https://github.com/ocaml/ocaml-ci-scripts/issues/123)


Because your site has the Cross-Origin Embedder Policy enabled, each embedded iframe must also specify this policy

Next.js 13 app deployed using Amplify: https://main.d2hzcewjwit905.amplifyapp.com
When trying to convert a file, a request is made to https://main.d2hzcewjwit905.amplifyapp.com/_next/static/chunks/795.ceab62a43c56068c.js which gives the following error:

I'm not sure how to resolve this. I have set Cross-Origin-Embedder-Policy to require-corp for all paths in next.config.js  (unless source: "/(.*)" doesn't match all paths? I tried source: "/" too and got the same issue).
async headers() {
    return [
      {
        source: "/(.*)",
        headers: [
          {
            key: "Cross-Origin-Embedder-Policy",
            value: "require-corp",
          },
          {
            key: "Cross-Origin-Opener-Policy",
            value: "same-origin",
          },
          {
            key: "Access-Control-Allow-Origin",
            value: "*",
          },
        ],
      },
    ];
  },

The source code for my project which includes my next.config.js file can be found here: https://github.com/CrypticSignal/av-converter-amplify
Reproduction steps

Go to https://main.d2hzcewjwit905.amplifyapp.com
Open devtools and go to the network tab
Try to convert a file
After a few seconds, you should see a request being made to https://main.d2hzcewjwit905.amplifyapp.com/_next/static/chunks/795.ceab62a43c56068c.js and the error in the screenshot above.

OS: Windows 11
Browser: Chrome v117.0.5938.149 (Official Build) (64-bit)

I was having the issue described but when I changed next config to path you used "/(.*)" the error disappeared!
I also had a look at how you are loading the ffmpeg files and looks like we are doing the same!
The issue I have now is the multithread version is slower than the single thread version lol

I never ended up using this as it crashes on mobile devices for my use case.
I also noticed the multithread url has an extra -mt i didn't notice, but not much improvement seen!
Let me know if you want the source code i had working!


Remove Scan Page (Out of Scope) and Address Dependencies

💡 Summary
The current state of the project includes the “Web Page Scan” feature, which is considered out of scope for our project goals. Additionally, we need to ensure that all dependencies, especially related to routing, are properly addressed.
Motivation and context
Eliminates confusion to the process to pull vulnerability data for organizations.
Implementation notes

Remove Web Page Scan
Update header
Update routing
Remove any other code dependencies

Acceptance criteria
How do we know when this work is done?

[ ] Add simple screen shot to ticket for proof
[ ] Ensure that all necessary dependencies are addressed
[ ] Specifically focus on any routing-related dependencies.
[ ] Update Playwright Tests

Dependencies:

CVE data is fetched locally through a scan (cveSync). Need a way to automatically load this data during the docker build if this data cannot be loaded afterwards.
Any other scans that were manual or scheduled through the UI must now be made in the backend.

@schmelz21

@aloftus23, @rapidray12 - scheduling a breakout session to discuss.


gh test action: authenticate against GHCR

GHCR must have some abuse system in place that blocks lots of connections done in an anonymous way. This seems to be causing our GH action testing policy-server to fail.
This commit ensures we perform authenticate operations against GHCR. The GITHUB_TOKEN is used to authenticate against ghcr.io
The testing done on my fork seems to prove this change helps. Unfortunately we will not know for sure until we merge this fix here.

Yes, all the tests are green :partying_face:
I'll push some esthetic fixes, let's hope everything will be green again :crossed_fingers:

Nope, this is now broken...

I think we should rewrite the test to use a mock, like @raulcabello and @viccuad just did with their sigstore PRs
Another approach, would be to run a docker registry next to the tested code, by using testcontainers. This is what oci-distribution uses to implement reliable e2e tests

Argh, what a find. Thanks. If we are going to do mocks, it would be nice to get them into testcontainers, so we can reuse them easier in other places. I suppose they would be accepted upstream, but I think testcontainers doesn't implement the concept of layers yet..

For unit tests I would mock all network calls we do in download_policies, these test would run very fast and are more reliable.
For integration or e2e tests we can use testcontainers

Let's go with the mock approach, as discussed during the daily :+1:

I don't see this issue happening anymore. Closing the PR, we can reopen it later on if needed


Multiple receipts scanning is stuck and does not show image preview

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:  @HaleyJacobson
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1718719341535539
Action Performed:

smart scan a receipt taking photo of the receipt
Submit the expense
Go to the report details and view the image

Expected Result:
Scanning receipt image is showing
Actual Result:
Black image is showing
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Unable to reproduce myself. Adding retest-weekly label

Unable to reproduce
https://github.com/Expensify/App/assets/43996225/e6f153e8-c7d0-4a7e-97db-cbe172b9b19b
https://github.com/Expensify/App/assets/43996225/580ed10d-b92e-468b-b836-a1e2752348e6


Enable RabbitMQ Queue mirroring

https://www.rabbitmq.com/ha.html
Make sure queues are mirrored and rabbitmq-ha queue master node failure, restart or sudden kills are tolerated.

This PR was really merged.


Unexpected behaviour onOpen onClose in v2.0.0

Hi,
I've just switched from 1.16.x to v2.0.0 alpha and I have an unexpected behaviour with onOpen and onClose props.
this is my drawer component :
         <Drawer
                ref={"drawer"}
                type="static"
                content={<Menu items={items} {...this.props} navigate={this.navigate}/>}
                openDrawerOffset={0.35}
                styles={{main: {shadowColor: "#000000", shadowOpacity: 0.3, shadowRadius: 15}}}
                onOpen={() => this.setState({drawerOpen: true})}
                onClose={() => this.setState({drawerOpen: false})}
                captureGestures={false}
                tweenDuration={100}
                negotiatePan={true}
                panThreshold={0.08}
                panOpenMask={0.35}
                disabled={this.state.drawerDisabled}
                tweenHandler={Drawer.tweenPresets.parallax}>
```

I need to set state with drawerOpen [true or false] when drawer open or close to open or close it programatically with this method :

```
toggle(){
        const fn = this.state.drawerOpen ? 'close' : 'open';
        this.refs.drawer[fn]();
    }
```
On previous version everything works well but now :
 When I open the drawer (programatically or with swipe gesture) the component re render and the drawer close automatically.

Maybe i missed something but i've tried all props and i can't make it works.

Thanks for this amazing module v2 is a really smooth

DavoCg

I figured out that new feature added by #110 fix my problem (even if it make animation a little bit less smooth)

@DavoCg there was a bug in the implementation around #110 that is now resolved :+1:


Implement Search Functionality in GitHub Copilot Chat Window in Visual Studio Code

The current design of the GitHub Copilot chat window in Visual Studio Code does not include a search feature for reviewing previous chat history. As a result, users have to manually scroll through the history to find past interactions, or repeat their questions, which can be inconvenient. Implementing a search functionality in the chat window will significantly enhance the user experience by allowing users to efficiently locate previous interactions.

https://github.com/microsoft/vscode-copilot-release/issues/124


FollowsUserLocation and showUserLocationButton not working (both platforms)

Is this a bug report?
YES

Have you read the Installation Instructions?
YES
Environment

Steps to Reproduce

I have this is my MapView configuration
          <MapView
            ref={ map => { this.map = map }}
            style={styles.map}
        	initialRegion={initialRegion}
        	showsUserLocation={true}
        	followsUserLocation={true}
            showsMyLocationButton={true}
             >
           </MapView>

Expected Behavior

The "my location" button is showing
The map follows user location

Actual Behavior

The location button is not showing on iOS, on Android it is showing when device is rotated to landscape. The button on Android doesn't seem to be working though.
The follow location is working on iOS, but no on Android

Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

+1   I'm going through the same problem
"react": "16.6.3",
"react-native": "0.58.5",
"react-native-maps": "^0.23.0",

+1 Same problem
"react": "16.6.3",
"react-native": "0.58.5",
"react-native-maps": "^0.23.0",

+1 Same problem
    "react": "16.8.3",
    "react-native": "0.59.3",
    "react-native-maps": "^0.25.0",

@CesarDenis @joaodematejr Any updates?

+1  same
  "react": "16.8.6",
  "react-native": "^0.61.1",
  "react-native-maps": "0.25.0",

+1 same
    "react": "16.8.3",
    "react-native": "0.59.3",
    "react-native-maps": "^0.25.0",

I looked into the source and it seems that followsUserLocation isn't supported in Google Maps, unless I'm missing something? I'll create an issue.

same problem
  "react": "16.8.3",     "react-native": "https://github.com/expo/react-native/archive/sdk-35.0.0.tar.gz",     "react-native-maps": "0.25.0",

followsUserLocation only works on iOS using Apple Maps, not Google's

I opened an issue a while ago regarding the documentation needing to be updated at https://github.com/react-native-community/react-native-maps/issues/3206

Closing, please check https://github.com/react-native-community/react-native-maps/pull/3371


GitHub Sync

This is an automatically generated pull request.

Merging on red to clear the backlog since we know about the one failure.


Refresh a PR after the PR is edited

I think there is a state that after a PR is edited and submitted a couple of times without refreshing the page eventually an error code 422 is sent back from the GitHub API. I'm guessing it's a state thing. If you go back to the dashboard and edit again it won't happen but worth the time to fix even if its a fairly unlikely workflow event.

I didn't encounter this issue once during my testing. So seems like this issue is being addressed during the multiple refactoring that taken place. I am closing this issue, but if you see this issue again please feel free to reopen the issue.


[melodic][noetic][dashing][eloquent][foxy][dunfell][gatesgarth[hardknott][master] rc-dynamics-api: unblacklist and fix

When working on rolling upgrade in https://github.com/ros/meta-ros/pull/806 it needed the same blacklist, but then I found out what was supposed to provide js_embed and now we can fix it for all distributions.
Rolling will have a fix in https://github.com/ros/meta-ros/pull/806 together with the distribution upgrade which introduces this recipe there.

Pushed to other branches as well:
afb4f94ca..6ceadfa0a  dunfell -> dunfell-next
3b644e44d..d3b4ec8da  gatesgarth -> gatesgarth
3b644e44d..d3b4ec8da  gatesgarth -> gatesgarth-next
4c8cc09c9..4753e7905  hardknott -> hardknott
4c8cc09c9..4753e7905  hardknott -> hardknott-next
4c8cc09c9..4753e7905  master -> master
4c8cc09c9..4753e7905  master -> master-next


Opened files forward arrow

Issue Type: Bug
Hi,
whenever opened multiple files .for ex:if i opened 10 files at a time, moving files navigation  arrow is not displaying to move the hidded files into display  right or left in the top.
VS Code version: Code 1.36.1 (2213894ea0415ee8c85c5eea0d0ff81ecc191529, 2019-07-08T22:58:59.274Z)
OS version: Windows_NT ia32 6.1.7601

/needsMoreInfo
Does this also happen with VS Code Insiders? Download: https://code.visualstudio.com/insiders/
Please attach a screenshot.
(Note that you are on an older version of VS Code, please update.)


ImageResizer blocks thread

PluginManager: THREAD WARNING: exec() call to ImageResizer.resize blocked the main thread for 150ms. Plugin should use CordovaInterface.getThreadPool().
Hi when we are going through multiple resizes, we get the message that the thread has been blocked.
Is this something you are looking into or planning to fix?

Oh at the moment I havn't seen it. You are welcome to create a pull request ao I can merge it.