Sort peers & fix bug with filtered peer status

This is to improve the initial connection to the network. More recently, there's been a lot of connect/disconnect activity. It now looks for the best peers based on highest Block Height, and lowest Delay.
@luciorubeens I couldn't use orderBy - it just didn't work (silently failing). Instead, I did this. I also didn't store in the serviceProvider as suggested, so i check on the first index of getGoodPeer and sort then, once.
Feedback welcome!

Removing the filter as it's a copy of another (does it twice)

I believe I'm done making changes now. There was a bug where it filtered down peers based on status, but it didn't use this filtered down list when making the initial connection to a peer (on first load of the desktop client).
Also now sorting peers based on Height DESC, Delay ASC

I think getting the same version from the current peer does not make sense because if the selected peer is using an earlier version the user will get stuck in that version

@luciorubeens would that be possible without manually changing the networks? It all starts from the initial network which would be the master wouldn't it?:
https://github.com/alexbarnsley/ark-desktop/blob/f168b793e5b0b10e704a633bb1a461ba35a04d3a/client/app/src/services/network.service.js#L84-L114
Maybe I've misunderstood, I though it connected to the master network first to get the peers. I'll double check when I'm at a machine.
I'm happy to take it out though if you think it will be an issue!

yeah, but suppose we updated the ark-node, and this default peer has not updated yet.. I'm just making sure that we don't have to manually update this. I'll merge but I think we can improve a bit more.. +5! 👍🏻

That's fair enough @luciorubeens 😁 I'll have a think about whether there's a better way to check version numbers. Perhaps any matching version or greater would work? That way you're not stuck on an old version. Does that sound like an option?


Make TyphoonAssemblyActivator implicit

Rather than instantiating an activator like:
MiddleAgesAssembly *assembly = [MiddleAgesAssembly assembly];
[[TyphoonAssemblyActivator withAssemblies:@[assembly, anotherAssembly]] activate];

Then allow:

//This activates using the concrete collaborator classes specified in properties
MiddleAgesAssembly *assembly = [[MiddleAgesAssembly assembly] activate];

//This activates overriding a concrete collaborator class
MiddleAgesAssembly *assembly = [[MiddleAgesAssembly assembly]
    activateWithCollaborators:@[anotherAssembly]];

Allow overriding one or more collaborators, eg NetworkProvider with TestNetworkProvider
Collaborators expressed as a protocol must be specified manually.

Rationale:
Don't especially want to change the API again, but benefits are:

Tighter API
Avoid confusion such as ##337

Drawbacks:
Changes API again. . . Typhoon 3.1 feature?

Implemented. Intend to update docs and push as 3.1 - TyphoonAssemblyActivator is deprecated by this. One outstanding feature:

When we inject an assembly we currently simply inject TyphoonComponentFactory. Its better to inject the activated assembly as properties of collaborators work correctly here.

Example:

//Inject TyphoonComponentFactory
@property ApplicationAssembly *assembly;

assembly.modelProvider //Resolves to TyphoonCollaboratingAssemblyProxy

//Inject activated assembly
@property ApplicationAssembly *assembly;

assembly.modelProvider //Resolves to activated ModelProvider

Added convenience methods:

[ApplicationAssembly activate]; //Equivalent of [[ApplicationAssembly assembly] activate];

. . . not sure if these are needed?


Compatible Device "Orvibo CT10W"

Hello,
just a short Info about a compatible device.
This device is using the ESP8266 and looks realy nice: https://www.orvibo.com/en/product/xiaofang.html
The PIN-Out is:
//  0 = fw update
//  2 = ?
//  4 = button (input)
//  5 = IR Rx (input)
// 12 = blue LED (output)
// 13 = ?
// 14 = IR Tx (output)
// 15 = red LED (output)
Picture of PCB:
http://bplagens.bplaced.net/ESP8266/Top.PNG
http://bplagens.bplaced.net/ESP8266/Bottom.PNG
My ArduinoIDE settings:
http://bplagens.bplaced.net/ESP8266/ArduinoIDE.PNG
At Aliexpress the price is aprox 16€.
At Amazon i payed ~30€
BR Bernd

How did you open the device?

Hi,
after work i will made a picture for you how i did the job.
But basicly just a knife and soft-force.
BR Bernd
Am Do., 25. Apr. 2019 um 20:56 Uhr schrieb Adrian Böckenkamp <
<EMAIL_ADDRESS>
How did you open the device?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mdhiggins/ESP8266-HTTP-IR-Blaster/issues/248#issuecomment-486796681,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALM5CCN2K3GG6NB3AR4FFL3PSH5GFANCNFSM4GYXRASQ
.

Thank you so much! :-)
I already removed the base plate successfully. I am just unable to get the PCB out of the plastic case. Also, the switch is still covered by some "(elastic) plastic cover" and I don't know how to get it out of it without destroying the PCB and/or plastic cover/case...

Hi Adrian,
the "trick" is: do not remove the base-plate (i made the same mistake) you
have to lift the black-cube from the silver-base
At http://bplagens.bplaced.net/ESP8266/ i have add a few pictures how i
managed it to open the enclosure.
Be carefull - on the inner-side of the black-cube are some clips - two
where broken :-(
But this is not a big point - it still fits!
[image: image.png]
When the black-cube is removed - you can remove very easy the pcb - it just
clipped into the base of the silver lower part.
BR Bernd
Am Fr., 26. Apr. 2019 um 12:53 Uhr schrieb Adrian Böckenkamp <
<EMAIL_ADDRESS>
Thank you so much! :-)
I already removed the base plate successfully. I am just unable to get the
PCB out of the plastic case. Also, the switch is still covered by some
"(elastic) plastic cover" and I don't know how to get it out of it without
destroying the PCB and/or plastic cover/case...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mdhiggins/ESP8266-HTTP-IR-Blaster/issues/248#issuecomment-487016384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALM5CCKQO4KT4ZKA2VCRNZ3PSLNLVANCNFSM4GYXRASQ
.

Thank you very much! I successfully managed to open it. However, my ESP does not seem to enter flash mode even though I connected "GP0" to "GND". I am not even sure, if my PCB has an ESP8266 since it differs from yours (nevertheless, its label says "orvibo 0w8266-02z"):

Even worse and for whatever reason, I am not able to reset the device anymore (by pressing the "S1" button at the bottom until the red LED blinks quickly). When I power up the device, after some time, the blue LED (sometimes?) lights up shortly but then remains off (forever?). Nothing seems damaged and according to my multimeter, the device seems to have power (measured 5 and 3.3V at the appropriate pads respectively). I've also tried to connect via serial monitor (115200 bauds) but nothing is printed. Also, no Wifi hotspot is created (anymore) and it does not connect to my router anymore. (I used the HomeMate app to test the device once.)
Strange: the LEDs do not light up when I power it (either using the micro USB port or via serial 3.3V interface). :-/
Any ideas what I can try? Initially, I intended to backup the original firmware using esptool but this seems impossible now and obviously, I am even unable to flash this repo's firmware... :-(

I managed to repair my board. And I can confirm that it actually has an ESP8266. ;-)
For others: please note that the pin header holes have a contact spacing of 2mm (not 2.54mm) so would need something like this (or its appropriate male counterpart).

Thanks a lot for this, I did not know it was ESP8266 based.

May I ask , I have flashed the standard tas build to Orvibo , but now want it to learn IR codes ,, a little unsure how to go about that .I use domoticz as the base home automation
Probably better to put tas ir build on unit .just learning codes

@mutantxxxx  I am about to follow the same path and install Tasmota.
I presume 'learning' is a process of sniffing IR commands, I found "Receiving IR Commands" on https://tasmota.github.io/docs/Tasmota-IR/ which looks promising.
Would you mind sharing how you flashed Tasmota on your magic cube?

May I ask , I have flashed the standard tas build to Orvibo , but now want it to learn IR codes ,, a little unsure how to go about that .I use domoticz as the base home automation
Probably better to put tas ir build on unit .just learning codes

See IRReceived and IRSend. I think "Learning" is basically sending what you've received using the IRSend command

@HerrPausL @CodeFinder2
I don't really manage to open the case (I already damaged it).
Should I try to lift the black-cube from the silver-base as shown in
1)

or
2)

If you have pictures of yours opened it will be most helpful (the link from this post gives a 404 error).

You should separate the black cube from the silver base, as shown in picture 1

Sorry,
i did not managed to find the pictures right now.
This pictures where made quickly - i hope it will help you:

At all 4 sides you will have 2 lugs which will snap into the cover (and a little bit of glue in the middle). The best way is to use a sharp knive.

Thank you both for your input. I managed to open it and flash Tasmota and now the fun begins integrating it in Home Assistant.

Hello,
sorry for the broken link!
After work I will try to find the old pictures.
If I'm honest, I can't remember exactly where it was to open - so I'll have
to look at the pictures as well.
BR Bernd
Am Di., 8. März 2022 um 13:50 Uhr schrieb Malakai-DB <
@.***>:

@HerrPausL https://github.com/HerrPausL @CodeFinder2
https://github.com/CodeFinder2
I don't really manage to open the case (I already damaged it).
Should I try to lift the black-cube from the silver-base as shown in
1)
[image: 01]
https://user-images.githubusercontent.com/66525206/157241087-e5a6a339-2b23-456f-b12c-b634d9e5d8a8.jpg
or
2)
[image: 02]
https://user-images.githubusercontent.com/66525206/157241119-ec48f84c-49b9-4d3e-88e6-df6abb073238.jpg
If you have pictures of yours opened it will be most helpful (the link
from this post
https://github.com/mdhiggins/ESP8266-HTTP-IR-Blaster/issues/248#issuecomment-487078940
gives a 404 error).
—
Reply to this email directly, view it on GitHub
https://github.com/mdhiggins/ESP8266-HTTP-IR-Blaster/issues/248#issuecomment-1061743345,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALM5CCPT3RTWENWG2NM7F5DU65EIPANCNFSM4GYXRASQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

I managed to repair my board. And I can confirm that it actually has an ESP8266. ;-)
For others: please note that the pin header holes have a contact spacing of 2mm (not 2.54mm) so would need something like this (or its appropriate male counterpart).

Hi @CodeFinder2 , how did you managed to repair the board, mine lits up with faint blue light. Any tips.

I managed to repair my board. And I can confirm that it actually has an ESP8266. ;-)
For others: please note that the pin header holes have a contact spacing of 2mm (not 2.54mm) so would need something like this (or its appropriate male counterpart).

Hi @CodeFinder2 , how did you managed to repair the board, mine lits up with faint blue light. Any tips.

Sorry, this is too long ago, I don't really remember and I am not using the device anymore. 😐


TVer olympic live archive URLs not working

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Japan
Provide a description that is worded well enough to be understood
yt-dlp tver module does not support olympics 2024 /live/ (archive) version of the links, falling back to generic handler.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://tver.jp/olympic/paris2024/live/FBLMTEAM11------------SFNL000100--/']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [e7d73bc45] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19044-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg N-109332-g45ab5307a6-20221201 (setts), ffprobe N-109332-g45ab5307a6-20221201, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.35.5, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1830 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds)
[generic] Extracting URL: https://tver.jp/olympic/paris2024/live/FBLMTEAM11------------SFNL000100--/
[generic] FBLMTEAM11------------SFNL000100--: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] FBLMTEAM11------------SFNL000100--: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://tver.jp/olympic/paris2024/live/FBLMTEAM11------------SFNL000100--/
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1626, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1761, in __extract_info
  File "yt_dlp\extractor\common.py", line 740, in extract
  File "yt_dlp\extractor\generic.py", line 2526, in _real_extract
yt_dlp.utils.UnsupportedError: Unsupported URL: https://tver.jp/olympic/paris2024/live/FBLMTEAM11------------SFNL000100--/

2024年8月31日をもちましてパリオリンピック特設サイトは終了しました。
ご愛顧、ありがとうございました。
5秒後にTVerトップに切り替わります。

The topic page for Olympic Games has been closed.

🤡


Peer dependency error when updating to 2.4.0-canary.5

What version of this package are you using?
2.4.0-canary.5
What operating system, Node.js, and npm version?
MacOS - Ventura 13.4
Node 18.16.0
npm 9.5.1
What happened?
I got a peer dependency error from NPM when trying to update to 2.4.0-canary.5 for next-translate and next-translate-plugin:

What did you expect to happen?
I expected to be able to upgrade without any problems.
Are you willing to submit a pull request to fix this bug?
Sure, though I don't really understand why the peer dependency error is occurring.

Here is my package.json:
{
  "name": "next-translate-example",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "dev": "next dev",
    "build": "next build",
    "start": "next start",
    "lint": "next lint"
  },
  "dependencies": {
    "@types/node": "20.3.1",
    "@types/react": "18.2.13",
    "@types/react-dom": "18.2.6",
    "autoprefixer": "10.4.14",
    "eslint": "8.43.0",
    "eslint-config-next": "13.4.6",
    "next": "13.4.6",
    "next-translate": "^2.4.0-canary.5",
    "postcss": "8.4.24",
    "react": "18.2.0",
    "react-dom": "18.2.0",
    "tailwindcss": "3.3.2",
    "typescript": "5.1.3"
  },
  "devDependencies": {
    "next-translate-plugin": "^2.4.0-canary.5"
  }
}

I'm at a loss as to why the peer dependency check is failing. If I'm reading it correct, next-translate-plugin is indicating a peer dependency of next-translate 2.3.0 or greater — which our version of 2.4.0-canary.5 clearly satisfies.

I think is related wit this npm issue for canary versions https://github.com/npm/cli/issues/4958 probably in the last npm version this is already fixed. However dont worry I'm going to update the peerDependencies before the release. For now to try the canary version you can force, update npm or use yarn or pnpm


Standard_Replication Tasks and Peeklock : Getting workflow saved successfully message along with validation error while trying to save When events are available for replication" trigger.

Describe the Bug with repro steps
Prerequisites:
Create a service bus namespace with two queues.
Create an event hub instance with two hubs.
1.Create a new Stateless logic app in standard, open the designer.
2.Add a event hub "When events are available for replication" trigger
3.In the trigger, enter the event hub name of your second event hub in "Event hub name"
4. Add the event hub "Replicate events" action. Enter the event hub name of your first event hub.
5.Save the workflow.
Expected: Workflow should save successfully.
Actual: Getting workflow saved successfully message along with validation error while trying to save When events are available for replication" trigger but workflow is not saving when we move to overview tab and come back to designer we can see workflow is not saved.
What type of Logic App Is this happening in?
Standard (Portal)
Are you using Preview Designer or GA Designer
Preview
Workflow JSON
{
  "definition": {
    "$schema": "https://schema.management.azure.com/providers/Microsoft.Logic/schemas/2016-06-01/workflowdefinition.json#",
    "actions": {
      "Replicate_events": {
        "type": "ServiceProvider",
        "inputs": {
          "parameters": {
            "eventHubName": "eh1",
            "skipAlreadyReplicated": false
          },
          "serviceProviderConfiguration": {
            "connectionName": "eventHub",
            "operationId": "replicateEvents",
            "serviceProviderId": "/serviceProviders/eventHub"
          }
        },
        "runAfter": {
          "builtin:newWorkflowTrigger": [
            "SUCCEEDED"
          ]
        }
      }
    },
    "contentVersion": "<IP_ADDRESS>",
    "outputs": {},
    "triggers": {
      "When_events_are_available_for_replication": {
        "type": "ServiceProvider",
        "inputs": {
          "parameters": {
            "eventHubName": "eh2"
          },
          "serviceProviderConfiguration": {
            "connectionName": "eventHub",
            "operationId": "receiveEventsForReplication",
            "serviceProviderId": "/serviceProviders/eventHub"
          }
        }
      }
    }
  },
  "connectionReferences": {
    "eventHub": {
      "connection": {
        "id": "/serviceProviders/eventHub/connections/eventHub"
      },
      "connectionName": "adw",
      "api": {
        "id": "/serviceProviders/eventHub"
      }
    }
  },
  "parameters": {}
}

Screenshots or Videos

Browser
Edge
Additional context
Environment Details:
URL: https://aka.ms/emastaging
Version: "2.30501.1.1"

@preetriti1 this is something we plan to implement where when workflow validation fails that we plan to prevent workflow saving, right?

We are not actually saving the workflow (so not a functional bug), just the message is wrong. I fixed the message but it will only be available in next release. We will still have to see why they are getting this validation error.

@vgouth - I am not able to reproduce this issue in staging for a newly created app with just adding trigger and action. Can you please try it again and reopen if this still reproduce?

@preetriti1- Yes, I am not able to repro this issue in stating environment. But there is difference in version for stating portal ( 2.30410.1.12) and request URL portal (2.30501.1.1).  I am able to reproduce the above issue in request URL with current version (2.30501.1.5).

NOTE: @Eric-B-Wu -we don't have an option to reopen this issue, request you to please reopen it.

Staging Portal should be 2.30501.1.5, the request deployment had been updated with some fixes from 2.30501.1.1. If this is reproable in staging currently (post fixes) then I'll reopen it unless @preetriti1 this is fixed in 0508 release?

Issue not repro in latest release.  Closing this issue
Environment Details:
URL: https://aka.ms/emastaging-flowstip1
Version: 2.30508.1.2


Fix/daemon path in dev mode

For Tapyrus, The default daemon path is ~/.tapyrus/prod-{networkid} or ~/.tapyrus/dev-{networkid}
This pull request fixes daemon path for Dev mode.

@Yamaguchi can you rebase this?


How to grab all days in current month, and process them?

Hi,
This isn't a bug report as such - I'm just trying to figure out if this is possible. So I have an inline calendar I want to use to show available dates for a room booking:
    window.VARS.ViewOtherDateDatepicker = flatpickr("#embed-calendar-avail input.date", {
        minDate: "today",
        inline: true,
        dateFormat: "d-m-Y",
        onDayCreate: function(dObj, dStr, fp, dayElem){

        }
    });

What I want to be able to do is grab ALL the dates  in the current visible month, and then send that off to a server script to check the availability. Then that would return a JSON object, and I can go through and update each of the  dates that are not available. I found the onDayCreate() function that I could use, but I really don't want to as it's going to mean I need to do 29-31 AJAX requests every single time the month changes (which is a waste of server resources, and the users). Is there a way around this?
Thanks!
Andy

Do something like this:
  window.VARS.ViewOtherDateDatepicker = flatpickr("#embed-calendar-avail input.date", {
        minDate: "today",
        inline: true,
        dateFormat: "d-m-Y",
        onReady() {
           const dates = Array.prototype.map.call(fp.days.childNodes, d => d.dateObj);
           const availableDays = someAjaxRequest(dates);
           Array.prototype.forEach.call(fp.days.childNodes, dayElement => {
             // do something with it, based on availability
             // add "disabled" class to mark as disabled
           });
        }
    });

Thanks for the reply. Unfortunately I had to go back to just showing one month, as there seems to be some issues:
https://ibb.co/7VPMWnW

It shows the 1st of Dec as a Sunday - when if you look at the calendar in Windows, it shows Saturday is the 1st?

The 29/30/31 all go weird and don't align under the days they are supposed to :/

Its a shame - as that would have worked really well. Any chance of fixing those bugs? I took a look around the code, but I'm not familiar enough with it to fix unfortunately.
Thanks
Andy

@chmln I think I've got it:
https://github.com/flatpickr/flatpickr/issues/1266#issuecomment-444386746
You have:
        // prevMonthDayClass = isMultiMonth ? "prevMonthDay hidden" : "prevMonthDay",
        // nextMonthDayClass = isMultiMonth ? "nextMonthDay hidden" : "nextMonthDay";

But commenting that out and replacing with:
        prevMonthDayClass = "prevMonthDay not-active",
        nextMonthDayClass = "nextMonthDay not-active";

And it now works like a charm:
https://ibb.co/qCyNh1M
Is there any reason you had that code in there?

Do something like this:
  window.VARS.ViewOtherDateDatepicker = flatpickr("#embed-calendar-avail input.date", {
        minDate: "today",
        inline: true,
        dateFormat: "d-m-Y",
        onReady() {
           const dates = Array.prototype.map.call(fp.days.childNodes, d => d.dateObj);
           const availableDays = someAjaxRequest(dates);
           Array.prototype.forEach.call(fp.days.childNodes, dayElement => {
             // do something with it, based on availability
             // add "disabled" class to mark as disabled
           });
        }
    });

I just tried that, and unfortunately it doesn't work:

(35) […]
0: Date 2019-01-27T00:00:00.000Z
1: Date 2019-01-28T00:00:00.000Z
2: Date 2019-01-29T00:00:00.000Z
3: Date 2019-01-30T00:00:00.000Z
4: Date 2019-01-31T00:00:00.000Z
5: Date 2019-02-01T00:00:00.000Z
6: Date 2019-02-02T00:00:00.000Z
7: Date 2019-02-03T00:00:00.000Z
8: Date 2019-02-04T00:00:00.000Z
9: Date 2019-02-05T00:00:00.000Z
10: Date 2019-02-06T00:00:00.000Z
11: Date 2019-02-07T00:00:00.000Z
12: Date 2019-02-08T00:00:00.000Z
13: Date 2019-02-09T00:00:00.000Z
14: Date 2019-02-10T00:00:00.000Z
15: Date 2019-02-11T00:00:00.000Z
16: Date 2019-02-12T00:00:00.000Z
17: Date 2019-02-13T00:00:00.000Z
18: Date 2019-02-14T00:00:00.000Z
19: Date 2019-02-15T00:00:00.000Z
20: Date 2019-02-16T00:00:00.000Z
21: Date 2019-02-17T00:00:00.000Z
22: Date 2019-02-18T00:00:00.000Z
23: Date 2019-02-19T00:00:00.000Z
24: Date 2019-02-20T00:00:00.000Z
25: Date 2019-02-21T00:00:00.000Z
26: Date 2019-02-22T00:00:00.000Z
27: Date 2019-02-23T00:00:00.000Z
28: Date 2019-02-24T00:00:00.000Z
29: Date 2019-02-25T00:00:00.000Z
30: Date 2019-02-26T00:00:00.000Z
31: Date 2019-02-27T00:00:00.000Z
32: Date 2019-02-28T00:00:00.000Z
33: Date 2019-03-01T00:00:00.000Z
34: Date 2019-03-02T00:00:00.000Z

Date wise, I have Feb and March 2019 showing, so at the minimum I need ast feb to 31st March 2019 :/

@youradds the fp.daysContainer field contains all the month containers with days.
I'll get you a quick snippet in a bit

I think this should do it:
fp.days.querySelectorAll('.flatpickr-day:not(.prevMonthDay):not(.nextMonthDay)').forEach(element => {
    console.log(element.dateObj);
})

Where fp is your flatpickr instance


[doc,site] Site/landing changes

[Only consider the final 2 commits, this PR will be rebased once #17323 is merged,  which makes up all but the last 2 commits here.]
REVIEW BUT DO NOT MERGE
We want to coordinate the merge of new website changes, so that suitable precautions can be taken for a safe deployment.

This PR consists of changes to the landing page site (opentitan.org) that were worked on out-of-tree by a number of people. The primary work was to change the site-header and add new pages in preparation for upcoming further features.
There has also been some minor refactoring to the existing code, and some styling tweaks.

Known Issues

Mobile experience has issues which are yet to be addressed.

Mobile experience has issues which are yet to be addressed.

@marnovandermaas If you have any time available, would you mind checking this out? I think the wrap behaviour just needs looking at, or even going full hamburger :hamburger:

@jwnrt I've included most of the changes to the landing page site here from the refactor repo (minus the stats, which can come back later). Would you mind checking out if anything made it in that is questionable / needs further work? I know this is kind of a unfocused task/ask, but any ideas you have would be helpful. Cheers

Mobile experience has issues which are yet to be addressed.

@marnovandermaas If you have any time available, would you mind checking this out? I think the wrap behaviour just needs looking at, or even going full hamburger hamburger

An initial implementation here: https://github.com/marnovandermaas/opentitan/tree/site/landing_changes
Couple of issues with it:

Hamburger menu jumping to the left when going to smaller screens
Animation of hamburger menu should be less fancy
The menu itself does not respond to the light/dark theme

Mobile experience has issues which are yet to be addressed.

@marnovandermaas If you have any time available, would you mind checking this out? I think the wrap behaviour just needs looking at, or even going full hamburger hamburger

An initial implementation here: https://github.com/marnovandermaas/opentitan/tree/site/landing_changes Couple of issues with it:

Hamburger menu jumping to the left when going to smaller screens
Animation of hamburger menu should be less fancy
The menu itself does not respond to the light/dark theme

Thanks for starting on this @marnovandermaas, it's much appreciated, and should be a good thing to build on. It's going to be nice to have a consistent mobile experience between both the landing-site and the book. :+1:

rebased

Looks good, thanks for putting all this together
Only issue I can see is that we've moved the skyline without moving the logo so it's not aligned any more
Should we leave out the skyline/logo change from this PR? I think it should be fine, we can add it later when we want to fit those metrics in this section

Should we leave out the skyline/logo change from this PR? I think it should be fine, we can add it later when we want to fit those metrics in this section

Whoops yes that should come later. Is that just a case of removing the changes to the static .svg files, or is there other things needed too?

Is that just a case of removing the changes to the static .svg files, or is there other things needed too?

I think that should be fine, but I can see some other changes to margins etc. too, so not too sure.
I have a branch which adds these changes in a way that keeps all the CSS separate, would you mind if I submitted that as a PR?
@hcallahan-lowrisc

Is that just a case of removing the changes to the static .svg files, or is there other things needed too?

I think that should be fine, but I can see some other changes to margins etc. too, so not too sure.
I've made a branch with these changes split into separate CSS files, let me know what you think #17462

Sorry I'm confused, is that PR to be made on-top of this PR? or instead of it?

rebased

Closing in favour of #17462


Fixed #6587: C-API exports again CV_VERSION* macros

resolves #6587
What does this PR change?
Export as in OpenCV 2.4 CV_VERSION macros for C-API.

Just include opencv2/opencv.hpp in your application before legacy and deprecated "C" headers.

@alalek Please, check again this PR. I'm asking for C-API with C compiler.
As you said me in #6585 c/h files should not include directly hpp files. Moreover, this change implies to modify old OpenCV applications. Obviously new apps should be implemented with the current c++ API.
Moreover, If I apply your proposal, I get compilation error as expected:
In file included from /usr/local/include/opencv2/opencv.hpp:46:0,
                 from main.c:3:
/usr/local/include/opencv2/core.hpp:49:4: error: #error core.hpp header must be compiled as C++
 #  error core.hpp header must be compiled as C++

code:
#include <stdio.h>
#include <opencv2/opencv.hpp>
//#include <opencv/cv.h>

int main(int a, char** v){
  printf("%d\n",CV_MAJOR_VERSION);
  return 0;
}

compilation:
gcc -Wall main.c -o main -I/usr/local/include/opencv -I/usr/local/include -L/usr/local/lib  -lopencv_imgproc -lopencv_core -lm

So, please check again this PR and suggest me other ways to fix the issue.

@jlguardi Please see my comment in issue


fix typo 'font-channel' in grant_types.rst

What issue does this PR address?
'font-channel' should be 'front-channel'
Does this PR introduce a breaking change?
no
Please check if the PR fulfills these requirements

[x] The commit follows our guidelines
[ ] Unit Tests for the changes have been added (for bug fixes / features)

Other information:

thanks!


7.2.0

The writing to the workspace settings will be fixed with this release as you know.

If project detection creates to much fuzz and support for you just remove it.

I totally missed it. I don't know what's happening with vscode but now when I use the open accent it doesn't put the space automatically as before.

I hoped you could provide the Angular controller icon before this release.

About the project detection, I know @JimiC, and I'm not thinking about removing it for the moment. I think it's a good utility but maybe it shouldn't be enabled by default (just thinking about alternatives). I'd like to find the time to discuss it with you in Slack. Maybe during the week if that's fine for you.

From my perspective, Angular icons shouldn't be enabled by default, simply because the majority of the projects aren't Angular based.

Yes, maybe you're right. Definetely, you have a strong point. I've created the Angular controller icons but I think that maybe we should deliver them once we have Angular 1 detection solved. What do you think?

I've finally published the controller icons in 7.2.0.
Just as a side note, the guy of the awful review has reconsidered and has given us 5 stars again! 😁


IP Group update preview

How to enable the Preview feature?
https://learn.microsoft.com/en-us/azure/firewall/ip-groups#parallel-ip-group-updates-preview

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

ID: 9f9791cb-ba50-98bf-8831-5e03b568c9f3
Version Independent ID: 9f4d1a49-9c81-b0b8-ee85-87c5f8a0dae3
Content: IP Groups in Azure Firewall
Content Source: articles/firewall/ip-groups.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@nitbhatt0 Thanks for your feedback, we shall get back to you shortly

@nitbhatt0 I have looped in the author vhorne in our internal discussion.
They shall update the document soon

Article updated
#please-close


Add display names using @display annotation and add user defined errors

Purpose

Add @display annotation to the Azure Cosmos DB connector.
Enhance error handling in Azure Cosmos DB connector.
Fix https://github.com/wso2-enterprise/choreo/issues/2954

Goals

Add display name for client
Add display name for listener
Add display names for remote operations
Add display name for parameters in the remote operations
Add display name for the return value of the remote operations
Provide additional information about errors.

Approach

Add labels via @display annotation which is available in Ballerina to all clients, remote operations, function parameters and return types.
Provide additional information about errors using user defined errors

Automation tests

Integration tests

Done for all operations implemented

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment

Ubuntu 20.04 LTS
Ballerina SL Alpha 2
JDK 11

Closing as this is an Outdated PR. All changes are included in  https://github.com/ballerina-platform/module-ballerinax-azure-cosmosdb/pull/4


*MapVertexColor without color data in buffer makes model disappear with WebGL warnings.

Setting diffuseMapVertexColor (for example) to true, if vertex buffer data does not have color data, model will not be rendered and will spam WebGL errors.
Same as it used to be with UV channels.

Does it help? https://github.com/playcanvas/engine/pull/305


Bazel implicit opts

@mcy suggested not passing in copts and linkopts we'd use for every opentitan_binary but instead making them implicit to the macro, so I've added a default and overridden it where hello_world needs an extra copt.

One of these commits is present in another PR but important for considering the content of this PR. I'll stage this as Ready for review when that one is accepted and this is rebased.


Django runserver error : cannot import name User

Dear all
I've run my django server thanks to the following command :
./manage.py runserver ipaddress:port
When I'm trying to connect to the webserver from a web navigator, my Linux machine displays the following error message :
File "/usr/lib/python2.7/wsgiref/handlers.py", line 85, in run
self.result = application(self.environ, self.start_response)
File "/usr/local/lib/python2.7/dist-packages/django/contrib/staticfiles/handlers.py", line 67, in call
return self.application(environ, start_response)
File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/wsgi.py", line 206, in call
response = self.get_response(request)
File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 187, in get_response
resolver, sys.exc_info())
File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 234, in handle_uncaught_exception
if resolver.urlconf_module is None:
File "/usr/local/lib/python2.7/dist-packages/django/core/urlresolvers.py", line 341, in urlconf_module
self._urlconf_module = import_module(self.urlconf_name)
File "/usr/local/lib/python2.7/dist-packages/django/utils/importlib.py", line 40, in import_module
import(name)
File "/home/admincatmaid/catmaid/django/projects/mysite/urls.py", line 13, in
admin.autodiscover()
File "/usr/local/lib/python2.7/dist-packages/django/contrib/admin/init.py", line 29, in autodiscover
import_module('%s.admin' % app)
File "/usr/local/lib/python2.7/dist-packages/django/utils/importlib.py", line 40, in import_module
import(name)
File "/home/admincatmaid/catmaid/django/applications/catmaid/admin.py", line 11, in
from catmaid.control.importer import importer_admin_view
File "/home/admincatmaid/catmaid/django/applications/catmaid/control/importer.py", line 16, in
from guardian.models import Permission, User, Group
ImportError: cannot import name User
Cheers,
Mathieu

I could not reproduce the error here.
make sure you have installed all the dependencies, and try again.
pip install -r requirements.txt
pip install -r requirements/local.txt
pip install -r requirements/test.txt

I've already install all the dependencies
Babel==1.3
Django==1.6
Jinja2==2.7.3
Landscape-Client==14.12
MarkupSafe==0.23
PAM==0.4.2
PyYAML==3.10
Pygments==2.0.2
South==1.0.2
Sphinx==1.3.1
Twisted-Core==13.2.0
Werkzeug==0.10.4
alabaster==0.7.3
amqp==1.4.6
anyjson==0.3.3
apt-xapian-index==0.45
argparse==1.2.1
billiard==<IP_ADDRESS>
celery==3.1.17
chardet==2.0.1
colorama==0.2.5
configobj==4.7.2
coverage==3.7.1
django-adminplus==0.3
django-celery==3.1.16
django-debug-toolbar==1.3.0
django-devserver==0.8.0
django-extensions==1.5.2
django-guardian==1.2.5
django-jsonfield==0.9.13
django-pipeline==1.4.7
django-taggit==0.13.0
docutils==0.12
futures==2.2.0
html5lib==0.999
kombu==3.0.24
numpy==1.6.1
psycopg2==2.6
pyOpenSSL==0.13
pyserial==2.6
python-apt==<IP_ADDRESS>ubuntu1
python-debian==0.1.21-nmu2ubuntu2
pytz==2015.2
requests==2.2.1
six==1.5.2
snowballstemmer==1.2.0
sphinx-rtd-theme==0.1.7
sqlparse==0.1.15
ssh-import-id==3.21
stevedore==0.14.1
urllib3==1.7.1
virtualenv==1.11.4
virtualenv-clone==0.2.4
virtualenvwrapper==4.1.1
wsgiref==0.1.2
zope.interface==4.0.5
Maybe some tools are not compatible with my Django's version ? (1.6)

@cody7914 exact. Probably many libraries had not been sufficient time to upgrade to Django 1.8.
I can tell only for cookiecutter-django. Currently, cookiecutter-django only provides support for Django 1.8.
Django 1.8 is a LTS (Long Term Support) version. LTS[1] version guarantees security updates and bug fixes for 3 years after release date(01/04/2015).
So it's a good idea to update your projects or start on new project with Django 1.8
create a new virtualenv and install the dependencies using the command and try again
pip install -r requirements.txt
pip install -r requirements/local.txt
pip install -r requirements/test.txt

[1] https://docs.djangoproject.com/en/1.8/internals/release-process/#long-term-support-lts-releases


feat: design module

Description

This PR adds a new Design module that should host all shared custom views like the BlurView. This will come in support for the modularization!
Related Issue
#233
Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] Review requested

Screenshots

Good work, much cleaner :]


Fixes #189 - Expose SQS message's ReceiptHandle to allow changing message visibility

Do we want just this field, or do we want a fix with full access to the message metadata?

@AnthonySteele We can add the fix now and phase it out when Message metadata aka Envelope is added later.

If you think that's worthwhile, sure. I don't know if/when envelopes will be worked on, so if this is all we get in 5.0 it's worthwhile.

@AnthonySteele Since this PR is not a breaking change it can be added to existing 4.x. We could still add message Metadata to version 5.0. right?

Yeah, so if @ssemyonov creates a PR for this against the master branch we can get it merged and get a release out, we can cherry pick this later if we don't get around to message envelopes (which I think we will get done) in v5.0

You are both correct, that's a good plan.

@ssemyonov Can you please submit your PR against master?

Will submit PR to master


UseCors & UseRouting / UseAuthentication

Please put in the documentation that calling UseCors must happen before UseRouting and USeAuthentication. Our team spelt half a day to found out why we were having authentication issues on Preflight requests.
Refering to his page: https://docs.microsoft.com/en-us/aspnet/core/signalr/security?view=aspnetcore-3.1

@brechtvhb ... Recommend that you close this and re-open it using the This page feedback button and form at the bottom of the topic. It will add metadata that automatically pings the author ... you'll get a faster response.


Built of Electron, but not cross-platform?

If this is built on Electron, why did you not released any binaries for Windows and Linux?

Hello @sapioit Currently working on the cross platform version of Yoda. Would be up in 3 weeks. Cheers. Please feel free to post any ideas you have about integrating within Yoda :)

A "Random" button, for each category may be a good idea... or a few color themes, to choose from...

@whoisandie Any update on the cross platform version? I don't see a v2 branch on Github.

?


Flaky std lib test: can_get_input_message_predicate_length

As it can be seen from this run can_get_input_message_predicate_length test is failed, but after re-running the job (here), the test passed

Assuming this hasn't happened recently, the last examples are so old the links expired. Closing since this is no longer actionable.


Check error handling if the Ethereum adapter container or the blockchain network is down. Apparently the dsu-wizard do not respond anything


用node example/starter-bot.js在macbook上运行的时候报错。请问如何解决？

用node example/starter-bot.js在macbook上运行的时候报错。请问如何解决？
------——————————————————————————————
fuxs-MacBook-Pro:wechaty frankgo$ node examples/starter-bot.js
19:24:44 INFO Wechaty  start() v0.22.6 is starting...
19:24:44 INFO PuppetManager<EMAIL_ADDRESS>please wait ...
npm: npm
npm:  WARN checkPermissions Missing write access to /Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules
npm: npm
npm:  WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm: npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm: npm
npm:  WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm:
npm: npm
npm:  ERR! path /Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules
npm: npm
npm:
npm: ERR!
npm:
npm: code
npm:  EACCES
npm: npm
npm:
npm: ERR!
npm:
npm: errno
npm:  -13
npm: npm
npm:
npm: ERR!
npm:
npm: syscall
npm:  access
npm: npm
npm:
npm: ERR!
npm:  Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules'
npm: npm
npm:  ERR!
npm:   { [Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules']
npm ERR!   stack:
npm: npm ERR!    'Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules'',
npm: npm ERR!   errno: -13,
npm ERR!
npm:    code: 'EACCES',
npm ERR!
npm:    syscall: 'access',
npm ERR!
npm:    path:
npm: npm ERR!    '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules' }
npm: npm
npm:  ERR!
npm: npm
npm:  ERR!
npm:  The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm: npm ERR!
npm ERR!
npm:  If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm: npm ERR! the command again as root/Administrator (though this is not recommended).
npm:
npm: npm
npm:  ERR! A complete log of this run can be found in:
npm
npm: ERR!
npm:      /Users/frankgo/.npm/_logs/2019-03-03T11_24_46_996Z-debug.log
{ Error: Command failed: npm install<EMAIL_ADDRESS>npm WARN checkPermissions Missing write access to /Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm ERR! path /Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall access
npm ERR! Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules'
npm ERR!  { [Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules']
npm ERR!   stack:
npm ERR!    'Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path:
npm ERR!    '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules' }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).
npm ERR!      /Users/frankgo/.npm/_logs/2019-03-03T11_24_46_996Z-debug.log
at ChildProcess.exithandler (child_process.js:294:12)
at ChildProcess.emit (events.js:182:13)
at ChildProcess.EventEmitter.emit (domain.js:441:20)
at maybeClose (internal/child_process.js:962:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:251:5)

killed: false,
code: 243,
signal: null,
cmd: 'npm install<EMAIL_ADDRESS>' }
19:24:47 ERR Wechaty start() exception: Command failed: npm install<EMAIL_ADDRESS>npm WARN checkPermissions Missing write access to /Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm ERR! path /Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall access
npm ERR! Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules'
npm ERR!  { [Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules']
npm ERR!   stack:
npm ERR!    'Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path:
npm ERR!    '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules' }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).
npm ERR!      /Users/frankgo/.npm/_logs/2019-03-03T11_24_46_996Z-debug.log
{ Error: Command failed: npm install<EMAIL_ADDRESS>npm WARN checkPermissions Missing write access to /Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm ERR! path /Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall access
npm ERR! Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules'
npm ERR!  { [Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules']
npm ERR!   stack:
npm ERR!    'Error: EACCES: permission denied, access '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path:
npm ERR!    '/Users/frankgo/Developer/wechaty/node_modules/wechaty/node_modules' }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).
npm ERR!      /Users/frankgo/.npm/_logs/2019-03-03T11_24_46_996Z-debug.log
at ChildProcess.exithandler (child_process.js:294:12)
at ChildProcess.emit (events.js:182:13)
at ChildProcess.EventEmitter.emit (domain.js:441:20)
at maybeClose (internal/child_process.js:962:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:251:5)

killed: false,
code: 243,
signal: null,
cmd: 'npm install<EMAIL_ADDRESS>' }
fuxiaos-MacBook-Pro:wechaty frankgo$ fuxiao node examples/starter-bot.js
-bash: fuxiao: command not found
fuxiaos-MacBook-Pro:wechaty frankgo$ root node examples/starter-bot.js
-bash: root: command not found

Problem solved...first you need to disable VPN service and then use:
"sudo npm install --save<EMAIL_ADDRESS> --unsafe-perm=true" to fix the access permission problem..before running npm start!!

Cool， then could you help me to close this issue if there aren't any questions?

sure..let me close it nwo..thanks.


Add PR Gate pipeline

Add PR Gate pipeline for image service.
Test job of PR Gate:
http://rackhdci.lss.emc.com/job/image-service/job/PR_Gate/
Test job of Build Base Image for CI/CD:
http://rackhdci.lss.emc.com/job/image-service/job/BuildBaseImage/
@panpan0000 @changev @anhou @iceiilin @nortonluo @sunnyqianzhang

test this please

test this please

BUILD image-service » PR_Gate #74 : FAILURE

test this please

BUILD image-service » PR_Gate #78 : FAILURE

BUILD image-service » PR_Gate #85 : FAILURE

BUILD image-service » PR_Gate #86 : FAILURE

test this please

This works for me :+1:

This PR as a Pair-Program task, and get several rounds of review and +1.
so as a committer, I will merge it in .


Params error should limit to specific call in call chain

From @unional on August 16, 2016 6:39

VSCode Version: 1.4.0
OS Version: OSX El Capitan

In the example above, only the last call (.option()) has invalid parameter.
It would be better if the error only limit to that function call.
Copied from original issue: Microsoft/vscode#10573

From @alexandrudima on August 16, 2016 6:45
@unional What language are you using? Can you also provide a reproducible case in textual form (good for copy-paste)

From @unional on August 16, 2016 7:28
Sure. It's TypeScript

let x = []
  .concat()
  .entries(1)

From @unional on August 16, 2016 7:33
That's strange:

From @unional on August 16, 2016 7:36
Ok. Know why. I'm using<EMAIL_ADDRESS>

From @alexandrudima on August 16, 2016 7:31
@unional In this case, I believe the error is showing at the right spot:

From @alexandrudima on August 16, 2016 7:54
Oh, cool

this seems to be working as intended in latest TS 2.0 drop.


Dev - disable lat long logging

Expected Behavior
I want to disable the 'lat' 'long' logging in the dev build.
Actual Behavior
{'distance_unit': '', 'distance': '', 'current_position': (52.061011021257656, 4.2319219650586595), 'last_position': (52.06107408529108, 4.231919556713865)}
Steps to Reproduce
Run the dev build.
Other Information
OS: CentOS 7
Git Commit: 350148e8443e35a6f5b74169b7ac5fd26815fe7b
Python Version: Python 2.7.5

#3484 is the same issue

In colored_logging_handler.py you can add:
if event == 'position_update':
    return

under
# Replace message if necessary
if event == 'catchable_pokemon':
    message = 'Something rustles nearby!'

And it'll stop updating with that message.

#3484


[DOCS] parse_json: Objects do not keep key order

Document JSON not guaranteeing key order.
Closes https://github.com/godotengine/godot/issues/16099.

We should explicitly specify that arrays keep order. At least that's how it should be.

Should this be mentioned in http://docs.godotengine.org/en/latest/classes/class_json.html too?

Would be good yeah, many users seem to be puzzled at how Godot handles JSON because they actually don't really know the JSON spec.

Will update this then.

Added same information to JSON/JSONParseResult class docs.


moves data asset record to repo

closes #384
Some discussion points we need to agree upon:

What should the default filename be (.aind.metadata.json)?
What should the required fields be (currently id, created, last_modified, location, metadata_status)? should we include curated, quality_controlled?
Should extra have some validation rules (for example, expect AindCoreModel)? or can we keep it general

David:
(1) what do we use for _id? is it currently Code Ocean asset id?
(2) do we need to explicitly version this?
For (1) I don't have a strong feeling. I guess it might be nice to have it explicitly be  a code_ocean_id, in which case _id could be a plai
For (2) I think yes, and maybe it should just be the aind_data_schema library version.

David:
(1) what do we use for _id? is it currently Code Ocean asset id? (2) do we need to explicitly version this?
For (1) I don't have a strong feeling. I guess it might be nice to have it explicitly be a code_ocean_id, in which case _id could be a plai
For (2) I think yes, and maybe it should just be the aind_data_schema library version.

For 1, this is an index of our data assets that we maintain in our buckets. It will have nothing to do with Code Ocean directly. It will either be a uuid or be the same as the location field. We can add a field to link to Code Ocean if we want. For 2, it can have a schema_version the same as the other fields. We can use S3's versioning to track updates to the json file directly and maybe pass that along to a "record_version" field or something.

I'm good with all of this.

Suggest we call this DataAsset and data_asset.py, unless others feel strongly.

We'll handle adding metadata fields (such as subject) in another ticket (once Saskia's refactor is complete)

Suggest we call this DataAsset and data_asset.py, unless others feel strongly.

I do feel strongly. I think we should call it DataRecord. We use data asset already to refer to the data asset - it will be confusing to call this data asset too.

Suggest we call this DataAsset and data_asset.py, unless others feel strongly.

I do feel strongly. I think we should call it DataRecord. We use data asset already to refer to the data asset - it will be confusing to call this data asset too.

Just renamed it. Let me know if there's any other comments.

Sorry, we just pow-wowed at decided:

let's just call it Metadata
it would be defined in metadata.py
and it would generate metadata.json


Feature/elastic experiment

See companion pull request in the Dockstore repo for readme details.
This is going to need major refactoring, but demonstrates some of what we want to accomplish.

Something seems strange, lots of old DAG code is in there.

Yeah, there bunch of strange merge conflicts when updating from develop.


fix to #1485 - enable GearsOfWar sql tests disabled by recent test change

Issue was that GearsOfWar uses SqlLogger to validate the expected query, and with recent change, the logger could contain database creation SQL on top of the actual query that was tested. Fix is to clear log after the database has been created

@ajcvickers

:shipit:

merged - 3ae3a838c019ab28e17336557ad17334accfaf40


Splitting Otterwiki Docker setup into multiple containers?

I heavily modified the Otterwiki Docker / Docker Compose setup for my own production use. Basically the setup now consists of three containers instead of just one:

web (uwsgi)
nginx
db (postgres)

This meant that supervisord was no longer required/desired, so I ripped that one out as well. The end result is probably not general-purpose enough to include as-is in this project.
That said, would you be open to including this type of Docker setup in the Otterwiki Git repository? It could probably live side-by-side with the old Docker setup without too much fuzz. Most of the code repetition would be in the Dockerfile.

Hey @mattock, first of all thank you for the work and thoughts you bring to this project!
A first step in this direction has been done, there is the docker/Dockerfile.slim whis is running only the uwsgi is running, serving both the app and the static files, rendering nginx unnecessary and therefore supervisord.
This is not documented yet, but I'll make up for it as soon as possible. An example docker-compose.yaml for a complex setup like the one you proposed would be useful. Contributiuons are welcome!

Hey @mattock, first of all thank you for the work and thoughts you bring to this project!

Some backstory: I looked into a gazillion of different wikis and after getting very frustrated with our "prime candidates" (xwiki and wiki.js) I had a pause. Then, by accident, I found Otterwiki which looked very promising. In fact, I added Otterwiki to the Wikipedia's "List of wikis" article so that others might more easily find it.
Otterwiki seems to do what we at the OpenVPN project needs and I always try to contribute back whenever possible. So I'm glad if I can be of assistance!

A first step in this direction has been done, there is the docker/Dockerfile.slim whis is running only the uwsgi is running, serving both the app and the static files, rendering nginx unnecessary and therefore supervisord.
The redimp/otterwiki(version)-slim image is beeing released along the redimp/otterwiki(version) image on hub.docker.com.

Noted.

This is not documented yet, but I'll make up for it as soon as possible. An example docker-compose.yaml for a complex setup like the one you proposed would be useful. Contributiuons are welcome!

I'll try to make my "production" setup as non-invasive as possible, so that it could potentially be merged. It already works fine afaics. I will see how to minimize divergences between it and the existing development Docker setups so that I can send a reasonable PR your way.

I've added the documentation how to make use of the -slim image variant to the Installation Documentation. Thank you @mattock for bringing this up!

Thanks! I'll check out the -slim variant once I get back to Otterwiki setup again. I may come up with some follow-up PRs.


delayInitToCheckForSearchAdscalled everytime?

Hi. I am just curious, it seems if you use branch.json config then delayInitToCheckForSearchAds is called every time the app opens. Is that correct? Would it make sense to only run it on the first app open since that would be the only time they came from the app store?

Cool, thanks!


Create jar for releases

We can eventually set up some script that when you create a release, it compiles it into a library jar. That way people can just add the .jar file to their project instead of copying and pasting the file. With that same script you can also do unit testing on push.

Gradle task buildJar will now build a fat jar of the library


Is there a way to remove those requests rather than redirecting them?

Everytime ungoogled-chromium just redirects those request like chrome webstore(https://chrome.9oo91e.qjz9zk/) and google services(https://support.9oo91e.qjz9zk/chrome?p=syncgoogleservices).
Is there a way to remove them rather than redirecting them to qjz9zk domain?

Replacing the urls with that domain is basically removing them, the domain will no be resolved as it doesn't exist. Removing all functionally related to every google url in the source would be impossible to maintain, replacing the urls with something other than valid urls might cause undefined behavior.

doesn't exist

Unless Google registers it :D

Can you completely remove these, cus its no use as it is of now, so why not remove it

The only reason I can think of not to do this is the added maintenance burden for no significant benefit.

Well, I can't deny that. Umm, can you help me with the second part?

can you help me with the second part?

Unfortunately, not, not me at least.

I can dig into this a bit and see if it's something that can be added to remove-uneeded-ui.patch.  There's no reason to remove every instance of substituted links in the ui, but it's nice to remove the more egregious areas.  The passwords section is something I've been aware of and put off for too long.

I can dig into this a bit and see if it's something that can be added to remove-uneeded-ui.patch. There's no reason to remove every instance of substituted links in the ui, but it's nice to remove the more egregious areas. The passwords section is something I've been aware of and put off for too long.

:) nice bro

@Ahrotahn Hey, I need a small help. I downloaded chromium using depot_tools earlier. But when i checked the modification readme, i saw that i have no option to use this downloaded one. I'm not that experienced, so can you help me with using the downloaded one. #

I'd recommend checking out the instructions on the repo for your platform:  https://github.com/Eloston/ungoogled-chromium/blob/master/docs/platforms.md
There's usually a writeup on the build process that's easier than attempting to set up your own.

I hope your question has been answered, otherwise please let us know.
Closing this issue for now.


ci: Monitor for scheduled CodeBuild job failures

Problem:
Scheduled CodeBuild job failures against master should generate an alarm or ticket (or both).
Proposed Solution:
CloudWatch (failure) events trigger a lambda, which checks the type of failure and opens a GitHub issue, if it's a new failure.

Done using a lambda triggered by a codebuild state change event. (Checked into CodeCommit)


feat(cxl-ui): add cxl-scroll-top component

@anoblet nice one, IMO this should be a part of Vaadin because use-case range of it is wide, I'm surprised they didn't develop this yet.


[camera] Is it possible to record a video while using imageStream with camera plugin?

Hello. I am trying to use the camera plugin for two purposes in my app:

Allow the user to record a video
Process the video alongside recording with some AI models

I tried to use the camera plugin with two controllers.
  CameraController _cameraRecordingController;
  CameraController _cameraImageStreamController;
.
.
.
// initiate controllers
      if (_cameraRecordingController == null ||
          !_cameraRecordingController.value.isInitialized) {
        _cameraRecordingController = CameraController(
          CameraBloc.cameras[0],
          ResolutionPreset.high,
          enableAudio: false,
        );
        await _cameraRecordingController.initialize();
        await _cameraRecordingController.prepareForVideoRecording();
      }

      if (_cameraImageStreamController == null ||
          !_cameraImageStreamController.value.isInitialized) {
        _cameraImageStreamController = CameraController(
          CameraBloc.cameras[0],
          ResolutionPreset.medium,
          enableAudio: false,
        );
        await _cameraImageStreamController.initialize();
      }
.
.
.
// start recording and imagestream one -by-one
        await _cameraRecordingController.startVideoRecording(event.filePath);
        await _cameraImageStreamController.startImageStream((img) {
          // Do image processing here
        });
.
.
.
.
// finally stop the controllers
        if (_cameraImageStreamController.value.isInitialized)
          await _cameraImageStreamController.stopImageStream();

        if (_cameraRecordingController.value.isInitialized)
          await _cameraRecordingController.stopVideoRecording();

The two controllers are for recording and imagestream respectively.
Do you think this is the right approach of using this plugin for my use-case (I was getting some weird behavior when doing this ex. the camerapreview was hung on a single frame many times)? Is there a better way of doing this (even if not using this plugin)?

Hi @ankitmundada
It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


$stateChangeSuccess on $rootScope is fired before $destroy is called on previous scope.

This is somewhat similar to issue angular-ui/ui-router#2485 but the difference being $stateChangeSuccess is registered on $rootScope rather than the $scope of the controller.
In this case, state change success event on root scope is fired before $destroy is triggered on the previous scope.
Is this event order expected/desired or should this order even matter?
https://jsfiddle.net/SHarprit/zLnayf00/

This is expected behavior.  Read this comment in the linked issue: https://github.com/angular-ui/ui-router/issues/2485#issuecomment-184234578
You can see it easily in the source code: https://github.com/angular-ui/ui-router/blob/legacy/src/viewDirective.js#L206-L208
In legacy codebase, views are updated in response to a $stateChangeSuccess
Closing because I've answered the question


Add strategy for ERC721 contract holding an ERC20 token balance

Changes proposed in this pull request:

Adds a strategy for granting voting power to holder of a 1/1 ERC721 proportional to an embedded ERC20 token balance held in the ERC721 contract

Closing this as stale, feel free to reopen if needed


Broken DBT 18 Update: Ubuntu, DBT, Brew

Describe the bug
Currently, I get the below stack trace when running dbt --version while trying to upgrade to dbt 18.1. I imagine this has something to do with my brew/python setup environment but am not quite sure exactly how to debug after about a day of brute forcing various solutions. Thanks for your help.
Steps To Reproduce

Install brew
Install Git and DBT via Brew
Run dbt --version

Expected behavior
dbt --version to return the updated version
System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[x ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
Traceback (most recent call last):
  File "/home/linuxbrew/.linuxbrew/bin/dbt", line 33, in <module>
    sys.exit(load_entry_point('dbt-core==0.18.1', 'console_scripts', 'dbt')())
  File "/home/linuxbrew/.linuxbrew/bin/dbt", line 25, in importlib_load_entry_point
    return next(matches).load()
  File<EMAIL_ADDRESS>line 77, in load
    module = import_module(match.group('module'))
  File<EMAIL_ADDRESS>line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 790, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/dbt/main.py", line 11, in <module>
    import dbt.version
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/dbt/version.py", line 100, in <module>
    installed = get_installed_version()
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/dbt/version.py", line 29, in get_installed_version
    return dbt.semver.VersionSpecifier.from_version_string(__version__)
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/dbt/semver.py", line 100, in from_version_string
    return cls.from_dict(matched)
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 618, in from_dict
    cls.validate(data)
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 957, in validate
    schema = _validate_schema(cls)
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 183, in _validate_schema
    schema = schema_cls.json_schema()
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 942, in json_schema
    return cls._json_schema_recursive(
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 913, in _json_schema_recursive
    schema = cls._schema_defs_from_cache(definitions)
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 900, in _schema_defs_from_cache
    schema = cls._collect_json_schema(definitions)
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 863, in _collect_json_schema
    properties[target_field], is_required = cls._get_field_schema(
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 809, in _get_field_schema
    field_schema, required = cls._get_schema_for_type(
  File "/home/linuxbrew/.linuxbrew/Cellar/dbt/0.18.1_1/libexec/lib/python3.9/site-packages/hologram/__init__.py", line 783, in _get_schema_for_type
    raise ValidationError(f"Unable to create schema for '{type_name}'")
hologram.ValidationError: Unable to create schema for 'Optional'

The operating system you're using:
Ubuntu 18.04.5 LTS (GNU/Linux 5.4.0-1028-aws x86_64)
The output of python --version:
Python 3.9.0 (I do have multiple installs of python which it seems to repoint to depend on a change in bashprofile)
Additional context
$ python --version
Python 3.9.0

$ brew --version
Homebrew 2.5.8
Homebrew/linuxbrew-core (git revision 071501; last commit 2020-11-03)

My best assumption is the issue lies here, but not sure.
$ whereis dbt
dbt: /home/linuxbrew/.linuxbrew/bin/dbt

$ whereis python
python: /usr/bin/python3.6 /usr/bin/python3.6-config /usr/bin/python2.7 /usr/bin/python2.7-config /usr/bin/python /usr/bin/python3.6m-config /usr/bin/python3.6m /usr/lib/python3.6 /usr/lib/python3.8 /usr/lib/python2.7 /usr/lib/python3.7 /etc/python3.6 /etc/python2.7 /etc/python /usr/local/lib/python3.6 /usr/local/lib/python2.7 /usr/include/python3.6 /usr/include/python2.7 /usr/include/python3.6m /usr/share/python /home/linuxbrew/.linuxbrew/bin/python3.9 /home/linuxbrew/.linuxbrew/bin/python3.9-config /home/airflow/onedrop-scripts/transform/dbt/dbt-env/bin/python /usr/share/man/man1/python.1.gz

$ whereis brew
brew: /home/linuxbrew/.linuxbrew/bin/brew

$ which dbt
/home/linuxbrew/.linuxbrew/bin/dbt

$ which python
/usr/bin/python

$ which brew
/home/linuxbrew/.linuxbrew/bin/brew

Thank you so much

duplicate issue


SSR SourceMap not loaded when debugging

Describe the bug
Currently it looks like when debugging the node process with source maps enabled (node --inspect --enable-source-maps ./source.js) that the source maps are not actually loaded.
Reproduction
Load any file via ssrLoadModule with a debugger.
You will see that the debugger shows compiled code instead of the source.
eg:
source.js
const { createServer } = require('vite')

(async () => {
  const server = await createServer({ server: { middlewareMode: true } });
  await server.ssrLoadModule("./debug.js");
})();

debug.js
debugger;
console.log("debugging", import.meta.url);

Then run
node --inspect-brk --enable-source-maps ./source.js

And finally connect your favorite debugger.
System Info
Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:
  System:
    OS: macOS 10.15.7
    CPU: (4) x64 Intel(R) Core(TM) i7-7567U CPU @ 3.50GHz
    Memory: 504.64 MB / 16.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 14.16.1 - ~/.nvm/versions/node/v14.16.1/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.12 - ~/.nvm/versions/node/v14.16.1/bin/npm
  Browsers:
    Edge: 90.0.818.51
    Firefox: 87.0
    Safari: 14.1
  npmPackages:
    vite: ^2.2.4 => 2.2.4

Used package manager: npm

Before submitting the issue, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the docs.
[x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[x] Provide a description in this issue that describes the bug.
[x] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[x] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

@DylanPiercey yes.

I tried this PR against vite-plugin-ssr's test suite. They fail because the following piece of code doesn't work anymore.
// Check whether `val` is a plain JavaScript object
function isPlainObject(val) {
  return val && val.constructor === Object;
}

The vm.runInNewContext seems to create a second Object value.
I couldn't find an alternative way to implement isPlainObject(). I could dig deeper and maybe find a solution, but this is a (slight) problem for vite-plugin-ssr.
If I remove all isPlainObject() instances, then all tests go green though.
The --enable-source-maps flag is still Stability: 1 - Experimental. Not being able to trace errors is one of the most painful experience; I'm not sure we want to be experimental here?
Also, forcing my users to use --enable-source-maps is not ideal. I'm wondering if that would also apply to SvelteKit users: would they need to svelkit dev --enable-source-maps? Once Node.js native Source Map support stabilizes, they'll probably then remove the flag.
It would be super neat to get rid of ssrFixStackTrace, but maybe we should wait a bit for Node.js's Source Map support to stabilize?

Hi,
Is there currently any plan or roadmap to fix this issue  ?
I am currently working on a project with SvelteKit and one of the only things I'm missing painfully is the option to debug server side code.
Currently the only option is to put console.logs everywhere but on the server side especially in complex scenarios that's a big pain point.
In my opinion this is critical for a better developer experiance, especially people that come from other languages will turn their back on something like svelteKit when they find out that they can't debug properly, instead have to use the console output.
Thanks in advance for help ;D

This is a huge development workflow blocker - would be great to get fixes merged in!

@gsxdsm Now there's a solution: https://github.com/cyco130/vavite/pull/17
It may one day make it into Vite itself, maybe.

Thanks cyco! Checked out vavite - very cool! I've been trying to get it working with Sveltekit (or at least the node-loader package into vite) but not having any luck. Will keep tinkering with it.

but not having any luck

Here's an example repo. Feel free to open an issue at vavite's repo or hit me on Rakkas's Discord channel.

That worked! Thank you so much!!


Serialization of Schema breaks swagger-ui Try out section for making requests

Why did you do serializing type property as Set in https://github.com/swagger-api/swagger-core/blob/master/modules/swagger-core/src/main/java/io/swagger/v3/core/jackson/mixin/Schema31Mixin.java#L45?
I believe it should be with Object type which maps to String or Set depending on values on serializing step.
It breaks how swagger-ui works in the line https://github.com/swagger-api/swagger-ui/blob/2be8b26f7c9709a2b4b23bc3b57fc68a15e81ad4/src/core/utils/index.js#L453 - you cannot send any requests in Try out.

The problem is on our side.


Adp cif naming

Add aliases for Uiso and Biso to follow the CIF naming convention
Add default ADP (Uiso=0) if it is not defined when atom site object is created

Addressed in https://github.com/EasyScience/EasyCrystallography/pull/64


Don't scrape examples from library targets by default

What does this PR try to resolve?
Based on some early feedback about the scrape-examples feature, both documentation authors and consumers did not consider examples useful if they are scraped from a library's internals, at least in the common case. Therefore this PR changes the default behavior of -Zrustdoc-scrape-examples to only scrape from example targets, although library targets can still be configured for scraping.
How should we test and review this PR?
I have updated the docscrape tests to reflect this new policy, as well as the Unstable Options page in the Cargo book.
r? @weihanglo

Thanks for suggestions, just incorporated both.

Thanks!
@bors r+

:pushpin: Commit eb829cfb35c36da1d68f2d146303c43b56f30031 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit eb829cfb35c36da1d68f2d146303c43b56f30031 with merge 74c7aab19a9b3a045e7af13319409a9de2cf4ef7...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 74c7aab19a9b3a045e7af13319409a9de2cf4ef7 to master...


Entity List template CSV gets opened and saved by Excel as not UTF-8

Steps to reproduce the problem

Download an Entity List template CSV
Double click to open in Excel
Populate a cell with non-ASCII characters (e.g. Hélène)
Save from Excel
Upload the Entity List CSV

Screenshot

Expected behavior
Excel opens and saves the file as UTF-8
Central version shown in version.txt
versions:
67bd0ad3def294a89fc17a8be8e1001a37303f80 (v2024.1.0-15-g67bd0ad)
+3372015d4fe4ba34598ab5e4552c1ec1e11ee205 client (v2024.1.0-41-g3372015d)
 3ae8a69011f3839b60ecc9fdda770faa408d16d4 server (v2024.1.0-35-g3ae8a690)

Other notes (if any)
This is related to https://github.com/getodk/central/issues/752. I think that in this case we can add the BOM because this template exists just to make it easier to input data. We generally don't expect the template to be ingested into data pipelines or edited by scripts.

I tried to reproduce this error, but I have the same problem, which I think has been in the past with Libre Office on ubuntu.
I saved the file as UTF-8 and when trying to upload it to central i get this error:

Excel saves file with semicolon instead of comma.
I have Excel version from 2007

Can you please share the file saved by Libre Office.
For Excel, I think your system "List separator" setting is ; or Excel decimal separator is ,.
List separator in windows:

Click the Start button and then Control Panel
Open the dialog box for changing Regional and Language settings
Find the List separator setting
Enter ,

Decimal and thousands separator in Excel:

File > Options > Advanced
Set decimal separator to . and thousands separator to ,

More details: https://www.ablebits.com/office-addins-blog/change-excel-csv-delimiter/

@sadiqkhoja The instruction was helpful, now everything is saved correctly.
But is this fix available on staging?
I was able to reproduce the case from @lognaturel and also other cases with polish signs:

colors6.csv
I also see that my version of staging is different:
versions:
51a3e10fdf188473ce38b24313962c674188c9ff (v2024.2.1-4-g51a3e10)
+ba422a03b6ec987536f1c4d8388d09e8f0ce67b3 client (v2024.2.1-39-gba422a03)
+663e3b9e0c0f2bc10037d3a83f951494b4e27b64 server (v2024.2.0-97-g663e3b9e)

The file colors6.csv doesn't have BOM character in it. I am wondering how that file was created/edited.
On a separate note semi-colon separated CSV file is parsed and uploaded correctly. File attached.
colors<PHONE_NUMBER>3146.csv
@srujner I would like to do a sync session with you to see what's happening. Sent you an email invite.

@sadiqkhoja I accepted the meeting


Identify and invite target stakeholders to Dec workshop

https://docs.google.com/document/d/1UPd1OH7kv8si5kriJXrDeGJp5ECLgJJvYwPq3pAZdBE/edit?usp=sharing
https://hackmd.io/sGV_FSsNQAutPq-Si4-Uyg?both

@dingaaling and project manager (replacing Harry) to discuss this after Jen is back

Team to review agenda, as discussed in today's weekly meeting.

Invitation Template:
Subject: Invitation for UK Climate Projections (UKCP) bias correction workshop
Dear [INSERT NAME HERE],
On December 11th 2023, The Alan Turing Institute will host an in-person workshop from 10:00-16:00 at our offices in the British Library (London) focused on UKCP bias correction methods. This event aims to bring together researchers and practitioners working with climate projections data to explore different approaches to bias correction across public health, climate science, and engineering domains.
This workshop is being co-organised with the Met Office UKCP team including Dr Fai Fung and Dr James Pope and will feature a resource that Turing is co-developing with their team: clim-recal, an open repository of method comparison and guidance for bias correction of UKCP18 products. This project is led by Professor Gavin Shaddick and Professor Mark Birkin.
Given your expertise and your work with climate data, we would like to invite you to attend the workshop. We sincerely hope you will be able to join us and would appreciate if you could let us know by 30 November if you can attend.
For more information about the workshop and project, please refer to this workshop document, and please do not hesitate to reach out if you have any other questions.
Best wishes,
Fai Fung (Met Office) and Jennifer Ding (The Alan Turing Institute)
Workshop co-organisers

Target people to invite:
From Turing/Turing partners

Project Team: Ruth, Greg, Jen, Griff, Bowen, Sophie, Monica
Alisha Davies
KCL researchers?
UA researchers?

From Met Office:

Fai Fung
James Pope
Laura Dawkins

From Met Office Partners:

Environmental/engineering consultancies
NERC Research institutes
University-based researchers
Infrastructure operators

Invite List (Target: 20)
Project Team (6)

[ ] Ruth Bowyer
[ ] Greg Mingas
[ ] Griffith Rees
[ ] Bowen Zhang
[ ] Sophie Arana
[ ] Monica Vakil-Dewar

Met Office (3)

[ ] Fai Fung
[ ] James Pope
[ ] Laura Dawkins

Met Office Stakeholders (4)

[ ] Env/engineering consultancies
[ ] NERC Research institutes
[ ] University based researchers
[ ] Infra operators

KCL (2)

[ ]  Geographer
[ ] Heat Physiologist

Wider Turing (5)

[ ] Health GC?
[ ] E&S GC?
[ ] UA

Need for a hybrid component

Our target participant:

UKCP18 user
Familiarity with Python or R
Interest in bias correction

Final participant list by 24 Nov


If rustup toolchain dir includes non-directory entries, espup install panics

Bug description
When a rustup toolchain dir managed by espup (ie: ~/.rustup/toolchains/esp) includes a non-directory entry (ie: a normal file) at the top level, espup panics when it attepts to remove that file using remove_dir_all()
To Reproduce
Steps to reproduce the behavior:

espup install
touch ~/.rustup/toolchains/esp/hello.sh
espup install
Note the panic

Expected behavior
espup should either return an error or (as it's already wiping out the directory anyhow) remove the file.
Environment

OS: nixos
espup version: 0.5.0

Thanks for reporting the issue! Just opened #346 fixing this, now when reproducing the issue you get:
Error: espup::remove_directory

  × ⛔  Failed to remove '/home/esp/.rustup/toolchains/esp/hello.sh'.

Do you mind trying the fix to validate it works?
cargo uninstall espup
cargo install espup --git https://github.com/SergioGasquez/espup --branch fix/rustup-panic
mkdir ~/.rustup/toolchains/esp/ && touch ~/.rustup/toolchains/esp/hello.sh
espup install

It seems to kind-of resolve the issue, though there's still a panic that might be related before that (after some repeated uninstall attempts).
[2023-09-07T14:50:18Z INFO ] 🔧  Uninstalling Xtensa Rust toolchain
[2023-09-07T14:50:18Z WARN ] ⚠   Installation for 'Xtensa Rust' failed, retrying
[2023-09-07T14:50:18Z INFO ] 🔧  Uninstalling Xtensa Rust toolchain
[2023-09-07T14:50:18Z WARN ] ⚠   Installation for 'Xtensa Rust' failed, retrying
[2023-09-07T14:50:18Z INFO ] 🔧  Uninstalling Xtensa Rust toolchain
[2023-09-07T14:50:18Z WARN ] ⚠   Installation for 'Xtensa Rust' failed, retrying
[2023-09-07T14:50:18Z INFO ] 🔧  Uninstalling Xtensa Rust toolchain
[2023-09-07T14:50:18Z WARN ] ⚠   Installation for 'Xtensa Rust' failed, retrying
thread 'tokio-runtime-worker' panicked at 'called `Result::unwrap()` on an `Err` value: SendError { .. }', /home/cody/.cargo/git/checkouts/espup-c286dba105940cd2/fe2e25d/src/toolchain/mod.rs:248:32
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Error: espup::remove_directory

  × ⛔  Failed to remove '/home/cody/.rustup/toolchains/esp/hello.sh'.

Given that we're already removing all the other files under this directory, it probably would make sense to resolve by removing the top-level files as well.

Given that we're already removing all the other files under this directory, it probably would make sense to resolve by removing the top-level files as well.

Just updated the branch to delete everything! Mind validating it?
cargo uninstall espup
cargo install espup --git https://github.com/SergioGasquez/espup --branch fix/rustup-panic
mkdir ~/.rustup/toolchains/esp/ && touch ~/.rustup/toolchains/esp/hello.sh
espup install

Looks good to me 👍


[$500] Chat - Some skin colors are only affecting one of the faces

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Open a chat
Open the emoji picker
Scroll down until you see couples with a heart above them
Switch between all skin colors

Expected Result:
All skin colors should affect both of the faces or just only one of them.
Actual Result:
The yellow color is appecting both of the faces. All other skin tones are only affecting the right face.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/43996225/332e5e9e-2e18-45e2-bf0d-b44210097b08

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0105e72483a5f5d73f
Upwork Job ID:<PHONE_NUMBER>764239360
Last Price Increase: 2024-01-30

Proposal
Please re-state the problem that we are trying to solve in this issue.
The yellow color is affecting both of the faces. All other skin tones are only affecting the right face.
What is the root cause of that problem?
Did some investigation related to the EmojiPicker component and found out the bug is related to the index used to fetch the filteredEmojis based from the selectedSkinTone. From the index of the preferredSkinTone in the EmojiPickerMenu, the system uses an index system that ranges from [-1 to 4]. And the index generated from the selected skin tone is used to Map to the emojis types which is an array gotten from assets/common.
Currently there is 25 items in the array for the emoji with the couple_kisses and other related emojis, The problem is that the mapped emoji from the preferredSkinTone index is mapping to the wrong index in the emojis types.
What changes do you think we should make in order to solve the problem?
If we can make a complete refactor of the emojis assets and place the right emojis based on the preferredSkinTone selected from the user, we fix the problem. Therefore, if the preferredSkinTone is having an index of 4, we are mapping the types from the refactored emojis to equal the right emoji types[4].
I made a refactor of the emojis assets file and also made some regression tests locally to make sure all works fine as well.
P.S: In the creation of the emoji assets creation, i noticed there were some emoji items which were not of use anymore and were collecting memory in the array. I removed those emojis to make sure we have a consistent emoji system across the application and to prevent future bugs from reoccurring.
Attached is the solution working on my local machine awaiting approval.
https://github.com/Expensify/App/assets/47699250/7ace505d-f759-4766-a123-5475f612bf57
@kavimuru Please have a look. Thank you.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01457ed5bd560af031

Reviewing now @MelvinBot!

Thanks for the proposals everyone. Both proposals have the correct RCA and the solution is more or less the same as well.
Since @dukenv0307 was the first one to propose the solution, I think we should proceed with there proposal.
🎀 👀 🎀 C+ reviewed

@allroundexperts  Yeah i agree with you 💪. But seems there is an auto-assignment from bot already 😂

@dukenv0307 - I've paid you via Upwork and closed the contract.
@allroundexperts - I'll check back in later today for the checklist and reg test steps, then move your payment forward as well. Otherwise, I'll check tomorrow morning if I can't.

@allroundexperts - Can you complete the checklist answers here as soon as you can so we can close this out?

Checklist

I wasn't able to pinpoint the exact PR which caused this. It seems like we never considered adding all skin colours for every emoji.
N/A
N/A
I think a regression test might not be strictly needed here since this is a really minor design thing with no impact on the app functionality. If we still decide to add it, then the steps given in the OP look good enough to me!

Thank you @allroundexperts - I'll finish up and I agree with you regarding the reg test.

Payment summary

[x] Contributor - @dukenv0307 - Paid $500 via Upwork
[ ] C+ - @allroundexperts - Owed $500 via Manual Requests

@JmillsExpensify - Money request incoming from @allroundexperts for $500.

@allroundexperts - Closing this out now but feel free to put in the request since the payment summary is above.


Check if codecov likes 2 unit test tasks

Replace #198
Proposed Changes

Java unit tests in a separate gh action

Closed in favour of #213

^
|

/close


No Refactor button in Animation editor

No Refactor button is added in the Animation drop down menu.
I'm using Godot 4.3 and this is the only addon I have in my project.
I downloaded the addon through AssetLib and restarted Godot just incase that mattered.
Since I tried this addon only with 4.3 I do not know if this is an issue specific to 4.3

Hi, can you give me a minimal reproduction project? And just to confirm, you've activated the plugin in the Project Settings, right?

Well, this is embarrassing. I didn't realize i had to activate it. I haven't messed with addons at all before this.
It's working now, though I had to do a little refactoring to avoid naming conflicts.
Thank you for your quick response.


Enable AAD authentication for agent pool only clusters

What this PR does / why we need it:
This PR enables AAD support in API version 2018-03-31
Exert of changes to API are:
 "properties": {
    "enableRBAC": true,
    "aadProfile": {
        "clientAppID": "<NATIVE_APP_ID>",
        "serverAppID": "<WEB_APP_ID>",
        "serverAppSecret": "<WEB_APP_Secret>",
        "tenantID": "<AAD_TENANT_ID>"
    }
}

Special notes for your reviewer:
API version 2018-03-31 will enable AAD integration using Webhook authentication method (and is not enabled in ACS Engine generated templates yet). ACS Engine generated templates support AAD integration via OIDC authentication and this remains as is.

@mboersma  @jackfrancis  @slack PTAL. Thanks!

@jim-minter fyi - we should probably be using AADProfile, too.


Horizontal tabs for groupings

Is there a plan to integrate this setting to this package?
https://novapackages.com/packages/digital-creative/resource-navigation-tab

Hi! PR-s are welcomed. :)

will definitely do this. :)

Hi!
Version 2.6.0 now adds support for subpages and nova-tabs.
Give it a try! Good luck!


Intersection type does not subtype with union type that contains the same intersection

TypeScript Version:  3.3.3
Search Terms: intersection type, union type
Code
I was trying to simulate nominal types with unique symbol like this:
export type AccountID = number & typeof AccountID;
declare const AccountID: unique symbol;

declare const id: AccountID;
const test: AccountID | undefined = id;

A narrower reproducing case is this:
declare const a: number & string;
const b: (number & string) | undefined = a;

Expected behavior: This example should pass type checking.
Actual behavior:
This example fails with:
Type 'number & string' is not assignable to type 'undefined'.

Playground Link

Interestingly, intersecting with an object type works:
declare const a: number & {};
const b: (number & {}) | undefined = a;

Which solves my original use case:
export type AccountID = number & {type: typeof AccountID};
declare const AccountID: unique symbol;

declare const id: AccountID;
const test: AccountID | undefined = id;

Still posting a report in case this behavior was unknown.

number & string is never, same as number & symbol - values of that kind cannot exist. Naturally you cannot assign from such a value into any type.
See #202 for alternate workarounds.

In this case, I’m using intersection types for their algebraic properties and not their set-theoretic properties. Replacing number & string with never does not error. Should the never case error or the number & string one pass?
declare const a: never;
const b: never | undefined = a;

When encountering (number & string) | undefined, the expansion of (A & B) | C to (A & C) | (B & C) occurs before the number & string -> never reduction. So TS sees
declare const a: never;
const b: (number & undefined) | (string & undefined) = a;


php artisan infyom:rollback $MODEL_NAME$ $COMMAND_TYPE

This command is not working properly. Givin error file_get_contents(C:\xampp\htdocs\laravel-infyom\resources\views/layouts.menu.blade.php): failed to open stream: No such file or directory
Which is not creating menu in layouts/menu.blade.php, so we did changes in /vendor/infyomlabs/laravel-generator/src/Common/GeneratorConfig.php at line :542 to $this->addOns['menu.menu_file'] = config('infyom.laravel_generator.add_on.menu.menu_file', 'layouts/menu.blade.php');
Then it starts working

Hello @Sanjeev-Singla
we have just tested it and its working fine for us.

may be you have did custom changes, that's why its not working


Document lock isn't cleared when user leaves edit mode

The document lock should be cleared when the user is no longer editing the report. However, we heard from a user that this is not the case. Two users tested editing the same report, and also viewing the same report. They said:

Caroline and I tested both options. When she was in the document in "Viewing" mode and when she was not logged in the ATP tool. In both cases we got the warning.

I'm not sure how the second scenario is possible because users can only see published documents when they're not logged in. I assume they are using production?
We should investigate why the report locks are not being cleared.

FYI @naomatheus

Adding this issue as P2 fix to FY23.2.3.   @bwbaker1 use Camille or Ashley to test this and better describe the issue so we know status and can add to next sprint

Hi @bwbaker1, this should be fixed now.

@wrynearson Excellent. Will test asap and let you know how it goes.

@wrynearson We tested this and could not reproduce the original error. This is ready for production.
Could we change the color of the warning message box to red? Just to make it stand out a bit more.

@wrynearson ⬆ okay. There was a little concern when we were testing this that some users may click through the warning without reading it. I think that is a low risk though. This issue is ready for production. Thanks for the feedback.

Thanks @bwbaker1. If we get user feedback that they're not acknowledging the warning, we can reassess