Cannot load the string 'null' in seed data

Issue
Issue description
I am being passed the string value 'null' in upstream data, and I would like to use my mapping file, maintained by checked in seed data, to turn it into something more pleasant to read like 'Unknown'.
However, I can't get the literal string 'null' into my seed table!
All of the following rows result in a NULL value rather than a non-null string value of 'null' after seeding:
key,original,mapped
a,null,Unknown
b,'null',Unknown
c,"null",Unknown

System information
The output of dbt --version:
dbt --version
installed version: 0.12.2

The operating system you're running on: OS X
The python version you're using: Python 2.7.10

thanks for the report @ianterrell! My recommendation would be to use some sentinel value besides null (like literal:null) for these values, then use a model to rename these values back to null.
I think the implementation you're describing is totally reasonable, but dbt's CSV parser is explicitly configured to treat "null" string literals in seeds as representations of null data points. You can see this configuration in the agate type tester code.
I think both approaches (treat null as the string "null", or as a missing value null) are reasonable implementations, and I think some nonzero number of people would be upset if we just flipped the implementation! It's conceivably possible to configure this on a per-seed basis, but that's not really something I'm interested in supporting in dbt.
I'd say: since seed files should be version controlled, your best bet is going to be to use some value other than null, then post-process the seed in a model. If that approach doesn't work for you, it could be that some tool other than dbt seed is the right one for this use case!

What if the column in the seed is INTEGER but we still want to pass null not quoted. What I don't like in CSV are empty values. I would prefer either null or 'null' to be treated as nulls. Is it possible?


Add missing verb to the role of node lifecycle controller

What type of PR is this?
/kind bug
What this PR does / why we need it:
Node lifecycle controller in yurt-manager maybe miss rbac settings.
Which issue(s) this PR fixes:
Fixes #1934
Does this PR introduce a user-facing change?
NONE

@crazytaxii charts/yurt-manager/crds/apiextensions.k8s.io_v1_customresourcedefinition_blockaffinities.apiregistration.k8s.io.yaml file need to be ignored.

/lgtm


fix: incorrect gradient for element on homepage 👷🏻♂⚒

Close #1490
Checklist:

[x] I have linked the PR to the correct issue.
[x] I have read the Contribution Guidelines
[x] I have built and tested the changes, and they do not break or show any errors.

@Vaishnavi-Patil2211 Please merge this PR.


[Error] Failed to get remote module base address

I have gotten this error message since the most recent update. I looked at all the closed tickets saying they have the same issue but saw no solution. I am using the most recent version of the unlocker and have done a clean installation of both Genshin and the launcher.

duplicate of #435, you're not using the latest version


[4.18] Update Node Cache after Successful Clean/Service

This commits makes sure we call update_node_cache
after we finish a successful cleaning/servicing.
Change-Id: I62403120c758caac38a4d2b3912a9c43f65161cc
4.19 will be added tomorrow when we do the sync

/hold
let's wait for the sync to 4.19

/unhold

/lgtm

oh god, I pushed to master this one instead of release-4.18 🤕


make sharding manager pluggable and document clustered setup

I am new to atlas and trying to configure and play with it. In the Documentation the Netflix Publish API pushes to the Mirrors. How does the publish API know what the mirrors are? I know in the documentation it uses Edda but then how does the publish API know where the Edda API is?

There are a several ways it can be configured. For the sample config it runs a single node in-memory configuration.
I need to finish merging a couple of the internal parts with oss and document the clustered setup. We currently used edda internally, but it forked and is significantly different than the oss edda right now. I'm trying to refactor some of those parts so it is more pluggable and can actually be used outside of our internal setup. This is taking a bit of time because it isn't strictly beneficial internally so it is a spare time project. I'm about half way done with this refactor though.

Any update on this?

+1 ?


[types] Mainnet ChainId 0 -> 1 to handle accidental initialization.

Motivation
Users might accidentally initialize the ChainId field to 0, hence reserving ChainId 0 for accidental initialization and starting mainnet at 1.

:umbrella: The latest upstream changes (presumably d57f43bde43b7a1ab9ad4026c97c0b5cafa48f70) made this pull request unmergeable. Please resolve the merge conflicts.

#5189 takes care of this.


Add Wi-Fi firmware partition support for Pico 2 W

This adds the ability to store and load the Wi-Fi firmware for Pico 2 W in a partition. It can be enabled by adding pico_use_partition_firmware(<exe_name>) to your CMakeLists.txt, which will embed a compatible partition table in the binary, and output lots of firmware UF2s to use (all called <exe_name>_firmware_..._.uf2,  eg ..._firmware_w for Wi-Fi only, ..._firmware_wb for Wi-Fi and Bluetooth, ..._firmware_w_tbyb for TBYB). You can also create your own partition table and use that.
A Wi-Fi firmware partition is detected as having the ID 0x123456789abcdef0, and the UF2 family_is for Wi-Fi firmware blobs is 0x12345678 - these should probably both be changed to something else before merging? The default firmware partition starts at 3500K into the flash - should this be changed to depend on PICO_FLASH_SIZE_BYTES? The default firmware partition is also duplicated with A/B partitions in the same location in flash - this is required to ensure a signature check is performed before loading the Wi-Fi firmware, as there's no way to call the bootrom to check the signature of a single partition (unless chaining into it), you can only call pick_ab_partition.
The Wi-Fi firmware blob is marked in it's image_def as an RP2350 Risc-V executable, and the partition is marked as ignored_during_riscv_boot - this ensures that it can work with TBYB (as TBYB only works for executable image_defs), and that signature checks are performed before loading the firmware when Secure Boot is enabled (because signature checks are performed for all executable image_defs in a partition that is not marked ignored_during_arm_boot). This is slightly clunky, but seems to work robustly.
Supercedes #1850, as it now includes that function in this PR, and shows a use for it.

pinging @peterharperuk as he did a lot of the Wifi-related stuff for Pico 1.

Works nicely. I think we need to fix the Pico W (rp2040) build errors in src/rp2_common/pico_cyw43_driver/cyw43_driver.c
It would be nice if it gave an error if you're a numpty like me and put pico_use_partition_firmware after pico_add_extra_outputs, but that could be improved later?

It would be nice if it gave an error if you're a numpty like me and put pico_use_partition_firmware after pico_add_extra_outputs, but that could be improved later?

Have added in separate PR #2054

I think we need to fix the Pico W (rp2040) build errors in src/rp2_common/pico_cyw43_driver/cyw43_driver.c

I've added a check that it's not RP2040 in the CMake function, so it'll throw a fatal error at that point if you try to build a binary with pico_use_partition_firmware for RP2040

If the Bazel checks are getting in your way, you can add the two new files to a filegroup and file an issue+leave a TODO to support this in Bazel and assign it to me.

If the Bazel checks are getting in your way, you can add the two new files to a filegroup (which will silence the error) and file an issue+leave a TODO to support this in Bazel and assign it to me.

Thanks, have done - I couldn't actually assign the issue to you, but have tagged you in it

A Wi-Fi firmware partition is detected as having the ID 0x123456789abcdef0, and the UF2 family_is for Wi-Fi firmware blobs is 0x12345678 - these should probably both be changed to something else before merging?

Is that still going to be happening? I guess choosing randomly-generated IDs is much less likely to produce an accidental collision than a "nice" ID like 0x123456789abcdef0 ? And should those IDs be added to a header-file, to allow other UF2s to make use of this Wi-Fi firmware partition?

A Wi-Fi firmware partition is detected as having the ID 0x123456789abcdef0, and the UF2 family_is for Wi-Fi firmware blobs is 0x12345678 - these should probably both be changed to something else before merging?

Is that still going to be happening? I guess choosing randomly-generated IDs is much less likely to produce an accidental collision than a "nice" ID like 0x123456789abcdef0 ? And should those IDs be added to a header-file, to allow other UF2s to make use of this Wi-Fi firmware partition?

Yes - I was looking for ideas. We could go with 0x776966696669726d (hex for wififirm) for the partition ID, and just use the data family_id we already have?

Yes - I was looking for ideas. We could go with 0x776966696669726d (hex for wififirm) for the partition ID, and just use the data family_id we already have?

I've got no opinions on this so it's probably something that you and @peterharperuk ought to decide. Would MicroPython be able to use the same Wifi firmware partition as C-SDK code? (or is that another one of my stupidly naive questions? :laughing: )

Would MicroPython be able to use the same Wifi firmware partition as C-SDK code? (or is that another one of my stupidly naive questions? 😆 )

Yes, they definitely could - there's no C-SDK specific code in the Wifi firmware partition, it's just a raw binary blob, so should work with MicroPython too.

On a vaguely-related note, the doxygen at https://github.com/raspberrypi/pico-sdk/blob/develop/src/rp2_common/pico_bootrom/include/pico/bootrom.h#L834 talks about an "explicit buy flag" - is that the same thing as the TBYB flag referred to elsewhere? (If so, it might be worth editing that doxygen for consistency?)

I've noticed an issue with the Risc-V build of the firmware UF2 - for some reason the compilation is putting a programme header in the ELF with offset of 0x0000 in the file loading to 0x1ffff000, when the Arm compiler correctly puts this at offset 0x1000 in the file loading to 0x20000000. @Wren6991 or @kilograham any ideas why the Arm and Risc-V compilers might be compiling the wifi_firmware.S file differently?

I've now got it working for Risc-V by storing copies of the firmware ELFs in the SDK, and then copying those into the build when necessary. It also needs increased bootrom stack size. This is tested and working fine.
However, I'm not sure if this is actually a solution that we want to put in the 2.1.0 SDK release, so it might be best if we don't merge this for 2.1.0 and I can investigate further to fix the Risc-V firmware build?


updating pre-commit with autopep8

Precommit-Verified:<PHONE_NUMBER>b3b5a1a6bbb1f17d38866e9658fe85a7b61410f0cc6c51e6faf3ee

Codecov Report

Merging #263 (ebef751) into main (802240c) will increase coverage by 1.30%.
The diff coverage is 85.18%.

@@            Coverage Diff             @@
##             main     #263      +/-   ##
==========================================
+ Coverage   80.29%   81.60%   +1.30%
==========================================
  Files           4        4
  Lines         873      875       +2
==========================================
+ Hits          701      714      +13
+ Misses        172      161      -11

Impacted Files
Coverage Δ

vmware_exporter/defer.py
97.22% <ø> (ø)

vmware_exporter/helpers.py
72.72% <20.00%> (+11.53%)
:arrow_up:

vmware_exporter/__init__.py
100.00% <100.00%> (ø)

vmware_exporter/vmware_exporter.py
81.43% <100.00%> (+0.33%)
:arrow_up:

Continue to review full report at Codecov.

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


JWM-2.4.2 ?

Hi @joewing
Since some good bugfixing has happened recently, is there a timeline for 2.4.2?
Regards.

I didn't have anything in mind, but I guess there's no harm in doing a release.
I'll try to get one out today or tomorrow.

Thanks Joe!


adaptor fasta

Dear all,
I couldn't find the adaptors of my samples in the adaptor directory. I used NEBNext® Multiplex Oligos for Illumina. Does it require a new fasta?
Thanks for your help in advance!
Best,
Yuling

Hi Yuling, we currently only support Truseq or Nextera adapters. For more details please refer to https://github.com/fl-yu/CUT-RUNTools-2.0/issues/24.
I am not familiar with that Oligos, but  I guess you will probably need to contact those people to get such adaptor sequences.

Thanks for the reference. May I ask how to generate the fasta file for adaptor sequence? I have the document for the sequence.
Best,
Yuling

https://international.neb.com/-/media/nebus/files/manuals/manuale6440.pdf?rev=903fc5e904cc4b2b9843b3b1f7d27f02&hash=7E9F7AC01683A3D41DFF5CEC33B6C156
This page contains the adaptor sequences for this kit, if you have interest.
Thanks!

Good to know you got the adaptor sequences working. I will close this issue and please feel free to reopen it once have further issues with the adaptor trimming.

Hi,
I also end in this issue: we also use NEBNext Oligos for library prep and sure we know the sequence and can provide a fasta files withe these based on the information of the pdf. Unfortunately I do not understand how to add this fasta to your pipeline in a way that the correct oligos for PE trimming will be selected. Can you please explain how this can be done?
Thank you,
Christian

Hi, I apologize that I was asking for support too early: I learned from the NEB website that basically 'The NEBNext libraries for Illumina resemble TruSeq libraries and can be trimmed like TruSeq:'
https://www.neb.com/en/faqs/2021/01/15/what-sequences-need-to-be-trimmed-for-nebnext-libraries-that-are-sequenced-on-an-illumina-instrument
I found the sequence of the oligos listed there in your and trimmomatic adapter sequences https://github.com/usadellab/Trimmomatic/blob/main/adapters/TruSeq3-PE-2.fa listed as:

PE1_rc
AGATCGGAAGAGCGTCGTGTAGGGAAAGAGTGTA
PE2_rc
AGATCGGAAGAGCACACGTCTGAACTCCAGTCAC

The only difference is that the oligos listed at NEB are one nucleotide shorter. While fastqc indeed showed little amount of Illumina Universal Adapter in the original fastqc, this was removed after trimming with your pipeline. I hope this comment helps.
Best,
Christian


public/newRegEmail.php missing!?

Took some time to install as the SQL.txt as it has some errors. After a while managed to fix them and successfully run the script. Now when I try to login I get "Object not found!"  The address line shows "public/newRegEmail.php " and there is no such file in the directory.

I'm not davidberriman, but it looks like he was trying to implement an e-mail confirmation system.
A temporary work around is to comment out the following lines in fg_membersite.php in the includes folder.
 1013:			  //if($confirm == "y"){ 1014:				  return true;//DON'T COMMENT THIS LINE OUT 1015:			//  }else{ 1016:				//  $_SESSION['confirmCode'] = $confirm; 1017:				  //$this->RedirectToURL("newRegEmail.php"); 1018:				  //return false; 1019:			  //}
This will default all users to being confirmed at login, even if they aren't, and then sends them to the logged in home page.

hi @Synthacon
I have found problems (sql.txt) to create the tables
What's the way to create?
What are the changes to be done?


Perform apt diffing by default.

Also move diff type checking up.
Before we would download all the images before failing with an invalid diff type.

Ref #86


Il2Cpp.perform not being called?

Hi!
Why is this perform method seems not called at all?

Used emulator x86 memu, frida-server also x86
command  "frida -Uf com.unity.games -l _agent.js --no-pause"
The game has Unity version: 2017.4.40f1 is this not supported? (i dont know unity version range)
i've tried to edit that "moduleName" in "Il2CppBase/base.js " to desired il2cpp target.so and auto compile with cmd "npm run watch" because the game load different name
but still not work. thank you :)

Hi, emulators are problematic; it doesn't work because they use different ways to load a shared library.
However, you could try adding --realm emulated to the Frida command and see if it works.
If doesn't I can already tell you there's nothing I can do about this

Hi, emulators are problematic; it doesn't work because they use different ways to load a shared library.
However, you could try adding --realm emulated to the Frida command and see if it works.
If doesn't I can already tell you there's nothing I can do about this

So i ended up trying with my android phone But got the same result.
could be the game has some protection? do i have wrong set-up?
do u have a link to example game working. i just wanted to try
thank you for your quick time :)

@ifoverload What game did you try?

@ifoverload What game did you try?

com.vng.mlbbvn used armeabi-v7a version as always..

It looks like that game has a protection - e.g. calling any il2cpp api will result in an access violation error. There's nothing I can do about this, unfortunately


Question: original material color names

Hi all, thanks for this great project!
I have a simple question: why did you implement your own color palette instead of using the spec?
Right now I'm trying to adopt a project on material-ui for react to materialize because of their inline-styles and your color naming is a bit confusing.
What is the mapping between your palette (base color, light, dark) to google's one? (that table with numbers)
UPD: Is there any possibility to switch to google's spec to be more consistent with material design?
Thanks!

Are you looking at the same _color.scss file as I am? We copied the hex values exactly.

@acburst Hex values are correct, I meant only the naming: amber-100, amber-200...

Agreed, I think it's both convenient and odd that the names are like .red.lighten-5. It would nice to also have .red500 and .redA200 and so forth. When I'm referencing colors from the spec and then go to materialize I have to convert red200 to lighten-2 or is is lighten-3?

The project could support both naming conventions. I personally don't mind the green-lighten-2, etc.
For example
green-lighten-3 === green-200 === #A5D6A7

green accent-4 === green-a700 === #00C853

blue-grey lighten-1 === blue-gray-400 === #78909C

etc.

Since the color classes are already very large when it comes to css output, we are not going to be supporting alternate naming conventions

This is a little confusing. The Materialize naming convention is an alternate naming convention.
So will you be changing to use he standard MD spec or leaving it with the current alternate convention?

I agree with @djensen47 here. The naming convention is confusing. Even though it would be a breaking change, I think switching to a schema like green-200 would make much more sense, since, as @djensen47 already pointed out, it is not always clear which materialize color is representing which color in the specs.


renovate: always update lvh in tandem with lvh-images

lvh-images may need a new version of lvh to boot. Always upgrade lvh when upgrading images.
Always update lvh in tandem with lvh-images

/test

/test

/test


Implement IInstrument for NI-Digital

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
AB#2531939

Creates a implementation of IInstrument for NI-Digital.
Tests for the 4 combinations of create/attach/detach/close

These changes are based on Paul's PR, #288 .

Automated tests pass.

Most of the VI descriptions for VIs in the Instrument.lvclass contain stale information from the interface VIs. I would recommend just deleting the descriptions since these VIs are really considered implementation details at this level and not public APIs. The same issue exists for the DCPower instrument class.

I've deleted the VI descriptions (via "Properties -> Documentation -> VI Description" for each DCPower and Digital VI) as requested.


KV store: master breakdown to hostname, port, ipv4, ipv6

Master entries are now listed in KV as follows:
mysql/master/mycluster:master.host.com:3306

^ as before, and then adding:
mysql/master/mycluster/hostname:master.host.com
mysql/master/mycluster/port:3306
mysql/master/mycluster/ipv4:<IP_ADDRESS>
mysql/master/mycluster/ipv6:<whatever>

This breakdown allows the user to pick their own preferred representation of the master. For example, HAProxy has better be populated with the IP address of the master rather than its hostname.

Deploys well, works as expected, raft included.

TODO: documentation


Regression, guest CognitoIdentityCredentials breaks when guest support is disabled.

My user pool does not use guest Credentials, the new support for guest CognitoIdentityCredentials broke my use case.
I error out here, since the calling code is not expecting the 'no current user' return.
     AuthClass.prototype.currentSession = function () {
         if (!this.userPool) {
             return Promise.reject('No userPool');
         }
         var user = this.userPool.getCurrentUser();
         if (!user) {
             return Promise.reject('No current user');

Higher up in configure, pickupCredentials() is not expecting an error.
    if (userPoolId) {
        this.userPool = new CognitoUserPool({
            UserPoolId: userPoolId,
            ClientId: userPoolWebClientId
        });
        this.pickupCredentials();

Adding an unauthenticated role to my identity pool did not fix this.
I am using manual setup and the configure call is failing.
Amplify.configure({
  Auth: {
    identityPoolId: 'us-east-1:.... /
    region: 'us-east-1',
    userPoolId: 'us-east-1_....',
    userPoolWebClientId: '71hfs....',
  },
});

I'll try again with a configuration file from Mobile Hub

This problem appears to be timing related. When I make changes to try and pin it down it goes away.  Here's screen shot of the call stack.

It may not be a regression, instead changes in my app may have altered the timing of how the code runs.

Hi Jon, it's possible that your code is trying to use the auth module before it's signed in, or established a session, which in fact should be handled though. Are you using the react library and Authenticator or using just plain JS for this?

@jonsmirl I am able to reproduce in Angular with the latest. Looking into it more now.

@jonsmirl it seems like this occurs when using await syntax on Auth.currentSession(). It's not actually failing, but the promise is getting rejected (as it should), but if you use the await syntax this will just produce an error, which technically it should (since there is no session). Are you expecting more to receive some sort of empty object or undefined for session? If you use standard promise syntax it works as expected i.e.
Auth.currentSession()
    .then(data => console.log('this will have a session if available') )
    .catch(err => console.log('this will fire if no session')

How ever if you use an async function
let session = Auth.currentSession();

This will throw an error
ERROR Error: Uncaught (in promise): No current user

I think this is the source of my troubles, I tried to put a Greeting into the page header.
I am only using the higher level React components.
class Header extends React.Component { // eslint-disable-line react/prefer-stateless-function
  render() {
    return (
      <div>
      Hello
        <Authenticator hideDefault={true}>
          <Greetings />
        </Authenticator>

Here is my startup...
Amplify.configure({
  Auth: {
    identityPoolId: 'us-east-1:...',
    region: 'us-east-1',
    userPoolId: 'us-east-1_C...',
    userPoolWebClientId: '71...
  },
});

// Create redux store with history
const initialState = {};
const history = createHistory();
const store = configureStore(initialState, history);
const MOUNT_NODE = document.getElementById('app');

const render = (messages) => {
  ReactDOM.render(
    <Provider store={store}>
      <LanguageProvider messages={messages}>
        <ConnectedRouter history={history}>
          <App />
        </ConnectedRouter>
      </LanguageProvider>
    </Provider>,
    MOUNT_NODE
  );
};

',
},
});
// Create redux store with history
const initialState = {};
const history = createHistory();
const store = configureStore(initialState, history);
const MOUNT_NODE = document.getElementById('app');

const render = (messages) => {
  ReactDOM.render(
    <Provider store={store}>
      <LanguageProvider messages={messages}>
        <ConnectedRouter history={history}>
          <App />
        </ConnectedRouter>
      </LanguageProvider>
    </Provider>,
    MOUNT_NODE
  );
};

I can hit this with nothing but the Amplify.configure() call in my app and using the remote debugger.  The remote debugger is altering the timing which causes the problem to occur. If I use the in-browser debugger I don't hit it.
I also logged into my Cognito user pool so that I have a current user, I still hit this problem when using the remote debugger.

@jonsmirl there was a fix that went out for this yesterday and was published to npm v0.1.22 (or latest master) should address this let me know if you still run into it.

Still there
jonsmirl@ubuntu-16:~/aosp/web3/web3$ yarn add aws-amplify-react aws-amplify | grep aws
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>
I rebuilt the Webpack DLL for sure
jonsmirl@ubuntu-16:~/aosp/web3/web3$ npm run build:dll

<EMAIL_ADDRESS>build:dll /home/jonsmirl/aosp/web3/web3
node ./internals/scripts/dependencies.js

Building the Webpack DLL...
Hash: 0093d8d89b46be85332e
Version: webpack 3.5.5
Time: 5830ms
Asset     Size  Chunks                    Chunk Names
reactBoilerplateDeps.dll.js  6.78 MB       0  [emitted]  [big]  reactBoilerplateDeps
chunk    {0} reactBoilerplateDeps.dll.js (reactBoilerplateDeps) 5.67 MB [entry] [rendered]
jonsmirl@ubuntu-16:~/aosp/web3/web3$ yarn start
yarn run v1.3.2
$ cross-env NODE_ENV=development node server
Server started ! ✓
Access URLs:
Localhost: http://localhost:3000
LAN: http://<IP_ADDRESS>:3000
Press CTRL-C to stop
webpack built cebf1a8d7c1c2e6f4143 in 3148ms
webpack building...
webpack built 1ad3f17112d016bf02e3 in 180ms
webpack building...
webpack built 7ef5c27c57d2c229087a in 83ms
webpack building...
webpack built e10a3d31a0f0e326cd74 in 91ms
webpack building...
webpack built c3f30841fadb4e8072db in 132ms

If I delete the files in, node_modules/aws-amplify/dist$ rm *, then the source maps correctly.
\
This is similar error in another place

If I tell my IDE not to stop on uncaught exceptions, well then the errors go away.  But there shouldn't be uncaught exceptions.  Other people maybe not be noticing this because they don't have this turned on in their debugger.

Hi @jonsmirl yes, so this does make sense, as in my comment above, the promise rejection is occurring "correctly" since you are trying to get an authenticated user and/or session when there is none, to prevent your uncaught exception you would surround it with a try/catch:
try {
    this.session = await Auth.currentSession()
catch(e) { console.log('no session') }

Again this only occurs when using await syntax see comment above.

I pushed my basic demo app out to git hub.
https://github.com/jonsmirl/web3
yarn install, yarn start
Open browser and debugger, turn on pause on uncaught exception.
Click the Login tab, click the Home tab, click the login tab
You should hit the exception.
The only interesting code is in:
https://github.com/jonsmirl/web3/blob/master/app/app.js
https://github.com/jonsmirl/web3/blob/master/app/containers/LoginPage/index.js

I asked a friend who is a Javascript expert, he did not look at the code but had experienced similar issues. His suspicion is that since async things like currentUserPoolUser() are getting called from the render() function the calls are being re-entered (from another need to render) before the async part of the first call finishes. When they get re-entered, the promise from the first entry gets lost which is what causes the 'uncaught exception' errors. He says this is why he uses things like Redux to keep the async activities independent of the rendering.

Newer version should be more accommodating to the exceptions, for now if you use async await it will throw an exception on rejection so you need to surround it with a try catch, or use then.catch similarly. We have more exception handling stories in our backlog and we'll track this one as part of that as well

I  saw on Auth have user object when signedIn:
try {
     const { accessToken, idToken, refreshToken } = Auth.user.signInUserSession
     const { jwtToken } = accessToken
     console.warn('this.session: ', jwtToken)
} catch (e) { console.log('no session') }

Look like docs missing update or I'm missing things here?
Retrieve Current Session
Auth.currentSession() returns a CognitoUserSession object which contains JWT accessToken, idToken, and refreshToken.

Refer: https://aws.github.io/aws-amplify/media/authentication_guide#using-withauthenticator-hoc


Why only .NET core was mentioned?

Why only .NET core mentioned? Earlier we used to have .NET code. Can it be mentioned in the documentation? this helps people like me who still work in .NET

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

ID: 60e56e69-1c44-2056-ad5f-319535e4b525
Version Independent ID: 49fa6098-0b8e-ebf5-efe7-f50f3604e325
Content: Azure Quickstart - Upload, download, and list blobs in Azure Storage using .NET
Content Source: articles/storage/blobs/storage-quickstart-blobs-dotnet.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

@aditya365 Thanks for the feedback! We are currently investigating and will update you shortly.

@aditya365 - Thanks for your feedback. I've updated the article to include setup information for .NET in the Prerequisites.
The reason that we recommend .NET Core in this article is so that users running on any platform (Windows, Linux, or MacOs) can use it to build a .NET application with Blob storage.
Note that the .NET Framework is installed with Visual Studio. So if you are using Visual Studio, you already have the .NET Framework. The method calls shown in the article are available in both .NET Core and the .NET Framework.

#please-close


Question: Does it support Instance Methods?

Hello, I am curious if this has support for instance methods, and what would be syntax to use it.

No need to answer, I'm a retard apparently

I wanna know


⚠ Minecraft has degraded performance

In a7f7943, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1844 ms

Resolved: Minecraft performance has improved in 951f617.


Explainable AI Integration for Time Series Model Interpretability

Description
This feature aims to enhance time series analysis by integrating the Explainable AI (XAI) package into the workflow. Users will be able to visualize temporal data, understand the factors influencing model predictions, and generate comprehensive reports. The integration includes applying stationarity tests, estimating model parameters using ACF and PACF plots, and utilizing the Explainable AI package to interpret model outputs. All visualizations and explanations will be saved as images within the same Colab notebook and compiled into a PDF report.
Problem it Solves

Lack of Interpretability in Time Series Models: Traditional time series models like ARIMA or SARIMA often act as "black boxes," making it difficult for users to understand how input features affect predictions.
Difficulty in Visualizing Temporal Feature Importance: Users struggle to identify which time-based features (e.g., trend, seasonality) are most influential in their models.
Inefficient Reporting Process: Manually generating and compiling visualizations and explanations into reports is time-consuming and prone to errors.

Proposed Solution

Explainable AI (SHAP) Integration: Leverage the SHAP (SHapley Additive exPlanations) library to explain the feature importance of time series models. SHAP values help users understand the individual contribution of features, even for complex models like SARIMA, ARIMA, or any tree-based regression model used in time series forecasting.
Temporal Feature Importance: Implement time-specific explanations where SHAP values can highlight how certain time periods or lags contribute to the predictions.
Model Explainability Visualization: Generate SHAP summary plots and force plots, illustrating how different features impact model outcomes across time. This will be crucial for both forecasting and general time series modeling tasks.
Report Generation: All visualizations, including SHAP plots and time series plots, will be automatically compiled into a PDF report. This allows users to have a structured output summarizing both the temporal patterns and the explainability analysis of the model.

Alternatives Considered

LIME: Another explainability method, LIME (Local Interpretable Model-agnostic Explanations), was considered, but SHAP provides more reliable global explanations, especially for tree-based models and regression models, making it a better fit for time series.
Manual Feature Analysis: Instead of using SHAP, manual feature importance analysis using correlation metrics or simple feature elimination could be implemented, but these would be less sophisticated and not as visually informative as SHAP.

Additional Context

Yes @sharayuanuse go ahead!
feel free to reach out,
update the core explaianbleai codes such that user can perform time series models efficiently with explainableai, as it's currently compatible with scikitlearn models

@ombhojane can you review this PR.


Just want to comment the vim.nofity, because it is pretty annoying when we save the file everytime.

Honestly do not understand your requirement.
I did setup autcmd just to emulate the not supported file saving, and even when the commented modified_date header is present, no vim.notify pops up on my setup.
Hovewer when I do this via keyboard shortcut, it simply displays me the vim.notify.
I suggest leaving it as is for now and close this PR.
P.S.: Please next time sync your fork, before opening an MR, for details see here.

I think the problem is, when there is no header in an unsupported file (e.g., JSON), it always shows the notification. However, I agree to just keep it like this. Sorry for the fork, will sync the fork first for the next pull request.


fix(runtime-core): the vnode created by createVNode, in the template, can't reactive

close #6151
fix(runtime-core): the vnode created by createVNode, in the template, can't reactive

Fix Preview


Fixes to Aligned Input Events

What's changed, or what was fixed?
In the Aligned Input Events article,

Added missing commas.
Removed an extra "in".
Added backticks where missing.
Removed a space before a comma.
Added missing semicolons in code snippets.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


[Bug]: Platinum shows the bookmark bar even when it's turned off.

Steps to reproduce the issue?
What I did was:
I build Platinum from source, then added two different bookmarks. They showed up, even when I had turned them off.
Note: When I reopened the browser, they were hidden again, but as soon as I added another they all showed up again.
Note: I'll try to fix this as well.
What happened?
I expected my bookmarks to be hidden until I enabled them.
Your environment (if applicable)
No response
Version
Dev from commit 8c7f869

Fixed in this pull request: https://github.com/redstone-scratch/Platinum/pull/4

I'm using the latest source code and can't reproduce this bug.

I'm using the latest source code and can't reproduce this bug.

It got fixed in commit 2cfa290. If you're using that version, then yes it got fixed. Also this might be broken in Catalyst too, so try to reproduce it there.


[ANCHOR-809][SEP-31] Add funding_method to SEP-31

Description

Add funding_methods to /info response. This field is mandatory for the anchor.
Add funding_method to POST /transaction param. This field is mandatory for wallet.

Context
SEP-31 changes scoped in https://github.com/stellar/stellar-protocol/pull/1567
Testing

./gradlew test

Next
The SEP-6 change will be handled in a separate PR.

By the way, develop is now the 3.0 branch, we should be merging into that.


Allow trying Firezone without signing up

Need a temporary "demo mode" of some sort to try Firezone without signing up that we can use for launch day. This was the most important thing we need according to HN. This can be turned back off or completely removed the next day.
Some thoughts:

What does it mean to try Firezone without an account?
Maybe we do this on staging? That will keep it from affecting current customers, but they'll need to update their Clients' settings...
Maybe just limit it to the admin portal? I imagine signing in from a Client will be hard to do without an account, so we can maybe just disable logins for accounts in "demo mode".

Perhaps it could be "SSH to a machine with no exposed ports" as an MVP.

This sounds like a pain but I think if we're creative we can come up with something. assigning the team for crowd-sourcing ideas!

@firezone/engineering

So we'd like a way for a free user to run, say, a Client on one of their systems and a Gateway on the other, and see that they can SSH (or even HTTP) from one to the other?
It's too bad we need root for both Clients and Gateways, that will be a little scary for the security-conscious. We could bundle an HTTP server with a Gateway as a demo, that would be less scary than an sshd

How difficult would it be to:

Make the demo slug a thing
Pre-create a site with a few gateways
Disable revoking gateway tokens and any other destructive action
Pre-create some resources like ifconfig.net etc, don't allow to change anything about it
Users "login" with email but don't require a password and can thus continue straight away
There is only one group Everyone and each "login" adds that user to that group

If we control the resources, we pretty much limit what they can do with that free account and they can't really hammer our gateways with heaps of traffic.
This "demo" slug could also be useful for something more permanent when you want to demo firezone on somebody's device :)

We can make signup transparent, you follow up a link and get an account without an email attached to it and can use everything right away. Then you can add email to make sure you won't lose the account when signed out. We will also need a way in the portal to show a client sign-in code or something similar to that.

Users "login" with email but don't require a password and can thus continue straight away

An advantage of this would be that we could build up an email list of users that have tried firezone and follow-up with a newsletter or something. Sign-up to that mailing list could be part of the T&C of using the demo!

We can make signup transparent, you follow up a link and get an account without an email attached to it and can use everything right away. Then you can add email to make sure you won't lose the account when signed out. We will also need a way in the portal to show a client sign-in code or something similar to that.

Ah yes, good call -- presumably some folks would actually like to continue using their account after trying things out. We would need some way to start an account in demo / trial mode then. Perhaps they can start on the Team plan.
We might need some way to prevent dupe sign ups, or just disable this special "sign up without email" workflow after Launch day.

presumably some folks would actually like to continue using their account after trying things out.

Isn't the account bound to the slug you signed in with? If you actually want to use firezone, they'd properly sign-up afterwards, no? Or am I misunderstanding the model here?

@thomaseizinger we will generate an account with a slug, but the identity of the admin will be something we need to build (like secret identity that just has a code/password, which can be used to sign in to the portal and client). In the portal we show a warning that the user needs to replace that with actual email. We can put everyone on Trial during the first days and then make it default to free accounts afterwards.

@thomaseizinger we will generate an account with a slug, but the identity of the admin will be something we need to build (like secret identity that just has a code/password, which can be used to sign in to the portal and client). In the portal we show a warning that the user needs to replace that with actual email. Fortunately, everything is built with assumption that IdP doesn't return emails so it should be pretty easy.

Do we need to generate so many different slugs? Why not set up one demo slug that people can use without a password (or some other auth mechanism)?
I'd imagine when people want to "Try Firezone", they have little interest to log into an AWS account and spin up an EC2 instance for a gateway to be able to see their IP change when they use ifconfig.net or something like that. It would be a much more powerful demo if they can:

Download the Android / iOS app
Select demo slug
Enter any email
Be logged in as "unverified"
Go to ifconfig.net and see their location change

No need to set up gateways or define resources.
Then, if they are interested in how it works, they can log into the portal with the same email and "browse" around (but not change anything).

Trial / demo users won't be able to see each other's email addresses, right? Or any other user-entered string? Someone will definitely put something offensive in any text field

@thomaseizinger because if we have one demo slug then all users will be in the same account and won't be able to try Firezone for real, they won't deploy their own gateways (everything will be accessible for anyone) - I believe we should try to convert the audience to get some ROI rather than just mark a checkbox "can use without sign up". And making most of the UI read-only with proper error messages is more work than creating accounts without emails.

Trial / demo users won't be able to see each other's email addresses, right? Or any other user-entered string? Someone will definitely put something offensive in any text field

If they are in the same account and read-only they will see everything under that account, unless we also hide some fields based on user role (which is even more work).

@thomaseizinger because if we have one demo slug then all users will be in the same account and won't be able to try Firezone for real, they won't deploy their own gateways (everything will be accessible for anyone) - I believe we should try to convert the audience to get some ROI rather than just mark a checkbox "can use without sign up". If they are not going to deploy a gateway at all - it's not our target audience but at least we will be fair to them and not fake anything. And making most of the UI read-only with proper error messages is more work than creating accounts without emails.

Yeah, converting some users would definitely be good!
I am just cognisant of the number of steps a user has to go through before they can use firezone. Signing up is one thing, having to deploy your own software is a much bigger step that e.g. anybody coming to the Launch HN post from their phone sitting on a sofa isn't going to do.
The demo slug would be more of an appetizer to have them set up a real account where they then can setup their own gateway etc.
I don't know the backend codebase so maybe there are better ways other than making things read-only yet they come to an already set up environment with resources and a gateway?
What if we deploy a separate portal instance where things are read-only hardcoded? I remember @jamilbk said that this could be something that will be thrown away after so we don't have to modify prod code.

Personally, I think letting each user have their own account is a better way to go, being that we've set everything up in the portal to have a boundary at the account level.  It feels like it would be the least amount of work, rather than trying to figure out how to let a lot of new users share an account.  Along with that, running a completely separate portal would require everyone to adjust their client settings to point to the temporary portal, which is not the end of the world, but it does introduce friction and then would require them to reset their client settings if they decide to use the prod portal.
All that said, I do recognize how much effort it is to get all of the pieces setup just to try out Firezone.  I know we all know this, but at a minimum you need to have a machine to run a gateway and then also install a client on another machine.  And ideally you'd need to run at least one other machine as a resource.
I'm wondering if we can split the difference and also make it kind of a game.  What if we made a small CTF game that if the user finds the flags we offer a discount of some sort? Each user would get a free Firezone account on prod and the only thing they would need to do is to download and install the client. Then we could run a cluster of machines that will run the gateways and resources for the game.  We'd need to add a few things in the portal for spinning up one of the "Firezone Hosted Gateways", but those machines could be shared between accounts since each account would have it's own instance of the Firezone Gateway running on one of them.  We'd have instructions for how to create a Site, how to spin up a Gateway (although this would be kind of simulated, because we'd have a button to deploy one of our hosted Gateways), and how to add a Resource.  Then they would need to add the first resource, which could be an HTTP server with some type of vulnerability they need to figure out to find the address of the second resource to add and then with the second resource that would give them the flag for the discount.  Even if they don't figure it out it would show them the basics of how to use Firezone.
I recognize this may be a decent amount of work, but I think just about anything we do is going to be a decent amount of work.  In doing this, after the 'launch', we'd at least have all new users in their won accounts that they could keep long term if they wanted and it wouldn't require everyone to spin everything up just to try it out.
Thoughts?

All that said, I do recognize how much effort it is to get all of the pieces setup just to try out Firezone. I know we all know this, but at a minimum you need to have a machine to run a gateway and then also install a client on another machine. And ideally you'd need to run at least one other machine as a resource.

To get how firezone works, we can use any resource on the Internet, right? Something like ifconfig.net that shows the location of your IP address makes it clear enough that traffic is now going through the gateway. To demonstrate split-DNS, we can also tell them to use a 2nd, similar website that that isn't a resource and they'll see that their IP for that one is different!

I'm wondering if we can split the difference and also make it kind of a game. What if we made a small CTF game that if the user finds the flags we offer a discount of some sort? Each user would get a free Firezone account on prod and the only thing they would need to do is to download and install the client. Then we could run a cluster of machines that will run the gateways and resources for the game. We'd need to add a few things in the portal for spinning up one of the "Firezone Hosted Gateways", but those machines could be shared between accounts since each account would have it's own instance of the Firezone Gateway running on one of them. We'd have instructions for how to create a Site, how to spin up a Gateway (although this would be kind of simulated, because we'd have a button to deploy one of our hosted Gateways), and how to add a Resource. Then they would need to add the first resource, which could be an HTTP server with some type of vulnerability they need to figure out to find the address of the second resource to add and then with the second resource that would give them the flag for the discount. Even if they don't figure it out it would show them the basics of how to use Firezone.
I recognize this may be a decent amount of work, but I think just about anything we do is going to be a decent amount of work. In doing this, after the 'launch', we'd at least have all new users in their won accounts that they could keep long term if they wanted and it wouldn't require everyone to spin everything up just to try it out.

I love this idea! I played through https://gandalf.lakera.ai/ recently and really enjoyed it. The idea of "Firezone Hosted Gateways" is good! We already have the terraform modules, right? How much effort would it be to add a button that says "Deploy Demo Gateway" that is perhaps shut-down again after a day or so? That would take some of the friction out.
A CTF game could be additional thing we build on top later?

These are all good suggestions. To get this shipped super quickly though I thought of the following:

We have a demo mode. Can be enabled / disabled via Domain.Ops (or possibly just by reverting the PR since this will just be a one-time thing)
Arrive at app.firezone.dev/new_demo_account: we create<EMAIL_ADDRESS>admin
Session is only good for one hour
We display banner message warning them about session, link to Actors to create a new Actor for themselves or to sign in from a Client

Another option is to spend a little bit of time on a new self-hosted docker-compose.yml that is geared towards spinning up locally for demos.

We have a demo mode. Can be enabled / disabled via Domain.Ops (or possibly just by reverting the PR since this will just be a one-time thing)

Arrive at app.firezone.dev/new_demo_account: we create<EMAIL_ADDRESS>admin

Does the demo mode come with a pre-deployed gateway or do they have to deploy there own?

@thomaseizinger We were discussing this on standup yesterday with @bmanifold and I think adding a pre-deployed Gateway to each account when it's provisioned is quite a large amount of infra work. We'd also have to make the site readonly so no Resources can be added to it, otherwise we end up with liability issues.
I was thinking the local docker-compose up thing could work, but unfortunately the client auth process seems to require HTTPS so that approach is going to be tricky as well.
See #5687.
The way we can "hack" the above for desktop clients is to inject a service account token into the secure keychain for each OS prior to the app launching. I'll think more about that... maybe it can be done with a simple script that does the following:

docker compose up
Generate a service account + token with mix ecto.seed or similar
Inject the token into the expected place in the keychain (Apple has CLI utilities for this)
Prompt the user to install the app for his/her platform and configure the control plane URL

That's incredibly hacky though.

Maybe we can get away with storylane.io

I think the consensus here was:

Add a simple 'demo' landing page - https://app.firezone.dev/try. Has a big button "Launch a demo instance" which creates an account with 1 actor<EMAIL_ADDRESS>and signs them into the admin portal.
For "demo" accounts, show a banner (#1732) with:

Welcome to your Firezone demo! You may sign in to this admin portal and using a Firezone Client using the slug <account_slug> and the email<EMAIL_ADDRESS>This account will be removed after <creation + 24 hours>. Create another admin user to keep this account active.

For demo accounts, show a new, default tab in the Deploy Gateway page that has a minimal docker compose file


Currency conversion

Currently, 100 nok in eur will be matched by the conversion regex but nothing will be done because the DuckDuckGo API doesn't do currency conversion.
Maybe find some alternate API that can do what we need?

Use this: http://www.ecb.europa.eu/stats/eurofxref/eurofxref-daily.xml
If you need to go from GBP to USD do it by going to EUR first. There's basically 0 arbitrage on FX these days.


Unable to associate Basic SKU Public IP to a Network Inteface

[Enter feedback here]
I have an existing VM that doesn't have a public IP but now needs one. When I create a new Basic SKU public IP and attempt to associate it to the NIC of the VM I get the error message below. This error is due to public IP being created in a different zone than that of the VM/NIC (zone 1)
I can create a Standard SKU public IP which would allow me to select the specific zone of the IP but I don't need a standard SKU and don't want to pay more for standard vs basic and also would prefer not to re-deploy the VM just to add a public IP. Is there another way to do this?
Failed to save IP address changes for network interface 'xxx-yyy-zzz-nic'. Error: Compute resource xxx-yyy-zzz-001 has a zone constraint 1 but the PublicIPAddress /subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/resourceGroups/rgname/providers/Microsoft.Network/publicIPAddresses/xxx-yyy-zzz-pip used by the compute resource via NetworkInterface or LoadBalancer has a different zone constraint Regional.

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

ID: 1f66d367-2dad-6595-cf3f-ffe91faee858
Version Independent ID: a46ae2e3-a5ba-e268-48fd-66248fc4b96f
Content: Create a public IP - Azure PowerShell
Content Source: articles/virtual-network/create-public-ip-powershell.md
Service: virtual-network
GitHub Login: @blehr
Microsoft Alias: blehr

@greendx, Thanks for reaching out. We will investigate and update this thread.

@greendx, as of now this feature is not available in basic SKU IP address and the same has been documented in Azure docs here,

Only Standard SKU Public IP can be zone pinned.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

That documentation is poorly worded because it makes it sound like
availability zone scenario is optional.
If you’re deploying a VM with a public IP into East US region it’s
impossible to use basic SKU public IP because you’re not able to not use an
availability zone even if you’re only deploying a single VM. Therefore, you
should update the document to state that Basic SKU public IP is not
supported with VMs in regions that have availability zones.
On Wed, Oct 28, 2020 at 6:48 AM suvasara<EMAIL_ADDRESS>wrote:

@greendx https://github.com/greendx, as of now this feature is not
available in basic SKU IP address and the same has been documented in Azure
docs here,
[image: image]
https://user-images.githubusercontent.com/44890829/97425918-c215be80-1938-11eb-963d-86942e064543.png
Only Standard SKU Public IP can be zone pinned.
We are closing this issue for now. If there are further questions
regarding this matter, please reply and we will gladly continue the
discussion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/64801#issuecomment-717851237,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRCJBWV6AUWQ2ADVTBPXMLSM7ZG3ANCNFSM4S2VJRDA
.


Additional tag requests for Cost Explorer

Added unique API requests for OCP on cloud, OCP on AWS, and OCP on Azure. When the Cost Explorer's "Perspective" menu changes, these additional requests are mad for tags.

https://issues.redhat.com/browse/COST-1338

Codecov Report

Merging #1959 (5e41e97) into master (95a1129) will increase coverage by 0.38%.
The diff coverage is 54.54%.

@@            Coverage Diff             @@
##           master    #1959      +/-   ##
==========================================
+ Coverage   71.16%   71.54%   +0.38%
==========================================
  Files         372      375       +3
  Lines        6231     6857     +626
  Branches     1122     1471     +349
==========================================
+ Hits         4434     4906     +472
- Misses       1679     1930     +251
+ Partials      118       21      -97

Impacted Files
Coverage Δ

src/api/tags/tagUtils.ts
27.77% <25.00%> (-7.23%)
:arrow_down:

src/api/tags/awsCloudTags.ts
66.66% <66.66%> (ø)

src/api/tags/azureCloudTags.ts
66.66% <66.66%> (ø)

src/api/tags/ocpCloudTags.ts
66.66% <66.66%> (ø)

src/api/tags/tag.ts
100.00% <100.00%> (ø)

...c/pages/costModels/createCostModelWizard/table.tsx
34.28% <0.00%> (-10.16%)
:arrow_down:

src/components/i18n/i18nProvider.tsx
57.14% <0.00%> (-9.53%)
:arrow_down:

src/pages/state/maintenance/maintenanceState.tsx
66.66% <0.00%> (-8.34%)
:arrow_down:

.../pages/costModels/createCostModelWizard/markup.tsx
27.27% <0.00%> (-8.03%)
:arrow_down:

...ges/costModels/costModelsDetails/utils/toolbar.tsx
77.50% <0.00%> (-7.80%)
:arrow_down:

... and 131 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 95a1129...5e41e97. Read the comment docs.


Update olHelpers.js

Sorry,
I had forgotten to add the $http.
I had modified it in my fork, but I hadn't done here.
Sorry again, but I'm not git expert.
Bye,
Matteo.

Fantastic @kratosmat, thank so mucho for the PR.
I Have edited some JSCS checks and compiled a new version with your feature in the last commit.
Cheers.


Feature: Setting to split result processing from scheduling

This is an offshoot of this ticket, but a different use case: https://github.com/sensu/sensu/issues/1537
The way sensu works today, Scheduling is a function of the leader.  The leader can also do result processing.
Scenario one: after adding the aforementioned "butt load" of servers, if there is any kind of monitoring event [massive outage, massive set of new checks to be scheduled] and result processing becomes overwhelmed, scheduling of new checks on the system will start to suffer / be delayed / stop.
The second scenario impacted by this design is the way Handlers are implemented.  From a plugable interface POV, handlers are perfect.  From a large system POV, handlers are overwhelming and can cause catastrophic failure of a BIG environment quickly.
The third scenario impacted by this design is the way config reload are performed.  Doing a graceful shutdown of a sensu server that is overwhelmed by result processing (docker or physical) will cause checks to not get scheduled.  This is extremely problematic when it comes to metrics and loss of metrics due to checks not getting scheduled.
I would like to see a setting get implemented where the scheduling leader can be setup so it will never process results.  The only thing that leader ever does is schedule checks to go out > systems will process > results will hit the rest of the cluster.  In an HA configuration, this should be part of the election process and not pinned to a specific node.  The registry you just built should be a perfect mechanic for tracking this kind of thing and making sure something is always scheduling.
This  could be expanded even farther to the queue / redis database.  Any result that has a status change gets a new message thrown into the queue / redis.  A role could be elected as a percentage of the entire server pool - 1 that are handler processors for events.  This would cut down any events of any single part of the system getting overwhelmed and having GLOBAL impact to the entire server environment.
If there is only a single server int he cluster, then obvi, it holds all the roles anyway and this logic goes out the window.
Expected Behavior
Better segmentation of roles for larger environments will lead to more stability and more consistent check scheduling.
  We are going to be eclipsing 1000 machines currently split between 2 sensu environments to mitigate this flaw.  We will need to continue to spin up new environments to mitigate this impact.
Your Environment
1000 machines running 16M checks / metrics per day

We intend to split up the Sensu server leader tasks, allowing multiple Sensu servers to contribute, spreading the load amongst them https://github.com/sensu/sensu/issues/1537

@portertech The spirit of this ticket is centered around more than pure splitting.  Today, scheduling can be compromised by result processing / TTL processing / keepalive processing.  This ticket takes role splitting a bit more granular by requesting that: Scheduling can be siloed off to its own server & result processing (in a multi server environment) cannot happen on the box scheduling checks.  Today, at scale, there are many scenarios we face where parts of the system can obliterate functionality on other parts of the system.  This is forcing us into situations where we need to have multiple sensu implementations running simultaneously rather than taking the benefits of the core ideals of sensu (horizontal scalability).
I would be happy to have a more in depth conversation about what we are seeing if we can get together on something like slack (or the dreaded telephone...).

@portertech any update on this?

@portertech are you going to be at the conference?  Can we chat there about this ticket?

@gwagner I am going to be at the Sensu Summit in August 👍 The team at Sensu Inc is making significant progress on the next generation of Sensu (2.0), which stays true to the core principles and features of Sensu, but takes a very different approach (particularly with the distribution of work). I am reluctant to make these sorts of changes with Sensu 1.x, as they will be superseded by the upcoming version. I recently published a blog post about Sensu 1.0 and the 1.x roadmap, here: https://blog.sensuapp.org/sensu-core-version-1-0-is-now-available-c93b61977933. I hope to publish a blog post about 2.0 and what we are building, very soon! Will I see you at the Sensu Summit? I would love to show you what we're building and have this conversation under its light 👍

I will be there and i am speaking :)

Closing this issue as Sensu 2.x is a complete rewrite and fully distributes these tasks 👌 https://github.com/sensu/sensu-go


the .gitignore file is outdated

the .gitignore file contains historical path names, it should contain
BUILD
mbed-os
*.py*

this should be checked in other examples too.

Thanks for raising this , we shall update all examples

This would be my proposal:
# Default Build Directory
.build/
BUILD/
.mbed
venv/

# mbed build system
mbed-os/
mbed_settings.py
mbed_config.h
*.pyc

# exporters
GettingStarted.html

Internal Jira reference: https://jira.arm.com/browse/IOTCORE-1403

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.

Apologies, this was closed in error by the automated bot - re-opening.


Repository size > 3 gig

You might consider reducing size to speed up downloads

Thanks for the suggestion. There is a compromise between accuracy and size of the repository. You need about a 1 GB of data to run optimizer properly. I suppose we can remove the history for them and save some space. Also, LFS is an option. But quite expensive on GitHub considering the current traffic.

Maybe you could store the data in a different place and provide a simple script to download it. Maybe bintray.com could be a good match? If you only need the data to run the optimizer is not a bad idea in my opinion


Add support for DescribeNetworkInterfaces tag filters

Closes #619

Codecov Report

Merging #4057 (6a6bd9a) into master (7147c5f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #4057   +/-   ##
=======================================
  Coverage   94.98%   94.98%
=======================================
  Files         465      465
  Lines       50450    50453    +3
=======================================
+ Hits        47918    47921    +3
  Misses       2532     2532

Flag
Coverage Δ

servertests
55.87% <66.66%> (+<0.01%)
:arrow_up:

unittests
94.85% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

moto/ec2/models.py
96.16% <100.00%> (+<0.01%)
:arrow_up:

moto/ec2/responses/elastic_network_interfaces.py
98.24% <100.00%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


New School

file.zip

@TeMp3r126 I am sorry, the request is incorrect. Please send a request to<EMAIL_ADDRESS>and describe which email domain you would like to add.
I have to close the request without merging. You can take a look on what a correct request looks like.


Additional .editorconfig settings

Description

We're getting a few inconsistencies around formatting in PRs recently, so it'd be best to write them into the .editorconfig and put them in the conventions doc.

[ ] Line width (120 seems to be default on Rider and it fits on the GitHub browser diff side by side.)
[ ] end_of_line =
[ ] dotnet_sort_system_directives_first = true
[ ] dotnet_separate_import_directive_groups = false
[ ] csharp_indent_case_contents = true
[ ] csharp_indent_case_contents_when_block = false
[ ] csharp_indent_block_contents = true
[ ] csharp_indent_labels = one_less_than_current
[ ] csharp_preserve_single_line_statements = false
[ ] csharp_style_namespace_declarations = file_scoped
[ ] dotnet_style_prefer_is_null_check_over_reference_equality_method = true:suggestion

YAML

[ ] ij_yaml_indent_sequence_value = false
[ ] YAML indent 2 spaces

These ones are additional Resharper configs that are not currently defined in our .editorconfig, but are default in Rider.
resharper_braces_for_while = required
resharper_csharp_insert_final_newline = true
resharper_csharp_keep_existing_enum_arrangement = false
resharper_csharp_wrap_before_binary_opsign = true
resharper_csharp_wrap_parameters_style = chop_if_long
resharper_indent_raw_literal_string = indent
resharper_keep_existing_declaration_block_arrangement = false
resharper_keep_existing_embedded_block_arrangement = false
resharper_keep_existing_linebreaks = false
resharper_place_accessorholder_attribute_on_same_line = false
resharper_place_simple_embedded_statement_on_same_line = false
resharper_resx_insert_final_newline = false
resharper_shaderlab_insert_final_newline = false
resharper_show_autodetect_configure_formatting_tip = false
resharper_space_within_single_line_array_initializer_braces = false
resharper_T4_insert_final_newline = false
resharper_use_heuristics_for_body_style = true
resharper_use_indent_from_vs = false
resharper_vb_insert_final_newline = false
resharper_wrap_before_eq = true
resharper_wrap_chained_binary_expressions = chop_if_long
resharper_xmldoc_insert_final_newline = false
resharper_xml_insert_final_newline = false

Discord Thread https://discord.com/channels/310555209753690112/1235489625771081759

end_of_line = crlf

Why???

0x6273 does have a point about the CRLF setting, as #26401 introduced a workflow that will fail when there's anything with CRLF in the PR, or the repo at that HEAD pointer for that matter. It's a choice that you will have to make, but if CRLF is chosen, then the workflow will have to be updated accordingly.
See also .gitattributes, which I believe also has functionality for managing this.

Ah I thought they had a problem with the end_of_line setting itself, since the settings are what the discussion is about. The values are either the current default values in the IDE or (in this case) what was in the .editorconfig, but commented out.

The workflow actually won't fail since in the git index it's all lf. Currently you can checkout and use whatever line ending you want in your worktree as long as commits are lf.
If we are going to force everyone to use a specific line ending style in the worktree like the issue author seemingly wants, it should be lf so that it's consistent with index.

I wanted to know why it was commented out in the .editorconfig, so I put it on the list to get arguments for or against.
I don't have a lot of experience with the problems that line ending can or do cause, so I've put it on the list for others such as you to discuss whether we want it in the file or not.
Please stop assuming maliciousness. I'm not forcing anything here. Since it's a problem, it'd help if you could list out the problem clearly so that we can have comments why it isn't being included for future reference.


integration/API improvements

Biggest wins here are in the integration touchpoints. Should be much easier to get started.
Added Hello World sample and a basic configured sample to demonstrate.

LG with setDispatcher and its inverse public, and with a unit test covering https://github.com/square/flow/commit/439536dd4116063548dcf99511eb96105673e9e9

@rjrjr Added the requested test, made dispatcher methods public and fixed build tools for travis.  merge time!


Windows docker mount volume not work if path contains "conf.d"

docker run -p 888:80 -it -v //g/src/docker/rainloop/conf/rainloop.conf:/etc/nginx/conf.d/rainloop.conf --name nginxtest nginx
it raise exption:
[crit] 1#1: pread() "/etc/nginx/conf.c/rainloop.conf" failed (21: Is a directory)
os: windows 10
when i cut "con.d" ,it works fine, but i dont knwon how to solve this problem
please let me know if you fix it.
thanks very much.
best regards.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

The reason that fails, is related to what I commented on your question in https://github.com/boot2docker/boot2docker/issues/846#issuecomment-167531624. Because the boot2docker virtual machine cannot find the file, docker will create a directory named rainloop.conf inside the container.
Either put your files below the c:\users directory on your Windows machine, so that they are shared with VirtualBox, or manually add extra mounts to the virtual machine, as mentioned in the boot2docker README
I'll close this issue here, because this is not a bug, and better discussed in the boot2docker or docker machine issue tracker


Coverage

Get coverage report and badge

signed up for coveralls based on badge on powercalc

Codecov added to CI pipeline


Made action button colors customisable

Also made the buttons darker by default

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

Project coverage is 98.83%. Comparing base (d7e41d0) to head (a239fec).
Report is 17 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage   98.83%   98.83%
=======================================
  Files          12       12
  Lines         431      431
  Branches       70       71    +1
=======================================
  Hits          426      426
  Misses          5        5

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


no lib, make error.

[root@*~]# cd amiitool
[root@ amiitool]# make
"make" lib -C /root/amiitool/mbedtls CFLAGS="-DMBEDTLS_CONFIG_FILE='"/root/amiitool/configs/mbedtls.h"' -Wall -pedantic -O2 -fsigned-char"
make[1]: Entering directory '/root/amiitool/mbedtls'
make[1]: *** No rule to make target 'lib'.  Stop.
make[1]: Leaving directory '/root/amiitool/mbedtls'
make: *** [Makefile:57: mbedtls] Error 2
I looked for install lib on arch linux on google.com but not find it.
Where i can install lib and other on arch linux?

Did you also check out the mbedtls submodule?

I found other issue, git submodule update --init --recursive
make and make install.


Make ConfigOption names into an Enum

    Is there any reason we couldn't just make these an enum? Then we could make the `set_x` and `get_x` methods type safe.

The rationale here is to make the configuration code more "rust like" and follow the Rust way of strongly typed when possible
Originally posted by @thinkharderdev in https://github.com/apache/arrow-datafusion/pull/4492#discussion_r1039443698

So I started bashing this out, with the incremental way like:

/// DataFusion specific configuration names.
pub enum ConfigName
{
    /// Configuration option "datafusion.execution.target_partitions"
    TargetPartitions,
    /// Configuration option "datafusion.catalog.create_default_catalog_and_schema"
    CreateDefaultCatalogAndSchema,
    /// Configuration option "datafusion.catalog.information_schema"
    InformationSchema,
    /// Configuration option "datafusion.optimizer.repartition_joins"
    RepartitionJoins,
    /// Configuration option "datafusion.optimizer.repartition_aggregations"
    RepartitionAggregates,

    /// Configuration option "datafusion.optimizer.repartition_windows"
    RepartitionWindows,
...}

But then I was thinking, if we are going to change the API anyways, why not go all the way with something fully statically typed and removing ConfigDefinition entirely:

/// DataFusion specific configuration names.
pub enum ConfigValue
{
    /// Configuration option "datafusion.execution.target_partitions"
    TargetPartitions(usize),

    /// Configuration option for arbitrary user defined data
    UserDefined {
      name: String,
      value: Option<ScalarValue>
    },
...}

impl ConfigValue {
  /// Return the name of this configuration value
  fn name(&self) -> &str {
    match self {
      Self::TargetPartitions(_) => "datafusion.execution.target_partitions",
      ...
    }

  /// Return the human readable description for this configuration value
  fn description(&self) -> &str {
    match self {
      Self::TargetPartitions(_) =>
                    "Number of partitions for query execution. Increasing partitions can increase \
                 concurrency. Defaults to the number of cpu cores on the system.",
...
  }

  /// set the value of the configuration value
 fn set_value(&mut self, new_value: ScalarValue) ->Result<()>{
     match (self, value) {
      (Self::TargetPartitions(v), ScalarValue::UInt64(Some(new_value))) => *v = new_value,
      (Self::TargetPartitions(v), _)  => return Err("Expected uint64 for {} but got {:?}", self.name(), new_value)).
...
}

But before I go crank that through the process I wanted to get some feedback if that was a desirable way to go. I would retain the ability to store arbitrary name/value pairs in the metadata.
What do you think @thinkharderdev @yahoNanJing @andygrove @avantgardnerio ?

I have a feeling that there is a egg and chicken problem in case of ConfigValue, for example property name is necessary to get value from env, but in order to get name it is necessary to have instance of ConfigValue because name fn requires &self. I don't see a way to avoid having separate definitions and values.
More strongly typed approach could be to define all properties as types,
struct TargetPartitionsDefinition { name: &'static str, description: &'static str } impl { parse(...) -> ConfigValue }
enum ConfigValue { TargetPartitions(usize) }

:point_up: what he said :)
I'd like to be able to use Default::default() and struct update syntax if possible.
BTW, thank you @alamb for taking on this thankless task. Recurring conflicts regarding multiple PRs each changing config in their own way is why I still have two of my own sitting in limbo. It will be nice to have this sorted.

struct update syntax

One potential issue with struct update syntax, is any additional config is technically a breaking change. Unfortunately ..Default::default() doesn't work with #[non_exhaustive] as you might add a private field. See here

I don't have a strong opinion either way. The one thing I do like about @alamb original enum proposal is that is that we get a nice way to associate each config with a config key and description that would maybe a allow a more generic way of handling the decoding from CLI params and spitting out documentation/help text. Although you could probably do the same with a macro in the struct case.

I don't think I have a strong opinion either as long as we retain the current capabilities, specifically:

Docs get generated from the config definitions
Custom key/value pairs can be used so that platforms building on DataFusion can leverage the same mechanism
Configs can be serialized easily

I had a play around and came up with this - https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=8c1ababec354db7118d3ff5ffd878f1e
I think it meets all of the above requirements, whilst being a "simple" structured type.
Let me know what you think, if people are happy with it I'd be happy to get something like it integrated with DataFusion.

Let me know what you think, if people are happy with it I'd be happy to get something like it integrated with DataFusion.

I think it looks good to me 👍

I had a play around and came up with this - https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=8c1ababec354db7118d3ff5ffd878f1e
I think it meets all of the above requirements, whilst being a "simple" structured type, and not requiring a proc-macro.
Let me know what you think, if people are happy with it I'd be happy to get something like it integrated with DataFusion.

Nice, seems like a good approach


[Update Request]: Microsoft.AzureCLI 2.62.0

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Microsoft.AzureCLI
Package Version
2.62.0
Please describe the changes you would like to see
There's a new version available (2.62.0). See https://learn.microsoft.com/en-us/cli/azure/install-azure-cli-windows?tabs=azure-cli

https://github.com/microsoft/winget-pkgs/pull/162494

#162494


Create Database Model

As a developer
I need create a high level database model
So that the team has a consistent high level understanding of what our resource's schema will look like
Details and Assumptions
The inventory service keeps track of how many of each product we have in our warehouse.
At a minimum it should reference:

product id
quantity
restock levels
condition of the item (i.e., new, open box, used) as an enumeration.
and their corresponding datatypes

*Restock levels will help you know when to order more products.
*Being able to query products by their condition (e.g., new, used) could be very useful.
Document to be updated: https://dbdiagram.io/d/648a3907722eb77494fa8edc
Acceptance Criteria
Given the database model is decided
When we develop the inventory database
Then we can reference the design document for tables, columns, relationships and datatypes

Should this be labeled with technical debt instead of enhancement? As the user won't be aware of the DB model?

Agreed. I wasn't sure about that either but technical debt makes sense to me.

I think the initial revision (https://dbdiagram.io/d/648a3907722eb77494fa8edc) looks good to me


126 search previous processes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

76 of 76 (100.0%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 91.16%

Totals

Change from base Build<PHONE_NUMBER>:
0.6%

Covered Lines:
1619

Relevant Lines:
1776

💛  - Coveralls


GITHUB_TOKEN?

I read the little doc... and the issues, but I've found no mention on what the GITHUB_TOKEN is supposed to be. I can guess it, but nevertheless a single little paragraph explaining how to create and use it would save a lot of time of guessing and try and error.
I would like the README to be automatically updated (i.e. no PR). How that is supposed to be configured? Again: a simple example in the doc would greatly improve the usability of this project.
Thanks

https://docs.github.com/en/actions/security-guides/automatic-token-authentication

And i am not sure what you meant on how to configure without PR its right in the first steps.


Rudimentary AArch64 architecture support

This a very basic support for the AArch64 architecture where I could link a trivial.C test case (2 relocation types are needed) on the real hardware :confetti_ball: I included parsing of the -m option in args.rs where default architecture is selected based on target_arch. Later on, the option value is converted to the proper Arch implementation and I've added more template Arch arguments in elf_writer.rs.
Please review the changes before I'll carry on :)
Related to #43.

Nice! This looks like a great start. Did you want to merge this or wait until it's more complete?
BTW, I was able to measure a small slow down for x86-64. Somewhere around 1%, which is close to the limit of what I can measure, however it does seem reasonably consistent. That doesn't need to block this though, since we can always make RelocationSize an associated type later - assuming that's the cause of the slowdown that is.

Thanks! I've extended the implementation where all non-TLS and non-dynamic relocation types should be supported now. There are plenty of failing tests, but I think it's a good starting point. Can we please merge it?


Support for custom Valves

Hi,
I created a custom Tomcat Valve and configured it in context.xml. I tried putting the Velve implementation class in classpath when launching, but I'm getting java.lang.ClassNotFoundException:
java -cp .jar -jar webapp-runner.jar --context-xml context.xml --port 8888 theWar.war
When the Valve implementation is added into the webapp-runner.jar, everything is working as expected.
Is there a way to specify jar(s) where custom Valves are implemented so they can be used by Tomcat?
Thank you.

Can you try:
java -cp webapp-runner.jar:yourapp.jar webapp.runner.launch.Main --context-xml context.xml --port 8888 theWar.war

Yes, this works! Thank you.


WORKDAY function is not available in Power-Fx

In Excel, we have WORKDAY function which returns a number that represents a date that is the indicated number of working days before or after a date (the starting date). Working days exclude weekends and any dates identified as holidays. Use WORKDAY to exclude weekends or holidays when you calculate invoice due dates, expected delivery times, or the number of days of work performed.

This function is not currently on our roadmap, but it is nice to have. Feel free to contribute to the project.

Working on this feature. I'll keep this thread updated.

Working on this feature. I'll keep this thread updated.

Please note that @m365solutioninsights already send us a PR on this: https://github.com/microsoft/Power-Fx/pull/2638
I'll find time this week to review it.

Ah, ok. Thanks you for letting me know, Anderson.
Looking for a way to contribute. I'll look for another good one to start with.


Support for Raspberry Pi

Hey there,
I would like to use the nextcloud-exporter with raspbian ...
Do you have any advice how to do that?
Thanks for your help.

Hi,
The nextcloud-exporter should run fine on raspbian without any modifications, you just need to compile it for a different architecture as the processor in the Raspberry Pi uses a different one than the CPU in your computer.
Luckily cross-compiling is very simple in Go, you just need to set the GOARCH environment variable (and GOOS if you are not on linux) to the architecture you need. I have a script to do that in one of my other projects because I use that one on a RPi myself. That script might just work for you if you have Go installed and are using Linux...
That said, I plan to provide pre-built binaries for different architectures for the next version which will happen soon.
Have a nice day! :sunny:

Dear Robert,
thanks for your message. I tried using your script but it failed with the following error:
pi@raspberrypi:~/go/src/github.com/xperimental/nextcloud-exporter $ GOOS=linux GOARCH=arm go build -v -ldflags="-s -w" .
github.com/xperimental/nextcloud-exporter
# github.com/xperimental/nextcloud-exporter
./main.go:67: config.InfoURL.Hostname undefined (type *url.URL has no field or method Hostname)

I am a Go newbie so I have no idea what the problem is. Maybe there are some libraries missing on my system?
Thanks again,
Klaus

I succeeded building the exporter with go 1.11 on another system :-) Raspbian has go 1.7 ... maybe this was the problem.


only year can't choose the future

hello can this library only year can't choose or scroll on the future?
thank you

@fajaranugrah You need to set the "maxDateRange" to now time then you won't be able to select the date in the future for the day, month and year also... This is an ideal solution to let users select the date of birth which cannot be a future date
new SingleDateAndTimePickerDialog.Builder(context)
            .defaultDate(nowTime)
            .maxDateRange(nowTime)
            .display();

@maksumon its work thank you


Add option to enable ExecuteMany WOH to set workflow properties

This adds the ability for the ExecuteMany workflow operation handler to set workflow properties as output rather than creating a new track.
This is similar to the support for workflow properties for ExecuteOnce introduced in MH-12286.

Closing as this is incorporated in #1234


Implement gRPC check

The gRPC liveness check is in alpha in Kubernetes v1.23 and behind a feature gate. Once it is not behind a gate it needs to be supported here in order for the checks to continue to be a superset of the Kubernetes probes.
It could also be implemented sooner and placed in a similar alpha state.

Update this feature is now in beta in Kubernetes v1.24 but is still behind a feature gate.


Issues with integer division

Hey Dylan,
I'm not really sure if you intend to support Python 3, but I just wanted to bring to your attention that this expectation on the integer division no longer holds in Python 3. You need to use the operator //.
https://github.com/dylan-campbell/bpnpnet/blob/21642c830c0544cd71ddb1c02873be2dfc025436/model/model.py#L36
The good thing is that // is an explicit integer division in python 2 and 3, so making that replacement will not affect legacy support.
Cheers

Thanks Sérgio, good pick up!
PyTorch 1.5.1 and older did integer division for the / operator, but I believe this has since changed. I've replaced it with //.
Cheers,
Dylan

Closing with a55eb7684515ba4fa9ae9e0110500994b9f83175


Feature extraction implement

Implement feature extraction logic for face recognition

feature extraction algorithms have been implemented to effectively extract more and more features from the input images iteratively - they are part of the CNN's functionalities


There are valid use cases for repeating a feature in a combo

https://commanderspellbook.com/combo/2062-2584-4133/
The general implementation of this combo requires two "damage whenever a creature enters" features. At least one of them must have the "source is creature that ETBd" attribute in order to meet Soul Collector's criterion, but the second does not have this restriction.

Duplicate of #546


Add contributing instructions

create a file called CONTRIBUTING.md

do you want me to work on this?

do you want me to work on this?

i can take this one


[Request] Fallback to default language for TMDB responses

I suppose Elementum doesn't have Catalan translation, cause it falls back to English for menus. But on genres submenu, text on item is missing even the English
It doesn't occurs when kodi set to Spanish or English, does make well the change
Not sure if occurs for other languages not included on the addon

Names of genres comes from the TMDB, probably, they are not having that language. I will look if it's possible to fallback to default language.

@Shut3n  do you have problems with language for, say, list of movies, or movie information (when you browse through Elementum)? Or it's only about genres list?

It's only about genres text, no text appears when kodi is set in Catalan language (it appears well when set to English or Spanish)

Added fallbacks in v0.0.61.


Spell Muncher TypeError: Cannot read property 'name' of undefined

FoundryVTT: 0.8.6
Importer: 2.1.3, 2.1.4
Receiving the following error when running Spell Muncher. Item muncher works fine.
main.js:5947 DDB Importer | ERR > TypeError: Cannot read property 'name' of undefined
    at http://localhost:30000/modules/ddb-importer/dist/main.js:26588:24
    at Array.map (<anonymous>)
    at parseSpells (http://localhost:30000/modules/ddb-importer/dist/main.js:26587:4)
    at async Function.parseSpells (http://localhost:30000/modules/ddb-importer/dist/main.js:29392:7)

Hi, if you're on Discord can you reach out so I can get some debug info you don't want to post on github :)
If not, do you have any homebrew spells in your collection? (And I'm going to assume you're using D&D 1.3.3?)
Thanks

Ok weird, it's working now and I didn't change anything other than update some modules today, including this to 2.1.5. Maybe your most recent change fixed it our DDB had a bug on their end?


Add RBAC example

Add RBAC example

:+1:

Coverage decreased (-3.1%) to 43.945% when pulling 9b85d8a3288e3ea869defea30bd278354296127c on rbac-example into 215a87058e6959023e032d484939c9dcb8bb7d5b on master.


Added ReflectionOnly methods and unit tests

Hi. Take a look. I am short on time, so I might have cut a corner or two, but everything works in my particular case.

Looks great. Thanks for your contribution!

You are welcome.

I'll get this into nuget probably late this week.


Add API documentation for uploading new custom emoji

See https://github.com/zulip/zulip/commit/3eeec94d036c606942bfa2ed17d8488417ffc312 for an example API docs commit; we may also need to extend the Python bindings in python-zulip-api to add a function for this first.
Tagging as a priority since we've gotten a lot of requests for this (the use case is bulk-importing custom emoji from Slack).

@zulipbot claim

https://zulip.readthedocs.io/en/latest/tutorials/documenting-api-endpoints.html is helpful context.

@zulipbot claim

Sorry, I don't have free time right now because of my final exams. I'll certainly get back here when its done
Thanks :)
@zulipbot abandon

@zulipbot claim


v2.0 Extend Line

Please see image above - Ideally, I want 10000 to be my zero line with a line going across as If it was zero, so I can change thickness and colour. I've tried many things, but can't get this to work
Alternatively, if I can get a line drawn across like in this example: http://jsfiddle.net/7a4hhzge/4/ then this should be enough for my requirements.
I have got as far as extending the Line (see below), but the code would only draw the dots and line and not lines to link up the dots. The line I commented out I thought would draw the lines(commented out as I haven't got syntax right - might not be there?) and then the code that follows is additionally drawn.
Chart.elements.Line = Chart.Element.extend({
        draw: function() {
                //Chart.elements.Line.prototype.draw.apply(this, arguments);  //draw existing lines?

               //my additional line to draw
                ctx.beginPath();
                ctx.moveTo(0,0);
                ctx.lineTo(300,150);
                ctx.stroke();
            }
        });

Any help much appreciated. Keep up the good work.

@andi-b the grid lines are drawn by the scale. The base class for all scales is in src/core/core.scale.js. To style an particular grid line, you could modify the scale so that the following properties are arrays
gridLines: {
    color: , // currently a string. if given as as array of strings, map array to ticks
    lineWidth, // currently a number. if given as an array of numbers, map to ticks
}

You should use Chart.helpers.getValueAtIndexOrDefault to make your life easier. You could modify that function to allow a sparse vector which means that not every grid line needs to be styled in the array. The new getValueAtIndexOrDefault would be:
function(value, index, defaultValue) {
	if (value === undefined || value === null) {
		return defaultValue;
	}

	if (helpers.isArray(value)) {
        // Check for undefined to allow a sparse vector
		return index < value.length && value[index] !== undefined ? value[index] : defaultValue;
	}

	return value;
},

The biggest downside of this method is that it requires knowledge of the number of ticks (since the array index is used). Alternatively you could build a mapping of value -> line settings. I'd be more than happy to merge a PR adding this functionality.

Thanks for the response - I tried colouring the lines with an array, but didn't seem to work, although the second part of your response has led me on a better track. I amended the following in Chart.js to get me closer:
FYI: this.options.gridLines.negative = 10000;
// Draw the chart area
if (this.options.gridLines.drawOnChartArea) {
	this.ctx.moveTo(chartArea.left, yLineValue);
	this.ctx.lineTo(chartArea.right, yLineValue);
	if(label==this.options.gridLines.negative){
		this.ctx.strokeStyle = "red";
		this.ctx.lineWidth = 2;
	}
}

But for some strange reason it colours each horizontal line that follows:

@andi-b I think you need to reset the canvas style after drawing the red line. Changing the if as follows should do it. The scale was written to not add lots of resets of strokeStyle and lineWidth when nothing is changing.
if (label ==  this.options.gridLines.negative) {
    this.ctx.strokeStyle = 'red';
    this.ctx.lineWidth = 2;
    setContextLineSettings = true; // reset next time
}

Brilliant - I didn't realise I was so close :)
Many thanks

Closing since resolved.


[BICEP] - deployCustomPolicyMonitoring - InvalidCreateRemediationRequest

What happened? Provide a clear and concise description of the bug, including deployment details.
When avdDeployMonitoring and deployCustomPolicyMonitoring = true, deployment fails with the following error
Please provide the correlation id associated with your error or bug.
194ecb07-59d8-44ab-89a0-1f7914565ff4
What was the expected outcome?
Azure policy should have been created
Relevant log output
"details": [
                    {
                      "code": "InvalidCreateRemediationRequest",
                      "message": "The request to create remediation 'policy-set-avd-diagnostics-1' is invalid. The policy assignment '/subscriptions/###########/resourceGroups/MYRGNAME/providers/Microsoft.Authorization/policyAssignments/policy-set-deploy-avd-diagnostics-to-log-analytics' assigns a policy set definition. Remediations must specify a single policy definition reference ID within the policy set definition."
                    }

@OrionWithrow-OHIT thanks for reporting the issue, we are reviewing it.

@yshafner

@danycontre @OrionWithrow-OHIT
I ran the LZA with monitoring and the monitoring policies and had no issues.

Unable to reproduce the issue or get comments from requestor.


Frontend refactoring

beep boop closing to try and force github to recognise there's a new commit 🫠


Initialize m_lastPoint; fixes intermittent test failure.

Bah.

On merge, the build status here will cascade to 20170125-merge-develop.


Why am I even thinking of reducing if-fi syntax?

No, should have I noticed earlier that what I was doing was nothing but to put my finger to your pie.


opt Simplify config file of meta-service

Remove some entries in the default doris_cloud.conf which may make users hard to understand.
Problem Summary:
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[x] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[x] No colde files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Release note

None

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[x] Add branch pick label

run buildall


*-DbaDbOrphanUser - Improve the workflow and fix some bugs

Type of Change

[x] Bug fix (non-breaking change, fixes #6459 )

The original bug in the linked issue was just the missung $users = $User because the names of these parameters are not the same in all commands.
While testing I found that Get-DbaDbOrphanUser only lists the orphan users from windows logins so I changed that code. Does anyone know why there was a -eq $false?
And then I changed Repair- and Remove- to use Get- to reduce duplicate code. Therefor Get- now returns the SMO of the user as well (Like Get-DbaDbRoleMember does).
I don't know if this commands handles all cases correctly. I found this link that might help: https://www.mssqltips.com/sqlservertip/5221/different-ways-to-find-sql-server-orphaned-users/
I always tell my students and clients how to avoid orphan users so I don't have any interest in these commands. But if someone has the time and the interest - there is room for improvement in these commands.

Maybe look at #7253 as well...

Ok, all tests for Remove-DbaDbOrphanUser fail - just have no time to work on that today or tomorrow. Will continue next week...

The code LGTM. If anyone else approves, I'm happy to merge. I believe @ClaudioESSilva did a lot of orphan work.

sweet, thanks so much!