Open Sample Errors on Car Wheel Sample

Description
Using "Open Sample" on samples that use in() or mm() (or equivalent) causes errors. When I opened the Car Wheel sample, I got an error becuase the units were in mm(), but it is expecting inches. We need to also bring in the units along with the sample.
Version
v0.25.4
Recordings/Screenshots

Ah interesting! That makes sense, but it does complicate things. In the web we can just override the unit settings if you think that makes sense. In the desktop app, what should we do?
Like if I have a project with mm as my units for a project, I expect the user won't want to overwrite their project's units, right? Do we need a way to transform samples that use any unit function (like in() or mm()) to be the sample but using a different unit system? That sounds super hairy. Or do we only allow creating a new project if the file uses a unit function, because that means its units actually matter?

Or transform all raw literals in the KCL to use the unit function in the sample's setting if any unit function is used anywhere within that file? So because the car wheel sample uses an mm() we transform it to use in() on every literal value when we import?

how many uses of mm()/in() are there in the samples?
Do we just remove them all for samples?

When users import a sample, I'm fine with overwriting their units for that project, perhaps with some message "This project will switch your units to in" or something like that. If they are asking for a sample, I feel they should be okay with getting all of the sample (toml included)

There's only this one I believe @Irev-Dev

Off of the top of my head, this makes me wish that every file could declare at the top which units it uses. This would allow you to import/copy a file from another project, and it would just work, regardless of what units you use. Mixing and matching wouldn't be a problem. Maybe it's not a best practice, but it would help ease friction with remixing.

We did have the idea of putting project level settings in a special comment at the top of the files, (eventually including dependencies too). The idea being would help folks share minimal reproducible examples, so that sounds good.

But at the same time, if there's two usages, maybe we just remove those, instead of implementing a whole new feature to fix this problem.

I'm liking @jtran's logic that the global units is a config setting in the KCL, that way it carries with the part.

adding my opinion from slack, ABSOLUTELY DO NOT ABUSE THE mm() in() functions where not necesary. We already ahve a way of knowing the units, its the project.toml, kcl-samples has that information, I say:

read the project.toml for the projhect
if users units differ prompt with a modal "This file is in inches cahnge project to use inches?" then they get the choice
We need the same functionality for text to cad

in the future with assemblies we could do something like @jtran suggests, but also mixing units among lots of files seems like a bad idea and a source of pain in general, like imagine in our repo we had some analogue of this, it seems insane

I can read the project settings of the sample and provide warning of the sort @jessfraz is talking about easily.

Is it common for projects to be in one unit system by default, but then need subsystems or parts within it to use another unit system? Like is it common to have a project assembly in imperial that has one part in metric? Because that points toward needing some part- or file-specific unit definition.

i like @jtran suggestion but i dont want to add a second way of getting the units for files without really thinking it thru since we will be stuck with it forever, like 1. are we sure we even need that, 2. is there a better way, like lets actually put thought into it, not that @jtran didn't (maybe hes been sitting on this for days) i just dont want to do it as a fast fix response

because i imagine a scenario where users NEVER use the project settings and instead put inches at the top of every file, like oye...

Yeah 100% we should think about it for a while. We have a plan for a UX bug fix that requires no deep changes

For the purposes of this bug, we will overwrite the current user's project settings to match the sample if they don't match the current settings. We'll warn the user on the confirmation step so that they can cancel before they complete the import.

Absolutely. Let's be careful and think through. I haven't done that yet. The reason it came to me is it's often nice to have everything self-contained in a single file. (There's even been proposals to put Cargo.toml contents in .rs files to make single-file scripts.) But maybe it doesn't apply to units. I don't want the format to encourage bad practices.

in most cases i think they'd overwrite, even better, when you load the thing that says "open in new project" tell them about the units then so they can open in a new prohect with those units.

I've broken that out into #3969


[FEATURE]Adding a reduced set of Adjectives(English)

Based on this Reddit post, there is a desire out there for a reduced set of Adjectives to use for TokenMold. I thought I'd offer up the listing that u/VagabondVivant offered for inclusion in the next update to TokenMold. The listing can be found, in text form, here.
I've attached their 571 word listing as a FVTT pack db to make things easy to import if you'd consider it. I zipped it so that I could attach it here.
adjectivesreduced.zip
Thanks (still love your work!).

Updated the list (removed some words that didn't really work, added a few new ones: https://textbin.net/AzIA7sUopE

i'm fine with adding them and possibly even as default (cause its a more curated list).
though i'll wait a bit until you're all settled on one set of adjectives :sweat_smile:

Please consider retaining the larger database as an option though. Thanks.

i'm fine with adding them and possibly even as default (cause its a more curated list).
though i'll wait a bit until you're all settled on one set of adjectives 😅

I thought I was done (I had only edited it once I realized it could go into the mod), but now that I have the option to refine it some, my inner perfectionist wants to take one final crack at it. I'll give it one last edit tonight and update the thread when it's ready ready.  😁

Please consider retaining the larger database as an option though. Thanks.

yes ofc!
just thinking about the new one being the defaullt (for new users)

Okay it's done. 700 words, tidied and capitalized. I'm sure it could be longer but I'm done. 700 is plenty. Enjoy!

Thanks for your work!
List was added with Token Mold 2.9


security_enc conversation

While an email and or a URL is the absolute minimum it may also be prudent to allow for encryption method for communications  maybe gpg and keybase
For me I'm on the fence, such as following the K.I.S.S (Keep It Stupid Simple) principle, arguably the email and or URL is enough to signpost (job done ?) the next
declaire the encryption presence with security_enc followed by a schema identifier and the key/id
Some examples( We spitballing):
GPG Short Key ID
security_enc= GPGSK:AAAAAAAA
GPG Long Key Example
security_enc= GPGLK:AAAAAAAAAAAAAAAA
GPG Fingerprint
security_enc= GPGF:AAAA AAAA AAAA AAAA AAAA  AAAA AAAA AAAA AAAA AAAA
Keybase Fingerprint
security_enc= KBF:AAAA AAAA AAAA AAAA AAAA  AAAA AAAA AAAA AAAA AAAA
My thoughts are, having it as a supported parameter is a secondary question to the main question:
Where can I have a security conversation ? - do we want to introduce another question that is how can I speak to security, or should that be left as part of step two in a conversation ?
Open for thoughts

Alright so, I'll close this, if your email address is declared, feel free to provide a fingerprint as above or an additional URL to the PGP/txt with the full key / reference


Global styles are inserted after component styles again in v5

This issue is similar to 1582, but now for v5.
@import rules are loaded in the same style block as other styles, not in a separate style block as they should be. This was fixed in 1577, but it is broken in v5 🙂
Environment
System:

OS: macOS 10.14.6
CPU: (8) x64 Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz
Memory: 1.27 GB / 16.00 GB
Shell: 5.3 - /bin/zsh

Binaries:

Node: 10.16.0 - /usr/local/bin/node
Yarn: 1.16.0 - /usr/local/bin/yarn
npm: 6.9.0 - /usr/local/bin/npm

npmPackages:

styled-components: ^5.0.0-beta.8 => 5.0.0-beta.8

Reproduction
In this codesandbox you can see the style block styled-components exports contains all styles and the @import rule at the bottom of it, and not in a separate style block above all other style blocks.
Steps to reproduce

Create new codesandbox for React
Install<EMAIL_ADDRESS>Create a styled component
Create GlobalStyles with a @import rule.
Inspect the <head /> of the document and see the <style /> block generated by styled-components.

Expected Behavior
Have 2 styles blocks in the <head />. One for the @import rule and one for other css.
Actual Behavior
Only 1 style block that contains all css and the @import rule somewhere in the middle.

I was checking out other issues and PR's in the repo and found 2357. So it is probably just not implemented yet, and that is why it is not working.
Right now I can fix this issue by creating my own <style /> block in my index.html file and import the fonts there, if someone was wondering.
Keep up the great work! 👍


Added The-Epic-Snake-Game w/ Pictures and Instructions

Sorry, but there are already two snake games in the repo. You can see #43 if you want to improve on that.


Adding dev-tool to the repo

This is the current iteration of dev-tool. It serves as a replacement for some scripts that I wrote to make the samples process easier to manage. The hope is that this command will be easy to extend and use for any purpose that we wish, so I hope everyone will have some time to take a look at it and ask questions.
I added tree-nesting at the suggestion of @richardpark-msft , and I think it has made the command a lot more semantically usable. The tree in src/commands mirrors the tree of commands.
The gist of extending the command is:

create a new TS file with a commandInfo export that follows the CommandInfo interface (mark it with as const for great type-checking)
add a mapping from some index.ts sub-command file to your new command.

There is a full developer manual and tutorial in the README.
Currently, only Text Analytics is adapted. I will test it in this PR and then if it succeeds, I will add another commit to this PR that converts all users of common/scripts/prep-samples and common/scripts/run-samples to use dev-tool instead and removes the old scripts.
(CC @xirzec @jonathandturner )
The new samples development workflow looks like this:

dev-tool samples dev
dev-tool samples run samples/typescript/src/analyzeSentiment.js (for example)

The CI process has not changed.
@bterlson There is a type in src/util/commandBuilder that is pretty ugly. It does the work of making a command handler's options types strong (it is basically a type wrapper around the output of minimist). If you have any ideas about how to clean this up a little bit, I am all ears.

 Need this when doing sanity check for releases; currently I am reusing the old script: https://github.com/ljian3377/azure-sdk-for-js/blob/sample_sanity_check/sdk/storage/execute-samples.js

@ljian3377 not yet on either of those points, but we have plans to use this tool to build a solution that will enable both of those easily and without having to copy anything around.


Callback option for when upgrade notice is triggered

I'd like to log to ganalytics when this lib detects the browser is below the configured threshold. I can best log the event if there was a callback option in the outdatedBrowserRework constructor configuration. I'd recommend using the property name onOutdated.

Sure! Send a PR.
Note the code for this project has to be ES3.

Seems like this PR is related https://github.com/mikemaccana/outdated-browser-rework/pull/32


Vulnerable Regular Expression

The following regular expression used to parse dates specified as strings is vulnerable to ReDoS:
/[0-9]*['a-z\u00A0-\u05FF\u0700-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF]+|[\u0600-\u06FF\/]+(\s*?[\u0600-\u06FF]+){1,2}/i

The slowdown is moderately low: for 50.000 characters around 2 seconds matching time. However, I would still suggest one of the following:

remove the regex,
anchor the regex,
limit the number of characters that can be matched by the repetition,
limit the input size.

If needed, I can provide an actual example showing the slowdown.

For clarification, this is in the matchWord regex, in /src/lib/parse/regex.js here.
Here is a railroad diagram of the regular expression.  From this we can see that the grouping with repetition is related to parsing Arabic characters.   It would be helpful if someone who understands both regular expressions and Arabic language could take a crack at this.
An overview of ReDoS is also helpful.

Ill take this is no one has already started

@hamiltondanielb all yours :)

@cristianstaicu
If needed, I can provide an actual example showing the slowdown.

I'd like to see the some example if it's possible :)

@Drag0s sent it on your private email.

FYI, this was added to NSP (see here), so this is probably going to start breaking people's builds soon.
@hamiltondanielb - Did you get anywhere looking into this?

Just had our build break 👯

It's sad that the issue had to become public before it was fixed. Issue opened on Sep 8, NSP advisory published today. @cristianstaicu perhaps you should've reminded the maintainers about the disclosure deadline to give this some momentum.

@mattgrande there is a meta-ness to this: turns out the version of nsp we were pinned to (2.8.1) depends on moment (via joi) so it was reporting a vulnerability on its own dependency:

Upgrading to nsp 3.1.0, resolved this because the dependency is no longer there - so beware of that if you don't directly depend on moment.

Is there a fix for this yet?

Please advise any fix available?

No fix has been published yet.
Please, if you're interested in getting updates from the maintainers, subscribe to notifications for updates to this issue by clicking the "Subscribe" in the right-hand column.

To add an nsp exception for this, add a .nsp file:
{
  "exceptions": [
     "https://nodesecurity.io/advisories/532"
  ]
}

Hi @westy92. I have the following code.
var tasksMethods = require('gulpfile-ninecms');
gulp.task('nsp', tasksMethods.nsp);
Its not picking up the .nsprc file exception. It keeps giving me the vulnerability error for moment.
I added the file on root of the project. Is there anything I am missing to do?

@Dexterslab we're using gulp-nsp, which works fine when the .nsprc is in the project directory (same level as package.json).  Maybe try using gulp-nsp directly?

@cristianstaicu @mattgrande is this happening in Luxon as well?

We'd appreciate it the fix for this can be expedited, now that it has been logged in nsp it is failing our builds.

Bump for fix

Can I suggest the maintainers to lock this convo?
For everyone waiting for a fix, just hit the subscribe button, and you’ll be notified.
If anyone want to undertake the fix, feel free to open a PR.

There is a PR for this already https://github.com/moment/moment/pull/4326

Does someone need to notify nsp this has been patched? https://nodesecurity.io/advisories/532

This has been fixed in version 2.19.3 . Please update accordingly.
I'm not sure how to notify NSP -- @cristianstaicu could you or others help here?

cc @evilpacket @lukekarrys @nlf @wraithgar

I emailed<EMAIL_ADDRESS>
Hello,
The currently outstanding moment.js vulnerability (https://nodesecurity.io/advisories/532) was recently patched and the fix released in v.2.19.3.
What's the process for reporting status updates on advisories?  This email address is the best avenue I could find.
Thanks!

Just got a reply that the NSP DB should be updated with 2.19.3 as the fixed version.
Verified on my end that 2.19.3 no longer alerts.


Default nrfx PWM interrupt prio results in assert

NRFX_PWM_DEFAULT_CONFIG_IRQ_PRIORITY defaults to 7 on all nRF52 SOCs
enabling CONFIG_ASSERT=y results in following assert on boot:
ASSERTION FAIL [prio <= ((1 << 3) - 1)] @ /home/janc/devel/external/zephyr/arch/arm/core/irq_manage.c:114:
invalid priority 7! values must be less than 7

The IRQ priority values from "nrfx_config_*.h" files are not used in Zephyr.
And they are not supposed to be used by any application built on Zephyr.
We'll be refactoring these files soon and such non-relevant entries will be removed from them to avoid confusion like this.

All clear now. Thanks!


Custom aggregator: weighted average

I would like to implement a weighted average using a fixed column as the weight value. That is, I could specify in advance that I want a specific column to serve as the weight. I am happy to alter the source of either dash-pivottable or react-pivottable but I'm not sure where to begin.
Does someone have a suggestion of how to or an example of implementing custom aggregation functions?
Thanks,
Matt

I think it's better to first fork react-pivottable and try your experimental feature; if it achieves what you are looking for, you can share it with the Dash community (community.plotly.com) and start a PR in the react-pivottable repo!
As for custom aggregators, I'm not immediately sure if it is exposed in react-pivottable. A good place to start might be this post, then diving directly into the react-pivottable's source code to locate where the aggregators are defined.


enhance(schemas): Support date subsitution via string replacement

As followup from https://github.com/dendronhq/dendron/discussions/2207, we will use string replacement instead of lodash to support date variable substitution. Since we don't need javascript evalaution, we find this will be simpler. We will still use lodash default interpolate delimiter
Testing
Create template as follows:
Year is <%= CURRENT_YEAR %>
Month is <%=CURRENT_MONTH %>
Hour is <%=CURRENT_HOUR%>

This links goes to [[daily.journal.<%=CURRENT_YEAR%>.<%= CURRENT_MONTH%>.<%= CURRENT_DAY%>]]

Creating note with template leads to:
Year is 2022
Month is 01
Hour is 13

This links goes to [[daily.journal.2022.01.26]]

Madge
Before: 130
After: 130

Code looks good to me - but we should have a corresponding docs change to detail the new format?


Aliases with SCSS composes behaves strangely

I have this inside one of my scss files:
.red {
  composes: block from 'components/Block.scss';
  background-color: red;
}

which causes fuse-box to give me the following error:
{ Error: ENOENT: no such file or directory, open '/Users/justinnel/Documents/Projects/Cyberlane/fusebox-test/src/components/components/Block.scss'
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '/Users/justinnel/Documents/Projects/Cyberlane/fusebox-test/src/components/components/Block.scss' }

My assumption is that how aliases are working, does not cover the use case of composes within SCSS?
A project with the problem can be found here: https://github.com/Cyberlane/fusebox-test

@Cyberlane i don't think that's FuseBox, i think it's sass reporting

@nchanged it behaves correctly when compiled with WebPack when using modulesFolder.
Is there an extra param I perhaps need to set which FuseBox feeds to sass?

you can try importer : true with macros
https://fuse-box.org/plugins/sass-plugin#macros

That doesn't seem to make a difference unfortunately.
I did however stumble across this, so perhaps it's a clue as to where the logic should live?
https://github.com/webpack-contrib/css-loader#importing

It should work with macros, create one that points to any folder, like @ for example, there are examples

FuseBox v4 Major Upgrade
As of today, the latest major version of FuseBox v4 is available in fuse-box@next. FuseBox has undergone a full re-write from scratch, learning from its mistakes in the previous versions.
There are a lot of improvements, speed and config wise.
Get started here
We are still working on the documentation and polishing before the major release. We are getting very close. And we need help testing.
npm install fuse-box@next

Most of the issues that I am closing have been resolved, however, if you find that your issue is still relevant in v4 don't hesitate to re-open or (better) create a new one with v4 configuration.


Introduce new navbar

This navbar is the same that was introduced in the Documentation site, relative issue https://github.com/netdata/learn/issues/1621


Change maximum allowed value for Integer Type in Custom Fields

Environment

Python version: 3.9.1
Nautobot version: v1.0.2

Proposed Functionality
The maximum value of an Integer used in a CustomField should be bigger than<PHONE_NUMBER>.

Use Case
I'd like to have a custom field ASN, which by nature needs to go up to<PHONE_NUMBER>.

Database Changes
Unknown

External Dependencies
None

Hmmm great idea. This would likely be as "simple" as changing CustomField.validation_minimum/validation_maximum fields from PositiveIntegerField types to PositiveBigIntegerField. Of course we'd still have to account for testing, and to make sure that the schema migration for the field type also includes a data migration. Because of this, we likely wouldn't be able to implement this until v1.2 at the earliest.

I can work on this.

@Joezeppe thanks Sammy, go for it!


Ensure blas produces pkg config files

Motivation for this change
Some Haskell packages use PkgConfig-Depends: blas and currently the blas package doesn't produce these.
Things done

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

[ ] NixOS
[ x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"

I tried to use this but
nix-shell -I nixpkgs=~/nixpkgs -p nix-review --run "nix-review wip"
...
Has to be execute from nixpkgs repository

and
nix-shell -p nix-review --run "nix-review wip"
error: file 'nixpkgs' was not found in the Nix search path (add it using $NIX_PATH or -I), at (string):1:13

[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @
There doesn't seem to be a maintainer (at least I couldn't find one in the default.nix)

@GrahamcOfBorg build blas
@GrahamcOfBorg build haskellPackages.blas-ffi haskellPackages.blas-hs


[IMPAC-631] use ngStyle when binding values

IE does not like angular expressions in html attributes like style. @ouranos

👍


Upgrade to Fragment.onAttach(Context)

API 23 introduced Fragment.onAttach(Context) and deprecated Fragment.onAttach(Activity). We should look into using the newer version (while maintaining backwards compatibility).

Support libraries added this as well (but did not deprecate the Activity version)

would it be ok for me to just change onAttach(Activity) to onAttach(Context) in my RxFragment class? Because it is not receiving onAttach(Activity) lifecycle events on lower api levels.

I think it's fine to just depend on the back-compat call for a while.


string interpolated parameter generates different function call

coffescript generates the following code:
source:
fun "hello"
  bla: "blub"

compiled:
fun("hello", {
  bla: "blub"
});

but using string interpolation we get a different result
source:
fun "hello#{i}"
  bla: "blub"

compiled:
fun(("hello" + i)({
  bla: "blub"
}));

it is no big deal, cause i can work around this by appending a "," after the first parameter. but the code without "," is nicer and can be read more fluent.
version: 1.2.0
is this the expected behaviour or kind of a bug ?

That first example:
fun "hello"
  bla: "blub"

... is actually the bug. It should require a comma there after the string in order to get the current output you're seeing. Commas are required to separate parameter lists.

I've encountered this issue as well.  Is there going to be an effort to enforce the comma requirement?  Unless code is being parsed consistently, there's going to be more sloppy code the breaks when this is changed.

Yes, we should fix it to disallow the initial example. This smells like a "feature" that was slipped in because it's technically unambiguous -- but that doesn't mean we should allow it.

In the grammar, it appears that there is an OptComma before TERMINATOR, and all lines of ArgList are governed by the same grammar rules.  An arglist like the following works:
x a,
  b
  c

but this does not:
x a
  b
  c

How is it that the first line of arguments is parsed differently?
I'm probably just not familiar enough with the code, but I imagine whatever's happening here makes the difference.


[grpc] update to 1.35 or latest

Library name: gRPC
New version number: 1.35
Other information that may be useful (release notes, etc...)

@kisan68, grpc has been upgrade to 1.37 in vcpkg, could you please get latest source and try again?

When I try  ".\vcpkg.exe search grpc", I can see that grpc is at version 1.33.1#3
Do I need to wait a while after the GitHub sync?

When I try ".\vcpkg.exe search grpc", I can see that grpc is at version 1.33.1#3
Do I need to wait a while after the GitHub sync?

You can run the command "git pull" to update vcpkg, if it still fails, please clone vcpkg again

Thank you. I tried a simple client-server example and it works.


TypeError: Cannot read property 'offsetWidth' of null

Hello,
I'm getting this error, even that it is working nice, but idk how to fix it. I'm running sapper/firebase
Uncaught (in promise) TypeError: Cannot read property 'offsetWidth' of null
    at isVisible (InfiniteLoading.svelte:145)
    at attemptLoad (InfiniteLoading.svelte:232)
    at scrollHandler (InfiniteLoading.svelte:225)
    at InfiniteLoading.svelte:308

I have the same problem.
In my case the error occurs because I add the 'InfiniteLoading' component via {#if items.indexOf(item) == 0} at the beginning of a Virtual List.
I think the source code should be optimized that way:
function isVisible(element) {
	return element && element.offsetWidth + element.offsetHeight > 0;
}

I have the same problem.
In my case the error occurs because I add/remove the 'InfiniteLoading' component via {#if items.indexOf(item) == 0} only at the beginning of a Virtual List.
I think the source code should be optimized that way:
function isVisible(element) {
	return element && element.offsetWidth + element.offsetHeight > 0;
}

This also suppresses the error when the component is destroyed with a pending fetch.
Uncaught (in promise) TypeError: element is null
    isVisible InfiniteLoading.svelte:855
    attemptLoad InfiniteLoading.svelte:234
    loaded InfiniteLoading.svelte:187

Hello there!
Hope this answer isn't too late...
I think this might be an issue with timing 🤔
When the component first renders, it sets thisElement to the container element using bind:this={thisElement}.
Now, if the scroll handler get's triggered BEFORE thisElement is assigned a value, it throws an error.
Not sure if or how this can happen...
Other than that I have no clue why this error occurs 😅
Maybe someone could provide an example code? Or some way for me to reproduce this?
Anyways, I've made @thebestgin suggested changes now. Hope that helps...

I think this might be an issue with timing 🤔 When the component first renders, it sets thisElement to the container element using bind:this={thisElement}. Now, if the scroll handler get's triggered BEFORE thisElement is assigned a value, it throws an error. Not sure if or how this can happen...

In my case element was destroyed by the time isVisible was called. Thanks for the fix!

I had a similar problem,my solution was to display the InfiniteLoading component on demand,meaning:
1.I get first batch of results from api.
2.Bellow i display a button,when clicked will set a viariable (showInfiniteLoading) to true.
3.I display InfiniteLoading component when showInfiniteLoading is true (and also hide the button,because there is no more use for it).


Data visualisation cheat sheet: geom_jitter() is not limited to continuous x continuous variables

Thanks for the great cheat sheets.
For the ggplot2.1 cheat sheet, geom_jitter() is listed under 2 variables:  continuous x, continuous y.
This can be a bit confusing for datavis beginners, as jittering is particular useful for discrete x, discrete y situations.
I find the continuous x, continuous y case is more of an advanced border case, so I would suggest to move the geom_jitter() into the other section in order to make the discrete use case more visible.

Anybody looking at this?


Suggestion: Radio towers - reinforcements - POIs

Short Issue Description
We played the new Lib a lot in rather small groups (2-7 players usually).
We noticed that the impact of disabled radio towers for our attacks has a pretty high impact. A disabled radio tower usually means that its somewhat a cakewalk to take a location, while its way more of a struggle if an active radio towers brings in reinforcement.
Thats definatly nice, but the gap between "easy" without a tower and "challenging" with an active tower seems to be too broad.
Suggestion

Some important locations (like airfields, the main towns, bases/bases with vehicle unlocks etc) should always be considered to have active radio towers to keep the battle a challenge.
(We're not sure, but it seems that main towns get constant reinforcements already? I ask, because we also had some suprising easy main town battles with 2-3 players)

Even with a disabled tower there could be a random chance that reinforcements show up. Would spice up things.

Thanks for considering. Keep up the great work! =)

The important targets are always covered by more than one radio tower in the maps I've provided.
And with 2-3 players you'll have a "easy" experience due to the "adapt to playercount" functionality, which is enabled by default.
Concerning the general thing you mention, I can only say that 96 will aim to everything "enemy related" stuff, which isn't touched until now and is still on the state of 924.
After my upcoming exams I'll start on 96 and I'll provide discussions in the discord concerning my thoughts and provide also the experimental versions as before.
I guess I can create a 96 which will suit the community, as the community will be involved.


ionic 2.1.18 does not install properly in Fedora 25

From @tonydiep on January 6, 2017 15:9
Ionic version:  (check one with "x")
[ ] 1.x
[X] 2.x
I'm submitting a ...  (check one with "x")
[X] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Ionic 2.1.18 upgrade or fresh install fails on Fedora 25 leaving ionic unuseable.
I suspect it's related to npm config set prefix assuming /usr vs /usr/local

Errors start with
npm ERR! enoent ENOENT: no such file or directory, rename '/usr/lib/node_modules/.staging/ansi-79f49b0e' -> '/usr/lib/node_modules/ionic/node_modules/cordova-lib/node_modules/ansi'
n

It seems install package is assumes /usr/lib/node_modules is where node_modules are kept
using npm config set prefix /usr/local works BUT other errors crop up as the rest of the node_modules are elsewhere
Symbolic link from /usr/lib/node_modules to /usr/node_modules does not work

Expected behavior:

npm install -g ionic or npm upgrade -g ionic works
Steps to reproduce:

Related code:
insert any relevant code here

Other information:

workaround is to install<EMAIL_ADDRESS>
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
insert the output from ionic info here

Copied from original issue: driftyco/ionic#9900

Experiencing the same issue on macOS

After double checking, uninstalling them again resolved the issue
$ npm -g remove ionic cordova
$ npm -g install ionic cordova

I found a workaround here: https://docs.npmjs.com/getting-started/fixing-npm-permissions
Using option 2 in the link, I moved global npm modules to my home directory.

In redhat and centos linux, npm config get prefix showed '/usr'
I wonder if the installation is assuming /usr/local?


Retain property values on entities after deletion

Is there a built-in mechanism for cloning entities? I can't seem to find one, nor can I find docs on this.
Use case: I have an entity that I'm about to delete, but afterwards I need to notify observers that the entity has been deleted. Deleting the entity is resetting dates to DateTime.MinValue, so I can't just use the same entity instance that is passed into DeleteObject. Perhaps this is a bug? I'm using <IP_ADDRESS>.
Does adding cloning support make sense? If so, could it just be generated code too?

I'm not sure about adding cloning support - it would mean pushing a requirement on to developers of partial class extensions to ensure that they also have cloning support in there. I wonder if there is a different way of handling your use case though as I can see how it would be a useful thing to have.

Good point about entity extensibility. The way I dealt with this is to change my entity->contract extension method helpers to produce immutable values rather than simply wrapping the entity and exposing it via the contract interface. This entails deep cloning, but it's done at a natural point in the code base. Not only does this solve the problem, but I think it's just a better design anyway because now all data from my services is immutable.
Tying this back to the use case I mentioned, I simply call ToContract() on the entity before I delete it. Thus, I have a deep clone of it that conforms to my contract which I can then use to notify listeners after a successful deletion.
Feel free to close this if you like.

I'm going to leave it open as a reminder that it would be good to have some story on post-delete property accesses - it seems reasonable to expect to be able to access the properties of an entity after it has been deleted from the store for as long as you are holding on to the entity handle.

Going to bump this out of the 1.11 release as it might be better to think about it in the context of adding Deleted / Deleting delegates to the BrightstarEntityContext.


Consumer release resources has a problem.(Consumer.Close() or Dispose())

Description
When release the consumer's resources,I try to use Consumer.Close() or Dispose().But this method conflict with Consumer.Consume() method.And this error can not capture.I need your help.The follow is my code.
1.Build a consumer
var consumerConfig = new ConsumerConfig
{
GroupId = Guid.NewGuid().ToString(),
BootstrapServers = context.CurrentOptions.Addresses,
EnableAutoCommit = false,
StatisticsIntervalMs = 5000,
SessionTimeoutMs = 6000,
AutoOffsetReset = AutoOffsetReset.Earliest,
EnablePartitionEof = true
};
_consumer = new ConsumerBuilder<Ignore, string>(consumerConfig).Build();
2.Start listening and receiving messages.
private void StartListen()
{
const int commitPeriod = 5;
Task.Run(() =>
{
while (!_cts.IsCancellationRequested)
{
try
{
var consumeResult = _consumer.Consume(_cts.Token);
if (null != consumeResult)
{
if (0 == consumeResult.Offset % commitPeriod || consumeResult.IsPartitionEOF)
{
_consumer.Commit(consumeResult);
}
if (null != consumeResult.Message)
_context.CurrentMessageManager.EnqueueMessage(MessageInfo.Create(consumeResult.Value, consumeResult.Topic));
}
}
catch (Exception ex)
{
//写日志
}
}
}).ConfigureAwait(false);
}
3.Release resources,but conflict with step2 var consumeResult =_consumer.Consume(_cts.Token);
this error for memory.
public void Dispose()
{
if (null != _cts)
{
_cts.Cancel();
_cts.Dispose();
_cts = null;
}
if (null != _consumer)
{
_consumer.Unsubscribe();
_consumer.Close();
//_consumer.Dispose();
_consumer = null;
}
    }

How to reproduce
Checklist
Please provide the following information:

[x] Confluent.Kafka nuget version: 1.0.0-RC1
[x] Apache Kafka version: kafka_2.11-2.1.0
[ ] Client configuration:
[x] Operating system:windows10
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

wait for the consume loop to finish before disposing the consumer.
if you're having problems still, please provide a complete working minimal example and i'll update it.

I do not know when the consume loop is finish.There is a someone uncomfortable solution in which I set a thread await just like Thread.Sleep(CancellationDelayMaxMs) before disposing the consumer.
step 1.
var consumerConfig = new ConsumerConfig
{
GroupId = Guid.NewGuid().ToString("D"),
BootstrapServers = "brokerlist",
EnableAutoCommit = false,
StatisticsIntervalMs = 5000,
SessionTimeoutMs = 6000,
AutoOffsetReset = AutoOffsetReset.Latest,
EnablePartitionEof = true,
CancellationDelayMaxMs = 1//set CancellationDelayMaxMs
};
_consumer = new ConsumerBuilder<Ignore, string>(consumerConfig).Build();
step 2. disposing the consumer
Thread.Sleep(1);
_consumer.Close();
In fact,I do not wanna to sleep the thread.If there is any better way, please give a reply.Thanks.


Elo support

Any idea of having support for ELO Credit Card?

I’ll look into it for v1.2

@PawelDecowski If need any help, let me know. :)

@ceasbz Do you have any sources that provide IIN ranges? I can’t find anything reliable.

@PawelDecowski Yes, see here: https://github.com/jessepollak/payment/issues/27 this table are updated with the last version :)

This is just a comment by you on another project. Not that I don’t trust you ;) but where did you get the list from?

@PawelDecowski Here in our company, we use the ELO Payment. And every update, I receive a table with the current IIN ranges. :)

OK. As long as they don’t collide with other cards, I’ll add them.

I am searching where is the documentation about that.
Em qua, 19 de out de 2016 às 12:25, Pawel Decowski<EMAIL_ADDRESS>escreveu:

OK. As long as they don’t collide with other cards, I’ll add them.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PawelDecowski/jquery-creditcardvalidator/issues/82#issuecomment-254828547,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGvOxbUE6eQTFIFFNkNBvR8Lys09weYYks5q1ig_gaJpZM4I6_lf
.

@PawelDecowski I am searching where is the documentation about that. :)

Are ELO card numbers Luhn-validated? According to this, they’re not.

Please, add support do ELO! <3

Hey @PawelDecowski sorry for delay.
I don't remember about that, do you have any progress?

@ceasbz If you still need this, follow the solution just below. Valeuuuuu :)
....
,{
name: 'aura',
range: '5078',
valid_length: [19]
}, {
name: 'elo',
range: '4011, 401178, 401179, 431274, 438935, 451416, 457393, 4576, 457631, 457632, ' +
'504175,<PHONE_NUMBER>78,<PHONE_NUMBER>99, ' +
'627780, 636297, 636368, 636369, ' +
'650031-650033,<PHONE_NUMBER>51,<PHONE_NUMBER>39,<PHONE_NUMBER>38,<PHONE_NUMBER>98,<PHONE_NUMBER>18,<PHONE_NUMBER>27,<PHONE_NUMBER>20,<PHONE_NUMBER>79,<PHONE_NUMBER>19,<PHONE_NUMBER>58',
valid_length: [16]
}, {  // aura and elo must inserted above visa_electron
name: 'visa_electron',
range: '4026, 417500, 4508, 4844, 4913, 4917',
valid_length: [16]
},
...

Ty @mateuspadua

:)

@mateuspadua Como vc alterou esse json? Obrigado.

I’m hesitant to add this as these ranges collide with Maestro and Visa. I’m closing it until someone can show a reliable source of IINs.


Multiline code breaks up arrow history functionality

Example:
scm> (+ 1 2)
3
scm> (
)
()
scm> # up arrow takes me to the multi line ( )s but I can't get to the first expression anymore

Hold Ctrl to scroll through multi-line history items.
Also, RTFM (and by manual I mean the "Keyboard Shortcuts" section of the message that's displayed when you open the interpreter)

It was only added in the rewrite last semester. The old interpreter stored each line as a separate history entry.

+1
Literally just had this same conversation with Jen IRL 😂

Though actually - can there be a line mentioning that it's command for Mac users?

I think it should be Ctrl regardless of platform (neither the Dart docs nor the web api docs seem to indicate the property I'm using being dependent on platform)


mobile search not working anymore

you should fix this thank you in advance

Same problem.

Same problem.

did u fix it?

Same problem.

remove line 86 mobile search works for me now
checkBingLogin(browser, isMobile)

Same problem.

remove line 86
checkBingLogin(browser, isMobile)

Same problem.

remove line 86
checkBingLogin(browser, isMobile)

Same problem.

DO NOT COMMENT OUT LINE 86 if you do POINTS_COUNTER will not work properly.
Instead try this:
-How to Fix Mobile:
Comment out around line 134-138 with double quotes above and below
""
#Refresh page
browser.get('https://bing.com/')
#Wait 5 seconds
time.sleep(10)
#Update Counter
""
-How to run Script:

Same problem.

remove line 86
checkBingLogin(browser, isMobile)

DO NOT COMMENT OUT LINE 86 if you do POINTS_COUNTER will not work properly.
Instead try this: -How to Fix Mobile: Comment out around line 134-137 with three double quotes above and below """ #Refresh page browser.get('https://bing.com/') #Wait 5 seconds time.sleep(10) """

Hi, do you have any problem with mobile searches? Im using your fork but in headless mode it will loop in the first search. But when I make it with the chrome tab open i can make it work closing the bing full screen cookies notice. Maybe thats the issue in headless mode and you could check it to fix it, thanks.

There is an issue sometimes with mobile and desktop searches. I put a re-try function for both. If the re-try does not work I restart the script and it seems to work.
Il'll look into it more.

Found the issue. Wehn trying to complete mobile searches the user was not fully logged in.
Code has been updated.
If you would like to check it out my github link is: https://github.com/AlexyGalaxy/Microsoft-Rewards-Farmer

Found the issue. Wehn trying to complete mobile searches the user was not fully logged in.
Code has been updated.
If you would like to check it out my github link is: https://github.com/AlexyGalaxy/Microsoft-Rewards-Farmer

Perfect! I'll try it today, thanks for your fork :)

Fixed with v3.0 #188


Data is shown as gb

Describe the bug
Data in the details page is shown as gb.
To Reproduce
Navigate to the details page
Expected behavior
The units should be coherent:
GB (international units)
GiB (digital information units)
Additional context
Add any other context about the problem here.

Looks like a duplicate of #347 and also does not provide much information. Please place all the bugs content in a single bug with enough information to clearly understand the issue. Screenshots are helpful.
Please use terminology of the application "details page" is unclear. Is the AWS or Openshift related.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 856

Questions ?
Please refer to the Backport tool documentation


CompletionHandler image not loaded yet.

Hi, I'm trying to use your great api but in order to do so I need the completionHandler to only be called after the image was already set to the UIImageView inside a CollectionView item so that I can resize my cell. How do you recommend me to do so?

The problem was on my end. Great api, sorry for that.


Make StaticClusterManager handle Replica_Unavailable server error

support handling Replica_Unavailable and Replica_Available in
StaticClusterManager
introduce StateResourcePolicy into Replica
add tests in StaticClusterManagerTest for verification

Codecov Report

Merging #1082 into master will decrease coverage by <.01%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##             master    #1082      +/-   ##
============================================
- Coverage     68.59%   68.59%   -0.01%
+ Complexity     4828     4827       -1
============================================
  Files           382      382
  Lines         30512    30535      +23
  Branches       3890     3890
============================================
+ Hits          20929    20944      +15
- Misses         8529     8535       +6
- Partials       1054     1056       +2

Impacted Files
Coverage Δ
Complexity Δ

.../com.github.ambry/clustermap/PartitionManager.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...a/com.github.ambry.clustermap/PartitionLayout.java
71.75% <75%> (ø)
35 <0> (ø)
:arrow_down:

...main/java/com.github.ambry.clustermap/Replica.java
71.83% <77.27%> (+4.52%)
21 <2> (+3)
:arrow_up:

....github.ambry.clustermap/StaticClusterManager.java
77.33% <83.33%> (+0.41%)
83 <1> (+2)
:arrow_up:

...in/java/com.github.ambry.clustermap/Partition.java
84.81% <0%> (-3.8%)
25% <0%> (-1%)

...rc/main/java/com.github.ambry.store/BlobStore.java
83.88% <0%> (-1.54%)
76% <0%> (-2%)

.../com.github.ambry.router/GetBlobInfoOperation.java
82.38% <0%> (-1.14%)
38% <0%> (-1%)

...b.ambry.network/BlockingChannelConnectionPool.java
62.6% <0%> (-0.87%)
7% <0%> (-1%)

...in/java/com.github.ambry.network/SocketServer.java
82.78% <0%> (-0.41%)
13% <0%> (ø)

...ain/java/com.github.ambry.router/PutOperation.java
89.61% <0%> (-0.2%)
80% <0%> (-1%)

... and 3 more

Continue to review full report at Codecov.

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

addressed comments from @vgkholla .
./gradlew build && ./gradlew test succeeded.


chore(formatting): format dune files in esy format

The inconsistent formatting of some of the dune files was slightly bugging me. I discovered dune format-dune-file, and thought it would be a good idea to add it to our formatting script.

dune 2 already format dune file so might not be needed to roll our script.

@Et7f3 very true, but didn't know what the plans were for upgrading to dune 2. This can be removed if/when we do that.


Show SaveTokens in OAuth setup and then later retrieval

There should be a brief sample showing SaveTokens = true and then showing token retrieval in a controller (e.g. to make an API call).
https://github.com/aspnet/Security/issues/970
https://docs.asp.net/en/latest/security/authentication/sociallogins.html
Storage:
    app.UseFacebookAuthentication(new FacebookOptions()
    {
        AppId = Configuration["Authentication:Facebook:AppId"],
        AppSecret = Configuration["Authentication:Facebook:AppSecret"],
        SaveTokens = true,
    });

@HaoK do you have a retrieval sample?

Extraction would be as simple as:
   context.Authentication.GetTokenAsync("Facebook", "<tokenName>"

@HaoK And the Identity version?

With identity:
Basically after login, to update the tokens:
            // Sign in the user with this external login provider if the user already has a login.
            var result = await _signInManager.ExternalLoginSignInAsync(info.LoginProvider, info.ProviderKey, isPersistent: false);
            if (result.Succeeded)
            {
                // Update any authentication tokens if login succeeded
                await _signInManager.UpdateExternalAuthenticationTokensAsync(info);

To retrieve:
  await userManager.GetAuthenticationTokenAsync(user, "Facebook", "<tokenName>")


Refactor _done of Watching in compiler

based off of #3933 but could be made independent.
What kind of change does this PR introduce?
refactoring
Did you add tests for your changes?
No
If relevant, link to documentation update:
N/A
Summary
Simplify the _done method of Watching in Compiler
Does this PR introduce a breaking change?
No

happy to give the handler a Stats Object and just mark a todo to maybe remove that in a later version?
That way we can get rid off this PR :).

Thanks


Tests don't work

Hi there,
I stumbled across this repo after looking for a RISCV toolchain for Bazel.  It looks like you've made some changes to directory hierarchy that haven't been updated in the tests.  The diff below fixes those:
$ git diff
diff --git a/tests/lib/sieve.c b/tests/lib/sieve.c
index 8a1e7a7..8d173d7 100644
--- a/tests/lib/sieve.c
+++ b/tests/lib/sieve.c
@@ -1,5 +1,5 @@
 #include <stdint.h>
-#include "lib/sieve.h"
+#include "tests/lib/sieve.h"

 uint8_t primes[SIEVE_SZ];

diff --git a/tests/riscv32/BUILD.bazel b/tests/riscv32/BUILD.bazel
index e364006..ffc02ef 100644
--- a/tests/riscv32/BUILD.bazel
+++ b/tests/riscv32/BUILD.bazel
@@ -19,7 +19,7 @@ cc_binary(
     ],
     deps = [
         "memmap.ld",
-        "//lib:sieve",
+        "//tests/lib:sieve",
     ],
 )

diff --git a/tests/riscv32/notmain.c b/tests/riscv32/notmain.c
index 3c1f5bd..af678ba 100644
--- a/tests/riscv32/notmain.c
+++ b/tests/riscv32/notmain.c
@@ -1,5 +1,5 @@
 #include <stdarg.h>
-#include "lib/sieve.h"
+#include "tests/lib/sieve.h"

 extern int semihost_call(int, int);

However, having fixed those I'm still struggling to get the examples to compile.  Do I need to manually specify the toolchain on the Bazel command line?  I'm running $ bazel build --platforms=//platforms/riscv32:opentitan //tests/riscv32:sieve and it's trying to use my system installed GCC.

Ergh, there's a workspace within a workspace.  Works when you run it from the correct directory.  Sorry for the spam...


[docs] Revise README.md

Basically we don't need the return inside the switch statement, it will jump us out of the circuit.

@googlebot I signed it!

Thank you, @Mudassar045!


Added bitwise operations to Math

The math codeunit helps a great deal with mathematical operations, but the one thing that Dynamics has always lacked is bitwise operations that are sometimes used in e.g. communication and validation with external systems.

Added a test codeunit; not sure whether you wanted me to add an app.json for that. Secondly, I could not determine a codeunit number for the test, since I am not sure which logic is behind that.

@PeterConijn that's great effort from your side and we are grateful to have contributors like you!
However, for this particular functionality it is wise to wait for it to be available in the language as the current solution isn't performant and does not cover all the cases.
That said, it's also crucial to acknowledge that we cannot share any details on when bitwise operations will be available in the AL language. What we are also willing to do in the meantime, however, is to expose bitwise operations in some of the .net libraries we use and then expose these in System Application via .net calls. How does that sound?
Of course, if you feel too strong about your contribution, we will reconsider it :)

Hi Maria,
That sounds like a wonderful idea and will solve a lot of challenges. Thank you!
Peter Conijn
Application Developer
t: +31742051199 | m: +31627332165
e: @.*** | w: www.dysel.com
From: Maria Zhelezova @.>
Sent: Wednesday, 17 August 2022 22:11
To: microsoft/ALAppExtensions @.>
Cc: Peter Conijn @.>; Mention @.>
Subject: Re: [microsoft/ALAppExtensions] Added bitwise operations to Math (PR #19505)
Alert: This is an external email.
@PeterConijnhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPeterConijn&data=05|01|p.conijn%40dysel.com|2a28e534181f4796d3a208da808ca1af|99b5255df73d44ff8fed0bf580b21df3|0|0|637963638746849701|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=bk1vBRRH%2BN8ScR%2FcyXHHP%2B6M1vTZj0sfTmKMtuzkKzY%3D&reserved=0 that's great effort from your side and we are grateful to have contributors like you!
However, for this particular functionality it is wise to wait for it to be available in the language as the current solution isn't performant and does not cover all the cases.
That said, it's also crucial to acknowledge that we cannot share any details on when bitwise operations will be available in the AL language. What we are also willing to do in the meantime, however, is to expose bitwise operations in some of the .net libraries we use and then expose these in System Application via .net calls. How does that sound?
Of course, if you feel too strong about your contribution, we will reconsider it :)
—
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FALAppExtensions%2Fpull%2F19505%23issuecomment-1218445537&data=05|01|p.conijn%40dysel.com|2a28e534181f4796d3a208da808ca1af|99b5255df73d44ff8fed0bf580b21df3|0|0|637963638746849701|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=umm1ivPNvlg3MM%2FnbPFXQC8BybbRBaGOq40m1t7%2Fdj4%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALI7UOXBN2JPIMBLT2AK2XDVZVBN5ANCNFSM547XHIMA&data=05|01|p.conijn%40dysel.com|2a28e534181f4796d3a208da808ca1af|99b5255df73d44ff8fed0bf580b21df3|0|0|637963638746849701|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=sjn7uOa7WY26GVc6G6DUHQvg9jez5krQ71tLwwl8XtM%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Hi @PeterConijn, thanks for your contribution!
I just have some concerns about the performance of the current implementation (as well as potentially unhandled edge cases).
Would you be open to change the implementation to just wrap the existing C# functionality?
Here is an example:
  procedure BitwiseOr(Left: Integer; Right: Integer): Integer
  var
      Result: DotNet BigInteger;
      DotNetLeft: DotNet BigInteger;
      DotNetRight: DotNet BigInteger;
      ResultInt: Integer;
  begin
      DotNetLeft := DotNetLeft.BigInteger(Left);
      DotNetRight := DotNetLeft.BigInteger(Right);
      Result := Result.op_BitwiseOr(DotNetLeft, DotNetRight);
      Evaluate(ResultInt, Result.ToString());
      exit(ResultInt);
  end


Plus/Minus symbol should be easier to see

Removes the color:white for the + and - symbols on the sidebar
Before:

After:

I like it! Merged.


Reimplement python embedding dataplane logic using the Python3_Script class and enhance the logging of Python Errors.

Python embedding is done in 3 spots in MET:
(1) vx_data2d_python library for gridded data
(2) ascii2nc for point obs
(3) stat_analysis for MPR data
(2) and (3) use the Python3_Script class, while (1), implemented first, does not.  This makes the implementation details a bit confusing and more difficult to maintain.  This task is to refactor (1) to also use the Python3_Script class.
In addition, enhance the Python3_Script class to check when an error occurred.  Call PyErr_Occurred() and PyErr_Fetch() to retrieve the PyObjects containing the error details.  Print the results to the mlog output so that the error messages are included in the log files.

I don't know if it is related, but (2) prevents ascii2nc to compile: https://github.com/NCAR/MET/commit/20fcf4f31bffb5b9393f93de7bf00af7785d9122#r37494338


Fix route requirements bug

Fix spec in #1787
This was a bug introduced by commit 9f4ba67.
The commit replaces options[:route_options].clone.merge(...) with options[:route_options].clone.reverse_merge(...).
That causes disappear of the requirements in namespace.

Amazing, merged.

Thanks @darren987469! Looks great.


Partition large disks

This PR correctly partitions disks larger than<PHONE_NUMBER>040 bytes (~2199 GB).
The XXLarge 4095GB (4 TiB) disks cannot use DOS partition table format and
instead must use Guid Partition table format (GPT). parted needs to be used
instead of fdisk to achieve this.

need to run npm run links -- --branch master before merging


Don't look up master address in DNS, use hostname directly

@bparees

LGTM

Fixed. We now use a separate environment variable for master service name, called '$MYSQL_MASTER_SERVICE_NAME'. PTAL

lgtm.
[test]

[test]


Add test for unknown recovery transaction attempt

Changes:

Adds TestEmailApiModule, which provides test doubles for testing EmailApiModule dependent modules.
Adds a unit test to AlertsController tests to verify a notification is sent to the verified signer emails when an unknown recovery transaction is attempted.

I am going to merge this directly as it is purely test-focused and a prerequisite to test https://github.com/safe-global/safe-client-gateway/pull/922.


Rename os-ansible-deployment to openstack-ansible

Per https://review.openstack.org/#/c/200730/, the stackforge/os-ansible-deployment project will be moved to openstack/openstack-ansible.  This is scheduled to happen on September 11, 2015. (See Andreas Jager's comment at Aug 27 2:04 PM in that same review).
This is a priority issue, since the installations will be broken until this addressed once the change happens.
This will affect the following files and directories in addition to the os-ansible-deployment submodule directory, per an ack check.
.gitmodules:1:[submodule "os-ansible-deployment"]
.gitmodules:2:	path = os-ansible-deployment
.gitmodules:3:	url = https://git.openstack.org/stackforge/os-ansible-deployment
README.md:4:# os-ansible-deployment integration
README.md:8:[os-ansible-deployment](https://github.com/stackforge/os-ansible-deployment)
README.md:20:[os-ansible-deployment](http://git.openstack.org/cgit/stackforge/os-ansible-deployment/tree/?h=juno)
README.md:27:The Kilo branch of os-ansible-deployment does not include any rpc-maas
README.md:43:run and then rerun the appropriate plays in os-ansible-deployment after this
README.md:67:2. Unless doing an AIO build, prepare the os-ansible-deployment configuration.
README.md:69:     `cp -R os-ansible-deployment/etc/openstack_deploy /etc/openstack_deploy`
README.md:96:To run an upgrade of an existing os-ansible-deployment installation:
rpcd/etc/openstack_deploy/user_extras_variables.yml:37:rpc_repo_path: /opt/rpc-openstack/os-ansible-deployment
rpcd/etc/openstack_deploy/user_extras_variables.yml:88:# This relies on the apt_package_pinning role from os-ansible-deployment.
rpcd/playbooks/ansible.cfg:13:# Set the path to the folder in os-ansible-deployment which holds the dynamic
rpcd/playbooks/ansible.cfg:15:inventory = ../../os-ansible-deployment/playbooks/inventory/
rpcd/playbooks/ansible.cfg:17:# Set the path to the folder in os-ansible-deployment which holds the dynamic
rpcd/playbooks/ansible.cfg:19:#hostfile = ../../os-ansible-deployment/playbooks/inventory/
rpcd/playbooks/ansible.cfg:21:# Set the path to the folder in os-ansible-deployment which holds the
rpcd/playbooks/ansible.cfg:23:library = ../../os-ansible-deployment/playbooks/library/
rpcd/playbooks/ansible.cfg:25:# Set the path to the folder in os-ansible-deployment which holds the roles
rpcd/playbooks/ansible.cfg:27:roles_path = ../../os-ansible-deployment/playbooks/roles/
rpcd/playbooks/ansible.cfg:29:# Set the path to the folder in os-ansible-deployment which holds the
rpcd/playbooks/ansible.cfg:31:lookup_plugins = ../../os-ansible-deployment/playbooks/plugins/lookups/
rpcd/playbooks/group_vars/all.yml:16:rpc_repo_path: /opt/rpc-openstack/os-ansible-deployment
rpcd/playbooks/roles/patcher/defaults/main.yml:17:patcher_dest_dir: "/opt/rpc-openstack/os-ansible-deployment"
rpcd/playbooks/roles/rpc_maas/defaults/main.yml:246:# maas_pip_packages: Packages we need but are not built by os-ansible-deployment
rpcd/playbooks/roles/rpc_maas/defaults/main.yml:254:#                        os-ansible-deployment
scripts/deploy.sh:16:source /opt/rpc-openstack/os-ansible-deployment/scripts/scripts-library.sh
scripts/deploy.sh:17:OSAD_DIR='/opt/rpc-openstack/os-ansible-deployment'
scripts/linting.sh:29:if [[ -f "os-ansible-deployment/dev-requirements.txt" ]]; then
scripts/linting.sh:30:  pip2 install -r os-ansible-deployment/dev-requirements.txt || pip install -r os-ansible-deployment/dev-requirements.txt
scripts/upgrade.sh:20:OSAD_DIR="$BASE_DIR/os-ansible-deployment"
scripts/upgrade.sh:37:# Do the upgrade for os-ansible-deployment components
tox.ini:7:exclude=os-ansible-deployment/*,f5-config.py

Closing as all PR's have merged and the string os-ansible-deployment no longer appears in the tree.


Add first paragraph from Readme

Wondering if adding the first paragraph from the the modules page would be good to give some more information on the Tweets

Thanks for bring this up.
Actually, I like the current short message.

Yes, it could be overkill. Just a thought.


DeadlineExceededException in acquireCameraImage()

I used the following code in the AugmentedFace example to access the CPU image. It worked fine before Aug 1st but started to throw "DeadlineExceededException" 2 seconds after running the app.
Did I change something wired or was there an update from ArCore? What is the appropriate way to call acquireCameraImage() in Sceneform? Thanks!
Frame frame = arFragment.getArSceneView().getArFrame();
if (frame == null) {
      return "Frame is null";
}

synchronized (frameImageInUseLock) {
      try (Image image = frame.acquireCameraImage()) {
        if (image.getFormat() != ImageFormat.YUV_420_888) {
          throw new IllegalArgumentException(
              "Expected image in YUV_420_888 format, got format " + image.getFormat());
        }
        //do something with image...
      } catch (NotYetAvailableException e) {
        Log.e(TAG, "NotYetAvailableException", e);
      } catch (DeadlineExceededException e){
        Log.e(TAG, "DeadlineExceededException", e);
      } catch (Exception e) {
        Log.e(TAG, "Image copy exception", e);
      }
    }

For those faces the same issue, it may be wise to access the camera image via GPU texture instead.

hello michael, I am new in ARCore. Will you please provide some reference for accessing camera image, as I am facing the same issue. I am not aware of GPU textures. Thanks in advance.

Hi, maybe you could get some useful information from the computervision_java example in the arcore-android-sdk repository. I've copied a few files from that project to retrieve the GPU image.


Create "Folder references" instead of "Groups" to organize source files

At least optionally. At least for files that we add into the project with "Include files" command explicitly. My use case is that I keep everything synchronized in Finder and Xcode anyway, so if I regenerate project - it will recreate the needed project structure again. Creating "groups" instead of Folder references does not make sense - just creates extra work for developers.

@maximkhatskevich I could look into exposing a property on the project what do you think ?
project.added_folders_type = :folder_reference

@jcampbell05 yes, something like this is exactly what I would need.
Maybe something like
project.add_folders_as = :reference # alternative/default value is ":groups"

@jcampbell05 probably this is not needed, for now, because it produces a strange issue in Xcode 7.3.* http://stackoverflow.com/a/36429157


Unbreak build with custom location of libyajl.so

... with that patch https://github.com/SpiderLabs/ModSecurity/commit/c9293993270bd177aed69e8001bd2964d93481fc

merged ;) thanks.


az resource update interprets string value as mathematical expression

Describe the bug
When using az resource update to set tag values, any tag value that appears to be a mathematical expression is interpreted as one, and the result of the expression is stored as the tag value.
For instance, I am trying to set a tag to a date value formatted as "2020-11-16". The string is interpreted as a subtraction, and the tag's value is set to "1993".
Is this the intended behavior? I haven't found any documentation that indicates that it is, and this doesn't happen when using az resource tag.
To Reproduce

Use az resource tag to create a new tag with a value of "10-1" on any existing resource (az resource tag --tags tagName="10-1" -g resourceGroupName -n resourceName --resource-type resourceType --query "tags", with the correct parameters substituted)
Observe that the tag is correctly set to "10-1"
Update the tag's value using az resource update, still with the same value of "10-1" (az resource update --set tags.tagName="10-1" -g resourceGroupName -n resourceName --resource-type resourceType --query "tags", with the same parameters substituted)
The tag's value will be set to "9" instead of "10-1"

Expected behavior
The tag's value is set to the exact string value provided ("10-1" for the above example), or this behavior is noted in the online docs.
Environment summary
CLI version: 2.14.2 (installed with apt-get)
OS version: Windows 10 build 19042.630
Shell type: Bash on windows (WSL 1)

ARM


the blog link not working

hello , crazyhottommy
I want to visit your blog, but the links are not working now. Do you change it ?
Links : http://crazyhottommy.blogspot.jp/search/label/CpG
http://crazyhottommy.blogspot.jp/search/label/MeDIP-seq
FF

Hi,
it is working for me.
try these http://crazyhottommy.blogspot.com/2013/12/tet1-enrichment-at-cpg-rich-promoters.html
http://crazyhottommy.blogspot.com/2014/01/medip-seq-and-histone-modification-chip.html
Tommy


gulp build-docs error

Have created 2 adjacent sibling folders UI and Docs
UI folder has been populated with Semantic-UI-Docs

However , when I try to run "gulp build-docs" inside the UI folder , I am getting an error message :
"gulp file not found" . Any suggestions on how to rectify this ?

UI folder should be clone of UI repo, not npm install


Add possibility for sending data to class A device without queuing packet

In some cases it is necessary to be able to send a packet to the device at the current receiving an uplink packet from the device.
If the server did not send the packet, server must not put the packet to the queue and send it next time, because the data is no longer relevant.
This feature is needed, for example, for synchronization time on class A devices.

The reason why this is not possible is because LoRa Server handles the sending of the application payload async. E.g. when LoRa App Server takes a long time to respond, LoRa Server is still able to respond to the device in the Class-A receive window(s). The delay is configurable in loraserver.toml.

This feature is needed, for example, for synchronization time on class A devices.

I suggest using the DeviceTime mac-command for this (you need GPS time-synced gateways). This is the most accurate way to sync the device time and is supported by LoRa Server.

882 5.9 DeviceTime commands (DeviceTimeReq, DeviceTimeAns)
883 This MAC command is only available if the device is activated on a LoRaWAN1.0.3 or later
884 compatible Network Server. LoRaWAN1.0.2 or earlier servers do not implement this MAC
885 command.
886 With the DeviceTimeReq command, an end-device MAY request from the network the current
887 network time. This allows the device to synchronize its internal clock to the network’s clock.


Remove references to RapidSMS scheduler and LoggerMixin

The former was removed, the latter is deprecated. This is based on top of #17; until this is merged the diff is best viewed here: https://github.com/caktus/rapidsms-decisiontree-app/compare/multitenancy...no-scheduler-or-logger

Made against the wrong branch.


[Feature Request]: human_readable_id for text_units

Do you need to file an issue?

[X] I have searched the existing issues and this feature is not already filed.
[X] My model is hosted on OpenAI or Azure. If not, please look at the "model providers" issue and don't file a new one here.
[X] I believe this is a legitimate feature request, not just a question. If this is a question, please use the Discussions area.

Is your feature request related to a problem? Please describe.
I noticed that in the \query\structured_search\local_search\system_prompt.py, there is a prompt to cite the reference:
"Person X is the owner of Company Y and subject to many allegations of wrongdoing [Data: Sources (15, 16), Reports (1), Entities (5, 7); Relationships (23); Claims (2, 7, 34, 46, 64, +more)]."

where 15, 16, 1, 5, 7, 23, 2, 7, 34, 46, and 64 represent the id (not the index) of the relevant data record.

However, if we examine create_final_text_units.parquet, there is no human_readable_id for us the to query like
text_units_df = pd.read_parquet("create_final_text_units.parquet")
text_units_df  = text_units_df ["human_readable_id" = source_id]

which is relatively unintuitive as it's clearly stated that the id (15,16) represent the id (not the index). Moreover, for entities, relationship, claims, there is human_readable_id field for querying. For reports, there is community field that can serve as human_readable_id.
Describe the solution you'd like
Add a human_readable_id for text_units for text_units dataframe.
Additional context
No response

Agree that this seems confusing. The human-readable ID for text_units is assigned from the index when we load the data for query. Text units were likely added to the query context after the others and hadn't had an incremental ID assigned during indexing so we used the default. Will add to backlog so this is more clear.


Update Suitcase documentation to reflect GSoC changes.

https://docs.odk-x.org/suitcase-intro/#using-odk-x-suitcase
Changes include :

updated Screenshots to cover the changes in the UI
document changes in GUI
any other changes

@linl33 please assign this issue to me.


Enable status cake alerting

Context
We currently don't have status cake alerting set up for AYTQ or Check.
Changes proposed in this pull request

Add an alert for each service
Set the terraform version in .tool-versions to match the terraform config
Update the status cake alert group. The previous setting was an old group related to DQT.

Guidance to review
I ran a plan with this change added to the preprod vars and got this output.
 statuscake_uptime_check.alert["access_your_teaching_qualifications_prod"] will be created
  + resource "statuscake_uptime_check" "alert" {
      + check_interval = 30
      + confirmation   = 2
      + contact_groups = [
          + "288912",
        ]
      + id             = (known after apply)
      + locations      = (known after apply)
      + name           = "access-your-teaching-qualifications.education.gov.uk"
      + paused         = false
      + regions        = [
          + "london",
          + "dublin",
        ]
      + trigger_rate   = 0

      + http_check {
          + enable_cookies   = false
          + follow_redirects = true
          + request_method   = "HTTP"
          + status_codes     = [
              + "204",
              + "205",
              + "206",
              + "303",
              + "400",
              + "401",
              + "403",
              + "404",
              + "405",
              + "406",
              + "408",
              + "410",
              + "413",
              + "429",
              + "444",
              + "494",
              + "495",
              + "496",
              + "499",
              + "500",
              + "501",
              + "502",
              + "503",
              + "504",
              + "505",
              + "506",
              + "507",
              + "508",
              + "509",
              + "510",
              + "511",
              + "520",
              + "521",
              + "522",
              + "523",
              + "524",
              + "598",
              + "599",
            ]
          + timeout          = 40
          + validate_ssl     = false
        }

      + monitored_resource {
          + address = "https://access-your-teaching-qualifications.education.gov.uk/health"
        }
    }

  # statuscake_uptime_check.alert["check_a_teachers_record_prod"] will be created
  + resource "statuscake_uptime_check" "alert" {
      + check_interval = 30
      + confirmation   = 2
      + contact_groups = [
          + "288912",
        ]
      + id             = (known after apply)
      + locations      = (known after apply)
      + name           = "check-a-teachers-record.education.gov.uk"
      + paused         = false
      + regions        = [
          + "london",
          + "dublin",
        ]
      + trigger_rate   = 0

      + http_check {
          + enable_cookies   = false
          + follow_redirects = true
          + request_method   = "HTTP"
          + status_codes     = [
              + "204",
              + "205",
              + "206",
              + "303",
              + "400",
              + "401",
              + "403",
              + "404",
              + "405",
              + "406",
              + "408",
              + "410",
              + "413",
              + "429",
              + "444",
              + "494",
              + "495",
              + "496",
              + "499",
              + "500",
              + "501",
              + "502",
              + "503",
              + "504",
              + "505",
              + "506",
              + "507",
              + "508",
              + "509",
              + "510",
              + "511",
              + "520",
              + "521",
              + "522",
              + "523",
              + "524",
              + "598",
              + "599",
            ]
          + timeout          = 40
          + validate_ssl     = false
        }

      + monitored_resource {
          + address = "check-a-teachers-record.education.gov.uk/health"
        }
    }

Link to Trello card

Checklist

[x] Attach to Trello card
[x] Rebased main
[x] Cleaned commit history
[x] Tested by running locally (ish)

 Review and update status cake alerting for all services


remove maxFiles

STOP! If your issue is with URI rewriting, read the wiki first:
http://code.google.com/p/minify/wiki/CommonProblems#URIs_are_re-
written_incorrectly_in_CSS_output
Please remove this message from the issue.

Minify version:
PHP version:

What steps will reproduce the problem?
1.Change the config.php with $min_serveOptions['minApp']['maxFiles'] = 60;
2.11 files in f=params
3.400 not work

Expected output:

Actual output:

Did any unit tests FAIL? (Please do not post the full list)

Please provide any additional information below. If this has to do with URI
rewriting, please include your DOCUMENT_ROOT, the

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 29 Mar 2010 at 9:44

Did you enable the FirePHP logging to make sure the 400 is due to too many
files?
http://code.google.com/p/minify/wiki/Debugging

The option is merged in here:
http://code.google.com/p/minify/source/browse/tags/release_2.1.3/min/lib/Minify/
Controller/MinApp.php#26

You might just insert a die($cOptions['maxFiles']) after that to see that the
option
is being merged correctly.

Original comment by<EMAIL_ADDRESS>on 2 Apr 2010 at 2:52

Changed title: maxFiles in config.php ignored
Changed state: NeedInfo

I think I'm just going to remove this feature in future releases.

Original comment by<EMAIL_ADDRESS>on 22 Apr 2010 at 5:30

Changed title: remove maxFiles
Changed state: Accepted

Removed in R411

Original comment by<EMAIL_ADDRESS>on 9 May 2010 at 4:48

Changed state: Fixed


Add draft contributors section

Fixes: #349
This first draft is not working 100% yet. We will need to figure out how to add a GH token to the Netlify build. I guess another option would be to commit content/en/contributors_include.html and create a GH action which updates it regularly through running the ./hack/contributors.py script and file a PR.
Thoughts?

I'd also need some help beautifying the section somewhat.

I would put the contributors below the community information, as the information has more value to a visitor than the faces of all the lovely people that got us here.

Done.
Now I need help hooking this up. AFAICS our options are:

put a GH token into Netlify somewhere, run script on every build
run script by a cron job in a GH Action (weekly?) which generates a PR to update content/en/contributors_include.html

I would opt for a weekly job that opens a PR.

Thanks @stefanprodan - will take a look!

Appears to be working: https://github.com/dholbach/website/runs/2931141709


Cookie sessions

Now using cookie-sessions to store user credentials and basic twitter profile data.
Deployed version

[Current coverage][cc-pull] is 69.68%

Merging [#94][cc-pull] into [master][cc-base-branch] will increase coverage by 3.60%

@@             master        #94   diff @@
==========================================
  Files            17         20     +3
  Lines           168        188    +20
  Methods          40         46     +6
  Messages          0          0
  Branches         26         28     +2
==========================================
+ Hits            111        131    +20
  Misses           57         57
  Partials          0          0

Powered by Codecov. Last updated by [d4e29e4...6f4a198][cc-compare]
[cc-base-branch]: https://codecov.io/gh/watson-developer-cloud/personality-insights-nodejs/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/watson-developer-cloud/personality-insights-nodejs/compare/d4e29e422e33d468e95808d9494397e41aac6039...6f4a198e24229a1a049d429ba474aff25c2b43e8
[cc-pull]: https://codecov.io/gh/watson-developer-cloud/personality-insights-nodejs/pull/94?src=pr


Update SQL queries to match new schemas

Motivation and Context
Slight changes to the SQL queries to update the schemas and column names
What has changed
Changes to the schema and column names
How to test?
Check that the SQL queries are correct and bring back data
Links
https://trello.com/c/RVgSGgNO
Screenshots (if appropriate):

 Generic Survey - DAP/MI (3)


feat: build and push docker image to ghcr

Closes #22

Build package from this branch: https://github.com/open-component-model/ocm-controller/pkgs/container/ocm-controller
Now I'll remove this branch as target.


How to disable an M2 NVMe drive from appearing in macOS?

Hello!
Is there a possibility to disable a specific M.2 slot? I am assuming I should be using a custom SSDT (but don't know how)

See chapters 1 and 2.
You need to find the PCI path of the device in th DSDT. If it contains the method _STA you can disable it pretty easy via SSDT.
I suggest asking for help on a Hackintosh forum or on reddit.


Get package SAS URI incorrect permission

The application permissions show as "Machine.CollectForensics"; however, the true permission is "Machine.Read.All" or "Machine.ReadWrite.All".
'code': 'Forbidden', 'message': 'The application does not have any of the required application permissions (Machine.Read.All, Machine.ReadWrite.All) to access the resource.'

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

ID: 5a8946a2-a2ea-1dc7-a6ce-13c56d11b485
Version Independent ID: 9de7a1fa-d8d9-4ea6-33fc-7e50528b1c04
Content: Get package SAS URI API
Content Source: microsoft-365/security/defender-endpoint/get-package-sas-uri.md
Product: w10
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@MastaDisasta Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/6073
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @MastaDisasta Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Does this expand an already existing service
Yes
Link to a game or software
https://automatisch.io/
Links for the download
https://github.com/automatisch/automatisch
Links for the install docs
https://automatisch.io/docs/guide/installation

Read the main repo please.


Fixed Issues #338 and #295

Related Issue
Not Proper View For User
Closes: #338 and #295
Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots Before

Screenshots After

@sourabhsikarwar  I have updated please do check and merge

Every time code I am facing conflicts , please look and try to merge as soon as possible @sourabhsikarwar

@sourabhsikarwar , Fixed Conflicts , please check and merge

@sourabhsikarwar  Please merge this PR it has been 3 weeks

Bhai take a pull from the update code. I have been saying this on discord for a while. Your PR will cause a lot of changes in the project.
Kindly update it with latest code and share images for that. @Harshu467

Bhai take a pull from the update code. I have been saying this on discord for a while. Your PR will cause a lot of changes in the project. Kindly update it with latest code and share images for that. @Harshu467

Bro  check out my commits I have taken pull

This issue and PR are pending since a month please check out @sourabhsikarwar

Its been toooo much time for this PR to be merged and review , Conflicts are coming because of addition or updating of part others are doing which I am also doing and their PRS are been merged and that why I am facing this issue , It is my humble request from maintainers and project owners please check out , humble REQUEST. I am closing this PR and creating again for betterment , Thank YOu, I have updated code as every time I took pull before checking


Render tiles from OHM PBF

See ohm-20180612.osm.pbf 750 MB download here: https://greeninfo.slack.com/files/UAAD2PM5Z/FBKD5RN95/ohm-20180612.osm.pbf (private slack msg)

To import into OpenMapTiles, place in data/ directory and run normally.

This is in place. See also #1 which continues this work.


Linked Visio file does not match the displayed architecture diagram

Currently links to https://archcenter.blob.core.windows.net/cdn/hybrid-network-hub-spoke.vsdx which appears to be an older, out of date version.
Compare to the current image: https://docs.microsoft.com/en-us/azure/architecture/reference-architectures/hybrid-networking/images/shared-services.png

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

ID: 87139748-938e-638e-de1f-94f8becfe1da
Version Independent ID: b96371e6-defb-3e70-792a-2c0cfdc25b7d
Content: Implementing a hub-spoke network topology with shared services in Azure
Content Source: docs/reference-architectures/hybrid-networking/shared-services.md
Service: guidance
GitHub Login: @telmosampaio
Microsoft Alias: pnp

Uploaded correct version of the Visio

I'm still not seeing the correct Visio diagram when using the download link.

@Trey432 - Are you looking at the "Shared Services" tab on the Visio? The file has two diagrams, hub-spoke and hub-spoke with shared services. (I just downloaded it and they look the same to me unless I'm missing something 😺 )

Yeah, my mistake.  I didn't realize that there was another tab.  Thanks for the quick response!


Don't re-tag if no new commits are found

Could be a new option for the current major and a new default for the next major
Related:

https://github.com/fregante/ghatemplates/issues/19

https://github.com/fregante/daily-version-action/blob/fb1a60b7c4daf1410cd755e360ebec3901e58588/index.js#L34
Already the case. The workflow should continue without creating further tags.


Less deps

Codecov Report

Merging #127 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #127      +/-   ##
=========================================
- Coverage   95.81%   95.8%   -0.02%
=========================================
  Files           2       2
  Lines         311     310       -1
=========================================
- Hits          298     297       -1
  Misses         13      13

Impacted Files
Coverage Δ

loggo/loggo.py
95.77% <100%> (-0.02%)
:arrow_down:

Continue to review full report at Codecov.

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


Cannot Install:  run.sh: file does not exist

Failed to install add-on
(-)
COPY failed: file not found in build context or excluded by .dockerignore: stat
run.sh: file does not exist
RPi 3B+ with latest version of Home Assistant OS:

I think the reason why this is happening is because these files (finish, run) are missing the .sh shell script extension. But I haven't tested this hypothesis yet.

fixed it!
https://github.com/SamirJouni/hassio-hotspot


Website: Nav panel toggle covers complete page

Describe the bug
On the website (https://hielkeminecraft.github.io/OpenNoteBlockStudio), if the window is not wide enough the navigation menu is shown on the left and can be toggled (see screenshot).
The problem is that the clickable area for the toggle action covers the complete screen (absolute, doesn't move when scrolling). This prevents the user from clicking any other links, such as the download link.
To Reproduce
Steps to reproduce the behavior:

Open https://hielkeminecraft.github.io/OpenNoteBlockStudio
Resize the window until the nav button in the top left appears (see screenshot)
Make sure you are at the top of the tab and have not scrolled down
Try clicking anywhere, including on the download buttons
:x: The navigation menu opens

Expected behavior
Only the navigation button should be clickable, not the complete / or parts of the page.
Screenshots

I did this and it happened.

Fixed in 79568262f15e16c56e259a908798d6cac2669b6e.


Cerberus+1 damage fall on

Right now we don't support Cerb's damage fall on nonsense. Amoral_Support in the Discord has collected data for us.

Make this work somehow
Probably will need to talk to @gothfemme to make sure it displays correctly from web

Here is amoral_support's data sheet


Translation to Portuguese Brazil

Hello, this is my translation to Portuguese Brazil V11. Thanks
pt-BR.json

Thank you very much, the translation is now included with v3.0.1


Smart Heat pump

Device normal info

Poolex
Jetline Premium 95

{
"active_time": xxx,
"biz_type": xxx,
"category": "ktkzq",
"id": "50705042e8db84d07225",
"ip": "xxxxxxxxxxxxxxxx",
"lat": "xxxxxxxxxxxxxxxx",
"local_key": "xxxxxxxxxxxxx",
"lon": "xxxxxxxxxxxxxxx",
"model": "",
"name": "Poolex Jetline Premium 95",
"online": true,
"owner_id": "",
"product_id": "",
"product_name": "热泵控制器",
}

Yesterday, Home Assistant 2022.2 was released and it contains a new diagnostic tool.
There is now a "Download Diagnostics" button on each Tuya device on the device page. Clicking that button will download diagnostics information for that specific device. Drag the downloaded file into this issue.
This will help us learn about your device and provide details for implementing/adding support.
Thanks!

Hello,
Same here
have you found an way ?


Ip adress is wrong when using uBlock Origin

The public-ip package fetch the public Ip Adress from icanhaz and ipfy services. When using ublock origin (and maybe others ad blocker), those domains are blocked, and the ip adress cannot be retrieved.
This leads to onyxia filling the ip adress with <IP_ADDRESS> in services, blocking all access to services when security ip is enabled (by default)

https://github.com/rdegges/ipify-api/issues/18
https://github.com/easylist/easylist/issues/2810

Damn it. Ok then I will use Onyxia API to fetch the IP then

Fixed in 0.11.40. Cool feature side feature. If the ip changes due to user switching to mobile data for example, the ip is reactively updated.

Thanks for reporting @micedre


CO_CANsend implementation

I'm using @sicrisembay  implementation of the CO_CANsend function for the ESP32 and have found an issue where the code fails (FREERtos tx task  enters infinite loop).
The question in this repo however is regarding the implementation of the CO_CANsend function. Here is sicris current implementation: https://github.com/sicrisembay/CANopenNode_ESP32/blob/97b26ad6dba7db224a637944037bdfb3fd9bc3b4/port/CO_driver.c#L354
I managed to get the code to fail when it sends two CO_NMT_sendCommand commands quickly, this will increment the CANmodule->CANtxCount value for each call and later in the tx task it will be decremented just once and hence the infinite loop, on the count > 0 check.
In case the buffer is full, it will just overwrite the buffer (and previous message is effectively lost), is this the implementation intended for this function? besides the bug here, I see esp32's by default has 5 tx buffers for the function when calling the lower level can send funcion, but this implementation is clarly not taking advantage of it in this case.
I really have two questions here:

what the behavior of the CO_CANsend should be when the canopennode buffer if full, should it still overwrite or just return the error without overwritting?
should the esp32's buffer be used when possible, even in this case? I guess this is done but only when using different canopennode's buffers.

@sicrisembay :
I guess the CO_CANsend function should wake the TX task before returning and have the buffer cleared, is this the intention here? --- ehmm, after writing this sentence I went and checked why this would not be happening and realized that the task calling CO_CANsend had higher priority (7) than the tx task (5) so I modified tx task priority to 15 and now it is working properly. I'm wondering if we could add a guard on the CO_CANsend implementation to detect the tx task priority is lower than calling task priority. I'm not really sure since default 5 does not seem to be "high"

@glmnet, thanks! I've fixed this (https://github.com/sicrisembay/CANopenNode_ESP32/issues/3#issuecomment-2394852375)


Estimated time to merge a PR is 13 minutes; is that the fastest we can do?

Investigate how much we can make the merging process faster without affecting the benefits. 13m for a PR merge is too long.

I'm trying new runners:

2xlarge takes 10m (down from 12m).
4xlarge takes 7m.

I'm not really sure if that's worth it, but we can try it for a while. I'm starting with 4xlarge because 2xlarge definitely isn't worth it. And the machines are spun up on demand so they shouldn't be too expensive...

Yes too expensive for the time gain. Thank you for trying those out

I've dropped us back to the defaults for now. TBH, we likely just need to speed up the tests a bit.

We could narrow down the test set in the merge queue. Most of our repos run zero tests in that phase.
Getting our tests would be nice as well but the above is lowest cost.


User interface changes as requested by client

Basic Information
Client said the student leaders would prefer the current sessions table above the queues

Closed by #35


So who is going to do a NEW Better OneTab?

obviously everyone is here because it would be nice & better to actually have a better OneTab ... who do not support their product either (and there is no Forum for it), and to add the obvious missing functions that the OneTab developers do NOT do ever.
btw, i just found another one (missing function) today (that many of you have prob already thought of) .... send OneTab groups (or single tabs) to Bookmarks

@decard123 I'm about to fork this repository and look into updating the codebase. However, I'm also looking into volunteers for assisting in maintaining the codebase in addition to myself.
What languages do you know?

hi there
i only just came back here so months late
only English
but it seems apparent to be that this Project for a Better Onetab is DOA.
is this wrong somehow?
and Onetab itself is totally unsupported in my experience
which is weird because it just needs a bit of serious tweaking to make it brilliant
do you know anything more about any of this?
fyi i am NOT a 'developer' myself, but sw and web design and development for 20 years: i just can't code any more apart from a bit of PHP (my coding dates back to COBOL)
i was/am so cranky about Onetab, i was thinking of hiring developers to reverse engineer the whole thing - with the missing enhancement; but i lack money.
kind regards
don