Lock down IP object shapes

As the DHT keeps IPv4 and IPv6 addresses separate, the family property introduced in https://github.com/compact-encoding/compact-encoding-net/pull/1 isn't needed for distinguishing them.

nice and simple 👍

@kasperisager can you check if we pass the address from on('message') directly to other apis as well, re shapes (that one has family also, which makes sense)

@mafintosh It's passed on to internal APIs, but it doesn't look like the family property ever reaches public API surface.

For this callsite, for example, I think we specifically want the family property to ensure monomorphism: https://github.com/hyperswarm/dht/blob/737c46286670beddd53f226795f2b1e8e8c93356/lib/socket-pool.js#L15
That handler is also invoked from the DHT client and server sockets where the addresses contain the family property.

@kasperisager ya makes sense. was referring internally where we use that address, but let's check after merge


Avoid eval for Rollup.js

Hi guys,
I'm currently using the branch-sdk with rollup.js but I get the following error:

rollup: Use of eval (in /home/gp2mv3/www/covevent/refonte2/ionic/node_modules/branch-sdk/dist/build.min.js) is strongly discouraged, as it poses security risks and may cause issues with minification.
See https://github.com/rollup/rollup/wiki/Troubleshooting#avoiding-eval for more details

Why is eval used ? Isn't there other solutions ?

@Gp2mv3, thanks for bringing this to our attention. We aren't using eval in any of the source code, but it does get introduced by the google closure compiler during build. Is this something that is blocking you from using the sdk, or is it just a warning?

It's just a warning but it can discourage people to use the SDK as it warns for security.                                                                                De<EMAIL_ADDRESS>31 octobre 2016 2:12 AMÀ<EMAIL_ADDRESS>à<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Re: [BranchMetrics/web-branch-deep-linking] Avoid eval for Rollup.js (#393)  @Gp2mv3, thanks for bringing this to our attention. We aren't using eval in any of the source code, but it does get introduced by the google closure compiler during build. Is this something that is blocking you from using the sdk, or is it just a warning?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.

@Gp2mv3 after looking into it a little more, we determined that this is most likely caused by using JSON.parse in a couple of locations. We have a safejson.parse method that should be used instead. We will do some testing and hopefully get this fixed in the next deploy. Thanks for notifying us on this!

Guys you're awesome ! Your customer service and relation with developers are really cool. :wink:

Thanks @Gp2mv3! I'll let the team know

Hi @Gp2mv3, We believe this issue should be resolved with our latest WebSDK release (v2.12.0). Can you update and let us know if you still see an issue. I will close this out for now, but it would be great if you can update to latest and let us know if it is resolved or you still face an issue.

I updated to 2.12.0 but still get the warning:
[17:46:48]  rollup: Use of `eval` (in
            /home/gp2mv3/www/covevent/refonte2/ionic/node_modules/branch-sdk/dist/build.min.js) is strongly discouraged,
            as it poses security risks and may cause issues with minification. See
            https://github.com/rollup/rollup/wiki/Troubleshooting#avoiding-eval for more details

@Gp2mv3 thanks for working with us on this. I think we need to remove any reference to the closure compilers json file. I'll keep you posted.

Eval was removed as a dependency in newer versions, please reopen if you find that this isn't the case


Timestamps missing in feeds

Posts in feeds should show timestamps, they currently do not.

i did this ages ago lol


distinct smilies color

i want to set distinct color for all 5 smilies, how can i do it. Any help would be appreciated

Currently no api for setting color  to separate smiley. But will be made sooner, please look for the next update :) Thank you!

Will you solve this issue?
Thank you! ;)


Add resizing

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation
[ ] Other

Description
This pull request adds the ability to resize scalar assets. If no configuration for resizing is passed, no resizing will be performed. If ResizeOptions from sharp are passed as configuration, resizing will be performed and the configuration will be passed to sharp directly.
Resizing is only written to the file if the resulting file size would be lower than the original file size. I think this is a point which could be discussed.
Checks

[x] PR adheres to the code style of this project
[ ] Related issues linked using fixes #number N/A
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Lint and build have passed locally by running pnpm lint && pnpm build
[x] Code changes have been verified in local
[x] Documentation added/updated if necessary

Additional Context

I don't want to sound rude, but is there any feedback here?

Does it work? Not sure if the maintainer is active

Looks good to me @sebb0. I'll publish a release shortly. Thanks for the patience!

@FatehAK thank you!

Resizing is only written to the file if the resulting file size would be lower than the original file size. I think this is a point which could be discussed.

For the above case, do you reckon we will face issues if say user is resizing the image larger than the original @sebb0 ?

Resizing is only written to the file if the resulting file size would be lower than the original file size. I think this is a point which could be discussed.

For the above case, do you reckon we will face issues if say user is resizing the image larger than the original @sebb0 ?

In my opinion, the purpose of this plugin is to optimise image assets, mainly to save bandwidth.
I would simply want to use my high resolution images in a project and reduce them to a certain image size (plus all the other optimisations in this plugin) to save bandwidth. I don't care about the image size if the storage size of the image is smaller before passing through the plugin.
I have implemented this according to my needs.
If you think that users of this plugin want to enlarge image sizes and also want to be able to rely on this enlargement happening, then we probably need to tweak this.
Translated with DeepL.com (free version)

I have a quick question regarding this matter. If you pass the resize options as { width: 600, height: 700 } and you have images of varying dimensions, wouldn't all images be resized to exactly 600x700 pixels? Is that the expected behaviour you want?
Also, I developed this plugin with the primary goal of optimizing images rather than transforming them. Mixing these together makes things super messy. There should be other plugins available that handle image transformation as a preliminary step before this optimizer plugin runs as part of Vite's build pipeline.
@sebb0

@sebb0 awaiting your response for the above query before I go ahead with the merge. Thanks again!

The whole thing happened so long ago that I had to take some time to familiarise myself with the subject again.
I've thought it all over again and don't really see any problems. Your original concerns regarding the enlargement of assets can be dispelled with the [options.withoutEnlargement] parameter from the resize function of sharpen.js.
There are also suitable parameters for preserving the aspect ratios of the images.  The default behaviour is to keep the aspect ratio.
Unfortunately, I cannot dispel your concern about mixing two functionalities that you believe should not be mixed. But in the end it's your project and your decision. 😀

Just chiming in here—I would just like to mention that the publishing of this resize functionality would enable me to switch from vite-imagetools and use this plugin for all image optimization in my project.
Right now, I'm using vite-imagetools for resizing JPEGs, and vite-plugin-image-optimizer for optimizing SVGs. But I do think it would be great to have both handled by the same plugin (and without any benchmarks to prove it, could potentially minimize overhead in building my Vite project).
And regarding the separation of concerns matter: I'm of the mindset that resizing is a form of image optimization (maybe not other types of image transformations, but I think resizing definitely counts). Because the end goal of optimization is to cut down of file size, which image resizing expressly does. Moreover, I think the practicality of having this (also given that it's a relatively small amount of code) would sufficiently outweigh any drawbacks regarding principle.
My only question would be: does this PR allow for image resizing on a per-import basis? Because different images in my project need to be resized to different sizes. Vite-imagetools handles this through query params on the imported path, e.g.:
import myImage from './src/images/my-image.jpg?w=320';
import myImageRetina from './src/images/my-image.jpg?w=640';

My only question would be: does this PR allow for image resizing on a per-import basis?

No it was not in my scope. But personally, I like this idea.


The default certificate length is too small

When trying to run npm run start-dev it complains about SSL, that the length is too small

I just instead edit their code to use localhost, check the commit history
[https://github.com/deadlyjack/acode-plugin/pull/39]


Add metadata server to zedrouter take 2.

Currently implements services to return the external IP for the associated network instance and the hostname of the app instance
We had to back out the first take since it was creating an interface with the IP address <IP_ADDRESS> which meant all packets got routed locally to itself, whether the origin was from the application instances or from the host itself, and in GCP the host needs to be able to talk to an external DNS server at that IP address.
Thus take 2 instead runs the server on the bridgeIP address and has fixed SNAT and DNAT rules which only apply to packets coming from the application instances on the bnX bridge.

Summoning @deitch and @aw-was-here  to comment on my cloud-init musings above. Pretty please? ;-)

Summoning @deitch and @aw-was-here  to comment on my cloud-init musings above. Pretty please? ;-)


Remove swift file from normal iOS framework

Fixes https://github.com/CocoaLumberjack/CocoaLumberjack/issues/658

This will broke other swift project. How about revert #647

This does need resolving as otherwise non-swift projects require the swift dylib which adds to the bundle size. There could perhaps be two separate framework targets.

I think that two targets makes sense as less projects stay Objective-C-only.

So revert #647 (db6a623) that merges CocoaLumberjack, CocoaLumberjackSwift can solve the issue. I think.

I've just added a new commit which adds a separate swift framework target. I've tested on both swift and non-swift projects. There is no test target for the non-swift framework though.

The advantage of (partially) reverting #647 would be ensuring that most users won't even notice something happened in between.

@rivera-ernesto Right. If create new target or anything else, It will make users notice it.

@sinoru Could you try to make a pull request for that?

@rivera-ernesto Okay. I make it right now.

And thanks for pointing out this issue @nrbrook!

I just create new pull request (#690)

@nrbrook Please give #660 a try and tell us if that works for you.


Define order of git urls

Hi,
is there a way to define order of git urls in git_url_box?
Our members use 99% HTTPS links and that should be first to show up in list.
At moment it seems that ordering of git urls is hardcoded.

Done in commit https://github.com/jbox-web/redmine_git_hosting/commit/332ce2b03893264cab53b79181ee56841704faef. Will be in 1.1.0 release.


Add support for Structured Data in Arbitrary Message Signing

The user should be able to send a structured message (Clarity Value) to be signed.
See https://github.com/hirosystems/stacks-wallet-web/issues/1051

Functionality test passed. We do have a known issue : https://github.com/hirosystems/stacks-wallet-web/issues/2400.


Home Page hero announcement

select the latest event from the blog page, hashtag, or announcement. then show it on the home page hero.

it has done with adding latestEntry


cra pwa and initial assets

CRA with typescript and progressive web app features
favicons, logos and manifest file update

some tasks will be moved with other project issues. so we are moving forward


Auto select identity based on X-Original-To / Delivered-To / X-Envelope-To / To / Cc

Hi,
I have a catchall domain and I use one address per service, with some I need to reply to. So I configured a few identities in K9Mail. Let's say I have configured<EMAIL_ADDRESS>and<EMAIL_ADDRESS>and id1 is the default.
When I reply to a mail sent to<EMAIL_ADDRESS>k9mail selects<EMAIL_ADDRESS>as identity for the reply. I'd expect it to select<EMAIL_ADDRESS>since mail was addressed to that email (visible in To/Cc sometimes, and always on X-Original-To in my setup).
There are multiple headers which can define the original email it was adressed to, but K9 could maybe match a few headers against the identity list ? Priority would be X-Original-To / Delivered-To / X-Envelope-To / To / Cc I suppose. Not sure if there are more potential headers doing this job.
Is it already done and I am missing something maybe ?
If not, consider this a feature request ! ^^
Thanks !
Environment
K-9 Mail version: 5.010
Android version: 4.4.4
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

From a quick glance, this is currently based on the To and also Cc headers (see relevant code). I can't see a downside of also checking X-Original-To or Delivered-To, so we could simply add that.

Ah yes now I see it was happening on mailing list, where I don't appear in the To header.
That'd be great, thanks !

Oh yes, this would probably resolve my issue indeed ! :-)
Cheers

I would like to work on this

I would like to work on this

@Valodim I can't actually understand the issue here. Can you please explain a bit more?

Fixed by #2454

Thank you very much !


OLS 0.1.2 snapshot-to-catalog with snapshot ols-dlgtp

Description
OLS 0.1.2 snapshot-to-catalog with snapshot ols-dlgtp
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/lgtm
/approve


Adds nemo to the mentor list

Added a curriculum as well.

Looks fine. Please rebase it with master.

Is the book complete? Or lacks chapters mentioned in curriculum? If latter, then you need to mention this and also mention that this is an experimental course.

Done. Should be good to merge.


I cound't able to use luarocks with lua 5.3

I am running Ubuntu Server 15.04 on VM , I used these commands to install lua and luarocks
sudo apt-get install unzip
sudo apt-get install gcc
wget http://www.lua.org/ftp/lua-5.3.1.tar.gz
tar zxf lua-5.3.1.tar.gz
cd lua-5.3.1
sudo make solaris test
sudo make solaris install
reboot
wget http://luarocks.org/releases/luarocks-2.2.1.tar.gz
tar zxpf luarocks-2.2.1.tar.gz
cd luarocks-2.2.1
./configure; sudo make bootstrap
sudo luarocks install luasocket
lua -lsocket -- It errors.

Use make linux for building Lua on Linux instead of make solaris.

It didn't work. I also getting this problem. when i remove lua
make uninstall lua it still searching for lua how to remove variable or unset it ?

Typing hash -r (or starting a new shell) will make bash forget that there was a lua in /usr/local/bin.
What do you mean by "It didn't work"? What's the error message? Do you have all required development packages installed (e.g. libreadline-dev)?

I check libreadline-dev it's installed. I used make Lunix install to install Lua and I am pretty sure i installed luasocket.
When i do lua -lsocket it errors me here.

What lua headers do you have installed?
Please paste the output of:
grep "LUA_VERSION" $(find /usr/ -name lua.h)

@daurnimator: lua_gettop is a function in all Lua versions since 5.0 at least, so it can't be a header version issue. The Lua API just isn't available for the extension module, which means that the lua executable used wasn't built with make linux. (Another hint for that is that readline isn't working.)
Possible reasons I can think of:

a missing make clean before make linux
a missing sudo before make install

I promise i will learn every single detail about unix and lua library but dont make me suffer from lua 5.1 :^(

@quiloos39 type which lua in your terminal. Does it show /usr/local/bin/lua?

Yes

Then type ldd /usr/local/bin/lua and show us the output please.

Copy and paste of the text will do. No need for a screenshot ...

Sorry i cant copy paste through Vm.

Did you use make linux? the lack of io.popen suggests not.

I use generic

I use generic

Use make linux

And as mentioned before, you need libreadline-dev installed for make linux to succeed:
sudo apt-get install libreadline-dev

I cant believe it worked i don't know what i did wrong. Probably libreadline-dev i am pretty sure i downloaded before but it seems it fixed my problem some how thanks all for helping.
I don't to topics here so i will ask here last time. I installed luasocket successfully but when i install luasec its erroring me.

when i install luasec its erroring me.

As it says in the error message, you need openssl/ssh.h.
On ubuntu, it's in the 'libssl-dev' deb.

So what i suppose do in this case ?

So what i suppose do in this case ?

Install libssl-dev (sudo apt install libssl-dev) then install luasec (sudo luarocks install luasec)

Thanks for all @siffiejoe I apologize to you i didn't understand at begging.


feat(network): Add stabilitytestnet network and FREE currency

Summary
This PR adds support for Stability Testnet network.
Changes

supportedChains
networks
Blockchain icon img

Issues
NA
For further information, please reach the following email<EMAIL_ADDRESS>

hi, can fix this CI and update this repo after tt-verfy has been updated, because there is a dependency on tt-verify repo.

hi, can fix this CI and update this repo after tt-verfy has been updated, because there is a dependency on tt-verify repo.

ready to go, thanks!

@isaackps
tt-verify PR: https://github.com/TradeTrust/tt-verify/pull/12
We will take care of bumping the dependency once it's merged

@isaackps tt-verify dependency updated


documentation: Add checkup API document

Add a document to describe the API between the kiagnose framework
and the checkup.
Signed-off-by: Orel Misan<EMAIL_ADDRESS>

Correct me if I am wrong, but isn't this a rewriting of the [Checkup Installation] (https://github.com/kiagnose/kiagnose/pull/93/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R63) in #93?
If so, perhaps it would be better to join the 2 PRs so that the README could link to the checkup-API doc?

This document is intended for checkup authors.
It should depict the API between the framework and a checkup.

Please try to avoid numbering when its not a must, it will make the it more "accessible"
and not feel like reading a contract

Isn't this document describing an API?

Rebase.

Change: Answered review comments.


Skull of guldan quest completed at start

Closed by https://github.com/AzerothWarsLR/WarcraftLegacies/commit/985787c50977547bb754d9a1efbf6472ed7d472c.


TypeError ufunc 'isnan' / 'isinf'

Hello,
I'm trying to use causal-learn (pc and fci at the moment) but I'm stuck with input data errors.
My dataframe has no nans nor infs, as checked with both pandas and numpy, but running either algorithm reports error
TypeError: ufunc 'isnan' (or 'isinf') not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
The full stack trace is as follows (where data types can also be seen):
reading csv...
                df.dtypes=cpu_utilization    Float64
memory_usage       Float64
pixel                Int64
fps                  Int64
consumption          Int64
dtype: object
                df.isnull().values.any()=np.False_
                np.isinf(df).values.any()=np.False_
        learning structure...
                df.shape=(12748, 5)
0it [00:00, ?it/s]
Traceback (most recent call last):
  File "/home/stefano/PyCharm-projects/Playground-causal-learn/scripts/tu_wien_stuff.py", line 126, in <module>
    process_file(f"{args.filepath}/{file}", columns, args)
  File "/home/stefano/PyCharm-projects/Playground-causal-learn/scripts/tu_wien_stuff.py", line 81, in process_file
    model = learn_structure(np_data, args.nrows, filepath)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/stefano/PyCharm-projects/Playground-causal-learn/scripts/tu_wien_stuff.py", line 46, in learn_structure
    model_graph, model_edges = fci(df, mvpc=True, indep_test="mv_fisherz")  # TODO ufunc 'isnan' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/stefano/.pyenv/versions/3.11.3/lib/python3.11/site-packages/causallearn/search/ConstraintBased/FCI.py", line 738, in fci
    independence_test_method = CIT(dataset, method=independence_test_method, **kwargs)
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/stefano/.pyenv/versions/3.11.3/lib/python3.11/site-packages/causallearn/utils/cit.py", line 32, in CIT
    return FisherZ(data, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/stefano/.pyenv/versions/3.11.3/lib/python3.11/site-packages/causallearn/utils/cit.py", line 142, in __init__
    self.assert_input_data_is_valid()
  File "/home/stefano/.pyenv/versions/3.11.3/lib/python3.11/site-packages/causallearn/utils/cit.py", line 81, in assert_input_data_is_valid
    assert allow_nan or not np.isnan(self.data).any(), "Input data contains NaN. Please check."
                            ^^^^^^^^^^^^^^^^^^^
TypeError: ufunc 'isnan' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

Hi, if possible, could you please provide a minimal example of code and data to reproduce this error? I haven't got this on my end.

Sorry for the latency, here it is a gist with a MWE and example dataset that generates the error:
FYI, in the MWE I'm using causallearn.search.ConstraintBased.PC, but the same error happens with fci.
As an aside, the same MWE but using ges or lingam gives a different error: AttributeError: np.matwas removed in the NumPy 2.0 release. Usenp.asmatrix instead. (I have numpy 2.1.1 installed)


[camera] Handle empty grantResults on permission request

This PR handles a special situation where the grantResults array could be empty when asking for permissions.
Fixes https://github.com/flutter/flutter/issues/116047
Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[X] I signed the CLA.
[X] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[X] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is test-exempt.
[X] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@camsim99 Unfortunately the Mac_x64 check keeps failing. There is nothing new to rebase on

@davidmartos96 I re-ran it because I think it's a flaky test, unfortunately. It looks like it passed!

@davidmartos96 I re-ran it because I think it's a flaky test, unfortunately. It looks like it passed!

Great!


Large Refactor to Convert pkg/middleware-analyzer into a Consumable Library

What this PR does / why we need it

This is a significant change in this repo. This does the following:

makes it so that the code/structs/interfaces in pkg/middleware-analyzer acts as a library for things that need to be implemented suck that you can create your own middleware component
the code that lives in cmd/example-middleware-analyzer is a representation for an implementation for using the pkg/middleware-analyzer library/interfaces/etc
the code that lives in cmd/example-simulated-client-app is a representation for what the client/UI/etc would need to implement
the application-specific structs and messages that this cmd/example-simulated-client-app interface/UI/etc would use are now located in pkg/middleware-analyzer
the example pkg/middleware-analyzer behavior has now been modified so that only the 5 most recent references to trackers will be provided instead of an unending list of past references
renamed symbl-dataminer to symbl-proxy-dataminer
renamed example-your-middleware to example-middleware-analyzer
renamed simulate-web-client-app to example-simulated-client-app

Additional work to be done in a follow up PR:

modify the application-specific struct/messages so that the most recent 5 references will be sent as a list instead of 5 individual messages
implement 5 most recent references for entities, topics, etc
[long-term] remove cmd/example-middleware-analyzer and cmd/example-simulated-client-app into a separate repo and replace with a "generic" (aka default) shell of an implementation (aka just print a message).

Which issue(s) this PR fixes

NA
Describe testing done for PR

Verified functional has been unchanged since previous version
Special notes for your reviewer

NA

Gave community notification of the change here: https://symbldotai.slack.com/archives/CB0QF76PK/p1675186821013949
Going to merge based on lazy concensus.


chore(src): upgrade to openresty <IP_ADDRESS>(WIP)

Summary
We need the related patches of OpenResty <IP_ADDRESS> to pass the ci tests.
Change logs:

remove apple m1 compatible code (<IP_ADDRESS> has already integrated it)

Reopen When OpenResty <IP_ADDRESS> is officially released.


IA-2292 500 when using a csv with semicolon in batch user creation + add projects and user roles

Explain what problem this PR is resolving
Related JIRA tickets : IA-2292 and IA-2289
Self proofreading checklist

[X] Did I use eslint and black formatters
[X] Is my code clear enough and well documented
[X] Are there enough tests

Changes
Field Updates :
Mandatory columns  have been updated to make it less error prone.
New column " user_roles " and " projects " have been added.
Mandatory columns are : ["username", "password", "email", "first_name", "last_name", "orgunit", "profile_language", "dhis2_id", "permissions", "user_roles", "projects"]
I added a check of csv delimiter to handle files separated by both coma and semicolon. For some reason sometimes ne Sniffer doesnt work and result into a error. So the code check first with Sniffer, if it fails it try a " manual " check by parsing the file.
Also, more tests.
How to test
Explain how to test your PR.
Test with a CSV created with Excel.
Upload a CSV file with semicolon separator, it should work. Also test the new fields. Use one of the working templates to test.

@hakifran  I just did a test and I don't have the error. Can you describe me what you are doing ?

I just found a bug that I have to fix.

Juste une question : pourquoi il m'affiche ce message bizarre tout en haut de cette page ? juste parce qu'il y  a un ; dans le nom de la branche ?

@mathvdh Oui c'est juste à cause du " ; " c'est pas bien grave.


Fix root path of coverage upload

The repo files were moved to a subfolder which confuses the coverage UI
Setting the base-path allows to keep the sources in the subfolder of the default checkout folder.
Note: The subfolder was required because we need to put BOOST_ROOT somewhere and only the PROJECT folder is guaranteed to be writeable.
See e.g. https://coveralls.io/builds/46260109/source?filename=repo%2Finclude%2Fturtle%2Fcleanup.hpp for an example of the issue

Coverage decreased (-97.8%) to 0.0% when pulling a4d72a5bf47deb3e1e4a57bca43f41158daff247 on Flamefire:fix_coverage into b9a1a4a4b75817366099ef02261cedc84ec8ccba on mat007:master.


Anomaly explanation

This PR adds descriptions from https://www.elastic.co/guide/en/elasticsearch/reference/master/ml-get-record.html to the Anomaly.ts file.

Fixed the issue with the validator so this PR has been merged, thanks all for the patience.


Generate CDN SDK for 2020-09-01 AFDX support

Regenerate CDN SDK for 2020-09-01 AFDX support https://github.com/Azure/azure-rest-api-specs/tree/7a71c15c380d7b9061496698f3679a19aa63191d/specification/cdn/resource-manager/Microsoft.Cdn/stable/2020-09-01

This request for an SDK for our upcoming AFDX feature.

This request for an SDK for our upcoming AFDX feature.

/azp run net - mgmt - ci

/azp run net - mgmt - ci

@ChenglongLiu What feature is this code being re-generated for? Are new properties being added? Thank you.

@ChenglongLiu What feature is this code being re-generated for? Are new properties being added? Thank you.

@ChenglongLiu What feature is this code being re-generated for? Are new properties being added? Thank you.

@nisha-bhatia  AFDX is a new service and will public review on 02/18. Serval resources are added in AFDX. We need to release the new version SDK first for service sign off.

@ChenglongLiu What feature is this code being re-generated for? Are new properties being added? Thank you.

@nisha-bhatia  AFDX is a new service and will public review on 02/18. Serval resources are added in AFDX. We need to release the new version SDK first for service sign off.

@nisha-bhatia, can we now merge this now that all the checks have passed? Our development is currently blocked on this PR.

@nisha-bhatia, the SystemData property was added in the below PR. Its an ARM property and not specific to our service.
https://github.com/Azure/azure-rest-api-specs/pull/11655/files#diff-81f6c7aad1a5b5fcf16bd0e7ddeb95be5e384f50d43e3081c31b4e7c9ed8b856

@jorinmejia Could you please provide the Swagger link where the SystemData property is introduced for this method? I don't see it in the linked Swagger PR.

@jorinmejia Could you please provide the Swagger link where the SystemData property is introduced for this method? I don't see it in the linked Swagger PR.

@nisha You can find the SystemData at line 5452 of cdn.json
https://github.com/Azure/azure-rest-api-specs/pull/11655/files#diff-81f6c7aad1a5b5fcf16bd0e7ddeb95be5e384f50d43e3081c31b4e7c9ed8b856R5452
It's read only system data
 "SystemData": {
      "description": "Read only system data",
      "type": "object",
      "properties": {
        "createdBy": {
          "type": "string",
          "description": "An identifier for the identity that created the resource"
        },
        "createdByType": {
          "$ref": "#/definitions/IdentityType",
          "description": "The type of identity that created the resource"
        },
        "createdAt": {
          "type": "string",
          "format": "date-time",
          "description": "The timestamp of resource creation (UTC)"
        },
        "lastModifiedBy": {
          "type": "string",
          "description": "An identifier for the identity that last modified the resource"
        },
        "lastModifiedByType": {
          "$ref": "#/definitions/IdentityType",
          "description": "The type of identity that last modified the resource"
        },
        "lastModifiedAt": {
          "type": "string",
          "format": "date-time",
          "description": "The timestamp of resource last modification (UTC)"
        }
}

@nisha-bhatia, this SystemData property is also a requirement by ARM for RPs. We had to include it in our swagger to pass validations and other tests when merging into the Azure REST Specs repo.


Disable Spark Catalog caching for integration tests

Fix #482
While working on #444, I couldn't get integration tests working since using spark.sql to count the number of data files returns the wrong result. This becomes more bizarre when Python's iceberg table state is not the same as Spark's iceberg table state.
This PR adds a simple test to verify that "python's iceberg table snapshot id" is the same as "spark's iceberg table's snapshot id".
The culprit is the spark.sql.catalog.catalog-name.cache-enabled setting, which defaults to True and caches table metadata.
Spark sql calls will use the cached iceberg metadata instead of the updated one.
spark.sql.catalog.catalog-name.cache-enabled	true or false	Whether to enable catalog cache, default value is true

https://iceberg.apache.org/docs/latest/spark-configuration/#catalog-configuration

Thanks @kevinjqliu I think this change makes sense. I don't think there's ever a reason on the Python side where we want to have the spark caching enabled. On the Iceberg Java side we do have tests which validate the caching catalog behavior when it's enabled/disabled so we don't need to test that through PyIceberg (I think). I've triggered CI if it passes, I'll go ahead and merge

Great idea @kevinjqliu ! Thanks for adding this

Sweet, thanks @kevinjqliu! I'm going to go ahead and merge this now.


rotation gizmo can be selected and removed, causing crashes

Describe the bug
The rotation gizmo can be selected when using the right-click+drag multi selection. Doing this the gizmo can actually be deleted, which causes the game to crash when selecting anything after that, trying to disable it in the toolbar or pressing the rotation hotkey.
This does not happen with the move gizmo, it is completely ignored by the multi selection.
Steps to reproduce the issue

select any unit
enable the rotation gizmo
deselect
use right-click + mouse drag over the just deselected unit, the rotation gizmo should now be selected
delete the selection
selecting anything or disabling the rotation gizmo will now crash the game

Log file
No response
Minimal reproduction
No response

Check if that does the trick.

nope, didn't fix it
https://user-images.githubusercontent.com/37846872/145842737-ced4e66a-2904-44a7-a17c-3668bc12a03d.mp4


Face recognition in the Voynich manuscript

I wanted to capture the faces from the so-called Voynich manuscript in order to categorize them later according to similarities. Face_recognition recognizes some faces but by far not all. Is the quality of the drawn faces too bad ?
Here is an example page:
https://www.jasondavies.com/voynich/#f84r/0.537/0.491/2.14

你好，人脸识别，可以教学吗？ 我出10万人民币学费，tg：@miao678


Container border-radius and border don't work if before_update method is used

Duplicate Check

[X] I have searched the opened issues and there are no duplicates

Describe the bug
If the before_update method is used, border-radius and border  params for container don't work
Code sample
Code
import flet as ft

# 0.19: OK
# 0.21.2 and > : KO
class Test(ft.Container):
    def __init__(self):
        super().__init__()
        self.border_radius = 15
        self.width=100
        self.height = 100
        self.bgcolor=ft.colors.BLUE_100
        self.border=ft.border.all(3,ft.colors.BLUE_ACCENT_700)
        self.content=ft.Icon(name=ft.icons.ADD)

    def before_update(self):
        print("doing something before update")

def main(page:ft.Page):
    page.theme_mode = ft.ThemeMode.LIGHT
    page.add(Test())

if __name__ == "__main__":
    ft.app(target=main)

To reproduce
simply run the code and compare with flet 0.19 or less, or with before_update() commented
Expected behavior
No response
Screenshots / Videos

Captures
Wrong:

Good:

Operating System
Windows
Operating system details
W11
Flet version
21.2 and >
Regression
Yes, it used to work in a previous Flet version (please specify the version in additional details)
Suggestions
No response
Logs
Logs
[Paste your logs here]

Additional details
No response

You need to add a super call:
    def before_update(self):
        super().before_update()
        print("doing something before update")


Translate training/_index.html in Korean.

This is a Feature Request
What would you like to be added
Translate training/_index.html in Korean.
Why is this needed
The following files are not translated into Korean.

[ ] /en/docs/training/_index.html

Comments
Page to update: https://kubernetes.io/docs/training/
I am translating this document.
The translated document will be submitted to dev-1.17-ko.7 branch.
/language ko

/assign

@ysyukr Only bug fix will be accepted in dev-1.17-ko.7.

@gochist I know that the purpose of the dev-1.17-ko.7 branch is bugfixes and current updates.
During the update of dev-1.17-ko.7(#19712 ), the training added to docs/home was reflected as /training based on the URL, but when checking the preview, it was changed to /ko/training and it was judged that translation of the traning document is necessary.
https://deploy-preview-19712--k8s-dev-ko.netlify.com/ko/docs/home/

@ysyukr That's right. Thank you!

/close


Stable 1.5.1 rc

We've been lax on getting PRs applied to the stable branches.  Here's an update for stable-1.5 . based on fixes which landed on master since 1.5.

/test

e8a8e0db Merge pull request #1111 from sboeuf/bump_kernel
16bd9833 Merge pull request #1264 from Ace-Tang/master
b96ca223 Merge pull request #1260 from stefanha/fix-vhost-user-blk-detach
1d79338a Merge pull request #1247 from nitkon/leakyPods
60f7c4f4 Merge pull request #1189 from devimc/topic/fixCpuCgroup
22cee2d0 Merge pull request #1220 from jcvenegas/delete-force-not-fail-non-container
731ff7b1 Merge pull request #1242 from teawater/golock
816ea428 Merge pull request #1202 from Pennyzct/unit-test
7f2b2da7 Merge pull request #1239 from chavafg/topic/update-k8s
268ddc59 Merge pull request #1237 from chavafg/topic/update-qemu-lite-hash
136b188f Merge pull request #1235 from nitkon/debug
d07297c1 Merge pull request #1232 from jcvenegas/clear-rootfs-latest
d7b6b250 Merge pull request #1205 from lifupan/asyncevents
1c27ab79 Merge pull request #1225 from teawater/fixmakeinstall
6431f1f2 Merge pull request #1066 from sameo/topic/state-storage
319a98fd Merge pull request #1175 from nitkon/master
a3a3d1a5 Merge pull request #1216 from grahamwhaley/20190206_remove_pullapprove
d3c63e66 Merge pull request #1212 from sameo/topic/ut-noise
f92ca1d9 Merge pull request #1199 from devimc/topic/snapBanner
5f1ca164 Merge pull request #1178 from yyyeerbo/wip
530360d5 Merge pull request #1186 from nitkon/Hotplug
29dae85a Merge pull request #1181 from sameo/topic/network-interface
91c454da Merge pull request #1168 from bryteise/hypervisor-cmd-defaults
eb92f498 Merge pull request #1193 from grahamwhaley/20190129_CODEOWNERS
7cb2f31c Merge pull request #1173 from lifupan/criconfig
4092eba6 Merge pull request #1187 from devimc/topic/fix1185
726df9da Merge pull request #1162 from jcvenegas/fix-1161
6f2c0366 Merge pull request #1164 from liwei/fix-factory-status
1353499e Merge pull request #1115 from nitkon/P9SMT

/test


update registration to support latest version w/signoff

Notes for Reviewers
This PR fixes #12155
Merge this Meshkit PR before merging this PR.
Signed commits

[ ] Yes, I signed my commits.

@Jougan-0 As discussed, both comps and rels will be registered from the latest version of the model.
To do that, the ability to generate relationships from Integrations sheet for all models needs to be incorporated before this gets merged correct?
Otherwise, there will be issues where all models don'thave relationships.

Yes, that's what we're currently doing. However, I added a fail-safe to check if the latest version lacks relationships, and if an earlier version has them, we'll use that instead. Generally, we register the latest version, but since the logic for generating relationships isn't implemented yet, this approach will work without breaking anything. Moreover, this method should continue functioning smoothly in the future as well.
The code won't require updates, as it includes an additional check on the latest version.


Could you add licenses to the preference datasets after reward model labeling on huggingface?

Hi authors,
Thank you for the nice works and release of the datasets. Could you add corresponding licenses to the released datasets, namely https://huggingface.co/datasets/princeton-nlp/llama3-ultrafeedback-armorm and https://huggingface.co/datasets/princeton-nlp/llama3-ultrafeedback? Correspondingly it has no licenses, and it will be greatly helpful to add like MIT license to let us officially use them.

Thanks! We have added the MIT license. For the dataset creation, we used random seeds 13, 21, 42, 79, and 100 with a temperature of 0.8 for sampling to obtain 5 responses per prompt. I hope this helps!
Best,
Yu

Thank you for the prompt replies! Closed the issues!


Added Seo Schema to Pages

Once again bothering with another PR :) . Um in PR #19 I did not include description meta. So including that with this PR. But I am attaching another SEO related thing which is adding schema. I have followed instruction from https://keithpblog.org/post/hugo-website-seo/. Feel free to Close the PR if you think it irrelevant.

Maybe this will helphttps://developers.google.com/search/docs/guides/intro-structured-data

Maybe we can add some flags in config to enable or disable based on user preference?

Maybe you are right, but I don't want this theme to be complicated. Theme's readme says:

It's built for bloggers who want a simple and focused website.

#26 is merged:
Since now people can add anything in html head, this feels not so necessary, so it can be closed.


ConfigMap, handle chomping

Summary
Fixes rancher/dashboard#5958

Technical notes summary

We decided to tackle this issue by forcing our yaml files to be saved with blocks style indicator '|+' which means that all newlines are preserved.
This strategy will cover all cases, since the users can decide to remove or add newlines themselves and avoid us to implement additional UI elements to control the yamls style.
To make it easier for the users to check the newlines and possibly allow them to remove or add new lines, we implemented space&line-breaks markdowns inside the value fields of the CofigMap resources.
Areas or cases that should be tested

ConfigMap edit view.

Areas which could experience regressions

Every place where we create and save resources as yaml file.
Screenshot/Video

Technical notes for the reviewers

The block style indicator inside the yaml file is made after the yaml file is create by js-yaml; the library does not currently allow to change block's indicators: https://github.com/nodeca/js-yaml/issues/495
Config Maps are now saved always as yaml file, since there is no ways to define block's indicators inside a json object.

References

Yaml, blocks indicators: https://yaml-multiline.info/
Codemirror guide: https://codemirror.net/5/doc/manual.html
Js-yaml: https://www.npmjs.com/package/js-yaml

Thanks for your review @cnotv .
I resolved the bug you mention here:

There is an issue with indexing the component using the key, as the value will point all to the same empty value ('') key. It works correctly if the key is defined.
Kapture.2023-04-26.at.16.54.42.mp4

but this is still an open point:

~Also I am not so sure, do we treat all the files as YAML or what?~ I tried it out with an HTML file I used for another PR and I crossed a small issue at the beginning of the content while converting it. Not so sure how much meaningful it.
Kapture.2023-04-26.at.17.15.06.mp4

And I wanted to do some tests, since it could be a regression. Could you please provide me the original yaml file you used to replicate it ?
For the other minor issues, I added more comments on the code and made some changes.
Other open point, unit tests are missing, I will open a new issue for those if you agree.

I used an HTML file to be created with the config map which you can find here.
For the tests we can rather don't create them at all if we have to make an issue, as it makes no sense. It's up to you as we do not have any definition about to deal with this 😅

Everything seems working fine now. Only thing but legit, if someone does not put a key and then switch to YAML, it will not show the changes.

The field is considered invalid if the key is missing, so it's discarded. Changes are also discarded when you put only value in a ingress rule and then click save.


Improve ListItemProperty

These changes do some minor improvements to ListItemProperty
and also let the user find out current index of the item tracked.

We use my prefix for fields. Also, we use only @Nullable annotation and not @NotNull. I merge it in, and make this fixes myself to save time but please make code conform to the code style next time.

Ok. I just notices another issue: I specified version on the dependency in pom.xml which should not be done. Could you please remove it too?

Also, we use only @Nullable annotation and not @NotNull

Sorry, I didn’t know that. There are a bunch of @Nonnull in all the repositories that we use, so I thought that was acceptable.

And since you removed all the @Nonnull, the dependency on findbugs should probably be dropped too as it was needed only for this annotation to work.


Fails to parse output from pg_dump?

I'm not sure if I'm using this wrong, but DDLParser() fails to parse the attached PostgreSQL schema output from pg_dump
schema.txt
Traceback:
~/.cache/pypoetry/virtualenvs/capgras-r404Yw5Z-py3.8/lib/python3.8/site-packages/simple_ddl_parser/parser.py in run(self, dump, dump_path, file_path, output_mode, group_by_type, json_dump)
    228             Dict == one entity from ddl - one table or sequence or type.
    229         """
--> 230         tables = self.parse_data()
    231         tables = result_format(tables, output_mode, group_by_type)
    232         if dump:

~/.cache/pypoetry/virtualenvs/capgras-r404Yw5Z-py3.8/lib/python3.8/site-packages/simple_ddl_parser/parser.py in parse_data(self)
    167                 if final_line or new_statement:
    168                     # end of sql operation, remove ; from end of line
--> 169                     statement = statement[:-1]
    170                 elif num != len(lines) - 1 and not skip:
    171                     # continue combine lines in one massive

TypeError: 'NoneType' object is not subscriptable

Please feel free to close this issue if you don't intend to support this use-case.

@bjmc hi, thanks for reporting the issue! I will work on it today & release a fix after.

@bjmc thanks for the sample - I added support for bunch cases based on your example. Now it parsed and you can get output. Please, use version 0.22.6 it is released - https://pypi.org/project/simple-ddl-parser/

Wow, thanks for that incredibly fast response! I'll give the new version a test tomorrow and if it works, I'll close this issue.
Thanks for your work on this project @xnuinside - it's very ambitious, and as far as I can tell, no one else is even attempting to offer something similar.

This is a big help. It parses without throwing an exception now! I think there's still room for improvement in understanding the meaning of the Postgres schema. For example, the DDL contains

ALTER TABLE ONLY public.accounts ADD CONSTRAINT accounts_username_key UNIQUE (username);

But in the parsed output, we see:
   {'name': 'username',
    'type': 'character varying',
    'size': 50,
    'references': None,
    'unique': False, <======= should be True?
    'nullable': False,
    'default': None,
    'check': None},

But the bug I reported is fixed, so closing this issue.

@bjmc   I created separate issue for this - https://github.com/xnuinside/simple-ddl-parser/issues/101 . I will fix it in future release.


Add HTML meta tags

Meta tags for:

Initial device zoom/scale (responsiveness)
utf-8 encoding
IE=edge compat (don't let new IE browsers emulate old ones)

Bra feature!


[python] Allow adata.X to be None

Issue and/or context: Issue #1672. Extends PR #1762.
Changes:
Notes for Reviewer:

This is stacked atop the blocking PR #1766 until that PR is merged, at which point this PR will be on main.

Codecov Report
All modified lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

apis/python/src/tiledbsoma/io/ingest.py
87.44% <100.00%> (+0.01%)
:arrow_up:

... and 76 files with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.

@johnkerl In the very first line you refer to both issue and PR as 1762.  Did you mean 1762 and 1763 ?


CLIP image processor

What does this PR do?
Adds image processor class for CLIP model
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] 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?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


feat/add debug capabilities

Fixes #4

feat: add debugger support
feat: add debugging logs for primary work

Codecov Report

Merging #39 (4f2cf0a) into main (24c379b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #39   +/-   ##
=======================================
  Coverage   98.55%   98.55%
=======================================
  Files           6        6
  Lines         207      207
  Branches       60       60
=======================================
  Hits          204      204
  Misses          3        3

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


Support json type

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Fixed issues
-

👍


KEDA Validation should prevent the creation of a ScaledObject with duplicate value for spec.advanced.horizontalPodAutoscalerConfig.name

Proposal
KEDA's validating admission webhook should prevent the creation of a ScaledObject if another ScaledObject already exists in the namespace with the same value for spec.advanced.horizontalPodAutoscalerConfig.name.
If more than 1 ScaledObject exists with the same value for spec.advanced.horizontalPodAutoscalerConfig.name, KEDA operator keeps updating the underlying HPA object continuously.
Use-Case
No response
Is this a feature you are interested in implementing yourself?
No
Anything else?
No response

Reasonable idea


Add a the ability to add link buttons to a message

Good example of call api required - https://github.com/slackapi/python-link-button-example/blob/master/example.py. Extended to work a bit nicer.
Additionally, will likely be similar to post_message with additional specification for the button(s)

Tbh I don't see what needs to actually change here? At most a wrapper class for buttons?

Ah I see now - the web server bit

So this has blown out to implementing attachments in general with only implementing the link button specifically

@TRManderson There is a bit of functionality and requirements required (which as far as I have seen haven't being made anywhere. I am using this as a guide https://api.slack.com/docs/message-attachments#when_to_use_attachments. The actual api call will be quite simple but I want it to be simple af to add attachements (such as buttons) to a message

I'm confused. Isn't this just bot.post_message(channel, text, attachments="stuff")?

How does one currently format those attachments tho? @mitchmcdee

isn't it in the document you linked? https://api.slack.com/docs/message-attachments#attachment_structure

I am trying to get a step away of needing to go into the docs, and providing an easy (and quick) way to add attachments like buttons

I reckon you'll just end up reading the docs anyway tbh

Ya this just seems like an abstraction of one facet of attachments and trades off reading their docs for reading yours. Additionally, they have pretty good docs already, and the structure looks pretty straight forward, so I don't think the abstraction is necessary

what you could do though is whip up an example of using buttons, say for !poll or something, and that'd be a nice demonstration of attachments/buttons :^)

@mitchmcdee I am currently doing so. The idea originally came from @jdcaperon's work on the umart command for having links to the pages, as the links were quite long we are looking to not clutter screen real estate

Check out Max's Hoogle PR (#312) for ezmodo link formatting


SetNoCache interference

I am implementing Asp.NET Identity alongside IdentityServer with ResourceOwner grant type. Inside my custom IResourceOwnerPasswordValidator I use the standard Asp.NET Identity's SignInManager PasswordSignInAsync function to validate the user. This function however sets the "Cache-Control" header to "no-cache". This conflicts with the TokenResult's ExecuteAsync function at the end of the pipeline which also tries to set this header, resulting in a duplicate key error.
Is there a workaround to this I might have missed, or should I just inject the context in my IResourceOwnerPasswordValidator and remove the header again?

IIRC this method in aspnet id sets a cookie? you don't want to sign in the user - just validate the password.
@brockallen we need to add an implementation of that interface to the aspnetid sample

Hi @Zinshis - I just looked over Identity code and didn't find anywhere that set cookies. I might be mistaken, but PasswordSignInAsync is probably what you need if you are trying to integrate user authentication from it with IS4. I have an example of how this is done on my repo. Contributions are welcome.

It does set a cookie
https://github.com/aspnet/Identity/blob/dev/src/Microsoft.AspNetCore.Identity/SignInManager.cs#L185
you rather want to call CheckPasswordAsync
https://github.com/aspnet/Identity/blob/dev/src/Microsoft.AspNetCore.Identity/SignInManager.cs#L253

PasswordSignInAsync calls CheckPasswordAsync internally but also performs other housekeeping tasks. I think then that it is a viable option to call either depending upon the use case.
https://github.com/aspnet/Identity/blob/dev/src/Microsoft.AspNetCore.Identity/SignInManager.cs#L236

You don't want to set a cookie for a resource owner password credential request. That does not make sense - you only want to validate the password.

I am trying to find where in PasswordSignInAsync is the actual cookie set. There is also the SignInAsync method that calls Context.Authentication.SignInAsync. And I think that is what sets the cookies.

Context.Authentication.SignInAsync does.

So it makes sense that if one avoids using SignInAsync but instead calls either PasswordSignInAsync or CheckPasswordAsync, no cookies will be set.

Looking at IS4 Host UI, how is the flow supposed to go for resource owner password credential request? How about the others? That would be useful documentation. Particularly the semantics and usage of LoginController.cs and any other integration point. I know for sure that HttpContext.Authentication.SignInAsync is being called during the MVC and API Sample. How about the other workflows? Client Credentials Grant? Extension Grants? Federated Auth / Google? I know there is more. I am just getting started. Thanks!

Sure - once the code is done we will work on docs.
Here's a starting point:
https://github.com/IdentityServer/IdentityServer4/tree/dev/docs

Thank you both for the extensive feedback. I'll use the CheckPasswordAsync method as suggested.


Problem with list-keys [JIRA: CLIENTS-1015]

I have the following code:
stream = c.ts_stream_keys('EUClaptimes')
for key_list in stream:
   print key_list
# print stream

stream.close()

That gives the following error messages:
Attribute error: ‘str’ object has no attribute ‘name'
If I amend it to:
stream = c.ts_stream_keys('EUClaptimes')
# for key_list in stream:

 #  print key_list
print stream
stream.close()

I do get a stream object
The table name etc exists as I am using it elsewhere on my jupyter notebook.

Checking this now with Riak TS 1.4

Just ran into this too.  The documentation is wrong, and you'll need to pass a Table object in instead of a string for the table name:
table = riak.Table(c, 'EUClaptimes')

stream = c.ts_stream_keys(table1)
for key_list in stream:
    print key_list

stream.close()

Looks like ts_stream_keys() is the only one that is missing this check, it'll need the lines:
        t = table
        if isinstance(t, six.string_types):
            t = Table(self, table)

added to it.

Fixed, or closed via GitHub issues.
[posted via JIRA by Alexander Moore]


ISPN-16417 Fix maxOccurs in XSD schema for index-sharding

https://issues.redhat.com/browse/ISPN-16417

The actual fix is https://github.com/infinispan/infinispan/pull/12766/files#diff-9456ba8256c16859b00c8bf588480c95f20574dcf9ee57bc14189d1afa683b7aL1148
The rest is cleaning up some ambiguity

In case you didn't see, I described a couple of other issues in the XSD in https://issues.redhat.com/browse/ISPN-16417.

Thanks @dalbani , I've addressed the other issues

Failures are unrelated

Merged, thanks!


patch - fix iomodule.c for VS 2017

VS 2017 has a new Windows SDK
iomodule.c needs windows.h to compile
this supplies a patch to add window.h to the module
related
Fix CMake Build for MSVC 15 2017 #233

Do we have stepwise instructions for this task, or should I set aside time to reverse engineer "what to do"?

The idea is to create a branches named X.Y.Z-patches based of the upstream X.Y branch. All relevant patches would be applied to this branch.
Patch added to the python-cmake-buildsystem repo would be manually generated from this branch.


Workspace removed in 4.1.1 even with shallowClone: false

Hi there,
First thanks for offering me a smooth way to leave Capistrano and use JS :)
The https://github.com/shipitjs/shipit/releases/tag/v4.1.1 release - specifically shipit-deploy - doesn't seem to implement https://github.com/shipitjs/shipit/commit/6ba6f0077b5a677aaf5b27bcffb419e142f67178 properly, at least in the lib task:
Config
shipit.initConfig({
    default: {
      repositoryUrl: process.env.DEPLOY_REPO_URL,
      workspace: process.env.DEPLOY_WORKSPACE,
      branch: 'master',
      keepReleases: 5,
      // shallowClone is correctly set to false
      shallowClone: false,
      ignores: [
        '.env',
        'node_modules',
        'node/node_modules',
        'shipitfile.js',
        'bootstrap/cache',
        'public/.htaccess',
        'public/storage',
        'public/robots.txt',
        'storage/app',
        'storage/logs',
      ],
      shared: {
        overwrite: true,
        dirs: [
          'bootstrap/cache',
          'storage/app',
          'storage/logs',
        ],
        files: [
          '.env',
          'public/.htaccess',
          'public/robots.txt',
        ]
      }
    },
    staging: {
      branch: 'master',
      deployTo: process.env.DEPLOY_STAGING_DEPLOY_TO,
      servers: {
        user: process.env.DEPLOY_STAGING_SSH_USER,
        host: process.env.DEPLOY_STAGING_SSH_HOST,
        port: process.env.DEPLOY_STAGING_SSH_PORT
      },
      keepReleases: 1,
    },
  })

Output
Removing workspace "/Users/michel/tmp/deploy/dentalsirh"
Workspace removed.
Finished 'deploy:update' after 15 s

node_modules/shipit-deploy/lib/tasks/deploy/update.js
 let removeWorkspace = (() => {
      var _ref8 = _asyncToGenerator(function* () {
        // I guess the condition should be here, but it isn't.
        shipit.log(`Removing workspace "${shipit.workspace}"`);
        yield (0, _rmfr2.default)(shipit.workspace);
        shipit.log(_chalk2.default.green('Workspace removed.'));
      });

      return function removeWorkspace() {
        return _ref8.apply(this, arguments);
      };
    })();

    (0, _extendShipit2.default)(shipit);

    yield setPreviousRelease();
    yield setPreviousRevision();
    yield createReleasePath();
    yield copyPreviousRelease();
    yield remoteCopy();
    yield setCurrentRevision();
    yield removeWorkspace();

Also I don't know if you're looking for contributors to help or not, but I'm totally in if so.
Cheers,

Hi @superbiche, yes I am always looking for help for Shipit! If you can fix it, please submit a PR!

Hi @neoziro, great, I'm back from vacation, just have a few emergencies to tackle then I'll PR the fix.

Should be fixed in #200

@btkostner indeed but there was no version bump since your fix was merged!
@neoziro could you publish a new version? Thanks


Fix symlink creation

Should fix #23, #89 , #100
According to the docs on windows we should define symlink type

:tada: This issue has been resolved in version 1.1.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cask request: Cloudapp

Cask details
Name:
CloudApp
Homepage:
https://www.getcloudapp.com/
Download URL:
https://www.getcloudapp.com/download/mac
Description:
Screen shot and video sharing software.

https://github.com/caskroom/homebrew-cask/blob/master/Casks/cloud.rb


Camera spins unevenly at <=100 FPS rendering speed

Once the displayed CPU time reaches ~= 10ms, the camera spins at uneven speed for me.
https://github.com/rerun-io/rerun/assets/9785832/e74b2825-828c-4f2b-be40-bb753093de09
Rerun Version

rerun_py 0.12.0 [rustc 1.74.0 (79e9716c9 2023-11-13), LLVM 17.0.4] x86_64-unknown-linux-gnu release-0.12.0 afa112e, built 2024-01-09T17:56:39Z

Desktop (please complete the following information):

Ubuntu 22.04

Maybe also adding here that the same scene didn't have this issue back in 0.8 and that the spinning speed has increased noticeably since then.
https://github.com/rerun-io/rerun/assets/9785832/8a343624-5783-4727-9e64-1ebf91984f8b

@emilk not sure this is a "quick issue". The code hat does the spinning looks like this:
            orbit_eye.rotate(egui::vec2(
                -response.ctx.input(|i| i.stable_dt).at_most(0.1) * 150.0,
                0.0,
            ));

so if high framerates cause issues here it implies that stable_dt isn't working properly

I tried out some more versions and it seems like this has changed from 0.11 -> 0.12.
Each frame takes ~2x as long, the rotation speed is higher, and the speed seems to be inconsistent.

The docs explains it pretty well: https://docs.rs/egui/latest/egui/struct.InputState.html#structfield.stable_dt

Ooh, it seems stable_dt is broken in egui! Will fix :)

Fix coming in https://github.com/emilk/egui/pull/3832


[SPARK-36885][PYTHON][FOLLOWUP] Fix drop subset inline type hint

What changes were proposed in this pull request?
Fix drop subset inline type hint
Why are the changes needed?
it should be same with DataFrame.dropna:
https://github.com/apache/spark/blob/90003398745bfee78416074ed786e986fcb2c8cd/python/pyspark/sql/dataframe.py#L2359
Does this PR introduce any user-facing change?
No
How was this patch tested?
UT

cc @itholic can you review this one please?

Merged to master.


⚠ Bofrost.de has degraded performance

In aff02da, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1297 ms

Resolved: Bofrost.de performance has improved in 5ec0e67 after 15 minutes.


Aria-hidden needed for modals in v3.3.5

Based on W3C rules, non-visible elements should have the aria-hidden=true attribute. A hidden bootstrap modal should have this attribute, and then the attribute removed when it's visible. aria-hidden is not redundant to display:none as stated in this PR. https://github.com/twbs/bootstrap/pull/16158 Screen readers do not interpret display:none.
http://www.w3.org/TR/aria-in-html/

Screen readers do not interpret display:none.

yes, they do. In which screen reader/browser combination is this not the case?

http://www.w3.org/TR/wai-aria/states_and_properties#aria-hidden
If an element is only visible after some user action, authors MUST set the aria-hidden attribute to true.
Note: Authors are reminded that visibility:hidden and display:none apply to all CSS media types; therefore, use of either will hide the content from assistive technologies that access the DOM through a rendering engine. However, in order to support assistive technologies that access the DOM directly, or other authoring techniques to visibly hide content (for example, opacity or off-screen positioning), authors need to ensure the aria-hidden attribute is always updated accordingly when an element is shown or hidden, unless the intent of using off-screen positioning is to make the content visible only to screen reader users and not others.
Apparently any screen reader that doesn't access the DOM through a rendering engine would need make use of aria-hidden.  I don't have enough experience with them to say how prevalent such readers are, although the spec does seem to mandate using aria-hidden rather than relying on display or visibility.

That advice is now fairly outdated. In the upcoming WAI-ARIA 1.1 (currently in working draft) that bit has been removed - see http://www.w3.org/TR/wai-aria-1.1/#aria-hidden

here is start of thread where change in requirement was discussed https://lists.w3.org/Archives/Public/public-pfwg/2014Dec/0035.html

:+1:


Crash in com.google.fira.worker / [FIRAPBMeasurementBundle initWithMessageInfo:]

Xcode version: 8.3.3
Firebase/Auth (4.1.1):
Firebase/Database (4.0.2):
Firebase Product: Auth and Database
So far we can't reproduce the problem. It happens on many devices since we updated to Firebase 4.1.1.
I can only say that we have a lot of other crashes as well and I have no clue why.

FIRAMeasurement
FLeafNode initWithValue:withPriority
FSnapshotUtilities nodeFrom:priority:withValidationFrom:atDepth:path
GAI threadMain
FSnapshotHolder init
FIRInstanceID
FIRASqliteStore

We don't do much with firebase for now. We just auth a user as anonym and write two numbers to the user database. On app launch we read back one of the numbers. So nothing special here. We never could reproduce this problem on our production machines. We even get sometimes users with a NULL as userID.
Attached is the crashlog.
crashlog.txt

The attached crashlog indicates FirebaseAnalytics

Mh, we only use:
pod 'Firebase/Core'
pod 'Firebase/Auth'
pod 'Firebase/Database'
But we ran Google Analytics at the same time. Could this lead to problems?

The Firebase pods have a dependency on FirebaseAnalytics. You'll see it being pulled in if you look at Podfile.lock.
It should be fine to include Google Analytics at the same time.

Ah I see it - thanks! Any idea why we get all those crashes? We have problems with Firebase from the beginning but the last 4.1.1 update brought in even more problems.

We are now getting new crashes from firebase coming in every few minutes.
Latest are:
[FIRASqliteStore recordsForQuery:parameterValues:error:withFilter:]
[FIRASqliteStore endTransaction]
[FSnapshotUtilities nodeFrom:priority:withValidationFrom:]
[FSnapshotHolder updateSnapshot:withNewSnapshot:]
[FSnapshotUtilities nodeFrom:priority:]

thanks for reporting. Can you please provide the content of the Podfile.lock? Thanks

Here is the Podfile.lock
Podfile.lock.txt

@frunny Thanks! Will you confirm that the Podfile.lock.txt attached is the one used for the app build that created the crashlog attached to the initial bug report with the crash in com.google.fira.worker?

I did some pod update meanwhile but there was no update available.
So yes, it's the one with the crashes.

This crash (the one with the stacktrace in the original post) is occurring in some low-level pb-copying logic. It's going to be really difficult to pin down the cause without repro steps or a ton of information, which is why this issue has stagnated a little bit. The exception/crash message and code seem to be missing from the crash log--can you post them here? By crash message I mean whatever the mach message or signal is (i.e. EXC_BAD_ACCESS/SIGABRT) and the code shoul dbe some hex number next to the message (i.e. 0x8badf00d).

@frunny is this crash still occurring in newer versions of Firebase?

Those are separate crashes in Firebase Database, tracked at #281 (and others).
I'm closing this issue, since it seems to be resolved.

This bug still happens to me.
This is my stacktrace:
Crashed: com.google.fira.worker 0  libsystem_platform.dylib       0x18f4b0e88 _platform_memmove + 296 1  SpadesKingLite                 0x101955464 pb_field_set_to_default (pb_decode.c:808) 2  SpadesKingLite                 0x101955464 pb_field_set_to_default (pb_decode.c:808) 3  SpadesKingLite                 0x101954c28 pb_message_set_to_defaults (pb_decode.c:845) 4  SpadesKingLite                 0x101954bdc pb_decode (pb_decode.c:959) 5  SpadesKingLite                 0x102777510 firapb_decode_measurement_config + 1952580 6  SpadesKingLite                 0x10277b76c -[FIRAPBMeasurementConfig initWithBuffer:] + 1969568 7  SpadesKingLite                 0x102742d84 -[FIRADatabase queryMeasurementConfigWithError:] + 1737656 8  SpadesKingLite                 0x10275f770 -[FIRARemoteConfig measurementConfigFromDatabase] + 1854884 9  SpadesKingLite                 0x10275dde8 -[FIRARemoteConfig initWithGMPAppID:database:measurementConfig:] + 1848348 10 SpadesKingLite                 0x10275db58 -[FIRARemoteConfig initWithGMPAppID:database:] + 1847692 11 SpadesKingLite                 0x10274a71c __48-[FIRAMeasurement startMeasurementOnWorkerQueue]_block_invoke + 1768784 12 libdispatch.dylib              0x18f2ae9a0 _dispatch_client_callout + 16 13 libdispatch.dylib              0x18f2af6cc dispatch_once_f + 56 14 SpadesKingLite                 0x10274a340 -[FIRAMeasurement startMeasurementOnWorkerQueue] + 1767796 15 SpadesKingLite                 0x10274a10c -[FIRAMeasurement setEnabledOnWorkerQueue:] + 1767232 16 SpadesKingLite                 0x102760454 __52-[FIRAScheduler scheduleOnWorkerQueueBlockID:block:]_block_invoke + 1858184 17 libdispatch.dylib              0x18f2ae9e0 _dispatch_call_block_and_release + 24 18 libdispatch.dylib              0x18f2ae9a0 _dispatch_client_callout + 16 19 libdispatch.dylib              0x18f2bcad4 _dispatch_queue_serial_drain + 928 20 libdispatch.dylib              0x18f2b22cc _dispatch_queue_invoke + 884 21 libdispatch.dylib              0x18f2bea50 _dispatch_root_queue_drain + 540 22 libdispatch.dylib              0x18f2be7d0 _dispatch_worker_thread3 + 124 23 libsystem_pthread.dylib        0x18f4b7100 _pthread_wqthread + 1096 24 libsystem_pthread.dylib        0x18f4b6cac start_wqthread + 4
More info:
issue_crash.txt
This is my Pod files:
Podfile.txt
Podfile.lock.txt

This is a slightly different crash, but also occurs in nanopb. Try updating your nanopb pod to the latest version (0.3.8).


0.7 automation

from 0.7 to 1.0.0 I have an automation that sends out amessage when the washmachine is finish it looks like this
id: Tvattnotis
alias: Tvättnotis
description: ''
trigger:

platform: state
entity_id: sensor.bosch_wau28pehsn_68a40e5d595e_bsh_common_status_operationstate
from: Running
to: Completed
condition: []
action:
service: tts.speak
data:
media_player_entity_id: media_player.hemgrupp_2
message: Hej! Vill bara meddela att tvätten är färdig och det är nu dags att hänga upp den på tork
target:
entity_id: tts.google_sv_se
mode: single

but now in 1.0.0 it won't work anymore so whats the new From & to?

You probably had server translation enable in the config.  In v. 1.0.0 you need to select that translation mode in the integration configuration options in the UI.

You probably had server translation enable in the config. In v. 1.0.0 you need to select that translation mode in the integration configuration options in the UI.

yes I have that in configuration.yaml with the client ID & client secret , so just remove the other and fix it in the UI? but keep client ID & client secret?


scheduled function trigger fires every 2 mins, if it does not get a response.

Is this a BUG REPORT or FEATURE REQUEST?: Not sure (might be some config i missed)
What happened: The cronjob triggering the scheduled function every 5 min, waits for 2 minutes for a response from http://.default.svc.cluster.local:8080. If the response takes more time that that, a new trigger is fired immediately in the cronjob. I have set --timeout=180, so I expect the function to complete its execution (including 2 minute sleep + some logic), and return the response to the req in the handler function. But the trigger job pod terminates the curl request before the res could be sent.
What you expected to happen: The trigger pod should wait for timeout seconds for a response from the actual function pod, failure of which should report cronjob failure .
How to reproduce it (as minimally and precisely as possible): Running a nodejs function with a scheduled trigger. I upload the function through a zip file containing all the function code.
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version): 1.8.7
Kubeless version (use kubeless version): 0.3.4
Cloud provider or physical cluster: Azure AKS

Hi @tapanhalani,
I am not able to reproduce your issue, I am seeing that the pod generated by the job remains waiting for a response at least 3 minutes:
$ kubectl logs -f trigger-infinite27-1519660200-jcxth
* Rebuilt URL to: http://infinite27.default.svc.cluster.local:8080/
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to infinite27.default.svc.cluster.local (<IP_ADDRESS>) port 8080 (#0)
> GET / HTTP/1.1
> Host: infinite27.default.svc.cluster.local:8080
> User-Agent: curl/7.56.1
> Accept: */*
>
  0     0    0     0    0     0      0      0 --:--:--  0:02:59 --:--:--     0< HTTP/1.1 408 Request Timeout

What is the output of the pod that gets generated in your case? You can locate the pod that is running the curl executing kubectl get pods -a. It should be named trigger-<your-function-name>-XYZ
Also, are your jobs being terminated? What is the output of executing the following?
$ kubectl get jobs -w
NAME                            DESIRED   SUCCESSFUL   AGE
trigger-infinite27-1519661400   1         0            12m
trigger-infinite27-1519662000   1         0            2m

Basically the problem is not necessarily function failure, but the jobs are not following the schedule . Say a function is completing successfully in 2 min, and is scheduled every 5 min, then in my case, after completing at 2 min, it will not wait , and immediately fires a new trigger, with the previous trigger pod showing "Terminated with error" and the error log as shown below.
The output of pod trigger-kafkaendtoendmonitoring-1519715700-zxh2m  is as follows.
* Rebuilt URL to: http://kafkaendtoendmonitoring.default.svc.cluster.local:8080/
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to kafkaendtoendmonitoring.default.svc.cluster.local (<IP_ADDRESS>) port 8080 (#0)
> GET / HTTP/1.1
> Host: kafkaendtoendmonitoring.default.svc.cluster.local:8080
> User-Agent: curl/7.56.1
> Accept: */*
>
  0     0    0     0    0     0      0      0 --:--:--  0:02:00 --:--:--     0* Empty reply from server
  0     0    0     0    0     0      0      0 --:--:--  0:02:00 --:--:--     0
* Connection #0 to host kafkaendtoendmonitoring.default.svc.cluster.local left intact
curl: (52) Empty reply from server

The output of "kubectl get jobs" is like this:
NAME                                            DESIRED   SUCCESSFUL   AGE
trigger-autoassigntrainingstousers-1519715400   1         0            12m
trigger-autoassigntrainingstousers-1519716000   1         0            2m
trigger-groupservice-1519714800                 1         0            22m
trigger-kafkaendtoendmonitoring-1519635900      1         1            22h
trigger-kafkaendtoendmonitoring-1519636200      1         1            22h
trigger-kafkaendtoendmonitoring-1519715700      1         0            7m
trigger-kafkaendtoendmonitoring-1519716000      1         0            2m
trigger-offerservice-1519715400                 1         0            12m
trigger-offerservice-1519716000                 1         0            2m
trigger-sessionservice-1519716000               1         0            2m
trigger-sessionservice-1519716120               1         0            9s
trigger-userenrolledeventservice-1519715400     1         0            12m
trigger-userenrolledeventservice-1519716000     1         0            2m

So the jobs are terminating successfully.

So apparently this is not a problem with the timeout but with the function not returning a response (probably due to an error during the function execution). When a job fails, the default behavior is to retry it until it reaches the maximum timeout (set by the --timeout flag) that's why you are seeing a new job being triggered.
Does the function log shows any information? (You can retrieve it executing kubectl logs -l function=kafkaendtoendmonitoring)
You can also try to run the curl command manually, this should succeed in order to have a successful job execution:
▶ kubectl run test --rm -it --image kubeless/unzip@sha256:f162c062973cca05459834de6ed14c039d45df8cdb76097f50b028a1621b3697 --command sh
If you don't see a command prompt, try pressing enter.
/ # curl -v kafkaendtoendmonitoring.default.svc.cluster.local:8080

The function log does not show any errors. I am doing res.end('Function completed successfully') at the end of the function. Also, I ran the "kubectl run" command and did the curl inside, with the same result "Empty reply from server". I am not sure what I might be doing wrong.

Closing this issue since it appears to be a function issue. Please let us know if we should reopen it.


scene.pick get wrong result at same position

I‘m  trying to get entity by scene.pick。 I found that sometimes it get wrong result.
i use code like:
  setInputAction(function(event) {     console.log(event.position)     let model = viewer.scene.pick(event.position);     if (model && model.id) {       console.log(model.id.id) } },  ScreenSpaceEventType.LEFT_DOWN)
In the map, i clicked green position three times

then i got three different entity:

Is there something I did wrong?

Hi @QianWu184 - are you able to share a Sandcastle that reproduces this issue?

Hi @QianWu184 - are you able to share a Sandcastle that reproduces this issue?

Thanks your attention,but i'm afraid i can't provid Sandcastle to you（It's too long. Basically, it's building 100 entities, arranging them like a grid, and trying to get one of them by LEFT_DOWN）.And is there any possible reason for this problem?

@QianWu184 This doesn't seem like correct behavior but it's tough to diagnose the issue.. I was unable to reproduce the behavior in this Sandcastle with multiple models arranged in a line.
Are you able to identify which entities are being picked when you are clicking?

@QianWu184 This doesn't seem like correct behavior but it's tough to diagnose the issue.. I was unable to reproduce the behavior in this Sandcastle with multiple models arranged in a line.
Are you able to identify which entities are being picked when you are clicking?

In fact, each Entity is just a polygon geometry. I just want to click and drag the Entity where the green dot is located. But the results obtained through viewer.scene.pick the previous few times are always wrong. For example, in the image above, I want to get the corresponding Entity by clicking on the green position. But these three numbers indicate the three different results when I clicked on the green dot.The yellow background is just the effect I made to confirm the current actual selected result。

@QianWu184 This doesn't seem like correct behavior but it's tough to diagnose the issue.. I was unable to reproduce the behavior in this Sandcastle with multiple models arranged in a line.
Are you able to identify which entities are being picked when you are clicking?

This problem does not seem to exist when the Entity has only one row or I click on the edge of the Entity. But if you arrange them into multiple rows and click on the Entity in the middle, this problem seems to occur.

Here's a different Sandcastle that has 100 entities in a 10 x 10 grid.  What are you doing differently?  I'm testing on Windows 10 using Chrome.

Thanks @GatorScott - in macOS/Chrome your Sandcastle works for me as well.

Here's a different Sandcastle that has 100 entities in a 10 x 10 grid. What are you doing differently? I'm testing on Windows 10 using Chrome.

This can also run successfully here. Maybe I should check if my  code has offset the coordinates

Hi @QianWu184. In my work, i encounter the same problem which you stated. Have you solved this problem yet?


PR to resolve issue #1140 - AttributeError: module 'tensorflow._api.v2.train' has no attribute 'Optimizer'

This PR has a minor modification to remove the error due to tensorflow version (>=2) incompatibility for optimizer.py.
Due to deprecation of  'tf.train.Optimizer', an improvement is made which allows the optimizer to run smoothly with tf version>=2(tested locally with 2.3.1) .
Suggestions are appreciated.Thanks

Changes:

Provided python 2 support for _run_split_on_punc  (tokenization.py)
Provided tf2  options in modeling.py and modeling_test.py

:shipit:

any expectations for when this patch will be merged?

Changes:
1. Provided python 2 support for _run_split_on_punc  (tokenization.py)

2. Provided tf2  options in modeling.py and modeling_test.py

For tensorflow 2.16.x, python throws an error message when calling modeling.BertModel()
AttributeError: Tensor.name is undefined when eager execution is enabled.

Essentially, the issue is with name = tensor.name on line 911 in modeling.py.


Add Fatuma file to interested tech writers directory

This PR:

adds a markdown file with my information to the interested technical writers directory.

Hi Fatuma! Unfortunately your bulleted lists will break the table. Please reformat into running text.

Hi,
Thanks for the heads up.
It should be fixed now :)


No Videos Found!

This only started happening today but I can't download any videos. It just gives me an error on any link (triple checked to make sure they're right) saying "No videos found!"
I need to figure out how to get these vods downloaded as it's my job to get videos edited for the client

Also, the newest version does not work. It just open the link on chrome.

Any other alternatives? my client need to have videos done by tomorrow

Kraken API v5 being removed.
Lots of Twitch extensions stopped working. Until everything isn't updated to the new API nothing will work.

@Abalieno What are you talking about? TL 2.0.2 was released exactly for that. There are no V5 dependencies. And since TL 2.0.2 works perfectly I must assume that you are on an old version of TL.
https://blog.twitch.tv/en/2021/07/15/legacy-twitch-api-v5-shutdown-details-and-timeline/

February 7, 2022 – 1-hour shutdown starting at 11:00am PDT
February 9, 2022 – 2-hour shutdown starting at 11:00am PDT
February 14, 2022 – 4-hour shutdown starting at 11:00am PDT
February 16, 2022 – 6-hour shutdown starting at 11:00am PDT
February 21, 2022 – 24-hour shutdown starting at 11:00am PDT
February 28, 2022 – v5 permanent shutdown

This is what's going on right now 😄

My bad then, I haven't checked a new version. But I started seeing problems on other things and here too it was being reported for the same timeframe, so I thought it was related.
Good to now it's not affecting newer TL

At least they are consequent and on time. I honestly did not expect that 😄


Whisper

Feature Request
Is your feature request related to a problem? Please describe.
Support OpenAI Whisper
Describe the solution you'd like
OpenAI has released Whisper and I would like this library to support it
Additional context
I am willing to submit a PR

@ADefWebserver Just got back into the office today. I might have it done before then tbh


More specific imports in config.ts

@ManuCutillas no worries! Btw, are you thinking of updating the npm package anytime soon? Thanks!

@angelnikolov, today I will try to upload it at npm.

@ManuCutillas  any update on this? Thanks!

@angelnikolov , Sorry, I've had a lot of work and I have not been able to update the npm package. This weekend I update npm. And this time will be true, sorry for the inconvenience. ; )

@angelnikolov Updated with "ng2-responsive": "^0.8.4". Thanks Angel.

Thanks!

@ManuCutillas it seems that my changes have been reverted in your last commit?


Close Dialog programatically

Hello, first of all. Thank you for this amazing package. It has helped me provide an elegant and beautiful alerts to my app.
I have faced a situation where I need to close the dialog dynamically, after  certain action. For example, when fetching data from an API is complete, I would want to close the "loading" dialog. I have seen the 'autoCloseDuration' option but there's really no way of knowing how much time such an action would take.
Kindly add the feature to close the dialog after an action. For example:
var alert = QuickAlert(options);
if(condition) {
alert.show();
} else {
if(alert.isOpen){
alert.close();
}
}

same problem
i want to close alert and start another

I found out that using Navigator.of(context, rootNavigator: true).pop(); closes the dialog. But I believe it would be better if the package had a close method

Hello @bennito254 & @kururu-abdo 👋
Thank you for using the package and your love towards the package. This enhancement has been already added to the current sprint of feature upgradations. Will be out with an update soon.
I used to handle these type of cases like this:
QuicklAlert.show(options); // Loading Alert Launched
....
// Do other asynchronous actions
.....
Navigator.pop(context); // Pop out the Loading Alert

We can do same thing multiple times before we return a value (if working in a function) or change the state.
Till the new update rolls out, help yourself doing the same or any other workarounds.
Happy Coding ✨

Thank you and Happy Coding

I found out that using Navigator.of(context, rootNavigator: true).pop(); closes the dialog. But I believe it would be better if the package had a close method

Many thanks, I can also close the message box with this code.

Hello @bennito254 & @kururu-abdo 👋 Thank you for using the package and your love towards the package. This enhancement has been already added to the current sprint of feature upgradations. Will be out with an update soon.
I used to handle these type of cases like this:
QuicklAlert.show(options); // Loading Alert Launched
....
// Do other asynchronous actions
.....
Navigator.pop(context); // Pop out the Loading Alert

We can do same thing multiple times before we return a value (if working in a function) or change the state.
Till the new update rolls out, help yourself doing the same or any other workarounds. Happy Coding ✨

Hello!
Is this update abandoned?


User management

Hi @waifung0207,
You did an awesome work. Congratulations. I have an issue if it is possible.
I have codeigniter membership script which has only one login page for users and admin users.
On my script when you type http://localhost/codeigniter/admin you go nowhere. But if you type this after logged in as admin from common login page, you can access to admin page. This is a very good idea to hide admin panel from public.
Another issue is login with social media. There is a library named hybridauth. Is it possible to add this library to ci_bootstrap_3 if not I will give it a try.
Again thank you very much for this very awesome work.

Thanks for raising the issue.
I understand the concept of single Login page to hide Admin Panel from public, but currently I use separate tables for Frontend & Admin users (so you will see 8 database tables are created, each modules have 4 tables due to Ion Auth) which I think have better security; in addition I use slightly different setup for Frontend users (login by email) and Admin users (login by username), so I would rather keep the login page apart. But of course you can customize the setup according to your own usage.
Alternative method to "hide" the Admin Panel may be: just rename the admin module with random number (e.g. admin5252798) which is used in some other tools / frameworks (like PrestaShop as far as I know).
Social login is not planned to add into the repo yet, but I would keep it in mind and perhaps someday I will consider it.
Thanks again :)


CB-17555: Remove the usage of Entitlement CDP_DATALAKE_BACKUP_ON_RESIZE

Currently backup on resize is under an entitlement. This might get us into a bad situation if the customer is not granted this entitlement but is granted entitlement to resize.
Resize is performed but the data will be lost. Having an entitlement would be of value if we have a validation to check if there was a backup that is available to restore from.
As such logic is not available we should not have a situation where back might be skipped.

I'm finding it hard to understand the situation where we have accidentally ended up with the configuration you described. The purpose for the entitlement was to allow a customer to disable a backup in the situation where they didn't want to take the backup ahead of the resize, because they already have that backup. We'd want to keep this as an option correct?


Not working with MotionJOY sync

I am able to connect and sync my PSmove  controller with MotionJoy using below BT dongle.
Motionjoy loads its own driver and replaces the BT driver. this way any BT dongle will work.
unable to recognize the move controller wireless Using either below BT dongle its driver or motionjoy.
Adapter MAC f8.8d.50.dd.9.0
Manufacturer Infineon Technologies AG
HCI Version 6.1d86
LMP Version 6.1d86

i was able to solve my issue by removing all the drivers i could find for my BT dongle and let windows10 handle it. i had to repeat this couple of times

It sounds like we'd have to do some custom work to interface with the MotionJoy driver. We currently just use the windows bluetooth stack directly for controller pairing and HIDAPI for getting data to and from the controller. Is there any reason you need to use the MotionJoy driver with the PSMove controller? Perhaps there is some bit of functionality we could add to PSMoveService instead?


Add filters to allow range searches for coordinateUncertaintyInMeters

Currently blocked by https://github.com/gbif/occurrence/issues/152

This was deployed 3 weeks ago, sorry, I forgot to tell everyone.
http://api.gbif.org/v1/occurrence/search?coordinateUncertaintyInMeters=0,100
http://api.gbif.org/v1/occurrence/search?coordinateUncertaintyInMeters=0,*
http://api.gbif.org/v1/occurrence/search?coordinateUncertaintyInMeters=100000,*
Could you also update the API documentation in the portal?


Easy way to create a Kaggle dataset from a Huggingface dataset?

I'm not sure whether this is more appropriately addressed with HuggingFace or Kaggle.  I would like to somehow directly create a Kaggle dataset from a HuggingFace Dataset.
While Kaggle does provide the option to create a dataset from a URI, that URI must point to a single file.  For example:

Is there some mechanism from huggingface to represent a dataset (such as that from load_dataset('wmt14', 'de-en', split='train') as a single file?  Or, some other way to get that into a Kaggle dataset so that I can use the huggingface datasets module to process and consume it inside of a Kaggle notebook?
Thanks in advance!

Hi @hrbigelow , I'm no expert for such a question so I'll ping @lhoestq from the datasets library (also this issue could be moved there if someone with permission can do it :) )

Hi ! Many datasets are made of several files, and how they are parsed often requires a python script. Because of that, datasets like wmt14 are not available as a single file on HF. Though you can create this file using datasets:
from datasets import load_dataset

ds = load_dataset("wmt14", "de-en", split="train")

ds.to_json("wmt14-train.json")
# OR to parquet, which is compressed:
# ds.to_parquet("wmt14-train.parquet")

We are also working on providing parquet exports for all datasets, but wmt14 is not supported yet (we're rolling it out for datasets <1GB first). They're usually available in the refs/convert/parquet branch (empty for wmt14):


Prepare cred provider utils in packaging-common for NuGetAuthenticateV1

Task name: NuGet Authenticate / packaging-common
Description: Preparing packaging-common for NuGetAuthenticate changes. First PR for the changes in this PR that must be split: https://github.com/microsoft/azure-pipelines-tasks/pull/15993
Documentation changes required: Docs PR: https://github.com/MicrosoftDocs/azure-devops-docs-pr/pull/4462
Added unit tests: (Y/N) N
Attached related issue: (Y/N) N
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

/azp run


Cannot read property 'getLanguages' of undefined

My code is from the demo
import React, { Component } from 'react';
import {
    Text
} from 'react-native';
import I18n from 'react-native-i18n';

class Demo extends React.Component {
    render() {
    return (
      <Text>{I18n.t('greeting')}</Text>
    );
  }
}

// Enable fallbacks if you want `en-US` and `en-GB` to fallback to `en`
I18n.fallbacks = true;

I18n.translations = {
  en: {
    greeting: 'Hi!'
  },
  fr: {
    greeting: 'Bonjour!'
  }
};

export default Demo;

I am getting

Thank you in advance! :)

I think you need to install react-native-i18n as well as ex-react-native-i18n

@SvetoslavSlavov Is the package correctly linked?

I got this error when I forgot to restart the app after installing and linking the package. Solved by running again.

It Worked perfectly for a long time.I'm experiencing g this error suddenly. How to resolve this


Promote ReferenceGrant to Beta

What would you like to be added:
Based on discussion in recent community meetings, we believe that ReferenceGrant has met the required graduation criteria to graduate to beta. This will require updating the types to match what we've done for GatewayClass, Gateway, and HTTPRoute, with one exception: The storage version will need to stick to v1alpha2 in this initial release.

@robscott I can take this one

Thanks!
/assign @nathancoleman

@robscott is the plan to move ReferenceGrant into the standard channel as well?

@nathancoleman yep, resources that have graduated to beta should also be part of the standard channel.


Generate PDF using puppeteer

Pro

PDF rendering is ready with almost zero styling adjustment.

Con

Puppeteer does not work on Vercel.
Also does not work with Docker.

Alternative: #26.

Superseded by #26.