Output directory can be unexpectedly deleted

Describe the bug
In reconftw.cfg, if dir_output is set to the same directory as where the script is running, the output directory will be deleted as a consequence of https://github.com/six2dez/reconftw/blob/main/reconftw.sh#L1741.
Expected behavior
Since output directory and working directory are the same, do nothing.
Additional context
I believe it could be fixed by adding a simple check like:
[[ "$dir" != "$dir_output" ]] && rm -rf "$dir"

Fixed on dev https://github.com/six2dez/reconftw/commit/315bde87eceacafeedbf3c6cc621995d7cc8a3eb


Add a new status code "SERVER_INACTIVE" to distinguish with "SERVER_UNAVAILABLE"

Note: It's not a secure practice to let clients know the exact server status (turms-gateway is inactive or the turms servers behind are unavailable). So we keep the current behavior.


ENH: Allow dict as the argument to Index.map (GH13517)

[X] closes #13517
[X] tests added / passed
[X] passes git diff upstream/master | flake8 --diff
[X] whatsnew entry

[Current coverage][cc-pull] is 84.33%

Merging [#13518][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master     #13518   diff @@
==========================================
  Files           138        138
  Lines         51107      51109     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          43103      43105     +2
  Misses         8004       8004
  Partials          0          0

Powered by Codecov. Last updated by [bf4786a...2a67719][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/bf4786a15f51d14ccf1318e44ae080868d8595ee...2a677196169f706f78c60fc14de0c1d76f4b40eb
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13518?src=pr

I would ideally like to merge the Series.map and Index.map impl and move it to Base.map like most other methods. You can move tests to tests/test_base/py:TestIndexOps. This will make these more consistent.

can you rebase / update?

closing, but pls reopen / comment if you can continue


add get_lexer_by_doctype

(Original issue 128 created by None on 2006-10-27T14:25:07+00:00)
I started playing with this yesterday and I like it very much.
it occur to me why not add this?
it will be great for all those "Template Language" lexers
what do you think?
Reported by Jorge<EMAIL_ADDRESS>

(Original issue was assigned to gbrandl)
(Original issue 128 last updated on 2006-11-13T21:11:59+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by None on 2006-10-31T15:13:03+00:00)
@gbrandl
by doctype I was referring either to the HTML doctype or a namespace that the engine uses, it depends on the templating engine but it will be (from the lexer's perspective) the same since it's a way to identify the lexer for mix of html+extension.
@blackbird
actually this was step one of my idea, if you look at the current "Filename patterns"
and get_lexer_for_filename you will see that those apply only to file extensions.
so my idea as a whole was as you suggested make a 3th method that will wrap both and will evertually be the guess_lexer you suggested.
ones this was implemented

(Original comment by gbrandl on 2006-10-29T06:44:40+00:00)
Okay, closing this. You can use the guessing mechanism for a similar effect.

(Original comment by None on 2006-10-27T14:38:15+00:00)
I would vote for "wontfix" because it would only affect xml based template languages. But i would like to have something like guess_lexer so that it automatically recognizes lexers. Each lexer provides a analyse_text method that returns an float between -1 and 1. The lexer with the highest return value wins the battle and lexes the content. If the highest value is -1 then it would use the plain text lexer.

(Original comment by gbrandl on 2006-10-27T14:35:14+00:00)
what exactly do you mean by "doctype"?

(Original comment by None on 2006-10-27T14:25:41+00:00)
sorry fixing metadata


More than one perfusion

Describe the bug
Subject id 661278 has 2 perfusions from LAS database. This is not possible as a perfusion can only be performed once. This might be a big issue this can be happening for other subjects. Find out how fake perfusions are being tracked in LAS and update procedures mapping in labtracks endpoint. Also look into whether there are other subject ids with multiple perfusions
To Reproduce
Steps to reproduce the behavior:

search subject id 661278 with aind-metadata-service
See 2 perfusions

Expected behavior
Only one perfusion
Additional context
Add any other context about the problem here.

from the LAS email it sounds like there is a field in LabTracks for canceled procedures that we need to be looking at


Cannot read property 'getAttribute' of undefined

I've tried using both the jquery.card.js and the card.js files.  I've tried to include them at the bottom of the page.  I am trying to use this in a modal so I am wondering if this is the issue.  Here is my modal.

×

Add Contract

Credit Card Number:

Expiration:

Name:

CVV:

Cancel
Add Contract

and here is my script at the bottom of my php file.

As mentioned in the title I get the Cannot read property 'getAttribute' of undefined in the console.  Am I missing something?

Hello, so I've done some more testing and this is what I've come up with.  When I use form it works fine but it doesn't seem to let me use names/ids/classes to choose the form.  Only when I remove the name/id/class does it work.  Is this something that can work or no?

@SimplySynced
I'm actually getting a similar error, and I'm hoping mine is simply because I wasn't using the CVC... however, I'm not setting the form selector, so not sure what our difference is to get the same error.
Thanks!

@SimplySynced I believe my issue is related to using *ngIf and my elements don't exist yet...

I'm getting the same error. Also an Angular (4.x.x) application.

@BallisticPain i have the same issue, if my element doesn't exist, it throws this error. Do we have the solution for this yet?

Any solution?

you need to have all four fields on the form, if you wana skip any of them then you need to hide it. If you use *ngIf it wont render and it will complain.
in order to hide it you can use [hidden]="anyExpression"

Thanks @muneer-memon

You need to add ID to the form
new Card({ form: 'form', ----> new Card({ form: '#formID',

guys using Angular 5 just make sure to new up Card function in the ngAfterViewInit() lifecycle hook:
  ngAfterViewInit() {
    const card = new Card({
      form: '.card-form',
      container: '.card-wrapper',
    });
  }

@stefmabo @ziemerz

Same issue with Vue. Card() is invoked inside mounted() but it seems it's too soon.

I had the same issue just now. I noticed I was loading the card before the DOM has loaded 🤣

@jcjollant Do you know what do I need to do to make it work with Vue or Nuxt?
Im getting an error, TypeError: Cannot read property 'getAttribute' of undefined
This is the code I have:
<template>
  <form>
    <input type="text" name="number" />
    <input type="text" name="first-name" />
    <input type="text" name="last-name" />
    <input type="text" name="expiry" />
    <input type="text" name="cvc" />
  </form>
</template>

guys using Angular 5 just make sure to new up Card function in the ngAfterViewInit() lifecycle hook:
  ngAfterViewInit() {
    const card = new Card({
      form: '.card-form',
      container: '.card-wrapper',
    });
  }

@stefmabo @ziemerz

You can also use ngOnChanges to initialize the card object.

Same issue with Vue. Card() is invoked inside mounted() but it seems it's too soon.

I have the same problem, any idea how to fix it?

@DerekCrosson We have our init call in created and within a Vue.nextTick() call.

was anyone able to fix this?????

Check this in your html

Check this in your html

Yeah no, it is, but it needs to be inside it's own form for some reason in a woocommerce payment method.
I was able to fix it by creating a sub-form and then copying the element to hidden inputs inside the main woo form 🤷♂

@hounw can you put in the gist and post here the code you used in WooCommerce? I'm having trouble implementing it on WooCommerce too.

@marcos-alexandre82 I was wrong I must've been something else that I ended up fixing, it CAN be inside a simple div.
the standard code worked for me, what issue are you facing?

@hounw are you using in a payment plugin for WooCommece?
I'm trying to implement it in my plugin, but it doesn't work properly. It only displays the skins of the Hipercard and Troy cards. If I try to use another card it remains gray.
For him to display the image of the card, I had to wrap the JS code with the jQuery function of WooCommerce, because otherwise, when the page is finished loading, the card disappears.
Can you help me?
These are the JS codes:
card.js
woo-rede.js
And this is the PHP code for the form:
integration-rede-for-woocommerce.php
Tks

@marcos-alexandre82 that's weird... by instinct I'm pretty sure it's some other issue but here's how I ended up doing it... we only have mc/visa/amex so I don't know about all others but those three seem to work
https://gist.github.com/hounw/2083e9b3e252222ed55987c9d20f3579

@hounw Thank you! I will analyze your code and try to implement it in mine ;)

There are is now wrappers or documentation for Vue, Angular, React etc so closing this ticket

@jcjollant Do you know what do I need to do to make it work with Vue or Nuxt?
Im getting an error, TypeError: Cannot read property 'getAttribute' of undefined
This is the code I have:
<template>
  <form>
    <input type="text" name="number" />
    <input type="text" name="first-name" />
    <input type="text" name="last-name" />
    <input type="text" name="expiry" />
    <input type="text" name="cvc" />
  </form>
</template>

me too, did you solve?

@jcjollant Do you know what do I need to do to make it work with Vue or Nuxt?
Im getting an error, TypeError: Cannot read property 'getAttribute' of undefined
This is the code I have:
<template>
  <form>
    <input type="text" name="number" />
    <input type="text" name="first-name" />
    <input type="text" name="last-name" />
    <input type="text" name="expiry" />
    <input type="text" name="cvc" />
  </form>
</template>

me too, did you solve?

There are is now wrappers or documentation for Vue, Angular, React etc so closing this ticket

Not everyone uses these frameworks. Seems like a valid issue.


Request: PCB sides in png/jpg format

Cloud someone upload both sides of the pcb in png/jpg format?
Thx in advance

or a better format such as svg, step, kicad, eagle....


Exclude *

Hey,
I think it would be very helpful if there was a way to exclude everything.
I have the following problem:
I have a potential infinitely nested resource (not really, but I don't know the levels (yet)).
Pages with includes:

meta
tags
pages (subpages) with includes

meta
tags
pages (...)

Without default includes I would have to send an api request with includes=pages.meta, pages.tags, pages.pages, pages.pages.meta, pages.pages.tags, pages.pages.pages, ... this is pretty crazy.
However when displaying a menu I only want to get pages and maybe pages.pages but certainly no meta or tags. Manually excluding them would mean the same craziness, just in an exclude statement.
So my hope would be either an includes=pages.,pages.pages.,pages.pages.pages.* or a similar exclude.
Is there another good or even better solution that I did not notice?

So my hope would be either an includes=pages.,pages.pages.,pages.pages.pages.* or a similar exclude.

This is a little confusing to me. Can you show a little more clearly what your ideal end solution would look like?

Hey, so basically my problem is the following:

given you have a resource that can be related to others of its type e.g. pages can have subpages, which are itself a resource of the type pages

other resources are related to the resources as well, e.g. pages might have a meta resource relationship, or a relationship to images resources

What I want is to get the main resources pages without relationships apart from pages (subpages), which itself should again have only pages (subpages of the subpages).
I hope this makes sense? 😕
What my idea would be is something like this:
// with defaultIncludes meta, images and pages
api.com/pages/12323442354?exclude=pages.meta,pages.images

// without any defaultIncludes
api.com/pages/12323442354?include=pages

However this should ALSO effect the included pages to only include pages and NOT images and meta.
There might be complications though, when you actually want to include defaultIncludes for subpages but maybe you could do:
// without any defaultIncludes
api.com/pages/12323442354?include=pages,pages.pages,pages.meta,pages.images

and this would than included the desired items for all following subitems? Or there could be a parameters like nestedIncludes=true (which is a very bad name) that would mean that the nested resources of the same type would use the same include rules?
Maybe you have an even better idea?


Feature/51 No more NiFi HTTP edition

Transfer functions are a simple way to create data flow extensions for EDC: implement an HTTP endpoint that handles data transfer according to the documentation and configure the Transfer Functions extension.
The HTTP endpoint can be written in any language and hosted on cloud infrastructure (such as an Azure Function or AWS Lambda) for scale-out. In the future, we can support Keda for on-premise or DIY scale-out.

good to go


CI failures

Currently tests are failing on https://github.com/kairos-io/provider-kairos/. This card is to make them green again and identify all the issues:

[ ] https://github.com/kairos-io/provider-kairos/pull/37

Closing in favor of opening dedicated issues when we spot them. our best bet is on https://github.com/kairos-io/provider-kairos/pull/37


build script for mac fails

the script BuildAllMac.sh fails with the following errors:
xcodebuild: error: The project named "zint" does not contain a scheme named "zint". The "-list" option can be used to find the names of the schemes in the project.
xcodebuild: error: The project named "zxing" does not contain a scheme named "zxing". The "-list" option can be used to find the names of the schemes in the project.
./BuildAllMac.sh: line 7: cd: Src/External/apriltag-2015-03-18/: No such file or directory
./BuildAllMac.sh: line 8: ./BuildLib.sh: No such file or directory
chmod: Src/External/swigwin-3.0.2/swig: No such file or directory
xcodebuild: error: 'Src/Solutions/Xcode/XToolsClient/XToolsClient.xcodeproj' does not exist.

@niklaswolf did you find a resolution to this issue?

No, unfortunately I didn't find a solution :/
Von<EMAIL_ADDRESS>Gesendet: Freitag, 18. August 2017 15:07
An<EMAIL_ADDRESS>Cc: Niklas<EMAIL_ADDRESS><EMAIL_ADDRESS>Betreff: Re: [Microsoft/MixedRealityToolkit] build script for mac fails (#78)
@niklaswolfhttps://github.com/niklaswolf did you find a resolution to this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/MixedRealityToolkit/issues/78#issuecomment-323348653, or mute the threadhttps://github.com/notifications/unsubscribe-auth/APR5n06uHeMQ6jYNHyTtZ4IXAnu4OXBCks5sZYyegaJpZM4MuL-3.

Ok so I managed to build for Mac using an older version of HoloToolkit (December 2016) that still includes the Apriltag source. I had to build zint and zxing separately using the included XCode Projects first.


Fix 'Num Predict Fim' for LMStudio and Oobabooga API

OpenAI-like APIs expect 'max_tokens' parameter instead of 'n_predict'
Current behavior is:

twinny sends:

Request body:
{
  "prompt": "<fim_prefix>\nexample:<fim_suffix><fim_middle>",
  "stream": true,
  "temperature": 0.2,
  "n_predict": 128
}

Oobabooga starts inference with 'max_new_tokens': 16,

Expected behavior is:
2. Oobabooga starts inference with 'max_new_tokens': 128,
I'm not sure if the error applies to LMStudio, but I couldn't find any mentions of 'n_predict' in it's APIs.
P.S. I haven't tested this MR 😉

LGTM, thanks!


Extremely slow PHPMyAdmin

PhpMyAdmin works extremely slow, each query takes ~2 sec.
ISSUE TYPE

Bug Report

Checklist

[x] .env file is attached
[x] docker-compose.override.yml is attached (if it exists)
[x] Custom configs from cfg/ dir are attached (if customized)
[x] docker-compose logs and log/ output is added
[x] I've looked through the docs: https://devilbox.readthedocs.io/en/latest/
[x] I've looked through existing issues: https://github.com/cytopia/devilbox/issues
[x] I've read troubleshooting: https://devilbox.readthedocs.io/en/latest/support/troubleshooting.html

OS / ENVIRONMENT

Host operating system and version: OSX
(Windows) Native Docker or Docker Toolbox:
Docker version: Docker version 19.03.13, build 4484c46d9d
Docker Compose version: docker-compose version 1.27.4, build 40524192
(Linux) Is SELinux enabled?:
What git commit hash are you on?: 4b23dd6d6dad7ade702c3d80d076368f1e962864

SUMMARY

STEPS TO REPRODUCE
I'm using php 7.3 with NGINX & MySQL 8
I've enabled xdebug using
// cfg/php-ini-7.3/xdebug.ini
; ############################################################
; # Devilbox php.ini: Xdebug example
; ############################################################
;
; Information
; -----------
; * Do not edit this file (it belongs to git)
; * This file show a possible Xdebug example configuration
; * If this file is enabled, it will overwrite the current Xdebug settings
;
; How to enable?
; --------------
; * Copy this file to another file with ".ini" extension
; * Only files with ".ini" extensions will be applied by PHP
;
; PHP.ini configuration
;
[PHP]

; Xdebug
; Use these settings to enable Xdebug for PHP
; Make sure to read up on Xdebug some settings might significantly slow down requests.
; The following is just an example configuration and should be adjusted
xdebug.default_enable   = On
xdebug.profiler_enable  = On
xdebug.remote_enable    = On
xdebug.remote_autostart = On
xdebug.remote_connect_back = 0
xdebug.remote_host      = <IP_ADDRESS>
xdebug.remote_handler   = dbgp
xdebug.remote_port      = 9000
xdebug.idekey           = PHPSTORM
xdebug.remote_log       = /var/log/php/xdebug.log

; vim: set ft=dosini:

And added custom config for mysql
//cfg/mysql-8.0/custom.ini
[mysqld]
sql_mode=ONLY_FULL_GROUP_BY

[mysqldump]
;quick

What I've tried:

adding ,cached to the MOUNT_OPTIONS
remove my custom config (it made phpmyadmin a bit faster, but not enough, I guess that xdebug make it a bit slower)

EXPECTED BEHAVIOUR
PhpMyAdmin should work faster
ACTUAL BEHAVIOUR
MYSQL speed is OK when accessing from my app (node & php) it is slow just in PhpMyAdmin
OTHER INFORMATION

Start command

$ docker-compose up bind httpd php mysql redis

File and user permissions (Linux & MacOS)

$ id
uid=501(felixm) gid=20(staff)

$ ls -la

$ ls -la data/www

Thanks for the bug report. This however is not an issue with Devilbox itself, but with the crappy Docker implementation on MacOS(X). Issue was reported on March 2016 and is still unfixed in this very moment: https://forums.docker.com/t/file-access-in-mounted-volumes-extremely-slow-cpu-bound/8076
The above issue has to do with mounted volume performance being really slow on MacOS(X). As phpMyAdmin consists of a huge amount of files, it is generally slow on MacOS(X).
The only solutions/workarounds I can provide (and some you will probably not favour) are listed below:

Setup docker-sync
Use Adminer (single file, way faster than phpMyAdmin)
Switch to Linux (docker runs native there and is damn' fast on it) ;-)
Use a Linux VM and docker inside (should still be faster than OSX)

Thanx for the quick reply.
I'm not sure how docker-sync helps with phpmyadmin

Hi @cytopia  can you explain why phpmyadmin (which is a vendor file inside the PHP docker) is affected by the mac performance issue?
From what I understand, the issue should occur when using mount from host to docker, no?

@felixmosh phpMyAdmin is not inside the PHP container. It resides in the repository and is being mounted here: https://github.com/cytopia/devilbox/blob/master/docker-compose.yml#L174

@felixmosh phpMyAdmin is not inside the PHP container. It resides in the repository and is being mounted here: https://github.com/cytopia/devilbox/blob/master/docker-compose.yml#L174

Wow, thanx for the quick answer 🙏🏼, OK now it make sense.
Another question, Can I "build" some how the images and just run them? without the need to sync / mount?

This is also possible, but requires some steps:

Build PHP images and copy the whole .devilbox/ directory to /var/www/default inside the container during build [1]
Build Nginx/Apache images and copy the whole .devilbox/ directory to /var/www/default inside the container during build [2]
Remove this line from docker-compose.yml https://github.com/cytopia/devilbox/blob/master/docker-compose.yml#L174

[1] https://github.com/devilbox/docker-php-fpm
[2] https://github.com/devilbox/docker-apache-2.2 | https://github.com/devilbox/docker-apache-2.4 | https://github.com/devilbox/docker-nginx-mainline | https://github.com/devilbox/docker-nginx-mainline

Thanx will try it.

I've tried your idea,
What I've done is:

Created a Dockerfile for php (Dockerfile-php)
It looks like:

FROM devilbox/php-fpm:7.3-work-0.112
COPY .devilbox/ /var/www/default

Created a Dockerfile for nginx (Dockerfile-httpd)
It looks like:

FROM devilbox/nginx-stable:0.33
COPY .devilbox/ /var/www/default

Built both
Changed the images in docker-compose.yml
Remove the lines which mounts .devilbox (one for php & one for httpd)
run docker-compose kill && docker-compose rm -f && docker-compose up httpd bind mysql php redis

All starts without any noticeable error.
When I navigate to localhost I see the dev panel.
When I navigate to localhost/vendor/phpmyadmin-5.0 I get the phpMyAdmin but it is still slow 🤷🏽♂


Ansible readme

This PR is intended to perform a couple of different cleanups and additions to the READMEs and docs in this repository post-migration to galaxy including:

[ ] Remove mention of roles
[ ] Rework navigation (Reduce number of READMEs):

Top-level README points to ansible (once), gpfs, openstack (once)
docs/README points to each ansible (is this needed??)
docs/ansible/ has no README
ansible/README points to docs/ansible

The discussed cleanups of requirements.yml and playbooks discussed for this PR are now in https://github.com/manics/infrastructure/pull/5

Travis will need https://github.com/openmicroscopy/infrastructure/pull/247

Merging as an initial cleanup to match the various deleted files. This will need further work soon.


Look into TI metadata cleanup

DevEx/OpEx
Locally and in deployed environments we're creating metadata with every run. This includes Automated Tests that run daily and are getting grouped by the same Ids. We need to look into having some cleanup task that may be scheduled or run manually
Tasks

[ ] Something.

Additional Context
https://flexion.slack.com/archives/C055XTF22B0/p1727717357268009

We may close this story if we can avoid storing metadata for tests that don't need it: https://github.com/CDCgov/trusted-intermediary/issues/1377


Async threads limited across all processes

I'm not actually sure if this is a bug, but it was behavior I didn't really expect so I wanted to check in to see if I just had the wrong initial idea.
I am working with a project that is running two queues. We have a process running for each queue on our server using something like python manage.py process_tasks --queue=first and python manage.py process_tasks --queue=second. We are also running with BACKGROUND_TASK_RUN_ASYNC = True and setting our thread max to 2.
When we do this, we noticed that if the first queue starts executing two tasks, then our second queue doesn't execute anything. I dug into the code a bit and realized that the async thread count is currently across all processes, rather than on a per-process basis. This is because the count is based on the number of total locked tasks in the database.
Is this the expected behavior, or should the thread count be on a per-process basis?

Hi @camuthig
Yeah, as you mentioned "async thread count is currently across all processes".
This is why we extended django-background-tasks with the async routines after adoption the project and only focused on one queue setups.
If you have a possibility to test a "multi queue async setup", feel free to contribute your solution.

Thanks for the response. Our team has moved to running multiple instances of the process_taskscommand for each queue and are avoiding the threads to create a more consistent behavior. If I can find some time, I will try to put together a PR that might support this behavior though.


[UICop] Rule AW0005 should not apply to RoleCenter actions

Describe the bug
The AW0005 rule to check the Image property on groups / actions should not be raised for page/pageExtension objects of PageType 'RoleCenter', since the images for the groups / actions are not shown in the webclient on the rolecenter page.
Expected behavior

Either integrate images in the RoleCenter like it's supported in the webclient ribbon, or in the RTC menusuite.
Either remove the AW0005 rule for objects of PageType 'RoleCenter'

Screenshots

Versions:

AL Langauge: 3.0.115791

The fix for this issue has been checked in to the master branch. It will be available in the bcinsider.azurecr.io/bcsandbox-master Docker image starting from platform build number 32644.
If you don’t have access to these images you need to become part of the Ready2Go program: aka.ms/readytogo
For more details on code branches and docker images please read:
https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/


Update README.md: cocotb[bus] install option is no longer available

Install via cocotb-bus is recommended instead.
-> See https://github.com/cocotb/cocotb/commit/aa6865a1a672f96ef8cf5dc35ee952b0385ab4fe

Thanks for the fix. Due to the dependencies in cocotbext-axi, it actually was still working as expected but a warning was shown.
I have added the change to the CI script.


My startbot.bat it not opening it just closes instantly

@yayeet461 start the bot through the terminal


First tests for $.style

Initial tests for $.style on elements, arrays, and ELEMENT._.style and ARRAY._.style.

And that's 100 unit tests, everyone. Awesome.

good stuff

The :100: wasn't intentional haha!


Add tutorial for LOJ-1051 (Good or Bad)

I want to write the English tutorial for LOJ-1051

@im-nayeem hello dear, you have been assigned.


This is garbage, it can't be used at all

containerCustomStyle is undefind

Amazing contribution... it's open-source therefore you have the power to fix your own problem and contribute back.


Check correct value for case

It looks like this may have just been a copy-paste error. I believe that the ignoreExternals should be checked instead of the depth.

Thanks!


Fix issues found during testing of v0.4.0 with Falcor

The code should now compile cleanly with warnings as errors for VS2015 with W3.
Most of the changes had to do with propagating a real pointer-sized integer type through code that had been using int.

This PR now has a bunch more fixes than just warnings-as-errors, changing title to match.
Basically this is a follow-up to fix all the bugs discovered along the way when testing this work inside Falcor. Mostly it involved fleshing out the lowering pass to be more exhaustive, and then also fixing an annoying bug that I'm surprised didn't come up earlier.


[stable/redis-ha] Init container - refactor

What this PR does / why we need it:

more output on init container to debug/review cluster recovery (which master, which sentinel, time for accessing sentinel)
fixed a bug where ping fails if redis port is not standard 6379
reduce redundancy
more robust - use retry - cluster may not be back in time

Which issue this PR fixes

ping has no port defined - which fails master ping if port is not default 6379

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/mychartname])

@ebgc  We're going to have the same problem with this one when I merge your other one. Lemme know when you're around.


IOS connect VPN errors: A networking error occurred. If this happens again, both please submit feedback.

hello,I met a problem,After input the correct service, choose the ADD service, and then click CONNECT, prompt A networking error occurred. If this happens again, both both please submit feedback.
I implore you to give me analysis this problem, what should I do, to be connected to the VPN.

@GitHubVYChao sorry you've been having problems on iOS.
This will help us debug the issue:

In the Outline app, try to connect.
After trying to connect, click on Submit Feedback.
If you include your Github username then we will be able to look at debug output and continue discussion here.

Thanks!

Closing due to inactivity. @GitHubVYChao if you'd like to discuss more, please re-open this issue.


Bug: link in EntriesOverTime does not work for inheritance_filter

Example:
https://sysndd.dbmr.unibe.ch/Entities/?sort=entity_id&filter=lessOrEqual(entry_date,2021-08-01),any(category,Autosomal recessive)

Problems seems to be that it is filtered for category instead of inheritance term. This should instead be conditional based on the selection.

The same applies for inheritance_multiple


Update card-fancy-example.html

Fetch the Shiba Inu picture over https instead of http.
Mixed content makes Chrome unhappy.

I signed it!

Duplicate of #7800

Duplicate of #7800


Bug fix/fix temp file usage

Scope & Purpose
Prevent accessing the temporary path before a user-specified temporary path could be read via --temp.path.
This PR fixes a regression introduced in devel about 2 weeks ago.

[x] Bug-Fix for devel-branch (i.e. no need for backports?)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)

Testing & Verification
This change is already covered by existing tests, such as scripts/unittest paths_server.
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/9072/

Tests blue


Docs: Packages installed in the layer are different that the ones from aws_lambda_powertools[all]

Summary: Converting discussion into a doc issue so we can prioritize it in this iteration. We answered the discussion on why that is.
Discussed in https://github.com/awslabs/aws-lambda-powertools-python/discussions/1807

Originally posted by ericbn January  4, 2023
Hi! 👋
Below is the list of installed packaged I get with a lambda using the arn:aws:lambda:us-east-1:017000801446:layer:AWSLambdaPowertoolsPythonV2:17 layer, obtained with this code:
import pkg_resources

def lambda_handler(event, context):
    print(*sorted(f"{i.key}=={i.version}" for i in pkg_resources.working_set), sep="\n")

Output:
aws-lambda-powertools==2.5.0
aws-xray-sdk==2.11.0
awslambdaric==2.0.4
boto3==1.20.32
botocore==1.23.32
fastjsonschema==2.16.2
jmespath==0.10.0
pip==22.0.4
pydantic==1.10.2
python-dateutil==2.8.2
s3transfer==0.5.2
setuptools==58.1.0
simplejson==3.17.2
six==1.16.0
typing-extensions==4.4.0
urllib3==1.26.9
wrapt==1.14.1

And these are the packages I get by installing aws_lambda_powertools[all,aws-sdk]==2.5.0:
❯ python3.9 -m venv venv
❯ . venv/bin/activate
❯ pip install 'aws_lambda_powertools[all,aws-sdk]==2.5.0'
❯ pip freeze --local
aws-lambda-powertools==2.5.0
aws-xray-sdk==2.11.0
boto3==1.26.42
botocore==1.29.42
fastjsonschema==2.16.2
jmespath==1.0.1
pydantic==1.10.4
python-dateutil==2.8.2
s3transfer==0.6.0
six==1.16.0
typing_extensions==4.4.0
urllib3==1.26.13
wrapt==1.14.1

Here's the difference (removed pip and setuptools from the left, which is the list from the layer):

Questions:

Shouldn't the package versions all be the same between the layer and the aws_lambda_powertools extra(s)?
Shouldn't awslambdaric and boto3 already be included in aws_lambda_powertools[all] (the "all" extra), or some other extra that is intended to match the packages in the layer?

I'm assuming it's a good practice to add aws_lambda_powertools[all]==2.5.0 to the dev dependencies and then deploy your lambdas only with extra dependencies beyond those already included as dev, because I was assuming those dev dependencies would be the exact same ones already provided in the arn:aws:lambda:us-east-1:017000801446:layer:AWSLambdaPowertoolsPythonV2:17 layer. In other words, I was expecting the diff above to perfectly match. I see it's not the case.
(Side question: Any plans on updating the boto3 version in next releases of the layer and the package?)

Now merged. Attaching screenshots in case anyone else drops by before it's released.


jfrog rt build-publish --dry-run deletes local build information

Describe the bug
When executing a jfrog rt build-publish command with --dry-run flag the gathered local build information is deleted
This leads to
[Error] open ...\AppData\Local\Temp\jfrog\builds\...\partials\details: The system cannot find the file specified. when following up with a real publish after the dry run
To Reproduce

Gather build information: jfrog rt <command> --build-name=bn1 --build-number=5
Check in temp folder that build information has been gathered (%userprofile%/AppData/Local/Temp/jfrog/builds)
Execute dry-run jfrog rt build-publish --dry-run bn1 5
Check temp folder again to see that gathered information is gone
Also execution of build-publish without --dry-run afterwards fails since expected files are not existing anymore

Expected behavior
jfrog rt build-publish --dry-run does not delete gathered local build information
Versions

JFrog CLI version:
jfrog version 1.36.0
JFrog CLI operating system:
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.18362 N/A Build 18362
Artifactory Version: (should be irrelevant)
Artifactory Enterprise
6.18.0 rev 61800900

Hi @7DataBits,
Thank for reporting this issue.
We recently released version 1.38.0 which includes a fix.
We'd appreciate your feedback for it.


Getting ICU error when trying to build with globalization

Description
Ubuntu image is not built with required ICU packages needed for dotnet build:
Process terminated. Couldn't find a valid ICU package installed on the system. Set the configuration flag System.Globalization.Invariant to true if you want to run with no globalization support.
Setting environment variable to:
$export DOTNET_SYSTEM_GLOBALIZATION_INVARIANT=1
is not viable as dev requires globalization.
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Current agent version: '3.236.0'
Current image version: '1.0.3'
This image is using the March Build of the Ubuntu OS.
Is it regression?
NA
Expected behavior
It should be able to build with Globalization using ICU on Ubuntu
Actual behavior
Fails due to missing packages
Repro steps

Run build

Hi @DevOpsAzurance - Thank you for bringing this issue to our attention. We will look into this issue and will update you after investigating.

hi! just sharing that we today started to get builds failing due to missing icu in macos images
could that be related to original issue and any of recent changes in images?

Hi @DevOpsAzurance - After my observation from your issue description, you can try to follow below workflow to resolve the ICU issue on Ubuntu runners for .NET builds which will install the required ICU packages before the build.:-
steps:
  - name: Install ICU packages
    run: sudo apt-get update && sudo apt-get install -y libicu-dev

This might ensures that the .NET build can proceed with full globalization support.

Hi @DevOpsAzurance - Since we haven't heard back, we'll assume your issue is resolved and will close this issue for now. Please feel free to reach out to us for any other queries. Thanks.

Thanks for the workaround @Prabhatkumar59 🙂
I see that the same issue is also present in the ubuntu-24.04 runner image, so I was therefore wondering if we can expect later runner images to include the ICU packages as in 20.04, or should we rely on installing the ICU packages from now on?

Hi @DevOpsAzurance - Since we haven't heard back, we'll assume your issue is resolved and will close this issue for now. Please feel free to reach out to us for any other queries. Thanks.

Can you reopen this issue?
We run our image rebuilds bi-monthly and this was not retested since then.  We are using Azure DevOps so this was added if running DotNetCore builds in a linux OS:
  - task: bash@3
    displayName: 'Install ICU Globalization on Linux'
    condition: eq(variables['Agent.OS'], 'Linux')
    inputs:
      targetType: 'inline'
      script: 'sudo apt-get update && sudo apt-get install -y libicu-dev'

which returns this:
Starting: Install ICU Globalization on Linux
==============================================================================
Task         : Bash
Description  : Run a Bash script on macOS, Linux, or Windows
Version      : 3.246.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/bash
==============================================================================
Generating script.
Script contents:
sudo apt-get update && sudo apt-get install -y libicu-dev
========================== Starting Command Output ===========================
/usr/bin/bash /agent/_work/_temp/08239187-3f97-449d-b051-d7db41128859.sh
Get:1 file:/etc/apt/apt-mirrors.txt Mirrorlist [142 B]
Hit:2 http://azure.archive.ubuntu.com/ubuntu noble InRelease
Hit:3 http://azure.archive.ubuntu.com/ubuntu noble-updates InRelease
Hit:4 http://azure.archive.ubuntu.com/ubuntu noble-backports InRelease
Hit:5 http://azure.archive.ubuntu.com/ubuntu noble-security InRelease
Hit:6 https://packages.microsoft.com/repos/azure-cli noble InRelease
Hit:7 https://packages.microsoft.com/ubuntu/18.04/prod bionic InRelease
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
libicu-dev is already the newest version (74.2-1ubuntu3.1).
0 upgraded, 0 newly installed, 0 to remove and 51 not upgraded.

Finishing: Install ICU Globalization on Linux

but results in this during the build:
  Process terminated. Couldn't find a valid ICU package installed on the system. Set the configuration flag System.Globalization.Invariant to true if you want to run with no globalization support.
     at System.Environment.FailFast(System.String)
     at System.Globalization.GlobalizationMode.GetGlobalizationInvariantMode()
     at System.Globalization.GlobalizationMode..cctor()
     at System.Globalization.CultureData.CreateCultureWithInvariantData()
     at System.Globalization.CultureData.get_Invariant()
     at System.Globalization.CultureInfo..cctor()
     at System.Resources.ManifestBasedResourceGroveler.GetNeutralResourcesLanguage(System.Reflection.Assembly, System.Resources.UltimateResourceFallbackLocation ByRef)
     at System.Resources.ResourceManager.CommonAssemblyInit()
     at System.Resources.ResourceManager..ctor(System.Type)
     at System.SR.get_ResourceManager()
     at System.SR.GetResourceString(System.String, System.String)
     at System.SR.get_XmlConvert_BadUri()
     at System.Xml.XmlReaderSettings.CreateReader(System.String, System.Xml.XmlParserContext)
     at System.Xml.XmlReader.Create(System.String, System.Xml.XmlReaderSettings, System.Xml.XmlParserContext)
     at System.Xml.XmlReader.Create(System.String, System.Xml.XmlReaderSettings)
     at System.Xml.Linq.XDocument.Load(System.String, System.Xml.Linq.LoadOptions)
     at System.Xml.Linq.XDocument.Load(System.String)
     at Vsxmd.Program.Main(System.String[])
  Aborted (core dumped)
##[error]/home/AzDevOps/.nuget/packages/vsxmd/1.4.5/build/Vsxmd.targets(17,5): Error MSB3073: The command ""/home/AzDevOps/.nuget/packages/vsxmd/1.4.5/build//../tools/linux-x64/Vsxmd" "" "" "True"" exited with code 134.
/home/AzDevOps/.nuget/packages/vsxmd/1.4.5/build/Vsxmd.targets(17,5): error MSB3073: The command ""/home/AzDevOps/.nuget/packages/vsxmd/1.4.5/build//../tools/linux-x64/Vsxmd" "" "" "True"" exited with code 134. [/agent/_work/5/s/SourceCodeRepo/src/Services/FNOL_PolicyInfo.Domain/FNOL_PolicyInfo.Domain.csproj]

Build FAILED.

This is the latest Ubuntu build, which is strange as the install ICU step says it is already at the latest and marking it as manually installed. Seems like something is there, but then fails anyways.

Thanks for the workaround @Prabhatkumar59 🙂
I see that the same issue is also present in the ubuntu-24.04 runner image, so I was therefore wondering if we can expect later runner images to include the ICU packages as in 20.04, or should we rely on installing the ICU packages from now on?

I am ok with running that install step, but so far not working for me.

Thanks for the workaround @Prabhatkumar59 🙂
I see that the same issue is also present in the ubuntu-24.04 runner image, so I was therefore wondering if we can expect later runner images to include the ICU packages as in 20.04, or should we rely on installing the ICU packages from now on?

I am ok with running that install step, but so far not working for me.

Hi, may I ask if your issue has been resolved? I am encountering the exact same problem.

Thanks for the workaround @Prabhatkumar59 🙂
I see that the same issue is also present in the ubuntu-24.04 runner image, so I was therefore wondering if we can expect later runner images to include the ICU packages as in 20.04, or should we rely on installing the ICU packages from now on?

I am ok with running that install step, but so far not working for me.

Hi, I am encountering the exact same problem, and I solve it by change:
runs-on: ubuntu-latest
to:
runs-on: ubuntu-20.04


Histogram in single-point is too fine

The histogram is too sensitive to large jumps in numbers in a particular class (because a high-employment block was found in the 18 minute band, say). We should smooth it, because all the jobs in a block aren't concentrated at 18 minutes: some may 17, while some may be 19. Perhaps just bin by five-minute intervals on the client side?

This will be fixed when we merge the complete-pointsets work and the updated cluster.

We bin to five-minute intervals on the client side now.


python3.pkgs.pyelftools: fix aarch64 build

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python3.pkgs.pyelftools

Result of nixpkgs-review pr 118593 at 660f1851 run on aarch64-linux 1

5 packages failed to build:

pwndbg
pwntools (python38Packages.pwntools)
python38Packages.manticore
python39Packages.manticore
python39Packages.pwntools

2 packages built successfully:

python38Packages.pyelftools
python39Packages.pyelftools

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.


Evaluate APIs for live option data

TDAmeritrade API
https://developer.tdameritrade.com/apis
e*Trade API
https://apisb.etrade.com/docs/api/market/api-market-v1.html#/definition/getOptionChains
Options Route
https://optionsroute.com/
tradier brokerage
https://brokerage.tradier.com/pricing
Article Top 4 Stock and Options Market Data APIs in 2021
https://techbum.io/best-free-api-for-stock-market-data/

API

Features

TDAmeritrade

option chain & strategy chain, place orders, price history, quotes, transaction history
e*Trade
option chain, option expire dates
Options Route (commission-free options trading, $0.50/contract)

tradier brokerage

I use TOS so I can start evaluating TDAmeritrade.

Ok, I also use TOS.  I have written a script to access some of TDAmeritrade's api.  Navigating OAUTH2 was the difficult part for me.  I'll post the code to /tdameritrade


Add XI - Freedom Dive (Low Pitch) by Raftk

Song Name
XI - Freedom Dive (Low Pitch)
Artist Name
Raftk
Source
Youtube
Youtube Link
r_Ni3DPwf_c
Direct File Link
No response
Song ID
735623
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 55274070
Downloaded: Yes

accept


Add CSS File to Bower main

Wiredep (and some other automated bower tools) rely on CSS files being included as part of the main configuration in the bower.json file. The bower.json spec also indicates that styles should be included if they should be used in conjunction with the file. So this just adds it to the bower.json file.

Bower support is discontinued, please download via npm instead.


better error message when yang import missing

Currently it looks like this:
panic: file does not exist - line 10, col 2 - line 5, col 2

would be better to at least include the missing file name

solved


Docker image

Description
Create a docker image to use Quetz.
Containerization is a different way to use an application, see #44 .
I also created a repo for Quetz under the TheSnakePit organization on DockerHub.

If it's OK, I will transfer the ownership to QuantStack members
If not, I will delete it

TODOs

[ ] Update the package requirement in Dockerfile to use the CLI
[ ] Transfer the ownership of the TheSnakePit DockerHub org to @wolfv or other QuantStack members
[ ] Add GitHub action(s) for building the image and publishing it. Maybe it implies a pipeline to build the conda package and publishing it on conda-forge if the Dockerfile relies on a conda package installation
Discussion

Base Quetz and Boa images on Mamba image to be released.

Good question, I never used miniforge.
It seems that the miniforge image is based on the Ubuntu bionic image while miniconda3 is based on a Debian one. I guess Debian is lighter.
Does miniforge add anything compared to miniconda?
We should keep a default one for the moment then switch to a mamba or umamba as soon as available don't you think?

looks good. I wonder if we can also add e.g. a github action to somehow automate this process?
One issue is that with this model, we need to wait for the release to be available through conda before we can push the new docker image.
Maybe we should build the docker image from source, and then run a github action whenever a new tag is available and push it to the registry?

We should create a GitHub action yes!
I'm not familiar with this (more used to GitLab) and I haven't the permissions for that on the repo.
We should do that when after having modified the entry point to use the CLI recently introduced
Jupyter images are built exactly the same way using conda packages so we should understand how they are working in there pipeline to wait for the conda package to be available.
Building from sources is also an option. I prefer using the artefact to force consistency.

This PR has been in draft for quite some time. I needed a docker image, so I created a new Dockerfile and made a new PR: https://github.com/mamba-org/quetz/pull/393
My personal opinion is that building from source is not an issue. When you can have the Dockerfile in the source repository, that's actually what you should do.
If you want to use a released version (like the conda package), you should have the Dockerfile in its own repository. But I think it's a bit more work to have a separate repository when you manage the official one.

we have a nice docker file for now from @beenje :)


Headers.prototype.getSetCookie

🔎 Search Terms
headers getSetCookie
🕗 Version & Regression Information
On version 5.1.6, the lib.dom.d.ts has the following interface:
interface Headers {
    /** [MDN Reference](https://developer.mozilla.org/docs/Web/API/Headers/append) */
    append(name: string, value: string): void;
    /** [MDN Reference](https://developer.mozilla.org/docs/Web/API/Headers/delete) */
    delete(name: string): void;
    /** [MDN Reference](https://developer.mozilla.org/docs/Web/API/Headers/get) */
    get(name: string): string | null;
    /** [MDN Reference](https://developer.mozilla.org/docs/Web/API/Headers/has) */
    has(name: string): boolean;
    /** [MDN Reference](https://developer.mozilla.org/docs/Web/API/Headers/set) */
    set(name: string, value: string): void;
    forEach(callbackfn: (value: string, key: string, parent: Headers) => void, thisArg?: any): void;
}

This is missing getSetCookie: https://developer.mozilla.org/en-US/docs/Web/API/Headers/getSetCookie
⏯ Playground Link
https://www.typescriptlang.org/play?#code/MYewdgzgLgBAFgUwIYBMECcIwLwzAgdxgAlk1MAKASgChRJYIEoBhEEAawEsEd4yMEAHQBzZgGVmbTj2pA
💻 Code
const headers = new Headers()
const setCookie = headers.getSetCookie() // <-- Property 'getSetCookie' does not exist on type 'Headers'.(2339)

🙁 Actual behavior
Property is not defined
🙂 Expected behavior
The property should be defined.

@RyanCavanaugh should this be moved to https://github.com/microsoft/TypeScript-DOM-lib-generator ?

I've not heard of that project. Sounds like it 👍

Looks like it's already in https://github.com/microsoft/TypeScript-DOM-lib-generator/blob/9a39f1049cfcceec4d76d3ffc0481bf2d63f1f51/baselines/dom.generated.d.ts#L13448-L13449
but I don't understand the flow of how that one gets merged in to TypeScript.

Actually, it looks this is already in this repo too: https://github.com/microsoft/TypeScript/blob/a0e0104656415b6c6d18c6ba1100ae42157a4f09/src/lib/dom.generated.d.ts#L13428-L13429
So I'm not sure why it's missing in newer versions.

After some detective work, I found out that 5.2.1-rc already has this, if you are using VSCode, make sure you are using the local version:

@kentcdodds Looks like this is fixed. The playground link is showing the current annotation now.

Thank you for confirming!

I was getting the same error but at build time and not in VS code. Fix was to yarn add -D @types/web to the project.


Add config.pyi stub

Adds config.pyi stub.
If this is merged after https://github.com/microsoft/picologging/pull/92 we need to also include this in setup.py setup() function.

Codecov Report

Merging #100 (9f2a9cf) into main (324ff44) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #100   +/-   ##
=======================================
  Coverage   88.93%   88.93%
=======================================
  Files          11       11
  Lines        2069     2069
=======================================
  Hits         1840     1840
  Misses        229      229

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


bump arrow from 6.0.0 to 6.0.1

Feature or Bugfix

Bugfix

Detail

quick PR to use the latest release from Apache Arrow 6.0.x branch

Relates

Release notes: https://arrow.apache.org/release/6.0.1.html

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 5444efeeff419f67c93c0cd76b4acc8574576564
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


MQ EM issue

I tested out the mq library and for some reason it seems to be producing incorrect mqs.
It was unable to calculate breakpoints in ems for smaller sized screens. I tested this on a iphone x and also on virtual devices in google dev tools.

style.scss

@include mq($from: desktop) {
    body{
        background-color:$background1;
        font-family:$fontFam;
}

When the scss was compiled, it resulted in the following scss:

style.css

@media (min-width: 61.25em) {
  body {
    background-color: orange;
    font-family: "Open Sans", sans-serif;
  }
}

I was trying to display orange as the background for screen sizes greater and equal to the desktop size.
Below is a link to a screenshot illustrating the effect.
https://user-images.githubusercontent.com/23007265/62267171-e494f100-b3f0-11e9-8b5d-999546457dbf.png

Thank you for using sass-mq - can you provide a project on GitHub or CodeSandbox (or other platform) so I can try reproducing the bug?

I have shared access to the repo to you.

Thank you! Have you tried adding a viewport meta to the head of your document?
<meta name="viewport" content="width=device-width, initial-scale=1" />

Also, please don't edit breakpoints in the library itself (the documentation explains how to override the mq-breakpoints map the right way).
Closing this, feel free to reopen if this still doesn't work.


🐛 BUG: D1 local databases are not created with database_id.

Which Cloudflare product(s) does this pertain to?
D1, Miniflare
What version(s) of the tool(s) are you using?
3.18.0
What version of Node are you using?
20.7.0
What operating system are you using?
Linux
Describe the Bug
Local databases are no long created with the database_id.  They are being created at .wrangler/state/v3/d1/miniflare_D1DatabaseObject/{hash}.sql.  This makes it very difficult to use with explorer tools. Its also resulting in runtime errors (see below).
[[ env.dev.d1_databases ]]
binding = 'DB_ACCOUNTS'
database_name = 'accounts'
database_id = 'ACCOUNTS'
preview_database_id = 'ACCOUNTS'
migrations_table = 'migrations'
migrations_dir = 'migrations'

 NO_D1_WARNING=true wrangler d1 execute DB_ACCOUNTS --local -c env-dev.toml -e dev --file ./0000.sql

I expect would expect the database to be created in .wrangler/state/v3/d1/ACCOUNTS/db.sqlite
It used to work like this, not sure when this changed or if its intentional, but it makes life much harder.
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
✘ [ERROR] Error: D1_ERROR: no such table: accounts

      at D1Database._sendOrThrow (cloudflare-internal:d1-api:66:19)
      at async D1PreparedStatement.raw (cloudflare-internal:d1-api:178:32)
      ... 5 lines matching cause stack trace ...
      at async jsonError
  (file:///---/.wrangler/tmp/dev-aqgzxk/index.js:55135:12)
  {
    [cause]: Error: no such table: accounts
        at D1Database._sendOrThrow (cloudflare-internal:d1-api:67:24)
        at async D1PreparedStatement.raw (cloudflare-internal:d1-api:178:32)
        at async _b81.get
  (file:///---/.wrangler/tmp/dev-aqgzxk/index.js:26068:18)
        at async handler
  (file:///---/.wrangler/tmp/dev-aqgzxk/index.js:33517:21)
        at async handle
  (file:///---/.wrangler/tmp/dev-aqgzxk/index.js:30041:22)
        at async Object.handle
  (file:///---/.wrangler/tmp/dev-aqgzxk/index.js:30041:22)
        at async Object.fetch
  (file:///---/.wrangler/tmp/dev-aqgzxk/index.js:55027:16)
        at async jsonError
  (file:///---/.wrangler/tmp/dev-aqgzxk/index.js:55135:12)
  }

Having this same issue. New SQLite database is being created and not using the one I ran migrations on

@mrbbot Thanks for pointing me to the changelog.  I need to follow them more closely.  I'm using drizzle-kit and drizzle-studio and making use of the hashing function solved it for me.
Here is the new drizzle config:
import crypto from 'node:crypto'

import type { Config } from 'drizzle-kit'

const uniqueKey = 'miniflare-D1DatabaseObject'

function durableObjectNamespaceIdFromName (name: string) {
  const key = crypto.createHash('sha256').update(uniqueKey).digest()
  const nameHmac = crypto.createHmac('sha256', key).update(name).digest().subarray(0, 16)
  const hmac = crypto.createHmac('sha256', key).update(nameHmac).digest().subarray(0, 16)
  return Buffer.concat([nameHmac, hmac]).toString('hex')
}

export default {
  schema: './src/db/accounts/*',
  out: './drizzle',
  driver: 'better-sqlite',
  dbCredentials: {
    url: `./.wrangler/state/v3/d1/${uniqueKey}/${durableObjectNamespaceIdFromName('ACCOUNTS')}.sqlite`
  }
} satisfies Config

Migrations work as well. I will close this for now, but the changelog said that this might change in the future... It might be nice to expose this hashing function in the miniflare package so you can change how it works under the covers without it breaking our implementations.
Maybe something like:
type StorageClass = 'D1Database' | 'R2Bucket' | 'KVNamespace' | 'Cache'

export function getLocalStore (storageClass: StorageClass, name: string) {
  const uniqueKey = `miniflare-${storageClass}Object`
  const key = crypto.createHash('sha256').update(uniqueKey).digest()
  const nameHmac = crypto.createHmac('sha256', key).update(name).digest().subarray(0, 16)
  const hmac = crypto.createHmac('sha256', key).update(nameHmac).digest().subarray(0, 16)
  const hex = Buffer.concat([nameHmac, hmac]).toString('hex')
  const prefix = getPrefixFromConfig(storageClass) // Get the storage prefix from where ever it lives
  return `${prefix}/${uniqueKey}/${hex}.sqlite`
}

getLocalStore('D1Database', 'ACCOUNTS') // .wrangler/state/v3/d1/miniflare-D1DatabaseObject/{hex}.sqlite


"x509: certificate signed by unknown authority" when bot encounters a funimation url

Someone in my server posted this link https://www.funimation.com/shows/fatestay-night-unlimited-blade-works/ and the bot responded with this message
`Download Failure
Gave up trying to download
https://www.funimation.com/shows/fatestay-night-unlimited-blade-works/
after 3 failed attempts...
Download Failed - Error Downloading URL Response
ERROR: Get "https://www.funimation.com/shows/fatestay-night-unlimited-blade-works/": x509: certificate signed by unknown authority
discord-downloader-go v1.4.2-a — discordgo v0.22.0`
I'm running the bot in a docker container and it only seems to have this issue with funimation links

@nmills3 Can confirm this is a Docker-specific issue. No issues when running on Windows but I encountered the same error in a Docker container.

It would be possible to manually add the missing certificates but that doens't seem like a good long term solution

@nmills3 Not sure what changed but this seems to be fine now in docker containers.


QE core dumped when report "Resume interrupt holdoff count is bad (0)…

… (xact.c:2907)"
The root cause is:
AT AbortTransaction(),  we will disable interrupt processing by calling  "HOLD_INTERRUPTS();" and "RESUME_INTERRUPTS();", these 2 functions will change "InterruptHoldoffCount" to indicate that we are in such a section. When in such a section, we can't report error, because it will set  InterruptHoldoffCount to 0.
On the other side, from the user demand, when transaction abort, we can't report error if temp file cannot be deleted, we need to just report warning.

+1

+1


Revisit some rules regarding react

Allow more than one component per file
Allow stateless components
Allow arrow functions in props
Remove _ prefix in private methods (syntax for private methods will be #, see proposal)

/cc @andregoncalvesdev
We should probably align this with #15 in a major release.

Allow more than one component per file
Allow stateless components
Allow arrow functions in props

These are currently in master.
Regarding Remove _ prefix in private methods (syntax for private methods will be #), this is already on stage 3 but only available on babel v7 beta. We should revisit this when babel v7 is published and close this issue.

@andregoncalvesdev I think the arrow functions in props is still disallowed, see: https://github.com/moxystudio/eslint-config/blob/85144fd1f56cdb4aef728baf1d0951c823edad5e/rules/react.js#L51

Handled in #26


fork() return error checking

Thanks!
This one, and #1282, and much smaller and managable to review and merge :-)


[Feature request] Model hub

An modelhub as huggingface transformers have too.
Basicly the same as huggingface transformers models hub is, just for ludwig.
Filtered by input and output features, so with the time more and more pretrained models are getting built with ludwig and the training process takes a lot less time than before.
The filtering could be done with the defitionion file as must have for the model description.
For the basic tasks like text classification, image classification etc I could train and share the models.

This is a lovely idea and something I've been thinking about for a log while. I would also want to provide pretraines encoders, decoders and combiners.
This has not ahppened so far mostly because of the time it requires, but it is definitely something I want to do.
If someone from the community is interested in helping on this, I would be glad to direct and support them!

Hey, sounds good.
We don't have time to help you with programming but we would upload pretrained models as much as we can.
At the moment we are especially building CV models.
I'd love to support this sinces Ludwig is a great opportunity to give prototyping tasks to my colleagues who don't have any idea about DL ;)
A pretty lazy solution would be an GitHub repo with json file containing links to this models.

I would proably study how Hugging Face does it, as their solution works very well, and try to replicate it (or improve over it) to get this right once and for all, avoiding the super lazy solution, but I'm open to suggestions ;)

@w4nderlust
my first suggestion for a model hub
example usage:
base url with github pages: "https://flozi00.github.io/modelhub/" + folder (models / encoders / decoders / combiners) + username + modelname
https://flozi00.github.io/modelhub/models/flozi00/titanic_model.json
https://github.com/flozi00/modelhub

Thank you, this is a very nice starting point :)
n alternative for the entries:
{
	"type": "model",  // model, encoer, combiner, decoder
	"path": "http://models.a-ware.io/titanic.zip",
	"ludwig_version": "0.3-alpha",
	"dataset_name": "titanic",
	"dataset_path": None,  //optional, for public datasets
	"tags": {...},  //useful for search in the future
	"description": "This model was trained from the ludwig tutorials",
}

I am thinking about replacing the description with an markdown file as huggingface does for the description, comments, code snippets and model definition file in yaml format.
The type is already defined with the folders, is it necessary to do it twice ?
Today I would start with generating an merged list of all files, is it okay when the type get filled there automatically ?
When the list is ready I would start with an simple UI to display the models and making an search mask

I am thinking about replacing the description with an markdown file as huggingface does for the description, comments, code snippets and model definition file in yaml format.

Sounds good, curious to see how it looks like after the change ;)

The type is already defined with the folders, is it necessary to do it twice ?

I wold say yes, i can see how it can seem redundant, but if in the future we want to place a little elasticsearch instance in front of it to do search, having the field may be useful.

Today I would start with generating an merged list of all files, is it okay when the type get filled there automatically ?
When the list is ready I would start with an simple UI to display the models and making an search mask

That sounds great, thank you!

I tried this out and it seems like you can't create an PR using the GitHub Apis this way.
I found examples where users cloned repos or committed to existing but I found no way to create an PR programmatically without automating the browser using selenium or something similar.
What's the biggest size of an zipped model ?
GitHub allows uploading files smaller than 100mb, with Ludwig 0.2 I didn't find an model bigger than this.
I will try the hf_transformers branche later and check the size.
In this case we could use an GitHub repo + GitHub pages as download server, hopefully it is not forbidden for such an use case ^^

I tried this out and it seems like you can't create an PR using the GitHub Apis this way.
I found examples where users cloned repos or committed to existing but I found no way to create an PR programmatically without automating the browser using selenium or something similar.

Got it. It's a shame, but it's fine to do without ;)

What's the biggest size of an zipped model ?
GitHub allows uploading files smaller than 100mb, with Ludwig 0.2 I didn't find an model bigger than this.
I will try the hf_transformers branche later and check the size.
In this case we could use an GitHub repo + GitHub pages as download server, hopefully it is not forbidden for such an use case ^^

I don't think it's forbidden, but 100mb is a pretty small size. It may be fine for now, but probably in the future we want users to either post the models and we manage saving them somewhere and keeping hte links, or they can provide a link to download the zip and we have some code that downloads it. I would prefer the first option, but it looks like it requires infrastructure.

Edit:
Otherwise I would take the username from GitHub oauth, create an seperated folder named by the username and store the uploaded model there.
Then the model is online and only the model card is missing.

That may be fine, but the 100mb limit would still be there.

That may be fine, but the 100mb limit would still be there.

There is no limit in this case.
We take the username by GitHub oauth, create an folder named by this, outside of the repo.
The models get stored on the hosting server then.
It's pretty similar to HF modelhub, anyone can upload models but the model cards needs to be reviewed from the team.
Which way to use ?
Hosted by GitHub pages or custom infrastructure ?

The following might be a suggestion:
https://github.com/flozi00/modelhub/tree/master/models/flozi00
In this version the paths are relative and you can host it on any server, you just have to enter the base url in the client.
Only the UI needs PHP support.
The models can be checked before they are merged, the functionality can be done somehow via the github actions.
Via git lfs (https://docs.github.com/en/github/managing-large-files/configuring-git-large-file-storage) it should be possible to manage files larger than 100 MB, I would have to deal with that.

This looks convincing :) If lfs allow for bigger files, that sounds pretty nice to me.

Okay, then I will check the automated test using Github actions, trying to make the css looking better and then we could start integrating it into ludwig.

import dload

def download_model(model_type, model_name, base_url = "https://models.a-ware.io/", base_path= "/tmp/ludwig/"):
    dload.save_unzip(base_url + model_type + "/" + model_name + ".zip" , base_path + model_type + "/" + model_name )

This library handles all the necceassary steps, the above function can successfully download and unzip the model from model hub.

Looks pretty cool! Only thing I suggest: when creating paths, use os.path.join() as it will respect the current os path separator (the current code won't work on windows, but with my suggestion it will for instance).

So, only the ludwig implementation is missing now.
The models can be stored everywhere (I will upload them to our S3 storage), the URL needs to be given in models json.
Every commit to the repo gets synced to the webserver instantly.
I will try to make the CSS looks better, could you insert the download function to ludwigs code ? I don't know where to do so exactly.
Actually I am training an address validation model for my work, after this I will train on kaggle datasets again.


Rename "is QR + PIN available" API to "get preferred auth method".

After a discussion with the IOs team and OneAuth team we decided to change:

The API to return a more general PreferredAuthMethod enum rather than QRPINAvailability. This is to attempt to be extensible in case some other similar method is created as part of current discussions, such as NFC.
The admin config will reflect this - we'd ask for "preferred_auth_config= QRPIN" rather than "qrPinAvailability = true".

Tested with
Authenticator and MSAL test app in this pipeline
Artifacts: https://identitydivision.visualstudio.com/Engineering/_build/results?buildId=1259288&view=results
test plan: https://identitydivision.visualstudio.com/Engineering/_testPlans/define?planId=2827166&opId=1986&suiteId=2827225


MicroPython + PyO3 + #![no_std] - call for participation

This is an idea born out of a conversation I had earlier today at EuroRust.
Rust has a lot of success in the embedded space. There is also MicroPython to bring Python to these tiny devices (useful for certain applications). This begs the question, can PyO3 support MicroPython, should it, and who would benefit from it?
If you think the answer is that PyO3 should support MicroPython and you'd be interested in making it happen, please get involved! I think before we definitely commit to adding support to PyO3 we'd need to understand how much complexity it would add for us as maintainers, what features we would be able to support, and how we can test it in CI.
Some observations:

It looks like MicroPython has a completely different C API, with unique restrictions. See https://docs.micropython.org/en/latest/develop/natmod.html. If it makes sense, we could write a shim in pyo3-ffi to keep the complexity there, but I haven't looked in detail at the API, maybe it's too different.
It looks like packaging for MicroPython is completely different to PyPI wheel packaging (again see the above link). Probably maturin would need to consider adding some support.
#![no_std] is probably useful for these targets. PyO3 could add a #![no_std] feature regardless of whether we decide to accept MicroPython support. I don't think it would be too bad, just needs someone who wants it to investigate it.

It's great if we can use pyo3 + MicroPython on embedded devices!

any updates?


Pulls

Final CI for a batch of PRs:
#12392
#12420
#12460

Dependency Tree Analyzer Output:
New Dependencies:

io.smallrye:smallrye-health-2.0:jar:1.0.2:compile

Removed Dependencies:

io.smallrye:smallrye-health:jar:1.0.2:compile

Major Version Changes:

org.eclipse.microprofile.health:microprofile-health-api:jar:1.0:compile -> 2.0.1

CC @wildfly/prod


Please put SOMEWHERE easy to find that building requires Gradle 1.5

For new developers, please give them the heads up that building the project requires Gradle 1.5. We could add a "+" to the gradle.build file so that Gradle versions 1.5 or newer can be used, but that might lead to unpredictable and non-reproducible errors because different builds could use different versions of Gradle.

Hm, perhaps I'm missing something. I see we have a requirement:
    dependencies {
        classpath 'com.android.tools.build:gradle:1.5.0'
    }

… but yet I seem to be using Gradle 2.8 just fine:
~/src/zulip-android$ ./gradlew
:help

Welcome to Gradle 2.8.

To run a build, run gradlew <task> ...

To see a list of available tasks, run gradlew tasks

To see a list of command-line options, run gradlew --help

To see more detail about a task, run gradlew help --task <task>

BUILD SUCCESSFUL

Total time: 4.371 secs

I too tried to compile with Gradle 2.8, which was right after I cloned the
repo, and it failed. Then I added Gradle 1.5 so that I have both Gradle 2.8
and Gradle 1.5. Now it compiles just fine. This is weird...
On Sat, Mar 26, 2016 at 4:42 PM, Luke Faraone<EMAIL_ADDRESS>wrote:

Hm, perhaps I'm missing something. I see we have a requirement:
dependencies {
    classpath 'com.android.tools.build:gradle:1.5.0'
}

… but yet I seem to be using Gradle 2.8 just fine:
~/src/zulip-android$ ./gradlew
:help
Welcome to Gradle 2.8.
To run a build, run gradlew  ...
To see a list of available tasks, run gradlew tasks
To see a list of command-line options, run gradlew --help
To see more detail about a task, run gradlew help --task
BUILD SUCCESSFUL
Total time: 4.371 secs
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip-android/issues/31#issuecomment-201927429


Db 5039 - Error reporting from Tablo is incorrect

Note:  This PR is a matching set for one in databasin_web.
Fixed to no longer issue "could not transform points" error when importing csv with duplicate column names. Can add more specific errors as we figure out what those are. Added code in tablo to the append method as well to report errors when the dataset is edited, however could not actually produce any errors at all!
Errors:
Transform points error - Create a csv spreadsheet with bad lat/long values:
River Name,Entry Id,Longitude,Latitude,Team,Temperature,Author,Date,Is Celsius
Columbia 'D' River,4,9999.5555,45.2266,Team A,55,John Doe,01/11/2016,1
Duplicate columns error - Create a csv spreadsheet with a duplicate column:
River Name,Entry Id,Longitude,Latitude,Team,Temperature,Author,Date,Is Celsius,Is Celsius
Columbia 'D' River,4,-122.5555,45.2266,Team A,55,John Doe,01/11/2016,1,1

Coverage decreased (-0.2%) to 50.485% when pulling 8caab603223e0ebc09a2c9e4dc15ab7eccf33c85 on DB-5039 into cba6f382fa5c8877163f78cdd1643830748eb1ac on master.

Coverage decreased (-0.2%) to 50.485% when pulling cf128e39211f357beedc356c3ddea32eb9634f1b on DB-5039 into cba6f382fa5c8877163f78cdd1643830748eb1ac on master.


Agent.TempDirectory Clean Verbiage

Agent.TempDirectory is cleaned after each job in a pipeline. I had some files being put in this directory in the first job in a pipeline. After that job execution completed, the files were removed and unable to be used in subsequent jobs.

@awickham10 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Adjust udunits2 dependency

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #14.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[x] Bump the build number if needed.

Fixes #14

@conda-forge-admin please rerender


[models] Support animated models

Proposal for animated models
I have been using raylib for a little toy project and I love it, it works really well. There's one thing I miss though, and that's support for animated models. Loading and parsing 3d animated models can be a lot of work, so I thought that integrating Assimp into raylib wouldn't be a bad idea. That way we would also gain support for a parsing a million different file formats. Assimp is written in C++ but comes with a clean C api. I'd be happy to help with implementing this.
At this point, I also want to mention something else: Is it really such a good idea that we pass structs by value to functions like DrawModelEx? Sure it's easier for beginner programmers to understand, but I think it hurts the library more than it helps. It's especially confusing for seasoned C programmers, wo generally considered this a bad practice.

Hi @Lisoph! Nice project!
I know 3d animated models support is one of the weakness in raylib but no plans to integrate Assimp.
Assimp is a HUGE library with lots of dependencies and one of the goals of raylib is just removing external dependencies, all required libraries (mostly single-file header-only) are integrated with base code.
I'm looking for some simple solution, probably in the line of IQM models animation support, I know @culacant is working on it (single-file header only?). In the meantime my recomendation is just using Assimp or any other 3d models lib at user side, not directly integrated in raylib.
About DrawModelEx(), you're right, Model struct has grown a lot since first version. Changing it now requires some internal redesign... I'm evaluating the benefits of that change because, despite I agree with the theory (value vs reference), I'd like to know what's the impact of that change in performance terms... I bet is lower than expected...

Sure it's easier for beginner programmers to understand

It might be possible to maintain API compatibility while still passing by reference if we typedef an array, e.g. for Model:
typedef struct Model {
    Mesh mesh;              // Vertex data buffers (RAM and VRAM)
    Matrix transform;       // Local transform matrix
    Material material;      // Shader and textures data
} Model[1];

This could be an option as well…

@procedural did a quick profiling sample comparing pass by value vs pass by pointer. It seems there is not much difference between both... actually pass by value works unexpectly faster.

I don't think that profling sample is a very good metric. It's trivial for the compiler to optimize that function, no matter how the data is passed. To get a true idea of performance we would have to simply try it out on raylib, ideally in prebuilt library form.
Still, I played around with that profiling sample on godbolt and found that pass-by-value forces the compiler to generate a memcpy call, to copy the data onto the function's stack (even on -O3), while for pass-by-reference no such call is generated. Tested with gcc 8.1 and I marked procedure as __attribute__ ((noinline)).
Here's my sample code:
Expand
enum {BYTES = 10000};

struct TYPE {
  unsigned char bytes[BYTES];
};

extern struct TYPE output;

#define BY_VAL 1

#if BY_VAL == 1
void __attribute__ ((noinline)) procedure(struct TYPE input, struct TYPE * output) {
  for (int i = 0; i < BYTES; i += 1) {
    output->bytes[i] = input.bytes[i] + input.bytes[i];
  }
}
#else
void __attribute__ ((noinline)) procedure(struct TYPE const *input, struct TYPE *output) {
  for (int i = 0; i < BYTES; i += 1) {
    output->bytes[i] = input->bytes[i] + input->bytes[i];
  }
}
#endif

int main() {
  for (int i = 0; i < 10000; i += 1) {
    struct TYPE input;
    // commented out for less assembly
    // for (int j = 0; j < BYTES; j += 1) {
    //   input.bytes[j] = rand() % 255;
    // }
    #if BY_VAL == 1
    procedure(input, &output);
    #else
    procedure(&input, &output);
    #endif
  }
}

@Lisoph

I don't think that profling sample is a very good metric.

Is it?  For #define BY_VAL 0 results are similar but slightly slower at -O0.
Can I merge your changes under unlicense (Public Domain) license so you could test too?
I added Visual Studio 2017 project file if someone wants to try it on Windows.

@procedural sure, feel free to use the code. I'll check it out at home, thanks for the VS2017 project files!

I don't know how to use Chrome's Tracing, but I ran the profiling program myself (VS2017) and the pass-by-ptr is faster for me in both Debug and Release builds.
I create another example on Godbolt which does something similar to our benchmark and the pass-by-ptr wins here as well. The assembly for by_ptr and by_val is nearly identitcal (only some reordering), but the invocation is quite different. by_ptr simply passs

Hi @Lisoph, actually, we were discussing this issue further with @procedural on his C programming Discord channel. There was some measure precission issues and effectively pass-by-ptr is faster than pass-by-val. But difference was not that big, if I remember correctly it was around 50ns for a 10Kb structure.

Anyway, I think this pass by pointer debate deserves it's own issue.

Agree, just opened it: https://github.com/raysan5/raylib/issues/563
PD. About the animated models issue, I'm working now through @culacant IQM implementation to try to integrate it into raylib in some way...

Yo! I'm interested in doing 3D with raylib as well! I was just wondering about the status of this. I'm wondering why it might be a good idea to go with the IQM format instead of something more standard, like OBJ (assuming OBJ can support animations), DAE, or OGEX (for a hopefully more future-proofed standard driven by community needs)?
Also, I was wondering if you were dealing with possible issues regarding the design of an animated mesh struct and how that might work with raylib's current API. Seems like you'd be good to stick a list of animations onto the Model struct, import the animations, and then transform the verts from there, but I imagine I must not fully grasp all of the complexities.

I chose IQM and the ascii-counterpart IQE because it is basically OBJ with animation data added in.
Keep in mind I had no idea what animations were when i made the loader, and the readability of the files was a big help in figuring out what actually needed to happen to make stuff move on screen.
Turns out there's not a whole lot of choices in formats if you look at ease of implementation.
The doom 3 md5 format is easy enough, but it's getting pretty old and blender doesn't play nice with the exporters I found.
FBX would be great to support because everyone's using it, but I couldn't find alot of documentation since it's supposed to be closed source. (Blender reverse engineered it tho, so it can't be that hard to figure it out)
Alot of the newer would-be standards like OGEX, DAE and glTF are more focused on stuffing alot of useless specialized data in their formats like physics and lights, which is great if you plan to use them but it makes the format way more complicated than it needs to be if you just want animations.
The good part of all this is that once the structs for the animated meshes and animations are in place it wouldn't be that hard to write loaders for different formats, because from what I saw most formats basically store the same data, with a couple minor differences (like local rotations instead of global etc).
Ray was looking at ways to put my code in Raylib in a way that makes sense and doesn't break everything else.

@culacant Ah, I see, I see. That might be a good middle-ground, then, especially if it's inherently simple to load and 3d modelers should have stable exporters. Thanks for the explanation!
@raysan5 Great to hear! Hope to see animated models soon~

Working sample added on commit https://github.com/raysan5/raylib/commit/198a02352764a78eb0fc3b94c3c8418d2d0c1432
API requires some changes and review...

Came across gITF today
It seems like a super-set of what's needed here, so maybe way too complicated?

I don't know, since glTF is JSON based we could simple ignore the stuff we don't care about while parsing. It looks as though going with glTF would force us to write our own parser, because I couldn't find any C libraries to do that. While it helps that glTF is based on JSON, we would still need a JSON library.
glTF definitely is the most modern 3d model format that I know of. A lot of software already supports it out of the box.

It looks as though going with glTF would force us to write our own parser, because I couldn't find any C libraries to do that. While it helps that glTF is based on JSON, we would still need a JSON library.

Yeah, lots of C++ libraries but no C ones except AssetKit which is under development.  However, since it's JSON it should be fairly straight forward.  I've seen the minimal-gltf-loader code. It creates buffers, loads data and makes links within structure; basically a deserializer, but nothing complicated.

glTF definitely is the most modern 3d model format that I know of. A lot of software already supports it out of the box.

Yeah, I came here looking for GLTF support for a personal C++ project.  It's supported by many engines and companies already.  Why we should all support glTF 2.0 as THE standard asset exchange format for game engines by Godot explains the benefits quite well.

Nice read. Undoubtely glTF is a great format to be supported by raylib...

Regarding a JSON parser: I came across jsmn. It's quite low level and requires a lot of manual work, but in exchange it is extremely small, easy to integrate and even quite flexible and fast. It might be a good base should we decide to roll our own glTF parser.

@Lisoph already started working on glTF v2.0 support! I'm using cgltf, looks amazing! Actually, it embeds JSMN in a single header file!
Implementation requires taking care of https://github.com/raysan5/raylib/issues/596 also. Need some time to figure out the best way to redesign this, it's quite a big change...
Unfortunately, animation is not supported by the loader yet...

Quick update on animation support. I'm currently implementing multi-mesh and multi-material support and, as long as Model struct is been changed, I also added some data for animation. Here there are the proposed new structs:
// Model bone pose (transform)
typedef struct Pose {
    Vector3 translation;    // Translation
    Quaternion rotation;    // Rotation
    Vector3 scale;          // Scale
} Pose;

// Model type
typedef struct Model {
    Matrix transform;       // Local transform matrix

    int meshCount;          // Number of meshes
    Mesh *meshes;           // Meshes array

    int materialCount;      // Number of materials
    Material *materials;    // Materials array
    int *meshMaterial;      // Mesh material number

    // Animation data
    bool animated;          // Model is animated
    int boneCount;          // Number of bones
    Pose *basePose;         // Pose by bone
} Model;

// Model animation
typedef struct Animation {
    int boneCount;          // Number of bones (joints)
    int frameCount;         // Number of animation frames
    float frameRate;        // Frame change speed
    Pose **framePoses;      // Poses array by frame and bone
} Animation;

I reviewed the data needed by IQM but I was wondering if I'm missing some important piece of data to be added to model to properly support animations. Feedback, please!

After some reading and a more careful analysis, here it is a new structs proposal for dealing with animation, probably some element still missing (maybe struct Bone?) but it seems pretty clear to understand:
// Transformation properties
typedef struct Transform {
    Vector3 translation;    // Translation
    Quaternion rotation;    // Rotation
    Vector3 scale;          // Scale
} Transform;

// Animation key frame
typedef struct KeyFrame {
    int boneCount;          // Number of bones (joints)
    Transform *poses;       // Bones transformations
    //float timeStamp;        // Keyframe time
} KeyFrame;

// Model type
typedef struct Model {
    Matrix transform;       // Local transform matrix

    int meshCount;          // Number of meshes
    Mesh *meshes;           // Meshes array

    int materialCount;      // Number of materials
    Material *materials;    // Materials array
    int *meshMaterial;      // Mesh material number

    // Animation data
    bool animated;          // Model is animated
    KeyFrame basePose;      // Model base animation pose
} Model;

// Model animation
typedef struct Animation {
    int frameCount;         // Number of animation frames
    KeyFrame *frames;       // Key frames (bones poses)
    //float frameRate;        // Frame change speed
}

I really care for structs naming, I want to be be clear and comprehensive, I had doubts between KeyFrame or AnimFrame but I feel KeyFrame is clearer, after all, one key-pose could be independent of animation.
Any piece missing? Please, feedback!

Big update in commit https://github.com/raysan5/raylib/commit/d89d24c5e8930c18a93a6403e26914a9e2b23b84!
Finally, the implementation selected has been:
// Transformation properties
typedef struct Transform {
    Vector3 translation;    // Translation
    Quaternion rotation;    // Rotation
    Vector3 scale;          // Scale
} Transform;

// Bone information
typedef struct BoneInfo {
    char name[32];          // Bone name
    int parent;             // Bone parent
} BoneInfo;

// Model type
typedef struct Model {
    Matrix transform;       // Local transform matrix

    int meshCount;          // Number of meshes
    Mesh *meshes;           // Meshes array

    int materialCount;      // Number of materials
    Material *materials;    // Materials array
    int *meshMaterial;      // Mesh material number

    // Animation data
    int boneCount;          // Number of bones
    BoneInfo *bones;        // Bones information (skeleton)
    Transform *bindPose;    // Bones base transformation (pose)
} Model;

// Model animation
typedef struct ModelAnimation {
    int boneCount;          // Number of bones
    BoneInfo *bones;        // Bones information (skeleton)

    int frameCount;         // Number of animation frames
    Transform **framePoses; // Poses array by frame
} ModelAnimation;

Still some doubts about having BoneInfo data duplicated in Model and ModelAnimation... but it definitely makes sense and, actually, it can be used to verify if an animation fits in a model.
Some work left on animation functions implementation.

Sounds good @raysan5. :smile:

Multiple functions added and some reviewed in commit https://github.com/raysan5/raylib/commit/92733d6695e0cdab3b42972f2cd6ed48d98ec689.


MockDirectory.SetCreationTime throws NullReferenceException

Attempting to set the creation date of a mock directory results in a null reference exception. Here is an NUnit test that demonstrates the issue:
[TestCase(true)]
[TestCase(false)]
public void TestDirectorySetCreationTime(bool realFileSystem)
{
  const string PATH = @"C:\test123";
  IFileSystem fs = realFileSystem ?
    (IFileSystem) new FileSystem() :
    (IFileSystem) new MockFileSystem();
  fs.Directory.CreateDirectory(PATH);
  fs.Directory.SetCreationTime(PATH, DateTime.Now);  // throws if fs is a MockFileSystem
  fs.Directory.Delete(PATH);
}

More information about the exception:
System.NullReferenceException was unhandled by user code
  HResult=-2147467261
  Message=Object reference not set to an instance of an object.
  Source=System.IO.Abstractions.TestingHelpers
  StackTrace:
       at System.IO.Abstractions.TestingHelpers.MockFile.SetCreationTime(String path, DateTime creationTime)
       at System.IO.Abstractions.TestingHelpers.MockDirectory.SetCreationTime(String path, DateTime creationTime)
  InnerException:

I am new to GitHub (and Git itself), but will attempt a fix.

The source of the null value is the MockFileSystem.GetFileWithoutFixingPath method:
private MockFileData GetFileWithoutFixingPath(string path)
{
    lock (files)
    {
        MockFileData result;
        files.TryGetValue(path, out result);
        return result;
    }
}

The call files.TryGetValue(path, out result) does not find the path C:\test123 because it doesn't end with a backslash. Consequently the return value is null, which results in a NullReferenceException later in the code.
The issue can be fixed with the following 2-line change to GetFileWithoutFixingPath:
private MockFileData GetFileWithoutFixingPath(string path)
{
    lock (files)
    {
        MockFileData result;
        files.TryGetValue(path, out result);
        if (result == null)
            files.TryGetValue(path + @"\", out result);
        return result;
    }
}

All unit test still pass, including the one I added for this issue. However, it appears the code generally expects a trailing backslash on directories, and I am leery of short-circuiting this behavior. Any thoughts or comments about the proposed change would be appreciated.
Below is the additional unit test:
[Test]
public void MockDirectory_SetCreationTime_ShouldNotThrowWithoutTrailingBackslash()
{
    const string PATH = @"C:\NoTrailingBackslash";
    MockFileSystem fs = new MockFileSystem();
    fs.Directory.CreateDirectory(PATH);
    fs.Directory.SetCreationTime(PATH, DateTime.Now);
    fs.Directory.Delete(PATH);
}


File reference breaks with array in vNext

If you have a file reference, where the file only contains an array, the parser will fail pulling it into the file.

Submitted a PR with the fix. PR is now merged to vnext branch.
https://github.com/Azure/vdc/pull/136


wrong idiom if &iq.Bind == nil.     It's always not nil.

Bind is struct in struct , so check &iq.Bind == nil is wrong, the sub-struct always have some address
var iq clientIQ
if &iq.Bind == nil {
  log.Printf("&iq.Bind is nil\n")
} else {
  log.Printf("&iq.Bind is not nil\n") // Always printed !
}

Proposed fix:
change in clientIQ definition Bind bindBind to Bind *bindBind, likes:
type clientIQ struct { // info/query
    XMLName xml.Name `xml:"jabber:client iq"`
    From    string   `xml:"from,attr"`
    ID      string   `xml:"id,attr"`
    To      string   `xml:"to,attr"`
    Type    string   `xml:"type,attr"` // error, get, result, set
    Error   *clientError  // Pointer to struct instead nested struct
    Bind    *bindBind    // Pointer...
}

And also change checking if the 'bind' field used to:
if iq.Bind == nil {
  return errors.New("<iq> result missing <bind>")
}

New version memory usage in Win32,
current version unsafe.Sizeof(clientIQ)=136
after my fix unsafe.Sizeof(clientIQ)=56 (in case <iq query...)
So if optional parts are not present, only nil pointer saved.
By the way, in the sruct  changed tags:
previous version was xml:",attr", I use xml:"from,attr"
It's important because go xml parser is case sensitive and without the change, did not parse
from, to, type and id

I don't see any dereferencing of iq.Bind anymore in the code, so I'm going to close this.


Make sure ADD_ONION cannot bind to any of the control ports

For policies that allow ADD_ONION commands, we need to make sure that it cannot bind to any of the control ports (that is neither the original control port, nor any of the filter ports).

fixed https://github.com/subgraph/roflcoptor/commit/4dbb28159af576784b8415726c0b3f8204b2085f

this issue is most likely not fixed... now that i'm actually writing unit tests i should have this mess sorted soon.

i wrote unit tests for this feature; here
https://github.com/subgraph/roflcoptor/blob/888e6c00a83b0710326f8961c59e6e6304cdaf70/session_test.go#L435-L463
i've fixed some bugs with it... but maybe it's still not good enough?

shouldAllowOnion now has full test coverage. i've since then fixed several bugs with it and added more tests.
i'm pretty happy with it now but i wonder if i've overlooked some detail that could allow a client program talking to our filter daemon to create an onion service which connects to a control port?


White rhino UI fixes

Increase font size of "2 admires" to fix jumpy feeling text
https://user-images.githubusercontent.com/6754223/201377982-968c8644-a94b-4e60-923e-2042213da4e8.mov
Fix alignment of header text on NotificationsModal

@tbezman for the admires, can we make the behavior:

"kaito admired this"
add a like
"kaito and 1 other admired this"
someone else likes it
"kaito and 2 others admired this"

instead of (2 admired this, 3 admired this, etc.)
if this is a bigger lift we can add this to the white rhino linear board


Erlang integration

The following features are missing:

Intelisense

[ ] Completion
[ ] Signature
[ ] Docs (can reuse all of the Elixir infrastructure)

Coding

[ ] Defining functions
[ ] Defining records (needs to consider how it makes cells as stales
[ ] Other functionality from the Erlang shell?

i can pickup this, still needs?

Yes, there are some PRs you can check out for discussion. :)

https://github.com/livebook-dev/livebook/pull/2806 - Hopefully this will cover some use cases.


Application doesn't start

Using OSX El Capitan 10.11.4, Mapbox Studio Classic shows a loading spinner then immediately gets stuck on a blank screen. Any ideas?

Bump.
Having the same problem on the same OS. I tried to build from source, but that doesn't work either.

My coworker and I have the same problem.  I'd like to modify some styles for use in our Atlas Server, but this issue make that pretty difficult.

I believe the issue is that it's not hitting the right Atlas Server IP. Is there a way to disconnect Atlas Server from Mapbox Studio if the program is not starting?

Tried to run on my Windows 8 Desktop, here is the output
K:\ElectSrc\mapbox-studio-classic-mb-pages>npm start

<EMAIL_ADDRESS>start K:\ElectSrc\mapbox-studio-classic-mb-pages
./index.js

'.' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "start"
npm ERR! node v4.4.7
npm ERR! npm  v2.15.8
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: ./index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script './index.js'.
npm ERR! This is most likely a problem with the mapbox-studio package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ./index.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs mapbox-studio
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls mapbox-studio
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     K:\ElectSrc\mapbox-studio-classic-mb-pages\npm-debug.log
K:\ElectSrc\mapbox-studio-classic-mb-pages>npm bugs mapbox-studio
K:\ElectSrc\mapbox-studio-classic-mb-pages>

@logindmartin the easiest way to get started on Windows is to download the installer package:
https://www.mapbox.com/mapbox-studio-classic/

This was solved when the hanging request I described above finally timed out and new connect page appeared.  So, in conclusion, just wait longer!

I resolved mine (built from source) on Ubuntu and OSX by doing the following:
First, installed nvm and switched to node version 0.10.
From your home directory, rm -rf .mapbox-studio (clears cache and other junk)
Then, from your mapbox-studio-classic directory, rm -rf node_modules
sudo npm install
npm start


Dependency has vanished

gosubli.me/something-borrowed/gocode is not resolving, so I can't download/run the code for this repo.
Would this project be open to using one of the available vendoring schemes (e.g. gb vendor)?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35586155-dependency-has-vanished?utm_campaign=plugin&utm_content=tracker%2F15718991&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F15718991&utm_medium=issues&utm_source=github).

Um, I don't think that is a dependency of limetext. That's an autocompletion daemon, perhaps a dependency of the editor you are using?

Perhaps to be more explicit, I don't have gosubli.me/something-borrowed/gocode downloaded in my $GOPATH or anywhere else for that matter, and I can build and run lime-qml just fine.

Also, I think github.com/nsf/gocode is the same thing, so if you go get github.com/nsf/gocode that might resolve your issue.

Ahh - I had an older clone which had lime-packages pre 9889d9c769f889a8b51209dbdd19c46f98d4c47e - which referred to gosubli.me. For some reason, go get -u was trying to update the dependency before downloading new code for lime-qml.

I can reproduce this with a clean GOPATH for lime-html. Will open an issue there instead.