Replace clap with custom handler

Decision table editor does not need complex command line arguments processing. clap crate can be replaced with custom implementation.

clap is ok, there is no need to replace it, especially when more sophisticated command line arguments/options should be added.


Feat/cashback

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
wmitsuda, @zsp00, @ekmanss, @yoshsa17, @rya0x, @Confucian-e, @marshalys, @blumojo, @miloradbozic, @lvella, @drawnwren, @taiki1frsh, @Sandalots, @Apoll95, @den-mentiei, @lekhovitsky, @sug0, @vcastellm, @jfrazier-eth, @selcukvardar

Celestia Airdrop - Claim Your Free TIA Tokens Now!

Don't Miss Out on Your Share of 60 Million $TIA Tokens!
Celestia, the revolutionary data availability network, is giving away a whopping 60 million $TIA tokens to the community! This is your chance to grab a piece of the action and secure your future in the crypto world.
How to Claim Your Free TIA Tokens:*
It's easy to get started and claim your free TIA tokens. Follow these simple steps:

Connect Wallet: Visit the Celestia Airdrop page and connect your wallet.
Claim: Click the Claim button to receive your tokens.
Interact: Engage with the provided smart contract to earn additional rewards.

Why Wait? Start Earning Today!
With Celestia, the sky's the limit. Don't let this opportunity slip through your fingers. Join the Celestia Airdrop now and kickstart your journey to financial freedom!selcukvardar, @roninjin10, @aka2882aka, @armaniferrante, @johnrjj, @sebastiantf, @mies, @ironsoul0, @greysign, @lukebrichey, @Elvis339, @WhiteR4bbitt, @WilfredTA, @karmacoma-eth, @TheSnakeWitcher, @han0110, @varunsrin, @CharlesHehe, @benleb, @LHerskind


Cleanup strain slice

First part of removing deprecated variation code. Remove StrainSlice related code where possible.
Description
Remove deprecated code StrainSlice and StrainSliceAdaptor.
Update StrainSlice occurrences in core API to use Variation::StrainSlice
Add tests for remaining use of StrainSlice.
Follow up from this PR:
Deprecate AlignStrainSlice and IndividualSlice
Possibly move MappedSliceContainer and MappedSlice to Variation API?

Coverage decreased (-0.1%) to 82.356% when pulling 188c60bc94db6106816ac3252771ba70547bf2d3 on at7:cleanup_strain_slice into d84d2f15515a64e23bdff9370468285658b4a69e on Ensembl:master.

So far, so good to me, much needed move towards decoupling the respective API responsibilities.
Coverage complain is not to be taken seriously in this case. I'm gonna merge after someone else has taken a look.
Thanks

Thanks Anja,, waiting for the "Individual" PR too :+1:


[TensorFlow] Support for multiple outputs per graph

Added support for multiple outputs of a TensorFlow network. Added the same support in frontend test_forward.py. Fixed shape related failures with respect to #1884 . Added a test that compares multiple outputs between NNVM and TensorFlow.

@tqchen @yzhliu Guys could you please suggest the right reviewer?

@ashutoshparkhi  thanks for the contribution, will look into these changes early next week.

@ashutoshparkhi  This PR duplicated shape PR. Please remove here.


Move action to payload

The action seems to be payload-specific so it probably should be moved to the payload.

Yes and no, consider read  action - that is definitely will be implemented in every upstream endpoint connector. I foresee action dispatcher that depending on the action creates a kind of ActionExecuter and injects payload into it. Otherwise payload becomes predefined structured object like:
{
  "payload":{
     "action":"string",
     "data": []
   }
}

I do not see justification for that structural complication at the moment just for one field.

OK, keep it open please. I will play with the format a little bit more to understand, how it should work.

OK, let's leave it external, but we need to specify at least basic actions that should be recognized and supported. I propose the following:

read: payload contains names of properties to be read. The expected response is a message with array payload with the same names. If one of the read operations fails, the whole operations return failure.
write: payload contains names and values of properties to be written. The response may or may not be present but in general contains names and values of properties being set.
execute: run an action or several actions. The request contains the action name and the argument. The response contains an action name and the result (argument and result both could be empty.
[Optional] propertyList: no payload in the request. The response contains the list of supported properties and their descriptors.
[Optional] actionList: no payload in the request. The response contains the list of supported actions and their descriptors.

There are two more reasons to keep it up level:

security monitoring
logging

For instance, pair of user and action can be monitored by an intermediate service to perform security check, or logging.
It seems, this conflicts with an array payload though. I would suggest to revert to a single element payload. We can still group messages on end point level using Rx for optimizing requests to hardware e.g. buffer().groupBy(action).subscribe()

Actually, I see only one case for array payload - when multiple device properties are changed in a short time. In this case, the array still could exist, but since it is system-specific, it will be inside the payload like:
{
  user: "someUser",
  payload: {
    device: "deviceName",
    properties: [
      {
         name: ...,
         value:
      },
      ...
    ]
 }
}

I have update RFC-1 accordingly, see 1e613cb

OK

I had to reopen the issue because after playing for some time with Tango to DF converter, I found out that there is no reasonable way to use tango-base actions outside of tango. I've found out that it is quite easy to move action to payload in magix-tango-connector. So I suggest that this change should be made in RFC.

This will also require changes in JS and Python clients


Update Risley Christmas visit slots

Additional visit slots:
24th 0900-1115 and 1400-1600
31st 0900-1115 and 1400-1600
Unbookable:

Christmas Day
Boxing Day
28th December
1st January

Please test locally

Tested locally


Page.setCookie() not working when value is mixed with characters of different types

Steps to reproduce
Tell us about your environment:

Puppeteer version<EMAIL_ADDRESS>as well as<EMAIL_ADDRESS>Platform / OS version: macOS
URLs (if applicable): Not necessary
Node.js version: v10.18.0

What steps will reproduce the problem?

Initialize puppeteer and get page object;
Set cookie with following codes:

const fakeCookieObjectThatWouldRaiseError = [{
	domain: '.google.com',
	name: 'nickname',
	value: ' 你好'
}];

const fakeCookieObjectThatWouldPass = [{
	domain: '.google.com',
	name: 'nickname',
	value: ' 你好'
}];

const fakeCookieObjectThatWouldPassToo = [{
	domain: '.google.com',
	name: 'nickname',
	value: 'English'
}];

try {
	await page.setCookie(...[fakeCookieObjectThatWouldRaiseError]);
} catch (e) {
	console.log(e);
}

API with https://pptr.dev/#?product=Puppeteer&version=v5.2.1&show=api-pagesetcookiecookies

What is the expected result?
setCookie could work and set Cookie correctly.
What happens instead?
The calling failed with following message:
Protocol error (Network.setCookies): Invalid cookie fields

Discussion
I could give several examples to explain the bug. Besides the example described above, we can see that the following value property with pure Chinese characters or pure English characters can both got the expected result.
So it's just a situation that when value is characters with space and Chinese characters mixed together.
const fakeCookieObjectThatWouldPass = {
	domain: '.google.com',
	name: 'nickname',
	value: ' 你好'
};

const fakeCookieObjectThatWouldPassToo = {
	domain: '.google.com',
	name: 'nickname',
	value: 'English'
};

try {
	// This is OK.
	await page.setCookie(fakeCookieObjectThatWouldPass, fakeCookieObjectThatWouldPassToo);
} catch (e) {
	console.log(e);
}

If it's designed to raise error at this situation, I think it should be added to unit test cases https://github.com/puppeteer/puppeteer/blob/main/test/cookies.spec.ts
Or we should make some judgements with user input parameters in value property, to forbid throw invalid error to users directly.
Otherwise, I can just guarantee expected result with encodeURIComponent operation once I test value property contains Chinese characters:
const getSafeCookieValue = (rawString: string): string => {
    if (/.*[\u4e00-\u9fa5]+.*/.test(rawString)) {
        return encodeURIComponent(rawString);
    } else {
        return rawString;
    }
};

I have a similar issue when trying to set a cookie containing binary data.
await page.setCookie({
  name: cookieName,
  value: new Buffer(cookieValueBase64, 'base64').toString()
});

Any suggestions how to supply this kind of cookies to the browser?

@dimatomp As I said in the issue, maybe you could try encodeURIComponent API to see if your code works.

Did not work for me - the buffer could be sent but was not parsed on the recipient side.
I ended up converting the buffer into hex representation and then just putting '%' in between - it worked :)
let urlEncodedCookie = '';
for (let i = 0; i < cookieHex.length; i += 2)
  urlEncodedCookie += `%${cookieHex.substr(i, 2)}`;
await browser.setCookies({
  name: cookieName,
  value: urlEncodedCookie,
});

I have the same issue when trying to set a PEM public key as a cookie value. It seems to not like the \ns

I have the same issue, latest npm install version, the cookie looks like this:
output of
        console.log(cookies, 'cookies');

is:
[
  {
    name: 'app_opened',
    value: 'j��@�W\rmWh�\x15\x100�[censored]$\x16\x00�\n�\x19�J2r\x0F�\t���',
    expires: 0,
    domain: 'app.[censored].com',
    path: '/'
  }
] cookies

@arthurwolf  try to add secure: true to the cookie object

@arthurwolf try to add secure: true to the cookie object

This works for me, Thanks .


Add support for Thermor Equateur 3 - AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint (io:AtlanticElectricalHeaterWithAdjustableTemperatureSetpointIOComponent)

[x] I have read the Readme, including the Advanced section regarding debugging.

Device information
Thermor "Equateur 3" electrical heaters, they are partially discovered (binary sensors: occupancy and contact, sensors: temperature and cumulative electric power consumption as well as some scenes to change target temperature).
But no climate entity created for that device.  I am using the Cozytouch Bridge (not the Tahoma Box).
(io:AtlanticElectricalHeaterWithAdjustableTemperatureSetpointIOComponent - HeatingSystem - AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint)
Device details
(io:AtlanticElectricalHeaterWithAdjustableTemperatureSetpointIOComponent - HeatingSystem - AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint)
`2021-01-13 14:20:35 DEBUG (MainThread) [custom_components.tahoma.coordinator] Initialized DataUpdateCoordinator with 0:00:30 interval.
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Finished fetching TaHoma Event Fetcher data in 0.109 seconds
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Unsupported TaHoma device detected (io:AtlanticElectricalHeaterWithAdjustableTemperatureSetpointIOComponent - HeatingSystem - AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint - io://0806-0588-9588/12338497#1)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:TemperatureInCelciusIOSystemDeviceSensor - TemperatureSensor - TemperatureSensor - io://0806-0588-9588/12338497#2)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:ContactIOSystemDeviceSensor - ContactSensor - ContactSensor - io://0806-0588-9588/12338497#3)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:OccupancyIOSystemDeviceSensor - OccupancySensor - OccupancySensor - io://0806-0588-9588/12338497#4)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:CumulatedElectricalEnergyConsumptionIOSystemDeviceSensor - ElectricitySensor - CumulativeElectricPowerConsumptionSensor - io://0806-0588-9588/12338497#5)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Unsupported TaHoma device detected (io:AtlanticElectricalHeaterWithAdjustableTemperatureSetpointIOComponent - HeatingSystem - AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint - io://0806-0588-9588/15461267#1)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:TemperatureInCelciusIOSystemDeviceSensor - TemperatureSensor - TemperatureSensor - io://0806-0588-9588/15461267#2)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:ContactIOSystemDeviceSensor - ContactSensor - ContactSensor - io://0806-0588-9588/15461267#3)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:OccupancyIOSystemDeviceSensor - OccupancySensor - OccupancySensor - io://0806-0588-9588/15461267#4)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:CumulatedElectricalEnergyConsumptionIOSystemDeviceSensor - ElectricitySensor - CumulativeElectricPowerConsumptionSensor - io://0806-0588-9588/15461267#5)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Unsupported TaHoma device detected (io:AtlanticElectricalHeaterWithAdjustableTemperatureSetpointIOComponent - HeatingSystem - AtlanticElectricalHeaterWithAdjustableTemperatureSetpoint - io://0806-0588-9588/8228744#1)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:TemperatureInCelciusIOSystemDeviceSensor - TemperatureSensor - TemperatureSensor - io://0806-0588-9588/8228744#2)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:ContactIOSystemDeviceSensor - ContactSensor - ContactSensor - io://0806-0588-9588/8228744#3)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:OccupancyIOSystemDeviceSensor - OccupancySensor - OccupancySensor - io://0806-0588-9588/8228744#4)
2021-01-13 14:20:36 DEBUG (MainThread) [custom_components.tahoma] Added TaHoma device (io:CumulatedElectricalEnergyConsumptionIOSystemDeviceSensor - ElectricitySensor - CumulativeElectricPowerConsumptionSensor - io://0806-0588-9588/8228744#5)
2021-01-13 14:21:06 DEBUG (MainThread) [custom_components.tahoma] Finished fetching TaHoma Event Fetcher data in 0.063 seconds
2021-01-13 14:21:36 DEBUG (MainThread) [custom_components.tahoma] Finished fetching TaHoma Event Fetcher data in 0.062 seconds

Thanks for reporting, @browetd! It looks like a duplicate of https://github.com/iMicknl/ha-tahoma/issues/304. I will try to see if I can add support for this device in the coming days.

Thanks for reporting, @browetd! It looks like a duplicate of https://github.com/iMicknl/ha-tahoma/issues/304. I will try to see if I can add support for this device in the coming days.

It would be great !  Thanks !

It would be great !  Thanks !

Tracked in #304.

Tracked in #304.


Created the basic job_application model.

closes #41

pytest check fail

yeah, I'm checking why is that. because I didn't change the recruiter and student files at all.

pytest check fail

done.

Squash the commits

Sine I see changes in the jobboard model and migrations files that are not explained in the pr description, I cannot safely merge this. Furthermore, I don't expect to see migration files being changed once they were merged.

I did not quite understand you.
Do you want me to add in the PR description exactly what I deleted in these files? General description of
deleting application from jobboarb is not enough?  Or is the problem that it appears in the commit and not in the main description?

For now the biggest issue seems to be that similar changes were done in another PR and you have a conflict

For now the biggest issue seems to be that similar changes were done in another PR and you have a conflict

yeah, I'm on it.


So which version to use for RGB20s?

Sorry for the inconvenience but I'm getting confused with so many versions and I don't know which one I have to download for RGB20s

None.  I don't officially support the RGB20s.  Powkiddy had their own version of ArkOS for the RGB20s.  If you want an OS that's officially supported, try UnofficialOS.  It's based off of JelOS.


Cannot connect to ssh server within Dev container: "no matching host key type found"

What happened?
I was attempting to run the Visual Studio demo
https://www.devspace.sh/docs/ide-integration/visual-studio-code#tldr
The following output was emitted in Visual Code
[18:07:00.313] Log Level: 2
[18:07:00.317]<EMAIL_ADDRESS>[18:07:00.317] linux x64
[18:07:00.382] SSH Resolver called for "ssh-remote+my-app.devspace", attempt 1
[18:07:00.383] "remote.SSH.useLocalServer": true
[18:07:00.383] "remote.SSH.path": undefined
[18:07:00.384] "remote.SSH.configFile": undefined
[18:07:00.384] "remote.SSH.useFlock": true
[18:07:00.384] "remote.SSH.lockfilesInTmp": false
[18:07:00.384] "remote.SSH.localServerDownload": auto
[18:07:00.384] "remote.SSH.remoteServerListenOnSocket": false
[18:07:00.384] "remote.SSH.showLoginTerminal": false
[18:07:00.385] "remote.SSH.defaultExtensions": []
[18:07:00.385] "remote.SSH.loglevel": 2
[18:07:00.385] "remote.SSH.enableDynamicForwarding": true
[18:07:00.385] "remote.SSH.enableRemoteCommand": false
[18:07:00.386] "remote.SSH.serverPickPortsFromRange": {}
[18:07:00.386] "remote.SSH.serverInstallPath": {}
[18:07:00.390] SSH Resolver called for host: my-app.devspace
[18:07:00.390] Setting up SSH remote "my-app.devspace"
[18:07:00.393] Acquiring local install lock: /tmp/vscode-remote-ssh-830fe282-install.lock
[18:07:00.395] Looking for existing server data file at /home/mark/.config/Code/User/globalStorage/ms-vscode-remote.remote-ssh/vscode-ssh-host-830fe282-6261075646f055b99068d3688932416f2346dd3b-0.92.0/data.json
[18:07:00.395] Using commit id "6261075646f055b99068d3688932416f2346dd3b" and quality "stable" for server
[18:07:00.398] Install and start server if needed
[18:07:00.400] PATH: /home/mark/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/snap/bin:/home/mark/.arkade/bin
[18:07:00.401] Checking ssh with "ssh -V"
[18:07:00.413] > OpenSSH_9.0p1 Ubuntu-1ubuntu7, OpenSSL 3.0.5 5 Jul 2022

[18:07:00.416] askpass server listening on /run/user/1000/vscode-ssh-askpass-1a8bc7dc310f347190d98ad2e0efccf6abb0b234.sock
[18:07:00.416] Spawning local server with {"serverId":1,"ipcHandlePath":"/run/user/1000/vscode-ssh-askpass-5d3ee3236e4d694a41359a1cc794011c6810bd11.sock","sshCommand":"ssh","sshArgs":["-v","-T","-D","46003","-o","ConnectTimeout=15","my-app.devspace"],"serverDataFolderName":".vscode-server","dataFilePath":"/home/mark/.config/Code/User/globalStorage/ms-vscode-remote.remote-ssh/vscode-ssh-host-830fe282-6261075646f055b99068d3688932416f2346dd3b-0.92.0/data.json"}
[18:07:00.416] Local server env: {"SSH_AUTH_SOCK":"/run/user/1000/keyring/ssh","SHELL":"/bin/bash","DISPLAY":":1","ELECTRON_RUN_AS_NODE":"1","SSH_ASKPASS":"/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/local-server/askpass.sh","VSCODE_SSH_ASKPASS_NODE":"/usr/share/code/code","VSCODE_SSH_ASKPASS_EXTRA_ARGS":"--ms-enable-electron-run-as-node","VSCODE_SSH_ASKPASS_MAIN":"/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/askpass-main.js","VSCODE_SSH_ASKPASS_HANDLE":"/run/user/1000/vscode-ssh-askpass-1a8bc7dc310f347190d98ad2e0efccf6abb0b234.sock"}
[18:07:00.423] Spawned 22365
[18:07:00.567] > local-server-1> Spawned ssh, pid=22374
[18:07:00.570] stderr> OpenSSH_9.0p1 Ubuntu-1ubuntu7, OpenSSL 3.0.5 5 Jul 2022
[18:07:00.581] stderr> Unable to negotiate with <IP_ADDRESS> port 10341: no matching host key type found. Their offer: ssh-rsa
[18:07:00.582] > local-server-1> ssh child died, shutting down
[18:07:00.589] Local server exit: 0
[18:07:00.589] Received install output: local-server-1> Spawned ssh, pid=22374
OpenSSH_9.0p1 Ubuntu-1ubuntu7, OpenSSL 3.0.5 5 Jul 2022
Unable to negotiate with <IP_ADDRESS> port 10341: no matching host key type found. Their offer: ssh-rsa
local-server-1> ssh child died, shutting down

[18:07:00.590] Failed to parse remote port from server output
[18:07:00.593] Resolver error: Error:
	at g.Create (/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:583821)
	at t.handleInstallOutput (/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:582473)
	at Object.e [as tryInstallWithLocalServer] (/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:623084)
	at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async /home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:642212
	at async t.withShowDetailsEvent (/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:645932)
	at async /home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:621556
	at async T (/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:618062)
	at async t.resolveWithLocalServer (/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:621171)
	at async t.resolve (/home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:643540)
	at async /home/mark/.vscode/extensions/ms-vscode-remote.remote-ssh-0.92.0/out/extension.js:1:722342
[18:07:00.598] ------

Specifically this line indicates a problem connecting the ssh server running within the container
[18:38:15.520] > local-server-1> Spawned ssh, pid=25714
[18:38:15.526] stderr> OpenSSH_9.0p1 Ubuntu-1ubuntu7, OpenSSL 3.0.5 5 Jul 2022
[18:38:15.541] stderr> Unable to negotiate with <IP_ADDRESS> port 11302: no matching host key type found. Their offer: ssh-rsa
[18:38:15.542] > local-server-1> ssh child died, shutting down

What did you expect to happen instead?
I expected so see coder connect via SSH without incident
How can we reproduce the bug? (as minimally and precisely as possible)
git clone https://github.com/loft-sh/devspace-vscode-example.git
cd devspace-vscode-example
devspace use namespace moc-test1
devspace dev

Local Environment:

DevSpace Version: 6.1.1
Operating System: linux (Ubuntu 22.10)
ARCH of the OS: AMD64

Kubernetes Cluster:

Cloud Provider: other (minikube)
Kubernetes Version: v1.25.3

Anything else we need to know?
$ code --version
1.73.1
6261075646f055b99068d3688932416f2346dd3b
x64

$ code --list-extensions
ms-vscode-remote.remote-ssh
ms-vscode-remote.remote-ssh-edit
ms-vscode.remote-explorer

Discovered a work-around.
Added "HostkeyAlgorithms" and "PubkeyAcceptedAlgorithms" configuration to the generated ssh configuration file
$ cat ~/.ssh/config
# DevSpace Start my-app.devspace
Host my-app.devspace
  HostName localhost
  LogLevel error
  Port 10620
  IdentityFile /home/mark/.devspace/ssh/id_devspace_rsa
  StrictHostKeyChecking no
  UserKnownHostsFile /dev/null
  User devspace
  HostkeyAlgorithms +ssh-rsa                     # Added
  PubkeyAcceptedAlgorithms +ssh-rsa       # Added
# DevSpace End my-app.devspace

Hey, @myspotontheweb.
Thanks for reporting this issue! We really appreciate it!
We are looking into it.
Are you satisfied with the workaround you have discovered?
Thanks!

We have updated our dependencies and this should be fixed with the next release of DevSpace


can't able to add two specific packages

Steps to Reproduce

can't able to add  two specific packages in the project it gives a lot of  error

packages are
flutter_downloader: ^1.1.7
admob_flutter: ^0.3.1
Logs

ms] > Task :app:transformDexArchiveWithExternalLibsDexMergerForDebug FAILED
[   +3 ms] D8: Program type already present: android.support.v4.app.INotificationSideChannel
[   +3 ms] 61 actionable tasks: 5 executed, 56 up-to-date
[        ] FAILURE: Build failed with an exception.
[   +1 ms] * What went wrong:
[        ] Execution failed for task ':app:transformDexArchiveWithExternalLibsDexMergerForDebug'.
[   +1 ms] > com.android.builder.dexing.DexArchiveMergerException: Error while merging dex archives:
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\2.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\3.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\4.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\5.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\6.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\7.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\8.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\9.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\10.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\11.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\12.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\13.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\14.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\15.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\16.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\17.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\18.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\19.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\20.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\21.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\22.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\23.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\24.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\25.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\26.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\27.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\28.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\29.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\30.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\31.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\32.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\33.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\34.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\35.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\36.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\37.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\38.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\39.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\40.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\41.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\42.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\43.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\44.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\45.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\46.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\47.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\48.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\49.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\50.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\51.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\52.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\53.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\54.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\55.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\56.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\57.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\58.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\59.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\60.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\61.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\62.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\63.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\64.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\65.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\66.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\67.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\68.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\69.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\70.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\71.jar,
C:\Users\Superman\Dev\Create\doc\bookconda\build\app\intermediates\transforms\dexBuilder\debug\72.jar
[  +15 ms]   Learn how to resolve the issue at https://developer.android.com/studio/build/dependencies#duplicate_classes.
[   +2 ms]   Program type already present: android.support.v4.app.INotificationSideChannel
[        ] * Try:
[        ] Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan
to get full insights.
[   +1 ms] * Get more help at https://help.gradle.org
[        ] BUILD FAILED in 15s
[ +437 ms] Running Gradle task 'assembleDebug'... (completed in 16.5s)
[   +1 ms] *******************************************************************************************
[        ] The Gradle failure may have been because of AndroidX incompatibilities in this Flutter app.
[        ] See https://goo.gl/CP92wY for more information on the problem and how to fix it.
[   +1 ms] *******************************************************************************************
[   +3 ms] "flutter run" took 24,753ms.
[        ] "flutter run" took 24,753ms.
Gradle task assembleDebug failed with exit code 1

#0      throwToolExit (package:flutter_tools/src/base/common.dart:24:3)
#1      _buildGradleProjectV2 (package:flutter_tools/src/android/gradle.dart:478:5)
<asynchronous suspension>
#2      buildGradleProject (package:flutter_tools/src/android/gradle.dart:346:14)
<asynchronous suspension>
#3      buildApk (package:flutter_tools/src/android/apk.dart:34:9)
<asynchronous suspension>
#4      AndroidDevice.startApp (package:flutter_tools/src/android/android_device.dart:377:13)
<asynchronous suspension>
#5      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:364:54)
<asynchronous suspension>
#6      HotRunner.run (package:flutter_tools/src/run_hot.dart:253:39)
<asynchronous suspension>
#7      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:428:37)
<asynchronous suspension>
#8      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:559:18)
#9      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:77:64)
#10     _rootRunUnary (dart:async/zone.dart:1132:38)
#11     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#12     _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#13     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#14     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#15     Future._complete (dart:async/future_impl.dart:473:7)
#16     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#17     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:28:18)
#18     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:294:13)
#19     RunCommand.usageValues (package:flutter_tools/src/commands/run.dart)
#20     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:77:64)
#21     _rootRunUnary (dart:async/zone.dart:1132:38)
#22     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#23     _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#24     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#25     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#26     Future._complete (dart:async/future_impl.dart:473:7)
#27     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#28     _AsyncAwaitCompleter.complete.<anonymous closure> (dart:async-patch/async_patch.dart:33:20)
#29     _rootRun (dart:async/zone.dart:1124:13)
#30     _CustomZone.run (dart:async/zone.dart:1021:19)
#31     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#32     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#33     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#34     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:115:13)
#35     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:5)

No issues found! (ran in 7.8s)

[√] Flutter (Channel stable, v1.5.4-hotfix.2, on Microsoft Windows [Version 10.0.17763.557], locale en-IN)
    • Flutter version 1.5.4-hotfix.2 at C:\src\flutter
    • Framework revision 7a4c33425d (7 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\Superman\AppData\Local\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = C:\Users\Superman\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[√] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 33.3.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] IntelliJ IDEA Ultimate Edition (version 2017.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2017.3.4
    • Flutter plugin version 29.0.1
    • Dart plugin version 173.4700

[√] VS Code, 64-bit edition (version 1.35.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.1.0

[√] Connected device (1 available)
    • Redmi Note 5 Pro • 93e9909b • android-arm64 • Android 8.1.0 (API 27)

• No issues found!

Did you try androidX migration?

Did you try androidX migration? https://goo.gl/CP92wY

I try but it is prompting  "No Usages Found in the Project"

I also try to manually migrate but nothing changes except now I am getting error some error I just imported two package nothing in the code just example pre-populated code try this if you want to see

I try to migrate with these packages and it gives me more error
''' cupertino_icons: ^0.1.2
html: ^0.13.3+3
http: ^0.12.0
bubble_bottom_bar: ^1.2.0
cached_network_image: ^0.6.0
auto_size_text: ^2.0.0+2
flutter_widget_from_html_core:
path_provider:
flutter_downloader:
scoped_model: ^1.0.1
flutter_pdfview: ^1.0.0+4
flutter_slidable: ^0.5.3
share: ^0.6.1+1
flutter_fluid_slider: ^1.0.1
admob_flutter:
'''

thank you @hemnathmouli  doing wrong in the process of  androidX migration  now i manage to do it thank you very much for your help

@ShashankSirmour I don't know why but that sounds like you're ironic.

@hemnathmouli  sorry if I said something wrong I.I just thanks you for advice as I am not considering androidX to create that much problem thank you again as of your suggestions I followed the process again and did it thank you again.


wiki

@ZeroKnight
i just want to say thank for what you did on wiki page.

Oh, sure thing! I was taking notes while I figured out how things worked and realized that they ended up being pretty comprehensive, so I decided to share them. Hopefully, I didn't get anything incorrect :)
Cheers!


test: Tests for OmniContainer

SUMMARY

Moving the OmniContainer.tsx file  to OmniContainer folder
Split into different files

TEST PLAN
No change in behavior, all tests must pass.
Notes: In the type of omnibar there are no props id or className. However, in the original javascript implementation, the prop className were being used. I checked that in execution both attributes were send to <input />.
So to be able to change the file to typescript and implement the tests I split Ominibar in another file, with the same interface that was being used in OminiContainer.
I marked component Omnibar as @deprecated and create a test to ensuring that the input will have the id that was sent...
To work correctly, I had to use the "// @ts-ignore" tag before the id attribute.
I believe that the correct fix for this is a PR in omnibar types or find some other lib that does the same and can meet the implementation requirements.

@ktmud If you can review this PR, I will appreciate it :-D

Codecov Report

Merging #13305 (87af48e) into master (29d6420) will decrease coverage by 14.76%.
The diff coverage is 85.00%.

@@             Coverage Diff             @@
##           master   #13305       +/-   ##
===========================================
- Coverage   77.20%   62.43%   -14.77%
===========================================
  Files         872      574      -298
  Lines       45101    20705    -24396
  Branches     5435     5441        +6
===========================================
- Hits        34820    12928    -21892
+ Misses      10158     7567     -2591
- Partials      123      210       +87

Flag
Coverage Δ

cypress
?

javascript
62.43% <85.00%> (+0.09%)
:arrow_up:

python
?

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

Impacted Files
Coverage Δ

...set-frontend/src/common/components/Modal/Modal.tsx
100.00% <ø> (ø)

...tend/src/components/OmniContainer/getDashboards.ts
37.50% <37.50%> (ø)

...et-frontend/src/components/OmniContainer/index.tsx
96.42% <96.42%> (ø)

...-frontend/src/components/OmniContainer/Omnibar.tsx
100.00% <100.00%> (ø)

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/reducers/index.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/filters/components/Time/types.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...et-frontend/src/dashboard/containers/Dashboard.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 509 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 29d6420...87af48e. Read the comment docs.

BTW, do we plan to keep investing in this OmniBar feature?  I think it's very useful and could have great potential. We may even insert it to all pages and let users search for any entities. (An alternative would be a global search bar that's always visible in the global nav).
cc @hughhhh @junlincc @willbarrett

With feature flag OMNIBAR turned on, it is supposed to show a dashboard list like in #6745 But I'm not seeing anything even though there are API requests fired.

@ktmud  To develop I removed the feature flag check and tested the bar (before doing this "refactor" and tests). It no shows anything, I thought it was a work in progress...
Can you check if when I did the refactor I accidentally inserted a bug? or if it was already like that on the master. (I don't know how to activate feature flags in this project rsrsrs)

It's broken in master, too.  cc @hughhhh
Here's the documentation on turning on/off feature flags:
https://superset.apache.org/docs/installation/configuring-superset#feature-flags

It's broken in master, too. cc @hughhhh

Do you think that in this case we can merge it anyway and create an issue to fix it?
As the component was split and changed a lot, a fix on the master (on the component as it was) may end up having to redo this job.

I think it's OK to merge once CI errors are fixed..

I think it's OK to merge once CI errors are fixed..

Done!

@yardz @ktmud I'm getting an npm conflict with this package. It only accepts<EMAIL_ADDRESS>and we're on v 16.14. The packages hasn't been updated in 2 years. Do you think we can remove this feature and add it back in at some point with better functionality? cc @yousoph

I'd vote for removing it for now since it's not working anyway...
cc @hughhhh


The body of a release is overwritten with the content of body_path or body

Prelude
Hi,
I'm using this action to create a release.
I provide a body to that release that contain the check sum of the released artifacts.
Those check sum aren't stable since we don't have reproducible build at the moment.
Using body_path
That body come from a generated file that is provided within the field body_path.
It appends that I run the workflow twice and the body wasn't updated of the second run.
It's currently configured like so:
      - name: Create release
        if: github.event_name == 'push'
        uses: softprops/action-gh-release@c9b46fe7aad9f02afd89b12450b780f52dacfb2d
        with:
          draft: true
          body_path: release-summary.md
          name: Release v${{ needs.version.outputs.full }}
          tag_name: v${{ needs.version.outputs.full }}
          files: foobar.md
          fail_on_unmatched_files: true
          generate_release_notes: false
        timeout-minutes: 5

Using body
I've also tried to use body instead of body_path with no luck, the body of the release is still not updated.
      - name: Create release
        if: github.event_name == 'push'
        uses: softprops/action-gh-release@c9b46fe7aad9f02afd89b12450b780f52dacfb2d
        with:
          draft: true
          body: ${{ steps.summary.outputs.output }}
          name: Release v${{ needs.version.outputs.full }}
          tag_name: v${{ needs.version.outputs.full }}
          files: foobar.md
          fail_on_unmatched_files: true
          generate_release_notes: false
        timeout-minutes: 5

What I'm expected
I would expect that the body is updated.

I've done some testing on my side, I've created a small repo for that https://github.com/FirelightFlagboy/action-gh-release-test


RB change objects need custom inspectors

The RB change objects need better inspectors to interact with them. For example, the composite changes should show a tree with the children, and method changes should show a difference view to the current source in the image.


ec2_elb_facts can cause API rate limit issues

there is no way to throttle the ansible module nor does the ansible module provide a way to back off/retry if an API limit is hit.  we should either:
a. make an update to the module so it does do back off/retry
b. re-write this task to not use this module.
using this module is a bit problematic for another reason:  #563

we should watch this: https://github.com/ansible/ansible/issues/13796

@coffeepac - just checking, did you want this ticket to remain open to track the long term resolution to #677 I believe the "fix:" text in the description caused this to automatically close on merge.

I do not want this to remain open.  We have another issue open to stop using this module entirely.


Update router documentation to include atc-router docs links

Summary
SUMMARY_GOES_HERE
Full changelog

[Implement ...]
[Add related tests]
...

Issues resolved
Fix #XXX

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Can't run add-on behind corporate proxy

Preflight Checklist

[X] I have installed the latest version of Power Query SDK.
[X] I have checked existing resources, including the common issues and the release notes.
[X] I have searched for similar issues.

Power Query SDK
0.2.2
Regression From
No response
Platform
Windows
Architecture
i86
OS Version
WINDOWS 10
VSCode version
1.77.0
PQSdkTool Path
No response
Bug Description
I have been unable to get the add-on to work within my corporate network and facing an error: Failed to update PowerQuery.sdkTools package due to unable to get issuer certificate.
I have adjusted the proxy settings in VSCode as per the below:
Proxy: http://[REDACTED]
Proxy Authorization: untouched
Proxy Strict All: unticked
Proxy Support: on
System Certificates: ticked
Any advice would be very much appreciated!
Steps to Reproduce
I have adjusted the proxy settings in VSCode as per the below:
Proxy: http://[REDACTED]
Proxy Authorization: untouched
Proxy Strict All: unticked
Proxy Support: on
System Certificates: ticked
Actual Experience
As I navigate to a project folder and select "Evaluate current file", I get the error message: Failed to update PowerQuery.SdkTools package due to unable to get issuer certificate.
Expected Experience
I would expect for the add-on to run and to build my custom connector.
Additional Context
Running behind a corporate proxy.

@albertli-msft we should honor the common VS Code proxy settings (http.proxy, http.proxySupport, and others) in the calls we make to nuget.

@pelicangris you might be able to workaround this problem using the nuget.exe command line tool.

Download nuget.exe from https://www.nuget.org/downloads and save it locally (i.e. c:\tools)
Configure the default proxy settings to use for nuget.exe (stackoverflow example, nuget reference)
Test nuget.exe from the command line to make sure the proxy settings are working
In VS Code, set the powerquery.sdk.externals.nugetPath configuration setting to the directory containing nuget.exe (c:\tools)

When you override powerquery.sdk.externals.nugetPath the tool will make nuget calls via nuget.exe, rather than calling out directly to the REST API. We made the direct REST calls our default behavior to make it easier to get started.

Just wanted to report that installing nuget and adjusting the nugetpath in the VSCode config did the trick. Thanks for the pointer!

Hi @pelicangris,
Thanks for using Power Query SDK,
Currently we only support use the http client to seize the nuget package behind the proxy w/o using nuget.exe.
I am using fiddler proxy here as an example:
Configure the setting item "http.proxy" following the pattern of  http://:@ like http://1:1@<IP_ADDRESS>:8888 in my local fiddler case.

And, also, leave the nuget.exe path blank:

Then the extension should be able to pick up the nuget package behind the proxy like in the video clip attached:
https://user-images.githubusercontent.com/69623692/235090116-44cdf3b1-cc4c-4f7e-b510-ed961b338565.mp4
The extension of version 0.2.3 is able to seize the nuget package behind my fiddle proxy server of basic auth.
btw, the extension would also respect the "http.proxyAuthorization" config item if you need it.
Have a good one. 😀


Change styles for basemap layer

Related to: #13091

cc @CartoDB/design

Super fast @rubenmoya, thanks!!
Looks pixel-perfect on the image but can we test it in staging? 🙏
<3

leaving it in a ded for acceptance

Amazing job guys, just tested and works like a charm ✨
❤BANDERA❤


Asynchronously updatable, centrally hosted CLI configuration

Problem
Once a release of the Tanzu CLI is installed, information that could affect the use or behavior of the CLI is not available until an explicit update of the CLI binary.
The only exception is the plugin inventory available in the central repository containing information about versions of plugins and plugin groups that are available to the user, because the CLI has been implemented to check for updates on and fetch new versions of the inventory.
Other types of information that could benefit from similar treatment include

feature flag activation/deactivation
general organization of the CLI command tree, changes to the provider of any specific command
population of default cli settings
whether there is a new version of the CLI
availability of a new EULA
general notices/tips (motd - message of the day)
tuning of behavior (such as timeouts) after telemetry gathered from the field
tuning of the type of telemetry information to capture
if any common endpoints (central repo, telemetry, TMC) needs to change
plugins could provide some information that the CLI would be coded to take into consideration
plugins could also benefit from being able to store their own information

Such information could be stored in the inventory DB, or in an artifact alongside it, in the central repository.  Using this approach enables us to leverage existing implementation as well as the documented procedure for supporting air-gapped/internet-restricted environments.
Desired Outcome
The CLI should be able to retrieve and persist a set of central configuration from the central repository (or any replaced repository prepped to contain it, e.g., for air-gap). Once persisted, any CLI or plugin functionality implemented to react to the introduction of or changes to this data will be able to adjust its behaviour based on the latest state.
Future Goals
Allowing plugins to use this new Central Configuration.

This has been completed, mostly by #700 and #707


Add Virtual dom support for dom manipulation in flutter-web or make it as performant as Angular

Flutter is really helpful for creating Cross-Platform apps, Dom manipulation seems much slower in flutter then React/Angular. Adding Virtual dom support can really boost Dom manipulation performance.

Hi @rajneeshraghav , Flutter framework only partially rerenders its graph so for the build part of
the build/layout/paint cycle, best suggestion is to setState at granular level. When you call setState it will mark everything up to the highest repaint boundary (see RepaintBoundary widget).
On the paint side of things, thank you for raising issue, when you paint the exact same Picture at the leaf level we have some optimizations but there is still a lot of ongoing work on our side to make things faster.
The most helpful thing for our team is to get snippets of real code, showing us where we have bottlenecks to optimize.
If you have some sample scenario with small repro case, please reopen issue with code. Closing this for now. Thank you for feedback!


added doubleTap zoom-in / out,  with animation

Description
added doubleTap zoom-in / out,  with animation

@nne998 Tony, could you give me push permissions to your fork? This post describes how to do that: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork That way I can share my changes with you and we can work on this PR together. Thanks!

While testing the double tap code, I came across this existing bug that has nothing to do with the new double tap stuff: https://github.com/thesmythgroup/easy_image_viewer/issues/27

@nne998 Tony, could you give me push permissions to your fork? This post describes how to do that: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork That way I can share my changes with you and we can work on this PR together. Thanks!

I enabled the option, please check if it's ok ?

@nne998 Thanks Tony, I've pushed my changes up. Would you be able to fix the code so that the existing unit tests pass and also write additional unit tests?

@nne998 I looked into this some more. The main issue is that InteractiveViewer and GestureDetector don't play nicely with one another. Wrapping an InteractiveViewer in a GestureDetector (or vice versa) results in a delay when trying to pinch & zoom. This is nicely shown here: https://github.com/flutter/flutter/issues/64156#issuecomment-1028213139
I really don't want to add such a delay to easy_image_viewer. A possible solution is to use Listener instead of GestureDetector. But that involves recognizing a double tap gesture ourselves (which is not that hard).

Thank you Tony, I can take another look next week. One more thing we'll need is a way to opt-in to this new behavior, the same way we can opt-in to "swipe to dismiss" using  swipeDismissible: true. Have a great weekend!

@nne998 Hey Tony, I've updated the PR to make the new behavior optional through the new doubleTapZoomable argument. Do you want to give the latest code a quick try? I think this should be ready to merge and to release. Thanks for your help on this!


Automate the addition of keywords to show notes frontmatter

https://gohugo.io/content-management/front-matter/#keywords

9838425


fix(DIST-269): Fix autofocus of full-screen form in Safari

Bug DIST-269 in Safari is solved by code inspired by #53 from @prilutskiy

[BOT] Preview available: https://embed.typeform.com/c104e786b8914d5248fd8c82c8820bcc8af52ad2/embed.js

[BOT] Preview available: https://embed.typeform.com/6635c4e259c3a57fe5a8b6316980aac060c352a0/embed.js

:tada: This PR is included in version 0.23.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

This worked on localhost, but it does not work when using embed lib from AWS.


Star and unstar messages; show starred status

This PR is built on top of #446 .
The alignment of the star marker is currently only ideal for default text (see screenshots at bottom). Unfortunately some markup introduce large margins, so the alignment is not attractive for a message that starts with a header:

Ideally the top margins of messages are collapsed, but that is outside the scope of this PR.
In this PR the star is offset from the top by 4px, which more or less matches visually what the Figma at https://www.figma.com/file/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=813%3A28811&mode=dev tries to achieve (the star shape centered with the first line of text), but will need to be readjusted (in addition to the margin fix) when we match the default text style to the guide (we currently use 14px and the mockups use 17).
Fixes: #170

@gnprice ready for review again!

Thanks for the revision! All looks good to me now modulo a small tweak: https://github.com/zulip/zulip-flutter/pull/449#discussion_r1462827049
Otherwise this is just pending #446 getting finished and merged.

@gnprice thanks for the review! comments have been addressed and I noticed there weren't tests for the visibility of the icon in the message so I added those as well.

Cool, thanks. All LGTM now pending #446.

#446 has been merged; rebasing this and merging.


Sm/isogit-perf

What does this PR do?
perf improvements from changes in iso-git
What issues does this PR fix or reference?
@W-10839310@
https://github.com/isomorphic-git/isomorphic-git/pull/1514
https://github.com/forcedotcom/cli/issues/1427

QA:
with EDA project mentioned in issue
toolbelt:
sfdx force:source:tracking:reset -p  3.89s user 2.51s system 104% cpu 6.111 total

initial performance:
 ➜  time sfdx force:source:beta:tracking:reset -p
Reset local tracking files.
sfdx force:source:beta:tracking:reset -p  241.25s user 19.92s system 105% cpu 4:06.70 total

linked through plugin-source:
 ➜  time sfdx force:source:beta:tracking:reset -p
Reset local tracking files.
sfdx force:source:beta:tracking:reset -p  14.46s user 5.33s system 156% cpu 12.651 total

dreamhouse-lwc
toolbelt
sfdx force:source:tracking:reset -p  0.86s user 0.31s system 72% cpu 1.600 total

initial
sfdx force:source:beta:tracking:reset -p  3.65s user 1.10s system 128% cpu 3.708 total

linked through plugin-source
sfdx force:source:beta:tracking:reset -p  7.95s user 2.50s system 161% cpu 6.473 total

All were slower than TB's implementation, and the "improved" version was slower in dreamhouse, but markedly faster for larger projects


Cmake's MSVC target doesn't define USE_MKL or expose an option

Here's the place for your question, suggestion, a feature request or brief
description of the problem. If you are submitting a defect report please fill
the sections below. For everything else feel free to remove everything below
the line.

Environment
Intel MKL-DNN includes hardware-specific optimizations and may behave
differently on depending on the compiler and build environment. Include
the following information to help reproduce the issue:

CPU make and model (try lscpu)
OS version (uname -a)
Compiler version (gcc --version)
MKLROOT value (echo MKLROOT=$MKLROOT)
CMake version (cmake --version)
CMake output log

Steps to reproduce
Use cmake to generate visual studio projects.
Actual behavior
USE_MKL is not defined.
Expected behavior
USE_MKL should be defined or at least there should be an option.

Hi @TechnikEmpire,
Could you please elaborate on the issue?
Cmake should check whether Intel MKL is available and if so it defines USE_MKL by itself. See this line.
Do you set MKLROOT when you call cmake?
Also if you download small Intel MKL-ML into external/ directory cmake should be able to find the library as well.

@mfomenk defines it where exactly, just in case I'm missing something. Sorry in advance if I've raised a false alarm, but for me it was not defined despite cmake finding MKL. I had to see it as a preprocessor variable manually but again I could have goofed something up here so let me know where I should have found that definition.

It seems that I miss something... so sorry for the repetition.
According to MKL.cmake if cmake finds MKL then it adds -DUSE_MKL to the compilation flags (see this link). In theory you should be able to find this definition in the compile output of MSVS or just on the screen if you cmd / msbuild.
Could you please dump the output of CMake and the output of how build goes?

I'm using cmake to generate visual studio 2017 project files. There's no USE_MKL defined in the preprocessor section of the project files. That was my point but I'll check to see if it gets added into the command line somewhere else.

Any updates?

@emfomenk After we merge 260 I'll reset and look at this again.

@TechnikEmpire,
Any news on this issue?

Will close due to lack of activity.
Please feel free to reopen.

@emfomenk Sorry for the long delay. This probably was never an issue, and I can confirm as of master (88c4882) that this is not an issue. When configured correctly from CMake, USE_MKL and friends are set properly.

Great!
Thanks for letting me know :)


registry key HKEY_LOCAL_MACHINE\SOFTWARE$77config run as administrator ?

The configuration is located in HKEY_LOCAL_MACHINE\SOFTWARE$77config and is writable by any process without elevated privileges. The DACL of this key is set to grant full access to any user.？
13:59:08 Failed to create registry key HKEY_LOCAL_MACHINE\SOFTWARE\$77config Try to run as administrator

Failed to create - by whom?
This registry key can only be created with administrator privileges, which is done by r77. Then, the DACL is set up properly.
So, you need to actually install r77, not just inject several processes. Then you have this key all set up.


Add cellular status to telem server and telem

add MAVSDK/build/default/src/mavsdk_server/src/mavsdk_server

@potaito
do you see any issues with this?
the mavsdk_server generated works with sitl but not after flashing .px4 on fmu (could be unrelated)


New build is not Windows friendly

Select one ...  (check one with "x")
[ ] bug
[ ] feature request
[x] enhancement

Expected behavior

"npm run build" should be error-free on Windows.
Actual behavior

"npm run build" fails on Windows. "del ./dist" and the cpy npm tasks are troulbesome on Windows. Running on Linux is fine though.
Reproduction of behavior

Include a link to the reproduction scenario you created by forking one of the Clarity Plunker Templates:

Environment details

Angular version: 4.x.x

Clarity version: master https://github.com/vmware/clarity/commit/eca43905caf18cdcec3f0a81d61fd370051d9c80

OS and version:
Windows 7

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

@jeeyun Could you please take a look at this?

I'll take a look at it.

Turns out del and cpy don’t play nice with windows
https://github.com/sindresorhus/cpy-cli/issues/9
I can get del-cli to work but nothing doing for cpy. If we need to support windows (I think yeah, even though I don’t want to) it seems like we will need to change the delete/copy tools commands for our build. From the author:
I couldn't care less about Windows, but feel free to change it if you do.

How about using shx?

@chunghha - that looks interesting. Thanks!
I'll take a look and see it it works a bit more seamlessly for *nix && Windows.

Ran into issues with many things on my windows VM. The related thing to his issue is that the replace command we are using seems have a problem with $npm_package_version - it seems windows wants %npm_package_version% but I can't confirm that for sure as I'm getting an error that is quite troublsome to debug: cp: copyFileSync: could not write to dest file (code=ENOENT)` .
A possible solution might be to use crossvar
I'm going to move this to the backlog for now and mark it as contributions welcome for someone who has access to an ideal Windows dev setup for node.

So far this still works on Linux, will test on Windows.
https://gist.github.com/anonymous/99cf282d9573d9ec423a0be260896703

The above error is from Windows 10/node 8.7.0. But, from Windows7/node 6.11.4, the above error disappears. Just mention that here for record now. Since I tested with node 8.7.0 on my Linux, I am not sure node version is related to the error during webpack on Windows 10.
With Windows7/node 6.11.4, one error I am getting is from this code in clarify-icons-svg.js
SHAPE_SETS.forEach((setName) => {
        shell.exec(`cd dist/clarity-icons/shapes; zip -r ${setName}.zip ./${setName}/*; rm -r ./${setName}`);
    });

The error is "The system cannot find the path specified." 7 times so taking care of the cd part, then error say

'zip' is not recognized as an internal or external command

so basically, need to re-write the clarify-icon-svg.js to make it seamlessly work on Windows.

@hippee-lee when you have time, please take a look at this.
https://gist.github.com/chunghha/5cc00f1021d2aaf26277ce2667c4095e
This is a working version on my Windows7/node 6.11.4 along with the shx/cross-conf-env/replace-in-file changes above. I haven't fully verified contents of all-shapes.zip are matched with the one from Linux but svg icons in other shape zip files looked OK.
When I can get a chance (probably this weekend), I will create a branch on my fork repo and do more tests with Windows 7/10 and Linux.

Thank you for continuing to look into this! What I see in your gist looks like a viable approach.
If you want, feel free to submit a PR with your suggested changes. Then, I can pull it on my end and test it out my Windows VM.

Will this be merged anytime soon?

Hi @vinayakpatil -
I'm really sorry but I haven't had time to come back to this. I'll update the PR this issue is linked to with feedback this week.

Any updates on this?

Actually, we just merged a revamp of our build tooling. It uses the Angular CLI for most of the work so it might do better on Windows. Would you mind testing against master and if you have issues report back?

I'm assuming that the new build tooling addressed this.


Another approach to Math.abs()

hi I like your code for ReverseInteger. Why do you use the Math.abs()?  Here is another approach with less code:
public int reverse(int x) {
    int rev = 0;
    while(x != 0){
        rev = rev*10 + x%10;
        x = x/10;
    }

    return rev;
}

hi Nice Solution, did you test x = -12 and x=<PHONE_NUMBER> cases?


fix(hover): check what editor the request was sent from

This fixes the multiple hovers issue @bryphe noted in #1973. The request now takes an editor, and the view checks that it is the same when displaying.

/azp run

Just realized that the same thing would happen for signature help, so I tested it and it does. I'll fix that in this PR too


Disable linter prefer-regex-exec

Summary
The autofix from String.match to RegExp.exec may introduce bugs when the linter cannot correctly infer the flags of the expression to ensure it remains stateless
Example where the flag is not inferred:
export class Foo
    private static VALIDATION_REGEX = /something/g; // Should be inferred as statefull

    public validate(value: string): void {
        // This match should be replaced by `.exec` since it will make consecutive calls to this method break,
        // but the linter cannot infer this correctly and does the change automatically
        if (value.match(Foo.VALIDATION_REGEX) == null) {
            // So something with instance variables
        }
    }
}

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Thank you!


How to use this in Ros Melodic？

Hi！Thanks for your awsome contribution.
if i want to compile and use this code in ubuntu 18.04&Ros Melodic，should i change something？
hoping your reply！
こんにちは！あなたの素晴らしい貢献に感謝します。
このコードをubuntu18.04Ros Melodicでコンパイルして使用したい場合、何かを変更する必要がありますか？
お返事をお待ちしております！

Thank you for your comment.
I only have Ubuntu 20, but I ported it to ROS1. In my experience, Noetic and Melodic are compatible.
There are probably still some bugs, but please feel free to make additional comments when you do.
https://github.com/Ar-Ray-code/YOLOX-ROS/tree/noetic

Thanks a lot! I try this branch soon and then update the issue.
Best wishes！

If you have any other problems, please make a new issue.

Thanks a lot! I try this branch soon and then update the issue.
Best wishes！

Hi feelingXj， have you  try  it  on Melodic？


Add Err::<(&T, ErrorKind)>::to_owned for T=str and [u8]

Hi.
Almost all combinators return Err<(&str, ErrorKind)> or Err<(&[u8], ErrorKind)>. I added an implementation of Err::to_owned for &str and &u8 to cast Err<(&str, ErrorKind)> to Err<(String, ErrorKind)> and Err<(&[u8], ErrorKind)> to Err<(Vec<u8>, ErrorKind)>

Coverage increased (+0.3%) to 86.604% when pulling e6b0d3e26a7204358e24c5db565ba8cf245ca7d5 on kpp:err_to_owned into 851706460a9311f7bbae8e9b7ee497c7188df0a3 on Geal:master.

that will be useful, thanks!


Update SITCOMTN-010.tex

Hi Keith,
Just spotted a minor until chapter 3.1.
Best,
Holger

Thank you for the catch!


Unsolicited infinite freezes

I've long been wondering (a few months) why my ST3 was randomly freezing, indefinitely unresponsive, and eventually have to manually quit out via Task Manager. I finally had enough and chose to track down the issue by systematically disabling and re-enabling my packages one by one. It seems as though this is the only package where this behaviour with ST3 occurs.
I don't think it's completely random. I've noticed that most often, it happens when I close a tab, either with an MMB click or CTRL+W.
My setup is as follows:

Windows 10 Pro (Update 20H2)
Sublime Text 3 v3211

I usually have 3+ different project folders added to the project pane, if that's of any relevance

Packages

SFTP
SublimeLinter (with SublimeLinter-eslint)

I love this package to bits and I want to do more to fix this. If you have any idea about any logs I can submit that would help, or anything else for that matter, please let me know.

Not sure where this is coming about. I've been using it recently in a project and haven't gotten any issues while closing tabs. You can open the console with ctrl+ `, or open Discord's console with the instructions below, not too sure how to fix this if there is a problem. Might be something holding up the IPC.

ctrl + shift + i to open Discord's Console (option + cmd + i on mac)

I could see the following on_close event handler causing at least smaller freezes since it is the only blocking call, but I wouldn't expect it to freeze indefinitely.
https://github.com/Snazzah/SublimeDiscordRP/blob/61755f229c8b1acfca6fedb3e70a9f1acfacc420/drp.py#L333-L339
@Snazzah, when is the condition expected to trigger? When all views of a window have been closed?

@FichteFoll Yes, but I need to fix this. At the moment, it just goes back to the idle presence when it is closed.

I've been using 66d30eddbd595ff6e1e6ce6e67df31c14c440475 since the last comment and I have not gotten any freezes as described.


Add support for SENTRY_RELEASE

ref: https://github.com/getsentry/sentry-php/issues/804

One nit. Other than that LGTM.
thanks for contributing!

Thanks @mtsmfm !

@mtsmfm FYI we'll make a release on Monday morning UTC+2.


Silent data drop failure

Just this with integer instead of bigint
SELECT count(*) AS impressions,
avg( (jstimestamp::bigint/1000) - (connectStart::bigint/1000) ) AS avg_time,
min( (jstimestamp::bigint/1000) - (connectStart::bigint/1000) ) AS min_time,
max( (jstimestamp::bigint/1000) - (connectStart::bigint/1000) ) AS max_time,
percentile_cont(0.99) WITHIN GROUP (ORDER BY ( (jstimestamp::bigint/1000) - (connectStart::bigint/1000) )) AS p99_time
FROM js_stream;
then insert values like<PHONE_NUMBER>423,999123123123123 which I just randomly picked to test out by hand and couldn't figure out why it wasn't working.
via @efrazier

This isn't going to be possible without significant rework because the error happens in the stream scan, not the insert process. The error is stilled logged in the logs, which isn't ideal but at least it's not completely silent. If this issue begins to comes up frequently, we can revisit.


Tor connections

Hello,
Kmd coin is privacy coin!
it's good idea to add in settings : tor connections ?
Regards

I'm not sure marketmaker will work very well over Tor due to requiring it's own ports on a public IP.
@jl777?

no idea, never used tor

We have decided not to support this. It just won't ever be feasible.

Marketmaker makes a lot of connections, this would be very slow because of the way Tor works.
We would be a big burden on the Tor network because we call both marketmaker and various network services on a frequent interval.

Marketmaker does not use TLS for Electrum servers so the exit nodes could mess up your trades, and even potentially reverse trades if your are trading with the entity that controls the Tor node. Which isn’t as unlikely as it sounds if they can see all your traffic, they could spin up Bobs with low prices for coins they can see you are interested in. - @lukechilds


Server setup

I set up the web server with certain security functionalities and the ability to render dynamic web pages.


Separate Spawnpoints per Team

This is a request to have Spread Players be a toggleable option in the lobby. My group prefers to be separated for our bingo games and while the vanilla command below doesn't completely prevent running into other players it does a good enough job that I don't think doing something custom to spread us further is needed.
I've been doing that by manually running the following command just after the countdown finishes (would probably be better before the countdown starts, but I didn't want to risk interfering with the countdown).
This spreads players out in a 1000 block radius and respects teams.
spreadplayers ~ ~ 1000 1000 true @p
https://minecraft.fandom.com/wiki/Commands/spreadplayers
You'd probably want to add your predicates to distinguish between in-game and spectators

Yep, I think something similar has been suggested before. It doesn't look like it has made it's way to github though, or at least I cannot find it.
The main issue I have with the approach you are suggesting is that it renders the skybox effectively useless. If this was implemented, there should be some way to preview each team's spawnpoint.
Since the teams are created after pre-generation starts, this would cause some issues as there is no way of knowing how many teams will be needed, and just always generating 16 spawnpoints would be quite wasteful and lengthen pre-generation significantly.
So, ideally we would initially just generate 1 skybox and only when someone joins a second team, we would generate an additional one if the setting is active and teleport all players to the correct skybox once it's ready.
This would btw also have the additional advantage that the marking functionality of item frames becomes more useful as each team could be using a local version.
Some considerations and edge cases:

Cycling through cards from the skybox after teams have already been split should generate multiple skyboxes immediately and teleport each team once their respective skybox is ready
When joining a game from the lobby, you are currently assigned a random team. With this setting active, it should probably only pick existing teams to avoid generating another skybox unnecessarily.
When a team is no longer used, we would ideally free up the skybox and re-use in case another team is created, even if it has a different color.

This could be skybox #3, while 4 skyboxes exist.

One open question:
I'm not really sure how the compass and coordinate display should work. Currently, you get teleported to very high coordinates to implement random spawn points, and the displayed coordinates are relative to your spawnpoint.
For cross-team communication it would be confusing. But most likely, it would make sense to just display coordinates relative to the team's spawn point.
This would also mean, for each new team we could just generate a completely new spawn point, which makes it effectively impossible, or at least extremely low odds, for teams to run into each other.
Finally, I think no spread should still be the default, mostly because it seems the most fair as everyone gets the exact same starting conditions.

(I think I may have changed the original idea slightly, as it doesn't seem to keep teams together. There is an option to spread by team in spreadplayers though, and I don't think it would make much sense to spread all players over spreading teams).

Yeah I guess if players want that overview from the skybox that would complicate things. For my group in particular we don't mind a blind spawn location, but I know that won't apply to everyone. We mostly play blind mode so I think of not knowing what spawn looks like an extension of that mode.
I know in other datapacks I've played the card and team selection are done in the lobby, and they essentially created a skybox for each team when the game started. Waiting to create those skyboxes and pre-generate until the game starts would eliminate some of the issues you've laid out (along with cutting down on world file size since currently if you keep randomizing the cards it's saving a lot of unused region files). That would probably require a big rewrite or reshuffle though. I agree that making this opt-in is the way to go.
But yeah we're fine with just running a spreadplayers command for our purposes in the meantime. I'm happy to provide feedback if you come up with any other ideas for this.

Giving a heads-up overview is pretty nice for some of the target groups. Selecting teams from the lobby and only showing spawn at the very end is what version 5.0 used to do, and it's certainly easier to implement.
Reverting back to the old behavior would be a step backwards imo for a - from my pov - relatively niche feature.


train.py not accepting sample rates other than 16000 | ValueError: could not broadcast input array from shape (1,32000) into shape (1,16000)

After fixing/bypassing issue #14, python train.py seems to dislike sample rates other than 16000:
Traceback (most recent call last):
  File "train.py", line 114, in <module>
    train(args)
  File "train.py", line 96, in train
    callbacks=[csv_logger, cp])
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training.py", line 819, in fit
    use_multiprocessing=use_multiprocessing)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 235, in fit
    use_multiprocessing=use_multiprocessing)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 593, in _process_training_inputs
    use_multiprocessing=use_multiprocessing)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 706, in _process_inputs
    use_multiprocessing=use_multiprocessing)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/data_adapter.py", line 952, in __init__
    **kwargs)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/data_adapter.py", line 747, in __init__
    peek, x = self._peek_and_restore(x)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/data_adapter.py", line 956, in _peek_and_restore
    return x[0], x
  File "train.py", line 46, in __getitem__
    X[i,] = wav.reshape(1, -1)
ValueError: could not broadcast input array from shape (1,32000) into shape (1,16000)

Issue solved by using the latest master branch.

python train.py complains after the 1st epoch:
2020-07-08 06:54:04.006436: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libnvinfer.so.6
2020-07-08 06:54:04.007359: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libnvinfer_plugin.so.6
2020-07-08 06:54:05.018369: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcuda.so.1
2020-07-08 06:54:05.022380: E tensorflow/stream_executor/cuda/cuda_driver.cc:351] failed call to cuInit: CUDA_ERROR_COMPAT_NOT_SUPPORTED_ON_DEVICE: forward compatibility was attempted on non supported HW
2020-07-08 06:54:05.022401: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:169] retrieving CUDA diagnostic information for host: myuser
2020-07-08 06:54:05.022407: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:176] hostname: myuser
2020-07-08 06:54:05.022506: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:200] libcuda reported version is: 450.51.5
2020-07-08 06:54:05.022525: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:204] kernel reported version is: 440.100.0
2020-07-08 06:54:05.022531: E tensorflow/stream_executor/cuda/cuda_diagnostics.cc:313] kernel version 440.100.0 does not match DSO version 450.51.5 -- cannot find working devices in this configuration
2020-07-08 06:54:05.022746: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2020-07-08 06:54:05.027262: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2020-07-08 06:54:05.027662: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x56528f60c990 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-07-08 06:54:05.027675: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/librosa/filters.py:196: FutureWarning: norm=1 behavior will change in librosa 0.8.0. To maintain forward compatibility, use norm='slaney' instead.
  FutureWarning)
WARNING:tensorflow:sample_weight modes were coerced from
  ...
    to
  ['...']
WARNING:tensorflow:sample_weight modes were coerced from
  ...
    to
  ['...']
Train for 94 steps, validate for 10 steps
Epoch 1/30
93/94 [============================>.] - ETA: 0s - loss: 2.7065 - accuracy: 0.20032020-07-08 06:54:14.433587: W tensorflow/core/common_runtime/base_collective_executor.cc:217] BaseCollectiveExecutor::StartAbort Invalid argument: logits and labels must be broadcastable: logits_size=[16,21] labels_size=[16,20]
	 [[{{node loss/softmax_loss/softmax_cross_entropy_with_logits}}]]
WARNING:tensorflow:Can save best model only with val_loss available, skipping.
94/94 [==============================] - 8s 85ms/step - loss: 2.7012 - accuracy: 0.2015
Traceback (most recent call last):
  File "train.py", line 118, in <module>
    train(args)
  File "train.py", line 100, in train
    callbacks=[csv_logger, cp])
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training.py", line 819, in fit
    use_multiprocessing=use_multiprocessing)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 395, in fit
    total_epochs=1)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 128, in run_one_epoch
    batch_outs = execution_function(iterator)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 98, in execution_function
    distributed_function(input_fn))
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/eager/def_function.py", line 568, in __call__
    result = self._call(*args, **kwds)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/eager/def_function.py", line 638, in _call
    return self._concrete_stateful_fn._filtered_call(canon_args, canon_kwds)  # pylint: disable=protected-access
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/eager/function.py", line 1611, in _filtered_call
    self.captured_inputs)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/eager/function.py", line 1692, in _call_flat
    ctx, args, cancellation_manager=cancellation_manager))
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/eager/function.py", line 545, in call
    ctx=ctx)
  File "/home/myuser/anaconda3/envs/audio/lib/python3.7/site-packages/tensorflow_core/python/eager/execute.py", line 67, in quick_execute
    six.raise_from(core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError:  logits and labels must be broadcastable: logits_size=[16,21] labels_size=[16,20]
	 [[node loss/softmax_loss/softmax_cross_entropy_with_logits (defined at train.py:100) ]] [Op:__inference_distributed_function_9095]

Function call stack:
distributed_function

Do you have any suggestions?

check len(args.src_root) and len(set(label_train))
They should be the same.
I guess a class must be missing from label_train. Basically your model thinks there should be 21 classes but the labels are being produced as if there are 20.

Easy solution would be to change on lines 89 and 91
len(set(label_train)) to params['N_CLASSES']
But this doesn't ensure all classes are present in the training data. I'll add a warning for it this weekend.
At least I think this is the issue you are having.

I made a warning to check for cases where not all classes are present in the train / val splits. Also the classes are set from the true n_classes present in src.root. This should fix your problem, but I'm not 100% sure.

Closing since this should be fixed now. If you're getting a warning, consider increasing data size, so all class labels are present in the training split.

I am using the latest train.py but it still doesn't accept sample rates other than 16000.
Traceback (most recent call last):
File "train.py", line 120, in
train(args)
File "train.py", line 103, in train
callbacks=[csv_logger, cp])
File "C:\Users\solab\anaconda3\envs\AudioClass\lib\site-packages\tensorflow\python\keras\engine\training.py", line 108, in _method_wrapper
return method(self, *args, **kwargs)
File "C:\Users\solab\anaconda3\envs\AudioClass\lib\site-packages\tensorflow\python\keras\engine\training.py", line 1063, in fit
steps_per_execution=self._steps_per_execution)
File "C:\Users\solab\anaconda3\envs\AudioClass\lib\site-packages\tensorflow\python\keras\engine\data_adapter.py", line 1117, in init
model=model)
File "C:\Users\solab\anaconda3\envs\AudioClass\lib\site-packages\tensorflow\python\keras\engine\data_adapter.py", line 916, in init
**kwargs)
File "C:\Users\solab\anaconda3\envs\AudioClass\lib\site-packages\tensorflow\python\keras\engine\data_adapter.py", line 786, in init
peek, x = self._peek_and_restore(x)
File "C:\Users\solab\anaconda3\envs\AudioClass\lib\site-packages\tensorflow\python\keras\engine\data_adapter.py", line 920, in _peek_and_restore
return x[0], x
File "train.py", line 47, in getitem
X[i,] = wav.reshape(-1, 1)
ValueError: could not broadcast input array from shape (352800,1) into shape (16000,1)``
Any suggestions?

I am still having this issue with the new code:
This is my error code:
ValueError: could not broadcast input array from shape (352800,1) into shape (16000,1)
This is the error code from Ma5onic:
ValueError: could not broadcast input array from shape (1,32000) into shape (1,16000)
Do you have any idea why this happens?


support completion ~uname without slash

this enhances the #86

However maybe it's better to complete uname after type ~. And add / after typed ~uname.

Just like you said, ~ without a slash should complete username instead.


can not use "postcss-nested" ?

@Hilbertangers Can I have your webpack config file to reappear the issue ?

@titancat

@Hilbertangers I have reappeared this issue, I need some time to fix it.
ps: Your PR is welcome

Sorry, I can't give pr because I haven't written a postcss plugin, but I think you need to delete the @callFn statement, maybe you can get inspiration from postcss-functions

@Hilbertangers
You're right.😁
I made some tests a while ago, the reason for the issue is that postcss-nested does not recognize @callFn keyword.
Maybe I should try another implementation.Thanks for your advice

Have you tried to define postcss-define-function before postcss-nested?

@Hilbertangers Please try the method mentioned by @equinusocio ?


Is there a way to remove empty directories recursively?

Is there a way to remove empty directories recursively?

Please try etcdctl rmdir command (https://github.com/coreos/etcd/tree/master/etcdctl#deleting-a-key).
rmdir is removes the key if it is an empty directory or a key-value pair

i was thinking of something like lets say
key:                                value:
/path
/path/ta
/path/ta/key                        test
/path/to
/path/to/key                        test
/path/towards
/path/towards/key                   test

$ etcdctl rm /path/ta/key
$ etcdctl rm /path/to/key
$ etcdctl ls --recursive

/path
/path/ta
/path/to
/path/towards
/path/towards/key

$ etcdctl <somecommand> /path

where <somecommand> will delete both /path/to and /path/ta since these are empty dirs. I'm basically looking for a command to clean up empty folders.

If you want to delete recursively you can do etcdctl rm /path/to/dir --recursive.
But if you want to delete only empty ones, you have to do manually.

Then I have to do manually. I see, thanks!

No problem. Please ask us if you have any questions. Thanks.


Update foundation.tooltip.js to enable to parse a postionClass when c…

Before submitting a pull request, make sure it's targeting the right branch:

For documentation fixes, use master.
For bug fixes, use develop.
For new features, use the branch for the next minor version, which will be formatted v6.x.

If you're fixing a JavaScript issue, it would help to create a new test case under the folder test/visual/ that recreates the issue and show's that it's been fixed. Run npm test to compile the testing folder.
Happy coding! :)
…reating Tooltips using js
Changed code so the positionClass checks inside this.options rather than just looking at the elements css

Seems reasonable to me


Fix paths when moving contents of  to body fragment

Also un-inline scripts that were auto-inlined by hydrolysis
Fixes #332
Fixes #319

LGTM


Cannot clone object '<shapicant._pandas_selector.PandasSelector object

I am trying to use shapicant as a feature selector within a sklearn pipeline as follows:
explainer_type = shap.TreeExplainer
shap = PandasSelector(clf_dim, explainer_type, n_iter=20, random_state=0)

pipe_dim = Pipeline(
    steps=[
        ('enc', encode),
        ('dim', shap),
        ('clf', clf),
    ],
)

However, when trying to fit the pipeline, I get the error:

TypeError: Cannot clone object '<shapicant._pandas_selector.PandasSelector object at 0x00000158C66CD190>' (type <class 'shapicant._pandas_selector.PandasSelector'>): it does not seem to be a scikit-learn estimator as it does not implement a 'get_params' method.

and therefore can't use it in the pipeline, unfortunately,
Is this something that can be fixed, perhaps by inheriting from the sklearn class TransformerMixin?

Hi @RNarayan73, sorry for the late response.
shapicant is not designed as a scikit-learn transformer, so it is not currently possible to use it within a scikit-learn pipeline. One reason is that scikit-learn transformers usually return data as numpy array, whereas shapicant is designed to work on DataFrame structures (although in the latest versions of scikit-learn some functions are now able to return a pandas DataFrame).

Hi @manuel-calzolari wondering if this is now compatible with scikit-learn 1.2?

@manuel-calzolari
SKLearn 1.2 now offers the ability to retrieve feature names from transformers.
Perhaps you may want to put converting this to a transformer, on a backlog like you have proposed for sklearn-genetic?
Regards,
Narayan


Fix Get Request with Body

Setting "Body" doesn't work for cpr::Get #166.
Fix sending a GET request with a body.

Thanks for the PR @joshleeb!


remove redundant closures

I'm honestly not sure about most of these. Yes, these closures aren't necessary, but people don't generally think about code like this. The thought process is generally something like this:

"This function accepts a closure that accepts an argument, so I'll write that out"
"How do I interact with the argument to produce the desired output. What function should I call?"

(2) often involves using autocomplete on the argument to discover the proper method.
The style here isn't a natural part of that process. It feels like adding a "step 3" to the process of "how do i compress this". I personally don't think thats worth it. This type of compression actually makes these things harder to understand imo.

I'm honestly not sure about most of these. Yes, these closures aren't necessary, but people don't generally think about code like this. The thought process is generally something like this:

"This function accepts a closure that accepts an argument, so I'll write that out"
"How do I interact with the argument to produce the desired output. What function should I call?"

(2) often involves using autocomplete on the argument to discover the proper method.
The style here isn't a natural part of that process. It feels like adding a "step 3" to the process of "how do i compress this". I personally don't think thats worth it. This type of compression actually makes these things harder to understand imo.

I guess its a matter of taste. Coming from more imperative languages originally I had the same thinking. The more time I spend working with functional approaches the more I think 'mapping' through a named function is a better fit for the mental model.
I don't think this change is totally clear cut, which is why I split this PR out from #2934

Yeah definitely a matter of opinion. I prefer to keep these as they are for now, so I'm closing this out. But feel free to start a github discussion if you feel strongly about it. If theres relative consensus, we can reconsider.


Cannot set default strategy more than once

It doesn't work with the Kibana 7.2.0. The error "Cannot set default strategy more than once" is raising at the code:

  server.auth.default('keycloak');

error log:

{"type":"log","@timestamp":"2019-08-12T03:33:28Z","tags":["fatal","root"],"pid":1,"message":"{ AssertionError [ERR_ASSERTION]: Cannot set default strategy more than once\n    at new AssertionError (internal/assert.js:269:11)\n    at Object.exports.assert (/usr/share/kibana/node_modules/hoek/lib/index.js:736:11)\n    at module.exports.internals.Auth.default (/usr/share/kibana/node_modules/hapi/lib/auth.js:73:14)\n    at Plugin.init [as externalInit] (/usr/share/kibana/plugins/keycloak-kibana/index.js:202:7)\n    at process._tickCallback (internal/process/next_tick.js:68:7)\n  generatedMessage: false,\n  name: 'AssertionError [ERR_ASSERTION]',\n  code: 'ERR_ASSERTION',\n  actual: false,\n  expected: true,\n  operator: '==',\n  isBoom: true,\n  isServer: true,\n  data: null,\n  output:\n   { statusCode: 500,\n     payload:\n      { statusCode: 500,\n        error: 'Internal Server Error',\n        message: 'An internal server error occurred' },\n     headers: {} },\n  reformat: [Function],\n  [Symbol(SavedObjectsClientErrorCode)]: 'SavedObjectsClient/generalError' }"}
kibana           |
kibana           |  FATAL  AssertionError [ERR_ASSERTION]: Cannot set default strategy more than once

You have to disable security plugin from kibana:
# disable x-pack security (required)
xpack.security.enabled: false

I've added info to readme.