Differential checks (downloads) a very big (not uploaded) folder in the root

I have the following structure in the root of my bucket:
img/
css/
very-big-folder/
index.html
In my local folder the very-big-folder doesn't exist. However, it still looks like it is downloaded to be evaluated for differential matches.
I'm using version 0.9.4.

This is strange. I need to take a deeper look at it, can't really see what it can be (besides a bug) from the top of my head. Can you try the latest version?

I've tried it now with the latest version, and the same thing happens.
Can you add an "exclude" list for differential checks maybe ?

Hi, sorry for the slow reply. I haven't had the time to investigate, I still suspect it's a bug in the code that should be able to be fixed without adding features. However, I could be wrong, and in this case adding an exclude list would be a great solution.

This is what I see:
In the upload task, you list the contents of the bucket - https://github.com/MathieuLoutre/grunt-aws-s3/blob/0.14.0/tasks/aws_s3.js#L806 recursively (according to https://github.com/MathieuLoutre/grunt-aws-s3/blob/0.14.0/tasks/aws_s3.js#L329-L331)

listObjects will indeed look at everything in the bucket that starts with the given prefix (which in the case of the differential upload is the dest).  In some cases dest can be the root and hence the whole bucket, but even so, listObjects isn't supposed to download anything, particularly when uploading, it's only listing the files on the bucket and uses MD5 hash returned to check what exists already.
Is your file downloaded on your hard drive or is it listed as part of the log?

So if I have a folder in my destination bucket (which is not in the input folder on my computer), and that folder has millions of files in it - it'll take a lot of time to go over all of these files, even though that folder doesn't interest me as I don't upload anything to it.
So we do must have an "exclude" list of what to ignore - even from just listing it for MD5 hashes.

Right, makes sense now! I guess if you can't avoid using the root as a dest, you do need to be able to exclude some paths. However, I'm not sure how to best do that:

I can only list files/folders using a prefix
if there's not prefix it'll scan everything
in your case your config doesn't seem to have a yield a list of prefixes
When listing with no prefix, AWS doesn't provide a way to filter the results/exclude a path

A potential solution would be to list to the top files/folders and try to work out the exclude from there but it seems like a pretty crazy thing to do (what about depth? and if you have a million top folders?)
Maybe it would come down to be able to provide the folders to check with differential rather than the ones to not check? How does that sound?

If the user provides the source folder - why not list only the root (using the prefix), and then only the folders that are in the source folder ?
So if my local source folder is something like this:

FolderOne
FolderTwo
FileOne.ext
FileTwo.ext

and my S3 bucket contains:

myPrefix/FolderOne
myPrefix/FolderTwo
myPrefix/FolderThree
myPrefix/FileOne.ext
myPrefix/FileTwo.ext
myPrefix/FileThree.ext

and the prefix is "myPrefix", you list all files (not recursively) in S3 under "myPrefix", and then you only list (recursively) myPrefix/FolderOne and myPrefix/FolderTwo, but not myPrefix/FolderThree as it's not in the source folder.

Ping

Sorry I've neglected this issue, I haven't had much time to focus on this project lately.
Can you try to console.log(unique_dests) on L805 and L800 of the latest version? (https://github.com/MathieuLoutre/grunt-aws-s3/blob/0.14.1/tasks/aws_s3.js#L805, https://github.com/MathieuLoutre/grunt-aws-s3/blob/0.14.1/tasks/aws_s3.js#L800)
I'd like to see what the current filtering system isn't doing well. If there's nothing in the root folder that you're trying to upload, it shouldn't scan the whole thing.


Update BioC2015 instructions to use C2D Bioconductor.

http://googlegenomics.readthedocs.org/en/staging/workshops/bioc-2015.html

@mbookman I'm going to merge this now but please let me know what feedback you have - thanks!

LGTM


Added updateWhenRebuild to rebuild for some reason.

In my use case, I need to create a single-selection effect, so that each time an item is clicked, the appearance of the other items also needs to be updated simultaneously (if necessary). After a quick look at the source code, I realized that the refresh was not happening in the didUpdateWidget function; instead, it was using the previously cached widget. Therefore, I added the updateWhenRebuild property for this purpose. However, this is a dreadful PR as it modifies the SDK version. Is there any other way to meet my requirements?

@parcool  Hi, i understand your needs, SortableWrap will provide a setState capability to the caller for making any changes to the view.


NFSP CNN + Updated Imperfect Recall Dark Hex

NFSP part is not fully complete, mainly uploading not to have multiple PRs for the same purpose.
IR-DarkHex updates work fine on the tests. When trying to generate new playthroughs, I get a weird shape on ObservationTensorShape,  It works well when I remove the function and inherit the old one (even though the numbers are wrong in that case), so I think the issue is on
https://github.com/deepmind/open_spiel/blob/7e6473cc6e64b1131b79553e4823ff3910f58c95/open_spiel/games/dark_hex.cc#L341-L343
I am not using num_cells_ since it acts weird when I make it public or protected instead of private. Addressing #789

Yeah, I am using the updated ones for my experiments right now, but couldn't find time to clean up. Will do it in couple of weeks hopefully.
Yeah subclass will be better I think, I will do that.

Closing for now, feel free to re-open a new PR.


Update chaos-bugbounty-list.json

Added entry for BugPoC Bug Bounty program

Thank you @rykkard for the PR.


Add auto tagging

Mailgun has decent analytics, it would be good to auto-tag messages going out by default with a general, non identifying tag or tags related to that source.

See d9280a20f7e81cf9460368a6c196d7b0ba85631d


refactor(project): split update method into four individual methods (DSP-1612)

resolves DSP-1612

After discussing with Ivan about the best way to go about this, we will close this PR without merging. Instead we will stick to the old way of having one method that deals with updating the project entity as a whole. The event model has also been changed to reflect this.


Upgrading gradle to 4.0.1

The left-shift operator (<<) is deprecated, and the recommendation is
to use doLast instead.

Codecov Report

Merging #342 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #342      +/-   ##
===========================================
+ Coverage     72.65%   72.7%   +0.05%
  Complexity      409     409
===========================================
  Files            81      81
  Lines          2000    2000
  Branches        164     164
===========================================
+ Hits           1453    1454       +1
+ Misses          477     476       -1
  Partials         70      70

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/google/api/gax/grpc/ApiExceptions.java
25% <0%> (+12.5%)
1% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


tflite label_image: add -llog and update doc

-llog needed for Android after 2c623eaa
update label_image.md
a. reflect changes
b. where to get test model and data
c. describe --cxxopt=-DTFLITE_PROFILING_ENABLED

Nagging Assignee @protoget: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 60 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 75 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

close this because lite is not longer in contrib/. Will send another one later


blockbuster for replicon Chr1 exited with status -11

Hi!
I met the error (blockbuster for replicon Chr1 exited with status -11) when I run PEAKachu, then I ran this ciommand #blockbuster.x -minBlockHeight 10 -print 1 -distance 1 my.bed# and it is successful. Could you help me about ma error?
Thank you very much!

Hi,
Did you run blockbuster.x on the BED file for Chr1 generated by PEAKachu or some other file?
How did you install PEAKachu and blockbuster, via conda?

Hi!
Thanks for your reply! I installed PEAKachu and blockbuster via conda. And i find some error in my bam file later, when I use right bam file ,there is no error. Thanks again!


chore: bump cardano-node to 8.1.2

Closes # .
Pre-review checklist

[X] All code has been formatted using our config (make format)
[X] Any new API features or modification of existing behavior are covered with tests
[X] The template (templates/ctl-scaffold) has been updated
[X] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Changed, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

Currently, I am facing  two problems:

Empty vector from Ogmios 6.0.0 fails (fixed in c0adae3c1 by filtering out empty vectors).
Unit test EvaluateTransaction fails.


Incompatible Devices (Bricked)

I love this hack. thank you so much for developing it.  However it would be handy if you could add a signature detector to run after the backup, that could warn us for know incompatible devices.
I understand the warnings and all and I take full responsibility for the 3 devices that I just bricked.  They were all glued case devices that I hadn't been willing to pry open. Two of them were different styles but both were based on the TYWE2S module.  One of them (https://www.amazon.com/gp/product/B07CXL9ZRG) now constantly toggles one of the 4 relays once per second, while the other (https://www.amazon.com/gp/product/B07B4CNJKG) appears totally dead (I am going to see if I can resuscitate both of these with soldered connections).  The third is a dual plug (https://www.amazon.com/gp/product/B076HKHSSX) with a module labeled NX-SP201 and it appears totally dead as well (I can't find a pinout for this one, so it is probably really dead).
I have had great success with TYWE3S modules.

I had a similar experience when I flashed my power strip, where one of the relays toggled really fast for a minute or so. It might not be bricked, don't give up on it just yet!
After that I was able to continue on to configuration by connecting to the sonoff-* AP. Configuration should be similar to #50 and #71

I had devices that have seemed dead until the correct module was set in the tasmota config. Are they connecting to wifi, and can you access their web page?

I have a dual plug that also has the NX-SP201 marking - https://www.amazon.com/gp/product/B07F6X4KX3/ref=ppx_yo_dt_b_asin_title_o00__o00_s00 looks the same as the one you posted.  I got both flashed with tuya convert, but one bricked while adjusting module settings in tasmota.  working cautiously with the second one from the dual pack.

Two smartplugs https://www.amazon.de/JinvooSmart-packungen-mit-fernbedienung-versandkostenfrei/dp/B07BNJTYJW/ref=sr_1_4?ie=UTF8&qid=1549454568&sr=8-4&keywords=jinvoo+smart+steckdose converted that both worked well for a couple of hours on tasmota. One died after a configuration reset and one by itself. The only thing I can see is a faint red light when plugging in that immediately goes off. Did not try to flash them yet using a serial cable. I am puzzled what could be the problem here, devices were working properly as S20 device.

I have one of those tuya water leak sensor.
I have drawn the schematics and made some comments on.
All info so far is here:
https://github.com/uspass/Tuya-Water-Leak-Sensor
Any info is welcome.
Thanks,
us

I have converted four SP201 , details at (https://github.com/SynAckFin/TuyOTA/wiki)

Two Epicka https://www.amazon.com/EPICKA-Mini-Wireless-Compatible-Assistant-Required/dp/B07BW9QB1G dead after sucessfully flashing a Gosund SW2. No lights, no connection and no ping.

OK, I tried it on the second NX-SP201 that I had and everything worked correctly.  So, I decided to try working on the first one again.  It had been unplugged for about 5 days now.  Plugged it back in and tried to find it on the network but could not see it on my router.  Then I happened to run TasmoAdmin to try to add a couple of other plugs and guess what it found it.  Somehow, the NX-SP201 was showing up in my router with the host name "--".  I have a couple of -- devices on my network already (I think they are my DirecTV receivers), so I just wasn't seeing it.  My router, unfortunately doesn't help here, as when it assigns the DHCP addresses it does so randomly, so I can't just look at the largest IP in the list.  Anyhow, all of my devices are now working.

I have converted four SP201 , details at (https://github.com/SynAckFin/TuyOTA/wiki)

I checked out your link and noticed you had an issue with XS-SSA06 RGB.  I've been able to flash 8 of them with no issues.

@onebyside , https://github.com/ct-Open-Source/tuya-convert/wiki/Compatible-devices indicates Epicka is compatible. did you have any issues during flash ?

I flashed a Gosund successfully and then failed on 2 Epicka 1000. They both need some serial  flash attention. I never got through the backup portion of the procedure to begin with.

Did you end up flashing some Epicka's? I still have 2 of the 4 unflashed and Im itching to flash them.

From: qwertangel<EMAIL_ADDRESS>Sent: February 21, 2019 10:30 AM
To: ct-Open-Source/tuya-convert
Cc: onebyside; Mention
Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72)
@onebysidehttps://github.com/onebyside , https://github.com/ct-Open-Source/tuya-convert/wiki/Compatible-devices indicates Epicka is compatible. did you have any issues during flash ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ct-Open-Source/tuya-convert/issues/72#issuecomment-466043152, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArYrk52oXrjXaAEaGnQYvkvR1hSi2HC3ks5vPruSgaJpZM4aftT7.

Yes, I finally got them loaded using the OTA method.
Thanks,
Tim
From: onebyside<EMAIL_ADDRESS>Sent: Monday, February 25, 2019 6:16 PM
To: ct-Open-Source/tuya-convert<EMAIL_ADDRESS>Cc: Tim Hughes<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72)
Did you end up flashing some Epicka's? I still have 2 of the 4 unflashed and Im itching to flash them.

From: qwertangel <notifications<EMAIL_ADDRESS>Sent: February 21, 2019 10:30 AM
To: ct-Open-Source/tuya-convert
Cc: onebyside; Mention
Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72)
@onebysidehttps://github.com/onebyside , https://github.com/ct-Open-Source/tuya-convert/wiki/Compatible-devices indicates Epicka is compatible. did you have any issues during flash ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ct-Open-Source/tuya-convert/issues/72#issuecomment-466043152, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArYrk52oXrjXaAEaGnQYvkvR1hSi2HC3ks5vPruSgaJpZM4aftT7.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ct-Open-Source/tuya-convert/issues/72#issuecomment-467238264, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFrGo_N35CGmAEwPB0twJA_v6LVb8deKks5vRHyogaJpZM4aftT7.

Yes, I finally got them loaded using the OTA method. Thanks, Tim From: onebyside<EMAIL_ADDRESS>Sent: Monday, February 25, 2019 6:16 PM To: ct-Open-Source/tuya-convert<EMAIL_ADDRESS>Cc: Tim Hughes<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72) Did you end up flashing some Epicka's? I still have 2 of the 4 unflashed and Im itching to flash them.
…
________________________________ From: qwertangel <notifications<EMAIL_ADDRESS>Sent: February 21, 2019 10:30 AM To: ct-Open-Source/tuya-convert Cc: onebyside; Mention Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72) @onebysidehttps://github.com/onebyside , https://github.com/ct-Open-Source/tuya-convert/wiki/Compatible-devices indicates Epicka is compatible. did you have any issues during flash ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#72 (comment)>, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArYrk52oXrjXaAEaGnQYvkvR1hSi2HC3ks5vPruSgaJpZM4aftT7. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#72 (comment)>, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFrGo_N35CGmAEwPB0twJA_v6LVb8deKks5vRHyogaJpZM4aftT7.

what OTA method ?

@demtat can you please share how did you flash your epickas ? I have 3 epicka's waiting to hear from you.. :-)

@qwertangel This project, tuya-convert, is the OTA method.
@demtat Glad you recovered all your devices. I'll close this issue now, if anyone has further questions please open a new issue.

I am confused. Is OTA still possible even after epicka's stop responding ?

I think your in the same boat as I. 2 dead and need serial recovery.
Get Outlook for Androidhttps://aka.ms/ghei36

From: qwertangel<EMAIL_ADDRESS>Sent: Tuesday, March 5, 2019 8:45:04 PM
To: ct-Open-Source/tuya-convert
Cc: onebyside; Mention
Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72)
I am confused. Is OTA still possible even after epicka's stop responding ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ct-Open-Source/tuya-convert/issues/72#issuecomment-469932440, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArYrk842ma7PiZhBNklBWJ-a4zDBQSEQks5vTx2ggaJpZM4aftT7.

The OTA worked the first time, but I had an issue where my router wasn’t listing the IP that it clearly gave out via DHCP.  It showed up in TasmoAdmin and I was able to get the IP, log in, and complete the setup.
The Configuration is:
Generic
GPIO4   – Led1i
GPIO12 – Relay1
GPIO14 – Button1
Thanks,
Tim
From: qwertangel<EMAIL_ADDRESS>Sent: Tuesday, March 5, 2019 7:45 PM
To: ct-Open-Source/tuya-convert<EMAIL_ADDRESS>Cc: Tim Hughes<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72)
I am confused. Is OTA still possible even after epicka's stop responding ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ct-Open-Source/tuya-convert/issues/72#issuecomment-469932440, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFrGoxPj0Or5ZU8AooWNalCLvANxQuIkks5vTx2hgaJpZM4aftT7.

I finally flashed the last 2 epicka ota successfully. The first 2 will have to be serial flashed. I have to say the instructions aren't completly clear for newbs on parts of the process.
Get Outlook for Androidhttps://aka.ms/ghei36

From: demtat<EMAIL_ADDRESS>Sent: Wednesday, March 6, 2019 8:45:19 AM
To: ct-Open-Source/tuya-convert
Cc: onebyside; Mention
Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72)
The OTA worked the first time, but I had an issue where my router wasn’t listing the IP that it clearly gave out via DHCP. It showed up in TasmoAdmin and I was able to get the IP, log in, and complete the setup.
The Configuration is:
Generic
GPIO4 – Led1i
GPIO12 – Relay1
GPIO14 – Button1
Thanks,
Tim
From: qwertangel<EMAIL_ADDRESS>Sent: Tuesday, March 5, 2019 7:45 PM
To: ct-Open-Source/tuya-convert<EMAIL_ADDRESS>Cc: Tim Hughes<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ct-Open-Source/tuya-convert] Incompatible Devices (Bricked) (#72)
I am confused. Is OTA still possible even after epicka's stop responding ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ct-Open-Source/tuya-convert/issues/72#issuecomment-469932440, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFrGoxPj0Or5ZU8AooWNalCLvANxQuIkks5vTx2hgaJpZM4aftT7.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ct-Open-Source/tuya-convert/issues/72#issuecomment-470112284, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArYrk3-vzIGQwH-U8QBQHt3l8oiXfnzxks5vT8ZvgaJpZM4aftT7.

@onebyside good to hear that ! any specific part you want to share which I should look out for when I flash mine ?

@qwertangel   Both of the Epicka's I did had "system upgrade userbin check: user2 0x81000"
Flash2 (moves the conversion software to user2)
and then Flash3
there wont be any lights on the plug after either flash  (dont worry its not dead)
Check you wifi after and you should see sonoffXXXX

I flashed a Gosund successfully and then failed on 2 Epicka 1000. They both need some serial flash attention. I never got through the backup portion of the procedure to begin with.

What did you use as the tasmota configuration for your Gosund SW2? Can't figure it out.

I used the epicka config

I used the epicka config

Couldn't get the dimmer to work. You are talking about this switch? https://www.amazon.com/dp/B07X1QRC23

Did you not ask me about the gosund sw2 and what config I used?


When a sidebar is the parent, the child can not be made full screen with height 100%

Problem:
When putting child nodes under CopilotSidebar, these nodes can't be made height: 100% easily because of the way the DOM node are generated.
Proposed Solution:
In CopilotSidebar, change the parent DOM nodes so that the child elements can be set to height: 100%.

I would definitely love to work on this issue. Is it possible @mme to assign me to this???

I would like to solve this issue.  If possible @mme assign it to mee for devFest AI ?

You can use CSS to ensure that the navigation bar stays fixed at the top even when the user scrolls down the page in order to solve the sticky navbar issue. Code is included after each steps.
Steps to Fix the Sticky Navbar:
Step 1: Find the Navbar Element Locate the HTML element that symbolizes the bar for navigation. The appearance could resemble this:
...
Step 2 : Utilize the Stickiness CSS: The navbar in your CSS file has to have the position: sticky or position: fixed property applied to it.
For position: sticky:
.navbar {
position: sticky;
top: 0; /* This keeps it at the top of the viewport /
z-index: 1000; / Ensure it stays on top of other content */
}
This will make the navbar sticky and maintain its position at the top as you scroll down.
For position: fixed (if you want the navbar always visible even after scrolling):
.navbar {
position: fixed;
top: 0;
width: 100%; /* Ensures the navbar takes up full width */
z-index: 1000;
}
Step 3: Verify Page Content Modifications: In order to prevent overlap, you must provide some padding to the page content if you choose to use position: fixed, which will remove the navbar from the standard document flow:
body {
padding-top: 60px; /* Adjust this value based on your navbar height */
}
You can test this in your local environment before submitting a pull request to the CopilotKit repository. Let me know if you need any more assistance!
this will fix issue.


ExampleJob has not 'register' method defined.

Reading a basic example on your readme it seems that the easiest way to define a custom job is to run
class ExampleJob extends Job {
  run() {
    this.logInfo("Hello world from: " + this.data.name);
  }
}

ExampleJob.register();

What I have done:

Created a test app with Meteor <IP_ADDRESS>; classy-job 0.5 and vsivsi:job-collection  1.4.0.
execute sudo meteor --port 80 run and in a new terminal window sudo meteor shell.
run the snippet above which returned the following:

TypeError: ExampleJob.register is not a function
    at repl:1:-50
    at packages/shell-server/shell-server.js:458:25
    at /home/masha/.meteor/packages/promise/.<IP_ADDRESS>ih0cfk++os+web.browser+web.cordova/npm/node_modules/meteor-promise/fiber_pool.js:32:39

What is the easiest example which creates and runs the ExampleJob?

You should never ever run meteor with sudo.

OK I thought that I needed to add the job-collection package as well. After removing it it seems to work

I'm using sudo to access port 80. Any ways to achieve this without sudo?

You should not be developing on port 80. :-)

Yes, you should not be adding  job-collection package itself because this package uses the same name (Job) for the base class, while job-collection uses it for their jobs.


Add the command for starting markdown preview to command palatte

I would like to be able to start a markdown live preview pane via the command palette. I won't remember the keyboard shortcut.

Sorry, I forgot about that. I'll do it soon (can't right now)


[v6] Create Custom Routes

How can i create custom routes for validating roles or refresh my JWT token for example?
I tried with below function, but no sucess yet :(
import React from 'react';
import { Routes, Route, Navigate } from 'react-router-dom';

import { useAuth } from '../contexts/auth';

import Dashboard from '../pages/Dashboard';
import Employees from '../pages/Employees';
import EmployeeInfo from '../pages/Employees/info';

function AppRoute({ children, ...rest }) {
  const { RefreshToken } = useAuth();

  return (
    <Route
      {...rest}
      render={() => {
        return RefreshToken() ? children : <Navigate to="https://www.google.com" />;
      }}
    />
  );
}

export default function AppRoutes() {
  return (
    <Routes>
      <AppRoute path="" element={<Dashboard />} />

      <AppRoute path="/employees" element={<Employees />}>
        <AppRoute path="/:id" element={<EmployeeInfo />} />
      </AppRoute>

      <AppRoute path="*" element={
Not found!
} />
    </Routes>
  );
}

There is no render prop for <Route /> in RRv6. Either you use children, or element.


platform integration:add with --states option fails

The --states option doesn't seem to work any way I try to specify it on the command line:
platform integration:add --type=hipchat --project=<PROJECT-ID> --room=<ROOM-ID> --token=<HIPCHAT-TOKEN> --states=pending,in_progress,complete

Always results in an error like this:
 [HttpException]
  An API error occurred.

  Details:
  [url] https://us.platform.sh/api/projects/<PROJECT-ID>/integrations
  [status code] 400 [reason phrase] Bad Request [message] The server could
  not comply with the request since it is either malformed or otherwise
  incorrect. [detail] {"states":"Invalid values:
  pending,in_progress,complete, not in accepted values complete, in_progress,
  pending"} [title] Bad Request

I've tried a variety of different formatting options:
--states pending,in_progress,complete
--states=pending,in_progress,complete
--states="pending,in_progress,complete"
--states "pending,in_progress,complete"
--states="pending, in_progress, complete"
--states=[pending,in_progress,complete]
--states="[pending,in_progress,complete]"
Also, changing the order seems to have no effect:
--states="complete, in_progress, pending"
With error message:
 [detail] {"states":"Invalid values: complete, in_progress,
  pending, not in accepted values complete, in_progress, pending"}

The only way I'm able to specify the states is by NOT adding the command-line option, and walking through the interactive prompt:
States to report (--states)
A list of states to report, e.g. pending, in_progress, complete
Default: complete
Enter comma-separated values (or leave this blank)
> pending,in_progress,complete

Fixed in v3.29.0 (updated the dependency as a side-effect of the user:add feature)


update implementers and verifier websites

@gauravping, @jischr, @awoie, @troyronda please add urls to your verifiers that people can test against if you plan to maintain them.

@Sakurann  url to spruce verifier: https://app.vp.interop.spruceid.xyz/

Please add Ping Identity verifier at https://verifycredential.pingone.com/verifier.html

@Sakurann please add Gen verifier https://demo-rp.stg.trustbloc.dev/verifierqr


Implement methods

Implements most of the functionality needed for methods in noir.

Parsing, name resolution, type checking, and backend support are included. The last of which is achieved via the type checker desugaring method calls into normal function calls once the method in question can be resolved. So no method call nodes should be present once we reach the evaluator.
We now parse into ast::UnresolvedTypes and ast::Type is moved to hir::Type. UnresolvedType is for types that contain struct names that have yet to be resolved by name resolution. Upon name resolution, they are replaced with hir::Types in which any structs are properly linked to their definition.
Methods of a struct are stored in the struct type itself.

Notably absent from this PR is:

Fully-qualified method syntax, i.e. crate::foo::bar(obj, param). Only obj.bar(param) is currently supported.
self parameters. This isn't too much of an issue though since methods desugar into a function call syntax, the object becomes the first parameter anyways. We only lose on the convenience of not needing to specify the type of self in the method definition currently.

Edit: Both fully-qualified method syntax and self parameters are now included in this PR

Added a commit implementing 'self' and 'Self'.
Currently, I have it so that using Self outside of an impl will attempt to lookup a type in scope with the name Self. This matches our previous behavior and rust's I believe. If desired though, it is also easy enough to make using self or Self outside of an impl an error with a message explaining you can only use them in an impl.

Still not sure what the CI error was that was causing the exit status 1 for ld. Changing to macos-latest from ubuntu-latest passes the CI but doesn't get us any closer to figuring out the original issue.


Rgb role

Make role to flex, soo pidroid changes the role every minute

Suggestion sounds interesting, however that seems like the fastest way to get rate-limited into oblivion.


Connect database to interest quiz

The interest quiz is hard coded with if/else statements, it would be more efficient to connect the quiz to the database to pull out places from the database.

Estimate: 400


Adding Dockerfile

I noticed this repository is missing a dockerfile, and this old pr tried to fix that but did not succeed.
I have not tested this Dockerfile with many plugins, maybe some dependencies are still missing, but it'll provide a nice base for future expansions. It allows building of images with preinstalled addons via the --build-args NSTALL_PLUGINS="..." flag.

I think it's because the Dockerfile can be found here: https://github.com/getgrav/docker-grav

Maybe link it in the Readme? I had trouble finding the repo through google and this projects readme...

I agree the documentation could mention it.
Not sure about you but on my end it does show as the first result on Google:

Oh well, seems like I did in fact not do proper research^^


Add static builds and use in docker

Problem

See #337 for reasoning
Solution

Instead of running esy release etc we build deku statically and use a leaner scratch container. We might want to use busybox or something with a shell available.
Related

#337 - Alternative to

The build time of this is better, but just to confirm, there is caching right?

There should be caching, now that the Dockerfile is better structured we should have more opportunities for it as well. Could you mark it as non-essential for PR merges if it's too slow?


fetch refresh not working

Versions

nuxt:  v2.15.4
node: v14.15.4

Reproduction
data() {
    return {
      comments: [],
    };
  },
activated() {
    if (this.$fetchState.timestamp <= Date.now() - 10000) {
      console.log('fetchState');
      this.$fetch();
    }
  },
async fetch() {
    this.comments = await this.$strapi.find("comments/articles:" + this.page);
  },
props: {
    page: {
      type: Number,
      default: "",
    },
  },

What is Expected?
Updating data
What is actually happening?
Nothing

@danielroe It seems that I just misunderstood the principle of this hook, I thought it would work like a long pool, but it only works when the page is reloaded or switched to it and does not allow a fetch to be made if it was made 10 seconds ago


Docs

Hey, I was passing by and I didn't see any examples. It would be pretty nice if you guys show what it can do so we can have an idea of it's capabilities without having to go town first. 🙂

Hi there, we put a few guided examples in the api documentation: https://docs.rs/al-jabr/0.1.0/al_jabr/
Are these insufficient, or did you expect them to be in the README?

Hum, I see. They are sufficient. I guess just an info saying where to find the examples would be great. 😊
PS: Just a question: do you expect this library to grow in the future, or you're gonna keep it like it is, I'm just asking because there are libraries like Algebrite in javascript that has like "higher-language" purposes?

Hum, I see. They are sufficient. I guess just an info saying where to find the examples would be great. 😊

Sure thing. As a quick, tip, at the top of most readme's for crates, there is a "doc" button that looks like this:

This will link you to the documentation for the crate.

PS: Just a question: do you expect this library to grow in the future, or you're gonna keep it like it is, I'm just asking because there are libraries like Algebrite in javascript that has like "higher-language" purposes?

We do intend to continue to add features to al-jabr. If there is a feature you think we missed, feel free to add an issue , and we will implement it for you.


[stable/prometheus-operator] Default dashboards are not created using helm template

Describe the bug
None of the default dashboards are created
Version of Helm and Kubernetes:
Helm 2.14.0, 2.14.3
K8S 1.13.8, 1.14.2
Which chart:
stable/prometheus-operator
What happened:
No dashboards are created
What you expected to happen:
This configmap should be filled with all the files in charts/stable/prometheus-operator/templates/grafana/dashboards/
How to reproduce it (as minimally and precisely as possible):
helm fetch stable/prometheus-operator --untardir ./charts --untar
helm template charts/prometheus-operator -x templates/grafana/configmap-dashboards.yaml

or
helm install --name=test stable/prometheus-operator
kubectl get cm | grep grafana

Anything else we need to know:
the configmap is looking for json files, while all of the templates are yaml - However changing json -> yaml doesn't fix this at all. I don't really understand how this ever worked, according to the docs

Some files cannot be accessed through the .Files object, usually for security reasons.
Files in templates/ cannot be accessed.

These values are on by default
grep dashboard charts/prometheus-operator/values.yaml  -C3 | grep -i enabled
  enabled: true
  defaultDashboardsEnabled: true
      enabled: true
      enabled: true
      defaultDatasourceEnabled: true

Many moons ago (~8) there was a reference to dashboards living at ./dashboards - e.g.
git rev-list --all | xargs git grep prometheus-operator/dashboards | grep . - did they get lost when the chart moved? I'm not really sure how to go about fixing this. Am I missing something obvious?

FYI I found out through slack that helm template --kube-version can apparently be used for setting this Capability. It does not set other things though, see https://github.com/helm/helm/issues/3589. Meanwhile, you can now set the chart value kubeTargetVersionOverride to force generation for a target version (#16994)

Hello, How did u fix the issue?

Set kubeTargetVersionOverride in your helm values
E.g. kubeTargetVersionOverride: 1.14.6


problems with restic backup during update of s3 storage

Dear restic team,
I have restic in use as backup client with s3 repositories on a netapp storage.
During an update of the netapp software is run on the storage, restic runs into problems. The storage seems to be giving http 500 error codes to the restic client. restic does show it encounters errors and retries, however the backup finishes successfully with exitcode 0. A subsequent repo check shows errors in the repository (many packs are claimed to be missing). I have attached a log of this below.
My questions are:

how come restic is missing packs in this situation?
why does restic give exitcode 0, when the backup apparently is inconsistent?
does restic backup do HTTP PUT before each HTTP HEAD?

In the current situation, the repository errors could be fixed by a combination of prune/rebuild_index. However, I have had similiar situations in which also snapshots were corrupt and had to be removed manually.
I would be very gratefuly if you can give me any feedback on this matter.
Output of restic version
0.12.0
How did you run restic exactly?
restic backup, parameters s3 repository and certificate
What backend/server/service did you use to store the repository?
netapp S3
Expected behavior
Either retry writing until all backup data is written successfully/consistently to the repository or abort the backup job altogether with an error.
Actual behavior
The backup finishes with exitcode 0 but a subsequent restic full check shows the repository has inconsistencies. See the attached:
log.txt
Steps to reproduce the behavior
It can be reproduced during updates of the s3. Please contact me for further information.
Do you have any idea what may have caused this?
A short outage of the connection to s3; however I think restic should handle this differently/more robust.
Do you have an idea how to solve the issue?
Try to handle the write errors more robust or abort the backup job with error.
Did restic help you today? Did it make you happy in any way?
When no errors occur, restic always makes me happy ;)

restic does show it encounters errors and retries, however the backup finishes successfully with exitcode 0.

The "Save(<data/a25b144eb3>) returned error, retrying after 552.330144ms: wrote 0 bytes instead of the expected 5178542 bytes" log output is merely a warning that an upload has failed and restic has to retry it. restic will try up to 10 times and fails if these retries are not enough to resolve the problem. Or in other words, the storage system told restic after a few retries that the file has been stored properly. None of the files for which restic had to retry the upload, are reported as missing later on. So it looks like the failed retries themselves are not related to the problems later on.
The list of missing data files returned by the first check run is far larger than the number of different data files for which restic had to retry the upload. What's curious is that the rebuild-index afterwards only removes 35 files from the index and not the 605 files reported as missing by check. The files removed by rebuild-index also all start with 7 or 8.
The second check run the reports not referenced in any index exactly 35 times. And yes, these are exactly the files which rebuild-index removed just before. The other not existing pack files look like a subset of the earlier list.
Given the apparently inconsistent file listings received by restic, it looks like the storage system is to blame. As the number of missing files has been shrinking, my guess would be that some kind of reindexing has taken place after the update.

My questions are:
* how come restic is missing packs in this situation?

* why does restic give exitcode 0, when the backup apparently is inconsistent?

* does restic backup do HTTP PUT before each HTTP HEAD?

Apparently the storage did not include these files in the listing of the bucket
All errors during the backup were resolved by retrying the operation
I'm not sure what you mean here? Are you referring to the read-after-write consistency usually used by S3 storages? restic won't try to read a file before successfully uploading it. More specifically it will call PUT or DELETE while uploading.

check currently only prints shortened pack IDs unless you patch the source code. But you can take a look at the data folder: the first two characters refer to the folder in which the pack file is stored. Inside that folder look for a file whose name starts with the shortened ID. ecc2d0c7 would be stored in data/ec/ecc2d0c7

I am backing up from a Windows restic 0.12.1 to a remote rclone v1.53.3-DEV.
And almost every time the remote internet connection resets or dynamic IP changes I am missing a pack. Is there some way to debug this?
I get the expected retries on the Windows machine but also a return code of 0 afterwards. Usually reindex and repeating the backup fixes it.
Should I open a new issue after gathering more data and logs?

@clst Please open a new issue. And include the command line you use to run restic. I'm not really sure what a "remote rclone" looks like.


Comparison to lattices

I see partial-order has some additional combinators, not present in http://hackage.haskell.org/package/lattices, but is there any other reasons to prefer one over another?

Hi,
I'm not familiar with the lattices package, but it seems to me that they are rather different. partial-order really focuses on that: defining a type class for partial orders & providing instances for several types. Since partial orders induce a notion of equality, my package also provides some convenience functions like elem or nub.
It doesn't introduce the notion of lattices.
Also, it provides instances and functions for e.g. testing for the subset-relation of lists out of the box:
Prelude> import qualified Data.PartialOrd as PO
Prelude PO> [3,1] PO.<= [1..10]
True
Prelude PO>

Does this answer your question?

Partially,
lattices has Algebra.PartialOrd too. And there is instances for subset partial order for Set.
There isn't a partial order for lists, but if added it probably be subrange relation (i.e. isInfixOf)

Ok, as I understand it we can summarize: The packages have some overlap, but don't provide equivalent functionality. What do you think, shall we close the issue?

lattices has Algebra.PartialOrd too. And there is instances for subset partial order for Set.

Some comparison between the interfaces in lattices and partial-order is here:

https://github.com/haskellari/lattices/issues/117

(Unfortunately, neither interface fits my needs: connectedness of nodes in a DAG is a partial order, but it is relative to a context (the DAG), a nodes in a particular DAG do not make a good Haskell type.)


Generate deb package on release #94

Closes #94

@Marwanmhks are you able to try this out on your fork?  It’s tricky to iterate on it in this repo as it’ll only work from master and will create spurious releases as we test.
Thanks again for helping with this.

okay let me check it out and get back to you

Okay so I was able to automate the deb packaging upon release but I am have a problem with the artifact upload, I would appreciate any hints.

@Marwanmhks I think you almost had it, nice work.
I had a try on a fork (see here) and was able to get it working with the following:
      - name: Build Debian package
        if: startsWith(matrix.build, 'linux-gnu')
        run: |
          cargo install cargo-deb
          cargo deb --target ${{ matrix.target }} --deb-version ${{ needs.create-release.outputs.trip_version }}
          case ${{ matrix.target }} in
            aarch64-*-linux-*) DPKG_ARCH=arm64 ;;
            arm-*-linux-*hf) DPKG_ARCH=armhf ;;
            i686-*-linux-*) DPKG_ARCH=i686 ;;
            x86_64-*-linux-*) DPKG_ARCH=amd64 ;;
            *) DPKG_ARCH=notset ;;
          esac;
          echo "DPKG_ARCH=${DPKG_ARCH}" >> $GITHUB_ENV

      - name: Upload assets to Github
        if: startsWith(matrix.build, 'linux-gnu')
        uses<EMAIL_ADDRESS>        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        with:
          upload_url: ${{ needs.create-release.outputs.upload_url }}
          asset_content_type: application/vnd.debian.binary-package
          asset_path: target/${{ matrix.target }}/debian/trippy_${{ needs.create-release.outputs.trip_version }}_${{ env.DPKG_ARCH }}.deb
          asset_name: trippy_${{ matrix.target }}_${{ needs.create-release.outputs.trip_version }}_${{ env.DPKG_ARCH }}.deb

If you can update the PR based on this (and squash it down to a single commit) then we should be good to merge it.

Merged, thank you for your contribution @Marwanmhks !


VSTS - Azure File Copy Failing in Hosted VS2017 Agent

We started experiencing failures this morning on our releases that do VM copies that use the Hosted VS2017 Agent.
2018-04-25T12:04:11.7300580Z ==============================================================================
2018-04-25T12:04:13.7570287Z ##[error]Required: '*****************************' service endpoint URL
2018-04-25T12:04:13.7768451Z ##[section]Finishing: Copy Files
When running in the Hosted build agent (not VS2017) it succeeds but it has a warning about a breaking change:
2018-04-25T12:13:14.0833171Z ##[warning]Breaking change notice: In upcoming release, top level properties, DataDiskNames and NetworkInterfaceIDs, will be removed from VM object because they are also in StorageProfile and NetworkProfile, respectively.
The Hosted VS2017 agent was working fine yesterday.  What might have changed here?

@afrlee This is reproducible when at the time of saving Release definition endpoint is there but before release someone has deleted the same endpoint you had used in release definition. Can you please verify your endpoint ?  Can you please share your contact details so we will debug this further ?


Expose RequestContext.createContext() for use in Elysia request decorator?

Elysia doesn't follow the app.use((req, res, next) => {}) pattern for middleware, so it's not immediately obvious how to properly associate each request with a fresh identity map fork.
Decorating the request with a RequestContext might be an option:
const app = new Elysia()
    .decorate('ctx', RequestContext.createContext(orm.em) as RequestContext)

However, .createContext() is private in the type declaration.
Would it be make sense to mark it public, or is there a better alternative that I'm unaware of?

Perhaps it's as simple as orm.em.fork()?


Attempt to de-reference a null object when refreshing from flow

Hi,
I'm facing a strange issue when rolling up in a refresh context from a flow.
11:09:56.211<PHONE_NUMBER>)|FATAL_ERROR|System.NullPointerException: Attempt to de-reference a null object
Class.RollupCalculator.DecimalRollupCalculator.handleSumOrCount: line 589, column 1
Class.RollupCalculator.performRollup: line 205, column 1
Class.RollupAsyncProcessor.getCalculator: line 1347, column 1
Class.RollupAsyncProcessor.getUpdatedLookupItemsByRollup: line 1250, column 1
Class.RollupAsyncProcessor.process: line 694, column 1
Class.RollupAsyncProcessor.performWork: line 350, column 1
Class.RollupAsyncProcessor.QueueableProcessor.execute: line 457, column 1
I checked the log file but I'm unable to determine which one is the null object referred, and apparently this error is blocking all the rollups from beign calculated. I'm on version 1.5.83. Please find the whole log file attached and let me know if anything else is needed to debug this problem. Thanks
07L0D00000Gwj7lUAB.log

found the problem, one of the metadata records had a string default value instead of a decimal one. Now I have a different problem: when the rollup refresh is triggered by the flow, the rollups are not updated 90% of the times (strangely sometimes they are updated but I couldn't understand why), but if I use the recalculate rollups button it works everytime

Hey @segdan, happy to try to help. Prior to being able to speculate on what the issue is, would you be able to update the Org Default Rollup Control custom metadata record to enable Is Rollup Logging Enabled and generate a log file to upload here? The log should show what's going on. Typically, there will be three logs generated when you update a record with Apex rollup - one with Origin of Aura, which is the sync part of the process. Then there's a QueueableHandler / blank Origin log - that's the one that probably has the info we need to examine. There's another log with the same origin as the second one; this is the RollupFinalizer and you don't need to include it. The aura and first of the two queueable logs would be very helpful though!

Hi @jamessimone , thanks for the quick feedback. Here are the two files, let me know if you need anything else. These logs were generated in one of the attempts where the rollups didn't update. If this can help, I'm specifically checking the Total_ARR_Override__c field on OpportunityLineItem, which should have been updated to 211000, and instead has the old value of 263000. I guess also the other rollups are not updated, but this one for sure is not.
07L0D00000GwjnPUAR.log
07L0D00000GwknTUAR.log

Thank you! I will have a look.


Tyf/fix rms

Motivation and Context

Description
修复RMS和FA
Use cases (Optional)

BC-breaking (Optional)

Checklist
Before PR:

[ ] I have read and followed the workflow indicated in the Contributors.md to create this PR.
[ ] Pre-commit or linting tools indicated in Contributors.md are used to fix the potential lint issues.
[ ] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] New functionalities are covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

[ ] CLA has been signed and all committers have signed the CLA in this PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Bonbon-Tang:x: rootroot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Epic: .NET Core is supported in air-gapped environments/private clouds

User Stories under this Epic:

[ ] As the developer of an service deployed to an air-gapped environment I can rely on .NET Core being available in my private cloud/air-gapped environment.
[ ] As an enterprise I can easily determine the latest .NET Core update I need, fetch it and use it for local deployment in my air-gapped/private cloud environment.

It would be great if you could consider the experience for stand alone, single computer, air-gapped development and deployment as well.

The work is not funded at this point, this will likely move to 7.0.

ITNOA
Any plan for .NET 8?


CanvasRenderingContext2D.imageSmoothingEnabled is missing from the d.ts files.

TypeScript Version:  1.8, nightly, release candidate (2.0.2) all affected.
Currently targeting es6.
Code
var canvas = document.createElement("canvas");
var context = canvas.getContext("2d");
context.imageSmoothingEnabled = false;

Expected behavior:
CanvasRenderingContext2D.imageSmoothingEnabled should be a property representing whether the canvas should smooth the image or keep it pixelated. The browser vendor forms are present, but not the prefix-less form.
Actual behavior:
Property imageSmoothingEnabled does not exist on type CanvasRenderingContext2D

https://github.com/Microsoft/TypeScript/issues/7846
take a look this issue

This is still experimental and is vendor-prefixed in several major browsers.
You can add it to the interface in your own file if you'd like to not support those browsers :wink:
interface CanvasRenderingContext2D {
  imageSmoothingEnabled: boolean;
}

I just figured this wouldn't be WAI because that logic doesn't really make too much sense.
There are 4 or 5 main browsers, right? Chrome, Firefox, IE, Opera, etc. They all have their own prefixed version. Therefore, each prefixed variant will only work on one browser. If you're going to support all of the prefixed variants, why not support the non-prefixed version? Future-proofed until you have to remove the others (if that even ever needs to happen), and for the people ahead of the game and already using the non-prefix version (me, making a game w/ electron: IE only chrome support necessary).
Also, chrome complains if you use the webkit variant vs unprefixed, and I refuse to leave warnings like that. That means it's my non-futureproof bug. And I don't like bugs.
As for the interface suggestion, I already added imageSmoothingEnabled to my es6 lib.d.ts, just figured it'd be nicer to point it out here as well, so I/others don't have to use a workaround. Can't say I agree with the final verdict, but I may be missing something. Anyway, thanks for your time.

The problem with providing the non-prefixed property is that it looks like any other property and people will be led to believe that it's standardized. The prefixed properties don't have this problem -- they're obviously vendor-specific and should be coded against appropriately.

Aha, I see. That makes sense. As someone who checks all browsers and caniuse sites and stuff when developing for webscale applications, that didn't cross my mind. I'm assuming you've thought of simply providing documentation notes? Or maybe info blurbs when compiling properties like that, "This property is not supported in all browsers." Or something? Just a thought.

When I add the interface extensions I get an error. TS 2.1.3
interface CanvasRenderingContext2D {
  imageSmoothingEnabled: boolean;
}

Error TS90010: Type 'CanvasRenderingContext2D' is not assignable to type 'CanvasRenderingContext2D'. Two different types with this name exist, but they are unrelated.
  Property 'imageSmoothingEnabled' is missing in type 'CanvasRenderingContext2D'

Should I be using extends or something? I'm wondering why TS isn't combining my 2 interfaces because they have the same name.

@dannyfritz alternatively inside a declare global { block


Add autoindentation

Detect indentation character/size.

I'm open to techniques to do this. Not willing to use a regex for obvious reasons. Could possibly done by scanning tokens.

The strategy taken by detect-indent seems robust--for each adjacent pair of lines (skipping empty lines), record the difference in indent. Pick the most used indent type.


Added intelligent function when paste labels to another task

Resolved #919
Resolved #3717
Motivation and context

[x] Added intelligent paste labels function (when user pasts a text, the function filters all the "id" fields.
[x] Added confirmation when delete labels via Raw editor (it was implicitly before)
[x] A bit adjusted behaviour of labels editor in general

How has this been tested?
Manual testing
Checklist

[x] I submit my changes into the develop branch
[x] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[x] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

@klakhov Could you please review the patch? Try to value the solution and find cases when it does not work

@nmanovic
Maybe additionally to delete on the server label/attribute IDs when create a new task/project?
And when updating a task/project, to delete any unknown IDs?

@bsekachev I've tried patch and it works as expected. No errors occur after labels copying.
Also I think its better to hide deleted labels/attributes paragraph if list is empty:

Btw why do we need to remove carriages if textarea does not contain them?
// remove all carriage characters (textarea value does not contain them)
replaced = replaced.replace(/\r/g, '');

@nmanovic
I did the proposed changes above

@klakhov

Also I think its better to hide deleted labels/attributes paragraph if list is empty:

Sure, I did.

Btw why do we need to remove carriages if textarea does not contain them?

It is related with textarea properties selectionStart, selectionEnd. They don't consider \r characters (maybe it is a bug, I don't know). Also when we enter something manually it does not add \r characters to the field (and does not add it when we fill out the field from a labels specification). So, I've decided just to ignore all such characters from clipboardData because they can appear only from this place.

@bsekachev , I'm find with the PR. The only concern I have for now is about update operation and removing id. Could you please share your ideas why you think the implemented solution is right?

@dvkruchinin
Could you please prepare a test with the following scenario:

From an existing task copy all labels and attributes (from the Row tab of the labels editor) (It must have at least one task and one attribute)
Go to the project create page, in the labels editor paste the copied text (ctrl + v)
Inserted JSON copy must not contain any "id":  fields
A project can be created with no errors

@dvkruchinin
Could you please prepare a test with the following scenario:

@bsekachev
Sure. A test will be prepared.


Update shubhra-kar.md

Added some thoughts.

Added some thoughts.
Signed-off-by: Jono Bacon<EMAIL_ADDRESS>

Signed-off-by: Jono Bacon<EMAIL_ADDRESS>


Fix issue #1882 , Generate a Unique random number of length 6

Add function to generate a unique number of length 6
Add tests to confirm the length of the generated number is 6

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #1882
Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[x] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

Can we use a verifiable number generator based on existing algorithms that can be validated?

For future work, it feels like we have a somewhat arbitrary set of functions in the RuleFactory class, am I missing some pattern or should we reorganize these at some point?

For future work, it feels like we have a somewhat arbitrary set of functions in the RuleFactory class, am I missing some pattern or should we reorganize these at some point?

We will reorganize the functions in future.

@pld @ellykits I have created an extension file just incase some functions are shared or you want to move the logic there.

@pld @ellykits I have created an extension file just incase some functions are shared or you want to move the logic there.

Thanks, let's wait until we thought through the structure before we create this file, I've removed it for now

@pld kdocs added.


Add renovate bot support to repo

Currently this repo has no renovate bot support. It could make sense to add this here to automatically update to new docker image version etc. See also https://github.com/owncloud-docker/helm-charts/pull/60.
@wkloucek Opinions?

Yes please! :pray:

Originally posted by @d7oc in https://github.com/owncloud/ocis-charts/issues/437#issuecomment-1830164020

@d7oc somehow the renovate comment also doesn't work. But I didn't have time to investigate

Comment? Which comment?
https://github.com/owncloud/ocis-charts/blob/d73fe65204ebf14e888bdfcb996ee9660d4a5a5b/charts/ocis/Chart.yaml#L13-L14

Ah. I take a short look

https://github.com/owncloud/ocis-charts/pull/438 should fix this. Unfortunately we'll only see that this is finally working after it's merged to master.

Looks good now

@d7oc are we done here?

Yes from my side it's done. Closing it.


🔨  Jinja metadata for UN IGME

Hey @Marigold this is a bit of a WIP but I'm trying to figure out Jinja templates. I think I am along the right lines but it doesn't seem to be adding the metadata to the variables as I expect. Probably missing something silly but if you could take a look when you get a chance that would be ace!

Hey @Marigold this is a bit of a WIP but I'm trying to figure out Jinja templates. I think I am along the right lines but it doesn't seem to be adding the metadata to the variables as I expect. Probably missing something silly but if you could take a look when you get a chance that would be ace!

Ah, I think it's because the section at the top e.g. where I have indicator, that should in fact be variable names? I probably need to restructure the data so that there is a column for each indicator...

You were pretty close @spoonerf! Here is the corrected version (took me a while, working with templates can get complex)
definitions:
  common:
    presentation:
      topic_tags:
        - Child & Infant Mortality
  sex: |-
    <% if sex == "Both sexes" %>

    <% elif sex == "Male" %>
    male
    <% elif sex == "Female" %>
    female
    <%- endif -%>
  wealth_quintile: |-
    <% if wealth_quintile == "All wealth quintiles" %>
    all wealth quintiles
    <% elif wealth_quintile == "Poorest quintile" %>
    the poorest 20% of the population
    <% elif wealth_quintile == "Second poorest quintile" %>
    the second poorest 20% of the population
    <% elif wealth_quintile == "Middle wealth quintile" %>
    the middle 20% of the population in terms of wealth
    <% elif wealth_quintile == "Fourth poorest quintile" %>
    the second richest 20% of the population
    <% elif wealth_quintile == "Richest quintile" %>
    the richest 20% of the population
    <%- endif -%>
  unit_of_measure:
    unit: |-
      <% if unit_of_measure == "Deaths per 1,000 live births" %>
      deaths per 1,000 live births
      <% elif unit_of_measure == "Deaths per 1000 children aged 10" %>
      deaths per 1,000 children aged 10
      <% elif unit_of_measure == "Deaths per 1000 children aged 15" %>
      deaths per 1,000 children aged 15
      <% elif unit_of_measure == "Deaths per 1000 children aged 1" %>
      deaths per 1,000 children aged 1
      <% elif unit_of_measure == "Deaths per 1000 children aged 20" %>
      deaths per 1,000 children aged 20
      <% elif unit_of_measure == "Deaths per 1000 children aged 5" %>
      deaths per 1,000 children aged 5
      <% elif unit_of_measure == "Number of stillbirths" %>
      stillbirths
      <% elif unit_of_measure == "Stillbirths per 1,000 total births" %>
      stillbirths per 1,000 total births
      <% elif unit_of_measure == "Number of deaths" %>
      deaths
      <%- endif -%>
  indicator:
    display_name: |-
      <% if indicator == "Neonatal mortality rate" %>
      The estimated number of deaths of << sex >> children aged less than 28 days, per 1,000 live births, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate 1-59 months" %>
      The estimated number of deaths of << sex >> children aged between 1 and 59 months, per 1,000 live births, in <<wealth_quintile>>.
      <% elif indicator == "Infant mortality rate" %>
      The estimated number of deaths of << sex >> children aged under one year, per 1,000 live births, in <<wealth_quintile>>.
      <% elif indicator == "Under-five mortality rate" %>
      The estimated number of deaths of << sex >> children aged under five years, per 1,000 live births, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 10-14" %>
      The estimated number of deaths of << sex >> children aged between 10 and 14, per 1,000 children aged 10, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 10-19" %>
      The estimated number of deaths of << sex >> children aged between 10 and 19, per 1,000 children aged 10, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 15-19" %>
      The estimated number of deaths of << sex >> children aged between 15 and 19, per 1,000 children aged 15, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 15-24" %>
      The estimated number of deaths of << sex >> children aged between 15 and 24, per 1,000 children aged 15, in <<wealth_quintile>>.
      <% elif indicator == "Child Mortality rate age 1-4" %>
      The estimated number of deaths of << sex >> children aged between 1 and 4, per 1,000 children aged 1, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 20-24" %>
      The estimated number of deaths of << sex >> children aged between 20 and 24, per 1,000 children aged 20, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-24" %>
      The estimated number of deaths of << sex >> children aged between 5 and 24, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-9" %>
      The estimated number of deaths of << sex >> children aged between 5 and 9, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Stillbirths" %>
      The estimated number of stillbirths, in <<wealth_quintile>>.
      <% elif indicator == "Stillbirth rate" %>
      The estimated number of stillbirths, per 1,000 total births, in <<wealth_quintile>>.
      <% elif indicator == "Neonatal deaths" %>
      The estimated number of deaths of << sex >> children aged less than 28 days, in <<wealth_quintile>>.
      <% elif indicator == "Deaths 1-59 months" %>
      The estimated number of deaths of << sex >> children aged between 1 and 59 months, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <% elif indicator == "Mortality rate age 5-14" %>
      The estimated number of deaths of << sex >> children aged between 5 and 14, per 1,000 children aged 5, in <<wealth_quintile>>.
      <%- endif -%>
tables:
  igme:
    variables:
      observation_value:
        title: Observation value
        unit: |-
          {definitions.unit_of_measure.unit}
        display:
          name: |-
            {definitions.indicator.display_name}
      lower_bound:
        title: Lower bound
        unit: |-
          {definitions.unit_of_measure.unit}
        display:
          name: |-
            {definitions.indicator.display_name}
      upper_bound:
        title: Upper bound
        unit: |-
          {definitions.unit_of_measure.unit}
        display:
          name: |-
            {definitions.indicator.display_name}

A couple of comments:

In Jinja templates you should use << ... >> to reference dimension values, not <%
Dynamic placeholders must not contain whitespace, i.e. use {definitions.unit_of_measure.unit} and not { definitions.unit_of_measure.unit } (not sure why it doesn't work...)
I had to change definitions.indicator.display.name to definitions.indicator.display_name because YAML doesn't like the dot

The template itself is good after fixing those small issues. I've tried running etl un/2023-08-16/igme --grapher and got proper display names in Admin.
(When inserting to grapher, I got the error Variables used in charts will not be deleted automatically. That typically means variables have changed short names. If you struggle with fixing it, ping me and I'll be glad to help)

Amazing thanks @Marigold! I'll try and figure out the next steps and almost certainly ping you. I think it's likely the short names will be different, but I think we've got round that before by using chart revisions?

but I think we've got round that before by using chart revisions?

Exactly. By the way it'd be great if you could ping me after this PR is ready before doing chart revisions. I'd like to test syncing of changes from staging servers to production.

but I think we've got round that before by using chart revisions?

Exactly. By the way it'd be great if you could ping me after this PR is ready before doing chart revisions. I'd like to test syncing of changes from staging servers to production.

Hey @Marigold - I think we're at this stage now, I believe the only issue remaining is the Variables used in charts will not be deleted automatically one.

I looked into the Variables used in charts will not be deleted automatically error. It happens because variables now use different (short) names.
You should first rebase or merge this PR onto the master. I'd also recommend reordering index levels to be consistent with the current version to make chart revision easier
tb = tb.reorder_levels(["country", "year", "unit_of_measure", "indicator", "sex", "wealth_quintile"])
tb_youth = tb_youth.reorder_levels(["country", "year", "unit_of_measure", "indicator", "sex", "wealth_quintile"])

If you make this change and commit back, staging server will rebuild ETL and you should see an updated dataset there.
Once you have that, the best way to fix this error is to map those variables with
STAGING=igme-metadata etl-wizard charts

and then approving them in chart revisions.
I tried to do it myself, but it's not as straightforward as some old variables don't exist in the current dataset (something with middle income group and possibly others).
With the new workflow, you can approve chart revisions on the staging server, then I can sync them to production (after we merge this PR).

Amazing, thanks @Marigold!
I had messed up some of the variables, making variable matching impossible, but sorted that out now. I have approved the chart revisions on the staging server. So I think we can merge this now? I will wait to hear from you before merging.


[Bug]: Landing page links should use router

The landing/welcome page links to documentation pages should use <Link> elements to allow navigation within the SPA.
Currently, when clicking on a documentation page, the application reloads.

@kgryte Related issue: Links in the landing page to other packages sometimes are broken due to missing version information
Steps to reproduce:

Hard refresh of https://stdlib.io/docs/api/latest
Do a normal refresh by clicking the reload button of the browser
Click on one of the package links; they will look like this: https://stdlib.io/docs/api/undefined/@stdlib/random/base/

Proposed solution:
If the version is missing, should we impute latest instead of going with undefined.

Thanks for noticing this!

Hmm...I am not able to reproduce on my end.

Yeah, I cannot get the version to be undefined. And that it would be so is a bit odd, as props.version should always be defined.

Hmm, maybe I didn't have the latest version of the docs website cached yet. Cannot reproduce anymore, see below gif for how the problem looked like.

However, even though not reproducible, we could act defensively and add a default property to the <Welcome /> component:
Welcome.defaultProps = {
    'version': 'latest'
};

Adding sensible defaults in case data is missing may be a good idea to do more generally .
I also noticed that we do not use PropTypes for run-time property type validation during development.It's best practice to use such type checking in React projects and am wondering why we didn't do so (I assume we must have had a Slack chat about it).

Re: PropTypes. This is the first I am hearing about it, and I don't recall any discussions. Not opposed to using in the application.
Re: defaultProps. Somewhat leery about doing this too much, as defaults, even if sensible, can mask bugs. For these components, we are assuming that the version propagates down the component stack. If it isn't, that tells me that the assumption is being violated, and, if things just "work", may be hard to track down where, when, and how that assumption is violated.

See https://github.com/stdlib-js/www/commit/23fdbc6e9fe22dbe3434efc4fe62c0b57768b361 for an example of added property types. I would be happy to add them throughout the app. It's quite helpful for debugging during development and all bigger React projects I know of make use of it.

As these are runtime checks, we probably first need to add tooling support to allow building the app in development mode. Currently, I think we always build in production mode.

Nicer would be not having to build at all but use react-dev-server for hot module reloading.

Not sure what is involved with setting that up. You have experience with it?

Let me fiddle with it!

🙏

Re: propTypes. To be consistent with elsewhere, for static properties, added a JSDoc comment: https://github.com/stdlib-js/www/commit/2f604eb172440809478fe8eaf4888a09cb33881f.
Otherwise, if you want to add those across the application, would be appreciated!

Sounds good, will do!

@Planeshifter Am still unable to reproduce the undefined version bug. Not clear to me where this might be coming from.
In client.jsx, we explicitly check for the absence of a version in the URL and fallback to a default value which is then threaded throughout the application:
https://github.com/stdlib-js/www/blob/78ff87f708d927953a6cb7959f9f957a49856bb1/src/client.jsx#L84-L87
For SSR, the route we'd be hitting above is /docs/api/:version, but in that route, we have access to a version URL parameter. I've confirmed locally that the version is correctly inserted into the URLs.
https://github.com/stdlib-js/www/blob/78ff87f708d927953a6cb7959f9f957a49856bb1/lib/server/lib/routes/version/version.js#L58
So not clear to me where things are going awry. Tis possible that this is a hydration issue of some kind when loading from the service worker.

Ah...wait. Found a potential issue source for things to not be passed down correctly. Namely, with react-router.

Followed the same steps as outlined above--hard refresh followed by refresh--and logged the value of match.params.version in _renderWelcome and all logged values are latest.
As this seems elusive to reproduce and there aren't any obvious code paths where version can be undefined, will go ahead and move on. Can always revisit in a separate issue thread if we run across this again.

I should state wrt the OP, the landing page now uses <Link> elements for internal routing.


Change AccessController to be post-match

Codecov Report

Merging #2037 into master will increase coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2037      +/-   ##
==========================================
+ Coverage    69.6%   69.75%   +0.15%
==========================================
  Files         830      830
  Lines       40079    40079
  Branches     5402     5402
==========================================
+ Hits        27895    27958      +63
+ Misses      10323    10260      -63
  Partials     1861     1861

Impacted Files
Coverage Δ

...not/controller/api/resources/AccessController.java
100% <ø> (ø)
:arrow_up:

.../pinot/core/query/reduce/InAndNotInComparison.java
61.29% <0%> (-19.36%)
:arrow_down:

...e/impl/dictionary/LongOnHeapMutableDictionary.java
82.97% <0%> (-12.77%)
:arrow_down:

...mpl/dictionary/DoubleOffHeapMutableDictionary.java
73.68% <0%> (-7.02%)
:arrow_down:

...not/core/query/pruner/DataSchemaSegmentPruner.java
53.65% <0%> (-4.88%)
:arrow_down:

...e/operator/dociditerators/BitmapDocIdIterator.java
60.71% <0%> (-3.58%)
:arrow_down:

...impl/dictionary/FloatOffHeapMutableDictionary.java
92.98% <0%> (-1.76%)
:arrow_down:

...not/broker/requesthandler/RangeMergeOptimizer.java
88.7% <0%> (-1.62%)
:arrow_down:

.../broker/routing/HelixExternalViewBasedRouting.java
90.45% <0%> (-0.71%)
:arrow_down:

...manager/realtime/LLRealtimeSegmentDataManager.java
69.43% <0%> (-0.2%)
:arrow_down:

... and 18 more

Continue to review full report at Codecov.

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


Support for invoke local

This adds support for the lifecycle hooks to make sls invoke local -f <funcName> work.
This fixes #8

Would you mind adding some docs to the readme about this feature?

@schickling can you tag a release for this change and publish to npm? What kind of docs are you looking for? The current state of serverless invoke local not working in conjunction with this plugin just seems like a bug. I don't think the functionality needs to be explicitly documented.

Done!

Thanks!


Feature redis user

Q
A

Documentation
no

Bugfix
no

BC Break
no

New Feature
yes

RFC
no

QA
no

Description
This PR is adding the possibility to authenticate with Redis with a custom username. The ACL feature is available in Redis since Version 6.
The username can be passed as an resource option.

created new pull request


Extra carefull deploy procedure (#2999)

Pull request opened by the merge tool on behalf of #2999

Merged into branches iso4 in c8ce2495d616c69aee8da5360255322a4d28089d


Add RWP

https://learn.microsoft.com/en-us/azure/architecture/reference-architectures/reliable-web-app/dotnet/pattern-overview?WT.mc_id=dotnet-90136-dotnet
/azure/architecture/reference-architectures/reliable-web-app/dotnet/pattern-overview

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

ID: a2eab542-a420-d6c5-d314-a420837cf602
Version Independent ID: 48c12168-54f3-0b83-f136-565f483a03e9
Content: Deploy ASP.NET Core apps to Azure App Service
Content Source: aspnetcore/host-and-deploy/azure-apps/index.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @bradygaster
Microsoft Alias: bradyg

cc @codemillmatt


Steve/yarn pnp

Coverage remained the same at 75.362% when pulling a10640af6195650c41214d6a25a0b954b99312a0 on steve/yarn-pnp into c447b95ccccd19fbc6ab8cf0e378aac11c026a7a on master.


Need help creating requests using raw XML

Hello,
thanks for your work, appreciate this a lot and I think it is exactly what I need.
I have a SOAP interface in business context, where I need to pass raw XML, generated/modified per request, to several methods. I would like to use your ParallelSoapClient to do async parallel requests to one specific method, where each request has a (slightly) different XML "body". Is this possible? Thanks in advance,
Oliver

Thank for Oliver :) I’m very glad that this would help other people with similar challenges.
Yes you could always change the xml in every request and launch all in parallel.
If you need some assistance doing proof of concept I’m glad to give you a hand.
Put some code in gist and let me have a look and check the examples and unit tests it cover most of the use cases and could help you.

Hi !
Thanks for your quick response.
I am just sharing a simple testcase from one of my testcontrollers .. please dont mind me ripping it out of its context, I suppose you`ll get the idea what I am trying to do based on the code. Actually it is a Method to grab base64 encoded files via a webservice. Those files are related to specific records, where one record may have multiple documents attached. I am trying to download them in parallel asyncronously therefore.
please dont worry about me using (for dev purposes) my own namespaces, as the environment (typo3) requires me to do so, and we have the company policy to strictly distinguish between official sources and our extensions and devs.
example_class_function.txt

Hi !
Don`t worry : I managed to solve this myself.
Just call the __doRequests function with the xml as request and the other necessary params and save the results as a new array-item. then set the parallelSoapClient->soapRequests to this array.
Works fine, thank you !!

Awesome :) i was just writing an example.
it would be great if you would be able to contribute some example while you implementing the work :) to inspire and help other devs on how to use the library 👍


Colourisation: params not coloured

procedure division using by value a as binary-long
                         by params b as string occurs any.

Fixed in 4.4.4


Use screens instead of explicit back list

Instead of keeping track of state
have a header to show which resource we're looking at

I tried:
bluepyentity --bucket bbp/somatosensorycortex explorer "https://bbp.epfl.ch/neurosciencegraph/data/e0ad3eb2-0c4a-4097-9d99-5cc9625ab13f"
and then go to that link,

then back and then try to go forward on the same link and got this:

The issue was probably there earlier, but that is the first thing I tried :)

The issue was probably there earlier, but that is the first thing I tried :)

No, I think it's a regression, I'll have a look sometime.


Can't use reserved word as element or attribute name

In the specification for Compact RELAX NG it says that:

It is not necessary to quote a keyword that is used as the name of an element or attribute or as datatype parameter.

Unfortunately it doesn't seem to work here. Minimal example:
element string { text } # comment

Running the test suite gets us this stack trace:
Traceback (most recent call last):
  File "test.py", line 23, in runTest
    root = rnc2rng.load(f)
  File "/Users/dargueta/git/rnc2rng/rnc2rng/__init__.py", line 4, in load
    return parser.parse(f=f)
  File "/Users/dargueta/git/rnc2rng/rnc2rng/parser.py", line 529, in parse
    return parser.parse(lex(src), state=State(fn, src))
  File "/Users/dargueta/.pyenv/versions/rnc2rng_35/lib/python3.5/site-packages/rply/parser.py", line 62, in parse
    self.error_handler(state, lookahead)
  File "/Users/dargueta/git/rnc2rng/rnc2rng/parser.py", line 504, in error
    raise ParseError(t, s.fn, ln, col, line)
rnc2rng.parser.ParseError: in tests/simple-escape.rnc [1:9]
element string { text } # comment
        ^

Same thing for attributes:
element test { attribute string { text } }

gives us:
Traceback (most recent call last):
  File "test.py", line 23, in runTest
    root = rnc2rng.load(f)
  File "/Users/dargueta/git/rnc2rng/rnc2rng/__init__.py", line 4, in load
    return parser.parse(f=f)
  File "/Users/dargueta/git/rnc2rng/rnc2rng/parser.py", line 529, in parse
    return parser.parse(lex(src), state=State(fn, src))
  File "/Users/dargueta/.pyenv/versions/rnc2rng_35/lib/python3.5/site-packages/rply/parser.py", line 62, in parse
    self.error_handler(state, lookahead)
  File "/Users/dargueta/git/rnc2rng/rnc2rng/parser.py", line 504, in error
    raise ParseError(t, s.fn, ln, col, line)
rnc2rng.parser.ParseError: in tests/keyword-attribute.rnc [1:26]
element test { attribute string { text } }
                         ^

The crash happens in both Python 3.5 and 2.7.

Thanks for your detailed report! It should be fixed in master. Let me know if you've found any other issues; or if not, let me know if you would like a release version with these fixes.

I found one other issue. I have a fix for it but I can't push or create a pull request.


changing sample download times from iso8601 to excel friendly format

Don't merge this yet, I'd like to keep it decoupled from the changes in GCMON-296

@davidmsibley no merge?

sorry, now we can merge it.

:+1:


Populated the visit tab on registration app

The visit tab now has the visit history of the patient

Coverage decreased (-0.02%) to 26.006% when pulling df89a102ae7f9ec8f2347d848de6c255e14e18ce on ningosi:master into 79961e2a061b391d2e6e8c2e8ada504f2378f009 on esaude:master.


[Update Request]: Microsoft.EdgeWebView2Runtime 122.0.2365.92

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Microsoft.EdgeWebView2Runtime
Package Version
122.0.2365.92
Please describe the changes you would like to see
No response

Close with reason: This should be resolved now;


Can I get IP for a user ?

Hi Everyone,
I'm making some bots that may be used by fake users using the same IP address.
I would like that have a way to determine the IP Address OR Mac address of someone using my telegram bots.
Do you guys know a way to do that ?
If yes, Can you point me to the direction?
Thank you for your return !
Best regards !

You can't. Telegram not send personal data of users in updates.


Add BodyOrQuery param type

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
Or, I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Example
Here's a self-contained minimal, reproducible, example with my use case:

from typing import Optional
import uvicorn
from fastapi import FastAPI, Depends
from pydantic import BaseModel

app = FastAPI()

class Item(BaseModel):
    p_int: Optional[int] = None
    p_float: Optional[float] = None
    p_str: Optional[str] = None

class BodyOrQuery:
    def __init__(self, item: Item = None, query: Item = Depends()):
        # merging here

        item_dict = {}
        if item is not None:
            item_dict = item.dict()

        self.items = {
            **{k: v for k, v in item_dict.items() if v is not None},
            **{k: v for k, v in query.dict().items() if v is not None},
        }

@app.post("/")
def body_or_query_using_depends(body_or_query: BodyOrQuery = Depends()):
    return body_or_query.items

Description
There is a related issue (https://github.com/tiangolo/fastapi/issues/1543) but that was a question and this is a feature request.

Tornado supports getting arguments from the body or query params.  We're trying to convert a tornado app to fastapi and will need to support this existing behavior.  My workaround above shows one way we could possibly support body or query arguments in the same method, but it has the drawback of making both sets of argument types optional and ends up being a lot of code to write for each path operator.  Also, if in the example above, p_int was required, that would not show up in swagger or pydantic, and we'd have to do our own validation.
The solution you would like
I would like it to have a BodyOrQuery parameter type that could take arguments from either input:
from fastapi import FastAPI

app = FastAPI()

@app.post("/")
def body_or_query(p_int: int = BodyOrQuery(), p_float: float = BodyOrQuery(), p_str: str = BodyOrQuery()):
    return {"p_int": p_int, "p_float": p_float, "p_str": p_str}

I'd be happy to work on implementing this as a PR, and was wondering if this feature was something you'd consider adding to the framework.  I've only looked in the codebase minimally, and I did start to work on this, but I don't yet have a good feeling for how feasible this even is, and wanted to gauge interest here before I worked on this any more.
Describe alternatives you've considered
The best alternative I've come up with is the one shown in the example above.
Alternatively, maybe there could be a way to define multiple path operators that respond to the same method (e.g. POST), but would have different signatures.  I don't really think this is viable, but figured I'd mention it.  I guess it would look like this:
@app.post("/b_or_q")
def only_body(body: Item):
    return body

@app.post("/b_or_q")
def only_query(p_int: int, p_float: float, p_str: str):
    return {"p_int": p_int, "p_float": p_float, "p_str": p_str}

If this even worked, which it doesn't (it always matches the first one), I think it would still have the drawback that you couldn't mix and match body/query args for the same argument.
Environment

OS: Linux / macOS]:
FastAPI Version [e.g. 0.3.0]: 0.60.1
Python version: 3.8.3

Is this something that you'd like to support within the module and would accept a PR for @tiangolo?

Sorry to revive something this old.  Did this ever get added/considered?


Complying with scss-lint for some mxin files

Updated some mixin files to comply with the scss-lint rules.

Hi @naicko!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

Changes must be made to the original JS source code file(s), not just the generated concatenated JS file(s).

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)


[VS Code] [Audio] Calling module not available. Cannot find module 'slimcore-2.0'

<!--
Attach verbose logs as follows:
1. Press F1 (or Ctrl-Shift-P), type export

@verdulife Could you try uninstalling the audio extension and reinstalling it? This issue may indicate that they install process failed for some reason. In the meantime, I'm going to close this as a duplicate of #827. Thanks!


Metal GFX Backend: RWBuffer does not work as expected with slang-test

Related to: #4291
tests/compute/atomics-buffer.slang does not produce correct output with Metal target, RWBuffer data is never filled (or not read correctly). The data read back is all 0s, this may be a sign that a shader failed to launch.

Putting this to low priority as this is only due to a limitation in gfx abstraction and not due to any compiler issues. Texel buffers are a less commonly used feature so it is OK to put on-hold its support it in gfx for now.


[Doc] Improve geo-replication example

Search before asking

[X] I searched in the issues and found nothing similar.

What issue do you find in Pulsar docs?
Explanation from the replication clusters could be improved, after reading several times the geo-replication I was a little bit lost.
What is your suggestion?

I think the first example should be just between 2 clusters. And put an example with more differences in the names. Sometimes it is difficult to follow us-west, us-cent, and us-east clusters. I would rename them to Original and Replicated.
There could be another second example with 3 clusters but easier to follow.
I missed in the explanation of geo-replication that replication behaves like a “subscription” in Cluster Origin that connect as a producer  into Cluster Replica. So that explains what happens on next command, why you execute it in Origin and the information you need to provide.

$ bin/pulsar-admin clusters create \
  --broker-url pulsar://<DNS-OF-REPLICA>:<PORT> \
  --url http://<DNS-OF-REPLICA>:<PORT> \
  replica-cluster

Any reference?
N/A
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Question:
There are two pulsar clusters(A and B) and we enabled the Geo-Replication. e.g. my_tenant/my_namespace/my_topic, we can produce the data to the A cluster and consume the data from the B cluster. How to delete this topic? If we can delete the topic, can you share the details?

@Hongten Sorry, I'm not sure about your question, do you want to delete the topic in both clusters?

@Hongten Sorry, I'm not sure about your question, do you want to delete the topic in both clusters?

yes, correct. Can you advise? thanks.

There has been some improvement lately and I didn't have time to improve it I will close this and review it more.


Rework settings storage to save space

Currently, a user's settings are stored via browser sync storage. Redirect settings are stored in an item called wikiSettings, and search engine settings are stored in an item called searchEngineSettings.
The data looks like { "id": "setting", "id2": "setting", ... }.
Currently, Chrome has an 8kb limit for each sync item. If a user has changed the settings for every wiki, that can take just over 7kb of storage - so we are quickly approaching that 8kb limit.
Need to come up with ways to avoid hitting this limit. Some ideas:

Instead of storing full setting names (redirect, alert, etc.), use abbreviations (could be single letters).
Split settings by language, but I don't think this is ideal (and doesn't really solve the problem long-term given how many English wikis there are).
Switch to using local storage, but I think this should be a last resort as it's nice to be able to backup settings and have them synced across devices.

Best option is likely to compress the data. We can use the Compression Streams API that is available across all major browsers as of May 2023.
Nice guide:
https://dev.to/samternent/json-compression-in-the-browser-with-gzip-and-the-compression-streams-api-4135

Best option is likely to compress the data. We can use the Compression Streams API that is available across all major browsers as of May 2023.
Nice guide by @samternent:
https://dev.to/samternent/json-compression-in-the-browser-with-gzip-and-the-compression-streams-api-4135
A quick test took our 7kb JSON to ~2.5kb.

Now compressing wiki settings in commit https://github.com/KevinPayravi/indie-wiki-buddy/commit/ac91d586b89a9c012fcd0d6f1105163eef529927


Spin.min and IE 9

The page will not load in IE 9.  We get the following error in Spin.min
spin.min.js?v=1.4.3&_=1428616895060, line 2 character 1070

We looked into this and have been able to reproduce the issue with the developer version of the Swipe template and IE9. We are not going to be able to fix that issue but this should not affect you too much as in fact that you are using is intended for development only.
Swipe comes in two version:

user download http://links.esri.com/storymaps/swipe_template_zip (not intended for customization or css customization)
developer download https://github.com/Esri/swipe-map-storytelling-template-js/archive/master.zip (intended for deep customization)

If you have made customization to the application, when deploying production application, you should follow the procedure here https://github.com/Esri/swipe-map-storytelling-template-js#developer-guide to create a production version that won't be affected by this issue.
If you have note made any customization to your shortlist or a customization that does not require deep modification of the Javascript, you should be using the developer download that won't have this issue http://links.esri.com/storymaps/swipe_template_zip


Reduce the number of open file handles

Installing thousands of bundles within a process can cause undefined behavior if the OS open file limit has been reached. In our use case, we reached this open file limit on Windows.
The reason is we read the metadata for the bundle through mz_zip_reader_init_file in BundleResourceContainer::BundleResourceContainer, which keeps it open until the Bundle is uninstalled (BundleResourceContainer's dtor is called that closes the zip via mz_zip_reader_end)
Proposal
In order to resolve the issue on all platforms i.e. to allow thousands of bundles to be installed into applications:

This solution will close the file handle for the bundle if it only contains a manifest file. The idea is that with only a manifest.json file, clients are unlikely to use another CppMicroServices resource API to read meta-data from the .zip file.

If the resource API is used after the bundle is installed, the file will remain open until the bundle is uninstalled.

I would like to add another proposal:

Bundle resources are accessed using the BundleResource and BundleResourceStream classes. As long as an instance of one of these classes exists, the file remains open. When the last instance for a specific bundle is destructed, the file is closed.

This approach means that after bundle installation, no file handle is kept (the manifest contents is read into memory and not accessed from the zip anymore).

@jeffdiclemente I think it would make the special manifest.json only check obsolete. Anyway, the related PR is a good improvement and we can incrementally improve the file handling.

@saschazelzer ok, if it simplifies the code and keeps the file handle count down, i'm all for it. I'll look into doing it.