(import org.greenrobot.greendao.annotation.Unique never used) Fixed issue#567

#567 #479

@NasiaKouts why we have two another commits link to #479 issue ?

@herau solved only issue #567


How to change TabbarIOS icon's  size?

 <TabBarIOS.Item
          selected={this.state.selectedTab == 'profile'}
          style={styles.icon}
          icon={require("./images/profile.png")}
          renderAsOriginal
          selectedIcon={require("./images/profile1.png")}
          iconImageStyle={{width: 23, height: 23, resizeMode:'contain'}}
          iconSize={20}
          title=''
          onPress={() => {
              this.setState({
                selectedTab: 'profile'
              });
          }}>
           <View style={{flex:1}}>
              <Navigator
                initialRoute={{
                  id: 'Profile',
                }}
                renderScene={
                  this.navigatorRenderScene
                }
              />
            </View>
          </TabBarIOS.Item>

Here is my code and i can't change size of the require image

Hey @Anu-Ujin, thanks for reporting this. Unfortunately we need more details to help. For example, what did you expect and what did you get as a result? If you think this is a bug in the project itself, can you make sure to fill out the new issue template that we provide when you create a new issue?

In this case, the component being displayed is an actual native UITabBar. Therefore we're limited in terms of how much it can be customized, especially when it comes to the presentation of icons. If you need to present a tabbar that is sufficiently different from the native iOS tab bar, you may want to write a JavaScript component that matches your look instead.

how to change the icon size?

@jjhesk i used this
icon={{uri: ProfileIcon, scale: 2}}
and
var ProfileIcon's image must converted by https://www.base64-image.de/

I'm having the same issue. The problem I'm encountering is that I'm saving my icons @3x for retina displays, but they get rendered 3x larger than they should on the screen. You can't use iconSize when using require() to import the PNG. Any workarounds for this?


Using a custom CA for SSL

Can you please provide recommendations on how to add a custom CA to be use for SSL?
I'm using Linode Postgress DB Cluster and it requires a custom CA to validate the Cert.
It will be also ok if I can skip the cert validation

Hello,
Can this help? https://github.com/alexandrainst/node-red-contrib-postgresql/discussions/81

Thank you, that solved the issue


Port configuration from local.settings.json not respected in dotnet-isolated mode

Software Configuration

Visual Studio 2022
.NET 6

Azure Functions Version : 4
Runtime : dotnet-isolated
There is host configuration is local.setting.json
"Host":{
   "CORS":"*",
   "LocalHttpPort":7073
}

Now if I run function with httptrigger using F5  of Visual Studio , it is still running on 7071. Also when I remove CORS then it create problem. It means that configuration read but port setting not applied when running. If I run from commandline func start then it run on 7073 so why it is not running when I direct run from Visual Studio.

Closing as duplicate of https://github.com/Azure/azure-functions-core-tools/issues/823


Fixed the WorkflowExecutionContext to properly set the status of an ended task

Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:

Fixes the WorkflowExecutionContext to properly set the status of an ended task, which was being wrongly set to the workflow instance's status

@JBBianchi FYI


Add fallback for current user

Fix #122

There is a test failing, needs investigation

It is strange. The action for commit 9a611eb passed testing. In the next commit beb784c, which failed testing, I just changed the fallback Uid and Gid. These two fields have no crosss-reference in your project, so I think it should have no impact on testing result.
I reran test multiple times on my machine and they all passed.
Maybe the testing code itself has some problem?

I reran the job manually and it worked. Really weird! I'm going to merge this.
Thanks for your contribution


Request: specific explanation of quick bookmark icon

Hi!
I may be missing something obvious to others, but thought I'd ask.
This is an excellent add-on - I gave 5 stars.
I can't understand what the quick bookmark icon is for though.
If the only difference is that it removes a bookmark with a single click, I'll just keep using the default one so I don't accidentally delete a bookmark.
Or is there a difference between pressing Ctrl+D vs hitting the default "bookmark this page" (blue) star in the address bar?
Is it because your add-on can replace some functionality but not that of the default star?
Maybe I'm not understanding some feature of the Other Bookmarks forlder.

Sometimes it wouldn't allow me to delete bookmarks within it.
It even repopulated itself with bookmarks from several years ago sometimes.
Possible synching with my Android phone...?
Regards,
James

Hello!
First, thank you for your review! Regarding your question, you have come to the right place. All questions are welcome here! 😃
The quick bookmark icon role is indeed to be able to bookmark a page in a single click or to remove that bookmark, again with a single click. The icon is an optional feature, and it might not be useful for you. I personally use it quite often when I bookmark pages during my web searches, as it saves me one click for every bookmark. However, it is true that it makes it easy to remove a bookmark by mistake, so I have opened a dedicated issue for this problem (#21).
The quick bookmark icon settings are completely independent from the settings of the built-in bookmarking system of Firefox (aka, the default blue star). In both cases, you are able to choose the default bookmark folder you prefer (and they can be different).
Using the Firefox built-in Ctrl+D shortcut is the same as clicking on the default blue star.
Regarding the Other Bookmarks folder, from my understanding it is a just a regular folder. You might have been experiencing syncing issues, which happens sometimes.
Hope it helps! Let me know if you have more questions!

Considering the question answered and closing the issue. Please feel free to reopen it if you deem it necessary. 😃


Proposal: Rework metric instantiation to get-or-set from the registry

At present (v0.19.0 and previously) the prometheus_client.registry.Registry class is secondary in terms of API control flow to the prometheus_client.metrics.MetricWrapperBase and subclasses in that a caller says prometheus_client.Counter() and doing so implicitly first instantiates a new metric and then registers the names it defines into the global registry.
This makes the default behavior of the library code-reloading unsafe. Today if you reload code which invokes Counter(), you get a new counter instance which encounters a name conflict when it tries to register itself.
It would be better if the primary interface first performed a get-or-set against the specified registry so that reloading would re-fetch the original counter functionally as a singleton.
The real solution is that we should be explicitly managing and lifecycling registries, but the library default behavior of using a shared global registry creates this pitfall.

We've adopted this among other changes at work in an attempt to mitigate this behavior while preserving the existing interfaces.
0004-Make-metrics-reloading-safe-with-bad-memoization.patch

My concern with a get-or-set feature is what if two files actually tried to create identical metrics? Then you could get unexpected behavior if the second usage started changing the metric defined in the first file. I would prefer to error in that case, but maybe I am missing something?
Another option could be to make it easier to clear an entire registry (including the default registry). Would that also help your reloading use case?


Switch to yarn

Because enough is enough

We don't need to switch to yarn & yarn.lock.
We can just remove the npm dep and that's it.

And if we are gonna switch to yarn for other reasons then we need to be careful about transitioning from package-lock.json to yarn.lock.
We need to carefully review lockfile changes to make sure someone isn't sneaking in a malicious dep.

Yeah, just wanted to switch to yarn sooner than later as it's consistent with all the other NodeJS repos. And the grpc npm bug (where sometimes it's being built/copied in the Docker image, and sometimes it isn't) turned out the right moment to pull the trigger.
Removed the npm dependency simply because it's not required as you earlier suggested, it isn't related to this specific switch of course.

we need to be careful about transitioning from package-lock.json to yarn.lock

How do you feel about yarn import?

We need to carefully review lockfile changes to make sure someone isn't sneaking in a malicious dep.

$ wc -l package-lock.json
    7798 package-lock.json

😐

Merging this as per our discussion on Keybase, primarily in favor of two reasons:

Yarn is helping us get rid of the pesky GRPC bug, where sometimes it's built and included in the final Docker image (resulting in +300mb image size), and sometimes it isn't (#43).

It's consistent with the entire Umbrel stack. NodeJS 12.16.3 + Yarn.


Dropouts

#34  Implemented Dropouts using Tensorflow's Dropout Wrapper around rnn_cell and dropout function for embedded_input.

Hi, there is an infer variable which is essentially !training. Only one of the two should be used. I think training is a better name, so maybe use that instead everywhere?
Thanks for your contribution.

Please check ^

Hi, will this pull request be merged soon?
Using appropriate dropouts will definitely make a more robust model.

@sherjilozair This looks good. Are you going to merge?
@anantzoid, do you think you can send this dropouts PR for word-rnn, https://github.com/hunkim/word-rnn-tensorflow/? I'll merge right away. Thanks in advance.

@anantzoid This PR has merge conflicts.

@hugovk This is resolved.

The latest Tensorflow stuff I tried has options for both input_keep_prob and output_keep_prob.  So I'll give a read through:

https://www.tensorflow.org/api_docs/python/tf/contrib/rnn/DropoutWrapper
https://www.cs.toronto.edu/~hinton/absps/JMLRdropout.pdf

After better understanding dropout, I'll try to add an option for both input and output on the CLI and get this tested and merged.  Thanks!

Okay, going to try to get this merged with:

both input and output keep probabilities available as CLI options
express the dropout rate p as defined below (as a probability [0,1] of keeping the input or output
I don't think its a perfect match to have input_keep_prob = "input layers" and output_keep_prob = "hidden layers" depending on the architecture depth etc, but good enough start for now.

Reference
https://www.cs.toronto.edu/~hinton/absps/JMLRdropout.pdf:
Dropout introduces an extra hyperparameter—the probability of retaining a unit p. This
hyperparameter controls the intensity of dropout. p = 1, implies no dropout and low values
of p mean more dropout. Typical values of p for hidden units are in the range 0.5 to 0.8.
For input layers, the choice depends on the kind of input. For real-valued inputs (image
patches or speech frames), a typical value is 0.8. For hidden layers, the choice of p is coupled
with the choice of number of hidden units n. Smaller p requires big n which slows down
the training and leads to underfitting. Large p may not produce enough dropout to prevent
overfitting.

Almost got it, the only odd thing I see is this line was changed in the PR:
output = tf.reshape(tf.concat(1, outputs), [-1, args.rnn_size])

Which raises the error:
TypeError: Expected int32, got list containing Tensors of type '_Message' instead.

So I put it back to what the original was, which seems to work and dropout appears in the Tensorboard graph.  This is what it was, and still is:
output = tf.reshape(tf.concat(outputs, 1), [-1, args.rnn_size])

Going to add a quick patch to extend to both input and output dropout CLI args then merge.

Thanks all, I could use some code review especially right here:
https://github.com/sherjilozair/char-rnn-tensorflow/blob/master/model.py#L47-L49


Closes when maxmized on bigger Extended screen

Terminal gets automatically closed when I try to maximize the screen on extended screen.

/feedback
this is almost certainly /dup #3303, but we'll un-dupe it if it looks different enough. thanks!


Circuit python

With Circuit Python MCU is only visible and only works terminal
If I try opening explorer there is no files, and if I try to create one, app crashes
Found with esp32c3 and esp32s2, both works correctly with PC

CircuitPython has a different files system than  MicroPython. I will work to support it soon
Thanks


Navbar Component

Creating navbar components using Prime faces
Sample Output.

With reference to this sandbox code which is officially from the prime vue, I created my components.
https://codesandbox.io/s/gallant-worker-mivksd
So If I created something similar to this I am not able to render the menu items in the navbar. without the menu items, I am able to render the navbar.

:model = headerMenuItems.
here the headerMenuItems is the list of menu items, which is going to be rendered. but when I use :model in my code the whole page is getting crashes.
NOTE: Even though these menu items are rendering they will be not editable since the values are coming from the place where the Vue instance is created. Similar to this issue #29


Why num_depth is (sample_interval*(2i+1)/2 + depth_min)

Congratulations, what a excellent job!
But I have a question about sample interval.
For dtu_io.py line 526-527 in datasets folder:
sample_interval = depth_range / 4.0
sample_depth = []
for i in range(4):
     sample_depth.append(np.ones_like(depth) * (sample_interval * (i + i + 1) / 2.0 + depth_min))

The sample depth acutally is  a decimal,  Why can not be an integer like this：
sample_interval = depth_range / 4.0
sample_depth = []
for i in range(4):
     sample_depth.append(np.ones_like(depth) * (sample_interval * ( i+ 1) + depth_min))

Look forward your reply!
Sincerely!

Hi, we use the mid-point of each bin as the sample depths. So sample_interval * (i + i + 1) / 2.0 actually computes the mid-point of i-th bin.


[Feature] Allow passing custom props to button

With the react-query devtools I can pass in custom props to make adjustments to it. In particular, it's good to be able to pass a className or a custom style to override some defaults, like done here:
<ReactQueryDevtools
  position="bottom-left"
  toggleButtonProps={{
    className: 'noPrint',
    style: { transform: 'translateY(-3rem)' },
  }}
  panelProps={{
    className: 'noPrint',
  }}
/>

None of the placements of the react-hook-form devtools button work for me as there are fixed toolbars at the top and the bottom of our app. So I need to be able to pass in a transform to nudge it upwards, like I am able to with the react-query devtools.

This is an important thing that shouldn't be too difficult to add. Is there a contribution guide? I currently can't use hookform devtools. After opening it on a form, I literally can't close it.


Relative images not found

Thanks for writing this script. I can successfully import text Markdown files with it. However, in testing this it's failing to locate relative images.
I have a directory structure on Mac ~/Desktop/test/ where the contents of test are a file Favorite tweet – PizzoniGiada – June 17, 2020 at 0901AM.md and a folder Favorite tweet – PizzoniGiada – June 17, 2020 at 0901AM which contains a single image image_1.jpeg.
Running using Python 3.8.3 and the latest version of md2notion installed via pip3 install md2notion, I am getting the following:
ELSLABM-155961:Desktop stratfordm$ python3 -m md2notion {token_2} https://www.notion.so/Test-0192acd694d443ec924a2a5de270c90d /Users/stratfordm/Desktop/test/Favorite\ tweet\ – PizzoniGiada\ –\ June\ 17\,\ 2020\ at\ 0901AM.md
Initializing Notion.so client...
Getting target PageBlock...
Uploading /Users/stratfordm/Desktop/test/Favorite tweet – PizzoniGiada – June 17, 2020 at 0901AM.md to Notion.so at page Favorite tweet – PizzoniGiada – June 17, 2020 at 0901AM.md...
Uploading ImageBlock, 4/7 (57.1%)ERROR: Local image '/Users/stratfordm/Desktop/test/Favorite%20tweet%20%E2%80%93%C2%A0PizzoniGiada%20%E2%80%93%20June%2017,%202020%20at%200901AM/image_1.jpeg' not found to upload. Skipping...
Uploading TextBlock, 7/7 (100.0%)EL

The end result is a note created but without the image attachment.
Hope this helps.

Couple things:

I see Windows-esque paths (/Users/stratfordm/Desktop/test/) so perhaps I used an incorrect pathing module.
URLEncoded image links, perhaps a similar problem to #13

Will give a look next time I'm on computer

Hmm, so after looking at this further, it looks like links and images with spaces are actually supposed to be interpreted as text by CommonMark

So my suggestion is it's probably better to rename your attachments, but for the sake of utility, it might be nice if md2notion checked the URLs for file:// syntax as well as URL decode and potentially try all three strings as paths to find a file.

Alright, this should now work for you in v2.1.1. We can reopen if it doesn't

Thanks! I'll take a look presently…


Update to release 3.1.0

Make the necessary release updates including README.md

Codecov Report

Merging #471 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #471   +/-   ##
=======================================
  Coverage   60.12%   60.12%
=======================================
  Files          49       49
  Lines        3155     3155
=======================================
  Hits         1897     1897
  Misses       1258     1258

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 9512ecf...f5d11fa. Read the comment docs.


Remove the "gridLayoutManager" field and declare it as a local variable in the relevant methods.

Description:
Private fields only used as local variables in methods should become local variables. When the value of a private field is always assigned to in a class' methods before being read, then it is not being used to store class information. Therefore, it should become a local variable in the relevant methods to prevent any misunderstanding.
Location:
src/main/java/de/dennisguse/opentracks/settings/SettingsCustomLayoutEditActivity.java
Category:
Maintainability

Issue was solved by moving the private field as a local variable in the method it is used in.
Commit: https://github.com/Reva-B98/OpenTracks-Winter-SOEN-6431_2024/commit/9fe972f6b9fec1b44b7ab8b530021e83b53aebd6


Merge Feature/ci into development

Run cargo check, fmt, clippy, test etc. on events

Decided that a development branch is overkill. Just merge stuff into main.


Update viewportarray.cpp

GPU does not support multi viewports and exit directly.

vks::tools::exitFatal already includes an exit with the exact same error code, so I don't think this is necessary at all.


#195: Option to enable case insensitive wildcard mapping

Issue link
#195 Dozer partially case sensitive in >=5.5.1
Purpose
Adds an option "wildcard-case-insensitive" to ignore the case of fields in wildcard mappings. The behavior is undefined if a single source or target class has multiple fields that only differ in case.
Open Questions and Pre-Merge TODOs

[ ] Issue created
[ ] Unit tests pass
[ ] Documentation updated
[ ] Travis build passed

Don't know why the OSGI tests fail on Travis, they run fine locally.

OSGi due to a test flake.
@orange-buffalo ; can you review? i'll try to review sometime this week.


Fix Cannot read property 'loaders' of undefined (fixes #43)

:+1:

@muffinresearch sorry I misse this - thanks again! I'll tag for npm shortly


Revert channel name back to 'stable'

Signed-off-by: Renuka Fernando<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

hi @J0zi  and @mvalarh,
Need some help :)
In a previous PR[1] we changed the channel name and added a new channel. Seems the channel rename is not reflected in [2] but the new channel has successfully added.
Also added a new version to the old channel [3].
Since changes have not been applied to the old channel, wished to revert the name back to 'stable' with this PR. Could you help to get this done please?
[1] https://github.com/operator-framework/community-operators/pull/3710/files
[2] https://operatorhub.io/operator/api-operator/stable/api-operator.v1.2.2
[3] https://github.com/operator-framework/community-operators/pull/4012/files

/merge possible


Update CREATE STATISTICS doc with missing variants

Fixes #4241, #4253, #4180.
Summary of changes:

Update CREATE STATISTICS doc with the following variants:

CREATE STATISTICS foo FROM bar
CREATE STATISTICS auto FROM bar
CREATE STATISTICS foo FROM bar AS OF SYSTEM TIME '-4h'

Added information about automatic stats collection to the CREATE
STATISTICS page (via an include)

Updated instructions for deleting statistics, added to both CREATE
STATISTICS and SHOW STATISTICS pages (via include)

Updated railroad diagram and parameter descriptions for CREATE
STATISTICS v2.2

Updated sidebar and page titles to remove extraneous "experimental"
text, since it's already clearly marked in the various documents via
an include.

Used a proper emdash (!) in a few places.

On Jan 31, 2019, at 4:15 PM, rytaft<EMAIL_ADDRESS>wrote:
This is great, Rich! I have a few comments:

Thank you! And thanks for reviewing this PR.  Responses and a couple of questions below.

On the CREATE STATISTICS page:
I think you can get rid of the warning saying "this is an experimental feature". I don't think the interface/output are likely to change. (Maybe you could call it a beta feature similar to the cost-based optimizer page…?)
Done.
You can get rid of the item under "Considerations" saying that old stats must be deleted by the user. We now do this automatically (there are still some cases where a user may want to delete stats, though -- e.g. if they manually created a column stat that turned out not to be useful and is not one of the stats we refresh automatically)
Done.  I deleted that section altogether (but left the instructions for deleting stats where they were for reference when needed).
Under the Automatic Statistics section, you can take out the part about manually creating stats to avoid overloading the system. I've fixed that issue now. Users should still manually create stats for tables that they don't expect to update frequently, though, since I haven't yet added logic to do that automatically. Enabling automatic stats will currently only create statistics for tables that are updated regularly. (I'm going to be working on this next, so there's a chance I could also get this feature in before the branch is cut Monday…)
I’ve removed the part about manually creating stats.

Should I still add the instruction that users should still manually create stats for tables they don’t expect to update frequently?  Or did the potential change you mentioned make it in?

You can take out all references to the name auto. Since we're using that name for automatic stats, I think it's better if users just aren't aware of it. They should create their own names instead. (I told you to include it before because of the potential for overloading the system -- but that's not a concern anymore).
Done.  All references to auto have been removed from the docs.
The name for the setting has changed. It's no longer sql.defaults.experimental_automatic_statistics. Now it's a cluster setting and called sql.stats.experimental_automatic_collection.enabled. (So you'll need to run SET CLUSTER SETTING sql.stats.experimental_automatic_collection.enabled=true)
Done.  All references to this setting have been updated.
For the AOST example, I'd suggest using a more realistic example than 4h -- maybe 1m? The reason to use AOST in this case is just to avoid interfering with the production workload. There's no benefit to having stats that are really old.
Done.  Updated to use ‘1m’ as you said.
On the SHOW STATISTICS page:
I think you can also get rid of the experimental feature warning here.
Done.
Otherwise everything looks good.
Yay, thanks!
Sorry this is a lot of changes -- things keep changing since I'm actively working on this stuff! Let me know if any of this is confusing -- happy to discuss offline too.

No worries, change is good.  I think other than my question above I’m not too confused about anything you wrote (at least that I’m aware of, but let me know 🙂)

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e47c38328a846387505a61f4614d47e07ed9cfa1/

Great -- thanks! Do you have links you want me to review again?
Regarding creating stats for tables that aren't updated frequently: I did get a PR in that creates stats for all tables (including read-only tables) when the nodes start up (https://github.com/cockroachdb/cockroach/pull/34519). So I think the main advice we should give to users now is that they should restart their servers after enabling the sql.stats.experimental_automatic_collection.enabled cluster setting. (Of course this probably doesn't matter if they are doing an upgrade -- presumably they would be cycling the machines anyway...)

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2b2e7c59273522975870f95c3fc0944a8d9d89c6/

On Feb 4, 2019, at 2:45 PM, rytaft<EMAIL_ADDRESS>wrote:
Great -- thanks! Do you have links you want me to review again?

Yes please -  Here is the updated link to CREATE STATS which also includes the information below re: restarting after the setting):
CREATE STATISTICS http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2b2e7c59273522975870f95c3fc0944a8d9d89c6/dev/create-statistics.html

Regarding creating stats for tables that aren't updated frequently: I did get a PR in that creates stats for all tables (including read-only tables) when the nodes start up (cockroachdb/cockroach#34519 https://github.com/cockroachdb/cockroach/pull/34519). So I think the main advice we should give to users now is that they should restart their servers after enabling the sql.stats.experimental_automatic_collection.enabled cluster setting. (Of course this probably doesn't matter if they are doing an upgrade -- presumably they would be cycling the machines anyway...)

OK, I updated the automatic stats instructions with the above information - should be visible on the page linked above.

Nice! Really minor: in the AOST example you call the stats yesterday_employee_stats. Since the time is just one minute ago, why not call them employee_stats?
Another minor suggestion: maybe make the info about automatic statistics more prominent? I'm going to enable automatic statistics by default for this alpha, so we may want to make it clear how to turn them on and off.  If someone wants to turn auto stats off and also get rid of all the stats, they will need to first set that cluster setting to false, then delete the stats (using the instructions you've provided), and then restart their machines to clear the stats caches.

Also one more thing: This command fails because sql.stats.experimental_automatic_collection.enabled is a cluster setting, not a session setting:
> SET sql.stats.experimental_automatic_collection.enabled=true

instead it should be:
> SET CLUSTER SETTING sql.stats.experimental_automatic_collection.enabled=true

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/da97f8ca38a4ef51c09797fce93fe2a41c72f5bd/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f249afa2c478a8875f21724884893d538a5826c6/

On Feb 4, 2019, at 3:55 PM, rytaft wrote:
Nice! Really minor: in the AOST example you call the stats yesterday_employee_stats. Since the time is just one minute ago, why not call them employee_stats?

Fixed.  Thanks for catching that.

Another minor suggestion: maybe make the info about automatic statistics more prominent? I'm going to enable automatic statistics by default for this alpha, so we may want to make it clear how to turn them on and off. If someone wants to turn auto stats off and also get rid of all the stats, they will need to first set that cluster setting to false, then delete the stats (using the instructions you've provided), and then restart their machines to clear the stats caches.

Great idea - I’ve bumped the “Automatic Statistics” section up a level on the CREATE STATS page so it has its own TOC link.  And on the main CBO page I’ve moved it to the top of the “Examples” section.
I’ve also:

Updated the settings page to note that the default for the setting is now ‘true’.
Added your instructions above for turning off automatic stats
updated 'SET sql.stats.XXX' to ‘SET CLUSTER SETTING sql.stats.XXX’ as you noted in another comment in this thread (Thank you for this catch!)

Link to latest build of CREATE STATS page is here http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f249afa2c478a8875f21724884893d538a5826c6/dev/create-statistics.html

LGTM! I'll let you know if for some reason the final PR to enable auto stats doesn't get merged before the alpha branch is cut, but otherwise we should be good to go.

On Feb 4, 2019, at 4:49 PM, rytaft wrote:
LGTM! I'll let you know if for some reason the final PR to enable auto stats https://github.com/cockroachdb/cockroach/pull/34529 doesn't get merged before the alpha branch is cut, but otherwise we should be good to go.

Hey Becca, looks like that one merged - did it make it into the alpha release?
If so I’ll pass this along for Docs team review now.

Yep! The alpha branch hasn't been cut yet, so it's definitely in. Thanks for your work on this!


[CM_BE_20] Fixing link flow

 Here is an overview of what got changed by this pull request:

Clones removed
==============
+ test/In.ProjectEKA.HipServiceTest/Link/LinkPatientControllerTest.cs  -2

See the complete overview on Codacy


SECURITY_CONTACTS: please include developers of kubectl

The current SECURITY_CONTACTS document in kubectl simply includes the Kubernetes Product Security Team. Please replace this list with kubectl developers who should be contacted by the Kubernetes Product Security Team in case of a security disclosure.
For example you might choose a subset of the approvers from OWNERS

/assign soltysh
/assign pwittrock

/remove-lifecycle stale

never mind, this is fixed. https://github.com/kubernetes/kubectl/blob/master/SECURITY_CONTACTS

gah, it is not fixed, the kubernetes security commitee is still listed as the security contact. Please fix!

@pwittrock @soltysh any recommendation on what to do here? kubectl really deserves to have SECURITY_CONTACTS I think.

/remove-lifecycle stale

@philips sorry, I missed this issue earlier, fix in https://github.com/kubernetes/kubernetes/pull/82261


Set up CI with Azure Pipelines

Description
Enables unit testing for devel and master pipelines
Related Issue
Resolve #435
Motivation and Context
Switch unit testing from appveyor to Azure pipelines
How Has This Been Tested?
Successfully ran existing unit tests on Linux / Windows / MacOS
Screenshots (if appropriate):
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

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

Checklist:
Go over all the following points, and put an x in all the boxes that apply. If you're unsure about any of these, don't hesitate to ask. We're here to help!

[ ] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTION document.
[X] I have updated the CHANGELOG file accordingly for the version that this merge modifies.
[X] I have added tests to cover my changes.
[ ] All new and existing tests passed.

I'm assuming this relates to issue #435? @jaapbrasser

That's correct @rfitzhugh, the pull request was auto generated by Azure Pipelines, so it did not follow the correct format of pull requests.
I have updated the pull request message


css 404 error

ACCESS [2019-08-30T20:00:26.628Z] "GET /tools/static/js/main.dd792898.chunk.js HTTP/1.1" 404 - 0 146 0 0 "CLIENTIP" "Mozilla/5.0 (Windows NT 10.0; WOW64; Trident/7.0; rv:11.0) like Gecko" "1a446aa0-13f4-428a-b397-3059844385b4" "ffiec.cfpb.gov" "CLUSTER-IP:8080"

My guess is that this was an inflight request as the new release was deployed. I can verify that all tools correctly return the right css on request.


Mismatched tags and problems

🐛 Bug Report
Some problems have their tags mismatched.
To Reproduce
Open some certain problems by vscode-leetcode extension, e.g., 1011.
Expected behavior
Array, Binary Search
:heavy_check_mark: Both leetcode.com and leetcode-cn.com have these two correct tags.

by leetcode.com

by leetcode-cn.com
Extension Output
tree, depth-first-search
:x: VSCode-leetcode extension gives these two mismatched tags.

by vscode extension
Your Environment

os: Windows 10
extension settings: almost default
nodejs version: 12.18.3
vscode version: 1.55.2
extension version: 0.17.0

+1 Have the same issue with problem #1029


Printing shows extra lines on Print or ExportToBitmap

I am trying to print the following map (from LayoutControl)

But the actual output is different from expected one as below

There are extra lines and missing lines in resulting print.
I am creating the LayoutMap control programatically as below
layoutElementMap = new LayoutMap(MapToPrint);
layoutElementMap.Size = mapSize;
layoutElementMap.Location = startPosition;
layoutElementMap.Scale = (long)MapScale;
layoutControl.AddToLayout(layoutElementMap);

The map on the LayoutControl is fine but when I tried to print this to printer or export to the Bitmap it shows some lines as mentioned above image.
What might be the problem?

@jany-tenaj  do you have any idea what is happening with my code?

Are you using the latest code? If not try that first.
Does the line that is to much continue to the right? It looks like it goes through 374 also.
If so I'd guess your border and your contained aren't aligned correctly. But no idea where that problem comes from exactly. Check out whether your can print your map with the demomap print window. If so have a look at the code of that window to find out what you're doing differently.


Observer decorator in TSX throws type error when updating project from 3.3.0 to 3.5.x

I'm working in a code base which has a react class decorated with the observer decorator. In Mobx-react 3.3.0 this worked perfectly, but after installing 3.5.x it throws a type error on compilation with typescript 1.8.10.

Unable to resolve signature of class decorator when called as an expression.
Type 'ComponentClass' is not assignable to type 'void'.

Here's a minimal example which recreates the issue.
@observer class Foo extends React.Component<void,void> { 	render() { 		return (

); 	} }

Quick check, what about React.Component<{}, {}> (that's what I always use)?

Yes, that does work.

Strange. Component state should allow void https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/react/react.d.ts#L16
@Haubach, are you using that type definition for React? Works fine for me.

Yes, that is the react type definition I'm using in the example repo.
I've just verified the behavior on my colleagues machine.

It is unclear to me why void isn't allowed by the type system, could be investigated further but is {} instead of void an acceptable work around? (Personally I would argue that it is more correct as well, props aren't absent, but just empty)

The decision to use void over empty object was taken some time ago where we encountered an issue in another code base, but I don't remember the specific reasoning. For now, this regression is not a serious issue and hopefully it will continue to be a non-issue if you don't intend to continue to support the 3.3.0 behavior.


GetItems throws 500 error. Error code - 103

I've integrated one drive. I'm getting a 500 response error for past two days.
https://skyapi.onedrive.live.com/API/2/GetItems
{"error":{"code":103,"debugMessage":"The given canary was not valid.","isExpected":true,"stackTrace":"6e0d716691d635f42:zVCkWx+WIP/8xepxq6/8c2RH4LQYAsgEb1g+Bh5Z96HZMkpNLJO3U4UCp8LYeXJGS4m2qr7hnEvx8E98Dc/SgYcs+W6eaeUOybmMw4Oos3Im1kqUdodTZpuyCKAQ7ZPq3Q6MHrjOFLky0W/YGX/Mhm0yiWzrlqpZe8Vips78JILwsph8MRSfwlkHCmdBCNSyca15FnSBCJ819k38ZlbRsI/PUQzGzNBFYYmqYQuq7Bws8pmBUrTlY3nW9v3BvXWU/izxjB4LAnNNtigueQs6WIS3z6/ZgQBLxj0JnSKgx/555c3uQp7lQ3vY8idp2HSBRrkiwwB8A4hQ85zP8kmQNy6Tn34zPW5BceFldNN31qBe7ECNa8mI4sl5DHoIeEGRwd/DmCZ4YM70z0N7m2Ht9xtYACSGCQy+Q5KVst4p0hCL9dXe9gP06qfdUyjTp6MX4OhWgve4qJsBlVyM/wxGJQ2ToQjCEfE9MUwFNkUX6Rqq/1xVrUmMbZHOtS8shxhyHuFkMZn63hDYeKxNSQO0VaFFjhIfyTQUC2j+4g0mrm0Os7RLZ8xfnCQBsfkcvGsG5idsIdlR43gL/NHItIZLYEa5NuTOFdwJwczcQCuGr1UYxftEj68q8MSK27yariTciRw/MyS4x7YxEdlJKqb6lUX8RMPVGEEfwE+w/Ow6+9vlFfgmXyFtj3rlf4jwklh3"}}

Hi Santhosini,
It looks like you're trying to use an unsupported private API. To enable this scenario, you need to use the Microsoft Graph APIs, for example List Files.

We are using this library for the file picker - https://js.live.net/v5.0/wl.js
So as a part of it, when I try to use WL.fileDialog it throws a 500 error for the following request -
https://skyapi.onedrive.live.com/API/2/GetItems


Add organoid and spheroid to tissue, cellType to PDX template

Details in 10/22/2021 email chain discussion

LGTM!


"nextra/components"' has no exported member 'Card'.ts(2305)

Following docs:
https://nextra.site/docs/guide/built-ins/cards
import { Cards, Card } from "nextra/components";
But getting:
Module "nextra/components"' has no exported member 'Card'.ts(2305)
It seems there were recent changes on the Cards and I can't see it in the node modules anymore

update to latest nextra

update to latest nextra

I'm on "nextra": "latest"

same error, i installed the latest nextra and i'm getting the same error


TypeError: this.slides[index] is undefined

Hi,
I'm getting the below error:
TypeError: this.slides[index] is undefined animatedLayers = this.slides[ index ].animatedLayers,
I've written a js code to read Slides information form a SharePoint list and build the HTML dynamically. When the page loads, my js file needs to run first to build the HTML. Then the jquery.sliderPro.js should run to build the slideshow.
So I've added the below interval to each ;(function( window, $ )  to check if the HTML is created and ready.
var checkExist = setInterval(function() {    if ($('.sp-slide').length) {          console.log("Exists!");       clearInterval(checkExist);    } }, 100); // check every 100ms
If I put an alert("Show something"); anywhere in the SlidePro.js functions, then all works fine.
If I remove the alert, then I'll get the error.
This is happening because there is  setTimeout() inside loop. when I put alert, browser will wait for user action therefore it works.
Please advise how to fix this issue.
Thanks,
Kate

Any respond?

I've managed to fix this issue!

For future reference, this error is frequent when there are no slides in the slider.
Cheers.

@davidghi
I am facing this error when I have less than 5 slides in the slider.
How to fix this error? I don't want to see the error in console.

How to use slider pro with IONIC?


Unable to open study in worklist; stuck loading

After a while I succeeded in setting up the Orthanc server and meteor server. I uploaded some DICOM images into Orthanc, which I can confirm by going to localhost:8042. However, the Echo test in Orthanc fails. I did not follow the Orthanc tutorial, but the OHIF documentation from this link:
https://github.com/OHIF/Viewers/wiki
I changed the paths in localhostOrthanc.json so that the cornerstone packages are found, and ran the shell script in OHIFViewer/bin/localhostOrthanc.sh
Whenever I run the meteor server, and go to to the worklist, all studies from the Orthanc server are correctly uploaded, including all parameters, except the modality tab. The following error message is shown in the orthanc terminal:
E0217 14:33:47.062190 OrthancInitialization.cpp:520] No modality with symbolic name: null
Did I forget to create a configuration file? Or to modify an existing one? Because all other parameters in my worklist are shown correctly. Also the modality parameter is displayed correctly in the Orthanc DICOM server, but not in the meteor server.
Any help would be greatly appreciated.

Hi there,
I don't think you've done anything wrong.
We have the same issue but haven't spent much time trying to solve it yet. Orthanc's built-in UI page uses their REST API, whereas we/you are using the Orthanc dicomweb plugin to retrieve the data. I'll post here if we find a solution for it.
Cheers,
Erik

Could you try to pull from master again, there were some bugs in there that I fixed around the time of your post.

Thanks for the headsup @swederik !
Something went wrong with cloning the project in Netbeans IDE. I had not made the OHIF clone a netbeans project yet. Apparently therefore I cannot pull the master at the moment. If you are really interested, I can still try to test the OHIF viewer. However, since I learned to make use of Netbeans IDE, it is kind of a necessity to be able to make a netbeans project out of the Viewer clone. So far I had some trouble with that.
I did not make a lot of changes, but I am leaning towards using the chafey cornerstone framework. I need a stable core to build my web application on. I did not know OHIF is still under such heavy development. My plans are to create a webapplication in which you can view DICOM images, perform statistics, enhance communication between multiple type of users and more.
Do you have any suggestions or pros/cons for using OHIF instead of the cornerstone framework?

I just tried to run the application after your changes but I still got an error message inside the viewer.

Cornerstone is a set of libraries for image viewing. What we are building here is an entire application using the Cornerstone libraries (see https://github.com/OHIF/Viewers/wiki).
Yes it is under heavy development. It is functional but many things will change and we don't have much documentation at the moment.
The current master branch is working, though (as far as I know). You can test it here:
http://viewer.ohif.org/
http://lesiontracker.ohif.org
What error did you receive?

I've found out that my proxy is not set up correctly. Because of that I have no CORS enabled and my images won't load. Whenever I try to run the proxy server, I get the following error message (which means that the port is already in use):
Retrieving docker default machine IP address...
./setupLocalOrthanc.sh: line 3: docker-machine: command not found
Copying nodeCORSProxy.example to nodeCORSProxy.js
sed: can't read s/localhost:8042/:8042/g: No such file or directory
Installing node http-proxy from npm...
/home/martin
└──<EMAIL_ADDRESS>npm WARN enoent ENOENT: no such file or directory, open '/home/martin/package.json'
npm WARN martin No description
npm WARN martin No repository field.
npm WARN martin No README data
npm WARN martin No license field.
Running proxy server...
In a separate tab/terminal, go to /main and run ./bin/localhostOrthanc.sh to start the Meteor server
events.js:154
throw er; // Unhandled 'error' event
^
Error: listen EADDRINUSE :::8042
at Object.exports._errnoException (util.js:856:11)
at exports._exceptionWithHostPort (util.js:879:20)
at Server._listen2 (net.js:1227:14)
at listen (net.js:1263:10)
at Server.listen (net.js:1359:5)
at ProxyServer.listen (/home/martin/node_modules/http-proxy/lib/http-proxy/index.js:135:16)
at Object. (/home/martin/NetBeansProjects/DRAsys/Viewers/etc/nodeCORSProxy.js:10:4)
at Module._compile (module.js:413:34)
at Object.Module._extensions..js (module.js:422:10)
at Module.load (module.js:357:32)
This is my proxy script:
// This sets up a proxy to work around CORS.
// See http://chafey.blogspot.be/2014/09/working-around-cors.html
// For more information
var http = require('http'),
httpProxy = require('http-proxy');
var proxy =  httpProxy.createProxyServer({
target: 'http://<IP_ADDRESS>:8042',
auth: 'orthanc:orthanc'
}).listen(8042);
proxy.on('proxyRes', function(proxyReq, req, res, options) {
// add the CORS header to the response
res.setHeader('Access-Control-Allow-Origin', '*');
});
proxy.on('error', function(e) {
// suppress errors
console.log(e);
});
Do you have any suggestions?

Are you running standalone Orthanc or inside Docker?
You can't proxy something on the same port and IP. Those examples were for Orthanc in Docker, and replaces localhost with the docker machine IP. For standalone Orthanc there is a file called OrthancProxy.js which you can run with node to use localhost port 8043 for requests to Orthanc.
Sorry for the confusion, the etc scripts need to be renamed and cleaned up.

Thanks so much, that explains a lot. I have set-up a standalone Orthanc indeed. I'm still trying to get it working using the OrthancProxy. I replaced the .listen(8042) with .listen(8043) and running the proxy in the terminal with: node OrthancProxy.js. The CORS header is still missing when I check the console window in my browser.
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://localhost:8042/.....

Ah yeah again my fault. The Meteor JSON config file should point to 8043 for WADO URI requests, since they need to go through the proxy.

@swederik Thanks a lot! I just got it working with pointing the proxy to port 8043 for WADO URI requests. Now I can finally start building further on this existing framework. Once again, thanks for all your assistance.

Great! Glad to hear it.
The script change is just because you were running the .sh from the bin directory. I should make it more clear that we normally run those from the app's main directory (e.g. LesionTracker or OHIFViewer)

I'll leave the issue open until our Wiki is updated.

The wiki now states to run the scripts from one of the application directories (https://github.com/OHIF/Viewers/wiki/Getting-Started#getting-started).

Hello Experts:
i do have install OHIF following all instructions
Installed Docers , Meteor , And running OHIF
The all study list list are appearing fine even in OHIF study list pages and orthonc webview also have list perfect BUT when i m clicking on image to view in DIcomViewer is just loading and nothing show to me ,
Please guid if i am missing any thing,
i did check my ports also open 8042, 4242, 3000
just stuck from last 2 weeks on same thing and tried my best at every point .
same time i m check the logs there is no error i have attached couple image for more clear idea


BINANCE supports switching between formal and test chains for ease of testing

BINANCE supports switching between formal and test chains for ease of testing

we only support mainnet at this moment


Add a const_parse function to parse a subset of semver at compile time

Would you be interested in a PR adding a const_parse (or similarly named function) which would be able to parse a subset of semver string versions (specifically those with no build/prerelease information) at compile time? This would be a convenience function to allow users to do something like
semver::Version::const_parse(env!("CARGO_PKG_VERSION"));

which could fail to compile, but wouldn't cause runtime errors, meaning they wouldn't have to deal with an additional Result (which I think is quite valuable).
It should also be possible to parse the entire semver version at const time, including the prerelease and build metadata, but that might require restructuring the backend and a memory cost that may be unacceptable for this crate, but I would be interested in adding that as well.

I think I would prefer not to build this into this crate. But if you end up implementing this as a separate crate, I would be interested to see the result.

Hello! I've been messing around with the konst crate and stumbled upon this issue. I tried adapting the existing parse implementation to make it const:
use konst::{option::and_then, parsing::ParseError, slice::get, try_, Parser};
use semver::Version;

pub const fn const_parse_version(s: &str) -> Result<Version, ParseError> {
    let parser = Parser::new(s);
    if parser.is_empty() {
        return Err(parser.into_other_error(&"Empty string"));
    }

    let (major, parser) = try_!(numeric_identifier(parser));
    let parser = try_!(dot(parser));
    let (minor, parser) = try_!(numeric_identifier(parser));
    let parser = try_!(dot(parser));
    let (patch, _parser) = try_!(numeric_identifier(parser));
    Ok(Version::new(major, minor, patch))
}

const fn dot(parser: Parser) -> Result<Parser, ParseError> {
    parser.strip_prefix('.')
}

const fn numeric_identifier(parser: Parser) -> Result<(u64, Parser), ParseError> {
    let mut len = 0;
    let mut value = 0u64;

    let bytes = parser.remainder().as_bytes();
    while let Some(&digit) = get(bytes, len) {
        if digit < b'0' || digit > b'9' {
            break;
        }
        if value == 0 && len > 0 {
            return Err(parser.into_other_error(&"Leading zeros"));
        }
        match and_then!(value.checked_mul(10), |value| value
            .checked_add((digit - b'0') as u64))
        {
            Some(sum) => value = sum,
            None => return Err(parser.into_other_error(&"Overflow")),
        }
        len += 1;
    }

    if len > 0 {
        Ok((value, parser.skip(len)))
    } else if parser.is_empty() {
        Err(parser.into_other_error(&"Unexpected end"))
    } else {
        Err(parser.into_other_error(&"Unexpected character"))
    }
}

I think it works in theory, but because Identifier manually implements drop (something I discovered after implementing this :sweat_smile:), I don't think Version is really usable at all in const contexts.
Just using the existing const constructor runs into the following error:
const VERSION: Version = Version::new(1, 2, 3);
const MAJOR: u64 = VERSION.major;

error[E0493]: destructor of `Version` cannot be evaluated at compile-time
 --> src/lib.rs:2:20
  |
2 | const MAJOR: u64 = VERSION.major;
  |                    ^^^^^^^     - value is dropped here
  |                    |
  |                    the destructor for this type cannot be evaluated in constants

Not sure if there's really any way around this :(

You can change VERSION to static to avoid the "destructor cannot be evaluated at compile time" issue.
static VERSION: Version = Version::new(1, 2, 3);
const MAJOR: u64 = VERSION.major;

(This requires rustc 1.83+)


Support delimiter requests for container listings

Currently, trying to list a ProxyFS "container" through the pfs middleware ignores delimiter query params, causing all objects in the container to be listed.
This would probably be fairly cheap to do for delimiter=/, but the more general case still requires that we walk the whole tree. Maybe it'd be enough for 90% of use cases to support / and 400 any other delimiters?
Fixing this should solve several Swift and Swift3 functional tests (as well as make large volumes manageable for Swift API users).

Changes included in tag <IP_ADDRESS> and merged into development + stable branches.


[Bug] Wrong padded values of nn.pad with pad_mode='reflect'

Expected behavior
nn.pad in the following Relay program should output the same result as its counterpart in NumPy, given input [[1., 1.]]:
def @main(%x: Tensor[(1, 2), float32]) {
  nn.pad(%x, 0, pad_width=[[1, 1], [1, 1]], pad_mode="reflect")
}

Actual behavior
nn.pad in TVM outputs different result from pad in NumPy:
TVM:
 [[ 4.5864499e-41 -7.9918004e-28  4.5864499e-41 -7.9918004e-28]
 [ 1.0000000e+00  1.0000000e+00  1.0000000e+00  1.0000000e+00]
 [ 2.2953549e-40 -7.9916771e-28  2.2953549e-40 -7.9916771e-28]]
NumPy:
 [[1. 1. 1. 1.]
 [1. 1. 1. 1.]
 [1. 1. 1. 1.]]

Padded values in the result given by nn.pad in TVM seem random and differ between two runs.
Environment
macOS 12.4. Compiled using Clang 13.1.6 with LLVM support. TVM commit 8341e33.
Steps to reproduce
import numpy as np
from tvm import relay, transform, cpu, IRModule
from tvm.contrib.graph_executor import GraphModule

x_shape = (1, 2)
x_input = np.ones(x_shape, dtype='float32')
y = relay.nn.pad(relay.var('x', shape=x_shape), [[1, 1], [1, 1]], pad_mode='reflect')
mod = IRModule.from_expr(y)
with transform.PassContext(opt_level=0):
    lib = relay.build(mod, target='llvm')
gmod = GraphModule(lib['default'](cpu()))
gmod.run(x=x_input)
print('TVM:\n', gmod.get_output(0))
print('NumPy:\n', np.pad(x_input, [[1, 1], [1, 1]], mode='reflect'))

I have tried other input shapes. It seems that this bug occurs when any of the shape dimensions is 1. I guess there is something wrong in the TOPI implementation of nn.pad.


add handling for jenkins XSRF

hello
we have a Jenkins with XSRF
I've allowed myself to add a simple example of how to handle the XSRF token to your client.
I would be happy if the code or something similar is included in your client
I've added a few log messages, so you can see what the plugin is doing at work
Please feel free to change my code, it is just a suggestion

Codecov Report

Merging #23 into master will decrease coverage by 1.39%.
The diff coverage is 76.47%.

@@            Coverage Diff            @@
##           master      #23     +/-   ##
=========================================
- Coverage   86.51%   85.12%   -1.4%
=========================================
  Files           2        2
  Lines          89      121     +32
=========================================
+ Hits           77      103     +26
- Misses         10       13      +3
- Partials        2        5      +3

Impacted Files
Coverage Δ

jenkins.go
83.33% <76.47%> (-1.15%)
: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 ead7714...18ff37d. Read the comment docs.


Add new method

Add new method which will allow to set or change a custom alphabet after instantiating the class. This can be useful for dynamically adjusting the encoding.

@microsoft-github-policy-service agree


item_append_file

I found a start to this function and finished it off. If this isn't what you were planning on doing, no worries... I'm happy changing how I implement this later if there's a different way this should be done.

please increment version number (minor increment) and commit it. then we will merge


For comparison operations, check units

When comparing to a value, if it's a quantity, convert to the cube's unit
first.  Needed to be done for all comparison operations.
NOTE that there is an outstanding bug: the "value" placed into the lamda function
does not get sliced when the rest of the SpectralCube gets sliced.

This is now built on top of jybeam since there are some redundant changes.
I don't know what to do about the travis failures, since I can't reproduce them.


tighten security privs

INVENTORY_USER should not have GRANT EXECUTE ON sys.DBMS_SCHEDULER privs

This is required to run the pl/sql version of the inventory service.  Is there another way that you have in mind?

can admin user (or yet another devoted user) do this? not a huge deal I guess but would make security all the tighter.


Cleanup shell for rule 2294

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:
Ref #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/hold until #460 is merged.


Bug when creating leaves for next year

Hi:
When a new leave request is created for the next year, the number of days used for the type is not checked against the max of allowed days of the type.
For example, if I have defined a type named "X" that has a max number of days of 1.
Let's suppose that I ask 1 day of type X for next year and it's granted. When I ask again for 1 day of type X for next year, the system let me ask for it, and it can even be granted.
Best Regards!

Hi @enriquetoledano! Thank you for reporting this issue. I will have a look into it. It definitely a bug.

Fixed and deployed on TimeOff.Management.
@enriquetoledano, thank you again for reporting!


String matching regex doesn't seem to work very well

Same goes for block comments. Those tend to be hard to write in regex, especially without non-greedy parsing.

Turned out I was just typing stuff wrong. For anyone interested, a working regex is:
#[regex = "\"([^\"\\\\]|\\\\.)*\""]


Person Validaton Accept VAT value

Inside the page http://localhost/it/account/address-book/3/edit
as a Person If I try to fill the sylius_address_taxCode with a vat number the validation accepts the data.
I expected that the taxCode was validated from the plugin. Mabye I am missing something?
Best Regards
Fabiano

Hi @fabianobonomini, validation depends also on the selected country. Make sure to select Italy as country. Then the tax code should accept only valid Italian tax code (codice fiscale) and not VAT numbers (partita IVA).

I selected italy. but the bug still there
On Tue, 7 Dec 2021, 08:53 Manuele Menozzi, @.***> wrote:

Hi @fabianobonomini https://github.com/fabianobonomini, validation
depends also on the selected country. Make sure to select Italy as country.
Then the tax code should accept only valid Italian tax code (codice
fiscale) and not VAT numbers (partita IVA).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webgriffe/SyliusItalianInvoiceableOrderPlugin/issues/9#issuecomment-987659625,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV5P4EC2CKX2SX4PR3A3UDTUPW4PJANCNFSM5JQQBEDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sorry @fabianobonomini, you're right and I was wrong. After looking at the code I remembered that we did it intentionally to not introduce too much complexity.
Yes it's true, individual Italian tax code should not accept VAT numbers (but it should for companies) btw we chose to not enforce this validation to keep it simple.
Anyway if you want provide a PR for that we'll be glad to review and merge it.


Matrix permissions on Folder

All in the title

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

Replaced by #407.


HelpWanted: Remove last character then find index of sep string

I do some string parse work.
Remove last character then find index of sep string.
I really want is: Does there have a easy way(less code and better performance) to resolve my problem?
If don't, How can I compare the three way I wrote(both memory used and performance)?
There are what method I found:
https://play.golang.org/p/2H84WYu2Mp
https://play.golang.org/p/JyFWZTtTfe
https://play.golang.org/p/VpHzAZXk6b

Thanks for your time!

Small question: I found there are many method for help []byte and string in strings, bytes.
Why there don't have same methods help []rune?

Close this. Found blog talk about this.
https://blog.golang.org/strings


Add Wi-Fi stats to API

Endpoint:
GET - http://{ip}/api/1/wifi_status
Results:

{"wifi_ssid":"BASE64_REDACTED","wifi_signal_strength":94,"wifi_rssi":-43,"wifi_snr":33,"wifi_connected":true,"wifi_infra_ip":"REDACTED","internet":true,"wifi_mac":"MAC_REDACTED"}

I'd like to contribute this - I've started working on it now and should have a PR soon. Just creating the issue so I don't step overtop of anyone else!

At the time I only added what I found useful to see in Home Assistant. Would WiFi signal strength be useful? Feel free to create a pull request for it here and in the Home Assistant repository as well :)


Create palomon.txt

This Pull Request resolves Issue #
Description:
In this Pull Request we have done the following additions/edits for the code:
Note: Delete all the text including this!

Make sure you replace #<IssueNumber> with the particular Issue that you are referencing with!
Make sure you describe the Issue properly and ask as many people as possible to review your Pull Request so that it can be merged as soon as possible!

Are there any updates here or should we close the PR and label it as spam?

Are there any updates or should we close this PR as spam?


What are the APIs available with HCX and what payload to be sent while calling the APIs?

A. HCX team has published specifications which are available to all and the team must go through it before integration. Please find the specifications at
https://docs.swasth.app/v0.7-draft/what-is-hcx


Feature/optional accounts

This adds the option to not require a username and/or password when new accounts are created. Makes it extra easy for guests to get started--especially if they've already gotten into the party mood 🥂
Admin accounts still always require a password.
Unfortunately this is based on my YouTube feature branch, which had already added a mechanism for allowing certain prefs to be seen by non-admin users.

More awesome work here. With the YT coupling I'm going to go ahead and close this, but definitely would like to see a guest feature!


Method that take params T[] values should have overload that takes IEnumerable values

just saying.

Fair comment, and something we are fixing in our 2.0 branch :+1:


15632 Rename to MultiClusterEngine

Signed-off-by: Zachary Kayyali<EMAIL_ADDRESS>https://github.com/open-cluster-management/backlog/issues/15632

/hold

/unhold


Add a basic REPL

A REPL will be very useful for programmers and is somewhat expected these days. The REPL will have to support lots of features such as support for multiple lines, in-built commands, support to edit previous commands and code, and autocompletion just to name a few. This will probably get its own Kanban board in the Github projects tab. For now, a basic REPL loop should suffice. Additional features can be added as the project grows.

Here is some food for thought: the REPL can probably be created in a different programming language and become separate from the compiler. We could use something like Inquirer.js for the CLI and it might be a much better choice for this task.

Another thought is to use a crate called dialoguer and console to create REPL. The packages look really good. They might even be used to create an installer.


[5.5] Add Route::redirect() method

This adds a new Route::redirect() method that handles simple redirects from one route to another. This implementation works with route caching because it uses an internal RedirectController.
The method accepts three arguments $url, $destination, and $status. The status is optional and defaults to 301.
Route::redirect('contact_us', 'contact', 302);

The implementation is based off my package Laravel route macros with one minor difference, the controller makes use of __invoke instead of a handle method. This is because when writing the test I couldn't find a way to attach the Route to the Request instance, and the __invoke method is handed each "parameter" - for some reason this works with Laravel's test suite but $request->route('parameter') does not 🤷♂ . I can't think of any reason this __invoke approach will cause any problems.
✌

Is it useful to pass along the current query string?

Yes definitely. I did a naive test for this early on but it was hardcoring the query into the destination URL. What do you think about also passing through the headers? Is there a way to create a redirect from an existing request object?

If we needed a route::redirect method to redirect an URL to another, why don't we use that route directly? I don't think this any useful.

Because you may need to keep that old route available for backwards compatibility, but a new route is handling the request. I've had to do this many times with applications that have a long enough shelf life.

If we need to keep that old route available for backwards compatibility, but a new route is handling the request, we simply need to define the two routes. If the old route is to be unavailable, just give out the new route.

@brayniverse what do you mean hard-coding the query into the destination URL?

I tried Route::redirect('hello?name=jane', 'greet') which I realised wasn't evidence that query strings are persisted across redirects. I wasn't ready for this PR to be merged because query strings don't work with this implementation. I could append the results of $request->all() if it's not empty, but it doesn't feel quite right. Is it possible to create a redirect from a request object?

would it be possible to get this added to 5.4 aswell?

@rs-sliske you can back-port this functionality using macros. However, if you don't want to bother with that I've got a [route macro package] (https://github.com/brayniverse/laravel-route-macros) which adds this functionality for Laravel 5.1 onwards.

@brayniverse i realised not long after replying that this wasnt the right solution for my use case, if i need in future i will look into that package (or just use 5.5). thanks :)

If there are multiple routes with the same uri but different http verbs, this applies to all of them. Any chance a way of specifying the desired http verb? Or perhaps a route name?

@joaomnb it'd be quite easy to allow an additional parameter to specify the verb - use self::$verbs as the default. You would need to use addRoute directly and pass the new verb(s) parameter. You'd be able to specify a single verb (as a string) or an array of verbs. Fancy giving a PR a go? I'd be happy to help if you need anything.

I’ve never done it before, but yeah! Seems like a good chance to start 🙂 Thanks!!


Feature/djangotest fixes

fix default test discovery pattern
fix manage.py file check

Wow, I don't know how I missed those. Thanks for the patch!


add icon

Purpose of this pull request
add logo

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Hi, Can you explain in detail what problem this PR wants to solve?

Hi, Can you explain in detail what problem this PR wants to solve?

No, this is not to solve any problem, just want add an icon to IDEA for funny.
after this, when you open the project you can see the icon instead of the S word.

LGTM


KCL doesn't lazy eval dictionary lookups

Bug Report
1. Minimal reproduce step (Required)
schema MySubSchema:
  a: int
  b: str
  c: {str:str}
  d: str = c[b]  // comment this line out to have this succeed.

schema MyTestSchema:
  field1?: [MySubSchema] = [
    {a: 1},
  ]

instance = MyTestSchema{
  field1 =  None
}

This results in an error.
2. What did you expect to see? (Required)
I expect no errors, since I override field1 to None in instance, the templated field should never be evaluated.
If I modify the example slightly, such that I comment out the highlighted line above, it passes, even though there are omitted required fields.
3. What did you see instead (Required)
5 |   d: str = c[b]
  |  'UndefinedType' object is not subscriptable with 'UndefinedType'

4. What is your KCL components version? (Required)
kpm 0.3.6

@steeling Good Catch! 👍 Thank you for your feedback and we will consider optimizing the lazy eval process and provide it in the next version.


Lossless Data Streaming

Love this library! Would like to be able to easily not lose starting or ending keys. This PR adds:

Ability to both stream inner data array items iteratively while keeping track of data read
Use the data read to construct the full dataset when finished
Unit tests for associated function/class
GitHub Actions for running the unit tests

@dyule @danielyule What do you think or the proposed new functions? Is there anything I should change?


Expose script samples to the Sample Solution Gallery

🎯 Aim
The aim of this PR is to prepare script samples in CLI repo so that they will get exposed to the sample solution gallery.
❗Remarks
Whats important is that our work is in sync with @pkbullock changes in script sample repo done in this issue https://github.com/pnp/script-samples/issues/398.
👨💻Todo
What needs to be done:

[x] refactor todo samples
[x] refactor tenant samples
[x] refactor teams amples
[ ] refactor spo samples
[x] refactor pp samples
[x] refactor planner samples
[x] refactor graph samples
[x] refactor flow samples
[x] refactor aad samples
[ ] change some samples names in sample.json so that they won't be duplicated with script samples done in PnP PowerShell and coming to the solution gallery from script sample repo

Track progress on branch
https://github.com/Adam-it/cli-microsoft365/tree/refactor-script-samples-for-sample-gallery

Reference list here: https://github.com/pnp/script-samples/issues/398

Consider adding a CLI friendly no preview images much like the community rocks one. I don't think your samples have images. It could make easier to identify samples in adoption gallery.

Consider adding a CLI friendly no preview images much like the community rocks one. I don't think your samples have images. It could make easier to identify samples in adoption gallery.

Thanks for that very good comment.
I think I already added like a general preview image even to the current single script exposed from CLI which is the Todo cleanup done by Garry but I will for sure double check it 👍


Bug: The game can’t launch with non-empty profiles

After I've installed the mod and set up profiles, Minecraft started to crash when I’m trying to launch it and it says the problem is with CustomHUD.
Here’s the crash report: crash-2021-06-10_14.14.22-client.txt
And one of my profiles: profile2.txt (the other one is the same except for it doesn't have lines 10-15)
Idk if that could happen because of other mods, but before that I’ve messed with the Notes mod. I wanted it to run on 1.17 and I know nothing about mods, so I just changed the version in its fabric.mod.json hoping that would work... Minecraft crashed after I clicked on its settings in Mod Menu, and now I can’t launch the game with CustomHUD. Having or not having Mod Menu and Notes doesn’t seem to change anything though, so that could be a coincidence.

So it appears the problem is with my profiles, everything is fine when they’re empty or with the example from General Structure. At first I thought that’s because of empty sections, I didn't notice but the game crashes even without them. Right now my only profile looks like this:
==Section:TopLeft==
{bx}, {by}, {bz} ({facing} {facing_towards_pn_sign}{facing_towards_xz})
Looking at {{tbx, "{tbx}, {tby}, {tbz}", "..."}}
Biome: {biome}

==Section:TopRight==
Day {day}

and it still doesn’t work.

If you're using the latest version (v1.2.2), downgrade to v1.2.1. I'm about to release the fix as 1.2.3.

Fixed in v1.2.3


Run Portus using docker-compose

This commit adds the possibility to run Portus and its database using docker-compose. This is a different take for issue #81.
I'll add the documentation tomorrow. In the meantime feel free to comment on it.

Cool!
I had to add MYSQL_DATABASE: portus_development to the environment variables for the web container in the docker-compose.yml - then ran docker exec -it portus_web_1 rake db:migrate RAILS_ENV=development which setup the DB.

It is simpler than #116 and I like that!
I see this is for development, right? Otherwise we should think about SSL and secrets configuration and apache+passenger.
DB configuration:  What's the plan there? Should the user add the portus user and the database to the mariadb container manually?
Add "RUN rake db:migrate"
I think you are missing installing docker-distribution ,right?
may it be this line?
RUN apt-get update -qq && apt-get install -y
and then I would also add a
RUN apt-get update && apt-get upgrade
so that there are the latest security fixes
but the idea looks great!

I would say this is pretty sufficient for getting this off the ground for development. Having the ability to specify DB host, user, port, and name  secrets via ENV variables on the Portus container would be a nice next step towards making it closer to production ready.
SSL maybe could be accomplished by adding haproxy, or nginx reverse proxy in front. Otherwise just documenting how to mount in certs (via docker -v)  and providing a config for SSL is fine. Maybe even generate a pair of self signed certs that are used if custom certs are not specified.
By adding the MYSQL_DATABASE ENV variable to the mariadb section of docker-compose.yml it bring the DB setup one step closer.
The rake db:migrate would likely need to be added to an init script for the portus container image if you want to setup the rest automatically. It wouldn't work in a RUN during build of the portus image, as the DB container would not be linked at that time.
Per https://docs.docker.com/articles/dockerfile_best-practices/ doing apt-get upgrade in a Dockerfile is not reccomended.

regarding migrations, we could reuse this
https://github.com/SUSE/Portus/blob/master/config/environments/production.rb#L80
we should make sure thought database is configured when we start rails....
I wasnt' aware of running apt-get upgrade is not recommended but now I understand why so many images in docker hub have insecure packages...
How would you set the ENV variables in production?

I see this is for development, right? Otherwise we should think about SSL and secrets
configuration and apache+passenger.

Yes this is for development purposes right now. We can iterate later over it to make it ready for production.

DB configuration: What's the plan there? Should the user add the portus user and the database to > the mariadb container manually?

The mariadb automatically creates a root user with the password mentioned inside of the docker-compose.yml file. Then the db is setup up manually by the user. This is part of the documentation I'm writing.

Add "RUN rake db:migrate"

We cannot do that because at build time the ports container is not linked against the db one. This has to be run manually. This is part of the doc I'm writing.

I think you are missing installing docker-distribution ,right?

No, that's on purpose. We can take advantage of the official registry container for that. I have to figure out how to add it to the mix.

may it be this line?
RUN apt-get update -qq && apt-get install -y
and then I would also add a
RUN apt-get update && apt-get upgrade
so that there are the latest security fixes

These are leftovers , I'm going to drop them as proposed inside of another comment.

I would say this is pretty sufficient for getting this off the ground for development. Having the ability > to specify DB host, user, port, and name secrets via ENV variables on the Portus container would > be a nice next step towards making it closer to production ready.

Yep, we can focus on that with a dedicated iteration.

By adding the MYSQL_DATABASE ENV variable to the mariadb section of docker-compose.yml it > bring the DB setup one step closer.

What would be the purpose of that?

Per https://docs.docker.com/articles/dockerfile_best-practices/ doing apt-get upgrade in a Dockerfile is not reccomended.

Point taken, I'm going to drop them

I then ran docker exec -it portus_web_1 rake db:migrate RAILS_ENV=development which setup the DB.

You aren't supposed to do that. You should use:
docker-compose run web db:migrate

This is documented inside of the doc I'm currently writing.

Added documentation and asciicast

I missed that there was a db:create available. The MYSQL_DATABASE variable was just how I pre-setup the Portus db in the mariadb container.
docker-compose run web db:migrate makes sense, exec was just how I did it before your docs were available.
So all is good here ;)

Do you imagine that it would be possible also to start the registry:2.0 in docker-compose as well?

LGTM

Do you imagine that it would be possible also to start the registry:2.0 in docker-compose as well?

Yes, I plan to do that with a separate pull request


SeatConsumptionData is empty

I have altered the FloatingLicenseFeatureDetail filter slightly to be able to detect hyphenated names in the user list and I can now see SeatConsumptionData custom object being filled but it never gets added to FloatingFeatureHash and is therefore blank when the script is finished. ($FloatingFeatureHash | ?{$guid -eq $_.guid}) is coming up empty every time.
Any ideas?

@tfrohe, can you show me what your regex looks like now?

<Filter id="FloatingLicenseFeatureDetail">(?:^\s+)([\w'-]+) ([\w'-]+) ([\w'-]+) (\(\w+\.\w+\)) \((?=\S*['-])([\w'-]+)\/(\w+) (\w+)\)\, (start|stop) (.+)</Filter>

@tfrohe I need to see the exact output from lmstat. Can you run the following (modifying as appropriate) and attach the output?
lmstat -c port@host -a > lmstat.txt

lmstat.txt
Please see attached. As we have a large set of features, I have shortened the file to 2 features and replaced all of the server, user and PC names. They all still follow the pattern of the real names. I cannot provide real names unfortunately.


set color dynamically

Hi PitPik!
Your colorpicker is pretty nice and I really like it. But I have a question: how to set a color dynamically? I use 'div' variation.
Thanks!

Hi @MikeSha,
Thanks for participating and your positive comments ;o)
Sorry, but I don't quite understand what you're about to do.
What do you mean by 'dynamically' and what is a 'div' variation?
Cheers
Peter

nevermind :) I've found the solution :)
2015-07-05 15:21 GMT+03:00 Peter Dematté<EMAIL_ADDRESS>
Hi @MikeSha https://github.com/MikeSha,
Thanks for participating and your positive comments ;o)
Sorry, but I don't quite understand what you're about to do.
What do you mean by 'dynamically' and what is a 'div' variation?
Cheers
Peter
—
Reply to this email directly or view it on GitHub
https://github.com/PitPik/tinyColorPicker/issues/27#issuecomment-118617609
.


No script named 'picopayments-cli'

Ubuntu 16.04.1 LTS
Python 3.5.2
Installed both from the source (tests passed) and the CLI (both as root and as a sudoer).

$ picopayments-cli --testnet status
Traceback (most recent call last):
  File "/usr/local/bin/picopayments-cli", line 4, in <module>
    __import__('pkg_resources').run_script('picopayments-cli==0.0.20', 'picopayments-cli')
  File "/usr/local/lib/python3.5/dist-packages/pkg_resources/__init__.py", line 743, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/usr/local/lib/python3.5/dist-packages/pkg_resources/__init__.py", line 1490, in run_script
    raise ResolutionError("No script named %r" % script_name)
pkg_resources.ResolutionError: No script named 'picopayments-cli'

Re-install attempt (after which I still get the above error):
$ sudo pip3 install picopayments-cli
Requirement already satisfied: picopayments-cli in /usr/local/lib/python3.5/dist-packages
Requirement already satisfied: json-rpc>=1.10.3 in /usr/local/lib/python3.5/dist-packages/json_rpc-1.10.3-py3.5.egg (from picopayments-cli)
Requirement already satisfied: micropayment-core>=0.4.3 in /usr/local/lib/python3.5/dist-packages (from picopayments-cli)
Requirement already satisfied: pyelliptic>=1.5.7 in /usr/local/lib/python3.5/dist-packages (from picopayments-cli)
Requirement already satisfied: ecdsa>=0.13 in /usr/local/lib/python3.5/dist-packages (from picopayments-cli)
Requirement already satisfied: pycoin==0.70 in /usr/local/lib/python3.5/dist-packages (from picopayments-cli)
$ sudo pip3 list | grep picopayments-cli
DEPRECATION: The default format will switch to columns in the future. You can use --format=(legacy|columns) (or define a format=(legacy|columns) in your pip.conf under the [list] section) to disable this warning.
picopayments-cli (0.0.20)

Can you try again after upgrading? I fixed one issue but am not sure if it will help you.
sudo pip3 install picopayments-cli --upgrade


Performance issue

Really happy to see clipper lib rebirthed, and some results are obviously better than clipper v1 lib(6.4.2).
However, according to my tests, the performance is about 2 times slower than clipper v1, both clipping, and offsetting(c++).
I just pull the latest code from the main branch (9/24/2022), whether it is normal circumstances in the current clipper v2 lib?

Here is the code of my situation
bool judge_collide_path(const Path64& crect, const Path64& path)
{
Clipper64 c;
c.AddOpenSubject(Paths64({ path }));
c.AddClip(Paths64({ crect }));
Paths64 closed_paths, open_paths;
c.Execute(ClipType::Intersection, FillRule::EvenOdd, closed_paths, open_paths);
return (bool)open_paths.size();
}
bool judge_overlap(const Path64& crect, const Paths64& crects)
{
Clipper64 c;
c.AddSubject(Paths64({ crect }));
c.AddClip(crects);
Paths64 solution;
c.Execute(ClipType::Intersection, FillRule::EvenOdd, solution);
return (bool)solution.size();
}
bool judge_overlap(const Path64& crect, const Path64& prect)
{
Clipper64 c;
c.AddSubject(Paths64({ crect }));
c.AddClip(Paths64({ prect }));
Paths64 solution;
c.Execute(ClipType::Intersection, FillRule::EvenOdd, solution);
return (bool)solution.size();
}

Under some circumstances Clipper2 can be a little slower, though I'd be really surprised if it's twice as slow.
However, when comparisons are done under load (with polygons containing many vertices that create many intersections), Clipper2 way out-performs Clipper1. The library includes benchmark testing (in all 3 languages) which will verify this.
Anyhow, please submit code with data where you're seeing this degraded performance. Cheers.

ok, I will write a test code with code

ok i will write a test code with my data

Test 1
starting Clipper1 ...
4.95 secs
starting Clipper2 ...
2.63 secs

Test 2 - random polygons
Clipper1: 5,000 vertices
44.3 secs
Clipper2: 5,000 vertices
5.41 secs

clipper_compare.zip

clipper_test.zip
here is th .cpp file that test the performance between v2 and v1

Test 1
starting Clipper1 ...
4.95 secs
starting Clipper2 ...
2.63 secs

Test 2 - random polygons
Clipper1: 5,000 vertices
44.3 secs
Clipper2: 5,000 vertices
5.41 secs

clipper_compare.zip

my standalone .cpp file:
clipper_offsetting_test.zip
I see my polygons are much simpler than yours,
And I also found out that clipper2 is much better with very complex polygons.
Whether it is too simple data for clipper2 to perform well?

Honestly I'm mystified as to why you see this as a good test. The 3 paths in the test each simply contain dozens of overlapping collinear segments so you're only testing 180 degree segment joins, and very unlikely to represent a "real world" problem. But I concede that in this test, Clipper1 is about twice as fast as Clipper2.
Nevertheless Clipper2 offsetting will generally be a little slower that Clipper1 (usually by about 20%, as you'll see in the attachment below). I'm not yet exactly sure why, but I suspect it's due to several factors. Firstly, offset 'squaring' in Clipper2 now occurs at exactly delta distance. (In Clipper1 algorithm was faulty though it was also a bit faster.) Also, once the initial offsetting is completed, the paths are then clipped to remove self-intersecting "dog ears". This isn't a complex clipping task so Clipper2's performance improvements under load won't be apparent. And Clipper2 is a little slower that Clipper1 with simpler tasks simply because it does a lot more work removing rounding artefacts from clipping solutions.
offset_compare.zip

ok, thank you for your reply, now this resolves my confusion.

Honestly I'm mystified as to why you see this as a good test. The 3 paths in the test each contain dozens of overlapping collinear segments so you're only testing 180 degree segment joins. So while this is very unlikely to represent a "real world" problem, I concede that in your test, Clipper1 is about twice as fast as Clipper2.
Nevertheless Clipper2 offsetting will generally be a little slower that Clipper1 (usually by about 20%, as you'll see in the attachment below). I'm not yet exactly sure why, but I suspect it's due to several factors. Firstly, offset 'squaring' in Clipper2 now occurs at exactly delta distance. (In Clipper1 the algorithm was faulty though it was a bit faster.) Also, once the initial offsetting is completed, the paths are then clipped to remove self-intersecting "dog ears". This isn't a complex clipping task so Clipper2's performance improvements under load won't be apparent. And Clipper2 is a little slower that Clipper1 with simpler tasks simply because it does a lot more work removing rounding artefacts from clipping solutions.
offset_compare.zip


VAT reverse charge for B2B sales

What I'm trying to achieve
As a merchant I'd like to be able to sell products to EU-based businesses.
This requires not charging VAT if checkout's customer is VAT-registered company from another EU country.
The gist of the feature consists of 3 parts:

ensuring that buyer is a EU-based business (validating a VAT identification number (VATIN));
if they are, deducting VAT from checkout totals;
modifying the invoice to indicate that reverse charged VAT was applied.

Additional info can be found at https://europa.eu/youreurope/business/taxation/vat/cross-border-vat/index_en.htm#withintheeu
Describe a proposed solution
I've chosen to write a custom plugin due to, mainly, limited time (about a month till this feature has to be ready).
However, a custom plugin is not the ideal solution, because this feature had been requested in the past, so, maybe, it's worth to make it part of the Saleor core instead.
Thus, if Saleor team considers this a worthwhile contribution, I'd like to get some pointers on how to approach it and will attempt to make a PR that implements VAT reverse charge in Saleor core, otherwise I'll stick to the custom plugin.
If someone could take a brief look at plugin's code and reply to this issue whether the approach the plugin takes is sensible or not, that'd be greatly appreciated.
Here it is: https://github.com/blender/saleor-vatrc/
The plugin does the following:

adds metadata fields vatrc.vatin and vatrc.vatin_validated to checkout;
attempts to validate metadata field vatrc.vatin once during calculations of checkout and lines totals using VIES;
if validation succeeds, it stores vatrc.vatin_validated for later use;
if vatrc.vatin_validated exists and VAT is included into the prices, it "deducts" the VAT from the checkout and lines totals.

The plugin implements 2 methods:

calculate_checkout_line_total;
calculate_checkout_total.

Additional changes to the invoices
This also requires changes to invoice templates, which aren't part of the custom plugin:

VATIN must be displayed near the billing address;
"Excl. VAT" must be present next to a total;
a note indicating that this is an invoice to which reverse charge was applied should be present.

Other solutions I've tried and won't work
Apps won't work because this requires validating additional bits of billing address and changing checkout/order amounts, which shouldn't be part of any front-end/API flow.
(In project's discussions it was mentioned that custom plugins are no longer recommended, however, again this is not the kind of functionality that fits apps' use cases, I believe.)
Screenshots or mockups

We are working on the dynamic tax calculation webhook that should solve your case which should be released in around two weeks from now
#9526
Apps won't work because this requires validating additional bits of billing address and changing checkout/order amounts, which shouldn't be part of any front-end/API flow.
That is completely up to you if you store such logic as long it is running server-side. Here is an example of how we build apps at Saleor using Vercel and Next.js https://github.com/saleor/saleor-app-template


Anyone can open a chest

If you shift right click with an empty hand on a locked chest, it will open it and you can remove items!

Yes, a patch is in progress ?

Having the same issue on SF 3392

Will the author also fix this bug?

The PR https://github.com/ichorpowered/latch/pull/38 may have fixed this issue, could any of you test this for me?
https://github.com/ichorpowered/latch/releases/download/v0.5.0/latch-0.5.0-lock-fix-S7.1.0.jar

The PR #38 may have fixed this issue, could any of you test this for me?
https://github.com/ichorpowered/latch/releases/download/v0.5.0/latch-0.5.0-lock-fix-S7.1.0.jar

OK。It has been repaired. Thank you

I can't build with gradlew, any updates on this??


docs: Fixed service list command in clustermesh affinity guide

While testing the guide, I discovered that the command to list the
services and preferred backends didn't work because it didn't specify
a namespace. This commit adds -n kube-system to the command so it
works out of the box.

Thanks, I translated all the command manually from my aliases such as ksysex (kubectl exec -n kube-system) and missed some :(


Removal of Dagger-android

Hi Fabio! Congratulations on your repo, it's a really complete project.
I've seen that in April you decided to remove the dependency of dagger-android. I wonder what was the reason, was it due to difficulties in testing with espresso?

Hi, thanks! Glad you like it!
I have removed dagger-android to try to use component dependencies in a multi module project. After the porting (I am using the same approach in some "real" projects as well) I think that using component dependencies is easier when you have to manage multiple components. This post https://proandroiddev.com/using-dagger-in-a-multi-module-project-1e6af8f06ffc is a really good starting point about this subject.

I see your point! Thanks for answering me.
We have a multi-module project that is somewhat large and we're thinking about moving to dagger-android indeed to avoid having to implement a lot of interfaces on the CustomApplication that are used to add the subcomponents to the main component.


QA Org 2gp Does Not Use Managed Configuration

Critical Changes
Changes
Issues Closed
New Metadata
Unpackaged Metadata
Deleted Metadata
Testing Notes
Run the qa_org_2gp flow. This made qa_org intelligently work with unmanaged vs managed.

Merging and testing after merge.


consider documenting contributor expecations, project goals

Thank you for putting in the time to create this library @PoiScript!
I am considering using it in another project but am not sure about your goals for this project. I propose creating a simple contributors guide and perhaps describing your goals for the project.
GithHub's guide has some best practice recommendations that could help here.
Again, thank you!

Thanks for your suggestion! I created docx-rs as a side project for resolving some problems I encountered at work. So it actually don't have any particular goal at first but making my own scripts works. :(
After some thought, to keep the docx-rs projecting evolving, I think the goal may be like providing a strong-typed interface for generating, parsing and modifying docx files.
To achieve this goal, a rewrite is necessary. I do have some cool ideas about that, like storing all xml element into indextree arena to make sure everything laid out in a single Vec (I will probably create a new crate to do so) and keeping a buffer locally to reduce String allocations. However I don't have time to implement that at this present time... So I will keep the issue open for a while (maybe 3-4 months) until I come back to rewrite this.

Certainly @PoiScript! The joy of side projects! Thank you for the update.

providing a strong-typed interface for generating, parsing and modifying docx files

This is an excellent goal. I think it would provide benefit to to users in the Rust community who want to work with docx. One use case I have is to generate docx files from a dotx. From what I have read in the specification, this should be straightforward to implement.

I do have some cool ideas about that, like storing all xml element into indextree arena to make sure everything laid out in a single Vec

Sounds neat. Not sure if it is relevant but I have been using roxmltree to parse document.xml and it has an excellent API.
I'll keep an eye on the project 😄

Sounds neat. Not sure if it is relevant but I have been using roxmltree to parse document.xml and it has an excellent API.

Thank you for the recommendation. I had never heard of roxmltree or xmlparser before. I definitely take a look at.

One use case I have is to generate docx files from a dotx. From what I have read in the specification, this should be straightforward to implement.

Could you provide a link to the specification? It would be best if you can also upload a example dotx file. Thanks in advance!


Improve role idempotence and avoid duplicate object creation at Borgbase.com

I'm currently testing to replace our existing Borg backup role with this one and the repos hosted at Borgbase and some issues I ran into as a new user:

idempotence for cron - fix proposed in PR #54
if you rerun the creation of a repo and it already exists, it should be made consistent with the requested state IMHO (and it that case, you can probably drop the create mode). This appears to be included in PR #53.
Same for SSH keys: seen the fact that there's an API, it should probably be possible to check if an SSH key is already defined on borgbase.com and to avoid to recreate it?

What's your idea about these proposed changes?

Sounds all sensible. I'll look into your PRs and merge them later today.

Sounds all sensible. I'll look into your PRs and merge them later today.

Thanks for the useful changes! I've merged one PR and commented on the other. Just needs minor style adjustments in the Python code.

Thanks for the useful changes! I've merged one PR and commented on the other. Just needs minor style adjustments in the Python code.


Rebuild with private pkgconfig flags

This PR rebuilds the library with a patch that implements lscsoft/lalsuite!696 to prevent overlinking in downstream libraries. I also added extra dependencies for the python library, because I can't figure out how to prevent the overlinking there (lscsoft/lalsuite#165):

use private pkgconfig variables
added missing dependencies
only link GSL libraries we actually use

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@skymoo, any objections to this one?

Non from me, we've made this change upstream but it's not in any release as yet.

Just released a new upstream release so it's probably best rejecting this MR and just updating to the new release as it contains these changes already.


UseSelection and could not load type 'HotChocolate.Utilities.TypeInfo'

I'm trying the example here, just with Npgsql.EntityFrameworkCore.PostgreSQL instead of Microsoft.EntityFrameworkCore.Sqlite:
https://dev.to/michaelstaib/get-started-with-hot-chocolate-and-entity-framework-e9i
This works fine:
public class Query
{
    public IQueryable<Student> GetStudents([Service] FleetFlowDbContext fleetFlowDbContext) =>
        fleetFlowDbContext.Students;
}

However if I add [UseSelection] annotation on top of GetStudents() i get this error when opening Playground:

An unhandled exception has occurred while executing the request.
HotChocolate.SchemaException: For more details look at the Errors property.
Could not load type 'HotChocolate.Utilities.TypeInfo' from assembly 'HotChocolate.Types, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. (ExampleProject.Query)

HotChocolate.* version: "11.0.0-preview.141
HotChocolate.Playground version: 10.5.2

Use selection is not yet supported in 11.
Also, you should not mix dependency verisons :)

I am closing this issue since the question has been answered.