Problem with automatic_optimization=False and terminate_on_nan=True

🐛 Bug

When automatic_optimization = False and terminate_on_nan = True, an exception is raised when checking for nan values. This is due to None being passed in as the value for loss to self.detect_nan_tensors. It looks like the code on master has already changed from what I'm seeing in 1.0.3, so I don't know if this has somehow been fixed or not. The problem seems to be that the AttributeDict returned from train_step has loss=None.
Please reproduce using the BoringModel and post here

https://colab.research.google.com/drive/1qQmP6BwQk--rBXC7W45y0mn6QK39IPcc
Expected behavior
Don't crash when automatic_optimization = False and terminate_on_nan = True

I discovered this because the loss was showing up as nan in the progress bar, and I was trying to figure out why I was getting nan. I dug some more, and it looks like this is itself a bug. I've inspected the loss and network parameters over several steps, and there are no nans. So there seems to be a problem in the logging somewhere, that if you're using automatic_optimization=False you get nan being logged as the loss in the progress bar.

Thanks @catalys1 you are correct, however recent changes should have resolved this issue since the nan check only runs if using automatic optimization:
https://github.com/PyTorchLightning/pytorch-lightning/blob/master/pytorch_lightning/trainer/training_loop.py#L779-L789
In #4204 we'll make it clearer that you should report values within the training step via the docs :)

Hi @catalys1,
in def training_step you can maybe overpass the nan issue by updating the running_loss directly:
self.trainer.train_loop.running_loss.append(loss)
in my case, no more nan whenautomatic_optimization=False


Proposal: Create a website enabling users to check and share their NFTs from their wallets, as well as airdrop to others in the community based on their NFT ownership rights.

Hello! I want to create a website that provides a convenient way for users to check which NFTs they own in their wallets.
With just a few clicks, users can connect their wallets to my website and instantly see a list of the NFTs they are holding. Additionally, I support members of my community in organizing airdrops based on their NFT ownership rights.
Whether it's sharing the latest collectible versions or tokens with friends or rewarding active members in the community, our platform supports easy and secure NFT transfers. Explore and join the vibrant world of digital collectibles!

https://github.com/klaytn/klaytn-dapp-toolkit/issues/18
You can continue building on your previous DApp and earn the bounty.


change action result execute to return void promise

Coverage remained the same at 100.0% when pulling 8ca33fb7f3a54a7a66c9e974f6b2eaec9e1aa405 on ktutnik:master into 4930f5cb07d8091aab3a8229736560659f043586 on kambojajs:master.


DepthwiseConv2D is slower than Conv2D

I saw the similar issue in
https://github.com/tensorflow/tensorflow/issues/42172
As in this post, when the batch size is larger, DepthwiseConv2D can be faster than Conv2D.
However, if I have to set the batch size to 1, is there any method to speed up DepthwiseConv2D?

I saw the similar issue in
#42172
As in this post, when the batch size is larger, DepthwiseConv2D can be faster than Conv2D.
However, if I have to set the batch size to 1, is there any method to speed up DepthwiseConv2D?

The version I used is tensorflow 2.3.1

I saw the similar issue in
#42172
As in this post, when the batch size is larger, DepthwiseConv2D can be faster than Conv2D.
However, if I have to set the batch size to 1, is there any method to speed up DepthwiseConv2D?

The version I used is tensorflow 2.3.1

@cookingbear What platform are u running with that ?
Are u running using gpu/cuda, or pure CPU. ARM  or x86 platform ?

@cookingbear What platform are u running with that ?
Are u running using gpu/cuda, or pure CPU. ARM  or x86 platform ?

I tested it on x86 using cpu only

I tested it on x86 using cpu only

@cookingbear Is this still an issue for you? Can you please test with recent TF2.6 and tf-nightly and let us know whether the issue persists. Here is a gist for reference. Thanks!

@cookingbear Is this still an issue for you? Can you please test with recent TF2.6 and tf-nightly and let us know whether the issue persists. Here is a gist for reference. Thanks!

thanks for your attention. I would test again with the newest version.


Issue # 306 : Sourcing org admin is navigated to textbook tab on click of back button available in view details page of nomination user.

Need to work


Display only uncorrectable offenses

Is your feature request related to a problem? Please describe.
In my project, most offences are automatically corrected by github action. I need to display to a developer only offences that are not automatically correctable without actually fixing them because I don't want to let developer know about things he doesn't need to know.
Describe the solution you'd like
Add --display-only-uncorrectable console option that would only display offences that can not be corrected.
Describe alternatives you've considered
I tried --format=json and filter the offences manually via files[].offences[].correctable but it requires too much formatting code on top.  It is possible though to implement --filter option that would allow for some flexible filtering like jq

This seems pretty specific. Could you instead do with a custom formatter?
class CustomFormatter < RuboCop::Formatter::ClangStyleFormatter
  def file_finished(file, offenses)
    offenses = offenses.reject(&:correctable?) unless ENV["CI"]
    super
  end
end

Adding --format CustomFormatter to .rubocop will then automatically use it. You also need to add that file to the require section of .rubocop.yml for it to work.

The following solved my problem:
# frozen_string_literal: true

require 'rubocop/formatter/json_formatter'
require 'rubocop/formatter/clang_style_formatter'
require 'rubocop/formatter/progress_formatter'

module SkipCorrectableRubocopFormatter
  def file_finished(file, offenses)
    offenses = offenses.reject(&:correctable?)
    super
  end
end

[
  RuboCop::Formatter::JSONFormatter,
  RuboCop::Formatter::ClangStyleFormatter,
  RuboCop::Formatter::ProgressFormatter,
].each do |klass|
  klass.send(:prepend, SkipCorrectableRubocopFormatter)
end

Making sure all formatters used in certain env do not output the correctable offences.
I would say the better idea would be to have --filter CustomOffencesFilter separated from --format. But I will leave it for core team to decide.

Yeah, it isn't as nice as it could be. If something does get added, I would personally go with --display-only-uncorrectable instead of something new alltogether.
Perhaps all the other --display-only- can be consolidated under a single switch like --display-only so that you can write --display-only=correctable/--display-only=uncorrectable,etc. (not sure, I think all these should be exclusive to each other). It's getting a bit much.


Clean up the crypto parameters

Remove exra parameters and normalize the names, like depth and maxDepth

Edit the comments for GetMaxRelinSkDeg

Looks like depth and maxDepth have already been removed.


Typo in Japanese translation for report_issue_action

Expected Behavior

A correct Japanese translation for report_issue_action would be "問題やバグを報告する".
Current Behavior

app/localizations/ja.json defines it as "問題やハグを報告する"
Possible Solution

The Japanese word for "bug" is "バグ", not "ハグ". So it would be sufficient to edit ja.json.
Context

Code Sample

Your Environment

Software
Version(s)

Package

Runtime

Package Manager

Operating System

Fixed in 1.1.5.


Traceur overwriting environment Object.keys

It seems that when requiring Traceur in Node, the runtime will always overwrite the native Object.keys, which isn't ideal.

It is done for symbols and map/set polyfills. We should add a runtime check before doing these overrides.


Check if arcore is supported.

When checking device, that does not support arcore it returns SupportedNotInstalled, instead of UnsupportedDeviceNotCapable.

Which phone are you testing on? You get SupportedNotInstalled if the phone is supported by the ARCore apk isn't installed.

Xiaomi mi a1, I get SupportedNotInstalled, then proceed to start AR session and arcore says that I need to install arcore, redirect to arcore play market page, which says your device is not supported.

@inio Any idea?

You’re right, it should.  That doesn’t make sense.  If you could capture a logcat that would be helpful.

Same issue for Asus ZenFon3, with Android 7. Get from the checkAvailability(this) SUPPORTED_NOT_INSTALLED but when opening arcore in the playstore to install, cant install and get the message "your device isn't compatible with this version".

Close the issue due to lack of response. Please reopen if the issue still exist.


[GSSoC'23] New project - Ping Pong Score Keeper using HTML, CSS and Vanilla JS

Hi, I'm a contributor in GirlScript Summer of Code'23 and I would like to contribute to this repository by adding a project of my own, which is a ping-pong score keeper

assigned to u and In PR include deploy link or screenshot to assess your work. Dont forget to follow contributing.md guidelines after reviewing PR by mentor

Okay got it

@Dharmesh177 How much time do I have to make a PR?

before Ending time of GSSOC23 you can complete it (nearly 2-3 month)

Oh okay, thank you so much

It is advisable to complete it with in 10-15 days

Cool


Consider relaxing Unpin bound on Sink impl for FramedWrite

It appears that the Sink trait implementation for tokio_util::codec::FramedWrite requires only AsyncWrite and not AsyncWrite + Unpin. Is there a chance that futures_codec::FramedWrite could do the same?
As of today, I cannot invoke StreamExt::forward() on a futures_codec::FramedWrite which wraps a non-Unpin type.

I tried to restructure and refactor this crate, WIP version @ https://github.com/YZITE/futures-codec

For future readers: since the above repo no longer exists, there is now a similar crate at silvanshade/async-codec-lite.

ah sorry, moved it here: https://github.com/YZITE/futures/tree/main/crates/codec


Reload overlays on hash group change.

The hashes are grouped into arrays dependent on whether or not the
overlay has to refresh/change when they are triggered. So from settings
-> administration, nothing needs to change since they’re in the same
group/overlay. However going from administration -> subscriptions will
run the function to display the correct overlay.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

@brockwhittaker this seems to fail the casper tests; can you investigate?

This was merged via the settings redesign branch.


feat: self-update for default pip installs

Summary
This is a proof of concept for updating uv installations that were installed using the system default pip. e.g. pip install uv.
Code is a little rough right now and is not implemented for pipx installations. I wanted to first discuses this implementation before going forward. So thorough review is appreciated!

Test Plan
I was unsure how to make a unit test for this but currently tested using the following commands:
pip install .
uv self update

Possible related to #2338 and #2386


Port patches for upstream cluster-api to v1beta1

Port all the patches made to upstream cluster-api to v1beta CRDs and new controller’s code (build on top of latest patch of v1.0.1)

Ported all BR+etcd patches to v1.0.1 tag to this branch https://github.com/mrajashree/cluster-api/commits/branch-1.0.1
Going forward we'll use this fork to create patches and use those in build-tooling repo


Fix Retrieve id issues

Once a user copies the text and submits it, they will get a retrieve id, that's great but once the same user still copies the second text and submit it, they will get a new id, the old one vanished, my suggestion would be to show a list of retrieve ids, so the user has all the ids of all text they have submitted, so they can easily retrieve the text they want.
I want to add that fix to the project.

@jeswanthmukesh20  kindly assign this issue to me 🚀


Reads and writes do not update the position of a buffer

Hi,
While both reads and writes update a ByteBuffer's position (at least of that from java.nio), this doesn't seem to be the case when reading from and writing to a buffer created by buf/allocate.
Example:
=> (require '[octet.core :as buf])
=> (def b (buf/allocate 24))
=> b
#object[java.nio.HeapByteBuffer 0x54bf0a6b "java.nio.HeapByteBuffer[pos=0 lim=24 cap=24]"]
=> (buf/read b (buf/byte))
0
=> b
#object[java.nio.HeapByteBuffer 0x54bf0a6b "java.nio.HeapByteBuffer[pos=0 lim=24 cap=24]"]

If this behavior is intended, then it should at least be documented somewhere.
If I have missed something, because so far I've only skimmed through the documentation, then please let me know.
Edit:
Judging from the code at https://github.com/funcool/octet/blob/master/src/octet/buffer.cljc#L178-L190,
this seems intentional. It would be good to add this information and the reason for this to the documentation, as it may surprise other people as well.

Hi @tkriik , you are judging well, this is intentional.
Regardless that the library is based on impure mutable data, the library uses at less surprise principle and does not performs unexpected mutations.
I know that java.nio api  by default mutate the position, but it also offers method for do it without mutation; and octet is not a api for java.nio (in fact it works with nio, netty and typed arrays in js), so expecting that it will work in the same way is a wrong assumption.
In any case, I agree that a little mention on the documentation is not going to do any bad.


Update hydrate-reminder

Are you down bad and feeling extremely thirsty?
Fear not my fellow adventurers for hydrate reminder v1.1.0 is here!

Your build.gradle changes aren't building because you've changed all the versions to ones which we don't have permitted in our dependency verification. I recommend you reverse those changes.


Update dependency flutter_colorpicker in pubspec.yaml

Hi 👋
according to this  upstream issue a update to 5.0.0 of the dependency flutter_colorpicker might solve a issue caused by new(er) Flutter versions.
I'd be very grateful if you could look into this any time soon.
Just wanted to say that this is a great package 👍 I'm currently learning about Flutter and really love it.
Regards, Jean-Luc

Great, glad you like it. And thanks for opening this PR. Does the sample app included in the package still work as intended after upgrading?

Thank you for your fast anwser. Sorry my fault I did not tested the sample. Hope it works.

Hi! I'm encountering this issue trying to use this package. Is there something I can do to help move this PR along to fix it?
Thanks.

Hey, yeah I'm happy to have someone jump in and fix this. I think you just need to update any of the packages in the pubspec and make sure the sample app still works as intended... Meaning that you can dynamically change the color and flip light and dark.


[WIP] Cache domain searches per session

Cache the searched instances and methods in a single session.

What clears the cache?  On that note, what does this actually save?  database queries are cached at the session level already, so I would expect that if the code were asking for records, then this wouldn't benefit from anything.

@mkanoor What is the current thinking with this one?  Do we want to close for now and re-open later if needed or is there still value in pursuing it?

Closed Temporarily


[FEAT] biometric for sending

has anyone tested this on android?

https://github.com/bitpay/bitpay-app/pull/271


Certificate issue

A scan (see here for details and background) just revealed the APKs at your releases are signed using a debug key. As that has security implications, may I ask you to please switch to a proper release key, and provide the corresponding APK signed with it? Thanks in advance!

@j-hc any word?

sure. updated and uploaded a signed build.

Thanks! Integrated, and the new cert pinned. Update protocol:
$ iod repo get com.jhc.volumetile
com.jhc.volumetile: looking for 'https://api.github.com/repos/j-hc/VolumeTile/releases'
com.jhc.volumetile: checking tag '2'
com.jhc.volumetile: lastRelNo set to '2', checking for files
com.jhc.volumetile: Upstream file date (2024-02-20 20:11) is newer than ours (2023-04-15 13:35).
com.jhc.volumetile: returning ['2','https://github.com/j-hc/VolumeTile/releases/download/2/app-release.apk',1708456269]
com.jhc.volumetile: 1/2, https://github.com/j-hc/VolumeTile/releases: https://github.com/j-hc/VolumeTile/releases/download/2/app-release.apk
- Grabbing update for com.jhc.volumetile: OK
- Checking 'repo/com.jhc.volumetile_2.apk' for libraries and malware …
- Checking the app's AndroidManifest.xml …
! repo/com.jhc.volumetile_2.apk contains signature block blobs: 0x504b4453 (DEPENDENCY_INFO_BLOCK; GOOGLE)

If you want to get rid of that DEPENDENCY_INFO_BLOCK blob, that's easy:
android {
    dependenciesInfo {
        // Disables dependency metadata when building APKs.
        includeInApk = false
        // Disables dependency metadata when building Android App Bundles.
        includeInBundle = false
    }
}

For some background: that BLOB is supposed to be just a binary representation of your app's dependency tree. But as it's encrypted with a public key belonging to Google, only Google can read it – and nobody else can even verify what it really contains.
And if you want an icon to be shown, please point me to it :wink:


Update cloning docs to reflect beta for 1.16

Updates the feature status for VolumePVCDataSource to 1.16/beta in the
docs.
Associated KEP update:  https://github.com/kubernetes/enhancements/pull/1147

feature-gates.md also needs to be updated

@j-griffith is this tied to any enhancement?

/lgtm
/hold
until feature is merged

/milestone 1.16

@j-griffith is this tied to any enhancement?

@simplytunde Yes, associated with update of KEP (merged):
https://github.com/kubernetes/enhancements/pull/1147

/hold cancel

/lgtm cancel

/lgtm

/milestone clear

@j-griffith  looks like the enhancement will be in 1.16. can you close this?

Confused, this should be in 1.16 right?
We're waiting on docs to give a final review/approval

sorry about that. Yes, it is still going into 1.16

/milestone 1.16

/approve


Question: How to make optional field to become required again?

some shema are shared. to make it weakner, some fields are optional. but in some cases such as fucntion args, I hope some fields to be set as required again. I didn't find a method to achieve this.

The newest v3 alpha<EMAIL_ADDRESS>contains an unwrap method for doing this.
z.string().optional().unwrap(); // ZodString


can not detect the network adapter

Is there an existing issue for this?

[X] I have searched the existing issues.

What's the problem?
my installed version is windows. i have installed npcap. but when i open the app. the app cannot detect the network adapter. the only released selection is a address <IP_ADDRESS>
How did you install the app?
downloaded one of the provided packages
Operating System
Windows
Additional context
No response

Hi,
Does this mean that the initial page on the left only shows a single option?
Could you kindly provide a screenshot?

hi, this is the  screenshot
Giuliano Bellini @.***> 于2023年8月30日周三 16:52写道：

Hi,
Does this mean that the initial page on the left only shows a single
option?
Could you kindly provide a screenshot?
—
Reply to this email directly, view it on GitHub
https://github.com/GyulyVGC/sniffnet/issues/361#issuecomment-1698762343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKGZSHQ32SM23KMFKUQMWSTXX35N3ANCNFSM6AAAAAA4D2AGLE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I don't see your screenshot.
Anyway, are you sure you installed the latest version of npcap? (1.76)

Different sources online suggest to reinstall npcap in cases like this one. (example: https://ask.wireshark.org/question/14276/with-wireshark-32-on-windows-with-npcap-the-only-interface-is-the-loopback-interface/)

Had the same problem, every time I restart my computer the npcap(1.76) is installed... but is not recognized by Sniffnet - a simple reinstall, the npcap, and it works.

Amazing.
Thanks @l-33ter for the report!

Closing the issue.
@freedom6927 feel free to reopen if you don’t solve by reinstalling npcap.


Support Multiple Currencies

This system should support multiple currencies and 1 basket per culture.
This should be possible by moving the currency setting to the root node and changing the session key to make use of the culture name

To make this non-breaking, the process will be changed to have a single currency by default managed in the Settings Section.
This however can be overridden by adding a currency to each root node.
(Documentation to follow)


New Install is missing module 'Packet"

Hi all,
I have installed a new Raspberry to use the TallyArbiter code. But after the install im run the code node index.js(at the installed directory) and the following is showing up on the terminal:
`pi@TallyServer:~/tallyarbiter $ node index.js
internal/modules/cjs/loader.js:638
throw err;
^
Error: Cannot find module 'packet'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
at Function.Module._load (internal/modules/cjs/loader.js:562:25)
at Module.require (internal/modules/cjs/loader.js:692:17)
at require (internal/modules/cjs/helpers.js:25:18)
at Object. (/home/pi/tallyarbiter/index.js:5:18)
at Module._compile (internal/modules/cjs/loader.js:778:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)`
and i have downloaded it directly from GitHub by using this ommand (git clone https://github.com/josephdadams/tallyarbiter )
Im using the Raspberry pi 3B as Server
Node JS version V10.21.0
NPM version V6.14.8
Can somebody explain/help me to fix this?
Thanks in advance, Wesley

You should run npm install before executing node server.js. Does that fix your issue?

Hi Guido,
Thanks for your help! Indeed i have forgot to install the npm!


Fixing #616 bug on NSAttributedStringExtensions.

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 4.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

Fixes #616

This two builds running just don't feel right 😭
Should just be one :)) #613

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 540 tests, with 0 failures (0 unexpected) in 16.362 (20.600) seconds

:book:
tvOS: Executed 506 tests, with 0 failures (0 unexpected) in 3.446 (4.188) seconds

:book:
macOS: Executed 370 tests, with 0 failures (0 unexpected) in 1.522 (2.047) seconds

  Generated by :no_entry_sign: Danger


How to deal with CORS Problem  ?

I got 'No Access-COntrol-Allow-Origin header is present on the requesed resource Origin' response
while using ajax request azkaban API .
What can I do to avoid this CROS problem in azkaban.
thanks.

ok，I got it.
add permit in doGet and doPost Method.


Invalid homepage

Can you please fix the homepage URL in both plugin descriptions? Right now it shows https://github.com/blackleg/nb-native-notifications/quote Then please reupload to Maven central and NetBeans Plugin Portal.

@jkovalsky How can I check if it was fixed? I can't see these urls in both sites.

Well, I think that 1.2.3 version of your https://plugins.netbeans.apache.org/catalogue/?id=42 plugin already contains correct https://github.com/blackleg/nb-native-notifications URL. Thanks!


salt: 2019.2.0 -> 3000.1

Motivation for this change
salt-minion service failed for me since a while with the following error, so I upgraded salt itself and python from 2 to 3.
The error is gone now and minion and master work as expected.
Process Process-1:
Traceback (most recent call last):
  File "/nix/store/mdnyphb6385xrcx2qw2g43bq81x1p67m-python-2.7.17/lib/python2.7/multiprocessing/process.py", line 267, in _bootstrap
    self.run()
  File "/nix/store/mdnyphb6385xrcx2qw2g43bq81x1p67m-python-2.7.17/lib/python2.7/multiprocessing/process.py", line 114, in run
    self._target(*self._args, **self._kwargs)
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/scripts.py", line 157, in minion_process
    minion.start()
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/cli/daemons.py", line 348, in start
    self.minion.tune_in()
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/minion.py", line 1053, in tune_in
    self._bind()
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/minion.py", line 953, in _bind
    self.event = salt.utils.event.get_event('minion', opts=self.opts, io_loop=self.io_loop)
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/utils/event.py", line 146, in get_event
    raise_errors=raise_errors)
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/utils/event.py", line 274, in __init__
    self.connect_pub()
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/utils/event.py", line 386, in connect_pub
    io_loop=self.io_loop
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/transport/ipc.py", line 264, in __new__
    client.__singleton_init__(io_loop=io_loop, socket_path=socket_path)
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/transport/ipc.py", line 666, in __singleton_init__
    socket_path, io_loop=io_loop)
  File "/nix/store/lxxxwhm55201l8w524bhzqxi3ln364qb-salt-2019.2.0/lib/python2.7/site-packages/salt/transport/ipc.py", line 292, in __singleton_init__
    self.unpacker = msgpack.Unpacker(encoding=encoding)
TypeError: __init__() got an unexpected keyword argument 'encoding'
Exception AttributeError: AttributeError("'IPCMessageSubscriber' object has no attribute '_refcount_lock'",) in <bound method IPCMessageSubscriber.__del__ of <salt.transport.ipc.IPCMessageSubscriber object at 0x7a276f80fed0>> ignored

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Salt 3000.2 and 2019.2.4 are out, fixing two CVEs: https://docs.saltstack.com/en/latest/topics/releases/3000.2.html

In that case, I suggest:

[ ] this PR should be renamed 2019.2.0 -> 3000.2
[ ] After this is merged, create 2x new PR with backports to [19.09] and [20.03] (see CONTIBUTING.md)

I'm not sure if the backports should be of 2019.2.4 or 3000.2. I don't know enough about this package to know how much breaking changes there are b/w 2019 & 3000. @Mic92, thoughts?

I would backport 2019.2.4.

This is now being exploited in the wild. https://github.com/saltstack/salt/issues/57057

Thanks for the reference @mweinelt, I’m lucky that I run salt in a vpn only.
I tried to backport 2019.2.4 but it does work neither for 20.03 nor for 19.09:
builder for '/nix/store/zpn8dbhf03rr94lc0n1lcb6mkr7ydp29-salt-2019.2.4.drv' failed with exit code 1; last 10 log lines:
  Finished executing setuptoolsInstallPhase
  installing
  Executing pipInstallPhase
  /build/salt-2019.2.4/dist /build/salt-2019.2.4
  DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date. A future version of pip will drop support for Python 2.7. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
  Processing ./salt-2019.2.4-py2-none-any.whl
  Requirement already satisfied: futures>=2.0 in /nix/store/04wmd8vzgvm9das2hf5sr2c84k30v29s-python2.7-futures-3.2.0/lib/python2.7/site-packages (from salt==2019.2.4) (3.2.0)
  Collecting PyYAML<5.1 (from salt==2019.2.4)
    ERROR: Could not find a version that satisfies the requirement PyYAML<5.1 (from salt==2019.2.4) (from versions: none)
  ERROR: No matching distribution found for PyYAML<5.1 (from salt==2019.2.4)
cannot build derivation '/nix/store/mj2nlpk5w2026z69c2zmdhgb8661m8j6-salt-pepper-0.7.5.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/h12n1kdfniibcsf6hjv419ncnqry2q1c-env.drv': 2 dependencies couldn't be built
[0 built (1 failed), 128 copied (348.4 MiB), 78.1 MiB DL]
error: build of '/nix/store/h12n1kdfniibcsf6hjv419ncnqry2q1c-env.drv' failed
2 packages failed to build:
pepper salt

3000.2 builds and seems to work fine in 19.09 (didn’t test thoroughly).
Does anyone know if the salt-minion in 20.03 and 19.09 actually works?

Does anyone know if the salt-minion in 20.03 and 19.09 actually works?

Yep, is outdated and seems to work.
❯ git reset --hard release-20.03
HEAD is now at c0137ebba71 Merge pull request #86604 from LibreCybernetics/backport-keybase-upgrades
❯ nix-build -A salt
these paths will be fetched (9.22 MiB download, 47.56 MiB unpacked):
  /nix/store/02ra46sj0kh4alkz76cd3vnfmcdprd5b-salt-2019.2.0
copying path '/nix/store/02ra46sj0kh4alkz76cd3vnfmcdprd5b-salt-2019.2.0' from 'https://cache.nixos.org'...
/nix/store/02ra46sj0kh4alkz76cd3vnfmcdprd5b-salt-2019.2.0
❯ cd result/bin
❯ ls
salt  salt-api  salt-call  salt-cloud  salt-cp  salt-key  salt-master  salt-minion  salt-proxy  salt-run  salt-ssh  salt-syndic  salt-unity  spm
❯ ./salt-minion --version
/nix/store/02ra46sj0kh4alkz76cd3vnfmcdprd5b-salt-2019.2.0/lib/python2.7/site-packages/salt/scripts.py:198: DeprecationWarning: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date.  Salt will drop support for Python 2.7 in the Sodium release or later.
salt-minion 2019.2.0 (Fluorine)

For some reason v2019.2.4 requires an older PyYAML version, ours is at 5.2 already, but I can't see that requirement in their repository.

Since this is a standalone application, you can override the pyyaml version
to 5.X and build with that. I'm not able to post an example right now, but
a search for overridepythonattrs should show an example.
On Sun, May 3, 2020, 9:31 AM Martin Weinelt<EMAIL_ADDRESS>wrote:

Does anyone know if the salt-minion in 20.03 and 19.09 actually works?
Yep, is outdated and seems to work.
❯ git reset --hard release-20.03
HEAD is now at c0137ebba71 Merge pull request #86604 from LibreCybernetics/backport-keybase-upgrades
❯ nix-build -A salt
these paths will be fetched (9.22 MiB download, 47.56 MiB unpacked):
/nix/store/02ra46sj0kh4alkz76cd3vnfmcdprd5b-salt-2019.2.0
copying path '/nix/store/02ra46sj0kh4alkz76cd3vnfmcdprd5b-salt-2019.2.0' from 'https://cache.nixos.org'...
/nix/store/02ra46sj0kh4alkz76cd3vnfmcdprd5b-salt-2019.2.0
❯ cd result/bin
❯ ls
salt  salt-api  salt-call  salt-cloud  salt-cp  salt-key  salt-master  salt-minion  salt-proxy  salt-run  salt-ssh  salt-syndic  salt-unity  spm
❯ ./salt-minion --version
/nix/store/02ra46sj0kh4alkz76cd3vnfmcdprd5b-salt-2019.2.0/lib/python2.7/site-packages/salt/scripts.py:198: DeprecationWarning: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date.  Salt will drop support for Python 2.7 in the Sodium release or later.
salt-minion 2019.2.0 (Fluorine)
For some reason v2019.2.4 requires an older PyYAML version, ours is at 5.2
already, but I can't see that requirement in their repository.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/85561#issuecomment-623110795, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNZYI4FXO6SWNFYE2BVBMLRPVW3FANCNFSM4MLZJ46A
.

Interesting that 2019.2.0 works on 20.03, thanks @mweinelt, I thought it might suffer from the same bug as on unstable.
salt 2019.2.4 requires an old PyYaml as specified here.
I made prs with overrides for 19.09 and 20.03 in #86651 and #86653.

LGTM.
❯ nixpkgs-review pr 85561
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/85561/head:refs/nixpkgs-review/1
remote: Enumerating objects: 44, done.
remote: Counting objects: 100% (44/44), done.
remote: Compressing objects: 100% (10/10), done.
remote: Total 53 (delta 34), reused 40 (delta 34), pack-reused 9
Unpacking objects: 100% (53/53), 13.36 KiB | 285.00 KiB/s, done.
From https://github.com/NixOS/nixpkgs
   0dc3e485a11..af62dc53559  master               -> refs/nixpkgs-review/0
   06e55a145b3..435713eac22  refs/pull/85561/head -> refs/nixpkgs-review/1
$ git worktree add /home/hexa/.cache/nixpkgs-review/pr-85561/nixpkgs af62dc53559c419d570df5bf9367a2e5d0c2a1d4
Preparing worktree (detached HEAD af62dc53559)
Updating files: 100% (21630/21630), done.
HEAD is now at af62dc53559 kismet: 2016-07-R1 -> 2020-04-R2
$ git merge --no-commit 435713eac2237abc1708497cf71a621fc49cc5ea
Automatic merge went well; stopped before committing as requested
$ nix build --no-link --keep-going --option build-use-sandbox relaxed -f /home/hexa/.cache/nixpkgs-review/pr-85561/build.nix
[4 built, 13 copied (61.0 MiB), 2.2 MiB DL]
https://github.com/NixOS/nixpkgs/pull/85561
1 package built:
salt

$ nix-shell /home/hexa/.cache/nixpkgs-review/pr-85561/shell.nix
these paths will be fetched (0.05 MiB download, 0.28 MiB unpacked):
  /nix/store/d27ihabal1lgajnjj56i0kg02fnavs5a-bash-interactive-4.4-p23-dev
copying path '/nix/store/d27ihabal1lgajnjj56i0kg02fnavs5a-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...

[nix-shell:~/.cache/nixpkgs-review/pr-85561]$ ./results/salt/bin/
salt                  .salt-call-wrapped    .salt-cp-wrapped      .salt-master-wrapped  .salt-proxy-wrapped   .salt-ssh-wrapped     .salt-unity-wrapped
salt-api              salt-cloud            salt-key              salt-minion           salt-run              salt-syndic           .salt-wrapped
.salt-api-wrapped     .salt-cloud-wrapped   .salt-key-wrapped     .salt-minion-wrapped  .salt-run-wrapped     .salt-syndic-wrapped  spm
salt-call             salt-cp               salt-master           salt-proxy            salt-ssh              salt-unity            .spm-wrapped

[nix-shell:~/.cache/nixpkgs-review/pr-85561]$ ./results/salt/bin/salt-minion --version
/nix/store/bghqfj3cdk2s5qwzvl6kf4sq1vab8id9-salt-3000.2/lib/python3.7/site-packages/salt/ext/tornado/httputil.py:107: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3,and in 3.9 it will stop working
  class HTTPHeaders(collections.MutableMapping):
salt-minion 3000.2

[nix-shell:~/.cache/nixpkgs-review/pr-85561]$ ./results/salt/bin/salt-master --version
/nix/store/bghqfj3cdk2s5qwzvl6kf4sq1vab8id9-salt-3000.2/lib/python3.7/site-packages/salt/ext/tornado/httputil.py:107: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3,and in 3.9 it will stop working
  class HTTPHeaders(collections.MutableMapping):
salt-master 3000.2

Result of nixpkgs-review pr 85561 1

1 package built:
- salt


[New Feature]: Publish ogc-api-python-client to PyPi

From Mike on Slack:
FYI, unity-py, a pypi installable library relies on https://github.com/unity-sds/unity-sps-ogc-processes-api-python-client - this is know. But without the releases in api-python-client also being published we cannot publish our pacakges. Essentially, pypi doesn't let you reference a git repository as a dependency, probably because it can go away without notice. The only solution i can imagine to this is to also publish the ogc-api-python-client to pypi as well- this is rpetty light weight, adn we did it with the 1.0.0 release previously to get around the same issue
unity-sds/unity-sps-ogc-processes-api-client-python
would it be alright to publish the ogc-api-client buids as part of the release process?
https://github.com/unity-sds/unity-sps-ogc-processes-api-client-python/blob/b272f1cc24fe25347ccc21df8fc67ba4ec97838a/[pyproject.toml](https://github.com/unity-sds/unity-sps-ogc-processes-api-client-python/blob/b272f1cc24fe25347ccc21df8fc67ba4ec97838a/pyproject.toml)#L3
pyproject.toml
version = "1.0.0"
yeah, we'll update and publish ours later. you'll need to update your pyproject.tom to be the correct version, as https://pypi.org/project/unity_sps_ogc_processes_api_python_client/ already has a v1.0.0

i've published a version previously to make this work (1.0.0) and so currently i "own" that pypi repository. I can either provide the publish tokens (i think they are org secrets in github) or add someone (a good idea) to own the repo so they can also  publish items with their own keys.

@drewm-jpl : could you please include documentation on how to generate a new version of the ogc-api-python-client if needed? And how to publish it to PyPi.

This is complete: https://pypi.org/project/unity-sps-ogc-processes-api-python-client/

Instructions on how to publish a PyPI package: https://packaging.python.org/en/latest/tutorials/packaging-projects/#generating-distribution-archives


Missing licenses

Add license for the code and the used geonames file

Added the license and the reference to the data source.


Browsing matches on map

It might be interesting to have a map-view on matches to one's needs to browse those by vicinity.

implemented


Unify VM Operations queue handling and set queue_name

Move the queue handling for VM operations into common methods, set the
queue_name, and queue operations after check_policy_prevent.
Any operations run from a check_policy_prevent_callback were being run in the context of the automate role which was preventing these ems_operations from being run on an operations worker.
Issue: https://github.com/ManageIQ/manageiq/issues/19543

I don't suppose there's any chance of documenting these methods is there? I know that you understand it all at a glance, but it would help me and potentially future maintainers. :)

I'd love to document all of the "interface" methods (e.g. start, start_queue) but that is unrelated to the changes being made here.  Would rather go through all the operations not just the ones that I happened to have to update here and have a PR just to add documentation.

@djberg96 https://github.com/ManageIQ/manageiq/issues/19546


Make an Open Source project popular

https://skerritt.blog/make-popular-open-source-projects/

https://opensource.guide/starting-a-project/


About applying NER on prediction for single sentences

Hi,
Thanks for a nice piece of work. Highly appreciated.
You mention in "nerds/nerds/models/bert.py" :
           NOTE: this is an experimental NER that did not perform very well, and
        is only here for reference purposes. It has been superseded by the
        TransformerNER model, which offers the same functionality (and improved
        performance) not only with BERT as the underlying language model (as this
        one does), but allows other BERT-like language model backends as well.

May I ask where i can find the TransformerNER model?
Thanks a lot

@marcusau both the BertNER (bert.py) and TransformerNER (transformer.py) are in the same models directory, but currently only in my fork of the project (https://github.com/sujitpal/nerds). I am working with the core developers of this project to have these models folded in here.


DB-5472: CSV: Aggregate field type not being validated

Adds support for the Empty field type and also allows for sending in
the csv_info object to override any guesses made by the CSV parser.

Coverage decreased (-1.3%) to 46.448% when pulling d5f4675bc5a7c8c457782cee9fd8349d713d63bf on DB-5472 into 9ef920feb93e4f78a7c82da96f2a5c52e145fa23 on master.

Please hold off on reviewing this just yet. I created this pull request to have a link to add to the Data Basin pull request. (I didn't figure you'd all be chomping at the bit to review stuff that I didn't add you to.) I'll let you know when it's all set.

Coverage decreased (-1.2%) to 46.469% when pulling 25a78c5c7873f21713199e5822e3ea227a6cd2da on DB-5472 into 9ef920feb93e4f78a7c82da96f2a5c52e145fa23 on master.

Okay, should be good. The coveralls went down due to more code with no tests. Created Issue in Tablo for that.

Okay. It's all set now. :)

Coverage decreased (-1.2%) to 46.469% when pulling b60d3127d4a5c46d07e5acce2642e4babe105db2 on DB-5472 into 9ef920feb93e4f78a7c82da96f2a5c52e145fa23 on master.

I went ahead and added unit tests in this PR. Hopefully, that doesn't throw things off too much.

Coverage increased (+3.3%) to 51.051% when pulling 43a3d9aaf060a3e7d9615e1c279dce25b8723c70 on DB-5472 into 9ef920feb93e4f78a7c82da96f2a5c52e145fa23 on master.

Coverage increased (+3.3%) to 51.051% when pulling 3e04a40e46ede67057b47f66c7125722d7bf7080 on DB-5472 into 9ef920feb93e4f78a7c82da96f2a5c52e145fa23 on master.


Revamp project handling UI

The current project handling UI grew organizally from whatever happened to already be there, and people find it confusing.  It should be replaced by something that acts more intuitively.  Some improvements would be:

A tree view whose parts can be expanded and collapsed at will, instead of having only one currently expanded path.
Reordering and moving projects by drag and drop (with perhaps a sort button that puts the projects back in order, by time or alphabetically?).
A search or filter bar to look for projects based on substrings of the name.

All of this fits into the current model (with some added metadata for ordering).  It's just a client-side revamp.

I going to try. Looks, need to configure local auth for this.

What I see is we have several different variables which represents current folder tree. It contains window.allProjectNames, window.allFolderNames and window.nestedDirs. It looks over-complicated.
Together these variables represent what user currently see, but hidden parts of tree have no representation. There is no single structure which reflects all user directory tree with all projects.
I guess this structure could be something like
[
  { type: "directory",
    name: "folder1",
    entries: [
      { type: "directory",
        name: "folder2",
        entries: [{type: project, name: "project1"}, {type:"project", name: {project2"}]
      }]
  }
]

That sounds good to me, sure.  So step 1 here could be to switch the representations, but leave the UI the same.  This needs to be done very carefully, because there's a lot of code that depends on the existing representation, and it would be easy to break a bunch of stuff.

Ok, I'll go this way.

Project paths have next structure DWh/DWhBei51A4TKXgNYuoiZdig/DrQ/DrQI0gpIFuQMxlrqBj3qHKw/SMa/SMaLGg9zLg--LjR7kMpDfYg.cw. For what purpose is these 3-symbols prefix dirs DWh, SMa?

What is the history of this decision?

These three-letter prefixes originally came about because directories were getting too large.  They are certainly overkill for a single user's project directory, since you're going to load them all anyway.  I didn't notice when the original folder-structure pull request was sent.
I'd be okay with doing a migration if you want to fix this.  The best way to do this is probably to add the migration code to the beginning of codeworld-server.  You'd need to fix all the server code to work with the new path structure at the same time.

Yes, I want fix this. I'm not sure what means "to the beginning of codeworld-server".
I think migration should be run once with logic update, between old server down and new server up. Maybe as different binary file or even bash script.

If I were running the only server, then a one-time migration script would be fine.  But that's not the case.  I wouldn't want us to end up in a situation where someone else runs git pull and rebuilds, and suddenly their server is broken.
If you'd like to write a separate script for the migration, that's okay; but it needs to be added to run.sh so we don't have to ask everyone hosting a server to take an extra step, and therefore it should be okay to run every time the server is started.  (For example, it can check if the migration has already happened, and exit right away if so.)

Yes, got it.

I'm reopening this, because even though the implementation now uses a tree view, the UI still looks like a stack of buttons just as it always did.  So this didn't really fix the root issue.


Receiving 403 response when downloading dynamodb_local_latest.tar.gz from s3 bucket

Hey :wave:
My first contribution here - seems to be a regression of https://github.com/99x/dynamodb-localhost/issues/37
latest tar.gz was updated yesterday
<ContentsContents>
  <Key>dynamodb_local_latest.tar.gz</Key>
  <LastModified>2023-02-02T18:02:28.000Z</LastModified>
  <ETag>"265a5d3d71889672d30e2bce91e83387-6"</ETag>
  <Size>42739735</Size>
  <StorageClass>STANDARD</StorageClass>
</Contents>

But starting dynamodb fails with
Started downloading dynamodb-local from http://s3-us-west-2.amazonaws.com/dynamodb-local/dynamodb_local_latest.tar.gz into .../.dynamodb. Process may take few minutes.
✖ Uncaught exception
Environment: linux, node 14.17.0, framework 3.24.1 (local), plugin 6.2.2, SDK 4.3.2
Credentials: Local, "..." profile
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issues

Error:
Error: Error getting DynamoDb local latest tar.gz location undefined: 403
    at ClientRequest.<anonymous> (/.../node_modules/dynamodb-localhost/dynamodb/installer.js:29:15)

Unsure who owns the s3 bucket, but it is not reachable programmatically, but reachable through browser download https://s3-us-west-2.amazonaws.com/dynamodb-local/dynamodb_local_latest.tar.gz
Checking if we are missing any headers

This appears to be an issue where the file is not accessible via HTTP, but is available via HTTPS. This is causing issues in downstream packages as well such as serverless-dynamodb-local.

That's our case - what we did to mitigate the issue is to upload to our s3 bucket and update the config.json

https://github.com/99x/dynamodb-localhost/pull/78#pullrequestreview-1325061394 fixes this issue

For now, I made a work around.
Instead of call  serverless dynamodb install you can use following script:
curl <download URL> --output dynamodb.tar.gz
mkdir ./.dynamodb
tar -zxf dynamodb.tar.gz --directory ./.dynamodb

You can find the download URL for your region here. (https://docs.aws.amazon.com/amazondynamodb/latest/developerguide/DynamoDBLocal.DownloadingAndRunning.html)
Just make sure to select .tar.gz file.

Fixed in https://github.com/99x/dynamodb-localhost/pull/78 🎉

Why mark this as fixed? The PR is still open.

Also, when will this become available via NPM?

I am still getting this error:
Started downloading dynamodb-local from http://s3-us-west-2.amazonaws.com/dynamodb-local/dynamodb_local_latest.tar.gz into /home/user/Desktop/tractatus/api-management-service/.dynamodb. Process may take few minutes.
✖ Uncaught exception
Environment: linux, node 20.2.0, framework 3.32.2 (local) 3.32.2v (global), plugin 6.2.3, SDK 4.3.2
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issues

My package.json:
"devDependencies": {
    "serverless-dynamodb-local": "^0.2.40",
    "serverless-python-requirements": "^6.0.0",
    "serverless-wsgi": "^3.0.1"
  }


Add button to get more info related to inheritEnv setting when prompt is dispalyed

As a user of VS Code I won't know what the setting inheritEnv is.
It would be good to add a button that the user could click to get more information.
Its just like any other prompt, LS does not work and we switch to Jedi, and we add a button where users can get more info about this.
E.g. what is LS and what is .NET core. Same here.

We might want to do this quickly because this is only a one-time prompt. We don't want to show this prompt again with More info button.

We might want to do this quickly because this is only a one-time prompt. We don't want to show this prompt again with More info button.

Doesn't make sense. Personally I dont see any harm in adding the button for the benefit of the user.
As a user i don't know what inhertiEnv is! I will click no or go open my browser google this, find the right issue/blog that explains this and then come back.
Now open VS Code and the prompt has gone away!

What I meant was that for the users who already have seen the prompt, we don't want to show the prompt again with More info button.

I'd always leave it in there.

See #7861 this did not work.

Interesting, it worked for me:

Since it worked for @luabud and @karthiknadig (see https://github.com/microsoft/vscode-python/issues/7607#issuecomment-543884896) I'm gonna mark this issue as ✅ validated ✅ and close it.


Library That Could Be Useful for Designing the Optimizer

Hey there! I just found this and thought it was interesting because I've been thinking about what it would look like to build essentially an LLVM replacement in Rust, not that I have the time or any compiler experience to do so. :)
Still, I found this library called egg that could be used for creating the optimizer using equality saturation. After walking through the tutorial I was so amazed by how well it seemed like it could work that I wanted to write a compiler just so I could test out the possibilities for optimization using egg.
Anyway, this is probably way pre-mature being that it doesn't look like you have code generation yet ( Edit: Oh, wait maybe you do have LLVM codegen ). Take it or leave it, but I thought the idea might be useful. ;)

Always happy for people to give me some fun reading material - I'll take a look thanks!

Oh and yeah I do have LLVM code-generation - the debug info doesn't work yet, I need to actually reimplement the mess that is target ABI function signature, and I really need to build some binaries, do a load library, and make sure some stuff works.
I really want to use cranelift for the target code - but since I know LLVM pretty well I thought I'd start with what I know (and what seems to be the logical first integration these days).

Yeah, I would love to have codegen written in Rust for a project, but the security aspect of cranelift is what makes me hesitant to use it over LLVM for anything security sensitive:

LLVM still seems like the most solid option out there for now, but cranelift is exciting to look out for.

Disclaimer: I know nothing about writing compilers, I just look around and entertain the idea of writing cool stuff in different fields. :wink:


sd_setAnimationImages not working

   imageView.sd_setAnimationImages(with: [targetURL!])

     imageView.animationDuration = 3.0

     imageView.startAnimating()

@grandmasteraziz please use the issue template that you deleted and fill in all those fields as we do need them so we can properly respond.


relays in on state

My relays are staying on when not active.
They are working when i do flash for each one and correct pins are assigned.
Using pi 3 b+ and 5v 8 channel relay board.

ok the relays are working wrong way round, off when in pgm state in tally arbiter.

low on relays now worked it out, lol,,,


Create a Angular/React/Blazor FrontEnd

It would be great if we implement a frontend SPA instead of using Swagger for everything.

I like that idea, I haven't worked in front-end for a while, but I would like to help

Hello @caetanomb,
There is an Angular Frontend created by @fals that I would like to update to the latest API version.
I am definitely not good at frontend and I am looking for some guidance.
Could you to take a lead on this?

I'm away form front-end for a while. Blazor looks promising, I added for my CQRS project. I could help to build using it, I don't know react, and I'm not much likely to work with angular anymore.

That's a better a idea!

Cool!!.. It looks like then I need to learn Blazor

If we wanted to go react I'd be willing to help - I would recommend it as it is pretty much the leading framework today.

If we wanted to go react I'd be willing to help - I would recommend it as it is pretty much the leading framework today.

Don't forget about Vue, that already #2 framework today.

I am totally newbie on frontend so I am gonna publish the backend in azure then see how to implement the client.

One option could be to develop a front-end with GatsbyJS, a static site generator powered by React and GraphQL.
Be aware that just because it is a static site, it is not prohibited to include dynamic content such as authentication. 😄
The main advantage would be to be able to deploy it for free on GitHub Pages using Azure DevOps.

If we wanted to go react I'd be willing to help - I would recommend it as it is pretty much the leading framework today.

Don't forget about Vue, that already #2 framework today.

+1


SQLSmith failure connecting to terrier

I am trying to use SQLSmith to conduct fuzzing tests for the terrier. However, the following error occurred upon the connection.
# sqlsmith --target="host=localhost port=15721 dbname=terrier"
SQLsmith v1.2.1
ERROR:  Procedure not registered

Meanwhile, I tried using SQLSmith for Postgres, which worked normally. I have ported out the SQLSmith query log and Postgres CSV log with the accepted queries for troubleshooting purpose.

BTW, the logs I dumped are the first 100 queries.

Same error we've seen in some other tools trying to speak Postgres: select version() fails.

version() was added, closing.


Candle abstraction

I created abstraction of Candle and IndexedCandle class using interfaces IOhlcvData and IIndexedOhlcvData. Mostly done using Resharper's refactoring techniques. The amount of changes is quite large, but consist mostly of substitution of specific class for interfaces.
What I am not so sure about two things:

Naming of the two new interfaces. Instead of IOhlcvData it could be just IOhlcv and IIndexedOhlcv respectively.
Another shadowing in IIndexedOhlcvData. There already was shadowing using in generic version of IIndexedObject so I used it too to make it compatible with rest of the code. Would like to hear your ideas about it.

Fixes #10

@LadislavBohm First, thank you for your PR, you're quite fast with that 👍
For the naming, I agree with using the name IOhlcv instead of the longer name IOhlcvData. The IIndexedObject is mainly targeted for IOhlcv object, using IIndexedOhlcv is also reasonable & I also agree with that.

@LadislavBohm I've just resolved the conflicts & merged it to v3.1 branch (the branch that i'm developing for new features & bugfixes for the next release), you may switch to that branch as well 😄

Thanks, I wanted to take a look at the conflicts and possibly rename those interfaces this weekend. You did the hard work of resolving the conflicts so now renaming can be as simple as two refactoring commands 😃

@LadislavBohm I have renamed those classes 😄 Thanks for all your contribution 👍


remove tabs outline

it looks bad in Chrome (OSX)
http://take.ms/RLxfs
PS  grunt-contrib-uglify  was missing in package.json

Aaaaww, thanks :heart_eyes: I had the same here on Linux and didn't know how to! Thank you boys!


Record layout tests failing for sparc-v9-solaris-eabi:Clang

It looks like Sparc v9 is now treated as a 32-bit system, which is causing the layout tests to fail. Unclear if this is because the layout tests themselves are wrong or if the problem is in aro or std.Target

It's most likely because of https://github.com/ziglang/zig/pull/20960/commits/db8f00e277016e12495bcc5217090052eea69eb3 (part of https://github.com/ziglang/zig/pull/20960). What that code was doing before was just super wrong. The way you determine bitness is from the arch component (sparc vs sparc64). It's perfectly valid to run sparc32 code on v9 CPU.

Ok I dug into things a bit more. In 0001_test.c we have the following, for target sparc-solaris-eabi (with v9 cpu model)
typedef struct {
    int a;
    long b;
    void* c;
} X;
_Static_assert(sizeof(X) == 24, "");

Previously it worked because that was treated as a 64 bit target, now it fails because it's a 32 bit target. The test ultimately comes from here: https://github.com/mahkoh/repr-c/blob/master/repc/tests/testfiles/0001/input.txt which produces https://github.com/mahkoh/repr-c/blob/master/repc/tests/testfiles/0001/output/sparcv9-sun-solaris.expected.txt for sparcv9-sun-solaris (note 64 bit longs and pointers).
That C program roughly corresponds to this Rust program:
// rustc test.rs --target=sparcv9-sun-solaris
#![no_main]
use std::mem;

#[repr(C)]
struct X {
    first: std::os::raw::c_int,
    second: std::os::raw::c_long,
    third: *mut std::os::raw::c_void,
}

const _: () = assert!(24 == mem::size_of::<X>(), "size of X");

Which fails the assert if you change the 24 to something else.
I'm not sure what the conclusion is here - one answer is "the test target name is wrong" and we should change instances of sparc-v9-solaris-eabi to sparc64-v9-solaris-eabi in the layout tests. The other possible answer is "the layout tests themselves are wrong", but given how the test data was generated in the first place (https://github.com/mahkoh/repr-c/tree/master/repc/tests), I don't see how that's possible - supposedly it matches what was generated by clang itself.

one answer is "the test target name is wrong" and we should change instances of sparc-v9-solaris-eabi to sparc64-v9-solaris-eabi in the layout tests.

That's probably it .sparc64 => .sparcv9, // In LLVM, sparc64 == sparcv9..
https://github.com/ziglang/zig/commit/fb0692334ed64a995690c21525fc3e729e63f424

Note that sparcv9 and sparc64 should mean the same thing in a GNU target triple. Zig doesn't have sparcv9 (to avoid the very confusion we're seeing here), only sparc and sparc64.


Android misses bssid and ssid fields in the networkChanged event

How it works now:
2017-01-21 11:31:35 - DEBUG thaliMobileNativeWrapper: 'networkChanged: {"bluetoothLowEnergy":"on","bluetooth":"on","wifi":"on","cellular":"doNotCare"}'
...
2017-01-21 11:31:39 - DEBUG thaliMobileNativeWrapper: 'networkChanged: {"bluetoothLowEnergy":"on","bluetooth":"on","wifi":"on","cellular":"doNotCare","bssidName":"70:cd:60:ff:5a:07","ssidName":"\"S\""}'

How it should work:
2017-01-21 11:31:35 - DEBUG thaliMobileNativeWrapper: 'networkChanged: {"bluetoothLowEnergy":"on","bluetooth":"on","wifi":"on","cellular":"doNotCare","bssidName":null,","ssidName":null}'
...
2017-01-21 11:31:39 - DEBUG thaliMobileNativeWrapper: 'networkChanged: {"bluetoothLowEnergy":"on","bluetooth":"on","wifi":"on","cellular":"doNotCare","bssidName":"70:cd:60:ff:5a:07","ssidName":"S"}'

bssidName and ssidName should be null when device is not connected to AP.
ssidName should not be wrapped in quotes.

We never ever sent null from Android to node via JSON. We use standard JSONObject.put method. Just look at it https://android.googlesource.com/platform/dalvik/+/f2f7880a40aed1c8d9c27db49226e47396556aac/libcore/json/src/main/java/org/json/JSONObject.java#253

PR #1747


AutocompleteInput can not be changed once a value is selected

There appears to be a regression with AutocompleteInput in the latest 1.3.0 release once a value is selected it cannot be changed or cleared.

seconding
ie.
<ReferenceInput label="User" source="userId" reference="users">
  <AutocompleteInput optionText="name" translateChoice={false} />
</ReferenceInput>

if you try to change the value by typing, it just resets to default (record) value
it reacts on clicking on the item in list though

Fixed by #1093

Please release soon 👍


Unit tests

Added tests for ArcMap and Pro
Updated readme

@dhagens please review and merge

@jmccausland, add Esri headers to test files.

@jmccausland, you can add the following instead of using in every method:
//Use ClassInitialize to run code before running the first test in the class
[ClassInitialize()]
[TestCategory("ArcMapAddin")]
public static void MyClassInitialize(TestContext testContext)
{
bool blnBoundToRuntime = ESRI.ArcGIS.RuntimeManager.Bind(ESRI.ArcGIS.ProductCode.EngineOrDesktop);
Assert.IsTrue(blnBoundToRuntime, "Not bound to runtime");
        IAoInitialize aoInitialize = new AoInitializeClass();
        aoInitialize.Initialize(esriLicenseProductCode.esriLicenseProductCodeAdvanced);
    }

@dhagens updated

Does this addin really require an Advanced License (esriLicenseProductCodeAdvanced) ?  If so may want to highlight in Readme

I was wondering when someone was going to say something about that

Per @csmoore suggestion, try using a lower license level to initialize.
aoInitialize.Initialize(esriLicenseProductCode.esriLicenseProductCodeBasic);

@dhagens already did before you commented

Already deleted before you commented.  :)


Update searchQuery on input change to handle Enter key press correctly

Cypress regression tests are failing because they search for the theme icon to enable dark mode at the start of the run. @miladsoft , once you are done, I'll update the test to use a data-cy attribute and ensure there are no actions searching for a specific icon (as there shouldn't be any).

Cypress regression tests are failing because they search for the theme icon to enable dark mode at the start of the run. @miladsoft , once you are done, I'll update the test to use a data-cy attribute and ensure there are no actions searching for a specific icon (as there shouldn't be any).

fixed

How bigger is the app if we pull the icons from resource?

about 5 kb

Ok I suppose this is not a big deal

@itailiors when you have a free moment check why the test fails and fix it


is there any example for crowdsale of non-mintabletoken?

in the example, it use MintableToken.
but i wanna use a standardtoken.

Use an AllowanceCrowdsale and allow the crowdsale to transfer _amount of tokens. or use a normal Crowdsale and transfer() some tokens to the crowdsale in your deploy script like
myToken.transfer(MyCrowdsale.address, _amount);

Also, please join us in the slack channel for questions like these :)


Remove author

Remove "author" line from all files, as it is not common practice.

Completed with #85


MDL component not accessibility complince

It seems like most of the component are not web accessibility compliance as now ARIA roles define for example: Dialog not having role="dialog" etc....
https://getmdl.io/components/index.html#dialog-section

We understand the accessibility concerns with MDL. There are a lot of little holes spread around that when you put it all together there can be a block of swiss cheese. However MDL is not longer in active development. The project has evolved into Material Components for Web which is putting a much heavier focus on accessibility from the initial concept phases of building the components.
On the note of Dialog itself though, we don't need to manually handle roles and things. The native dialog element in browsers that support it does this already. For people who need to use it in browsers that don't support that element, you may need to take the appropriate care of applying aria roles as needed.


CI: Use 2.4.6, 2.5.5, 2.6.2, jruby-<IP_ADDRESS>

This PR updates the CI matrix.

Coverage decreased (-0.9%) to 91.581% when pulling 24a20e85b555f581b54f41efb74e2ac13d13df01 on olleolleolle:patch-1 into 3b456fd0d927f79e9fd365583e20203c7eca7d96 on nning:master.


visualization_config_id now in the visualization_data object

I think this fixes the issue, though was not sure if routes needed to be changed or if that would all happen in previews/extractors or pyclowder.

I think this should also include the functionality that hitting DELETE endpoint for visualization_config should delete any vis_data associated with that config.

This should be the case now both ways.


Receiver Side ACL plugin and Chain Engines example

Why are these changes needed?

This PR includes the following change:

A receiver side ACL plugin, that can intercept request and return an error code, bypassing the server logic.
A example of chaining multiple engines, with document.
A fix about upgrade logic, which tries to flush all engines iteratively until nothing left. This enables engines that,  when being flushed, send other request backwards.  Previous implementation only requires requests to be sent in  dataflow order.
Some other minor fixs.

Checks

[ ✅] I've run scripts/clippy.sh to lint the changes in this PR.
[ ✅] I've included any doc changes.
Testing Strategy

[ ?] Release tests
[ ] This PR is not tested :(

@libertyeagle Could you double-check if the changes in src/phoenixos/src/runtime/upgrade.rs make sense to you? I think currently it implies that we should trust the engine developers to never block the runtime (given that we already trust engine developers, so it should be fine?) Would there be any other possibility that the flushing stage was blocked indefinitely?

LGTM! Thanks.


Should repo be named pint-proj, not print-proj?

Just a question...

Thanks, I have fat fingers, apparently... Renamed.


Global Components Unrecognized (Vue 3 + Typescript)

I set some global components, import then at different ways (a global file that import all, using require, and directly into main, like you can see in the picture:

But no matter how I import it, never find the global components.

I suppose that work with vue 2, but not with vue 3.

+1

+1

I believe this is what you're looking for? If you place a file named vueconfig.json in your project root and start editing it, VSCode will know which options to suggest. There is an option called globalComponents but unfortunately, I cannot find not much documentation on this whole file.


whatsapp is not install

it is showing error after uploading file

please check if the file is valid csv in the instructed format in readme. I will also try to handle this error

please upload s sample file
Thanks


HowTo: Create a Request

Hello,
I am trying to create a request using the code bellow. I am successufully connecting but I cannot manage to call the "chia_getWallets" RPC method from c# even if the Chia dapp works (https://github.com/Chia-Network/chia-wallet-connect-dapp-test/blob/main/src/contexts/JsonRpcContext.tsx##L751).
Can someone give me a hint how to make the call for the Chia app (https://www.chia.net/downloads/)?
        public static async Task Connect()
        {
            var options = new SignClientOptions()
            {
                ProjectId = "a97dbb2756d03b9160843f6ec3ecb989",
                Metadata = new Metadata()
                {
                    Description = "Chia Connect",
                    Icons = new[] { "https://s2.coinmarketcap.com/static/img/coins/200x200/9258.png" },
                    Name = "ChiaConnect",
                    Url = "https://chia.org/",
                },
                Storage = new InMemoryStorage()
            };

            var client = await WalletConnectSignClient.Init(options);

            var connectData = await client.Connect(new ConnectOptions()
            {
                RequiredNamespaces = new RequiredNamespaces() {
                    { "chia",
                        new RequiredNamespace() {
                            Methods = new[] {
                                "chia_getWallets"
                            },
                            Chains = new[] {
                                "chia:mainnet"
                            }
                        }
                    }
                }
            });

            Console.WriteLine(connectData.Uri);
            var sessionData = await connectData.Approval;

            try
            {
                var testRequest = await client.Engine.Request<JsonRpcRequest<JsonRpcPayload>, JsonRpcResponse<JsonRpcPayload>>(sessionData.Topic, new JsonRpcRequest<JsonRpcPayload>(){Method = "chia_getWallets"});
            }
            catch (Exception ex)
            {
                Console.WriteLine(ex.Message);
            }
        }

I'm using something like this:
var request = new WalletsRequest("ADDRESS", false);
var response = await _client.Request<WalletsRequest, dynamic>(sessionData.Topic, request);

where
[RpcMethod("chia_getWallets")]
public class WalletsRequest
{
    public string fingerprint { get; set; }
    public bool includeData { get; set; }
    public WalletsRequest(string fingerprint, bool includeData)
    {
        this.fingerprint = fingerprint;
        this.includeData = includeData;
    }
}

and the response is dynamic because I can't figure out the type

Thank man, works as charmed.

Thank man, works as charmed.


async and non-blocking ?

async and non-blocking ?

yes https://github.com/openresty/lua-nginx-module#ngxsockettcp


Checkstyle tests don't work with a generated checkstyle.xml

I am trying to run checkstyle tests with a generated checkstyle.xml instead of a file in the source tree. It seems like there is something wrong with the rule that isn't allowing this use case.
I wrote a minimal reproducible example and pushed it to a repo here: https://github.com/vinnybod/bazel_java_example/tree/generated-checkstyle-config
For the minimal reproduction, the macro just copies the checkstyle config from the source tree into the bazel-out runfiles and the label of that genrule is what gets passed into the checkstyle_config.
INFO: Invocation ID: 300a43af-8451-4adb-b49f-a69bfe52a210
INFO: Analyzed 7 targets (2 packages loaded, 11 targets configured).
INFO: Found 6 targets and 1 test target...
FAIL: //:entrypoint-checkstyle (see /private/var/tmp/_bazel_vrose/17aacbd88bcaf1ce194e6bbdb78e4a29/execroot/__main__/bazel-out/darwin_arm64-fastbuild/testlogs/entrypoint-checkstyle/test.log)
INFO: From Testing //:entrypoint-checkstyle:
==================== Test output for //:entrypoint-checkstyle:
/private/var/tmp/_bazel_vrose/17aacbd88bcaf1ce194e6bbdb78e4a29/sandbox/darwin-sandbox/106/execroot/__main__/bazel-out/darwin_arm64-fastbuild/bin/entrypoint-checkstyleexec.runfiles/__main__/entrypoint-checkstyleexec: line 5: cd: bazel-out/darwin_arm64-fastbuild/bin: No such file or directory
================================================================================
INFO: Elapsed time: 0.795s, Critical Path: 0.70s
INFO: 5 processes: 3 internal, 2 darwin-sandbox.
INFO: Build completed, 1 test FAILED, 5 total actions
//:entrypoint-checkstyle                                                 FAILED in 0.5s
  /private/var/tmp/_bazel_vrose/17aacbd88bcaf1ce194e6bbdb78e4a29/execroot/__main__/bazel-out/darwin_arm64-fastbuild/testlogs/entrypoint-checkstyle/test.log

Executed 1 out of 1 test: 1 fails locally.

Thanks for the really clear minimal repro! This should be fixed by https://github.com/bazel-contrib/rules_jvm/pull/227

Wow thanks for the turnaround on that @illicitonion ! I just tested it out on my larger repo and everything appears to be working.


How to resolve the  exception in Adioso Agent?

I have created a Adioso agent.I have defined username and password.While running it throws following exception in the log tab
Exception during check:  undefined method `gsub' for nil:NilClass

Just wanted to add a +1 on this... I've reconfirmed log-in information is correct but I get the same result as rselvi above. Also, I attempted to find any information on the Adioso API on their website to no avail, and when trying to directly HTTP GET the Adioso API, my correct log-in info is rejected. Is the AdiosoAgent working for others? Does their API work for others?

Circling back on this... I've reached out to Adioso to see if the API is still open / available. Doing a little more tracing makes me doubly suspicious that the AdiosoAgent itself is working fine and that the end-point (Adioso) is the issue.

Thanks for digging in @Ishfal.  I don't use Adioso, so have no way to test it.

I'm getting the same error with the Adioso Agent. Login credentials are confirmed and I've tested the search on the Adioso website without issue.
'undefined method `gsub' for nil:NilClass'

It seems like they've turned off their API.  See also https://github.com/cantino/huginn/issues/1088

Closing since we removed the Agent 😄


writing to LiquidTemp.sensor

All is configured correctly (temp shows up in serial) but the powershell script does not write to LiquidTemp.sensor, nor does the FanControl plugin work.

Hi, thanks for cloning repo :-) can you make sure that ps script is using the same com port as serial reader?

As far as I'm aware it is. Using COM5


Go2 Improve the way of package import

The go package desgin
The package is important.  Go use file path and package name (more like a namespace) to define a unique package. It is a great design that easy to use and understand.
What is the problem ?
Also go package is easy to use, but when we import the package and use the element of the package. we have to write the package name first. I think it is boring and some kind of unnecessary. After we import the package usually think current go file have all element of the package.
Proposal
No need to clarify the package name. Only if there is a naming conflicts with other package or local package, then we need to write the package.
Such as C# namespace using.
Some tips

I know some one will point out that go can import package with a point in front of package name. Like blow.

import (
    . "math"
)

But there will report an error when the name of the element such as function, struct and var conflicts with the one in the package "math". And write a "."  So just don't mention it ,Thinks.

Most of the time there is no naming conflicts. Right ?
Some times the package name is not as same as the last directory' name that maybe a little confusing. Most of time we don't need and care about the package name.
This does not break existing code.

 So just forbid modules circular dependency and open package cycle import.
Many people  merge the all files in one package just to avoid the error of the package cycle import. It is quite annoying. It didn't have a positive impact.
I know the chances of changing this are low. But I just want to say it !!!

Go package cycle import will cause single package can not be reused in another projects, because it has relative to everything in current project.


Doc suggestion: How to target a path

Sometimes, you don’t want to target a whole domain, but only a path.
You could add this to the documentation:
How to target a specific path
CSS
You can use the @-moz-document Mozilla Extension:
/* Full path */
@-moz-document url-prefix(http://www.w3.org/Style/) {
  /* CSS here */
}

/* Regex */
@-moz-document regexp("^https?:\/\/www\.w3\.org\/Style\/.*") {
  /* CSS here */
}

Documentation: https://developer.mozilla.org/en/CSS/@-moz-document
JavaScript
You can test the window.location object:
// Search for a string
if (window.location.pathname.indexOf('/Style/') === 0) {
 // JS here
}

// Regex
if (/^\/Style\/.*/.test(window.location.pathname)) {
 // JS here
}

Documentation: https://developer.mozilla.org/en/DOM/window.location

I like!

Landed 7d003c04205b6c1ca51c9c35f7745fd6eeb18280
Thanks!


Testing checklist for nebari:v2022.10.1

Testing Checklist
Use/modify this checklist to capture the platform's core services that need to be manually tested .
Manual testing: core services
If the integration tests for all of the cloud providers are successful, that is a good sign!
However, the following core services still need to be manually validated (until we can automate them).
At minimum, the following services will need to be tested:

[x] Log into keycloak as root user

[x] Add a user

[x] Log into conda-store and create

[x] a conda environment in a shared namespace and,
[x] a conda environment in your personal namespace

[x] Launch dask-gateway cluster, test auto-scaler and

[x] Validate that the dask-labextention is working

[x] Create a basic CDS Dashboard
[x] Open VS-Code extension

[x] Add the Python extension
[x] Create a .py file and run it

Tested on GCP deployment based on 2022.10.1rc2


Investigate robot out of capsules

We saw occasions where the capsules did not encapsulate all robot link entirely.

I found the problem source:
This is caused by simulating gravity.
This is how the capsules look in position [2.5, 1.5, 0.0, 0.0, 0.0, 0.0] with simulating gravity:

And by disabling the gravity simulation in gazebo, by clicking the robot, then select L_X_E on the left side panel, and disabling the gravity checkbox (for every Link L_1_E to L_6_E), this problem can be avoided:
This is what it looks like after disabling gravity.  The arm is now fully encapsulated:

So the problem is not caused by the capsule generation or sara-shield, but by the robot having different joint positions then specified

Ah, then this is a problem of the joint impedance control. We should ask them to adjust the gravity compensation.

https://github.com/ADVRHumanoids/concert_description/issues/26
Continuation of this issue with IIT :)


Feature: GraphQL Relay Node Interface

What does this PR do and why?
Describe in detail what your merge request does and why.
This PR updates the Group type to implement the Relay Node interface. This allows us to query Groups via Node or Nodes queries. This will be really useful later on when we add additional types as it will give our clients the ability to query different types of objects within a single query.
To achieve this I updated the Group type to implement the Node interface.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other pull requests.
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.

Login with<EMAIL_ADDRESS>then navigate tohttp://localhost:3000/graphiql`
Query for a group that the user has access through the node interface{
  node(id: "gid://irida/Group/18") {
    ... on Group {
      name
    }
  }
}

See the result:{
  "data": {
    "node": {
      "name": "Bartonella"
    }
  }
}

Query for a group that the user does not have access to through the node interface{
  node(id: "gid://irida/Group/18") {
    ... on Group {
      name
    }
  }
}

See an error message{
  "data": {
    "node": null
  },
  "errors": [
    {
      "message": "An object of type Group was hidden due to permissions",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "node"
      ]
    }
  ]
}

Look at the docs and try the nodes interface the same way, while searching for the same groups.

PR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

[x] I have evaluated the PR acceptance checklist for this PR.

Everything seems to be working, but I'm not sure I understand how the gid's get made/used. If I enter
{
  node(id: "gid://anytext/Group/2") {
    ... on Group {
      name
    }
  }
}

It still responds with with the data node as if anytext was irida. Is this intended?

rails/globalid is used to generate the ids and find models based off of the ids. Since this PR didn't introduce the GlobalID functionality (it was added in the first GraphQL PR), I can create an issue to enforce that the appname of the passed GlobalID matches the configured appname.


how can i attach a file to a long message?

how can i attach a file to long message?
like this?
https://i.stack.imgur.com/0u9hc.png

@MBoretto @noplanman @jacklul can you suggest anything guys?

Have you already tried with this? https://github.com/akalongman/php-telegram-bot/blob/master/src/Entities/Document.php

Document caption has a limit of 200 characters.
Trying to figure out if that is the document caption in the screenshot or something else...

This might be something they didn't release yet to the API, it looks like linked content but also like attachment. Maybe there is some kind of new field that allows links to be attached?

@jacklul I agree, it's not a document caption. It looks like a preview when sharing a link, maybe they'll be adding that to the Document entity?
@hamidamel Where did you take that screenshot from?

Link can be attached with zero width space utf symbol.
Test this message, for example:
$data = [
            'chat_id' => $chat_id,
            'parse_mode'=> 'HTML',
            'text'    => 'So, is there a secret way? ',
        ];

@chuv1 Thanks, before you wrote this comment i solved my problem like you do, bu I used your sample for white space trick.thats really useful.
we can set parse_mode to markdown and use like this:
$data = [
    'chat_id'    => $chat_id,
    'parse_mode' => 'HTML',
    'text'       => '['white-spaces'](url)',
];

thank @MBoretto @noplanman @jacklul @akalongman for your comments and specialy thank to you @chuv1


Assets and UsedAssets

How do you differentiate between asset and used asset?
e.g. if the applicant has cash CHF 200'000 and thereof he uses CHF 150'000 for financing the property as "Eigenmittel".
do you enter
CHF 200'000 in the "assets" in "financialSituation" as "assetType" cash
and CHF 150'000 in the "usedAsset" in "FinancingRequest" as "assetType" cash and "assetUsageType" withdraw?
If so, this makes sense. For new participants in the Common API community, this should be documented somewhere.

I can say it worked for CS.

good setup. thank you

Proposed description in "assets" in the object "financial Situation":
'Array of all assets the applicant possesses, incl. the assets which will be used for the financial request.'
Proposed description in "usedAssets" in the object "Financing Request":
'Array of all assets which will be used for the financial request. At least, all of these used assets must also be recorded in the array of assets in the financial situation of one applicant.'


Update the helm labels to new app.kubernetes.io namespace

Describe the feature you'd like to have
Update the helm charts labels to the new app.kubernetes.io namespace:
https://helm.sh/docs/topics/chart_best_practices/labels/#standard-labels
What is the value to the end user? (why is it a priority?)
Helm chart is up to standard, what the users expect.
How will we know we have a good solution? (acceptance criteria)
All the helm labels updated to the new namespace.

Waiting on #570 to get merged, will then pick this up ASAP

@wilmardo iic this issue can be closed. Please feel free to reopen if you think otherwise. +1

@humblec The PR implementing this is still open and waiting on documentation. This is still is a valid open issue :)

#765 is still open, not stale


Update to 1.25 and modernize recipe.

Checklist

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

@conda-forge-admin please rerender

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

Closes https://github.com/conda-forge/r-aqp-feedstock/pull/10


[Question]Fixed size array deserialization

Why does deserializer trait delegate fixed size array deserialization to generic method?

Do you have a concrete use case where this is problematic?

Here is such use case.
Bincode crate cannot deserialize fixed size array.
This method is not implemented there, so the default method is used for deserialization. It's no wonder it cannot handle this case properly. At the moment bincode just crashes.

I tried it out locally and can confirm that the bincode examples work when delegating to deserialize_seq. Sadly this is a breaking change, because there is the possibility (although unlikely) that someone is depending on this. Then again, the docs sound to me like it should be delegating to deserialize_seq, thus this is a bug we can fix even if it's a breaking change?

I think code relying on bug is a bugged too. :)

Thanks! Last one question - could you make the new patch release when this PR will be merged?

https://github.com/TyOverby/bincode/pull/76 fixes this in bincode. We can reopen #392 when we are ready for 0.8.

I am closing this in favor of #219 which proposes removing all default implementations from the Deserializer trait. That is another way to have prevented the bincode bug.


[BUG] kubectl drain node gets stuck forever

Describe the bug
In case when drain the node by the command kubectl drain, it gets stuck forever with log
error when evicting pods/"instance-manager-r-dac5b6e6" -n "longhorn-system" (will retry after 5s): Cannot evict pod as it would violate the pod's disruption budget
To Reproduce
Steps to reproduce the behavior:

Setup the Kubernetes cluster and deploy the Longhorn system with v1.3.0-rc3.
Create a LHV and do nothing.
Use the command kubectl cordon [worker node] and then kubectl drain --force --ignore-daemonsets --delete-emptydir-data --grace-period=10 [worker node].
And it gets stuck.

Expected behavior
The drain could finish.
Environment

Longhorn version: v1.3.0-rc3
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: v1.23.6+k3s1

Number of management node in the cluster: 1 control-plane
Number of worker node in the cluster: 3 workers

Node config

OS type and version: Ubuntu 20.04.4 LTS
CPU per node: 2
Memory per node: 4G
Disk type(e.g. SSD/NVMe): NVMe
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): KVM
Number of Longhorn volumes in the cluster: 1

cc @longhorn/qa

@cchien816 Could you help verify this? because this was fixed by @mantissahz as well.

For me it's draining but it takes 5min or so with kubectl. About that Cannot evict pod as it would violate the pod's disruption budget
kubectl describe pdb instance-manager -n longhorn-system
Name:           instance-manager-e-4c4a2936
Namespace:      longhorn-system
Min available:  1
Selector:       longhorn.io/component=instance-manager,longhorn.io/instance-manager-image=imi-b6f5233f,longhorn.io/instance-manager-type=engine,longhorn.io/managed-by=longhorn-manager,longhorn.io/node=<IP_ADDRESS>
Status:
    Allowed disruptions:  0
    Current:              1
    Desired:              1
    Total:                1
Events:                   <none>

I have 3 workernodes. But sadly i'm not able to drain using Ansible (module: kubernetes.core.k8s_drain)
"msg": "Failed to delete pod POD NAME HERE due to: Too Many Requests"
I guess this module works over API.

Tested on longhorn manager-head ec91a689daef211cd0435f840d7fed2f4fb2ae51
Scenarrio 1 - Pass

Setup the Kubernetes cluster and deploy the Longhorn system with master-head
Create a LHV and do nothing.
Use the command kubectl cordon [worker node] and then kubectl drain --force --ignore-daemonsets --delete-emptydir-data --grace-period=10 [worker node].
Drain work well

Scenarrio 2 - Stucked

Setup the Kubernetes cluster and deploy the Longhorn system with master-head
Create a LHV, PVC and deployment pod mount to PVC
Write data to deployment pod
Use the command kubectl cordon [worker node] and then kubectl drain --force --ignore-daemonsets --delete-emptydir-data --grace-period=10 [worker node].
Drain will stuck with log Cannot evict pod as it would violate the pod's disruption budget.

In scenario 2 , If run kubectl delete poddisruptionbudgets -n longhorn-system --all and drain node again, drain node will complete smoothly. And uncordon all nodes, can observe that pod data still keep consistency
cc @innobead , @c3y1huang , @mantissahz

Scenarrio 2 - Stucked

Setup the Kubernetes cluster and deploy the Longhorn system with master-head
Create a LHV, PVC and deployment pod mount to PVC
Write data to deployment pod
Use the command kubectl cordon [worker node] and then kubectl drain --force --ignore-daemonsets --delete-emptydir-data --grace-period=10 [worker node].
Drain will stuck with log Cannot evict pod as it would violate the pod's disruption budget.

Is the volume still attached when draining node?

After discussed with @c3y1huang , I understood that Longhorn not allowing drain while volume is still attached.
Update test result
Tested on longhorn manager-head ec91a689daef211cd0435f840d7fed2f4fb2ae51
Result : Pass

Setup the Kubernetes cluster and deploy the Longhorn system with master-head
Create a LHV and do nothing.
Use the command kubectl cordon [worker node] and then kubectl drain --force --ignore-daemonsets --delete-emptydir-data --grace-period=10 [worker node].
Drain work well

For me it's draining but it takes 5min or so with kubectl. About that Cannot evict pod as it would violate the pod's disruption budget
# kubectl describe pdb instance-manager -n longhorn-system
Name:           instance-manager-e-4c4a2936
Namespace:      longhorn-system
Min available:  1
Selector:       longhorn.io/component=instance-manager,longhorn.io/instance-manager-image=imi-b6f5233f,longhorn.io/instance-manager-type=engine,longhorn.io/managed-by=longhorn-manager,longhorn.io/node=<IP_ADDRESS>
Status:
    Allowed disruptions:  0
    Current:              1
    Desired:              1
    Total:                1
Events:                   <none>
P.S there are more similar rows, i didn't copy all of them here.

I have 3 workernodes. But sadly i'm not able to drain using Ansible (module: kubernetes.core.k8s_drain) "msg": "Failed to delete pod POD NAME HERE due to: Too Many Requests" I guess this module works over API.

Hi @impsik ,
Would you mind create another ticket for tacking issue about unable to drain node when using Ansible? thank you

@cchien816 , sorry I've misled you. Going through Updating the Node OS or Container Runtime and seems we are expecting the engine process to migrate during drain.
https://github.com/longhorn/longhorn/issues/1631#issuecomment-708842694

Update test result
Tested on longhorn manager-head ec91a689daef211cd0435f840d7fed2f4fb2ae51
Result : Pass
Scenario 1

Setup the Kubernetes cluster and deploy the Longhorn system with master-head
Create a LHV and do nothing.
Use the command kubectl cordon [worker node] and then kubectl drain --force --ignore-daemonsets --delete-emptydir-data --grace-period=10 [worker node].
Drain work well

Scenario 2

Setup the Kubernetes cluster and deploy the Longhorn system with master-head
Create a LHV, PVC and deployment pod mount to PVC
Use the command kubectl cordon [worker node] and then kubectl drain --force --ignore-daemonsets --delete-emptydir-data --grace-period=10 [worker node], kubectl uncordon  [worker node] sequentially on each. We need at least 1 schedulable node for migrate.
In step 3 can observe deployment pod will migrate to schedulable node during node draining
After all node drain and uncordoned, can observe that deployment pod's data still keep consistent


Feature request : Linux-ELF/glibc toolchain support

Is ckb-vm support Linux-ELF/glibc toolchain ? I need some function like pthread in my ckb script, but only  Linux-ELF/glibc support pthread library.

Could you provide more insights why glibc and pthread library are important to write code for on chain verifiers in CKB?
We believe that glibc and pthread are too heavy. Especially pthread, CKB will not expose any operation system resources such as threads, locks to the VM.

Due to the design of CKB VM, threading will probably never be a supproted feature, there might be a way to support glibc but so far we failed to see a proper use case. It might be better if you can expand on your requirement here, a different approach might exist.


Number of requested results 4 cannot be greater than number of elements in index 3

Describe the bug
ERROR - Error: Number of requested results 4 cannot be greater than number of elements in index 3
Browser and Version

Chrome
112.0.5615.137

Screenshots
image

This was fixed in 76162a9ffa8314013fadb04217e09a688a635e97
It happens because what you are trying to load is not generating at least 4 documents which is the default number of retrieved documents by ChromaDB.
If you fix the split it could help (maybe the separator or the chunk size).
We are working on better tooling for debugging this kind of things.

Is this one already available via pip or should I custom build it?
Thanks for the answer!

We'll release a new version today with many fixes and some new features.


Replaced transaction's amount is shown twice in "unconfirmed_balance"

Background
My node recovered the commitment amount a couple of days after a force-close. The first transaction showed up in unconfirmed_balance (lncli walletbalance), as expected. However, lnd decided to replace this transaction by a transaction paying a higher fee rate. Now, unconfirmed_balance shows the double of the amount.
Your environment

version of lnd: lnd 0.15.0-beta.rc6
which operating system (uname -a on *Nix): Linux server 5.10.0-13-amd64 #1 SMP Debian 5.10.106-1 (2022-03-17) x86_64 GNU/Linux
version of btcd, bitcoind, or other backend: bitcoind v23

Steps to reproduce
Let lnd recover commitment amount after force-close. Let lnd replace (cheap) transaction by (more expensive) transaction. Run lncli walletbalance.
Expected behaviour
Amount is shown in unconfirmed_balance.
Actual behaviour
Double of the amount is shown.

Probably related and/or duplicate of https://github.com/lightningnetwork/lnd/issues/5244.

Yes, or rather #5225.
PS: Now I have 3x the amount :)


Usage of import/export is confusing and should be hidden

Right now all language bindings (and the CLI) support import and export modes. Figuring out what this means within each language generator is confusing and easily forgotten. I don't like how this has shaped up. Internally I think this distinction is necessary for configuring what sort of bindings are generated where, but ideally this would not be a concern of an end-user.
I'm not sure how to fix this though since *.witx files are fundamentally only intended to describe "half" where they're describing either what a wasm module is importing or what a wasm module is exporting. Hosts may want to prefer to say "here's the adapter module that I'm working with" and have everything generated from that. Wasm programs seem like they still want to separate imports/exports, though.
That may actually be the fix? We could have hosts deal with imports/exports all at once where you just say "here's the shape of the wasm module I'm generating bindings for" but the import/export terminology and split sticks around for guests compiled to wasm.
In any case I haven't given this too too much thought but I wanted to at least jot down some ideas and the fact that this is an issue I'd like to fix.

I'm going to close this in favor of https://github.com/bytecodealliance/wit-bindgen/issues/314 and the advent of *.world files.


Database is not selected automatically

redis://user:password@localhost:6379/2

Using this URI to connect to Hiredis, the database is not selected automatically. It always defaults to DB 0. Inspecting the Client object after creation, the instance variable @db has the correct value.
@db = '2'

But when I'm doing a manual select of the DB, it works like a charm.
con = EM::Hiredis.connect('redis://user:password@localhost:6379/2')
con.select('2')

Thanks Marc

If sounds like a bug. I'm not going to have time to look into this for a while, but the following code in client.rb should mean that the database is selected. Maybe you can debug a little?
@connection.on(:connected) do
  @connected = true

  select(@db) if @db

Cheers, Martyn


⚠ ClamAV has degraded performance

In a4abaf7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 929 ms

Resolved: ClamAV performance has improved in a691c15.