Reload the content of a turbo-frame

We would like to trigger the reloading of a turbo-frame which has been lazy loaded with the src attribute.
How can we achieve that? If there's nothing built-in, would be something you would consider in a PR?
As a reference: https://discuss.hotwire.dev/t/how-to-auto-reload-frame/1872/5

To me this issue can be closed nevertheless. Your suggestion works, and I opened a PR if we want to implement the reload() method. Thanks.

The above mentioned technique doesn't work when loading="lazy"

Any ideas for reloading content of loading="lazy" frames?

@marselmustafin You can reload the content manually using this.element.reload() inside a stimulus controller attache to that turbo frame or you can do document.getElementById('turbo-frame-id').reload()

Can I somehow mark a turbo-frame as "incomplete" so it reloads the next time it's displayed?

@firedev

Can I somehow mark a turbo-frame as "incomplete" so it reloads the next time it's displayed?

I just implemented @jorgegorka's suggestion in the stimulus controller I use to hide/show a modal with its content loaded via turbo frame with src:  It works great.  Right after display I call the .reload() so there's zero delay but new content will indeed get loaded on top after a few ms.
A fancier alternative could be to load the content with a subscribed turbo stream and then broadcast any changes server side.  I'm loading such a small amount of data, and it's only shown temporarily so for me just reloading every time is fine.

@duffyjp Hey here is how I did it in the end https://gist.github.com/firedev/cedf0ea3153e790498da5c72b33fcd76

Here's a stimulus controller that resets a turbo-frame when it is removed from view.  Awesome for dialog elements.
import { Controller } from "@hotwired/stimulus";

export default class extends Controller {
  /*
  A controller that resets a lazy turbo frame when it disappears from view.
  Usage:
      <dialog>
        <turbo-frame id="frame_name" data-controller="reloadframe" src="<url>" loading="lazy">Loading...</turbo-frame>
      </dialog>
  */

  started = false

  connect() {
    this.initialContent = this.element.innerHTML;
    this.src = this.element.src
    this.element.addEventListener("turbo:frame-load", this.frameLoaded.bind(this));
    this.intersectionObserver = new IntersectionObserver(this.intersect)
  }

  frameLoaded() {
    if (!this.started) {
      this.started = true
      this.intersectionObserver.observe(this.element)
    }
  }

  resetTurboFrame() {
    this.element.removeAttribute("complete")
    this.element.setAttribute("loading", "lazy")
    this.element.innerHTML = this.initialContent

    if (this.started) {
      this.started = false
      this.intersectionObserver.unobserve(this.element)
    }
  }

  intersect = (entries) => {
    const lastEntry = entries.slice(-1)[0]
    if (!lastEntry?.isIntersecting) {
      this.resetTurboFrame()
    }
  }
}


JSONDecodeError over running usellm chat.py

Opened the repo in github codespaces installed the usellm using "pip install usellm" command. After succesfull installation, used this command changed the directory to examples (i.e., cd examples) , and executed command to run the chat.py (i.e., python chat.py) file to get familiar with this LLM models. Unfortunately there is no output, all i could find was "JSONDecodeError"

Issue is sorted


Input range slider

I want to implement this in the drink editor:
https://strukturedkaos.github.io/kickdrop-drops/drops/sliders/
To do something like this:

Can you check that script out and import the required js (I attempted but since the page loads async it didn't work). I've already added the CSS.
I want to implement this on the drink page as well as in editor and eventually we can plug this into the search somehow and allow people to filter/query based on what tastes they desire :)
This would be really awesome!
I've come up with this list of tastes, if you can think of any descriptors let me know.

Alcohol (content)
Fruitiness
Sweetness
Spiciness
Bitterness
Sourness
Saltiness / Savoriness
Creaminess
Bubbliness
Dryness

@sirtastic : let's postpone this. I feel that I'm getting a lot of new requirements, too many, and these things should have informed the model layer. Flashbacks of life at Pinney. :rage1: I have to keep redesigning the back end and refactoring the front end (js).
If/when you feel that this is a must, please explain the entire usage if there is more. (I don't suppose you want just a set-and-show feature for this.)

@sirtastic : I'm not against it. I think it might be cool. But I'm busting my hump, and I can't take any new developments right now.

Postponed. Adding to Phase II Milestone.


Object prototype may only be an Object or null

I tried all the solutions suggested at https://github.com/mvayngrib/react-native-randombytes/issues/5
I did (react native > 0.29.0)

npm i --save-dev rn-nodeify
rn-nodeify --install --hack
linked to react-native-randombytes

Now I can not import react-native-crypto.

check out https://github.com/mvayngrib/rn-nodeify/issues/18

@mvayngrib  As far as I understand, that "hack" has already been pushed to NPM?
I just npm install rn-nodeify and I can see that https://github.com/mvayngrib/rn-nodeify/commit/9917779ed9a461c6c43ba410b9f0bf6372c6f31a is already there.
Yet when I ran it on android in windows I still run into this issue somehow? (A friend of mine was developing on a Mac and he is not seeing any issues). I mainly needed this so a dependency of our app does not break on my when it does require("crypto")
I am using react-native 0.32.1

@liuzhen2008 just to make sure, do you need the crypto module? If not, you may be able to just exclude crypto: https://github.com/facebook/react-native/commit/5710b230a2a0f2b29553fbf94af5147a2e030f44

@liuzhen2008 released 7.0.1 with fix for windows by @mhofman: https://github.com/mvayngrib/rn-nodeify/commit/67642d26c08440a117971396c6a4ee3dc1c524b7

@mvayngrib Thanks. I will to install rn-nodeify again once i get home.
But to answer your first question. My react native application uses a "sockjs" module. And that module requires "crypto".
Apparently I didn't need to use rn-nodeify on a Mac. Really odd.

@mvayngrib Just tested. The same error still occurs. same stack as @huutrancl posted

@liuzhen2008 hm, i've never tested on windows (are you using windows now?). Usually when errors persist it means i need to rerun the rn-nodeify script. For example after you npm i <something>, you should re-run rn-nodeify. Hope it helps...

I'm also stuck at this error on Windows.

So I wonder how can I solve this error ?
I occur this error now...

no answer for this issue ? i have this problem with react-native-tcp and react-native-crypto !
is there any solution ?

@omidnavi I have solved this problem, you can try to do this.

@yreenchan  Thanks for your response , could you please tell give me a complete walkthrough ?

@yreenchan Oh its not needed 👍 Everything is working now , Thank you , you made my day.

@omidnavi You're welcome !


fix(content-block-simple): multiple style bugs

Related Ticket(s)
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5883
Description
Multiple style bug fixes for content block simple

Closing. All fixes included in https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/5943


Request for openmpi implement

I am using deepmd-kit v3.0.0b3 offline that requires openmpi >=4.1.6, which contains plumed with default module config (from conda-forge channel) and lammps (compatible with deepmd-kit).
However I wanna use some modules that are turned off by default in plumed. So I have to reinstall plumed in above conda env.
Then I install plumed from deepmodeling channel, which is built by this plumed-feedstock. But the problem is that plumed built from this feedstock is based on mpich implement, which seems conflict with openmpi required with deepmd-kit. The detail reports is that:
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
failed

LibMambaUnsatisfiableError: Encountered problems while solving:
  - package deepmd-kit-3.0.0b3-cuda120py311hdc0cc46_mpi_openmpi_0 requires openmpi >=4.1.6,<5.0a0, but none of the providers can be installed

Could not solve for environment specs
The following packages are incompatible
├─ deepmd-kit 3.0.0b3.*  is installable with the potential options
│  ├─ deepmd-kit 3.0.0b3 would require
│  │  └─ openmpi >=4.1.6,<5.0a0  with the potential options
│  │     ├─ openmpi 4.1.6 would require
│  │     │  └─ mpi 1.0 openmpi, which can be installed;
│  │     └─ openmpi 4.1.6 conflicts with any installable versions previously reported;
│  └─ deepmd-kit 3.0.0b3 conflicts with any installable versions previously reported;
└─ plumed 2.8.2**  is not installable because there are no viable options
   ├─ plumed 2.8.2 would require
   │  └─ mpich >=4.1.1,<5.0a0  but there are no viable options
   │     ├─ mpich 4.1.1 would require
   │     │  └─ mpi 1.0 mpich, which conflicts with any installable versions previously reported;
   │     └─ mpich [4.1.1|4.1.2|4.2.0|4.2.1|4.2.2] conflicts with any installable versions previously reported;
   └─ plumed 2.8.2 conflicts with any installable versions previously reported.

Therefore, if there is an openmpi implemented plumed in the deepmodeling channel, I may be able to reinstall plume with all modules in the above conda environment, and then the problem can be solved.
Thanks and look forward for your reply.

The deepmodeling channel has been deprecated. No new packages will be added anymore.


RDKTV-12796:Move iarmmgrs oem patches

Reason for change: mfr enum rename
Test Procedure: mfrmgr test cases
Risks: Create None
Signed-off-by: Rahamataunnisa shaik<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Rahamataunnisa shaik seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

raised https://github.com/rdkcentral/rdkservices/pull/3757/


Feature 643 log commands with nesting to indicate call depth

Description:
This PR causes Pipette commands to be prepended with a *. Each star indicates that a pipette command is nested call to a previous call.
E.g. if aspirate calls dispense The log will read aspirate, *dispense
For the following pipette command:
p200.distribute(55, pcr_plate.wells('A1'), pcr_plate.rows('1'), touch_tip=True, blow_out=True)

The robot.commands() call prints,
 '*Transferring 55ul from <Well A1> to <WellSeries: <Well A1>..<Well H1>>',
 '**Picking up tip from <Deck><Slot A1><Container tiprack-200ul><Well A1>',
 '**Aspirating 165.0 at <Deck><Slot C3><Container pcr><Well A1>',
 '**Touching tip',
 '**Dispensing 55.0 at <Deck><Slot C3><Container pcr><Well A1>',
 '**Touching tip',
 '**Dispensing 55.0 at <Deck><Slot C3><Container pcr><Well B1>',
 '**Touching tip',
 '**Dispensing 55.0 at <Deck><Slot C3><Container pcr><Well C1>',
 '**Blowing out ',
 '**Touching tip',
 '**Aspirating 165.0 at <Deck><Slot C3><Container pcr><Well A1>',
 '**Touching tip',
 '**Dispensing 55.0 at <Deck><Slot C3><Container pcr><Well D1>',
 '**Touching tip',
 '**Dispensing 55.0 at <Deck><Slot C3><Container pcr><Well E1>',
 '**Touching tip',
 '**Dispensing 55.0 at <Deck><Slot C3><Container pcr><Well F1>',
 '**Blowing out ',
 '**Touching tip',
 '**Aspirating 110.0 at <Deck><Slot C3><Container pcr><Well A1>',
 '**Touching tip',
 '**Dispensing 55.0 at <Deck><Slot C3><Container pcr><Well G1>',
 '**Touching tip',
 '**Dispensing 55.0 at <Deck><Slot C3><Container pcr><Well H1>',
 '**Blowing out ',
 '**Touching tip',
 '**Drop_tip at <Deck><Slot D2><Container point><Well A1>']

Check List:

[x] integration tests pass and have sufficient coverage of new changes (npm run integration)
[x] added unit tests if necessary

Coverage decreased (-0.03%) to 90.577% when pulling 651a020d39b4f77c29a9fc2b04ea95aa11bf224f on feature-643-log-commands-with-nesting-to-indicate-call-depth into 23be266c1e622641680518d7fd85e246658808c8 on develop.


Implements #89 by adding a FileResponse class

This Pull Request adds a new class FileResponse which will asynchronously stream file data back to a client.
It can be used in two ways:

A shared pointer to an instance of a FileResponse can be returned from a PathHandler to stream that file back
Another Response object can instantiate a FileResponse and use its respond method to send file data back on a provided ResponseWriter shared pointer

In either situation, the file data is read and queued up for transmission on the server thread to prevent stalling other requests. If zlib support is enabled and it is explicitly enabled within the constructor, the data response can be compressed with the deflate method.
When the file being sent back is uncompressed, file resuming with the Range header is supported. Also, the If-Modified-Since, If-Unmodified-Since, and If-Range headers are supported, using the file's last modification time to determine this. Support for these headers required the addition of a webtime string parser to be added to the StringUtil class.
When the file being sent back is compressed, it is not compressed ahead of time but while it is being loaded from the disk. As a consequence, the final data size is unknown so a Content-Length header is not provided and the Range header is no longer supported though the If-Modified-Since and If-Unmodified-Since header will still be acknowledged. If the client supports compression but would rather have file resuming ability, compression is disabled if the incoming Accept-Encoding header does not have deflate in it.
This Pull Request implements #89

Personally I would consider if it's possible for your application to use a server that specialises in file serving (nginx is good) because it can probably do it much more efficiently (ie using sendfile(2), pre-gzipped files, etc).
That said, I'm not against this as long as it doesn't increase the attack surface for apps that just want to use websockets (which of course is the main point of seasocks).
Also I quickly grepped your patch and didn't see any Vary headers, you'll probably want to add a "Vary: Accept-Encoding": https://blog.stackpath.com/accept-encoding-vary-important

I just realised my comment may have come off sounding discouraging, sorry for that. I actually do think this is a worthwhile patch!
Also, this was just my opinion and I of course don't speak for @mattgodbolt nor about the direction he is planning for seasocks.
Cheers!

Personally I would consider if it's possible for your application to use a server that specialises in file serving (nginx is good) because it can probably do it much more efficiently (ie using sendfile(2), pre-gzipped files, etc).

Definitely would have used an engine like that if most of the things that my application was doing was returning files but for the most part it's programmatic. Person accesses address, C++ function is called, data is returned. So far Seasocks has been a fantastic match for the common case. It's just that a few addresses return an actual file from the filesystem which is why I need this thing. Doesn't need to be fancy, just work well enough.
I'll look into the Vary header, thanks for the great link

Added the Vary header to the response so it will be cached correctly.

Wow, you've put a lot of work into this! I've added some implementation related comments.

I've added some implementation related comments.

… which are not visible until Submit review is pressed …

@offa how do you feel about this change? I skim-read the comments and all looks OK to me...are we waiting on the author to address anything?

I think some of the comments should be addressed before. And we should get some tests in. But actually there's nothing dramatic, just a few things which have to be done.
We / I can help @scallopedllama to fix them of course. But which way do we go?

Contribute via PR to this PR
Contribute directly here (can we?!)
Merge to a development branch and once finished to master

I'm fine with all of them. We could also merge this to master and then work on it, but this is probably not the best way to go. @scallopedllama / @mattgodbolt  what do you think?

Another ping @scallopedllama :) Thanks again for this, would be great to get it merged in. I'd prefer working off-master (another branch in this project) is fine, if we need to work on the repo here.

Sorry for the delay on this, I haven't had the time to work on the changes.
I'll look into getting some time to work on it soon!
On Mon, Oct 8, 2018 at 4:54 PM Matt Godbolt<EMAIL_ADDRESS>wrote:

Another ping @scallopedllama https://github.com/scallopedllama :)
Thanks again for this, would be great to get it merged in. I'd prefer
working off-master (another branch in this project) is fine, if we need to
work on the repo here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mattgodbolt/seasocks/pull/94#issuecomment-427975957,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALVdi3_cCiglEnsP1WxHawPliRIdHL4ks5ui7uQgaJpZM4V0aeF
.


Fix compiler warnings

Currently, compiling liner emits several warnings, which should be fixed.

Sorry, didn't see #77


onPanResponderTerminationRequest never called in Android

Description
Hello,
I'm trying to achieve card swipe swipe ui similar to Tinder. But the variation is that I have scrollview inside a card. So basically scroll-view inside pan responder view. So on android after pan gesture is triggered scrollview takes over and onPanResponderTerminate is called but onPanResponderTerminationRequest false is not respected.
onPanResponderTerminationRequest is never gets triggered. I belive this is the issue with React Native PanResponder. It will be very helpful if this gets addressed.
Version
0.64.0
Output of react-native info
System:
OS: macOS 11.6
CPU: (8) x64 Apple M1
Memory: 22.01 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 14.16.1 - /usr/local/bin/node
Yarn: Not Found
npm: 7.11.2 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.10.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 15.0, DriverKit 20.4, macOS 11.3, tvOS 15.0, watchOS 8.0
Android SDK:
API Levels: 28, 29, 30, 31
Build Tools: 28.0.3, 29.0.2, 30.0.2, 30.0.3, 31.0.0
System Images: android-31 | Google Play ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7583922
Xcode: 13.0/13A233 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_292 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.1 => 17.0.1
react-native: ^0.64.2 => 0.64.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Have a ScrollView insider a View with PanResponder handlers and try moving View in (try swipe right/left).
Pan gesture is triggered but immediately interrupted which cause onPanResponderTerminate  to trigger. As scrollview takes over and becomes a responder now
Snack, code example, screenshot, or link to a repository
No response

HI,  was it resolved with this question ? And I have this question, need your help

HI, was it resolved with this question ? And I have this question, need your help

Unfortunately not.

onPanResponderTerminationRequest is not called on Android so setting:
onPanResponderTerminationRequest: () => false will only call onPanResponderTerminate in Android.
You could have similar logic between onPanResponderRelease and onPanResponderTerminate

Can confirm this bug too.  onResponderTerminationRequest (and onPanResponderTerminationRequest) aren't called on Android when a scroller wants to become the responder, so when you're trying to respond to drags that overlap with scrollers, the drag gets stolen from your responder without asking.  This is a problem for card-style UIs, where you do want to allow scrolling, but trigger a different gesture if the user drags down while the scroller is already at the top.
PanResponder's onShouldBlockNativeResponder seems to have no effect on this, and the documentation doesn't mention it either.

strangely, i have found that wrapping the view and applying the pan handlers to an Animated.View (instead of a standard View) actually prevents the ScrollView from taking over.

Why even bother reporting bugs?

I am running into this as well. Did anyone find a solution?

same issue here. ... any update on this?


'docker port' man page example uses wrong syntax

In man page for docker-port, the examples section says to type 'docker ps test' but currently, docker ps takes 0 arguments.
The examples should instead say
docker ps -a | grep test

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO


remove python3 syntax error (PEP8 E999)

Corrected except statement to use Python3 as syntax.
Added parentheses around print statement.

These changes still remain unmerged.


Separate table into its own component and add to dynamic components

Related #1399
The metadata type table can render a datagrid or table clarity component depending on context. Instead these components should be clearly separated and add TableComponent directly into dynamic-component.ts.
Since table is already mapped to DatagridComponent, we will need to think of a reasonable type name.

I think there are some example of how this is done in the code base but just to make sure
https://angular.io/guide/dynamic-component-loader and https://github.com/lenriquez/octant/blob/master/web/src/app/modules/shared/components/view/view-container.component.ts
will be another example?


Added new value to LinkType parameter in Add-UnifiedGroupLinks.md

Closes https://github.com/MicrosoftDocs/office-docs-powershell/issues/7687.

Docs Build status updates of commit bd81a8a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Add-UnifiedGroupLinks.md
:white_check_mark:Succeeded
View (exchange-ps)

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

@dariomws, closing this PR. See Issues #7207 #7689. I investigated with the product team, and that  EventSubscribers value is exposed in error, and you always get an error when you try to use it.
Closing


Bug: python sockets sometimes fail and the service needs to be restarted

Investigates why the sockets are failing and find a fix or workaround to avoid service shutdowns.

Happened over the weekend:
Windows PowerShell
Copyright (C) 2016 Microsoft Corporation. All rights reserved.

PS C:\Users\ubi> .\Desktop\foodaid.ps1
2017-08-17 12:16:38,082 - werkzeug - INFO -  * Running on http://<IP_ADDRESS>:8080/ (Press CTRL+C to quit)
Exception in thread Thread-1:
Traceback (most recent call last):
  File "C:\Python36\lib\threading.py", line 916, in _bootstrap_inner
    self.run()
  File "C:\Python36\lib\threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\ubi\source\foodaid\bot.py", line 107, in _loop_messages
    events = self._slack.rtm_read()
  File "C:\Python36\lib\site-packages\slackclient\_client.py", line 126, in rtm_read
    json_data = self.server.websocket_safe_read()
  File "C:\Python36\lib\site-packages\slackclient\_server.py", line 155, in websocket_safe_read
    data += "{0}\n".format(self.websocket.recv())
  File "C:\Python36\lib\site-packages\websocket\_core.py", line 293, in recv
    opcode, data = self.recv_data()
  File "C:\Python36\lib\site-packages\websocket\_core.py", line 310, in recv_data
    opcode, frame = self.recv_data_frame(control_frame)
  File "C:\Python36\lib\site-packages\websocket\_core.py", line 323, in recv_data_frame
    frame = self.recv_frame()
  File "C:\Python36\lib\site-packages\websocket\_core.py", line 357, in recv_frame
    return self.frame_buffer.recv_frame()
  File "C:\Python36\lib\site-packages\websocket\_abnf.py", line 340, in recv_frame
    self.recv_header()
  File "C:\Python36\lib\site-packages\websocket\_abnf.py", line 288, in recv_header
    header = self.recv_strict(2)
  File "C:\Python36\lib\site-packages\websocket\_abnf.py", line 375, in recv_strict
    bytes_ = self.recv(min(16384, shortage))
  File "C:\Python36\lib\site-packages\websocket\_core.py", line 427, in _recv
    return recv(self.sock, bufsize)
  File "C:\Python36\lib\site-packages\websocket\_socket.py", line 93, in recv
    "Connection is already closed.")
websocket._exceptions.WebSocketConnectionClosedException: Connection is already closed.

This seems to be a bug in Slack API according to this issue.


Question Regarding Meta-train of PyTorch Implementation

Hi, thanks for your interesting work. I have a question regarding meta-training step in PyTorch implementation. In line 167-169 of 'meta-py' the parameters of the classifier are updated according to the equation #3 in paper using the loss for training samples. Also, the loss for validation samples (according to equation #4 and #5 in the paper) should be back-propagated to do the second loop of meta-learning. However, in the released PyTorch implementation, I can see only the forward pass using validation samples, and the loss is not back-propagated to optimise SS parameters & theta (equation #4 and #5). I would really appreciate it if you could clarify this for me. Thank you very much.

Hi,
Thanks for your interest in our work.
The back-propagating is conducted as follows,
https://github.com/yaoyao-liu/meta-transfer-learning/blob/d4ab548fe4258bab8e1549c8e1c9be175d52afe1/pytorch/trainer/meta.py#L169
And the optimizer is defined as follows,
https://github.com/yaoyao-liu/meta-transfer-learning/blob/d4ab548fe4258bab8e1549c8e1c9be175d52afe1/pytorch/trainer/meta.py#L61-L62
where self.model.encoder.parameters() are the SS parameters and self.model.base_learner.parameters() is theta.
So the SS parameters and theta are optimized by the meta loss.
If you have any further questions, feel free to add additional comments.
Best,
Yaoyao

Thank you very much for your prompt response.


ToC

This page is missing from the ToC. Can it be added back in if the doc is still current?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ae68aa1a-26f8-007c-c353-619521edf569
Version Independent ID: 577d5c30-903a-29aa-561e-6abdacccf360
Content: Virtual machines in an Azure Resource Manager template - Azure Windows Virtual Machines
Content Source: articles/virtual-machines/windows/template-description.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

@dcbrown16
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@cynthn
Can you please check and add your comments on this doc update request as applicable.

I've added this file to the Windows TOC. Pull request will probably merge tomorrow. Thank you!
#please-close


Remove extra debug log statements[BUG]

Describe the bug
Log statements that don't provide that much information:
INFO:marqo.tensor_search.throttling.redis_throttle:Beginning throttling check. API request is of type SEARCH

To Reproduce
Steps to reproduce the behavior:

Make a search
Observe output

Expected behavior
This information is visible from uvicorn log anyway, so we don't need it.

Resolved by https://github.com/marqo-ai/marqo/pull/319


added test

-### Using jest

Add index.test.js to check for data

Hey Eric,
Thank you for the feedback. We will revert accordingly.


Implement a TFTP interface on top of TcpConnection

This will be useful for doing various TFTP things.

Do this: https://github.com/vkottler/runtimepy/issues/143 first / as well.

https://github.com/vkottler/runtimepy/pull/237 one of the tougher climbs


GPG key generator

Is your feature request related to a problem? Please describe.
You can generate ssh keys and sync them with remote servers. However you should also include a way to make gpg keys and setup git/remote keyservers with it

This option is now available by simply typing in tos gpg or for short tos g If you want to know all available options use tos gpg help


DLLs in the PATH are not searched

Version
Microsoft Windows [Version 10.0.22621.819]
WSL Version

[X] WSL 2
[X] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
Run the following program (windows executable) from WSL1/WSL2 bash. Specify arbitrary DLL file name located in PATH environment variable.
When executed from cmd.exe,  DLL is searched from PATH and is successfully loaded.
When executed from WSL1/WSL2 bash, DLL is not searched from PATH and not loaded, as if SetDefaultDllDirectories() is called.
#include <stdio.h>
#include <windows.h>

int main(int argc, char **argv)
{
    if (argc == 1) {
        fprintf(stderr, "usage: %s DLLNAME\n", argv[0]);
        return 1;
    }
    HMODULE h = LoadLibrary(argv[1]);
    if (!h) {
        fprintf(stderr, "LoadLibrary failed: %d\n", GetLastError());
        return 2;
    }
    FreeLibrary(h);
    return 0;
}

Expected Behavior
DLL in the PATH is searched.
Actual Behavior
DLL in the PATH is not searched.
 Just DLLs in the PATH are not loaded as if
standard PATH search is blocked by the exe via SetDefaultDllDirectories() . If I launch win32 exe via "cmd.exe /c ～" from wsl bash, then DLLs in the PATH are loaded.
This used to work in the past. I found this issue recently (within a month or so). Currently I'm using WSL <IP_ADDRESS>.

Thank you @nu774.
Unfortunately, I can't reproduce the issue if I compile and run the sample program you shared:
./loaddll.exe
usage: loaddll.exe DLLNAME
root ➜ Debug ./loaddll.exe user32.dll
root ➜ Debug echo $?
0
root ➜ Debug ./loaddll.exe doesnotexit.dll
LoadLibrary failed: 126

I suspect that either the dll you're trying to load is being loaded because it's in the current directory or fails to load with WSL because of access issues.

@OneBlue
Hey, don't close the issue!
You couldn't reproduce the issue because you picked user32.dll. Indeed it's in the PATH, but it's in the system directory where the loader searches specially.
https://learn.microsoft.com/en-us/windows/win32/dlls/dynamic-link-library-search-order
You have to pick DLL in the PATH other than system directory (and application directory).

Fair point.
I can't reproduce the issue with a dll in the path either though (C:\tools is in my %PATH%)
root ➜ Debug /usr/bin/file /mnt/c/tools/wsldeps.dll
/mnt/c/tools/wsldeps.dll: PE32+ executable (DLL) (console) x86-64, for MS Windows
root ➜ Debug ./loaddll.exe wsldeps.dll
root ➜ Debug echo $?
0

That's strange... I can reproduce this in two PCs
The following is captured by procmon.exe.
Here, we are trying to load avformat-58.dll.
On WSL, apparently only a few of paths are tried and it gives up.

On windows 11 host (uname -a shows 4.4.0-22621) it's NG.

I believe I'm having this same problem. Running exe files from WSL does not find DLLs in my path, or in the working directory. It only finds DLLs in the same directory as the exe.

I believe I am encountering the same problem on WSL2.


[v1.9] daemon: Avoid blocking datapath on node discovery

Previously, the datapath relied on node discovery completing. With a
kvstore configured, this meant that node registration would also need to
complete first.
If the kvstore is deployed as pods intended to be managed by Cilium,
then this creates a chicken and egg problem. The kvstore pod cannot come
online because the datapath is blocked. The datapath is blocked because
the agent cannot register the node into the kvstore. And finally, the
node cannot be registered into kvstore because kvstore is not online.
Here's how the issue manifested itself:
$ kubectl -n cilium describe pods etcd-operator-59cf4cfb7c-288qx
Name:           etcd-operator-59cf4cfb7c-288qx
Namespace:      cilium
Priority:       0
Node:           gke-chris-form3-cluster-default-pool-3f30c3b5-zd0f/<IP_ADDRESS>
Start Time:     Fri, 15 Jan 2021 14:56:24 -0800
Labels:         io.cilium/app=etcd-operator
                pod-template-hash=59cf4cfb7c
...
Events:
  Type     Reason                  Age   From               Message
  ----     ------                  ----  ----               -------
  Normal   Scheduled               83s   default-scheduler  Successfully assigned cilium/etcd-operator-59cf4
  Warning  FailedCreatePodSandBox  22s   kubelet            Failed create pod sandbox: rpc error: code = Unk
5bf622367" network for pod "etcd-operator-59cf4cfb7c-288qx": networkPlugin cni failed to set up pod "etcd-op
ent client after 30.000000 seconds timeout: Get "http:///var/run/cilium/cilium.sock/v1/config": dial unix /v
Is the agent running?
  Normal  SandboxChanged  21s  kubelet  Pod sandbox changed, it will be killed and re-created.

To break the chicken and egg problem with the datapath, we need to split
the "node discovery" ((*NodeDiscovery).StartDiscovery()) into two
separate phases: (1) local node state is initialized and (2) node
registration which includes syncing to the kvstore if that's configured.
The (*NodeDiscovery).Registered channel will close when (2) completes.
The new channel (*NodeDiscovery).LocalStateInitialized will close when
(1) completes.
This will unblock the datapath from the implicit dependency on the
kvstore and allow the kvstore pods to come online (Cilium generates
endpoints for them), while node registration into the kvstore continues
on in the background.
Fixes: 43997f5f0 ("loader: Wait for node configuration to generate datapath")
Related: 7045103aa ("daemon: Move KVStore initialization earlier")
Co-authored-by: André Martins<EMAIL_ADDRESS>Co-authored-by: Joe Stringer<EMAIL_ADDRESS>Co-authored-by: Paul Chaignon<EMAIL_ADDRESS>Co-authored-by: Kornilios Kourtis<EMAIL_ADDRESS>Signed-off-by: Chris Tarazi<EMAIL_ADDRESS>Fix bug Ciilum hangs with kvstore configured

test-backport-1.9

test-backport-1.9

test-backport-1.9

test-backport-1.9

test-missed-k8s

test-missed-k8s

I've dropped the last commit from this PR which was incrementing the timeout for the k8s jobs. The CI had passed so there's no need to re-run it again.

I've dropped the last commit from this PR which was incrementing the timeout for the k8s jobs. The CI had passed so there's no need to re-run it again.


feature: search blog posts and trace search logs

[x] prepare Elasticsearch (on Docker)
[ ] connect to Elasticsearch from webapp
[ ] insert articles to DB and Elasticsearch (from webapp?)
[ ] prepare and apply mapping templates for blog articles
[ ] output search logs
[ ] collect search logs by fluentd
[ ] prepare fluentd-plugin-elasticsearch
[ ] inserts search logs to Elasticsearch
[ ] prepare mapping templates for search logs
[ ] (optional) show search logs on Kibana

[ ] install Kuromoji Analyzer for taming Japanese text.

fixes in #11


Default region

This is a rebase and minor tweak of @noggleboggle 's work in https://github.com/rusoto/rusoto/pull/521 .  Fixes https://github.com/rusoto/rusoto/issues/497 .
Did a quick test and it works as expected.

Appveyor is being fussy due to https://github.com/appveyor/ci/issues/1426 .


CodeSee take 2

Hey @jleven, I wasn't able to push to your branch, hence the new PR.
The issues we ran into on the call seem to be resolved - the only wrinkle on our end was that process/browser wasn't in all the places it needed to be.  Now it is.
Closes: #40694

Take 2. Lights, camera, action!
I'll try to poke at this sometime today. If you're around I'll probably have hundreds of questions when I pull this down.

Should be good to go - I played with it a bit during the meeting. Seems quite helpful.


Nice sample!

I added this sample to https://github.com/jsuarezruiz/xamarin-forms-goodlooking-UI
Good job!

thank you


Update README.md

Fix change log since last PR was mistakenly added to the release.

I missed that the release had just been made to this repo, so I didn't see that the PR wrongly wanted to merge in it at the wrong place. This PR resolves this by moving the change log entries to the Unreleased section.

This change is

Reviewed 1 of 1 files at r1.
Review status: :shipit: complete! all files reviewed, all discussions resolved

Comments from Reviewable

Codecov Report

Merging #210 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #210   +/-   ##
===================================
  Coverage    81%    81%
===================================
  Files        14     14
  Lines      1214   1214
  Branches     10     10
===================================
  Hits        990    990
  Misses      214    214
  Partials     10     10


Table: fix params on docs

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's contributing guide (中文 | English | Español).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

table.d.ts 文件不需要更新一下吗？

只是文档写错了，代码没改。看了一下 table.d.ts，少了一些属性，我加一下。

@ziyoung 好了


fix typing on search/match/findall/sub/...

you can run mypy tests and python tests/api/__init__.py to verify typing are both valid at runtime and typing level.
this should suppress https://github.com/ddn0/google-re2-stubs/pull/4 , patch are included in this PR.

Looks like I misunderstood how re2 handle inputs and you misunderstood how AnyStr works here.
I find that methods on re2.compile(str(...)).match/findall/sub/... can handle bytes input, so it dones't need to be AnyStr. For python stdlib re, pattern and string must have same type but it's not true for re2, re2 can run bytes pattren on string or string pattern on bytes, which means re.findall("1", b"0 1 2") doesn't work but re2.findall("1", b"0 1 2") works.
And AnyStr is a TypeVar and in class _Regexp(Generic[AnyStr]) it will be narrowed base on the type of re2.compile(), but not narrowed at function/method level by the input string.
This make most methods in _Regexp have wrong typing, I'll send a seprated PR to fix them later

A good thing on Python typing is that even original package doesn't have typing, you can still use typing stub.
If you do not want to maintain this package I can keep maintaining it. re2 looks pretty stable so I guess we won't need to change this package very much.

Thanks!
I learned a lot more about constrained types.
I just had some style issues. And to speed up convergence, I just applied my suggestions to your PR.
Let me know if you have any objections by Tuesday end (anywhere on earth "timezone"); otherwise, I'll merge the PR as it is and prepare a new PyPI release with these changes.

LGTM.
sadly we can't export Match and Regex

new pypi release?

Done!


expose get_icon_from_buffer

I might be missing something, but it looks like it was an oversight not to expose the get_icon_from_buffer function through the application?

I think I was holding off on that until we figured out a way to do that from linux, though I honestly can't remember. Right now your patch is failing on CI due to missing that. I guess we could throw a stub function in the linux code, so it'd at least work on windows?

For Windows and Mac. As far as I can remember, I have already fixed that in context of https://github.com/qdot/systray-rs/pull/22

Closing as this is exposed on windows (on master) and macos (on the macos branch) right now, still gotta figure something out about linux.


take optional query_group as command line arg

Small PR that adds support for setting the queue that the queries run on.

Thanks Josh - good addition.


Update spelling mistake in README.md

Thanks for raising this @jennygrahamjones!
I moved your commit to #548 to get our CI running.


Fix generate config typo

This fixes an import typo, allowing the client to be created through params rather than from config file.

Thanks for submitting this fix


fix fix http url with incorrect character notation

Proposed changes
Single replica load, compaction, cloning or others, may result in a 404 error if the parameter name contains illegal symbols
Issue Number: close #xxx

run buildall

run buildall


Brain wide map link

This will send to the general page of the project. I don't know what the current link was supposed to do.

It's supposed to lead to a file directory structure showing their source storage pattern

Thank goodness we copied the directory structure explicitly into the example (minus the ...) but IDK what good the generic link will do for helping explain the use case - maybe just no link at all then?

Oh, IDK - the link still works for me: https://ibl.flatironinstitute.org/public/

It does work for me locally. Maybe it should go on the ignore list, most likely they blocked access to bots.

The CI checker on other PRs has been flickering but not regularly failing - usually if I rerun it once it's fixed itself, so might just be slow server response times or something

All right, let's give it some time. I think that if it keeps being annoying for a month or so, then the link should get the honor of being in the ignored list.

@h-mayorquin BTW the IBL link here has been stable for a while it seems so I think we can close
Problem lately has been SSL certificate of FicTrac: https://github.com/catalystneuro/neuroconv/actions/runs/8649412894/job/23715562511?pr=800#step:6:456
This is why I usually give it a week or more of consistent failures before even thinking about opening a PR to try to address them ourselves

OK, a week of consistent failure seems like a good criteria to open a PR. That's great. Let's close.


CameraUpdates overwrite each other (no parallel camera updates)

Note
This is probably rather a missing feature than a bug. However, the API suggests that this should be possible, but it turns out that it isn't.
Description
Any new camera update posted to MapLibreMap::easeCamera cancels the old camera animation and replaces it with a new camera animation.
To Reproduce
Calling e.g.

val zoomTo = CameraUpdateFactory.newCameraPosition(CameraPosition.Builder().zoom(18).build())
map.easeCamera(zoomTo, 1000)

delay(500)

val moveTo = CameraUpdateFactory.newCameraPosition(CameraPosition.Builder().target(12.0,15.0).build())

map.easeCamera(moveTo, 1000)

will cancel the zooming to zoom=18 after 500ms and instead (rather than additionally) move to the given target position. I.e. any new camera position update animation will overwrite the last.
Expected behavior
The animations of different properties of the camera (zoom, tilt, position, rotation, ...) should run in parallel if they don't conflict each other.
Video
In this video, we see that the position is being updated in an animation every second or so to focus on the current location. When the user taps the zoom-in button, the move-to animation is cancelled or the other way round (the zoom-in animation is cancelled), depending on the timing:
https://github.com/user-attachments/assets/4d6529fa-9de5-44bb-aafc-ee21fe746fd1
Important: This is not only an issue with zoom vs position. At any time, the user may also press a button to change the map's rotation (the compass button) or tilt (changing from (car-)navigation-mode to just-follow-me-mode) while the position is continuously updated. Also, it is possible for the continuously updated position to interrupt a zoom-to or zoom-back animation as seen in the video in #2789
Platform information (please complete the following information):

Platform: Android, any version
Version 11.2.0

Additional context
Originally reported in https://github.com/streetcomplete/StreetComplete/issues/5860 and in https://github.com/streetcomplete/StreetComplete/issues/5857 (for padding-animation)

(Regarding the labels: This is probably rather a missing feature, but an issue for cpp-core, as animations are implemented in native code.)

Would just including the options from the previous update in the new one be an option?

You mean the previous animation target? I.e. animation A starts with zoom by +1 (zoom 17 to 18), then while animation A is still running, animation B starts with a new position?
If your suggestion is to include the zoom target of 18 in the animation B that replaces animation A: I think this behavior would be suboptimal and can lead to unexpected results that feel like bugs.
Imagine the animation A had a duration of 1s, but animation B has a duration of 4s. Then suddenly, also the zooming-in would slow down a lot. And even if the duration of animation B was just 1s, too, and animation B was started 0.5s after animation A was started, that would still result in the animation time of the zoom doubling.
The correct way to handle parallel animations, I'd say, is to animate each separate property (zoom, tilt, rotation, position, padding) separately and only replace a running animation with a newly posted animation if it animates the same property.
FYI, I wrote in the chat today, a slightly different approach:

Back when I used Tangram-ES for display of the map, I remember that that library also didn't support parallel camera animations. (Initially, they did, but it was kind of buggy so they removed it without replacement.) So, what I did was to implement this replacing the normal animations API: CameraManager.kt
In a nutshell, I re-implemented all animating of the camera with Android ObjectAnimators. They animate a set of camera properties to the target value(s), always starting from the current camera properties and cancelling any prior animations whose animated properties overlap on start. (E.g. new animation of zoom+tilt cancels prior animation of zoom+rotate but doesn't cancel prior animation of position). I think when a gesture was detected, I just called cancelAllAnimations on that class.


tinyusb_soc.py - ValueError: Memory map has been frozen

I am trying to use tinyusb_soc.py to evaluate this core. That part of the repository seems to be unmaintained but I had to try it as I could not find any other examples.
I am greeted with this exception on line 87. The memory map of SimpleSoC seems to be already fixed when returning from the constructor.
Should I dig further into this path? Or is there other documentation or examples for evaluating Luna as a SoC peripheral?

In latest Amaranth assigning a memory map to a bus freezes the memory map preventing any modification afterwards.
Fix: luna/gateware/soc/memory.py line 88
    self.bus = wishbone.Interface(addr_width=self.local_addr_width, data_width=data_width, granularity=granularity)
    self.bus.memory_map = memory.MemoryMap(addr_width=self.bus_addr_width, data_width=granularity)
    self.bus.memory_map.add_resource(self, size=2 ** addr_width)

becomes:
    self.bus = wishbone.Interface(addr_width=self.local_addr_width, data_width=data_width, granularity=granularity)
    memory_map = memory.MemoryMap(addr_width=self.bus_addr_width, data_width=granularity)
    memory_map.add_resource(self, size=2 ** addr_width)
    self.bus.memory_map = memory_map

Thank you for reporting this!
We have an open PR bringing SimpleSoC up to date that is waiting on review:
https://github.com/greatscottgadgets/luna/pull/188

Thank you for updating the SimpleSoC core! Looking forward to it.


Connecting with "wss://"

I've compiled the library successfully and connecting to the localhost with "ws://" URI works fine.
But when I try to connect to the external server (which uses encryption) "wss://" fails with the following:
[2016-05-08 17:07:27] [connect] Successful connection
[2016-05-08 17:07:28] [error] handle_read_http_response error: websocketpp.transport:7 (End of File)
What are the steps I need to take in order to connect to a server with TLS encryption? Thanks!

You need to recompile the lib with SIO_TLS macro to make it a secured socket.io client.

Thanks for the reply reply but could you show me how? I've tried make CXX-flags=-DSIO_TLS to no avail


OTA Agent : eImageState consistency

Describe the bug
The value of the OTA Agent member eImageState is not consistent.
The value is set upon  validation of the OTA (rejected or accepted).
The value is not updated when a new OTA is processed.
When the OTA agent request activation of the image, the value return is what was set last ( Rejected or Accepted ) which is inconsistent with the OTA Agent initialization value : eOTA_ImageState_Unknown.
Expected behavior
When the OTA Agent requests activation, the eImageState provided by OTA_GetImageState should be consistent and equal to eOTA_ImageState_Unknown.
A new state could be made to identify a new image eOTA_ImageState_New.
To reproduce

Add a printout of the eImageState before performing activation.
Perform an OTA. Before calling activation, the value of eImageState should be 0.
Without any reboot after successful validation, perform another OTA
Before calling activation, read the value of eImageState.

Additional context
The OTA are performed on ESP32.
Thank you!

Patch to set the eImageState before calling the activation callback :
0002-OTA-Agent-image-state-inconsistent-fix.patch.txt

Hi,
Please let us know if you still have this issue on latest Amazon FreeRTOS release.

We're closing this issue for now, if you have further questions you can reopen this issue.


The "cancel search" icon could be improved

In my opinion, the new icon for "cancel search" looks like a person icon.
A suggestion would be a "stop sign". Maybe something like this?

PS: thanks for working VSCode and keep improving the UX for millions of developers out there! ✨

cool. thanks for sharing the iterations and reconsidering this.
As you know, this icon is not that important since icon appears only when the search is taking a long time - usually when the computer is a bit slow or you're searching in huge folders.
All the best

@gianpaj Thanks again for the feedback. I'm going close this issue since this is as-designed and to also keep the number of issues in our inbox on a manageable level. We look at the number of votes an issue has received and the number of duplicate issues filed. More information can be found on our issue grooming post.


[backport] Warn NaN in FP16 mode in sentiment example

Backport of #7356

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 1f19f105ff38e8a599761acee53b21421804a780, target branch v6) failed with status FAILURE.

LGTM! The CI failure is not relevant to this change.


Josephus problem recursion in C++

code for Josephus problem

I want to work on this in c++. Please assign me @Anshul727

Hello, Can you please assign this issue to me?

@ritikbanger @Anshul727 pls assign me this issue I have the codes

Hello Anshul & Ritik,
Can you assign me this issue, I'd like to contribute.


Version is not supported in this release

I have created an RPM for a python script using the docker image v1.7.5
When I try and instantiate the service after activating I get the following message
sbx-n9kv-ao(config)# nxsdk service-name FibrePorts.py
Error: Service FibrePorts.py Version is not supported on this release, check supported versions

This application works with a previous RPM I created using docker image v1.
Also tried example application
sbx-n9kv-ao(config)# nxsdk service-name pbwMonitor
Error: Service pbwMonitor Version is not supported on this release, check supported versions

I am testing this on a N9000v
Nexus 9000v is a demo version of the Nexus Operating System

Software
  BIOS: version
  NXOS: version 9.2(1)
  BIOS compile time:
  NXOS image file is: bootflash:///nxos.9.2.1.bin
  NXOS compile time:  7/17/2018 16:00:00 [07/18/2018 00:21:19]

Which NXOS versions are supported

Can you please attach

the output of "show nxsdk internal service" command from your switch?
the rpm package name for your application?

Closing this issue. Please reopen if you are still encountering this issue.


Mission Leave, Join and Status update

What does this PR do?

[ ] Implement the action to leave a mission
[ ] Implement the action to join a mission
[ ] When all the actions are triggered they change the status of the badges

Issue implemented above
#10
#9
#6

Thank you @helio3197


Label for security PRs

I've reported this several times in the past, such as:

https://github.com/dependabot/feedback/issues/493#issuecomment-497760182
https://github.com/dependabot/dependabot-core/issues/1998
https://news.ycombinator.com/item?id=27933085

We'd like a label for all PRs that Dependabot makes to address security fixes. The label could be customizable or just hardcoded to security as pre-GitHub-native Dependabot used to do. That way we could do a GitHub Issues search like is:open is:pr label:security org:<our-org> and see/merge them all quickly. (We also have a lot of automation we wrote for security PRs from Dependabot from before GitHub-native removed this functionality.) This is a pretty big pain point for us.
The impression I had from the conversation in the third link is that some solution here should be possible. Is that still the case?

Hi @JacobEvelyn,
Thank you for your feedback!
We are now tracking this feature as part of a larger one internally around custom label creation for Dependabot.
Right now, the Dependabot team is unable to take on this work, but we will try to prioritize this and other community feature requests in the next quarter!

Related:

#5825

We had some internal discussion on this, and the summary was:

If users opt into adding labels on their PRs, that's on them and we don't mind supporting it. We just don't want to add that functionality by default because it can have the effect of disclosing vulnerabilities before maintainers are ready to address them.

However, from an implementation perspective, we want to avoid the business of managing labels wherever possible... it is a morass of edge cases.
It also makes it harder to keep PR creation logic separate from "generate the update" logic... rather keep that interface simple if possible (although it may be impossible here) so that we can keep it pluggable... long term, I'd like for us to extract out the GitLab, BitBucket, ADO, etc PR creators to separate plugins that the community can maintain. They are the ones who are impacted if a feature is missing/broken, so they'll best know what to add/how to test and we don't have the cycles to do that.
A better solution IMO for security labels is if we could figure out a way for https://github.com/dependabot/fetch-metadata/ to post-apply the label... But I don't think we expose the necessary data in a way that fetch-metadata can access it?
Perhaps we could wire in some metadata to:

https://github.com/dependabot/dependabot-core/issues/4680

Such that security advisory link is exposed to the GitHub Actions workflow for users to do whatever they want with that metadata? I haven't really dug into the guts of fetch-metadata, or passing metadata through GitHub actions workflows, so not sure what's possible from an implementation perspective.
Anyway, no concrete next actions just yet, but wanted to keep folks in the loop.

Such that security advisory link is exposed to the GitHub Actions workflow for users to do whatever they want with that metadata?

This is probably more relevant for GitHub ecosystem only because for all other ecosystems user would need to provide the advisory data beforehand anyway. So as far as severity label creation go, we would already know all metadata regarding the severity and potentially other data which we might want to expose as labels.

I'd like for us to extract out the GitLab, BitBucket, ADO, etc PR creators to separate plugins that the community can maintain

I think this would do the trick for non GitHub users of dependabot-core library


Currently there is the array field _circulation.holdings
Do we want to encode implicit information about the type of that hold (loan/request) in the start_date attribute and the position? (If start_date is older/equal to today, and the holds is at position 0 -> active loan) (mind import this though)

Coverage remained the same at 100.0% when pulling 54ada5395c906fea810b820e676e83e834f26d4e on mvesper:gim_second_entities into d88e7a22607883855bcc299577077f462c4dc553 on inveniosoftware:master.

I'm not sure about OVERDUE and IN_PROCESS states. Can you remove them for now to make the PR simpler and easier to validate?

Reduced transitions:
               lose_item
   +-------------------+----------+
   |                   |          |
   |                   v          |
   |   +------------+MISSING      |
   |   |                          |
   |   |return_missing_item       |
   |   |                          |
   +   v       loan_item          +
ON_SHELF+------------------------>ON_LOAN
       ^                          +
       |                          |
       +--------------------------+
              return_item

@mvesper we can merge it when you strip the Circulation prefix from class names.


Library project dependency

I build android projects with gradle but use vim+eclim+eclipse to write code. I tried to generate eclipse dependencies with this plugin, but eclipse still can't resolve that dependency. It assumes that there is a jar file in library's bin folder, which is doesn't exists. How can I workaround it?

@kazufukurou there should actually be a jar file in the bin folder of the dependency if that dependent project compiled successfully. Fix any errors in the dependent project first (e.g. perhaps it's using an SDK version that's not installed?)

I use gradle to build app. It doesn't create jar files in bin folder, instead it creates an aar somethere in build folder. I want to be able to create proper eclipse project config with one command, so I could work with eclim.


Cuda error on initialzing Embedding instance in a spawned subprocess aka a celery background task.

Hi , thanks for this awesome lib. Am facing an issue when initializing an Embending instance in a celery background task , below is error trace .
Heres a sniplet of the Celery code:
from txtai.embeddings import Embeddings

@celery_app.task
def ftp_indexer():
    vetorizer_model: str = "BAAI/bge-base-en-v1.5"
    _config = {
	"path": vectorizer,
	"autoid": "uuid5",
	"graph": {
		"approximate": False,
		"topics": {}
	},
	"content": 'sqlite',
	"sqlite": { "wal": True},
   }
   embeddings = Embeddings(_config)

Error Trace:
File "/app/taskflow/integrations/integration.py", line 175, in __init__
taskflow-1  |     embeddings = Embeddings(default_config)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/txtai/embeddings/base.py", line 88, in __init__
taskflow-1  |     self.configure(config)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/txtai/embeddings/base.py", line 737, in configure
taskflow-1  |     self.model = self.loadvectors() if self.config else None
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/txtai/embeddings/base.py", line 888, in loadvectors
taskflow-1  |     model = VectorsFactory.create(self.config, self.scoring)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/txtai/vectors/factory.py", line 44, in create
taskflow-1  |     return TransformersVectors(config, scoring) if config and "path" in config else None
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/txtai/vectors/base.py", line 39, in __init__
taskflow-1  |     self.model = self.load(config.get("path"))
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/txtai/vectors/transformers.py", line 33, in load
taskflow-1  |     return PoolingFactory.create({"path": path, "device": deviceid, "tokenizer": self.config.get("tokenizer"), "method": method})
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/txtai/models/pooling/factory.py", line 46, in create
taskflow-1  |     return ClsPooling(path, device, tokenizer)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/txtai/models/pooling/base.py", line 42, in __init__
taskflow-1  |     self.to(self.device)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1152, in to
taskflow-1  |     return self._apply(convert)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 802, in _apply
taskflow-1  |     module._apply(fn)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 802, in _apply
taskflow-1  |     module._apply(fn)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 802, in _apply
taskflow-1  |     module._apply(fn)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 825, in _apply
taskflow-1  |     param_applied = fn(param)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1150, in convert
taskflow-1  |     return t.to(device, dtype if t.is_floating_point() or t.is_complex() else None, non_blocking)
taskflow-1  |   File "/root/.local/lib/python3.10/site-packages/torch/cuda/__init__.py", line 288, in _lazy_init
taskflow-1  |     raise RuntimeError(
taskflow-1  | RuntimeError: Cannot re-initialize CUDA in forked subprocess. To use CUDA with multiprocessing, you must use the 'spawn' start method

Thanks once more.

Thanks for the suggestion , yes that did solve the issue .


Should we return boolean in the isApplePayAvailable and isGooglePayAvailable method?

I think these methods should return boolean.
And we can use the returned attributes to change the UI behavior.
if (!Stripe.isGooglePayAvailable()) {
  return null
}
return (
  <button>Pay with Google</button>
)

Can we replace these behavior?

Thanks. Capacitor Plugin's method must be Promise base. so I think  is**Available should be resolve/reject .

I had the same thoughts @hideokamoto, but I understand @rdlabo. However here is a some sugar code I use to getting a boolean on a call:
function boolifyPromise(promise) {
  return promise.then(() => true).catch(() => false);
}

//
const isApplePayAvailable = await boolifyPromise(Stripe.isApplePayAvailable());
if (!isApplePayAvailable) {
  return null;
}


Organize output

Issue to track the organization of console output.

[ ] Map all exceptions to specific error codes
[ ] By default output mimimal logging to stdout
[ ] Introduce quiet option (-q --quiet) to output RDF result only
[ ] Introduce verbose options (-v --verbose) to output additional logging
[ ] Log errors to stderr

If I may: Could -q and -v be thought of as mutually exclusive options?
My personal preference would be a default of RDF output only with the option of additional output for debugging, i.e. -v.

Hmm yeah, I was doubting which route to take here. Could you elaborate on why that would be your preference?

Sure :-) I think it is primarily a question of convenience: if everything goes well (exit code 0), the output graph is written to stdout without any further logging output or configuration flags needed.
In my workflow, I control things with Python subprocesses and directly continue with what stdout returns.
If the output seems strange (although the engine did not throw an error), then I would further investigate with -v to see what happened.
I think in case of a non 0 exit code, it would be convenient to directly write an error message to sdterr. Not sure if there could be categories of errors like things done wrong by the user (syntactically invalid Turtle, missing things in the mapping etc.) or things that the engine cannot process properly for some reason. I am primarily familiar with REST APIs and there you have categories which indicate that something was the user's fault or things like internal server errors expressed with HTTP status codes.
It might also be worth looking at https://github.com/RMLio/rmlmapper-java#cli.
For me, having two implementations available to process my RML mappings with is a real plus, especially if their APIs are quite similar (it makes it easier to exchange one for the other in your workflow).

If you like, I'd be open for a short call this week to discuss this. Maybe also with @semanticfire? He also made a WAR file which I find useful.

Sure :-) I think it is primarily a question of convenience: if everything goes well (exit code 0), the output graph is written to stdout without any further logging output or configuration flags needed. In my workflow, I control things with Python subprocesses and directly continue with what stdout returns.
If the output seems strange (although the engine did not throw an error), then I would further investigate with -v to see what happened.

Cool. I just wonder if this will be true for the majority of users. But I guess it is simple enough to add -v(vv) for more verbosity.

I think in case of a non 0 exit code, it would be convenient to directly write an error message to sdterr. Not sure if there could be categories of errors like things done wrong by the user (syntactically invalid Turtle, missing things in the mapping etc.) or things that the engine cannot process properly for some reason. I am primarily familiar with REST APIs and there you have categories which indicate that something was the user's fault or things like internal server errors expressed with HTTP status codes.

Yeah idea would be to have error codes for specific errors and have the message output to sterr.

It might also be worth looking at https://github.com/RMLio/rmlmapper-java#cli.

I'll take a look.

For me, having two implementations available to process my RML mappings with is a real plus, especially if their APIs are quite similar (it makes it easier to exchange one for the other in your workflow).

If you like, I'd be open for a short call this week to discuss this. Maybe also with @semanticfire? He also made a WAR file which I find useful.

Sounds good. I'm available this week.

Sounds good. I'm available this week.

Great, what about Wednesday or Thursday afternoon at 14h?

Both times work for me

Great, I've sent you an invite for tomorrow!


Adding a VM implementation detail around expected import signatures.

This allows modules to snoop what importers are expecting the exporters to provide. This information is optional and not included in all cases as not all importers have the information available.

Nit: const reference instead of const pointer?

No references in C :)


Added scroll table and normal table css

Based on #911 I added basic table css for desktop version.
Because the comparison table is just to big I added option to display scroll for table on desktop and then mobile version for mobile devices.

@NejcZdovc aside from my two review comments I did test and this does seem to resolve the issue from #911.

@skipjack I don't see your review comments

@NejcZdovc sorry looks like I forgot to submit, you should be able to see them now.

@skipjack added overflow to all tables for desktop and looks good


[uuencode, uudecode]: fix uuencode, uudecode and write their tests

Few changes have been done on uuencode.rs
The crate that was used i.e "uuencode", it seems that it outputs the permission symbols always as "644", which should be in fact the permission symbol of the file being "read", so i've removed the usage of uuencode from uencode.rs and implemented the historical encoding i.e the encoding used by uuencode as default(works well)

the algorithm for historical encoding is mentioned at : https://pubs.opengroup.org/onlinepubs/9699919799/utilities/uuencode.html

btw @jgarzik, do we plan on using spaces for the uuencode
what i mean by that is, i was reading
https://en.wikipedia.org/wiki/Uuencoding and
https://news.ycombinator.com/item?id=38343748
what i found out was the encoder algorithm mentioned in the posixutils specification
doesn't mention anything regarding and makes one use the ascii character at 32 i.e "SPACE"
but the wikipedia and the thread mentions that some encoder uses the character ` instead of the SPACE so that,
so currently its upto us which one we choose, use the character ` instead of space or use the SPACE itself at the default ASCII position of 32
FYI: is preferred(and also most tools are implemented to replace spaces with that character, including the official implementation at sharutils), the advantage of over SPACE is that it allows the string to be not truncated because of some tool
So, the question is do i use the default SPACE at the ascii position 32 or the character `

Postel's Law is the meta-answer:
"the robustness principle is a design guideline for software that states: "be conservative in what you do, be liberal in what you accept from others". It is often reworded as: "be conservative in what you send, be liberal in what you accept".
uuencode should produce what is maximally likely to be accepted by the widest range of tools.
uudecode should accept what is "out there on the Internet" as existing UUE format.

hey @jgarzik, uuencode and uudecode are ready to be used now


WIP: Adding manifest parameter to 'copy_from_files 'method

Subject: Adding manifest parameter to 'copy_from_files 'method
Feature

Feature

Detail

New parameter for redshift.copy_from_files() : `manifest

Relates

#1142

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 2ad9114b6bbc37e049da43ec7de6eee6f11d60dc
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

One point I was unsure on, is path still required for the method even when using a manifest uri ?

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: f4e7d5873c293936c4db7f77d9d0a584aaa4effb
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@malachi-constant looks like it's either path or manifest

Just a thought - maybe manifest could be a boolean param that would determine if path is just an S3 path or path to the manifest? Otherwise making path optional would work just as well since it wouldn't be a breaking change

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 40638b72ddab998de155854f9114c0d93739a7ed
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 9261ae6d5eef954ed25e1678e7b93e729624f076
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 726c27002367f3abf79db3c9555969fcedfd5d7e
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 80cdaad2d68b8392d17ed7b8d41dd6667ecd0ad1
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Avoid duplicates in ids and entities for Entity store

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[X] Support request
[ ] Other... Please describe:

Current behavior
Given this set of data:
[{id: 'doc-0001', name: 'My document name'}, {id: 'doc-0001', name: 'My document name duplicated'}]
and this model:
interface DocumentModel {    id: ID;    name: string; }
when I use query.getCount() I got 2 instead of 1, that happen because ids (inside of my state) is not removing duplicates.
Expected behavior
non duplicated in both ids and entities
Minimal reproduction of the problem with instructions
https://stackblitz.com/edit/angular-facb25
Environment

Angular version: 7.2.0
Akita: 3.1.2 (But i tested on latest stable version and still happen)

Browser:
- [X] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

Why do you have duplicates entities in your store?

Before adding the data into the store, filter duplicates. Use [lodash](https://lodash.com/docs/4.17.11#uniqBy) or pure JS.


Small error in CDK Workshop Code?

:bug: Bug Report
What is the problem?

Reproduction Steps

Verbose Log

Environment

CDK CLI Version:
Section:
Browser:
Language:

Other information

@RajivKYadav Thanks for submitting this over here. (For readers: this was originally a bug in the CDK Guide repo. See https://github.com/awsdocs/aws-cdk-guide/issues/105)

@RajivKYadav I can't reproduce this problem. I've run through the CDK Workshop again and I did run into a problem at the same spot, but it turned out I had some stale package (in particular, my DynamoDB Table Viewer wouldn't upgrade to the latest version because I had a handful of modules stuck at 0.3.8). Once I corrected this, the code in the Workshop worked without modification.
Try deleting your CDK package and reinstalling. On Windows this is:
rmdir /s /q %appdata%\npm\node_modules\aws-cdk
npm install -g aws-cdk

Then update everything:
npm update
npm update -g

I also had to stop my npm run watch window and restart it.
If you can, give it another try. I don't think there's actually a problem with this step.

HI Jerry,
Thanks for the follow up.  I'll give this a try over the weekend and get
back to you.  What you are saying would make sense as it was the viewer
that was giving an error and looks like that might have been fixed in the
updated version of the viewer.
Best Regards,
Rajiv
(508)817-6147
On Thu, Aug 15, 2019 at 6:59 PM Jerry Kindall<EMAIL_ADDRESS>wrote:

@RajivKYadav https://github.com/RajivKYadav I can't reproduce this
problem. I've run through the CDK Workshop again and I did run into a
problem at the same spot, but it turned out I had some stale package (in
particular, my DynamoDB Table Viewer wouldn't upgrade to the latest version
because I had a handful of modules stuck at 0.3.8). Once I corrected this,
the code in the Workshop worked without modification.
Try deleting your CDK package and reinstalling. On Windows this is:
rmdir /s /q %appdata%\npm\node_modules\aws-cdk
npm install -g aws-cdk
Then update everything:
npm update
npm update -g
I also had to stop my npm run watch window and restart it.
If you can, give it another try. I don't think there's actually a problem
with this step.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/aws-cdk-intro-workshop/issues/70?email_source=notifications&email_token=AHYEYKDAF7E3AIA6P4FRQOTQEXNUZA5CNFSM4IL5NJWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4NG7EY#issuecomment-521826195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHYEYKDCVJCU7NGF42FLUFLQEXNUZANCNFSM4IL5NJWA
.

This was fixed in #84 , can we close this @eladb ?


New command: Get-JAR

I'm working on a few spaces at the same time, and the current jars at are on a specific space sometimes get mixed up.
At the moment there is a comfortable way to get the current manifests
I would like to be able to do something like this:
cf get-jar appname -s spacename
Thanks!!

Hi @ytupman
Can you provide more detail? I have absolutely no idea what you're asking for. jars of what?
Is this specific to Java applications? (and you know the cf CLI is generic to cater for many programming languages?)
Thanks,
Dies Koper
CF CLI PM

Hi @dkoper
Sure, i meant to extract the java applications that are upload to the CF.
Hope you understood me now. :)

You can use the cf files command to download individual files from your application. Or cf ssh if your app is on Diego.
There is also a CC API to download application bits (http://apidocs.cloudfoundry.org/231/apps/downloads_the_bits_for_an_app.html) - you can use cf app APPNAME --guid to retrieve your app's guid and pass it to cf curl to start the download.
Does that help?


[Azure Search] adding filter property for Autocomplete in SDK

includes generated code from swagger spec from azure-rest-api-specs:master
added custom code to AutocompleteParameters for POST flavor of the API
added tests for the property

@brjohnstmsft FYI.

@ishansrivastava90 The build failure appears to be a bug in another Azure service's SDK. I'll merge from master again to resolve it. Sorry for the inconvenience.

Will merge once builds are passing

@brjohnstmsft @dsgouda The CI/Travis-CI check is failing consistently with the message,
_"E: Failed to fetch https://packages.microsoft.com/ubuntu/14.04/prod/dists/trusty/main/binary-amd64/Packages.gz  Hash Sum mismatch
E: Some index files failed to download. They have been ignored, or old ones used instead.
The command "sudo apt-get update" failed and exited with 100 during." _
Any ideas as to get this to pass?

@ishansrivastava90 We're taking a look

@dsgouda Both @brjohnstmsft  and I are going to be OOF for the 1st part of the next week. If the issue is determined and fixed before we are back, feel free to merge the PR.

@ishansrivastava90 Please pull lkatest changes from master and update this PR, that should fix the CI issues


Provide a Security Policy

closes #233
Use the text from the go-algorand project and direct users to the same resources.

BLOCKER: AlgoKit is not mentioned within the "Assets in scope" section of the Immunefi bounty page for Algorand:
https://immunefi.com/bounty/algorand/

It looks like the foundation is a different legal entity, so the text needs to be modified anyways.

AlgoKit isn’t in scope for the bug rewards program.

AlgoKit isn’t in scope for the bug rewards program.

I see. I've updated the document, you can either merge or take this as a template to provide a document.

Hi Abe, thanks for bringing this up, we're in the process of creating a disclosure form for the security policy, will commit this once it's done and we can then merge the pr

in the process of creating a disclosure form for the security policy

I think github has some decicated functionality for this, which in combination with a contact email should serve all cases.
Note that you do not need to everything in one chunk. You can e.g. use this PR initially, then later update it (with your form, or with githubs security-report functionality).
Anyway, leaving it to you to decide.
Please feel free to use/modify or close this PR as you see fit.

@Loedn, after rethinking this: the PR can go in "as-is", and you can add a link to your form once it is ready.
This is really nothing special.


Added bower.json file

Hi!
I have added the bower.json file, so gulp build tools can (wiredep for example) automatically inject this file.

Awesome thanks!


[SPARK-43489][BUILD] Remove protobuf 2.5.0

What changes were proposed in this pull request?

Spark does not use protobuf 2.5.0 directly, instead, it comes from other dependencies, with the following changes, now, Spark does not require protobuf 2.5.0 (please let me know if I miss something),

SPARK-40323 upgraded ORC 1.8.0, which moved from protobuf 2.5.0 to a shaded protobuf 3
SPARK-33212 switched from Hadoop vanilla client to Hadoop shaded client, also removed the protobuf 2 dependency. SPARK-42452 removed the support for Hadoop 2.
SPARK-14421 shaded and relocated protobuf 2.6.1, which is required by the kinesis client, into the kinesis assembly jar
Spark itself's core/connect/protobuf modules use protobuf 3, also shaded and relocated all protobuf 3 deps.

Why are the changes needed?

Remove the obsolete dependency, which is EOL long ago, and has CVEs CVE-2022-3510 CVE-2022-3509 CVE-2022-3171 CVE-2021-22569
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Pass GA.

https://github.com/apache/spark/blob/46332d985e52f76887c139f67d1471b82e85d5ca/connector/kafka-0-10-assembly/pom.xml#L62-L66
This one should be 2.5.0 before, I think we should manually specify its version to keep the behavior unchanged

Thank you for doing this with nice analysis. Ya, definitely, this was the goal. BTW, could you send an email to dev@spark because this is an important removal of dependency, @pan3793 ?

+1, Agree

@dongjoon-hyun @LuciferYang FYI, I have sent a mail to the mailing list
https://lists.apache.org/thread/xzrov348c3dq0d8jwcxf4j7fk7lb3r92

https://github.com/apache/spark/blob/46332d985e52f76887c139f67d1471b82e85d5ca/connector/kafka-0-10-assembly/pom.xml#L62-L66
This one should be 2.5.0 before, I think we should manually specify its version to keep the behavior unchanged ?

@LuciferYang The version here does not matter. What's matter is the <scope>provided</scope>, which is used to exclude deps from the assembly jar.

Merged to master


CrashReporter: Clicking on bitcoin amount header in Payments breaks MBHD

Clicking on the bitcoin amount column header creates the stacktrace:
e":"Uncaught exception","logger_name":"org.multibit.hd.core.error_reporting.ExceptionHandler","thread_name":"AWT-EventQueue-1","level":"ERROR","level_value":40000,"stack_trace":"java.lang.ClassCastException: com.google.common.base.Present cannot be cast to org.bitcoinj.core.Coin\n\tat org.multibit.hd.ui.views.components.Tables$3.compare(Tables.java:282) ~[classes/:na]\n\tat

Looks like the Optional isn't getting explored correctly.

Updated the Comparator for the Coin column t reflect that it is now actually an Optional.
Awaiting review and closing

Verified working. Closing.


LandingPage: UL in top section needs better margins

The paragraph immediately before the list also needs to either get more space from the paragraph above it, or needs to be separated from the UL and given the same formatting as the rest of the text in the section.


Implement /get_limit Endpoint

Description
We need to implement the /get_limit endpoint in the Vault backend. This endpoint is responsible for retrieving the account spending limit.

Response:

Status Code: 200
Description: Account current spending limit
Content Type: application/json
Body:

{
  "limit": "0x0123456789abcdefABCDEF0123456789abcdefABCDEF0123456789abcdefABCD"
}

Notes:
The spending limit is represented in hexadecimal format.
Example
// Import necessary modules
import fastify from 'fastify';

// Create Fastify instance
const app = fastify({ logger: true });

// Define the get_limit endpoint
app.get('/get_limit', async (request, reply) => {
    try {
        // Logic to retrieve the spending limit goes here

        // Return the spending limit
        return { limit: "0x0123456789abcdefABCDEF0123456789abcdefABCDEF0123456789abcdefABCD" };
    } catch (error) {
        // Handle errors
        console.error('Error while retrieving spending limit:', error);
        return reply.status(500).send({ error: 'Internal Server Error' });
    }
});

Tasks:

[ ] Implement the /get_limit endpoint.

yes please do

Hello Lucas, I've set up the project locally on my machine, but I need clarification on the database schema I am to work with.
I've checked the current database schema and non appeared to have a limit field.

This has to be indexed by apibara it will get added later, can you create a mock table that has address as primary key and limit ? Add a todo somewhere to index that from apibara and I’ll create an issue (or if you know how to work with apibara you can directly do it)

Alright, I will create a mock table with the fields you specified.
I'm not familiar with apibara, But I'll check it out .

Hi @LucasLvy I created a PR #61.
It's ready for review.


Let account filters apply to -b

Using -b prints all accounts, regardless of current filters.
Only accounts that are included in the current filters should be included

-b, -f, -q, and -a should all just be refactoring into a single print options feature so we can mix and match as we like. Filters are applied inconsistently to all print styles.


Encode the source page URL when building the web intent URL

Hi!
As described in issue #17 it is safer to encode the URL before building the web intent URL (also see https://developer.twitter.com/en/docs/twitter-for-websites/tweet-button/guides/web-intent.html). Ended up using encodeURIConponent instead of encodeURI just like in this example from Codepen: https://codepen.io/sandradudley/pen/KMgJNR
Thanks.
Xavier

Thanks for your contribution @XavRsl  ! I'm looking to merge it and publish new version

Everything good, last version is up on NPM : https://www.npmjs.com/package/share-selected-text
Thanks a lot !


Fix Bazaar cloning in makepkg

This is because makepkg was requiring the cloned repository to be removed across builds, because it was erroneously thinking that the repository is not a real clone of $source. More info in the commit message.

This has been reported upstream: https://bugs.archlinux.org/task/43448.


Added default value for with_locals option for sentry

We are currently generating a lot of errors since the with_locals option seems to be missing from our sentry client options.
I added a default value if this option is not present in our client options.

@isra17 for review!


mbt build error: could not execute the "make -f Makefile_20210305162355.mta p=cf mtar= strict=true mode=" command: exec: "make": executable file not found in %PATH%

Dear Colleagues,
I am trying to run an "mbt build" on a CAP project.
I am getting the following error:
C:\Users\I072405\Documents\workspace-spring-tool-suite-4-4.9.0.RELEASE\tesSAPBA>mbt build
[2021-03-05 16:23:55]  INFO Cloud MTA Build Tool version 1.1.0
[2021-03-05 16:23:55]  INFO generating the "Makefile_20210305162355.mta" file...
[2021-03-05 16:23:55]  INFO done
[2021-03-05 16:23:55]  INFO executing the "make -f Makefile_20210305162355.mta p=cf mtar= strict=true mode=" command...
Error: could not build the MTA project: could not execute the "make -f Makefile_20210305162355.mta p=cf mtar= strict=true mode=" command: exec: "make": executable file not found in %PATH%
Please provide the following info:

MBT Version: Cloud MTA Build Tool version 1.1.0
OS Version: Windows on My local machine
Project Link: https://github.com/atanumallik1/CAP_BAsic

Steps to Reproduce:

clone the project
run mbt build

Could you please help me how to fix the error.

Hello @atanumallik1,
Please see the following: https://sap.github.io/cloud-mta-build-tool/makefile/
Regards,
Alla


[SP-3694] Backport of PDI-16221 - "file" option of Kitchen and Pan do not work in the file system (7.1 Suite)

@pamval @mchen-len-son @mdamour1976 please review

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/kitchen/Kitchen.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.auth.core.impl.DefaultAuthenticationConsumerFactory

Branch Change:  + 7.14%
Complexity Change:  + 7.69%

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.39%:small_red_triangle_down:
Complexity Change: -.51%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Instruction Change:  + .09%

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.35%:small_red_triangle_down:
Complexity Change: -2.70%:small_red_triangle_down:
Instruction Change: -.18%:small_red_triangle_down:
Line Change: -.48%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:


RequestRejectedException

What version of pinpoint are you using?
master
Describe the bug
RequestRejectedException occurs in certain environment after updating spring security version from 5.3.6 to 5.4.6.
Although it does not occur all the time, temporary workaround to allow all header values in Spring HttpFirewall might be useful.
06-16 18:44:22.022 [nio-9000-exec-3] ERROR o.a.c.c.C.[.[.[.[dispatcherServlet]      -- Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Request processing failed; nested exception is org.springframework.web.socket.server.HandshakeFailureException: Uncaught failure for request http://pinpoint.navercorp.com/agent/activeThread.pinpointws; nested exception is org.springframework.security.web.firewall.RequestRejectedException: The request was rejected because the header value "ì<86>¡í<98><9c>ì§<84>" is not allowed.] with root cause
org.springframework.security.web.firewall.RequestRejectedException: The request was rejected because the header value "ì<86>¡í<98><9c>ì§<84>" is not allowed.
at org.springframework.security.web.firewall.StrictHttpFirewall$StrictFirewalledRequest.validateAllowedHeaderValue(StrictHttpFirewall.java:739) ~[spring-security-web-5.4.6.jar!/:5.4.6]
at org.springframework.security.web.firewall.StrictHttpFirewall$StrictFirewalledRequest.access$000(StrictHttpFirewall.java:605) ~[spring-security-web-5.4.6.jar!/:5.4.6]
at org.springframework.security.web.firewall.StrictHttpFirewall$StrictFirewalledRequest$1.nextElement(StrictHttpFirewall.java:647) ~[spring-security-web-5.4.6.jar!/:5.4.6]
at org.springframework.security.web.firewall.StrictHttpFirewall$StrictFirewalledRequest$1.nextElement(StrictHttpFirewall.java:637) ~[spring-security-web-5.4.6.jar!/:5.4.6]
at org.springframework.security.web.firewall.StrictHttpFirewall$StrictFirewalledRequest$1.nextElement(StrictHttpFirewall.java:646) ~[spring-security-web-5.4.6.jar!/:5.4.6]
at org.springframework.security.web.firewall.StrictHttpFirewall$StrictFirewalledRequest$1.nextElement(StrictHttpFirewall.java:637) ~[spring-security-web-5.4.6.jar!/:5.4.6]

org.springframework.security.web.firewall.RequestRejectedException: The request was rejected because the header value "ì<86>¡í<98><9c>ì§<84>" is not allowed.

This happens when ServerHttpRequest.getHeaders() tries to validate all headers (even the ones I don't use) via spring-projects/spring-security#8644, and it seems like the custom nginx filter puts korean characters into the header value.
Since I found the source of these weird characters, this commit is not needed, thereby closing this issue.

@ga-ram Thanks for the further info link.


Implemented SAH BVH with Binning, minor fixes

There are some implementation quirks (with respect to parameters) which must be properly defined in constexpr constants instead of hardcoded.
-Implemented SAH BVH with Binning based on PBRT
-Fixed ray-triangle intersection by making epsilon lower for the det > epsilon case.
-Normals now also reversed to proper coordinates.
-BBoxes Union() method now fixed.

@stijnherfst  Merge?


How to compile a axs_testing.js that contains all tests?

Hi all!
I just found out about this repo. Great stuff! I'm trying to implement it in our Jenkins environment, using phantomjs, and it kind-of works. However, the bundled axs_testing.js only contains a few tests, and I would love to implement more tests. For example, it doesn't test for contrast or a missing <title> element. How can I tell grunt which tests to include when re-compiling axs_testing.js?
Thanks for the help!

Anyone?

I'm getting warmer to the actual problem. The axs_testing.js file pointed out on the README.md (https://raw.github.com/GoogleChrome/accessibility-developer-tools/stable/dist/js/axs_testing.js) is a file generated on a codebase from 2014 (http://github.com/GoogleChrome/accessibility-developer-tools/tree/3d4893b4ecd0eb8f4765e04479213d04b240f3e0), the stable build.
However, the axs_testing.js included in the git clone is way newer and misses a lot of tests, although these tests are still available in the src folder. Why are they not longer included in the axs_testing.js?
Is this a bug in the closure-compiler?

@BarisW did you ever uncover a solution to this issue? When I build the project and run it, it seems to not catch much. I'm not sure if I'm just missing something or if it's still not functioning correctly. I tried to run the script linked in your second post but it requested some additional parameters for which I haven't previously been prompted.

Looks related to https://github.com/GoogleChrome/accessibility-developer-tools/issues/356. You can point to a specific version like this https://raw.githubusercontent.com/GoogleChrome/accessibility-developer-tools/v2.11.0/dist/js/axs_testing.js (2.11.0 seems to have all the tests included)


Add pagination links to STAC output of DAPA endpoints

This should follow the best practices described here:
https://github.com/radiantearth/stac-spec/blob/789e90fba4c7119bc925d26f75f47bacf44c91e6/best-practices.md#using-relation-types
i wonder if we should have a separate issue for making the STAC response follow the 'stac-spec' ?
https://stacspec.org/en/about/stac-spec/

#70 is merged.