import SelectDateWidget error

problem
working with both git versions of django (2.0.dev20170222001300) and django-bootstrap3 (branch dev commit 465e61570453d7633a4bb122b22ee5334ad378d8) I encountered this error:
    django-bootstrap3/bootstrap3/renderers.py", line 14, in <module>
    from django.forms.extras import SelectDateWidget
    ImportError: No module named 'django.forms.extras'

Solution
SelectDateWidget is now in django.forms.widgets so editing renderers.py would work for me:
#from django.forms.extras import SelectDateWidget
from django.forms.widgets import SelectDateWidget

cf. #374 :)

This is fixed with the changes in #374.


distinct property in IFindOptions

Hello, i'm trying to do a findAndCountAll on one of the models and i've added an include property however i'm getting an error from the typings that distinct is not part of the property.
BenefInsuranceTransaction.findAndCount({             where: params,             limit: take,             offset: skip,             attributes: ['Id','User','Ind','Stats'],             include: [{model: BenefInsuranceDetail, required:true, where:{DeleteInd:0}, attributes:['BenefInsDetailID']}],             distinct: true         })
I need to use distinct in order to get the correct count because it's the only solution that I've found to get the correct count for paging.
Thanks.

@azalpacir Does this work for you?

hello,
Sorry, I'm not sure how I will be able to use the distinct function in the link you gave because i need to count only if it has the associated model. hope you can enlighten me.
thanks.

Hey @azalpacir, sry for the late response. Can you tell me the exact SQL query you want to achieve?

Hello,
I'm trying to use the findAndCountAll to be used for the pagination of the records. However when using the said function with the "include" property, I'm getting an incorrect count, which is an existing issue in sequelize. The solution was to add a "distinct" property to the main query in order to get the correct count but the sequelize-typescript model does not allow me to add the "distinct" property because it's not included in the IFindOptions interface. When I checked the sequelize library the "FindOptions" has the "distinct" property.

Ah sry, I misinterpreted your code example. You‘re right it is missing. I will mark this issue as a bug.
For now you could use distinct like so {['distinct']: true} or {['distinct' as any]: true}

This still isnt working and the solution of using {['distinct']: true} or {['distinct' as any]: true}
does not work


Decompressed branch spawns at an angle for eccentric orbits

on escape trajectory decompressing branch invokes angry Kraken

Fixed.


STM32L4 ADC: Change hardware trigger configuration

Summary
Change the way that hardware trigger configuration word for regular and injected channels are done. With this change we can setup hardware trigger without assign a timer to trigger ADC.
Impact
No impact is expected.
Testing
This changes was tested on nucleo-l432kc and ADC example app.

Hi @daniel-p-carvalho here is a board config error:
====================================================================================
Configuration/Tool: nucleo-l496zg/nsh,CONFIG_ARMV7M_TOOLCHAIN_GNU_EABI
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Disabling CONFIG_ARMV7M_TOOLCHAIN_GNU_EABI
  Enabling CONFIG_ARMV7M_TOOLCHAIN_GNU_EABI
  Building NuttX...
In file included from ./stm32_adc.c:36:
Error: /github/workspace/sources/nuttx/arch/arm/src/chip/stm32l4_adc.h:221:6: error: #error "ADC1 External trigger must be enabled"
  221 | #    error "ADC1 External trigger must be enabled"
      |      ^~~~~
ERROR: arm-none-eabi-gcc failed: 1
       command: arm-none-eabi-gcc -M '-fno-common' '-Wall' '-Wstrict-prototypes' '-Wshadow' '-Wundef' '-fomit-frame-pointer' '-finstrument-functions' '-ffixed-r10' '-ffunction-sections' '-fdata-sections' '-g' '-march=armv7e-m' '-mtune=cortex-m4' '-mthumb' '-mfloat-abi=soft' '-isystem' '/github/workspace/sources/nuttx/include' '-D__NuttX__' '-DNDEBUG' '-D__KERNEL__' '-Wno-cpp' '-Werror' '-pipe' '-I' '/github/workspace/sources/nuttx/sched' '-I' '/github/workspace/sources/nuttx/arch/arm/src/chip' '-I' '/github/workspace/sources/nuttx/arch/arm/src/common' '-I' '/github/workspace/sources/nuttx/arch/arm/src/armv7-m' ./stm32_adc.c
make[2]: *** [/github/workspace/sources/nuttx/boards/Board.mk:95: .depend] Error 1
make[2]: Target 'depend' not remade because of errors.
make[1]: *** [Makefile:199: .depend] Error 2
make[1]: Target 'depend' not remade because of errors.
make: *** [tools/Unix.mk:583: pass2dep] Error 2
make: Target 'all' not remade because of errors.
/github/workspace/sources/nuttx/tools/testbuild.sh: line 290: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory


Please add leaflet.draw to CDNJS

Would be very useful to have leaflet draw on cdnjs, thanks.

Leaflet draw is now automatically available on cdnjs.

can someone give an example of a map with draw using cdnjs?

actually I was able to do it by updating an example by d3noob
http://bl.ocks.org/d3netxer/0067027199f1d618680e7d7a3f1b754f


[mobile]make location header lookup stricter

Commit Message: make location header lookup stricter
Additional Description:
Risk Level: Low
Testing: existing tests
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: android only

/assign @abeyad

is there a way we can write a test for this?

I modified the existing test to cover this situation.

@abeyad does this LGTY?


Make compatible with Python 3

This module isn't compatible with Python 3 because Python 3 does not support calling .encode() and .decode() on strings.
However both Python 2 and Python 3 support the codecs module to encode and decode hex strings.

Hey, thanks for this! I haven't had a chance to run the tests yet, but I looked at the changes - I see you also changed some parts to use math.log() instead of bitwise operations.  Is that also for python3 compatibility?


Feature Request: Should support IDP initiated SLO.

Would be cool if this IDP also supported IDP initiated SLO.  Would need to provide a hook so that when a user logs out of the IDP it sends the request to all the authenticated SPs

Like this you mean? https://github.com/sportngin/saml_idp/pull/41 or am I missing something? If so can you link to some documentation for anyone who glances at this and decides to pursue it?

#41 provides a lot of the functionality for SLO.  What i was looking for is a means to initiate a logout from the IDP itself.  Perhaps providing a means in the controller to initiate logout from the IDP side rather then waiting for a logout call from the SP.  It may already be in there and may just need documentation but looking through the code i didn't see anything obvious, but i could have missed it.

@Yanchek99 after looking at the docs, does this gem support IDP initiated logout?
I want to implement such thing (IDP initiates a logout that is then handled by SP so that in the end we are logged out both from IDP and SP) in my app but can't find much documentation about how to approach this using this gem

@beniutek from the research I have done on this gem, it does not support IDP initiated logout.

@beniutek From what I have found in using this gem, it does not currently support IDP initiated logout.

thanks for the reply. I'll be implementing that on my own then

@beniutek did you create an implementation? Could you give any pointers on how to add this functionality?


Can I buy you a coffee?

This is godsend. Can I buy you a coffee?

Yeah sure! 🤩
Here is the link: https://github.com/sponsors/PJijin


I am all-powerful

at least until the bugs get fixed.

@ags131 I promise no more PRs after this one.

just checking if you can merge a negative-scored PR.


Remove "message" from json by status=pass

According to
https://github.com/exercism/docs/blob/main/building/tooling/test-runners/interface.md#message
at "status":"pass", message should be null or omitted. We omit the message.
Fix #151

Note: When this gets merged I will need to update tests in test runner, see
https://github.com/exercism/fortran-test-runner/pull/7


Interactive SpyQL

This is v0 for interactive spyql. It implements the following new things:

Q class and q function in interactive.py: This is the class/function that the user can add in their script
InteractiveProcessor in processor.py: row yielding loaded by the Processor
InteractiveWriter in processor.py: writer that adds the output in a list

Other Important changes:

init_vars, go and _go now have user_query_vars as a valid input
Processor can now be explicitly defined as interactive

Misc:

ifs changed to elifs where needed

Issue: #64
To test this python tests/interactive_test.py.

Thank you so much for contributing @yashbonde !
I think what you wrote is great. I have several suggestions that I will include when reviewing the PR but in the meanwhile I would like you to think in the following things we might want to do:

allow querying files and not just data structures. e.g. we might want to read a very large file, do some aggregations that would result in a smaller output, and return the result (otherwise, we would need to load the large file into an in-memory datastructure)
allow writing the result to a file, and not necessarily limit the output to a data structure (I haven’t a strong option about this one)
support multiple data structures types on input: list of dicts, pandas, …, or at least offer ways to convert commonly used data structure to a standard one that spyql can handle
have different output data structure types or return an object with functions to convert to other formats
have a writer which is a lambda function, letting to the lambda function deal with the outputs rows, one by one

I am not saying that a first approach to interactive spyql should cover all these, but at least we should be mindful of decisions we make about these and eventual other relevant topics. Pandas is super popular but I have been avoiding it due to the large number of dependencies...
Let me know what you think!

Hi,
Your points make sense. I would go about it this way:

Supporting multiple sources in the interactive form: This has to be done by expanding the grammar a little bit:
a. when using a python data structure: FROM xxx becomes FROM {xxx} (not the curly braces). We will know immediately what are the python inputs expected at runtime from the user. This will set prs["from"] = "PYTHON"
b. we can add checks in the Q.__call__ that see what datatype the object is, this will determine the Processor to use, ex. PandasProcessor, PolarsProcessor
c. FROM and TO can be written down as (relative) filepaths: FROM ./data.csv TO ./data.json. This will set prs["from"] = "JSON/CSV/TEXT" and can be processed as it currently being processed. Difference being that the if TO is not provided then the information is stored to the memory and returned as python data structure (sounds like a fancy file reader ;-) )
Returning a new structure (SpyOutput) can be a cool strategy of further exporting it wherever the user wants. It can have function to send it anywhere out.to_csv("filepath") or out.to_list(), etc. I think I'll work on it once the core logic mentioned above is working.
Writer as a lambda? I don't get it, but pure functional approach is something I am happy to work with.

Things I'll implement:
→ I am trying to get all the tests in main_test.py working interactively
→ Pandas and Polars in working condition

Writer as a lambda? I don't get it, but pure functional approach is something I am happy to work with.

Vary basic (actually silly) example:
SELECT col1 FROM range(10) TO lambda(row): print(row)
This would allow users to have custom writers very easily... without having to extend the writer and register it (not possible at the moment). But I guess this is for a future iteration if it makes sense.

CLI tests are still failing and I don't know why

This was fun, I think we can make some serious progress in this direction if we are able to support:

Purely functional approach so multi-thread (Interactive and CLI) and multi-core (CLI only) combined can be used reliably (see vmap/pmap in jax). This is completely possible but will require a big refactor to make it completely functional
Figure out how to use multiple files and "JOIN" them.
Support wider I/O with the package in terms of:

Auto replace larger operations with numpy/jax arrays to utilise more compute
dataframes and rely on their internals for many of the operations, will require writing a new DataFrame class that can have a make_dataframe that takes in the string and tries to get it actual Dataframe object.
Networking support ex. SELECT * FROM aws.s3://object can automatically download/stream the files
Connecting with an actual DB (like Sqlite3 or Postgres) ?? (still unsure about this)

Personally I think these are the places where it can be super useful:

Default CRUD app logic can be made really simple by chaining a bunch of spyql queries

CLI tests are still failing and I don't know why

I will check what's happening, might be some dirty detail :-)

I really want this to be used more widely, but the fundamental problem is that Python is inherently slow. So if I have 2GB of data that I have to query again and again it would make sense to load that into a DB.

True. spyql is great if your analysis fits into a single query (or a chain of queries). If you need to repeat different queries over the same (large/big) data, spyql might not be the best solution. In that case, depending on the size of the data, I would jump to either PostgreSQL (and spyql helps here allowing to easily import data into the DB using the TO sql) or Spark-SQL (typically in standalone mode after an initial conversion of the data to parquet).

The strong spot of spyql is:

When dataset size is low (need benchmarking and numbers on this)
When querying is a one time process.

True :-) If the dataset is small or if don't need to do multiple different queries on the same large data.
Here's a simple benchmark for json data. spyql is head to head to jq.

More info in my FOSDEM22 presentation:

Slides
Presentation recording

Personally I think these are the places where it can be super useful:

Default CRUD app logic can be made really simple by chaining a bunch of spyql queries
Please mention if you can come up with more!

I think that CLI can leverage the command-line much better than python alone for data processing. In this context, spyql is great for:

Ad-hoc querying / data analyses
Converting file formats (e.g. CSV, JSON) and importing data to DBs
Setup simple data pipes
Monitoring data streams
Transforming/analysing outputs from shell commands

This was fun,
Yes!! :-D

I think we can make some serious progress in this direction if we are able to support:
Great points you have here.

Purely functional approach so multi-thread (Interactive and CLI) and multi-core (CLI only) combined can be used reliably (see vmap/pmap in jax). This is completely possible but will require a big refactor to make it completely functional

This seems exciting. My main goal now is making spyql feature-rich while having a decent performance. I am trying to understand what could lead to higher adoption rates. Most of the feedback I am getting is about features for making query writing easier like: auto-complete (including column names) and syntax highlighting. So, I am planning to tackle IPython integration, and the work you are doing put us already half-way there. See for instance: https://github.com/catherinedevlin/ipython-sql

Figure out how to use multiple files and "JOIN" them.

For now I would keep this very simple. Load a JSON into memory (as a dict) and then do SELECT my_json[my_foreighn key] FROM .... Basically, this allows for equi-joins using dict lookups.

Support wider I/O with the package in terms of:

Auto replace larger operations with numpy/jax arrays to utilise compute more effectively

This seems a nice entry for the backlog, given that I don't think performance is an adoption barrier right now.

dataframes and rely on their internals for many of the operations, will require writing a new DataFrame class that can have a make_dataframe that takes in the string and tries to get it actual Dataframe object.

Interesting

Networking support ex. SELECT * FROM aws.s3://object can automatically download/stream the files

This is already possible with the CLI using the ASW cli, and piping stdout to spyql. But I agree that this will make it much easier.

Connecting with an actual DB (like Sqlite3 or Postgres) ?? (still unsure about this)

Also possible with the CLI by piping the output of DB clients, but again, handy to be included for interactive use.

Storage of information in some highly optimised format such as Apache Parquet can fix the issues with large dataset size and mmap of pyarrow and numpy and numpy to jax can enable zero-copy, high-performance processing.

This is definitely something to consider on the mid-term.

If we go by the logic that we keep spyql as the Front end and build plugins that can be used with other systems (pull from S3 process data on GPUs and return a dict object).

This is very interesting... definitely something to explore, especially if anyone could contribute with plugins!

Note sure if these pointers are useful, but I learnt a lot hacking this. It's super cool 👍

Super useful and insightful! I am super happy with both what you are doing and also hearing your perspective!
Thank you!!

@dcmoura go for it! I have pushed all my changes in yashbonde:master. You can merge it into a feature branch as it would make more sense.

Piping through CLI for AWS etc. makes way more sense than adding them here unless someone really requests it.

I was not aware of the Performance metrics but they are very good and your talk demonstrated far more than I had in mind. I work with data pretty rarely, mostly it's building a new a ML pipeline (apologies for the shameless plug 😛) and so I was a bit concerned about the metrics as I don't have experience in the domain.

I read a bit on LINQ and it is a very fascinating idea. Ability to effectively query complicated data structure from within a language (even if using another language which is a standard) can reduce the amount of time and code required.

... � So, I am planning to tackle IPython integration, and the work you are doing put us already half-way there.

I actually was looking for something in notebooks then landed back here. I may have put you half way there but you came up with the whole thing so you deserve far more credits. I'll take a look at notebook thing as well!
Best,
Yash

Thank you @yashbonde :-)
LINQ in C# is really cool, as it seems to be handled at compile-time (and not a query in a string evaluated at runtime).
I also noticed how they changed the order of the clauses to match the execution order (an option that I am also wondering if should be implemented).
About Jupyter plugins for SQL (and actually much more) you can also take a look to this other project: https://github.com/fugue-project/fugue
I'll ping you when I am done or if I have any question!
Thanks,
Daniel

Hi @yashbonde . Yesterday I have managed to finishing fixing the code and do some very much needed refactoring to eliminate code/logic replication. The code was breaking on lots of places and took me some hours to fix everything (I had to revert several changes).
Next time, please interrupt coding when you detect that tests are not passing. Every time I change something I run make test to quickly check if all tests are passing and run black to format the code before committing.
Your contribution was important and timely but please reach out to me when tests start breaking next time, OK?
I am taking the opportunity to make further changes/refactorings, so I will take a bit longer to submit the PR, but if you are interested you can check the changes I have made here: https://github.com/dcmoura/spyql/compare/interactive
Thanks

Got it, will take care of it from next time!

Hi @yashbonde , the PR is finally out!
https://github.com/dcmoura/spyql/pull/68
Feel free to provide feedback. Thank you for your help!


build: workaround older CMake limitations

Add a workaround for older CMake which would not emit the import library
into the correct location.

@swift-ci please smoke test

@swift-ci test


Always use the latest stable Rust version on Travis

We're pinning to that version intentionally to make sure we're aware when we drop support for that version of Rust. The nightly and beta tests should handle tests against more modern versions.

Ah, I see...

I realized it's probably best to have a consistent snapshot of both values so I modified this just a bit. Publishing now: https://github.com/sfackler/r2d2/commit/731b389a05b90e235de3dcd6cd155853fd062b10

Indeed, having a consistent snapshot is the best way to go. Thanks! Just curious though, what is the purpose of the _p?

We can make all of the other fields public. The private field allows us to add new public fields in the future without breaking backwards compatibility.

Neat! I didn't know this trick. I'm glad I asked :)


Pin version of VM2 to version w/o vulnerability

Upgrades version of VM2 to a version that fixes the 'Sandbreak' vulnerability: https://www.darkreading.com/application-security/critical-open-source-vm2-sandbox-escape-bug-affects-millions

Hm, nice catch but this might be worth thinking 🤔 about an alternative solution: Turning this into an actual Juice Shop challenge. Not sure if that's possible in safe way (maybe double-sandboxing or something) but just fixing might be a missed opportunity in our very special Juice Shop case.

Ah, sorry, I didn't see this was the Chat Bot repo actually. Here it already is intended to be vulnerable, but I'm not sure what underlying vuln we really use.

@bkimminich Just unsure of the process to get this live: I see that juice-shop's package.json is pulling in the latest patch for juicy-chat-bot which will include this, but does there need to be a release on juice-shop before this change is reflected in docker images?


Add to npm registery

We should be in this list https://www.npmjs.com/search?q=jquery-

Done!
https://www.npmjs.com/package/jquery-ziptastic


No prebuilt wheels for v1.2.0 on pypi?

Hello, was it intentional to stop publishing prebuilt wheels to pypi? Just wondering, because the instructions still say the wheel packages include pandoc for Windows and Mac OS X, but the wheels are not listed under the files section on pypi. https://pypi.python.org/pypi/pypandoc/1.2.0
When using the latest version 1.2.0 my builds/checks fail with the pandoc not found error...using download_pandoc() or reverting to v1.1.3 gets them working again.

The problem is simply that I don't have a working package builder right now.
Apart from that, the new release makes it easier to download the required pandoc binary via an included API method. I tried to keep the language in the README as "neutral" as possible (e.g. if I get my package builder running again, I think it would be nice to build the wheels with pandoc binaries again). If you hve any suggestions how I could make it better, please comment here or file a PR :-)

Ahh, ok. I do think it is rather convenient to have pandoc in the wheels, especially when testing builds and packaging on Windows using appveyor. What were you using for your builds before? Can you point me towards what was, and now is not working?

https://github.com/janschulz/package-builder/tree/master/recipes/pypandoc
That repo builds conda packages and I hacked in the wheel building in the build scripts for the pypandoc conda package. But now the conda packages are build on conda-forge and not in my private channel). I could probably resurrect the pypandoc recipe and remove the rest...
If you are using conda (with the conda-forge channel) on appveyor, then you can install pandoc from there (conda is installed on appveyor, you just need to add the right path to PATH). The other way is to pip install pypandoc and then just do the oneliner to install pandoc (on windows you don't even need to set path, pypandoc searches the default path).

Hmm, no, I haven't used conda. Does conda also push packages to pypi? If the pypi upload is already happening separately, perhaps the script building the wheel could be executed by travis ci, using the deploy feature, when a version tag is pushed to the master branch?

no, not by itself. conda is an alternative package manager (nice because it uses precompiled packages, so you can install numpy without setting up a compiler first) and conda-forge has a infrastructure (or better scripts which let you build infrastructure) to build conda packages on CI services (Win, Mac, Linux). I just hacked in the wheel building and upload (via twine).

Since you support download_pandoc in setup.py, it looks like something as simple as this might work, in .travis.yml?
deploy:
  provider: pypi
  user: ...
  password: ...
  distributions: sdist download_pandoc bdist_wheel
  on:
    tags: true
    branch: master

@lorengordon Did you try the .travis.yml change that you are suggesting?  I need to ensure support of Win and Mac users.  I too am still using 1.1.3 until a work-around is found.

I have not yet. I was just figuring out a couple days ago how to incorporate the new download_pandoc() function in my projects to workaround it.
I suppose I could mirror this project, update travis.yml there, enable travis in my account, and try deploying to pypi under another name. Doubt I'll get to it for at least a couple weeks, but I'll report back here if it works.

@lorengordon I don't mind adding you as a contributor to the project if you want avoid mirroring. Maybe less hassle.

Oh, it's no hassle. I just don't want to accidentally deploy a broken package to pypi under this project. I haven't really worked with travis's deploy feature yet, so I'd like to work separately while I figure out how it works. If I get it working, I'll be happy to submit the PR here. :)

Sure, that will work. :+1:

I wonder if a service like pandoc-extras/pandoc-portable would help embedding the binaries. Basically it provides a portable version, that upon unzip you can grab the pandoc and pandoc-citeproc executables.
I didn't finalize the folder structure yet, and @jgm seems to be hesitant on a 3rd parties distribution of binaries like this. But if there's a need, there will be a way for this to work out (we can even provide the .zip in the official jgm/pandoc).
Alternatively, we can stick to pandoc_download only and deprecate this feature.

To get this working, we would need to add macosx tests via travis. I haven't yet looked deeply how to convert the travis setup to run builds on both linux and macosx, but it should be possible. If not, there is always a Circle CI which could take over the linux/source builds.

I have a quick skim on the travis setup in pypandoc, it seems that it is doable on Travis' osx build. e.g. see [Testing Your Project on Multiple Operating Systems - Travis CI](https://docs.travis-ci.com/user/multi-os/#Python-example-(unsupported-languages)) on how they splits the tox over different builds.
From my experience, it seems the most tedious part is to get the LaTeX build environment working on it, basically because MacTeX is too large to be build in Travis, BasicTeX is too small to include the packages you need, then you need to install those by tlmgr.

On Windows the following works for me. Since 1.1.3 will install pandoc I install that version first.  Then I upgrade pypandoc.  It is a bit of a hack.
pip install pypandoc==1.1.3
pip install pypandoc --upgrade

@nueverest Note that there is now a "API" way to install pandoc after installign pypandoc: pypandoc.download_pandoc() will usually do the right thing and make pandoc available to pypandoc...

so no wheel support for win/osx anymore ? the README still says there are ?

This might be a way to set up a autobuilder for wheels: https://github.com/joerick/cibuildwheel If someone wants to have a go at that, feel free to send PRs. I'm currently a bit swamped to do the CI dance for that :-/

https://pypi.org/project/py-pandoc/ seems to be a better way to handle this.


Graph rbac api get manager

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] The PR targets the latest branch.
[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] Apache v2 license headers are included in each file.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: UdiRaitblat sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Can one of the admins verify this patch?


core(TranslateService): load translations if they have not been loaded before (#1193)(#1266)

When the use(...) method is called nothing happens if the current language is the same as the one passed as parameter. This causes that after changing the language in one instance of the translate service the rest of instances don't fetch the translations if they don't use isolate = true since the current language is the same as the one passed as parameter.  That is why workarounds like this
translateService.currentLang = '';
translateService.use ('en');

worked. Removing the first return statement leaves the service check if it has the translations and gets them if they have not been loaded before. It fixes (#1193)(#1266) between other issues I am not able to find now.

also fixes #602 #444  #425

Sad that library seems to be abandoned


An ability to link to a file and line and share that link with other person working on that project?

My and my coworker are working on a project together.
He asks me where is xyz coming from. I investigate, and find the place. I should be able to right click and do copy link to the line and send him the link, which when he clicks opens up the file and highlights the line. Sort of like github similar feature where you can link to file number in a file. Assuming he is working on a same project as well.
I'm fine if it requires specifying some field jsconfig or such.

Check out some extensions like Open in Github, and GitHub PR extension which shuold have copy line location
fyi @alexr00

Ok I'll it a go

this is actually pretty integral part of workflow. being able to link to exact file in a project. It'd require all parties to have an extension installed which is unlikely.

Checkout live share.
We will not make this a part of vscode code. VS Code tries to be lean and minimastic.


GO program to convert celsius to fahrenheit

Why:

Closes #2287
What's being changed:
I have added a method to convert C to F in Go. I also wrote a simple main method to see it's example usage.

@Ladam0203,
You are pushing changes from the master branch. Please create a separated branch and push your changes! You can refer steps in make changes.
Thank you!


adr-log -d . does not output output log to stdout

When reading the documentation, I expect that adr-log -d .  outputs the log to stdout. Currently, it prints the help.

Fixed in version 1.0.1


[Bug]: Do not show green notification during LNURL withdraw

Describe the bug
When trying to withdraw from LNmarkets you have green notification which is confusing.
We need to change it to yellow and change the text, mention that payment is still in process and we are waiting for it.
Reproduce

Go to lnmarkets withdraw
withdraw using LNURL-withdraw
See Success notification but not the confetty

Screenshots / Recording
No response
Operating system
iOS, Android
Bitkit version
No response
Log output
No response

This is the current notification

Verified in release-v1.0.7 f0cde24


Update LOGO_LICENSE.md with license cleanly integrated into text and copyright symbol added

This change integrates the license into the text and adds the copyright symbol.

@Riteo what about renaming LICENSE.txt to LICENSE.md like a lot of other repositories?

Thanks for the contribution. I agree with some of the changes, but not all. Indeed changing it to a plain text file would make sense.
I went ahead and made my own update of the docs: #83095.
In general, we advise against new contributors making minor changes to repository documentation like this. This requires a lot of effort from maintainers to help shape the contribution so that it fits our PR workflow and style guides, for very minor changes.


Feature request: count badges on instruments channel sidebar

I would love to have object count badges on instrument bar icons like:

number of users online in selected channel
number of starred messages here
number of attachments
number of pinned messages
Surely not all badges will really be useful and it may have sense to let user or admin decide which ones to have displayed.

@OrdinaryGuy Please could you reopen this request on the main Rocket.Chat app, as this is a feature that will be implemented there, rather than just for the desktop app.
https://github.com/RocketChat/Rocket.Chat


Re-enable --turbo in Next.js pages example

--turbo was disabled in the Next.js pages examples in ce18565 (#410), due to https://github.com/vercel/next.js/issues/70262. After the fix lands on Next.js 14, we'll be able to enable --turbo back.

The disabled --turbo ended up not going to canary because the fix was already deployed in a new Next.js version at that point.


Customer name not sent to nets

Hi
We have a Vendr installation with version 2.0 of this provider.
It works fine but customer name is not shown in Nets.
Do we need to do something special to make this happen ?

So I need to fill out these fields ?

I did look at the other issue, but wasn't sure what I need to do in order to make it work.

Yes, otherwise it doesn't set shipping address and I believe it's a requirement from Nets to store consumer on the payment.
https://github.com/vendrcontrib/vendr-payment-provider-nets/blob/v2/dev/src/Vendr.Contrib.PaymentProviders.Nets/Easy/NetsEasyOneTimePaymentProvider.cs#L275-L288
Hopefully in future versions of Vendr, we get a bit more control of properties, e.g. tabs, groups, make specific properties mandatory and maybe even decide the property editor to use @mattbrailsford

HI Bjarne
I tried setting the properties but ended up with this error:

All the fields have been set, and I have ensured that there are a value in all of those properties.

I can see others have had this error but I am unsure what the solution was.
Hope you can help me :)

Does phone property contains a valid format, e.g. "+4512345678"? Currently it split the first 3 characters to use as prefix and the ones as number.
Phone isn't required but when set is must both contain a value prefix and number.
I also think Terms URL is mandatory.
Does the log show a more specific error?

Terms url has been set.
I removed phone number and now I goes to the payment page.
Thanks for your help.

Great.. I wonder if which format the phone number was specified?

It is a danish only page so just 8 digits :)

For example it works with "+45", but with "45" the request fails.

Strange enough Nets shows "001" in the example here, but it fails. Also "045" doesn't seem to be valid.
https://developers.nets.eu/nets-easy/en-EU/api/payment-v1/#v1-payments-post

Not the first time Nets has bad description or badly updated documentation.
My client just wanted the customer name not so much phone and all that :)

Yeah, the documentation has been updated, but still there are various parts which isn't fully updated.
We could probably adjust the code to add a + in case phone property value doesn't start with a plus.


Finished Reactime v22.0.0

🎉🎉Finished Reactime v22.0.0!! 🎉🎉
https://medium.com/@kelvinmirhan/reactime-real-time-debugging-timeless-results-3f163b721d01
Co-authored-by: AndrewByun<EMAIL_ADDRESS>Co-authored-by: jimmally<EMAIL_ADDRESS>Co-authored-by: kelvinmirhan<EMAIL_ADDRESS>

LETS GO!!!!!


Edge Chromium App mode

I've seen this issue already #251 which is from 2020, and Im asking my self, ¿Will you add that feature to the versions to come? Because it will be pretty handy to be able to use Edge in App mode with a fresh install of the package.

Looks like this is a dead project. There are so many open pull request.
I was also looking for it. Then I forked it and created a code change to support app mode in edge.
I created a pull request and waiting for it's merge https://github.com/python-eel/Eel/pull/744 , however,  until this get merged , you can use my forked repo. https://github.com/yehigo/Eel

@Meenapintu Yes unfortunately none of the previous maintainers have capacity to do so any more. Do you want to maintain the main repo?

@ChrisKnott just to throw my head in the ring here, I'd be happy to contribute to contribute toward maintaining eel if that helps.


mat-icon does not render in currently selected mat-select > mat-option

Bug, feature request, or proposal:
Bug
What is the expected behavior?
To render the selected mat-icon in a mat-select component
What is the current behavior?
mat-icon does not render when an option is selected in a mat-select but does render in the mat-option
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz: https://stackblitz.com/edit/angular-select-option-matprefix-icons-issue?file=app/form-field-overview-example.html
What is the use-case or motivation for changing an existing behavior?
To visually see the selected mat-icon in a mat-select
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular v6.1.3
Angular material v6.4.5
Angular CDK v6.4.5

That's because the select only looks at the text value for an input to determine what to show in the trigger. You can customize the trigger text yourself with an icon using the mat-select-trigger. There's an example of how you can do it in the docs.

It is a bug. mat-select-trigger only solve some problem. But if I want to use i18n to translate what is in the mat-select-trigger what should I do?
It is easy to translate the innerText in the mat-option and use the default behavior of mat-select to display the text. But when it comes to mat-select-trigger, because there is variable in this tag, how should I translate this then?

I, i solve this problem with the next code:
into html
`
Libellé
<mat-select-trigger>
    <mat-icon>{{ getSelectedValue() }}</mat-icon>
</mat-select-trigger>

<mat-select #selectIcon class="mat-select-icon" [(value)]="selectedIconItem">
    <mat-option [value]="0">
        <mat-icon>text_fields</mat-icon>
    </mat-option>
    <mat-option [value]="1">
        <mat-icon>title</mat-icon>
    </mat-option>
    <mat-option [value]="2">
        <mat-icon>format_bold</mat-icon>
    </mat-option>
</mat-select>

`
into ts
@ViewChild('selectIcon') selectIcon; getSelectedValue() {     if (this.selectIcon && this.selectIcon.options && this.selectIcon.options.length > 0) {         return this.selectIcon.options.find(             item => {                 return item.value === this.selectIcon.value;             }         ).getLabel();     }     return ''; }
if this can help someone.


Add type definitions for the client

Given the popularity of TypeScript, it would be good to add type definitions.

Added in bullet-train-client and react-native-bullet-train 0.2.3, enjoy!

I'm getting a build failure when using TS as-of version 0.2.3 in Angular.
7016: Could not find a declaration file for module 'bullet-train-client'. '/Users/crutchcorn/git/Coopervision/CVExtreme/node_modules/bullet-train-client/index.js' implicitly has an 'any' type.
  Try `npm install @types/bullet-train-client` if it exists or add a new declaration (.d.ts) file containing `declare module 'bullet-train-client';`

2 import bulletTrain from 'bullet-train-client';
                          ~~~~~~~~~~~~~~~~~~~~~

However, renaming the bullet-train-client.d.ts file to index.d.ts in node_modules fixes this for me

Another fix for this would be to add the following to package.json:
  "types": "./bullet-train-client.d.ts"

However, I think that the isomorphic import would also break. I would suggest moving both d.ts files into the same index.d.ts file (since they're namespaced, it should work properly) and then adding it to the types field in package.json

Ah strange, but I totally see what you mean about combining. I think ideally I'd look to also have some automated webpack approach for this since react-native, isomorphic AND the general lib have the same types.
If you're offering to do that quickly @crutchcorn that would definitely be appreciated!

Yup! It's somewhat blocking stuff at work, so let me see if I can't just fix this issue quickly :) Forking now!

Thanks, apologies about that! It's strange because running the client locally command clicking jumps to the .d.ts instead of the lib code which signifies it's working.
Regardless, I'll keep an eye out for this, do some quick sanity checks and publish a version as soon as I see it!

Right! Very weird, the differences between the TS language provider (for IDEs) and the compiler 🤷♂
Either way #31 has been opened to help resolve this

Also, instead of having a webpack fix, what I might suggest is doing something like:
shared_types.ts:
export interface Test {
    example: string;
}

Then, in each of the instances you need those types:
export * from './shared_types;

That way, you can have a single source-of-truth without having to deal with webpack copy messiness

Version @0.2.4 released, thank you again!
Will close for now, but reopen if you come across anything TS related.


Add a little warning to the UI if a non-standard license is used

Users may define any text for the license field via the metadata of an Rmd file. The UI should show a warning, e.g., a orange frame around the license field and a text below it, if a non-standardised license is used.
Related to #93

This is my current approach:


MSBuild version comparisons

Docs around comparing versions in MSBuild. Includes the old way (silently in conditions using System.Version.TryParse) and the new (explicit property function that behaves more like people expect).
Fixes #6127.

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

Hi @rainersigwald - Would you like to respond to the queries in the pull request?
Thanks.

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

@ghogen - Hello! Can you please triage and work to close this? Thanks!


fix(types): isConnected / isInternetReachable should be boolean not true

Overview

This is a small fix that I identified when I was trying to mock the library at my jest + typescript tests.
All checks still working as expected.
Hope it can be useful for everyone.
Test Plan

:tada: This PR is included in version 11.3.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@mikehardy or @matt-oakes, maybe I'm missing something here but were the previous types not used for type guards? i.e isConnected should only be true when there is an internet connection conversely isConnected: false isInternetReachable: false are the only possible states when not connected to the internet?

Connection and reachability are distinct, the captive portal or VPN with routing protection will be connected but not reachable. Is that what you meant?
I don't think you can be disconnected and reachable but there are three states not 2

Thanks for the reply @mikehardy, and no what I meant was is the type change in this PR correct?
My understanding is the previous types (before this change) were setup to provide type guarding to consumers of this package. There are two distinct NetInfo States, thos being NetInfoDisconnectedStates and  NetInfoConnectedStates these are combined as into the NetInfoState as NetInfoState = NetInfoDisconnectedStates | NetInfoConnectedStates.
If the users device is in the disconnected state then isConnected and isInternetReachable keys can only ever have values of false and thus the previous type (before the change in this PR) reflected this, it being:
interface NetInfoDisconnectedState<T extends NetInfoStateType> {
  ...
  isConnected: false;
  isInternetReachable: false;
  ...
}

the change in this PR changes this to be:
interface NetInfoDisconnectedState<T extends NetInfoStateType> {
  ...
  isConnected: boolean;
  isInternetReachable: boolean;
  ...
}

But this isn't a true reflection of the possible states for when a device is disconnected because isConnected and isInternetReachable can't ever be true when in a disconnected state.
The same is the case for NetInfoConnectedStates where isConnected can only ever be true and thus it's type reflected that.

@JDMathew Thanks for your arguments and the explanation, and it makes sense, but for that, we need to assume that the isConnected should be the same value as the isInternetReachable, but it is not valid because, as @mikehardy said, you can be connected but not reachable, then one of them can be in a different state.

@JDMathew Thanks for your arguments and the explanation, and it makes sense, but for that, we need to assume that the isConnected should be the same value as the isInternetReachable, but it is not valid because, as @mikehardy said, you can be connected but not reachable, then one of them can be in a different state.

I think we're missing each other here. I agree that "you can be connected but not reachable" (connected state), I'm arguing that "you can't be disconnected and reachable" (disconnected state). However the new types in this PR suggest that you can be disconnected and reachable.
The three viable states are:
When disconnected:
  isConnected: false,
  isInternetReachable: false,

OR
When connected but internet is not reachable
  isConnected: true,
  isInternetReachable: false,

OR
When connected and reachable
  isConnected: true,
  isInternetReachable: true,

The only state never possible is (not connected but internet is reachable:
  isConnected: false,
  isInternetReachable: true,

These three viable states can be broken into two types, those being when the user is connected or when the user is disconnected (represented by isConnected: true or isConnected: false). These states are therefore in the type form:
interface NetInfoDisconnectedState<T extends NetInfoStateType> {
  ...
  isConnected: false;
  isInternetReachable: false;
  ...
}

interface NetInfoConnectedState<T extends NetInfoStateType> {
  ...
  isConnected: true;
  isInternetReachable: boolean;
  ...
}

This is a more true or "strict" type than the updated types in this PR as it represents the only possible connected states through type narrowing rather than a generic boolean.

I totally agree with @JDMathew, it is also my understanding that the usage of true and false was intentional, in order for TypeScript to allow to infer proper set of types when doing this:
const state: NetInfoState = ...:

if (state.connected) {
  ... // By using `true` and `false` in the types, TS can properly infer here which type elements are available or not (since NetInfoState is a discriminating union type in that case via the `isConnected`).
}

The last suggestion by @JDMathew above seems to be the more correct one.

@mikehardy or @matt-oakes, appologies for pushing this so hard but I really do believe this is a regression. Please see above:

I totally agree with @JDMathew, it is also my understanding that the usage of true and false was intentional, in order for TypeScript to allow to infer proper set of types when doing this:
const state: NetInfoState = ...;

if (state.isConnected) {
  // By using `true` and `false` in the types, TS can properly infer here which type elements are available or
  // not since NetInfoState is a discriminating union type (in this case via the `isConnected` element).
}

The last suggestion by @JDMathew above seems to be the more correct one.


JSON language server error not showing error stack

VSCode Version: 1.44.0
OS Version: Darwin x64 19.4.0

Steps to Reproduce:

Don't know where this error is comming from since I can't check the stacktrace.
Open project with error
if i close code completly then reload code, this error is not showing.

Does this issue occur when all extensions are disabled?: No

Can you add a screenshot?


Flash attn

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

Please state [minor] in your commit message next time if it's a small problem :)


Fixes 2308 - Added service principal warning message

PR Summary
Fixes #2308 - Added service principal warning message
PR Checklist

[x] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[x] Summary: This PR's summary describes the scope and intent of the change.
[x] Contributor's Guide: I have read the contributors guide.
[x] Style: This PR adheres to the style guide.

Learn Build status updates of commit 6d10742:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-8.3.0/authenticate-azureps.md
:white_check_mark:Succeeded
View (azps-8.3.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Arnold trace set support

This adds support for specifying Arnold traces sets using standard gaffer sets with a "render:" prefix on them. The implementation is intended to one day support set membership in other renderers (for instance RenderMan's "grouping:membership" attribute), but only once they are updated to use the new Renderer API.
A couple of things to note :

The second commit isn't strictly necessary for this PR. I added it thinking I could use it in RenderSets but turned out to need something more specialised. I've left it in because I can imagine it being useful, especially from Python.
The last commit should be squashed with the previous one before merging, but I've left it on its own because I think it's worthy of discussion. Arnold appears to treat objects which have not been placed in any trace sets as if they are in all trace sets, which I find really unhelpful. For instance, you can start with an object in no sets at all, and use a shader tracing the "foo" set, and the object will be visible. Then you put the object in the "bar" set and suddenly it disappears, even though its membership of "foo" is unchanged from before. I can't see how this is helpful in any way, but it's documented here so doesn't seem to be considered a bug. Although I generally advocate for the renderer backends to present renderers in their default behaviour as much as possible, I really think this is one case where we might want to make a correction. Any opinions?

This seems like a case where we absolutely want to modify the default behaviour.  The way the docs word it doesn't sound so much like they think this is desired behaviour, as some sort of backwards compatibility or something.  I guess if we wanted to double check, we could take a look at what Mtoa does?  The docs kinda sound like they're suggesting outputting a null set on objects with no other sets?

Yeah, it's a case of replacing trace_sets == [] with trace_sets == [ "somethingNobodyWillEverWantToUse"]. Those docs suggest "", but I was wondering about "none" - any preference?

No preference here. A blank string or "none" sounds good, I guess the word none makes sense if you don't want to get confused using Python to test if the trace set is a completely empty list or that the first member string itself has nothing in it.

Doesn't matter much to me.  I would tend to use all-caps and/or underscores to emphasize the uniqueness of this name?  "NONE"?  But that's just a naming convention thing, up to you.

I think the GitHub Markdown screwed the formatting of my first attempt (I've edited it since). I was suggesting using underscores to make a value of "__none__" - sound OK?

Sure, sounds good.

OK, I've rebased that change in.


Add new debug icon

Issue

None

Overview (Required)

I made new debug icon

Links

#349

Screenshot

release
debug

LGTM! 💯 Good work 👍

@u-nation thx! :)


[Inspector] Broken in master

@jaredly - clicking on any View appears to have no effect

(also, very minor but the text "Tap something to inspect it" on the overlay should probably be white)

cc @a2, maybe related to view hierarchy flattening?

If we can get this fix in before 0.8.0-rc release which we hope to do tomorrow, that would be ideal @jaredly

cc @frantic

cc @shayne

@vjeux looks like the objc side is messed up -- I'll look into it

@vjeux now fixed internally

Could you guys sync the fix shortly? We can always include it with rc.2 but would be nice to include it with the rc since the issue's already been fixed.

yeah I'll get on it

thanks @jaredly - waiting on this then we will ship the 0.8.0-rc release :heart_eyes:

Thanks @jaredly!

@jaredly - fantastic, RC is ready to go =D


[Feature Request] Cli command: Import

It would be great if there was the ability to import collections via the command.  Specifically importing an OpenAPI spec via the cli could be super useful.
For example, something like this:
$ bru import http://example.com/api/spec.json

I also think this would be very useful.
I had a bit of a look at the code. Right now it looks like the importing logic is coupled with the app. I would like to move that somewhere more general and make it available over the CLI.
Is it ok if I start working on this?


Support existing VCN in other compartment

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Support the case that you want to use an existing VCN - which is in a compartment different from the one where you wat to create the cluster
It almost works .... by specifying a vcn_id , but
9:   vcn_id       = var.create_vcn == true ? module.vcn[0].vcn_id : coalesce(var.vcn_id, data.oci_core_vcns.vcns[0].virtual_networks[0].id)
│     ├────────────────
│     │ data.oci_core_vcns.vcns[0].virtual_networks is empty list of object

Right, that's because the data source is looking the VCN up in the same compartment: https://github.com/oracle-terraform-modules/terraform-oci-oke/blob/main/datasource.tf#L7
Can you please elaborate on your use case?

Hi, yes I actually do not need the datasource at all - Im happy to specify  var.vcn_id
Changing it to
vcn_id       = var.create_vcn == true ? module.vcn[0].vcn_id : coalesce(var.vcn_id, try(data.oci_core_vcns.vcns[0].virtual_networks[0].id,""))
for example would solve my issue :) . MAybe as well for the next two lines on the routes ?
The underlying use case is that the existing VCN is centrally maintained in a central network compartment

Ah I see. Looks like you've figured it out already :)
Can you please submit a PR?

https://github.com/oracle-terraform-modules/terraform-oci-oke/pull/564


how to deal with NA in memory correction

@thomas-muench
The picarro measurement data can contain NA values, e.g.:

Line
Inj Nr
Identifier1
d(18_16)Mean
...

51
1
A
1

52
2
A
1

53
3
A
1

54
1
B
NA

55
2
B
NA

56
3
B
NA

57
1
C
2

58
2
C
2

59
3
C
2

We don't have access to a true value for B when calcuating the memory corrected values for C. How should the memory corrected values for C be calculated?

This is indeed a major problem. If only single injections for one sample are NA, then memory coefficients and the approximated true value for the previous sample can be still calculated using na.rm = TRUE - as already done. But when all injections for this sample are NA, as you illustrate, there is no way of calculating the true value for this sample.
I guess the question is then why and how the measurement of this sample failed:
If, for any reason, no water was injected at all into the device cavity for this sample, then one could argue that the true previous value should be obtained from the last successfull sample before, since water of that sample was last in the cavity.
If water was injected but the entire sample measurement failed for other reasons, I am afraid that then we do not have any means to do the correction. All we can do is to use the uncorrected value for approximating the true previous value, and maybe issue a warning for this sample.
Single injection failure is relatively frequent, whereas I cannot recall if I ever encountered entire sample failure. I would suggest to ask Mikaela (and/or Alexandra) how often they experience this issue and if it is related to total injection failure or to other reasons.

Most often measurements are NA because no water was injected at all. Therefore it makes sense to use the last known delta true for the memory correction of the next sample.

Most often measurements are NA because no water was injected at all. Therefore it makes sense to use the last known delta true for the memory correction of the next sample.

This decision was implemented in EarthSystemDiagnostics/piccr#29; closing this issue.


chore: bump testcontainer version

Issue
N/A
Description
Update testcontainer lib to have access to the latest kafkacontainer classes.
https://java.testcontainers.org/modules/kafka/

📚  View the storybook of this branch here

Need more reflexion


Create forloop to apply finite differential operator

this is just a small task to get a forloop to apply the finite differential operator to a general grid of values

the loops have been writted for the first and second differential, although one needs to have a change in variable. See commit 551cda0


Remove SearchMsmarco

Since we're in the middle of a Lucene 8 -> 9 transition, this is a good time to get rid of deprecated code.

Codecov Report

Merging #1962 (42562e3) into master (0fc578a) will increase coverage by 0.55%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1962      +/-   ##
============================================
+ Coverage     60.14%   60.70%   +0.55%
  Complexity     1061     1061
============================================
  Files           178      177       -1
  Lines          9903     9812      -91
  Branches       1371     1359      -12
============================================
  Hits           5956     5956
+ Misses         3441     3350      -91
  Partials        506      506

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


[regression] building and publishing a Dockerifle throws read/write error on closed pipe

While trying to run this:
import { connect } from<EMAIL_ADDRESS>
// initialize Dagger client
connect(async (client) => {

  const contextDir = client.host().directory(".")

  const image = await client.container().build(contextDir).publish("ttl.sh/foo")

  // print output
  console.log("Image pushed to" + image)
}, { LogOutput: process.stderr })

with a simple Dockerfile it fails to run with the following error:
#10 ERROR: failed to push ttl.sh/foo: failed to copy: io: read/write on closed pipe
panic: input:1: container.build.publish failed to solve: failed to push ttl.sh/foo: failed to copy: io: read/write on closed pipe

Seems like a regression.

can't reproduce in curret main . Closing for the moment.


Remove < rule from default

Operator < and > is usually use individually for comparison. Using it as default will be annoying when user just want to input < or <<. This PR remove it from the default preset.

Could you fix tests?

I've read through this tests. It seems like the search.spec, syntax.spec, fast_wrap.spec and jump_next.spec tests are not affected by this PR. I don't know why they all fail.

Approve for GitHub Actions.

Hm... I guess the failing test modifys the editor environment. I'll improve the test infrastructure.

I'll investigate this PR on local environment.

This PR is correct. I'll investigate workflows. Merging.


[SPARK-15472][SQL] Add support for writing in csv, json, text formats in Structured Streaming

What changes were proposed in this pull request?
This patch adds support for writing in csv, json, text formats in Structured Streaming:
1. at a high level, this patch forms the following hierarchy(text as an example):
                <OutputWriter>
                      ↑
             TextOutputWriterBase
                 ↗          ↖
BatchTextOutputWriter   StreamingTextOutputWriter

                    <OutputWriterFactory>
                        ↗          ↖
BatchTextOutputWriterFactory   StreamingOutputWriterFactory
                                              ↑
                              StreamingTextOutputWriterFactory

The StreamingTextOutputWriter and other 'streaming' output writers would write data without using an OutputCommitter. This was the same approach taken by SPARK-14716.
2. to support compression, this patch attaches an extension to the path assigned by FileStreamSink, which is slightly different from SPARK-14716. For example, if we are writing out using the gzip compression and FileStreamSink assigns path ${uuid} to a text writer, then in the end the file written out will be ${uuid}.txt.gz -- so that when we read the file back, we'll correctly interpret it as gzip compressed.
How was this patch tested?
FileStreamSinkSuite is expanded much more to cover the added csv, json, text formats:
test(" csv - unpartitioned data - codecs: none/gzip")
test("json - unpartitioned data - codecs: none/gzip")
test("text - unpartitioned data - codecs: none/gzip")

test(" csv - partitioned data - codecs: none/gzip")
test("json - partitioned data - codecs: none/gzip")
test("text - partitioned data - codecs: none/gzip")

test(" csv - unpartitioned writing and batch reading - codecs: none/gzip")
test("json - unpartitioned writing and batch reading - codecs: none/gzip")
test("text - unpartitioned writing and batch reading - codecs: none/gzip")

test(" csv - partitioned writing and batch reading - codecs: none/gzip")
test("json - partitioned writing and batch reading - codecs: none/gzip")
test("text - partitioned writing and batch reading - codecs: none/gzip")

Test build #60230 has started for PR 13575 at commit bc28f41.

Test build #60230 has finished for PR 13575 at commit bc28f41.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60230/
Test PASSed.

Merged build finished. Test PASSed.

@marmbrus @tdas @zsxwing , would you mind taking a look? Thanks!

@lw-lin sorry we haven't visited this pr in the past, but the goal of this pr will be accomplished by https://issues.apache.org/jira/browse/SPARK-17924 (as a side effect). It reduces a lot of duplicated code as well.

No problem here. Spark-17924 is super great (I've been watching it all the time)  and thank you for bringing that up!

No problem here. Spark-17924 is super great (I've been watching it all the time)  and @rxin thank you for bringing that up! :-D


Error while using "bench --site [site_name] console" in python 3.8

While trying to open the ipython console using bench command the following error is thrown.
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/usr/lib64/python3.8/codeop.py in __call__(self, source, filename, symbol)
    134
    135     def __call__(self, source, filename, symbol):
--> 136         codeob = compile(source, filename, symbol, self.flags, 1)
    137         for feature in _features:
    138             if codeob.co_flags & feature.compiler_flag:

TypeError: required field "type_ignores" missing from Module

Solution is to bump up the version of ipython to >7.3 in the env of the bench. this is know issue in the and is filled here:https://bugs.python.org/issue35894 and here:https://github.com/ipython/ipython/issues/11590.


Simple VOD workflow shouldn't call describeendpoints every time

https://github.com/aws-samples/aws-media-services-simple-vod-workflow/blob/master/7-MediaConvertJobLambda/convert.py
Here, our example shows calling describeendpoints every time. This is problematic since the request rate limit is 1 per second.
The example should have a spot for a cached or manually entered endpoint.

This issue is related: https://github.com/aws-samples/aws-media-services-simple-vod-workflow/issues/38


[FIX] Check for correct response type to prevent undefined sequence_n…

…umber

@pedroslopez I think, that we already fixed the problem #330 with this commit: https://github.com/it-projects-llc/pos-addons/commit/a3a8685143a0e0cdc107a2a57aeedf5899460053 which makes similar thing: prevents to set empty sequence_number value
@GabbasovDinar Am I right?

@yelizariev I commented on the original issue (#330), but I'll quote it here just because it's related

This does fix the issue when it occurs when removing an order, but not when a revision_error is thrown on set_order

@GabbasovDinar can you confirm that? Can there be other responses that need to be handled same way as update_revision_ID?

@yelizariev , @pedroslopez Yes, I confirm this. The do_ms_update function did not take check when the server function returned a "revision error" and this fixes it.
Thanks, good work!

Great to hear!
Can this be back ported to the 9.0 branch? The same issue is present.


Request to have a development guide

Please consider adding the following basic content:

How to set up a development environment?
How to compile this project?
Which branch should we use when creating a PR?
If we have a code style guide?
others

https://docs.primihub.com/docs/core-concept/model
following this doc, you can reimplement the function which can meet your demands
to comiple primihub node, you just using make command and specifiy the platform what you need, such as:
TARGET := //:node
//:cli
//src/primihub/pybind_warpper:linkcontext
//src/primihub/pybind_warpper:opt_paillier_c2py
//:py_main
linux_aarch64:
bazel build --config=linux_aarch64 ${TARGET}
if you find bug during using primihub service, you are highly welcome to create PR to help us fix it, what you just do is fork primihub repo, and then fix the problem based on develop branch, and then create a pull request to our develop, specifiy a reviewer,   if the  CICD is passed, we will merger your PR as soon as possible,  and finally thanks for your contribute.


Move code from componentWillMount to class' constructor

Description
File ChatBot.js still uses componentWillMount() to run some of its logic.
But this lifecycle method is not recommended to be used anymore.
From what I looked into the code, there are no side effects running inside willMount() but wouldn't it be interesting to update the code, so we wouldn't face any warning when developing an app?
Screenshots

Closing because this can be solved by merging Pull Request #50


Support for multioutput model

Is it possible to train a multi-output model similar to the following example? I tried with a custom train_step function and multi output model similar to the one in example but get shape mismatch error.
ValueError: Shape mismatch: The shape of labels (received (<batch size>,)) should equal the shape of logits except for the last dimension (received (<batch size * number of outputs>, <number of classes>)).

@beat-buesser could you please propose a way to resolve this issue?

@beat-buesser could you please propose a way to resolve this issue?

Hi @aqibsaeed Thank you for your interest in ART!
I think theoretically this should be possible, but right now with ART's focus on single output models might require small modifications and extensions, depending on how far you would like to go with the multi-output model.
To get a better understanding of your plans, can you provide more details like the full stack trace and a code example for train_step?

Hi @aqibsaeed Thank you for your interest in ART!
I think theoretically this should be possible, but right now with ART's focus on single output models might require small modifications and extensions, depending on how far you would like to go with the multi-output model.
To get a better understanding of your plans, can you provide more details like the full stack trace and a code example for train_step?

Thank you for the response @beat-buesser. I tried many things but it is not possible currently due to the implementation of TensorFlowV2Classifier (see snippet below line 857). I am trying to train a model following this example  with PGD and AdversarialTrainer but replacing KerasClassifier with  TensorFlowV2Classifier. Here is the example train_step:
https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/99c7efa379d149194cc35c0a564685199f28b4e0/art/estimators/classification/tensorflow.py#L857
loss_object = tf.keras.losses.CategoricalCrossentropy()
def train_step(model, x, y_a, y_b, y_c):
    with tf.GradientTape() as tape:
        pred_a, pred_b, pred_c = model(x, training=True)
        loss_a = loss_object(pred_a, y_a)
        loss_b = loss_object(pred_b, y_b)
        loss_c = loss_object(pred_c, y_c)
        total_loss = loss_a + loss_b + loss_c
    gradients = tape.gradient(total_loss, model.trainable_weights)
    optimizer.apply_gradients(zip(gradients, model.trainable_weights))

Thank you for the response @beat-buesser. I tried many things but it is not possible currently due to the implementation of TensorFlowV2Classifier (see snippet below line 857). I am trying to train a model following this example  with PGD and AdversarialTrainer but replacing KerasClassifier with  TensorFlowV2Classifier. Here is the example train_step:
https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/99c7efa379d149194cc35c0a564685199f28b4e0/art/estimators/classification/tensorflow.py#L857
loss_object = tf.keras.losses.CategoricalCrossentropy()
def train_step(model, x, y_a, y_b, y_c):
    with tf.GradientTape() as tape:
        pred_a, pred_b, pred_c = model(x, training=True)
        loss_a = loss_object(pred_a, y_a)
        loss_b = loss_object(pred_b, y_b)
        loss_c = loss_object(pred_c, y_c)
        total_loss = loss_a + loss_b + loss_c
    gradients = tape.gradient(total_loss, model.trainable_weights)
    optimizer.apply_gradients(zip(gradients, model.trainable_weights))

@aqibsaeed These are modifications that I expected above, but more changes are probably required.  Do you plan to calculate loss gradients ontotal_loss in train_step? Did you also modify AdversarialTrainer and ProjectedGradientDescent accordingly?

@aqibsaeed These are modifications that I expected above, but more changes are probably required.  Do you plan to calculate loss gradients ontotal_loss in train_step? Did you also modify AdversarialTrainer and ProjectedGradientDescent accordingly?

Yes the gradients will be calculated using total_loss. I made changes in AdversarialTrainer but that's not the only thing that need to be changed. The implementation of fit in TensorFlowV2Classifier also needs to be changed accordingly.  https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/99c7efa379d149194cc35c0a564685199f28b4e0/art/estimators/classification/tensorflow.py#L857
That's the reason I opened this issue. Is there any easy way to make and train a multi-output model?

Yes the gradients will be calculated using total_loss. I made changes in AdversarialTrainer but that's not the only thing that need to be changed. The implementation of fit in TensorFlowV2Classifier also needs to be changed accordingly.  https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/99c7efa379d149194cc35c0a564685199f28b4e0/art/estimators/classification/tensorflow.py#L857
That's the reason I opened this issue. Is there any easy way to make and train a multi-output model?

I see. At the moment, these are the changes that I would currently expect to be required for multi-output models since ART focuses on single output models so far. But I find your case an interesting example for us to think about abstracting the data to enable more complex models including multi-output models in the near future.
It looks like you are getting close to make it work for your case. In which line is the ValueError in your first post raised?

I see. At the moment, these are the changes that I would currently expect to be required for multi-output models since ART focuses on single output models so far. But I find your case an interesting example for us to think about abstracting the data to enable more complex models including multi-output models in the near future.
It looks like you are getting close to make it work for your case. In which line is the ValueError in your first post raised?


chore: remove redundant parent arg

the parent argument is redundant for try_append_canonical_chain because the parent is always the canonical fork.

🤖 Generated by Copilot at 9b7b6f4
Refactor and document a function in the blockchain-tree crate. Improve readability and reduce unnecessary arguments.

Codecov Report

Merging #2796 (9b7b6f4) into main (7d36dea) will decrease coverage by 57.82%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main    #2796       +/-   ##
===========================================
- Coverage   71.40%   13.59%   -57.82%
===========================================
  Files         505      505
  Lines       65794    65794
===========================================
- Hits        46982     8945    -38037
- Misses      18812    56849    +38037

Flag
Coverage Δ

integration-tests
13.59% <ø> (-3.89%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/blockchain-tree/src/blockchain_tree.rs
0.00% <ø> (-84.11%)
:arrow_down:

... and 388 files with indirect coverage changes


Code Freeze 1

Closes #4

The build logs are failing – take a look at the below:
08:40:42.135  	Cloning github.com/MLH-Fellowship/pod-3.1.3-team-3 (Branch: staging, Commit: fd0b43f)
08:40:42.679  	Cloning completed: 544.604ms
08:40:42.710  	Analyzing source code...
08:40:43.752  	Installing build runtime...
08:40:46.756  	Build runtime installed: 3.004s
08:40:49.566  	Looking up build cache...
08:40:49.691  	Build cache found. Downloading...
08:40:51.760  	Build cache downloaded [43.25 MB]: 2068.337ms
08:40:52.843  	Installing dependencies...
08:40:56.345  	npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
08:40:56.350  	npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
08:40:56.355  	npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
08:40:56.363  	npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
08:40:56.363  	npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
08:40:56.364  	added 20 packages from 15 contributors in 3.085s
08:40:56.574  	87 packages are looking for funding
08:40:56.574  	  run `npm fund` for details
08:40:56.603  	Detected Next.js version: 11.0.1
08:40:56.604  	Running "npm run build"
08:40:56.806  	><EMAIL_ADDRESS>build /vercel/path0/gitdash
08:40:56.806  	> next build
08:40:57.519  	info  - Using webpack 5. Reason: Enabled by default https://nextjs.org/docs/messages/webpack5
08:40:57.668  	info  - Checking validity of types...
08:41:05.266  	Failed to compile.
08:41:05.266  	./pages/api/github.ts
08:41:05.266  	3:1  Warning: Assign arrow function to a variable before exporting as module default  import/no-anonymous-default-export
08:41:05.266  	./components/hero.tsx
08:41:05.267  	19:9  Warning: Custom fonts not added at the document level will only load for a single page. This is discouraged. See https://nextjs.org/docs/messages/no-page-custom-font.  @next/next/no-page-custom-font
08:41:05.267  	54:13  Warning: passHref is missing. See https://nextjs.org/docs/messages/link-passhref  @next/next/link-passhref
08:41:05.267  	./components/team.tsx
08:41:05.267  	121:20  Error: `'` can be escaped with `'`, `‘`, `'`, `’`.  react/no-unescaped-entities
08:41:05.267  	Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules
08:41:05.285  	npm ERR! code ELIFECYCLE
08:41:05.285  	npm ERR! errno 1
08:41:05.289  	npm ERR<EMAIL_ADDRESS>build: `next build`
08:41:05.289  	npm ERR! Exit status 1
08:41:05.289  	npm ERR!
08:41:05.289  	npm ERR! Failed at the<EMAIL_ADDRESS>build script.
08:41:05.289  	npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
08:41:05.297  	npm ERR!


[CSL-1453] Make benchmarks compile conditionally

So if they don't harm much for Windows, why not enable them?

My 2 cents: we should really have a separate Travis task dedicated to do just that (maybe regardless to the environment, ideally nix) so that we can keep each task relatively self-contained and fast to build 😉

My 2 cents: we should really have a separate Travis task dedicated to do just that (maybe regardless to the environment, ideally nix) so that we can keep each task relatively self-contained and fast to build :wink:

The problem is, it takes 40min to compile cardano-sl and then 5min to run the benchmarks. If you split that in two, you'll have almost double compilation times. Then we'd need to increase our workers for appveyor.

The problem is, it takes 40min to compile cardano-sl and then 5min to run the benchmarks.

Well, we wouldn't need to run the benches (not yet, at least), but ensure the code didn't rot and we can still compile it, yes. But I thought that with nix we were able to share dependencies (aka caching of those) between tasks of the same job? 😄

Also, @rockbmb, please add --bench --no-run-benchmarks (or whatever name these flags have) to travis.sh, to be run only for linux (not osx!)


Data needed ?

It seems at least the file base.mat is needed (I assume this is the average connectome they used for the synthetic model). Could you add back the data folder to just include the data/base.mat file ?

I've find the way to restore the file from git. Will do another PR.

FYI this is how I did
https://stackoverflow.com/a/1113140


creating an ACL from a default with an origin rule points the origin in new ACL to the default location

I create an ACL for a file inheriting (defaulting to) an origin predicate, the subject in the new ACL still points to the default location).
for example, starting with an empty https://gaia.solid.community/is.darcy/test/ folder and the following contents in https://gaia.solid.community/is.darcy/.acl:
@prefix : <#>.
@prefix is: <./>.
@prefix n1: <http://www.w3.org/ns/auth/acl#>.
@prefix acl: <http://www.w3.org/ns/auth/acl#>.
@prefix c: </profile/card#>.

:owner
    a n1:Authorization;
    n1:accessTo is:;
    n1:agent c:me<EMAIL_ADDRESS>    n1:default is:;
    n1:mode n1:Control, n1:Read, n1:Write.

:trusted-apps
    a acl:Authorization;
    acl:agent c:me;
    acl:accessTo <./>;
    acl:default <./>;
    acl:origin     <https://localhost>;
    acl:mode acl:Read, acl:Write, acl:Control.

And then running this code:
            const { AclApi, AclParser, Permissions} = SolidAclUtils;
            const { READ } = Permissions;
            let fileUrl = "https://gaia.solid.community/is.darcy/test/"; // a folder defaulting to its parent

            const fetch = solid.auth.fetch.bind(solid.auth);
            const aclApi = new AclApi(fetch, { autoSave: true })
            return aclApi.loadFromFileUrl(fileUrl).then(
                (acl) => acl.addRule(READ, 'https://giulio.solid.community/profile/card#me')

the resulting https://gaia.solid.community/is.darcy/test/.acl is the following:
@prefix acl: <http://www.w3.org/ns/auth/acl#>.
@prefix foaf: <http://xmlns.com/foaf/0.1/>.

<#owner> a acl:Authorization;
    acl:agent </profile/card#me><EMAIL_ADDRESS>    acl:accessTo <./>;
    acl:mode acl:Control, acl:Read, acl:Write.
<#trusted-apps> a acl:Authorization;
    acl:agent </profile/card#me>;
    acl:accessTo <./>;
    acl:mode acl:Read, acl:Write, acl:Control.
<https://gaia.solid.community/is.darcy/.acl#trusted-apps> acl:origin <https://localhost>.
<#Read-0> a acl:Authorization;
    acl:agent <https://giulio.solid.community/profile/card#me>;
    acl:accessTo <./>;
    acl:mode acl:Read.

As soon as the new ACL is saved the site loses control privileges on the subfolder.

As stated in the README, I don't intend maintenance of this repository. So if you want this to be added, either your or someone else will need to make a PR (which I'd merge if it looks good)
This would only require changes in solid-acl-parser (solid-acl-utils is only a wrapper for that). In particular, I think following things should be changed:

add origin option to AclRule.ts
parse it in AclParser.ts (_addQuadToRule and _ruleToQuads)

I think the implementation will be similar to accessTo and defaultForNew, so you could take a look at that and probably just copy it. And you could add a test case which shows that it works as intended.
I guess this would solve the inheriting problem.
If you also want to adjust this option, you could consider adding a addOriginRule method to AclDoc.ts similar to addDefaultRule


Non-unicode version? Wcwidth is a resource hog.

I understand the need for uni-code characters in modern applications, but for some which only need the standard ascii characters, calling Wcwidth repeatedly is needlessly eating up a ton of resources:

In this particular application, it alone is taking up 57% of total application CPU time.
It would be nice if there were a way to tell asciimatics to use len() instead. I'm going to create a fork and modify the code to allow for this switch and post the results.

I'd be interested in what your application is doing if you're happy to share...
When I first introduced unicode support, I spent a fair bit of time optimizing the logic so that all the samples still ran smoothly on my machine.  By the end I found that the cumulative time spent in wcwidth was between 5% and 20% of the application's CPU.
None of the ones I tried hit your numbers, so if you can share a bit more, I'd be happy to look into ways to reduce the impact.

Hi Peter,
I'm writing an algorithmic trading application. For it I created a custom log viewer widget that takes up the bottom 35% of my screen, so that I can keep an eye on my Logger calls. I initially considered using a StringIO object in the log view widget, and then pass that to the Logger instance and bind it to a StreamHandler, but instead I opted for a simple callback and custom formatter:

I could not use reduce_cpu=True with the log view widget in the Frame, as the logger gets updated quite frequently. And so the wcwidth/wcswidth functions were being called constantly.
But I added the option to use len() or other simpler logic under the assumption all characters are 1-character wide by adding
asciimatics.screen.set_unicode_support()
Which simply sets a global boolean flag. Not sure if that's the right namespace or not, but it made sense to me.
But it fixed the CPU use problem:

You can look at my fork if you wish to add the ability to disable unicode/wide-character support or not. I find it useful, but you may wish to force wider compatibility, which is totally fine. I just didn't need wide characters.
Thanks!

Also, I've encountered an issue where my keyboard/mouse input will no longer be responsive until I raise a ResizeScreenError to trigger resetting the wrapper.
Is there a way to "soft-reset" the input? I have attempted to forcibly set the focus to a specific widget, but it does not work.
Thanks!

OK - I think that there is any easy win here and then a harder one...

The easy one is just to use the unicode_aware flag that is already on the Screen to remove the call to wcidth().
The harder one is to use the same flag on the canvas to pass that through to the rest of the widget processing.

Making just the first change removes nearly all wcwidth usage in my samples and so I think is all that is probably all that is required here.  Now pushed to master - see https://github.com/peterbrittain/asciimatics/commit/68744d9c5184c1a751c2dd5ed973642c4ef59b73.
Also, you might consider using force_update() in your callback processing and then making your Log wait forever as its default refresh period.  This should reduce any unecessary redrawing when there are no updates.

Hi Peter,
Alright so I adapted my changes to use the unicode_aware flag already present in Screen, and had it pushed down using the frame._canvas object.
I'll create a pull request, as it really does improve performance for non-unicode applications.
I'll look into adapting my log viewer widget to force updates following callbacks as well.
Thanks for the input!

Pull request now merged.  Thanks.


Bug to LuckPerms

Hi i am find bug in to do your LuckPerms plugin. I use this command /lp user Test parent addtemp VIP 10m to add temp group in to the player what works properly or problem is in the time to expire VIP group. If player is to my server to unless group expire to add automatically add tempgroup to permanent group and when expire tempgroup to player  all the time is set VIP to permanent this is very big problém to my server Please fix this bug Thanks

Sorry, I don't really understand what you're saying.

Oh sorry for my English. Hello problem is when I write / lp user Test addtemp VIP 10d to play the VIP settings for 10 days and start counting it down and when it is 10 days to automatically remove the players from the Parent temp list, but when the player is activated Temporarily a VIP group and idem between worlds on servers like world events, etc. So the VIP Group is placed in the Parent permanent Group, which means that the VIP Group will remain in the group after 10 days of trouble.

You can use /lp user test removetemp vip to remove the temporary permission.

Yes but player me VIP group Parent Permamanent no TempParent.  In 10 days, the players will change the temporary VIP group to the stand without to use commands.

In which case, you need to make sure the temporary parent is removed before you add the permanent one.

Problem is in to the that I set Player Temp Parent group and set expire time 10 Day so VIP group remains to player to expired 10 Day VIP group is set to Parent Permanent Group this is problem which i have to my server

Sorry, I have no clue what you're trying to say.
Could you give me an example / screenshots?

Ok Screenshots
Screenhost 1 : https://ctrlv.cz/DybB - I set player Temporaly parent groups: Sponzor witch use /lp user test parent addtemp sponzor 10m - this is ok
Problem is here:
Screenhost 2 https://ctrlv.cz/VXdG - After 3 minutes. Plugin alone set Temporaly parent groups Sponzor to Parent group to world novyspawn
Screenhost 3 : https://ctrlv.cz/sbvO - Another 3 minutes. Plugin alone set group sponzor to world eventy in Parent groups
Screenhost 4  : https://ctrlv.cz/R34o After 10 minutes expire group sponzor and plugin remove sponzor but group sponzor is all the time set in to Parent Groups: and the player still has a sponsor group You can see the screenshot

The other groups in Screenshot 2 and screenshot 3 are not being added by LuckPerms.
Do you have Vault installed, or any plugins which edit permission data? I would guess it's caused by one of those.
This is definitely not an LP issue.

Yes vault is instaled to my server

I have no rights editing set through the Vault plugin

The only way that'd be happening is another plugin adding the groups through Vault.


[Snyk] Upgrade com.h2database:h2 from 2.1.214 to 2.2.220

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade com.h2database:h2 from 2.1.214 to 2.2.220.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 23 days ago, on 2023-07-05.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Not exploitable


SPFX to teams Issue : failed to sync solution to teams

Hi,
We are facing an issue in deploying the sharepoint framework webpart to teams. After deploying the solution to app catalog and clicking on the "Sync to teams" we are facing "failed to sync solution to teams" error and in console we are getting 400 response code.

Any help/Suggestions ?

@DeepikaEvuri  - Did you find the solution to this issue?

Go to Microsoft 365 Admin center by selecting Admin and enable "Allow sideloading of external apps" like the below:

you should find and remove it by opening Teams -> Apps -> Built for [tenantname]. After removing it here, you should be able to successfully sync the app once.


makinGIANTS/Banjen to projects

https://github.com/MakinGiants/android_banjo_tuner[makinGIANTS/Banjen^] - App that plays sounds helping to tune a brazilian banjo.

Thank you!

:+1:


Simple JS version for browser use

Is there any chance you could provide a simple JS version for browser use where webpack etc. are not available?

Hi @T-vK,
You can use Skypack to use SuperMemo on browsers.
https://www.skypack.dev/view/supermemo
Thank you!


feat(container, pod): default runAsNonRoot to true (#1143)

Backport
This will backport the following commits from k8s-24/main to k8s-23/main:

feat(container, pod): default runAsNonRoot to true (#1143)

Questions ?
Please refer to the Backport tool documentation

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1155

Questions ?
Please refer to the Backport tool documentation


Add cf router-groups command

This PR is to add cf router-groups command to CF CLI. This command lets user list router groups.
This is first in the list of stories scheduled for supporting tcp routes in CF. Pivotal tracker story link for this PR: https://www.pivotaltracker.com/story/show/100975070
(Required OAuth scopes for CF CLI client are added to cf-release.)
Regards,
@atulkc and @fordaz
(CF Routing team)

Looks great, merged

Thanks!


Support MPI Job in kubebench

To support MPI Job in kubebench, needs to resolve the issue in several projects - kubeflow, mpi-operator and kubebench.

Update rbac to grant kubebench service account permission to operate mpi jobs PR. https://github.com/kubeflow/kubeflow/pull/2442
Add mpijob_manifest_modifier to update manifest files (in this PR)
Add customized post_processor for MPI Job. Retrieving logs part is tricky, instead of using runner, since only launch-job has logs, I directly fetch pod logs from launch job after job complete.  (in this PR)
Use command array in mpi job spec.  PR https://github.com/kubeflow/kubeflow/pull/2434
Add StartTime and CompletionTime to chain argo workflow
(This can be done using successCondition: status.launcherStatus = Active  and  successCondition: status.launcherStatus = Succeeded, but anyway mpi-operator doesn't have timestamp to indicate job duration. PR https://github.com/kubeflow/mpi-operator/pull/89)

This change is

/cc @xyhuang

/ok-to-test

Just lint python files and push the revision. I will file an feature request to add pylint verification in hack scripts which will give better development experiences.

@xyhuang All upstream dependency issues are resolved. Please have another look.

hi @Jeffwan, thanks! this looks good to me, just some additional questions (we can probably address them in follow up PRs if needed):

does the example work for multiple workers? for that case you might want to make sure each one write to different files.
do you have an example kubebench job config like this https://github.com/kubeflow/kubebench/blob/master/kubebench/kubebench-examples/prototypes/tf-cnn.jsonnet ?

/hold
just to keep my comments open, please feel free to /unhold whenever you are ready to merge.

/lgtm
/approve

does the example work for multiple workers? for that case you might want to make sure each one write to different files.

Yes. MPIJob is a little bit different from TFJob. Only launch pod has logs which has been persistent. Behavior of multiple workers and one worker is same.

do you have an example kubebench job config like this https://github.com/kubeflow/kubebench/blob/master/kubebench/kubebench-examples/prototypes/tf-cnn.jsonnet

Now, I am using kubeflow/kubebench-job and then customize a job. I will add one more example in kubebench registry as you suggest in a separate PR.  We'd better release a new image version and this would be used in the example.

Should be fine to merge now. Example will come in separate PR.

/unhold

/hold cancel


fix(expect-util): fix comparison of DataView

Summary

Relate PR: https://github.com/vitest-dev/vitest/pull/3703
Test plan

Added

https://github.com/jestjs/jest/releases/tag/v30.0.0-alpha.1