Separate pyproject.toml location from root argument

PR Checklist

[x] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[x] If you've fixed a bug or added code that should be tested, add tests!
[x] Documentation in docs is updated

Description of changes
deptry allows passing the root directory as a positional argument, which gives some flexibility to projects that use different kind of structures.
This is for instance useful for projects that use an src directory structure, like Poetry or Hatch.
Unfortunately, the root directory passed as a command line means that the configuration file will be read from the directory, and not from the directory deptry is launched from, diverging, in my opinion, to what most other tools in the ecosystem do.
This PR is an attempt to fix that by always trying to load pyproject.toml from the location deptry is run from, both for reading [tool.deptry] configuration, or extracting the dependencies if a project uses PEP 621 or Poetry.
This means that when running:
$ deptry src

deptry will now search for pyproject.toml in the location it is run from, whereas currently, doing so will make deptry look for a pyproject.toml in src directory.
The downside is that this could break some projects that did explicitly want to find pyproject.toml in a root directory other than ., but I have a second PR (https://github.com/fpgmaas/deptry/pull/245) building upon this one that exposes the --config argument, making it possible to explicitly pass the location of pyproject.toml, giving both flexibility for passing a different root directory and allowing to continue supporting this use case, if it exists.

Codecov Report

Merging #244 (cfe17dc) into main (a2866fe) will decrease coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main    #244     +/-   ##
=======================================
- Coverage   95.7%   95.6%   -0.1%
=======================================
  Files         32      32
  Lines        915     906      -9
  Branches     203     200      -3
=======================================
- Hits         876     867      -9
  Misses        26      26
  Partials      13      13

Impacted Files
Coverage Δ

deptry/cli.py
93.4% <100.0%> (-0.2%)
:arrow_down:

deptry/core.py
97.6% <100.0%> (ø)

deptry/utils.py
86.6% <100.0%> (-4.7%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Keeping it as a draft for now, since this might be controversial, and I would like to have your opinion on the changes first.

As you mentioned, this does mean that we could potentially introduce breaking changes in other projects using deptry. I think it would be good to not create a release after merging this PR, and then - since the project is also approaching reasonable maturity - to create our first major release after merging both this PR and #245.
Do you agree?

Semantic versioning explicitly authorises introducing breaking changes in 0.y.z.
Personally, for now, I like that we are able to update the API of the CLI or the library's behaviour without having to give too much thoughts about it, even if it is still a good thing to not break possible current usages of it (like this PR + https://github.com/fpgmaas/deptry/pull/245 do, by offering alternatives), and of course, communicate about the changes we are making in the changelog.
If we want to respect semantic version, going into 1.0.0 would be a major shift in how we handle changes in the project, and I can see a few things that could be reworked a bit before that, especially considering that deptry is not yet used in major projects, so we might see multiple feature requests over time that could considerably change the current APIs or behaviour of the library.
There are IMO reasons why major libraries like black took quite some time before going into "stable", or why mypy is still on a 0.y versioning: it takes time to stabilise APIs based on users expectations, what possible changes we think of introducing as developers, and it's also a point of no return.
In any case, I agree that it would be best to tag a release once https://github.com/fpgmaas/deptry/pull/245 is also merged, but based on everything mentioned above, I would personally bump the version to 0.7.0, and mention both changes in the changelog. Of course, you might disagree with me about not going into 1.0.0, but in that case, I'd like that we at least talk about it in a bit more details, as this IMO requires careful planning.

Thanks, that makes sense. I was unaware that breaking changes can be introduced in version 0.y.z. Although in hindsight it makes sense :) I agree with your proposal of releasing 0.7.0 instead.


merge from template-sync/template-arlac77-github

.github/workflows/ci.yml

chore(deps): add 22.10.0 remove 22.9.0 (jobs.test-node.strategy.matrix.node-version)
chore: add 22.10.0 (jobs.release.steps.with.node-version)

.github/workflows/codeql_analysis.yml

chore: add 22.10.0 (jobs.analyze.steps.with.node-version)

.github/workflows/update_package_lock.yml

chore(deps): add 22.10.0 remove 22.9.0 (jobs.update_package_lock.strategy.matrix.node-version)

.github/workflows/update_readme_api.yml

chore: add 22.10.0 (jobs.update_readme_api.steps.with.node-version)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 87.005%

Files with Coverage Reduction
New Missed Lines
%

src/template.mjs
4
81.23%

Totals

Change from base Build<PHONE_NUMBER>7:
-0.1%

Covered Lines:
2534

Relevant Lines:
2901

💛  - Coveralls


Update README.md

Added a few links to GPTs and added a section for career and guidance

Hi @Anil-matcha ! I created three new GPTs and I wanted to add it to the collection!

A career coach to help with career pathing, job listings search, mock interviews, and resume review.
A cognitive behavioral therapy counselor to talk you through anxious and stressful situations.
A life coach to help you set goals, create plans, and build habits with reminders over SMS.

Thanks!


Improvements to user token management

When you add your token to lots of models the box they appear in gets quite big and on smaller screens can mean lots of scrolling. Can we cap the height and add a scroll bar?

Is it worth adding an "All permissions" button here?

Is it worth adding an "All permissions" button here?

I considered this, but as there's only two options it feels unnecessary to me. I'd be keen to hold off on an ALL option unless we add more options.


fixed "Additional references" section

Added another line break to fix the formatting of the "Additional references" section
Fixes #1519

Thanks, @bcwood !


README: fix a typo

Current coverage is 85.21%

Merging #9 into master will not affect coverage as of 010c56a

@@            master      #9   diff @@
======================================
  Files           44      44
  Stmts         2611    2611
  Branches         0       0
  Methods          0       0
======================================
  Hit           2225    2225
  Partial          0       0
  Missed         386     386

Review entire Coverage Diff as of 010c56a

Powered by Codecov. Updated on successful CI builds.

Thank you !


Don't crash on unknown content transfer encoding

See #2044
This avoids the crash. But to properly fix the issue we have to take the Content-Transfer-Encoding into account when collecting viewable parts.

So what will we get instead of a crash? An email with a blank body?

The body will be returned as-is, i.e. potentially transfer-encoded.


Make processing state buttons faster

After a transaction is confirmed, the processing state button shows confirmed.  But it takes a very long time for this to go away.  Need to improve the speed of dismissal

Is there a specific transaction you are testing? The speed varies, slowest is trading

Notice it in a few places, but trading and cancelling an order stand out....Takes like 15-20 seconds for the confirmed button to disappear....if we can get this to 2 seconds would make it feel much more snappy

Issue is that UI is waiting for SDK to send event in order to update user values like positions and PL. UI can't do that until SDK has processed the block data.

Need to standardize handling pending transactions. need to add transaction hash to pending queue entries. so that transaction can be looked up if status hasn't been updated in 10 blocks.

This issue appears to be exacerbated by the blocks being cached by alchemy. On mainnet this discrepancy wont happen to this extent.


[NT-1893] UI for Comment Composer

📲 What
Create a Comment Composer component that would be used to post comments to a specified project in the app.
🤔 Why
This is part of a new commenting experience we're adding to the application.
👀 See

✅ Acceptance criteria

Launch the app
Tap on any project card
Tap on comment section to show the new comments screen
Tap on the input area where you see "Leave a comment..."
As the keyboard comes up, the comment composer should slide up with the keyboard
Enter some texts in the new comment input area. The input area should grow as you type in more texts and should stop growing when the maximum height is reached and text view should be scrollable.
Pan down the keyboard and the composer should slide downwards along with the keyboard

⏰ TODO

[x] Find a way around the corner radius issue for the input container view
[x] Limit input characters to 9000
[ ] Use appropriate colours for the interface when they are ready from design team.
[ ] Replace raw strings with type-safe versions when copy is ready
[ ] Add Unit tests and screenshots tests.

A couple of observations:

When I select a project and enter the comments the avatar is occasionally not rounded appropriately (diamond oval shape)
When I select a project I've backed and enter the comments the "Only backers can leave comments" label is showing instead of the textview
When I select a project, enter the comments, swipe the page view controller and return back to the comments, the avatar/textview screen is no longer showing/accessible
The cells are using a grey selection style

Is there a separate ticket to add VMs and tests? I do see a lot of opportunities for testing here.

@tobitech you might have missed this question ☝

Is there a separate ticket to add VMs and tests? I do see a lot of opportunities for testing here.

@tobitech you might have missed this question ☝

Oh yeah, I missed it. There is no separate ticket for that it will be part of this PR. What I have is a ViewModel specifically for the composer component and tests for it. I will be pushing that shortly

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger


ci: pre-release sanity check for rules_nodejs 1.6.0

Just a sanity check that all the fixes for 1.6.0 haven't broken anything

1.6.0 released https://github.com/bazelbuild/rules_nodejs/releases


Support for how to show horizontal expand/collapse.

Describe the Bug
I bought the pro version and used the example: https://pro.reactflow.dev/examples/expand-collapse
I want to show this tree in horizontal flow. I tried with { sourcePosition: 'right',
targetPosition: 'left',} on the node, and   I added handle position accordingly right and left but it's not working as expected.
(See attached file

)
Would anyone help me on this?
Thanks
Your Example Website or App
http://localhost:8081/intelligence
Steps to Reproduce the Bug or Issue
Steps I described above:
`export const nodes = [
{
id: 'A',
sourcePosition: 'right',
position: {x: 0, y: 0},
data: {expanded: true,label:'MLK kili',class:['simple-root-node', 'text-center']},
},
{
id: 'B',
position: {x: 0, y: 0},
sourcePosition: 'right',
targetPosition: 'left',
data: {expanded: false,label:'Netherlands',class:['child-node', 'text-left']},
},
];
`
Expected behavior
As a user I expect the behaviours like attached Screenshot.

Screenshots or Videos
No response
Platform

OS: [ Linux]
Browser: [Chrome]
Version: [Version 114.0.5735.198 (Official Build) (64-bit)]

Additional context
Check above issue

I find something to set horizontal, that I don't know proper or not? After I set positions dynamically I got horizontal layout but it does not support separation between nodes.
Code:
`function useExpandCollapse(
nodes, edges,
{layoutNodes = true, treeWidth = 220, treeHeight = 100}) {
return useMemo(() => {
    const hierarchy = stratify()
        .id((d) => d.id)
        .parentId((d) => edges.find((e) => e.target === d.id)?.source)(nodes);

    hierarchy.descendants().forEach((d) => {
        d.data.data.expandable = !!d.children?.length;
        d.children = d.data.data.expanded ? d.children : undefined;
    });

    const layout = tree()
        .nodeSize([treeWidth, treeHeight])
        .separation(() => 1);

    const root = layoutNodes ? layout(hierarchy) : hierarchy;

    let mXstep = 50;
    let mYstep = 50;
    return {
        nodes: root.descendants().map((d) => ({
            ...d.data,
            // This bit is super important! We *mutated* the object in the `forEach`
            // above so the reference is the same. React needs to see a new reference
            // to trigger a re-render of the node.
            data: {...d.data.data},
            // targetPosition : 'left',
            // sourcePosition : 'right',
            type: 'custom',
            // position: isHierarchyPointNode(d) ? {x: d.x, y: d.y} : d.data.position,
            position: {  // dynamic position added OR using from initial nodes data.
                x:d.data.position.x,
                y:d.data.position.y + mYstep,
            },
        })),
        edges: edges.filter((edge) => root.find((h) => h.id === edge.source) && root.find((h) => h.id === edge.target)),
    };
}, [nodes, edges, layoutNodes, treeWidth, treeHeight]);`

If you want to display a horizontal flow, you need to swap x and y.
position: isHierarchyPointNode(d) ? {x: d.y, y: d.x} : d.data.position

We will update the pro example soon, in order to show a vertical and horizontal direction.


PatternFly LeftNav view has excessive padding

For some reason, we are adding a padding-top, on top of whatever padding the patternfly css uses:
[kui-theme-style] .pf-c-nav .pf-c-nav__list {
    padding-top: 1em;
}

with our rule:

without our rule:


useServiceAccountAuth curently requires a string not JSON

In the example text it shows
var creds = require('../token.json');
doc.useServiceAccountAuth( creds, function()
but this gives an error from Google saying key not found. Turns out it should be a string version passed in rather than the JSON.
A quick fix is just modify the docs but perhaps the userServiceAccountAuth should convert JSON to a string internally?
Below appears to work fine.
var creds = require('../token.json');
doc.useServiceAccountAuth( JSON.stringify( creds ), function( params )

If you pass a string then it must be the path of the module you want to require (ie. credentials.json). Otherwise pass

I've just a published a new version of this module which is a complete rewrite using Google's V4 Sheets API (v3 is being deprecated on March 3, 2020).
Docs available here - https://theoephraim.github.io/node-google-spreadsheet
I do believe this issue should be fixed in the new version.
Please test it out and if you find that there is still a problem, reopen the issue.
Thanks! 😄


feat: Wallet Instance Attestation

Wallet Instance Attestation
This PR adds the design review of the Wallet Instance Attestation including the request flow and format of the Wallet Instance Attestation Request and Wallet Provider Entity Configuration
resolve #4

[X] Ensure your files are written following RST specs (not MD!)
[ ] Italian version
[X] English version
[X] Example files
[X] Ask for review

it seems to me that we can definitively say that the flow proposed in this PR is not compliant to:
oauth-attestation-based-client-authentication
we should work on an analysis on the impacts and implication of this @asharif1990 @giadas @grausof. In which parts the absence of DPoP in oauth-attestation-based-client-authentication is considerable a requirement or overkill?


enh: add test for get int from service annotation

What this PR does / why we need it:
Which issue this PR fixes(if applicable):
refers https://github.com/kubernetes/cloud-provider-openstack/issues/2400
Special notes for reviewers:

Release note:

NONE

/ok-to-test

/lgtm

/approve


Fuckr will not logout, stays in background processes after program closed or crashes

@tomlandia Fuckr will not logout, it stays in the background processes after the program is closed or crashes (seems I am still on Grindr still). Multiple instances taking up resources. Have to do full shutdown to make them go away.

Yeah v1.6.1 is a shitshow sorry. It's not the one on Fuckr.me. Going to release 1.6.2 using nw 0.12.1.


feature request: Columns after filename

Did you check existing requests?

[x] I have searched the existing issues

Describe the feature
Make it possible to place column information after the filename.
Provide background
First of all, I absolutely love this plugin and am disappointed I didn't find it earlier. Truly helps me a lot.
I was interested in using the columns feature but I've noticed they always come before the filename, and there is no functionality to move them after. Although I do acknowledge this format matches ls better, my eyes are conditioned to look right at the start of the line from netrw; so having this option would be amazing.
What is the significance of this feature?
nice to have
Additional details
One way to implement this would be to make the "name" column placeable in the options.

Unfortunately this customization is impossible for oil. Because of how oil works, each line that gets rendered must also be parseable so that we can read back changes. Columns can each define how they get parsed, so it allows for fairly flexible formats (e.g. a single column can contain whitespace), but there is an issue with filenames. Filenames can be...anything. They can contain whitespace, or not, special characters, or not, and there's no way to look at a sequence of characters and determine when a filename has ended and something else has begun. For this reason, the filename must be located at the end of the line so that we can parse all of the values with known formats (columns) first, and then take the rest of the line as the filename.

Thanks for a descriptive reply, I can definitely see you have thought about this before. Just for my satisfaction, I'd like to ask if it's possible to do this if you check for columns backwards; perhaps a setting like this:
columns = {
    "icon",
    "permissions",
    "size",
    "mtime",
    reverse = true,
  },

or maybe
...
columns = {
    "icon",
    "permissions",
    "size",
    "mtime",
  },
reverse_columns = true,
...

Technically possible, but would add more complexity to the code than I think it's worth. Also doesn't play nice with some of the other mechanisms such as using <filename> -> <destination> for symbolic links and using virtual text for other information (e.g. original location of a trashed file)


Custom fields when viewing devices/racks

We have asset tags for all of our equipment.
The ability to add custom fields to the device/rack screen would be excellent.
It would be more beneficial (for reports etc) to have specific fields.
The option to make them required would be good too.
Thanks.

This was touched upon here:
https://www.reddit.com/r/networking/comments/4k89ty/i_built_an_ipamdcim_thing/d3iw6sr
There isn't a specific enhancement for custom fields yet but it certainly has merits

Ahh, excellent!
I'll close this for now.


Hook: add type attribute to spec

For vm-based runtimes, like Kata Containers, the hooks become
confused, some hooks may need run on the host side, but others
may need to run on the guest side.
Add type attribute identity where the hooks should run, the
host side or guest side, and let runtime treat them separately.
Signed-off-by: bin liu<EMAIL_ADDRESS>

how are these used by Kata?  Is there any open PR that uses the new attribute?

@giuseppe Indeed Kata only supports running hooks on the host side now. And Kata has a feature named guest hook that running in the VM. Users can add hooks to the guest image. When Kata containers start, the agent inside the guest will scan for hooks and add them to the OCI spec of the containers. Here is one issue https://github.com/kata-containers/kata-containers/issues/3729.
Adding the type attribute can make it easy to decide whether run hooks on the host or guest.
@flouthoc I understand what you are meaning. I have thought of some workaround, for example passing the host/guest flag by ENV field, or setting flags in annotations.
How about adding a new field annotations to Hook? It's will be more generic and extendable than fixed-named field names like type. Through annotations, users can pass more metadata to runtime for handling hooks.

Through annotations, users can pass more metadata to runtime for handling hooks.

The state object already has annotations.
https://github.com/opencontainers/runtime-spec/blob/main/runtime.md#state
Is this enough for your usecase?

ping @liubin

@AkihiroSuda Sorry for the late response, I'm considering closing this one as comments above.
And I'll try to think about other plans including your  with real-world use cases later.


onClick/onChange calls the value of rating, but not filling the star symbols

Hello!
I came across a problem when trying to use the react-rating. I could successfully call the values of the rating using onClick or onChange. However, the stars of the rating is filled in color only when clicked the second time on the same rating. Here is how my code looks like (in react hooks):
const [Rate, setRate] = useState(0);
return ( <Rating
emptySymbol={star}
fullSymbol={star}
onClick={value => {
setRate(value);
}}
/>)
Am i missing something here?
Thank you very much!

Hi @lixinge921111!
As you are keeping the state outside of the Rating component with the useState hook, you need to plug the Rate value to the Rating component. You just need to bind the initialRating to your Rate state:
  const [Rate, setRate] = useState(0);

  return (
    <Rating
      emptySymbol={star}
      fullSymbol={star}
      onClick={setRate}
      initialRating={Rate}
    />
  );

Check this codepen.

Thank you very much! We fixed the problem :)

It works now :) Thanks again!


Pull in edits for GEOSadas-5_27_1 into main

This is to pull in the latest bufrlib (also see #12 by @gmao-msienkie). We pull in this version as it has been tested.

Note: Labeling as non-zero-diff as the bufrlib update seems...extreme. But this is needed for GEOSadas and does not affect GEOSgcm. So it's a "safe" non-zero-diff


removed wro dependency and config

Coverage remained the same at 26.159% when pulling 8db0f3d6b3ef4b071e4cd3353dcda72a654ad9de on remove-unused-dependency into e7f5faf519fc9280d40e566cdf6bcf02dab1e645 on master.


PHP 64 bit signed int causing issues with Trace ID

The SpanConverter class uses hexdec to convert the traceId (both high and low is using 128bit ids) to an integer for the Thrift package, the issue is that PHP doesn't support unsigned ints, so any ID above 0x7FFFFFFFFFFFFFFF will be output as a float and will lose precision.
When converted back it will have changed the spanId
$original = 'ba3035a23b64558d';

echo "Original: {$original}\n";

$a = hexdec($original);

echo "As Int: {$a}\n";

$b = dechex($a);

echo "Back to Hex: {$b}\n";

will output

Original: ba3035a23b64558d
As Dec: 1.3416282260834E+19
Back to Hex: ba3035a23b645800

I have had to fall back to the zipkin exporter, even though I am pushing to Jaeger, as it uses strings for the ID and doesn't rely on the hex conversion

This would need to be fixed upstream in the Thrift PHP package as it expects integer values to TProtocol#writeI64($int).

Agreed, but how is anyone using the jaeger exporter at all (or Thrift PHP for that matter)? Are people just finding a way to limit Int64 generation in PHP to below the cutoff?

The core opencensus-php library only generates trace and span ids with 32 bits of randomness. For a standalone PHP application, it should be good enough. If it's in a SOA where the trace id is propagated and the trace id generated is 64-bit, then this will not work unless we can fix Thrift upstream (or switch Thrift implementations if there is one).

So from my example above the 64 bit int should be<PHONE_NUMBER>0834112909, which PHP cannot handle.
We can use BC Math to get the true value as a string.
function bchexdec($hex) {
$dec = 0;
$len = strlen($hex);
for ($i = 1; $i <= $len; $i++) {
$dec = bcadd($dec, bcmul(strval(hexdec($hex[$i - 1])), bcpow('16', strval($len - $i))));
}
return $dec;
}
That would return the correct value for my example id.
But it doesn't look like the Thrift library can support a string representing an I64.
I guess I could modify my upstream id generators to only use 32 bit ids, but as many are starting to use 128 bit IDs, would be good to get a fix.

@chingor13 I think this should be fixed by my PR

Fixed in #8


python310Packages.ghapi: 1.0.1 -> 1.0.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.ghapi is: "Python interface to GitHub's API"
meta.homepage for python310Packages.ghapi is: "https://github.com/fastai/ghapi"
meta.changelog for python310Packages.ghapi is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.3 with grep in /nix/store/vrllsd3fghpkbir8dnj7ymp7xczhsnnj-python3.10-ghapi-1.0.3
found 1.0.3 in filename of file in /nix/store/vrllsd3fghpkbir8dnj7ymp7xczhsnnj-python3.10-ghapi-1.0.3

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
ghdorker
python310Packages.ghapi
python39Packages.ghapi

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vrllsd3fghpkbir8dnj7ymp7xczhsnnj-python3.10-ghapi-1.0.3
ls -la /nix/store/vrllsd3fghpkbir8dnj7ymp7xczhsnnj-python3.10-ghapi-1.0.3/bin

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

3 packages built:

ghdorker
python310Packages.ghapi
python39Packages.ghapi

Maintainer pings
cc @fabaff for testing.

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

3 packages built:

ghdorker
python310Packages.ghapi
python39Packages.ghapi


Made mbedtls depend on oelibc.

The PR does the following:

Make mbedtls depend on musl headers and musl libc instead of corelibc headers and oecore.
Remove corelibc headers includes from oeenclave/oecrypto/mbedtls code.
Rewrite the mbedtls CMake build so the dependencies between the mbedtls libraries, oelibc and oecore
are correctly defined when installed.
Change the attestation test to use libc headers instead of corelibc.

bors try

bors try

bors r+


No way to set broker.SubscriberOptions

Upon looking at creating an example which reads messages from a broker, performs work on it (Such as writing to a database) and exposes grpc methods, I could not find a way to configure all possible broker options.
The idea was to do the following
package main

import (
	"flag"
	"log"

	"github.com/micro/go-micro/server"

	micro "github.com/micro/go-micro"
    "github.com/micro/go-plugins/broker/rabbitmq"
)

func main() {

    rmq := rabbitmq.NewBroker()

	// Create a new service. Optionally include some options here.
	service := micro.NewService(
		micro.Name("greeter"),
		micro.Broker(rmq),
	)

	// Init will parse the command line flags.
	service.Init()

	// Register handler
	tags.RegisterGreeterHandler(service.Server(), new(Greeter))

	micro.RegisterSubscriber("#", service.Server(), ProcessGreets, server.SubscriberQueue("greets"))

	// Run the server
	if err := service.Run(); err != nil {
		log.Fatal(err)
	}
}

func ProcessGreets(ctx context.Context, event *greeter.Greet) error {
	fmt.Printf("Received tag %v\n", event)
	return nil
}

It seems that the broker.SubscribeOptions struct allows for configuring certain properties of the broker such as AutoAck (There are even helper functions such as broker.DisableAutoAck()) however when trying to create a subscriber there seems to be no way to configure the subscriber broker.SubscriberOptions. As a consequence it does not seem to be possible to configure autoack at the moment and it is always true (Which might not be desired as it is clearly shown in the rabbitmq plugin which handles such scenario, should it be possible to configure it). The micro.RegisterSubscriber method allows for passing server.SubscriberOption which does not expose all possible options.
Should that option not be exposed for the service to configure it?

This change was merged to include subscribe options 67a738b
If you want to set broker options please preconfigure the broker before passing it in.

i found rabbitmq can not set the durableQueue of broker options. it only can set the name of queue and AutoAck.


fix bug in ss.find('0')

Solves:
opt = device.GetDeviceOption(mvnc.DeviceOption.OPTIMISATION_LIST)
File "/usr/local/lib/python2.7/dist-packages/mvnc/mvncapi.py", line 190, in GetDeviceOption
end = ss.find(0)
TypeError: expected a string or other character buffer object

Failed to build on Raspberry Pi 3B - Stretch
Don't know what went wrong, followed the steps.


Is there a way to inject middleware before hitting the operation logic?

First, I love this tool! Thanks a bunch for putting this together.
Second, I'm looking for something like a "preResponseHandler" of sorts. I'm looking to check a specific header on the request and potentially circumvent the operation if a condition exists. Is there something like that today?

Currently there is no equivalent of preResponseHandler.
Closest thing to what you're describing would be a security handler, but that's really meant to be specific for authorizing requests against OpenAPI security schemes.
The reason there's no catch-all preResponseHandler implemented is that I've had no need for it so far. All the pre-handling I've needed to do has been easy to do before calling handleRequest(), but thinking more about this I can definitely see cases similar to what you described where it would be nice to hook before calling the operation handler but still have access to the OpenAPI Context object.
I'll put this into the backlog of ideas! Feel free to submit a PR. Should be pretty easy to add this. :)


Packages aren't licensed properly

They should be following https://input-output.atlassian.net/wiki/spaces/EN/pages/819757219/Open-Source+License
This has actual annoying consequences: the various x2nix tools carry over the license metadata, and assume that these packages are therefore unfree, which requires some additional fussing to allow them. It would be simpler if they were just licensed properly!

I had 10 minutes so I made an issue. Perhaps if nobody else does anything I'll make a PR in a bit ;)


wayfinding: layer view

re: https://github.com/CartoDB/cartodb/issues/13127

acceptance
check the layer view has the new look as per the wayfinding document, and margins and colors won't break even in advance mode

retest this, please

Deployed to ded15, @CartoDB/design could you take a look please <3

The center map icon is black:

The margins are wrong in the widgets and edit feature tabs:

Back to development

Separation between arrow icon and breadcrumb should be 8px
“Layer list” should be “Layers list” (in the Edit geometry UI)
"Widget list" should be "Widgets list" (These are both a mistake on our end, sorry for that!)
Hover style of “Layer list”, “Widget list”, and “Back” shouldn't show an underline
The first layer color should be #11A579
The sync icon should stay fixed to the right

We're missing the breadcrumb in the Basemap options

If you're anywhere but the Layers list, and you access let's say, a geometry option... if you then click on "Layers list" in the breadcrumb to go back, it won't take you to the layers list but to where you were before. For example: go to the Basemap options, then click on a point in the map to edit it, then click on "Layers list" and see where it takes you.

Redeployed to ruis, everything should be fixed but the last point, which we can't do, because of how the navigation works @arianaescobar

Can you elaborate on that please? @rubenmoya

sure, this has already been addressed, and the problem might rely on the breadcrumb literal being wrong, rather than the behaviour
https://github.com/CartoDB/cartodb/issues/10472
note the "wherever the user would be before clicking on edit"

Then maybe we should always put "Back" in the breadcrumb instead... I'll think about it.

So, can we merge this tomorrow and change the breadcrumbs copy in the future if needed?

I would rather change it before merging.

Yeah, please, let's change them all to "Back" 🙂

On it! :) thank you!


[Suggestion] Reduce number of different components

The Components 13 (R 3K) and 14 (R 1K) on the BOM only appear once in the schematics to form a voltage divider.
It doesn't seem that there will be much current over this voltage divider, so i'd suggest reproducing it with a 10K and 3x 10K resistors.
That way, all resistors in the project would be 10K's.
Less rails required for a pick&place machine and no way to misplace when manual soldering.
Same method would probaply be possible for some capacitors.
Many of them seem to be only for buffering&filtering purposes without needing to be an exact value.

Good point! I'll look to change that. Should make things a little easier and cheaper to make.


No Connection to rTorrent

After the installation of rtinst, everything was working fine (ubuntu 20.04)
What I forgot, was that my big hard drive was not mounted at the right place for default configuration. So after an hour of downloading my hard drive got full.
I updated the .rtorrent.rc file accorcing to my setup and created the right folder in it.
method.insert = cfg.basedir,     private|const|string, (cat,"/home/jlafleur/data/torrents/")
method.insert = cfg.watch,       private|const|string, (cat,(cfg.basedir),"watch/")

#method.insert = cfg.logs,       private|const|string, (cat,(cfg.basedir),"log/")
#method.insert = cfg.logfile,    private|const|string, (cat,(cfg.logs),"rtorrent-",(system.time),".log")

session.path.set = (cat,(cfg.basedir),".session")
directory.default.set = (cat,(cfg.basedir),"download")

I rebooted the machine and checked with ps if rtorrent is running
jlafleur    2220  0.0  0.0   5736  2460 ?        Ss   14:30   0:00 SCREEN -d -m -S rtorrent rtorrent
jlafleur    2222  1.3  1.0 1065348 124416 pts/0  Dsl+ 14:30   2:02 rtorrent

Everything look like it's fine, but when I go to rutorrent, I got this error:
Bad response from server: (504 [error,getplugins])
I tried many things, and can't find what's wrong. (I checked that PHP is working well, I created an info.php file with phpinfo() inside of the file and I have the right information)
That's really strange.
Everything was working well before I changed the rtorrent.rc config file... I only changed the basedir path. I don't know what else to do.
Thank you.

This isn't directly related to rtinst and hence you won't get any further support with rTorrent/ruTorrent issues. Regardless, it could be as easy as file permission if you have not set correct file permission for the new directory which rTorrent is going to use... Verify that everything is correct setup. If that doesn't work, revert the changes done to .rtorrent.rc. Do you still get the same error? Have you tried to attach to rTorrent with screen? Do that to see if rTorrent is running and there are no issues. Enable logging for rTorrent if you need more information about what's going on. If you still don't have figued this out, ask over at rTorrent or ruTorrent depending on what you discover and where you think the issue is. Feel free to comment when you have figured the issue as other people might find it helpful 👍

It's effectively not related to rtinst. I have to say that this tool is working FLAWLESSLY on Ubuntu 20.04. Congrats for this awesome work.
I would propose a new feature tho, when installing, ask path for the .rtorrent.rc to configure downloads etc. Execpt that, everything is awesome.

That could be added, however, most people probably are going to change more to the file than just the download path. In my case, I change almost 20 lines. This can easily be done with sed, as example throttling download rate: sed -i "/^\s*throttle.global_down.max_rate.set = */ c\throttle.global_down.max_rate.set = $DLRATE" $RTORRENT_RC.


From 3.0.5 to 3.0.7, What is the difference with slidesPerView: 'auto'

I used to use the property slidesPerView: 'auto' to make a slide to reload effect with swiper v3.0.5.
But when I had update it to the version 3.0.7 it is not working any more. I have tried a lot of properties to figure what is the difference but I haven't find any solution to my problem.
This is my code that is working fine with v3.0.5:
new Swiper(element, {
        slidesPerView: 'auto',
        initialSlide: 1,
        longSwipesRatio: 1,
        wrapperClass: 'elementWrapper',
        slideClass: 'elementSlide',
        allowSwipeToNext: false,
        touchMoveStopPropagation: false
 })

I think the problem is with the 'allowSwipeToNext', When I removed it the swiper works fine. But I need it to prevent the user from swiping the element to the left side.
The second slide (The content of the element (the white slide)) should be the initial slide.
![Swipe to reload]
(http://s17.postimg.org/6le4lpajz/Screen_Shot_2015_05_29_at_8_01_36_PM.png)

I am not sure that i understood what you are trying to achieve. Do you have a live link or JSFiddle?

I uploaded it to this link:
http://novie.us/test
Better to run it in chrome DevTools mobile device mode.
The code of the swiper is here:
http://novie.us/test/app/novie-list/controllers/NovieListCtrl.js
At line 228
The problem is "allowSwipeToNext" when removed it every thing works fine, but I need it to prevent the user from slide to next slide. allowSwipeToNext works fine with 3.0.5.

Ok, it is actually correct. It just doesn't allow you to swipe/set to 2nd slide because of allowSwipeToNext: false flag. Instead of this flag, you may just set resistanceRatio:0 to prevent oversliding after 2nd slide

Closing as no further response or discussion


Add exhaustion to damage source and add some javadocs

SpongeAPI | SpongeCommon
Adds the ability to set and retrieve the exhaustion of a damage source. Also adds some Javadocs while I'm at it. I'm open to changes of the defaults for exhaustion and where I set the exhaustion when building the damage sources(refer to common PR).

Issues should be addressed, still looking for feedback(and testing) here and on the common PR.


Return $.cookie so AMD/CJS user can use it directly

By returning $.cookie you will allow AMD/CJS user to just
var $cookie = require('jquery.cookie');

instead of
var $ = require('jquery');
require('jquery.cookie');

which could be avoid because jquery.cookie has jquery as dependency.

I've merged #339, which is supposed to fix this.


[release-1.9] Fix echo readiness probes when bound to other IPs

This is an automated cherry-pick of #34375

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio_release-1.9
551189b0d21a71e1b1cf35a99b0fd8f484ad8e4e
link
/test integ-security-multicluster-tests_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Getting syntax error running exampleFunctional.php

When I plug in my credentials, I'm getting the following error:
PHP Parse error:  syntax error, unexpected '[' in /home/ec2-user/scripts/snapchat/src/snapchat.php on line 179
Any idea what I'm doing wrong?

@flyxx have you downloaded latests code? its working for me.

Yes, its fresh as of a few minutes before I posted the issue.

@flyxx are you running it locally? If so, have you installed the dependencies? https://github.com/mgp25/Snap-API/wiki/Dependencies
Im going to check the code anyway.

Yes, all dependencies are installed.  BTW, if you want to add the commands for RHEL/Centos/Amazon, they are:
yum install php-gd
yum install php-curl
yum install php-mcrypt
yum install php-zip

@flyxx What is your PHP version?

5.3

https://github.com/mgp25/Snap-API/compare/master...sallaben:patch-1 fixes this error

@sallaben thanks
Fixed: https://github.com/mgp25/Snap-API/commit/4891213d5f99ae648e17b34d21cbe57e10053372


java8

希望能支持java1.8

Hi there!
Closing this topic as it's not an issue with reproduceable steps.

@riyuexc Java 8 has reached end of life years ago and will not be supported. The minimum Java version for this plugin is Java 11, as mentioned in the README. If I do remember correctly Burp Suite also requires at least Java 11 for extension development.


CI: Switch to node 16

cc @billyvg I don't have any permissions or access to this action, but this cause issues in backend workflows:  https://github.com/getsentry/getsentry/actions/runs/5226902812

I don't have write access anymore


[HELP] In app purchase on tvOS

Hi guys, I'm having problems with completing a purchase on tvOS (in iOS is working perfectly)
The error that shows the Xcode log is:
[react-native-iap] Purchase Started 2022-12-27 16:32:25.649576-0600 App-tvOS[6629:167481] <SKPaymentQueue: 0x283831c80>: Payment completed with error: Error Domain=ASDServerErrorDomain Code=1003 "Ningún token disponible cuando se esperaba." UserInfo={storefront-country-code=NIC, client-environment-type=Sandbox, AMSServerErrorCode=1003, NSLocalizedDescription=Ningún token disponible cuando se esperaba.}
[react-native-iap] Purchase Failed
[react-native-iap] removedTransactions - countPendingTransactions 0
2022-12-27 16:32:25.702119-0600 App-tvOS[6629:167098] [javascript] 'PURCHASE ERROR', [Error: An unknown error occurred]
Both the ios App and the tvos app are using the same bundle ID.
I'm using a real Apple TV device.
Extra info:

"react-native"<EMAIL_ADDRESS>"react-native-iap": "^10.1.0"
"react": "18.0.0"

Running on iOS 16 and tvOS 16

This seems to be the problem: https://stackoverflow.com/questions/1717700/iphone-in-app-purchase-store-kit-error-1003-cannot-connect-to-itunes-store


Coq Platform 8.14~2022.01 on macOS 10.13.6

Unable to run Coq Platform 8.14~2022.01 on macOS 10.13.6 using the binary installer at:
https://github.com/coq/platform/releases/tag/2022.01.0
which states:
"This installer is only compatible with macOS 10.13 or higher."
However, attempts to run coqtop, coqc tools result in:
dyld: lazy symbol binding failed: Symbol not found: ____chkstk_darwin
Referenced from:
/Applications/Coq-Platform-8.14~2022.01.app/Contents/Resources/bin/coqtop
(which was built for Mac OS X 11.0)
Expected in: /usr/lib/libSystem.B.dylib
dyld: Symbol not found: ____chkstk_darwin
Referenced from:
/Applications/Coq-Platform-8.14~2022.01.app/Contents/Resources/bin/coqtop
(which was built for Mac OS X 11.0)
Expected in: /usr/lib/libSystem.B.dylib
Attempts to launch CoqIDE result in:
Dyld Error Message:
Symbol not found: ___kCFBooleanTrue
Referenced from:
/Applications/Coq-Platform-8.14~2022.01.app/Contents/Resources/bin/../lib/dylib/libgdk-3.0.dylib
(which was built for Mac OS X 11.0)
Expected in: /System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation
in /Applications/Coq-Platform-8.14~2022.01.app/Contents/Resources/bin/../lib/dylib/libgdk-3.0.dylib

Let's also repeat what you said on Zulip:

Looking around web, other applications have had this problem too, and that Xcode 13.1 will work. It is still available at
https://developer.apple.com/download/all/?q=Xcode 13

I was hesitant to add that to the issue because I haven't verified their claim, but it is worth noting.

On 20 Feb 2022, at 7:29 AM, Théo Zimmermann @.***> wrote:
Let's also repeat what you said on Zulip:
Looking around web, other applications have had this problem too, and that Xcode 13.1 will work. It is still available at
https://developer.apple.com/download/all/?q=Xcode 13 https://developer.apple.com/download/all/?q=Xcode 13
—
Reply to this email directly, view it on GitHub https://github.com/coq/platform/issues/220#issuecomment-1046227282, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH4PZKCPZMTNT2R3CELDHDLU4DNB5ANCNFSM5O3TEFQQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.

Current XCode is 13.2.1 (or so) and I doubt that there are changes in supported platforms between 13.1 and 13.2.1. it is likely more a question of command line options and/or the OS version you run it on.

I looked around a bit. As far as I can tell the XCode version just sets the default lower limit for the supported OS, but (again afaict) one can override this. The XCode gcc gives the following infos:
$ gcc --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)
Target: x86_64-apple-darwin20.6.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

$ gcc --help
OVERVIEW: clang LLVM compiler

USAGE: clang [options] file...

OPTIONS:
:
  -mmacosx-version-min=<value>
                          Set Mac OS X deployment target

I guess all one has to do is to issue -mmacosx-version-min=something. I am not sure I can inject this to opam via environment variables - afaik it does some environment sanitation.

See "https://github.com/ocaml/opam/issues/5064".

Besides I think one has to inject this into MacPorts / Homebrew as well ...

We should have updated the release notes of 2021.01 to acknowledge that the Coq Platform binary installer doesn't work with macOS 10.13, shouldn't we? We know that it works with macOS 10.15, but we have no information regarding macOS 10.14 IIUC.

I tested export MACOSX_DEPLOYMENT_TARGET=10.14 (see https://clang.llvm.org/docs/CommandGuide/clang.html) before running the Coq PlatForm script and this does work:
otool -l $(which coqc)
:
Load command 9
      cmd LC_BUILD_VERSION
  cmdsize 32
 platform 1
    minos 10.14
      sdk 12.1

@Zimmi48 @richfh : so what should I set? 10.13? I don't think it matters much for Coq and CoqIDE.
Btw.: It will be hard to guarantee that this works for everything, so I would say we recommend 11.0, but try to be compatible to 10.13.

On 29 Sep 2022, at 10:50 AM, MSoegtropIMC @.***> wrote:
@Zimmi48 https://github.com/Zimmi48 @richfh https://github.com/richfh : so what should I set? 10.13? I don't think it matters much for Coq and CoqIDE.

That would be great for me, assuming it breaks nothing else!
Unclear how many other users there are still on 10.13.
It is the end of the line update for one of my machines – to be replaced by a new machine running macOS 11.
— Rich

We have some data from the Coq Community Survey from February 2022: https://coq.discourse.group/t/coq-community-survey-2022-results-part-ii/1746


Made javaslang.Iterable package private

Fixes #808

Current coverage is 89.19%

Merging #809 into master will not affect coverage as of 65a7790

@@            master    #809   diff @@
======================================
  Files           77      77
  Stmts         8076    8076
  Branches      1738    1738
  Methods          0       0
======================================
  Hit           7203    7203
  Partial        376     376
  Missed         497     497

Review entire Coverage Diff as of 65a7790

Powered by Codecov. Updated on successful CI builds.


Can't access headers

Consider the following Sinatra app:
class App < Sinatra::Application
  post '/user/create' do
    api_key = headers[ 'X-Blah-Token' ]
    body    = request.body.read
    # ...
  end
end

How do I mock the needed header (i.e., X-Blah-Token)
describe 'foo' do
  include Rack::Test::Methods

  def app
    App
  end

  it 'completes successfully' do
    body   = { 'name' => 'joe' }.to_json
    header 'X-Blah-Token', '12345...'

    post '/user/create', body # => Doh! header arrives as 'HTTP_X_Blah_Token'
    expect ...
  end
end

@j4zzcat are you sure this has been resolved now?

@j4zzcat I am not sure I entirely understand your question, but there seems to be a confusion between request headers and response headers in your example.
Inside a sinatra handler, the method headers deals with response headers. It allows you to set a header for the browser to receive. You can read a header as well, but not a request header, only response header. A response header is rarely set at this point unless it was set by a previous middleware in your rack stack.
If you want to read a request header — one that is set by the browser or rack-test request — you can call it this way:
api_key = request[ 'X-Blah-Token' ]

Now on the request side, the way you send a request header is like you did as far as I know, or you can pass a hash as the 3rd argument of the post method.

Thanks for the explanation @mig-hub. 👍 I'll close this for now, feel free to reopen @j4zzcat if there's still an open issue here.


Unable to add watch variables when debugging Python

VSCode Version: Code - Insiders 1.20.0-insider (ed4c237467bfe7c37b173da782cb65e333484a13, 2018-01-26T06:08:15.259Z)
OS Version: Darwin x64 17.3.0
Extensions (16)

Extension
Author (truncated)
Version

vscode-generate-getter-setter
DSK
0.4.2

xml
Dot
1.9.2

gitignore
cod
0.5.0

vscode-eslint
dba
1.4.4

githistory
don
0.3.8

tslint
eg2
1.0.24

vscode-npm-script
eg2
0.3.3

docthis
joe
0.6.0

extension-manifest-editor
ms-
0.1.5

python
ms-
0.9.1

vscode-ai
ms-
0.1.6

azurecli
ms-
0.4.0

debugger-for-chrome
msj
4.1.0

typescript-hero
rbb
2.1.1

vscode-icons
rob
7.19.0

vscode-open-in-github
ziy
1.3.1

Steps to Reproduce:

Install the Microsoft Python extension
Debug the following python code:

for i in range(1,10):
    print(i)

Add a break point to the print statement
Try adding a variable in the watch window, for some reason it doesn't work

Clicking the + symbol once disables the + icon and nothing happens.

Reproduces without extensions: This is specific to the extension.
Original issue: https://github.com/Microsoft/vscode-python/issues/646

@isidorn the DAP spec says that the 'expression' attribute for the evaluateRequest must not be missing or undefined. So it is not the DA's responsibility to protect against this case.
See https://github.com/Microsoft/vscode-debugadapter-node/blob/b61cf04501cfe7356b695f99bc60961684f89a4f/protocol/src/debugProtocol.ts#L886

@weinand agree. Though ideally they should be more error tolernt and not leave pyhton process lying around :)

oh yes, definitely!


ProgrammingError while running SQL DML Statements

Driver version
2.0.913
Redshift version
PostgreSQL 8.0.2 on i686-pc-linux-gnu
Client Operating System
Linux
Python version
3.8
Table schema

Problem description

Expected behaviour: Redshift python driver should not throw any error for cursor.fetchall() while executing SQL DML statements.
Actual behaviour: Redshift python driver throws ProgrammingError for cursor.fetchall() while executing SQL DML statements.
For eg:
Run:

res_cursor.execute("create table sample_table_from_sm (col1 int);")
data = res_cursor.fetchall()

Output:
ERROR:c8e60ff5311f4a709f6ea83e092d3d4c:Error while executing query no result set
Traceback (most recent call last):
  File "/opt/conda/lib/python3.8/site-packages/redshift_connector/cursor.py", line 498, in __next__
    return self._cached_rows.popleft()
IndexError: pop from an empty deque

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/conda/lib/python3.8/site-packages/**********_sql_execution/redshift/connection.py", line 61, in execute
    data = res_cursor.fetchall()
  File "/opt/conda/lib/python3.8/site-packages/redshift_connector/cursor.py", line 456, in fetchall
    return tuple(self)
  File "/opt/conda/lib/python3.8/site-packages/redshift_connector/cursor.py", line 503, in __next__
    raise ProgrammingError("no result set")
redshift_connector.error.ProgrammingError: no result set
ERROR:root:Error executing SQL query:
create table sample_table_from_sm (col1 int);

no result set

P.S. the same issue occurs for an UPDATE query.

Error message/stack trace: Attached above
Any other details that can be helpful:

Python Driver trace logs

Reproduction code

Hi @avneeshn, are you familiar with the Cursor.rowcount property? I believe this may be what you're looking for. After executing a UPDATE or CREATE statement, Cusor.rowcount will provide the number of rows produced by the last call to execute(). Cursor.fetch__ methods return the result set returned from statement execution. Given CREATE and UPDATE statements do not produce a result set, raising an exception is the expected behavior per the DB-API spec

Thanks for the comment @Brooke-white. So, we are developing an SQL client for our users to run queries. So we have no way of knowing in advance if it is a DML statement, or DDL/SELECT query. That is why we are using cursor.fetchall() everytime.
Regarding your approach regarding Cusor.rowcount, I believe this property will be non-zero for both an UPDATE query and a SELECT query. Do you know how can we use this property (or another approach) to identify whether a call to cursor.fetchall() needs to be made or not?

I'd recommend adding try/catch logic around the call to fetchall(), where the catch statement catches redshift_connector.ProgrammingError


Can't install successfully

I've tried installing from pip3, the official deb release, and the ppa and I can't get any of them to work. Here's what happens after installing the .deb. I've tried putting empty JSON files at ~/.config/emoji-keyboard.json and ~/.local/share/recent_emoji.json (which it complains about if emoji-keyboard.json exists) but then it just starts complaining that configuration keys are missing (correctly, since the files are just {}). What am I doing wrong here?
$ emoji-keyboard
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/emoji_keyboard/emoji_shared.py", line 33, in <module>
    with open(settings_file) as fd:
FileNotFoundError: [Errno 2] No such file or directory: '/home/rsilva/.config/emoji-keyboard.json'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/emoji-keyboard", line 5, in <module>
    from emoji_keyboard import emoji_shared as shared
  File "/usr/lib/python3/dist-packages/emoji_keyboard/emoji_shared.py", line 35, in <module>
    except (FileNotFoundError, json.decoder.JSONDecodeError):
AttributeError: 'module' object has no attribute 'JSONDecodeError'

emoji-keyboard is written on python 3.5 and that's what it requires, you seem to have an older python version.
Anyway, I've fixed  this particular error in the master branch, could you try installing with pip3 again?
There's likely to be more bugs, but they're not hard to fix.

Thanks, it works now! I haven't seen any more showstopping bugs yet. 👍⬅ typed using emoji keyboard.

Great! If anything else pops up, let me know.


Removal of the Santa Hat on vscode insiders is very offensive to me

Removing the Santa Hat on vscode insiders is very offensive to me. Getting offended by one person is against my atheist religion. Bowing before SJW's as part of a product update is completely unacceptable. Please readd it immediately and make it your top priority. To me this is almost equally offensive as censorship.

VSCode Version: 1.42.0-insider
OS Version: Windows 10 1909

Steps to Reproduce:

Get offended.

Does this issue occur when all extensions are disabled?:

Too late! Spreading it wide on the internet of the hilarity of binding to an SJW troll.

Given that the article the original author linked was titled "What does Judaism say about Christmas", the site it is hosted on is a self-identified religious text repository, and the article itself describes how Christmas is now de factor secular holiday, I believe the original issue author was incontestably acting in bad faith.

Bring back the Santa hat!

Duplicate of #87291

You've changed this icon to special snowflake xD Great idea


Improving the performance

It's time to improve project's performance of redux-go

Since no support 1.7 & 1.8 anymore, I think can use sync.Map to help reducers update concurrency!

After pprof, I found it sync.Map just make redux slower, and pretty hard to improve current code now.


INFO log message including "HTTP" is misleading when using HTTPS

When using an HTTPS DNS, the following is logged:
INFO com.getsentry.raven.DefaultRavenFactory : Using an HTTP connection to Sentry.
This is unnecessarily alarming, if you are intending to use HTTPS

You're right, I'll fix, thanks!


Grip points don't work on Windwos

User claims that resizing doens't work on Windows OS

works now


Populate defaults in appConfig.js by postinstall

Summary
Currently, following the readme instructions results in everything being disabled. This fixes that by copying the defaults from ui_attributes.common.json into appConfig.js so that the defaults are consolidated in one place in the repo, while keeping the local separation of config.
Checklist

[x] Tested changes end to end
[x] Added PR Label "ready-for-review"
[x] Requested one or more reviewers for the PR

This looks great, the only thing is the features value populated in the appConfig is a json object but it should be a javascript definition

Yeah, this isn't great. I tried doing some processing to remove the unnecessary quotes, but it ended up being fragile. I'll look for some other solutions.


ping failed from node to underlay pod

Expected Behavior
ping from node to underlay pod gets replied
Actual Behavior
no reply
Steps to Reproduce the Problem

create underlay subnet and set u2oInterconnection: true (with namespace of test)
create pod in namespace of test
ping above pod from node

Additional Info

Kubernetes version:
Output of kubectl version:

WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.6", GitCommit:"b39bf148cd654599a52e867485c02c4f9d28b312", GitTreeState:"clean", BuildDate:"2022-09-21T13:19:24Z", GoVersion:"go1.18.6", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.6", GitCommit:"b39bf148cd654599a52e867485c02c4f9d28b312", GitTreeState:"clean", BuildDate:"2022-09-21T13:12:04Z", GoVersion:"go1.18.6", Compiler:"gc", Platform:"linux/amd64"}

- kube-ovn version:

1.11.0

- operation-system/kernel version:

**Output of `awk -F '=' '/PRETTY_NAME/ { print $2 }' /etc/os-release`:**
**Output of `uname -r`:**

Red Hat Enterprise Linux 8.7 (Ootpa)
4.18.0-425.3.1.el8.x86_64

<!-- Any other additional information -->

If disable u2oInterconnection, ping works fine.

Expected Behavior
ping from node to underlay pod gets replied
Actual Behavior
no reply
Steps to Reproduce the Problem

create underlay subnet and set u2oInterconnection: true (with namespace of test)
create pod in namespace of test
ping above pod from node

Additional Info

Kubernetes version:
Output of kubectl version:

WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short. Use --output=yaml|json to get the full version. Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.6", GitCommit:"b39bf148cd654599a52e867485c02c4f9d28b312", GitTreeState:"clean", BuildDate:"2022-09-21T13:19:24Z", GoVersion:"go1.18.6", Compiler:"gc", Platform:"linux/amd64"} Kustomize Version: v4.5.4 Server Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.6", GitCommit:"b39bf148cd654599a52e867485c02c4f9d28b312", GitTreeState:"clean", BuildDate:"2022-09-21T13:12:04Z", GoVersion:"go1.18.6", Compiler:"gc", Platform:"linux/amd64"}

- kube-ovn version:

1.11.0

- operation-system/kernel version:

**Output of `awk -F '=' '/PRETTY_NAME/ { print $2 }' /etc/os-release`:**
**Output of `uname -r`:**

Red Hat Enterprise Linux 8.7 (Ootpa) 4.18.0-425.3.1.el8.x86_64

<!-- Any other additional information -->

If disable u2oInterconnection, ping works fine.

please give detail information, because I can't reproduce the bug

ip route on node, the nexthop should be underlay subnet gateway

eg: <IP_ADDRESS> is the underlay pod
# ip r get <IP_ADDRESS>
<IP_ADDRESS> dev br-8ac9461a1c7a src <IP_ADDRESS>

ip neigh on underlay gateway, there should be underlay pod's ip arp on it.

# ip n |grep <IP_ADDRESS>
<IP_ADDRESS> dev br-8ac9461a1c7a lladdr 00:00:00:4c:93:bd STALE

tcpdump on your underlay pod, and see if the packets get into the pod

or  on the host nic for pod
# tcpdump -i 69cff00d2dbe_h -nel
tcpdump: verbose output suppressed, use -v[v]... for full protocol decode
listening on 69cff00d2dbe_h, link-type EN10MB (Ethernet), snapshot length 262144 bytes

4 see the ip route on underlay pod :
kubectl exec -it pod-underlay -- ip r

<IP_ADDRESS> is the underlay pod
<IP_ADDRESS> is the node

ip route on node, the nexthop should be underlay subnet gateway

[@node1 ~]# ip r get <IP_ADDRESS>
<IP_ADDRESS> via <IP_ADDRESS> dev br-network-v11 src <IP_ADDRESS> uid 0
    cache

ip neigh

[@node1 ~]# ip n |grep <IP_ADDRESS>
[@node1 ~]#

tcpdump on the underlay pod

[@node1 ~]# kubectl ko tcpdump test/test-pod-mirror -nn -v icmp
Defaulted container "cni-server" out of: cni-server, install-cni (init)
Defaulted container "cni-server" out of: cni-server, install-cni (init)
+ '[' veth-pair = internal-port ']'
+ kubectl exec kube-ovn-cni-p6c6m -n kube-system -- nsenter --net=/var/run/netns/cni-9578c974-8501-c9bb-3efe-1fcd5b8d9db9 tcpdump -nn -i eth0 -nn -v icmp
Defaulted container "cni-server" out of: cni-server, install-cni (init)
tcpdump: listening on eth0, link-type EN10MB (Ethernet), snapshot length 262144 bytes
11:59:27.029895 IP (tos 0x0, ttl 63, id 15783, offset 0, flags [DF], proto ICMP (1), length 84)
    <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 213, seq 11, length 64
11:59:27.029928 IP (tos 0x0, ttl 64, id 55380, offset 0, flags [none], proto ICMP (1), length 84)
    <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 213, seq 11, length 64

on the host nic for pod
[@node3 ~]# tcpdump -i 7d07ef2b37ff_h -nn icmp
dropped privs to tcpdump
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on 7d07ef2b37ff_h, link-type EN10MB (Ethernet), capture size 262144 bytes
12:11:32.101196 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 214, seq 1, length 64
12:11:32.101246 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 214, seq 1, length 64
12:11:33.109915 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 214, seq 2, length 64
12:11:33.109963 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 214, seq 2, length 64
12:11:34.133776 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 214, seq 3, length 64
12:11:34.133817 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 214, seq 3, length 64
12:11:35.158774 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 214, seq 4, length 64
12:11:35.158816 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 214, seq 4, length 64

tcpdump on the underlay subnet gateway
subnet gateway is managed by another team

ip route on underlay pod:

[@node1 ~]# kubectl exec -ti -n test test-pod-mirror -- ip r
default via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/23 dev eth0 scope link  src <IP_ADDRESS>

it turns out that your underlay pod can receive packets from the node, and gives a reply.

can you tcpdump on your node gateway or underly subnet gateway, and find if is there any reply packet?

when you turn on the u2ointerconnection switch. you should reboot your underlay pod, your pod's default route will change nexthop to subnet.status.u2ointerconnectionIP


Make sure passed in block devices are valid block devices

Right now one can pass in any xyz string in DEVS= and there are no error
checks and sfdisk tries to partition it and fails with odd error messages
like as follows.
INFO: Volume group backing root filesystem could not be determined
./docker-storage-setup.sh: line 463: * 2 - 2048 : syntax error: operand expected (error token is "* 2 - 2048 ")
ERROR: No valid volume group found. Exiting.
Instead check for valid block devices early and fail gracefully.
Signed-off-by: Vivek Goyal<EMAIL_ADDRESS>

CC @jwhonce @cgwalters @rhatdan @cevich Any review/testing is appreciated.

@rhatdan Ok, I took care of your comments. Though I had to create more patches in the process but that's all good cleanup.

Other then comment on Infor :+1:

@rhatdan Fixed Info(). Looking for your LGTM and I will merge it.

LGTM.

One think you might want to do with your functions is to output $* or $@ (Never remember which is which.
Then you could just do
Error -n This will not print a newline

WOuld work

Ok, got it. I think that improvement for some other time. Merged PR for now.

Yes I agree.


truncate function not defined for SPIFFS filesystem

Hello everybody,
truncate() function seems to be NOT implemented for spiffs components. If called, it returns -1
and errno is 88 which description in "\esp-idf\components\newlib\include\sys\errno.h" is
#define ENOSYS 88	/* Function not implemented */
Is it a problem of ESP-IDF or is it a problem of  SPIFFS component itself?
As a reference I link a similar issue which is solved (for FATfs):
[(https://github.com/espressif/esp-idf/issues/1980)]
Thanks

https://github.com/pellepl/spiffs/issues/107

Thanks @negativekelvin,
Do you suggest me to try implement the function and then pull request on esp-idf or to pull request to spiffs native repository (pelleel/spiffs)?


Fix invalid route with one segment showing spinner indefinitely

example: https://thirdweb.com/foo shows spinner forever instead of redirecting to a different page
I'm redirecting to 404 now, but we can also redirect to /explore or something ..

📝 Changed routes:

/[profileAddress]

(view full size)

Commit f330473c33277bf9d01e6bee7518192eb704c1f8 (https://thirdweb-qoig6oq3s.thirdweb-preview.com).


More memory/temporary table support

Is your feature request related to a problem? Please describe.
Drift now supports in-memory/temporary DB but as standalone DB.
I need to use in-memory/temporary DB as second DB(attached DB).
Describe the solution you'd like
One possible way for it is
Add option to disable auto table creation for in-memory/temporary table
and option to specify schema for Table
@DisaleAutoCreate
class InMemoryDbTable extends Table (

  @override
  String get schema = 'aux1';    // or   schema() method like alias()
)

  @override
  int get schemaVersion => 1;

  @override
  MigrationStrategy get migration {
    return MigrationStrategy(beforeOpen: (details) async {
      await customStatement('ATTACH DATABASE ':memory:' AS aux1');   // attach to memory DB
      InMemoryDbTable.createTable();  // manually create table
    });
  }

SQLite In-Memory Databases
thx.

Related to #1366 and #1159.
I agree that we should support something like this, but we need a clear way to separate the different schemas. Maybe this could also be a part of the database annotation:
@DriftDatabase(
  tables: [Users],
  additionalSchemas: {
    'aux1': [InMemoryDbTable]
  },
)


Windows: ls command not working

It hangs the actual tab. The shell is blocked even in new tabs.
(Option 2: Double-clickable app Download)

thanks for the bug report; can you try pwd ? i suspect that this is due to us not building the node-pty binary for windows properly -- to do this right, for now, we will need to build our windows distribution on windows; we are currently building the windows distribution on linux.
the underlying issue here is that node-pty-prebuilt has not kept up with releases of node-pty; the result is that we cannot rely on it, and have to build node-pty ourselves.

if my suspicion is true, the javascript console should show something like

cannot load lib/Debug/pty.node

For pwd is a bit different

And new tabs keep working:

BTW dir also fails, so probably we have different problems.

ls

pwd

thanks! upon further reflection, i suspect that the pwd bug is indeed the pty build issue; but ls is probably separately solvable. i'll take a look.

i think we might be good to go here, with #3332 and 5.5.8

it turns out that windows ls works fine when kui is launched with git-bash, but not if it is launched from powershell


EnderIO crashes when Mekanism is not installed (is this intentional)?

I get an instant crash when that mod is not installed with Iconduitbundle being the culprit
http://pastebin.com/fGGPGB5k

EnderIO{1.7.10-<IP_ADDRESS>4} [Ender IO] (EnderIO-1.7.10-<IP_ADDRESS>4.jar) Unloaded->Constructed->Errored

Use 365

I was trying to find the newest version but yesterday and today I was unable to find version 365, unless its not on the jenkins page :/

nvm I found it sorry for the troubles!


ci: add tests for conda install

conda install

Codecov Report

Merging #11 (e52bc0b) into dev (5447051) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #11   +/-   ##
=======================================
  Coverage   59.98%   59.98%
=======================================
  Files          56       56
  Lines        8015     8015
  Branches     1579     1579
=======================================
  Hits         4808     4808
  Misses       2783     2783
  Partials      424      424

Flag
Coverage Δ

unittests
59.98% <ø> (ø)

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

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 5447051...d4125bc. Read the comment docs.


Independent Cell AI

As of the latest update it appears you are able to control each cell mass independently, which is great, yet more than half the time the masses end up splitting up and eventually get absorbed or go out of range making you loose the mass. Can you set a proximity range for the cells so they stay grouped better? Also a way to turn on/off the independent cell AI would be great.

can't you just use follow mouse to regroup the cells?

@Linkaan You can but that would highly defeat the purpose of a bot. Currently the disadvantages of independent cell AI greatly outweigh the advantages because every time you get split, either to attack an enemy cell or by a virus, you will more often than not loose a lot of your mass because the individual cells will get absorbed by an enemy; which chances are wouldn't have happened had you been at your full mass.

The formula for how much time in seconds two cells will take to regroup is as follow:
30 + Combined_Mass * 0.2
I didn't implement this yet. I'll have to experiment with how a cell life time works.

This formula is not accurate for the official servers.

Do you have the right formula? I got it from http://agar.gcommer.com/index.php?title=Formulas#Time_To_Remerge

I am contributing there regularly and know who found that formula. He said it is correct if you have only 2 cells within a tolerance of +-4 seconds.

What I'm thinking is that each cell has a birth time. When you split, you are creating a new cell. That cell has a birth time. You can merge back when that cell has been old enough. I was thinking that the formula could tell you when any two cells could merge back.
One way to test if what I'm saying is true is to split. When you can merge back again, split right before you do. You now have 4 cells. The two parent cells should still be able to merge back while the two child cells won't be able to merge back.

Oh, that's interesting.
Did you record test data or just try it out? I'd like to analyze some more data to get exact results, as the formula is not precise enough for my purposes.
btw, missed your edit, Github only notifies on new posts :/

I simply tried it out. I have an idea about how the formula should look like. I'll try to write it when I have more time.


Typo on page

"The minimum prefix that you need to declare for the local network gateway is the host address of your BGP peer IP address on your VPN device. In this case, it's a /32 prefix of <IP_ADDRESS>/32.  "
There is a type the IP should read "<IP_ADDRESS>/32"

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

ID: ba8af16e-8da4-287b-0a4e-5ff77aa296d8
Version Independent ID: fb643fb3-eab2-110a-f378-ea091d9dddbb
Content: Configure BGP on an Azure VPN gateway: Resource Manager and CLI
Content Source: articles/vpn-gateway/bgp-how-to-cli.md
Service: vpn-gateway
GitHub Login: @yushwang
Microsoft Alias: yushwang

@kenleach Thanks for catching this! I have updated the IP, but the update will not show on the live document until 5/6.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Thanks so much

I really appreciate you keeping these pages up to date

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

ID: 9a9a5ad3-9618-8b83-2357-22c2ae0432fe
Version Independent ID: 7fa1c6a0-3968-b662-2f7f-748cb4c1bacf
Content: Xamarin Docs What's New (March 2019) - Xamarin
Content Source: docs/whats-new/2019-03.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @cillroy
Thanks for your feedback. We aim to publish a what's new page to coincide with each new release of Xamarin.Forms, so you can expect another one in December.


Updates the install manual to reference sdkman tool name over the older gvm name.

First patch to another project and found glide through sdkman.  Saw the install page references the gvm name and thought it would be good to keep up to date.  I raised issue-24 in the project to reflect the alteration.

Thanks for the PR. Big changes are coming to glide soon.


Replace content-editable divs in the overlays with a textarea

As per @viddo suggestion.

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Update link to samples / quickstarts

[ ] docs
[ ] quickstart UI

anywhere else?

i think I am done with that.


[Workshop, EN Localization] The service provider sees localization problems when creating a workshop if he has chosen EN Localization.

Environment: Windows 10, 64-bit, Google Chrome 116.0.5845.179,
Windows 10, 64-bit, Microsoft Edge 114.0.1823.67.
Reproducible: always.
Preconditions

Go to https://oos.dmytrominochkin.cloud/#/.
Log in to the site as a service provider.
Select the English localization of the site.

Steps to reproduce

Go to the "My workshops" page.
Click on the "Add workshop" button.

Actual result
When choosing EN localization when creating a workshop, the service provider sees: untranslated elements; hours of classes do not fit into the frame.

Expected result
When choosing English localization, the workshop creation page is completely translated, all interface elements are displayed correctly.
Labels to be added
"Bug", Priority ("pri: Low"), Severity ("severity: Low"), Type ("UI, "Localization").

Fixed in #2247 except second image selects/labels - they're coming from backend


Tip for making this work in React

If anyone is using this in React, here's how you get React's HTML manipulation to work nicely with this library:
  <style scoped key={styleString}>
        {styleString}
  </style>

Thanks! I might add this in the README.

If anyone is using this in React, here's how you get React's HTML manipulation to work nicely with this library:
  <style scoped key={styleString}>
        {styleString}
  </style>

Great tip, spent 2 days trying to figure out how to inject scoped style tag in a React library, and couldn't make this work.
@samthor:
Definitely add this to the README or it won't work on React, while it works perfectly with a unique BUT also dynamic key attribute.


Add value to hosted control plane namespace label

What this PR does / why we need it:
The label is dropped when reported in metrics if it doesn't have a value.
Checklist

[x] Subject and description added to both, commit and PR.

/lgtm


walletd, grpcd and databased launching not using runtime.spawning_services

I also noticed now that the walletd, grpcd and databased launching are not using runtime.spawning_services
Originally posted by @zkao in https://github.com/farcaster-project/farcaster-node/issues/670#issuecomment-1223854206

As of #693 we now track online services (including syncers, database, peer, and wallet).


WorkItemTrackingCommentsClient get_comments uses invalid API version

response = self._send(http_method='POST',
location_id='608aac0a-32e1-4493-a863-b9cf4566d257',
version='5.1-preview.1',
route_values=route_values,
content=content)
this causes to give a 404 Response
should be  version='5.1-preview.3',

The service shouldn't have broken api compat with version 5.1-preview.1.  I have passed along that feedback to the owning team.  The change to bump to M151 clients is in the works, which will switch these methods to using '5.1-preview.3'.  See #210

Fixed in release: 5.0.0b6
The methods in the WorkItemTrackingCommnetClient have been moved into the WorkItemTrackingClient in M151.


Add functionality to create and join rooms

For Create:  generate a random string and ask them for their nickname
For Join: ask for the string and ask nickname


Is there an existing issue for this?

[X] I have searched the existing issues.

Is your feature request related to a problem? Please describe.
想用一些中文CLIP模型，但towhee目前似乎不支持自定义模型哦
Describe the solution you'd like.
No response
Describe an alternate solution.
No response
Anything else? (Additional Context)
No response

@micrazy Towhee supports user model as well. You have two options below:

wrap the model inference with a function to work with other towhee operators. Here are some example codes:

def encoder(inputs):
    inputs = preprocess(inputs)
    embs = my_model(inputs)
    return embs.detach().cpu().numpy()

towhee.dc(['test']).runas_op(func=encoder).to_list()

towhee.models.clip also supports some multilingual models (support Chinese).  Check README in this page: https://github.com/towhee-io/towhee/tree/main/towhee/models/clip

tks, 我试了下确实可以，不过碰到了另外的问题：

tensor_normalize

tensor_normalize does normalize the input. You can check docs here: https://towhee.readthedocs.io/en/latest/builtin operators/tensor like.html . In your case, I think the problem is that you need to squeeze the vec before normalization. i.e. The proper shape should be (512,) instead of (1, 512), which affects the normalization result.

NB!


AttributeError: 'Index' object has no attribute 'tz..,probably because our research stock market is in China

Please help with handling this error,I did every other thing fine but the tz as I trace along:
if factor.index.get_level_values('date').tz != prices.index.tz:
raise NonMatchingTimezoneError("The timezone of 'factor' is not the "
"same as the timezone of 'prices'. See "
"the pandas method tz_localize.")
Any clue at all?

same problem

You need to set the column 'data' to index.
Here is what I have done to resolve the same problem.
import pytz
from pytz import timezone
dataset['PeriodEnd'] = pd.to_datetime(dataset['PeriodEnd'])
dataset.set_index('PeriodEnd', drop = False, inplace = True)
Mt_Gambier= timezone('Australia/Adelaide')
dataset.index = dataset.index.tz_localize('UTC').tz_convert(Mt_Gambier)
dataset.head()
You will get the result you want.

Hi Leona0616 ,
I've tried following up your guidance but not successful yet. My case the first dataset is multiindex, levels[0] is date in dtype='object', length=506) format, levels[1] is tickers' name, meanwhile the second dataset is single index, the date in dtype='datetime64[ns, UTC]', length=506, freq='C' format.

With thanks

Here is my case of 2 levels pandas index dataframe. Level[0] is datetime, level[1] is ticker.
dataset.index = dataset.index.set_levels([pd.to_datetime(dataset.index.levels[0]).tz_localize('UTC'),dataset.index.levels[1]])
Hope that helps

prices should not have a multi index.  You need to unstack into wide format, for example:
prices.open.unstack(level=1)

Hi Leona0616 , I've tried following up your guidance but not successful yet. My case the first dataset is multiindex, levels[0] is date in dtype='object', length=506) format, levels[1] is tickers' name, meanwhile the second dataset is single index, the date in dtype='datetime64[ns, UTC]', length=506, freq='C' format.  With thanks

have you solved that problem?i meet the same issue

For anybody who get into this problem, this is how I resolve the problem.
			market_cap	pe_ratio	total_operating_revenue
date	code
2019-01-02	000001.XSHE	1577.9608	6.4428	2.942300e+10
2019-01-03	000001.XSHE	1593.4142	6.5059	2.942300e+10
2019-01-04	000001.XSHE	1674.1151	6.7456	2.942300e+10
2019-01-07	000001.XSHE	1672.3981	6.7386	2.942300e+10
2019-01-08	000001.XSHE	1658.6617	6.6833	2.942300e+10

above is my factor df which name is series_facs_datas
#  factor df
series_facs_datas.index = series_facs_datas.index.set_levels([pd.to_datetime(series_facs_datas.index.levels[0]).tz_localize('UTC'),series_facs_datas.index.levels[1]])

# tick
price.index = pd.to_datetime(price.index).tz_localize('UTC')


Define communications strategy (blog posts, social media posts...) after a CVE is released

For CVE-2024-36129 we also published a blog post to spread the word and provide further ciontext about a vulnerability. This may be a good practice (specially for High/Critical security vuilnerabilities). Having a guideline about the content and timing of such blog posts as well as when they are advisable would be helpful.

I'll work on a paragraph for this as part of "Fix Disclosure Process" once we sort out #61 .


Attaching a mdf file.

I am new in docker.
I just wondering how can I attach/restore a mdf file or a bak file using a ubuntu terminal for mssql server?
thanks.

If you want to do it from the terminal you need to use a SQL command line utility like sqlcmd, sql-cli (mssql), usql, isql, sqsh, etc.
Let's say you used sqlcmd.  You would want to pass in a query like this:
sqlcmd -S  -U  -P  -Q "RESTORE DATABASE ......"
To gert sqlcmd you can install the package for Ubuntu by following the instructions here:
https://docs.microsoft.com/en-us/sql/linux/sql-server-linux-setup-tools#ubuntu
If  you want to do this as part of a container deployment strategy you need to create an image which includes the tools in the image.
An example that could help you is here:
https://github.com/twright-msft/mssql-node-docker-demo-app
In that particular example, I used sqlcmd to run a .sql script file but you can just run a command using -Q too.  Or you can put your attach/restore logic in a .sql script file if you want.
https://github.com/twright-msft/mssql-node-docker-demo-app/blob/master/import-data.sh
We are going to make this simpler in the future by implementing the feature requested in this issue: #4

In the CTP 1.3 release we included the tools in the image so you no longer need to create a special image that contains the tools.  @ely029 - Can we close this issue at this point and track making this easier to do as issue #4 ?


Show source button should point to repository source if GitHub, gitlab, or bitbucket is configured

Currently, our "show source" button points to the raw source file that sphinx bundles with a build. However this is a bit awkward for a few reasons:

Sphinx defaults it's html_sourcelink_suffix to .txt, which means that all source files have .txt appended to them by default.
Clicking show source generally opens a raw text version of the file, which is not particularly hemoful as it isn't highlighted etc.
The file is in isolation (sphinx just puts all files in a flat list in _sources) so you don't have any information about the file source relative to the other pages in the site

In cases where somebody has a git repository configured as well, i think it'd be much more helpful if we used "show source" to point to the repository version of the file. From there they'd get better highlighting, more context about the other files in the repo, and the ability to download the raw file. If no remote repository is configured, we could fall back to the source file in the build.

I have a mixed opinion on that specific feature of sphinx. To me the main use here is to view the raw code used in the page to check how they produced this or this output (new directive, extention I don't know about). In short I use it exacltly the same way as the "raw" button in github.
Advantage:

fast to load
show me exactly what is display and not the latest master not yet release file

cons:

no highlight

Now let's assume we do what you suggest. I won't list the advantages as you already did but focus instead on the drawbacks:

incompatibility with Sphinx normal behavior (can be surprising for newcomer)
display something that may not be sync with the currently displayed version
if the file is written in rst or md it will be pre-rendered by github and I won't see the underliying directives (which for me the only use of this button) forcing me to click on "raw" to finally access what I look for.

So even if I think it's not super useful (people can easily deactivate it) I think this modification will make us loose a very niche sphinx feature.
On the other hand what if we add a parameter for the "edit this page in ..." to make it a "see this page in ..." instead (default to the old behavior of course). In this case we are not changing Sphinx way of doing things and we can lead people to the github file being used ?

Maybe a "Show on GitHub" would solve this. Though note that we could construct the GitHub URL to avoid pre-rendering eg:
https://github.com/pydata/pydata-sphinx-theme/blob/main/README.md?plain=1
I think the main downside would be if the content of the page significantly differed from the source on GitHub. Though presumably that would be less of an issue as long as new releases happened (and in my experience many people use /latest as the default on RTD)

This feels unnecessary to me. We already provide an Edit this file that links to GitHub [^1], which will be nicely highlighted. Sphinx provides the Show page source for (I think) exactly the reason @12rambau mentions ("show me source for exactly what is displayed, not the latest main").  If there were a way to know which commit the codebase was on when the docs were rendered, and link to the version of that file at that point in history, then I'd be OK with the proposed change. But that seems hard / not exactly low-hanging fruit, verging on overengineering.

I think the main downside would be if the content of the page significantly differed from the source on GitHub. Though presumably that would be less of an issue as long as new releases happened

I am less confident about this than you are @choldgraf.  We have now 221 forks and 348 stars, and I have no idea how frequently all those folks release (MNE-Python cadence is ~4 months, matplotlib is ~6 months I think; those are the only two I know off the top of my head). But a lot can change in 4-6 months.
[^1]: though I just tried it on our site and got a 404... changing edit to blob in the URL made it resolve (see screenshot), but it wasn't an edit interface:
  ![no-edit](https://user-images.githubusercontent.com/1810515/220759265-196376a5-58fd-464b-84f2-0d3fa683e8f1.png)

2 VS 1 (sorry @choldgraf) do we wait for other inputs ?

Nah I'd say go ahead and wontfix it for now


Without tags/history can we fall back to the version file?

In my project we're setting project/VERSION and reading that in as the basis for file-based versioning using
[tool.setuptools-git-versioning]
enabled = true
version_file = "project/VERSION"
count_commits_from_version_file = true
dev_template = "{tag}.{branch}{ccount}"
branch_formatter = "utils:_branchFormatter"

(branch formatter is there to sanitize branch name to something valid for a version)
Now, if we do clone with depth=1, we get an invalid versions like 2024.1.0devNone because it couldn't find either a tag distance or a commit distance. It would be great if, when no distance was found it simply returned the contents of the VERSION file (or 2024.1.0dev0. Basically, I want it always to be something valid no matter how they've fetched the project. Is this something that needs a custom version callback?
The project in question has over 900 forks so if it doesn't "just work" in all common cases then contributors are going to hit annoying barriers.
I think my current workaround is going to be to leave setuptools-git-versioning out of the main repo (for safety/compatibility) and just install it in the CI build system, when generating our official releases. But if I could set it up to work safely in all places that would feel neater.

PRs are welcome


C Program to find Largest Element in an array

Can you also label the issue as "hacktoberfest-accepted" on merging the PR?

@suman-shah Can you Please label it so it can count in Hacktober Fest?


Allow java applications to set timeouts for commands

Issue Being Resolved

Fixes #22601

Change overview
Adds a SetTimeout and GetTimeout commant to java.
SDK itself gains a GetTimeout which was missing before (for symmerty - if I can set, I should be able to get in most cases).

Accepted for 1.0: platform fix.


#1120 Identify request is coming from web-bots or human and from which device(like tablet, mobile or desktop)

Used robot-detection for web-bots detection
Used django-user-agents for device detection

👍