[SPARK-15226][SQL]fix CSV file data-line with newline at first line load error

What changes were proposed in this pull request?
fix CSV file data-line with newline at first line load error.
How was this patch tested?
test as here describe;
https://issues.apache.org/jira/browse/SPARK-15226

Can one of the admins verify this patch?

I think if we really need to solve this problem, we need a option for unescaped quote handling.

In addition, I think a test is needed for this as well in CSVSuite. Also it would be nicer if we don't comment the original code. I blieve current change breaks the existimg tests.

@HyukjinKwon I run existing test against this patch and all pass. If need I will add a new test in CSVSuit.
And I think the only reason cause the bug is reading first line handling in the code. Because a row with multi-line in csv format is legal and the current csv datasource code using LIB com.univocity.parsers.csv also support it. Only when a data row with multi-line appear at file beginning will cause bug and I think this patch can fix it easily.

@WeichenXu123 External CSV data source supports this but has an issue for parsing unescaped quotes, here, https://issues.apache.org/jira/browse/SPARK-14103.
In this JIRA, I introduced the usage of UnescapedQuoteHandling to deal with the problem. So, if we need to support the original behaviour like the external CSV data source, we need an option to deal with the unescaped quotes.
Personally, I think we should not allow CSV parsing across multiple lines. CSV data source currently uses TextInputFormat which reads the data line by line. So, a record (across multiple lines) would mean a record across multiple HDFS blocks, which will end up with failing to read correctly.
So, I think we should not support this feature for now until we have a clear solution.

@WeichenXu123 Also, I realised the test in CSVSuite is only for data. If we have a test for header, this will fail. I mean some might want a header
["r1","r2","\"r"]

with the data below
r1,r2,"r
3",r4,r5
v1,v2,v3,v4,v5

while others might want a header below
["r1","r2","r3","r4","r5"]

@HyukjinKwon
En..current cvs load code use Hadoop LineRecordReader, so not allow a row split into mulit-lines, so I think the code should disable csv multi-line format, or should replace LineRecordReader with csv multiline supported reader.

@WeichenXu123 Currently it is possible to support multiple lines because the lines read from LineRecordReader becomes Reader (a byte stream) by StringIteratorReader which converts Iterator to Reader with manually inserted \n.
But supporting multiple lines has potential problems I said above and this is one of the reasons why I opened a PR here, https://github.com/apache/spark/pull/12268.


AR Engine/VR SDK Support for Overseas EMUI Devices

@dingsing2021 @Lixming  -  for overseas EMUI devices, is there a plan to have the model similar to Google Play Services for AR (AR Core) to help users to experience AR/VR?
Scenario:

User installs a WebXR browser via AG (on supported models like overseas versions of the models supported in China, including P30 Pro, P40 Pro, Mate 40 etc.)
As overseas device market do not have VR Engine/VR SDK as part of the file system the user will be prompted when launching the browser for the first time to download the VR Engine from AG. This is the AR Core experience -- where an AR app launch and requests AR Core download (if not available in the system). Once downloaded, Play Store Auto Update on WiFi will keep AR Core updated.

Note: WebXR experience for overseas market has to be limited to 3DoF because 6DoF tracking is not supported under Magic UI/EMUI. However, we think that should be acceptable now.
The main goal of this feature request is that there must be a working path for overseas developer to publish AR/VR app and users can easily experience it.

Closing due to no announcement to support overseas EMUI devices.


I have 2 pc streaming set up

I have 2 pc streaming set up
I been using pick/ban ui no problem
but I'm stuck on this one
http://localhost:9001/frontend
I though I only need to change localhost to ip address
http://<IP_ADDRESS>:9001/frontend
something like that but it doesn't show the overlay
when I put on chrome it does read as League Broadcast Hub
but it doesn't show the overlay
help me please

Workaround for now:

Download repo and open Overlays/ingame/variables.ts
Change backendUrl to your desired IP
In the folder Overlays/ingame open cmd and run npm run build
Copy Overlays/ingame/dist to your LeagueBroadcast Install/Frontend/ingame
Start LeagueBroadcast

I hope this works until I have the time to write a proper fix

I tried and I fail on step no.3 so I tried my way after I got an idea from ur help
I went to frontend > ingame and open main.js and search localhost and change to ip address and it work ^^
now I just need to figure it out how to edit the scoreboard for my own image

Both multi PC support and Overlay customization added in latest release

Hola, pudiste editar el fondo del scoreboard? :3


In Ruby 2.2.2 you get undefined method name for nil:NilClass when try to delete object from array

If you execute the test:
RAILS_ENV=test bundle exec rspec spec/requests/carto/api/imports_controller_spec.rb -e "returns oauth_valid false for not valid tokens and deletes them"
you are going to get fails in the tests but the root cause is:
undefined method name for nil:NilClass in
/home/ubuntu/www/production.cartodb.com/current/app/services/carto/data_imports_service.rb:128:in `delete_oauth'  /home/ubuntu/www/production.cartodb.com/current/app/services/carto/data_imports_service.rb:59:in `validate_synchronization_oauth'  /home/ubuntu/www/production.cartodb.com/current/app/controllers/carto/api/imports_controller.rb:30:in `service_token_valid?’

The problem is that the version prior to 3.2.22 don't support ruby 2.2.X so we need to upgrade to it.


[netdata] allow zero-len external route prefix

This commit allows a zero-len prefix ::/0 to be added as an external
route in Network Data. Such a prefix can act as a default route.

In new pushed commit added a testcase for zero len external route.


delete a running container

Netbox version
Plugin Version
Agent version
Docker API Version

3.7.1
1.6.2
20.0
1.43

Actual Behavior
Today it is possible to request to delete a running container in the plugin but it raises an error in the agent :
{"message":"You cannot remove a running container d199e8e066f49321ddea958aa8bfe20285f563e0140090fa80ea491e905de3f9. Stop the container before attempting removal or force remove"}
Desired Behavior
Possible options are :

prevent deletion running container from the plugin
force stop before deletion in the agent

opinions welcome

I would implement the feature inside the plugin. We should not force a stop in the agent if it's not asked to do so
The agent should only do what it is asked for and not implement some hidden tasks


Weird rotation in iPad?

So here is my code for creating the EAIntroView (using Swift)
    let titleColor = UIColor.blackColor()
    let descColor = UIColor.blueColor()

    let page1 = EAIntroPage()
    page1.title = "Some title"
    page1.desc = "Some deep"
    page1.titleColor = titleColor
    page1.descColor = descColor
    page1.bgImage = UIImage(named: "bg1")

    let page2 = EAIntroPage()
    page2.title = "page 2"
    page2.desc = "something again"
    page2.titleColor = titleColor
    page2.descColor = descColor

    let page3 = EAIntroPage()
    page3.title = "page 3"
    page3.desc = "again again again"
    page3.titleColor = titleColor
    page3.descColor = descColor

    let intro = EAIntroView(frame: self.view.bounds, andPages: [page1,page2,page3])
    intro.delegate = self
    intro.pageControl.pageIndicatorTintColor = UIColor.blackColor()
    intro.pageControl.currentPageIndicatorTintColor = UIColor.redColor()

    intro.showInView(self.view, animateDuration: 0.5)

But on my iPad, the following display in iPad:

How can I fix it? Thanks!

Also, it works fine on iPhone 6 Plus

Thanks for great demo of bug! I will take a look into it on a week.

Hi @ealeksandrov, I want to use in my iPad project. When this problem is solved? Thanks.

i have the same issue

Same issue here.
Does anyone know how to block rotate? I think this can be one workaround for this issue

Can anyone confirm this bug on device?
I struggled with debugging on simulator until tested it on my iPad 4. And it worked flawlessly, just as designed.

Tested:
iOS 9.3 simulators:

iPad 2 - working
iPad Retina - working
iPad Air - broken
iPad Air 2 - broken
iPad Pro - broken

Devices:

iPad 4 - working

Since I can't reproduce this on on my device and there is no possible reason for this behaviour - I consider it as iOS Simulator bug.

@ealeksandrov
My Device:
iPad Air (First Generation) - working
Thanks.

Thanks for reporting!
Closing as iOS Simulator bug.

iPad Air (First Generation) - broken :(

O_o
File a radar. I really can't see any reason for this behaviour. This is iOS autolayout issue.

I have the same problem. It seems the layout doesn't change so e.g. the portrait layout is still there even if you switch to landscape. Verified on iPad Air 2.
@ealeksandrov Can you block the rotation?

Will look for devices to test it.
Try to override shouldAutorotate in VC containing EAIntroView.

I can confirm this occurs with version 2.9.0 on an iPad Air running iOS 9.3.2.
It does change the layout on rotation, but it seems it shows the portrait orientation when rotating to landscape and vice versa.
A 'sort of' workaround is adding the pages again to EAIntroView after the rotation, but the animation remains incorrect of course.
It does however show the correct layout when first opening the intro view on any rotation.

I can't seem to reproduce this bug using your example app on the same iPad Air.
Tried out a couple of things, it doesn't happen.
I fail to see what's different in my project though.
I'm using EAIntroView in a presented ViewController.
This works perfectly in the example application.

Weird. Thanks for info, lets collect more feedback in this issue.

Can anyone update us on this bug with iOS 10? Particularly with projects and devices already experienced this issue on iOS 9.

Experiencing this problem on the iOS 10 simulator (for iPad Air)

Experiencing it on actual device as well (iPad Air 2) using iOS 9.3.1. Same as above, in that example works fine, but in project it does not.
I'll update to iOS 10 later and let you know if anything changes.

I also can reproduce this in iOS 10.1 simulator.
It looks like the problem is in UIDeviceOrientationDidChangeNotification handler. You're expecting there that all the frames have already changed after rotation. But it isn't always the case. An alternative would be to override layoutSubviews and do all the changes there.

Fixed by #191


Setup so that changes here can be automatically merged to downstream repos

https://github.com/marketplace/actions/fork-sync-with-upstream

combine with thsi
https://gist.github.com/moltar/afe0e541ac87ed07f1454520683140de


Add Logistic distribution

What are you trying to do?

I am trying to use Logistic CDF.
How does Gonum not allow you to achieve your goal?
Gonum does have a number of distributions but Logistic distribution is not one of them.
It is used in logistic regression and also particularly useful for rating calculations (e.g. Elo rating).
What version of Go and Gonum are you using?

go version go1.17.2 linux/amd64
gonum v0.9.3
Is this feature absent from the current master?
Yes
Are you able to help contribute the feature?

I suppose I can write an implementation but I am not familiar with gonum internals to do so efficiently.

Hi Maxim, thanks for your interest in Gonum. If you submit a PR with the initial implementation, I can help you to get it merged.

What are hard and soft requirements for implementation? I suppose it should be in distuv but is there something other than that?

@romanwerpachowski can you take a look at #1745?

Closed by #1745.


sched: fix kasan report error when open CONFIG_DEBUG_MM

Summary
sched: fix kasan report error when open CONFIG_DEBUG_MM
reason:

g_running_tasks = thread A
thread A exit (free thread A's tcb) -> thread B
thread B interrupt by irq
check g_running_tasks->flags -> kasan report used after free

rootcause:
g_running_tasks has't set completely when syscall hanppened
Resolve:
Use rtcb (get at ISR begining) instead
Signed-off-by: ligd<EMAIL_ADDRESS>Impact
DEBUG_MM
Testing
VELA

@pkarashchenko as the summary said that some code path(e.g. syscall) doesn't set g_running_tasks correctly.


[fix] prevent cached model from being overwritten [add] pass on **kwargs to load_model function

This PR makes the cached ModelLoader more robust preventing cached models from accidentally being overwritten. This is achieved by always returning a deepcopy instead of a reference to the cached model.

Now additionally includes a change to pass **kwargs to the TrainedModel load_model function for more flexibility. @KonstantinWilleke could you please have a look and approve if happy? (I extensively tested this PR)

Why do you want to pass **kwargs? The provided model table doesn't accept any additional arguments.

Why do you want to pass **kwargs? The provided model table doesn't accept any additional arguments.

I will open a PR to nnfabrik soon that introduces kwargs (I use them in my local version). On that note, I think the model cache actually would fit better to the nnfabrik repo; maybe something to be considered for the hackathon.


resource not found

when running the following code the first time in a cell in a Jupyter Lab notebook:
%use dataframe(0.8.1)
I get a the following exception:
JupyterIntegration:WARNING => A resource with classpath 'table.css' not found.
I have a custom exception handler installed, so not sure if it normally doesn't pop-up.
When I run same  %use dataframe(0.8.1) again, there is no error anymore and also the table rendering afterwards looks better.

Hi! What do you mean by a custom exception handler? What version of the Kotlin kernel? I couldn't reproduce this bug from the first try :(

versions used :
jupyter-client        7.0.6
jupyter-core          4.9.1
jupyter-server        1.11.2
jupyterlab            3.2.8
jupyterlab-pygments   0.1.2
jupyterlab-server     2.8.2
kotlin-jupyter-kernel <IP_ADDRESS>

I also checked a bit more my code and it is not the custom exception handler addThrowableRenderer(....) but rather the logging module. I see WARNING level errors in my output cell, but you should see it where ever you log the WARNINGS.

Hereby also two screenshots of the same table, once rendered after running the %use dataframe(0.8.1) the first time and once after running it for a second time. Clearly the CSS for the table is different.

@jbaron could you please check, is it reproduced on dataframe (0.10.0) or not if you have a time?
It will be helpful for us.

@zaleslaw So long ago, forgot all about this issue :)
But I can confirm that with dataframe 0.10.0 I don't get this error anymore. So this issue can be closed.

Yes, sorry for the long delay with answering your question.
Thanks for the confirming.
Hope in the future we will not keep issues without reaction for too long.
Have a good day!


Unable install on Windows

Getting this error:
> npm install utimes
npm ERR! code EBADPLATFORM
npm ERR! notsup Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"linux"} (current: {"os":"win32"})
npm ERR! notsup Valid os:  linux
npm ERR! notsup Actual os: win32

NodeJS v21.5.0
Windows 10

The package referenced in the error message<EMAIL_ADDRESS>is unrelated to this package. I performed a fresh installation of this package on Windows 10 and inotify was not found in the lock file nor node_modules.
Most likely, you have another package in your project that is calling upon inotify, but hasn't been installed yet. Running npm install utimes will install all other remaining dependencies in the project as well.

My mistake, I was using incorrect parameter: --save=dev instead of --save-dev


More Nix Update

I enabled tests by default in the nix package, simplified instructions, and checked that everything works fine. Ready for merge.
However, even using --enable-extended-functional-tests, ./test/functional/feature_any_asset_fee_rbf.py is not automatically run. — I think it needs be added to test/functional/test_runner.py.


Remove -Werror flag from docker_cmake in favor of Jenkins's warning post processor

We will rely on the Jenkins warning post processor to track trends in warnings added from commits.

I think that's a wrong approach unless one is able to see warnings on github. Otherwise who would even look at warnings if builds pass?

The 12.10.1 Trilinos source is not patched in the Docker base image, so each time you start a new container you would not be able to compile unless you apply docker/jenkins/trilinos_stk.patch.

So? Youc could simply patch it in container, or run it with docker-compose.How is it different? And is it the only reason?

I do not want -Werror to be the default.  I agree we need a way to back communicate that.  So do we with the execution time...

I do not want -Werror to be the default.

I'd like to understand why not. Not opposing but curious.


[Feature Request] Text selection

Please add a feature for selecting a part of texts and copy them .
standard text selecting is good .

Assalamu 'Alaikum, brother.
Here is a new release with text copying feature :https://github.com/frrahat/Quran-For-My-Android/releases/tag/1.4.0
Please inform, if you find any bug.

I find a bug at this feature . when I press the plus sign and enter an invalid format of Verse:Ayah , instead of showing an error or help ( similar to the main page ) it gets force close .

Jazallahu khairan, brother.
I will fix this and let you know of the update, insha 'Allah.

Assalamu 'Aalaikum, brother.
Here is a new update: https://github.com/frrahat/Quran-For-My-Android/releases/tag/1.4.1


[Slider] Update SliderClassKey types

Stop showing error for ts projects when adding thumbIcon and thumbIconWrapper classes, which does already have support.
https://material-ui.com/lab/api/slider/#css-api

👍


[SPARK-50352][PYTHON][DOCS] Refine docstrings for window/aggregation functions - part 3

What changes were proposed in this pull request?
Refine docstrings for window/aggregation functions:
1, make examples copy-pasteable;
2, specify the meaning of str input: either column name or literal string;
3, show both input and output columns;
4, add see also section for related functions;
Why are the changes needed?
1, improve the documentation;
2, improve the doctest coverage;
Does this PR introduce any user-facing change?
doc changes
How was this patch tested?
updated doctests
Was this patch authored or co-authored using generative AI tooling?
no

Merged to master.


SSL handshake error

running tests I get an SSL error. Support told me "Please make sure you are disabling all ECDHE ciphers as they will lead to this error.". Didn't find any EC in the code, so no idea how to fix.
Traceback (most recent call last):
  File "/Users/ben/trading/python-bittrex/bittrex/test/bittrex_tests.py", line 31, in test_handles_none_key_or_secret
    actual = self.bittrex.get_markets()
  File "/Users/ben/trading/python-bittrex/bittrex/bittrex.py", line 78, in get_markets
    return self.api_query('getmarkets')
  File "/Users/ben/trading/python-bittrex/bittrex/bittrex.py", line 67, in api_query
    headers={"apisign": hmac.new(self.api_secret.encode(), request_url.encode(), hashlib.sha512).hexdigest()}
  File "/Library/Python/2.7/site-packages/requests/api.py", line 71, in get
    return request('get', url, params=params, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/api.py", line 57, in request
    return session.request(method=method, url=url, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 475, in request
    resp = self.send(prep, **send_kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 585, in send
    r = adapter.send(request, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/adapters.py", line 477, in send
    raise SSLError(e, request=request)
SSLError: [SSL: SSLV3_ALERT_HANDSHAKE_FAILURE] sslv3 alert handshake failure (_ssl.c:590)

What platform and what version of Python are you running on?
I cannot reproduce this issue on either OSX or Linux, using 2.7.13

I had the same error. I installed pyopenssl, pyasn1 and ndg-httpsclient. And now it works.
pip install -U pyopenssl pyasn1 ndg-httpsclient


test-1

Hi vecps-bot, I'm the tosone here serving you. You can find the instructions on how to interact with me at
<>.

Hi tosone, I'm the vecps-bot here serving you. You can find the instructions on how to interact with me at
https://baidu.com.

/assign

/kind feature


style: simplify assignments using assignment operators

This PR refactors assignments in the analyzer.go.
See https://go-critic.com/overview.html#assignop

Thanks for linting the linter.
Do you want a patch version bump for this ?


keg: make guile's site-dir persistent

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully run brew tests with your changes locally?

Things that install Guile scheme objects should be doing so in HOMEBREW_PREFIX/share/guile/site, which should be persistent rather than a symlink to any formula's Cellar, not even guile's necessarily as Homebrew/homebrew-core@f061d86 managed to expose.
I believe guile isn't actually configured correctly to expect this directory, but since absolutely nobody has complained to date as far as I can find & gnutls seems to be the only formula using it this way the temptation is to leave it misconfigured & use this directory for site schemes.
I don't really know the deeper issues around messing with guile's configuration so going for a minimal fix seems the safest option.

First CI failure seems to be because Rubygems was having some issues. Rescheduled the build.

👍


Custom sentences for noise-contrastive-estimate.lua

How can the code be modified so as to load a custom set of sentences (like e.g. in char-rrn or torch-rnn), rather than the Google billion words (GBW) dataset?

@namp The GBW dataset uses the MultiSequence loader. Just wrap your custom sequences in one of those.

Thank you for your reply.
Perhaps I should had rephrased the question: What the custom corpus input format should be like (e.g. one sentence per line? would a file like this in do? )
billionwords.tar.gz contains already processed ".t7" files and it is not quite obvious.
Thanks again

You will probably have to write some code for your specific dataset, MultiSequence takes a table of tensors. You can use util functions from dataload to make things easier, see here, specifically dl.buildVocab and dl.text2tensor should help.

The link was really helpful - apologies for the question if the task is trivial. I've been working in python for nlp tasks and I'm not that familiar with Lua.
Many thanks


Incorrect level calculations

Apparently our getLevel function in server/src/api/util/levels.ts is wrong.
getLevel(174) should be 3 (currently 2)
getLevel(1154) should be 10 (currently 9)
This affect build detection for 10hp pures

Looking at the experience formula shows that dividing by 4 is only supposed to happen at the very end. Currently this same division is done during each call to the getXpDifferenceTo function, but the issue is actually hidden by the fact that the remainder is also returned. Most of the time this seems to yield the correct result in the end, but there are some corner cases where the decimal part breaks things.
The easiest solution is to simply account for the remainder and change this condition to (exp >= Math.floor(accumulated) && exp < Math.floor(accumulated + required)). As the remainder issue still remains however, one should at least consider using a lookup table instead to always get veritably correct values.
PS. The same code also seems to be in /app/src/utils/levels.js.
PPS. Oddly enough, it seems like the xp remainder formula on that same wiki article is also incorrect for this reason. However, there seems to be a bit more going on, as the result from the exponent calculation is immediately rounded down which seems a bit questionable.

one should at least consider using a lookup table instead to always get veritably correct values.

The initial implementation had a lookup table, but somewhere along the way someone (can't remember who) changed it and it seemed to work fine until now


Add test cases for Sort command

Add test cases for:

List Command
Sort Command
Sort Command Parser

Also, decoupled Sort Command from Undo Command

Closes #136


Revert "Offline Support via ServiceWorker"

Reverts TryGhost/Ghost-Admin#523
Temporarily reverting in order to get the next alpha release out. Unfortunately this has the unintended consequence of the service worker caching a 200 response for /ghost/ so when starting a new blog the redirect to /setup/one that the server sends is never picked up.
This would only affect developers using localhost/people trying to set up a blog multiple times on the same domain but as that's the target audience for the alpha it's probably best to avoid any confusion.

Coverage remained the same at 75.307% when pulling 0b9f0745193a31ffb7b07bc74d61bff387dc0064 on revert-523-offline-ghost into acf63486815fe2fbb36e95882dcae80c5f582b82 on master.


Use JSON output for az commands

Script was failing if az default output isn't JSON. Setting AZURE_CORE_OUTPUT at the top of the script to force the output of all az commands in the script to be in JSON format (without affecting the user's default)

/test-force-approve
(I don't believe aad-app-reg.sh is exercised anywhere in CI)


fix: move packed artifact between stages

build job did not publish anything, so produce the npm packed .tgz file from it to the release stage.
npm package was missing files due to release stage not having lib directory available after building

How about we just add the compile step to the release job just before running pack? The build job exists only to ensure that we can build the package after every commit/PR. We don't have to integrate it with release.
I'm not sure if artifacts from build will be available automatically in release or if we need additional steps for that and this might need additional hit and trial. So I'm thinking to leave build as is and add the compile command to the release job.

But of course if you have already verified this or are very sure this is all we need, happy to merge this as well.


Kani is unable to build some crates due to package naming

Here's an example:
git clone https://github.com/serde-rs/json
cd json
cargo kani

This results in:
error: There are multiple `serde_json` packages in your project, and the specification `serde_json` is ambiguous.
Please re-run this command with `-p <spec>` where `<spec>` is one of the following:
<EMAIL_ADDRESS><EMAIL_ADDRESS>Error: cargo exited with status exit status: 101

The problem is that we use cargo_metadata to examine which packages should be built, then pass each down with -p. But that is ambiguous (apparently!) For example:
cargo build -p serde_json

gives the same error.
We can build the specific package with the suggested fix:
cargo build -p<EMAIL_ADDRESS>
But not with kani:
cargo kani -p<EMAIL_ADDRESS>thread 'main' panicked at 'Cannot find package<EMAIL_ADDRESS>kani-driver/src/call_cargo.rs:203:22

This is because our code tries to interpret package specs as if they were just package names.
The ideal fix would be to pass these arguments like -p transparently down through to cargo build without interpreting them.
As a bonus, this would parallelize builds of multiple packages and targets again.

IIRC, the reason kani interprets this argument and those loops exist was that invoking cargo/rustc the "usual" way caused the native linker to be invoked, which fails on our rlibs. We'd presumably have to figure out how to avoid that.

I have a branch here that makes assess generate pkg-specs instead of package names as arguments to -p:

https://github.com/tedinski/rmc/commit/988bde2dfbe6ac4d1b9c69ced5e65e950de02dfa
https://github.com/tedinski/rmc/commits/pkg-spec

But this doesn't work for the above reasons (Kani expects names, not specs), so it's not useful yet.


Fix Internal server error on cube generation [Cauldron]

Added chunk load before cube generation.
Exception: http://pastebin.com/ngwKqU7e

Cauldron forces worlds to unload even if they are programmed to not do that. You can configure this in your cauldron config and tell it to keep the Compact Machines dimension loaded.
Cauldron is a dead project, not accepting fixes for compatibility with it, sorry.
If you can convince me this is a bigger issue than cauldron servers, i might reconsider it.


DRAFT: MACE interface

Avoid ase calculator and as many Julia -> Python -> Torch pipelines as reasonably possible without re-writing MACE for Julia.

I've decided to move this to a separate package instead to avoid requiring Python in NQCModels.jl.
Instead, this PR now makes the ASE and ASE friction interfaces usable with PythonCall.jl to fit in the structure of using PythonCall and CondaPkg.jl for packaging Python dependencies.


feat: Render HTML based content

Summary

Render HTML content on User statement and discussion
Display first post in a topic (previously hidden)
Imporve twitter card

Closes: https://github.com/snapshot-labs/workflow/issues/189
How to test

Go to a discussion page. http://localhost:8080/#/s:lido-snapshot.eth/proposal/0xa478fa5518769096eda2b7403a1d4104ca47de3102e8a9abab8640ef1b50650c/discussion
Should be formated similar to https://research.lido.fi/t/organize-the-lido-alliance-program-as-a-lido-dao-adjacent-borg/8173/3

To-Do

[x] Find if some case is missing

Incorrect escape on [ and ] char on https://bafybeib554rbng74f3apb2angnydsm4xmzl4aqnspgqtwjpnjwefnzozo4.on.fleek.co/#/s:aave.eth/proposal/0xf87cf0761b27becf6c8d18bbb457c9e6bf6b7aa436cdb0d197ad2d93a495ed04/discussion

On http://localhost:8080/#/s:aave.eth/proposal/0x9e2e3bd26866212d0cbd8e7cefcfa21eec9522202fd25b02456b8ff59371db08/discussion
Rendering of the table is missing, and the post is cut (missing everything after the table)

On http://localhost:8080/#/s:apecoin.eth/proposal/0xaa3b8b16b4e6193e8c0feb8d9fd0fed9873659bc27ffa4ec9f5792f323f8c766/discussion
Another escape issue, search for \-@Facilitators

http://localhost:8080/#/s:apecoin.eth/proposal/0xff1829f8e2fb778946ef83050858a54582da50da3de3acb67c74f5e38c1da0aa/discussion
Issue with bullet point, always giving 1\. instead of 1.

ul and ol does not have a padding bottom of 8px, compared to all other tags, making post ending by these tags closer to the footer than the other post

On http://localhost:8080/#/s:safe.eth/proposal/0x2c2ad686e94fd97903de8ee520b90d400c98a1985e31b21416da095b28373f18/discussion
Some _ are escaped, search for @amy\_safe

It is just here discord id:

ul and ol does not have a padding bottom of 8px, compared to all other tags, making post ending by these tags closer to the footer than the other post

Same issue with table missing bottom padding:

In http://localhost:8080/#/s:apecoin.eth/proposal/0xaa3b8b16b4e6193e8c0feb8d9fd0fed9873659bc27ffa4ec9f5792f323f8c766/discussion
In the penultimate post, the "-@Facilitators" is linking to the same host (localhost:8080)

Can we merge this?


Downloaded episodes aren't detected

When you download an episode on your ipad and play the media from the downloads, it's not detected by the software and the auto skip doesn't work. On the exact same device it works fine if you're streaming an episode.

The whole script works by monitoring playback server side and then skipping during the correct time window. I imagine downloading the episode makes the server side activity disappear and the playback session invisible. No way around that

Confirmed downloaded content is invisible to the server


Language request: Typst

Language Name 🖊
Typst
https://typst.app
Logo 📷

Upstream support ✅

[X] Does tokei already support the language in question?

Looks like Typst isn't available in the latest release of Tokei yet

It is not documented in the README, but it has been implemented already, at least on their default branch.

Implemented languages in current latest release: https://docs.rs/tokei/13.0.0-alpha.1/tokei/enum.LanguageType.html
This was released 4 months ago, Typst support was merged in last month.

Oh yeah, sorry, again, I was looking at their defaupt branch rather that at their releases.

No worries! Unfortunately we can't be on the "bleeding edge" by depending on the default branch or any git refs, as that would prohibit releases to crates.io (also, stability concerns).

I see! Feel free to close this issue as "not planned" then.

Oh, we can still eventually support Typst :slightly_smiling_face:
I just mean it won't be possible right away

Okay, thank you so much, then!

The glyph for this filetype will be available at codepoint \uf37f when new NF releases new version.


Add option to disable the use of SSL

Docs for https://github.com/iterative/dvc/pull/1428

👍 good to merge!

Thank you!


Replace TravisCI with CircleCI

We should move our continuous integration service over to CircleCI. It's got a lot of advantages over Travis.

I'd love to help with this effort!


Compressed Images

Currently, ZapCapture is producing images with compression. Output files should not be compressed.

Looks like it is an issue with the gif format itself, rather than the writer. Video should be written to a different format than GIF to preserve quality.


List of available version

Is there somewhere a list of all versions that can be used?
I just tried for example version IC-14.0.4 which seems to be not available.

Hi, the full list of available versions you can find at https://www.jetbrains.com/intellij-repository/releases and https://www.jetbrains.com/intellij-repository/snapshots.
Indeed, there are problems with 14.0.4 because sources archive is missing, I'll fix the plugin for this case, thank you for report.

0.0.38 is published


Bug: Site loading became super slow

Description
Here is my site: https://samsmerrygoround.github.io/
And here is the repository: https://github.com/samsmerrygoround/samsmerrygoround.github.io
The site used to load very quickly, but recently it has been loading slowly. It takes quite a minute to refresh and load all the content on the page. I'm not sure how to fix this issue. Any advice is greatly appreciated!

I faced the same issue, and found out it took too much time to fetch dependencies from cdn.bootcdn.net.
Setting the the sources to unpkg both in _config.yml and in _data/variables.yml fixed it for me.
Hope this helps !

After messing around with branches, eventually I made the changes as suggested and the problem is fixed. Counterintuitively, it was affected under MI Note 7/Firefox only. Again thank you so much!!!


feat(logs): Add error logs for io.Copy if error isn't EOF

It seems as though on io.Copy, an EOF constitutes the normal pattern of a successful copy, but I am checking for io.EOF anyway just to be safe.
If the error isn't an io.EOF, I think having a log message indicating what went wrong might be good for the user experience.
Let me know your thoughts, what do you think? @andydunstall
Right now in k8s as we are using piko in TCP mode, we are seeing the connection close on the agent side for some peculiar reason (not sure what it is yet), but the error log would be good to have as general diagnostics.
Also if the log messages can be better, let me know. I just went with something pretty general:
failure to copy...

Hi @yquansah, thanks for the contribution! As mentioned in another thread, TCP support is still quite new so great to find issues (at some point I'm hoping to look at TCP error handling, such as opening the connection to the forward address before considering the tunnel created)
What kinds of errors are you expecting to see other than net.ErrClosed? Trying this out I'm getting a lot of "error":"read tcp <IP_ADDRESS>:8000-><IP_ADDRESS>:56166: use of closed network connection" which probably shouldn't be logged, as that's expected behaviour? So I'm just wondering what other errors this could be useful to catch?

@andydunstall You are right. My thinking here was that it could be a catch all for unexpected errors, just as a safety net that users can turn on when necessary. A debug log as you mentioned would indeed make more sense here. I'll switch it to that.

@yquansah Added https://github.com/andydunstall/piko/pull/144 which includes your commits plus a bit of tidy up (shortens log line and removes EOF check) - if that looks ok will merge
(I would have pushed to this PR though as its a fork I don't think I can)

@andydunstall Indeed! I approved your PR.


Firebase reading and writing works on one phone but not on another

I have an app where I read and write from the firebase database (currently using 15.0.0). Everything works fine with a nexus phone but when using a different phone (moto g5), I am unable to read or write to the database. I have tried upgrading the dependency version to the latest with no success. I have had success with database communication using android 7.0 emulated and 8.0 on an actual device (nexus). Is there some firewall or privacy issue that I am missing that is preventing the moto g5 phone to not work when communicating with the database?

@brownninjaturtle sorry for the late response but I am going to close this issue because there is no evidence provided that it has to do with FirebaseUI.  Please try a more general forum (like StackOverflow) to get help with this.


Document consensus rules from Zcash spec: 7.1.2 Transaction Consensus Rules

TODO
Open a zcash/zips PR to split the spec section into multiple sub-sections. That way, there will be links to each sub-section, and the rules will be easier to find.
Tasks
For each consensus rule:

check that the rule is implemented in Zebra
make sure the latest version of the rule is quoted in the docs for the code that implements it
make sure the code implements the latest version of the rule

Split

[x] #3425
[x] #3426
[x] #3427
[x] #3428

Consensus rules
https://zips.z.cash/protocol/protocol.pdf#txnconsensus

Implementation

#1917

PRs:

#3069
#3076

#1980

PRs:

#2229
#2234
#2236
#2246

#1981

PRs:

#2371
#2437

#2103

PR: #2441

#2105

PR: #3039

@mpguerra this is a very large task, so I'd like to split it between multiple people.

@mpguerra this is a very large task, so I'd like to split it between multiple people.

We can possibly split it between the people that worked on the listed issues/PRs. Before we start working on this we should double check that there aren't any other issues/PRs that contributed towards implementing this section of the spec.

It would also be nice if these issues could be used by the auditors as a reference and so we should make sure the description is as complete and accurate as possible

@mpguerra this is a very large task, so I'd like to split it between multiple people.

We can possibly split it between the people that worked on the listed issues/PRs. Before we start working on this we should double check that there aren't any other issues/PRs that contributed towards implementing this section of the spec.

Thanks, please re-assign me to some of the sub-tasks after you've split this task.
Here is the split I'd suggest, because it matches the structure of Zebra's code (and also the spec):

transaction header (including expiry height)
transaction size (including item counts)
coinbase transactions (including coinbase-specific item counts and values)
shielded pools: the rule is specific to a shielded pool (this is different from the network upgrade range of the rule, even if the network upgrade has the same name as a shielded pool)

It would also be nice if these issues could be used by the auditors as a reference and so we should make sure the description is as complete and accurate as possible

This is unusual - I've never seen a project that keeps closed tickets up to date as the code changes. So I assumed that closed tickets are out of scope for the audits. Tickets are based on what we knew at the time they were closed. But we want to audit the current state of the software, so we need something that stays up to date.
So I think we should update the documentation next to the code we're getting audited. That way, we can maintain that documentation as we change the code, so it will be ready for future audits.
Is it confusing to use open tickets to track questions for the auditors?
Let's talk about auditing closed tickets and auditor questions at the Engineering sync.

It would also be nice if these issues could be used by the auditors as a reference and so we should make sure the description is as complete and accurate as possible

This is unusual - I've never seen a project that keeps closed tickets up to date as the code changes. So I assumed that closed tickets are out of scope for the audits. Tickets are based on what we knew at the time they were closed. But we want to audit the current state of the software, so we need something that stays up to date.
So I think we should update the documentation next to the code we're getting audited. That way, we can maintain that documentation as we change the code, so it will be ready for future audits.

I wasn't suggesting that we update the closed tickets. I think the tickets as they are can contain useful information about how and where a consensus rule was implemented in addition to the documentation within the code. This will obviously only be valid for the NU5 network upgrade and I assume we'll do an audit before any new network upgrade is scheduled.

All the sub-tickets in this ticket have been completed.


Plot neural responses

Summary:

Created media directory for storing atlas file. Used "mock.pickle" for now.

Added a form (NeuronInputParamForm) in form.py with validation code to handle user input. Used hidden fields for both stim_neu_id and resp_neu_ids.

Added code in view.py to get validated values for form fields, and then call wormfunconn functions to generate data for plotting neural responses.

New template file (plot_neural_responses.html) for form input and output.

Minor changes to navbar.html

Notes:

Ran Superlinter locally and passed. No new test code, although user input parameters can be easily verified as they are listed with graph in output section.

Tried to get main steps of the web app. in place. Used "vanillaSelectBox" and form hidden fields for selecting neurons. I am open to suggestions or a different approach.

Keep list_input_params.html and related code for now (as references for testing purpose). Plan to delete them later.

Need suggestions on how to handle exceptions while calling wormfunconn functions. I used "raised ValueError..." in views.py, but not sure if it's the right way to handle it. The current version of wormfunconn only worked for stim_type=rectangular.

I am adding some test code, and will push new commits next Monday. If you haven't reviewed this PR, it's better to wait till next Monday.

Just pushed a commit. Added error catch for generating html output ((plot_div) from ployly.offline.plot function in views.py. Since plot function runs validation and including plot.js in output as default, the html output should render a graph is no error is raised.  Also modified and added code to test.py - check form status and expected output or errors.
Ran Super-linter locally and passed. However, GitHub Superlinter failed with the error: Docker pull failed with exit code 1.

@fkang and @jcmatese I believe the linting failure was just a temporary issue, however, I noticed that we were still on v3 of superlinter and decided to take the opportunity to bump to v4 and use the "slim" image which might help speed things up. If you use that when running locally, I've noticed it does speed things up a bit. I'm taking a look at the PR now.

@fkang and @jcmatese : While I'm always happy to have feedback from @hepcat72, I don't want to overburden people with reviews. Since Rob is already working on https://github.com/PrincetonUniversity/neuron-graph, we can let him off the hook for reviewing these PRs (though your welcome to if you feel you have the time Rob).

It's totally fine with me. I'll let Rob to decide if he wants to review this PR.  I can wait till tomorrow morning.

I'm fine reviewing. Just wrapped up the updates to the TB data submission PR.  If in the future, I'm slow to review, I wouldn't be offended if you merge without my review.  Or wait, or poke-tag me.  Whatever you want to do is fine by me.

@hepcat72 Rob, thanks a lot for you detailed review and suggestions, especially the code for invalid reps_neurons.  I will make changes accordingly. Just address your main comments here instead of one-by-one:

form filed names and descriptions: I used those listed in Francesco's wormfunconn package. I won't make changes until I get feedback from him. His code can only handle one stim_type right now, so I expect more changes will be made on his end.

debug code: plan to do a final clean up when we are close to move the code to production. In the next stage of development/testing, I'll still need those lines of debug code.

stim_neu_id and resp_neu_ids selections: I want to use JavaScript code from VanillaSelectBox (a nice web front for multiple selections with auto suggestion feature). I tried different options and also had a discussion with John. The approach I used is the best I can come up with right now.

@lparsons I made some minor changes based on Rob's suggestions. Tested again with test.py and from web front. I had to fetch/merge the changes from the remote branch (from your PR7 initially) to my local branch before pushing new commits to GitHub. You may want to take a quick look before I merge this PR to main. Thanks.


Add back our custom implemetation of sync.WaitGroup

We have been using a custom implementation of sync.WaitGroup to fix a data race in the PacketConn code used by pion/dtls
See:

https://github.com/pion/udp/issues/74
https://github.com/pion/transport/issues/57#issuecomment-1694528604

It looks like sync.WaitGroup is in use in transport for connWG and there's no current issues around it being used, so I'll close this issue for now.


Use the webpack banner plugin for UMD builds

Waiting on this to land first: https://github.com/DefinitelyTyped/DefinitelyTyped/pull/14993


Filter component for tablets and Mobiles

https://www.figma.com/file/A0OEqjIWy6jL4Rk4kT8ddr/UX%2FUI_Workflow?node-id=3970%3A22390&t=Fa5B8Nj2Z1dvb0O0-1

@TvrtkoM What's the status on this?

Ok, I see it's marked "Done". Re-open if it's not done.


Store intermediate build files in version control to support binder experimentation

It would be nice to allow high-level experimenting in binder. The computer requirements are way to high to run the entire process, so storing the intermediate state files (e.g., forecast.rds, gbm_TRUE.rds) will allow higher level experimentation. I think we should be good on size, but we should check.

bbs_data.csv and bbsforecasting.sqlite are both > 100 MB uncompressed, which will prevent us putting them in git.


Style Image rotation is anti-clockwise on WebGL

I noticed that regularshape-canvas.png and regularshape-webgl.png are different.
But they both test, among other thing, the rotation option for ol.style.RegularShape, which is suppose to be clockwise.
The correction seems so easy (6648c82f8f301c7387fedaedaa4cdca8c9f08ad3) and the differences are so obvious that I wonder if there is a technical reason for that that I don't think about.

I'm not familiar with ol's source, even js, sorry.
But if the only problem is the rotation direction, clockwise or anticlockwise,  and you got the right position, that should be ok.  I think the equation are right:
clockwise x degree == anticlockwise -x degree, so x degree should be replaced by -x degree
sin(x)-->sin(-x) equals -sin(x),
cos(x)-->cos(-x) equal cos(x),
however,  the code in gberaudo's comment might be more readable and safe.


Venkatesh/test buildkite usersettings

:nut_and_bolt: Description: What code changed, and why?
Testing the buildkite failure issue for PR-5378

:chains: Related Resources
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[x] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[x] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[x] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[x] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[x] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

This is the test PR to monitor the buildkite cypress pipeline errors.


Auto-complete support

Pycharm Use code hints without triggering voice.

https://intellij-support.jetbrains.com/hc/en-us/community/posts/360008889679-Read-the-selected-auto-complete-result


Move away from fmt.Print and friends

Instead of using fmt.Print, which forces output to os.Stdout, this change moves us so that we use the fmt.Fprint family of functions, to allows us to control where output goes.
In places where we already have a cobra.Command, I have used cmd.OutOrStdout() out and friends to get the output stream we want to write to. For other components, The Console type now has a Handles() property that you can use to access Stdout, Stderr and Stdin.
This change also removes the ambient use of os.Stdout, os.Stdin and os.Stderr in as many places as possible. The only remaning locations are for setting up our context objects and some code in our process runner for the case where we are running an "interactive" command and we want to have our standard streams be used by the child. That will be cleaned up in a future change.

This has been pulled off from #849 with some changes.  The main goal of these changes is to stop ambient use of Stdout and friends.  I made some changes to the design such that this information now flows as a property of the Console object.  Thanks to @vhvb1989 for mentioning that the factoring felt wrong, it caused me to revisit it.  I think @wbreza will also like this a little more because we don't end up adding a ton of io.Writer arguments everywhere.

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

bash:
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879 --version '' --verbose

pwsh:
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/uninstall-azd.ps1' -OutFile uninstall-azd.ps1; ./uninstall-azd.ps1
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/install-azd.ps1' -OutFile install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879' -Version '' -Verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-879

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

bash:
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879 --version '' --verbose

pwsh:
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/uninstall-azd.ps1' -OutFile uninstall-azd.ps1; ./uninstall-azd.ps1
Invoke-RestMethod 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/install-azd.ps1' -OutFile install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879' -Version '' -Verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/879/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-879

@wbreza

It feels that maybe all calls to bring to stdout should be wrapped and using the Console.Message or add additional methods in there.

We can't use Message as is because it doesn't always write the output in some cases.  I'm not opposed to adding additional methods to console down the line that behave like the fmt. family of functions but use the values from the Handles() object, but it seems like accessing them directly for now is OK as well?
FWIW - I will want to use these handles in a future PR (so we don't have explicit calls to os.Stdout and friends in the command executor) so I will need the underlying streams to be accessible.


Added the updated_since and updated_until parameters to the survey_response query

It's pretty simple.
Please review and merge.
Thanks.

Sure


Implement IXMP4Backend

This PR is intended to aid @glatterf42 for developing model data features in ixmp4. It is not a complete plan or solution for migrating ixmp-based code, workflows, and research to ixmp4; such a plan would involve many additional elements that not in scope for this PR.
The PR:

Adds a IXMP4Backend class that allows message_ix (thus all existing code based on ixmp and message_ix) to talk to ixmp4's API. This functions as a shim between the established and new/developing APIs.
Makes minimal changes to allow ixmp.Platform backed by IXMP4Backend to be created and used.
Adjusts or parametrizes tests of the ixmp Platform, TimeSeries, and Scenario classes to run against both JDBCBackend and IXMP4Backend, ensuring the latter provides the same behaviour expected of the former per the test suite.

This is a precondition for migrating other code (for instance, in message-ix-models or message_data) to, first, use ixmp4 via an IXMP4Backend, and eventually, ixmp4 directly.

May serve as a basis for a parallel PR in message_ix.

Notes

ixmp4 is not published on PyPI for Python 3.8 and 3.9 and can't be installed in those CI jobs.

[ ] Separate ixmp4 from the tests dependency, and don't install it on those Pythons.
[ ] Adjust parametrized tests to not import ixmp4 or run against IXMP4Backend with those Pythons.
[ ] Document that IXMP4Backend is only available for Python >= 3.10.

How to review
TBD. This branch/PR may not necessarily be merged itself, but may serve as the basis for another PR that adds the IXMP4Backend.
PR checklist

[ ] Continuous integration checks all ✅
[ ] Add or expand tests; coverage checks both ✅
[ ] Add, expand, or update documentation.
[ ] Update release notes.

 Did you mean: '__le__'?

/opt/hostedtoolcache/Python/3.12.1/x64/lib/python3.12/site-packages/ixmp4/data/db/base.py:76: AttributeError

I guess this means ixmp4 does not support Python 3.12, although it is installable.


How can I create nested fragments with defaultValue?

Hi there,
I have the following model/fragments structure and I'm not sure how to handle the creation of nested fragments using defaultValue. Maybe the solution is obvious but I can't find it.
// app/models/document-template.js
export default Model.extend({
  name: attr('string'),
  settings: fragment('settings')
});

// app/models/settings.js
export default Fragment.extend({
  footer: fragment('document-element'),
  header: fragment('document-element'),
  margin: fragment('document-margin'),
  orientation: attr('string')
});

// app/models/document-element.js
export default Fragment.extend({
  center: attr('string'),
  content: attr('string'),
  left: attr('string'),
  right: attr('string')
});

I'd like for a new DocumentTemplate's settings to be populated with footer, header and margin by default. Is there a way to do this?
Thanks a lot! ❤

No activity in several years. Closing this.

I'm also interested by the subject 🤔, I'll try to find a way


CosmosDB: Missing option for storage capacity or incomplete documentation

With az cosmosdb collection create there is no way to set the storage capacity (in the Azure Portal either "fixed (10GB)" or "unlimited"). Since the command supports --partition-key-path which is only useable for unlimited (=partitioned) collections, I assume that the CLI always creates "unlimited" collections. If that's the case, then it should be documented. Currently it is quite confusing.

Hi @letmaik thank you for identifying this lack in documentation. At this point, we have deprecated the fixed storage option, so only "unlimited" collections can be created.


Add fix for old Python versions

When pinging Riak using Python 2.7.3, you see this:
Python 2.7.3 (default, Jun 22 2015, 19:33:41)
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import riak
/usr/local/lib/python2.7/dist-packages/riak/security.py:45: UserWarning: OpenSSL 1.0.1 14 Mar 2012 (>= 1.0.1g required), TLS 1.2 support: False
  warnings.warn(msg, UserWarning)
>>> nodes = [{'host':'<IP_ADDRESS>','pb_port': 10017,'http_port': 10018}]
>>> nodes
[{'host': '<IP_ADDRESS>', 'pb_port': 10017, 'http_port': 10018}]
>>> c = riak.RiakClient(protocol='pbc',nodes=nodes)
>>> c.ping()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/riak/client/transport.py", line 179, in wrapper
    return self._with_retries(pool, thunk)
  File "/usr/local/lib/python2.7/dist-packages/riak/client/transport.py", line 121, in _with_retries
    return fn(transport)
  File "/usr/local/lib/python2.7/dist-packages/riak/client/transport.py", line 177, in thunk
    return fn(self, transport, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/riak/client/operations.py", line 120, in ping
    return transport.ping()
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/transport.py", line 88, in ping
    codec = self._get_codec(msg_code)
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/transport.py", line 75, in _get_codec
    codec = self._get_pbuf_codec()
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/transport.py", line 52, in _get_pbuf_codec
    self.client_timeouts(), self.quorum_controls(),
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/feature_detect.py", line 163, in client_timeouts
    return self.server_version >= versions[1.4]
  File "/usr/local/lib/python2.7/dist-packages/riak/util.py", line 89, in __get__
    value = self.fget(obj)
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/feature_detect.py", line 215, in server_version
    return LooseVersion(self._server_version())
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/transport.py", line 80, in _server_version
    server_info = self.get_server_info()
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/transport.py", line 105, in get_server_info
    resp_code, resp = self._request(msg, codec)
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/transport.py", line 537, in _request
    resp_code, data = self._send_recv(msg_code, data)
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/connection.py", line 33, in _send_recv
    return self._recv_msg()
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/connection.py", line 158, in _recv_msg
    msgbuf = self._recv_pkt()
  File "/usr/local/lib/python2.7/dist-packages/riak/transports/tcp/connection.py", line 174, in _recv_pkt
    msglen, = struct.unpack('!I', msglen_buf)
struct.error: unpack requires a string argument of length 4

The comments in the code explain why.

👍 LGTM. If only people would update their stuffs.


Add support for artifact id in module import

See ceylon/ceylon@3599380cf3ef55ffa55ca9eed91bfb132c353d77 and ceylon/ceylon@81381228821bf7801fb35d9c1371fc9db74e20d0. Also, consider renaming repositoryType to just repository now that there’s a spec for it.

Done in branch 128.


Bootstrap-email styles applied to main application

I have installed the Bootstrap-email gem according to instructions and my main application styling has changed due to the addition of the styles. I assumed the styles of this gem would only be applied to the mailer where I specified the stylesheet link, but that is not the case.
Unfortunately I cannot provide an example due to workplace constraints, but I am hoping some insight might be available with the meager amount of information I am providing. Thank you.

Look at this page: https://bootstrapemail.com/docs/setup
You should be creating a new application stylesheet called application-mailer.scss for example and adding @import 'bootstrap-email'; to import the bootstrap email css. You should not be adding it to the main application.css file.
Is that not working for you?

I chose a different filename (email.css.scss), created the file in my
app/assets/stylesheets folder, and added the @import 'bootstrap-email'
directive. I configured my config/initializers/assets.rb to append the file
along with the other precompile assets, added my <&= "stylesheet_link_tag
'email', media: 'all' " %> statement in the head section of my
mailer.html.erb file, ran rake asset:clobber and rake asset:precompile, and
successfully tested my mailer.
What I can't figure out is how the bootstrap mailer styles are getting
wrapped up with my application.css.scss.
It is true that both style files are in the same folder
app/assets/stylesheets, is that the reason? I do think I have followed the
process as directed in the documentation, am I missing something?
On Tue, Jul 30, 2019 at 3:54 PM Stuart Yamartino<EMAIL_ADDRESS>wrote:

Look at this page: https://bootstrapemail.com/docs/setup
You should be creating a new application stylesheet called
application-mailer.scss for example and adding @import 'bootstrap-email';
to import the bootstrap email css. You should not be adding it to the main
application.css file.
Is that not working for you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stuyam/bootstrap-email/issues/33?email_source=notifications&email_token=AK4UY3PEO2BM7MWS3DCPGADQCDBA7A5CNFSM4IIASOP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3FRNIQ#issuecomment-516626082,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK4UY3OHHJ7ESHQVPI57N63QCDBA7ANCNFSM4IIASOPQ
.

I think I solved it.
I have been studying the Ruby on Rails guide for The Asset Pipeline, and
came across a few hints. I renamed my file from email.css.scss to
email.scss and moved it from app/assets/stylesheets to
lib/assets/stylesheets. Now my application is not including the email
styles, and so far my mailer is still working properly.
So I think in this case the answer is to use the lib/assets directory
instead of the app/assets directory for the bootstrap-email styles, because
Sprockets is going to precompile the subdirectories of app/assets whether
you like it or not.
I am very sorry to have troubled you, I will close my issue as solved.
On Tue, Jul 30, 2019 at 3:54 PM Stuart Yamartino<EMAIL_ADDRESS>wrote:

Look at this page: https://bootstrapemail.com/docs/setup
You should be creating a new application stylesheet called
application-mailer.scss for example and adding @import 'bootstrap-email';
to import the bootstrap email css. You should not be adding it to the main
application.css file.
Is that not working for you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stuyam/bootstrap-email/issues/33?email_source=notifications&email_token=AK4UY3PEO2BM7MWS3DCPGADQCDBA7A5CNFSM4IIASOP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3FRNIQ#issuecomment-516626082,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK4UY3OHHJ7ESHQVPI57N63QCDBA7ANCNFSM4IIASOPQ
.

Now if I can just get Outlook to render my badges, I will be a happy camper. :)


Possibility to export Contained HTML file with image URLs

Describe the feature you'd like
Is there an option to export book to "not contained HTML file" with original images urls instead of having them encoded to base64? I would like to create online docs (where book exported to html is a perfect solution) with images uploaded on my server where Bookstack runs.
Images encoded to base64 dramatically increase the filesize at the moment.
Possibility to have css file separated is the same feature for clear html output.
Did I miss any settings?
Describe the benefits this would bring to existing BookStack users
Clean html code with minimum size for online docs, with images hosted on server and separated CSS file.
Can the goal of this request already be achieved via other means?
No.
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundemental request

How long have you been using BookStack?
0 to 6 months
Additional context
No response

Hi,
and thank you for the response. Allright, we gonna test this REST API solution.
However, I think a checkbox with "don't convert images to base64" option in the settings would be great.

Since the above provided solutions should suit I'll close this off.

However, I think a checkbox with "don't convert images to base64" option in the settings would be great.

To be honest, that's not something I'd really look to add as that opens up a whole new category of options, which I try to avoid where possible.

:(


How to show Month name when pick date.

Example hear.

Thank you.

Not support that. You can reset the input value in your format after picked a date.

Not support that. You can reset the input value in your format after picked a date.

Thank chen.  I solved the problem by using events onChange.

can you send me any examples


Update to new sfstoolbox.org/en/3.0 permalinks

This upgrades all the permalinks to equations on http://sfstoolbox.org to the new http://sfstoolbox.org/en/3.0 version.

This is obsolete, given #103.


TOC applies to H1 headings, while H2 headings are indented

Before the commit

After the Commit

I personally feel that the TOC may be too cluttered, so we could restrict the amount of text for each section? (put a ...) Or at least make it scrollable like documented in the other issue

I also added scrolling (Only appears when headings are too long) . I made it take up approximately half the screen


Add new link

As title

Hey, @mmarif4u There was a conflict in your PR, so I just added the link separately in 68e67164eb964460de9c9cff715525a2803d2fa3
Thanks!


BENCH: Add random element access benchmark

Goal is to understand overhead vs accessing a python object directly.
!benchmark

lgtm


How to map new key?

How to map a new key to show controls?

In shader patch.yml look for Screen Toggle: 0xDC, it'll be under Developer:. The values are virtual-key codes: https://docs.microsoft.com/en-us/windows/win32/inputdev/virtual-key-codes Find the value on that page for the key you'd like to use and paste it there.
So if you want backspace for instance,

you'd put 0x08 as the value.
Sorry that it's not that user friendly to change. Changing it is a rare enough that developing a better way has never really been that high priority.


ESLPopup: positioning feature

As an ESL tech architect, I want to

see flexible mechanism of the position definition for ESLPopup
support different strategies of positioning ESLPopup

Done for ESLPopup beta


Restrict pilot and ingress controller

Restrict pilot and ingress controller to the namespace in which they run, by passing -a  to pilot discovery.
POD_NAMESPACE in ingress-auth fixes partially istio/pilot#1185.

/lgtm

/ok-to-test
/test all
/retest all
(no idea why the tests aren't started - @yutongz ?)

controller.go:60] Ingress controller running in namespace demo-test-7e65257c3fc74c10a883e4556b0, watching namespaces demo-test-7e65257c3fc74c10a883e4556b0
1 controller.go:60] Ingress controller running in namespace mixer-test-7cd966cd991f498ea98abac7e8, watching namespaces mixer-test-7cd966cd991f498ea98abac7e8

/test all

@andraxylia: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
16e1256011d7f737d962b613156b46d6c58b88eb
link
/test istio-presubmit

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

/test all

@andraxylia: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
9b82f556d8824fd594d9de47adee19f79158a722
link
/test istio-presubmit

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

@andraxylia: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
615d91667e26e1c009fbb1b0894a921c59ffc570
link
/test istio-presubmit

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

/retest

/lgtm
On Thu, Sep 7, 2017 at 6:40 PM istio-merge-robot<EMAIL_ADDRESS>wrote:

/lgtm cancel //PR changed after LGTM, removing LGTM. @andraxylia
https://github.com/andraxylia @ldemailly https://github.com/ldemailly
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/696#issuecomment-327946262, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH0qd4M87Z76KlAXfyeOLSLFcaB0Qm3uks5sgHDsgaJpZM4PQZhS
.

--
~shriram

Phew. 2 to go

Auth version is failing.

I think we can merge

e2e with auth is failing because productpage is crashing:
Error from server (BadRequest): previous terminated container "istio-proxy" in pod "productpage-v1-43061786-263s0" not found

"W0907 23:18:50.276] I0907 23:18:50.274614     889 testInfo.go:242] Fetching logs on mongodb"
How is this possible?

let's not merge yet because that would kill the auth job and we need the mixer logs of the auth job to understand the
W0907 22:56:59.687] [libprotobuf ERROR external/mixerclient_git/src/report_batch.cc:78] Mixer Report failed with: UNAVAILABLE

Is that caused by this test? If not merge and then reopen the mongodb pr as it might have some deps

it's unrelated to mongodb (mongo db is https://github.com/istio/istio/blob/748b513a16e6d18968bc13965bda860576626fc2/tests/e2e/framework/testInfo.go#L70 someone I guess is ahead of times)
the important thing is why mixerclient can't talk to mixer when we have auth setup
maybe there is a bug in the auth yaml somewhere ?

mongodb has not been introduced yet. So the question is how did that get into the infra? I have only pushed the mongo container to docker hub. Did not update the test scripts.

it doesn't affect anything, let's focus. we have a mixer client to mixer server problem
we will merge this PR as soon as the auth test is done running (and it will fail but also collect the logs we need)

@andraxylia: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-auth.sh
d744b259d737af20ca4e672698c70e5a5803850f
link
/test e2e-suite-rbac-auth

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


Merge tool / validator not checking waynode max limit

I'm merging a few of Lake Huron's ways and noticed that iD's merge tool won't check the 2000 nodes waynode limit  as per /api/capabilities. This has a few bad consequences during upload, as the API refuses this changeset and somehow iD bails off into an undefined state. Ideally, iD's validator should catch this before it hits the upload.
Testing: https://master.apis.dev.openstreetmap.org/edit?editor=id#map=18/44.87525/-81.34865

Merge 2 ways
Upload

Try to upload again

I think iD should just split these for the user without even bothering them.

I added a safety check to the merge operation, but there are still other situations where the maximum node limit could be exceeded:

Continuing a line
Double-clicking a way
Dragging a midpoint
Dragging a node onto a way
Clicking a way while drawing
-Clicking a way while placing a new node
Probably more…

Since these only add one node at a time, I expect they're less likely to exceed the node limit than merging long ways, but they still make for future work.


Telegraph->files not cleaned between 2 telegram requests

Hi,
Thanks for your library.
I found interesting bug when try to edit message after I sent message with attached audio file.
Long story short.
Telegraph facade object vendor/defstudio/telegraph/src/Telegraph.php
When we clone telegraph object, I believe, we need to clear files Collection, because it is an object and it will point to the same object in original Telegraph object.

I propose to add something like this to vendor/defstudio/telegraph/src/Telegraph.php
public function __clone(): void
{
    $this->files = Collection::empty();
}

@fabio-ivona Is it worth it to create a PR for such small fix in future?

Hi! sure! feel free to open a PR for this ❤

Hi! sure! feel free to open a PR for this ❤

Okay. I will add some tests first. Because solution is not so straightforward, as I thought.
We clone Telegraph object on every method call. If we clear files on every clone call, we loose attached file on following call.
For example,
Telegraph::photo(Storage::path('photo.jpg'))->message('test')->send();

Update:
I wrote test and discovered that problem is with TelegraphFake actually and not with Telegraph.
Reason is that TelegraphFacade is bind to factory, not instance of Telegraph object. So every time Facade creates new object:
class TelegraphServiceProvider extends PackageServiceProvider
{
    public function configurePackage(Package $package): void
    {
        $package
            ->name('telegraph')
            ->hasConfigFile()
            ->hasRoute('api')
            ->hasMigration('create_telegraph_bots_table')
            ->hasMigration('create_telegraph_chats_table')
            ->hasCommand(CreateNewBotCommand::class)
            ->hasCommand(CreateNewChatCommand::class)
            ->hasCommand(SetTelegramWebhookCommand::class)
            ->hasCommand(UnsetTelegramWebhookCommand::class)
            ->hasCommand(GetTelegramWebhookDebugInfoCommand::class)
            ->hasTranslations();

===>        $this->app->bind('telegraph', fn () => new Telegraph());
    }
}

But TelegraphFake is binded to instance to preserve sent messages between fake requests:
class Telegraph extends Facade
{
    protected static $cached = false;

    /**
     * @param array<string, array<mixed>> $replies
     */
    public static function fake(array $replies = []): TelegraphFake
    {
        TelegraphFake::reset();
===>        static::swap($fake = new TelegraphFake($replies));

        return $fake;
    }
....

I fix that and make PR https://github.com/defstudio/telegraph/pull/433. Please check and merge if it's ok. Thanks.

solved by @varemel  PR, thanks !


⚠ Jenkins Plugins Artifactory has degraded performance

In 65c5c88, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1221 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 4ac9c7d after 13 minutes.


How to install and use the program

Hello all, this might be too basic for you, but I would like to ask someone to explain how to use the program? I launch the .exe file and after a brief DOS  window flash nothing happens. I've placed the video files in the program folder and launched it, but nothing happened. Thank you very much.

You need to open the .yaml files with the .exe (drag and drop it on top). It'll use the settings specified by the .yaml to find the specified video files in the same folder and process the videos

Thank you very much for your prompt response. I really appreciate it! I have tried doing so but nothing happens. Do I need to place the video files in the root folder, and do I need to edit the file name in the .yaml file?
Thank you in advance!

The videos need to be in the same folder as the .exe and the filename under the input-files needs to match the video file you want to process. Turns out that just running the exe without dragging/dropping will work and trdrop will automatically use tr-config.yaml. Can you get a screenshot of the command window before it goes away?

Wonderful, you are amazing. Thank you for your quick response. I've edited the .yaml file with a Notepad, and upon dragging it on top of the .exe file it worked like a charm. Thank you very much!

No problem! Can't wait to see tests from you

Thank you! I will test it out a bunch and report back! Thank you for the program. From the workflow perspective, it works absolutely effortlessly! If I may, could I inquire if there is a way to assign labels to the output, next to the FPS counter, such as the name of the game? Thank you!

I wasn't the one that made trdrop, but you'll have to put the game's name in in post if you don't want the game's name to share the same line with the FPS counter

Understood. Thank you very much for your prompt replies. I will be in touch!


Use Fastify logos by default

This PR adds a configuration to modify the logo.
It also sets the logo of Fastify as the default.
Checklist

[x] run npm run test and npm run benchmark
[x] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

@mcollina out of interest do you have a screenshot of what it looks like with the logos added?

Here is it:


Feegrant module absent

What happened?
I was trying to authorize an account to pay fees using balance present in another account. I've seen this feature is available in cosmos-sdk, and the module seems to be there in the osmosis cosmos-sdk fork, but when tryint to use it with the CLI, it is absent.
I also tried creating a transaction template with gaiad and then adjust the values and sign it with osmosisd, but the message is not registered.
Is there any current alternative to use this feature or are there any plans to add it back with the sdk unfork?
Osmosis Version
19.1.0
How to reproduce?
In the CLI:
osmosisd tx feegrant grant --help
Module is not present.

There is not currently a plan to add it into Osmosis, due to complex interactions it has with other modules.


Par of Seqs to Seq of Pars Transformation

First try at the pass we discussed to deal with #840 .
Like we discussed, I tried to implement the most conservative possibility for this pass.
Furthermore, the attributes for the new seq block that I create is justir::Attributes::default().
I wasn't sure what to do for testing. Let me know if you had something else other than what I did.
It's also currently failing formatting tests so I'll make sure to fix that too.

@rachitnigam I know it's the weekend, but here's the update to the PR (just so you know I updated it).


node-usb not found for electron 8

The problem occurs when building the native dependency. The essential error message is this:
prebuild-install http 404 https://github.com/tessel/node-usb/releases/download/v1.6.2/usb-v1.6.2-electron-v76-win32-x64.tar.gz

There is no release of<EMAIL_ADDRESS>for electron-v76, as can be confirmed over at the releases page.
@kevinmehall Would it be possible to issue a release that supports electron-v76? When I look at the corresponding commit dce21022559e486f6e1245adea927bbd3d5b0454 for electron 6 and 7, it looks like editing .travis.yml might be enough, i.e. adding 8.0.0 to PREBUILD_ELECTRON_TARGETS.
(Mentioning serialport/node-serialport#2031 for similarity)

Yes, if it's supported in https://github.com/lgeiger/node-abi/ (used by prebuild), then you can just submit a PR adding it, and when I tag a release it will be built automatically.

PR: #354


Get All parents for selected node in Treeview

Hello,
Below is the tree view I have created with gijgo version 1.9.11, I want list of all parents that are are linked to selected child node

When I Select Dist 02 I want the result as Developer,Oem 01,Dist 01
When I select Oem 01 I want the result to be Developer
When I select Dist 01 I want the result to be Developer,Oem 01
I have tried with below code but it always return only Developer as a result
tree.on('select', function (e, node, id) {
         selectednode = node;
	 var result = tree.parents(id);
         alert(result.join());
});

Please help !

look the my answer for Treeview parents

Have you solved this problem ?
because I have the same problem


couldn't find file 'pdfjs_viewer/pdfjs/compatibility'

Hi, I've tried to remove the pdfjs_viewer-rails gem from my project but i'm facing the follow error
couldn't find file 'pdfjs_viewer/pdfjs/compatibility' with type 'application/javascript'

I already removed the gem from gem file, run the bundlecommand and remove the mounter from routes.rb but nothing changed

Same here!

forget about it


fix(components): [grid] repeatedly requesting data

PR
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our Commit Message Guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #1203
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@betavs 恭喜你成为OpenTiny 2023年度开源新星，请添加下OpenTiny小助手微信：opentiny-official，如已添加，请发一条信息给小助手，谢谢！


Callback feature

Addresses #716, #735, #736, #737.
To support webhooks, a proposed callback mechanism is added.  Essentially, callbacks are out-of-band operations against some remote server.  The callback subscription mechanism is not prescribed, as it is actually implementation dependent.  The mechanism for responses, however, is created by added a callbacks section to any operation.
The callbacks section has named callbacks, which are essentially operations.  The operations can be applied to a URL as extracted from the request options using a syntax similar to with the links technique.
An example below describes the GitHub webhook mechanism, with comments, for discussion.
paths:
  /repos/{owner}/{repo}/hooks/{id}:
    post:
      body: |
        description: Repository service hooks (like email or Campfire) can have at most one configured at a time. Creating hooks for a service that already has one configured will update the existing hook.
        Repositories can have multiple webhooks installed. Each webhook should have a unique config. Multiple webhooks can share the same config as long as those webhooks do not have any events that overlap.
        schema:
          $ref: '#/components/definitions/WebhookRequest'
      responses:
        201:
          description: Webhook created
          schema:
            $ref: '#/components/definitions/WebhookResponse'
      callbacks:
        createWebhook:
          $ref: '#/components/callbacks/createWebhook'
components:
  callbacks:
    createWebhook:
      # extract the url parameter from the request
      '$request.body.url':
        post:
          parameters:
          - in: header
            name: X-GitHub-Event
            type: string
            description: Name of the [event](https://developer.github.com/webhooks/#events) that triggered this delivery.
            required: true
          - in: header
            name: X-Hub-Signature
            type: string
            description: HMAC hex digest of the payload, using the [the hook's `secret`](https://developer.github.com/v3/repos/hooks/#create-a-hook) as the key (if configured).
          - in: header
            name: X-GitHub-Delivery
            type: string
            description: Unique ID for this delivery.
          body:
            name: payload
            schema:
              $ref: '#/components/definitions/Event'
          responses:
            200:
              description: webhook successfully processed an no retries will be performed
  definitions:
    WebhookRequest:
      required:
      - name
      - config
      properties:
        name:
          type: string
          description: Use web for a webhook or use the name of a valid service. (See /hooks for the list of valid service names.)
        config:
          $ref: '#/components/definitions/WebhookConfig'
        events:
          type: array
          items:
            type: string
            enum:
            - push
          default: [push]
          description: Determines what events the hook is triggered for.
        active:
          type: boolean
          description: Determines whether the hook is actually triggered on pushes.
    WebhookResponse:
      properties:
        id:
          type: string
        url:
          type: string
          format: url
        test_url:
          type: string
          format: url
        ping_url:
          type: string
          format: url
        name:
          type: name
        events:
          type: array
          items:
            type: string
            enum:
            - push
            - pull_request
        active:
          type: boolean
        config:
          $ref: '#/components/definitions/WebhookConfig'
        updated_at:
          type: string
          format: date-time
        created_at:
          type: string
          format: date-time
    WebhookConfig:
      type: object
        properties:
          url:
            type: string
            format: url
          content_type:
            type: string
            enum:
            - json
        additionalProperties:
          type: string
          description: Key/value pairs to provide settings for this hook. These settings vary between hooks and some are defined in the [github-services](https://github.com/github/github-services) repository. Booleans are stored internally as "1" for true, and "0" for false. Any JSON true/false values will be converted automatically.
    Event:
      anyOf:
      - $ref: '#/components/definitions/commit_comment'
      - $ref: '#/components/definitions/create'
      - $ref: '#/components/definitions/delete'
      - $ref: '#/components/definitions/deployment'
      - $ref: '#/components/definitions/deployment_status'
      - $ref: '#/components/definitions/fork'
      - $ref: '#/components/definitions/gollum'
      - $ref: '#/components/definitions/issue_comment'
      - $ref: '#/components/definitions/issues'
      - $ref: '#/components/definitions/member'
      - $ref: '#/components/definitions/membership'
      - $ref: '#/components/definitions/page_build'
      - $ref: '#/components/definitions/public'
      - $ref: '#/components/definitions/pull_request_review_comment'
      - $ref: '#/components/definitions/pull_request'
      - $ref: '#/components/definitions/push'
      - $ref: '#/components/definitions/repository'
      - $ref: '#/components/definitions/release'
      - $ref: '#/components/definitions/status'
      - $ref: '#/components/definitions/team_add'
      - $ref: '#/components/definitions/watch'

@OAI/tdc for discussion today ^^

This $request.body.url syntax doesn't seem to be defined. Something similar is there for the variable substitution in the link object, but there it is in {...}. I suggest unifying this (and defining it a bit clearer).

in the example, I think X-Hub-Signature should be X-GitHub-Signature

The proposal should address security considerations -- can a man in the middle or other attack alter the callback URL in the request in order to redirect sensitive data?

@DavidBiesack I'm not sure how you think this proposal introduces any security issues?  Please explain.

@OAI/tdc please review and add your feedback.  Would be great to get any other examples or feedback on this structure.

I'm all 👍 on this.

A few comments;

Although it's 'just a name' (and so could be any valid string, I assume) the name createWebHook as the identifier/name for the callback seemed odd to me. I wanted to log this in case there is a deeper reason for the name to be what it is

I might have used the term eventCallback or gitHubEventHook

Setting aside the syntax used in '$request.body.url':, its placement in the structure of the document leads me to think that the entire callback is part of a structure that hangs off this item.

If this interpretation is correct, I think this means that this update does not model 'standalone' webhooks declarations.
In the last OAI TDC discussion (that I was on) we spent some time discussing that it is useful and necessary to enable this. As I recall, the proposal was to allow callback declarations to exist within the components section without needing to be attached to a subscription API
In your [comment on #716] you had syntax like this

callbacks:
      # callbacks are sent to the url provided
      # this could also be a query param, etc.
      url: '$request.url'
      schema:
        type: object
        properties:
          a:
            type: string

... which could allow WebHooks to be specified standalone by allowing url: to be optional.

Discriminating between events: the anyOf: does a nice job of enumerating all of the different event structures that could occur. Can we also provide a way of specifying that X-GitHub-Event provides a discriminator?
On the syntax of '$request.body.url':, if I've understood the semantics, and checking the schemas, I think this should be '$request.body.config.url:'

Hi @DavidBiesack github does in fact use x-hub-signature for this specific item;
https://developer.github.com/webhooks/securing/

One more thought; could we add an element something like;
   subscriptionList: '$request.body.config.events'

Which uses your syntax to make it clear which element of the webhooks subscription interaction specifies the list of events required.

This implementation doesn't seem anything like what I'm see elsewhere.  I think calling things what they are in the spec, webhooks or events, make more sense than callbacks.  To me, I'd expect to define a set of events and then dictate which operations emitted those events.  Each event would describe the payload sent via the webhook and the expected response to return.
Here is an example of GitHub's Webhook documentation: https://developer.github.com/webhooks/  I realize this feedback might be a little late.

Here is a portion of our Event Hub definition updated with this callback proposal:


Feature registry zk

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: sharajava:white_check_mark: github-ygy:white_check_mark: slievrly:white_check_mark: xingfudeshi:white_check_mark: xuxiaofei4519:x: peijian.huangpeijian.huang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


webkit support in docker image

What would you like?
Support for webkit & its dependencies baked into the the existing docker  image
Why is this needed?
To have webkit support, This is useful for my use case as my org network doesn't connect to playwright's cdn
Other
No response

Related to https://github.com/cypress-io/cypress-docker-images/issues/526


feat(users): User model and authentication pattern

[ ] user model and instructions on how to include in migrations
[ ] cli to create admin user
[ ] authentication mechanism
[ ] integration with service authorization mechanisms

will implement client credentials grant flow
the authorization server will be this one, but need to document that a 3rd party should be used in production
ensure necessary abstraction so that external auth server is 0 friction
cli to create users, not super user
super user credential defined in environment (maybe doesn't even really need to be a user in the db).
if super user password is not defined, the super user is disabled - this can allow for a redeployment after admin users have been created.
admin users can create customer accounts
customer accounts authenticate and get a token that has expiry
api routes require token for viewing