Android | Line render error

I've problems with Lines when I create Line using Map.addMarker().
Large line:
If line is large and I do zoom in the lines disappear.
Small line
if line is small and I do zoom in the line modifies the width on map.
Code for reproduce it:
On Android demo (MainActivity) replace:
        Map<String, String> props = new HashMap<>();
        props.put("type", "line");
        props.put("color", "#D2655F");

        List<LngLat> line = new ArrayList<>();
        line.add(lastTappedPoint);
        line.add(tappedPoint);
        markers.addPolyline(line, props);

for this
        String style = "{\"width\":\"3px\",\"color\":\"red\",\"order\":500,\"style\":\"lines\",\"visible\":true}";

        List<LngLat> line = new ArrayList<>();
        line.add(lastTappedPoint);
        line.add(tappedPoint);
        Marker lineMarker = map.addMarker();
        lineMarker.setPolyline(new Polyline(line, new HashMap<String, String>()));
        lineMarker.setStylingFromString(style);

screenshots:

vidio:
record.zip
Thanks!

Its happening on my side as well, really irritating.
This should be a 5px wide line, but its totally huge!
String lineStyle = "{ style: 'lines', color: '#06a6d4', width: '5px', order: 2000 }";
lineMarker.setPolyline(new Polyline(taps, null));

As i see its happening at least since Release 0.10.0, i couldnt built the older releases.

Here is the fix: https://pastebin.com/TEk68XVb
I dont know why is modelScale added or why, but with the old marker->extent() it just works well.

Hey @vilinet thanks for checking this out. I believe there was a reason we didn't use extent() in these cases (it might have broken some other use case) but I will find out why.

Would be nice to find out. This error shows up when the zoom level is kinda high. As closer you go down, the line getting enormously wider.

Hi @matteblair, are there any updates for that? I am facing the same problem as @vilinet and would be happy to get the real fix or at least some reasoning for why you stopped using the extent() in favour of modelScale(). Thanks!

Hi @vilinet
Thanks for debugging this and suggesting a fix. It seems like we had the markers use the extent to begin with, but @cleeus found some interesting polygon vertex wrapping issues (#1639) which lead to a fix (#1776) for that specific problem.
Clearly, there is a regression here, as it popped up this issue you and others are running into.
I think the solution might involve a different approach which should address #1639 along with this issue. I will investigate this more and see if I can come up with a good solution.
Thanks

Hi @tallytalwar
I'm not sure that we need a different approach, so much as better follow through with using modelScale() instead of extent() in polylineStyle.cpp.

In order for us to cut a new release and avoid a regression, I am going to revert the changes from #1776 (sorry @cleeus) and open another issue that describes the behavior we have discovered here.
@wcrestfield Maybe the solution here is to refactor polylineStyle, but currently adding the 3rd-dimensional complexity to already not so trivial marker system (something what #1776 was doing) might not be the right approach, plus I do not think we ever thought marker system will be used in the way it's being used right now :D. We have been meaning to refactor the annotation system and with that the marker API for some time now, and cutting a major release will get us started on the same.
Thanks to everyone here for being patient, tangram-es project currently is in parttime dev mode, hence the noticed slowness!

@tallytalwar I don't think polylineStyle needs a refactor.  The regression stemmed from using extent() in some places instead of modelScale().  I haven't verified extrusion, but https://pastebin.com/SKEJFmNK resolves the line width issue.

@wcrestfield Thanks, I can give this a try later. Also extrusion examples are specifically related to Polygon styles, so I am not sure. Again I need to verify and collect more thoughts here before I can concretely answer on the correct solution.


Draft new release now that sourcemaps fixed upstream

https://github.com/swc-project/swc/pull/3003
Would be nice to draft a new release now that upstream has fixed sourcemap issues. I believe would close some other open issues in this repo.

Published


Allow different values for minecraft:collision_box and minecraft:selection_box

minecraft:selection_box is a Boolean or JSON Object component that defines the area of the block that is selected by the player's cursor.

minecraft:collision_box is a boolean or JSON Object that defines the area of the block that collides with entities.

I want to make the player able to pass through the block by setting the origin and size in the Model to Vector3::zero(), but I still want the player to be able to interact with the block

You can do this with permutations
->withComponent('minecraft:selection_box', CompoundTag::create()
                    ->setByte('enabled', 1)
                    ->setTag('origin', new ListTag([
                        new FloatTag(-4.0),
                        new FloatTag(0.0),
                        new FloatTag(-4.0)
                    ]))
                    ->setTag('size', new ListTag([
                        new FloatTag(8.0),
                        new FloatTag(13.0),
                        new FloatTag(8.0)
                    ]))),```


Change Passport authentication to demo redirecting dynamically

Passport's authenticate method can take static success and failure redirect paths, but it can also defer to a vanilla middleware function and do any response the developer likes — e.g. a dynamic redirect. See here for an example.

On that note, there is an undocumented Passport feature, successReturnToOrRedirect, that 1704-FS-RM discovered. Might also be nice to include. Or maybe that would be just one more thing students get confused by…


Add pattern example confirmation

This pull request adds a new section to GOV.UK elements - Patterns.
This shows how to use GOV.UK elements to create pattern pages.
This adds an example - the transaction end page, or confirmation page,
has guidance taken from the Hackpad and a link to the Hackpad.
Here is a screenshot of the patterns page:

Here is a screenshot of the confimation page:

This relates to this PR on the GOV.UK prototype kit, to add common patterns for prototyping: https://github.com/alphagov/govuk_prototype_kit/pull/35
cc. @timpaul @ @joelanman @rivalee.

I'm wondering if the first step is just to add highlight box to the elements? Maybe in the typography section?
http://govuk-elements.herokuapp.com/typography/


Webhook at Read the Docs not set (docs not built on each commit)

Hi,
I noticed that the webhook at Read the Docs is not yet set (probably that was not done because we imported the project manually). The documentation will not build on each commit if the webhook is not set. You can find instructions on how to set the webhook here: https://docs.readthedocs.io/en/stable/webhooks.html#webhook-creation.
As a quick fix, you can also trigger a manual build at Read the Docs to get the most up to date documentation.
Regards,
Rafael

This is done. Closing


showWhen/hideWhen not working Beta 6

showWhen/hideWhen is not working.

`
<ion-navbar *navbar primary hideBackButton>
<ion-buttons left>
    <button showWhen="android" (click)="onCancel()" >
      <ion-icon name="close"></ion-icon>
    </button>
    <button showWhen="ios" (click)="onCancel()" >Cancel</button>
  </ion-buttons>

  <ion-title>Add Location</ion-title>

 <ion-buttons end>
    <button showWhen="android" (click)="onSubmit()" >
      <ion-icon name="checkmark"></ion-icon>
    </button>
    <button showWhen="ios" (click)="onSubmit()" >Done</button>
</ion-buttons>

</ion-navbar>

`
Version 2 Beta 6
Cordova CLI: 6.0.0
Gulp version:  CLI version 3.9.0
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.6
Ionic CLI Version: 2.0.0-beta.25
Ionic App Lib Version: 2.0.0-beta.15
ios-deploy version: 1.8.6
ios-sim version: 5.0.6
OS: Mac OS X El Capitan
Node Version: v5.1.0
Xcode version: Xcode 7.3 Build version 7D175

So I looked into this some more, and it is because the button's display property is taking precedence over the showWhen/hideWhen classes. As a temporary fix, you should be able to add the following css to your app.core.scss file (or any other sass file that comes after the ionic imports):
// Applied by the showWhen directive
.hidden-show-when {
  display: none !important;
}

// Applied by the hideWhen directive
.hidden-hide-when {
  display: none !important;
}

I'll work on fixing this in the framework. Thanks for the issue!

Thanks once again Brandy, you're awesome!!


Deprecation of get_duplicates

Pandas get_duplicates is being dropped. It needs to be rewritten as stated in the testing output log when pandas brings up the deprecation.

This is being addressed in #101

Addressed in pull #103


eddkkks

asdddd

Testing ITFFF recipes:
github issue -> Cloud LittleBit -> Arduino Input
Only for demo purposes.


Design cross-shard communication and smart contract functionality

Description
Design of the Harmony's cross-shard communication and smart contract functionalities. The feature should be fully secure against all kinds of attacks so the users asset are not in danger when involved in cross-shard operations such as sending HRC20 across shards or making smart contract calls across shards. The feature should enable the composability of smart contracts across shards.
Context
Harmony is a sharded blockchain where there are 4 shards each running consensus separately. Currently harmony supports cross-shard ONE token transfer where users can send ONE tokens across shards. Smart contract runs in each individual shards separately as of now and are not able to communicate with smart contracts from other shards. For example, users can not send a HRC20 token that's in shard 0 to shard 1 if the HRC20 contract only lives in shard 0.
Harmony described the initial design idea for the cross-shard smart contract feature here for reference: https://docs.harmony.one/home/general/technology/transactions#cross-shard-smart-contract-transaction. The final design will be worked together by the bounty hunter/grantee and the harmony team.
Acceptance Criteria

[ ] Successful deliver of the design of the cross-shard smart contract functionalities with full technical details and specifications, including protocol design change, security analysis, performance considerations, deployment strategies etc.
[ ] The design should be reviewed by the Harmony core team and improved iteratively until final approval by the Harmony core team.

Reward
$75k in ONE token

Moved to grant application


Feedback Source

@Swiftwork good shout - added some extra info to the comment. I don't expect this property to be used by customers, but didn't want to introduce extra misdirection to avoid it being visible.


Add paginator for ConfigService select_resource_config

Hello maintainers! I'd like to add paginator support for ConfigService advanced queries (select_resource_config). I'm up for submitting a one-off PR (similar to #1520 and others), but I'm wondering if it's better to hold off in favor of a broader coordinated effort like the one proposed by @kapilt in #1462.
In either case I'm happy to pitch in, I get a ton of value out of this SDK :).

We do now have better generation support, but some things may still slip through the cracks for one reason or another. Those misses should go down over time. In the meantime feel free to open a pr if you find something missing.

Ok cool, a one-off PR sounds like the best option here then. Thanks @JordonPhillips .


❓ If frameProcessorFps is 5, shouldn't frameProcessor run 5 times per second?

Question
If frameProcessorFps is 5, shouldn't frameProcessor run 5 times per second? Assigning 1 to frameProcessorFps calls exactly once every second. However, if I assign a number greater than 2, it is called only once or twice a second.
Please see the code below (this is an excerpt of my project code made using reaction-native-vision-camera). The text value recognized in runOnJs is saved as ocr and the processRecognizedText is executed whenever the ocr value changes to useEffect. At this time, I guessed that the frame processor could not run as many times as I wanted per second because it took a long time to run processRecognizedText, but when I checked it, it took 0.001 seconds.
If so, I think processRecognizedText should be excuted 100 times if frameProcessorFps is set to 100, but why is it only excuted 1~2 times?
I would appreciate it if you could check if I misunderstood the function of frame processor Fps and if there was anything wrong set up! Thank you.
import React, { useEffect, useState } from 'react';
import 'react-native-reanimated';
import { runOnJS } from 'react-native-reanimated';
import {
  Camera,
  useCameraDevices,
  useFrameProcessor,
} from 'react-native-vision-camera';
import { scanOCR } from 'vision-camera-ocr';
(...)

const DataCaptureView = ({ navigation, route }: IProps) => {
(...)
  const [enabled, setEnabled] = useState(true);
  const [ocr, setOcr] = useState();
  const devices = useCameraDevices();
  const device = devices[cameraPosition];

  const frameProcessor = useFrameProcessor(frame => {
    'worklet';
    const data = scanOCR(frame);
    runOnJS(setOcr)(data?.result?.text);
  }, []);

  useEffect(() => {
    if (!Boolean(ocr)) {
      return;
    }
    enabled && processRecognizedText(ocr);
  }, [ocr, enabled]);

  const processRecognizedText = (text: string) => {
    (...)
  };

  return (
    <StyledView>
      {device !== undefined && hasPermission ? (
          <Camera
            style={{ flex: 1, width: '100%' }}
            frameProcessor={frameProcessor}
            device={device}
            isActive={isAppForeground && isFocused}
            frameProcessorFps={5}
          />
      ) : (
        <StyledView />
      )}
    </StyledView>
  );
};
const StyledView = styled.View`
  flex: 1;
`;
export default DataCaptureView;

What I tried
VisionCamera Version
2.13.5
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

I have exactly the same issue, setting frameprocessorfps to , let's say 30, and I only receive around 12-13 frames from native Android code. Any ideas?

Hey everyone, I think I might have some insight onto your issues here:

Firstly, on Android you can set the Frame Processor FPS as high as you like but internally it will be capped at the FPS CameraX decides on for its ImageAnalysis use case (in my testing this was around 15-20fps). There are ways to improve this FPS, but it requires setting up some custom capture requests with Camera2 through CameraX's interop API - maybe this is something we could think about sorting out for V3 so you can disable some camera features to improve the ImageAnalysis use case @mrousavy ?
Secondly, if you're running this on iOS I just fixed an issue that you can see in this PR #1288 that was causing it to skip ever other frame (if the frame processor callback was able to execute in real-time)

but it requires setting up some custom capture requests with Camera2 through CameraX's interop API

CameraX is limiting us more and more. We should talk about using Camera2 instead

Closing as this is a stale issue - this might have been fixed with the full rewrite in VisionCamera V3 (🥳) - if not, please create a new issue.


Display Reaction Time

An option to display a map's set reaction time instead of offset/jump distance in the level params.
Easily calculatable from jump distance: reactiontime = jumpdistance / (2 * njs) * 1000; (in ms)
Maybe change the existing jump distance setting to a dropdown menu to accommodate the extra option?

I considered that before but I feel like the only real purpose of the default offset is for mappers / giving mapping feedback tbh since most people nowadays override the offset anyways. I'll consider it for a future update tho

Personally, I don't really get much from the offset (to be fair I am not a mapper), and I get some info from jump distance, but reaction time is just so much more intuitive to understand. I appreciate you considering it!

Yeah but like knowing the maps default RT doesnt really give you much if you are overriding it anyways

I still think it would be useful information to have built into the song select menu, but that's true, for people who change everything anyways it's doesn't really matter.

Closing this as that info field isnt offset anymore anways but map age now


Picking up latest snapshots

Continued delays in bringing new content in.
Current state -> PR raised in Canton on Monday and approved yesterday (Tuesday) is still building.

Hi @gerolf-da a stamp on this one would be very helpful when you have time please, thank you.

Is Pipfile needed? I thought this was managed by nix?

I'm hoping Gary will add Pip configs to gitignore as they are popping up all over the place.

Let's do this @gerolf-da  :)

@garyverhaegen-da @gerolf-da this is a really dangerous fix of the build - in my view. The error comes from another PR, a completely different task, which is not finished and won't be finished for at least 2/3 weeks, maybe more. This means that 2.6.0 has to include all these changes or there will be missing documentation, and it's impossible to track this.

Update: this failure can't be fixed @garyverhaegen-da @gerolf-da.


A collection of small fixes

Document and cleanup ledger advance log. Don't acquire if validated ledger is too old and don't try to publish if we have no valid ledger.
Tolerate a ledger sequence number is pseudo-transactions. That will allow the server to parse the future pseudo-transaction change to avoid re-using transaction hashes
Reduce chattiness of some log outputs.
Reviewers needed. All changes are fairly trivial.

Looks good to me. One slight quibble, the commit message on e7aaf21 has a typo: RADME.md is missing an E.

Fixed by rewriting history.

Great comments!

@rec I hereby appoint you as the second reviewer. Does this get a thumbs up?

Well, so when do we use if and when don't we?!?  Now I have no idea!

:+1:

@rec: we use the if when the streamed data contains function calls that are expensive (e.g. convert accounts or amounts to string, generate JSON for some object, etc.)

I have rebased this against develop and squashed the fold commit. I'm removing the Rebase label.

Merged as cba19d7e23c7c6a74ee2c0eb2accaab2347e927f, f813cb2310ea18c9380073429cbe9ba91318cbe2 and ff7dc0b446ff6c08b26370b5595e6eb73913d181.


api design Q

should we make all methods + config funcs individual exports that relate to a central state instance?
versus how it is now where most utils are members + methods of the large Algonaut() class.

reference:
look how mobx does this w a global configure func (https://mobx.js.org/configuration.html#proxy-support)
import { configure } from "mobx"

configure({
    useProxies: "never"
})

then, instead of how we do it now,
it would look something like...
import {
  setNodeConfig,
  connect,
  sendAlgo
} from '@thencc/algonautjs';

// set node to use (globally)
setNodeConfig({
    BASE_SERVER: 'https://testnet-algorand.api.purestake.io/ps2',
    API_TOKEN: { 'X-API-Key': 'MY_KEY_HERE' }, // key is header, value is token
    LEDGER: 'TestNet',
    PORT: ''
});

// connect acct
let accts = await connect(); // uses inkey as single wallet default. similar to how it is now. + stores in localstorage

// construct a txn + sign it + send it
let t = await sendAlgo({
  to: '123',
  amount: 1000
});

idea:
import {
  AnyWalletState
} from '@thencc/algonautjs';

console.log(AnyWalletState.address); // would be populated on page load IF user had connected an acct to the dapp previously

NOTE: @youraerials hates this approach!
it moves the floor out for under ya and it's super hard to debug. i saw it with my own eyes!


[SPARK-7022][PySpark][ML] Add ML.Tuning.ParamGridBuilder to PySpark

Can one of the admins verify this patch?

Jenkins this is ok to test

Test build #30616 has started for   PR 5601 at commit 8b8a6d2.

Test build #30616 has finished for   PR 5601 at commit 8b8a6d2.

This patch fails Python style tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

class ParamGridBuilder(object):

This patch does not change any dependencies.

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

Test build #30630 has started for   PR 5601 at commit 9a8ce22.

Test build #30630 has finished for   PR 5601 at commit 9a8ce22.

This patch fails PySpark unit tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

class ParamGridBuilder(object):

This patch does not change any dependencies.

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

Test build #30647 has started for   PR 5601 at commit c9e2481.

Test build #30647 has finished for   PR 5601 at commit c9e2481.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

class ParamGridBuilder(object):

This patch does not change any dependencies.

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

Friendly bump for more comments :)

Can one of the admins verify this patch?

It looks good to me. We need to run Jenkins again to make sure it doesn't conflict with recent changes to the pipeline API.

test this please

Merged build triggered.

Merged build started.

Test build #31312 has started for   PR 5601 at commit c9e2481.

Test build #31312 has finished for   PR 5601 at commit c9e2481.

This patch fails Spark unit tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

class ParamGridBuilder(object):

This patch does not change any dependencies.

Merged build finished. Test FAILed.

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

test this please

Merged build triggered.

Merged build started.

Test build #31457 has started for   PR 5601 at commit c9e2481.

Test build #31457 has finished for   PR 5601 at commit c9e2481.

This patch fails MiMa tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

class ParamGridBuilder(object):

This patch does not change any dependencies.

Merged build finished. Test FAILed.

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

Should I rebase and update this? I'm not sure why the build is failing.

Yea, you should try rebasing. It looks like you're not the only one running into this.
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31639/console

test this please

Merged build triggered.

Merged build started.

Test build #31692 has started for   PR 5601 at commit c9e2481.

Test build #31692 has finished for   PR 5601 at commit c9e2481.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class ParamGridBuilder(object):

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

Merged build finished. Test PASSed.

LGTM. I've merged this into master and I'll send a follow-up PR to implement what @justinuang suggested.


bikeshed should warn if "Previous version" link is out-of-date (or just generate the "Previous version")

It would be helpful if bikeshed warned if the "Previous version" link in the spec header disagreed with the actual previous version on the TR page.  These links are frequently out-of-date (or missing) on editor's drafts.  (Even better, perhaps, would be if bikeshed could generate it automatically, perhaps with separate directives for either the most recent previous version, or all previous versions.)
I'm not sure how easy this is to do given what is in bikeshed's bibliographic database -- I suspect right now it's possible, but could be substantially easier if the biblio database had some additional data.

Hmm, this is an interesting suggestion!  As long as I can link the spec up with the biblio entry, this should be auto-generatable.  I'll investigate.

So for example, given the key css-flexbox-1 which pulls up the master entry in the biblio db, there are a number of entries like css-flexbox-1-\d{8} that correspond to the previous versions. I should be able to go search for those and output Previous Version lines appropriately.
That's a simple case, tho, where the spec didn't change names at any point.  A more complex one is, for example, css-inline-3, which was previously known as css3-linebox.  The biblio data does show css3-linebox as being obsoleted by css-inline-3, so it should be crawlable as well.
I think to make this reasonably fast I need to maintain a new biblio data file tracking obsoletions in the other direction (given a name, get a list of all specs obsoleted by this name, transitively).
If you have a Date metadata, I probably want to exclude any versions that don't predate that, so you can, for example, regen a WD after it's been published once without it trying to link to itself as a "previous version".


Memory usage too high

Hi,
Sorry, I have another suggestion: the memory usage of your OS.
When I launch the terminal and run "free -u" I can see 150MB of memory used !
Your "hobby OS" has only very limited functionalities, but 150MB is surprisingly more than the actual linux system I use.
Strange...

You are running a live CD system with a 64MB ramdisk, running a compositor, on an OS without COW support, running several Python processes. There is nothing particularly high about the memory usage given the circumstances.

I should be clear, I'm closing this because I don't consider 150MB to be issue-worthy memory usage. There are definitely areas where we can improve memory usage, but this is perfectly expected behavior as it is, and very much within reason for the system in its current state.


Is it possible to override the values of global addons

Hi Team,
Is it possible to override the value of global add-ons, within the story specific values?
Thanks in advance
Amit

What value do you want to override? Could you please provide more details?

I want to override the ReadMe file set in the addDecorator in the config.js. And I want to override this ReadMe set in config.js by the story specific ReadMe.
Thanks in advance.

Sorry but I didn't catch you. Can you please provide an example what you want to do?

In my config.js I am using addDecorator with readme addon and providing it with a default Readme file (DefaultReadMe). Now I am want to override DefaultReadMe in the stories if the story has its own readme (StoryReadme).
e.g. In my config.js I have a DefaultReadme stating "No Information about the component" now what will happen every story will get the same Readme (DefaultReadme) stating: "No information about the component". Now if any story has it's specific readme (StoryReadme) then I want to replace DefaultReadme with StoryReadme for that specific story,
I hope you will understand the example.
Thanks in advance

How do you add the default readme?

Below is the code for config.js
import { withReadme  } from 'storybook-readme'
import * as DefaultReadme from '../DefaultReadme.md'

addDecorator(withReadme(DefaultReadme ))

and what I want is that in the story
storiesOf('Test Story A', module)
  .addDecorator(withReadme (StoryReadme))

But I want as the story has its own readme file so the default one should be overridden.
Thanks

Going to implement this feature in next release

Thanks a lot.

This FR released in 4.0.2


Confusion on Custom Resource Loaders

Hi,
In the docs it's mentioned you can load from a source other than "the underlying file system".  In the code snippet, a custom resource reader is used but the LoadMap(string) method still takes a "path".
I have need to load a map from memory (via a memory stream, byte array, or span). I've tried implementing IResourceLoader but the LoadMap method still wants a path and I get various errors for giving it an empty path or fake path (System.ArgumentNullException: 'Value cannot be null. (Parameter 'path1')'); even though my resource loader implementation doesn't even read from the path.
Am I missing how to load a map from memory or does this need to be fixed or is it not planned to be added?
Any help would be greatly appreciated, thanks.

I'm still unclear on what the docs seemed to be doing (docs) in the sense that I don't see how it lets you load from a source other than a file using LoadWith.
However, for my personnel use case. One of the examples demonstrated how to load your own map from any source "manually". This has resolved my issue, so I'm closing this issue.


[otbn] Add ECDSA P-384 to end-to-end test

We currently have an end-to-end test for ECDSA P-256 at https://github.com/lowRISC/opentitan/blob/master/sw/device/tests/otbn/otbn_ecdsa_p256_test.c. Produce a similar test for ECDSA P-384.

This is probably superseded by the chip-level testplan. The SW guys can figure out exactly what they want to run to test their software :-)


min_density parameter deprecation in sklearn 0.16

Not sure if you were still maintaining this code base base but now fails with sklearn 0.16.0 due to:
"The min_density parameter is deprecated as of version 0.14 and will be removed in 0.16.
"and will be removed in 0.16.", DeprecationWarning)"

Hi, thanks for the heads-up, I’ll take care of it.

On Aug 27, 2015, at 4:05 AM, Michael Pearmain<EMAIL_ADDRESS>wrote:
Not sure if you were still maintaining this code base base but now fails with sklearn 0.16.0 due to:
"The min_density parameter is deprecated as of version 0.14 and will be removed in 0.16.
"and will be removed in 0.16.", DeprecationWarning)"
—
Reply to this email directly or view it on GitHub https://github.com/dclambert/pyensemble/issues/2.

Thanks
Locally, i've also added n_jobs=-1 to a lot of the classifiers to improve speed i.e Extratrees, sgd and RandomForest, not an issue but food for thought


"docker: invalid reference format: repository name must be lowercase."

Description:
I'm encountering an issue when trying to execute the following Azure CLI command through a GitHub Action:
az storage blob delete-batch --account-name mystorageaccount --source '$web' --auth-mode key

The error message I receive is:
Error: docker: invalid reference format: repository name must be lowercase.

Steps to Reproduce:
Execute the Azure CLI command (mentioned above) in a GitHub Action.
This error is consistently reproducible every time the action is run. However, interestingly, when the exact same action content is copied to another action, the error does not occur.
Expected Behavior:
The command should execute successfully without the "docker: invalid reference format: repository name must be lowercase" error.
Additional Information:
I'm using the latest version of Azure CLI as of the date of this issue.
This error only occurs with this particular GitHub Action and not when running the Azure CLI command elsewhere.
I would appreciate any help in resolving this issue. Thanks in advance!

Hi @ratmie, this is a known issue when the user is trying to pass environment variables with quotes to Azure/cli and we have already prepared a fix to this scenario. See https://github.com/Azure/cli/issues/103. Could you please check if this applies to you?

@ratmie, the fix has been merged to the master branch.  To check if your issue has been resolved, please switch to azure/cli@master.


feat: Resources for groups and memberships

Describe the feature request
I would like to be able to create and manage unleash groups and group memberships using terraform.
Background
Today we assign users to projects directly using unleash_project_access, but we're currently reworking the way we assign permissions/access to different systems.
We would like to be able to create a group for each team in the workspace where we manage teams (across different systems), and then look up that team from the workspace where we set up "projects" (across different systems) and grant that team access to the unleash projects they "own".
We're currently not able to achieve this, as there aren't resources to manage groups in unleash.
A workaround would be to output team-members from the "teams workspace" and read that from the "projects workspace", and assign users directly to each project, but I'd prefer to keep the way we manage access similar across the different services we use.
Solution suggestions
Implement unleash_group resource and data source. Membership could either be managed directly on the unleash_group resource, or using a separate resource. My preference would be using a separate resource, and that seems common among other providers as well.
I could look into implementing this myself, but it seems like the unleash-server-api-go library doesn't support groups api.

Thanks @JonasBak, this makes sense. We have added to the backlog for the terraform provider.


add / fix PHPDocs + fix typos

I tested my Simple PHP Code Parser project with laravel/framework and found a few typos and PHPDoc errors.
./simple-php-code-parser.phar php-code-checker src/ vendor/autoload.php --skip-mixed-types-as-error

This change is

Feel free to re attempt with the requested changes. Thanks.


python311Packages.rapidfuzz: 3.7.0 -> 3.8.1

Description of changes
Diff: https://github.com/rapidfuzz/RapidFuzz/compare/refs/tags/v3.7.0...v3.8.1
Changelog: https://github.com/maxbachmann/RapidFuzz/blob/refs/tags/v3.8.1/CHANGELOG.rst

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

2 packages marked as broken and skipped:

python311Packages.pynitrokey
python311Packages.pynitrokey.dist

8 packages failed to build:

poetryPlugins.poetry-audit-plugin
poetryPlugins.poetry-audit-plugin.dist
python311Packages.pdf2docx
python311Packages.pdf2docx.dist
python312Packages.autofaiss
python312Packages.autofaiss.dist
python312Packages.pdf2docx
python312Packages.pdf2docx.dist

88 packages built:

adl
anime-downloader
anime-downloader.dist
apostrophe
atomic-operator
atomic-operator.dist
creds
creds.dist
emacsPackages.lsp-bridge
ibus-engines.uniemoji
instawow
instawow.dist
knowsmore
knowsmore.dist
ledger-autosync
ledger-autosync.dist
mealie
mealie.dist
nitrokey-app2
nitrokey-app2.dist
osc-cli
osc-cli.dist
paperless-ngx
poetry
poetry.dist
poetryPlugins.poetry-plugin-up
poetryPlugins.poetry-plugin-up.dist
puddletag
puddletag.dist
pynitrokey
pynitrokey.dist
python311Packages.autofaiss
python311Packages.autofaiss.dist
python311Packages.cleo
python311Packages.cleo.dist
python311Packages.fire
python311Packages.fire.dist
python311Packages.fuzzywuzzy
python311Packages.fuzzywuzzy.dist
python311Packages.jarowinkler
python311Packages.jarowinkler.dist
python311Packages.jiwer
python311Packages.jiwer.dist
python311Packages.levenshtein
python311Packages.levenshtein.dist
python311Packages.nkdfu
python311Packages.nkdfu.dist
python311Packages.rapidfuzz
python311Packages.rapidfuzz.dist
python311Packages.syncedlyrics
python311Packages.syncedlyrics.dist
python311Packages.thefuzz
python311Packages.thefuzz.dist
trytond (python311Packages.trytond)
trytond.dist (python311Packages.trytond.dist)
python311Packages.videocr
python311Packages.videocr.dist
python312Packages.cleo
python312Packages.cleo.dist
python312Packages.fire
python312Packages.fire.dist
python312Packages.fuzzywuzzy
python312Packages.fuzzywuzzy.dist
python312Packages.jarowinkler
python312Packages.jarowinkler.dist
python312Packages.jiwer
python312Packages.jiwer.dist
python312Packages.levenshtein
python312Packages.levenshtein.dist
python312Packages.nkdfu
python312Packages.nkdfu.dist
python312Packages.rapidfuzz
python312Packages.rapidfuzz.dist
python312Packages.syncedlyrics
python312Packages.syncedlyrics.dist
python312Packages.thefuzz
python312Packages.thefuzz.dist
python312Packages.trytond
python312Packages.trytond.dist
python312Packages.videocr
python312Packages.videocr.dist
rapidfuzz-cpp
spotdl
spotdl.dist
sublime-music
sublime-music.dist
ulauncher
ulauncher.dist


[diem-framework] Added the emits specs to the Diem Framework

This PR adds the "emits" specs to:

ValidatorAdministrationScripts::set_validator_config_and_reconfigure
TreasuryComplianceScripts::preburn
TreasuryComplianceScripts::tiered_mint
PaymentScripts::peer_to_peer_with_metadata
DiemSystem::update_config_and_reconfigure
DiemAccount::tiered_mint
DiemAccount::preburn
DiemAccount::pay_from
AccountCreationScripts::create_validator_operator_account

Fixed the specification issue in:

TreasuryComplianceScripts::burn_with_amount

Corrected/added the expected events of some transaction scripts in the
Diem Framework
Motivation
To add the emits specs to the Diem Framework
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
cargo test

/land

:lock: Merge Conflict

/land


JsDoc cleanup

Description
4th TODO from https://github.com/scaffold-eth/scaffold-eth-2/issues/617#issue-2003940907, also fixes #617

Gj Shiv, thanks! 🙌  Left some comments

Thanks @technophile-04
Love clean-up PRs :D


⚠ Turnos 24x7 has degraded performance

In 51b3692, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 970 ms

Resolved: Turnos 24x7 performance has improved in 905bef2.


Ceedling cannot find the mock header file during the compilation of the _runner.c

The Xxx_Xxxxxxx_Xxxxx_V1.44 header file isn't correctly included in the list of files that Ceedling needs to mock.  My header file is named Xxx_Xxxxxxx_Xxxxx_V1.44.h, the mock file should be mock_Xxx_Xxxxxx_Xxxxx_V1.44.h. However, in the runner, it looks for mock_Xxx_Xxxxxxx_Xxxxx_V1.h. I am working with legacy code, so it would be very difficult to adjust the file names to ensure consistency to handle this correctly. Can I change Ceedling configuration to handle this correctly through its mock configuration settings?
Generating runner for test_XxxxxxxXxxxXxxxxxx.c...
Compiling test_XxxxxxxXxxxXxxxxxx_runner.c...
build/test/runners/test_XxxxxxxXxxxXxxxxxx_runner.c:10:39: fatal error: mock_Xxx_Xxxxxxx_Xxxxx_V1.h: No such file or directory
 #include "mock_Xxx_Xxxxxxx_Xxxxx_V1.h"
                                       ^
compilation terminated.

So someone is doing manual version control of the header files and (important) uses dot as separator, and now you want to configure a general usage tool for this bad practice? I would sure like to know which developers and library is that, so that I can stear clear.
Just a useless rant, but I do not think there is a configuration fix as Ceedling already uses .h for file-endings and if it cuts after first dot then its what you would consider a bug in parsing. Further more, this is part of CMock most likely.

You're absolutely right — having version numbers with dots in header file names can complicate things, especially when using tools like Ceedling and CMock. This is less about Ceedling's configuration flexibility and more about CMock's parsing limitations. If it encounters a dot and interprets it as a file extension delimiter, it could cause exactly the kind of issue I'm facing. Maybe you have some few suggestions that could help mitigate or work around this problem?

@anoshyn, What an interesting quirk in legacy file naming! I'll cut to the chase… This is now fixed in the upcoming 1.0.0 Ceedling release. The fix will be available in a new prerelease build soon.
Long story short, Ceedling juggles file extensions all over the place. Certain utility methods are quite handy for removing, changing, or adding file extensions to filenames. Ceedling makes assumptions throughout that filenames have a base name and then only a dotted filename extension. In a few key places, that assumption falls apart in cases like yours because of the utility methods I mention. In certain places those utility methods were used in brute force ways among filename slicing and dicing that gobbled up and replaced the dotted portion of the filename as you experienced. It may be bad practice to include dotted version numbers as already noted, but having multiple periods in filenames is legal. So, we should do our best to support this.


is it working with Wear OS ?

When I try to get my location on my watch, I always get this error message : "The location service on the device is disabled.", however it's enabled.
Thx

Are you sure that you set the correct permissions in the manifest file for wear os?
if you forget it just add this line to the android manifest file
<uses-permission android:name="android.hardware.location" />

Yes, it's already done, i've tested the same code on my Oneplus 6t and it works...

Hi @weissloic,
This is hard to say, I personally don't have access to a WearOS device and therefore haven't tested. The plugin is developed (and tested) for Android phone/ tablets, iOS phones and tablets and the web. I have not done anything specific to make sure WearOS is also supported.
If you could help out and maybe debug on you WearOS device and give feedback on where things might go wrong, I am more then willing to make changes to make sure WearOS is also supported correctly.

@mvanbeusekom  For Chinese devices, Google Play Store is missing

@jahnli thank you for your reply, but how does this apply to running the plugin on WearOS? I am not quite sure how your comment is relevant to this issue, could you explain this?

@mvanbeusekom   https://github.com/Baseflow/flutter-geolocator/issues/794
This is the question I want to ask,
I have a Chinese device, but when I use it, I am prompted that oogle Play Store is missing. May I ask if it can be solved

@jahnli you may of course but that is a separate issue and not related to WearOS. Posting your problem here will distract from the original issue and may lead to wrong assumptions / conclusions.

@mvanbeusekom  ok， I will delete the answer under this question, please help me https://github.com/Baseflow/flutter-geolocator/issues/794

Can confirm, speed api works perfectly well on android phone but throws an error about the service being disabled on any wearOS device.
Any updates or plans on this?

I would be motivated to add wearOS support to the android plugin, if you can point me in the right direction.

Hi @mvanbeusekom
I just realized now that you offered to help implement it :)
I only develop on Android Studio, using the provided WearOS emulator. It is quite simple to use and the error seems to be identical to the one @weissloic describes.
I can post the stacktrace I receive below:
D/FlutterGeolocator( 6377): Attaching Geolocator to activity
D/FlutterGeolocator( 6377): Creating service.
D/FlutterGeolocator( 6377): Binding to location service.
D/FlutterGeolocator( 6377): Geolocator foreground service connected
D/FlutterGeolocator( 6377): Initializing Geolocator services
D/FlutterGeolocator( 6377): Flutter engine connected. Connected engine count 1
...
E/FlutterGeolocator( 6377): Geolocator position updates started
...
I/flutter ( 6377): The location service on the device is disabled.
I/flutter ( 6377): #0      GeolocatorAndroid.getPositionStream.<anonymous closure> (package:geolocator_android/src/geolocator_android.dart:187:9)
I/flutter ( 6377): (elided 26 frames from dart:async)

There's not really anything useful in there as far as I can tell. I assume there's a different API that must be used for wearOS compared to regular android?
Does this help? Otherwise let me know what I can do in order to help.

So, no updates?

so you cannot get the Wear OS device's location in flutter??? because the location library is outdated too

Closing this issue as discussion pointed out that WearOS is indeed not supported at this time.
Please refer to #1197 for tracking WearOS support.


no lazy computes anymore

When implementing properscoring.brier_score, I saw that we dont test lazy computation, i.e. whether we get chunked results when entering chunked inputs. Therefore I added a few assert statements. But this error also occurs now for test_xr_threshold_brier_score_dask which had the line assert actual.chunks is not None already before I did my changes.
    @pytest.mark.parametrize('dim', AXES)
    def test_pearson_r_p_value_xr_dask(a_dask, b_dask, dim):
        actual = pearson_r_p_value(a_dask, b_dask, dim)
>       assert actual.chunks is not None
E       assert None is not None

Maybe it has something to do with the stacking in _preprocess? @ahuang11
Furthermore I get this warning, maybe there is a problem with np.rollaxis:
======================================= warnings summary ========================================
/Users/aaron.spring/anaconda3/envs/climpred-dev/lib/python3.6/site-packages/_pytest/mark/structures.py:332
  /Users/aaron.spring/anaconda3/envs/climpred-dev/lib/python3.6/site-packages/_pytest/mark/structures.py:332: PytestUnknownMarkWarning: Unknown pytest.mark.flaky - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/latest/mark.html
    PytestUnknownMarkWarning,

/Users/aaron.spring/anaconda3/envs/climpred-dev/lib/python3.6/site-packages/_pytest/mark/structures.py:332
  /Users/aaron.spring/anaconda3/envs/climpred-dev/lib/python3.6/site-packages/_pytest/mark/structures.py:332: PytestUnknownMarkWarning: Unknown pytest.mark.network - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/latest/mark.html
    PytestUnknownMarkWarning,

xskillscore/tests/test_deterministic.py::test_pearson_r_xr_dask[time]
xskillscore/tests/test_deterministic.py::test_pearson_r_xr_dask[time]
xskillscore/tests/test_deterministic.py::test_pearson_r_xr_dask[lat]
xskillscore/tests/test_deterministic.py::test_pearson_r_xr_dask[lat]
xskillscore/tests/test_deterministic.py::test_pearson_r_xr_dask[lon]
xskillscore/tests/test_deterministic.py::test_pearson_r_xr_dask[lon]
xskillscore/tests/test_deterministic.py::test_pearson_r_xr_dask[dim3]
xskillscore/tests/test_deterministic.py::test_pearson_r_xr_dask[dim3]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[time]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[time]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[time]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[lat]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[lat]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[lat]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[lon]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[lon]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[lon]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[dim3]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[dim3]
xskillscore/tests/test_deterministic.py::test_pearson_r_p_value_xr_dask[dim3]
  /Users/aaron.spring/anaconda3/envs/climpred-dev/lib/python3.6/site-packages/dask/array/core.py:1263: FutureWarning: The `numpy.rollaxis` function is not implemented by Dask array. You may want to use the da.map_blocks function or something similar to silence this warning. Your code may stop working in a future release.
    FutureWarning,

-- Docs: https://docs.pytest.org/en/latest/warnings.html

See https://github.com/raybellwaves/xskillscore/pull/18 for the new tests.

my mistake somewhere.


Improve text popup

To give it more style they should be in italics. If anyone have more suggestions please write them here
Also the /me command should appear as a popup too

What about :

Font size 10
Italic
Light gray, so it is a bit more aggressive

Nice, maybe reduce the font size to 8?
Does it work with /me?

I'll try 8, but it sounds too small for me.
I'm working on something else for /me: a speech "bubble", but in italic, and without the border, so it stick with the rest of the conversation.

Sounds good.
If 8 is too difficult to read then dump it

Sounds good.
If 8 is too difficult to read then dump it

ring
'e pushed?
doomp eet
click

Full screen picture at 8 for comparison

Hmmmm, yeah, too small

Here is my proposition for the emotes: instead of using the current popup messages, which look more like "notifications" to me, I implemented the emotes as speech bubble, in italic without border.

Looks good, i dont have any issues with it, you should commit it


Add airport routes api

Add Airport Routes API for Node and Python
Java to be added by @xianqiliu

changed to MAD <3 @tsolakoua


Feature: call $+5

Summary
Add (one-off) feature to match on call    $+5; often used in shellcode or for obfuscation.
Motivation
Help to detect shellcode and anti-disassembly techniques.

Hi @mr-tz ,
I am currently looking into capa internals.
If I understand correctly capa code, this could be solved by adding functions that extract Characteristic("call $+5") inside the capa extractors.
What should be the "scope" for this? I think this feature could be extracted at a basic block level or at an instruction level. Is there a preferred option (that could be related to performance for example)?

I think this should be extracted at instruction scope, like how fs/gs access is extracted.
There's potentially a minor performance difference, but I think without profiling results that show this is a bottleneck, we should use Instruction scope when appropriate.

All right, I should be able to start working on this in the next few days.
Feel free to assign me this issue.

Sounds great. Thank you! We're happy to help with any questions.

Hi,
I started working on this using the capa rule 64-bit execution via heavens gate to find an example where this feature should be extracted.
However, I am obtaining different results on this binary depending on the backend. I am using the test sample 79abd17391adc6251ecdc58d13d76baf.dll_.
With smda:
python3 scripts/show-features.py -b smda tests/data/79abd17391adc6251ecdc58d13d76baf.dll_ | grep 'call $+5'
insn: 0x10002396: characteristic(call $+5)
insn: 0x100023cd: characteristic(call $+5)

With vivisect:
python3 scripts/show-features.py -b vivisect tests/data/79abd17391adc6251ecdc58d13d76baf.dll_ | grep 'call $+5'
insn: 0x10002396: characteristic(call $+5)

I'm guessing from the few tests I made that vivsect is not disassembling the code around 0x100023cd.
To look further I would need other samples using this obfuscation technique.
@williballenthin @mr-tz Do you guys have in mind few more binaries using this obfuscation technique?
Also, I think that I won't be able to add the IDA extractor since it requires IDA Pro (from what I understood).

Awesome, thanks for looking into this!
In the test files I could locate this file:
946a99f36a46d335dec080d9a4371940.dll_:0x100015C7
On Virustotal I found the following (packed) files (attached with password infected call5.zip):
5b1102b10f8d0dee41b592bea0892ac78cecee1a3f5bc55c5176359634c6c002
efbf3f3546130f3a00304eb1873fb0b72b27988f162f5599024e49b3df864f2a
f99c836e89a6a181d292e1aad24d8667c6b53b870548dada3694e474b259d3e9
I hope that helps a bit. Please let me know otherwise.
We can add the IDA extractor, if you don't have IDAPython available.

So I made further test and it seems that the issue comes from vivisect or smda not being able to disassemble the code.
As example, here's the output of vivisect for the test sample 79abd17391adc6251ecdc58d13d76baf.dll_:

As you can see, the 0x100023cd: characteristic(call $+5) cannot be extracted with vivisect because it did not disassemble this part of the memory.
I also made some test on the other binary you provided and got these kinds of results:
❯ python3 scripts/show-features.py -b smda  ~/Downloads/call5/efbf3f3546130f3a00304eb1873fb0b72b27988f162f5599024e49b3df864f2a | grep 'call'

❯ python3 scripts/show-features.py -b vivisect  ~/Downloads/call5/efbf3f3546130f3a00304eb1873fb0b72b27988f162f5599024e49b3df864f2a | grep 'call'
insn: 0x004241b4: mnemonic(call)
insn: 0x004241b4: characteristic(call $+5)
insn: 0x004241b9: characteristic(calls from)
insn: 0x00424223: mnemonic(call)
insn: 0x00424286: characteristic(calls from)
insn: 0x00424261: mnemonic(call)
insn: 0x00424261: characteristic(call $+5)
insn: 0x00424266: characteristic(calls from)
insn: 0x00424276: mnemonic(call)
insn: 0x00424276: characteristic(call $+5)
insn: 0x0042427b: characteristic(calls from)
func: 0x00424223: characteristic(calls to)
func: 0x00424223: characteristic(calls to)

So I think that it is not really an issue as rule matching and feature extraction can be different depending on the backend used.

Thank you, @kn0wl3dge!


signOut not removing cookie

https://github.com/nuxt-modules/supabase/issues/114
I think that the release that removed useSupabaseAuthClient has reintroduced this bug
https://github.com/stlbucket/nuxt-supabase-client-signout-bug
That is a minimal reproduction of the issue.

This works for me:
async function signOut() {
   try {
      await supabase.auth.signOut()
   } catch (err) {
      console.error(err)
   } finally {
      reloadNuxtApp()
      navigateTo('/')
   }
}

This is working for me:
async function signOut() {
   try {
      await supabase.auth.signOut()
   } catch (err) {
      console.error(err)
   } finally {
      reloadNuxtApp()
      navigateTo('/')
   }
}

I can confirm this, tested with Chrome. After logout, when trying to access the restricted page, it still rendered the restricted page before redirecting. This does not happen when deleting the cookies manually.
As a workaround I manually delted both access-token and refresh-token on SignOut. This worked for me as I could not render the restricted page afterwards.
`async function signOut() {
try {
const { error } = await client.auth.signOut();
const authAccessToken = useCookie("sb-access-token");
authAccessToken.value = null;

const authRefreshToken = useCookie("sb-refresh-token");
authRefreshToken.value = null;

router.push("/login");

} catch (error) {
console.log(error.message);
}
}`

facing a similar issue and for some reason the router.push('/login') wasn't working after signout even after clearing cookies
expanding on solution above I created a function to clear all cookies then redirect to login
const useCookieHelpers = () => {
  const deleteAllCookies = () => {
    const cookies = document.cookie.split(';');

    for (let i = 0; i < cookies.length; i++) {
      const cookie = cookies[i];
      const eqPos = cookie.indexOf('=');
      const name = eqPos > -1 ? cookie.substr(0, eqPos) : cookie;
      document.cookie = name + '=;expires=Thu, 01 Jan 1970 00:00:00 GMT';
    }
  };
  return {
    deleteAllCookies,
  };
};

export default useCookieHelpers;

on my logout function
const client = useSupabaseClient<Database>();
const { deleteAllCookies } = useCookieHelpers();
const handleLogout = async () => {
    await client.auth.signOut();
    deleteAllCookies();
    location.replace(`${location.origin}/login`);
};


Poetry resolves differently when fetching packages through a local devpi mirror

[x] I am on the latest Poetry version.

[x] I have searched the issues of this repo and believe that this is not a duplicate.

[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: MacOS 10.14 Mojave

Poetry version: 12.10

Link of a Gist with the contents of your pyproject.toml file: https://gist.github.com/bjoernpollex/529e35cf10f4abe791cefc711319188c

Issue
Given the linked pyproject.toml, poetry will resolve differently (and incorrectly) when running against a local devpi mirror. Running poetry -vvv lock without custom package source against clean caches produces the following output:
Using virtualenv: /Users/bjoernpollex/Library/Caches/pypoetry/virtualenvs/poetry-futures-py3.6
Updating dependencies
Resolving dependencies...
   1: fact: poetry-futures is 0.1.0
   1: derived: poetry-futures
   1: fact: poetry-futures depends on isort (^4.3)
   1: selecting poetry-futures (0.1.0)
   1: derived: isort (^4.3)
PyPI: 5 packages found for isort >=4.3,<5.0
PyPI: Getting info for isort (4.3.4) from PyPI
PyPI: No dependencies found, downloading archives
PyPI: Downloading sdist: isort-4.3.4.tar.gz
   1: selecting isort (4.3.4)
   1: Version solving took 0.364 seconds.
   1: Tried 1 solutions.

Running with a local devpi mirror as a custom package source (uncomment the relevant lines in pyproject.toml, requires a local devpi-server running) gives this:
Using virtualenv: /Users/bjoernpollex/Library/Caches/pypoetry/virtualenvs/poetry-futures-py3.6
Updating dependencies
Resolving dependencies...
   1: fact: poetry-futures is 0.1.0
   1: derived: poetry-futures
   1: fact: poetry-futures depends on isort (^4.3)
   1: selecting poetry-futures (0.1.0)
   1: derived: isort (^4.3)
dev: 5 packages found for isort >=4.3,<5.0
dev: Downloading wheel: isort-4.3.4-py2-none-any.whl
dev: The cache for isort 4.3.4 is outdated. Refreshing.
dev: Downloading wheel: isort-4.3.4-py2-none-any.whl
   1: fact: isort (4.3.4) depends on futures (*)
   1: selecting isort (4.3.4)
   1: derived: futures (*)
dev: 22 packages found for futures *
dev: Downloading wheel: futures-3.2.0-py2-none-any.whl
dev: The cache for futures 3.2.0 is outdated. Refreshing.
dev: Downloading wheel: futures-3.2.0-py2-none-any.whl
   1: selecting futures (3.2.0)
   1: Version solving took 0.264 seconds.
   1: Tried 1 solutions.

For some reason, when running against the local mirror, poetry doesn't figure out that (1) futures is not needed for isort on Python 3, and (2) that futures 3.2.0 is not compatible with Python 3 (it will fail to install).
I would expect poetry to come to the same and correct solution in both cases.

It also seems strange that it downloads a Python 2 wheel, maybe that's why it gets the bad dependency information.

So, I'm not sure if this is a problem on poetry's side, or if the package has bad metadata. Basically, the metadata in the Python 2 wheel lists the futures requirement twice, once with an environment qualifier, and once without:
    "run_requires": [
        {
            "requires": [
                "futures"
            ]
        },
        {
            "environment": "python_version==\"2.7\"",
            "requires": [
                "futures"
            ]
        }
    ],

The Python 3 wheel lists the requirement only with the environment qualifier.

I also have this question. I have a local devpi mirror to speed up gitlab runner docker.
  - Installing futures (3.2.0)
    Installing from unsecure host: <IP_ADDRESS>
[EnvCommandError]
Command ['pip', 'install', '--no-deps', '--trusted-host', '<IP_ADDRESS>', '--index-url', 'http://<IP_ADDRESS>:3141/root/pypi/+simple', 'futures==3.2.0'] errored with the following output:
Looking in indexes: http://<IP_ADDRESS>:3141/root/pypi/+simple
Collecting futures==3.2.0
  Could not find a version that satisfies the requirement futures==3.2.0 (from versions: 0.2.python3, 0.1, 0.2, 1.0, 2.0, 2.1, 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.1.5, 2.1.6, 2.2.0, 3.0.0, 3.0.1, 3.0.2, 3.0.3,
3.0.4, 3.0.5, 3.1.0, 3.1.1)
No matching distribution found for futures==3.2.0

Commit 750e31d should fix the issue and will be released with next bugfix version some time next week.


Logger only prints to the console. not possible to integrate with actual logging framework

Currently loggerFactory is fixed to depend on Console.Service. It would be nice that we can integrate with logging framework
https://github.com/zio/zio-logging

In theory, you should be able to override loggerFactory in the ZparkioApp by implementing a new version of Logger.Service . That should work.


If-statement hightlighted even though it works in the pipelines

The extension always expects a task as the first property even though the Azure Devops pipelines accept if statement blocks.
This example runs without any issues in my pipeline yet the extension highlights it as an issue:

${{ if eq(parameters.includeReleaseTemplates, true) }}:

task: CopyFiles@2
displayName: 'Copy release files to $(build.artifactstagingdirectory)\dotnetPublish'
...

Seconding this, i get a highlighting error that states "The first property must be task" when using an if statement to conditionally include a task/template.

i have the same problem in my azure-pipelines.yml
maybe syntax highlighting is correct on templates only and not in entry pipeline yml?
variables:

${{ if .... }}:

Happens with 'each' too.

I have the same Problem:

If the fix proposed by @demings back in April isn't the fix, does anyone know what is?

Closing in favor of #187. I know this is a pain point, just working on consolidating issues right now so we can get a better sense of the outstanding areas to work on.


include "added in vX.Y.Z" for each function

When working on a project which specifies a Ramda version other than the most recent, one should be able to determine from the documentation whether a particular function is available. This need would be satisfied by ramda/ramda.github.io#10, but I'm interested in seeing each function's "age" regardless.

For the most part you can use git to find out in which tag src/<function>.js was added, but not so easy for ones added before 3a3285b.

I've wanted the same thing at times.  While this is probably worth doing, I hate adding still another JSDoc tag, and reducing further the signal/noise ratio of our source files.

It should be doable without a JSDoc tag if the git method is acceptable and figuring out the git tags when docs are built isn't too time consuming.

We could, but that might get tricky for renames, etc.  I'm really tempted to have one or several separate metadata files for such information, ones that would be required by the build to have an entry for every public function, but which don't end up in the source code file.
But perhaps it's just me that doesn't like all that stuff cluttering up my source code.

I don't think this needs special treatment: it could simply be a paragraph in the description. For example:

objOf :: String -> a -> StrMap a
Creates an object containing a single key:value pair.
Added in v0.7.0 as createMapEntry. Renamed objOf in v0.18.0.

If we do have it in the source file, I would rather a tag (or even two if necessary [@formerly or some such]), which we can more easily treat as standardized metadata, format independently, etc.
I just having a longing for cleaner source files.

I vote for adding the @since tag, which is official JSDoc: http://usejsdoc.org/tags-since.html

Nice find, @branneman! That's clearly the way to go. Now we just need someone with good Git skills to add this for each function. :)

I'll look into it!

I've got an initial try here: https://gist.github.com/branneman/f63a7ea4d867d539599d
I did not implement searching for the exact function versions from before ramda 3a3285b (is that ≤0.8 or ≤0.9?), that requires a different implementation. Do we actually want this? Or might a @since tag such as ≤ 0.8.0 be enough? I can put the @since tags in a PR right now, or I can implement the harder version, which will take me some more time.
I can look into the more precise older version tags as well, but if we don't really want it, that'll save me the effort.
When will the next version of ramda be there? It would be nice if this makes it in. Is there a deadline?

Aside:
I was playing around with ramda-fantasy for the first time, so forgive me if i screwed up with using IO correctly ;) To me, it seems that the implementation of getCommit() and getVersion() is too bloated. If I'm using IO/chain/map wrong, pointers would be appreciated.
The non-functional shorter version of the same code is available as well, here.

I can look into the more precise older version tags as well, but if we don't really want it, that'll save me the effort.

I would very much like to provide @since tags going all the way back to v0.1.0 or whatever the first release was. There's a straightforward manual approach for determining when early functions were added:

Open n tabs in a browser window, each displaying ramda.js as it existed at a particular tag (here's 0.2.0, for example).

For each function added before 3a3285b817ea1780e5e9a5fbc72f8cde0da90f78, determine the version in which it was added by searching for R.<name> = in each tab from left to right until a match is found.

When will the next version of ramda be there? It would be nice if this makes it in. Is there a deadline?

There's no deadline. I would love to release v0.18.0 this weekend, but I'm happy to wait for you pull request.
One thing I noticed when viewing your gist is that patch releases are incorrectly given credit for additions. Functions added in X.Y.0 are shown to have been added in X.Y.1 instead.

An approach that may work for all versions:

run git show-ref --tags to get our list of tags and sha's
order tags, start with earliest
run git show ${sha}:package.json > ${os_tmp_dir}/package.json
require the file from step 3
read the main key to get path to ramda
run git show ${sha}:${path_from_step_5} > ${os_tmp_dir}/ramda.js
require our ramda file from step 6, loop over module keys, every function that we have not seen yet we can add our since tag

Could also just require those temp files from memory. Sorry for the pseudo code if I've messed anything up/left anything out, played around a bit and this seems to work. I also like personally like @CrossEye's suggestion above of a separate meta data file that is included with the distribution, so that the doc builder repo can use it. Seems easier to maintain going forward and can potentially include other stuff for docs as needed.

I'm keen to release v0.18.0 this weekend, so I've put together a shell script based on @kedashoe's excellent suggestion. I've also pushed missing tags and addressed inconsistencies in the tag names from the pre-xyz days.
Do you mind if I take this over, @branneman? If you've done further work on your script we could collaborate. :)

Not at all @davidchambers! I haven't done any additional work yet.
I'd love to see your solution. If you don't mind me asking, why not a ramda/ramda-fantasy script? ^^

Not at all @davidchambers! I haven't done any additional work yet.

I'm pleased to determine that we haven't been duplicating our efforts.

I'd love to see your solution. If you don't mind me asking, why not a ramda/ramda-fantasy script?

I prefer Bash for scripts which need to interact with other Unix tools because synchronous code is simpler than asynchronous code (even if good abstractions are available). I'll include the script in the description of the pull request I'll open shortly. :)


remove un-owned hubs

There are a few hubs pins for hubs that don't have a hub manager.
Remove:
Wellington
Brisbane
Tongatapu

Done!


Attempting to put null into a typed array of Object (and derivatives) fails

Godot version
4.0a13
System information
Linux/Ubuntu 22.04
Issue description
TypedArrays of type Object (and derivatives) do not accept null. This is inconsistent with declaring a variable, typing it as Object or derivatives, and setting it to null.
Steps to reproduce

Make a TypedArray of type Object
Try to set any of its elements to null

Minimal reproduction project
No response

I made a minimal reproduction project based on what i understand of the bug but could not replicate it
test.zip
I might have made false assumption if it is the case could you please add a minimal reproduction project?

Facing the same issue. Reproducible:
var nodes: Array[Node] = []
nodes.resize(1)
print(nodes[0]) # Prints <null>
nodes[0] = Node.new()
print(nodes[0]) # Prints <Node#47445968338>
nodes[0] = null
print(nodes[0]) # Prints <Node#47445968338>

I tried working around the issue by using Array.remove_at and Array.insert, but that doesn't work either. Example:
var nodes: Array[Node] = []
nodes.resize(1)
print(nodes[0]) # Prints <null>
nodes[0] = Node.new()
print(nodes[0]) # Prints <Node#45298484672>
nodes.remove_at(0)
nodes.insert(0, null) # Silently fails with '_input: Condition "!_p->typed.validate(p_value, "insert")" is true. Returning: ERR_INVALID_PARAMETER'
print(nodes[0]) # Fails with "Out of bounds get index '0'"

So my other workaround is to use a Dictionary instead.


Visual Studio Code - Insiders: Running the contributed command: 'vscode-edge-devtools-view.launch' failed.

Hello, I am unable to run the extension, clicking "Open new tab"

I see the following message

Running the contributed command: 'vscode-edge-devtools-view.launch' failed.

vscode specs

Machine specs

Hi!  Thanks for reporting this.  We actually have a bug and a fix open for this already.  Going to close the issue.
Bug: https://github.com/microsoft/vscode-edge-devtools/issues/277
PR: https://github.com/microsoft/vscode-edge-devtools/pull/287

As a note, the workaround for this is to wait a couple more seconds and it should work.  The issue I was running into was that VSCode would generate the button before it could be instantiated on our end.
Let me know if waiting a couple seconds doesn't work and it stays broken.  I can re-open this issue in that case.

Hello @mliao95 .. waiting a couple of seconds doesn'tt solve the issue

Re-opening the issue.

Could you expand the error and post what it says?

I think this should be helpful?
note that I have node version: v12.20.2, do I need to upgrade nodejs maybe?

This is likely an issue the extension is having with ARM devices. We can reference #301 to track the progress on this bug.

Hi @abusada, we just released v1.1.6 of the extension which fixes the issue with ARM devices. If you're still unable to launch, let me know and we can re-open the issue. Thanks!


Object.keys support

From the docs (e.g. here) I can't really tell if Object.keys is actually supported in duktape. When I try to get all keys from an object it returns an empty array, so it seems there is some support, but maybe it's broken.

However, it seems as if that has to do with how the object has been created. The result from os.networkInterfaces() is constructed by duktape/C code. Pure JS code like var t = Object.keys({ a: 1, b: 2 }); however works as expected.

Object.keys() is supported so I'm not sure what could go wrong here. Object.keys() only returns own properties but that doesn't seem to be an issue. Currently Object.keys() doesn't support a Proxy object, not sure if that could be related?

Could you paste how the C code creates the interface object?

Pretty simple code:
duk_ret_t OS::networkInterfaces(duk_context* ctx) {
  duk_push_object(ctx);

  for (auto &entry : Platform::get().networkInterfaces()) {
    duk_push_string(ctx, entry.first.c_str());
    duk_idx_t arrayIndex = duk_push_array(ctx);

    duk_uarridx_t index = 0;
    for (auto &interface : entry.second) {
      duk_push_object(ctx);

      duk_push_string(ctx, "address");
      duk_push_string(ctx, interface.address.c_str());
      duk_def_prop(ctx, -3, DUK_READ_ONLY | DUK_DEFPROP_HAVE_VALUE);

      duk_push_string(ctx, "netmask");
      duk_push_string(ctx, interface.netmask.c_str());
      duk_def_prop(ctx, -3, DUK_READ_ONLY | DUK_DEFPROP_HAVE_VALUE);

      duk_push_string(ctx, "family");
      duk_push_string(ctx, interface.family.c_str());
      duk_def_prop(ctx, -3, DUK_READ_ONLY | DUK_DEFPROP_HAVE_VALUE);

      duk_push_string(ctx, "mac");
      duk_push_string(ctx, interface.mac.c_str());
      duk_def_prop(ctx, -3, DUK_READ_ONLY | DUK_DEFPROP_HAVE_VALUE);

      duk_push_string(ctx, "internal");
      duk_push_boolean(ctx, interface.internal);
      duk_def_prop(ctx, -3, DUK_READ_ONLY | DUK_DEFPROP_HAVE_VALUE);

      duk_push_string(ctx, "scopeid");
      duk_push_uint(ctx, static_cast<duk_uint_t>(interface.scopeid));
      duk_def_prop(ctx, -3, DUK_READ_ONLY | DUK_DEFPROP_HAVE_VALUE);

      duk_put_prop_index(ctx, arrayIndex, index++);
    }

    duk_def_prop(ctx, -3, DUK_READ_ONLY | DUK_DEFPROP_HAVE_VALUE);
  }

  return 1;
}

Object.keys() doesn't include non-enumerable properties so you should include DUK_DEFPROP_SET_ENUMERABLE to DUK_READ_ONLY.
If you don't want to do that, Object.getOwnPropertyNames() also includes non-enumerable properties.

Makes a lot of sense. Thanks for your quick help!


New Feature Idea: Command to send random, small programming challenges from codewars.com

I think this could be a fun feature; if someone invokes a command like !kata the bot sends a random kata from a db.
I have made a bare-bones implementation as a personal project, you could check it out at https://github.com/ShakyaMajumdar/KataBot/blob/master/bot/exts/challenges.py.
This what it looks like currently:

seems good to me :+1:

but are you storing the challenges manually?

No; there's a function which I call to update the database of stored challenges.
We could just loop this to be called at specified time intervals automatically

I had also wanted to include some way to test submissions from users, but unfortunately I couldn't think of any way to extract sample input and output from the API response and all katas don't have examples any way.

So have you like stored every single challenge there is in a db, and can update with a command, wont that be slow as there are like so many katas? though if we use asyncpg with postgres i think it wouldnt be that slow, if asyncpg doesn't lie about its speed

I extracted the latest 30 katas everytime I ran !update_kata_db, so I guess considering overlaps there are around 100 katas atm.
I don't think that's so large as to cause performance issues

Ah alright, seems like a good idea to me, though we would have to wait till we get a db setup

Fair enough
Maybe json could be a temporary solution?
In that case I might have to put aside my hatred for the format for the time being lol

no no no no not json :no_good:

well that's good news I think
any other ideas or do we put this on hold till the db gets set up?

i think we will have to wait for the db :((

sure, no probs.
I'll keep my eyes open on #93 then.

You can except the DB pr by next week, a lil busy with a hackathon this weekend

@ShakyaMajumdar would you like to start working on this?


Android Doesn't work

On Android it says The file associated with this item is not accessible by the server. Please verify that the file exists and the necessary drive is accessible."

Because the add isn't compile for android, sorry I have no plan to cover it ATM.

So Plex Android Client support is not in the works at all?

Oh my bad; I think you try to launch it on android.... sorry.
It does work on client it depends of the stream; I am working on a remux atm to all the stream issue.
Does it work on desktop with that channel?

Yes I am. I'm using vaders and I added the ts to the end of the url to indicate ts

Hey @12nick12 , did recently had to watch a match into my phone; didn't got any issues; could you try with another android device?

It doesn't work on the newer app with the new ugly UI. I just downgraded to an older version of the apps it it works flawlessly. Gotta love Plex breaking things.

It's not working again. It must be something to do with my IPTV provider. It's works great on Roku and the Web. Thats more than enough for me.


babe: use SelectChain for block import fork choice

This switches to use SelectChain for fetching the best block header in block import.

@sorpaas , @andresilva is this still  relevant?

I shared some comments relating to this on the Consensus Team channel. Since this is currently broken because of the light client I'd suggest we close it.


Padrino return 404 when "provides" is set and IE is used.

I find strange behaviour in padrino.
It seems router bugs.
Environment
Windows
% ruby -v
ruby 2.1.6p336 (2015-04-13 revision 50298) [i386-mingw32]

Linux
$ ruby -v
ruby 2.1.6p336 (2015-04-13 revision 50298) [x86_64-linux]

Source Code
get :text, provides: :text do
  content_type 'text/plain'
  'hello world'
end

get :image, provides: :jpg do
  content_type :jpg
  filepath =  File.join(Padrino.root, 'files/padrino.jpg')
  File.binread(filepath)
end

Whole source code is accesible from tunepolo/padrino_router_bugs.
$ git clone https://github.com/tunepolo/padrino_router_bugs
$ bundle install
$ ruby bin/image_server -h <IP_ADDRESS>

Symptom
Note that '<IP_ADDRESS>' is my PC's IP.
The response difference is caused by browser's 'Accept' header.

Google Chrome : text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8
Internet Explorer(localhost) : text/html, application/xhtml+xml, */*
Internet Explorer(IP address) : */*

[Google Chrome]

URL
Success

http://localhost:3000/text
○

http://localhost:3000/image
○

http://<IP_ADDRESS>:3000/text
○

http://<IP_ADDRESS>:3000/image
○

[Internet Explorer]

URL
Success

http://localhost:3000/text
○

http://localhost:3000/image
○

http://<IP_ADDRESS>:3000/text
×

http://<IP_ADDRESS>:3000/image
×

Confirmed but not with IE 11.0.9600.17905
Had to install Modified Headers Ext to Firefox and send Accept: / to get 404.

@ujifgc  Yes, you're right.
The problem happens when the Accept headers has only '/'.
So problem always happen when curl is used.
I've updated the issue title to describe the symptom more correctly.


Inline loading accessibility

Accessibility content update for inline loading, both text and images

Just FYI, I ran format to get all of the checks to pass.

I have got some image edits to this PR. I will create another PR to make those changes and link here for reference.

@laurenmrice can you take a look at my suggested wording above to address Abbey's concern, and incorporate into yours if that works for you?


cellrank.tl.kernels.VelocityKernel deterministic mode

Thourougly investigate why it doesn't match with our expectation (scvelo)

https://github.com/theislab/cellrank/blob/master/tests/test_kernels.py#L1026

Versions:
https://github.com/theislab/cellrank/tree/9eca5d883844dd72058045c1bbb84f56226ea729

Could you first check whether it matches on the pancreas? After all, there may be a problem with our generated AnnData objects? Keep me up to date on this one please, would be great if we could figure this out today somehow.

Awesome, thanks a lot for figuring this out. Remaining to do's from my point of view:

[ ] add a big comment to our code that reminds us of this finding in case we stumble upon this again in the future
[ ] modify test_pearson_correlations_{fwd/bwd} to only compare on the indices that we agree on. Pearson correlations for these indices should be the same between cellrank/scvelo
[ ] modify test_transition_probabilities_{fwd/bwd} to use cellrank's pearson correlations for the velo_graph. That should fix the discrepancies.


Tried to download(undefined): https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-linux-x64.tar.gz

when I run
 npm install -g node-inspector

throw  error
node-pre-gyp ERR! Tried to download(undefined): https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-linux-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI, glibc) (falling back to source compile with node-gyp)

Hi, I'm getting the same error.
Running on Mac OS X with 5.3.0 npm version.
Console output:
node-pre-gyp ERR! Tried to download(404): https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI, unknown) (falling back to source compile with node-gyp)

 CXX(target) Release/obj.target/debug/src/debug.o
../src/debug.cc:41:20: warning: 'Call' is deprecated [-Wdeprecated-declarations]
        v8::Debug::Call(context, fn);
                   ^
~/.node-gyp/8.4.0/include/node/v8-debug.h:192:3: note: 'Call' has been explicitly marked deprecated here
  V8_DEPRECATED("No longer supported",
  ^
~/.node-gyp/8.4.0/include/node/v8config.h:332:29: note: expanded from macro 'V8_DEPRECATED'
  declarator __attribute__((deprecated))
                            ^
../src/debug.cc:52:45: warning: 'GetDebugContext' is deprecated [-Wdeprecated-declarations]
        Isolate* debug_isolate = v8::Debug::GetDebugContext(Isolate::GetCurrent())->GetIsolate();
                                            ^
~/.node-gyp/8.4.0/include/node/v8-debug.h:208:3: note: 'GetDebugContext' has been explicitly marked deprecated here
  V8_DEPRECATED("Use v8-inspector",
  ^
~/.node-gyp/8.4.0/include/node/v8config.h:332:29: note: expanded from macro 'V8_DEPRECATED'
  declarator __attribute__((deprecated))
                            ^

npm debug.log:

"/usr/local/homebrew/bin/npm" "install" "-g" "node-inspector"
6216 verbose node v8.4.0
6217 verbose npm  v5.3.0
6218 error code ELIFECYCLE
6219 error errno 1
6220 error<EMAIL_ADDRESS>install: node-pre-gyp install --fallback-to-build
6220 error Exit status 1
6221 error Failed at the<EMAIL_ADDRESS>install script.
6221 error This is probably not a problem with npm. There is likely additional logging output above.
6222 verbose exit [ 1, true ]

Running:
npm install -g node-inspector
Gave pretty much the same error for:

nodejs  version 8.9.1
npm 5.5.1

node-pre-gyp ERR! Tried to download(404): https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI, unknown) (falling back to source compile with node-gyp)
  CXX(target) Release/obj.target/debug/src/debug.o
../src/debug.cc:41:20: warning: 'Call' is deprecated [-Wdeprecated-declarations]
        v8::Debug::Call(context, fn);

I have same error as @zeristor .

I have same error as @zeristor .

We have same error as @zeristor .

Hi, It looks like this package does not work with node 8(#1010). For fix just change node version in petstore/Dockerfile to 6.10.2 or less than 8.

We have same error as @zeristor .

+1, same error.

+1

Same error for:

npm 5.6.0
node 8.9.3

+1

+1

Same error for me.
npm 5.5.1
nodejs 8.9.3

+1

Same error for me.

same error

same error on node v9.2.0 or v8.2.0

Same here on macos with node v8.11.0

I am also facing same issue, node v9.1.0
Mac OS 13

Same here on macos with node v9.10.0

same error

same error with "gulp-node-inspector": "^0.2.1",
node-pre-gyp http 404 https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-darwin-x64.tar.gz node-pre-gyp ERR! Tried to download(404): https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-darwin-x64.tar.gz  node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI, unknown) (falling back to source compile with node-gyp)  node-pre-gyp http 404 status code downloading tarball https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-darwin-x64.tar.gz    CXX(target) Release/obj.target/debug/src/debug.o ../src/debug.cc:41:20: warning: 'Call' is deprecated [-Wdeprecated-declarations]         v8::Debug::Call(context, fn);                    ^
╰$ node -v v8.12.0
LTS node version should be supported.:)

Same error, how to resolve this?

npm install -g<EMAIL_ADDRESS>maybe ok

npm install -g<EMAIL_ADDRESS>maybe ok

Works in Ubuntu 16.04 Node 8.10.0 npm 5.6.0

npm install -g<EMAIL_ADDRESS>maybe ok

how is it work?


Escaping % characters in request path.

I am running into an issue where I still get the mismatched signature from AWS ES. The request looks like this:
PUT /products/product/foo%3A0234aa9d45c45284fbd3f5dde43e09fc95cf05dc
AWS says that it wants the canonical request to be:
PUT /products/product/foo%253A0234aa9d45c45284fbd3f5dde43e09fc95cf05dc
Basically, it looks like we need to escape % characters in the path, not just in the query string.

:+1:

A fix for this has gone into today's 0.7 release (github and pypi). Turns out S3 was the only one that needed special handling. Cheers again for the report.


How to leverage the maxWidth property of SebmGoogleMapInfoWindow?

I'm using the latest version of angular2-google-maps (0.14.0) and I'd like to use the maxWidth property of SebmGoogleMapInfoWindow, but it's unclear to me (from the documentation) how you would pass this configuration option. Is this only possible when calling SebmGoogleMapInfoWindow programmatically instead of creating it in a template like below?
  <sebm-google-map-marker *ngFor="let location of locations" [latitude]="location.latitude" [longitude]="location.longitude">
    <sebm-google-map-info-window [disableAutoPan]="false">
      InfoWindow stuff...
    </sebm-google-map-info-window>
  </sebm-google-map-marker>

@mensch it's possible like this:
<sebm-google-map-info-window [disableAutoPan]="false" [maxWidth]="myWidth">
      InfoWindow stuff...
    </sebm-google-map-info-window>

Feel free to reopen if you think there is a bug.

Thanks for the reply! I tried using maxWidth as an input (as well as zIndex), but I get the following error:
Can't bind to 'maxWidth' since it isn't a known property of 'sebm-google-map-info-window'.
For some reason isOpen and disableAutoPan do not trigger the same error, I'm using them without any problems.

Oh you're right, thats a bug! The binding is missing. We add that to the todays release (with angular 2.0.0 support)


Transcoding feature support

following transcoding  features validation can be added as an enhancements to vaapi-fits.

Transcoding  AVC test streams to AVC, HEVC, MPEG2 and MJPEG using HW decode->HW encode, HW decode->SW encode ,SW decode ->HW encode.
Transcoding  HEVC test streams to AVC, HEVC, MPEG2 and MJPEG using HW decode->HW encode, HW decode->SW encode ,SW decode ->HW encode.
Transcoding  MPEG2 test streams to AVC, HEVC, MPEG2 and MJPEG using HW decode->HW encode
Transcoding  VC1 test streams to AVC, HEVC, MPEG2 and MJPEG using HW decode->HW encode.

@haribommi,  have implement it on ffmpeg/gstreamer, can you have a verification and close it?

Implemented this feature with following patches:
https://github.com/intel/vaapi-fits/pull/17
https://github.com/intel/vaapi-fits/pull/34
https://github.com/intel/vaapi-fits/pull/39


Should we document Rights Statements and Licenses, and how?

About Rights and Licenses
In the possible information that CHIN will handle, there are to kind of data:

factual data and metadata, that are not subjected to intellectual property rights (at least in Canada), but some institutions still adds a license to such data (like CC0)
Images, photos, long texts and other kind of data that could be subjected to intellectual property rights.

The first question that arise is about who owns the licenses: Does CHIN owns the licenses of the factual data and metadata (and can therefore add a license to them)? If not, is it the providing institution who owns them?
For the images and other kind of data, it is the providing institution that holds the rights (or the artist in some cases).
When CHIN owns the rights, it can created a license, using the Creative Commons. If CHIN does not own the rights, it can simply declare the rights statements, using the RightsStatements.org
Modeling
Model-wise, rights and licenses can simply be stated using Dublin Core Terms.
In case of the factual data and metadata, the easiest way is to add a license or rightsstatements to the NamedGraph, like in the following diagram. In that case, all the data within the NamedGraph should be subjected to the license or rights stated in the NamedGraph.

https://drive.google.com/file/d/1mPOFRgl6zz6giX5YmwVeCQEWI9dmodyP/view?usp=sharing
For the Images, Texts or other data subjected to intellectual property rights, the same pattern using Dublin Core could be added, to specify the rights pertaining to the particular data.

https://drive.google.com/file/d/1y0fJxE11O4ldr1sFo8515LKLFQkbN0jV/view?usp=sharing

Question: why would you not use the rights properties and classes native to CRM?
ie Object -> P104 is subject to -> E30 Right and then link to your rights statement URI as a type.
I would argue that aside from being consistent with the standard, the reason to instantiate a Rights node is that there are different kinds of right and right can also have different temporal limitations. Therefore, if you don't instantiate such a node the long term maintenance of this rights information could become quite tricky.

I would not use a named graph (alone) to attach rights. While it describes the situation,  it's not enough for software agents (crawlers) who stumbled upon the resource to be confident they can use it. So you also better have in on each "main record" or entity (eg Artist, Artwork).
As for the whole dataset, it's best to use VOID (and DCAT), see http://vocab.getty.edu/doc/#License_Info for an example.
The next section shows how its attached to individual items,  though that is probably slightly wrong since the Getty does not hold any rights over a concept such as Rhyton, only over Getty's description thereof. But we wanted to keep it simple.

I think this is highly binded with #45 since the Named Graph level (dataset vs record) will impact the pattern.
@VladimirAlexiev So should we state the rights at the Named Graph level, since it's not convenient for crawlers and that it's not exactly right to say that a factual data is under a specific license? Does this kind of info is important for some data hubs? If so, can we explain that our datasets contain various types of licenses?
I would be happy with a right pattern that covers specific entities of our TM like images and long texts. I also agree with @Habennin that we should see the rights as a simple URI. We should state the right as a node to be able to describe it more precisely. @stephenhart8 was there a reason why we've decided to use Dublin Core in the first place?
Another question to all, should we have two different patterns for the "right" and the "right statement"? Following rightsstatements.org it seems like there is a slight difference between both.  I didn't explore what have been done around those questions, but it's probably something that @Habennin has already tackled. ;)

From the discussions in the Semantic Committee on issue #17 "How to model images and how to integrate image URLs?", the issue of rights (of images) has been touched on.
For the moment, dct:source is used to indicate the image bibliographic mention, but this kind of information should be refined. It has been raised that there should be a semantic distinction between the kind of sources (rights, licenses, etc.), so that more refined information can be documented. For the moment, the list of fields needed is:

Rights
Provider
License
Creator

More effort should be put into establishing the list of elements needed for recording what “sources” entails.
Other questions raised during the meeting:

What ontology should be used to model rights? Is CIDOC CRM enough?
Where to place the rights statement? In the case of images, should it be on the Visual Item? On the object? On the Digital Object?

It is worth noting that this question is important, even if it is not possible to publish the protected information. Indeed, it is still possible to document the object (image, text, etc.) without publishing it


⚠ ClamAV has degraded performance

In 296932c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 465 ms

Resolved: ClamAV performance has improved in 803a53f.


feat(notifyManager): batch method configuration

As discussed previously in #6592, adding a way to configure the batch scheduling behavior can be useful, e.g. to prevent unwanted rerenders in solid-js (although I haven't fully gotten to the bottom of why solid-query does this).
This pull request adds notifyManager.setBatchMethod(...)
Sets the method for batching. There are four methods:

{ type: 'timer', timeout?: number }:
The default is to use setTimeout with a timeout of 0. This has the effect of
scheduling a javascript task in the event loop some time later.
{ type: 'raf' }: This batches all updates in a requestAnimationFrame, so
that it runs before the next frame is rendered by the browser.
{ type: 'tick' }: This uses queueMicrotask to schedule a microtask to batch updates, which runs immediately after the current task is done.
{ type: 'custom', fn: (() => void) => void}: The custom method allows you to specify your own way of scheduling batches

This API was inspired by the redux autoBatchEnhancer API
I added some tests, and documented the notifyManager while I was at it. I would appreciate feedback, because I've not written many docs before and I'm not 100% sure if I documented all the functionality correctly.
I also added a demonstration in the solid-start-streaming example to demonstrate how setTimeout (the default batch method) can sometimes result in layout shifts solidjs. I don't know if this is due to the way how solid works or if solid-query can adapt to prevent this from happening. Either way, changing the batching method to 'raf' or 'tick' also prevents it from happening.

Although I would need to look deeper into this to be sure I suspect this may be beneficial for the Angular adapter too. setTimeout triggers change detection and queueMicrotask may well be more efficient. At the very least it's good for adapters to have options.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (29770f8) 39.12% compared to head (bb8b17a) 56.23%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #6600       +/-   ##
===========================================
+ Coverage   39.12%   56.23%   +17.11%
===========================================
  Files         161       99       -62
  Lines        6597     4088     -2509
  Branches     1297     1004      -293
===========================================
- Hits         2581     2299      -282
+ Misses       3665     1570     -2095
+ Partials      351      219      -132

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


FTP connector

I saw the list of connectors that on-premises data gateway supports, but there isn't FTP, so is not possible to connect a logic app to a FTP server on premises?

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

ID: 7892e678-a98f-73fa-7c54-1a8a921f35c7
Version Independent ID: 064c9db9-5e18-7266-1740-ada24a2086a1
Content: Access data sources on premises - Azure Logic Apps
Content Source: articles/logic-apps/logic-apps-gateway-connection.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@DanieleG90 There is a feature request for supporting this scenario which you could up vote for it to gain more traction.
While this is not supported, another way to setup access to on-premise data sources is by using an Integration Service Environment which is deployed into a VNET which you can connect to your on-premise network using VPN or ExpressRoute.
Another option would to leverage Hybrid Connection in Azure App Service which allows for secure access to on-premise resources similar to the On-Premise Data Gateway.
Since there is no doc update required here, 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.


How to prevent ack on message consume failure

Hello,
If I am consuming QOS 1 messages, and encounter an error consuming message, how can I prevent the message from being acked? If I have an uncaught exception inside of my client.UseApplicationMessageReceivedHandler handler will it redeliver ?
Thanks,
Colin

You can set MqttApplicationMessageReceivedEventArgs.ProcessingFailed to true. Then no ACK will be sent.

@chkr1011 Shouldn't that be the default when an exception is catched when QoS1 is requested?
So if you want QoS1 you need to do something like the following?
public async Task Handler(MqttApplicationMessageReceivedEventArgs arg)
{
    try
    {
        // Do your magic here
    }
    catch (Exception ex)
    {
        arg.ProcessingFailed = true;
        throw;
    }
}

I assume this question in properly answered.


first commit

My first pr for Hacktoberfest

Thank you for sending this PR and attending the event.
Here are some resources that'll help you in your Open-Source Journey - https://docs.google.com/document/d/1nZ4kwCXzbOXtJzvy5RaCzlA2UXs5TCD6N-TK_qAMpuo/edit
Stay Curious! and remember to have fun while learning ;)


HV: export crypto function kdf_sha256 and derive key from seed

Derive key for attestation keybox decryption
Signed-off-by: Chen, Gang G<EMAIL_ADDRESS>

please also update title and comments for naming changin.g

@GangSecurity  again - please also update title and comments for naming changing


Typo fix

Step 2 in Debugging has incomplete sentence.  Should be something like "Press any key to close the window."

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

ID: d684aa29-fb98-1d88-a930-89813a98eb17
Version Independent ID: db38dce1-01d0-dc84-da3f-67cd925dd134
Content
Content Source

Thanks! This is addressed in https://github.com/MicrosoftDocs/visualstudio-docs-pr/pull/1192, which you may not be able to see because it's in our private repo.