Add support for multiple BAMs

Also includes a fix for the soft-clip analysis to handle the case when no SV tool output is provided.

@sbandara @msahraeian please have a look. I will merge when both of you say okay.

I think it is good to go.
On Mon, Feb 15, 2016 at 12:37 PM, Marghoob Mohiyuddin <
<EMAIL_ADDRESS>wrote:

@sbandara https://github.com/sbandara @msahraeian
https://github.com/msahraeian please have a look. I will merge when
both of you say okay.
—
Reply to this email directly or view it on GitHub
https://github.com/bioinform/metasv/pull/98#issuecomment-184382042.

+1.


fix: don't update address of node in node-resources-controller

What type of PR is this?

/kind bug
What does this PR do?

Which issue(s) does this PR fix?

Fixes #338
Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

cherry-pick  to release-0.3.0

/lgtm


GPT-4-vision-preview does not throw error when the message content indicates one

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
I'd like to know if there is a way to detect responses as the ones listed below, as the openai cliend does not throw any exceptions when these occur?
“I am unable to view images directly.”
“Unfortunately, there seems to be an issue as the image is not visible or accessible to me at the moment.”
“I’m unable to provide real-time analysis or generate content based on the image provided as the capabilities have been disabled.”
“Given the lack of an actual image to analyse, (…)”
“Since the image cannot be displayed (…)”
“Unfortunately, I cannot assist with this request.”
“Unfortunately, there seems to be some misunderstanding as I do not possess the ability to physically see images or any type of attachments. Therefore, I am unable to provide an analysis based on an actual image.”
Here is a discussion on the same issue:
https://community.openai.com/t/calls-to-gpt-4-vision-preview-dont-produce-errors-but-it-says-it-cant-read-images/478203/7
Please suggest a way I can handle these grasefully.
To Reproduce
N/A
Code snippets
No response
OS
Ubuntu 20.04
Python version
Python 3.10.12
Library version
openai  1.6.1

Thanks for reporting!
This sounds like an issue with the underlying OpenAI API and not the Python library, so I'm going to go ahead and close this issue.
The community is indeed the correct place for issues like this.


Attempting clean install

I'm attempting to follow the docs for a clean install, but having issues getting things running.
After cloning the repo and changing into the OpenEats directory I'm copying the docker-prod.override.yml and env_prod.list from the samples folder. I'm running a Nginx reverse proxy on port 80 already, so I have updated the docker-prod.override.yml with:
version: '2.3'
services:
  nginx:
    ports:
      - "7000:80"

I'm making the following changes to the env_prod.list file:
...
ALLOWED_HOST=myers.isa-geek.org
HTTP_X_FORWARDED_PROTO=true

# Node config
NODE_ENV=production
NODE_URL=localhost:8080
NODE_PORT=8080
NODE_API_URL=http://myers.isa-geek.org
NODE_LOCALE=en

I've got my nginx file set to redirect as follows:
	location /openeats/ {
                proxy_set_header Host $http_host;
                proxy_set_header X-Real-IP $remote_addr;
                proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
                proxy_set_header X-Forwarded-Proto $scheme;
                proxy_pass http://<IP_ADDRESS>:7000;
        }

Creating a super user works as expected, as far as I can tell. I can't seem to browse the django admin interface at <IP_ADDRESS>:8000.
It doesn't seem the access to the api container is working to load recipes, either:
docker-compose run --rm api bash
ERROR: build path /opt/OpenEats/openeats-web either does not exist, is not accessible, or is not a valid URL.

I can load the site at http://<IP_ADDRESS>:7000, but no recipes are shown because it's unable to reach the api, net::ERR_CONNECTION_REFUSED in the Chrome Console.
Visiting the site through the hostname https://myers.isa-geek.org/openeats/ shows GET https://myers.isa-geek.org/bundle.js net::ERR_ABORTED in the console.
Checking the docker logs I see 2018-02-07  3:25:52<PHONE_NUMBER>83488 [Warning] Access denied for user 'root'@'localhost' (using password: NO) repeated in the openeats_db_1 container every few seconds.
Sorry for the long post, but I hope that including all this info will show a glaring hole in my configs. I think I should be trying from this repo and not the old one? I'm still playing around trying to get things to work, but if I'm honest I'm a bit lost in the interaction between all the containers and what is appropriate for configs in my scenario. Thanks in advance!

My brain is a bit fryed, so I'll leave a quick response now and give a lot more details tomorrow.

Yes you should be using this repo. I created it so I could spin up separate repos for each piece of code (ui, api, nginx, etc...) as I add more features and to reduce confusion about cloning the old repo.

Whenever you try and run docker-compose be use to add the -f flag of (docker-compose -f docker-prod.yml). If you run docker-compose without this then docker will try and build images from your local file system and think you are using the dev version. (Thinking about this more now, I should probably remove that from this repo or at least make thedefault docker file a production ready copy.)

do you mind doing a docker ps to see what containers are running?

The API also doesn't have bash anymore (you'll need to use sh)
docker-compose -f docker-prod.yml run --rm api sh
I'll fix this in the docs tomorrow ^^

Here's the docker ps -a output for the relevant containers.
docker ps -a
CONTAINER ID        IMAGE                      COMMAND                  CREATED             STATUS                    PORTS                         NAMES
34d9b5f435fe        openeats/openeats-nginx    "./start.sh"             12 hours ago        Up 12 hours               80/tcp                        openeats_nginx_1
d792e753b8e4        openeats/openeats-web      "npm start"              12 hours ago        Up 12 hours                                             openeats_web_1
d2b7d7a4b088        openeats/openeats-api      "/startup/prod-ent..."   12 hours ago        Up 12 hours                                             openeats_api_1
3d7536988e3a        mariadb                    "docker-entrypoint..."   12 hours ago        Up 12 hours (healthy)     3306/tcp                      openeats_db_1

Thanks for the update on the bash to sh change. I was able to run the manage.py commands. Had a couple warnings but seems like it was successful.
./manage.py loaddata news_data.json
/usr/local/lib/python3.6/site-packages/django/db/models/fields/__init__.py:1423: RuntimeWarning: DateTimeField News.pub_date received a naive datetime (2010-10-10 13:32:47) while time zone support is active.
  RuntimeWarning)
Installed 1 object(s) from 1 fixture(s)
/code # ./manage.py loaddata recipe_data.json
/usr/local/lib/python3.6/site-packages/django/db/models/fields/__init__.py:1423: RuntimeWarning: DateTimeField Recipe.pub_date received a naive datetime (2011-05-21 07:35:32) while time zone support is active.
  RuntimeWarning)
/usr/local/lib/python3.6/site-packages/django/db/models/fields/__init__.py:1423: RuntimeWarning: DateTimeField Recipe.update_date received a naive datetime (2011-05-21 07:35:32) while time zone support is active.
   This may take a minute...
Creating network "openeats_default" with the default driver
Creating openeats_api_1   ... done
openeats_db_1 is up-to-date
Creating openeats_web_1   ... done
Creating openeats_web_1   ...
Creating openeats_nginx_1 ... done
App started. Please wait ~30 seconds for the containers to come online.

$ docker ps
CONTAINER ID        IMAGE                            COMMAND                  CREATED              STATUS                        PORTS                NAMES
a1370569611f        openeats/openeats-nginx:latest   "./start.sh"             12 seconds ago       Up 11 seconds                 <IP_ADDRESS>:80->80/tcp   openeats_nginx_1
11e1a75402f9        openeats/openeats-web:latest     "npm start"              13 seconds ago       Up 12 seconds                                      openeats_web_1
266b56e00951        openeats/openeats-api:latest     "/startup/prod-entry…"   14 seconds ago       Up 13 seconds                                      openeats_api_1
2ec095a2afd6        mariadb                          "docker-entrypoint.s…"   About a minute ago   Up About a minute (healthy)   3306/tcp             openeats_db_1

$ docker-compose -f docker-prod.yml run --rm --entrypoint 'python manage.py createsuperuser' api
Starting openeats_db_1 ... done
Username (leave blank to use 'root'): ryan
Email address:
Password:
Password (again):
Superuser created successfully.

$ docker-compose -f docker-prod.yml run --rm api sh
Starting openeats_db_1 ... done
/code # ./manage.py loaddata course_data.json
Installed 4 object(s) from 1 fixture(s)
/code # ./manage.py loaddata cuisine_data.json
Installed 9 object(s) from 1 fixture(s)
/code # ./manage.py loaddata news_data.json
/usr/local/lib/python3.6/site-packages/django/db/models/fields/__init__.py:1423: RuntimeWarning: DateTimeField News.pub_date received a naive datetime (2010-10-10 13:32:47) while time zone support is active.
  RuntimeWarning)
Installed 1 object(s) from 1 fixture(s)
/code # ./manage.py loaddata recipe_data.json
/usr/local/lib/python3.6/site-packages/django/db/models/fields/__init__.py:1423: RuntimeWarning: DateTimeField Recipe.pub_date received a naive datetime (2011-05-21 07:35:32) while time zone support is active.
  RuntimeWarning)
/usr/local/lib/python3.6/site-packages/django/db/models/fields/__init__.py:1423: RuntimeWarning: DateTimeField Recipe.update_date received a naive datetime (2011-05-21 07:35:32) while time zone support is active.
  RuntimeWarning)
Installed 31 object(s) from 1 fixture(s)
/code # ./manage.py loaddata ing_data.json
Installed 15 object(s) from 1 fixture(s)
/code # exit

This all worked for me. Hitting localhost:7000 gives me the UI with the correct api results. Through here are a few notes:

I wasn't using a domain name for this, just my localhost.
The env file:

The secrets and passwords are not changed from the examples (you should ofc change these).
NODE_API_URL=http://localhost:7000 - This needs to be what the UI needs to call to get to the UI from your browser. There are rules in the nginx config that will route this to the api when calling /api.

Now on to your specific config.

First you won't be able to call the api directly because the ports are not exposed in production. Meaning the only way to get to the api is via the nginx proxy. So if you are trying to see if the api is working try http://<IP_ADDRESS>:7000/admin
Second NODE_API_URL=http://myers.isa-geek.org looks like it needs to be changed to NODE_API_URL=http://myers.isa-geek.org/openeats. If your domain is routing everything through /openeats, then the api needs to be called from that. The way you currently have it, the UI will try and call the api at http://myers.isa-geek.org/api/... when it should be calling via http://myers.isa-geek.org/openeats/api/...
Third, once you get the UI and API to work, there might be some issues with routing due to the UI routing on the base domain. There are some sections in the UI code that will try and do a redirect to, for example, /browse/. This will lead to http://myers.isa-geek.org/openeats/news/ -> http://myers.isa-geek.org/browse/. But we can address this issue once everything else is working.

Hope this helps!

Ok, I was able to get it working using localhost following the setup guide. I can reach the site using localhost:7000 and the api admin using localhost:7000/admin. Visiting http://<IP_ADDRESS>:7000 loads the web files but does not load any recipes and http://<IP_ADDRESS>:7000/admin returns a Bad Request (400).
I'm unable to get it to work using NODE_API_URL=https://myers.isa-geek.org/openeats (I forgot it should be https).
I've got env_prod.list set to the following:
# Django config
API_URL=<IP_ADDRESS>:8000
API_PORT=8000
DJANGO_SECRET_KEY= sdfsadfas32e98zsdvhhsnz6udvbksjdhfi4galshjfg
DJANGO_SETTINGS_MODULE=base.settings
DJANGO_DEBUG=False
ALLOWED_HOST=localhost
HTTP_X_FORWARDED_PROTO=true

# Node config
NODE_ENV=production
NODE_URL=localhost:8080
NODE_PORT=8080
NODE_API_URL=https://myers.isa-geek.org/openeats
NODE_LOCALE=en

When I visit the page I get an error GET https://myers.isa-geek.org/bundle.js net::ERR_ABORTED      myers.isa-geek.org/:22. It seems to be looking for the <!-- Core JS Bundle --> but can't find it?

The reason your domain isn't working is because the index is trying to get the bundle from /bundle.js not /openeats/bundle.js. I'm looking at the code now to see what it takes to change this. See: https://github.com/open-eats/openeats-web/blob/master/public/index.html#L22
Getting the IP working, you might need to add that to the allowed hosts. Right now I only have it able to accept one host. That can easily be changed here. For now go ahead and try: ALLOWED_HOST=<IP_ADDRESS>.

Thanks for looking into this. I was able to update the env_prod.list to allow access over the IP address by using:
ALLOWED_HOST=<IP_ADDRESS>
NODE_API_URL=http://<IP_ADDRESS>:7000

I fixed the issue with multiple allowed hosts here: https://github.com/open-eats/openeats-api/pull/1
It's basically the same, but if there is a comma, it will treat it like a new host. IE, <IP_ADDRESS>,<IP_ADDRESS>

# Database config
MYSQL_DATABASE=openeats
MYSQL_ROOT_PASSWORD=root

# Django config
API_URL=<IP_ADDRESS>:8000
API_PORT=8000
DJANGO_SECRET_KEY=sdfsadfas32e98zsdvhhsnz6udvbksjdhfi4galshjfg

DJANGO_SETTINGS_MODULE=base.settings
DJANGO_DEBUG=False
ALLOWED_HOST=localhost
HTTP_X_FORWARDED_PROTO=true

# Node config
NODE_ENV=production
NODE_URL=localhost:8080
NODE_API_URL=http://localhost:8000
NODE_LOCALE=en

I am getting this error message from my db container:
9:06:28 35 [Warning] Access denied for user 'root'@'localhost' (using password: NO)
Not sure how to fix it, I tried the steps in this thread.

That message is is a dead lead. Meaning that it doesn't mean anything. I need to remove it.
This thread is quite old so some things in here may not work as described anymore.
What specifically isn't working for you?

Are you trying to run this in prod or dev mode? Looks like dev based on ur config file
If you are using prod what's in your docker-prod.override.yml

Sorry to open this thread up again. I am struggling to get my server working on the LAN. I am sure it must be something to do with the API URL. but can't fathom what is wrong. I will paste my current env_prod.list below.
The default set-up worked fine, but I am trying  to set it up with LAN access. With this below set-up I am able to access the home page which renders, but no data is pulled in. Also when I try to go to http://<IP_ADDRESS>:8000/admin/ I get a "Bad Request (400)".
# Database config
MYSQL_DATABASE=openeats
MYSQL_ROOT_PASSWORD=##REMOVEDPASS##

# Django config
API_URL=<IP_ADDRESS>:8000
API_PORT=8000
DJANGO_SECRET_KEY=sdfsadfas32e98zsdvhhsnz6udvbksjdhfi4galshjfg
DJANGO_SETTINGS_MODULE=base.settings
DJANGO_DEBUG=False
ALLOWED_HOST=192.168.1.x
HTTP_X_FORWARDED_PROTO=true

# Node config
NODE_ENV=production
NODE_URL=192.168.1.x:8080
NODE_API_URL=http://192.168.1.x:8000
NODE_LOCALE=en

Any help would be greatly appreciated.


Id Attribute in converted HTML missing

When I give an Id to my text block it appears in the mjml but not in the converted HTML. Is there are a way to adjust the conversion that the Id appears on the HTML element (preferably on most outer on) as well?

I think I understand it know. The Problem is that MJML is not supporting Ids. And grapesjs-mjml is using the official parser to convert it to HTML. So I guess the parser is just ignoring/deleting the Id.
As a workaround, I'm added "css-class" as a trait. This will be converted to the class attribute in HTML.


ng-class

Please help me with adding a class , using ng-class in angular-formly.

Please be more specific with what you're trying to accomplish. We do add classes to formly-forms and formly-fields. What do you want to have an ng-class on? If you need it on the templates, which template library are you using, bootstrap or vanilla? What version of angular-formly are you using?

@rahil471, have you resolved your issue?

Hey Thanks for following up ,  this issue is solved.
I've another issue , I'll start a new thread if required.
I wanted to ask is that , I'm unable to retrieve form name in my controller, is it because of the <ng-from /> tag , or may be because the library is adding form by it self into the name attribute.
For eg: If i name the form as activateSimForm , the DOM shows it as name="activateSimForm form"

Yes, angular-fromly adds the name attribute for you and binds a form attribute for you to alleviate confusion. So, you'd do this instead:
<formly-form form="activateSimForm" model="simModel" fields="simFields"></formly-form>

If that doesn't solve your issue, please open a new issue :-)

Hey Thanks man ,
That worked.
I'm enjoying using this.


`length error running .nlp.findDates

.nlp.findDates throws `length errors on the following inputs
"5.24.01"
"December 18-January 2"
"memo of 10-5 1 page.doc"
Running it on the Enron emails, it is throwing `length on ~8% of the emails

Hi tepu,
The error was occurring as there wasn't picking up the format of the date correctly. This bug should be fixed now if you clone this repository again.
q).nlp.findDates"5.24.01"
2024.05.01 2024.05.31 "5.24" 0 4
q).nlp.findDates"December 18-January 2"
2018.12.01 2018.12.31 "December 18" 0 11
q).nlp.findDates"memo of 10-5 1 page.doc"
2010.05.01 2010.05.01 "10-5 1" 8 14

Let me know if you have any issues,
Thanks,
Fionnuala

Closing issue


Allow extensions to contribute Group Policy

As we are investigating Group Policy support, another request is for extensions to also add policies. Now there is a PolicyService we can build on top of, extensions can also expose their settings and promote them to policies. We would have to open up our setting contribution to allow declaring some settings as policies and then respecting it.

Priority of this needs to be visited.  Enterprises need to be able to set GPO on VsCode AND Visual Studio or what's the point?
Think it should be a shared don't care if vscode or visual studio or other implementation of live share but common gpo to set policy.  If I can do this for Visual Studio why not VSCode. https://learn.microsoft.com/en-us/visualstudio/liveshare/use/policies-visual-studio
Here is why  https://jsac.jpcert.or.jp/archive/2024/pdf/JSAC2024_2_3_sasada_hazuru_en.pdf


Limit printing of EarlGrey errors

Description
Currently when an action fails the entire UI hierarchy of the current app page is printed by EarlGrey.  Would like to be able to utilize the EarlGrey Error API to customize error output so test results are more readable to less technical team members and aren't filled with information that is unnecessary.  General improvement of error printing and ability to customize error output in general would be very useful, as currently I don't see a way of doing so other than outputting to a text file and parsing it at the end.

Any specific ideas? What do you count as necessary and what not?

The biggest issue on our end is the printing of the entire page's UI hierarchy on failed actions.  Being we use react-devtools for viewing the hierarchy when writing and debugging tests, printing out the whole hierarchy in the test result output is just somewhat excessive and makes the results of the tests quite cumbersome to navigate.
Also the stack trace of failed actions is somewhat unhelpful as it doesn't seem to reference a specific line of a test, it merely points to the line where the test is declared.  This makes the stack trace itself somewhat pointless as it's printing out everything except where the failure is actually occurring.
Just the ability to include/omit specific types of errors or omit certain types of error logging in general may be useful.  I know there is some functionality regarding this, but just being able to customize test output a bit more would be very useful.

+1. A flag to indicate whether or not to print the UI hierarchy on failure would suffice.

Any word on when/if some basic functionality relating to this will be able to be implemented?

You can just use a custom reporter for this. I wrote one that just strips out the UI Hierarchy, but it can be modified easily to strip everything besides test result. It is for Mocha but I am sure any test runner will have similar capabilities.
https://gist.github.com/cucumbur/e2fc35792c1677b97f23d35205b98550

@LeoNatan, is it okay if we reopen it, or you have other suggestions? I just don't know what to do in regards to this issue. :(

No. As I said in the other issue, I spoke with Rotem and we decided the current policy stands.


View grades of subattainments in the course results view

Instead of just a dialog showing the grade tree of a student, there should be a way to view the grades of all students per attainment as a table, for example:

Student
Exam
Project
Exercises

581041
5
4
5

127051
3
4
4

972919
0
5
5

How attainments at different levels are shown must be considered.

Let's first implement the view to show all attainments regardless of their level.


Updated documentation

So you asked for feedback on the setup process, and I thought some updates to the documentation might help other developers get set up. There is a tonne of awesome information in this repo, but it seems to have grown organically over time, so it's not necessarily the most organized.
There were two goals I had in mind for this pull request:

Make the README more of a sales pitch: "this is what fastlane is, this is why it's useful, this is how to install it, and here's where to go for more information."
Make the documentation easy to find, read, and reference.

I made some major changes to accomplish these goals:

Moved the documentation into a new /docs directory.
Moved a lot of the README into the /docs/README.md file, which is parsed by GitHub when you look at that directory.
Moved a lot of the README into other, more specific markdown files in the /docs directory.
Reorganized the Actions.md file to group all the actions into one of several categories: Building, Testing, Deploying, Modifying Project, Developer Portal, Using git, and Notifications. Keeping all of these straight was actually one of the biggest challenges for me when I was setting up fastlane, so hopefully this makes things easier.

There're also some smaller things I changed throughout, like consistently ending lists with periods, consistent capitalization, and consistent use of H1, H2, H3 in markdown files. I almost certainly missed some of these, but it's a start.
You can preview any changes on my fork.
This is just an idea of how things could be – really looking forward to hearing your feedback. If you like something, we can more more in that direction. If you don't like something, we can change it.

With them separated out you can also use these markdown files to generate a guides page on your website

Updated docs for add_git_tag based on https://github.com/KrauseFx/fastlane/pull/175

Hey Ash,
First off, thank you for working on this. It is super helpful and will help a lot of users.
Introducing the /doc directory is a great idea. I love how you improved and re-worked some poorly written sentences and code examples.
The Actions.md is perfect! I love it :heart_eyes:
I'd like to move the fastlane Toolchain entry from the docs/README.md to the project's README.
While you're at it, you can add a few links to eidolon as real world example setup :+1:
Thanks again for working on this, really appreciated :rocket:

Awesome, glad you feel it fits. Moving the fasten Toolchain back into the main README makes sense. I'll get it pushed up shortly.

I think it's ready to merge, right? Anything missing from your side? :+1:

All good to me!

Thank you so much :+1:


[com_google_fonts_check_STAT_strings] Should "Italic" be allowed for 'slnt' axis?

Observed behaviour
When using "Italic" as axisValue on the 'slnt' axis the check fails. I understand the purpose of this check for other axes but I don’t see a reasong why "Italic" can not be allowed for Slant axis.
Many times Slant axis will not only varies the inclination angle but also introduces changes on the glyphs shape and using other names like "Oblique" does not make sense.

Hi @RosaWagner I think this check is on Universal profile https://github.com/googlefonts/fontbakery/blob/0dd4631a35a31f1df903438a595e62fa589e1eba/Lib/fontbakery/profiles/universal.py#L766

Should we migrate it to Google Fonts profile, then?

See also: https://github.com/googlefonts/fontbakery/issues/2863, where the check was originally discussed/proposed

I think the check has value for the Universal profile, specially because is very often to see fonts compiled with Glyphs with bad STAT tables or where the parameters were not correctly set using AxisValue like "Light Italic" for wght axis.

On the Universal profile I think com.adobe.fonts/check/STAT_strings should be used instead of com.google.fonts**/check/STAT_strings.
In sumary because AF check accepts "italic" Axis Value Name on slnt axis.


ding dong bot demo

@univerone Please feel free to help yourself merge this PR when you feel it's OK.
After this PR has been merged, we can prepare to publish this repo to NPM soon.

ping @univerone
Please take a look at my reviews and let me know what you think, thank you very much!

ping @univerone
Please take a look at my reviews and let me know what you think, thank you very much!

Sorry to reply late !  I'll solve this pr ASAP!

It is fantastic that we can have this PR been merged!
The Whatsapp puppet will help our Wechaty community to get a huge step forward, thank you @univerone for your effort!


"node: command not found" every time terminal opens

Every time I open a new terminal window or start a new process I have to type nvm use --delete-prefix v10.13.0 before node is added to the path.
The README.md says I can use nvm alias default node to set a default node version to be used in any new shell, but this command doesn't do anything. Node is never on the path when a new process starts.

Operating system and version:
Mac OS X High Sierra
nvm debug output:

nvm --version: v0.33.11
$TERM_PROGRAM: iTerm.app
$SHELL: /bin/bash
$SHLVL: 1
$HOME: /Users/jason
$NVM_DIR: '$HOME/.nvm'
$PATH: $NVM_DIR/versions/node/v10.13.0/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/go/bin
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin17)'
uname -a: 'Darwin 17.7.0 Darwin Kernel Version 17.7.0: Wed Oct 10 23:06:14 PDT 2018; root:xnu-4570.71.13~1/RELEASE_X86_64 x86_64'
OS version: Mac 10.13.6 17G3025
curl: /usr/bin/curl, curl 7.54.0 (x86_64-apple-darwin17.0) libcurl/7.54.0 LibreSSL/2.0.20 zlib/1.2.11 nghttp2/1.24.0
wget: not found
git: /usr/local/bin/git, git version 2.10.1
grep: /usr/bin/grep, grep (BSD grep) 2.5.1-FreeBSD
awk: /usr/bin/awk, awk version 20070501
sed: illegal option -- -
usage: sed script [-Ealn] [-i extension] [file ...]
       sed [-Ealn] [-i extension] [-e script] ... [-f script_file] ... [file ...]
sed: /usr/bin/sed,
cut: illegal option -- -
usage: cut -b list [-n] [file ...]
       cut -c list [file ...]
       cut -f list [-s] [-d delim] [file ...]
cut: /usr/bin/cut,
basename: illegal option -- -
usage: basename string [suffix]
       basename [-a] [-s suffix] string [...]
basename: /usr/bin/basename,
rm: illegal option -- -
usage: rm [-f | -i] [-dPRrvW] file ...
       unlink file
rm: /bin/rm,
mkdir: illegal option -- -
usage: mkdir [-pv] [-m mode] directory ...
mkdir: /bin/mkdir,
xargs: illegal option -- -
usage: xargs [-0opt] [-E eofstr] [-I replstr [-R replacements]] [-J replstr]
             [-L number] [-n number [-x]] [-P maxprocs] [-s size]
             [utility [argument ...]]
xargs: /usr/bin/xargs,
nvm current: v10.13.0
which node: $NVM_DIR/versions/node/v10.13.0/bin/node
which iojs:
which npm: $NVM_DIR/versions/node/v10.13.0/bin/npm
npm config get prefix: /Users/Jason/.nvm/versions/node/v10.13.0
npm root -g: /Users/Jason/.nvm/versions/node/v10.13.0/lib/node_modules

- `nvm ls` output:

->     v10.13.0
default -> v10.13.0
node -> stable (-> v10.13.0) (default)
stable -> 10.13 (-> v10.13.0) (default)
iojs -> N/A (default)
lts/* -> lts/dubnium (-> v10.13.0)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.14.4 (-> N/A)
lts/carbon -> v8.12.0 (-> N/A)
lts/dubnium → v10.13.0

When node is not added to the path, what do npm config prefix and which npm print out?

I had this problem with a new laptop the other day.
I was trying to call node before nvm had been initialized - and I don't have node on my default path.
I moved this further up in my .bashrc
 echo loading nvm conf
 export NVM_DIR="$HOME/.nvm"
 [ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh" # This loads nvm

and then I added a .nvmrc file to my home directory.

@maxwellgerber nvm alias default node also works, in lieu of an .nvmrc file.

No, nvm alias default node doesn't do anything. I'm not using an .nvmrc file either.

@jtiscione you should need a prefix set. That implies you have either a prefix line in ~/.npmrc or an env var that’s setting it.

I don't have any ~/.npmrc file. What environment variable is doing it?
If I type npm config delete prefix and then don't set it to something, the "nvm is not compatible with the npm config "prefix" option" error comes right back.
Also, globally installed npm packages don't appear on the path anywhere.

What do you get with env | grep -i prefix?

It doesn't print anything.

Ok - what does npm config get prefix print out, prior to doing anything with nvm?

Well, before doing anything with nvm, npm config get prefix just prints "Command not found".
If I do nvm use --delete-prefix v10.14.1 then npm config get prefix prints "/Users/Jason/.nvm/versions/node/v10.14.1".
I'm having none of these problems on my other laptop... I'll have to compare them side by side.

I think i see the problem. $HOME has “jason” but your other paths have “Jason”. Is there a reason that the capitalization doesn’t match?

Didn't spot that- but this is a Macbook and the path is case insensitive. Commands in there still work regardless of the directory- i.e. /uSeRs/Jason/.nvm/versions/node/v10.14.1/bin/node -v prints "v10.14.1" (if it's there).

Well, on the Mac where it does work, the home folder's case does match- because when I set that machine up a year ago, I renamed the home folder to lower case. (Usually when you set up a mew MacBook the home folder ends up capitalized; you have to create a special System Administrator account from which to rename it.)

That might be the problem - what does whoami print out?

OK; I created another admin user, renamed my home folder from "Jason" to "jason" (since $USER, $PATH, and whoami output all have my name in lower case), logged back in as myself, blew away ~/.nvm, reinstalled nvm from scratch, and now all these weird problems seem to be gone, including the thing with global modules not making it onto the path.
But if this really is just a matter of case sensitivity weirdness for a user's home folder name on a Mac, I wouldn't expect to be the only one running into it.

I agree - i suspect that you have something in your dotfiles that references one capitalization, and you used a different one when first setting up the Mac. The typical case on a Mac is to use the same capitalization from account creation, everywhere, which (if my hunch is right) might be why others aren’t running into it.

I must have set up my work machine a little more carefully than the one under the Christmas tree...

I think we’ve figured out the cause, so I’m going to close this - happy to reopen if there’s more to be done.

I had this problem with a new laptop the other day.
I was trying to call node before nvm had been initialized - and I don't have node on my default path.
I moved this further up in my .bashrc
 echo loading nvm conf
 export NVM_DIR="$HOME/.nvm"
 [ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh" # This loads nvm

and then I added a .nvmrc file to my home directory.

however it make my iTerm2 so slow. is it my only problem?


How to run ALTS?

Dear publisher:
Recently I am trying to reproduce the ALTS model in your paper, but I don't know how to do it.
Could you please explain it in detail? Thanks for your share!

Hi @Arman1zz,
I listed the steps for how to reproduce the paper in the README at https://github.com/DanielVandH/ALTS?tab=readme-ov-file#steps-to-reproduce-the-paper. Can you please explain what might not be clear there? Basically you need to just download the files from the repository (see the green code button in the top-right above the files, click it and then download zip), and then run the files as mentioned in the README.

Hi @Arman1zz,

Where is the main entry of your code? Is it Paper_Code_Data.R?

The code is split into parts as described in the README. First, note that

The algorithms themselves are defined in ALTS_Functions.R.
The functions used for performing the case study are given in Case_Study_Functions.R.
Some other functions for running the simulations are in Simulation_Functions.R.

For reproducing the actual results, you do it in two steps:

Firstly, Paper_Code_Data.R is used to obtain the results for the paper, including the simulation study. So, yes, this is the main entry point.
Code for reproducing the figures is in Paper_Code_Figure.R. Use this if you want to see how all the data is used.

To run all this, open R with all these files in your working directory and it should just work. Note that (as far as I remember from when I last ran this code) Paper_Code_Data.R can take a bit of time to complete.

If I want to run your code with my own dataset(the same base model GEFC2012 proposed by Hong), how do I fit it with my data?

See the ALTS_Functions.R script. Specifically, the ALTS function does the model fitting. In that file it is hopefully sufficiently documented for you to understand how to use it. For example, in the case study, the function random_attack_ALTS on L122 of Case_Study_Functions.R works fits the data using our ALTS method in lines 125-131, using the dataset new_data (which is just the data you mention except with some attacked data) (this data, and the model formula, are defined in Paper_Code_Data.R, see L65-L135).
Hope that helps.

Is it Paper_Code_Data.R? I failed to run the Paper_Code_Data.R.

I saw that you added this while I was making my response but hopefully my point

To run all this, open R with all these files in your working directory and it should just work. Note that (as far as I remember from when I last ran this code) Paper_Code_Data.R can take a bit of time to complete.

will help address whatever error you saw. Otherwise, send the error so I can see what's going wrong

Hi@DanielVandH ,
Thank you very much for your patient and complete reply.
I'll take your advice well.
Thank you. Have a good life!

No problem, happy to help. I'll close this issue for now but feel free to ask if there are still problems.

Hi@DanielVandH ，
Errors occurs when I run the Paper_Code_Data.R.
More precisely, errors occurred in "Economic loss study"/ "Economic loss study: Smaller mu, larger sigma"/ "Economic loss study: Smaller mu, larger sigma"......
The errors from these code blocks all point to the function — "random_attack_all_simulations_grid_vals". And the same issue is also present in function ramp_attack_all_simulations_grid_vals.
My execution process is as follows:

Setwd workspace and Load the functions and libraries. (line 4-24) — success
Executive "Read in the case study data". (line 65-135) — success
Executive "Economic loss study". (line 140-149) —error

The error message is as follows: stop("'arg' must be of length 1")

I'll look into this soon


Add to F-Droid

Since the client is already "production ready" how do you feel about inclusion in F-Droid?

Hi there! Given the security requirements of Accrescent and the fact that it can update itself, I'm not interested in publishing it on alternative stores.

But for the first installation it would be nice to get it on f-droid. Also more people could notice it that way.

But for the first installation it would be nice to get it on f-droid.

You would still need to manually update it once from accrecent (and then never again from f-droid) for unattached updates to work.
For self updater to work at all it must be signed by the signing key of accescent not f-droid.

Maybe it makes more sense to add the apk from GitHub Release to Izzys repo?

Also more people could notice it that way.

Maybe it makes more sense to add the apk from GitHub Release to Izzys repo?

He's already addressed the option of putting it on any alternative store.

I'm not interested in publishing it on alternative stores.

https://github.com/accrescent/accrescent/issues/139#issuecomment-1868501539

For self updater to work at all it must be signed by the signing key of accescent not f-droid.

Use reproducible builds then. Allows you to use your own signing key instead of F-Droid's.


DownloadLocation is not the true location for downloads

Bug Report
Overview

By default, the Rest utility will download files, however, they are stored in a private "sub folder" (download_cache) and not in the actual download folder.
The correct location is reported by the API when the file is downloaded, but if an app wants to check if the downloaded files exist before downloading them again, it cannot as the download_cache is a hidden folder and the files do not exist in the actual downloaded folder.
To Reproduce
Steps to reproduce the behavior:

Leave the DownloadLocation as the default
Download a file
Check the folder set by the DownloadLocation
See no content only a new subfolder which contains the content.

Expected behavior

If a DownloadLocation is set, the files should be locatable in that folder using the "DownloadLocation" property

I don't understand how this is a problem if using the DownloadCacheDirectory.

I'll update the docs to be explicit about this quirk


Missing wavefront-spring-boot-bom dependency

org.springframework.cloud:spring-cloud-sleuth-otel-autoconfigure:jar:1.0.0-M7 uses com.wavefront:wavefront-spring-boot-bom:pom:2.1.1-SNAPSHOT but the wavefront version doesn't exist in maven repository
Error Stack:
Failed to collect dependencies at org.springframework.cloud:spring-cloud-sleuth-otel-autoconfigure:jar:1.0.0-M7: Failed to read artifact descriptor for org.springframework.cl
oud:spring-cloud-sleuth-otel-autoconfigure:jar:1.0.0-M7: Could not find artifact com.wavefront:wavefront-spring-boot-bom:pom:2.1.1-SNAPSHOT
Failed to collect dependencies at org.springframework.cloud:spring-cloud-sleuth-otel-autoconfigure:jar:1.0.0-M7
at org.apache.maven.lifecycle.internal.LifecycleDependencyResolver.getDependencies (LifecycleDependencyResolver.java:269)
at org.apache.maven.lifecycle.internal.LifecycleDependencyResolver.resolveProjectDependencies (LifecycleDependencyResolver.java:147)
at org.apache.maven.lifecycle.internal.MojoExecutor.ensureDependenciesAreResolved (MojoExecutor.java:248)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:202)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
We just added the sleuth otel dependencies and running mvn clean install. Where can I find the wavefront-spring-boot-bom:pom:2.1.1-SNAPSHOT dependency?  does this exist in any maven repo?

It's in repo.spring.io/snapshot however that's not good that we depend on snapshots. We'll need to fix this @jonatan-ivanov

Theoretically wavefront will release 2.2.0 tomorrow, I' going to update pom after their release.

2.1.1 was released (instead of 2.2.0) but it makes the build fail (2.1.1-SNAPSHOT does this too). I went back to the latest snapshot: 2.2.0-SNAPSHOT, we can upgrade once 2.2.0 is released.


Generate standalone TAGS.md

The current categories are great, and I'd like to enhance it by generating a standalone TAGS.md file to help users navigate the plugins.
Proof of concept (Python script):
import re
import requests

idx = 0;
with open("TAGS.md", mode="w") as target:
    with open("README.md", mode="r+") as f:
        for line in f.readlines():
            if idx == 5:
                break
            if line == "## External":
                break
            if line[0] == "-" and "github.com" in line:
                github_url = re.search("(?P<url>https?://github.com[^\s)#]+)", line).group("url")
                user = github_url.split("/")[3]
                repo = github_url.split("/")[4]
                target_api_url = f"https://api.github.com/repos/{user}/{repo}"
                req = requests.get(target_api_url, headers={"Accept": "application/vnd.github.mercy-preview+json"})
                topic_list = req.json()["topics"]
                if len(topic_list):
                    target.write(line)
                    for topic in topic_list:
                        target.write(f" [#{topic}](https://github.com/topics/{topic})")
                    target.write(".\n")
                    idx += 1

Proof of concept (generated TAGS.md file)

I don't think this will add much, everything will just be#lua, #neovim, #neovim-plugin etc.

I can add a few lines to filter out those generic tags.
It’s up to the maintainers, so feel free to close this issue.

That's true, if you can also make it a GitHub action that automatically adds it when a new line is added to a PR I'm fine with it.

Got it. I'll start working on the GitHub Actions part.

@3719e04 Any progress on this or should I close this issue?


users-groups: fix typo

Fix typo in the users.users.<name>.packages option description.
No tests/confirmations run because just a one-word typo fix in documentation.

thanks!


Save temporary rubocop file as binary, closes #167

See #167
Not exactly using line endings from the HAML (this would be the perfect solution). But at least we won't have to disable Style/EndOfLine to get rid of the false positives where rubocop reports CRLF offenses while the HAML source only contains LFs.

Thanks for digging into this issue, @agross.
I'm a little hesitant to start writing files in binary mode because that causes the content to be treated as ASCII-8BIT. This could potentially cause problems with HAML files containing unicode characters.
Rather than open up a potential can of worms, since the Style/EndOfLine cop really doesn't make sense to keep enabled for HAML documents (HAML-Lint should check line endings itself), can you change this pull request to simply disable the cop instead?
If you think I'm missing something that your solution also addresses, please let me know. Thanks!

Hi @sds, that's all valid points. I changed the PR to just disable Style/EndOfLine. What's a bit sad is that I now can't really test it. Rubocop for some reason doesn't report the offense in the test, even when run on Windows. I left it though for documentation. Let me know whether that's OK with you!


Is async_mqtt5::async_publish thread-safe ?

Hi !
When I call async_publish, the method below is invoked. Is this correct?
Is it thread-safe to use more than one thread for the I/O context?
https://github.com/mireo/async-mqtt5/blob/d52090f438906976c864698def2413b9d4ee097b/include/async_mqtt5/impl/async_sender.hpp#L228-L290
Pozdrav :) ,
Amar

Hi,
Asio-compliant objects (like our mqtt_client) are not thread-safe. This is intentional to avoid the overhead associated with mutexes and other synchronisation mechanisms in single-threaded applications.
Regardless, you can use more than one thread to run io_context. In that case,  you should make sure that the mqtt_client's functions are executed within the strand (an executor that guarantees that tasks assigned to it are executed in a serialised manner, preventing concurrent execution).
You can achieve that by:

constructing the mqtt_client with a strand object as its executor
dispatching all mqtt_client's function calls to that strand using post  or dispatch
ensuring that the async_xxx's completion handler's associated executor is that same strand

This is how you would call async_publish in a multi-threaded environment:
  boost::asio::strand strand = boost::asio::make_strand(ioc.get_executor());
  async_mqtt5::mqtt_client<boost::asio::ip::tcp::socket> client(strand);
  client.brokers("<your-mqtt-broker>", 1883);
  boost::asio::post(
	  strand,
	  [&client, &strand] {
		// Considering that the default associated executor of all completion handlers is the strand,
		// it is not necessary to explicitly bind it to async_run or other async_xxx's handlers.
		  client.async_run(boost::asio::detached);
	  }
  );
  boost::asio::post(
	  strand,
	  [&client, &strand] {
		  client.async_publish<async_mqtt5::qos_e::at_least_once>(
			  "<your-mqtt-topic>", "Hello world!", async_mqtt5::retain_e::no,
			  async_mqtt5::publish_props {},
			  [&client, &strand](
				  async_mqtt5::error_code ec, async_mqtt5::reason_code rc,
				  async_mqtt5::puback_props props
			  ) {
			  }
		  );
	  }
  );

Now, you can freely use as many threads as you'd like for your io_context and not worry about any concurrency issues.
Additionally, we have thoroughly covered this topic in our documentation. I'd recommend giving it a glance.
Hope this helps!


feat: add rust-feature to build without including ffmpeg and/or livekit

Codecov Report
Attention: 25 lines in your changes are missing coverage. Please review.

Comparison is base (fb785e6) 49.59% compared to head (523da7b) 47.37%.

Files
Patch %
Lines

...-godot-lib/src/scene_runner/components/material.rs
10.00%
9 Missing :warning:

...t/decentraland-godot-lib/src/scene_runner/scene.rs
0.00%
7 Missing :warning:

...ot-lib/src/scene_runner/components/video_player.rs
0.00%
5 Missing :warning:

...land-godot-lib/src/scene_runner/godot_dcl_scene.rs
40.00%
3 Missing :warning:

...aland-godot-lib/src/comms/communication_manager.rs
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #170      +/-   ##
==========================================
- Coverage   49.59%   47.37%   -2.23%
==========================================
  Files         139      139
  Lines       16841    16848       +7
==========================================
- Hits         8352     7981     -371
- Misses       8489     8867     +378

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


unable to run snakemake

I could  run ./foec_setup.sh -g /home/microbiology/FoEC2/16 and got the genome config file
Then i tried as you said and got error as below.
snakemake --use-singularity --use-conda --cores 6
Building DAG of jobs...
WARNING: underlay of /etc/localtime required more than 50 (74) bind mounts
WorkflowError:
Unable to find environment in container image. Maybe a conda environment was modified without containerizing again (see snakemake --containerize)?
Details:
Command 'SINGULARITYENV_CONDA_PKGS_DIRS=/tmp/conda/e7040f71-b17c-43e6-88f6-846cb42aa18c singularity --quiet --silent exec --home /home/microbiology/FoEC2  /home/microbiology/FoEC2/.snakemake/singularity/29c46ba7a831b1ad5bf4a5e5d5165eb7.simg sh -c '[ -d '''/conda-envs/50ba960368883e5775eedd03e35da544''' ]'' returned non-zero exit status 1.
WARNING: underlay of /etc/localtime required more than 50 (74) bind mounts
pls help me to solve the issue

Hi! I noticed that you are running the pipeline with --use-singularity. Could you try removing that, and just run the command as shown in the readme? https://github.com/pvdam3/FoEC2#usage
snakemake --use-conda --cores [N]


Make the sync feature as a default feature

Follow https://github.com/meilisearch/meilisearch-rust/issues/166
Check with @Kerollmops. We think it's better to use Arc everywhere instead of Rc to make the user simpler for everyone.
Even if Arc are sightly less performant than Rc

this package is only an HTTP client, so the most performance issues are coming from the network
it eases the usage for everyone
it's common to do this in Rust

This is already implemented under the sync feature. This feature replaces all the Rcs with Arcs. However, it is not enabled by default and that would indeed be better.

Ok, I understand better, I updated the title of the issue

I'd like to give this a try

Thank you @sccalabr

Closed by #179


Revise groupversionaliaskind

Please provide a description of this PR:
Make it more easier to use to contain both the original GVK and its alias.

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
07a73dcfcd1f706030cb773d0e10f7f3bb1c93bf
link
true
/test integ-security-multicluster_istio

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

/retest-required


App folder (sandbox) access attempt failed

For some reason I can't get the the client.ls command to use the mode I've set with Dropbox::API::Config.mode, it keeps returning the following error message:

1.9.2p290 :017 > client.ls
Dropbox::API::Error: App folder (sandbox) access attempt failed because this app is not configured to have an app folder.  Should your access type be 'dropbox' instead?
	from /Users/amotion/.rvm/gems/ruby-1.9.2-p290/gems/dropbox-api-0.2.0/lib/dropbox-api/connection/requests.rb:22:in `request'
	from /Users/amotion/.rvm/gems/ruby-1.9.2-p290/gems/dropbox-api-0.2.0/lib/dropbox-api/connection/requests.rb:41:in `get'
	from /Users/amotion/.rvm/gems/ruby-1.9.2-p290/gems/dropbox-api-0.2.0/lib/dropbox-api/client/raw.rb:26:in `request'
	from (eval):3:in `metadata'
	from /Users/amotion/.rvm/gems/ruby-1.9.2-p290/gems/dropbox-api-0.2.0/lib/dropbox-api/objects/dir.rb:9:in `ls'
	from /Users/amotion/.rvm/gems/ruby-1.9.2-p290/gems/dropbox-api-0.2.0/lib/dropbox-api/client.rb:25:in `ls'
	from (irb):17
	from /Users/amotion/.rvm/gems/ruby-1.9.2-p290/gems/railties-3.2.0/lib/rails/commands/console.rb:47:in `start'
	from /Users/amotion/.rvm/gems/ruby-1.9.2-p290/gems/railties-3.2.0/lib/rails/commands/console.rb:8:in `start'
	from /Users/amotion/.rvm/gems/ruby-1.9.2-p290/gems/railties-3.2.0/lib/rails/commands.rb:41:in `'
	from script/rails:6:in `require'
	from script/rails:6:in `'

But when I check in the console it seems the mode is set correctly:

1.9.2p290 :016 > Dropbox::API::Config.mode
 => "dropbox"

See anything obvious I might be missing?

I am having the same issue.
ruby-1.9.2-p0 > Dropbox::API::Config.mode
=> "dropbox"

ruby-1.9.2-p0 > client.ls
Dropbox::API::Error: App folder (sandbox) access attempt...

Access type is set to "Full Dropbox".

mattkanwisher's fork did the trick!


want to help maintain markdown-toc?

Thanks to some amazing sponsors like @ImZeus_XYZ I'm able to start spending more time with open source again. Drop a comment below if you want to help get this project in up to date.
Here are some reasons why this is beneficial to you:

This library is downloaded 6m times/mo. and gives you an opportunity to get experience maintaining a project that many people depend on
Companies love hiring developers who collaborate on open source projects
Your contributions would be warmly appreciated by me and the community

Hi @jonschlinkert, would love to help out. I've been using this for my personal site for a while, and tbh, got motivated to fix it because it was spewing newlines all over my articles every time a file was touched. Fixed that in #190!
Let me know what needs help. I was going through adding tests for issues that appear to be already fixed but never closed.

Hello @jonschlinkert !
I've opened a PR #192 that should fix 3 old important issues.
Would you like to check it out and possibly merge it?
Thank you for your work :)!


Command is missing the path to the cks directory for the model.ckpt-28252.pt

When I run the command
python convert_tf_checkpoint.py --tf_checkpoint /cks/model.ckpt-28252.index --bert_config_path /cks/bert_config.json --output_checkpoint model.ckpt-28252.pt
It works as expected. However, it writes the needed file (model.ckpt-28252.pt) to /workspace/bert. When I exit the container the file is deleted as well. The correct command should be
python convert_tf_checkpoint.py --tf_checkpoint /cks/model.ckpt-28252.index --bert_config_path /cks/bert_config.json --output_checkpoint /cks/model.ckpt-28252.pt

you didn't attach the local director to the container
use: docker run -it -v localdiretory:/csk this should to the job

ok I see the command is misleading:
python convert_tf_checkpoint.py --tf_checkpoint /cks/model.ckpt-28252.index --bert_config_path /cks/bert_config.json --output_checkpoint model.ckpt-28252.pt

should be
python convert_tf_checkpoint.py --tf_checkpoint /cks/model.ckpt-28252.index --bert_config_path /cks/bert_config.json --output_checkpoint /cks/model.ckpt-28252.pt

https://github.com/mlcommons/training_results_v1.0/compare/master...jqueguiner:patch-1


Update o2tvgo.py

Fix for "invalid OAuth token" error

Unfortunately, I cannot test it. I trust you ;-)


Improve Clarity of syncHandler Comment in Sample Controller

Overview
This PR proposes a minor yet significant update to the comment of the syncHandler function in the sample-controller. The aim is to enhance clarity and reduce potential ambiguity for developers who might not be deeply familiar with the Kubernetes controller pattern.
Changes
The original comment used the term "converge" to describe the process of aligning the actual state of a resource with its desired state. However, this term can be somewhat ambiguous, especially for newcomers or those not intimately familiar with Kubernetes' declarative approach. The proposed change in this PR replaces "converge" with a more explicit phrase to clarify the intended operation of the syncHandler function.
Original Comment
// syncHandler compares the actual state with the desired, and attempts to
// converge the two. It then updates the Status block of the Foo resource
// with the current status of the resource.

Updated Comment
// syncHandler compares the actual state with the desired, and attempts to
// align the first to the latter. It then updates the Status block of the
// Foo resource with the current status of the resource.

Rationale
The adjustment in the comment aims to make the description more accessible and straightforward, particularly for those who might be learning about Kubernetes controllers for the first time. This change is in line with best practices for code documentation, where clarity and precision are crucial for effective communication and understanding.
Impact
The change is limited to the comment and does not affect the functionality of the code. It is intended purely for educational and clarity purposes, enhancing the overall quality of the documentation in the sample-controller.

To be fair, this may be an overkill but the use of "converge" has bothered me to this point. The original comment in code uses "converge" while implicitly assuming that the reader would infer that the conversion point is at the desired state, which is very fair, yet still ambiguous.
Thank you for reading :joy:


check out the error in build.

(Emitted value instead of an instance of Error) Cannot find source file 'MutationObserver.js': Error: Can't resolve './MutationObserver.js' in '/Users/dong/iotex/iotex-explorer/node_modules/mutationobserver-shim/dist'

repro:
npm run start
check this error in list.

if it dos not affect at all, please close.


When querying metrics using Azure Monitor Query, specifics filters have to be used instead of getting data from all dimensions

Whenever querying metrics with Azure Monitor Query, the metric query must contain filters with specific dimensions to get accurate information.
When using the Portal there is no need of doing filtering 1 by 1, since the user can either apply a filter or use the split feature or both to see the multiple results from different dimensions.

Supporting splitting by <dimension_name> would make things easier since for example, in my team's current project the dimension is called process_id and there are more than 25 different process ids that I should know the name beforehand to query them 1 by 1 with multiple filters.
Adding the split feature would make things easier:

the user wouldn't need to know the dimension names beforehand
one query would replace one query per dimension name

Thanks for the feedback, we’ll investigate asap.

Hi @esttenorio  The split feature can be achieved in the filter by setting the value to * - for example, like "City eq '*'"
Let me know if you have any questions I'm keeping this issue open to update documentation/ sample.


Show metrics from github

If the package use github, try to get metric (star, fork, issue count) and display them in the package detail page

:+1: Solved, now with sync metrics from github and even use them in the search query sorting


Fixed footer for all screen sizes

Fixes : #649

@MastanSayyad
You will have to assign this PR to me else it would not get counted in my contribution to GSSOC
so kindly assign

Hey @Utsavladia leaderboard points are only counted when the merged PR has gssoc and level lables so it will not affect your points even if the PR isn't assign to you!


Version 0.4.1

Change-Id: I83e8d325e6775037f5ad500fe5bd03cbd012055c

@wolffcaleb this pull request is slightly confusing to me.  I already see the 0.4.1 changes reflected in the master branch.  Do you not see the same?

Going to close this, please re-open if I am misunderstanding something.

My apologies! I honestly am just trying to figure out Github. I was trying to get a piece of the code so I could test it and I figured a pull request would allow me to pull the code to my console. Could you explain to me how to utilize this code?
I would truly appreciate it!
Caleb

@wolffcaleb ah I think this is just a terminology issue.  Pull Request is what you'd submit if you wanted to add a patch to this repository.
I think you want to clone the code, which you can do with git clone https://github.com/firebase/FirebaseUI-Android

Greatly appreciated!
Thank you!
Caleb


Add Rapid Dominance to list of turn based strategy games

Note: I am the author

Sorry, but I cannot add every small game to the list.


Monitor the responsiveness of the heartbeat thread

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

Current dependencies on/for this PR:

master

PR #170   👈

This stack of pull requests is managed by Graphite.


Recognize "audio" property of h-entry

It'd be great to recognize the "audio" property and set it as a media enclosure, which would be one step closer to being able to turn an h-feed into a podcast feed!
Example feeds:

https://aaronparecki.com/tag/100daysofmusic
https://aaronparecki.com/tag/this-week-podcast

Note that u-audio is already an official Proposed Addition to h-entry: http://microformats.org/wiki/h-entry#Proposed_Additions and one more consumer (e.g. granary in addition to p3k) would help bump it from Proposed to Draft!
More: https://indieweb.org/audio

yes!

done! e.g.: https://granary-demo.appspot.com/url?input=html&output=atom&url=https%3A%2F%2Faaronparecki.com%2Ftag%2Fthis-week-podcast


chore: parameterize Synth

same as: #2435
but it uses jit::Result<Value>

waiting for https://github.com/tailcallhq/tailcall/pull/2442 to get merged


Swapping system theme on macOS breaks application color scheme

Version and Platform (required):

Binary Ninja Version: 3.1
OS: MacOS
OS Version: >=10.14 (Mojave)
CPU Architecture: *

Bug Description:
When swapping the system color theme (Dark -> Light or Light -> Dark), Qt tries to automatically swap color pallets for the program, but is only partially successful, resulting in a really strange, broken-looking BinaryNinja
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

Open BinaryNinja
Swap system theme in macOS' Settings panel

Additional Info:
Not reproducible on Windows 10 or 11 by swapping system theme.

No longer breaks when switching themes as of build 3567.
The theme will not be updated when changing system themes. See bug #1909 for the automatic switching feature.


Have you considered collaboration with other Open Projects?

As good as open formats are, a single implementation will always be far from ideal.
Have you considered reaching out to the communities at Inkscape, Penpot etc?
Also, slightly off topic, is .free a really recognizable name, why not i.e. .luna or .fui (free ui) .lui (lunacy ui) etc
Thanks for moving over from .sketch though!
Looking forward to more openings in the future! ^^
Would you consider showing up in FOSDEM 2025? 👀

Hi, thanks for your questions!
We will not push anyone to support FREE format. Also we will not advertise it in the near future. We think that format is already pretty good, but still need some time to be well cooked. We need to verify that we like it as it is, did no mistakes, and don't want to make breaking changes in future. Format should be rock stable. And when it will - we will advertise it.
If anyone will be interested in support of FREE format - we are open for communication and extension of format.
There is a lot of things is not described in a FREE format. For example - prototyping is poor, extensions metadata support is not yet considered, etc. Also we are open for criticism.
About name - we don't want to make format name to be highly related to Lunacy. Also we want format to have a nice, memorable, easy, best name. We hope that this will improve chances of getting format to be known, and supported in other apps and services.
About FOSDEM - Lunacy is not open source. Why? So, I think no.


Update README

Remove bintray reference as it's no longer in existance and artifacts are on maven central
Update branch references for builds and coverage

Coverage decreased (-0.007%) to 93.849% when pulling 2c782b8ce8aa2ca0aa2410d1d26dd3129c1e4c12 on tgianos:updateReadme40x into cf234e9de5bb6e54b81ea0d19762b77e6c0a9a1f on Netflix:4.0.x.


Crash after 5 seconds

micro is crashing when editing a commit message for git. git will open up micro with the default commit text/comments. After approximately 5 seconds micro crashes out. This happens without entering a single keystroke.
This is what I've been able to salvage from the terminal.
goroutine 12 [running]:
main.main.func6()
/home/zbyedidia/gopath/src/github.com/zyedidia/micro/cmd/micro/micro.go:483 +0xc6
created by main.main
/home/zbyedidia/gopath/src/github.com/zyedidia/micro/cmd/micro/micro.go:480 +0x170d

The status line looked like this before the crash, (if that helps).  /home/sbe/stardust-mycloud/.git/COMMIT_EDITMSG (1,1) unknown unix
sbe@simon_ubuntu:~$ mic --version
Version: 2.0.1
Commit hash: 7c71995
Compiled on February 12, 2020

Linux amd64 static build.
OS: Ubuntu 18.04.3 LTS
Terminal: Extraterm 0.49.1

Can you reproduce this using another terminal emulator or an older micro version?

micro 2.0.0 had the same problem yesterday. Older versions of micro didn't have this problem.
I forgot to mention that I'm connected to this Linux (virtual) machine from Windows over ssh.

That error is a bit strange since line 480 doesn’t exist in micro.go (the file is only about 350 lines long).

lmao

I've got more of the message now.

 Pre-2.0 micro is actually running here and it crashes on the updated config files.
thanks for looking at this issue so quickly though.

lmao


Does this work in browsers?

May i add my comment here?
I don't use and never used npm. How to make it work just as an ordinary js, by including it into jsbin for test?


Adds missing include and avoids unused variable

This commit fixes two small issues:
IAMF_decoder_private.h uses IAMF_extradata and IAMF_StreamInfo but does not include IAMF_decoder.h (And it does not appear to be transitively included, so this commit adds the include.
In IAMF_decoder.c, if binauralizer is disabled, there is a unused var ctx.  This PR moves it inside the code block where it's used to avoid that error.

@yilun-zhangs, @hkchung72 - this is just a small change to fix some nits building it in different environments. Can you please review.


crash of mitmdump as result of bad cookie

What's the problem?
I recently see mitmdump crashing with errors like
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/server.py", line 120, in handle
    root_layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/modes/transparent_proxy.py", line 21, in __call__
    layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/tls.py", line 273, in __call__
    layer()
TypeError: 'NoneType' object is not callable

mitmproxy has crashed!Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/server.py", line 120, in handle
    root_layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/modes/transparent_proxy.py", line 21, in __call__
    layer()
TypeError: 'NoneType' object is not callable

Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/server.py", line 120, in handle
    root_layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/modes/transparent_proxy.py", line 21, in __call__
    layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/tls.py", line 273, in __call__
    layer()
TypeError: 'NoneType' object is not callable
mitmproxy has crashed!
mitmproxy has crashed!

Please lodge a bug report at: https://github.com/mitmproxy/mitmproxyPlease lodge a bug report at: https://github.com/mitmproxy/mitmproxy

Please lodge a bug report at: https://github.com/mitmproxy/mitmproxy
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/server.py", line 120, in handle
    root_layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/modes/transparent_proxy.py", line 21, in __call__
    layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/tls.py", line 273, in __call__
    layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/http.py", line 162, in __call__
    layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/http.py", line 367, in __call__
    self.get_response_from_server(flow)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/http.py", line 485, in get_response_from_server
    if flow.response.stream:
AttributeError: 'NoneType' object has no attribute 'response'
[...]

mitmproxy has crashed!
Please lodge a bug report at: https://github.com/mitmproxy/mitmproxy
Traceback (most recent call last):
  File "/usr/local/bin/mitmdump", line 3, in <module>
    mitmdump()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/main.py", line 99, in mitmdump
    master.run()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/dump.py", line 342, in run
    return flow.FlowMaster.run(self)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/controller.py", line 122, in run
    self.tick(self.masterq, 0.1)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/flow.py", line 821, in tick

  File "/usr/local/lib/python2.7/dist-packages/libmproxy/controller.py", line 108, in tick
    self.handle(*msg)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/controller.py", line 129, in handle
    m(obj)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/dump.py", line 314, in handle_request
    flow.FlowMaster.handle_request(self, f)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/flow.py", line 1003, in handle_request
    return f
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/flow.py", line 914, in process_new_request
    if not pb:
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/flow.py", line 801, in do_server_playback
    flow.reply(response)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/models/http.py", line 396, in refresh
    c.append(self._refresh_cookie(i, delta))
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/models/http.py", line 355, in _refresh_cookie
    c = Cookie.SimpleCookie(str(c))
  File "/usr/lib/python2.7/Cookie.py", line 576, in __init__
    if input: self.load(input)
  File "/usr/lib/python2.7/Cookie.py", line 629, in load
    self.__ParseString(rawdata)
  File "/usr/lib/python2.7/Cookie.py", line 662, in __ParseString
    self.__set(K, rval, cval)
  File "/usr/lib/python2.7/Cookie.py", line 582, in __set
    M.set(key, real_value, coded_value)
  File "/usr/lib/python2.7/Cookie.py", line 457, in set
    raise CookieError("Illegal key value: %s" % key)
Cookie.CookieError: Illegal key value: f_?RK<
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/server.py", line 120, in handle
    root_layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/proxy/modes/transparent_proxy.py", line 21, in __call__
    layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/tls.py", line 273, in __call__
    layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/http.py", line 162, in __call__
    layer()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/http.py", line 367, in __call__
    self.get_response_from_server(flow)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/http.py", line 485, in get_response_from_server
    if flow.response.stream:
AttributeError: 'NoneType' object has no attribute 'response'
[...]

What's expected?
mitmdump should not crash but handle the problem.
Which versions are affected?
The master version of mitmdump from yesterday was used.
How to reproduce the problem?
I'm sorry but I do not know exactly how to produce this. I think it should be enough to set up a small server, e.g. with python, that sends a "bad" cookie. Now running mitmdump as a transparent proxy and capturing the data. Afterwards the crash should occur at the replay. But I had not the time to test this.
Why does this happen?
In my captured files there seems to be a bad cookie from match.rtbidder.net/match?[...]. I do not know if this is against any convention but it crashed mitmdump as a result of a Cookie.CookieError.
After modifying mitmdump to log every Cookie in libmproxy/models/http.py _refresh_cookie, it was clear that the following shortened line lead to the error:
uid=Kfyl<1f>]f_?RK<=x__<5/C#j>/KP8>=[...]mC72; path=/; domain=.rtbidder.net; expires=Thu, 15-Sep-2016 [...]
How could the problem be solved?
I modified _refresh_cookie in a way that everything expect the return is covered by a try except block for Cookie.CookieError. If the error occurs the original parameter str(c).strip() is returned. I know that the expiration time is not modified but I thought this is better than a crash.

Just trying to fix this - can you post the full cookie or a subset that triggers an exception when calling Cookie.SimpleCookie("cookievalue")?

Nevermind, got it - ">=A" does the job.


disappearing tx link when there's a message saying Transaction taking too long?

Redundant issue #33


Update libinjection & Mbed TLS

what
Update third-party dependencies (included in others directory): libinjection & Mbed TLS.
why
The versions included in ModSecurity have not been updated in a while:

libinjection currently references commit libinjection/libinjection@bfba51f from Dec, 2020.

This PR references the current head of the library (libinjection/libinjection@b9fcaaf), as the project isn't currently being tagged (the last one is from 2017).
Even though the project hasn't move a lot since then, there are at least a couple of changes that may be interesting:

libinjection/libinjection@a28aceec29932d216f487b09882667479de717ed
libinjection/libinjection@98cf343b0c94b7e9a94aadcaa94ef779f7e16633

A subset of Mbed TLS source code from (at least) Nov, 2016 is included in the ModSecurity repository.

This PR references the latest tag of the library (3.6.0, Mbed-TLS/mbedtls@2ca6c28) and instead of including their source code, uses a submodule (which makes it clear the version being used and simpler to update in the future).
A few interesting changes in the referenced subset of the library (base64, md5 & sha1) since the last update (all of them may not be relevant in ModSecurity usage):

Mbed-TLS/mbedtls@ab043350525833314b5a2e6d0d2efcd6510a8142
Mbed-TLS/mbedtls@1cc1fb05999aea8067e11f5c4f4fdb32dbe91036
Mbed-TLS/mbedtls@d1c98fcf5e04b8f6628ca951131aa1f7d67283de
Mbed-TLS/mbedtls@0963e6cfac2230d68c6ed1aa220ac41f096796ff

We should indeed update these libraries from time to time.
This PR dynamically uses the last version (master), right?
It seems they don't provide a release version. Do you (and other people contributing to this repo, I some some other ones) consider the master version as always stable?
Btw, the same should be done for v2 as well.

This PR dynamically uses the last version (master), right? It seems they don't provide a release version. Do you (and other people contributing to this repo, I some some other ones) consider the master version as always stable?

In both cases the PR references a specific commit hash in order to pin down the version included in libModSecurity until a new review and update is done.
With regards to libinjection, as the project is not currently tagged (the last tag is from 2017), I'm referencing the current head (libinjection/libinjection@b9fcaaf). Mbed TLS references the commit for tag 3.6.0 (git submodules don't currently support explicitly referencing a tag), Mbed-TLS/mbedtls@2ca6c28.

Hi @eduar-hte,
thanks again for this great PR!
Fortunately @fzipi is one of the maintainer/developer of libinjection project so I tagged him - may be he can help us.

Thanks @eduar-hte for this great work again.
A reminder to myself: add git submodule init and git submodule update commands to README.md.

What you normally do is use the checkout actions like this:
      - name: Checkout
        uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
        with:
          submodules: recursive

That will bring you what you want.

So, we should replace the hard-coded files in v2 with submodules as well, right?

What you normally do is use the checkout actions like this:
(...)

That's right, the submodules option simplifies checkout of the repository and also its submodules. However, I found out that the git describe command would not work when the submodules are initialized that way, so I replaced that (arguably simpler approach) with an additional step to do those steps manually as a workaround.
If you take a look at the build with the workaround, you'll see that the 'configure' step shows version info on MbedTLS & libInjection correctly:
 Mandatory dependencies
   + libInjection                                  ....v3.9.2-92-gb9fcaaf
   + Mbed TLS                                      ....v3.6.0
   + SecLang tests                                 ....a3d4405

However, if you look at an older build, you'll notice that the version information is missing:
 Mandatory dependencies
   + libInjection                                  ....
   + SecLang tests                                 ....a3d4405

Oh, for describe you might also want to use depth: 0

You're right, fetch-depth: 0 works! (I thought I had tested that before going for the workaround 🤷♂)
I'll create a simple PR to simplify the workflow again, thanks.


feat: Adding DPDK support in UPF

Description
This PR adds DPDK support to UPF.
Most important changes:

NADs creation has been refactored to support vfioveth type required by DPDK
StatefulSet network annotations creation has been refactored
NetworkAnnotation dataclass has 2 additional fields required by DPDK: mac and ips
DPDKStatefulSetUpdater has been added to handle StatefulSet updates required to run UPF in DPDK mode
New charm configs added

Design decissions:

DPDKStatefulSetUpdater is not a library, because at this point we are not aware of anyone else who could potentially leverage such a library. Regular class seems good enough at this point.
DPDKStatefulSetUpdater makes bess container a privileged container. That's because in DPDK mode, bess needs to access physical address of HugePages. Ideally IPC_LOCK capability should be enough, but it's not. Similar issue has been described here.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that validate the behaviour of the software
[x] I validated that new and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have bumped the version of the library

After deploying the charm with DPDK mode, bessd container can not start with following error.
2023-11-28T19:58:45.592Z [pebble] Service "bessd" starting: /bin/bessd -f -grpc-url=<IP_ADDRESS>:10514
2023-11-28T19:58:45.706Z [bessd] F1128 19:58:45.706557    53 bessd.cc:211] flock(pidfile=/var/run/bessd.pid): Permission denied [13]
2023-11-28T19:58:45.706Z [bessd] *** Check failure stack trace: ***
2023-11-28T19:58:45.727Z [bessd] F1128 19:58:45.727789    53 debug.cc:407] Backtrace (recent calls first) ---
2023-11-28T19:58:45.727Z [bessd] (0): /bin/bessd(_ZN6google15ErrnoLogMessageD1Ev+0xae) [0x55de3962323e]
2023-11-28T19:58:45.727Z [bessd] (1): /bin/bessd(_ZN4bess5bessd21TryAcquirePidfileLockEi+0x163) [0x55de38cf6863]
2023-11-28T19:58:45.727Z [bessd] (2): /bin/bessd(_ZN4bess5bessd19CheckUniqueInstanceERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE+0x49) [0x55de38cf6989]
2023-11-28T19:58:45.727Z [bessd] (3): /bin/bessd(main+0xe3) [0x55de38a65d73]
2023-11-28T19:58:45.727Z [bessd] (4): /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf1) [0x7f92cd8f5081]
2023-11-28T19:58:45.727Z [bessd] (5): /bin/bessd(_start+0x2c) [0x55de38cbbefc]
2023-11-28T19:58:45.727Z [bessd] *** Check failure stack trace: ***

After deploying the charm with DPDK mode, bessd container can not start with following Permission Deny error.
2023-11-28T19:58:45.592Z [pebble] Service "bessd" starting: /bin/bessd -f -grpc-url=<IP_ADDRESS>:10514
2023-11-28T19:58:45.706Z [bessd] F1128 19:58:45.706557    53 bessd.cc:211] flock(pidfile=/var/run/bessd.pid): Permission denied [13]
2023-11-28T19:58:45.706Z [bessd] *** Check failure stack trace: ***
2023-11-28T19:58:45.727Z [bessd] F1128 19:58:45.727789    53 debug.cc:407] Backtrace (recent calls first) ---
2023-11-28T19:58:45.727Z [bessd] (0): /bin/bessd(_ZN6google15ErrnoLogMessageD1Ev+0xae) [0x55de3962323e]
2023-11-28T19:58:45.727Z [bessd] (1): /bin/bessd(_ZN4bess5bessd21TryAcquirePidfileLockEi+0x163) [0x55de38cf6863]
2023-11-28T19:58:45.727Z [bessd] (2): /bin/bessd(_ZN4bess5bessd19CheckUniqueInstanceERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE+0x49) [0x55de38cf6989]
2023-11-28T19:58:45.727Z [bessd] (3): /bin/bessd(main+0xe3) [0x55de38a65d73]
2023-11-28T19:58:45.727Z [bessd] (4): /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf1) [0x7f92cd8f5081]
2023-11-28T19:58:45.727Z [bessd] (5): /bin/bessd(_start+0x2c) [0x55de38cbbefc]
2023-11-28T19:58:45.727Z [bessd] *** Check failure stack trace: ***

Is this strictly confined Microk8s that you're running there? If yes, it won't work.


Broken syntax highlighting on docs that render python notebooks.

tensorflow/docs related.
URL(s) with the issue:
Code example for MultiHeadAttention at:
https://www.tensorflow.org/tutorials/text/transformer?hl=en#multi-head_attention
and code example at:
https://www.tensorflow.org/tutorials/text/image_captioning?hl=en#create_a_tfdata_dataset_for_training
Description of issue (what needs changing):
Clear description
I actually wasted some amount of time for my own stupidity, but there seems to be a glitch in the syntax highlighting system for notebooks rendered on the docs.
It seems that the syntax highlighter parses // as comments in python code blocks, not floordiv, which it actually means.
This actually may cause some amount of confusion, especially for people coming from c++ world like myself
who tend to eye-parse away the // as comments and the broken syntax highlighter is encouraging this (+ it looks so natural!).
Request visuals, if applicable

Also in dark mode:

When the Jupyter notebooks are rendered as HTML, for python code, lang-python CSS class is used for the
 block instead of lang-py. The prettify library used at https://www.tensorflow.org/tutorials understands python code as lang-py (https://github.com/tensorflow/docs/blob/master/tools/tensorflow_docs/api_generator/parser.py#L503 - lang-py is used correctly when generating API documentation). When prettify doesn't recognize the language identifier, it falls back to default-code lexer which ends up being used when lang-python is specified (floor division operator is probably being interpreted as a C style comment).
When the underlying JavaScript function is invoked from the Google Chrome Developer Console with the correct language identifier, the generated output is correct as shown by the following snippet.
> window.prettyPrintOne('self.depth = d_model // self.num_heads', 'python')
"self.depth = d_model // self.num_heads"

> window.prettyPrintOne('self.depth = d_model // self.num_heads', 'py')
"self.depth = d_model // self.num_heads"

I could not find any information on how this repo is converted to the full fledged site at https://www.tensorflow.org/tutorials or how the Jupyter notebooks are converted to HTML. This change probably needs to happen in that converter.

Unfortunately, tensorflow.org (and others) use a version of code-prettify and that project is now archived. There are no plans to upgrade, at this time. Sorry


Instead of creating an iframe, just render the html part of the snapshot

Fixes AP-3900
What Changed
Previously, we created an iframe to render the full document snapshot, as rrweb requires re-open()-ing the document (and dropping all event handlers, including the ones the preview uses).
Instead, we can use rrweb to just render the html element part of the document and do an innerHTML replacement of the result. This seems to work fine.
The only thing we need to do is a minor hack to ensure #storybook-root and #storybook-docs are always available, as the WebView assumes they are there and errors if they are not.
@shilman - perhaps we can either make the WebView more resilient, or even allow overriding the web view from a preset somehow (that might be tricky).
How to test
Include this canary in a project, check that you can render stories and run Chromatic builds.
Change Type

[ ] maintenance
[ ] documentation
[ ] patch
[x] minor
[ ] major

📦 Published PR as canary version: 0.0.19--canary.14.df1452d.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

I tried the canary build with a project that has an additional ignored region that's absolutely positioned. See here: I tried the test-archiver build with an additional ignored region that's absolutely positioned here: https://www.chromatic.com/build?appId=65a69506bef475d432719ca8&number=2
I think we still have a problem with ignored regions that are out-of-flow. It doesn't look like the lower left ignored region is being caught. Moreover, it still doesn't appear that ignored regions are being computed correctly, because the topmost ignored region isn't actually ignored.
Refs: CAP-1441


add Valentine plugin

im not sure if i did everything right, the .yaml is definitely wrong. this is my first attempt so sorry!
https://github.com/Tote-Bag-Labs/Valentine

i only included the linux version here btw

Thanks for finding and contibuting this plugin.
Validation is erroring with:
TypeError: pkgVersion.files.forEach is not a function
If you compare to another yaml file you will see you are missing hyphens - at the beginning of lines, which turns an object into an array

You also need to nested it inside a folder called valentine, matching the folder structure of the other plugins.
And please add Mac and Windows files too: https://github.com/tote-bag-labs/valentine/releases/tag/v1.0.1
Let me know if you have difficulty with it, I could add it if you want.
Thanks!

i'll try get around to it soon, if the developers do not support it does that matter? how do i ensure tote-bag-labs has access to updating their own plugin to prevent malicious actors

@kmturley it works now

youll have to check if the mac files are documented right

Thanks this is great. I had to make a few adjustments:

Extra folder for plugin name
Update author name
Reordered platforms
Updated description

Opened as a PR here:
https://github.com/open-audio-stack/open-audio-stack-registry/pull/38


Remove RemoteApp specs

Hi @amarsi, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (amarsin). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@amarsi - Same question: Why is this being removed?


Update Navigation View to Consume Drop Shadows

Description

Checks OS version to use Drop Shadows if OS version is 21h1 or higher
Add ShadowCaster element to cast shadow outside from SplitView to avoid being clipped by PaneClipRectangle in SplitView
Add  animations to ShadowCaster

How Has This Been Tested?

Manual visual verification

Screenshots (if appropriate):

/azp run

Does this include the Inset shadow for when the Pane is open and not overlaying other content?

/azp run

Does this include the Inset shadow for when the Pane is open and not overlaying other content?

No, the shadow only appears for Minimal and Compact display modes when the pane is open.

/azp run

@karenbtlai I think these conflicts are with the animated icon visual verification file updates.  I would recomend merging in master, and taking the versions of the files from master, deleting all of your local verification file updates, and then rerunning the pipeline to get a fresh set up updates.

/azp run

/azp run

/azp run

/azp run


up() breaks if volumes are defined with long syntax in the compose file.

Pretty minor, so just a heads up.
Services just assumes the volume is described with short syntax and tries to split a string by ':'

Just implemented this on master branch.

Published v1.0.3


Add automated software tests

To keep up with rising complexity, automated SW tests could help as quality gate. If a suitable solution was implemented, a test driven approach for extensions would be encouraged.

@kozaka-tv Do you happen to know any good test frameworks for C#?

Not really.
https://www.lambdatest.com/blog/nunit-vs-xunit-vs-mstest/
https://saucelabs.com/resources/blog/nunit-vs-xunit-vs-mstest-with-examples

But I would take NUnit Test I think.

Alright, then it will be NUnit, will still take some time to refactor.
In order to have cleaner public interfaces, i'd pack fetching and parsing the json to a nested class, and the whole evaluation to another. Then i'd suggest focussing on the second nest class in testing.

I've already spend a few hours on the refactoring itself, should be finished soonish (as soon work permits me to continue)
I think this issue should be split into the refactoring itself, and then adding the test cases in a separate issue

What I do not like, is the global variables. I think it would be better, to have 1-2 record classes for the data.

For config
For variables like currentScene, sniffer status, etc
( 3) If needed... etc.)

And then, classes/methods could get those records as input. This way, we could call a method with any input, and test it.
https://www.baeldung.com/cs/global-variables
I do not know, that in SB we could use different files, like a real structured code, but most likely not. Only one file could be added and inside this, we have to do everything (this is the Program.cs). So we have to do everything in that single file. But test could be separated and, structured. Like one test the config read, the other the isRelevant Method or class, etc.
If, there would be an option, that somethhing assemble everything into one file, that would be great. Then we could split up Program.cs to smaller pieces, and then at release time (or during dev), we just assemble the Program.cs and then use it. That would be great. Idk that is this possible somehow....just thinking :)

They are not really global variables, they are still in CPHInline.
It seems that NUnit offers methods to directly access those private
attributes for testing as well, so this solves the problem most likely.
If you check the refactoring branch (still not finished though) it's
already separating scene interactions, parsing the records from Sniffer and
evaluating those records.
Am Mi., 22. März 2023 um 12:27 Uhr schrieb Kozaka @.***

:

What I do not like, is the global variables. I think it would be better,
to have 1-2 record classes for the data.

For config
For variables like currentScene, sniffer status, etc
( 3) If needed... etc.)

And then, classes/methods could get those records as input. This way, we
could call a method with any input, and test it.
https://www.baeldung.com/cs/global-variables
I do not know, that in SB we could use different files, like a real
structured code, but most likely not. Only one file could be added and
inside this, we have to do everything (this is the Program.cs). So we have
to do everything in that single file. But test could be separated and,
structured. Like one test the config read, the other the isRelevant Method
or class, etc.
If, there would be an option, that somethhing assemble everything into one
file, that would be great. Then we could split up Program.cs to smaller
pieces, and then at release time (or during dev), we just assemble the
Program.cs and then use it. That would be great. Idk that is this possible
somehow....just thinking :)
—
Reply to this email directly, view it on GitHub
https://github.com/Th0rstenf/Rocksniffer-to-Streamer.bot-extension/issues/46#issuecomment-1479389830,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJK4DPSTK3VZUTHHGOBXZDW5LOYLANCNFSM6AAAAAAVM73CNE
.
You are receiving this because you authored the thread.Message ID:
<Th0rstenf/Rocksniffer-to-Streamer.bot-extension/issues/46/1479389830@
github.com>


Update partner definition

Description of change
Update partner definition
Link to relevant ticket
https://dsdmoj.atlassian.net/browse/CRIMAP-401
Notes for reviewer
Screenshots of changes (if applicable)
Before changes:
After changes:

How to manually test the feature

Were we going to change these from stacked radios to inline radios? I seem to remember some conversation around this. No biggie.

Oh, I have a vague memory of that, but will have to check. I'll get this merged, then find the thread/ask/create a ticket


Bars render incorrectly after new data containing NA's has been assigned.

It appears that the DOM is being manipulated incorrectly if data that contained NA's was previously assigned to the mark. Note the missing bar in the example below:
from bqplot import *
import numpy as np

scale_x = OrdinalScale()
scale_y = LinearScale()
axis_y = Axis(scale=scale_y, orientation='vertical')

data = [[1,2,3],[4,5,7]]

bar = Bars(x=['A', 'B', 'C'],
           y=data,
           scales={'x':scale_x, 'y':scale_y},
           colors=['orange', 'blue'],
           type='stacked')

Figure(marks=[bar], axes=[axis_y])

bar.y = [[1,np.nan,3], [6,3,4]]

bar.y = data

@SylvainCorlay @maartenbreddels @ibdafna for awareness

I think it is a d3.js 5 transition inssue in Bars. We need to check that exit is properly called.

I working on it right now.

OK, so the issue comes from the fact that NaN were serialized to null before the switch to binary serialization, and are now properly serialized as NaNs.
The former was treated as zero, which incidentaly was giving the correct behavior, but now the NaNs propagate.