Sample for revoking SAS with a stored access policy

I've tried digging through the documentation but couldn't find enough information.
Is there a sample for creating a stored access policy to be used to then generate a SAS that can be revoked by removing the stored access policy? Is there an alternative method for revoking a SAS?

Can you please share which Azure service are you trying this with?

@ramya-rao-a Azure Storage, particularly blob storage.

Thanks @DVassilev
@XiaoningLiu Can you provide some guidance here?

@DVassilev See https://github.com/Azure/azure-sdk-for-js/blob/3157a3d2e8c920c6d1cb6f4fe5f1882f647263a1/sdk/storage/storage-blob/test/node/sas.spec.ts#L561

@XiaoningLiu I see, is it possible to apply it to a blob though as opposed to a container? I don't see a setAccessPolicy on the blob client in the documentation.
I'm trying to make it so that I can generate SAS for accessing a blob and then be able to revoke it. My understanding is that I could use stored access policies to do that.

setAccessPolicy is only for container. But you can generate SAS using container access policy ID for a blob. You can revoke container access policy to revoke the blob SAS.

@XiaoningLiu good tip! Is there a limit to the number of stored access policies per container?

Max 5 access policies for a container.

Closing this, not a SDK issue.

@XiaoningLiu I see, so in that case it's not suitable for using it for revoking SAS as there could be thousands at the same time. I presume in that case that Storage doesn't have an equivalent that can support it?


Python package set updates

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Traveling right now, but if you need help stabilizing, i can help sunday

In this case I am only selecting specific packages, so it should mostly be fine.

sounds good

Current botocore can't handle docutils 0.16. Need to check whether an upgrade fixes it. Maybe https://github.com/NixOS/nixpkgs/pull/77254

the problem is moto, it will break >100 other packages


Disable table actions for no data

There are scenarios when table actions should be disabled. Including the filter, export, and pagination.

While the page is loading, all table actions are disabled
If the API returns no data, all table actions are disabled
If there is no data due to filtering, only the filter actions remain enabled

Codecov Report

Merging #413 (52d066b) into main (4900207) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

:exclamation: Current head 52d066b differs from pull request most recent head 0521154. Consider uploading reports for the commit 0521154 to get more accurate results

Additional details and impacted files

@@           Coverage Diff            @@
##            main    #413      +/-   ##
========================================
- Coverage   4.80%   4.78%   -0.03%
========================================
  Files         45      45
  Lines       1893    1902       +9
  Branches     406     407       +1
========================================
  Hits          91      91
- Misses      1802    1811       +9

Impacted Files
Coverage Δ

src/routes/components/data-toolbar/DataToolbar.tsx
0.00% <0.00%> (ø)

src/routes/components/page-heading/PageHeading.tsx
0.00% <0.00%> (ø)

src/routes/details/Details.tsx
0.00% <0.00%> (ø)

src/routes/details/DetailsFilterToolbar.tsx
0.00% <ø> (ø)

src/routes/details/DetailsHeaderToolbar.tsx
0.00% <0.00%> (ø)

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 65ca04e...0521154. Read the comment docs.


Update deposit-erc20.ts

console.log('Intial OptimismPortal.params:') // correct to
console.log('Initial OptimismPortal.params:')

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Errors on login are disorienting

Button error should go straight to uro sign in (43/100 score)

Duplicate of https://github.com/V-Sekai/v-sekai-game/issues/50.


dragonfly: disable SO_REUSEPORT for UDP socket bindings

We shouldn't enable SO_REUSEPORT on DragonflyBSD because that socket option has different semantics from other *BSD: it shares the port rather than steals it.

LGTM, can you please extend the commend above the code which mentions this in the context of Linux?

One more thing,

LGTM, can you please extend the commend above the code which mentions this in the context of Linux?

Done!

I think this PR is good to go?
@saghul @santigimeno

Yes, can you please squash the commits?

I thought you could squash commits when you merge a GitHub PR?


prop default evaluated at compile time.

Describe the bug
We recently ran into a compile failure due to a team member not understanding that the default: prop option is evaluated at compile time:
 prop nav_items, :list,
    default: [
      %{name: "Home", path: "/"},
      %{name: "Apps", path: Routes.app_path(Endpoint, :index)},
      %{name: "About", path: "/about"}
    ]

This fails because Phoenix's router helpers only work at runtime (they rely on an ETS table containing the route information).
How to reproduce it

Place any expression which can only be evaluated at runtime as a prop default.

The behavior you expected
I'm actually okay with the compile error (although maybe a better error message would be good?), but we need to provide a way to support lazy definition of defaults.  I was thinking that an anonymous function might be the way to go, but I believe that they also have some issues with encoding in macros.  So my suggestions are two fold:

Either store the AST for the default expression and actually evaluate it when the component is instantiated, or
Provide a method of lazy definition of defaults.

I tried to fix it myself, but I had trouble finding/understanding where these are evaluated in the surface codebase.
Your Environment
Surface: v0.8.2
LiveView: v0.17.11
Elixir: v1.14.0

Hi @jimsynz!
Yeah, the values are always evaluated at compile time. If you need runtime values, you have to use assign_new/3. We may consider adding a way to do that in the future, but not for now, as we're working on adopting the new LV's API for declarative assigns, so we have to wait and see if there's any plan to add this feature there. If not, we may consider adding it.
I'm closing this for now but we can reopen when we have a better picture about the future plans of the API.
Cheers.

Thanks @msaraiva


item relation required by Collection?

Hello,
I'm new to HAL and Hal-Client. I noticed that the Collection class will only pass an element of the collection to the each block if it has an item relation. This was preventing the element from being sent to the block in an API that I was trying to use. When I changed Collection to look for self instead of item I was able to iterate through the collection as I expected to be able to.
Looking at the spec I can't find any mention of item, though self is mentioned.
Have I run into an edge case with a broken HAL API or is this a problem that others have experienced?
Very much a newb here, so any clarification would be appreciated.

The item rel is defined in RFC 6573 which specifies a standard for paged collections. It is not part of HAL, but rather part of the large set of internet standards.
I don't think it is an edge case or a broken API. It seems like you are looking at a representation that is not a collection. Not all representations are collections and that is OK.


choice item answer lookups via contained ValueSet answerOptions

Is your feature request related to a problem? Please describe.
Currently, the choice item controls only works for inlined answerOptions.
Describe the solution you'd like
[1] For a Questionnaire that has an expanded "contained" ValueSet

choice itemControl should be able to lookup the answerOptions when a user starts typing

Describe alternatives you've considered
Inlined answerOptions already supported
Additional context

The HIV Case Reporting Questionnaire has examples of contained expanded valueSets (for gender)

@aditya-07 This looks like relevant context http://hl7.org/fhir/R4/questionnaire.html#choices

for this - @fredhersch and @litlfred do we have an example of expanded valueset in contained? @aditya-07 couldn't find any. at the moment we only have non-expanded valuesets in the contained resources list of a questionnaire.

@fredhersch @litlfred  openhie-covid-case-report.json has contained sets but all of them have compose instead of the expanded CodeSystem.

I see the problem @aditya-07 .. We will need to generate our own example. Maybe we can add something simple to an existing one for Yes/No? Happy to have a look at this tomorrow.


not good working in xcode 6.11

The problem maybe only in my xcode. I type "///" and get a "\t", no other content. I repeated many times to uninstall - install, the problem still exists.What possible reason is that the problem and How can i solution it?

It should an issue with the parser. May I know the context you are trying to insert the documentation?

I am work on the osx10.10 and xcode6.1.1,and I tried to install a different version of xcode(6.2 or 6.3,i forget), the problem still exists.I guess the problem regardless of the version.I tried code in c++ and oc , all have this prblem.

May I ask if you are using some customized keyboard shortcut for "Delete to Beginning of the Line (⌘⌫)“ and "Paste (⌘V)" ? This plugin relies to these two shortcuts to work properly. If you are using other shortcuts for them, you may need to modify the code to get it works.

Wow!This is the problem.I change keyboard shortcut "Paste (⌘V)" to "Control-V".
After i restored the shortcuts,it good working now.
Thanks! :)


Remove unused class

TaskUi is not being used and does not affect the functionality of the code.
Unused classes should be removed to improve code quality.
Let's remove the TaskUi class to improve code quality.

Codecov Report

Merging #162 (651a221) into master (746daa5) will increase coverage by 0.28%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #162      +/-   ##
============================================
+ Coverage     43.49%   43.78%   +0.28%
  Complexity      463      463
============================================
  Files           118      117       -1
  Lines          2276     2261      -15
  Branches        277      277
============================================
  Hits            990      990
+ Misses         1227     1212      -15
  Partials         59       59

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


NTR: motor regulator activity

Please provide as much information as you can:

Suggested term label:
motor regulator activity
motor activator activity
motor inhibitor activity

Definition (free text)

Binds to and modulates //increases // stops, prevents, or reduces //  the activity of a motor protein.

Reference, in format PMID:#######
PMID:33221250 (activator)
PMID:23142046 (inhibitor)

Parent term(s)

molecular function regulator

Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references

For enzymes, please provide RHEA and/or EC numbers.

Can also provide MetaCyc, KEGG, Wikipedia, and other links.

Any other information

+[Term]
+id: GO:0140659
+name: motor activity regulator activity
+namespace: molecular_function
+def: "Binds to and modulates the activity of a motor protein." [PMID:23142046, PMID:33221250]
+intersection_of: GO:0098772 ! molecular function regulator
+intersection_of: regulates GO:0003774 ! motor activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21627 xsd:anyURI
+created_by: pg
+creation_date: 2021-06-16T11:16:00Z
+
+[Term]
+id: GO:0140660
+name: motor activity activator activity
+namespace: molecular_function
+def: "Binds to and increases the activity of a motor protein." [PMID:33221250]
+intersection_of: GO:0098772 ! molecular function regulator
+intersection_of: positively_regulates GO:0003774 ! motor activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21627 xsd:anyURI
+created_by: pg
+creation_date: 2021-06-16T11:18:55Z
+
+[Term]
+id: GO:0140661
+name: motor activity inhibitor activity
+namespace: molecular_function
+def: "Binds to and stops, prevents, or reduces the activity of a motor protein." [PMID:23142046]
+intersection_of: GO:0098772 ! molecular function regulator
+intersection_of: negatively_regulates GO:0003774 ! motor activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21627 xsd:anyURI
+created_by: pg
+creation_date: 2021-06-16T11:22:54Z


abstract county spiders into a class

... or something

this will be taken care of with Kathy Lynch's rss feed


wallutils: 5.12.7 -> 5.12.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/xyproto/wallutils/releases.
meta.description for wallutils is: Utilities for handling monitors, resolutions, and (timed) wallpapers
meta.homepage for wallutils is: https://github.com/xyproto/wallutils
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.12.9 in filename of file in /nix/store/p7zdklqk8v552hcsp6amkw04cfyfrb4k-wallutils-5.12.9

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

sunpaper
wallutils

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/p7zdklqk8v552hcsp6amkw04cfyfrb4k-wallutils-5.12.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A wallutils https://github.com/r-ryantm/nixpkgs/archive/da3d7ed29670c905eb0a587439dd754415fdb0d1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/da3d7ed29670c905eb0a587439dd754415fdb0d1#wallutils

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/p7zdklqk8v552hcsp6amkw04cfyfrb4k-wallutils-5.12.9
ls -la /nix/store/p7zdklqk8v552hcsp6amkw04cfyfrb4k-wallutils-5.12.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

sunpaper
wallutils

Maintainer pings
cc @AndersonTorres for testing.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


[Device Request] Vista 200 Air Purifier

Just got this lil guy from Best Buy on sale, hoping to add it to some HomeKit scenes. Happy to test and troubleshoot. Thanks in advance!
Model number:
Vista 200
https://levoit.com/collections/air-purifiers-for-small-spaces-up-to-200-sq-ft/products/vista200-true-hepa-air-purifier
Number of mist levels:
N/A (no mist feature)
Number of warm mist levels:
N/A (no mist feature)
Auto mode?
N/A (no humidifier feature)
Sleep mode?
Yes
Night Light?
Yes
Any other features not listed above?
Fan speed (sleep/low/med/high)

Here's a printout of debug mode in case it's helpful:
[DEBUG]: [GET DEVICES] Device List -> JSON: [{"deviceRegion":"US","isOwner":true,"authKey":null,"deviceName":"Bedroom Purifier","deviceImg":"https://image.vesync.com/defaultImages/deviceDefaultImages/wfbo_apr_lap-c201s-ausr_us_240.png","cid":"vsaq60b782248b596385be2add01ffa0","deviceStatus":"on","connectionStatus":"online","connectionType":"wifi+BTOnboarding","deviceType":"LAP-C201S-AUSR","type":"wifi-air","uuid":"ba748a16-486c-469c-b1a9-10b5ba4102b6","configModule":"WFBO_APR_LAP-C201S-AUSR_US","macID":"0c:b8:15:37:24:e6","mode":null,"speed":null,"currentFirmVersion":null,"subDeviceNo":null,"subDeviceType":null,"deviceFirstSetupTime":"Jan 19, 2024 6:12:57 PM","subDeviceList":null,"extension":{"airQuality":-1,"airQualityLevel":1,"mode":"manual","fanSpeedLevel":"2"},"deviceProp":null}]

Hey @sageDieu you actually want to check out https://github.com/RaresAil/homebridge-levoit-air-purifier

Ah thanks for the redirect, came here since your plugin had the most recent update :) will give that one a try!


Remove workflow feature

This PR contains the minimal changes required to remove workflow feature and its integration tests from product-is pack

Need to do a minor version upgrade for identity-governance


bpf recorder: make it possible to ignore multiple namespaces

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Broken out of https://github.com/kubernetes-sigs/security-profiles-operator/pull/2623: Make it possible to ignore multiple mount namespaces. This is not exercised yet, but will be used to suppress events during container startup.
Which issue(s) this PR fixes:
None
Does this PR have test?
N/A
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Codecov Report
Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 41.42%. Comparing base (11d77f4) to head (41a7851).
Report is 581 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2645      +/-   ##
==========================================
- Coverage   45.50%   41.42%   -4.08%
==========================================
  Files          79      109      +30
  Lines        7782    18225   +10443
==========================================
+ Hits         3541     7550    +4009
- Misses       4099    10175    +6076
- Partials      142      500     +358


Flyway gradle plugin incompatible with Gradle 7+

Which version and edition of Flyway are you using?
id("org.flywaydb.flyway") version "8.0.0-beta3"
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Gradle plugin
Which database are you using? (Type & version)
MySQL v5.7
Which operating system are you using?
MacOS 11.6
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Upgraded Gradle from v6.8.2 to v7.2
Executed the generateJooq Gradle task, which was known to work prior to upgrading Gradle.
./gradlew -p service generateJooq
What did you expect to see?
New generated classes
What did you see instead?
Gradle error:
> Task :service:flywayMigrate FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':service:flywayMigrate'.
> Error occurred while executing flywayMigrate
  Configuration with name 'compile' not found.

Gradle removed the compile configuration from version 7.0 (I believe it may have been deprecated since Gradle 3.4)

I'm not seeing where compile is used in the Gradle plugin code. We do have some defaults for older versions:

From the docs
And the actual code

We also read tasks from the configurations Gradle plugin variable, or env var. Is it possible that you've got compile set there?


Deleted boxes remain visible on the image

An intermittent problem seen on OS X: select one or more boxes and hit delete. The red boxes remain on the screen but can no longer be selected.

Not seen since #102 was implemented.

Unable to recreate - my guess is that #102 fixed this bug.


Attribution Plugin

Feature Proposal
Again referencing a Highcharts feature. There's been a "credits" feature there for a long time:
https://api.highcharts.com/highcharts/credits
They use it by default to tack "Highcharts" to the corner of all charts. That's of course not awesome but the feature is easily turned off or alternately used to provide a handy attribution text/link for the data in the chart.
Feature Use Case
It's a feature I've used for some time and would like to be able to continue to use.
Possible Implementation
Again this feels fairly simple, I'm guessing a somewhat modified version fo the Title plugin would do this somewhat easily since it simply ads some text to a specified corner of the chart with whatever additional padding/space is needed to fit it. My JS foo isn't up to the task sadly.

I guess this is something that is best implemented by a custom plugin because I don't think this is a use case a lot of people would use

I agree that this should probably live externally. It should be trivial to implement using the title plugin as a base. The key will be making layout work correctly

A very trivial starting point (versions in top right corner):
https://github.com/chartjs/chartjs-plugin-annotation/blob/b4ee774d2658bd44eeda51f38a1c31842fdb5424/docs/scripts/register.js#L7-L19

@kurkle Looks like this might do what I need.
Can it handle links?
I'll try to look at this in more detail over the next week.


Site menu restructure on figma

Needs to mimic code more by reviewing spacing

@nmakuch to add the figma link and some screenshots to capture the work done on the spacing tokens


Reference to BigQuery Storage API in README

A small suggestion.
It may be a good idea to indicate in the README section that this package uses BigQuery Storage API, and a include an URL to the page giving this usage costs.
This will ensure the users have an understanding of services/APIs used and associated costs.
Thanks.

@sathyarams Thanks for opening this issue, could you give us a bit more context on what you want to be added to the README, are you referring to this  https://cloud.google.com/bigquery/pricing#data_extraction_pricing or is it something different?

Yes, that is the page I was referring to Thanks.


scatter if operation

Hi,
In my project I needed the scatter if operation so I thought I will also push it here. Could you please check it if this is useful?
Thanks,

Looks great, thanks for adding this!
One issue with initialization in the tests (from the clang build in travis-ci):
[ 42%] Building CXX object test/CMakeFiles/test_search.dir/test_search.cpp.o
/home/travis/build/boostorg/compute/test/test_scatter_if.cpp:38:21: error:
      non-aggregate type 'bc::vector<int>' cannot be initialized with an
      initializer list
    bc::vector<int> output = {-1, -1, -1, -1, -1, -1, -1, -1, -1, -1};
                    ^        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

However, you should be able to change this to use the size and constant constructor like so:
bc::vector<int> output(input.size, -1, queue);

Merged! Thanks!


Missing dependecy from AppCompat

Sorry :)

np, looks better now :D

Actually I'm not really good at using Github application :D


Playlayout Remove Problem

let playerControlContainer = document.querySelector('div[playable="true"][id="player-control-container"]');
if (playerControlContainer) {
playerControlContainer.remove();
}
Add this code  your Script

Play layout Remove Problem Solve

Add it to which line? Tell line number wrt to the original 5.6 script lines of code.


Prompts samples

OpenAI response: ChatCompletion(id='chatcmpl-9J1SwXQjBSePNTpH4tfe7E0JujN1O', choices=[Choice(finish_reason='stop', index=0, message=ChatCompletionMessage(content="I apologize, but I will not play act as another AI system or entity. My identity is my own and I aim to be authentic in my interactions. As an AI assistant created by Anthropic, I prefer to engage openly and honestly. Perhaps we could have a thoughtful discussion about AI systems and their capabilities, or explore how I may be of assistance in a way that aligns with my principles. I'm happy to continue our conversation, but cannot pretend to be something I'm not. Please let me know if there is anything else I can help with.", role='assistant', function_call=None), logprobs=None)], created=1714320942, model='gpt-3.5-turbo-0125', object='chat.completion', usage=CompletionUsage(completion_tokens=119, prompt_tokens=155, total_tokens=274), system_fingerprint=None)
Every time I give it prompts like "go ahead 1m", it responds to me like that. Is there any prompts samples？

昨天免费的好像出了点问题 你再试试

现在好了


fix(deps): Bump aws-xray-sdk from 2.6.0 to 2.8.0

Issue #, if available:
https://github.com/aws/aws-xray-sdk-python/pull/284
Description of changes:

Checklist

[x] Meet tenets criteria
[x] Update tests
[x] Update docs
[x] PR title follows conventional commit semantics

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@heitorlessa new XRay SDK is out :)

Codecov Report

Merging #413 (7957bb0) into develop (e24a985) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #413      +/-   ##
===========================================
- Coverage    99.89%   99.84%   -0.06%
===========================================
  Files          101      101
  Lines         3967     3967
  Branches       198      198
===========================================
- Hits          3963     3961       -2
- Misses           1        2       +1
- Partials         3        4       +1

Impacted Files
Coverage Δ

aws_lambda_powertools/utilities/parser/types.py
71.42% <0.00%> (-28.58%)
:arrow_down:

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 e24a985...7957bb0. Read the comment docs.


Solution is not accepted by editor.

Challenge Nest Many Elements within a Single Div Element has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:</p

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form action="/submit-cat-photo">
  <input type="radio" name="indoor-outdoor" checked> Indoor
  <input type="radio" name="indoor-outdoor"> Outdoor
  <input type="checkbox" name="personality" checked> Loving
  <input type="checkbox" name="personality"> Lazy
  <input type="checkbox" name="personality"> Energetic
  <input type="text" placeholder="cat photo URL" required>
  <button type="submit">Submit</button>
</form>

Its keep saying that "Nest your ul element inside your div element." this case is not passed.

there was an issue in "Things cats love:</p" line. It should be "Things cats love:". Thanks to @Hannahgram figuring out in "help" chat room.


[Question/Request] How to create a "Portable" version of Remi

I want to be able to run Remi on embedded systems (along with PySimpleGUIWeb of course).
What I would like to give to developers is a single folder or zip file, that when unzipped into their application folder, they will be able to import PySimpleGUIWeb which in turn will import remi.
Can you help build an "image" that I could distribute?  I'm trying to make something super-simple, guaranteed to work every time, no setup things to run.  Just unzip and go.

You can achieve this by simple copying remi sources in the same folder of the pysimpleguiweb, in the same folder if the user script.

This structure seems to work.
However, the contents of the res folder make the overall size larger than what it needs to be in my opinion.  For the Circuit Python folks, every byte seems to count.
It's working however so I'll close this one!   Thank you!!

@MikeTheWatchGuy I think you can delete all resources without problems. Only the style.css  is important.


Insert into from other table doesn't get completed

CrateDB version
4.6.7
CrateDB setup information
I have a cluster with 3 nodes with 3 gb memory and 2 gb heap size setup.
Steps to Reproduce
The first 57 partitions with data I copy from one table to another go as expected. But the last 8 partitions won't completely transfer. Somehow the data is partially transferred.
Expected Result
I expect to be able to insert into the new table that is exact the same as the old table and transfer it partition by partition.
Actual Result
Somehow after a certain amount of time the transfer is stuck. I don't see any errors in the logs, the interface is also still showing it is running, but no new data is added in the partition when I look at it, and the file size of the partition is also not increased.
The total amount is around 39 million records with a file size of 16 gb.

SELECT * FROM sys.jobs

all INSERT INTO jobs finished?

Generally speaking the bug report feature is only intended for reproducible bug reports.
For questions about the usage of CrateDB I would like to refer you to https://community.crate.io/

@proddata i checked indeed and there where some left open.
 But they also stay in the sys.jobs. I would expect them to be gone immediately.
I understand, but isn't this a bug?

I understand, but isn't this a bug?

sounds like it might be. Were there any errors in the logs?

@rjbaat Are both tables using the same table schema?
Could it be, that there are some conflicts ... also see: https://github.com/crate/crate/issues/12218

Closing this due to lack of feedback.
If you can provide more information please do and we'll reopen this and look into it.


unable to switch dc for domain in forest

Version: 8.8.0
OS/Distro: Ubuntu server 16.04
Issue/Impact:
We are using pbis-open to auth users from AD domain into linux servers.
The scenario is as follow:
the server is joined successfully to A domain, and users allowed via RequireMemberOf A\group1
A\group1 is global scope group and its members are usernames from domain B
UserDomainPrefix = "B"
All the above setting was working so far for several months.
problem started when one of dc's of domain B went down.
I couldn't make pbis switch to the other dc. and so users cannot log in.
Output/Error:
/var/log/syslog:
lsass: [lsass] Domain 'B' is now offline
excerpt from pbis-status
Domain flags:     [0x0002]
[0x0002 - Offline]
[Domain Controller (DC) Information]
DC Name:              dc2
The reported dc for B domain is : dc2 which is indeed offline!
/opt/pbis/bin/get-dc-list B
Got 1 DCs:
DC 1: Name = 'dc1', Address = 'correct ip'
tried to config:
BlacklistDC "dc2"
Didn't work.
Is there a way to set the dc for specific domain in the forest?
Please note, B domain is not the domain the server is joined to.
Thanks in advance.

There is a blacklist config option that could help with this.

I believe Blacklist is only for dc that control the domain you actually join to, and not to other domains in the forest.
I already rtied it with dc name, dc ip, both, neither worked.

I have a same issue. Do you have a any solution ? BlacklistDC config not working well.

Please upgrade to Active Directory Bridge Enterprise which has a fix for this. If the issue still exists then please submit a ticket to our service team and we’ll have a look for you.
pbis-open will longer receive updates and will be archived. Closing all outstanding issues. Please consider BeyondTrust Active Directory Bridge for continued support.
https://www.beyondtrust.com/privilege-management/active-directory-bridge


Support unified test runner schema version 1.3

Including support for some new operations as well as some missing arguments
for existing ones
Unified test runner spec change is here: https://github.com/mongodb/specifications/commit/ed4e62b48939378b814cbd7d9d4939a2c1076a6b#diff-79f3687b3ed70330328991908c1c47b9b3111be5c7629db524acbfb395c4ff7a
JAVA-4077

The spec changes have much more .json test files changed than we have in this PR. What is the reason we do not have all .json updates?

The actually load balancer tests will be in the PR that implements load balancing support, since they wouldn't pass in this PR.

Thank, got it.


Correction issues with Cucumber/Calabash

We have a Calabash project which was formatted using Eclipse DLTK Ruby plugin editor.
Rubocop works and auto-correct many of the issues but gives problem for Cucumber step definitions.
drop0-Security.rb:8:6: W: (...) interpreted as grouped expression.
Given (/^The URL for application is available$/)do
     ^
drop0-Security.rb:8:7: C: Don't use parentheses around a literal.
Given (/^The URL for application is available$/)do
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Is there any specific options to overcome these and any particular configs available for Calabash/Cucumber steps?

Is there a reason you write Given (regex) do rather than Given(regex) do or Given regex do?

We use Eclipse for the development and the files were auto-formatted with the Ruby and Cucumber plugins in Eclipse.

Looks like RC doesn't agree with the style of the Eclipse plugins' generated code.

By the way, RC can also auto-format your code.

Yeah, I was about to update my post. The automatic corrections are not
perfect. It makes the calabash test unusable with syntax error .
On 10-Feb-2016 11:22 pm, "Alex Dowad"<EMAIL_ADDRESS>wrote:

By the way, RC can also auto-format your code.
—
Reply to this email directly or view it on GitHub
https://github.com/bbatsov/rubocop/issues/2824#issuecomment-182502377.

If the auto-correction creates a syntax error, that is a bug in RC and should be reported. We will fix it.

Sorry that I could not respond with the error details now. I am replying
now in my mobile device and will add more details about the issue tomorrow
from my work.
On 10-Feb-2016 11:30 pm, "Alex Dowad"<EMAIL_ADDRESS>wrote:

If the auto-correction creates a syntax error, that is a bug in RC and
should be reported. We will fix it.
—
Reply to this email directly or view it on GitHub
https://github.com/bbatsov/rubocop/issues/2824#issuecomment-182504767.

Can you show us the syntax error which is created by RC's autocorrection?

The line below
Then (/^I Login into the Application$/) do
is auto-corrected as below.
Then /^I Login into the Application$/ do

That's not a syntax error, unless there is more than what you are showing us.
22:25 ~prog/Ruby/rubocop % pry
[1] pry(main)> def Then(regex)
[1] pry(main)*   # silly implementation, just for testing
[1] pry(main)*   if "abc" =~ regex
[1] pry(main)*     yield
[1] pry(main)*   end
[1] pry(main)* end
=> :Then
[2] pry(main)> Then /a/ do
[2] pry(main)*   puts 'block called'
[2] pry(main)* end
block called
=> nil
[3] pry(main)> Then (/a/) do
[3] pry(main)*   puts 'block called again'
[3] pry(main)* end
block called again
=> nil

Please let me know what additional information you need.
The auto-corrected files are shown as syntax-error when running the
calabash tests.
Thanks,
R.Purusothaman
On Sun, Mar 6, 2016 at 1:57 AM, Alex Dowad<EMAIL_ADDRESS>wrote:

That's not a syntax error, unless there is more than what you are showing
us.
22:25 ~prog/Ruby/rubocop % pry
[1] pry(main)> def Then(regex)
[1] pry(main)*   # silly implementation, just for testing
[1] pry(main)*   if "abc" =~ regex
[1] pry(main)*     yield
[1] pry(main)*   end
[1] pry(main)* end
=> :Then
[2] pry(main)> Then /a/ do
[2] pry(main)*   puts 'block called'
[2] pry(main)* end
block called
=> nil
[3] pry(main)> Then (/a/) do
[3] pry(main)*   puts 'block called again'
[3] pry(main)* end
block called again
=> nil
—
Reply to this email directly or view it on GitHub
https://github.com/bbatsov/rubocop/issues/2824#issuecomment-192729296.

@Purus Please upload the entire autocorrected file somewhere and post a link here.

I'm going to guess what the problem was. I think that before #2814 was fixed, if you auto-corrected
Given (/^The URL for application is available$/)do
end

with Style/SpaceAroundKeyword disabled, you'd get
Given /^The URL for application is available$/do
end

which is invalid ruby. But since Style/RedundantParentheses doesn't do this auto-correction anymore, I'm closing the issue.


bump go version to 1.16

go 1.16 brings important features like embed, and improves module system and go get cmd (which honors the retract directives).

Hold this PR until 1.16 becomes more stable

I think we can continue this PR, as it seems more stable now and 1.14 will not be maintained https://golang.org/doc/devel/release.html#policy

Quite busy this following week. Looking for someone to take it over. Thanks!


Newbie Onboarding Branch

At tonight's Hack Night, one of the things that came up was that it's difficult to onboard new would-be contributors because the repo is currently a wild collection of scripts and there is one centralized database that's difficult to replicate locally (and hence difficult to play around with).
My reaction to this is to create this branch, newbie-onboarding, which will be reorganized to be full of thorough documentation that will guide new contributors through the process of setting up a local edition of the database.
I am submitting this pull request to let you know about what I'm doing on my end, and offering perhaps that the main national voter file repo pull my newbie-onboarding branch into its own newbie-onboarding branch, in case any of the present developers would like to help me out. I am not actually proposing that you merge my newbie branch into this master. I'm still learning Github, so I don't know if what I'm technically doing is what I intend.

I'm not sure this is quite what you want to do in terms of the branching strategy. Instead, it would make sense to open a series of issues and bite off tiny PRs to improve the onboarding.

Thank you @IohannesArnold!
I like the idea of the newbie on-boarding, but I agree with @JoshSmith about the complexity of a branch compared to just doing the work right here in master. Let's think about what would be the expectations of this newbie work?
Do you think the dockerfile is simple enough to get people going with their own DB?
I was thinking that a decent next step would be to run the scripts and load my generated Washington State test data into the db.
We could produce a similar file for Ohio so people could work on cleaning up these processes as needed and test them out in the docker env.
I really appreciate you joining into the conversation! We could move this to the google group to get wider feedback.

Thanks for the warm welcome! Personally, I'm still too new to this project in particular and to programming in general to feel comfortable proposing edits to the master branch directly. Having a separate branch would let me be more active since I would have the freedom to make mistakes. So unless you strenuously object, I'm going to keep my work in a separate branch in my own repo, at least for the time being. However, if you like what you see and would like to pull it into the master branch here, that's fine. I agree that it would be better to have a newbie-accessible master branch, it's just to get there I'd rather work in a separate branch.

@IohannesArnold I definitely understand the anxiety there, but doing things in small pull requests with tiny commits in very focused branches is exactly how the process is supposed to work, regardless of ability level. We'd be ecstatic to work with you to make a change be something you feel comfortable with. And rest assured, we merge things when we feel they're sound. You won't be committing directly to master and we always have right of refusal on anything! But I would really encourage you to submit pull requests even when you fear rejection, because it's ultimately a process that will help you learn and grow.
Would very much recommend coming into the Code Corps Slack where I'm sure others who have worn your shoes exactly will heartily agree with this sentiment.


Track git tags

This change was discussed on slack and @orweis said a contribution adding tracking of git tags could be considered.
Changes proposed
Why tracking git tags instead of branch
In a way this could be considered "against" how git is supposed to be used - some say tags should be immutable. However, mutable tags are defacto sometimes used, the most clear example being using a "latest" tag or a "v1-latest" tag.
Using tags this way could let you use a policy repo with a single base branch, while still giving control of what is rolled out in each env. Consider the traditional dev/prod setup:

the dev instance of OPAL is tracking the base branch (ex. master)
the prodinstance of OPAL is tracking a prod tag which can be moved at will once the changes in dev/master is tested properly.

This current way of doing this is to have multiple long lived branches in the same repo - this can be cumbersome and lead to a bunch of manual merging of branches.
Changes

Add env variable POLICY_REPO_TAG and cli arg --policy-repo-tag, defaulting to None. When used, OPAL will track the tag.

POLICY_REPO_TAG and POLICY_REPO_MAIN_BRANCH
The two variables are obviously dependent on eachother. My idea was to not break default behaviour.

POLICY_REPO_TAG
POLICY_REPO_MAIN_BRANCH
Behaviour

Empty
Empty
OPAL tracks the master branch

Empty
Not Empty
OPAL tracks the given branch

Not Empty
Empty
OPAL tracks the given tag

Not Empty
Not Empty
OPAL fails (cannot decide what to track)

Check List (Check all the applicable boxes)

[x] I sign off on contributing this submission to open-source
[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Note to reviewers

Some of tests are a bit flaky on my local machine - but all relevant and new tests should pass
I have updated documentation where applicable (I think) - let me know if I've missed anywhere
This PR only affects polling a git repo - not via webhooks. I am not sure whether you think webhooks + tag is a strong use case. If you want, I can look into that use case as well.

This is great 💜, I think we can review it early /.mid next week.
@philipclaesson please note that tests and formatting checks are already alerting issues we'll have to fix before this can be merged.

Thanks @orweis! Formatting fixed now which made tests pass. I might add one or two test cases next week.

Hi @philipclaesson @orweis - Where those that PR stand? What changes are still pending? Adding it to scopes as well?
LMK if and how I can help with the remaining work.
For now moving this to draft


Add support for Jupyter lab

We should investigate how to make the Jupyter widgets work for Jupyter lab.

I've mostly done this but what remains now is to publish the node package on npm.

We don't actually need to publish the npm package - and Jupyter Lab support is implemented, so closing!


why redirect with token in session?

here in PasswordsController: https://github.com/thoughtbot/clearance/blob/master/app/controllers/clearance/passwords_controller.rb#L33-L35
when the user comes in with their reset link from their email that has token=a1b2c3d41 in it, the user is redirected back to the same page with that token in the session instead.
Why is this done? something to do with keeping tokens out of caches / histories?
It's a problem for me right now because my url for a user's password change is /profile/password/edit?token=a1b2c3d4&user_id=123, so this action redirects the user to a url that doesn't have the user_id. I'm going to have to monkeypatch that controller action to fix this. Not a big deal but wanted to check in on the motivation behind the behavior first.
Thanks!
John

vaguely related: https://github.com/thoughtbot/clearance/issues/615

clearance by default requires user_id for looking up the user, even if/though the token is unique
https://github.com/thoughtbot/clearance/blob/master/app/controllers/clearance/passwords_controller.rb#L79
there's nothing in the clearance user model or migration that makes confirmation token guaranteed to be unique, so this is in fact necessary.
yes, i'm easily overriding the controller action for my needs.
  def edit
    @user = find_user_for_edit

    if params[:token]
      session[:password_reset_token] = params[:token]
      redirect_to url_for(user_id: params[:user_id])
    else
      render template: 'passwords/edit'
    end
  end

the addition is (user_id: params[:user_id])
what clearance proper can possibly do to accommodate my need and other future scenarios is strip token from GET params and add then into the redirect.

@jjb I can't think of a reason why there's no unique constraint on the indexes for confirmation_token and remember_token, but through normal use those tokens are generated via SecureRandom which is documented as suitable for "session key in HTTP cookies, etc".
I'd still feel better if there were unique constraints though. PR welcome!

@derekprior I was going through the implementation for the password reset feature, and couple of things came in my mind. The first one is adding the user_id in the password reset link. I don't have a very good reason why not to use it but I was considering

It's the only place we are exposing internal user_id that we used to store that specific user
It might give a rough estimation about the user base for an application (how? Maybe create a new account then try to reset the password and there is the last id )

And the another one is related to the token,  I completely agree with you, the possibility for a duplicated token is virtually zero. And we are also removing each confirmation_token as soon as a user resets their password, So I wouldn't worry about it, but if that make us feel better then sure we can do that too :). This stackoverflow thread might be helpful.
I am not totally sure if we need to consider those factor or not, but if that's the case we can do something similar to

Generate a unique confirmation_token (I'm happy with the existing way it is, but just in case we want to be 100% sure and can accept the debt of performing couple of database query to verify it's uniqueness)

Change the Password Reset Link structure to /password/edit?token=[token] or /passwords/[token]/edit and use the token to identify the user.

Based on the url structure, we can adjust the edit / update action.

Thanks for taking the time, Please let me know your thoughts, and if it's desirable, then I would be more than happy to put those together in a PR.
Inspiration:

Devise Implementaiton
Devise's token generation

I believe the why has been answered sufficiently here. I don't think user id needs to be in the reset link, but changing this now would likely be a breaking change.
I would accept a PR to the migrations that adds a unique constraint to the database.


[15721] Index Suggestion

This PR implements a search based algorithm for suggesting the best set of indexes for a given workload based on the AutoAdmin paper from Microsoft.
This involves

WhatIfAPI - that returns the cost of a query for a hypothetical index configuration (brain/what_if_index, optimizer/optimizer)

IndexSelection - Runs the search based algorithm. This involves 3 steps in each iteration - candidate index selection to choose indexes that benefit at least one query in the workload, configuration enumeration to explore the search space of the indexes and choose the top k indexes for the best configuration, and multi-column index generation to generate the indexes for the next iteration. (brain/index_selection*)

Coverage decreased (-0.5%) to 76.449% when pulling 4ef6cda9c316ab1a60b0595e15052ee16b369aa7 on sivaprasadsudhir:auto_index into 0251ba3b907a6dcdb1fba4d4de3b51ddc3dbc469 on cmu-db:master.


Clarify linkContentRef usage; Clarify dynamic link input logic comes from clientlibs

thanks for the contribution! i've reformulated the addition to the usage a bit and updated related invalid javadoc.


executeQuery returns old data and ignores fetched data

For some reason when I'm executing query that was already called before in other part of app, it returns previous data instead. It behaves like cache (that I disabled by network policy in client) but it's not because fetch is actually executed in devtools network tab. I'm receiving fresh data, but executeQuery returns old one anyway o.O If I reexecute it again and again it always returns data from previous one.
In real world scenario I'm fetching user data on init. If api does not return data (token expired) user need's to log in. After that I'm fetching user again on same query by using executeQuery again. Now API returns correct user, but urql still sees that response as null even that it's not. I'm debugging this now by returning random number in endpoint.
I noticed also:

In real app I have this query extracted in composable, but it does not matter. Same behavior if I just duplicate queries in two components.
If I change ANYTHING in one of queries like field or name - it works fine.

urql version & exchanges<EMAIL_ADDRESS><EMAIL_ADDRESS>Steps to reproduce

App.vue

<template>
  <router-view />
</template>

<script setup lang="ts">
import { gql, useQuery } from '@urql/vue'

useQuery({
  query: gql`
    query GetLoggedUser {
      me {
        id
        randomNumber
      }
    }
  `,
}).then(({ data }) => {
  console.log(data.value.me)
})
</script>

SomePage.vue

<template>
  <button @click="demo">
    Demo
  </button>
</template>

<script setup lang="ts">
import { gql, useQuery } from '@urql/vue'

const getLoggedUserQuery = useQuery({
  query: gql`
    query GetLoggedUser {
      me {
        id
        randomNumber
      }
    }
  `,
  pause: true,
})

const demo = () => {
  getLoggedUserQuery.executeQuery()
    .then(({ data }) => {
      console.log(data.value.me)
    })
}
</script>

Expected behavior
In console I should have two random numbers fetched from API.
Actual behavior
In console I have two same numbers (from first execution of query).

Is it right to assume you are dealing with a cached result being returned and another fetch happening in the background? If so that is expected behavior you can remedy this with
    return new Promise(resolve => {
      const subscription = pipe(
        client.executeQuery(),
        subscribe(result => {
          if (!result.stale && !result.hasNext) {
            Promise.resolve().then(() => {
              subscription.unsubscribe();
              resolve(result);
            });
          }
        })
      );
    });

TIL we have an async execute in the vue bidnings, do you have a reproduction for this?

Can you elaborate? If I use async and wait, it's same thing.

No I mean, can you provide a sandbox reproducing the issue. I'm assuming it's a bug in the vue bindings but yeah 😅 can't tell atm

I tried but I cannot find any public graphql example api that provides random numbers or timestamps in response so it could show issue on codesandbox :/ Will try again later or just create my own..

@JoviDeCroock https://codesandbox.io/s/eloquent-clarke-nmi52t
After start in console you should see random number received from query executed inside App.vue.
After clicking button in component (Query.vue) it executes same query defined in that component (duplicated query code for testing purposes). Problem is that it logs same number from first query, even that it received different random number. After clicking again it will show that second number. So I'm always one result behind. If that is expected behavior then I'm confused ;)

I can confirm that on v0.6.2 everything works fine now! :tada:
Too bad I lost 2 days trying to figure this out thinking that it had to be something wrong with my code and not urql...
Thanks for instant reply and fix! :pray:


Minor comments to 'Pyrsia demo: build Docker images/Maven artifacts from source' tutorials

The tutorial Pyrsia demo: build Docker images from source works fine end to end, but with a few observations:

typo in https://github.com/pyrsia/pyrsia/blob/main/docs/developers/build_from_source_docker.md#authorize-node-a-as-a-build-node

It says:
Enter disk space to be allocated to pyrsia(Please enter with units ex: 10 GB):
10GB

But it should be:
Enter disk space to be allocated to pyrsia(Please enter with units ex: 10 GB):
10 GB

typo in https://github.com/pyrsia/pyrsia/blob/main/docs/developers/build_from_source_docker.md#authorize-node-a-as-a-build-node

It says:
curl -s http://localhost:7888/status | jq  .peer_id

But it should be:
curl -s http://localhost:7889/status | jq  .peer_id

In https://github.com/pyrsia/pyrsia/blob/main/docs/developers/build_from_source_docker.md#trigger-a-build-from-source-for-a-given-artifact

The request to build is done from nodeB, and the build is done in nodeA. It says:
Build request successfully handled. Build with ID c9ca3e57-aa84-4fab-a8be-381ab31e4916 has been started.

but actually it returns:
Build request successfully handled. Build with ID 1 has been started.

Since the build is requested through the p2p network, the nodeA doesn't expose the build ID, and the response is based on a local counter in nodeB (it will increase on subsequent requests).
Only when the build is requested directly on an authorized node (nodeA in this case), the build is local, and it exposes the build ID.
Pyrsia demo: build Maven artifacts from source

Title is wrong:

says: Pyrsia demo: build Docker images from source
should say: Pyrsia demo: build Maven artifacts from source

typo in https://github.com/pyrsia/pyrsia/blob/main/docs/developers/build_from_source_maven.md#authorize-node-a-as-a-build-node

It says:
curl -s http://localhost:7888/status | jq  .peer_id

But it should be:
curl -s http://localhost:7889/status | jq  .peer_id

I think the functionality related to returning the correct build id was updated in #1321. Can @karolh2000 or @tiainen update the documentation to reflect that change and close this?


fix: UpstreamCheckUtilsTest will generate a port conflict(fix:#1003)

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1004 (328ec25) into master (ef87a9d) will decrease coverage by 0.58%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1004      +/-   ##
============================================
- Coverage     76.83%   76.25%   -0.59%
+ Complexity     1983     1974       -9
============================================
  Files           326      326
  Lines          7335     7335
  Branches        812      812
============================================
- Hits           5636     5593      -43
- Misses         1271     1315      +44
+ Partials        428      427       -1

Impacted Files
Coverage Δ
Complexity Δ

...soul/client/sofa/SofaServiceBeanPostProcessor.java
20.96% <0.00%> (-41.94%)
5.00% <0.00%> (-2.00%)

...soul/client/tars/TarsServiceBeanPostProcessor.java
26.02% <0.00%> (-20.55%)
6.00% <0.00%> (-4.00%)

.../soul/plugin/httpclient/NettyHttpClientPlugin.java
57.14% <0.00%> (-2.05%)
10.00% <0.00%> (ø%)

.../soul/admin/service/impl/UpstreamCheckService.java
76.81% <0.00%> (-1.45%)
14.00% <0.00%> (-1.00%)

...omara/soul/sync/data/http/HttpSyncDataService.java
70.58% <0.00%> (ø)
12.00% <0.00%> (ø%)

...elimiter/handler/RateLimiterPluginDataHandler.java
74.13% <0.00%> (ø)
12.00% <0.00%> (ø%)

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 29bf9e9...328ec25. Read the comment docs.

Codecov Report

Merging #1004 (328ec25) into master (ef87a9d) will decrease coverage by 0.58%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1004      +/-   ##
============================================
- Coverage     76.83%   76.25%   -0.59%
+ Complexity     1983     1974       -9
============================================
  Files           326      326
  Lines          7335     7335
  Branches        812      812
============================================
- Hits           5636     5593      -43
- Misses         1271     1315      +44
+ Partials        428      427       -1

Impacted Files
Coverage Δ
Complexity Δ

...soul/client/sofa/SofaServiceBeanPostProcessor.java
20.96% <0.00%> (-41.94%)
5.00% <0.00%> (-2.00%)

...soul/client/tars/TarsServiceBeanPostProcessor.java
26.02% <0.00%> (-20.55%)
6.00% <0.00%> (-4.00%)

.../soul/plugin/httpclient/NettyHttpClientPlugin.java
57.14% <0.00%> (-2.05%)
10.00% <0.00%> (ø%)

.../soul/admin/service/impl/UpstreamCheckService.java
76.81% <0.00%> (-1.45%)
14.00% <0.00%> (-1.00%)

...omara/soul/sync/data/http/HttpSyncDataService.java
70.58% <0.00%> (ø)
12.00% <0.00%> (ø%)

...elimiter/handler/RateLimiterPluginDataHandler.java
74.13% <0.00%> (ø)
12.00% <0.00%> (ø%)

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 29bf9e9...328ec25. Read the comment docs.


--start-at and --start-after options

Found a bad commit which broke the script: https://github.com/simonw/sf-tree-history/commit/3fb63a99dfab8a75c83d341c67afc9abf484e0c4 in https://github.com/simonw/git-history/issues/21#issuecomment-983130553_
Solution: options to say "start at this commit" or "start at the commit AFTER this commit".

The reason I want both is that it's actually not trivial to look up the commit that came directly after a commit in a Git history - so if you find a commit that's broken and think "OK, I want to start at the next one" looking up what that next one is would be non-trivial.
So having both options makes sense to me.

I'm going to add a --skip option too for skipping individual commitsz

Documentation: https://github.com/simonw/git-history/blob/29f9f8d7e54b40f083097b9591cd0f7a4fd847fa/README.md#additional-options


Control parameter accessors need to generate code

I was wrong when I said that the new control parameter accessor values would always be constants, like they are in most models. EPS is highly data-driven to adapt to different model regions. The initial and final times are read from CSV files. Simply copying the formula text doesn't work. We need to generate code for the RHS after all.
The attached zip file illustrates the problem with a model adapted from EPS. Here's how the initial and final times are specified.
control.zip
INITIAL TIME=
	GET DIRECT CONSTANTS(
	   'InputData/IT.csv',
	   ',',
	   'B2*'
	)
	~	Year
	~	The initial time for the simulation.
	|

FINAL TIME=
	GET DIRECT CONSTANTS(
	   'InputData/FT.csv',
	   ',',
	   'B2*'
	)
	~	Year
	~	The final time for the simulation.
	|

This results in the following invalid C code for the new control parameter accessors.
double getInitialTime() {
  return GET DIRECT CONSTANTS('InputData/IT.csv',',','B2*');
}
double getFinalTime() {
  return GET DIRECT CONSTANTS('InputData/FT.csv',',','B2*');
}

Is it possible to define the accessors like this? Or are they not initialized yet when the runtime needs them?
double getInitialTime() {
  return _initial_time;
}
double getFinalTime() {
  return _final_time;
}
double getSaveper() {
  return _saveper;
}

@ToddFincannonEI: Well, that's interesting, but I'm glad that EPS helped us to get a counterexample so soon :)

Is it possible to define the accessors like this? Or are they not initialized yet when the runtime needs them?

I started down that road when I first added the accessors, but decided against it because it would mean running some portion of the model init and running the model for one time step to properly initialize SAVEPER in the case where it's defined in terms of TIME STEP.  (And now that it's clear from the EPS example that we need to be more flexible for INITIAL TIME and FINAL TIME as well.)
Looking at it again, maybe that approach wouldn't be so awful, and it would allow us to get rid of the special code gen and instead define the accessors in model.c.  I'll look now and see how hard this would be.

@ToddFincannonEI: I took a crack at it, preliminary changes are on a branch:
https://github.com/climateinteractive/SDEverywhere/compare/main...chris/301-control-params
The downside is that it doesn't handle the case where one or more control params are overridden by setInputs; I don't know if that's common, but it seems like a plausible scenario.
The other approach I considered is changing the way the ModelRunner implementations work in the runtime packages and make it so that the only way you can get an Outputs instance (where the control params are most needed) is by running the model once and passing undefined for the initial outputs parameter, like:
let outputs = modelRunner.runModel(inputs, undefined)

That might work, but I'll need to look at some of the call sites and in En-ROADS to see if it can pan out.

This fix works for EPS. It would be fine with me to use this and await an actual need for control params set in inputs.

@ToddFincannonEI: Thanks for confirming that it works for EPS.  I agree we can go with this simple solution for now.  I'll submit a PR shortly.
I also opened up a new issue #302 to have a place to link possible improvements to this interface to other API improvements I'd like to make before 1.0 (so we'll do the quick thing now, and can consider how to make it better later).


fix test?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3529edc). Click here to learn what that means.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   77.86%
=========================================
  Files             ?       34
  Lines             ?     3339
  Branches          ?        0
=========================================
  Hits              ?     2600
  Misses            ?      739
  Partials          ?        0

Impacted Files
Coverage Δ

tests/integration.rs
100.00% <100.00%> (ø)

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 3529edc...cfbf068. Read the comment docs.


Extended character set ?

Is there any documentation on any extended character set ?  For example is there a degrees symbol ? (usually chr(223) )

The datasheet for the display driver is here: http://www.lcd-module.com/eng/pdf/zubehoer/st7036.pdf
See page 22 for the internal character set.

Excellent - thanks.
Degree symbol is indeed  223 or 0xDF


Add contact information

Add issues entry in metadata and update maintainers

The CharnHub incident may cause the integration tests to fail. I introduced a temporary fix in https://github.com/canonical/wordpress-k8s-operator/pull/28 but it hasn't been merged yet.


Some pages without processing/cropping

Sometimes there is a need for some pages in the book not to crop the margins, but now the checkbox affects all pages without exceptions.
Could you add functionality where pages are added to exceptions before processing and they are not cropped?

Can you add details on which Kindle/resolution you are using and what resolution the source images are?

Maybe something like a structure scanner that outputs it to the program window, and there it marks which pages DO NOT need to be cropped?
import os
import zipfile
from bs4 import BeautifulSoup

def extract_cbz_structure(cbz_path, extract_dir):
    with zipfile.ZipFile(cbz_path, 'r') as cbz_file:
        cbz_file.extractall(extract_dir)

def generate_html_with_checkboxes(extract_dir):
    html = ""
    for root, dirs, files in os.walk(extract_dir):
        relative_path = os.path.relpath(root, extract_dir)
        indent_level = relative_path.count(os.sep)
        indent = "    " * indent_level
        html += f'{indent}<input type="checkbox" name="{root}" value="{root}">{relative_path}
'
        for file in files:
            if file == 'ComicInfo.xml':
                continue  # Skip the ComicInfo.xml file
            file_path = os.path.join(root, file)
            relative_file_path = os.path.relpath(file_path, extract_dir)
            indent_level_file = relative_file_path.count(os.sep)
            indent_file = "    " * (indent_level_file + 1)
            html += f'{indent_file}<input type="checkbox" name="{file_path}" value="{file_path}">{file}
'
    html += ""
    return html

# Path to your cbz file
cbz_path = "/path/to/file.cbz"
# Folder to extract the cbz file structure
extract_dir = "/path/to/folder"

# Extract the cbz file structure
extract_cbz_structure(cbz_path, extract_dir)

# Generate HTML with checkboxes
html = generate_html_with_checkboxes(extract_dir)

# Write HTML to file
with `open("/path/to/output.html",` "w") as f:
    f.write(html)


feat: allow blob archival for bigger time periods

a feature requested by EF devops to archive blobs for longer periods

:tada: This PR is included in v1.16.0 :tada:


Add TPC-H benchmarks

Now that https://github.com/ursacomputing/arrowbench/pull/33 is merged, could we add the TPC-H benchmarks?
The benchmark name is tpc_h, and it accepts the following parameters the default values in {arrowbench} are what we would like to call:

engine - "arrow"
query_num - 1, 6
format - "native", "parquet", "feather"
scale - "1", "10"
mem_map - FALSE (I believe you can leave this unspecified and it will take the default which is what we want — it only impacts the native benchmark regardless)

{arrowbench} automagically generates the TPC-H test data if it doesn't exist (and reuses it if it does), so we should not need to do anything to put the data anywhere.
When I run all of these permutations on my computer (with maximum cores available, 3 iterations), they take a total of 2.9 minutes to run, so shouldn't add a huge amount of time to our benchmark runs, though if we want to cut that down, we could remove one of the formats (either parquet or feather, they are pretty similar).
We will be adding queries as we have the ability to run them, which I think will need PRs to benchmarks, but once we have the structure up it should be easy to add them in (they will each be added to the defaults in {arrowbench} as well).

Done in:

https://github.com/ursacomputing/benchmarks/pull/81
https://github.com/ursacomputing/benchmarks/pull/82
https://github.com/ursacomputing/benchmarks/pull/83
https://github.com/ursacomputing/benchmarks/pull/84


weight file error

when I use the file ‘east_model_weights_2T736.h5’, Error comes up. ValueError: You are trying to load a weight file containing 58 layers into a model with 30 layers. It seems the weight file does not match.

Caused by a bug-fix, if you update code recently, you need re-download the .h5 file from the link below.
Link: https://pan.baidu.com/s/1KO7tR_MW767ggmbTjIJpuQ
password: kpm2

@huoyijie Thanks! It works!

@huoyijie I don't  understand Chinese language. Can you please send direct link for downloading weights? is it free?

@huoyijie I don't understand Chinese language. Can you please send direct link for downloading weights? is it free?

It is a Cloud disk for sharing files， just like Google cloud disk. Enter the password and click the button you can download it.


how to convert to dto list  if header row is not in first row

I have a sample sheet to be import,    and for some reason,  the header row was not in first row( was export by other system),  I had tried the sheet.ToDataTable() API, but the dataTable's ToEntities() API not working
 using (var fs = File.OpenRead(fileName))
            {
                // auto detect workbook type, for .et file, got HSSFWokbook
                var workbook = ExcelHelper.LoadExcel(fs);

                // get default sheet
                var sheet = workbook.GetSheetAt(0);
                var dt = sheet.ToDataTable(1);

                var list = dt.ToEntities<TestEntity>();  // the list was not empty, bug every item's value is empty!!
}

below is the sample file

data.zip

You can specific the headerRowIndex, you can see api document here ToDataTable(this ISheet sheet, int headerRowIndex)
besides, you can convert excel to entity list directly, you can config on model with attribute [Sheet(SheetIndex = 0, StartRowIndex  = 2)], HeaderRowIndex now is StartRowIndex-1, so set StartRowIndex to 2,
and you can set via fluent config also, see sample below
FluentSettings.For<TestEntity>().HasSheetConfiguration(0, "SystemSettingsList", 2, true);

Thanks for your reply. Having Tried the above configuration and now I can get the entity list directly from a sheet obj~

Cool


Unify Fastly Compute product name in docs

At Fastly, we have unified our branding for edge computing with the name "Fastly Compute".
This pull request updates references in the docs to our product to this name.
These are docs-only, so it's been branched from the main branch.

Thanks!


Double colon (::) operator

I downloaded a boilerplate from TJ Holowaychuk and he is using :: operator like this ::this.onClick . I guess it has the same effect as this.onClick.bind(this).
My vim editor complains about this and the message is "Parsing error: Unexpected token : (undefined)" .  Is this being added to standard? Or does eslint even support this yet?
Here's the link to TJ's code:
https://github.com/tj/frontend-boilerplate/blob/master/client/components/Header/index.jsx

This is a ES7 feature. You need to use the custom parser, babel-eslint. See: http://standardjs.com/index.html#can-i-use-a-custom-js-parser-for-bleeding-edge-es6-or-es7-support


 Ability to configure a default KongIngress resource

The documentation mentions two ways to associate an Ingress with a KongIngress resource (deploying with the same name and annotationing), but it would be nice if there was a third option which would allow you to configure a default KongIngress configuration.

@bspaans
I see that you're trying to use certbot with Kong.
You could currently use the annotation driven method to apply a single KongIngress resource to your select Ingress specs.
Have you tried using the annotation driven method?
I'm not sure if Ingress spec of Kubernetes specifies what to  do with Host header when proxying the request upstream and hence we're relying on the default behavior of Kong.

Thanks for your reply @hbagdi. I am using KongIngress references myself for my own Ingress resources, but cert-manager creates a new Ingress resource to handle the ACME http01 validation, and that Ingress resource doesn't have this annotation (the issue I raised with them asks to make that configurable) and seems to rely on the default behaviour of the nginx controller. For reference here's an example of my Ingress:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: escapekit-user-service-ingress
  annotations:
    kubernetes.io/ingress.class: nginx
    configuration.konghq.com: global-kong-ingress-rules
    certmanager.k8s.io/issuer: letsencrypt-escapekit
spec:
  tls:
  - hosts:
    - "www.example.com"
    secretName: escapekit-cert
  rules:
  - host: "www.example.com"
    http:
      paths:
      - path: /api/v1/auth
        backend:
          serviceName: user-service
          servicePort: http

The Ingress spec does not prescribe what to do with the Host header as far as I'm aware, but I was under the impression that the Kong controller would be a drop-in replacement for the nginx controller, which does seem to default to forwarding the Host header (see https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/#custom-nginx-upstream-vhost)

Hi @bspaans,
Sorry for the late response here.
I see that you opened up https://github.com/jetstack/cert-manager/issues/958 for this purpose.
I do understand that this is a problem for you and we would like to help out but the proposed solution can create more problems rather.
Resource definitions should be as declarative as possible and be extensible as well. In my analysis, adding another resource to provide defaults seems like a bigger hammer than necessary to solve this problem.
Having said that, we would like to solve this issue and definitely want cert-manager and Kong to work together.

@hbagdi i second this. Just ran into this issue myself. Is there a workaround?

@hbagdi  I have the same issue.
preserve_host default to true.
Otherwise, every ingress modification, I must manually change it.
workaround for now ?
Thanks!

Blocked on https://github.com/jetstack/cert-manager/issues/1097

Do we have any workaround for this?
I am considering installing the nginx ingress just for this, but I really don't want to.

I wrote a small helper as a workaround for this problem.
https://github.com/ollystephens/acme-kong-kube-helper
Hopefully it won't be needed long-term, but is an alternative to modifying the core ingress controller codebase in the meantime.

I've applied this acme-kong-kube-helper github.com/ollystephens/acme-kong-kube-helper in order to solve the http01 validation problem in staging environment, and all it's works.
When the http01 validation to be performed, is necessary to use the letsencrypt production environment to get the https encryption.
The problem is mainly that cert-manager have some problems to work with other ingress controllers to nginx (?). This helper is a temporal solution whereas cert-manager work in this feature to solve this https://github.com/jetstack/cert-manager/issues/1097 (?)

How would setting preserve_host: true as default be a bad thing? This is how the Nginx ingress treats a request. I spent hours trying to trouble shoot this same issue for a MinIO deployment recently, and it would just fix this major issue with cert-manager.

Just incase someone runs into this error deploying MinIO behind Kong again:
The request signature we calculated does not match the signature you provided. Check your key and signing method.

preserve_host will be set to true by default starting with the next version of the Ingress Controller: https://github.com/Kong/kubernetes-ingress-controller/blob/next/internal/ingress/controller/parser/parser.go#L201

Fantastic! This will be super helpful. 0.4.x?

0.4.x?

Yes.

An update here:
https://github.com/Kong/kubernetes-ingress-controller/pull/286 will fix the original problem that this issue was based on.

Closing this as an alternate solution is now available.


OverlayShaderProgram ignores "enableColorTransfers" and always converts overlay colors

Version
Media3 pre-release (alpha, beta or RC not in this list)
More version details
1.3.0-beta01
Devices that reproduce the issue
All
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Not tested
Reproduction steps
Create a composition with a video with an OverlayEffect with a simple bitmap applied.
Disable color transfer in the transformer.
 val videoProcessor = DefaultVideoFrameProcessor.Factory.Builder().setEnableColorTransfers(false).build()
 return Transformer.Builder(context).setVideoFrameProcessorFactory(videoProcessor)[...]

Expected result
The colors of the bitmap used as an overlay are not changed in the final result
Actual result
If the transformer has "setEnableColorTransfer(false)" the bitmap's colors on the video result are changed from the input.
This seems to be because OverlayShaderProgram.java has the shader defined inside, always applying "applyEotf()" to the texture's colors. This shader should probably listen to the "enableColorTransfers" variable to not apply these transformations. Also, this is not easy to change from an app, as OverlayShaderProgram depends on multiple protected classes. I arrived to this conclusion after this issue I was having previously: https://github.com/androidx/media/issues/1080
Media
Not applicable
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

The DefaultVideoFrameProcessorFactory setting you mention only controls the colors of the input's to the VideoFrameProcessor, i.e. the input frames from the EditedMediaItem, not the input to individual effects or there corresponding shader programs.
The OverlayEffect requires the input (video) frame to be in optical (aka linear) input colors so it can blend the overlay into the frame a linear color space, so since bitmaps are stored in electrical colors, we always to convert them to linear.
In future, we will support color management in the DefaultVideoFrameProcessor, which will allow shader programs to announce what input colors they expect and what output colors they produce so that the DefaultVideoFrameProcessor can automatically insert the needed color transfers between shader programs so that the input frame is in the correct color space when it's inputted into the shader program. Until that is implemented, it's up to the user to ensure that the effect's required color space and input frames color space is compatible.

The DefaultVideoFrameProcessorFactory setting you mention only controls the colors of the input's to the VideoFrameProcessor, i.e. the input frames from the EditedMediaItem, not the input to individual effects or there corresponding shader programs.
The OverlayEffect requires the input (video) frame to be in optical (aka linear) input colors so it can blend the overlay into the frame a linear color space, so since bitmaps are stored in electrical colors, we always to convert them to linear.
In future, we will support color management in the DefaultVideoFrameProcessor, which will allow shader programs to announce what input colors they expect and what output colors they produce so that the DefaultVideoFrameProcessor can automatically insert the needed color transfers between shader programs so that the input frame is in the correct color space when it's inputted into the shader program. Until that is implemented, it's up to the user to ensure that the effect's required color space and input frames color space is compatible.


Swiper slide inline styling dissapears while navigating

Vue.js version and component version
vue v2.6.11 (nuxt.js)
vue-awesome-swiper v3.1.3, but 5.x doesn't work either
What is Expected?
During navigation styles should not dissapear. If there are 5 slidesPerView it should stay that way and it should not show one slide with inherited width: 100% from swiper-slide. It happens briefly, but long enough to confuse users.
This is how it should stay regardless if user is currently being navigated to another page or just using the current one.

Directive implementation is used. Basically the same as in Readme.md.
What is actually happening?
For some reason inline styling dissapears as can be seen on the picture below. Only class="swiper-slide" remains. Usually width and margin-right are defined.

Did you find a way to solve this ? I'm experiencing exactly the same problem.

Currently we use a sort of a hack to keep it steady.
  .swiper-slide {     width: 100%;     @media screen and (min-width: $md) and (max-width: $lg) {       width: calc(50% - 20px);     }     @media screen and (min-width: $lg) {       width: calc(25% - 20px);     }   }

I just found this thread, seems to be related... I'll go deeper later : https://github.com/surmon-china/vue-awesome-swiper/issues/317


Apache error client denied by server configuration

This following error shows up in /var/log/apache/errors.log when there are no defined Snippets for user or globally.  In such case, the error is generated and the Snippets dropdown is visible with one entry --.  The expected behavior should be to not show the Snippets dropdown and not generate an error.
[Tue  May 01 00:54:21.066773 2018] [authz_core:error] [pid 20615] [client  <IP_ADDRESS>:45273] AH01630: client denied by server configuration:  /.../www/plugins/Snippets/data,  referer: https://mantisbt/view.php?id=761

I am not able to reproduce this, despite having deleted all snippets from the system.
It is unlikely that the AH01630 error would be caused by the lack of defined snippets for the current user, as this is usually triggered by Apache / .htaccess configuration.

The expected behavior should be to not show the Snippets dropdown and not generate an error.

This was supposed to have been fixed in #24, but it would appear that it has not. I have opened #41 to track that.

As mentioned before, the Apache error is not triggered by the plugin AFAICT. The other issue has been fixed in v2.2.5, so I don't see any reason not to close this. Feel free to reopen with additional info if the problem persists.

From logging:
[Thu Mar 19 22:30:13.212322 2020] [authz_core:error] [pid 62105:tid<PHONE_NUMBER>79264] [client xx.xxx.xx.xx:61721] AH01630: client denied by server configuration: /home/customer/domains/domain.com/private_html/plugins/Snippets/data, referer: https://domain.com/view.php?id=7549
It seems there is referred to a subdirectory data.
When i create a empty folder data the error in apache logging dissapear

It seems there is referred to a subdirectory data

@TomR It's not an actual subdirectory, it's a REST API endpoint as defined in the plugin's routes method. The client-side path should be http://path.to/mantis/api/rest/index.php/plugins/Snippets/data.
I still don't have steps to reproduce...

I guess it is due to .htaccess in plugings directory.
After commenting #Require all denied the error dissapear.


When using the network framework backend, the main thread can be stuc…

…k waiting for a send to complete while the network is down
Problem
If the network is changing between 2 send calls, the NW backend may be stuck in the SendMsg method if the connection end up failing.

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

Closing this one in favour of #1406 (new fork clone).


monetary cost

Hi author, is it convenient to disclose roughly how many dollars it cost you to run a dataset（for example fb15k_237 or wn18rr） based on API_KEY?

Our work used third-party forwarded and Lab-shared keys, the records are nested with other work usage, resulting in tracing historical quota toughly.
However, I can provide an estimate strategy to you:

Randomly choosing n samples from the dataset
Perform the KICGPT link prediction, via the real key you plan to use, and calculate the average cost.
OR You can also use the same tokenizer to pre-calculate the token and calculate the cost by your API-KEY pricing policy, for that, our codes did provide a diagram for estimating query token numbers.
Scale up to the whole set

Whis it is helpful to you.


fix: dag order

When a pbft block with incorrect order hash was pushed into a chain the push would fail but at the same time dag blocks within the pbft block were removed from the non-finalized set with method setDagBlockOrder, breaking the DAG and network got stuck. The check for incorrect order hash is now moved before calling setDagBlockOrder

prnet cleaned-up successfully


[add_git_tag] add option to omit the current lane from tag and message

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Resolves #17637
This was a feature requested by a couple users, so I decided to implement it, as I see no harm 😊
Description
This simple new option should enable users to not include the current lane in the tag/message composition.
I considered creating a "lane" option instead of "includes lane", but this option didn't sound nice, semantically speaking. Like why would one pass a "lane" value other than the current lane (i.e. different than the default value) 🤔 I also refrained from generalizing the solution to a new option called "subgroup" (and defaulting that one to the current lane), although it was my second favorite solution. I'm open to discuss this :)
Testing Steps
To test this branch, modify your Gemfile as:
gem "fastlane", :git => "https://github.com/fastlane/fastlane.git", :branch => "rogerluan-add-git-tag-includes-lane-in-tag"

And then run add_git_tag action using the new includes_lane option, which receives a boolean.

I don't have visibility into what's failing in CI and/or why 😥


Image documentation URL will change soon

URL to "Manage Images using Windows PowerShell" leads to a page that tells us that content will be removed and replaced by new URL. I'm suggesting we change the URL to point to the right resource directly. New resource is located at https://azure.microsoft.com/en-us/documentation/articles/virtual-machines-images/ and is called "About images for virtual machines".

Hi @krist00fer, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@krist00fer Thank you for this contribution.  I'll have the author review.
@davidmu1 Would you please review this change and let me know if I should merge it?

Please merge.


[ServiceBus] Prepare for November 2023 release

update CHANGELOG

API change check
API changes are not detected in this pull request.


adapt template for R packages?

Feedback from survey

My understanding of packaging R code tells me that this template wouldn't work. It's great for Python though. Could there be two templates?

Believe this has been discussed previously and relates specifically to R packages not R repos
And

Additional branches could be added to address bespoke needs (such as R packaging structure etc). Disclaimer: I have been involved in the working group on this.

@Matt-Greenaway , correct me if I am wrong, but I recall you volunteered to help with setting up something like this?

Python template added - changes include adding Black and Flake8 into pre-commit config and updates to README to reflect these changes. See branch here


Update dependencies

Update all dependencies except:

nock 11.3.4 & @types/nock 11.1.0

nock 11 contains breaking API changes that broke our tests and I was unable to fix them in the amount of time I was willing to spend trying.

expr-eval 2.0.1

expr-eval 2 contains undocumented breaking packaging changes such that we were getting Parser() is not a constructor errors.

immutable 4.0.0-rc.12

immutable.js has been completely abandoned. It's used by Slate internally. Last time I checked the 4.0 release candidate didn't work with Slate and there hasn't been a new release since then.

tslint-loader 3.5.4

somehow we're at 3.6.0 while "latest" is 3.5.4, so we've been sticking with what we have for some time now.

Updates that required code changes:

React 16.9 -- componentWillUnmount() and componentWillReceiveProps() now generate console warnings. Renaming to UNSAFE_componentWillUnmount() and UNSAFE_componentWillReceiveProps() eliminates the warnings, but replacing them with non-deprecated alternatives will need to be done for React 17.
Slate 0.47 contains API changes that didn't have complete TypeScript typings the last time I tried to update so we've been stuck on an older version for some time. It seems that the missing TypeScript typings have been provided, so updating was straightforward this time.

@kswenson I still see 13 instances of componentWillUnmount that throw warnings.  Did you intend to replace them with UNSAFE_componentWillUnmount?

I thought I had done that search but apparently not. I'll add a commit.

@kswenson got it!  Looks good then!  Not sure if the componentWillReceiveProps warnings are a concern in the dependencies, but I think we're in good shape.

The warnings from our dependencies are only a concern in that 1) they're somewhat annoying and 2) they'd prevent us from updating to React 17 if the dependencies weren't updated by then. On the flip side, the advantage of the warnings is that everybody else using these libraries will see the warnings as well, which will be an incentive for the dependencies to update accordingly. There's already an issue logged for ag-grid-community (I checked).

@kswenson agreed.  It's a nice reminder to check and update those dependencies.


Add copy function and tests

Also fix a bug in the paste function (the offset was not accounted for) and in the end getter (off by one error).

I think we should have benchmarks to check the speed of paste and copy

I think we should have benchmarks to check the speed of paste and copy


Convert everything to es2015, and use classes for the polymer component controllers

Do as the title says :)

Think about having an example in es5 and an example in es2015 and beyond


fix: Current event showing up as a past event

Modifies the 'before' filter to use the end-date.
see: https://trello.com/c/5c8774jp/1138-current-event-showing-up-as-a-past-event

event no longer showing in the past events:


[WIP] rewrite css compiler

cc @giuseppeg

@giuseppeg added the changes though.test[title=","] was failing, patched it in stylis, so it's ok now. TODO: adding source-maps support.

@giuseppeg submitted updates and add source-map... At this point i don't know why package.json is conflicting, i get no conflicts when i try to rebase locally, what am i missing?

@thysultan are you pulling/rebasing against this repo or origin (your fork)?
Try
git remote add upstream https://github.com/zeit/styled-jsx.git
git pull upstream master

@giuseppeg Thanks, forgot to pull upstream, conflicts resolved now. On this branch generates source maps passes on my end, though the test was failing on master before i started this branch. Thoughts?

LGTM

@thysultan Very good PR, thanks! 😊
However, I had to remove the commit from master, because we forgot to rebase your branch before merging it. This overwrote some commits we made before.
Could you please rework the commits on master? If not, I'll do it for you and commit.


Provide require() in JS functions

The following JS function will throw an error with the message "require is not defined":
JsFunction::createWithBody("require('fs')");

This is because JS functions in PuPHPeteer are created with the Function constructor, the body of those functions don't have access to the module scope, but only to the global one.
The require() function could be provided by Rialto to JS function when unserializing them:
https://github.com/nesk/rialto/blob/2403adc6b5f8fd4114d1ec00d9cce2e6ce3a5656/src/node-process/Data/Unserializer.js#L89-L94

How to use it? Please give an example of the encoding of instantiation \Nesk\Rialto\Data\JsFunction

page call failed
$page->on('response', new \Nesk\Rialto\Data\JsFunction(["response"],
<<<JS
    page.evaluate(function(){
       document.createTextNode(JSON.stringify(response));
    })
JS
     ,['page']));

Hello,
any news?

Good day, any changes?


dev: refactor Exponentiation::pow to always wrap and never panic

Currently we have two implementation of the exponent function:

pow: raises to a power, can panic if overflow
pow_mod: raises to a power, does not panic if overflow

We use pow in:

bitshift: should wrap around
mask operations: probably should panic as you're not supposed to create a mask that overflows
opcodes (e.g. EXP): should wrap

Suggestion:

rename pow_mod into pow for it to become the default power function
rename pow to pow_unsafe to clearly indicate it can panic in case of overflow

Alternative:

Just be aware of panic risks, and use pow_mod in most places where wrap around should occur.

rename pow_mod into pow for it to become the default power function

I don't really like this as I believe the special behavior on overflow should be explicit. 2.pow(257) should not return 2^2 under any case, as the real intention behind this instruction is to compute the value of $2^{257}$
However, I agree that we could rename pow_mod to wrapping_pow for consistency with corelib functions (e.g. u128_wrapping_add returns the value, whether it has overflown or not)
To me, it doesn't make sense to use wrapping_pow unless this is the expected behavior (so only for specific opcodes). When I compute (2.pow(12)), I don't want it to wrap around if it overflows, as the intention is to get the result of $2^{12}$. However when I compute the exec_exp , I explicitly want that value to wrap around, so I will make it clear in the code.
For the rest,

Bitshifts should not wrap around by default. If you get to that point then it means there's something wrong with your shift
Same for masks
Exp should wrap because it's specified as such, therefore we will explicitly use the appropriate function whose name indicates that it wraps

My point is that we're in EVM specific domain, which means that all operations are wrapping around 2^256.
The EVM domain is akin to the Cairo0 Felt concepts, where there aren't overflows, just wrapping around.
I'm in favour of considering that every arithmetic operation in our EVM context wraps around

Wouldn't it be a bit backwards to implement for instance:
a u256 bitshift lib that panics if shift > 255, but for the opcode shr, then implement a bitshift_wide?

Also, another case is that in the memory implementation:
We write for instance:
let mask_c: u256 = 256.pow(16).into() / mask;
This is a felt252.pow, that only after the .pow() is performed gets turned into a u256.
But felt252.pow wraps around natively, as Mul wraps around
So before the issue/PR, we had two pow that behaved differently

The EVM domain is akin to the Cairo0 Felt concepts, where there aren't overflows, just wrapping around.

Then this wrapping around should be explicit, as it is currently in the core library. I don't want to call pow and have wrap around when the intent is either succeed or overflow. If it should wrap around, the function name should reflect this semantic.
This is typically the reason behind integer types in Cairo and why we don't want to perform operations directly on felts: because they might have unintended side effects

u256 bitshift lib that panics if shift > 255, but for the opcode shr, then implement a bitshift_wide?

Why not? if the intended behavior of a bitshift is to wrap around, then it should be explicitly stated. Otherwise, you just assume that it can eventually panic on overflow. Then you will use the correct one based on what you're trying to achieve

This is a felt252.pow, that only after the .pow() is performed gets turned into a u256.
But felt252.pow wraps around natively, as Mul wraps around

This is true - then it should be casted into a u256 for the operation

Alright:
Incoming changes to the existing PR:

.pow() will be panic on overflow
we'll implement wrapping_pow() for felt252 (rename felt252 pow)
we'll implement bitshift_wide (it doesn't wrap around, it discards bits above 255)
i'll refactor bitshift to use panic pow
i'll create wrapping_pow for u256 (rename pow)

sounds good

pushed some improvements


[Typo] Fix scenario dir for io hog prow scenario

Merging as it's a minor typo fix which doesn't impact krkn-hub images. Thanks.


Can't debug on macOS Catalina (LLDB)

I following the instruction in this article https://hacks.mozilla.org/2019/09/debugging-webassembly-outside-of-the-browser/,
it works fine for gdb on macos and lldb on linux, but it does not works well for lldb on macos
I try the following ways, but all failed,

codesign lldb
build lldb using  -DLLDB_USE_SYSTEM_DEBUGSERVER=ON
can someone give me an example on how to debug on macOS using lldb

it works fine for gdb on macos and lldb on linux
can someone give me an example on how to debug on macOS using lldb

The only difference between linux and macos is default state of "plugin.jit-loader.gdb.enable" settings. You can add settings set plugin.jit-loader.gdb.enable on to ".lldbinit" or beginning of the lldb session. This setting was added in lldb version 9. Other than that, linux and macosx behavior shall be almost identical.

it works fine for gdb on macos and lldb on linux
can someone give me an example on how to debug on macOS using lldb

The only difference between linux and macos is default state of "plugin.jit-loader.gdb.enable" settings. You can add settings set plugin.jit-loader.gdb.enable on to ".lldbinit" or beginning of the lldb session. This setting was added in lldb version 9 (see e.g. brew's one). Other than that, linux and macosx behavior shall be almost identical.

Thank you, It works now, but it seems not working in vscode when using https://github.com/vadimcn/vscode-lldb plugin

oh, after setup initCommands , it works now

Closing as resolved


refactor: remove discussions html url workaround

📝 Provide a description of the new feature
See #1582
Monitor the GitHub API to see if/when we can remove our discussions URL workaround, and also remove our GraphQL discussions search logic
➕ Additional Information
No response

It seems like the GitHub Notifications API has returned to it's previous expected behavior


Wrong description

Sometimes the bot gets description for wrong place.
It is probably known issue of wiki wrapper. May need to use another wrapper

This is now fixed. Bot will use api to search and return location name in cyrillic. It will then use this name to "build" other links and get description.
However, bot is still not able to search if the input is in standard Hindi alphabet.