change image starting position to top

hello, how can i change the image position?

Don't really understand your question.

i copy the styles and changed this
 @include object-fit(cover, bottom);

is that the way to go?

Oh now I understand.
Yeah the object fit must be changed in the css.
The styles are relevant for the behavior, to give you as much control as possible.
Thats why you can pass the relevant css classes as props. So you can style them the way you want.
However the masthead class, which contain the image, has always a height of 120%.  The 20% are nessasary for the parallax effect. Thats why you not always see a perfect crop of your image.

great! thank you!


[FEATURE] Adding Squeezenet in timm

Is there a plan for adding Squeezenet as part of timm library?

@nahidalam I was not planning to add that one as it doesn't look very compelling compared to many recen models in that size range, any particular reason?

Just because I was planning to use it and thought loading from timm is generally easier than torch hub

@rwightman can you share some smaller models in that size range, which are suited for embedded devices available in timm models.


[FLINK-5330] [tests] Harden KafkaConsumer08Test to fail reliably with unknown host exception

Using static mocking to reliably fail the InetAddress.getByName call with an UnknowHostException.
Furthermore, the PR decreases the connection timeouts which speeds the test execution up.

Looks good to me, +1

Merging this...


Support for multi-label classification

I am attempting to integrate a multi-label classification model in FAST Pathology, but experience some problems:
Ideally, it would be nice if all classes were listed as classes. Due to the model creating separate outputs, hence the need to create separate process objects and heatmaps, they now appear as layers. In addition, the colour attribute is not kept; they are all initialized as red.
This is the pipeline I am running:
PipelineDescription "Multi task patch-wise image classification trained from ADP dataset"
PipelineInputData WSI "Whole-slide image"
PipelineOutputData heatmap stitcher 0
PipelineOutputData heatmap1 stitcher1 0
PipelineOutputData heatmap2 stitcher2 0
PipelineOutputData heatmap3 stitcher3 0
PipelineOutputData heatmap4 stitcher4 0
PipelineOutputData heatmap5 stitcher5 0
PipelineOutputData heatmap6 stitcher6 0
PipelineOutputData heatmap7 stitcher7 0
PipelineOutputData heatmap8 stitcher8 0

Attribute classes "Tissue"

#"E;C;H;S;A;M;N;G;T"

### Processing chain

ProcessObject tissueSeg TissueSegmentation
Attribute threshold 85
Input 0 WSI

ProcessObject patch PatchGenerator
Attribute patch-size 256 256
Attribute patch-magnification 5
Attribute patch-overlap 0.0
Attribute mask-threshold 0.05
Input 0 WSI
Input 1 tissueSeg 0

ProcessObject network NeuralNetwork
Attribute inference-engine "OpenVINO"
#Attribute scale-factor 0.00392156862
Attribute model "$CURRENT_PATH$/../models/Model_prod.onnx"
Attribute dimension-ordering "channel-first"
Input 0 patch 0
ProcessObject stitcher PatchStitcher
Input 0 network 0
ProcessObject stitcher1 PatchStitcher
Input 0 network 1
ProcessObject stitcher2 PatchStitcher
Input 0 network 2
ProcessObject stitcher3 PatchStitcher
Input 0 network 3
ProcessObject stitcher4 PatchStitcher
Input 0 network 4
ProcessObject stitcher5 PatchStitcher
Input 0 network 5
ProcessObject stitcher6 PatchStitcher
Input 0 network 6
ProcessObject stitcher7 PatchStitcher
Input 0 network 7
ProcessObject stitcher8 PatchStitcher
Input 0 network 8

### Renderers
Renderer imgRenderer ImagePyramidRenderer
Input 0 WSI

Renderer heatmap HeatmapRenderer
Attribute interpolation false
#Attribute hidden-channels 0
#Attribute channel-colors "0" "green" "1" "green" "2" "magenta" "3" "red"
Input 0 stitcher 0

### Renderers
Renderer imgRenderer ImagePyramidRenderer
Input 0 WSI

Renderer heatmap HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "green"
Input 0 stitcher 0

Renderer heatmap1 HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "blue"
Input 0 stitcher1 0

Renderer heatmap2 HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "yellow"
Input 0 stitcher2 0

Renderer heatmap3 HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "black"
Input 0 stitcher3 0

Renderer heatmap4 HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "magenta"
Input 0 stitcher4 0

Renderer heatmap5 HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "white"
Input 0 stitcher5 0

Renderer heatmap6 HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "cyan"
Input 0 stitcher6 0

Renderer heatmap7 HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "brown"
Input 0 stitcher7 0

Renderer heatmap8 HeatmapRenderer
Attribute interpolation false
Attribute channel-colors "0" "red"
Input 0 stitcher8 0

Hmm, I guess this happens because we are unable to set the Attribute classes "Tissue" Attribute at the top of the pipeline for each output, or maybe I am wrong, @smistad?
I believe the results of the individual outputs should be stored on disk, and you should access them in FP, as you have defined 9 separate PipelineOutputData heatmapX stitcher 0 where X in range {0,1,...,8}. But I guess colour information and whatnot will not be stored, and perhaps in the View widget the displayed class names will be the same for all, maybe?

Sounds like what you really need is a tensor concatenation process object, so that you can concatenate all the output tensors from the neural network into one big tensor. Then you would only need 1 patch stitcher, 1 heatmap renderer, and have only 1 pipeline output data, and each channel will them be separate classes.
We could add this to FAST, but it is also very easy for you to just add this to your model. Take your trained model, and a append a concatenation layer at the end, so you end up with only one output. I have done this before with keras, so let me know if you need help to do it.
This will offcourse only work if all output tensors have the same size. Is that the case?

Works as intended now with latest artifact in ubuntu 20.04. Thanks!

Great! All credit to @smistad!
Then I believe this issue can be closed. A new release of FP will likely be published in the next upcoming weeks. Stay tuned!


Proposal: Move from Annotations to CRD

Motivation
Manage Cloudflare configuration outside the service lifecycle.
Current Status
Currently, the service is annotated with several annotations that determine the behaviour of the Service Controller. The controller adds a finalizer to the service preventing it from being deleted until the DNS records are deleted. If #43 is to be implemented, a new set of annotations come in, making it difficult to track all the possibilities.
Implementation
Create a CRD (ServiceTunnel? TunnelBinding like RoleBinding?) that binds the service and the tunnel to be served from. All configurations can be optional fields in the CRD. Finalizers can be on the CRD rather than the service, with an optional service finalizer to prevent deletion. For #43, a CRD (AccessTunnel) can be created that binds the referred service to the generated deployment.

This will also help to link the Tunnel to an existing service like the Ingress Controller's service. Also, when Load Balancer support is added, it will make adding it simple by inserting it right into the CRD.
https://github.com/adyanth/cloudflare-operator/pull/1#issuecomment-1123236806


Release cut 20.2 revised update develop

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

I'm sorry but I had to give up struggling further and further to get around the certificate issue on preview environment, and had to merge this straight on to develop. Otherwise the team would wait another whole day to see develop PRs back green.
:(((((


chore: add PostState::logs_bloom

add function to generate the logsBloom for a block based on all recorded logs

Codecov Report

Merging #2191 (40bcb34) into main (ccf540e) will increase coverage by 0.00%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main    #2191   +/-   ##
=======================================
  Coverage   72.16%   72.16%
=======================================
  Files         460      460
  Lines       55361    55371   +10
=======================================
+ Hits        39950    39958    +8
- Misses      15411    15413    +2

Flag
Coverage Δ

integration-tests
18.84% <0.00%> (-0.04%)
:arrow_down:

unit-tests
67.03% <0.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

crates/storage/provider/src/post_state.rs
90.26% <0.00%> (-0.96%)
:arrow_down:

... and 6 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Improve Scrolling

[x] Implement Scrollbar, as well as expose ScrollbarThemeData to MacosTheme
[x] Implement MacosScrollBehavior, which adds a macos styled Scrollbar to all scroll views when running on desktop

Closes #99
Pre-launch Checklist

[x] I have run dartfmt on all changed files
[x] I have run "optimize/organize imports" on all changed files
[x] I have addressed all analyzer warnings as best I could
[x] I have added/updated relevant documentation
[x] I have incremented the package version as appropriate and updated CHANGELOG.md with my changes
[x] I have run flutter pub publish --dry-run and addressed any warnings

Whoops, I thought I was on a different PR. @bdlukaa did I overwrite your changelog?

@GroovinChip yes

Sorry, go ahead and fix that up


Hex file with Yahboom Smart Robot Car will not build

This library is used for the Yahboom Smart Robot Car, and the program provided for that car uses bluetooth ("Extension bluetooth is incompatible with neopixel. Remove bluetooth and add neopixel?").
Any plans to remove the NeoPixel dependency?
This can be resolved by forking the neopixel and pxt-ws2812b libraries, then removing line 30 ("cpsid i ; disable irq") in sendBuffer.asm

Same here. I will try this to resolve

I have solved this bug！


Unable to install

Both the curl > sh and npx method are failing for me. The following gets stuck on "downloading installer" and never proceeds...
% curl --proto '=https' --tlsv1.2 -sSf https://sh.edgedb.com | sh
info: downloading installer

Whereas I get a Timeout error with npx:
% npx edgedb
No EdgeDB CLI found, downloading CLI package...
node:internal/deps/undici/undici:12344
    Error.captureStackTrace(err, this);
          ^

TypeError: fetch failed
    at node:internal/deps/undici/undici:12344:11
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async getVersionMap (file:///Users/mac/code/learn-anything.xyz/node_modules/edgedb/dist/cli.mjs:219:26)
    at async findPackage (file:///Users/mac/code/learn-anything.xyz/node_modules/edgedb/dist/cli.mjs:209:24)
    at async downloadCliPackage (file:///Users/mac/code/learn-anything.xyz/node_modules/edgedb/dist/cli.mjs:182:20)
    at async getCliLocationFromTempCli (file:///Users/mac/code/learn-anything.xyz/node_modules/edgedb/dist/cli.mjs:148:5)
    at async main (file:///Users/mac/code/learn-anything.xyz/node_modules/edgedb/dist/cli.mjs:42:10)
    at async file:///Users/mac/code/learn-anything.xyz/node_modules/edgedb/dist/cli.mjs:26:1 {
  cause: ConnectTimeoutError: Connect Timeout Error
      at onConnectTimeout (node:internal/deps/undici/undici:7492:28)
      at node:internal/deps/undici/undici:7448:50
      at Immediate._onImmediate (node:internal/deps/undici/undici:7480:13)
      at process.processImmediate (node:internal/timers:478:21) {
    code: 'UND_ERR_CONNECT_TIMEOUT'
  }
}

Node.js v20.11.0

Because neither of these scripts log what they are trying to download and where from, I am unable to confirm if something is broken with my DNS/Internet/SSL etc.

The installer is downloaded from https://packages.edgedb.com/<architecture>/edgedb-cli<extension>.
<extension> is .exe if you're on Windows or nothing elsewhere.
For <architecture>, the script runs uname -s and uname -m to determine the CPU type and OS, normalizes them, and combines them in the format <cpu-type>-<os-type>.
It looks like you're on a Mac, so your URL would be either https://packages.edgedb.com/dist/aarch64-apple-darwin/edgedb-cli if you have an Apple Silicon Mac or https://packages.edgedb.com/dist/x86_64-apple-darwin/edgedb-cli if you have an Intel Mac.
You might check your Mac's network settings and see if the firewall is on. If so, try disabling it and see if that helps.

You can also get more debug information from the npx version by setting DEBUG=* on the command line like:
DEBUG=* npx edgedb

Feel free to share your results here, or if the output is sensitive, feel free to email me at<EMAIL_ADDRESS>

The curl | sh method worked fine when I ran this using a different Internet connection (home v/s work).

The npx installation issue has been fixed in v1.5.7, thanks for the details!


Question: Undo previously suppressed inspection rule

Hi,
I have a problem were I told PyCharm to suppress an inspection rule but later decided I wanted it back. How can I do that? I'm not seeing anything for this in any files or PyCharm's inspection views in preferences.
Specifically the rule is ember/use-brace-expansion
Thanks!

sorry, I don't know how to answer this either. the ember/use-bracee-expansion is an ESLint rul that is unrelated to this IntelliJ/PyCharm plugin. you would have to look at the docs for the ESLint plugin for PyCharm.


ModuleNotFoundError: No module named 'llama_index.core'

[x ] I have checked the documentation and related resources and couldn't resolve my bug.
Describe the bug
Using version 0.2.4 requires llama index as dependency
Ragas version: 0.2.4
Python version: 3.10.13
Code to Reproduce
Any sample code
Error trace
Traceback (most recent call last):
File "/Users/markus/Code/xyz/evaluation/ragas_evaluation.py", line 4, in
from ragas.metrics import (
File "/Users/markus/Code/xyz/.venv/lib/python3.10/site-packages/ragas/init.py", line 2, in
from ragas.evaluation import evaluate
File "/Users/markus/Code/xyz/.venv/lib/python3.10/site-packages/ragas/evaluation.py", line 11, in
from llama_index.core.base.llms.base import BaseLLM as LlamaIndexLLM
ModuleNotFoundError: No module named 'llama_index.core'
Expected behavior
Llama index only as optional dependency
Additional context
Problem introduced in 0.2.4, 0.2.3 works without issues.

Yes! Same problem - you can see in version 0.2.4 it forces you to install LLamaIndex before using evaluation.py
Version 0.2.4
https://github.com/explodinggradients/ragas/blob/v0.2.4/src/ragas/evaluation.py#L11
Version 0.2.3
https://github.com/explodinggradients/ragas/blob/v0.2.3/src/ragas/evaluation.py

hey @sam-h-long @cirezd  that was a bug on our end - this has been removed with v0.2.5
I'm closing this now but let me know if this is still a problem


Remove BoundPods from Kubelet

This PR is changing Kubelet to use Pods instead of BoundPods.
This is partially addressing #5392.
cc @bgrant0607 @vmarmol @dchen1107 @erictune

This is looking good @wojtek-t, what do you think is missing? After this PR we can probably remove BoundPods entirely since they're only used here and you're removing 90% of its uses.

Thanks, this was really quick!
So, does this still accept data from file and http source which is in boundPod format?  It should.
A subsequent PR can make it so that we autodetect whether a file/http source is containerManifest, boundPod or pod.

LGTM.
@erictune Why we still want boundPod format from file and http source?

For backward compatibility of any containerVM users who are using this format?

We never advertise / publish that. ContainerVM users should either on containerManifest or Pod, but I am not sure other channels.

@erictune: I don't - I didn't check it yet.
BTW - do I understand correctly that in file channel we only accept "ContainerMainfest"?

We definitely want to accept pod as well as containerManifest.
Dawn says we can stop accepting boundPod.

Thanks for the feedback! PTAL
I updated this PR and I think it's ready for review.
I've run e2e tests and the result is:
[Fail] Shell [It] tests that guestbook.sh passes
Ran 25 of 27 Specs in 1027.552 seconds
FAIL! -- 24 Passed | 1 Failed | 2 Pending | 0 Skipped F0316 11:54:55.527483    8717 driver.go:87] At least one test failed
However, I think it's not related to my change - it might be related to #5431
I will check if #5452 will fix this problem and update this thread.

BTW - I'm going to completely remove BoundPod type in the subsequent PR.

PTAL

LGTM, but will wait on @erictune's LGTM as well :)

So IIUC, we're waiting for @erictune to LGTM, right?

Yep!

LGTM

Thanks Eric - I'm squashing the commits now.

Commits squashed - all tests are passing.

Merging, thanks @wojtek-t!


feat(sink): sink one-dimensional array data type to postgres and mysql

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Support array data type of primitive types (integer, float/double, varchar), we only support sink one-dimensional array to downstream. We can add support for more field types on demand in the future.
Convert array data type to a string for MySQL (ref), for example ARRAY['Value 1', 'Value 2'] will be converted to Value 1,Value 2

#9919
close #9918
Checklist For Contributors

[x] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Connector (sources & sinks)

Release note

Support Array data type of primitive types (integer, float/double, varchar), we only support sink one-dimensional array to downstream now.
For databases except for Postgres, the Array data type will be converted to a string

Comments are fixed. PTAL again. @fuyufjh @tabVersion
I think we can leverage StringUtils.join() to convert an Array into a String. The test cases have been updated to reflect the changes.


User journey

As a social enterprises who looks for suitable public sector contracts, I should be able to signup and login. I should also be able to fill in all the information required to access my profile then I should be able to browse the result page to see all related contracts available.
Acceptance Criteria:

[ ] the home page include description for the company, how the site works with two buttons for sginup as social enterprises or as public organization, testimonials and supported .
[ ] A social enterprise starts with a signup screen as a supplier with another option to login at the same screen.
[ ] signup should include business information, first name, last name, email address, password and phone number.
[ ] social enterprise should be able to add full information including:

business information.
upload logo.
social impact.
places to work at.
turnover.
policies and governance.

[ ] social enterprise should be able to access his profile after filling his full information.
[ ] profile preview include :

company information .
company description .
places that they can work at.
browse all contracts related.

[ ] social enterprise should be able to see results page with all contracts related.
[ ] social enterprise should be able to search for specific contract .
[ ] social enterprise should be able to apply for any contract he want.

For the sake of clearance, on the acceptance criteria list of yours, you should start each list item with the story/feature/page name.
for example:
 the home page include description for the company, how the site works with two buttons for sginup as social enterprises or as public organization, testimonials and supported .
this should look like:
 Home Page: the home page include description for the company, how the site works with two buttons for sginup as social enterprises or as public organization, testimonials and supported .

ok thank you, I will edit it @sultanassi95

sure,no problem


TS - Adding a barrel, that is, index.ts  rolling up exports into a single file

The Kiota Generation must also generate a index.ts  barrel file which would do the following:

[ ] Export models
[ ] Export the request configuration parameter types/ classes
[ ] Document the process to export

Example: https://github.com/microsoft/kiota/blob/main/http/typescript/fetch/src/index.ts
So the index.ts file would look something like:
export {UserRequestBuilder} from './users/item/userRequestBuilder';
export {Message} from '../../../models/microsoft/graph/message';

The purpose using an index.ts file is that the user of a generated library should be simply able to do a conventional import as follows:
// Example of a `GraphServiceLibrary` use

import {UserRequestBuilder} from "@microsoft/msgraph-sdk-typescript"

instead of (the current case):
import {UserRequestBuilder} from "@microsoft/msgraph-sdk-typescript/users/item/userRequestBuilder"

Thanks to @rkodev who brought this issue to my attention! He faced this issue while Raptor testing. @rkodev Can you please expand on this problem?
A challenge that I can think of while introducing the barrel:

Exporting modules with duplicate names. Example: \mail\users\item\messages\item\messageRequestBuilder.ts  and \mail\users\item\mailFolders\item\messages\item\messageRequestBuilder.ts

thanks for starting a conversation on this.
Kiota already supports doing things like that, we had to implement that for Ruby. It'd only need to be implemented for TypeScript (addition of a CodeNamespaceWriter, and special case for the naming convention in the PathSegmenter).
While I do support this for models (a complete recursive rollup), I'm not sure we should do it for request builders. You stated the naming conflicts, and that would also impact the bundler's ability to treeshake if all the imports in the customer's code are "root imports" on the package (vs selective imports like today).

I think the goal here is to have models being exported so you can use their types, but will people really use RequestBuilders as types? Or use them as ways to get to their data?
As a dev, I'd do something like this :
import { Message } from '@microsoft/msgraph-sdk-typescript`;

var message: Message = await graphClient.Me.Messages.GetAsync();
console.log(message.title);

I don't know if I'd need the MessageRequestBuilder in this case. And if this is the case, I'd do it in very specific cases, not necessarily with all my request builders and then getting the "right" request builder in the hierarchy might make more sense.

There are multiple aspects to that, I think a better example would be sending a message, which today would look something like this
import { Message } from '@microsoft/msgraph-sdk-typescript/models`;
import { SendMailRequestBody } from '@microsoft/msgraph-sdk-typescript/me/sendmail`;
import { MessagesRequestBuilder } from '@microsoft/msgraph-sdk-typescript/me/messages`;

const message = new Message(); //... set all the properties you want
const body = new SendMailRequestBody(); // set other action parameters
body.message = message;

const result = await graphClient.me.sendMail.post(body);

const page1 = await graphClient.me.messages.get();

const page2 = await (new MessagesRequestBuilder(page1.nextLink, requestAdapter)).get();

models are already flat for the most part, and because the classes/enums are one per file, naming conventions get us here without a barrel. The only exception to that would be namespaced models like education, some sharepoint stuff, intune etc where you'd need to index into the namespace under models.
request bodies, when they are not a model straight (component vs inline definition) live next to their request builders. But I feel like those are easy to discover. when you type graphClient.me.sendMail.post, if you f12 on post, it'll take you to the method definition, and you'll be able to discover where the type lives. Plus the package segmentation is really close to the fluent API path.
you will have access to all the request builders through the fluent API path, the only exception to that being iterating on pages/resuming delta. But we can facilitate that through a page iterator (which I believe we already have for the most part in the JS SDK) and without needing a barrel.

For all those reasons I haven't implemented a barrel to date for typescript. It does feel like its necessity is limited to edge cases we have other ways to solve for, and it could come at a serious drawback in terms of package size because of the import style and its implications on tree-shaking.

TS does not support package.json exports - https://github.com/microsoft/TypeScript/issues/33079

Reopening this as #1274 partially fixes this.
This feature needs more discussion and may need more work after considering limitations such as https://github.com/microsoft/TypeScript/issues/33079

Documentation should be available on how to manage exports.

We were under the impression this was already done. Reopening then.

Some of it is done, but I think enums are missing, order in indeterministic, and something else was missing last time I checked

This actually seems connected to the service library which is currently on hold... As we wouldn't need to do this for self-serve libraries. Am I missing something? Or do we want to generate a barrel file for self-serve also? @baywet

generation result is exactly the same between self-serve and service lib. Only the scope of the OpenAPI definition changes and some other parameters (namespace/client name, etc...).
This is something I've been pushing back on: having a "special generation" just for service libs. It would create differences of experiences between the two and increase maintenance burden.

Totally agree that it would be a mess. Now, do we need the barrel in self-serve / kiota-only mode?

I think we do to ease up the import story
without barrels
import { User } from 'graph/models/user';
import { Group } from 'graph/models/group';

with a barrel
import { Group, User } from 'graph/models';

I thought we already had that work. Ok, let's keep it.

We have it for classes/interfaces, but the order is indeterministic which makes reviews harder.
And I think we're missing it for enums

We are leaving it for post-GA for reconsideration. Current state is good enough.

a little update on this one:

reusable models are all grouped in a single index file per "namespace". This is in an effort to both simplify import paths, and reduce the number of modules/imports (to reduce the size)
path segments are now proxies with everything grouped in each path segment's index for the same reasons.

And rolling everything into a single barrel (models and sub-modules, fluent API paths and submodules) would lead to name collisions, so we won't be doing the last part.
Closing.


use MKL and OpenMP to speedup dense softmax

Merged.
Great thanks @tensor-tang!
Looking forward for new PR from you :)


Use multiple selection list boxes instead of panels

The add button is fine, move the remove and edit to contextual menus or key presses.
https://stackoverflow.com/questions/35561011/listboxitem-contextmenu-only-when-listbox-multiple-items-selected


DUPLO-26127 TF: Helm repo and release CRUD support

Overview
Added support for helm repository and helm release resources
Summary of changes
Added support for duplocloud_k8_helm_repository and duplocloud_k8_helm_release resources
This PR does the following:

Implemented CURD operation for duplocloud_k8_helm_repository
Added example for duplocloud_k8_helm_repository
Implemented CURD operation for duplocloud_k8_helm_release
Added example for duplocloud_k8_helm_release
Updated documentation

Testing performed

[ ] Using unit tests
[ ] Manually, on my local system
[✔ ] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-26127 TF: Helm repo and release CRUD support


Add psm2speclib conversion utility

This adds a tools/ directory to the repo, intended to contain conversion tools. Two tools are added here: the conversion tools psm2speclib and speclib2psm along with some documentation in the README.
These are written using Dask for its parallelisation/out-of-core capabilities, so that hopefully large amounts of data can be coped with (we will probably find out rather soon). In principle it makes it easy enough to write both tsv and parquet formats as in issue #97

Tests are passing, mass calculations are validated - and documentation in the README. @tvpham  Perhaps you could give a quick review?
Once this is merged I will refactor further and bring in the output_layer_to_speclib functionality in PR #96

@raar1 it looks good.


SCons cannot work with files containing UTF-8 encoded Unicode codepoints

This issue was originally created at: 2011-03-15 05:10:29.
This issue was reported by: russel.

russel said at 2011-03-15 05:10:29

SCons  HEAD as at 2011-03-15T09:00 cannot handle C and C++ files with UTF-8 encoded Unicode code points in the file name.  Trying to process a file with ï in results in:
scons: building terminated because of errors.
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 18:
ordinal not in range(128):
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Script/Main.py", line 1342:
    _exec_main(parser, values)
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Script/Main.py", line 1306:
    _main(parser)
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Script/Main.py", line 1070:
    nodes = _build_targets(fs, options, targets, target_top)
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Script/Main.py", line 1264:
    jobs.run(postfunc = jobs_postfunc)
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Job.py", line 111:
    self.job.start()
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Job.py", line 218:
    task.postprocess()
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Script/Main.py", line 289:
    tp.display(t)
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Script/Main.py", line 416:
    SCons.Util.print_tree(t, func, prune=self.prune, showtags=s)
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Util.py", line 278:
    print_tree(C, child_func, prune, idx, margin, visited)
  File "/home/Checkouts/Mercurial/SCons/bootstrap/src/engine/SCons/Util.py", line 270:
    sys.stdout.write(''.join(tags + margins + ['+-', rname]) + u'\n')

bdbaddog said at 2018-01-08 07:55:04

Is this resolved by scons 3.0.1 and py3?

Works for me with Python 3.6.3 with scons 3.0.1 on Windows 10.
> ls
Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----       24/02/2018     10:23            345 SConstruct
-a----       24/02/2018     10:21              0 test_ï.c
> cat .\SConstruct
Object(Glob('*.c'))
> scons
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
cl /Fotest_ï.obj /c test_ï.c /nologo
test_ï.c
scons: done building targets.

I suggest marking this issue as resolved/fixed.

@ajf58  - Could you create a test for this?

I'm going to close this, as suggested six years ago.  I've just recreated the test using a source file named mÉ.c and it works just fine.


Sorry, but I can only assist with programming related questions.

VS Code Version: 1.85.1
OS Version: Windows 11
Logs:

Steps to Reproduce:
1.I write a question about programming at any time
2.The answer does not respect the context
3.And if I'm lucky he answers me:
Sorry, but I can only assist with programming related questions.
In almost 50% of the questions
4.All my questions were about programming.
Thanks

how about providing the actual questions or providing logs (ctrl + shift + u) so the team can actually try to fix it, cause now they have nothing.

2023-12-23 17:34:01.033 [info] [chat fetch] url https://api.githubcopilot.com/chat/completions
2023-12-23 17:34:01.033 [info] [chat fetch] modelMaxTokenWindow 4096
2023-12-23 17:34:01.033 [info] [chat fetch] chat model gpt-4
2023-12-23 17:34:02.650 [info] [chat fetch] request.response: [https://api.githubcopilot.com/chat/completions], took 1617 ms
2023-12-23 17:34:05.791 [info] [streamMessages] message 0 returned. finish reason: [stop]
2023-12-23 17:34:05.792 [info] [streamChoices] request done: requestId: [83fa003d-9c20-4320-9555-da340cacef7d] responseId: [714cb302-4103-4e3b-9d68-71166afc26f2] model deployment ID: [x8d3c95500d67]
2023-12-23 17:34:08.481 [info] [chat fetch] url https://api.githubcopilot.com/chat/completions
2023-12-23 17:34:08.481 [info] [chat fetch] modelMaxTokenWindow 4096
2023-12-23 17:34:08.481 [info] [chat fetch] chat model gpt-4
2023-12-23 17:34:08.620 [info] [chat fetch] request.response: [https://api.githubcopilot.com/chat/completions], took 138 ms
2023-12-23 17:34:08.620 [info] [chat fetch] Request ID for failed request: ,77c06402-33fc-4174-bd77-8cf17d869e6a
The question:

Here an example
[cid:989eaaaa-b634-4757-8493-4c0165670d76]
Atte.
Javier A. Rosado Baltazar

De: BulawOw @.>
Enviado: viernes, 22 de diciembre de 2023 23:56
Para: microsoft/vscode-copilot-release @.>
Cc: jrosadob @.>; Author @.>
Asunto: Re: [microsoft/vscode-copilot-release] Sorry, but I can only assist with programming related questions. (Issue #700)
how about providing the actual questions or providing logs (ctrl + shift + u) so the team can actually try to fix it, cause now they have nothing.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-copilot-release/issues/700#issuecomment-1868206326, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJDHG5LSQ26KZFGZDGM35OLYKZP6BAVCNFSM6AAAAABBALC4OSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYGIYDMMZSGY.
You are receiving this because you authored the thread.Message ID: @.***>

Thanks for the logs @bulawow
@jrosadob We need the logs found by doing CMD/CTRL + SHIFT +U -> GitHub Copilot Chat. Conversation Id is just something unique to you.
A screenshot also helps. Images must be uploaded directly to GitHub as they're lost when uploaded via email

We updated the off-topic filter and see a significant drop in false-positive filtering in experiment results.
Closing as fixed, but please don't hesitate to file new issues in case you still run into this.


Svelte Doc: Confusion/error between useMutation and createMutation

Eden documentation regarding mutations and the examples provided inside the Git repository differ.
The left-side bar says createMutation() (which is the valid method for the svelte-side of things) but the code refers to useMutation.
Doc
Appears to be incorrect
https://github.com/ap0nia/eden-query/blob/e45550593389d2968a65233f90e6d6d84729d5f7/documentation/docs/eden-query/svelte/createMutation.md?plain=1#L88
Working demo
https://github.com/ap0nia/eden-query/blob/e45550593389d2968a65233f90e6d6d84729d5f7/examples/eden-svelte-query-basic/src/routes/mutation/%2Bpage.svelte#L6
The React example seems "ok", the doc and the example both refer useMutation method.

Oops, bad copy/paste job. Svelte should have the createMutation hook.


Add production guidance on connection pooling

The CockroachCloud docs make recommendations around connection pooling. We should vet those recommendations, update and expand them if necessary, and expose them in the CockroachDB Production Checklist as well, with adjustments as needed.
There's a general sense from SEs that it's more expensive to open connections than to maintain them and that a larger default pool (e.g., 50 or 100) per app instance may be best.
Once possible factor is password vs. cert-based auth. CC intentionally uses a cpu heavy hash algorithm (bcrypt) to make it difficult to brute force a password match.

Yep! @tim-o and Marc came up with the equation.

Thanks, @Amruta-Ranade. @tim-o, do you remember anything about the process and rationale for that equation?

@tim-o, ping.

@tim-o any background here would be helpful. The AppDev team is planning to design internal tests for max connection usage and connection pool recommendations.

@vy-ton @jseldess we got it from https://github.com/brettwooldridge/HikariCP/wiki/About-Pool-Sizing, Marc Berhault verified it at the time.
The optimal pool size is going to be workload and environment dependent, so we really should encourage people to use the formula as a starting point, and increase the pool size until they can identify the point of diminishing returns.

@vy-ton, I remember seeing you link to a document by @a-entin with different guidance on connection pooling. Do you know what I mean and, if so, can you add that here?

@vy-ton, can you provide a link to the source for more context?
In @a-entin's vmware doc, he also says:

The True concurrency-to-vCPU ratio should not exceed 4. Users are encouraged to implement connection pooling or similar techniques to ensure the number of always-active connections per CockroachDB node does not exceed 4 times vCPU count.

Additional notes from @a-entin:

“True” Concurrency is a characteristic of a workload where database clients open connections and execute SQL statements back-to-back, as in a stress test

As opposed to connections with a significant share of idle time, such as in interactive user connections, for example

Alternatively, “true” concurrency may be referred to as the number of "always-active connections" or as the "number of actively executing statements".

The True concurrency-to-vCPU ratio should not exceed 4. Users are encouraged to implement connection pooling or similar techniques to ensure the number of always-active connections per CockroachDB node does not exceed 4 times vCPU count.

The ratios provided are recommendations, not hard requirements. Experienced users with insightful understanding of their real-world workload should take these recommendations as a starting point and adjust them to best fit their application’s specific workloads. Development environments, for example, may or may not be provisioned with the same ratios as a production environment.

The Sessions UI page being documented with #8218 is relevant here, as it displays active connections (sessions).

@ianjevans, assigning this to you since you covered connection pooling in the new dev guide. Can you think about corresponding updates to the prod checklists?


No APP icon

Environment

OS: Windows
scrcpy version: 1.12.1
installation method: Windows release
device model: Redmi K20 PRO (Mi 9T PRO)
Android version: 10 (MIUI11)

Describe the bug
If I type ctrl+o, My app icon will disappear when my phone screen is turned off

Environment
OS: Windows, Ubuntu 1804
scrcpy version: 1.12.1
installation method: Windows release, snap
device model: Redmi K20 PRO (Mi 9T PRO)
Android version: 10 (MIUI11)

This problem also appears in my ubuntu system.

Which app icon? You mean scrcpy closes itself?
Please do the following steps:

clean your logcat: adb logcat -c
run scrcpy from the console
press ctrl+o to produce your problem
run adb logcat -d > logcat.txt

Then:

paste the output from the console here (between ```)
post your logcat.txt

Ah no ok it makes your launcher crash. Post your logcat.txt to confirm.

I'm sorry I didn't make my problem clear. There are bug videos and logcat.txt in this zip.
Bug_record.zip

If you switch the screen on again (Power off then power on), are the icons back?
Might be the same problem as #836 for chrome (the app does not draw if it thinks there is no display).

Yes, If the screen is on, I can see the icons.
So, I think scrcpy cannot solved this issue.


vendor:publish lack of error message

Laravel Version: 5.5.3
PHP Version: 7.1
Database Driver & Version:

Description:
php artisan vendor:publish  --provider=\foo\bar
Does not return any error message if the provider name is incorrect.
Steps To Reproduce:

This repo is for bug tracking. This isn't a bug so you can open a proposal in the laravel/internals repo

Yeh, the behaviour here is that is successfully published no resources.


Add debounce to search bar

The search bar currently sends a request after each keystroke. The request should be sent after the user stops typing for 1-2 seconds.

14677d4
Added button to confirm search instead. Might revisit later.


date-of-money to change text of label instead of replacing div

Make the date of money setting not replace the entire div but instead change the inner text of the label. I did this because the current styling of the replaced div does not take all the space and makes the mouseout event triggering unreliable.

I also considered not touching the div contents at all and just make the date of money a title attribute, to hover and find out.

Can you also make this apply to the date of buffering feature?

Went for #2427


Mego: Watch istikhara

Mego requests the watch of the watch_keyword istikhara. See the Metasmoke search here and the Stack Exchange search here.

istikhara has been seen in 3 true positives, 0 false positives, and 0 NAAs.

Coverage remained the same at 65.71% when pulling 4f1c66fd344757967b0c09cc43766920f02a516c on auto-blacklist-1518981443.198078 into 13aef79780c9b411be7e65e476e302b912d995f0 on master.

!!/approve


Receive "TypeError: QRCode is not a constructor"

var qr = new QRCode(typeNumber, correction, inputMode);
located angualr-qr.js line 121:

I am experiencing this as well.

Anyone having an issue with angular-qr; if you pass lowercase sting to QRCode(typeNumber, correction, inputMode); it will fail because the regex at angular-qr.js line 34:
    $scope.isALPHA_NUM = function(text){       var ALLOWEDCHARS = /^[0-9A-Z $%*+\-./:]*$/;       return ALLOWEDCHARS.test(text);     };
and it should be:
var ALLOWEDCHARS = /^[a-zA-Z0-9 $%*+\-./:]*$/;

Anyone having an issue with angular-qr; if you pass lowercase sting to QRCode(typeNumber, correction, inputMode); it will fail because the regex at angular-qr.js line 34:
    $scope.isALPHA_NUM = function(text){       var ALLOWEDCHARS = /^[0-9A-Z $%*+\-./:]*$/;       return ALLOWEDCHARS.test(text);     };
and it should be:
var ALLOWEDCHARS = /^[a-zA-Z0-9 $%*+\-./:]*$/;


Removing timeout settings that are set but never used

Coverage decreased (-0.008%) to 80.933% when pulling 730394c03dbeb0ffedebf3e2b50b16acbfc07887 on garrettjonesgoogle:alt into b6a87b0deb4dd2aaebaf528d9225c1927b5bb475 on GoogleCloudPlatform:master.

LGTM - did these used to be used? Or have they never been used at all?

Yes, they used to be used, but @vam-google 's PR https://github.com/GoogleCloudPlatform/google-cloud-java/pull/1738 stopped using them after RetrySettings was introduced. Now, going through RetrySettings (which can be set on ServiceOptions) is the correct thing to do.


LRA-415-art-survey-add-users-roles-policy

This is the whole tuna for authorization.
There are checks for what access a user has and what is displayed based on department roles.

When a user logs in they are authenticated via Shibboleth
Upon success they have session variable set that is labelled :uniqname
There are various methods in application_controller that will take the user's uniqname session variable return appropriate data items depending on the authorization that is required on a controller or a variable list.
The controllers that govern Role and Permission are specifically locked down to SuperUser status
Access controller can be a used by SuperUser or Department Administrator only
ArtItem controller is accessible by anyone who has an access in the system
The StaticPage#home is not protected and allows public viewing of the home page and it's message

The ROLE / PERMISSION / ACCESS bits

A role is set by a SuperUser to describe currently one of three types

SuperUser - see-er / editor /deleter of all things

Department Administrator - see-er / editor /deleter of department specific related items

Recorder - See-er / editor of department specific related items

A permission is matching a role to a department ie. Dean's - Recorder
Accesses are matching a uniqname to a permission

Since the system is s-o-o-o-o-o-o-o (too) complicated, it would be nice if you add some description. That would help to test the process. Please describe the authorization process.

See edited PR 'initial' comment

Let's start from the beginning. On my localhost, the roles table has three roles, permissions and accesses tables are empty.
I can login; I see no links.
But I can go directly to http://localhost:3000/art_items - I see no record, but the button [New art item] is available. I can create a record for any department. I can see the record.
I can access and update:

http://localhost:3000/page_informations)

The following URLs give different errors:

http://localhost:3000/annotations,
http://localhost:3000/appraisal_types
http://localhost:3000/departments

Not authorized (good!):

http://localhost:3000/accesses
http://localhost:3000/permissions
http://localhost:3000/roles

Can access and update without a specific permission

http://localhost:3000/page_informations)

The following URLs give different errors:

http://localhost:3000/annotations,
http://localhost:3000/appraisal_types
http://localhost:3000/departments

I updated the before actions to the ones that actually will do a proper √ for access approval.

How can I make my account a superuser? In rails console?
Right now without any permissions I can create a new art_item and see show pages.

I created Access records for my account.

If I go to http://localhost:3000/accesses I see my account and I can create a new record. But in both cases, I get a flash message 'Not Authorized."
The same is true for http://localhost:3000/permissions.
Is the super_user_department_admin_access_authorized! method correct? It displays 'Not Authorized' when it gives access or denies access. I created an um171712 account with recorder permission and for this account, authorization works correctly.
seed.rb file has errors (both syntax and logical). Do you want me to fix them? Or describe them here?

I created Access records for my account.

If I go to http://localhost:3000/accesses I see my account and I can create a new record. But in both cases, I get a flash message 'Not Authorized."
The same is true for http://localhost:3000/permissions.
Is the super_user_department_admin_access_authorized! method correct? It displays 'Not Authorized' when it gives access or denies access. I created an um171712 account with recorder permission and for this account, authorization works correctly.
seed.rb file has errors (both syntax and logical). Do you want me to fix them? Or describe them here?

Fixed the notification message and updated the seeds.rb

In the Department controller in the create method, when a new department is created permissions for this department should be created at the same time, I think.

Not necessarily

Not necessarily

Of course, not, but then after creating a department you have to create permissions for that department. Well, you'll take care of that in the admin interface ... which will be as complicated as the ROLE / PERMISSION / ACCESS process.

Please share your table structure so I can see what you have configured locally

Role.all.pluck(:id, :title)
  Role Pluck (0.5ms)  SELECT "roles"."id", "roles"."title" FROM "roles"
=> [[4, "SuperUser"], [5, "Department Administrator"], [6, "Recorder"]]

Permission.all.pluck(:role_id, :department_id)
  Permission Pluck (0.5ms)  SELECT "permissions"."role_id", "permissions"."department_id" FROM "permissions"
=> [[4, 26], [6, 26]]

Access.all.pluck(:permission_id, :uniqname)
  Access Pluck (0.6ms)  SELECT "accesses"."permission_id", "accesses"."uniqname" FROM "accesses"
=> [[1, "brita"], [1, "rsmoke"]]

User.all.pluck(:id, :uniqname)
  User Pluck (0.4ms)  SELECT "users"."id", "users"."uniqname" FROM "users"
=> [[2, "brita"], [3, "um171712"]]

User um171712 can login and go to http://localhost:3000/art_items, see [New art item] button, and go directly to http://localhost:3000/art_items/11

User um171712 can login and go to http://localhost:3000/art_items, see [New art item] button, and go directly to http://localhost:3000/art_items/11

I was expecting the result of the before_action :access_authorized! to be the equivalent of:
    if access_authorized!
      @new_annotation = Annotation.new(art_item: @art_item)
      @annotations = @art_item.annotations.order("created_at DESC")
    else
      redirect_to root_path
      flash.alert = "Not Authorized."
    end

Here is a good example where that did not work and a test would have captured the problem

@britaumich I refactored the access over the art_item controller to prevent unauthorized miscreants

General question:
@rsmoke  - Since "Super User" is not tied to any department, I think we might have to set some dummy department id value for its entry in the Permissions table. But this opens up the possibility of have multiple "Super User" entries in the permissions table which we don't want (  Since Permissions table is acting as a reference table ). Is there a way to limit the "Super User" entry in the Permissions table to 1 ?

I know you did not ask for my review again, but since I get emails I tested the branch.
If I log in as a recorder (or a super user) and try to create a new item, I get the following error (is @departments_list correct?)

To fix the @department_list error,
In the art_items_controller replace the set_departments_list method (lines 95-97) with
def set_departments_list
      @departments_list = current_user_departments
end

By the way, why do we need to reassign current_user_departments to another variable? Could you use current_user_departments in the art_item form instead of @departments_list? It works.

This is true but I like having the instance variable in the form

This is true but I like having the instance variable in the form

I agree, but then maybe you don't need [helper_method :current_user_departments] in the application controller which makes the current_user_departments method available in views.

I tested what is allowed for Recorders and Department Administrators. They both have access only to their departments, nothing else. They both have access to the archive button which is doing nothing right now and there is no delete button. We can define permissions to delete/archive later when we create the delete/archive process.

Similar to dependent: :destroy for :accesses in permission model, I think we might need it for department and role model as well. For example - If a certain department is removed, then we want all its permissions and accesses to be removed. Your thoughts ?

@anantaUM I added the dependent: :destroy on the has_many relationships. I also refactored the authorization to DRY it up and for readability

Everything seems to be working (different roles and permissions).
I don't think I like
class Department < ApplicationRecord
  has_many :art_items, dependent: :destroy

We still don't know what we are doing with deleting/archiving art items. But if an art_item is in the system it should not be deleted with a department automatically. It might be moved to a new owner or archived or something else. But not deleted.

Ananta wanted to smack his peepers on it one more time


Outgoing artifacts as provider of a task output cause the failure

Expected Behavior

Using the output property of a task as outgoing artifacts is allowed. For example:
def jarTask = tasks.register('jar', Jar)

def config = configuration.create('potato') { /* configure as consumable */ }
config.outgoing.artifact(jarTask.flatMap { it.archiveFile })

Current Behavior

Using such a task output provider will cause a crash because of the "query provider before task executes" check. The Configuration seems to be queried internally for the artifact file which happens to be done before the task graph is computed or executed. There are two ways to work around it which simply hide the issue: 1) create a temporary provider to fool the check or, in this particular case, use the TaskProvider<Jar> instead of a Provider<RegularFile> as internally archive task have special handling. Both solution are pretty bad as they rely on internal behavior of Gradle.
Context

Similar but not exactly this other issue: https://github.com/gradle/gradle/issues/15146
Steps to Reproduce

Your Environment

Build scan URL:

Just wondering, @lacasseio , do you think it is a duplicate of https://github.com/gradle/gradle/issues/16777#issuecomment-817303964 ?

It's close but not exactly the same. If I'm not mistaken,. it's possible to add a provider as artifact on a consumable configuration and if that provide has task dependencies, it will be pickup properly. The issue here is if the provider in question refers to a task output then the check will trigger because the provider is queried to figure out what's kind of provider it is. Creating a provider with task dependency that is not a task output if tricky. Archives task (zip, jar, tar, etc) have a special handling mostly for legacy reason where you can just pass the task provider and it will work. In the past, I did work around by creating an archive task just so I could defer the resolution but it's assume the final file is an archive file. 🙃

Please add the stacktrace :)

I believe this is still valid.

Can you provide a full reproducer for this? I have attempted to reproduce but this, but it seems to be working fine for me:
    def "test"() {
        given:
        settingsFile << "include 'producer'"

        file("producer/src/Foo.txt") << "Hello!"
        file("producer/build.gradle") << """
            plugins {
                id("base")
            }

            def jarTask = tasks.register('jar', Jar) {
                from(layout.projectDirectory.dir("src"))
            }

            configurations {
                consumable("potato") {
                    outgoing {
                        artifact(jarTask.flatMap { it.archiveFile })
                    }
                    attributes {
                        attribute(Category.CATEGORY_ATTRIBUTE, objects.named(Category, "foo"))
                    }
                }
            }
        """

        buildFile << """
            configurations {
                dependencyScope("conf")
                resolvable("res") {
                    extendsFrom conf
                    attributes {
                        attribute(Category.CATEGORY_ATTRIBUTE, objects.named(Category, "foo"))
                    }
                }
            }

            dependencies {
                conf project(":producer")
            }

            task copy(type: Copy) {
                dependsOn configurations.res
                from {
                    zipTree(configurations.res.incoming.files.singleFile)
                }
                into layout.buildDirectory.dir("unzip")
            }

            task verify {
                dependsOn tasks.copy
                doLast {
                    assert file("build/unzip/Foo.txt").exists()
                    assert file("build/unzip/Foo.txt").text == "Hello!"
                }
            }
        """

        expect:
        succeeds ":verify"
    }

I've also tried to reproduce this in a number of ways and using various versions of Gradle and I've not been able to recreate the behavior.  I suspect there's some detail missing with respect to what will trigger the behavior, but I haven't been able to find it.  I'm closing this for now, but happy to revisit if there's a good reproducer available.


[AIRFLOW-1182] & [AIRFLOW-1184]: Contrib Spark Submit operator should template fields & Contrib Spark Submit Hook does not split argument and argument value

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following

Airflow-1182 Contrib Spark Submit operator should template fields
Airflow-1184 Contrib Spark Submit Hook does not split argument and argument value

Description

[ ] Here are some details about my PR, :

added template fields to the SparkSubmitOperator : _application_args & _name making them usable with macro. This enable to pass params based on dag execution to the Spark application
split spark application options as they should be for Popen in spark_submit_hook. Without, the spark application is complaining about missing & malformed option. See jira issu 1184

Tests

[ ] My PR adds the following unit tests :

refactored tests for spark_submit_operator & spark_submit_hook.

Codecov Report

Merging #2289 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #2289   +/-   ##
=======================================
  Coverage   68.87%   68.87%
=======================================
  Files         142      142
  Lines       10969    10969
=======================================
  Hits         7555     7555
  Misses       3414     3414

Impacted Files
Coverage Δ

airflow/settings.py
88.88% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

This PR contains #2282 plus additional Jira  1184

Can you please cleanup your commit message (see guidelines) and see if you need to split the patch across two commits that address the jira's separately?

Is there any way to re trigger the travis build without a new push ?
the error is
11) FAIL: test_scheduler_run_duration (tests.SchedulerJobTest) (11s vs 5s)

and build is ok on my branch :
https://travis-ci.org/vfoucault/incubator-airflow ->bid 8 for same commit number (e60aab8)
thx.

You could try to do a force push of the same commit, this might trigger a new build.

No effect.  I'll go with a empty commit then

I think it is not a problem, the committers are aware of the fact that the tests are sometimes a bit flakey.

I think we're now, let me know.
kr.

@Fokko tests are not flaky any more. So no we don't accept non passing tests anymore. Exception could be due to changes to underlying dependencies (i.e. a version has changed).
LGTM

I think this needs reverting as it impossible to include an argument with an embedded space!
I don't think this is needed:
application="com.example.MyClass"
args=[
  "--foo=bar",
 "--qux=foo bar",
]
task = SparkSubmitOperator(
    task_id="spark_csv_parser",
    dag=dag,
    application="com-example-spark.jar",
    application_args=args,
    java_class="com.example.MyClass",
)

Previously to this PR this would do the equivalent of:
popen([
  "spark-submit",
  ...,
  "com-example-spark.jar",
  "com.example.MyClass",
  "--foo=bar",
  "--qux=foo bar"
])

But after this commit it drastically changes the last argument:
popen([
  "spark-submit",
  ...,
  "com-example-spark.jar",
  "com.example.MyClass",
  "--foo=bar",
  "--qux=foo",
  "bar"
])


Idea/Suggestion: Adds support to sync custom files

It would be a great idea if we can not only sync VS Code related files, but also custom files provided by the user via file path. This will be really useful specially for linters configuration files.
Right now you have a feature called Customized Sync but is intended just to remove files from syncing right? I propose the other way round, adding custom files to be sync, so that we can deploy them in any other machine. In order to be useful, user should have to indicate where to download this custom files in each machine.

Please give a proper scenario for your case along with files names and their path

Sure, lets consider a bunch of linters typically installed in VSCode through extensions: ESLint, StyleLint and HTMLHint for example, and also the widely used Beautify extension. All of them are customizable via configuration files:
.eslintrc
.stylelintrc
.htmlhintrc
.jsbeautifyrc

They could be located anywhere actually, cause linters will find for them recursively upwards in the folder tree. In my case, in order to keep common settings for all my project, I have them saved in
C:\dev\
Which is the root folder for all my development projects.
It would be nice to be able to tell Sync to consider these files as well, along with the VSCode setting files. So that I can synchronize them in different machines and different paths. Just to put an example:
MachineA: C:\dev\.eslintrc synched with MachineB: D:\Users\[username]\code\.eslintrc
Maybe paths should be stored in a new file (e.g.: customFileMapping) and also uploaded to Gist. This file will contain the different paths for each file in each machine.
The files provided in this scenario are just examples. The idea would be to allow whatever file (maybe you can apply restrictions if you desire, only text files or light files) in whatever path.

It would also be nice to be able to save launch configurations.
The launch file is in .vscode/launch.json.
It looks like this:
{
    // Use IntelliSense to learn about possible Node.js debug attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "node",
            "request": "launch",
            "name": "Debug Nightwatch (grunt)",
            "program": "/usr/local/bin/grunt",
            "args": ["tests:integration:chrome"],
            "cwd": "${workspaceRoot}",
            "stopOnEntry": false
        }
    ]
}```
Mote info there:
https://code.visualstudio.com/docs/editor/debugging#_launch-configurations

There would be something like syncCustomFiles with their name and path.
When users upload the files it will upload the files from the specific path and upon download it will replace those files in their path.

I think there's some difference to note betweetn vscode files (such as tasks.json, that is generated by the editor whenever you save a task) and custom per project/per language files such as .eslintrc.
Editor files can be saved by default if present, custom files would need a list provided by the user as ti which files they want to save.

Another great candidate is being able to sync theme files generated from https://github.com/be5invis/vscode-custom-css. In my instance I'm storing my theme under ~/.vscode/themes/theme.css; being able to sync that folder, or files within the folder, would be nice.

As a workaround for launch.json VSCode also supports storing launch configurations in it's preferences under "launch" key:
"launch": {
	"configurations": [ { "type": "node", ... } ]
}

I'd use this to sync my .code-workspace files (as discussed in https://github.com/shanalikhan/code-settings-sync/issues/460 )

It should work like:
customFiles:
[
"fileName1" : "pathOfFile",
"fileName2" : "/home/project/setting.json"
]

@boostio funded this issue with $60. Visit this issue on Issuehunt

@shanalikhan I'd like to contribute with this one. This extension is a must have for me since I have to use newly installed machines frequently (linux and windows). I use a lot of dot files configs.
I was thinking, once the file was downloaded it should be a command like >sync import $file_name$. For instance, I want to sync a frequently used .babelrc and use it in different projects, so I only have to mark that file in the sync settings and then import it wherever I want through the command palette.
What do you think? Please tell me your ideas and I will start to work on this as soon as I can

once the file was downloaded it should be a command like >sync import $file_name$. Please tell me your ideas and I will start to work on this as soon as I can

There is already issue opened #403  but code doesnt allow extensions to register commands CLI.
What you we do for to add something like this.
customFiles:
[
"babelrc" : "path/of/babelrc",
"fileName2" : "/home/project/setting.json"
]

When user uploads Settings Sync needs to upload file in path path/of/babelrc with name babelrc in GIST.
On downloading the file, Settings Sync will read the  babelrc and put in path/of/babelrc.

once the file was downloaded it should be a command like >sync import $file_name$

Once it is done, we can have new command in Advance Settings of Settings Sync Import File to workspace. That path/of/babelrc will be copied to the current workspace when user select this file from the list in visual studio code.

@shanalikhan I meant the command pallet (ctrl + shift + p in linux). Something similar to

What do you think? Since the imported file will be needed when you are in vscode more than importing it through CLI.
I will be working on this this weekend (tough week at work)

@tkrtmy has started working. Visit this issue on Issuehunt

@tkrtmy has submitted output. Visit this issue on Issuehunt

As @tkrtmy has send the initial PR with further work  required.
The details are mentioned at https://github.com/shanalikhan/code-settings-sync/pull/666#issuecomment-427581660
@b1tdust

@shanalikhan I fixed this issue. What should I do to get the bounty at issuehunt..?

i will transfer in this week.

v3.2 is released with this feature. Let me know if there is any problem

@shanalikhan has rewarded. Visit this issue on Issuehunt

Sorry to jump on a closed thread, but how is this supposed to work exactly? I have added a couple of custom files on one machine, I see the relevant lines added to syncLocalSettings.json and the files are uploaded to my private Gist, but then when I sync/download on a second machine syncLocalSettings.json does not get updated therefore the second machine does not know to download the two custom files (I don't see them created in the destination folder, even if the folder exists)

@carlocardella
syncLocalSettings.json is not synced. it is edited only local.
if you'd like to download & sync custom file in also other machine, you should decide file location and use >Sync : Add Custom Sync File path  option to specify absolute path on the second machine too.
After that, you can custom file download from gist to the second machine.

PUBLIC WIKI :
https://github.com/shanalikhan/code-settings-sync/wiki/Custom-Sync
Feel free to imprve this page where required.

Thanks, I figured that after a few tests but my initial guess (admittedly without fully reading the new docs) was that since the extension already synchronizes a number of settings (and does an excellent job at it, it saved me a few times already) and seeing the custom files are uploaded to the same Gist used for other settings, downloading on a new machine would have downloaded the custom files as well.
Sounds like a nice new feature/improvement request then 😊

I clean-installed VSCode, of course syncLocalSettings.json got deleted.  Since that file was missing , all paths of my custom files were gone. Then I thought it would be a good idea to sync syncLocalSettings.json itself as customFiles.
I imported it separately after recovering essential files.  Just an idea.


Update C-NEWTYPE-HIDE now impl Trait is shipped

I remember your comments on Reddit about it, now. 😄
WDYT?

LGTM. cc @dtolnay


Upgrade jackson to version 2.13

Closes #2412

Don't forget to update the JsonProvider in FrankFrameworkApiContext

I just did a search-in-project for com.fasterxml there are 4 hits in core/tests


Response body is empty on unauthorised responses

Expected Behavior
When creating a custom authentication handler the failure message is not returned in the 401 response from Ocelot. I would expect this failure message to be in the response body so that the API consumer could get a hint of why the authentication failed.
The authentication handler
public class MyAuthenticationHandler : AuthenticationHandler<AuthenticationSchemeOptions>
{
    public MyAuthenticationHandler(IOptionsMonitor<AuthenticationSchemeOptions> options, ILoggerFactory logger, UrlEncoder encoder, ISystemClock clock)
        : base(options, logger, encoder, clock)
    {
    }

    protected override async Task<AuthenticateResult> HandleAuthenticateAsync()
    {
        return AuthenticateResult.Fail("You shall not pass");
    }
}

The startup configuration
public void ConfigureServices(IServiceCollection services)
{
    services.AddAuthentication().AddScheme<AuthenticationSchemeOptions, MyAuthenticationHandler>("MyAuth", null);
    services.AddOcelot(Configuration);
}

ReRoute configuration
{
    "DownstreamHostAndPorts": [
        {
            "Host": "my.api",
            "Port": 80,
        }
    ],
    "DownstreamScheme": "http",
    "DownstreamPathTemplate": "/api/v1/Something/1",
    "UpstreamPathTemplate": "/api/v1/Something/1",
    "UpstreamHttpMethod": ["GET"],
    "ReRouteIsCaseSensitive": false,
    "AuthenticationOptions": {
        "AuthenticationProviderKey": "MyAuth",
        "AllowedScopes": []
    }
}

Actual Behavior
The authentication handler is working as expected but I am not able to add anything to the response body when I set the failure message on the AuthenticateResult. The body of the 401 response is always empty.
Specifications

Version: 14.1.0
Platform: .NET Core 3.1

Noticed the same issue

Noticed the same issue


[SPARK-15441][SQL] support null object in Dataset outer-join

What changes were proposed in this pull request?
Currently we can't encoder top level null object into internal row, as Spark SQL doesn't allow row to be null, only its columns can be null.
This is not a problem before, as we assume the input object is never null. However, for outer join, we do need the semantic of null object.
This PR fixes this problem by making both join side produce a single column, i.e. nest the logical plan output(by CreateStruct), so that we have an extra level to represent top level null obejct.
How was this patch tested?
new test in DatasetSuite

cc @marmbrus @yhuai @davies @liancheng @clockfly

Test build #59692 has started for PR 13425 at commit 56cf840.

Test build #59692 has finished for PR 13425 at commit 56cf840.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59692/
Test PASSed.

LGTM, merging to master and branch-2.0.


Add support for NSS key log export to ssl_client2/ssl_server2 for Wireshark debugging

Summary: This PR adds command line options nss_keylog=0/1 and nss_keylog_file=PATH to ssl_client2 and ssl_server2 which allow to print and export session keys in the NSS keylog format used e.g. by Wireshark for decryption of TLS traffic.
The feature is implemented using @RonEld's recent extended key export feature.
Wireshark usage:

Set a NSS keylog file FILE in Wireshark ( Edit -> Preferences -> Protocol -> (Pre)-Master-Secret log filename).
Run ssl_client2 or ssl_server2 with nss_keylog=1 nss_keylog_file=FILE while Wireshark is listening.
Use the Decrypted Traffic tab in Wireshark to inspect the decrypted traffic.

@Patater @k-stachowiak Thank you for your review! I've rebased this on top of development and pushed a commit addressing @k-stachowiak suggestion.
Could you please re-review?


Mail::Body#to_s uses wrong String encoding

Given an email encoded with a ISO-8859-1 charset, I'd expect the strings coming out of mail methods to be either:

Encoded with ISO-8859-1
Converted correctly to UTF-8

However, we see them returned with ASCII-8BIT encodings which doesn't seem right - and which causes problems on subsequent handling of those strings.
How to reproduce
The following script illustrates the issue:
require "mail"

mail = Mail.new('
Content-Type: multipart/alternative; boundary="_000_AM0PR09MB233743E6638C8665A901BF08CA220AM0PR09MB2337eurp_"

--_000_AM0PR09MB233743E6638C8665A901BF08CA220AM0PR09MB2337eurp_
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

R=F8dgr=F8d med fl=F8de p=E5!
')
text_part = mail.text_part

charset = text_part.charset
puts "charset: #{charset.inspect}"

body_string = text_part.body.to_s
puts "encoding: #{body_string.encoding.inspect}"

# This triggers a conversion of the string to UTF-8, which then fails.
require "json"
JSON.generate({body: body_string})

Running it outputs
charset: "iso-8859-1"
encoding: #<Encoding:ASCII-8BIT>
Traceback (most recent call last):
	2: from repro2.rb:22:in `<main>'
	1: from ~/.rvm/gems/ruby-2.6.5/gems/json-2.3.0/lib/json/common.rb:224:in `generate'
~/.rvm/gems/ruby-2.6.5/gems/json-2.3.0/lib/json/common.rb:224:in `generate': "\xF8" from ASCII-8BIT to UTF-8 (Encoding::UndefinedConversionError)

While the exception comes from inside the JSON gem, I'd risk assessing that the root cause is because body_string.encoding is #<Encoding:ASCII-8BIT> and not #<Encoding:ISO-8859-1>. To verify this we can add a
body_string.force_encoding(charset)

before performing the JSON encoding, which makes the script run without exceptions.
Versions

mail 2.8.0.edge
ruby 2.6.5 and 2.7

Possibly related issues
I did look through the existing issues, and while I wasn't able to find any that matches the issue exactly, there are quite a few that seem related. Many are older, though, and/or missing reproduction steps:

https://github.com/mikel/mail/issues/809
https://github.com/mikel/mail/issues/1216
https://github.com/mikel/mail/pull/921
https://github.com/mikel/mail/issues/902

This was originally reported in https://github.com/mikel/mail/issues/809 but closed based on a misunderstanding of later comments.

Nicely spotted, that is the exact issue we're seeing.
And misunderstandings so easily occur here, when we're talking about encodings, but not those encodings, the other encoding, which is really charset. 🙄 And it's especially hard because you can't visually tell the difference and in so many cases everything still works even though it is wrong.

It does seem, though, that using Mail::Message#decoded and Mail::Part#decoded, as @jeremy suggests in https://github.com/mikel/mail/issues/809#issuecomment-301986573, works. Perhaps not entirely as expected as it always returns a UTF-8 encoded String, but that does satisfy my expectation number 2 above.
That said, it still seems problematic that Mail::Body#decoded can return strings with a decidedly incorrect encoding, but I wonder if this might be more an issue of improving the docs around #decoded?

It does seem, though, that using Mail::Message#decoded and Mail::Part#decoded, as @jeremy suggests in #809 (comment), works.

Only if the email is multi-part. Otherwise, Mail::Message.parts returns [].

Perhaps not entirely as expected as it always returns a UTF-8 encoded String, but that does satisfy my expectation number 2 above.
That said, it still seems problematic that Mail::Body#decoded can return strings with a decidedly incorrect encoding, but I wonder if this might be more an issue of improving the docs around #decoded?

I agree, it's broken.

I'm running this issue too, while processing a message sent into the app from Cloudmailin.
Encoding::UndefinedConversionError: "\xC2" from ASCII-8BIT to UTF-8
So what's the solution?
What do we expect decoded to return in these cases? How can we fix it so that it does so?
(And why is this 5-year-old (#1216), very serious, app-crashing bug still unresolved?)

Not a proper solution to the underlying problem, but here's the quick workaround I came up with, in case it's helpful to anyone...
  html =<EMAIL_ADDRESS>
  private def fix_encoding(part)
    body_str = part.body.decoded
    # Why is the encoding sometimes ASCII-8BIT (and sometimes already converted to UTF-8) instead of
    # respecting the claimed encoding? Why does trying to encode it as UTF-8 result in:
    # Encoding::UndefinedConversionError ("\xE2" from ASCII-8BIT to UTF-8)? We may never know. (See
    # https://github.com/mikel/mail/issues/1413, etc.) But here is a workaround.
    logger.debug "body_str.encoding: #{body_str.encoding} (part.charset: #{part.charset})"
    unless body_str.encoding == Encoding.find('utf-8')
      new_encoding =
        if (body_str.encode('utf-8') rescue false)
          'utf-8'
        else
          part.charset
        end
      logger.warn "Using force_encoding to change from #{body_str.encoding} to #{new_encoding} (part.charset: #{part.charset})"
      body_str = body_str.force_encoding(new_encoding)
    end
    body_str
  end


Add timestamp option

It would be nice you could start the timestamp from a certain point so you could sync it to a recording. You would need a start and stop command that would control a background timer of some kind. Maybe similar to how you can set a build process, you could have a shortcut run a python script that would keep track of this...

After looking into this, it would have to be a python script as ST snippets don't support timestamps.


WEBAPP-348: Sanitize remote content

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword WEBAPP.

We need to sanitize all input is inserted directly into the dom. We probably also need to sanitize input which is used as href anywhere.
We could introduce a type like: SanitizedString. Our components could require you to sanitize it before usage.
But yeah there are probably a lot more places where we need to sanitize.

@klinzo Yes, that's true, but this task was about sanitizing html content that we inject into the dom.


在vue中如何进行Chart级别主题切换

官方例子里是使用theme: 'dark'参数配置:
const chart2 = new G2.Chart({
    container: 'c2',
    forceFit: true,
    height: 250,
    theme: 'dark'
  });

我这样写好像并没有什么效果:
<v-chart :forceFit="true" height="300" :data="data" :scale="scale" :theme="'dark'">
</v-chart>

请问应该怎么正确使用?

We don't support this feature right now. We will consider to add it in next version. Thank you.

We don't support this feature right now. We will consider to add it in next version. Thank you.

feature comming soon...

supported! use the lastest version .thank u


chore(ci): Set up codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@d9acfc1). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #103   +/-   ##
==========================================
  Coverage           ?   75.05%
==========================================
  Files              ?       27
  Lines              ?      854
  Branches           ?        0
==========================================
  Hits               ?      641
  Misses             ?      213
  Partials           ?        0

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 d9acfc1...3e97c2d. Read the comment docs.


Typescript doc visible for aliased imports, but not for single ones.

From @Elarcis on September 21, 2016 7:35

VSCode Version: 1.5.2
OS Version: Windows 7 Service Pack 1
Typescript version: 2.0.2

Steps to Reproduce:

have a myClass.ts file exporting a MyClass class. Write some /** */ doc for that class.
In another file, try the following two ways of importing identifiers:

import { MyClass } from 'myModule';
MyClass // When hovering MyClass, the written doc is not visible.

import * as myModule from 'myModule';
myModule.MyClass // When hovering MyClass, the written doc appears.

It seems logical that the doc should appear in both cases, is this due to Typescript 2?
Copied from original issue: Microsoft/vscode#12360

Repro'd with latest TypeScript release bits - release-2.0.5 branch.
Here is code to copy and paste.
test.ts
import { MyClass } from './myModule';
MyClass // When hovering MyClass, the written doc is not visible.

import * as myModule from './myModule';
myModule.MyClass // When hovering MyClass, the written doc appears.

myModule.ts
/**
 * Some documentation
 */
export class MyClass {

}

sending to TypeScript language service repo.

From @chrmarti on September 21, 2016 16:42
Can reproduce with the given example and TypeScript 1.8 in todays insider build.

This is currently a huge painpoint. Improving this would be a very big win.

I'd also like to point out that default imports don't get the (js)doc for that export either.
import SomeDefaultExport from './FooBar';

Hovering over SomeDefaultExport in both the import declaration as well as it's uses in code will just show "import SomeDefaultExport".
I'm not at my desktop at the moment, so I haven't tested this, but imagine the same problem would exist with imports like this:
import { Foo as Bar } from './FooBar';

I'll update this post when I'm able to test that.
My experience with this bug are in typescript, but I would imagine there would be the issue with javascript (that uses es2015 imports). I'll test that out too.

I've lost quite some time thinking I was doing type exports / imports wrong due to not seeing the docs on hover. 😅
What's the status on this? Waiting for someone from the community to work on it?

I'm the one who reported https://github.com/Microsoft/vscode/issues/28753. You can see the repro easily there.
This seems like something basic that should just work, especially consider that props work fine (and that seems like it'd be harder to support). :D But anyways, please fix/include the class/component-level docs inclusion soon!

Is this a duplicate of #5515?

The reported code works OK today - the documentation is displayed. cc @jakebailey

Yeah, this was a dupe of #5515; fixed in #19489.


fix: change rm -rf to rimraf

Currently, npm clean-lib fails on windows with cmd because it does not now about rm.
This change uses the already installed rimraf to do the job instead.

Alex, could you PTAL?


python311Packages.aiosmb: 0.4.8 -> 0.4.9

Changelog: https://github.com/skelsec/aiosmb/releases/tag/0.4.9
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 263118 run on x86_64-linux 1

14 packages built:

crackmapexec
crackmapexec.dist
python310Packages.aiosmb
python310Packages.aiosmb.dist
python310Packages.lsassy
python310Packages.lsassy.dist
python310Packages.pypykatz
python310Packages.pypykatz.dist
python311Packages.aiosmb
python311Packages.aiosmb.dist
python311Packages.lsassy
python311Packages.lsassy.dist
python311Packages.pypykatz
python311Packages.pypykatz.dist


data preprocessing

I'm just wondering what type of data preprocessing for SIF embedding I need to do for the sentences. For example,

do I need to remove punctuations? In the example, sentences don't have punctuations.
should I tokenize negations?
what other preprocessing needs to be done?
Thanks a lot!!

SIF depends on the embedding generated by other word embedding algorithms like GLOVE..

It is better to feed the data free of punctuation.
Not necessary.
Even if you dont remove stop words SIF will take care, but its generally better to clean the data.

You can see a comparison btw Fasttext and Word2vec using SIF here :
https://github.com/a11apurva/Sentence-Embeddings-benchmarking


Using root finder to solve a inverse function with the NN form Error

Thank you for the great job. I find it very helpful for me to solve the inverse function. However, I have a problem as follows:

I want to solve a inverse function with the form of a neural network, numerically. So I choose to use root finding algorithm.
Following the instructions in your document, here is my codes:

torch_model = torch.nn.Sequential(
    torch.nn.Linear(3, 1),
    torch.nn.ReLU(),
)

def func1(y, A):  # example function
    return A - torch_model(y)

A =  10*torch.ones((2,1)).requires_grad_() # output shape of the torch_model_1 is (2,1)
y0 = torch.zeros((2,3)) # the torch_model accepts input shape of (2,3)

# finding a root
yroot = rootfinder(func1, y0, params=(A,))

The torch_model accepts input shape of (batch_size,3) and the outputs (batch_size, 1)
Given the outputs, I want to find the inputs values. So, I choose to let the param A to be the given outputs and try to get the yroot.
But the code show the errors:
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Cell In[42], line 9
      6 y0 = torch.zeros((2,3)) #
      8 # finding a root
----> 9 yroot = rootfinder(func1, y0, params=(A,))

File [~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/optimize/rootfinder.py:93](https://file+.vscode-resource.vscode-cdn.net/media/lk/disk1/DL_lk/202205_RZ_GAN%E6%A8%A1%E5%9E%8B/model/EvoOpt/~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/optimize/rootfinder.py:93), in rootfinder(fcn, y0, params, bck_options, method, **fwd_options)
     91 pfunc = get_pure_function(fcn)
     92 fwd_options["method"] = _get_rootfinder_default_method(method)
---> 93 return _RootFinder.apply(pfunc, y0, pfunc, False, fwd_options, bck_options,
     94                          len(params), *params, *pfunc.objparams())

File [~/anaconda3/envs/dpc/lib/python3.8/site-packages/torch/autograd/function.py:506](https://file+.vscode-resource.vscode-cdn.net/media/lk/disk1/DL_lk/202205_RZ_GAN%E6%A8%A1%E5%9E%8B/model/EvoOpt/~/anaconda3/envs/dpc/lib/python3.8/site-packages/torch/autograd/function.py:506), in Function.apply(cls, *args, **kwargs)
    503 if not torch._C._are_functorch_transforms_active():
    504     # See NOTE: [functorch vjp and autograd interaction]
    505     args = _functorch.utils.unwrap_dead_wrappers(args)
--> 506     return super().apply(*args, **kwargs)  # type: ignore[misc]
    508 if cls.setup_context == _SingleLevelFunction.setup_context:
    509     raise RuntimeError(
    510         'In order to use an autograd.Function with functorch transforms '
    511         '(vmap, grad, jvp, jacrev, ...), it must override the setup_context '
    512         'staticmethod. For more details, please see '
    513         'https://pytorch.org/docs/master/notes/extending.func.html')

File [~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/optimize/rootfinder.py:302](https://file+.vscode-resource.vscode-cdn.net/media/lk/disk1/DL_lk/202205_RZ_GAN%E6%A8%A1%E5%9E%8B/model/EvoOpt/~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/optimize/rootfinder.py:302), in _RootFinder.forward(ctx, fcn, y0, fwd_fcn, is_opt_method, options, bck_options, nparams, *allparams)
    300     name = "rootfinder" if not is_opt_method else "minimizer"
    301     method_fcn = get_method(name, methods, method)
--> 302     y = method_fcn(fwd_fcn, y0, params, **config)
    304 ctx.fcn = fcn
    305 ctx.is_opt_method = is_opt_method

File [~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:181](https://file+.vscode-resource.vscode-cdn.net/media/lk/disk1/DL_lk/202205_RZ_GAN%E6%A8%A1%E5%9E%8B/model/EvoOpt/~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:181), in broyden1(fcn, x0, params, **kwargs)
    167 @functools.wraps(_nonlin_solver, assigned=('__annotations__',))  # takes only the signature
    168 def broyden1(fcn, x0, params=(), **kwargs):
    169     """
    170     Solve the root finder or linear equation using the first Broyden method [1]_.
    171     It can be used to solve minimization by finding the root of the
   (...)
    179            https://web.archive.org/web/20161022015821/http://www.math.leidenuniv.nl/scripties/Rotten.pdf
    180     """
--> 181     return _nonlin_solver(fcn, x0, params, "broyden1", **kwargs)

File [~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:123](https://file+.vscode-resource.vscode-cdn.net/media/lk/disk1/DL_lk/202205_RZ_GAN%E6%A8%A1%E5%9E%8B/model/EvoOpt/~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:123), in _nonlin_solver(fcn, x0, params, method, alpha, uv0, max_rank, maxiter, f_tol, f_rtol, x_tol, x_rtol, line_search, verbose, **unused)
    118     raise ValueError("Jacobian inversion yielded zero vector. "
    119                      "This indicates a bug in the Jacobian "
    120                      "approximation.")
    122 if line_search:
--> 123     s, xnew, ynew, y_norm_new = _nonline_line_search(func, x, y, dx,
    124                                                      search_type=line_search)
    125 else:
    126     s = 1.0

File [~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:277](https://file+.vscode-resource.vscode-cdn.net/media/lk/disk1/DL_lk/202205_RZ_GAN%E6%A8%A1%E5%9E%8B/model/EvoOpt/~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:277), in _nonline_line_search(func, x, y, dx, search_type, rdiff, smin)
    274     return (phi(s + ds, store=False) - phi(s)) [/](https://file+.vscode-resource.vscode-cdn.net/) ds
    276 if search_type == 'armijo':
--> 277     s, phi1 = _scalar_search_armijo(phi, tmp_phi[0], -tmp_phi[0],
    278                                     amin=smin)
    280 if s is None:
    281     # No suitable step length found. Take the full Newton step,
    282     # and hope for the best.
    283     s = 1.0

File [~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:295](https://file+.vscode-resource.vscode-cdn.net/media/lk/disk1/DL_lk/202205_RZ_GAN%E6%A8%A1%E5%9E%8B/model/EvoOpt/~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:295), in _scalar_search_armijo(phi, phi0, derphi0, c1, alpha0, amin, max_niter)
    294 def _scalar_search_armijo(phi, phi0, derphi0, c1=1e-4, alpha0=1, amin=0, max_niter=20):
--> 295     phi_a0 = phi(alpha0)
    296     if phi_a0 <= phi0 + c1 * alpha0 * derphi0:
    297         return alpha0, phi_a0

File [~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:263](https://file+.vscode-resource.vscode-cdn.net/media/lk/disk1/DL_lk/202205_RZ_GAN%E6%A8%A1%E5%9E%8B/model/EvoOpt/~/anaconda3/envs/dpc/lib/python3.8/site-packages/xitorch/_impls/optimize/root/rootsolver.py:263), in _nonline_line_search..phi(s, store)
    261 if s == tmp_s[0]:
    262     return tmp_phi[0]
--> 263 xt = x + s * dx
    264 v = func(xt)
    265 p = _safe_norm(v)**2

RuntimeError: The size of tensor a (6) must match the size of tensor b (2) at non-singleton dimension 0

Could anyone to give some suggestions? Thanks!

The rootfinder requires the input and output to have the same number of elements. In your case, the input y has (batch_size, 3) elements, while the output has (batch_size, 1) elements, so there is a mismatch of the number of elements.


Edits to Current drought conditions page

Jared suggested text to update the 2021 overview section of the Current drought conditions page.

[x] Work his suggested text into the whole: draft
[x] Notify @jaredwyee so he can get stakeholder buy-in/edits
[ ] Get approval or edits
[ ] Stage
[ ] Publish

@thepegisin Okay to close this?


getTextContent() Shows jumbled symbols instead of text

Scriptures - John Lesson 17.pdf
Link to PDF file (or attach file here):
Configuration:

Web browser and its version: Chrome
Operating system and its version: Linux / Mac
PDF.js version: 1.6.418
Is an extension:

Steps to reproduce the problem:

Simply load document and read text.
Real code where we do that: http://ttsreader.com
Try to upload the attached file.

What is the expected behavior? Text
What went wrong? Jumbled symbols instead.
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension): http://ttsreader.com
Thanks!

I get similar broken behaviour in a number of other PDF readers as well, e.g. in both Adobe Reader and PDFium (in Chrome).
The issue is with the PDF file itself, since the embedded fonts are missing the ToUnicode entries required for text-selection to work properly.
Closing as invalid, since we unfortunately cannot do anything about the missing data in the PDF file.


Fix deprecation warnings

Didn't know how to contribute to #29 so here's another PR. This brings use of AWS SDK up-to-date, removes all deprecation warnings but should keep direct S3-to-S3 copy feature intact.

Closing this for inactivity, and we no longer use this gem.


Patch to remove 'using namespace std' from headers

closes #139

My intention was to try this patch with remill (since that's where I first ran into the issue), but it looks like there are some additional patches there that break with this branch. I managed to fix the ArmThumb patch from remill and the only issue I ran into was that register is no longer supported in C++17. This patch seems to fix that.
Unfortunately now I run into a linker error from remill (which I think related to the fact that remill is on a different version this repo and the API changed?):
[100%] Linking CXX executable remill-lift-15
Undefined symbols for architecture arm64:
  "ContextInternal::restoreFromSpec(Element const*, AddrSpaceManager const*)", referenced from:
      remill::sleigh::SingleInstructionSleighContext::restoreEngineFromStorage() in libremill_arch_sleigh.a(Arch.cpp.o)
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

My intention was to try this patch with remill (since that's where I first ran into the issue), but it looks like there are some additional patches there that break with this branch. I managed to fix the ArmThumb patch from remill and the only issue I ran into was that register is no longer supported in C++17. This patch seems to fix that.
Unfortunately now I run into a linker error from remill (which I think related to the fact that remill is on a different version this repo and the API changed?):
[100%] Linking CXX executable remill-lift-15
Undefined symbols for architecture arm64:
  "ContextInternal::restoreFromSpec(Element const*, AddrSpaceManager const*)", referenced from:
      remill::sleigh::SingleInstructionSleighContext::restoreEngineFromStorage() in libremill_arch_sleigh.a(Arch.cpp.o)
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Here is my current WIP repo if you want to try: https://github.com/mrexodia/remill/tree/sleigh-using-namespace

Ah. Yeah, this patch is for the latest Ghidra 10.2 version, which has some internal sleigh API changes, and remill has not been updated yet. Remill also has some additional patches, which I have not looked into yet and seem to require updating, as you've found out.
I won't have time to look into this too much this week, but I will take a look when I am back.
There are a few places I will start to look based on updating to Ghidra 10.2 in other tools:

https://github.com/lifting-bits/sleigh/commit/f38eee41a289ac5fbf27c6951a3c65e8a48e7bac#diff-aed2f30b604fcd3c832b0255d6cd0ef5a7cca4cab0d8d2c77ee6d125d2fcf301
https://github.com/trailofbits/maat/pull/160/files#diff-7e5860b8ac02e6dc2650d19d18a9e3fb9ab939518a85d7b0f99dd67fca124216 (although these tests are failing for some reason now 🤔)
https://github.com/trailofbits/mishegos/pull/1462/files#diff-a0f9b268e448e17e90e505165fe08069883b1e3b5d0c7cac92065f501f923722 (to a lesser extent)

Thanks, I'll take a look to see if a lot of effort is required to update remill to the latest Ghidra version!

Ping @ekilmer

Any chance this could be merged? I tested and this does fix the issue. It's a blocker for building remill on Windows and also a blocker for https://github.com/trailofbits/maat/pull/97

Updated the patches for the latest commit on Ghidra's master branch. I assume that if things are compiling correctly and passing tests, then it works for you as well @mrexodia.
Would you be willing to write a test (or post some C++ code that I can integrate into CMake) that we could include in this repo to make sure we don't suffer regressions?


ClimateMappedAfrica Fix Zoom Bug

Description
This PR fixes a bug in ClimateMapped where the map zoom level was not included in the rootGeography configuration
Type of change

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

@koechkevin PR #991 deleted the RootGeography file where these changes had been made. When merging #996, I did not crosscheck that further modifications had been made to that file.

If it's a fix, don't wait for my approval, lets get it deployed @kelvinkipruto