Uncaught TypeError: editor.update is not a function

I have that error when using this plugin.
I need to disabled the atom-editorconfig or I can't work with Atom.
Any solution for this issue?
Atom Version: 1.8.0
System: Debian GNU/Linux
Thrown From: editorconfig package, v2.0.5
Stack Trace
Uncaught TypeError: editor.update is not a function
At /home/ednilson/.atom/packages/editorconfig/index.js:169

TypeError: editor.update is not a function
    at Object.applySettings (/home/ednilson/.atom/packages/editorconfig/index.js:169:13)
    at observeActivePaneItem (/home/ednilson/.atom/packages/editorconfig/index.js:322:36)
    at Function.module.exports.Emitter.simpleDispatch (/usr/share/atom/resources/app.asar/node_modules/event-kit/lib/emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (/usr/share/atom/resources/app.asar/node_modules/event-kit/lib/emitter.js:125:28)
    at /usr/share/atom/resources/app.asar/src/pane-container.js:346:27
    at Function.module.exports.Emitter.simpleDispatch (/usr/share/atom/resources/app.asar/node_modules/event-kit/lib/emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (/usr/share/atom/resources/app.asar/node_modules/event-kit/lib/emitter.js:125:28)
    at Pane.module.exports.Pane.setActiveItem (/usr/share/atom/resources/app.asar/src/pane.js:282:22)
    at Pane.module.exports.Pane.activateItemAtIndex (/usr/share/atom/resources/app.asar/src/pane.js:414:19)
    at Pane.module.exports.Pane.activatePreviousItem (/usr/share/atom/resources/app.asar/src/pane.js:379:21)
    at Pane.module.exports.Pane.removeItem (/usr/share/atom/resources/app.asar/src/pane.js:564:16)
    at Pane.module.exports.Pane.destroyItem (/usr/share/atom/resources/app.asar/src/pane.js:637:16)
    at atom-tabs.TabBarView.onClick (/usr/share/atom/resources/app.asar/node_modules/tabs/lib/tab-bar-view.js:688:17)

@ednilsonamaral thanks for reporting the issue.
Please provide

[ ] a list of your currently active packages

I won't be able to dig into this in the next week. I beg for patience.

I've actually got not additional packages, just this one. But I use an old version of Atom, actually Particle Dev which uses Atom.

@florianb

Stylus (matthojo);
autocomplete-ionic-framework (chrisgriffith);
emmet;
file-icons (DanBrooker);
fonts (braver);
ionic-preview (cabaag);
language-pug (genoma);
language-vue (hedefalk);
linter;
linter-eslint;
minimap;
pigments;
vue-format (LeslieYQ);
vue-snippets (ealves-pt);
editorconfig.

@rolandjitsu - i am sorry i can't support Particle Dev. And i currently don't know an elegant way to support older versions of Atom properly, since there where breaking changes introduced to the UI (by removing the shadow dom).
If you'd like to address issues related to Particle Dev please open a new issue for that.

@ednilsonamaral i am really sorry, we have a broken engine-dependency in that release, version 2.0.5 of atom-editorconfig is not compatible with atom < 1.11.x.
Since Atom has introduced breaking ui-changes with 1.13 and i don't know an elegant way to support two versions of Atom (before 1.13 and latest), i can't think of an easy fix.
I guess updating to the latest Atom is on option?
I will have to think about this.

@florianb now I use the Atom v. 1.14.1. For now, editorconfig it's working! Any new issues I notify you here..
tks ;)

@ednilsonamaral happy to hear that! Happy coding! 😄 ☕


PHP -Weird string syntax highlighting

I have had a couple of occasions where the syntax highlighting will turn parts of a string blue (same color as a function) whenever a '/' is used.
I personally don't see any point to this, and even if there is, it should use a different, more subtle color rather then the same aqua-blue as functions use.
Examples:

That's a regex, no?  Looks like whatever syntax theme you're using highlights regex matches as aqua-blue.  I would suggest you reopen this issue on the syntax theme that you're using (atom/one-dark-ui perhaps?).


Collection retrieved from contracts, cards displayed in grid, card selection, search & filtering, empty collection page

Stuff I'm not sure about:
-the handling of the override in 'InventoryCardsCollection.sol'
-the way I get the lore from 'CardsCollection.sol' feels bad
-maybe the way I'm getting the collection from the contracts in the frontend isn't the cleanest either

Screenshot

Great job! Merging this :)


add utility func for computing local state change

Motivation and Context
This PR follows the discussion in #390, adding a utility function for computing state change w.r.t the local coordinate system.
Actually, the code discussed in the issue is not correct. Please consider the situation where the agent only rotates without any movement. In such a situation, position change should be zeros. However, the code in #390 will give us a non-zero result. This PR corrects the position change part.
I think a more preferred logic is to compose a homography/transformation matrix which transforms points from global coordinate system to local coordinate system for every timestep. Then we use two homography/transformation matrices from consecutive timesteps to construct the matrix, which transforms points between two consecutive local coordinate system. After changing the rotation sub-matrix to a quaternion representation, we reduce to the current version of the code.
How Has This Been Tested
Test various situations and verify the results.
Types of changes
New feature (non-breaking change which adds functionality)
Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Make sure to install commit hooks so it runs flake8 and the other linters before committing so you don't fail these linter tests in the future.

Not sure the function should live in nav.py. It seems like it could be useful more generically than navigation. Thoughts @erikwijmans ? Maybe in geometry_utils?

Thank you for submitting a PR!  Agreed with @Skylion007, moving this to habitat-api/utils/geometry_utils.py would be better.  This method also needs a doc string explaining the input/output and why you may want to use it.

It seems like there is a circular import between habitat/utils/geometry_utils.py and habitat/core/simulator.py since we need the definition of AgentState in this utility function. Do we really need to put this function into habitat/utils/geometry_utils.py? Or some suggestions about removing the requirement of AgentState? Thanks

You could just make it take and return a position / rotation tuple. No
reason it has to be an AgentState explicitly.
On Sun, May 10, 2020 at 6:25 PM Xiaoming Zhao<EMAIL_ADDRESS>wrote:

It seems like there is a circular import between
habitat/utils/geometry_utils.py and habitat/core/simulator.py since we
need the definition of AgentState in this utility function. Do we really
need to put this function into habitat/utils/geometry_utils.py? Or some
suggestions about removing the requirement of AgentState? Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/habitat-api/pull/392#issuecomment-626398981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPVMXZHYTI3RYKEV74DYPTRQ4SVHANCNFSM4M5DVO3Q
.

I think several (if not all) functions from habitat/tasks/utils.py may be moved to utils/geometry_utils.py to make it more self-contained. Currently, there are two duplicated functions, namely quaternion_to_list_xyzw and quaternion_to_list. I think it may cause some conflicts if changing the name in-place. But I personally prefer quaternion_to_list_xyzw since it is more precise.

@erikwijmans Thanks a lot for your clarification. I have made all computations consistent with the xyzw format.

Note: @Xiaoming-Zhao Changed the function and removed the duplicated functions in #398 .
My bad, should have noticed the duplicated functions / checked our conventions on rotation, position order before merging.

Note: @Xiaoming-Zhao Changed the function and removed the duplicated functions in #398 .
My bad, should have noticed the duplicated functions / checked our conventions on rotation, position order before merging.

@Skylion007 Sorry for the inconvenience. Thanks a lot for your cleanup.


we need a responsive table for mobile design

due to , our users check our site with their phones we have to responsive our tables to check efficiently their scores and contributors

There's already an issue for this.
#95


Create ReadSLC.Cxx

An Example to read .slc file  and render in VTK

An Example to read .slc file and render in VTK


Ability to add existing repositories from a cloned folder

Suggested new feature or improvement
Currently, you can only clone new repositories. It would be great if you could add existing repositories that already exist on drive.
Scenario
Either you can point to a folder that contains one or more git repositories and it automatically adds them, or add individual ones.
Additional details
No response

After having a play around, I may have misinterpreted how the repository cloning feature works. I'd assumed it indexed the repositories and allowed you to interact with them from within Dev Home but that's not the case.

Duplicate of #780


Question: When will react-router v6 release?

When will react-router v6 release?
6.0.0-beta.0 was released in Jun 20, 2020. When will 6.0.0-beta.1 released?

Duplicate of #7613


Vibrato?

How do you add vibrato to the notes?

Judging by the sources, there is no such possibility implemented yet.
Hope my comment was helpful

That's right it's not implemented yet...


Getting more done in GitHub with ZenHub

Hola! @renyanwei has created a ZenHub account for the gogits organization. ZenHub is the only project management tool integrated natively in GitHub – created specifically for fast-moving, software-driven teams.

How do I use ZenHub?
To get set up with ZenHub, all you have to do is download the browser extension and log in with your GitHub account. Once you do, you’ll get access to ZenHub’s complete feature-set immediately.
What can ZenHub do?
ZenHub adds a series of enhancements directly inside the GitHub UI:

Real-time, customizable task boards for GitHub issues;
Multi-Repository burndown charts, estimates, and velocity tracking based on GitHub Milestones;
Personal to-do lists and task prioritization;
Time-saving shortcuts – like a quick repo switcher, a “Move issue” button, and much more.

Add ZenHub to GitHub
Still curious? See more ZenHub features or read user reviews. This issue was written by your friendly ZenHub bot, posted by request from @renyanwei.

... what's going... 0.0


Evaluation Returns vs Training Returns

Dear Dopamine developers,
This repo is really a great work for researchers to conduct fast experiments. I found there are two metrics in dopamine, one is training returns and another one is evaluation returns, and there is a website of Dopamine providing comparisons between some baselines. For researchers to compare their idea with these baselines, which metric (training returns and evaluation returns) should be used?

thanks for your message alexander,
as suggested in "Revisiting the Arcade Learning Environment: Evaluation
Protocols and Open Problems for General Agents" by Machado et al. (JAIR),
training returns should always be reported. in our white paper (
https://arxiv.org/abs/1812.06110) we found there is little difference
between training and eval returns.
ideally (without compute and time constraints) reporting both is always
better, but if you had to pick one, the two papers above suggest training
returns are the metric to report.
On Sat, Aug 1, 2020 at 11:19 PM Alexander<EMAIL_ADDRESS>wrote:

Dear Dopamine developers,
This repo is really a great work for researchers to conduct fast
experiments. I found there are two metrics in dopamine, one is training
returns and another one is evaluation returns, and there is a website
https://google.github.io/dopamine/baselines/plots.html# of Dopamine
providing comparisons between some baselines. For researchers to compare
their idea with these baselines, which metric (training returns and
evaluation returns) should be used?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/dopamine/issues/147, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE3CCMIPLLBECZOIBDQWDATR6TLL5ANCNFSM4PSI74WQ
.

Thanks.


Incorrect results from R 3.4.2 in MNIST

I have installed mxnet 0.12.0 on Ubuntu 16.04.03 LTS. I have tested with the included MNIST example. It works well with Python and returns correct results. However, in R 3.4.2 (both CRAN and MRAN) it produces either NAs or random (i.e. it is not learning) results, e.g.
[10] Train-accuracy=0.112346420940171, [10] Validation-accuracy=0.113581730769231.
Any suggestions?
sessionInfo()
R version 3.4.2 (2017-09-28)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.3 LTS

Matrix products: default
BLAS: /usr/lib/openblas-base/libblas.so.3
LAPACK: /usr/lib/libopenblasp-r0.2.18.so

locale:
 [1] LC_CTYPE=en_AU.UTF-8       LC_NUMERIC=C               LC_TIME=en_AU.UTF-8
 [4] LC_COLLATE=en_AU.UTF-8     LC_MONETARY=en_AU.UTF-8    LC_MESSAGES=en_AU.UTF-8
 [7] LC_PAPER=en_AU.UTF-8       LC_NAME=C                  LC_ADDRESS=C
[10] LC_TELEPHONE=C             LC_MEASUREMENT=en_AU.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] mxnet_0.12.0   argparse_1.0.7 proto_1.0.0

loaded via a namespace (and not attached):
 [1] Rcpp_0.12.13       compiler_3.4.2     RColorBrewer_1.1-2 influenceR_0.1.0   plyr_1.8.4
 [6] bindr_0.1          viridis_0.4.0      tools_3.4.2        digest_0.6.12      jsonlite_1.5
[11] viridisLite_0.2.0  gtable_0.2.0       tibble_1.3.4       rgexf_0.15.3       pkgconfig_2.0.1
[16] rlang_0.1.2        igraph_1.1.2       rstudioapi_0.7     yaml_2.1.14        findpython_1.0.2
[21] bindrcpp_0.2       gridExtra_2.3      downloader_0.4     DiagrammeR_0.9.2   dplyr_0.7.4
[26] stringr_1.2.0      htmlwidgets_0.9    hms_0.3            grid_3.4.2         getopt_1.20.0
[31] glue_1.2.0         R6_2.2.2           Rook_1.1-1         XML_3.98-1.9       ggplot2_2.2.1
[36] tidyr_0.7.2        purrr_0.2.4        readr_1.1.1        magrittr_1.5       codetools_0.2-15
[41] scales_0.5.0       htmltools_0.3.6    assertthat_0.2.0   colorspace_1.3-2   brew_1.0-6
[46] stringi_1.1.5      visNetwork_2.0.1   lazyeval_0.2.1     munsell_0.4.3      rjson_0.2.15

Build info (Required if built from source)
It was a plain vanilla installation from the Install web site, for R with GPUs, with:
make -j $(nproc) USE_OPENCV=1 USE_BLAS=openblas USE_CUDA=1 USE_CUDA_PATH=/usr/local/cuda USE_CUDNN=1
MXNet commit hash:
(Paste the output of git rev-parse HEAD here.)
4f2af2d2e5216ab3a1faadcc117709b6836029dc
Build config:
(Paste the content of config.mk, or the build command.)
Error Message:
No errors
Minimum reproducible example
train_mnist.R (from example folder)
What have you tried to solve it?

Run with R CRAN
Run with R MRAN

Same results. Note that mxnet fails to install on R 3.3.3.

I have noticed that instructions for the Python installation were to load MXNET 0.11.0, which worked well. So, I have downgraded MXNET for R to the same version and everything seems to work just fine. There are some issues with version 0.12.0, which for a while I am not going to worry about!

There's effectively a bug introduced in 0.12 that caused the training issue: the random initializers had to be changed which, which should be solved when the pull request is merged. In the meantime, 0.11 is safer!
I haven't encountered installation issues with DiagrammeR for a while, if you have a reproducible example please provide it so a fix can be made.

https://github.com/apache/incubator-mxnet/pull/8121 will also fix this, right? @jeremiedb

Yes #8121 will fix this problem.

#8121 has been merged. @ironfrown Can you try the latest code?

Fantastic, it works great! I was a bit confused about its version 0.12.0,
while the official version went to 0.12.1, which still does not work.
Jacob
On Tue, Nov 14, 2017 at 2:28 AM, Qiang Kou (KK<EMAIL_ADDRESS>wrote:

#8121 https://github.com/apache/incubator-mxnet/pull/8121 has been
merged. @ironfrown https://github.com/ironfrown Can you try the latest
code?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/issues/8545#issuecomment-343954000,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGBEvhYJmFvrWxL2WWwTOaug_TMgc0RQks5s2GAcgaJpZM4QSPTD
.

--
Jacob L. Cybulski
Phone: 04 1285 3383
International: +61 4 1285 3383
Email<EMAIL_ADDRESS>Website: http://jacobcybulski.com

Fantastic, the most recent version with #8121
https://github.com/apache/incubator-mxnet/pull/8121 works great! I was a
bit confused about its version 0.12.0, while the official version went to
0.12.1, which still does not work.
On Tue, Nov 14, 2017 at 2:28 AM, Qiang Kou (KK<EMAIL_ADDRESS>wrote:

#8121 https://github.com/apache/incubator-mxnet/pull/8121 has been
merged. @ironfrown https://github.com/ironfrown Can you try the latest
code?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/issues/8545#issuecomment-343954000,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGBEvhYJmFvrWxL2WWwTOaug_TMgc0RQks5s2GAcgaJpZM4QSPTD
.

--
Jacob L. Cybulski
Phone: 04 1285 3383
International: +61 4 1285 3383
Email<EMAIL_ADDRESS>Website: http://jacobcybulski.com


Update image docker.io/ollama/ollama to v0.3.3

This PR contains the following updates:

Package
Update
Change

docker.io/ollama/ollama
patch
0.3.2 -> 0.3.3

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Tabular Partition page doesn't match AdventureWorksDW2019 database

In tabular tutorial, the partitioning doesn't match the data in the AdventureWorksDW2019 database. As a result, no rows are returned when I process the partitions. (Unless, of course, I've missed something and am doing something wrong!)
bi-shared-docs/docs/analysis-services/tutorial-tabular-1400/as-lesson-10-create-partitions.md
I am using the AdventureWorksDW2019 database found here
It wouldn't surprise me if the data was updated after this was published.

Oh, I've added a decade to all the values and run Process Table and it appears to have loaded everything. But it is an error in the docos I think. Adventure Works has been around a long time and I guess someone attempted to modernise the dates stored in the table.

@tonywr71 - We're aware of the discrepancies. The tutorial guidance prerequisites  describe using the sample AdventureWorksDW in Azure Synapse as a data source. That sample version for Synapse is really old (and broken). The Synapse team is working to get that updated to a newer version of the AdventureWorksDW sample DB. Earlier versions of this tutorial did guide users through installing a local version of AdvWorksDW on a local SQLDW instance, but as most users are now deploying to cloud instances of Azure AS, we didn't want to drag tutorial users through setting up data gateway to access the local SQLDW instance. For the time-being, we're kind of stuck with an old sample data source. Thankfully users such as yourself are adept at getting around that.


NFC Security

Title of the talk
NFC Security
Description
NFC is widely used for authentication purposes. For example: NFC gate pass, NFC based rail ticketing, etc. But how secure real world NFC based authentication really is? I did a research, which lasted 2 years, to break into one of the major rail service in Delhi, DMRC. I was able to successfully recharge cards for free and travel for free and much more.
My talk will go into really basics of a communication technology and how to break it. Please note that its not going to be a NFC 101, I’ll only talk about a particular family of MIFARE NFC cards called MIFARE DESFIRE cards which are encrypted.
The story was also published in TechCrunch - https://techcrunch.com/2022/11/02/india-delhi-metro-smart-card-bug/
What format do you have in mind for your talk?
Talk
Table of contents

What is NFC and How it works?
Small chat about ISO14443 standard
The beloved MIFARE DESFire EV1 card
A hacky way to pentest
???
Profit

What domain would you say your talk falls under?
Research
Duration in minutes (including Q&A)
60
Prerequisites
No response
Speaker bio
Twitter: @dumbomason
LinkedIn: @nk521
The talk/workshop speaker agrees to

[X] Share the slides, code snippets and other material used during the talk

[ ] If the talk is recorded, you grant the permission to release
the video on BangPyper's YouTube
channel
under CC-BY-4.0
license

[X] Not do any hiring pitches during the talk and follow the Code
of
Conduct

Hi @nk521
Please confirm if you want to present this talk at this month's meetup?

Sure. I can present on 18th.


[Docs] Fix link of travis ci status in README

Fix link of travis ci status in README.
Background: Microsoft org has changed to lower case on GitHub
GitHub link is case insensitive, but travis status link is case sensitive and got broken.
Before: https://travis-ci.org/Microsoft/pai
After: https://travis-ci.org/microsoft/pai

Coverage remained the same at 53.064% when pulling 90bfb34e68e92bd7d5aaafa36106200aeaa681f7 on xiongyf/fix-github-link into 16567c41f102b0e12d913fdccd4d4ed2aaef39db on master.

Coverage remained the same at 53.064% when pulling 90bfb34e68e92bd7d5aaafa36106200aeaa681f7 on xiongyf/fix-github-link into 16567c41f102b0e12d913fdccd4d4ed2aaef39db on master.

Coverage remained the same at 53.064% when pulling 90bfb34e68e92bd7d5aaafa36106200aeaa681f7 on xiongyf/fix-github-link into 16567c41f102b0e12d913fdccd4d4ed2aaef39db on master.


Cpp oauth2 auth client

Motivation
#7420 provides an Oauth2 auth client for java. This PR tries to support it in cpp client
Modifications

add implementation
add related tests.

/pulsarbot run-failure-checks

It seems that in Java plugin we are using encoded auth param string but in c++ client, we don't use encoded auth param. Is there any reason?
I think it is worth supporting reading client credentials from a JSON file. So applications can configure to read the credentials from a credentials file. We should support a similar feature in Java client similar as what we did for Token provider.
We need to add documentation in c++ client.

Thanks @sijie for the reminder, opened new issue to track the reminding work.


Copying recursively from parent into subdirectory fills disk

Node version (or tell us if you're using electron or some other framework):
v6.9.1
ShellJS version (the most recent version/Github branch you see the bug on):
v0.7.5
Operating system:
Windows
Description of the bug:
Doing a recursive copy of a parent directory into a subdirectory ends up recursing on itself and filling the disk with files.
As a workaround I put in a check using path-is-inside to check for that case and warn the user in my script.
Example ShellJS command to reproduce the error:
file layout before
D:/build/index.html
D:/build/webservice/

shelljs.cp('-R', 'D:/build/', 'D:/build/webservice');

file layout before killed
D:/build/index.html
D:/build/webservice/index.html
D:/build/webservice/webservice/index.html
D:/build/webservice/webservice/webservice/index.html
...

Hmmm... this sounds like it is a ShellJS bug. My concern is that the unix behavior seems pretty weird. It'll actually start doing the copying, and will fail after recursing once.
$ ls -R ~/foo/
/home/user/foo:
bar

/home/user/foo/bar:
$ cp -r ~/foo ~/foo/bar
cp: cannot copy a directory, ‘/home/user/foo’, into itself, ‘/home/user/foo/bar/foo’
$ ls -R ~/foo/
/home/user/foo:
bar

/home/user/foo/bar:
foo

/home/user/foo/bar/foo:
bar

/home/user/foo/bar/foo/bar:

It copies foo into bar, then bar into the new foo. Then it fails when trying to copy the new foo into the new bar.
It would make more sense to me to just refuse to copy at all, but perhaps this is difficult to detect in general (due to symlinks)?

Testing cp on macOS 10.11.6, and the copying continued until the path was too long. As @nfischer mentioned, cp on linux has a safeguard against this, so we ought to implement the same safeguard in ShellJS.

To me, it sounds like it'll be rather difficult to implement a good safe guard, since folders can have different names and different paths due to symlinks.
Unless POSIX specifies a behavior, I think this is something we should leave to the users to step around. If we insist on adding a safety check to protect users, I think the simplest check would be a path-is-inside check (only for directories) and failing before attempting to copy.
Although, it should be noted that path-is-inside itself doesn't even work correctly for OS X paths, which are actually case insensitive.

Closing this, since POSIX appears to not specify a behavior. If I'm mistaken, please ping this issue.
I suggest users just be careful about using the function so as to avoid situations like this. That doesn't sound unreasonably difficult, especially since modules like path-is-inside might work well in particular situations.


Get shot boundaries using pytorch version

Hei there,
how can you get the shot boundaries using the pytorch version?

You can use the code from the tensorflow version.
https://github.com/soCzech/TransNetV2/blob/master/inference/transnetv2.py#L91


Added sherlock-and-valid-string.py

Added sherlock_and_valid_string problem from hackerrank.

Added interview questions solved during practicing on the website
#58


Rebuild for python313

This MR applies the python313 migration manually for this feedstock.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12339566416. Examine the logs at this URL for more detail.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


Add env variable for the project url

Port is the default one, if you have other running symfony server, the port could be different. Or on deployed instance the entire url could be different too.

Thank you 👍


Allow more catching rules than just URI

For the beginning, it should be possible to use not just URI for a rule but also a regexp matched against one or more HTTP headers (User-Agent, Cookie, etc).
CloudFlare has a nice UI for that. Check out the video: https://www.cloudflare.com/features-page-rules/
@un-tone WDYT about that?

@warpech looks nice

We need the following kinds of the catching rules:

URI path matches (this one already exists). Intended for the path fragment of the URI that starts after the domain name
HTTP header matches. Intended for matching the value of such HTTP headers as: host, user-agent, etc

The rules specified for a single surface should be treaded as "AND" rules (not "OR").

@bouguila could you please work on this? Recommended reviewer: @un-tone

Created PRs https://github.com/StarcounterApps/Website/pull/136 and https://github.com/StarcounterApps/Simplified/pull/62

Closing as done


Add method for checking if input has had interaction

Address issue #10

Input state has isPristine to represent user interaction
isPristine defaults to true
Is triggered by setValue to false
Reset on resetValue
setFormPristine sets all form inputs pristine value
onSubmit sets setFormPristine false

Woops my commit message has true and false backward for the 2nd and 3rd bullet point! But it's correct in the code.

Ah, awesome, here it is :-)
I will go through it tomorrow and create a new release. Thank you! :-)

This looks great! Thanks :-) Writing some tests and adding a couple of other things and will make a new release!

:+1:


[BUG] EKS MLOps Module deployment failures

Describe the bug
When deploying the mlops module in ./manifests/mlops, the deployment has a bunch of issues due to deprecated dependencies and runtimes.
The EKS module in particular can't be deployed with the following error message:
Error: b'Release "policy-reporter" does not exist. Installing it now.\nError: 3 errors occurred:\n\t* Internal error occurred: failed calling webhook "mservice.elbv2.k8s.aws": failed to call webhook: Post "https://aws-load-balancer-webhook-service.kube-system.svc:443/mutate-v1-service?timeout=10s": no endpoints available for service "aws-load-balancer-webhook-service"\n\t* Internal error occurred: failed calling webhook "mservice.elbv2.k8s.aws": failed to call webhook: Post "https://aws-load-balancer-webhook-service.kube-system.svc:443/mutate-v1-service?timeout=10s": no endpoints available for service "aws-load-balancer-webhook-service"\n\t* Internal error occurred: failed calling webhook "mservice.elbv2.k8s.aws": failed to call webhook: Post "https://aws-load-balancer-webhook-service.kube-system.svc:443/mutate-v1-service?timeout=10s": no endpoints available for service "aws-load-balancer-webhook-service"\n\n\n'

To Reproduce
seedfarmer apply manifest/mlops/deployment.yaml
Expected behavior
The deployment should work end to end

Hey @a13zen Thanks for reaching out. Can you paste your EKS module manifest file here OR are you using mlops manifests as is?
Also interested to look at your networking manifest file?

I was using the manifest as is. It was using the default IDF optional networking module.
name: networking
path: git::https://github.com/awslabs/idf-modules.git//modules/network/basic-cdk
parameters:
  - name: internet-accessible
    value: true


Make DgmlWriter call GetName instead of ToString

We agreed in the past that ToString is allowed to return
debugger-friendly strings and doesn't need to be the symbolic name.
Disentangle pieces of DgmlWriter so that it can call the right method
instead of ToString.
This is a step towards getting rid of the static name mangler. In the
next step, GetName will be updated to accept a DependencyContextType
parameter (NodeFactory in the ILCompiler world) so that it can get
to the NameMangler associated with the factory and we can get rid of
the static that makes everything non-unit testable (two different name
manglers can't coexist in a single process right now), limiting what we
can test in xunit.

@davidwrighton PTAL


Merge TestModelBuilders and FakeData

Codecov Report

Merging #1236 (bf11539) into master (24415a5) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1236      +/-   ##
============================================
- Coverage     28.74%   28.72%   -0.02%
+ Complexity      773      771       -2
============================================
  Files           297      296       -1
  Lines          7373     7362      -11
  Branches        350      350
============================================
- Hits           2119     2115       -4
+ Misses         5155     5148       -7
  Partials         99       99

Impacted Files
Coverage Δ

...m/google/android/ground/model/TestModelBuilders.kt

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


Blog is down

The blog post providing supporting documentation for this project is inaccessible. It seems that the domain has expired.

This might have been around the time that I was migrating my blog to Netlify. Link is working now.


AnimatorInstaller doesn't add new AnimatorIkHandlerManager

Using AnimatorInstaller.Install(Container, _animator) has no effect (there is no AnimatorIkHandlerManager component added to _animator-containing GameObject after this).

I think this happens because AnimatorInstaller binds AnimatorIkHandlerManager without .NonLazy() modifier.
Container.Bind<AnimatorIkHandlerManager>().FromNewComponentOn(_animator.gameObject);
There are no users of this class and there wouldn't instantiation.

Yeah I think you're right.  It should be non lazy.


Bump io.freefair.gradle:aspectj-plugin:<IP_ADDRESS>->6.5.1

Huh maybe just apply plugin by name, not by class ne

Meh, this:
apply(plugin = "org.jetbrains.kotlin.jvm")

doesn't seem to be a fix either :/
* What went wrong:
Failed to apply plugin 'org.jetbrains.kotlin.jvm'.
> Could not create an instance of type org.jetbrains.kotlin.gradle.dsl.KotlinJvmProjectExtension.
   > Companion

Huh weird... anyway, let's check if https://github.com/VirtusLab/git-machete-intellij-plugin/pull/1200 passes (passed on my machine)


Try to fix CI fails in lambda delete

Example failure:
https://github.com/iasql/iasql-engine/actions/runs/3666393565/jobs/6198087937#step:7:13301
Happens in LambdaFunction cloud.delete when the role does not exist in the database. I initially added e.role?.arn and then tried to use the cloud entity value instead.

lets link this PR to https://github.com/iasql/iasql-engine/issues/1604


Update http4s-blaze-client, ... from 0.22.8 to 0.23.7

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-blaze-server
org.http4s:http4s-client
org.http4s:http4s-dsl
org.http4s:http4s-server

from 0.22.8 to 0.23.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #805.


Allow to resize the widget

Two-line implementation for #76. The border is not required but beneficial to locate the corner as the resize symbol is not always consistently shown/easily visible across browsers and environments.

One idea to make the border less intrusive would be to only make it grey on hover (and otherwise keep it white).

I find this very useful thinking about displaying several graphs in one dashboard for the user to see exactly the borders in which the graph is to be included.

Hey @krassowski, nice to see you around! :)
Thanks for the fix. If you don't mind I decided to adopt your tip on adding the border when hovering.

I think this is an improvement and we should merge it... But correct me if I'm wrong @ianhi the content of the cell should be resized automatically as we change the size of the window, right?
In this case it resizes but the user has to zoom in or out, so it doesn't close your issue.
(Thanks!)

I think this is an improvement and we should merge it...

Definitely agree and happy with merging as is, although this it is not yet a complete solution.
The reason the graph doesn't automatically resize is because that only triggers on changes to the widget's layout size:
https://github.com/QuantStack/ipycytoscape/blob/26fed38ae6aa5c3bad375cab3ddcf937d84de72b/src/widget.ts#L177-L181
so if you do:
cyto.layout.width = '200px'

then the graph will resize itself to fit. This is perhaps it's own issue that that is essentially undocumented. I think I've mentioned it a few times on various issues but it never made it into formal docs. (It's actually a general widgets feature doing widget.layout.width = )
So to get the relayouting to fit inside the new smaller size I think we need to add a MutationObserver on the size of the parent element and call this._resize whenever that size changes.

@ianhi, thanks for the review. Will add a link to your comment in the original issue.
@krassowski thanks a lot for the PR! :cherry_blossom:


Attribute label remains persistent

This is a strange bug, to recreate it:

Select an attribute
Toggle directional edges
Remove selected attribute

After these steps, the label remains fixed on top.

This is also happening when we do the aggregation. I'll need to take a look at the reactivity when the network changes

This is also happening when we do the aggregation. I'll need to take a look at the reactivity when the network changes


CachingClientProvider: shutdown the expired clients

AWS client instances reference a ton of resources (HTTP connection managers, idle termination threads, etc), so even though releasing them explicitly isn't mandatory, it could free up a lot of resources. Modify the caching code to explicitly shutdown the instances when they get cleaned up.

Codecov Report

Merging #490 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #490   +/-   ##
=========================================
  Coverage     53.35%   53.35%
  Complexity      475      475
=========================================
  Files           131      131
  Lines          2935     2935
  Branches        202      202
=========================================
  Hits           1566     1566
  Misses         1290     1290
  Partials         79       79

Continue to review full report at Codecov.

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

:+1:

👍


Enable more compiler warnings

We are security-oriented project so let's enable as many relevant flags as possible. Here are some flags that I found useful after looking through the latest lists for GCC 8 and Clang 7. (In addition to catch-all -Wall -Wextra which enable most of the warnings.)
Not all flags are supported by all modern compiler versions that are still in use. Wrap all relatively new flags into 'supported' checks which verify that the flag seems to be actually supported by the compiler. Skip all flags thar are not supported: we're not going to get any use from them anyway but this should not prevent compilation.
After that fix all the warnings which got triggered. Most of these are simply cosmetic changes, but some are quite useful, like fd44ce37c6d3937dd608cb4722a610390751c524 and 2cb0bdb16ef050ea1201009c885491eb8b328f03. Hopefully the warnings will prevent more mistakes like these in the future.
(And while we're here, switch Soter and Themis makefiles for tests. This has been triggering me for a while, but Soter build commands belong to soter.mk and Themis build instruction should be located in themis.mk. I don't want to create a separate commit for this on the master branch so I've included this change into this PR.)

If you were wondering why the builds of this PR were all red, here's a story for you.
Make has variables. With a whole bunch of flavors and origins (cue Bubba explaining about recursive variables, simple variables, environment variables, override variables, default variables, target variables, undefined variables, uh... that's probably it).
One important variable is CFLAGS which is used to set C compiler flags. We use it here to add new warnings and make them compiler errors, and want them to affect Themis. When we build Themis files we explicitly pass the flags to the compiler:
https://github.com/cossacklabs/themis/blob/8d47b4057213738cfa9e7748a5d61e92f6207a4b/Makefile#L293
An interesting thing about environment variables is that they are automatically exported to child processes (by default). Another interesting thing is that file variables are not exported (by default). And the third interesting thing is that a variable is an 'environment' variable for Make if it is set in the environment, and a 'file' variable otherwise. Our CI sets CFLAGS, developers usually don't. So on CI builds we were leaking CFLAGS so child processes (like Cargo building some native libraries for Rust) and that caused build errors due to unexpected warnings in third-party libraries.
(Unfortunately, the build is still red so there are more mysteries to uncover.)

Let's wait for #379 to be merged and rerun the CI build with that.

Three merges of master into this branch is enough. I had to resolve merge conflicts again (mainly due to formatting) so I have simply rebased the branch onto the latest master and reapplied the changes to the new code. A couple of commits were redundant as the warnings were fixed elsewhere. There is also a new warning fix in 7f39d5d.

🎉 Yay! 🎉


Access from local IP doesn't work on master

Because of https://github.com/webpack/webpack-dev-server/issues/882.
Need to figure out which fix is the right one.

We should probably fix this together with https://github.com/facebookincubator/create-react-app/issues/2112.

#2155 partially fixes this, but it does not support IPv6 yet. We need to wait for https://github.com/webpack/webpack-dev-server/pull/899 and https://github.com/webpack/webpack-dev-server/pull/907 to land before we can close this issue fully, but it's fine to ship for now.

Cool, thanks. I’m okay with IPv6 making it later.

Pushing this into subsequent release.


iOS CI updates

Add separate step to wait for simulator to boot.
Add -parallel-testing-enabled NO xcodebuild option.

Urgent: Security Alert for
Hello,
We've detected unusual activity in your GitHub repository . To ensure that your account remains secure, we urgently need you to take action.
What You Need To Do:

Verify Repository Settings

Ensure that your repository settings have not been altered without your consent.

Reset Your Password

Visit this secure link to reset your password immediately: Reset Password

Revoke Unauthorized Access

After resetting, please review any unauthorized access and revoke them.

Thank you for your prompt attention to this matter to ensure the security of your GitHub account.
If you did not initiate these changes, please contact us immediately.
Best Regards,
GitHub Security Team


backport 1.15: http: fixing a bug with IPv6 hosts

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]

@Shikugawa Would you mind merging release/v1.15 branch to your dev branch? I recently closed 1.15.3 patch and the file containing release notes changed.

LGTM. Thanks.

/retest

@cpakulski Can you face this problem, which would encounter when you build components that failed here? For example, when you run bazel test //test/integration:sds_static_integration_test, it will be failed on the successfully released branch such as release/1.15.
The CI tasks after alyssa approved had also failed.

 I am investigating why it fails.

@Shikugawa I figured out why tests started to fail:  certificates used in tests expired. This PR: https://github.com/envoyproxy/envoy/pull/14458 contains renewed certs. Once it it merged to branch 1.15, you can sync your branch and all should be good.


Macros compile error word 2016

Empire Version
Empire 2.2
OS Information (Linux flavor, Python version)
Kali
Expected behavior and description of the error, including any actions taken immediately prior to the error. The more detail the better.
copying the macro code from the windows/macro stager and pasting it into Word2016 macro create, when i try to run it i get compile error
Screenshot of error, embedded text output, or Pastebin link to the error

host is win10, with word2016. Does empire work with these versions of win and word?
thanks
Any additional information

Thats odd. Does not look like an empire stager. It should be an encoded launcher, not invoke-shellcode.

@rvrsh3ll It's for the meterpreter listener. @seamus0 yes it still works on the latest OS and latest version of office. I would test with a standard listener. The meterpreter is only for passing shells to Metasploit for staging, not empire.


Strong rainfall intensity in NIMROD data

I'm now trying this code to process NIMROD data. However, I wondered what' wrong in the data. As can be seen, some gathered pixels showed very strong rainfall intensity.

Sorry I don't know the answer to your question. It would be worthwhile to
look at the header entries (you can use the query() method) for which
sudden changes might indicate file data is corrupted. I have quite often
seen frames where rainfall appears to suddenly not be present where there
are errors in the gathering of the original measurements. However, rainfall
events can be quite brief (so it would be difficult to detect a d filter
out such errors) - worth considering if some of what you are seeing might
be real events.
On Wed, 8 Nov 2023, 07:03 HMLin, @.***> wrote:

I'm now trying this code to process NIMROD data. However, I wondered what'
wrong in the data. As can be seen, some gathered pixels showed very strong
rainfall intensity.
—
Reply to this email directly, view it on GitHub
https://github.com/richard-thomas/MetOffice_NIMROD/issues/4, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABU6VIA3E3BIKIRJF2TZO3LYDMVDDAVCNFSM6AAAAAA7CL4JD6VHI2DSMVQWIX3LMV43ASLTON2WKOZRHE4DEOBZGMYTSNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


update last_commit_lsn in commit_config

HomeObject call HS is_ready_for_traffic before put/get/del, and if it return false, HO will reject requests. In HomeStore, we use m_commit_upto_lsn>=m_traffic_ready_lsn to determine if it is ready, so we need to update m_commit_upto_lsn every time raft commit a log. Currently we only update it in app log, so this pr add the update in config log.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 65.99%. Comparing base (1a0cef8) to head (fad812f).
Report is 116 commits behind head on master.

Files with missing lines
Patch %
Lines

...rc/lib/replication/repl_dev/raft_state_machine.cpp
66.66%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #622      +/-   ##
==========================================
+ Coverage   56.51%   65.99%   +9.48%
==========================================
  Files         108      109       +1
  Lines       10300    10994     +694
  Branches     1402     1508     +106
==========================================
+ Hits         5821     7256    +1435
+ Misses       3894     3010     -884
- Partials      585      728     +143

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


Fix mx dns lookup

Fixes failure in MX DNS lookup when one MX record fails to get an IP address.

Thanks! I have updates in the v0.1.2 branch that fixes the travis errors, so don't worry about that.
Also, inserting an empty array on DNS Error may cause problems using that data. Perhaps we should no create an item. (return nil instead of [] and to a .compact on the resulting array?)

Thank you, @afair.  Sounds good to me.  Updated with that change. :)

Hi @afair,
Should I leave this PR on master or move it to v0.1.2, just wondering when you plan to do your next release.
Cheers,
-Laura

Thanks, @lauraeci, I'll handle it. I have other work as well.


ignore files in /lang

Hi,
I'm using laravel-vue-i18n package: https://github.com/xiCO2k/laravel-vue-i18n
And as the doc says:

During the npm run dev execution time, the plugin will create some files like this php_{lang}.json on your lang folder.

Given that laravel-translations-checker will already be checking .php lang files, is it possible to ignore this php_{lang}.json files please?

Hi @valh1996 so this is currently not supported, let me see if I can hack this up for you.

Just merged the pr and will create a new version for you, should be available in the next hour or so.

Closing for now, but if it does not work feel free to re open.


Adds support to the Image_streamer_golden_image resource

Description
Adds support to the Image_streamer_golden_ resource
Issues Resolved
#99
Check List

[x] New functionality includes testing.

[x] All tests pass ($ rake test).

[x] New functionality has been documented in the README if applicable.

[x] New functionality has been thoroughly documented in the examples (please include helpful comments).

[x] Changes are documented in the CHANGELOG.

RESOURCES.md missing. PS. I believe Plan Script is also missing from it.

:+1:
I will add this docs for Plan Script, OS Volume, and Golden Image

I will add the docs for Plan Script, OS Volume, and Golden Image

I added the Resources for the Puppet Module for HPE Image Streamer. Can you check again?

@fgbulsoni @tmiotto I made the suggested changes.
Can you review again?


feat(error): add error handling helper

Proposal for helper error handling:

compile time type checking for expected errors
runtime type matching for expected errors

import { makeErrors, makeApi, matchErrorByPath, Zodios } from '@zodios/core';

const commonErrors = makeErrors([{
  status : 401,
  schema: z.object( ... )
},{
  status: 'default',
  schema: z.object( ... )
}
]);

const api = makeApi([{
  mehod: 'get',
  path: '/users/:id',
  ...
  errors: [
    ...comonErrors,
    {
        status: 404,
        schema: z.object({
          message: z.string(),
          specificTo404: z.string(),
        })
     }
  ]
},{
  mehod: 'get',
  path: '/users',
  ...
  errors: comonErrors
}]);

try {
  result = await zodios.get('/users/:id', { params: { id: 1 } });
} catch(e){
  const match = matchErrorByPath(zodios.api, 'get', '/users/:id', e);
 if (match.type === ZodiosMatchingErrorType.ExpectedError) {
   if(match.status === 404) {
     // error is now correctly inferred and typesafe at runtime
     match.error.response?.data.specificTo404 // correctly inferred
     //                             ^? type = string
   }
 }
}

Hi, @ecyrbe, I appreciate you came up with a solution so fast. Here is what I thought:
I am against on ZodiosMatchingErrorType
I don't think ZodiosMatchingErrorType is a good idea.

With ZodiosMatchingErrorType, as an zodios user, I need to know too much.
In catch block, I usually only care about handling a ZodiosMatchingErrorType.ExpectedError.
If I ever care about any case of ZodiosMatchingErrorType, it better to check each case individaully.
Why matchErrorByPath and check match.type === ZodiosMatchingErrorType.ExpectedError again?

I thought the aforementioned code example could be rewritten in a cleaner way:
try {
  const result = await zodios.get("/users/:id", { params: { id: 1 } });
} catch (err) {
  // Redundant
  // I thought it should never be on production
  if (err instanceof ZodiosError) {
    return;
  }

  // Redundant
  // It should definitely be a AxiosError, right?
  if (!axios.isAxiosError(err)) {
    return;
  }

  // see next section for isErrorByPath
  if (!zodios.isErrorByPath("get", "/users/:id", err)) {
    throw err;
  }

  // err.response.data is now a union of schemas defined by makeErrors
  if (err.status === 401) {
    // handle
    return;
  } else if (err.status === 404) {
    // handle
    err.response.data.specificTo404; // correctly inferred, and response definitely exists
    return;
  }

  // edge case we fail to think of
  throw err;
}

In short, if a user cares about all cases of ZodiosMatchingErrorType, he/she should check for each one instead of rely on a super versatile zodios.matchErrorByPath
A ErrorResponse to replace ZodiosMatchingErrorType
I would suggest a dedicate ErrorResponse that either extends AxiosError or native Error.
The flow is like:

axios get response and it an error response and throw err
zodios parse err successfully
then thrown new ErrorResponse(err)

ErrorResponse is representing a parsed error response and is capable of being narrowed.
And by doing so, the api zodios.matchErrorByPath() can be changed to a type gaurd  zodios.isErrorByPath().
More about zodios.isErrorByPath()
zodios.isErrorByPath() is a type gaurd  that doing two things

check if a given err is a ErrorResponse
check if a given err is produced from a given request (identified by method and path).

By only checking it in catch block (as removing the codeblocks I marked redundant), it can effectively imply the error is parsed safely and axios.isAxiosError.

@QzCurious Thank you for the feedback,
For the redundant things, they are definitly possible :

validation is also done by default on production builds, if you want you can disable it, but zodios will not magically do it for you.
unfortunately, errors can be anything because you can have memory issues, plugin issues, any knind of errors, because this is javasccript

I like the isErrorByPath idea.  it's definitly simpler, i agree. I was not thinking about typeguards at all.
I'll update the PR with this idea.
One thing though, i'm thinking making this independent of zodios instance and making this a free form function:

this will allow it to be used in hooks context easier
better tree shacking to have shorter bundle size if you do not use it

@QzCurious this is now available in v10.4.0 of @zodios/core

@ecyrbe
By ctrl-f ZodiosMatchingErrorType on Files changed tab, you may now safely remove ZodiosMatchingErrorType from the codebase?

validation is also done by default on production builds, if you want you can disable it, but zodios will not magically do it for you.

I decided to only validate response on development. Because, for my experience, backend would produce a slightly off schema data from time to time. Either it's in purpose, ex: lose the date-range constraint a little bit, or the source of data is not easy to be sanitised.
In consequence, I also prefer not to do any side effect on zodios response schema (zod transformation). For this, I might raise a issue to discuss about if we should separate validation and transformation in future.

unfortunately, errors can be anything because you can have memory issues, plugin issues, any knind of errors, because this is javasccript

That's a good call. I usually not overly handle each possible error case. And the uncaught error would eventually be thrown by the latest throw statement:
  // edge case we fail to think of
  throw err;

And if any error is thrown by the latest throw, I would figure out how should I deal with it at that time.


Docs: fix typo in CloneSetup.md from repo to repository

Description
Typo in CloneSetup.md file:
I would change :
repo  -> repository
Screenshots

🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

Can you assign this issue to me

Hi @Yash-Sajwan24 , you can start to work on this

I have addressed and submitted the pull request. @tamalCodes Please look at it whenever you can using the following link: https://github.com/MilanCommunity/Milan/pull/1167.


Upscale not working

Hi,
as the title says, upscale is not working.
I have no clue why and I don't know how to enable it.
I am searching for Infos since days now, but I cannot find any.
Even if I try the "example-2-high-quality.png", I still get only the normal resolution.
Can you give me any hints on what I'm doing wrong?
Thanks!

@phail216 You close this issue as completed. Can you please say if you found a solution?

@Anoromi Hi, I did not "found" a solution, it just started working after restarting and reloading multiple times. Although I have no clue why it did not work in the first place.


use custom vector allocator to prevent exceptions

Since some programs will compile with -fno-exception and we want zero chance of this library causing the program to abort, switch to a custom Allocator for the vector storage of the uv_buf_t.
In order to do this we need to change the API to return the pointer instead of a reference to the vector.

Using https://github.com/nodesource/nsuv/pull/24 instead since it requires fewer memory allocations.


build automatic dependencies installation solution for domainlab and snakemake

from the official documentation of snakemake, https://snakemake.readthedocs.io/en/stable/getting_started/installation.html
the dependencies are installed via conda and mamba.
For ease of user, either we want a docker or another dependencies resolution scheme for one line command installation
@schoersch , @xinyuejohn , how have you installed snakemake on the cluster? also using mamba? or a minimal installation via pip works as well?
Since mamba will create new conda environment, then one have to install all dependencies of DomainLab into the new conda environment?

@smilesun Currently, snakemake is installed using pip on the cluster. Maintaing a single "requirements.txt" for pip installations should be enough, right?


Add precision param for pdot

Currently pdot has no precision parameter. When it is lowered we lose information about the default_matmul_precision decorator and the precision is always default. This PR adds in the precision parameter and plumbs it to the lowered dot_general.

Can you add a test?

Added a test that looks at the jaxpr and the vmapped jaxpr.

Instead of checking the jaxpr, can we just check the behavior (of whether the precision is preserved)?

Updated to check jaxpr text instead of looking through the data structure.


Demo app does not work

The demo app link in the README gives an error

Wasn't down, just sleeping.


Extract supervision controller to a standalone module

Coverage remained the same at 56.403% when pulling e36612cb638f515c5e45d22ef5189f7dbaf16a32 on extract-supervision-controller into 6e12bc21301bdd6a56541f3203a141aa5b1ca3f3 on master.


First vagrant up db fails

mbp-13-2:p4-kitchen-sink localadmin$ vagrant up db
Bringing machine 'db' up with 'docker' provider...
==> db: Docker host is required. One will be created if necessary...
    db: Vagrant will now create or start a local VM to act as the Docker
    db: host. You'll see the output of the `vagrant up` for this VM below.
    db:
    db: Importing base box 'phusion/ubuntu-14.04-amd64'...
    db: Matching MAC address for NAT networking...
    db: Checking if box 'phusion/ubuntu-14.04-amd64' is up to date...
    db: Setting the name of the VM: p4-kitchen-sink_default_1421246112634_19859
    db: Clearing any previously set forwarded ports...
    db: Clearing any previously set network interfaces...
    db: Preparing network interfaces based on configuration...
    db: Adapter 1: nat
    db: Adapter 2: hostonly
    db: Forwarding ports...
    db: 22 => 2222 (adapter 1)
    db: Running 'pre-boot' VM customizations...
    db: Booting VM...
    db: Waiting for machine to boot. This may take a few minutes...
    db: SSH address: <IP_ADDRESS>:2222
    db: SSH username: vagrant
    db: SSH auth method: private key
    db: Warning: Connection timeout. Retrying...
    db:
    db: Vagrant insecure key detected. Vagrant will automatically replace
    db: this with a newly generated keypair for better security.
    db:
    db: Inserting generated public key within guest...
    db: Removing insecure key from the guest if its present...
    db: Key inserted! Disconnecting and reconnecting using new SSH key...
    db: Machine booted and ready!
    db: Checking for guest additions in VM...
    db: Configuring and enabling network interfaces...
    db: Mounting shared folders...
    db: /vagrant => /Users/localadmin/Desktop/p4-kitchen-sink
    db: Running provisioner: shell...
    db: Running: inline script
    db: stdin: is not a tty
    db: OK

    db: Reading package lists...
    db: Building dependency tree...
    db:
    db: Reading state information...
    db: The following extra packages will be installed:
    db:   aufs-tools cgroup-lite git git-man liberror-perl lxc-docker-1.4.1
    db: Suggested packages:
    db:   git-daemon-run git-daemon-sysvinit git-doc git-el git-email git-gui gitk
    db:   gitweb git-arch git-bzr git-cvs git-mediawiki git-svn
    db: The following NEW packages will be installed:
    db:   aufs-tools cgroup-lite git git-man liberror-perl lxc-docker lxc-docker-1.4.1
    db: 0 upgraded, 7 newly installed, 0 to remove and 148 not upgraded.
    db: Need to get 7,864 kB of archives.
    db: After this operation, 36.7 MB of additional disk space will be used.
    db: Get:1 http://us.archive.ubuntu.com/ubuntu/ trusty/universe aufs-tools amd64 1:3.2+20130722-1.1 [92.3 kB]
    db: Get:2 http://get.docker.io/ubuntu/ docker/main lxc-docker-1.4.1 amd64 1.4.1 [4,420 kB]
    db: Get:3 http://us.archive.ubuntu.com/ubuntu/ trusty/main liberror-perl all 0.17-1.1 [21.1 kB]
    db: Get:4 http://us.archive.ubuntu.com/ubuntu/ trusty-updates/main git-man all 1:1.9.1-1ubuntu0.1 [698 kB]
    db: Get:5 http://us.archive.ubuntu.com/ubuntu/ trusty-updates/main git amd64 1:1.9.1-1ubuntu0.1 [2,627 kB]
    db: Get:6 http://us.archive.ubuntu.com/ubuntu/ trusty/main cgroup-lite all 1.9 [3,918 B]
    db: Get:7 http://get.docker.io/ubuntu/ docker/main lxc-docker amd64 1.4.1 [2,090 B]
    db: dpkg-preconfigure: unable to re-open stdin: No such file or directory
    db: Fetched 7,864 kB in 16s (489 kB/s)
    db: Selecting previously unselected package aufs-tools.
    db: (Reading database ... 74120 files and directories currently installed.)
    db: Preparing to unpack .../aufs-tools_1%3a3.2+20130722-1.1_amd64.deb ...
    db: Unpacking aufs-tools (1:3.2+20130722-1.1) ...
    db: Selecting previously unselected package liberror-perl.
    db: Preparing to unpack .../liberror-perl_0.17-1.1_all.deb ...
    db: Unpacking liberror-perl (0.17-1.1) ...
    db: Selecting previously unselected package git-man.
    db: Preparing to unpack .../git-man_1%3a1.9.1-1ubuntu0.1_all.deb ...
    db: Unpacking git-man (1:1.9.1-1ubuntu0.1) ...
    db: Selecting previously unselected package git.
    db: Preparing to unpack .../git_1%3a1.9.1-1ubuntu0.1_amd64.deb ...
    db: Unpacking git (1:1.9.1-1ubuntu0.1) ...
    db: Selecting previously unselected package cgroup-lite.
    db: Preparing to unpack .../cgroup-lite_1.9_all.deb ...
    db: Unpacking cgroup-lite (1.9) ...
    db: Selecting previously unselected package lxc-docker-1.4.1.
    db: Preparing to unpack .../lxc-docker-1.4.1_1.4.1_amd64.deb ...
    db: Unpacking lxc-docker-1.4.1 (1.4.1) ...
    db: Selecting previously unselected package lxc-docker.
    db: Preparing to unpack .../lxc-docker_1.4.1_amd64.deb ...
    db: Unpacking lxc-docker (1.4.1) ...
    db: Processing triggers for man-db (<IP_ADDRESS>-1) ...
    db: Processing triggers for ureadahead (0.100.0-16) ...
    db: ureadahead will be reprofiled on next reboot
    db: Setting up aufs-tools (1:3.2+20130722-1.1) ...
    db: Setting up liberror-perl (0.17-1.1) ...
    db: Setting up git-man (1:1.9.1-1ubuntu0.1) ...
    db: Setting up git (1:1.9.1-1ubuntu0.1) ...
    db: Setting up cgroup-lite (1.9) ...
    db: cgroup-lite start/running
    db: Setting up lxc-docker-1.4.1 (1.4.1) ...
    db: docker start/running, process 4096
    db: Processing triggers for ureadahead (0.100.0-16) ...
    db: Setting up lxc-docker (1.4.1) ...
    db: Processing triggers for libc-bin (2.19-0ubuntu6) ...
==> db: Syncing folders to the host VM...
    db: Mounting shared folders...
    db: /var/lib/docker/docker_1421246198_40531 => /Users/localadmin/Desktop/p4-kitchen-sink
A Docker command executed by Vagrant didn't complete successfully!
The command run along with the output from the command is shown
below.

Command: "docker" "ps" "-a" "-q" "--no-trunc"

Stderr: time="2015-01-14T14:36:39Z" level="fatal" msg="Get http:///var/run/docker.sock/v1.16/containers/json?all=1: dial unix /var/run/docker.sock: permission denied. Are you trying to connect to a TLS-enabled daemon without TLS?"

Stdout:
mbp-13-2:p4-kitchen-sink localadmin$
mbp-13-2:p4-kitchen-sink localadmin$
mbp-13-2:p4-kitchen-sink localadmin$ vagrant up db
Bringing machine 'db' up with 'docker' provider...
==> db: Docker host is required. One will be created if necessary...
    db: Docker host VM is already ready.
==> db: Warning: When using a remote Docker host, forwarded ports will NOT be
==> db: immediately available on your machine. They will still be forwarded on
==> db: the remote machine, however, so if you have a way to access the remote
==> db: machine, then you should be able to access those ports there. This is
==> db: not an error, it is only an informational message.
==> db: Creating the container...
    db:   Name: db
    db:  Image: mysql
    db: Volume: /var/lib/docker/docker_1421246198_40531:/vagrant
    db:   Port: 23306:3306
    db:
    db: Container created: 24f627e380d05ac7
==> db: Starting container...
==> db: Provisioners will not be run since container doesn't support SSH.

Vagrant is no longer actively supported.


dynamic enable special tokens

remove "encode/decode_special_tokens" parameters in encode/decode method, add global enable_special_tokens option and turn on depend on the existence of add_tokens.json

This would be a breaking change to the API. And quite a few use-cases rely on controlling when added tokens are encoded or decoded. E.g. not encoding special tokens removes the need to sanitize user input, and it's necessary if you want to be able to encode something like </s> as text rather than as a control symbol.


feat: Scrub/ignore by predicate function

I'm trying to introduce Verify to our large existing code base and one area of friction is that we have thousands upon thousands of dynamic properties (auto-generated IDs) that we want to ignore or scrub, which all follow a similar pattern, they all end in ID. While it's possible to use reflection at startup to find all those properties and Scrub/Ignore them one by one, it's a bit wasteful as only a small subset of them will ever pass through a call to Verify, we have to pre-emptively ignore them ahead of time. For this, I think the ability to provide a callback function that gets called at the same point the normal "is property ignore/scrub dictionary" is a good approach. The callback can be based on either the string name of the property (or anything that's not a property like a dictionary key) or on the MemberInfo of the property if you need more information to determine if the property should be ignored or not.
I did not include benchmarks as I hope from the implementation it should be clear that there shouldn't be any performance impact from this, especially not if you don't opt-in to this feature, but let me know if you must have them.
Todo:

Docs I guess?

This is based on the discussion here:
https://github.com/orgs/VerifyTests/discussions/1345

@dotnet-policy-service agree [company="New Black"]

yep will need some docs.
can u also add static settings

@SimonCropp is that on public static partial class VerifierSettings? If so, those are included here. If not, then I'm afraid I need a pointer where exactly.
As for docs: happy to write some examples, but there's some auto-generation of docs going on and I couldn't get it to work.
What I tried was adding a #region ScrubMemberByPredicate around my tests and then adding this to the docs:
## Scrub members by predicate

<!-- snippet: ScrubMemberByPredicate -->
<!-- endSnippet -->

But if I then build the solution, it simply removes that section from the docs. What exactly is the flow to add docs?

@JulianRooze , it would be using https://github.com/SimonCropp/MarkdownSnippets, but I imagine the doc generation is handled by the workflow here


Rabin-Karp Algorithm

The Rabin-Karp Algorithm is a string-searching algorithm that uses hashing to find a pattern within a text. It works by calculating a hash value for the pattern and for each substring of the text with the same length as the pattern. If the hash values match, it checks the actual substring to confirm the match. This technique is efficient for multiple pattern searches or when dealing with large texts, as it reduces the number of string comparisons.
Please assign me

merged!


Handle FITS headers that have no history

otherwise would crash

The travis failure seems to be unrelated to the code.


Wrong variable in a_filter

Hi,
first of all thanks for the great monad library! I used it in a project and it was way more pleasant to compose monads than with the existing library of the BAP project.
I noticed a typo in the definition of a_filter, it should be C.cons curr ys instead since xs is the original list.
https://github.com/Denommus/monadic/blob/697fa651b49d51b018bb598fc0c0810754bdb1e5/library/monad.ml#L317

@addap hey, addap, sorry for the extreme delay, but I pushed an update for that on master.


TextEditingControllers

I have created in my controller a generator for the texteditingcontrollers in the forms as such
var textEditingControllers = {};

....

..

 addTextEditingController({String textEditingControllerToken, snapshot}) {
//    textEditingControllers.value.clear();
  print('Adding controller called from $textEditingControllerToken with $textEditingControllers');
    if (textEditingControllers[textEditingControllerToken] == null)  {
      print('fetching new');
      String textValue = snapshot?.info(textEditingControllerToken) ?? ''; // value if obtained correctly and set as initial value in the text field through the controller
    Rx<TextEditingController> textEditingController = new TextEditingController(text: textValue).obs;
    textEditingControllers.putIfAbsent(textEditingControllerToken, () => textEditingController);
      print(textEditingController);
       }

      return textEditingControllers[textEditingControllerToken];
  }

and in the textfield I am consuming the controller like this:
controller: steps.addTextEditingController(textEditingControllerToken: IncomeDetails.AFTERTAXSALARY, snapshot: viewSnapshot).value,
where steps is my controller;
IncomeDetails.AFTERTAXSALARY is a String representing the text field
snapshot hold the initial data of the text field (and the controller is successfully updating the initial value)
when changing the value of the text in the controller like:
steps.textEditingControllers[textEditingControllerToken].value.text = result;
my page does not rebuild (the page is a dialog),
if I exit the dialog and open it again I can see the changes in the text field.
is there anything wrong implementing this controller. I can provide more code if needed

Hi, try to use the onChange of your TextField/TextFormField and change the value from an RxString.
I think that using Rx<TextEditingController> will not work in any way, because it is a kind of reactive Controller that uses ValueNotifier, so you are not accessing or accessing its value directly.

thank you, I will try that

Here how it worked:
1- no observer (.obs)
in my controller :

var textEditingControllers = {};

.....
 addTextEditingController({String textEditingControllerToken, snapshot}) {
    if (textEditingControllers[textEditingControllerToken] == null)  {
      print('fetching new');
      String textValue = snapshot?.info(textEditingControllerToken) ?? ''; // value if obtained correctly and set as initial value in the text field through the controller
   TextEditingController textEditingController = new TextEditingController(text: textValue);
    textEditingControllers.putIfAbsent(textEditingControllerToken, () => textEditingController);
      print(textEditingController);
       }

      return textEditingControllers[textEditingControllerToken];
  }

for each text field i am creating a controller at the time of building unless it does exits I am assigning its value
In page one, I am consuming the controller
controller: steps.addTextEditingController(textEditingControllerToken: IncomeDetails.AFTERTAXSALARY, snapshot: viewSnapshot).value,
and requesting another page to do some calculations and send back a new value for the controller
GeneralDialog.show(child: PaymentFrequencyCalculator(IncomeDetails.FAMILYTAXBENEFIT));},
(A customised dialog the uses get from within it)
In the payment calculator, I am making updates on the controller
                      steps.textEditingControllers[textEditingController].text = result;
so this way I am able dynamically to create controllers and use them using a text value for each field.
last thing I am disposing the controllers by
  dispose: (temp) {
    steps.textEditingControllers.clear();
  },

Okay, can I close then?


Adding custom validators to completers

You had an API-breaking bug in your PR. Fixed in 105b1f1.

Hi @kislyuk, thanks for the fix. And of course, sorry for breaking the api.


distrobox: Using the GPU inside the container with --nvidia does not work

Describe the bug
distrobox added the --nvidia flag for container creation in 1.5.0 which allows containers to use the libraries and drivers of the host system for Nvidia GPUs.  (currently distrobox>=1.5.0 needs to be installed from the unstable channel)
However, this option does not seem to work on NixOS. It seems that distrobox relies on FHS paths to detect Nvidia files and libraries in order to bind mount them to the container.
Steps To Reproduce
Steps to reproduce the behavior:

Install distrobox (>=1.5.0) on a machine with an Nvidia GPU.
Create a container with --nvidia flag.
The container cannot access the GPU driver files that were installed in the host system.

Expected behavior
Containers created with the --nvidia should be able to leverage Nvidia GPUs without having to install the Nvidia drivers in the container as well.
Notify maintainers
@AtilaSaraiva

I am searching for solutions as well, can we somehow still install the necessary drivers in the container and somehow still be able to use the GPU, or the linking between host resource and the container is not possible?


Cannot load demo (OSX)

Hey,
I run into this error every time I try to run the demo.
Couldn't find example data in current working directory (svviz-examples/example1). Shall I download it and decompress it into the current working directory? Y/n:y
Traceback (most recent call last):
  File "/usr/local/bin/svviz", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/site-packages/svviz/app.py", line 371, in main
    run(sys.argv)
  File "/usr/local/lib/python2.7/site-packages/svviz/app.py", line 285, in run
    args = commandline.parseArgs(args)
  File "/usr/local/lib/python2.7/site-packages/svviz/commandline.py", line 104, in parseArgs
    inputArgs = checkDemoMode(args)
  File "/usr/local/lib/python2.7/site-packages/svviz/commandline.py", line 84, in checkDemoMode
    cmd = demo.loadDemo(which, args.auto_download)
  File "/usr/local/lib/python2.7/site-packages/svviz/demo.py", line 78, in loadDemo
    if not checkForDemo(which, autoDownload):
  File "/usr/local/lib/python2.7/site-packages/svviz/demo.py", line 68, in checkForDemo
    """Shall I download it and decompress it into the current working directory? Y/n:""".format(which))
  File "<string>", line 1, in <module>
NameError: name 'y' is not defined

Same thing happens when I do Y, n, N, or just enter. I've tried installing/uninstalling, and using pip and pip3.6.

Thanks for your bug report.
I think I've fixed this in the latest version on github. You can try installing this development version by running pip install -U git+git://github.com/svviz/svviz.git. Let me know if this fixes your problem.

Perfect. Fixed.
Thanks!


New version: AsyPlots v0.2.1

Registering package: AsyPlots
Repository: https://github.com/sswatson/AsyPlots.jl
Created by: @sswatson
Version: v0.2.1
Commit: 747400aa129eddcdaf169cd522f82f59a469f89b
Reviewed by: @sswatson
Reference: https://github.com/sswatson/AsyPlots.jl/commit/747400aa129eddcdaf169cd522f82f59a469f89b#commitcomment-35120103

Please add (at least)
[compat]
julia = "1"

to your project file and retrigger the bot.


Adding Options

I can't quite figure out how to get options working in ember-cli-build.js.
I've tried variations on, for example :
var app = new EmberApp({
"ember-cli-pickadate": {
'disable': [
1, 4, 7
]
}
});
Am I getting the format incorrect?

The only option supported in ember-cli-build.js is theme - that is a little misleading in the readme. I'll make it clearer.
For your use case, you'd need to pass the options directly to the component.
{{pick-a-date options=options}}

options:  {
  "disable": [1, 4, 7]
}

The options parameter is passed through directly to pickadate.js

Ah of course that makes sense, cheers :)
Although stil haven't got it quite working. I added an object as a computed property on my controller.
  extraPickadateOptions: Ember.computed(function () {
    return {
      min: new Date(2015,3,20),
      max: new Date(2015,7,14)
    }
  })
``
And I try and pass this into the component itself.
{{pickadate options=extraPickadateOptions}}

The object computes fine, but the component doesn't seem to get it in time. I get

Uncaught TypeError: Cannot read property 'onClose' of undefined

in the console. Any idea?

EDIT - I figured it out - I was forgetting to pass it through a parent component, I have nested coponent.
That's it, it's working great, thanks for your help! And cheers for wrapping the plugin :)

Awesome! thanks :)


Read/write performance issues

I'm seeing some performance issues while reading / writing data using the dask-snowflake connector.
Specifically, the to_snowflake call in the code below (taken from the coiled docs) sometimes runs in ~25 seconds and other times takes >1min.
import coiled
import dask
from dask_snowflake import to_snowflake

cluster = coiled.Cluster(
    name="coiled-snowflake",
    software="coiled-examples/snowflake",
    n_workers=20,
    shutdown_on_close=False,
    scheduler_options={'idle_timeout':'2 hours'},
    backend_options={'spot':'True'},
)

from dask.distributed import Client
client = Client(cluster)

ddf = dask.datasets.timeseries(
    start="2021-01-01",
    end="2021-03-31",
)

ctx = snowflake.connector.connect(
    user=os.environ["SNOWFLAKE_USER"],
    password=os.environ["SNOWFLAKE_PASSWORD"],
    account=os.environ["SNOWFLAKE_ACCOUNT"],
)

cs = ctx.cursor()

cs.execute("CREATE WAREHOUSE IF NOT EXISTS dask_snowflake_wh")
cs.execute("CREATE DATABASE IF NOT EXISTS dask_snowflake_db")
cs.execute("USE DATABASE dask_snowflake_db")

connection_kwargs = {
    "user": os.environ["SNOWFLAKE_USER"],
    "password": os.environ["SNOWFLAKE_PASSWORD"],
    "account": os.environ["SNOWFLAKE_ACCOUNT"],
    "warehouse": os.environ["SNOWFLAKE_WAREHOUSE"],
    "database": "dask_snowflake_db",
    "schema": "PUBLIC",
}

%%time
to_snowflake(
    ddf,
    name="dask_snowflake_table",
    connection_kwargs=connection_kwargs,
)

And the read_snowflake call in the code below ran in ~40s in @fjetter's original demo video but takes >4min. in my case.
import coiled
from dask_snowflake import read_snowflake
from dask.distributed import Client
import snowflake.connector

# spin up cluster
cluster = coiled.Cluster(
    name="coiled-snowflake",
    software="coiled-examples/snowflake",
    n_workers=20,
    shutdown_on_close=False,
    scheduler_options={'idle_timeout':'2 hours'},
    backend_options={'spot':'True'},
)

client = Client(cluster)

ctx = snowflake.connector.connect(
    user=os.environ["SNOWFLAKE_USER"],
    password=os.environ["SNOWFLAKE_PASSWORD"],
    account=os.environ["SNOWFLAKE_ACCOUNT"],
)

SCHEMA = "SNOWFLAKE_SAMPLE_DATA.TPCH_SF100"

example_query=f"""
SELECT

    C_CUSTKEY,
    C_NAME,
    SUM(L_QUANTITY) AS sum_qty,
    SUM(PS_AVAILQTY) AS sum_avail_qty,
    MAX(P_RETAILPRICE) AS max_retail_price

    FROM {SCHEMA}.CUSTOMER

        JOIN {SCHEMA}.ORDERS
            ON C_CUSTKEY = O_CUSTKEY

            JOIN {SCHEMA}.LINEITEM
                ON L_ORDERKEY = O_ORDERKEY

                JOIN {SCHEMA}.PART
                    ON P_PARTKEY = L_PARTKEY

                    JOIN {SCHEMA}.PARTSUPP
                        ON P_PARTKEY = PS_PARTKEY

    WHERE PS_SUPPLYCOST > 11

GROUP BY C_CUSTKEY, C_NAME
"""

connection_kwargs = {
    "user": os.environ["SNOWFLAKE_USER"],
    "password": os.environ["SNOWFLAKE_PASSWORD"],
    "account": os.environ["SNOWFLAKE_ACCOUNT"],
    "warehouse": os.environ["SNOWFLAKE_WAREHOUSE"],
}

%%time
ddf = read_snowflake(
    query=example_query,
    connection_kwargs=connection_kwargs,
)

I'm running this from a notebook on an EC2 instance because the snowflake.connector still doesn't run on M1.
I've asked @pavithraes to run the code above on her local non-M1 machine to see if she can reproduce.

how large is the compute warehouse? this sounds like you might've used a smaller warehouse than I did. I'm not sure if the video shows what I used back then but I would simply suggest to bump that up.
Either way, it would be helpful if you can find out what takes that much time (e.g. a performance report). The dask snowflake connector doesn't perform any computations itself so I strongly suspect this is on snowflakes side and the most likely source is the warehous size

I've asked @pavithraes to run the code above on her local non-M1 machine to see if she can reproduce.

Here are my timings:

to_snowflake -- 1min 26sec
read_snowflake & print head -- 16sec

Confirming that this was an issue due to warehouse size and closing this issue.


cssbuilder.CSSBuilder has no field or method CreateVlist

Hi :wave:
Thanks for your work on this library.
We've been using this for some time, however since the latest update we can't build our app anymore.
$ go build -o build/app .
# github.com/speedata/bagme/document
../../../go/pkg/mod/github.com/speedata/bagme@v0.0.0-20231122194752-d1948a8d651e/document/document.go:82:22: d.cssbuilder.CreateVlist undefined (type *cssbuilder.CSSBuilder has no field or method CreateVlist)

We detected this with golangci-lint after upgrading the library:
$ golangci-lint run
[REDACTED].go:25:2: could not import github.com/speedata/bagme/document (-: # github.com/speedata/bagme/document
../../../go/pkg/mod/github.com/speedata/bagme@v0.0.0-20231122194752-d1948a8d651e/document/document.go:82:22: d.cssbuilder.CreateVlist undefined (type *cssbuilder.CSSBuilder has no field or method CreateVlist)) (typecheck)
        "github.com/speedata/bagme/document"
        ^

And looking in the commit history, it was removed in commit 53a959b.
Our setup:

Go 1.22.4 linux/amd64
github.com/speedata/bagme v0.0.0-20231122194752-d1948a8d651e
github.com/speedata/baseline-pdf v0.0.0-20240621111510-9e70fb550141
github.com/speedata/boxesandglue v0.0.0-20240621111718-e173db49977c
github.com/speedata/css/scanner v0.0.0-20201005071706-2ed32c3ce553
github.com/speedata/gofpdi v1.0.18
github.com/speedata/hyphenation v1.0.2
github.com/speedata/textlayout v0.0.0-20230827181055-b7ff752e85ae

I will push my local changes as soon as possible, probably on Monday, I have changed a lot and it might break things, but I will correct the mistakes...

Thank you very much for the bug report!

I have updated some packages, but there are still some loose ends. I will keep working on it the next days hopefully


Add Bypass list support

First of all, I'd like to thank you for creating a great menu app, and I found it extremely useful. I would like to check the possibility if we can include the "bypass list" feature for the proxy? this is a standard settings in almost all the proxy configurations, and would be a much needed addition to this app.

Hello @sayanroy7, and thank you for your kind words! 🌟 I appreciate your suggestion about the "bypass list" feature for proxies, and I agree it could be a valuable addition. I'm pleased to inform you that I've already planned to include this feature in the upcoming updates. You can expect it to be part of the program in the next two or three weeks. I appreciate your patience and understanding. If you have any more feedback or suggestions, feel free to share them. Thanks again for your support! 🚀

Hello @sayanroy7,
I have added the bypass proxy list feature to the Proxy Manager app in the settings. You can download the new version from the releases.


Added support for use of units as subdirectory when dependencies are also subdirectories

Added check for gsl::gsl-lite to avoid find_package if targets already added by add_subdirectory(...) and did the same for fmt::fmt and ranges-v3, with out these changes, impossible to use library if dependencies are also subdirectories, complains about not being able to find package (when it no longer needs to)

Thanks!
Unfortunately, I have mixed feelings about that. On one side, I understand your point, and I see how this PR fixes your problem. On the other hand, doing dependency management via add_subdirectory is really wrong, and I wouldn't like to encourage it or complicate the code logic to provide support for such scenarios.

definitely agree, but the problem is 7 fold:

This library still advertises subdirectory support,
this library even advertises header only support, as in drop in place headers, an infinitely worse practice.
We still need escape hatches when our tools fail us, makes it hard to troubleshoot things that have nothing to do with your library on our end if we can't isolate the library from the package management chain.
VCPKG and conan only got "good" in 2017, many projects predate this, and many critical features weren't available in either until 2021.  Package management in C++ is very new, and there's still a lot of gotchas.  Certainly I wouldn't suggest the equivalent in something like Rust.
I haven't been able to use conan across two companies corporate networks, and it's left the conan devs scratching their heads, and leaves conan unusable in many nominal environments.
VCPKG is "roughly" universal, and I do use it, but it's offline functionality is rather... lacking, and getting export and --only_download to work for my problem has been problematic, and does not work right with manifest mode, which is also something everybody should just be using according to MS, but these problems leave us with out ideal solutions.
I need to use this project in an offline environment, the above solution fixes what would have taken months of work into minutes to get working.

The practical reality on the ground is why other libraries support such use cases out of the box, we have no other real option. Trust me, myself and the thousands of other engineers that work in these kinds of environments would much rather be using package managers, but unlike rust's cargo, or heck even PIP, neither Conan or VCPKG provide adequate solutions to these problems. IME, 99% of the people who have the same issues simply don't attempt to use libraries like this if they don't immediately work offline. I know at my company, I'm at least the fourth person to attempt to use this library, and the only person to attempt to actually stick with it despite the dev ops problems associated with it, and I was hoping this would kind of pave the way for further usage.


Why not use sentence-mask task in GLM-130B

Hi,
Thanks for your great job. The GLM paper GLM: General Language Model Pretraining with Autoregressive Blank Infilling has three pretraining tasks (span-mask, sentence-mask, doc-level-mask). Why GLM-130B only use span-mask and doc-level-mask tasks but discard sentence-mask task ? Thanks.

UPDATE.
I've read the issue about missing [sMASK], can you share the detail about since it harms the efficiency of pre-training. It means performance degradation or train speed become slow ? Any numbers provided will be good. Thanks.

Hello @mx8435, this has nothing to do with training efficiency, we just remove [sMASK] for simplicity because we didn't come up with a unified solution to split paragraphs into sentences for Chinese, English and code data.


14.2 Not Available on Docker Hub

Hi There,
Just thought I'd bring it to your attention that version 14.2 is not yet available on Docker Hub container registry. Perhaps a build issue? (I observe some of the project's previous action runs are failing: one, two, and three. Is this related?)
Thanks,
David

Thanks for the quick response! Sorry for not reading beforehand. Cheers!


SolidWorks 2017 SP5.0-generated STL model triggers DoesNotMatchExactly("facet normal")

Hi,
I used to be able to import SW-generated STL files using CADMesh 1.x a few years ago. Recently I updated to 2.0.2 and presumably also a newer version of SolidWorks, SW 2017 SP5.0, to restart the project. I am getting error

The CAD file appears to contain incorrect syntax:
Error around line 1: Facets are indicated by the tag 'facet normal'.

from DoesNotMatchExactly("facet normal"). Is there a setting I missed when converting SW parts to STL or is this a bug? I am also attaching the STL I tried here. Thanks!
-- Yen-Yung
update 200707: I can import your bunny.stl, so it must be something wrong with SW-generated STLs...
update 200708: found the problem being the first line of the STL file

solid assembly - 300K_can-1
facet normal -9.993034e-002 0.000000e+000 -9.949945e-001
outer loop
vertex 1.867302e+002 5.450940e+001 3.775120e+000
vertex 1.765321e+002 1.742470e+003 5.317435e+000
vertex 2.131847e+002 1.742470e+003 2.110749e+000
endloop
endfacet
...

It doesn't like the original name "assembly - 300K_can-1," nor "300K_can" or "300Kcan," but accepts "Kcan" and "kcan." I guess numbers and special characters are not allowed.

Thanks for reporting this. It looks like only names with letters are accepted. I will fix this right away.

@Yen-Yung The CADMesh.hh header file in the master branch should fix this problem for you now.


Testing and Deployment

[ ] Write unit and integration tests for your views, models, and forms.

[ ] Prepare for deployment using Docker.

[ ] Deploy your application on a platform like Heroku or DigitalOcean (optional).

using render for deployment
testing are being written together with development


[2017.3 Mac] Error when trying to quit unity + console has no theme

GetManagerFromContext: pointer to object of manager 'MonoManager' is NULL (table index 5)
This is an error I'm getting when trying to quite unity with Theme enabled project.
Also console doesn't get themed unless I re-apply the theme while the console is open.
This kinda makes the whole thing unusable for me.

This is also an issue on Windows Unity 2017.4.0f1 personal. Running the game is stil working and since I got used I get eye cancer running Unity without.
Fix would be really awesome!

I had the same issue on Unity2017.3.1f1 on Windows.
Somehow I got mine fixed.
You need to get the Codebase and re-build it yourself.
First you need to find UnityEditor.dll and UnityEngine.dll of your editor then drop it in Codebase/.Solutions/ .
Then you need to get

ThemeSystem.cs from "Codebase/Systems/Interface/Themes/Editor/"
AssemblyInfoThemes.cs from "Codebase/.Solutions/Properties/"
Again, drop them in Codebase/.Solutions/.
Before you can build, you'll need to change the name of "AssemblyInfoThemes.cs" to "AssemblyInfo.cs" then in the file, edit "AssemblyVersion" and "AssemblyFileVersion" as instruction commented in the file.
After build you'll see something like "ZiosTheme2017.x.x.dll" rename it to just "ZiosTheme.dll" and use this to replace the one in release version.

Earlier this year, I released a 2017.3 stopgap release for both Windows and Mac posted on the official Unity forums thread.  If I can get some feedback/conversation, open a dialogue, and get a few testers, I'll continue development further to Unity 2018 and missing styles for newer EditorWindows.


Enable querying of position index in event pool from an event handle

L0 event interface involves position index inside event pool. However, given an event handle, there is no way to tell what the index was, which complicates the situation when people freed the event and want to reuse the slot. It would be nicer to have an interface querying the index from the handler, or return it when free the event like zeEventDestroyAndReturnIndex(ze_event_handle, uint32_t &index)

hi @CaoZhongZ . Thanks for the comment.
When an application is creating an event, it needs to pass an index to the pool. So it is the expectation that the application can have a mapping of L0 event and index, to know which index is available for reuse.
More, if an application wants to reuse an event for a new workload, it doesnt need to actually destroy it and create it again, All that it needs is to call zeEventHostReset to reset it to state when it was first created.

Oh I see, thanks!


Fix auto resume for Honda Bosch, less spam

fix for Auto resuming on Honda Bosch radar. Spamming resume all the time causing jerking for no reason in heavy traffic, this change keeps car in a "Stopped" state. Car resumes like normal once the lead car actually starts driving forward. changes made to carcontroller.py and carstate.py.
I'm not the author of these fixes, but i always put them in my fork because it's too jerky without it.

15 is too large set that to 8 so accord or other vehicles that return lead_distance from radar_hud will resume too slow if its on 15.

seems like a lot of code for what it does. I wondering if there is an indication from stock camera/radar on when its a good time to start spamming. Some sort of "lead vehicle is moving away" bit we can use.

https://github.com/gregjhogan/openpilot/commit/a5a55c67dfd9e5edeec936e53af5b7c1bdbb803e


Trigger Postbuild Status XCS Magic Fix

magic in XCS dictates that postbuild triggers have to have a status of 2, otherwise they don't run. why? what are other values? nope. nada. you shall not know.

Result of Integration 1
Duration: 1 minute and 8 seconds
Result: Perfect build! All 49 tests passed. :+1:
Test Coverage: 52%.


Mac OS - No prompt to restart after downloading updates

This has to do with the default settings being set not to apply the update and therefore not notify the user. Furthermore, when changing the setting it was not correctly set in the program and was not applied until the program was restarted. The default is now to apply the update after being downloaded and the settings are properly set.


Remember me feature broken when 2FA enabled

Before submitting your issue, please make sure that you've checked all of the checkboxes below.

[x] You're running the latest release version of Deployer.
[x] Ensure that you're running at least PHP 7.0.8, you can check this by running php -v
[x] You've ran composer install --no-dev from the root of your installation.
[x] You've ran npm install --production from the root of your installation.
[x] You've ran rm -rf bootstrap/cache/* from the root of your installation.
[x] You have restarted the queue listener and node socket server.

Describe the bug
When 2FA is enabled, the "remember me" feature is broken.
I checked the source code, I noticed a variable naming bug.  Easy to fix.
// LoginController.php line 105
if ($auth->user()->has_two_factor_authentication) {
	$this->session->put('2fa_user_id', $auth->user()->id);
	$this->session->put('2fa_remember', $request->has('remember'));
}

And
// LoginController.php line 146
$remember = $this->session->pull('2fa_login_remember');

It's saved into the session as 2fa_remember and gathered as 2fa_login_remember
I could fix it myself via a pull request but I'd better wait you merge first otherwise I fear there will be many conflicts right?
Thanks

Hmm, good catch!

Fix merged


Mocha Suite Parse Error

2 macs, same Node version, same npm version, same mocha local version... mine works, her's fails. We even updated her Mac to El Capitan, same problem.
If we compare the suite variable after the pre-require event, mine appears correctly parsed, her's does not. If someone could tell me what this line of code does, maybe I could solve it. It's in mocha.js 's Mocha.prototype.loadFiles method.
suite.emit('require', require(file), file, self);
I ask because by the time gets to after the loadFiles method call in Mocha's run, my suite is correctly parsed, her's is not.
Mine:
suite: Suite {
  title: '',
  ctx: {},
  suites:
   [ Suite {
       title: 'Array',
       ctx: {},
       suites: [Object],
       tests: [],
       pending: false,
       _beforeEach: [],
       _beforeAll: [],
       _afterEach: [],
       _afterAll: [],
       root: false,
       _timeout: 2000,
       _enableTimeouts: true,
       _slow: 75,
       _bail: undefined,
       delayed: false,
       parent: [Circular],
       file: '/Users/jessewarden/Documents/_Work_Projects/Circus/test3/basic.spec.js' } ],

Hers:
suite: Suite {
  title: '',
  ctx: {},
  suites: [],
  tests: [],
  pending: false,
  _beforeEach: [],
  _beforeAll: [],
  _afterEach: [],
  _afterAll: [],
  root: true,
  _timeout: 2000,
  _enableTimeouts: true,
  _slow: 75,
  _bail: undefined,
  delayed: false,
  _events: { 'pre-require': [ [Function], [Function], [Function] ] },
  _eventsCount: 1 }

Apparently there is a deeper problem with Node path resolving on her machine, and mocha just revealed it... like all good testing should!


Fix a couple of warnings

A couple teeny little warning fixes:

Fix format warning in vjolt_collide.cpp. Even though the code works fine, it trips -Wformat warnings in GCC/Clang.
move(x) -> std::move(x) is to fix a dumb warning introduced with clang 16.

Worlds smallest PR :trollface: