Error on Pandas profiling - TypeError: concat() got an unexpected keyword argument 'join_axes'

TypeError: concat() got an unexpected keyword argument 'join_axes'

Please attach a screenshot of the options that you have chosen with the error message and the dataset that you are using.

Attaching the screenshot & file,
[image: image.png]
On Sat, Jun 20, 2020 at 12:45 PM Arunn<EMAIL_ADDRESS>wrote:

Please attach a screenshot of the options that you have chosen with the
error message and the dataset that you are using.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arunnbaba/pandas_ui/issues/10#issuecomment-646956242,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2RSD5YSDVMOH3HUSDELM3RXROYZANCNFSM4ODGUQZA
.

Hi  @HariprasadManimozhi, your attachment is inaccessible.

Hi Arunnbaba/Pandas_ui,
The attachment is a CSV file & it's loading fine.
Try to download & check & let me know if it works.
On Mon, 6 Jul, 2020, 2:42 pm Arunn<EMAIL_ADDRESS>wrote:

Hi @HariprasadManimozhi https://github.com/HariprasadManimozhi, your
attachment is inaccessible.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/arunnbaba/pandas_ui/issues/10#issuecomment-654114223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2RSD72KBLQSN5PEZZ3SQTR2GIN7ANCNFSM4ODGUQZA
.

Book1.xlsx
TypeError: concat() got an unexpected keyword argument 'join_axes'

Book1.xlsx
TypeError: concat() got an unexpected keyword argument 'join_axes'

I have attached the file <Book1.xlsx>, But the file I used in actual was in csv format

I see the problem , pandas_ui accepts only CSV files
On Wed, 8 Jul 2020, 11:17 am HariprasadManimozhi<EMAIL_ADDRESS>wrote:

Book1.xlsx
https://github.com/arunnbaba/pandas_ui/files/4888732/Book1.xlsx
TypeError: concat() got an unexpected keyword argument 'join_axes'
I have attached the file <Book1.xlsx>, But the file I used in actual was
in csv format
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/arunnbaba/pandas_ui/issues/10#issuecomment-655300804,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADRKCWRYBZ564CAU6JFXUPDR2QCAJANCNFSM4ODGUQZA
.

I see the problem , pandas_ui accepts only CSV files
…
On Wed, 8 Jul 2020, 11:17 am HariprasadManimozhi, @.***> wrote: Book1.xlsx https://github.com/arunnbaba/pandas_ui/files/4888732/Book1.xlsx TypeError: concat() got an unexpected keyword argument 'join_axes' I have attached the file <Book1.xlsx>, But the file I used in actual was in csv format — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#10 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADRKCWRYBZ564CAU6JFXUPDR2QCAJANCNFSM4ODGUQZA .

Any solution as to why not working?

Hi HariprasadManimozhi, as I said earlier please attach a screenshot of the jupyter notebook with the error message so that I can look into it. I can't find any screenshots in the thread.
I can only find the "[image: image.png]" string not the actual screenshot. but I can download your file.

https://user-images.githubusercontent.com/28645647/88040847-24f00080-cb67-11ea-9d8f-ba92ce873d17.png  The snapshot of error!

https://user-images.githubusercontent.com/28645647/88040847-24f00080-cb67-11ea-9d8f-ba92ce873d17.png The snapshot of error!

If above link doesnt work, use this https://we.tl/t-Khe62Eog8C

I can very well access the pandas profiling for the file which you shared. Please find the attached screenshot. I am confident that it's causing error due to your local settings of system environment.


Optimize no-wake path of AtomicWaker::wake

Introduce HOLDS bit which is true when there's waker in the cell
(option is Some).
wake operation immediately exits if HOLDS bit is not set (also
if WAKING bit is set which means another thread does wake right
now).
The patch includes a graphviz diagram to better understand state
transitions.
From the diagram, it's clear that there are only 6 states although
there are 8 possible bit set values:

EMPTY
HOLDS
WAKING
WAKING|HOLDS
REGISTERING
REGISTERING|WAKING

Register state transitions are red, and wake state transitions are
blue.
Solid line transitions at the beginning of function calls, and
dashed lines are second transitions.

CC @Pslydhh

I've updated the patch to do load(SeqCst) which should be good on x86, and sorry about other architectures.
But even with SeqCst wake operation should be cheaper than before under heavy load (when the number of wake calls is higher than the number of register calls).
If that is not good enough, I should probably give up. Memory ordering is hard.

@stepancheg

But even with SeqCst wake operation should be cheaper than before under heavy load (when the number of wake calls is higher than the number of register calls).

Can you say more about why you expect this to be the case? Previously, a no-op wake was an AcqRel fetch_or followed by a Release fetch_and, whereas now it's a SeqCst load. Without benchmarks I have no idea whether the perf loss on registration and on the "real" wakeup case is worth this.


ideas

Additional context
Add any other context or screenshots about the feature request here.
Is it possible to import from JLC also the assembly type tag? aka SMD or wave soldering ?
Also if it is a Basic part or not? this can really help a lot.
Also but this is more like a side topic i noticed that every part comes with its own footprint , even tho that footprint is a standard one this may make the lib grow in size very fast due to a lot of duplicates.
BTW i also found it a bit unclear how the parameters in the command line work so maybe a bit more explanations there would be nice

schematic == symbol for kicad here is another source of confusion since i understand schematic as being the entire circuit with op amps resistors and such
btw regarding footprint can u get a list in a script? since i think you can use string comparison (and that may be enough for most parts aka resistors and caps but i think it may work for many other standard footprintss)

@immortal-sniper1 meanwhile, you can use this simple bash script I wrote to do batch processing of multiple parts, to add the missing SMD attr: https://github.com/mikaelj/jlc-kicad/blob/19302e96aa4621850ff5f9dfa071db96adff1f28/jlcpcb-library.sh#L44


[BUG] Directories are not enumerated if directory has no trailing slash

Describe the bug
When Feroxbuster discovers a link with no trailing slash, it is not further enumerated.
To Reproduce

Save this simple example server as index.py:

from flask import Flask, render_template, request
import base64

app = Flask(__name__)

@app.route('/', methods=['GET'])
def index():
    return '', 200

@app.route('/test', methods=['GET'])
def test():
    return 'Hello world', 200

@app.route('/test/a', methods=['GET'])
def a():
    return 'Hello world', 200

@app.route('/test/image.png', methods=['GET'])
def image():
    return base64.decodebytes(b'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mNk+M9QDwADhgGAWjR9awAAAABJRU5ErkJggg=='), 200

Run python3 index.py
Run echo -e "a\n" | feroxbuster -u http://localhost:5000 -w /dev/stdin --force-recursion

Expected behavior
/test/image.png is discovered using HTML parsing. Then Feroxbuster also checks if any parent directories exists and enumerates these directories (i.e. /test and /test/a)
Traceback / Error Output
Only the link itself is discovered
 ___  ___  __   __     __      __         __   ___
|__  |__  |__) |__) | /  `    /  \ \_/ | |  \ |__
|    |___ |  \ |  \ | \__,    \__/ / \ | |__/ |___
by Ben "epi" Risher 🤓                 ver: 2.10.1
───────────────────────────┬──────────────────────
 🎯  Target Url            │ http://localhost:5000
 🚀  Threads               │ 50
 📖  Wordlist              │ /dev/stdin
 👌  Status Codes          │ All Status Codes!
 💥  Timeout (secs)        │ 7
 🦡  User-Agent            │ feroxbuster/2.10.1
 🔎  Extract Links         │ true
 🏁  HTTP methods          │ [GET]
 🔃  Recursion Depth       │ 4
 🤘  Force Recursion       │ true
───────────────────────────┴──────────────────────
 🏁  Press [ENTER] to use the Scan Management Menu™
──────────────────────────────────────────────────
DBG      0.198 feroxbuster sending ["http://localhost:5000"] to be scanned as initial targets
INF      0.198 feroxbuster::event_handlers::scans scan handler received http://localhost:5000 - beginning scan
INF      0.198 feroxbuster::scanner::ferox_scanner Starting scan against: http://localhost:5000
404      GET        4l       34w      232c Auto-filtering found 404-like response and created new filter; toggle off with --dont-filter
DBG      0.238 feroxbuster::filters::wildcard filtered out http://localhost:5000/a
DBG      0.238 feroxbuster::filters::container filtering response due to: WildcardFilter { content_length: Some(232), line_count: Some(4), word_count: Some(34), method: "GET", status_code: 404, dont_filter: false }
INF      0.239 feroxbuster::event_handlers::scans Added new directory to recursive scan: http://localhost:5000/
DBG      0.240 feroxbuster::extractor::container Parsed link "/test/image.png" from http://localhost:5000/
DBG      0.241 feroxbuster::filters::wildcard filtered out http://localhost:5000/test/
DBG      0.241 feroxbuster::filters::container filtering response due to: WildcardFilter { content_length: Some(232), line_count: Some(4), word_count: Some(34), method: "GET", status_code: 404, dont_filter: false }
DBG      0.242 feroxbuster::extractor::container Extracted File: FeroxResponse { url: http://localhost:5000/test/image.png, method: GET, status: 200 OK, content-length: 86 }
200      GET        3l        5w       86c http://localhost:5000/test/image.png
200      GET        1l        2w       54c http://localhost:5000/
INF      0.949 feroxbuster All scans complete!

Environment (please complete the following information):

feroxbuster version: 2.10.1
OS: Ubuntu 22.04.1

Other information
If the test route is defined as /test/ (with trailing slash), Feroxbuster does function as expected

hey there, thanks for submitting this!
What you're seeing is expected behavior (see logic below). forced recursion should be able to get the results you're after (the recursion logic is laid out in plain language on that page as well)
    pub fn is_directory(&self) -> bool {
        if self.status().is_redirection() {
            ...
        } else if self.status().is_success() || matches!(self.status(), &StatusCode::FORBIDDEN) {
            // status code is 2xx or 403, need to check if it ends in /

            if self.url().as_str().ends_with('/') {
                log::debug!("{} is directory suitable for recursion", self.url());
                return true;
            }
        }

Hi @epi052, thanks for your response. As you can see from the command provided above, I am already using --force-recursion. Is there anything else I should add?

Ooooh, missed that, my bad.  Lemme take a look, iirc that should be recursing then.

i started looking into this, but haven't arrived at a suitable solution yet. pinning for now


use GCM instead of CBC

This strengthens the cryptography used to protect token-signing keys. Fixes #98

pretty good, just a few little things

LGTM


ng-superclass: Angular v14

Can you add angular 14 as a compatible peer dependency please?

Hi Dmitry, I still get the peer error:
<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.


[Bug report] No response for a long time when building Thousands of markdown files

Description
hello, No response for a long time when building Thousands of markdown files...

Reproduction
Reproduction
Used Package Manager
yarn
System Info
info Initializing VuePress and preparing data...
⠼ Compiling with viteUse of eval is strongly discouraged, as it poses security risks and may cause issues with minification
Use of eval is strongly discouraged, as it poses security risks and may cause issues with minification

Duplicate of #994


Add Community Guidelines

I couldn't find anything related to community guidelines.
Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support
Can you add that?

In PR #47 we've added a CONTRIBUTING page to the docs. After merging the PR, this page will also be rendered in the sphinx documentation alongside the developer documentation page.

Here is the new docs page: https://harrispopgen.github.io/mutyper/CONTRIBUTING.html


add defendant name(s) to allocation tool

Alistair, this minor change to the allocation tool adds a column which squeezes up the others and makes it somewhat unattractive. can anything be done about from front-end

ok, i have just made that tweak. i think it would be useful to add tickets for you in backlog to look at the the styling for this page (allocation tool) and another ticket i am working on which is widening the expense section of the new/edit form and could do with some love. can chat about it after they are in.


Error on attempted move between rooms

I converted a game, ran it, attempted to move from one room to another for the first time, and immediately ran into an error:
Rags can not process the condition check correctly.  If you are the game author, please correct the error in this conditon:Character in dest room? check:CT_Character_In_Room - [char.name] - [tp: GetCount : Results] -

(This error doesn't come up when I run the game in RAGS Player 2.4.16 on Windows.)
Game download link: https://www.mediafire.com/?ctzxbijtmlfm5r3

Thanks for reporting the issue.
This should be fixed now, convert the game again and check.

It does indeed seem to be fixed. Thanks!


Backport for 1.3.2

Checklist

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

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

Aim for https://github.com/nanomsg/nng/tree/4f5e11c391c4a8f1b2731aee5ad47bc0c925042a - 1410 commits vs. https://github.com/nanomsg/nng/tree/v1.3.2 - 1399 commits


meta: stop supporting EL7

After internal discussions, we decided to not support EL7 due to where it is in its lifecycle.

[citest]


templating name for file - how to use author et al. + title?

Really nice plugin! Thank you for your work!
I have a question - does it is possible  to have a  name of literature note in the form
author (et al. if more than 1 or 2 authors) - year - title ?

I'll add a helper authorsShort that output authors in the format you've mentioned in next release, meanwhile you can do something like this:
<%= it.authors.first()?.lastName %>
<% if (it.authors.length > 1) { %> et al. <% } %>
- <%= it.date %> - <%= it.title %>


Low-code/No-code

Summary
I wonder how this would work, and what problems it might solve?
Provocation: If you want low-code, you still need to learn to code, and you also need to write a complex platform to run your low-code code. That platform needs much more skilled engineers to write and operate it. Ultimately, are things going to end up being more complex rather than less complex?
Use Cases
When would you use this?

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Source  and demo environment for a related effort.


Don't rebuild some tools for every versions of OCaml

cc @panglesd
Some tools like ocamlformat, dune-release and opam-publish don't seem to depend on OCaml's object files format and so don't need to be built again and again.
This is a potentially huge speed improvement, as these tools have a lot of transitive dependencies (41 for ocamlformat and 105 for opam-publish), slowing down the sandbox build significantly.

Done in https://github.com/tarides/ocaml-platform-installer/pull/110


Fix duplicate clize output and errors

Fixes two bugs with Clize CLIs:

(ERROR/3) Unknown interpreted text role "exc". - Fixed by adding a filter to cli.init.docutilitze.
Error messages printed twice. - Fixed by removing a print statement (and surrounding try block)

You can reproduce these things on master with the command bin/improver apply_lapse_rate which returns:
process:33: (ERROR/3) Unknown interpreted text role "exc".
process:40: (ERROR/3) Unknown interpreted text role "exc".
process:44: (ERROR/3) Unknown interpreted text role "exc".
process:45: (ERROR/3) Unknown interpreted text role "exc".
process:46: (ERROR/3) Unknown interpreted text role "exc".
process:33: (ERROR/3) Unknown interpreted text role "exc".
process:40: (ERROR/3) Unknown interpreted text role "exc".
process:44: (ERROR/3) Unknown interpreted text role "exc".
process:45: (ERROR/3) Unknown interpreted text role "exc".
process:46: (ERROR/3) Unknown interpreted text role "exc".
improver: Unknown command "apply_lapse_rate". Did you mean "apply-lapse-rate"?
Usage: improver command [args...]
process:33: (ERROR/3) Unknown interpreted text role "exc".
process:40: (ERROR/3) Unknown interpreted text role "exc".
process:44: (ERROR/3) Unknown interpreted text role "exc".
process:45: (ERROR/3) Unknown interpreted text role "exc".
process:46: (ERROR/3) Unknown interpreted text role "exc".
process:33: (ERROR/3) Unknown interpreted text role "exc".
process:40: (ERROR/3) Unknown interpreted text role "exc".
process:44: (ERROR/3) Unknown interpreted text role "exc".
process:45: (ERROR/3) Unknown interpreted text role "exc".
process:46: (ERROR/3) Unknown interpreted text role "exc".
improver: Unknown command "apply_lapse_rate". Did you mean "apply-lapse-rate"?
Usage: improver command [args...]

Testing:

[x] Ran tests and they passed OK

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ lib/improver/cli/__init__.py  -1

See the complete overview on Codacy


fix(ui): restructure server initializations

📑 Description
This PR fixes a performance issue which occurs when at least 4 packages are installed in a cluster + the user switches back and forth between package detail pages very quickly. The cause of this was that the used package client was not actually the cached one, because of the incorrect initialization.
✅ Checks

[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required

ℹ Additional Information

Code looks good, but I get a strange error when installing a package:
It looks like the installation failed, although it actually didn't. I think this is because we try to get the package info before the informer cache has been updated. 😕

Good point, didn't see it because it doesn't always appear, but often enough. Tried to fix it.


Milestone001:Mobile  first portfolio setup

The following functionalities were added in this feature update;

The markup for the toolbar and the headline were created. ✔
The two sections above were styled as per figma template. ✔
The readme file was updated. ✔

Thankyou Sir. 💘


Redesigned DM recipient headers

This is built on top of #416.
This PR updates the DM recipient headers to the new style. Text style and spacing was taken from the same sources as #416:

https://www.figma.com/file/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=538%3A20849&mode=dev
https://github.com/zulip/zulip-mobile/issues/5511

But given there were no mockups that included a DM recipient header the style used here also looked to how web renders these headers (in particular the color to use for the background, and icon to use for display).

Here showing behavior when the system requests a larger font:

@gnprice rebased on main and ready for review!

@gnprice ready for review again!

Thanks for the revision! All looks good — merging.


🌱 Rename upgrade managment test and remove unecessary passed msg

Rename upgrade management cluster to upgrade clusterctl and add the correct upgrade from release to the describe text.
Also remove passed message from the aftereach as it shows that even when the test fail

/test-e2e-upgrade-main-from-release-1-2

/test-centos-e2e-integration-main


Support inverse header match in AuthorizationPolicy

Describe the feature request
The AuthorizationPolicy matches traffic based on a number of conditions and then takes an action. However, it can't match the condition:
Request does NOT have header X.
We'd like to add support for this feature.
After looking around a bit, I found the part of Istio which translates the header match YAML to Envoy configuration:
https://github.com/istio/istio/blob/71f09fe8a8122f7b489b0305adf590bfbe0a0ba8/pilot%2Fpkg%2Fsecurity%2Fauthz%2Fmodel%2Fmatcher%2Fheader.go#L24-L56
From that, I found out that Envoy actually supports inverse matching:
https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/route/v3/route_components.proto#envoy-v3-api-msg-config-route-v3-headermatcher
And actually, this feature is used in VirtualServices, to implement the withoutHeaders field, which is exactly what we need:
https://github.com/istio/istio/blob/372f1427c41abba18639fc04dcf73a80177222cd/pilot%2Fpkg%2Fnetworking%2Fcore%2Fv1alpha3%2Froute%2Froute.go#L626-L630
I think the main difficulty is to decide on what the UX will be. Right now, conditions are written in pseudo-code, while in VirtualService they are written in YAML. I'd love to have some suggestion on how to express this inverse matching in AuthorizationPolicy.
BTW, you can assign this issue to me, I believe I can implement it once we agree on the UX.
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

a quick question, have you tried using the notValues for the negative matching? for example:
apiVersion: "security.istio.io/v1beta1"
kind: "AuthorizationPolicy"
metadata:
  name: "httpbin-header"
  namespace: default
spec:
  selector:
    matchLabels:
      app: httpbin
  rules:
  - when:
    - key: request.headers[foo]
      notValues: ["*"]

@yangminzhu thanks, this seems exactly like what I need! I somehow missed that this was available in the new Policy 😄

Closing as this should already be supported, feel free to reopen if you have other questions.


Issue with MixedVectorIntegrator after recent merge of #1989

I am assembling a MixedBilinearForm which has only boundary integrators on a 3D mesh. After merging #1989, this no longer works and I get the following incompatible dimensions error and backtrace:
Assertion failed: (a.Height() == b.Height() && a.Width() == c.Width() && b.Width() == c.Height()) is false:
 --> incompatible dimensions
 ... in function: void mfem::Mult(const mfem::DenseMatrix&, const mfem::DenseMatrix&, mfem::DenseMatrix&)
 ... in file: /Users/sjg/Documents/Repositories/qht-cem/build/extern/mfem/src/mfem/linalg/densemat.cpp:1984

Backtrace:
0) [0x10dc84a1e]: mfem::mfem_error(char const*)
1) [0x10dcc4993]: mfem::Mult(mfem::DenseMatrix const&, mfem::DenseMatrix const&, mfem::DenseMatrix&)
2) [0x10e082c83]: mfem::FiniteElement::CalcPhysDShape(mfem::ElementTransformation&, mfem::DenseMatrix&) const
3) [0x10dbbf48f]: mfem::MixedVectorWeakDivergenceIntegrator::CalcTestShape(mfem::FiniteElement const&, mfem::ElementTransformation&, mfem::DenseMatrix&)
4) [0x10ded92a2]: mfem::MixedVectorIntegrator::AssembleElementMatrix2(mfem::FiniteElement const&, mfem::FiniteElement const&, mfem::ElementTransformation&, mfem::DenseMatrix&)
5) [0x10debb0fa]: mfem::MixedBilinearForm::Assemble(int)

It seems the issue is inside CalcPhysDShape, the inverse Jacobian is 2 x 3 but the dshape matrix (trial_shape in MixedVectorIntegrator::AssembleElementMatrix2) is now only allocated with 2 columns (trial_vdim = 2), whereas it used to always have spaceDim columns. The test_shape matrix is still constructed with test_vdim = 3. I'm not using any of the new ND or RT collections, in this case the test space is a 3D ND space and the trial space is a 3D H1 space.

Just as a follow-up or potential solution, it looks like elsewhere like in VectorFEMassIntegrator::AssembleElementMatrix2, trial_vdim and test_vdim are computed as:
int trial_vdim = std::max(spaceDim, trial_fe.GetVDim());
int test_vdim = std::max(spaceDim, test_fe.GetVDim());

Maybe this is all that is required in this case too. Also tagging @mlstowell, thank you Mark!

Thanks for the quick bug report, @sebastiangrimberg!

Hi, @sebastiangrimberg,
Being a generalized bilinear form integrator I don't think we can use set::max in the same way here. The gradient of a scalar basis function will always be the space dimension so we should simply use that. We can make this change in MixedVectorWeakDivergenceIntegrator::GetTestVDim.
I'm concerned that similar changes may be necessary in other integrators which are based on the three generalized bilinear form integrators. I'll take a closer look this afternoon and create a PR to, hopefully, catch any similar bugs.
Best wishes,
Mark

thanks @mlstowell !

@sebastiangrimberg, if you get a chance, could you verify that PR #2895 resolves this issue in your code?

Thanks @mlstowell, I can confirm that this fixes the issue!
Closing this now, discussion continued in #2895.


Option to show stack traces on the 500 page

It'd be useful for debugging to have stack traces available on the 500 page to give an indication on where/why the error occured without having to dig into the logs, maybe in a expandable box. I'd even lean to having that option on by default.

I'd even lean to having that option on by default.

Perhaps for admin users? Similar to the report message.

Yeah, admins should always see it. I guess there could be options:
ui.SHOW_STACK_TRACES = true
ui.SHOW_STACK_TRACES_USERS = all | admin-only

Note that showing stack traces with error messages to the client has the potential to weaken the security of web applications, e.g, through CWE-79.

XSS should of course be avoided with HTML escaping. Other than that, my only concern is a "leakage" of file system paths, but I don't think this would give much of an attack vector of its own.

HTML escaping does not help since the server generally does not have control about the context in which the data are processed by the client.

I guess there could be options:

I'm with @veita here, even having an option to show the stack traces to everybody is a potential security flaw, not so much about your instance per se, as much about the application itself in general, leaving more room to malitious users to find a way in (potentially) all instances.
(I know the entire project is open source, but digging into Golang code to find a breach is much harder than having a direct path in shown directly by the server itself you're trying to breach 😉)

Stack trace could be shown if Gitea is run in development mode

Stack trace could be shown if Gitea is run in development mode

Would agree: temp move Gitea to dev -> solve issue -> go back to production

That would for example not really help developers that want to debug errors seen on try.gitea.io for example, unless we permanently run it on dev mode.
I still think it's kind of pointless trying to hide internals when the whole source code is open source. Paths could potentially be sanitized as well.

unless we permanently run it on dev mode.

I think it is in dev mode... correct me if I'm wrong @lafriks.

I still think it's kind of pointless trying to hide internals when the whole source code is open source.

As said before, it's not so much as the source is open, but rather as it would lead to a "come here, open entrance" message directly from your server...

I'm not sure what dev mode entails but I could see a toggle in the admin settings UI to temporary switch then run mode. Of course it should be turned off after debugging. We can't really restrict it to logged in admin users because some errors can happen before login.

as for api: #13115

as for api: #13115

Is this solved then?

I have not tesed that PR but I think it enables stack traces for admin users. I guess a option to show it to all users may still be handy sometimes, for example if an error is only reproducible under a certain user so admin could temporarily flip that option.
There should be a warning in the option description that it might expose secrets/internals so it has to be enabled with care. I'd call it ALWAYS_SHOW_STACK_TRACES or similar.

I think showing the stack trace to a normal user is a bad Idear!
It can print data the user should not have access to!
But If you are admin It should be fine

Yeah I guess admin-only is good enough and certainly would fit my requirement, so yes, I think this is resolved.


Line sort dedupe comma support

Currently if you add a list that is comma seperated it does nothing need to update with comma support.
i.e.
Olivia,
Ethan,
Sophia,
Liam,
Ava,
Jackson, Isabella, Lucas, Mia, Aiden, Harper, Elijah, Amelia, Logan, Abigail, Oliver, Charlotte, Noah, Emma, William, Grace, Luke
if organized in list on top of one-another it sorts but if side by side it fails and uses only the first word.

Hey, can I pick up this issue? Also, I think we can add two more things

Inline Data String Support

Input
Output

Beta Alpha Gamma
Alpha

Beta

Gamma

Remove Quotations

Input
Output

"Beta", "Alpha", "Gamma"
Alpha

Beta

Gamma

Hey, I have created a PR for this issue and its deployed on vercel.


requiring "bundler/setup" can modify Gemfile.lock

Requiring "bundler/setup" can modify Gemfile.lock.  In a separate communication with @mpapis, we learned that it should not.
This gist contains a script to reproduce the problem, and other relevant files.  Running that script shows (on my system) that Gemfile.lock can be modified by require "bundler/setup".

A quick note about my issue of losing gems from Gemfile.lock.  The issue still occurs for my company, so I went back to do some more digging.  I am able to reproduce it on a box running Debian "wheezy" (the previous stable version), but not on one running Debian "jessie" (the current stable version).  This is with identical commands, same version of Ruby, latest version of Bundler, etc.  It seems likely that Bundler was never at fault (we never doubted ya, Bundler), but that the cause was a bug in the kernel or in some C library that was fixed at some point.
Just in case you were worried about Bundler at all.  And in case anyone else with a similar issue runs across this.

Thanks for the update, @wconrad !


RFE: please tighten up FAQ entry How can I use a vault password with ansible-navigator

ISSUE TYPE

Doc issue

SUMMARY

when I filed #1559 I overlooked that for the example to work, the last two lines should be shown as one. Sorry, my bad, I could have fixed that alredy when I created #1560.

Seeing too often users disregarding security warnings just because they  find an example to copy and paste next to said warning, it would be nice if the FAQ removed the echo my_password >> ~/.vault_password example or at least moved it to second place

it might be useful to explain the use of bash's HISTCONTROL together with using an env var in a vault password client script a bit more in depth (so that users feel confident to prefer that method over passwords in cleartext on disk).

Since doing is better than just asking :nerd_face:  I'll open an PR shortly with commits to pick or discuss or drop.

Closed by #1572


FEATURE REQUEST

Description
fix the nav bar, it shouldn't scroll down with screen

Wanna Work upon[y/n] ??
no

@freecoursera I would like to work on this issue pls. assign it to me

@freecoursera  I want to work please assign me this

Thank you @vedantmistryy for assigning me this issue , I will be blessed if you  hacktoberfest accepted label to this issue

@DhairyaMajmudar issue fixed?

Yes @vedantmistryy sir almost complete

(Hehe)[https://github.com/freecoursera/freecoursera.github.io/pull/56]


YOURLS not re-reading config when running in Docker?

[X] I've read the guidelines for Contributing to YOURLS Projects
[X] This request isn't a duplicate of an existing issue
[X] I've read the docs and followed them (if applicable)
[X] This is not a personal support request that should be posted on the YOURLS Discourse community

Describe the bug
I'm using the change password plugin, along with YOURLS in a Docker container and /var/www/html/user mapped to a directory on the host. The plugin is updating config.php with the new user's password but logging out and logging back in still references the old password. The container has to be stopped and started to get the new password to work.
To Reproduce
Steps to reproduce the behavior:

Run YOURLS in a Docker container with the change password plugin active.
Log in and change the password. Confirm that the encrypted password has changed.
Log out and log in with the new password - failure.
Log out and log in with the old password - success.
Stop and restart the Docker container.
Log out and log in with the new password - success.

Expected behavior
YOURLS should be re-reading the config.php file.
Actual behavior
It appears that YOURLS only reads the config.php file upon startup and never again.
Versions
Latest YOURLS Docker container with the change password plugin installed on the host under the plugins directory.

This has initially be reported at https://github.com/vvanasten/YOURLS-Change-Password/issues/2 but it does look like the plugin is doing what it should be doing.


Connector for Biomedical Literature

Creates a connector for #254
Implementation details
The implementation involves the introduction of SimpleBiomedConfig, BiomedIngestDoc and BiomedConnector which ingests documents from the PDF Download.
At the moment, it downloads from the Individual Article Download  as tar.gz files but yet to work on the ingestion of such files.

Hi @cragwolfe I have this PR open which works for PDFs and can get a first review.
However, for the tar.gz files, I'm not quite sure what is desired. I have it in mind to extract contents of the tar.gz files and process them, is this the desired behaviour?
I can see there is a Filetype.ZIP in the code but I'm not quite sure what the objective is for compressed files.

However, for the tar.gz files, I'm not quite sure what is desired. I have it in mind to extract contents of the tar.gz files and process them, is this the desired behaviour?
I can see there is a Filetype.ZIP in the code but I'm not quite sure what the objective is for compressed files.

Hi @ajjimeno I'd appreciate your input on this too

I gave this some thought and I have an idea on how to handle compressed files.
We have a Connector (or something similar for such files), instead of them downloading the files, the decompress.
Hence, at this point of the code:
https://github.com/Unstructured-IO/unstructured/blob/7c619f045b2c7ffdd2f79158bbd986db1c096452/unstructured/ingest/main.py#L72-L89
After the processing of files (where compressed files will fail), we try to extract the files that were not processed (compressed files only).
In a pattern similar to:
        docs = self.COMPRESSED_CONNECTOR.get_decompressed_and_ingest_docs()

        # remove docs that have already been processed
        if not self.reprocess:
            docs = self._filter_docs_with_outputs(docs)
            if not docs:
                return

The call pass the decompressed files into the multiprocessing call as usual.
Please let me know if this makes sense and matches the intention for the project.

@HAKSOAT while Crag provides an answer. There are different ways in which files can be access from PMC. One is through the compressed files as you have seen already. An alternative way is to identify the PMIC ids of the PDFs and download each individual PDF file. I do not have a preferred way to do it, i.e. downloading the compressed files versus individual PDF documents.

@HAKSOAT while Crag provides an answer. There are different ways in which files can be access from PMC. One is through the compressed files as you have seen already. An alternative way is to identify the PMIC ids of the PDFs and download each individual PDF file. I do not have a preferred way to do it, i.e. downloading the compressed files versus individual PDF documents.

Thanks for the response @ajjimeno The download of individual PDF documents already works in this PR. I only worked on the compressed files because the issue document #254 asked that "The connector is able to process a document using the Individual article download." where articles refer to tar.gz files. If PDF suffices and this isn't necessary, I can take it out.

@HAKSOAT I see. In addition to the PDF, there might be additional data
files such as supplementary material linked to the document and available
in the tar.gz file, which is why there is a compressed version of the
article. It would be great to be able to get all the individual
documents linked to an article. I think processing the PDF documents is
already a good way forward. Being able to process the tar.gz files would be
really an interesting bonus. Thanks!
On Fri, Mar 10, 2023 at 11:12 AM Habeeb Shopeju @.***>
wrote:

@HAKSOAT https://github.com/HAKSOAT while Crag provides an answer.
There are different ways in which files can be access from PMC. One is
through the compressed files as you have seen already. An alternative way
is to identify the PMIC ids of the PDFs and download each individual PDF
file. I do not have a preferred way to do it, i.e. downloading the
compressed files versus individual PDF documents.
Thanks for the response @ajjimeno https://github.com/ajjimeno The
download of individual PDF documents already works in this PR. I only
worked on the compressed files because the issue document #254
https://github.com/Unstructured-IO/unstructured/issues/254 asked that
"The connector is able to process a document using the Individual article
download https://www.ncbi.nlm.nih.gov/pmc/tools/ftp/#indart." where
articles refer to tar.gz files. If PDF suffices and this isn't necessary,
I can take it out.
—
Reply to this email directly, view it on GitHub
https://github.com/Unstructured-IO/unstructured/pull/345#issuecomment-1463021150,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6BZDMHQ7S7GVIYDBTTJDDW3JWXTANCNFSM6AAAAAAVRZLEFQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the review. I will attend to them shortly.

On the .tar.gz vs. PDF question, IMO the PDF solution in this PR is very much sufficient!


Unreleased LinkedListAsArray to drafts.

I should at least find a better name for it. FIRO (First In Random Out) is possible, QueueWithDesist another.

I like FIRO :nerd_face:


Enhancement: Overload CollectionAssertions.OnlyHaveUniqueItems() with IEqualityComparer

Just ran into a case where I want to assert that all of the objects have identical values for all properties EXCEPT one in particular.  So my collection items should all be unique, but almost the same.
Throwing in an IEqualityComparer seems like an easy way to facilitate this...
Any other ideas on how I might compose this kind of assertion?

Why not use collection.ShouldAllBeEquivalentTo(otherCollection, options => options.Excluding(x => x.SomeProperty)?

Yeah, that's obviously much better -- must've missed those options before.  Thanks!


TextureArray/TextureLayered Mipmaps not working

Godot version
3.3.2
System information
Windows 10 (ver. 2004), GLES3, NVIDIA GeForce RTX 2080
Issue description
When creating a Texture array with Flag 1 (FlagsEnum.Mipmaps), no mipmaps are created for the layers, nor will existing mipmaps be transfered over.
Since it is possible to set this flag the expected behaviour would be that mipmaps will be created for all layers.
Steps to reproduce
Using C#:

Create a TextureArray

TextureArray textureArray = new TextureArray();
textureArray.Create(2048, 2048, 1, Image.Format.Rgba8, (int)TextureLayered.FlagsEnum.Mipmaps);

Add Image

textureArray.SetLayerData(myImage, 0);

If in a prior step mipmaps were created they wont get transfered over. ( myImage.GenerateMipmaps() )

Check if the Image inside the Texture got any mipmaps

bool hasMipmaps = textureArray.GetLayerData(0).HasMipmaps(); //will be false

Note:
You can also check for mipmaps looking at the size of the Data array of the Image using myImage.GetData().Count() or by checking with .GetMipmapOffset
Minimal reproduction project
No response

Related to/duplicate of https://github.com/godotengine/godot/issues/34312?

This was a lot of fun to explore.
The problem in your MRP lies in get_layer_data(). get_layer_data()` calls texture_get_data()`` which is defined here:
https://github.com/godotengine/godot/blob/f3d931410bfec2cf19f5fbec0286087f75bd8fb1/drivers/gles3/rasterizer_storage_gles3.cpp#L1005
texture_get_data() has a different pathway when dealing with layered textures. Unfortunately, it seems that whoever implemented that pathway forget about mipmaps. So texture_get_data() always returns an image without mipmaps when called on a Texture3D or TextureArray. This doesn't mean your image doesn't have mipmaps. The fact that mipmaps are missing from texture_get_data is a bug
You are right to point out that mipmaps are created on texture creation. In your MRP mipmaps are created. However if myImage doesn't have mipmaps, then the mipmaps in textureArray will be ignored when you call setLayerData(), which makes sense, you are only giving Godot a single mip level so it only sets a single mip level. Accordingly, even though you have mipmaps, you are likely never seeing them. This should be better documented.
The combination of these two issues made it look like your TextureArray was never creating mipmaps.
Below is an MRP that shows mipmaps working correctly with TextureArray:
var tex = load("res://icon.png")
var image = tex.get_data() # need to load as a texture otherwise mipmaps are not generated
var textureArray = TextureArray.new()
textureArray.create(64, 64, 1, Image.FORMAT_RGBA8, 1) # allocates space for mipmaps
textureArray.set_layer_data(image, 0)
print(textureArray.data["flags"]) # prints "1" because the mipmap flag is enabled
print(textureArray.get_layer_data(0).has_mipmaps()) # prints "false" because
$ColorRect.material.set_shader_param("tex_array", textureArray)

Shader code for the ColorRect:
shader_type canvas_item;

uniform sampler2DArray tex_array;

void fragment() {

	COLOR = textureLod(tex_array, vec3(UV, 0), 3.0*3.0*sin(TIME));
}

Thanks for looking into this!
This would have been a mayor performance issue.
I will test your MRP later today.

Can confirm this works.
I want to add that:

var image = tex.get_data() # need to load as a texture otherwise mipmaps are not generated

is not 100% correct as you dont need to load it as a texture.
Just call .GenerateMipmaps() on your image before adding it to the array.
This is useful when you dont import your image as a texture


java.lang.IllegalStateException

I have implemented RxDownloader and getting IllegalStateException when download Failed in RxDownloader$DownloadStatusReceiver.onReceive.

Hi sorry for the late reply. Can you give me a complete stack trace of that?


realsense d435, raspberry  - No device connected

Hi, i have realsense d435, raspberry 4 and the program, it runs in a loop, receives frames, but sometimes 1 time a day, 1 time in 2 days it freezes with the error "No device connected", restarting the program does not help, only restarting the device, which may be associated with?
  File "foo.py", line 191, in get_frame_from_realsense
    pipeline.start(config)
RuntimeError: No device connected

Thanks for answer!

Hi @SmirnovL  If a RealSense 400 Series camera in a long-run session remains within its recommended temperature range then if a problem occurs hours / days later the cause will typically be one that is not related to the camera hardware.   This could include a glitch on the USB port / USB cable, or a problem on the computer hardware or its operating system.
The official recommended maximum operating temperature is 35 degrees C, though the camera should be able to operate normally at up to 42 degrees C in real-world conditions before it may begin to experience problems above that temperature.
It may be helpful to perform a long-run session with the camera in the RealSense Viewer tool instead of your own program and monitor the ASIC and Projector temperatures to see whether they rise substantially over time.

@MartyG-RealSense how i can get temperature from python command? the camera is located in a small box with a raspberry and on a raspberry the temperature is 55 degrees

import pyrealsense2 as rs

pipeline = rs.pipeline()
config = rs.config()
config.enable_stream(rs.stream.depth, 1280, 720, rs.format.z16, 30)

cfg = pipeline.start(config)
depth_sensor = cfg.get_device().first_depth_sensor()

# Get device temperature
tempProj = depth_sensor.get_option(rs.option.projector_temperature)
print("tempProj: " + str(tempProj))
tempAsic = depth_sensor.get_option(rs.option.asic_temperature)
print("tempAsic: " + str(tempAsic))

@MartyG-RealSense hmm 54-58С, its not ok? =)

54-58 degrees C is not a healthy camera temperature and there is a good chance of problems occurring at that temperature range.  That kind of internal temperature may occur if the natural temperature of the location that the camera is being used in is strongly heating the camera casing, or if there is an electrical problem with the USB port or USB cable that is causing overheating.
If the camera casing is hot to the touch after only a few minutes of activating from a cold start then that would be a good indicator that there may be a USB electrical problem.

Hi @SmirnovL  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Add justfile for building wasm examples

Just writes the build_examples_wasm.bat as a justfile to get cross-platform shelling for free (I think)
The command to build the examples becomes: just build_examples

understandable, but I don't really get the security concerns tho, as it is almost 1-to-1 with having a bat file in the repo.
(the detail would be that https://github.com/casey/just usually shells out to sh and thus would be arguably more useful across systems. under the current system, Mac and Linux get the short end of the stick, and have to run the commands manually from the Windows batch file to build the warm examples)


core: update _rm_titles to account for title argument name bug

Issue : For functions which have an argument with the name 'title', the convert_pydantic_to_openai_function generates an incorrect output and omits the argument all together. This is because the _rm_titles function removes all instances of the the key 'title' from the output.
Description : Updates the _rm_titles function to check the presence of the 'type' key as well before removing the 'title' key. As the title key that we wish to omit always has a type key along with it.
Potential gap if there is a function defined which has both title and key as argument names, in which case this would fail. Maybe we could set a filter on the function argument names and reject those with keyword argument names.
No dependencies. Passed all tests.

[x] PR title: "package: description"

Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..."  for template changes, "infra: ..." for CI changes.
Example: "community: add foobar LLM"

[x] PR message: Delete this entire checklist and replace with

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change
Twitter handle: if your PR gets announced, and you'd like a mention, we'll gladly shout you out!

[x] Add tests and docs: If you're adding a new integration, please include

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/docs/integrations directory.

[x] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Additional guidelines:

Make sure optional dependencies are imported within a function.
Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
Most PRs should not touch more than one package.
Changes should be backwards compatible.
If you are adding something to community, do not re-import it in langchain.

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, hwchase17.

@baskaryan Let me know if it makes more sense to set an explicit check on function argument names to raise an error for argument names like 'title' and 'key'. Or if the above solution is good enough

@baskaryan @eyurtsev Let me know if any changes are required to move forward.

@baskaryan Added unit tests.
Also updated the logic for the function to account for the edge case that I had previously mentioned. The new logic should hold irrespective of the use of keywords like 'title', 'type' and 'properties' as Class variable names

@baskaryan @eyurtsev


jalik_ufs_1.UploadFS.Uploader is not a function

Dears,
I'm new to Meteor development here.
I'm using it as a backend server to my Ionic2 application so i'm using Angular2+Meteor
and i'm trying to upload image to server
here the tutorial I follow:
https://angular-meteor.com/tutorials/socially/angular2/files-and-uploadfs
but i get an error
Exception while invoking method 'uploadImage' TypeError: jalik_ufs_1.UploadFS .Uploader is not a function
I declared the UploadFS on my declaration file and compiler reads it. but when i run upload function it doesn't work.
also when i console.log(UploadFS) it return an object :
I20170124-22:47:19.728(2)? { store: { GridFS: [Function], Local: [Function] },
I20170124-22:47:19.728(2)?   tokens:
I20170124-22:47:19.729(2)?    { _makeNewID: [Function],
I20170124-22:47:19.729(2)?      _transform: null,
I20170124-22:47:19.730(2)?      _connection:
I20170124-22:47:19.731(2)?          Config: [Function],
I20170124-22:47:19.757(2)?   config:
I20170124-22:47:19.757(2)?    { defaultStorePermissions: null,
I20170124-22:47:19.757(2)?      https: false,
I20170124-22:47:19.758(2)?      simulateReadDelay: 0,
I20170124-22:47:19.758(2)?      simulateUploadSpeed: 0,
I20170124-22:47:19.759(2)?      simulateWriteDelay: 0,
I20170124-22:47:19.759(2)?      storesPath: 'ufs',
I20170124-22:47:19.759(2)?      tmpDir: '/tmp/ufs',
I20170124-22:47:19.760(2)?      tmpDirPermissions: '0700' },
I20170124-22:47:19.760(2)?   Filter: [Function],
I20170124-22:47:19.760(2)?   StorePermissions: [Function],
I20170124-22:47:19.761(2)?   Store: [Function] }

If you can tell me any solution for this issue.
regards.

Have you imported the UploadFS object ?
import {UploadFS} from 'meteor/jalik:ufs';

Yes I did

I have same issue, any news update  here ?


Uncaught ReferenceError: module is not defined at push.js

Expected Behaviour
I should be able to run my project after creating it using phonegap cli. I created my app using the push template.
Actual Behaviour
I was able to add the plugin-push without any problem. The plugin was added automatically to my config.xml and package.json.
However when i try to load my page on a browser, I encountered an error saying :

Uncaught ReferenceError: module is not defined at push.js:330
Uncaught TypeError: PushNotification.init is not a function.

Reproduce Scenario (including but not limited to)
Steps to Reproduce

Create a new app using "phonegap create myApp --template push"
Add the plugin Push Notification "phonegap plugin add phonegap-plugin-push"
Open the index.html file(under www) on a browser.

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
N/A
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
N/A
Cordova CLI version and cordova platform version
cordova 9.0.0
Plugin version
honegap-plugin-push  2.2.3
Sample Push Data Payload
N/A
Sample Code that illustrates the problem
module.exports = {
/**

Register for Push Notifications.

This method will instantiate a new copy of the PushNotification object
and start the registration process.

@param {Object} options
@return {PushNotification} instance
*/

init: function init(options) {
return new PushNotification(options);
},
Logs taken while reproducing problem
Uncaught ReferenceError: module is not defined
at push.js:354
(anonymous) @ push.js:354
index.js:36 Received Device Ready Event
index.js:37 calling setup push
index.js:41 calling push init
index.js:42 Uncaught TypeError: PushNotification.init is not a function
at Object.setupPush (index.js:42)
at Channel.onDeviceReady (index.js:38)
at Channel.fire (cordova.js:798)
at cordova.js:229

Solved!
The browser will give you this error if you are previewing your app using the index.html from the root folder of your application.
It should be compiled first to a specific platform so you can preview the app from the compiled files. This is the path in case of Android: platforms\android\app\src\main\assets\www.
Thanks for ignoring this question.This is just a minor mistake which can be answered easily. I thought phonegap has an active support but i am disappointed.


[uwp] Frame.Navigate( typeof(class_derived_from_Page)) crashes like a boss in Release

Describe the bug
frame.Navigate(typeof(survey_page)); crashes when survey_page is a simple class derived from Page, like this:
class survey_page : Page { ... }

But to make it more with a twist (i.e., let the programmers spend a few good hours cursing, and maybe they'll figure it out), this works perfectly in Debug mode.
To make things even cooler, the crash is one of those "You're f--ed" crash - even if you have a handler to catch ALL exceptions, it doesn't get called.
This situation is also never described in the docs, so I really don't know who's fault it is:

the "awesome" WinRT - my best is on this one
poor documentation
bug in UWP

Steps to reproduce the bug

create a class

 class survey_page : Page { ... }

Create the rest of the code, and rejoice since it all works in debug mode.
Do unpaid overtime figuring it out.

As a side note, if I create the class from the wizard, with a xaml behind it, it all works.
Screenshots

Windows 10 version
Saw the problem?

November 2019 Update (18363)

Device form factor
Saw the problem?

Desktop

Weird that this error only occurs on release. What exactly do you mean by

survey_page is a simple class derived from Page, like this:
class survey_page : Page { ... }

Is this page a class that should trigger that crash?

Weird that this error only occurs on release.

Not too weird, debug and release modes use different compilers:
https://blogs.windows.com/windowsdeveloper/2015/08/20/net-native-what-it-means-for-universal-windows-platform-uwp-developers/
Also, as mentioned there:

Local native compilation via the “Release” configuration will enable testing your application in an environment that is similar to what your customers will experience. It is important to test this on a regular basis as you proceed with development.

A good rule of thumb is to test your app this way periodically throughout development to make sure you identify and correct any issues that may come from the .NET Native compiler.

@jevansaks this sounds very familiar to me, does it ring any bells with you?

@kmgallahan Thank you for posting that article! I did not knew that the differences between debug and release were that big.

Weird that this error only occurs on release.

Not too weird, debug and release modes use different compilers:

Well, as a user of this -- this is insane. Clearly, you need to check your app in Release mode, but this sounds unacceptable.
I mean, even if Debug and Release behave differently, this event crashes in Release without any trace -- you can't log anything, how are you, as a developer find the cause of this? Luckly, I was able to narrow it down rather fast, but this is not always the case.

https://blogs.windows.com/windowsdeveloper/2015/08/20/net-native-what-it-means-for-universal-windows-platform-uwp-developers/

This is NOT compilation with .net native. It's in Release mode, without .net native.

Probably related to lack of XAML data going into the .NET Native compilation.

As said above, even this was true, it should fail in a better manner -- like, allow my app to handled an Unhandled exception. At this point, it's a sudden crash without any info

This is NOT compilation with .net native. It's in Release mode, without .net native.

Release mode for UWP apps uses .NET Native, it isn't something you opt into.

Weird that this error only occurs on release. What exactly do you mean by

survey_page is a simple class derived from Page, like this:
class survey_page : Page { ... }

Is this page a class that should trigger that crash?

I'm not sure. I did notice the public sealed partial class keywords on it, so I'm not sure, My class was not sealed, nor partial.
I do know that creating a Page class with the wizard (along with the xaml) fixed the issue.
The original class I was using was this:
class survey_simple_page : Page {
    private static ILog logger = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);

    private static survey_simple_page last_shown_page_;
    public static Action on_new_page;

    public survey_simple_page() {
        Loaded += Survey_simple_page_Loaded;
    }

    public static survey_simple_page last_shown_page => last_shown_page_;

    private void Survey_simple_page_Loaded(object sender, Windows.UI.Xaml.RoutedEventArgs e) {
        Loaded -= Survey_simple_page_Loaded;
        last_shown_page_ = this;
        on_new_page?.Invoke();
    }
}

I did add extra logging while investigating the error, and the constructor never got called.

@kmgallahan

This is NOT compilation with .net native. It's in Release mode, without .net native.

Release mode for UWP apps uses .NET Native, it isn't something you opt into.

@jtorjo You might try checking the .NET Native box, re-building and re-testing to see if it is related to having that disabled in a release build.

@StephenLPeters  this sounds like .NET Native missing the reflection info, which you can curate with rdinfo.xml.
But I see from the comments that disabling .NET Native doesn't change anything which is strange to me.
I wonder if "public" visibility modifier is the other difference?

I have tried reproducing the same behavior as @jtorjo outlined using, the Settings page of the XCG (branch available here) and I also got the app to crash, though it did fail in KernelBase.dll :

I did add extra logging while investigating the error, and the constructor never got called.

Same happened for me, luckily on my machine I was also able to reproduce this in debug.

@jevansaks

I wonder if "public" visibility modifier is the other difference?

The SettingsPage @chingucoding tested with is public sealed partial, so adding public doesn't seem to make a difference.

I did add extra logging while investigating the error, and the constructor never got called.

Same happened for me, luckily on my machine I was also able to reproduce this in debug.

@chingucoding Are you on the same win version as me? I'm on 1909 (18363.952)
It's definitely an improvement that it happens in debug for you. I have a hunch if you remove the public sealed partial, I won't happen in debug.

@kmgallahan

@jtorjo You might try checking the .NET Native box, re-building and re-testing to see if it is related to having that disabled in a release build.

I'm sorry I can't do that, since my app doesn't work well compiled with .net native -- up until recently, the compiler was crashing at build, and after that, my app would fail pretty fast at runtime. I did not have the time to investigate all the issues with ".net native", so I turned it off.

@chingucoding Are you on the same win version as me? I'm on 1909 (18363.952)

I tested this on Insider Preview version 19555. Maybe this is a bug related to Windows internal issues, since it fails in KERNELBASE.dll in my case and in CoreCLR.dll in your case.

I have a hunch if you remove the public sealed partial, the crash won't happen in debug.

Unfortunately (or luckily) this issue still happens in Debug when I remove the public sealed partial from the class definition.

Unfortunately (or luckily) this issue still happens in Debug when I remove the public sealed partial from the class definition.

Do you have ".net native" enabled in debug mode?

I'm sorry I can't do that, since my app doesn't work well compiled with .net native

This was for diagnostic testing purposes, rather than as a proposed workaround.
If it won't even build with .NET Native then I guess the test can't be done, unless a minimal repro app tries it out @chingucoding

Do you have ".net native" enabled in debug mode? (I don't)

Also, I'm testing x64 builds.

Using this configuration, the app crashes upon calling Navigate :

After a bit of head scratching, I think I finally understand what might be going on here. This is likely an issue with IXamlMetadataProvider and the way we do type activation. I get the callstack below with the repro that @chingucoding shared. It looks like we go though the type information system generated in the app by the markup compiler. However, if there is no xaml, the markup compiler probably does not generate type information for those types (look at the difference in the generated typeinfo.g.cs file under obj folder)?
The tricky part is the difference between debug and release builds. I believe the culprit there is a difference in the markup compiler choosing different paths based on debug/vs retail builds to speed up the debug builds.
You can cause the same crash in debug builds by adding this to your project file.
false
I believe instead of using the generated code for type information, we use reflection in this case which will work fine because reflection will see all the types (not just the ones defined in markup).
@danzil @RealTommyKlein as FYI. Can the reflection based provider be used in release ? or is that not allowed because of .net native ?.
>	Windows.UI.Xaml.dll!DirectUI::ActivationAPI::ActivateInstance(const CClassInfo * pType, IInspectable * pOuter, IInspectable * * ppInstance) Line 42	C++
 	[Inline Frame] Windows.UI.Xaml.dll!DirectUI::ActivationAPI::ActivateInstance(const CClassInfo *) Line 29	C++
 	Windows.UI.Xaml.dll!DirectUI::NavigationCache::LoadContent(HSTRING__ * descriptor, IInspectable * * ppInstance) Line 148	C++
 	Windows.UI.Xaml.dll!DirectUI::NavigationCache::GetContent(DirectUI::PageStackEntry * pPageStackEntry, IInspectable * * ppIInspectable) Line 99	C++
 	Windows.UI.Xaml.dll!DirectUI::Frame::PerformNavigation() Line 512	C++
 	Windows.UI.Xaml.dll!DirectUI::Frame::StartNavigation() Line 434	C++
 	Windows.UI.Xaml.dll!DirectUI::Frame::NavigateImpl(Windows::UI::Xaml::Interop::TypeName sourcePageType, IInspectable * pIInspectable, Windows::UI::Xaml::Media::Animation::INavigationTransitionInfo * navigationTransitionInfo, unsigned char * pCanNavigate) Line 381	C++
 	Windows.UI.Xaml.dll!DirectUI::FrameGenerated::Navigate(Windows::UI::Xaml::Interop::TypeName sourcePageType, IInspectable * pParameter, Windows::UI::Xaml::Media::Animation::INavigationTransitionInfo * pInfoOverride, unsigned char * pReturnValue) Line 494	C++
 	Windows.UI.Xaml.dll!DirectUI::Frame::NavigateImpl(Windows::UI::Xaml::Interop::TypeName sourcePageType, IInspectable * pIInspectable, unsigned char * pCanNavigate) Line 340	C++
 	Windows.UI.Xaml.dll!DirectUI::FrameGenerated::Navigate(Windows::UI::Xaml::Interop::TypeName sourcePageType, IInspectable * pParameter, unsigned char * pReturnValue) Line 476	C++
 	Windows.UI.Xaml.dll!DirectUI::Frame::NavigateImpl(Windows::UI::Xaml::Interop::TypeName sourcePageType, unsigned char * pCanNavigate) Line 327	C++
 	Windows.UI.Xaml.dll!DirectUI::FrameGenerated::Navigate(Windows::UI::Xaml::Interop::TypeName sourcePageType, unsigned char * pReturnValue) Line 512	C++

Yes, the reflection provider appears to be why this works in Debug but not Release.  The reflection provider won't work in Release builds since they use .NET Native, which has restricted reflection support.
To get the scenario working in Release, you can add the Windows.UI.Xaml.Data.Bindable attribute to the survey_page class so there will be generated type information for it in the IXamlMetadataProvider, and the Xaml framework can correctly use/navigate to it:
[Windows.UI.Xaml.Data.Bindable]
class survey_page : Page { ... }

Thanks @RealTommyKlein for providing the explanation.


Corrected typo "Bescrheibung" to "Beschreibung"

Thanks for the report and the fix.
We manage translations via https://poeditor.com/projects/view?id=221603 (and some custom export/import) code. I have updated the translation there and will import it shortly.
You are very welcome to create yourself an account on POEditor and fix any other errors you have spotted and/or contribute to expanding the coverage of the translations.

Thanks for the link. I created an account and will try to contribute some...


66 - checking STIX obj ID from TAXII server

issue #66

Codecov Report

Merging #92 into master will decrease coverage by 0.05%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
- Coverage   96.55%   96.49%   -0.06%
==========================================
  Files          53       53
  Lines        4875     4878       +3
==========================================
  Hits         4707     4707
- Misses        168      171       +3

Impacted Files
Coverage Δ

stix2/sources/taxii.py
33.33% <0%> (-1.29%)
:arrow_down:

Continue to review full report at Codecov.

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

Do we need to check the Id of returned objects twice? Seems like we're doing it both on line 129 and line 163.

@clenk : I think one is in get and the other is in all_versions.

Thank you, @mbastian1135!


Bot ignores channel ID and sends to #general

Exactly what it says on the tin. Now that I have the bot actually running, it doesn't seem to want to listen to me and send where it should. No matter what channel ID I give it, it pops everything off to #general anyway.
Nothing shows in the debug log that seems out of place - just Discord heartbeats and logs of messages being sent.

Just pushed a fix. Give it a try and let me know

All fixed with that commit! Thanks for your work. :)


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.10.89",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=450121075a6a6f1d50f97225d3396315309d61a1"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "450121075a6a6f1d50f97225d3396315309d61a1",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "hamradio: defer ax25 kfree after unregister_netdev\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.89 by commit 450121075a6a6f1d50f97225d3396315309d61a1. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002814


What is the status of the rp2040 port for Raspberry Pi Pico/W boards?

What is the status of the rp2040 port for Raspberry Pi Pico/W boards?

No more updated.

If you want to support for host driver, you can ask official.


Added UserName and Password support to heX16/mqtt-free-pascal component

Thanks for your work on the library!
Here is the pull request.
I just want to make you aware of some changes I made before you merge:

I was not happy that randomize was called in a component library. So I swapped that out with DateTimeToUnix(Time) which forces me to use the dateutils unit.
I also removed the duplicate set of TMQTTReadThread .FClientID because it already happens in TMQTTClient. FClientID as shown above.
I'm not sure if the username and password flags must be of type word, boolean actually makes more sense for the VariableHeaderConnect method. It was just hacked in to get it working.
Finally if FUserName and FPassword is an empty string('') I assume it is not set. This behaviour is borrowed from another pascal mqtt library.

Let me know what you want me to change before you accept the merge request.
Finally I was already using your forked library, you just did not enable issues for your github project so I created an issue in ZiCog repo.

I apologize for taking PR for so long - there was no time to analyze the changes.

I was not happy that randomize was called in a component library.

Yes, I agree that interfering in "random seed" is wrong.

I'm not sure if the username and password flags must be of type word

in that function is not a good way to work with bits (we inherited it from the original library). the right way is to set bits (https://wiki.freepascal.org/Bit_manipulation).

This behaviour is borrowed from another pascal mqtt library.

Thanks for the link to this library - I did not see it (it appeared later). Have you tried using it?

I only went looking for other mqtt libs when I realised mqtt-free-pascal did not support user auth.
The lib does look well documented and coded. But it depends on the authors own util library and then another crypto library.
So I opted to rather try adding user auth to mqtt-free-pascal as added by this merge request :)
Step next from my side will be to get mqtt-free-pascal on the online package manager and then maybe try to hack in SSL/TLS support.
But don't hold your breath. Because I'm actually actively trying to procure some freelance work, and that on its own seems like a full-time job! :/
Watch your issue tracker if I get the ball rolling I will create the issues for my next two goals!
Later,


fix: role update sync grouping policies

role update & delete sync grouping policies.

@tom2nonames fix linter:

@leo220yuyaodog

hi, Do we have any updates on this PR? @tom2nonames @hsluoyz @leo220yuyaodog

@notdu the PR author didn't respond to our code reviews for months and now the PR has code conflicts.
If you would like this feature, you can create a new PR (can refer to this PR's code or not), we can continue there

ok, i'll open a new PR and refer to this PR.

Replaced by: https://github.com/casdoor/casdoor/pull/1477


Create Kanaban board, and Assign tasks.

GitHub project: https://github.com/users/ibr5500/projects/6
Note: There are 4 members in this team.
member 1: @GOlivierNation
member 2: @NatiG25
member 3: @KanoCode
member 4: @ibr5500
Screenshot:

Hi Team,
While you made a great effort in this project, unfortunately, I cannot proceed to review your board.
Invalid Code Review Request
You have submitted a project, that does not contain a screenshot of the current state of the Kanban Board at the time of review request which is seen as a requirement here
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the Issue thread if something is not 100% clear.
Please, do not open a new Issue for re-reviews. You should use the same Issue submitted for the first review, either valid or invalid unless it is requested otherwise.

Hi Team,
In my opinion, you are ready to move on to the next step! ⏭
Good luck! 🎉
Feel free to leave any questions or comments in the Issue thread if something is not 100% clear.

Thank you @mcrd25 for the review


Add parameter serviceUrl

I added a parameter serviceUrl, for letting configure the service base URL.
This is useful when you're developing or when you host you're on version of the compiler services.
The default behavior is still the same.

@MonsieurV thanks for your contribution. Do you accept to rename the switch from serviceUrl to simply url because there are no other urls.  My proposal is :
	pflag.String("url", "", "The base url for the service. If empty, the default URL is used.")

@MonsieurV Two more things.

I prefere to not commit my .gitignore (I have one also) in this repo.
I prefer to keep my .goreleaser.yml as is, including the archives section you commented on.

Thanks in advance.

@kpym ok, that's done. I've squashed so the history is kept clean.

@MonsieurV Thanks !


./configure.sh fails on mac

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Mac OSX 10.14.3
TensorFlow installed from (source or binary): none
TensorFlow version (use command below): none
TensorFlow Addons installed from (source, PyPi): source
TensorFlow Addons version: latest master
Python version and type (eg. Anaconda Python, Stock Python as in Mac, or homebrew installed Python etc): conda
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
Is GPU used? (yes/no):
GPU model (if used):

Describe the bug
Running ./configure.sh tries to install nightly-2.0-preview that is not available for the mac.
`fluid:tensorflow-addons armando$ ./configure.sh
Collecting tf-nightly-2.0-preview (from -r requirements.txt (line 1))
Could not find a version that satisfies the requirement tf-nightly-2.0-preview (from -r requirements.txt (line 1)) (from versions: )
No matching distribution found for tf-nightly-2.0-preview (from -r requirements.txt (line 1))
Describe the expected behavior`

Depends on fixing of https://github.com/tensorflow/tensorflow/issues/27470

We currently have no way of providing python 3.7 whl's and are waiting for the new BUILD toolchain (see #119 )
Once the toolchain is complete... tf-nightly will begin publishing 3.7 wheels and our configure script will work again. Closing as there is nothing we can do here and as of today we do not support python3.7


Fix 2.47.0 release notes

Python SDK still supports protobuf 3.x (3.20.3) dependency
minor cleanup

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @tvalentyn @jrmccluskey


Remove Dave from team page

This replaces and closes #183.

@zachlatta, don't forget to assign someone to this : )


Sanitize TOTP secret to prevent binascii.Error: Non-base32 digit found

Slight usability problem with TOTP providers like ConEd - if the user enters a TOTP secret with trailing(*) white space, the exception binascii.Error: Non-base32 digit found is raised by the base64 module. For Home Assistant, the user sees Unknown error which can be resolved by trimming out spaces that might arise from copy and paste.  However, this isn't obvious to the user and seems like a problem with the integration.
See https://github.com/home-assistant/core/issues/101829#issuecomment-1783144419
(*) I only tested with trailing white space but I suspect leading white space would cause the same problem.
The Opower config flow in the Home Assistant integration should probably do the cleaning and/or have a check for a TOTP secret with invalid characters.
However, the Opower module itself could remove leading/trailing spaces from the secret and/or validate that the secret s a valid base32  string that can be decoded earlier during configuration.  (Right now I think the string that can't be decoded isn't detected until a login is attempted.)
Edit: Forgot to mention, ideally if the opower module detects that the TOTP secret can't be decoded, it should throw an error that will result in a more meaningful error message to the end user.

Fixed with https://github.com/tronikos/opower/commit/94723fa229c46df666d5ccd790e28712c9b4adc9

I have the same issue now with Opwer trying to have ORU in NJ while adding new integration in HA with the new version.
File "/usr/local/lib/python3.12/base64.py", line 236, in _b32decode
raise binascii.Error('Non-base32 digit found') from None
binascii.Error: Non-base32 digit found


Fix SPM Support for Xcode 11

Thank you for updating Package.swift file. However, I can't see why @available(iOS 10.0, *) are needed?

I have to test it again, because with the early version of Xcode 11 I have an issue where package where compiling for all platform and all os version :/ since Reusable make sense only on iOS this is why I put that protection in the first place.

Because this pull request seems to be inactive for quite some time now, I've closed it. If you feel this deserves some attention from me feel free to reopen it.


@EqualsAndHashCode changed behaviour when calling super

Previous version: 1.16.12
New version 1.18.4
After upgrade to newer (1.18.4) version, generated hashCode() method changed (and thus broke our test suite).
This is the generated hashCode in the previous (1.16.12) version:
public int hashCode() { 		int PRIME = 59; 		int result = 1; 		result = result * 59 + super.hashCode(); 		return result; 	}
and this is the hashCode (in the same class) after upgrade:
public int hashCode()   {     int result = super.hashCode();return result;   }
This newer code generates another value of hashCode for the same object (missing 'result * 59 + ' part). Consistent hashCode across versions is an imperative for our application.
Is there a config flag for this perhaps? Or is it a bug?

Disclaimer: Just a lombok fan here.
This change is fixing #1505 (according to my proposal). The original behavior was buggy as it returned different hashCode for equal objects, so I don't think, a configuration makes sense.
You might be able get the part back using a dummy field with value one placed first in the class. A method like @EqualsAndHashCode.Include private int dummy() {return 1;} might do, too (the position of the method matters). I'm not sure.
Replacing callSuper=true by a method returning super.hashCode() should probably work. Anyway, as a sure workaround, you can copy the generated code from delombok. It's ugly, I know.

Yes, I can see that the original implementation was a bit flawed - it shouldn't have included 'result * 59' blindly like that. We'll adapt to the correct version of generated code and try to handle the legacy with as little (bad) impact as possible.

Good catch by @Maaartinus - thanks!


Add install.sh for linux - see bash script in comment

Hello, I'm not sure where to put this for a pull request, but I hope that you can make use of this.
Actual Behavior
Cannot double-click jar files in linux (not voxelshop's fault)
Expected Behavior
Unzip then use voxelshop without terminal, or at least have install.sh script to get a GUI icon.

This script can be placed in the voxelshop directory in the zip file.
If PREFIX environment variable is not set, the script uses $HOME/.local (if EUID is 0 [running as root] then /usr/local instead)
The script uses following directories under the PREFIX according to standards: $PREFIX/lib/voxelshop/, $PREFIX/bin/, $PREFIX/share/applications/, $PREFIX/share/pixmaps/
I attached the script to avoid issues with copying and pasting from the web:
install.sh.tar.gz

You would also have to add the following to the linux zip builds:

You'll have add the existing resource/img/icons/application/icons/icon-48x48.png to a  voxelshop/share/pixmaps directory (or other location, see shortcut_path variable)
The attached shortcut file (tar.gz'd desktop file) is needed as well, in the voxelshop/share/applications directory (or other location--see icon_src_path variable)--the install script rewrites the Exec and Icon variables according to PREFIX before installing the shortcut.
com.blackflux.voxelshop.desktop.tar.gz

Hello @poikilos!
Thank you very much for your input here! Much appreciated and I would love to include this. Have a few questions first though:

How would one go about uninstalling (reverting the installation process)?
What did you base your script on? I'm assuming you didn't write this all from scratch?
Would you be able to create an example linux distribution zip file that I can use as an example? That would make it much easier for me to set it up.

Cheers, L~

There is no uninstall. This is expected for installing from source unless autotools or another framework is used for generating a manifest.
I wrote this all from scratch, with trivial syntax help from stackoverflow as usual. Some of it (such as application menu refreshing) is boilerplate from my other projects. I worked as a linux server admin for a while, and all installs that weren't packaged were automated like this. I wrote the script for VoxelShop because I like it so want to encourage development and help my friends install it.
See attached (same as your zip, but with necessary files and the install script--this would be a good way to distribute it to linux users)
voxelshop.tar.gz

Ok, now it creates an install.log and has an uninstall.sh. Enjoy...
voxelshop-1.8.24.tar.gz

@poikilos Sweet, thank you very much! I'll try to review and add it this weekend!

@poikilos Finally had time to review this. Looking good!
I've opened a pr here: https://github.com/simlu/voxelshop/pull/298/files
However ideally we would detect if we "should" install. I.e. fail on windows. The alternative is to have different distros, but I'm not a big fan. What do you think?

I'm not sure what you mean by "fail on Windows." This file would be excluded from windows builds if possible. Is that what you mean by different distros? I see if you want to avoid that. However, maybe just add a big banner for Windows users at the top of the sh file:
#!/bin/sh

# To install on Windows:
# * Make sure you have clicked "Extract All" first
# * right-click on the jar file, then click Send to, Desktop (Create Shortcut)

I've released it as a pre-release: https://github.com/simlu/voxelshop/releases
Let me know if everything looks good and I'll make it a full release!

Pretty good so far! I am sending a PR to deal with the following:

The icon works, but only if java is installed. Often only a "headless' java is installed in linux (such as Ubuntu and Fedora), leading to the program not opening. The best way to deal with this is to change the voxelshop bin to send a notification to the user if it fails to run.
/usr/local/share/pixmaps/voxelshop.png is not being installed, so though the shortcut works, the icon is blank.
I have verified that the PR resolves both issues.

Ignore the first PR on my repo, I accidentally submitted a PR to myself! The open PR #299 is the valid one.

@poikilos Merged and did another release. I'll mark it as a full release once you confirm that everything looks good!

Closing, but feel free to comment if there are any issues!


Split a string on matched tokens, but keep the delimiters

Hi,
I'm trying to split a string on a particular delimiter, but I want to keep the delimiter.
I'm quite new to Ruby-ish ecosystem, so it took me quite  a while to figure few things. However, I still couldn't achieve what I wished for.
Example (taken from http://stackoverflow.com/a/6964207)
s = "split on the word on okay?"

I've tried String#split, but it doesn't allow to keep delimiters (or so it seems, according to source code).
I've tried to play on the regexes like
s.split(/(on)/)

But I am unable to achieve:
["split ", "on", " the word ", "on", " okay?"]

I also tried the "Ruby-way" with the link provided over, but nothing again gave the desired output.
Any help is greatly appreciated,
A.
n.b. I would've normally asked this kind of question on SO but I'm not quite sure about the visibility of Crystal-lang related questions there

Oh, I didn't know that in Ruby if the argument given to split is a regex then the groups are returned in the result array as well. I'll do it.

Hmmm... In fact, maybe we can change things from Ruby, a bit. For example say you want to split by /(foo)+/ but without keeping the delimiter, in Ruby there's no way. So maybe we can add options to these methods.
class String
  def split(separator : Char, limit = nil, include_separator = false)
  end

  def split(separator : String, limit = nil, include_separator = false)
  end

  def split(separator : Regex, limit = nil, include_separator = false, include_groups = false)
  end
end

What do you think?
/cc @jhass

That'd simplify the way to do it, yes. I don't know if it is a common task, but I think doing this would make sense.

Regular expressions are already telling something, so maybe we don't need to introduce new arguments?

/on/ skips the delimiter;
/(on)/ injects the delimiter (captured);
/(?:on)+/ skips the delimiter (not tested, but expected since it groups but doesn't capture).

You can do this already http://carc.in/#/r/2ib so maybe we can keep it simple?

Oh and Ruby also has String#partition, which while only splitting once, has "keep the delimiter" a lot more obvious in its contract.

@jhass Thanks! Yes, let's keep it simple for now. I have the include_separator implementation in a stash in case we want to add it later.
I wonder what's needed for us to start using Stack Overflow... Probably nothing. Maybe we should use the crystal-lang tag as there's also crystal-reports.

I'd love to get a mod to give use the crystal alias though.

:+1: for a stackoverflow tag

I have enough rep to create tags; I just need to find out what question to ask. :)

Going to http://stackoverflow.com/tags?tab=new is not enough?

Nevermind; I just tagged the question asking if anyone has tried Crystal.
Behold...the Crystal tag: http://stackoverflow.com/questions/tagged/crystal-lang
Now somebody needs to ask a question on SO and use this tag within six months to prevent it from being deleted...

@kirbyfan64 Thanks!!
@vyp Thank you too, that'll help spreading that tag in Stack Overflow.

Nice for the SO tag!
@ysbaddaden Indeed, just reading at the Regex is clear to me that'd be what to expect from it.
@jhass Thanks for the link, I'll use this at the moment, although that I feel it seems hacky a bit, but hey, it works :)
Thanks guys, that answered my question!

The solution from @jhass isn't hacky, it's just a regular expression, but it sure is more complicated than having String#split reinject captured expressions !

@ysbaddaden Totally forgot about (?:...) for not capturing a group.
I think it would be fine then to copy Ruby's semantic. It's a simple change that adds flexibility and makes it easy to keep the delimiter, plus maybe other cases.

sweet!

:+1:


Jetpack Compose preview

Whats the best way to preview components if you use tiamat?
Since the lib encourage users to create a components like that:
val SomeComp by navDestination {
    val args = navArgs()
    val viewModel = ...
    // ... further comps ...
}

I tried to preview a component with the following code:
@Composable
fun <NavArgs> PreviewComponent(dest: NavDestination<NavArgs>, navArgs: NavArgs? = null) {
    // ... some setup ...
    val navController = rememberNavController(
        startDestination = dest,
        startDestinationNavArgs = navArgs,
        destinations = arrayOf(dest),
    )
    Navigation(navController)
}

But it fails with
java.lang.ClassCastException: class com.android.layoutlib.bridge.android.BridgeContext cannot be cast to class android.content.ContextWrapper (com.android.layoutlib.bridge.android.BridgeContext and android.content.ContextWrapper are in unnamed module of loader com.intellij.ide.plugins.cl.PluginClassLoader @7d9d7378)   at com.composegears.tiamat.RootDataStorageKt.rememberRootDataStore(RootDataStorage.kt:31)   at com.composegears.tiamat.Platform_androidKt.rootNavControllersStore(Platform.android.kt:12)
...

Hello, I prefer to separate navDestination logic and pure Compose UI. It allows to use Preview without any restrictions.
Please take a look some example:
https://github.com/ComposeGears/Valkyrie/blob/main/idea-plugin/src/main/kotlin/io/github/composegears/valkyrie/ui/screen/intro/IntroScreen.kt

@BioRyajenka
It is better to use smth like
val SomeComp by navDestination {
    val args = navArgs()
    val viewModel = ...
    ScreenContent(args.param1, args.param2, viewModel::onClick)
}

@Composable
fun ScreenContent(
    val param1 : Int,
    val param2: String,
    val onClick : ()->Unit...
){
  // screen logic without linkage to models, args and so
   ....
}

@Preview
fun ScreenPreview(){
    ScreenContent(stub params)
}

Yeah, I did it this way, but with complex logic/structure you also probably don't want to pass lambdas multiple levels down in many places.
Plus, some components are always expected to navigate to the same, and by using lamda you will duplicate code in all caller places.
So I ended up with a mix: extracted a function + passing demo data for the preview purposes, but also created custom dummy navController with reflection and pass it down instead of lambdas.

We will add some options to perview a screen...
Proposed syntax:

val SomeScreen by navDestination<Input>{...}

@Preview
@Composable
fun SomeScreenPreview(){
        ScreenPreview(SomeScreen, args=Input(), freeArgs = Smth, navResult = Smth)
}

Hey @vkatz , thanks for reply!
I think the ideal syntax would be not to introduce any syntax and allow existing rememberNavController and Navigation to work in preview mode, wdyt? Idk if thats achievable though

Taskt to do so already created, this conversation marked as enhancement and we have it on the road.


chore: promote test1 to version 0.0.2

chore: promote test1 to version 0.0.2
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign q1323829945
You can assign the PR to them by writing /assign @q1323829945 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

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

Test name
Commit
Details
Rerun command

verify
76011e91dc2e48de8ee4d8d6c378137069ba822c
link
/test verify

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 jenkins-x/lighthouse repository. I understand the commands that are listed here.


HTML

HTML 💻

User Story
As a boot camp student,
I want a Prework Study Guide website,
so that I can review all the information I learned throughout Prework.

Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed

CL: added HTML with PR on 022623 (Added HTLM code to application #7)
ヾ(⌐■_■)ノ♪


Run task to recursively remove *.rej files

These files are generated when updating the template and a file is ignored by _skip_if_exists. Until conflict resolution is changed in copier-org/copier#613, let's scrub these files away after updating the template.

I have created a VSCode task to do so in the meantime. This issue may become moot if we get rid of all skip_if_exists entries.

No longer an issue.


Migrate some dependency

Reference Issue
Fix #28
What does this implement/fix? Explain your changes.

Migrate to importlib from imp.
Done by : https://github.com/python/cpython/issues/104212
pysoundfile  is now deprecated and use soundfile. ( It was problematic if other packages used soundfile.)

Tested on python 3.10, 3.11, 3.12 and all worked fine.

LGTM, thanks!  I'll try to finish the CI cleanup after merging this.


feat: blobless blocksync

Summary
Assuming that blobs dominate in size within a block yet aren't required for state execution, we could formulate a mode of blocksync whereby nodes request and respond with blocks in a reduced form (without any blobs). The nodes would perform some verification that the provided set of transactions in this reduced block was complete and exhaustive and simply execute and store the reduced block.
I haven't given much thought to verification. Roughly, the responding node would need to provide the transactions in share form including the padding or next share (to prove the end of transactions) and the requesting node would use the data availability header to prove inclusion of those transactions.
The benefits of such a feature are reduced bandwidth and disk space.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Closing this as a duplicate of https://github.com/celestiaorg/celestia-core/issues/994