[Darwin] MTRDevice should support multiple delegates

This PR adds multiple delegate support for MTRDevice, and a unit test for it.
Also includes some comment fixes, logging additions, and MTRDevice os_unfair_lock usage cleanup for straight-forward lock/unlock situations.

@woody-apple
I've changed the API to "addDelegate:..." and "removeDelegate:" for the new multiple delegate support. The old "setDelegate:..." will still work seamlessly.


add <Input> component

We are introducing the <Input> component.

@mazmassa Niceeee job :) Very cool to see this.
Compare to the design system, I can't see the "hover" mode. Did you remove it from the 1st iteration on purpose?

I can't see the "hover" mode

Maybe you should set the dark mode because in light is soft.  Normally it's there: https://github.com/massalabs/ui-kit/assets/126461030/7889f57a-4d4c-4efc-9893-1ece42b3e44b

Did you remove it from the 1st iteration on purpose?

Sorry but I haven't got your point here! 🤔


Some display problems

The "days" on the page don't show up on the iPhone's Safari. Some Unicode characters are non-Chinese characters.

额 既然能打中文就不用生草机了
junze经历了两个月的玉米集训，又回到家乡H市了
所以不重新计数吗？

Due to the fact that I does not has a Macintosh to debug the website on Safari, I cannot patch it aimed to Safari.
I am trying to discover what leads to this bug on mdn.
Sorry for the inconvenience.

发件人: KHLFYY @.***>
发送时间: 2021年7月15日星期四 21:12
收件人: CrSjimo/junze
抄送: Subscribed
主题: [CrSjimo/junze] Some display problems (#1)
The "days" on the page don't show up on the iPhone's Safari. Some Unicode characters are non-Chinese characters.
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/CrSjimo/junze/issues/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AINYVMATXFL73YXD4D7YXUTTX3NERANCNFSM5ANQZITA.

已经修复了

发件人: KHLFYY @.***>
发送时间: 2021年7月17日星期六 22:27
收件人: CrSjimo/junze
抄送: Subscribed
主题: Re: [CrSjimo/junze] Some display problems (#1)
额 既然能打中文就不用生草机了
junze经历了两个月的玉米集训，又回到家乡H市了
所以不重新计数吗？
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/CrSjimo/junze/issues/1#issuecomment-881907337, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AINYVMDNAXE677RUKCRTH43TYGHNBANCNFSM5ANQZITA.

好消息
自定义中的天数在Safari上显示正常
但预设的俊泽和赵总的仍然显示NaN


Update documentation of new images pipelines

Issue Description
As described here:https://github.com/deeplearning4j/dl4j-examples/issues/420
We need to update
https://deeplearning4j.org/simple-image-load-transform
to reflect the use of PipelineImageTransform.

 Image support for the Transform process

https://github.com/deeplearning4j/deeplearning4j/pull/3499


Final Project Feedback

Hi Thomas Butterfield,  I am Kuldeep - one of the mentors helping with the grading.
Close to finishing but missing critical thing
Please add at least 5 successfully running Smart Contract test cases. That is one of the critical aspects of the project. You can use Github pages to deploy your front-end but if that is causing any issue, we are happy to test the front-end locally.
I understand that sometimes due to time, we are not able to finish what we started. But I will encourage you to take out some time and try it, please take help from your fellow BootCamp mates or ask us on Discord. We would like to see you with an NFT Ceritification.
If you'd like to fix these things, you can resubmit your Final Project. Most likely, we'll have to revisit it in January. But you're really close! Don't give up.
Before resubmitting, be sure to run it by a friend and see if they can interact with your project on the baseline requirements. Please ping<EMAIL_ADDRESS>when you believe you corrected the errors.
Thank you,
Kuldeep

hey kuldeep, thanks for the feedback, I have preceded to implement the testing script and now have 10 passing unit tests (atleast on my local instance...). I have also emailed Coogan as you said too, got an email back saying he is out of the office till 16th January, so im just letting you know here aswell that i believe i have corrected the errors and can resubmit it for marking.
Thanks
Tom

Please re-submit and let me know.

re-submit where? I have sent coogan an email but out of office till mid jan

i mean push the latest updates in this repo. but yeah please wait for Coogan to reply back when he is back in office.

ok thanks, yea i have pushed all the latest commits to the repo, cheers


LXC: Terraform plan error:

Referring to #234, there seems to be fixed an issue with manually deleted VM's... I have the same issue after manually removing a container from proxmox. I expected the provider to want to recreate the container as the state doesn't reflect the reality. This is also how it works for other providers I know (AWS/Azure).
It fails even on terraform destroy, because it already fails when planning.
By the way, manually removing the resource from the tfstate fixes it, but that should not the way to do it, I recon. :wink:
terraform plan output:
proxmox_lxc.master[0]: Refreshing state... [id=pve1/lxc/501]
╷
│ Error: Vm '501' not found
│
│   with proxmox_lxc.k8s_master[0],
│   on master.tf line 6, in resource "proxmox_lxc" "master":
│    6: resource "proxmox_lxc" "master" {
│
╵

Is this an easy fix? Do you need more info? The plugin log didn't really show a related error to me. This is the full log of the terraform plan.
Wed, 01 Sep 2021 05:41:19 +0200 INF github.com/Telmate/terraform-provider-proxmox/proxmox/util.go:141 > Enabling the capture of log-library logs as ithe _capturelog flag was detected
Wed, 01 Sep 2021 05:41:19 +0200 INF github.com/Telmate/terraform-provider-proxmox/proxmox/util.go:174 > Logging Started. Root Logger Set to level debug
2021/09/01 05:41:19 >>>>>>>>>> REQUEST:
GET /api2/json/cluster/resources?type=vm HTTP/1.1
Host: REDACTED:8006
User-Agent: Go-http-client/1.1
Accept: application/json
Authorization: PVEAPIToken=REDACTED
Accept-Encoding: gzip

2021/09/01 05:41:19 <<<<<<<<<< RESULT:
HTTP/1.1 200 OK
Content-Length: 1812
Cache-Control: max-age=0
Connection: Keep-Alive
Content-Type: application/json;charset=UTF-8
Date: Wed, 01 Sep 2021 03:41:11 GMT
Expires: Wed, 01 Sep 2021 03:41:11 GMT
Pragma: no-cache
Server: pve-api-daemon/3.0

{"data":[{"vmid":100,"name":"REDACTED","mem":1823934260,"id":"qemu/100","type":"qemu","uptime":94243,"cpu":0.00589525838096042,"maxmem":4294967296,"maxcpu":4,"maxdisk":34359738368,"diskread":165914684,"disk":0,"netout":13327344732,"node":"pve1","netin":13234704875,"template":0,"diskwrite":3695708672,"status":"running"},{"type":"lxc","name":"REDACTED","vmid":101,"mem":66879488,"id":"lxc/101","cpu":0.00119708843756034,"maxmem":1073741824,"uptime":94180,"maxdisk":4143677440,"diskread":2073321472,"disk":1391534080,"maxcpu":1,"status":"running","template":0,"diskwrite":0,"netout":6796563,"node":"pve1","netin":11063902},{"maxmem":1073741824,"cpu":0.00196508612698681,"uptime":94165,"type":"qemu","id":"qemu/102","mem":630513664,"vmid":102,"name":"REDACTED","status":"running","diskwrite":253689856,"template":0,"node":"pve1","netin":14232155,"netout":318369,"disk":0,"maxdisk":17179869184,"diskread":422935552,"maxcpu":1},{"uptime":94161,"cpu":0.0014738145952401,"maxmem":1073741824,"name":"REDACTED","vmid":103,"mem":876687360,"id":"qemu/103","type":"qemu","netout":1180860,"node":"pve1","netin":15963042,"template":0,"diskwrite":413847552,"status":"running","maxcpu":2,"maxdisk":17179869184,"diskread":801033216,"disk":0},{"netout":16430098,"node":"pve1","netin":651045340,"template":0,"diskwrite":1560629248,"status":"running","maxcpu":2,"diskread":1350863872,"maxdisk":26843545600,"disk":0,"uptime":94157,"cpu":0.00736907297620052,"maxmem":4294967296,"vmid":104,"name":"REDACTED","id":"qemu/104","mem":3216908288,"type":"qemu"},{"maxdisk":0,"diskread":0,"disk":0,"maxcpu":2,"status":"stopped","netout":0,"node":"pve1","netin":0,"template":0,"diskwrite":0,"type":"qemu","vmid":105,"name":"REDACTED","mem":0,"id":"qemu/105","uptime":0,"cpu":0,"maxmem":2147483648}]}

Any update ?

Please fix, also face same issue

Stale is so bad thing :(


Config re-sync instead of full wireguard restart

Created wireguard restart script, which compares wg0.conf part between [Interface] and first peer with previous version for changes, and, if there is any - restarts whole wireguard, if the only changes are done to peers, it calls "wg syncconf", eliminating client disconnection after pressing apply config problem.
Resolves #228, resolves #167

Since you are modifying a bash HEREDOC in the readme, insert a pwd via a shell variable instead of /DIR-TO-RESTART-FILE/ which not everyone will pick up on.

Instead of using: ExecStart=/usr/bin/systemctl restart<EMAIL_ADDRESS>use: ExecStart=/usr/bin/systemctl reload<EMAIL_ADDRESS>

I tried using systemctl reload instead of systemctl restart and reload caused my remote shell to never reconnect, while restart would reconnect after a few seconds.
I like the script though, Im using it on my machine!
I added another little thing to it, since I wanted my wg0.conf to be regenerated from the db each time I run the script.
...
config="/etc/wireguard/wg0.conf"
old_config="/etc/wireguard/wg0.conf.old"

rm $config
systemctl restart wireguard-ui
sleep 5
# Wait for the wg0.conf to be reloaded before continuing on with the script
echo "Config Regenerated"

if [ ! -f $old_config ]; then
...

If I understand this code right, it is fixed to wg0 interface.
But can not handle any other, if the name is not wg0 but wg1 or anyname.at.anytown2.
Right?
Should not be that rather a dynamic config parameter?
(Even the auto-install script is asking for a name.)


Roll back to old version of got

New versions of got do not play well with async listener. The bug
tracking this issue is here:
https://github.com/othiym23/async-listener/issues/112.
Got is only a dev dependency used by the system tests so relying on an
older version should be fine.

Windows x node 8 failure unrelated.


IoC container & Automatic resolution

IoC & Automatic resolution

Author(s): Frank

Introduction
One of Laravel greatest features is their IoC container. It is a powerful tool for managing class dependencies. Instead of defining the dependencies hard-coded, they are injected on runtime, which allows for a greater flexibility.
Motivation
One of my motivations is the fact that I can maintain my dependencies properly. Whenever I get more dependencies, I don't want code like this:
let vehicle = Vehicle(type: Car(engine: Engine(), wheels: Wheels()))
Another motivation is that I like to code behind an interface (or protocol). Instead of injecting implementations into my classes, I inject an interface. Whenever I want to swap implementations, I only need to do this at one place in my code.
Proposed solution
The Application class is the main class of Vapor. This class could be extended by a container. In my mind registering and resolving something to/from a container would look like this.
let app = Application()

app.bind('vehicle', Vehicle(type: Car())

let vehicle = app.resolve('vehicle') as? Vehicle
// Wil return the Vehicle class with Car as the type

The automatic resolution part, although I'm not sure in which way this would be possible in Swift
// bind the CarProtocol to an implementation of Car
app.bind(CarProtocol, Car)

class Vehicle {
  let _carProtocol: CarProtocol

   init(carProtocol: CarProtocol) {
        // Whenever a carProtocol is recieved, a Car instance will be passed in
        self._carProtocol = carProtocol
     }
}

I have absolutely no clue if this matches Vapor's philosophy and in which way it's possible, but I would like to open the discussion and see how other people think about it. Personally I'm a big fan of this implementation inside Laravel, but of course there is also magic going on, which might be a downside.

As discussed in chat a bit, I think we support a lot of the intentions of this feature. One addition we should make is to add storage the way we have on request/response for greater application extensibility via providers.

Yesterday I had a talk with @LoganWright about this and one of the issues we faced with this implementation was the fact that we were matching on strings. This could possibly result in runtime errors, which needs to be minimized as much as possible. However, we do want the flexibility of being able to switch implementations without modifying all of our other classes. A way we could implement this is by using extensions. This would look like the following.
extension Application {
   var userRepository: UserRepositoryProtocol{
       get{
          // do all the stuff you need to initialize and return a new instance of a UserRepository
        }
   }
}

This way you would be able to use it like this: app.userRepository.all(). However, I can image writing an extension for Application directly could be a bit tricky, so it might be a good idea to create a seperate Storage or Container class that you can write an extension for, so you'll get app.storage.userRepository.
I'm also still looking on how we can extends the current providers functionality, so it can be added to providers. @LoganWright , @tannernelson , do you might know a good solution for this?

What is UserRepositoryProtocol trying to achieve?

The UserRepositoryProtocol is used to force a class to adhere to a contract. When I'm accepting a concrete implementation instead of a protocol, it could be the case that I'm returning an implementation that does not match what's being used in f.e. the controller. Let me explain it by code.  All it knows is that there's a contract that it can call methods on, in this case the method all.
extension Application {
    var userRepository: UserRepositoryProtocol{
        get{
            return UserRepository()
            // do all the stuff you need to initialize and return a new instance of a UserRepository
        }
    }
}

public class UsersController : Controller{

    let droplet: Droplet
    let userRepository: UserRepositoryProtocol
    public required init(droplet: Droplet){
        self.droplet = droplet
        self.userRepository = droplet.storage.userRepository
    }

    public func index(){
        let users = self.userRepository.all()
   }
}

Now let's change the UserRepositoryProtocol to a concrete implementation. As you can see it's UserRepository. In this case the UserRepository still has the method all, so that's what I'm calling in my controller.
extension Application {
    var userRepository: UserRepository{
        get{
            return UserRepository()
            // do all the stuff you need to initialize and return a new instance of a UserRepository
        }
    }
}

public class UsersController : Controller{

    let droplet: Droplet
    let userRepository: UserRepository
    public required init(droplet: Droplet){
        self.droplet = droplet
        self.userRepository = droplet.storage.userRepository
    }

    public func index(){
        let users = self.userRepository.all()
   }
}

But imagine that I change the UserRepository to UsersMongoRepository. It would look like this:
extension Application {
    var userRepository: UserMongoRepository{
        get{
            return UserMongoRepository()
            // do all the stuff you need to initialize and return a new instance of a UserRepository
        }
    }
}

public class UsersController : Controller{

    let droplet: Droplet
    let userRepository: UserMongoRepository
    public required init(droplet: Droplet){
        self.droplet = droplet
        self.userRepository = droplet.storage.userRepository
    }

    public func index(){
        let users = self.userRepository.all()
   }
}

But the problem here is that the controller can not rely on the fact that it has a all method. I could decide to rename it to getAll. However, this will result in compile errors and to make this work I need to change the all in my controller to getAll. That's something I don't want. Whenever I change my implementation, I don't want to touch the rest of my classes to make it work.
I hope this explanation works for you :) If you have any questions, let me know!

If you need to use the all() function, then that function will have to be declared in the UserRepositoryProtocol. All of this seems like it would work. The only thing we need is a storage property on the droplet.
protocol UserRepository {
   static func all() -> [User]
}

extension Droplet {
    var userRepository: UserRepository? {
        get { return storage["userRepository] as? UserRepository }
        set { storage["userRepository] = newValue }
    }
}

final class MyUserRepository: UserRepository {
       static func all() -> [User] { return [] }
}

let drop = Droplet()
drop.userRepository = MyUserRepository()

Exactly! :) I will start working on this.
The only thing I'm still wondering is if we should create the storage object on the droplet itself, or create a separate class that holds the storage. This so that people can write an extension for that class, instead of directly for Droplet. So are there any potential problems when writing an extension for Droplet directly?

@frankieleef @tannernelson I think whatever we do, we should seek parity w/ HTTPMessage which uses storage: [String: Any]. I think that's a good introduction for now, but I'm also open to proposals for a type and what sort of benefits it could add. Right now the dictionary is a little loose, so if a new type could give us more stability, could be nice.
Summary

Let's do storage now
Continue discussion on custom object

This should be achievable with droplet storage now.


MySQLServer went away

[x] This issue isn't duplicated - you can check if it is by using the search bar located at the top left hand corner and select "Issues" on the left.
[x] This issue includes appropriate markdown for sections - e.g. code blocks for crash dumps.
[x] This issue is understandable - feel free to use your native language to write issues if you are not comfortable with English.

Issue description

Reproducing the issue

run MySQL. Windows still has bugs that it disconnects. if "went away" appears, it should use the cache, and later try to reconnect

Client information
PocketMine-MP Version:
Plugin Version:
PHP version: 7.0.13 (default)

Optional information

Feel free ti fix it yourself. I don't do MySQL


Performance of "workbench.action.goToSymbol" is slow for network mapped drives

Issue Type: Performance Issue
I'm not sure whether its due to the size of the project or if I'm using VSCode over a mapped network drive.
Whenever I try to jump to a symbol within the same file, the autocomplete just doesn't work.
If I try it on a local file, it works quite well.

VS Code version: Code 1.26.0 (4e9361845dc28659923a300945f84731393e210d, 2018-08-13T16:29:31.933Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7500U CPU @ 2.70GHz (4 x 2904)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.86GB (3.23GB free)

Process Argv
D:\Programs\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    4	    91	 10396	code main
    0	    65	  2188	   shared-process
    2	   146	  7648	   gpu-process
    4	    72	 12360	   window (Issue Reporter)
    0	   207	 12888	   window (Untitled-1 - Visual Studio Code)
    0	    64	 11176	     extensionHost
    0	    32	  5448	       C:\Users\twig\.vscode\extensions\ms-vsliveshare.vsliveshare-0.3.535\dotnet_modules\vsls-agent.exe --autoexit --pipe bebb86de87384751b93353a9fb31b667 --service https://insiders.liveshare.vsengsaas.visualstudio.com/
    0	    10	  8136	         console-window-host (Windows internal process)
    0	    36	 12444	     searchService
    0	    14	 13076	     electron-crash-reporter

Workspace Info
;

Extensions (4)

Extension
Author (truncated)
Version

reasonml
fre
1.0.38

python
ms-
2018.7.1

vsliveshare
ms-
0.3.535

vscode-code-outline
pat
0.2.1

I had a hunch that disabling node_modules would improve performance and it seems to have worked
maybe switch the algorithm a little so it favours current file?

Please provide a complete set of repo steps and some example code that demonstrates the problem


Exclude download-git-repo from forbidden words

This package already exists on npm, so if I understand the matter correctly, it should be allowed here, too:
https://www.npmjs.com/package/download-git-repo
Complements ad7eb6011049ed560053864df9040576d9d1180a. Required for https://github.com/DefinitelyTyped/DefinitelyTyped/pull/56874.

Ah, I see, so what does this mean for my PR to DefinitelyTyped (https://github.com/DefinitelyTyped/DefinitelyTyped/pull/56874) Do I have to submit it under a different name?


Support 254-bit inputs/outputs

Problem
Now the generate_proof in mopro-ffi doesn't support 254-bit inputs/outputs
(which is the snark scalar field in bn254)
https://github.com/oskarth/mopro/blob/018bb1e3075d6f75f5e8edf0037bceff3f62bfbf/mopro-ffi/src/lib.rs#L66-L69
Notes
The acceptable types are listed here: https://mozilla.github.io/uniffi-rs/udl/builtin_types.html
We can change the serialize_inputs
pub fn serialize_inputs(inputs: &SerializableInputs) -> Vec<Vec<u8>>

and the generate_proof in mopro-ffi
    pub fn generate_proof(
        &self,
        inputs: HashMap<String, Vec<Vec<u8>>>,
    ) -> Result<GenerateProofResult, MoproError>

(or other uint type e.g. u64)
Acceptance criteria

It should accept bigint like<PHONE_NUMBER>1839275222246405745257275088548364400416034343698204186575808495616
It should work with current tests
(There can be a test to input the bigint)

fixed by https://github.com/zkmopro/mopro/pull/24


When using address book on Aave governance payloads, flattening for verification generates too big file

Context
When flattening with Foundry a payload like the one HERE, forge flatten contractPath generates a really big Solitidy flattened file, due to the number of interfaces' dependencies of address book, and it even contains clash of dependencies because of duplications.
This is pretty inconvenient because as we know, it is worth it even to use address book in production payloads in some cases. But with this problem of flattening, and consequently not being able to do Etherscan verification, it is not possible to introduce address book.
Proposal
Needs some research. This would be solved if on flattening only the used interfaces dependencies (and I would say even better only the functions/structs used on those) would be included.

closing as there's not much we can do and we already removed some things to minify the included amount of artifacts.


Tutorials needed on connecting to and using INBO data from R

INBO hosts many databases, so tutorials are needed for scientific and scientific support staff on how to use them from within R.
These tutorials should focus on:

[X] connecting to the respective (spatial and non-spatial) databases / datawarehouses from within R, in a dplyr / DBI compliant way (rather than using RODBC). Interesting examples can be found at https://db.rstudio.com/.

to achieve reproducibility (transferability to docker, ...) and scalability (servers), methods should be agnostic of the user's environment as much as possible. E.g.: the user's location, the operating system, the network.
one connection method that works with all INBO (spatial and non-spatial) databases / datawarehouses is the preferred way to go.

[ ] best practices using each specific database, by clearly defining the available tables and fields, and providing key scripting examples.

There is an urgent need for the following INBO databases:

[ ] Meetnetten + Watervogels
[ ] Watina
[x] INBOVEG
[ ] GIS data (using GDAL; see vignette on sf and the raster package for practical implementations in R) - see also issue #29
[ ] INBO database on surface water

Further, the need is also urgent for these databases:

[ ] LIMS
[ ] BWK-database
[ ] INBODEM

This list is most probably incomplete. (So having an overview of data is another, related need.)

 I cite here the information provided by Carine Wils:

INBO  netwerk connecteren naar de nieuwe Read only databank van MercatorNet (= PostGIS databank). Daarin zitten een groot deel van de (vector)lagen van AIV (vroegere AGIV), maar ook van ANB – INBO – dep OMG (ex-dLNE+dRV) – OVAM – MOW – Erfgoed – VMM, ed.
Het gros van deze lagen is ook beschikbaar op onze S-schijf, maar dan in de vorm van compressed FileGDB's, dus niet zo handig voor niet-ArcGIS gebruikers.

Toegang  tot deze databank is mogelijk via je milieuinfo account (diegene waarmee je inlogt op JIRA), maar je moet daarvoor wel een mail sturen naar<EMAIL_ADDRESS>met vraag om toegang tot de RO databank Mercator.

Als extra ondersteuning voor de gebruikers zijn er van elke dataset op de nieuwe databank connectie/legende bestanden aangemaakt (LYR en QLR). Deze zijn te vinden op de MercatorNet website:  https://www.mercator.vonet.be/intern-ontsloten-data-lyr-qlr en zijn hier: https://github.com/MercatorNet/MercatorUpload in één keer te downloaden.

Meer info op deze website: https://www.mercator.vonet.be.

Voor de geïnteresseerden, de connectieparameters zijn als volgt:

• Database: mercator_ro_productie
• User: <milieuinfo user productie>
• Password: <eigen paswoord>
• Instance: 5613
• Hostname: db-pr.mercator.vonet.be

Just tested DBI/odbc and rodbc for a project:

RODBC: has better documentation.
odbc: is faster, but can't find more documentation than the same README that is copy/pasted everywhere. Still, it is the technology that is used in the RStudio connections pane, so I agree we should use that rather than RODBC.

Best for INBO would be to provide examples that can be added to the connections pane in RStudio. Or bundle it into something that can easily be installed for everyone?

With respect to odbc and DBI, some information from two talks at useR!2017 can be found here. (google doc; will jump to the odbc talk; DBI is next)

at https://inbo.github.io/tutorials/tutorials/r_database_access/ a introduction is provided about R access to databases

Woohaa! This rocks. I'll certainly test this further, for now I'll propose a few tweaks here or there.

Sorry for a late response, but I only now noticed this issue.
With respect to RODBC and DBI, our styleguide recommends to use RODBC to connect to SQL Server and Access, and DBI to connect to other databases.  For me as a Windows user, this seems the right guideline to follow:  the odbc-driver that is used with DBI for SQL Server on Windows causes a lot of difficulties with special characters.  (Apparently Linux machines have a better driver and do not have these problems.)
As @peterdesmet already mentions: the documentation in RODBC is better, and to be honest: I think the error handling in DBI is terrible.  Often I have to paste my query in SQL Server Management Studio to find out what is wrong.
For databases without any special characters, it could be ok to suggest to use DBI (e.g. advantage to use dbplyr), but otherwise I would recommend to test first in a Windows environment if all characters can be imported well.  The majority of the INBO users has Windows, so I think choices should mainly be based on this OS.

A way to circumvent problems with special characters, yet still use DBI is given in this function. The input parameter scient_name in that function often contains special characters. This parameter is passed on to glue_sql with {} and afterwards converted to the correct encoding with iconv before passing it to dbGetQuery(). The problem was solved by @Jo-Loos.

As you know, I knew this solution, and it works indeed for some special characters, but unfortunately not for all (e.g. characters for male, female, <= in one character,...).  (The examples indeed have an alternative and can be replaced in the database, but there will probably pop up others without alternative that give the same problem.)
But what actually worries me, is that this all makes it more complicated for starting or occasional R programmers to use DBI.  The whole process of connecting to a database and running a query is already complicated, and then they have to add another statement to translate these characters that are not presenting well,...  Furthermore, we already searched for weeks to find this (partial) solution, they might already search for some time before they figure out it is not their mistake (which can be demotivating) and ask for a solution.
So my main concern was not if there is a solution, but: if we know there are some difficulties with DBI that will make things complicated for less experienced users and there is an alternative (RODBC), will we still recommend DBI for using with SQL Server?  What is actually the advantage of using DBI over RODBC?  Except for the presence of a connection pane in RStudio, I don't see any arguments to choose for DBI in the discussion of this issue.
So please convince me with some good arguments why DBI is prefered over RODBC.
(I don't know why @ThierryO mentioned RODBC as a recommended package for SQL Server in the styleguide.  It seems he wanted to promote DBI (as he did for most database types), but for some reason he made an exception for SQL Server and Access.)

I was just pointing to that solution because it matters a lot where in your code the conversion to the correct encoding is done. For instance, encoding the input parameter and then passing it to glue_sql will not work.
I don't have strong opinions, about DBI or RODBC, but I find exploring the database via the connection pane a big plus and it is considered the "best in class" package by rstudio.

Something else to consider here is that the tidyverse package dbplyr is designed to work with DBI connections (see next coding club).

Fast forward to 2020: we now have the inbodb package to address some challenges! It would be good to have a pointer to inbodb in the tutorials website, both in the database-R tutorial and in the overview article of INBO software. @ElsLommelen can you give your opinion? (you're best placed to implement this as well)
Also the INBOVEG tutorial will need some updating @ElsDeBie
Beside that, I think we should further investigate / specify the need for functionality for specific databases, i.e. database-specific queries and preprocessing through R functions. This relates to the second sub-part of this issue, i.e. "best practices using each specific database".

Some ideas for the database-R tutorial:

one easy step to start with: replace inborutils by inbodb to guide users to the right database
it could be useful to mention in this tutorial that for some databases (INBOVEG and Florabank), ready-to-use functions exist to query the database, with of course a reference to the INBOVEG tutorial
setting up the connection yourself: for Windows users it is certainly useful to mention here to add parameter encoding = "latin1"
certainly for Access we should do some further tests:

are special symbols and complex data types an issue as well in DBI/odbc?
should we add a specific function in inbodb that overcomes these problems (if possible), or mention RODBC as an alternative package, with some advantages and disadvantages for both? (Access provides possibilities to explore the database and compose queries as an alternative to the connection pane and dbplyr)

maybe change the url https://inbo.github.io/tutorials/tutorials/r_database_access/ because the word access can be confusing as it mainly deals with SQL server?
would it be useful to add a short note on other database types (postgresql, sqlite,...) and the fact that dedicated packages exist to make this connection?
is there a tutorial on dbplyr, or should this be added here?
the tutorial is rather long, which might make it difficult to look up specific information.  Would it be an idea to split it in 2 (or 3) tutorials: one on a connection and one on queries, and maybe a third one as a general page that explains that one needs 2 steps: connection and queries.

INBO software: I'll add inbodb and a short description
I'll start working on a PR with the first idea for the database tutorial and the software and I'll wait on some more opinions for the other ideas.  Maybe it would be better if someone with experience on it would add something on dbplyr (if necessary)?
Maybe it is a good idea to involve @hansvancalster in this discussion as well?  (For the tutorials on Florabank and INBOVEG)

Considering the INBOVEG tutorial: maybe it better fits as a vignet in inbodb?

Fine ideas @ElsLommelen !

maybe change the url https://inbo.github.io/tutorials/tutorials/r_database_access/

perhaps r_database_sqlserver ? MS Access better goes into its own tutorial.

is there a tutorial on dbplyr, or should this be added here?

You could simply refer to https://dbplyr.tidyverse.org/index.html

split it in 2 (or 3) tutorials: one on a connection and one on queries

If you would do that, I'd prefer two tutorials rather than three, and which should clearly refer to each other at the beginning. Otherwise, the interconnection gets lost (the website doesn't provide tutorial-hierarchy) and in such case I think it's better to keep it together as one.

perhaps r_database_sqlserver? MS Access better goes into its own tutorial.

Because the general use is similar for all database types, I am not sure the best option is to give each database type its own tutorial.
Also, maybe (part of) the database tutorial should better be part of a vignet in inbodb...  I feel the whole idea should have some more considerations.  For now I will make some minor changes to direct people to the right package, and I will have a closer look at it later on.

I am not sure the best option is to give each database type its own tutorial

Sure, common parts better aren't duplicated.
Regarding the tutorials-website vs. vignette, I leave such preference to you, it is not that obvious to me what is best. It's certainly a good thing to have self-contained help within the package. If the tutorial (or parts of it) were to move to the inbodb package, I suggest to still leave a 'tutorial' here that directs to the inbodb package for more information.

Part of the tutorial describes methods not using inbodb, so this will stay on the tutorial website anyway.  Other parts can directly link to the relevant inbodb vignet.

Another issue with inbodb: where does it fit best on the INBO software website?  It certainly belongs to retrieve data, but it covers partly the technical aspect (connecting to databases) and it partly queries databases.  For now it only queries databases with biological data (and is put in this section), but as part of the INBO databases contain environmental data, the package inbodb may in future contain queries on environmental data as well.  Considering all this, where does it fit best?

Maybe in the table add a row retrieve data: general, or do not make that distinction in the table and just lump them in one row retrieve data? In the remainder of the article, the distinction is useful (and I guess also in the table, so I am in favor of adding a row).

I agree with Hans to add a row Retrieve data: general, above the current ones on that topic. You can then add a new subtitle in the text, under 'Retrieve data' and put inbodb there.

Are there any good sites about 'how to write a vignette'? I quickly searched and find this: https://kbroman.org/pkg_primer/pages/vignettes.html
http://r-pkgs.had.co.nz/vignettes.html
For tutorials, we have our: https://inbo.github.io/tutorials/create_tutorial/. I can imagine that we have to write more vignettes, perhaps it's also useful to make a tutorial for this? Or are there generel R rules?

Hi @ElsDeBie, I found Hadley Wickham's book very helpful; see the vignettes chapter: http://r-pkgs.had.co.nz/vignettes.html

Euh, no idea.  Easiest to generate a vignette is using the function usethis::use_vignette() and specify a name for your vignette.  (This function deals with all settings and the heading, you only have to push these changes to github, and of course add content to your vignette.)  That's about it for the technical part, I think.
And for the content, to me it seems the same as any other text: focus on a (potential) user group for your package and write down what they should really know to use your package.  I generally start with writing the goal of the vignette (and the focal user group), and try to use a lot of examples.

Hm, I should have looked better, you've listed r-pkgs already :wink: Anyway, thanks for the tutorial of Broman, seems interesting - didn't know it.

@ElsDeBie In the link by Hadley Wickham, just replace package name devtools with usethis if the function is unknown: some functions have moved to usethis some years ago, and for some months these functions are effectively removed from devtools.  Unfortunately a lot of documentation (even the develop cheatsheet) is not yet updated with these changes.

There is a 2nd edition of the book under development at https://r-pkgs.org. It seems that the authors did already make updates to the vignettes chapter at https://r-pkgs.org/vignettes.html.

Els Lommelen: > Another issue with inbodb: where does it fit best on the INBO software website? It certainly belongs to retrieve data, but it covers partly the technical aspect (connecting to databases) and it partly queries databases. For now it only queries databases with biological data (and is put in this section), but as part of the INBO databases contain environmental data, the package inbodb may in future contain queries on environmental data as well. Considering all this, where does it fit best?
--> Is it an idea to put between brackets the databases mentioned in this package? Otherwise users can be lost if they don't no where to search for ..."Retrieve data: general | inbodb (technical support, Inboveg, FLoradatabank)".
Or maybe beter to put a third column with 'concerning databases'
https://inbo.github.io/tutorials/articles/inbo_software/

@ElsDeBie it has both pros (more informative) and cons (needs maintenance). @ElsLommelen what do you prefer?
Adding the explicit databases (in the description) was actually her first approach; I suggested generalization :thinking:  (see https://github.com/inbo/tutorials/pull/171#discussion_r447543421).
Regarding the form: I would not insert package-specific information in the table, as the table intends to give a very global overview. Package-specific information is given in the text below. @ElsDeBie Maybe you can propose a better category name than 'Retrieve data: general', that would a better fit ?
Still another possibility could be to repeat the package under more than 1 category in the table.

@ElsDeBie I agree with @florisvdh and I would rather prefer to not mention specific databases for maintenance reasons.  When adding new databases to inbodb, it is easily added in for instance the readme of inbodb itself, but keeping the tutorials website up-to-date as well (and especially remembering to do so whenever we make changes in inbodb), will require an additional effort.  On the other hand it would be nice to name all these specific databases (inboveg and florabank for now) in the readme of inbodb, so visitors will immediately have this information while using the link in the tutorials website.


how to set Multiple Sessions per user?

This is not possible as each peer has a particular IP address and set of keys assigned.
You need to generate a new configuration file for each peer.


Programmatic scheduling for templates

Allow the Ergo logic for a template to indicate to the host environment when the template should be next triggered.

I propose that scheduled callbacks are expressed in terms of Intervals. This allows the Ergo logic to state that they need to be called back within a given time Interval. Please see https://www.joda.org/joda-time/key_interval.html for a description of time intervals.
concept Interval {
   o DateTime start
   o DateTime end
}

schedule myInterval;

Or:
schedule toInterval(myDateTime, myPeriod);

The schedule keyword would be similar to the emit keyword, in that it would allow the Ergo logic to add DateTime instances to an array that could be retrieved by the host environment after processing of the incoming request has completed.
It is then the responsibility of the host environment to callback the clause/contract for all the specified set of Intervals. If the host environment cannot satisfy any of the requests (e.g. the start of the interval is in the past, or the duration between the start and end of the interval is too short) the host environment may throw an exception.
The host environment may also use the interval information to sort/order the scheduled callbacks, potentially using different algorithms/queues/persistence for callbacks that are 5 years in the future with a period of 24 hours, vs intervals in 10 minutes with a period of 15 seconds.

See also: https://docs.oracle.com/javase/1.5.0/docs/api/java/util/concurrent/ScheduledExecutorService.html

How does the host environment know how to callback the clause / contract? For example, what is the payload of the scheduled request?

Good question! I'll defer the second one to @jeromesimeon who has more insight into the "full contract" questions. I'd imagined that we will add a new modelled transaction for these callbacks. Something like:
transaction ScheduledRequest extends Request {
}

This would allow the template developer to handle ScheduledRequest as they handle other incoming request types.

A few (not very informed or clear, so bear with me) reactions:

we should not try to do "real time" or "near real time" (which really means real time to people working on embedded systems)
whose responsibility is it to ensure the scheduling makes sense (not in the past, not conflicting, etc.)
whose responsibility is it to execute the call back?
The call back should definitely indicate what has to be called (a request to be sent? a direct clause call?)
I might model those as Obligations rather than ScheduledRequests, maybe... maybe something like:

event ScheduledRequestObligation extends Obligation {
  o Request request
  o Interval interval
}

or
event ScheduledCallObligation extends Obligation {
   o String clauseName
   o Params params
   o Interval interval
}

We might not want to let programmers do those kinds of call back without a story on concurrency (e.g., what happens when two call backs overlap, is there locking? is there a different kind of scheduling?

I would love to see a concrete use case... (scenario?)

A few more questions:

Do we need a cicero-scheduler package for execution
Is there Node.js npm package scheduler we can reuse or do we need to write our own
How do we test this feature?

Here is a schema proposal for automatic scheduled requests by templates.
These models should live in org.accordproject.cicero.runtime namespace at https://models.accordproject.org/cicero/runtime.html
import org.accordproject.time.Period from<EMAIL_ADDRESS>import org.accordproject.time.Day from<EMAIL_ADDRESS>
abstract event ScheduledRequestEvent {
  o Request request
  o DateTime until optional
}

event PeriodScheduledRequestEvent extends ScheduledRequestEvent {
  o Period period  // period.unit has type PeriodUnit which is already constrained to a maximum granularity of 1 day. :+1:
}

// In the future, we may also want to support other types of scheduled request, e.g.
event DateTimeScheduledRequestEvent extends ScheduledRequestEvent {
  o DateTime time
  o Period granularity
}

event DayOfWeekScheduledRequestEvent extends ScheduledRequestEvent {
  o Day day
}

event DayOfMonthScheduledRequestEvent extends ScheduledRequestEvent {
  o Number day
}

These events are emitted in the same way as obligations by contract logic. e.g.
emit PeriodScheduledRequestEvent
  request: Request { ... },
  period: Period {
    unit: "weeks",
    amount: 2
  }
};


Run sqlite statements

This works, but I don't love how it turned out. ResolvedRuntimeConfig knows nothing about the underlying runtime config type that is parsed from the runtime config source. This means that only the caller knows that the parsed runtime config has access to sqlite specific runtime config (and thus the list of labels -> connection creators). So we have to pass that information back down from the caller into the callee.
It's perhaps not 100% elegant, but it works. Perhaps we can think of a better way to structure this so that this round about data passing isn't necessary.

Could this be done in a ExecutorHooks::configure_app? That has access to AppState, and SqliteFactor's AppState should be able to execute without this separate logic.

@lann  - I made ExecutorHooks an async trait. It required constraining a few AppState's to be Sync, but I think that's fine?

It required constraining a few AppState's to be Sync, but I think that's fine?

We should probably have a blanket constraint on AppStates to be Send + Sync anyway; I'm pretty sure it is implicitly required already.

Also notable: the one impl of ConnectionCreator isn't actually async...

I made ConnectionCreator non-async, and I added some tests. I think this should be good to merge now (assuming CI passes).


Forks: Support for Flora

As requested by Digital Spaceport and AfelBirnelKreis

VOTE 4 ADDING FLORA

Yes, this would be much appreciated :)


ignores latest major version

What happened incorrectly?
The repository has the following tags, but the next version is 0.16.0 and not 1.x.0. Comprehension problem on my part?
0.0.1
0.1.0
0.10.0
0.11.0
0.12.0
0.13.0
0.14.0
0.15.0
0.2.0
0.3.0
0.4.0
0.5.0
0.6.0
0.7.0
0.8.0
0.9.0
1.0.0

Can you please provide a repository where we can reproduce this?

Unfortunately not, as it is a private repository. But we can have a look together if it helps.

Can you please setup a publicly accessible sample repository (or invite me to a private one)?

I'll try to recreate this in a separate repository that I can give you rights to.

Unfortunately, we can't get it cleaned up enough to share the repository. We'll have to look around for an alternative first. Thanks for the feedback.


Feat/read file

Description

Updated read file code for unstructured-io. The code also reads epub files using EbookLib

Related Issues

Solution and Design

Test Plan

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Docs update

Checklist

[x] My pull request is atomic and focuses on a single change.
[ ] I have read the contributing guide and my code conforms to the guidelines.
[ ] I have documented my changes clearly and comprehensively.
[ ] I have added the required tests.

 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.8 out of 9 committers have signed the CLA.:white_check_mark: luciferlinx101:white_check_mark: nborthy:white_check_mark: anisha1607:white_check_mark: Tarraann:white_check_mark: NishantBorthakur:white_check_mark: jedan2506:white_check_mark: Fluder-Paradyne:white_check_mark: AdarshJha619:x: boundless-asuraYou have signed the CLA already but the status is still pending? Let us recheck it.

created from main instead of dev


[Feature]: Golang skills development

Description
Learn Golang language :rocket:
Expected behaviour
Here is what you have to do :

[x] Follow the entire Go Tour
[x] Follow this PoC's Golang course until step 9
[x] Follow this PoC's Gin course until step 9
[x] Read this Golang Interface article
[x] Read this Golang Data Structures article

Environment
/
Proposed solution
/

All done !


ReactNode icons not supported, even though types specify you should use them

Using a ReactNode as icon gives me this error:

Here's my code:
<NativeTabs.Screen
        name="(home)"
        options={{
          title: "Home",
          tabBarIcon: ({ focused, color, size }) => (
            <Ionicons name="home" size={size} color={color} />
          ),
        }}
      />

Hey!
Thanks for opening the issue.
I've looked at your PR: https://github.com/fredrikburmester/streamyfin/pull/166/files
It looks like you are importing BottomTabNavigationOptions
import { BottomTabNavigationOptions } from "@react-navigation/bottom-tabs";

You should import
import { NativeBottomTabNavigationOptions } from "react-native-bottom-tabs/react-navigation"

This will give you correct typings. You can't pass ReactNode for the icon.
If you want to use react-native-vector-icons you can use getImageSourceSync from react-native-vector-icons:
import Icon from 'react-native-vector-icons/Ionicons';

const navIcon = Icon.getImageSourceSync('md-arrow-back', 24, 'white');

Ah, thank you for the amazing support!


ensure kube client Delete func works on rootScoped resources

The PR

fix bug in kube crd client to ensure it's delete func can work on rootScoped resources.
(though users can make a  "empty->default namespace" conversion themselves before using the func,
I think we'd better have protection in it, just like we did in Create/Get/Update)

Add delete step in CheckIstioConfigTypes to make it idempotent.
Otherwise, we'll get "already existing error" when running test TestTempWorkspace multi times:

    --- FAIL: TestTempWorkspace/istioConfig (0.03s)
    	config.go:459: Post(RbacConfig) => got rbacconfigs.rbac.istio.io "default" already exists

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ayj
If they are not already assigned, you can assign the PR to them by writing /assign @ayj in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @lichuqiang. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/assign @ayj

@ayj Can you please take a look?


Chore/secure docs gen

Ensure deterministic docs generation by enforcing jq version 1.7.
Also re-generate docs to make linters happy.

Summary by CodeRabbit

Chores

Added a version check for jq in the build process.

Documentation

Updated hash values in the rendered documentation for:

okp4-cognitarium
okp4-dataverse
okp4-law-stone
okp4-objectarium

size-limit report 📦

Path
Size

target/wasm32-unknown-unknown/release/okp4_objectarium.wasm
444.69 KB (0%)

target/wasm32-unknown-unknown/release/okp4_law_stone.wasm
648.68 KB (0%)

target/wasm32-unknown-unknown/release/okp4_cognitarium.wasm
835.63 KB (0%)

target/wasm32-unknown-unknown/release/okp4_dataverse.wasm
674.44 KB (0%)

:tada: This PR is included in version 5.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Datepickers are not generated (javascript calendar and clock)

Hello,
It's really an amazing work how you transformed the admin area with bootstrap, thank you so much for
sharing your great work.
But the javascript calendar and clock icons are not inject when using bootstrap with my admin area.
but when i disable bootstrap they appear here is the code:
 Today |
I'm using Python 2.7.3 and Django 1.7.1 in a python virtual environment, i already run the command
"python manage.py collectstatic" inside the virtual environment shell. it moved all static files to the
static directory of my site, and no files are missing according to the browser debugging logs.
so is the problem because of an incompatibility with the version of Django or python I'm using
or is it a missing feature in your integration.
Thank yous o much for your support and your very good work.

Hello, i have the same problem here if i add this line under settings.py
DAB_FIELD_RENDERER = 'django_admin_bootstrapped.renderers.BootstrapFieldRenderer', the date picker is gone im using django 1.7 and python 2.7.3 thru and nginex server.
thanks

I can confirm the same issue when adding the line :DAB_FIELD_RENDERER = 'django_admin_bootstrapped.renderers.BootstrapFieldRenderer'

Guys if any of you can point a meaningful error that would be easier to track down and eventually fix.

I hope this helps,
Simple installing and adding to the installed apps this package,  produce for a DateTimeField an output like this in the admin:

and it works just fine, but if as you suggest in the documentation for a better boostrap experience:
"For a full bootstrap3 experience you may want to use a custom renderer for the fields. There's one available in tree that requires the django-bootstrap3 application installed. You have to add to your project settings file:
DAB_FIELD_RENDERER = 'django_admin_bootstrapped.renderers.BootstrapFieldRenderer'"
once i install the django-bootstrap3 add the above constant in settings the output in the admin is like this:

as the original author of the issue suggested the problem is there is no clock icon nor calendar icon,
i am using python 3.4.2, django 1.7.7, django-admin-bootstrapped==2.4.0, django-bootstrap3==5.4.0, there is no console error, no not founds not anything

@LRPalacios thanks! now it's clear to me that the issue in django-bootstrap3 trying to render a SplitDateTimeWidget. Will try to fix.

Opened a pull request for django-bootstrap3 here:
https://github.com/dyve/django-bootstrap3/pull/253
Some changes are required on our side too

Hi there, could you please double check latest git works for you? thanks!

Hello xrmx,
Could you tell me which version that obtains the fix I should download?
In my admin page I see the datetimefield as a single text box and not as a "AdminSplitDateTime".
I'm using django_admin_bootstrapped==2.5.6, bootstrap3_datetime==2.2.3, django_bootstrap3==6.2.2, django 1.8 and python 3.4.
Thanks!

@iliashko 2.5.1 see milestone field in the right column, or just read the CHANGELOG. The bug here was about missing js calendar widget not missing input fields so please check if you can reproduce this with the shipped test project, i'd start with removing bootstrap3_datetime.

@xrmx , thanks for you reply.
I'm new to Django and trying to make my admin page to work properly with django-admin-bootstrapped.
I'm actually having trouble with modifying a DateTimeField to work with the bootstrap3_datetime widget, and i'm not sure regarding to where I should ask about bootstrap3_datetime.
I've posted it in stackoverflow but didn't get any answers. If you could take a look and point me to the right direction, that would help tremendously.
http://stackoverflow.com/questions/33445868/django-bootstrap3-datetime-widget-in-admin-site-doesnt-pass-form-data
Thanks again.

@iliashko you should ask to bootstrap3_datetime developers, asking here to fix your own issues it not how it is supposed to work. Good luck!


How to handle inputs consisting of sentence pairs?

Hi,
I tried attacking a victim model for SNLI with inputs being pairs of sentences. The  underlying HuggingfaceClassifier expects a string as input, hence I am wondering what is the expected way of handling sentence boundary in this situation.
I tried including a [SEP] token at the sentence boundary, but this resulted in the token being attacked sometimes, which seems to be an undesired attack.
Or a simple concatenation of the premise and hypothesis sentences should do the job without indicating the boundaries between the sentences?
Thanks for the help in advance.

We added an example. See examples/nli_attack.py .


Is the RSS / Atom feed gone ?

Technical Bug Report
Your Environment
Not relevant here.
Is this an issue with Rust, CSS, or HTML (or a combination)?
It looks like a server-side issue, so probably Rust.
Expected Behavior
There used to be a way to get an RSS / Atom feed of the Rust blog via https://blog.rust-lang.org/feed.xml . This was convenient for aggregating Rust news with other news in a single RSS feed reader like Thunderbird.
Current Behavior
The URL now errors out with a 404 "not found" error. Is the feature gone?
Step to Reproduce
curl https://blog.rust-lang.org/feed.xml

this is definitely an issue, but an issue for the blog repo :) i will move it.


Dependency updates

Updated 3rd party dependencies along with Angular Core and CLI packages.

Looks like I butchered the merge commit message not understanding what I was doing. I'm used to the GitLab interface ¯_(ツ)_/¯

Looks like I butchered the merge commit message not understanding what I was doing. I'm used to the GitLab interface ¯_(ツ)_/¯


Plugin has stopped working as of the latest release

Todos are not being rolled over.

If I create today's note then the todos aren't copied across
If I manually run the Rollover Todos Now action then nothing happens

Yesterday's note
## Day Plan

- [ ] Test  1
- [x] Sevgin
	- [x] PR
	- [x] Infra
- [ ] Test 2
- [x] Sam
	- [x] PRs
		- [x] Database, Doppler
	- [x] QA tests, create a task
- [ ] Test 3

Today's note
## Day Plan

Settings below

This worked until very recently, I assume it is the latest release.

Can you share the settings for your daily-notes-plug-in?

Great question, I think that I would have had Daily Notes configured and then added Periodic Notes later on for the weekly, yearly notes.
I've had this configured for ages now.
I've just turned off Daily Notes and recreated Today's Note and still get the same issue. (no todos rolled over, automatically or manually).
Anything else you'd like me to try?
Thanks

I've debugged the source a bit.
If I change
if (path.startsWith(`${folder}/`)) {
// to
if (path.startsWith(`${folder}`) || path.startsWith(`${folder}/`)) {

Then everything works perfectly.
I am not quite sure how your logic is working above around folder paths to make a PR.

I might have the same issue... I have enabled delete todos from previous day and those are deleted, but they aren't added to the current daily note.

Hello! I'm having the same issue and have been having it since I installed this plugin about a week or two ago. The setting for deleting to-dos does function properly, but nothing rolls over to the following day.
If you'd like me to provide some kind of information or try something to help diagnose the issue I'd be happy to give it a try. Here's my settings:

Thanks a bunch! This plugin would be very helpful for me so I'm looking forward to using it in the future.

Hey @PrinceHeadache and thanks for offering your help. It would in fact be very helpful to know the settings for your daily notes plugin (Daily Notes/Periodic Notes).

@tim-hilt
Sure thing! Hopefully this works:

Thanks!

Hello!
I am having the same problem since a few days ago.

I've debugged the source a bit.
If I change
if (path.startsWith(`${folder}/`)) {
// to
if (path.startsWith(`${folder}`) || path.startsWith(`${folder}/`)) {

Then everything works perfectly.
I am not quite sure how your logic is working above around folder paths to make a PR.

I'm pretty sure that was introduced by that line:
https://github.com/lumoe/obsidian-rollover-daily-todos/blob/1f20c0a758a4827ae2dd0791b62b599c8f5af4b7/src/index.js#L72
Because of that, the following logic is no longer needed:
https://github.com/lumoe/obsidian-rollover-daily-todos/blob/1f20c0a758a4827ae2dd0791b62b599c8f5af4b7/src/index.js#L108-L111
Here, we added a slash at the end of the daily-notes folder, but because of the first line I quoted, it basically checks if there are two slashes at the end, which will (hopefully) never be the case. I'll submit a PR soon and let this thread know.
If you know how to build and run an npm-project, I'd be very happy to have this tested by more than one person.

I updated to 1.1.5, but it still stopped working and became invalid.

I can’t comment on why it became invalid, but since my PR mentioned above is still not merged, there is no chance of the bug being fixed on any released version.

I tried the "hack" from @seagullmouse and rollover get deleted again.. but they are not added to the current daily note.
@tim-hilt did you test your merge?

Yes it works... but I found a new issue..

hi,i met the same problem.Sometimes it works well,but sometime does not.
here my setting

have turned off the official daily notes

@lumoe is there anything holding you back from approving #96?

@all please try the newest release 1.1.6

@ALL please try the newest release 1.1.6

It works again for me after updating :)

It works for me now, too. Thanks so much!


is sequence applying effects in backwards order?

Compare:
def allocatePersonS(name: String) = State { id: Int => (id + 1, Person(id, name)) }

scala>     val allocateThreePeople = for {
     |       p0 <- allocatePersonS("Alice")
     |       p1 <- allocatePersonS("Bob")
     |       p2 <- allocatePersonS("Charlie")
     |     } yield (p0, p1, p2)
allocateThreePeople: cats.data.State[Int,(Person, Person, Person)] = cats.data.State@24407ee8

scala>     val (id3, (p0, p1, p2)) = allocateThreePeople(initialId)
id3: Int = 3
p0: Person = Person(0,Alice)
p1: Person = Person(1,Bob)
p2: Person = Person(2,Charlie)

to:
val (id3, List(p0, p1, p2)) =
  List("Alice","Bob","Charlie")
    .map(allocatePersonS)
    .sequence[State[Int,?],Person]
    .apply(initialId)

// Exiting paste mode, now interpreting.

id3: Int = 3
p0: Person = Person(2,Alice)
p1: Person = Person(1,Bob)
p2: Person = Person(0,Charlie)

Haven't had a chance to check whether it's a problem with sequence or with ListInstances.listInstance or if it's something wrong that I've done; confirmation would be appreciated :)

Did you intend this to be opened in cats? :)

Crap, sorry!


Fixed malformed regex in pipeline rule

Original guide had a malformed regex in the pipeline's extraction rules, and simple copy-paste would result in a syntax error.
Errors, even if easily spottable, in a guide that could be used to complete newcomers (Such as I was a few weeks back) to a platform could turn them away.

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


Various init.d script fixes

There are two main fixes here:
-- don't redefine function is they are already defined by the distro.
-- don't pass -f to pgrep, which pattern matches against the entire line. Just match against the process name with a fully anchored pattern.
Tested on Ubuntu 14.04 and service start and stop remain fine, even when the config file is open in an editor.

CLA confirmed signed by @daachi
This PR adds a final anchor to the regex. The script has been tested, and InfluxDB is now properly controllable even when config file is opened in an editor.

Should fix https://github.com/influxdb/influxdb/issues/3524

@dgnorton @beckettsean

+1

There has been a lot of work in this area as a result of other work, I'll leave this to @rossmcdonald to decide if there is anything we can take from this.
Thanks for your help @daachi


fix(schema): warn if user provides vite.publicDir

🔗 Linked issue

Resolves #21831
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[x] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This PR is based on https://github.com/nuxt/nuxt/issues/21831#issuecomment-1611180881
I hope I'm doing it in right way and as @danielroe intended. 🙏
References:

https://vitejs.dev/guide/assets.html#the-public-directory

📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

@danielroe
Oh, sorry I couldn't reply to your comments.
Thank you for refactoring my PR and merging it!


Union functionality

Hi,
I added a nice functionality with the help of turf to union shapes:

userGroup is a featureGroup in my case

`function unifyGroup(userGroup){
var polyList = userGroup.getLayers();
var unionTemp = polyList[0].toGeoJSON();
for (var i = 1; i < polyList.length; ++i) {
unionTemp = turf.union(unionTemp, polyList[i].toGeoJSON());
}
var fullUnion = L.geoJson(unionTemp);
userGroup.clearLayers();
fullUnion.eachLayer(function (layer) {
userGroup.addLayer(layer);
});
return userGroup;
}`
Maybe you want to integrate it. 👍

Thx for comment, we will implement in the future a function with that.
Same as: #608


[Release-1.26] Support new integration artifacts for PR install

Is your feature request related to a problem? Please describe.

We need to backport a portion of https://github.com/k3s-io/k3s/pull/9185, to enable other release branches to be tested with the INSTALL_K3S_PR env var.

Sha256sum is not enabled on this branch, so no change was necessary


Update glossary.md - Kubernetes

https://github.com/haxtheweb/edtechjoker/pull/264/commits/23990b88a66cb0a9cf72cf6ae9f5d833278cc471


[INTERNAL] ComponentAnalyzer / XMLTemplateAnalyzer detection of views

Cherry-pick of https://github.com/SAP/ui5-builder/pull/808

Coverage increased (+0.5%) to 94.717% when pulling 18e15c3e7c21b69c7b4645db5661028dab28629a on next-fix-TypedViews into e99d58defb7b5e2e8e5de0a5c8a4b007c9092130 on next.


bitscope: create derivations with version info

Motivation for this change
So far the store paths produced by the bitscope derivations have been
version-less. Fix it.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Applied to master (6b834384f669a16008fdc6886e235939af206bf4).


Can't open platformio terminal

I've attempted to uninstall and reinstall both atom and platformio but still no luck. Every time I try to open the terminal I get the error attached.

read https://github.com/platformio/platformio-atom-ide-terminal/issues/584

@ivankravets please close this is a duplicate of https://github.com/platformio/platformio-atom-ide-terminal/issues/584


Add fuel consumption modes

Closes https://github.com/paritytech/wasmi/issues/702.

The diff coverage is 5.00%.

@@            Coverage Diff             @@
##           master     #706      +/-   ##
==========================================
- Coverage   79.52%   79.40%   -0.12%
==========================================
  Files         100      100
  Lines        8414     8426      +12
==========================================
  Hits         6691     6691
- Misses       1723     1735      +12

Impacted Files
Coverage Δ

crates/wasmi/src/engine/executor.rs
77.66% <0.00%> (-0.91%)
:arrow_down:

crates/wasmi/src/engine/mod.rs
81.02% <ø> (ø)

crates/wasmi/src/engine/config.rs
65.33% <16.66%> (-4.67%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@athei Gotta fix the performance regressions before we continue merging this PR ...

I think the benchmark regressions that we have seen on Wasm targets might be unusual noise. At least with the last commit we see significant improvements even though it is the same set of changes as another previous commit with which we have seen significant regressions ... not sure what to do but I guess it is best to simply go forward and merge this PR ...


How to sort the import statement which not contains import name

For Example, I want the import './SaveNFTModal.less'; to be last line.
My configuration:
{
  // Wiki: https://github.com/daidodo/format-imports/wiki/Grouping-Rules
  "tsImportSorter.configuration.groupRules": [
    ["^[@]", {}],
    ["^api", "^components", "^features", "^style", "^util"],
    "^[.]"
  ],
  "tsImportSorter.configuration.wrappingStyle": {
    // Max binding names per line before wrapping for imports. 0 for no limit.
    "maxBindingNamesPerLine": 0,
    // Max default and binding names per line before wrapping for imports. 0 for no limit.
    "maxDefaultAndBindingNamesPerLine": 0,
    // Max binding names per line before wrapping for exports. 0 for no limit.
    "maxExportNamesPerLine": 0,
    // Max names on wrapped lines. 0 for no limit.
    "maxNamesPerWrappedLine": 0,
    // Whether to ignore trailing comments when counting line length.
    "ignoreComments": false
  },

  "tsImportSorter.configuration.sortRules": {
    // Sorting rule for import paths. Valid values are 'none' or an array.
    "paths": ["_", "aA"],

    // Sorting rule for imported/exported names. Valid values are 'none' or an array.
    "names": ["_", "aA"]
  }
}

@daidodo

Hi @tjx666, thanks for the feedback!
The imports are classified as script and named imports: Types of Imports
And they are dealt with differently: flags
Example: How to customise script imports
Please tell me if you have further questions!

Thanks for so awesome project!


Incorrect limitations mentioned

The limitations section currently states that only 5 tenants can be linked using MTO while we do see an option of connecting up to 50 tenants in the portal itself.
Kindly correct this.
https://learn.microsoft.com/en-us/entra/identity/multi-tenant-organizations/multi-tenant-organization-overview#limits
[Enter feedback here]

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

ID: 94ff49b6-b5fe-df1a-c359-c81006998929
Version Independent ID: 7e765471-f9ba-61e7-e30a-bad2f16c66f6
Content: What is a multitenant organization in Microsoft Entra ID? (Preview) - Microsoft Entra ID
Content Source: docs/identity/multi-tenant-organizations/multi-tenant-organization-overview.md
Service: entra-id
Sub-service: multitenant-organizations
GitHub Login: @rolyon
Microsoft Alias: rolyon

@the-sentinental-guy
Thanks for your feedback! We will investigate and update as appropriate.

@Naveenommi-MSFT Please check and confirm soon. Need a confirmation that the limit (of 50) in the MTO portal is not conflicting with the limit (of 5) mentioned here.

@the-sentinental-guy Thank you for the follow-up. We are awaiting confirmation from the PG team to update this document.

@the-sentinental-guy
We will close this thread for now. Once I receive confirmation from the PG team, I'll update the status here. If you have any further questions regarding the documentation, please tag me in your reply, and I'll be happy to continue the conversation.


*: support granting a role to a user in v3 auth

/cc @xiang90 @ark76r

LGTM after fixing the comments.

@xiang90 updated for the comments, PTAL.

LGTM. I will get this merged tomorrow morning in US time. Thanks!

@xiang90 updated for the last comment. Thanks a lot for your review, good night :)


PowerShell script improvements

This PR makes the following improvements to this PowerShell script:

Renames the functions to have PowerShell approved verb-noun names. Uses alias to ensure the previous function names still work for backwards compatibility.
Adds comment based help to the functions so get-help will return useful output.
Adds cmdletbinding and supportsshouldprocess so default parameters such as -whatif and -verbose can be used.
Makes the parameters mandatory rather than using throw if no input is provided.
Puts $null on the left side of comparisons to avoid potential false positives.
Changes sections using write-error and return into the simpler throw which achieves the same result.

Note that these changes should still allow these functions to be used in exactly the same way as they were previously (no breaking changes), these changes just add the additional safety of the -whatif switch being available and other PowerShell best practice improvements.

Closing as this repo doesn't seem to be being maintained any more.

That sad. Your proposal was great.


Enable WGSP tests that works for Metal related to Semantics

This commit enables existing tests for WGSL that are enabled for Metal regarding the Semantics.
Closes https://github.com/shader-slang/slang/issues/5502

Just FYI, the commit message says "WGSP" instead of "WGSL".


[SVProgressHUD show] does not display

iOS11 xcode 9 at the time of page jump In jump target page viewdidload method perform [SVProgressHUD show] does not display

[SVProgressHUD show] , I found the view under the display view which show my control. But i can not find the way to resolve it.

Execute [SVProgressHUD show]  after VC viewDidAppeaer  is  display. So, How to solve this

@SionChen  try this in viewdidload:
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.1 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
[SVProgressHUD show];
});

There is a change that SVProgressHUD not showing when using UIAlertView, deprecated in iOS8, as a popup an alert.
Step:

[SVProgressHUD show] ;
[UIAlertView show];
or
[UIAlertView show];
[SVProgressHUD show] ;

Example: https://github.com/apptasklpaco/SVProgressHUDNotShowing

Some issue
CGFloat second = 0.3;
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (second * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
    [SVProgressHUD show];
});

second = 0.2 is not work but second = 0.3 is work for me.


(StoredProcedure) support for CURSOR/REFCURSOR OUT parameters is needed

❗ please do not add sensitive information in issues, you can provide extra information via email using issue number as reference ❗
Describe the issue
I have a use case where this feature is required so please implement it
Reporter
Ali

@tnleeuw can you pick this up with Ali


Use PostQuery for Timber::get_context()'s default return

Coverage remained the same at 97.326% when pulling dad2dc5bd94d5b7a83c31b5b8d8d12126fff8776 on 1.3/default_post_query into daf7be3dfeb126f79c68324b08fc274b261304bb on master.

Coverage remained the same at 97.326% when pulling dad2dc5bd94d5b7a83c31b5b8d8d12126fff8776 on 1.3/default_post_query into daf7be3dfeb126f79c68324b08fc274b261304bb on master.

Changes Unknown when pulling deca10f72f91a0cd3c1dc32fc1bbf9f79fe4405a on 1.3/default_post_query into ** on master**.

Coverage remained the same at 97.326% when pulling deca10f72f91a0cd3c1dc32fc1bbf9f79fe4405a on 1.3/default_post_query into a9088f35bf9317e60d201d2db046087389608ce1 on master.


Add COG conversion endpoint

Resolve #198
Adds two new endpoints:

api/geodata/imagery/image_entry/convert/<int:pk>: create a new converted image in COG format
api/geodata/imagery/converted_image_files/<int:pk>: Check status of converted image entry

To do

[x] fix gdal_translate in PATH on CircleCI
[ ] change test to validate that a COG is generated

https://pypi.org/project/tifftools/
https://github.com/OSGeo/gdal/blob/master/gdal/swig/python/samples/validate_cloud_optimized_geotiff.py

[x] change views to class-based views from DRF

The test is failing because gdal_translate is not in the current path in the tox test env when launched on CircleCI. I ssh'd into CircleCI and I can see gdal_translate under ~/project/.tox/test/bin/, but that is not in the tox environment's path for some reason...

I think the reason gdal_translate is failing is not because it can't be found, but because it is a small wrapper that redirects to the actual location of gdal_translate (deep in the GDAL python package), and therefore needs to be invoked with a shell (e.g., subprocess.check_call(cmd, shell=True).

Ah okay, I will try that.
FYI, it works when testing locally for me (within docker-compose), just not on CircleCI

Same issue ^. When ssh'ing into CircleCI, I notice that the bin dir is not in the path which is what leads me to https://github.com/ResonantGeoData/ResonantGeoData/pull/209#issuecomment-736846881
circleci@d8ac92da9f01:~$ cd project/.tox/
circleci@d8ac92da9f01:~/project/.tox$ ls test/bin/gdal_*
test/bin/gdal_auth.py  test/bin/gdal_edit.py        test/bin/gdal_pansharpen.py  test/bin/gdal_retile.py
test/bin/gdal_calc.py  test/bin/gdal_fillnodata.py  test/bin/gdal_polygonize.py  test/bin/gdal_sieve.py
test/bin/gdal_contour  test/bin/gdal_grid           test/bin/gdal_proximity.py   test/bin/gdal_translate
test/bin/gdal_create   test/bin/gdal_merge.py       test/bin/gdal_rasterize      test/bin/gdal_viewshed
circleci@d8ac92da9f01:~/project/.tox$ which gdal_translate
circleci@d8ac92da9f01:~/project/.tox$ test/bin/python
Python 3.8.6 (default, Nov 18 2020, 13:49:49)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys, subprocess
>>> subprocess.check_call('echo $PATH', shell=True)
/home/circleci/.local/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
0
>>> exit()
circleci@d8ac92da9f01:~/project/.tox$

So perhaps subprocess.check_call(..., shell=True) is executing a different shell environment than the one activated by tox where the ~/project/.tox/test/bin/ dir is supposed to be added to the path?

I switched from using a subprocess to using gdal's Python bindings for Translate and all is working

I am not going to add a test to verify that the produced COG is valid. IMO we don't need to test that as it is upstream functionality. We are passing the correct options to gdal_translate and we should accept what it returns, so the test currently just verifies that something is returned.

Going to squash and merge. More endpoints/views are coming in a follow up PR for #201  and we can use that as a place to modify any of this work if needed


Peer-to-peer-code-review

▶ You made a beautiful website.
▶  Your code is clean.
Constructive comments
▶ Kindly work on your tablet view to fit the screen. Because your website needs to be compatible with tablet users as well.
▶ For the laptop view, kindly work on your social icons. Because your page would be better if you adjust the height and width of the icons.

Thanks, guys! Will do...


Fix typo in publishing-with-visual-studio.md

Add an "a" to the following sentence:

... where the published application runs on machine that has the .NET runtime installed.

So it reads like this:

... where the published application runs on a machine that has the .NET runtime installed.

Merry Christmas and happy New Year!

Internal previews

📄 File
🔗 Preview link

docs/core/tutorials/publishing-with-visual-studio.md
Tutorial: Publish a .NET console application using Visual Studio

@tdykstra Can you review this PR? When this content is ready to merge, you must add '#sign-off' in a comment or the approval may get overlooked.

@tdykstra Can you review this PR? When this content is ready to merge, you must add '#sign-off' in a comment or the approval may get overlooked.


End Smelter not working

When i try to open the GUI for the end smelter nothing happens. its like im clicking on an interactive block but no GUI opens. The regular end furnace works fine but i need to use the smelter

Tested in latest version - works fine. Please provide your logs and other information


Can't use in a monorepo where each project has a different tsconfig.json

ts-jest: 22.4.6
Structure of project
<rootDir>/
|- packages/
   |- package1/
      |- tsconfig.json
      |- src/
      |- tests/
         |- tsconfig.json
   |- package2/
      |- tsconfig.json
      |- src/
      |- tests/
         |- tsconfig.json

Issue
In a monorepo where each project has a tsconfig.json I cannot run jest as it cannot find a tsconfig.json

Expected behavior
ts-jest should use the tsconfig.json file closest to the file being tested that is applicable for the file (the same way tsc and tslint work).

Do you use jest projects here, or are they simply in subfolders?

Have a look here.
I have posted my working solution there
https://github.com/facebook/jest/issues/3112


Prove multiple epoch keys

It would be useful to be able to prove information about multiple epoch keys from multiple epochs in a single proof.
For example, if I want to claim some reputation from a previous epoch (where I submitted an epoch key), I need to submit two proofs. One showing control of the old epoch key, one showing the new epoch key.
Having a single proof to show control of keys in different epochs solves this. One major problem with doing this, especially with reputation proofs, is the number of public signals. In a proof showing control of a past epoch key, and reputation in the current epoch, we have the following public signals:
oldEpochKey
oldEpoch
proveOldNonce
oldNonce

epoch
epochKey
proveGraffiti
minRep
proveNonce
nonce

We can combine a lot of these into a single field using #181

Hey team! Please add your planning poker estimate with Zenhub @AnthonyMadia @ChialiTsai @kittybest @vivianjeng @yuriko627

Hey team! Please add your planning poker estimate with Zenhub @AnthonyMadia @kittybest @vimwitch @vivianjeng @yuriko627


Release bitmap resources immediately

Release  bitmap memory immediately to prevent out of memory exception

Ignore  Jun 16 commits, it for my personal experiments. But clearBitmap() function change is significant, only this change allowed use this component without OutOfMemmory exception.

Are there any works going this?


Theme: Green on black in dark mode-- "matrix"

Qt-calculator-master_v1.zip

Sorry, still trying to learn github, what can I do to fix this?

https://github.com/Arsenic-ATG/Qt-calculator/pull/15


Dependency issues (pint and numpy)

The newest pint version (v0.20) broke its API my moving unit functions to a different location. Therefore, the particular import fails on:
RIAssigner/data/Data.py:5: in <module>
    from pint.unit import build_unit_class
ModuleNotFoundError: No module named 'pint.unit'

https://github.com/RECETOX/RIAssigner/blob/85bacb3678caaa4978339270c91bafe3fff4f8ba/RIAssigner/data/Data.py#L5
The easiest way to resolve this would be by pinpointing the version of pint to v0.19.2.
Another possibility is to update the import path for build_unit_class, but using the v0.20+ version of pint also allows to use newer version of numpy (v1.24.1 in particular), which causes another import issue not present when using v1.23.5:
  File "/usr/local/lib/python3.8/site-packages/RIAssigner/data/MatchMSData.py", line 3, in <module>
    from matchms import Spectrum
  File "/usr/local/lib/python3.8/site-packages/matchms/__init__.py", line 1, in <module>
    from . import exporting, filtering, importing, networking, plotting, similarity
  File "/usr/local/lib/python3.8/site-packages/matchms/importing/__init__.py", line 23, in <module>
    from .load_scores import scores_from_json, scores_from_pickle
  File "/usr/local/lib/python3.8/site-packages/matchms/importing/load_scores.py", line 2, in <module>
    from matchms.Scores import ScoresBuilder
  File "/usr/local/lib/python3.8/site-packages/matchms/Scores.py", line 7, in <module>
    from sparsestack import StackedSparseArray
  File "/usr/local/lib/python3.8/site-packages/sparsestack/__init__.py", line 2, in <module>
    from .StackedSparseArray import StackedSparseArray
  File "/usr/local/lib/python3.8/site-packages/sparsestack/StackedSparseArray.py", line 5, in <module>
    from .utils import join_arrays
  File "/usr/local/lib/python3.8/site-packages/sparsestack/utils.py", line 1, in <module>
    import numba
  File "/usr/local/lib/python3.8/site-packages/numba/__init__.py", line 43, in <module>
    from numba.np.ufunc import (vectorize, guvectorize, threading_layer,
  File "/usr/local/lib/python3.8/site-packages/numba/np/ufunc/__init__.py", line 3, in <module>
    from numba.np.ufunc.decorators import Vectorize, GUVectorize, vectorize, guvectorize
  File "/usr/local/lib/python3.8/site-packages/numba/np/ufunc/decorators.py", line 3, in <module>
    from numba.np.ufunc import _internal
SystemError: initialization of _internal failed without raising an exception

To be more precise, the bioconda package pinned pint dependency here, but we also need to update dev conda env config here in the repo.


Asynchronous http requests using tornado library

Signed-off-by: Nick Palombo<EMAIL_ADDRESS>

Outdated. The head branch has moved quite a lot since this PR. Closing.


Remove resource limits and requests in local development scenario

For local development (minikube) it's helpful to remove all resource limits and requests, so we can pack more components on the same machine

What do you think @mvladev about #255? Can't we close this one/won't it be resolved if we take your thought into account when doing #255, which we want to do anyways?


cleanup event handler addition/removal to not bork on source change

The current way the event handlers are registered / unregistered for text and audio track changes causes problems when switching between SourceHandlers. This change seems to prevent it, at least from my testing with contrib-dash and contrib-hls.
There should probably be some automated tests here checking to make sure that's actually the case.

I have tried this change locally myself it seems to fix the issue mentioned. The tracks are now getting removed when the plugin is disposed. Thanks 👍


fatal: refs/remotes/svn/trunk: not a valid SHA1

I am trying to convert the Splix repository https://svn.code.sf.net/p/splix/code/ and I get this:
$ svn2git --verbose --nobranches file:///...I.made.a.mirror.for.speed.../splix/svnmirror/
...
r258 = 73c93d9d2e85d34f6865402c5da399207ab17fc3 (refs/remotes/svn/tags/splix-2.0.0@259)
        M       ChangeLog
        M       THANKS
        M       rules.mk
r259 = a3dd33a9c671f37b9621e2bd913f0b084fe00716 (refs/remotes/svn/tags/splix-2.0.0@259)
Found branch parent: (refs/remotes/svn/tags/splix-2.0.0) a3dd33a9c671f37b9621e2bd913f0b084fe00716
Following parent with do_switch
Successfully followed parent
r260 = 53da06a13c87f5a880b7b255cd1ed8dc051bcae8 (refs/remotes/svn/tags/splix-2.0.0)
        M       THANKS
        A       ppd/scx4500.ppd
        A       ppd/scx4500fr.ppd
        A       ppd/scx4500pt.ppd
r262 = cdcd43d3ec479ce81bd3b7fef890b905a12ef25e (refs/remotes/svn/tags/splix-2.0.0)
        M       ppd/samsung.drv
        M       ppd/scx4200.ppd
        M       ppd/scx4200fr.ppd
        M       ppd/scx4200pt.ppd
        M       ppd/scx4500.ppd
        M       ppd/scx4500fr.ppd
        M       ppd/scx4500pt.ppd
r263 = 142c9556f9ed9f3aca2d217c3c8f09aafcd517d5 (refs/remotes/svn/tags/splix-2.0.0)
fatal: refs/remotes/svn/trunk: not a valid SHA1
update-ref HEAD refs/remotes/svn/trunk: command returned error: 128

command failed:
git svn fetch

This is a somewhat annoying repo since it doesn't have a trunk directory for the trunk.
Also the author apparently switched branches by deleting and copying around files.
svn2git isn't the only command that does weird thing with this repo. git svn clone --tags=tags --ignore-paths=branches file:///.../splix/svnmirror/ gitconversion quietly drops all commits after r263, for example.

im not sure how this fixed it for me, but i tried creating a new folder and playing with the --revision START_REV parameter to either start at the revision before or a few before the one that is failing and for some reason it worked. Then I was able to start from revision 0 (is that the first revision in svn?) and it finished without issue

im not sure how this fixed it for me, but i tried creating a new folder and playing with the --revision START_REV parameter to either start at the revision before or a few before the one that is failing and for some reason it worked. Then I was able to start from revision 0 (is that the first revision in svn?) and it finished without issue


Improve graphics and tiles

Maybe add themes?
We can have a minigrid theme, but also a different one.

Fixed by #53


Add Tests [Unit Tests and Instrumentation Tests]

Add tests to the app and Daraja Module

Linking this issue with a new PR that was just opened. I think it makes sense to also close this issue with the PR.
The major reason is the app is adopting a new architecture that will mean the sample app will be architecture pattern free, and not use any DI setup giving the user the ability to choose how best to use. We can maybe add samples snippets on how it can be used with different DI tools


Can we use both method?

Is there any way to use both method GET/POST at same time?

Hi @hellofresh01,
There is no way to use both methods at the same time yet.


fix(internal): fix clear script from the main package.json

Motivation
Currently the rimraf portion of the clear script from the main package.json is not working correctly.

More information: https://www.npmjs.com/package/@lerna/exec
The forwarding dashes (--) has been removed due to warning appearing on run:

Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
Successful run of clear command from main package.json file:

Related PRs
No.

thanks :)


Questions on plugin / extension system

On the wiki, there is a slashed "Editor Plugin"
Does it means the feature exist now?
How does the plugin system work?
The Plugins you planned exist as c++ codes and need compile? Or as gdscript executed at runtime(like Unity)?
The Godot editor is distributed in a single exe file (win64 for example). Without exposing the inner file structure, where to put those plugin files?
I would like to create an extension, to export objects on stage to a file,maybe xml, for use in other game engine. (Godot will only be a scene editor  in this workflow). Can I do that？ How can I do that?

http://www.godotengine.org/wp/community/
There are communities for asking something, facebook, forum, IRC.

there is a plugin/extension system but it needs cleanup and documentation.
This work will happen in Godot 1.2 (1.1 should come out soon)
On Wed, Apr 29, 2015 at 5:48 AM, volzhs<EMAIL_ADDRESS>wrote:

http://www.godotengine.org/wp/community/
There are communities for asking something, facebook, forum, IRC.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1763#issuecomment-97354970.

I've used a lot the current plugin system, and i created a tool with it.
And i only did it because i know how read a C++ code, and i learned how the system works, and how is hard to use it, because it miss cannot access several editor related functions.

yeah, it's not on purpose that you can't access them, but more like I have
no idea what kind of functions are needed.
the idea for 1.2 is to try working on figuring out which kind of APIs are
required for plugin developers
On Wed, Apr 29, 2015 at 12:17 PM, Franklin Sobrinho <
<EMAIL_ADDRESS>wrote:

I've used a lot the current plugin system, and i created a tool
https://github.com/TheHX/add_primitives with it.
And i only did it because i know how read a C++ code, and i learned how
the system works, and how is hard to use it, because it miss cannot access
several editor related functions.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1763#issuecomment-97463632.

I hope help with this matter in the future.


GPIO refactor (Simplify macro, reduce type states, and support interrupt)

Simplify macro
gpio.rs uses more generics than macro now
Collapse Input type arguments
An attempt to address #125
Support interrupt
Close #83 and #112

Super interesting attempt. I'll take a closer look in the next few days. But this approach seems promising!

Could you please add support for GPIO interrupts? I'm actually modding this now to do so. I made a standalone module a while back for this (In my PR), but am switching it to GPIO to be consistent with L4 and F4 HALs.

Could you please add support for GPIO interrupts? I'm actually modding this now to do so. I made a standalone module a while back for this (In my PR), but am switching it to GPIO to be consistent with L4 and F4 HALs.

GPIO interrupts support added.
Referred to f1xx-hal, f4xx-hal, and #200.

Clippy warnings should be fixed by #202

I'll take a closer look in the next few days. But this approach seems promising!

Next few months rather. I have this on my TODO list. I'm hopeful that I find time this week :)

Updated (add comments and move IntoAf.. traits to marker module).
This is ready to merge :)

Nice. Let's merge it :steam_locomotive:
Thanks for your hard work, again! :)


React Redux group project

Hi 👋,
In this pull request, We implemented the following requirements:

Create React App.
Install React Redux, Redux Logger and React Router.
Download the free image for the app logo.
Create routes and view components (Rockets, Missions, My Profile, Dragons [only if your team has 3 members]). Use  for the page navigation links and style active class to indicate which section/page user is currently on (underline active navigation link).
Create directories for all Redux state slice files (rockets, missions, dragons [only if your team has 3 members]).

@thecodechaser thanks!!


100+ non-colliding objects

Hello!
Is there a way I could create more than 100 objects which won't collide with each other, but only with the surrounding environment? Something like I've done here but using matter.js?
As far as I read the docs, you can only have up to 32 collision groups.
Thank you!

I think you can use let group = Body.nextGroup(true) to get a non-colliding group, then set body.collisionFilter.group = group of your 100 bodies to that same group?

Awesome! it worked! All the bodies which are in that group aren't colliding.
Out of curiosity, could you explain why it's still colliding with the walls? (it's the expected behavior, but I was wondering how the system works) 😄

@florinpop17 you can find the information in the documentation here http://brm.io/matter-js/docs/classes/Body.html#property_collisionFilter
The properties you're looking for are body.collisionFilter, body.collisionFilter.category, body.collisionFilter.group, and body.collisionFilter.mask
In short (if I understand and explain correctly):

Category is a number. If two bodies have the same positive category number, they always collide. If they have the same negative category number, they never collide (these are your 100 objects). Otherwise we check the group and mask (your walls have no group, so your walls fall under this 'otherwise' situation. See next bullet point).

The group and mask are bit sets. Each body belongs to one single collision group (one bit in the group bit set), and collides against any other body in a group that collides with the mask (any number of bits in the mask bit set). The default setting for the group and mask of every body is to collide with everything (this is why your walls collide with the 100 objects).

Yeah I think that's pretty much it, thanks @AndrewBrownK. To be fair, I'm not a huge fan of the approach and there's a discussion around making it better in #341.


24 Capabilities to Drive Improvement - typo

In header
] -> ) (link format broken)
In Have a lightweight change approval processes
processes -> process or a
In Check system health proactively
preemptively -> pre-emptively

Fixed in #28


Kernel dies when running from jupyter notebook?

After installing babel fish, when I run the library, I get: The kernel appears to have died. It will restart automatically. Is this normal? However, when running on the terminal I get a dictionary with the UAST tags, how can I run this client it in python's jupyter notebooks?

Hey @alonsopg - yes, one should be able to use it from Jupyter.
If you could you please also post the python/jupyter/python-client versions that you are having an issue with?

I reinstalled everything and now works properly, thanks!


Toggle for open on startup?

Is there a way to toggle open-on startup ? I didn't select when initially installing the app but now seems like there is no way to set it.

I think you might consider Menutools as abandonware - don't know if this might help you though #18.

Just reinstall it... it should work.

@navossoc, good to know that the project is still alive. :)
Unfortunately, in my case, reinstalling it didn't help. It won't start on windows startup.

make a new task in task manager and drop menutools depository in there, and there u go


How can I use raw PCM data directly as input?

I have an audio monitor hardware and I can get raw PCM data from it.
How can I use the data as fft plot input?
I'm checking the run() method in SamplingLoop.java file, is it the correct place that I can modify to use raw PCM data as input?

Yeah, that is the right place.
Btw is that hardware run in an Android system?

Yeah, that is the right place.
Btw is that hardware run in an Android system?

Thank you very much, I got it and your code works very fine!
No, our hardware is just a single integrated circuit board and it runs c program to collect audio signal, then encode them into PCM data and send to Android phone via UDP socket

Hi fhaefele, may be try look up the menu (settings->audio source), and try different source. If nothing worked, then the code might need some change.


BraveTracing should support custom names for the service name

Feature Request
Is your feature request related to a problem? Please describe
It is likely for multiple Redis installations to coexist in a production environment.
Currently the service name for spans created by the Lettuce is hardcoded to "redis" which means that every service (using Lettuce client) will report spans with the service name "redis", regardless if they are using different Redis installations.
Describe the solution you'd like
There should be a builder for a BraveTracing that would allow users to provide a custom name to use for the service name for spans.
Describe alternatives you've considered
Didn't consider any alternatives.
Teachability, Documentation, Adoption, Migration Strategy
Currently a BraveTracing is created like this:
BraveTracing.create(tracing)
This could be changed to something like this (no name override):
BraveTracing.builder(tracing).build()
Or like this (with name override)
BraveTracing.builder(tracing).serviceName("customNameHere").build()

Draft implementation here: https://github.com/lettuce-io/lettuce-core/pull/866 if it's easier to explain and discuss the proposed changes.

Awesome improvement. Thanks a lot for submitting a pull request. Tests are sort of flaky on Travis CI so don't get distracted by that.

I enhanced your PR with customization hooks for Span and Endpoint and applied a round of polishing. That's now on master.