Update JNI and private version to released 23.08.0 [databricks]

bump up JNI and private versions to 23.08.0
NOTE: I will trigger CI after JNI and private artifacts are available at maven central

build

please ignore failed spark333test (release private do not include snapshot shims)
mvn[compile,RAT,scalastyle,docgen] / package-tests (333, true) (pull_request)

We had test failures for this. Why is it that we didn't disable the 333 pre-commit?  Any PR after this for 3.8 is also not going to pass.

We had test failures for this. Why is it that we didn't disable the 333 pre-commit? Any PR after this for 3.8 is also not going to pass.

This should be the last change for pre-release, if any followups people could just ignore the optional snapshot shim test.
And yes, we could add some specific control to make sure we skip the snapshot shims in github actions.
Currently due to our auto-merge strategy, we will always update in multiple places to make sure that the change would not also fail/forget required shims the current dev branches.
If needed, we can consider adding some global VAR to control instead of oral notification or code changes in multiple branches


Better Examples of Security

Hi,
I've found the example of ITokenValidator, and understand how to inject and build one, but I'm struggling to understand how Claims are checked.
If this documentation exists, but is not brought together, I'd be glad to do it.

I've recently updated the docs and released a video on authorisation - which includes claims checking.
Let me know if that helps!

cheers , thanks
On Mon, Jul 29, 2019 at 8:25 PM James Randall<EMAIL_ADDRESS>wrote:

I've recently updated the docs and released a video on authorisation -
which includes claims checking.
Let me know if that helps!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JamesRandall/FunctionMonkey/issues/79?email_source=notifications&email_token=AAJVQEQMETWCNX2U4KNXPE3QB47ZHA5CNFSM4IEAU2YKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3BX3KQ#issuecomment-516128170,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJVQEV7D2KUENDOULBNFGTQB47ZHANCNFSM4IEAU2YA
.

--
Linkedin http://www.linkedin.com/profile/view?id=5170079&trk=tab_pro


Proposal: enforce minimal linting rules

I propose to follow the basic principles that were adopted for github.com/kubescape/kubescape and introduce minimal code standards to this repository.
See my PR proposal: #76
NOTE: I now realize that most of the impact here concerns the regostore, which has now migrated to github.com/kubescape/regolibrary. Since that one remains for now a plain copy of this repo, I'll propose similar improvements there.
cc: @matthyx @dwertent @vladklokun

Thanks. Yes, we are moving the package to the regolibrary.
/cc @kooomix please take a look into this

Thanks for the proposal, I'll look into it.

@kooomix   @dwertent  I am going to sync the new rego library with the same linting proposal shortly today

@kooomix  @dwertent  this is here: https://github.com/kubescape/regolibrary/pull/299
I've limited the scope to gitregostore for now, for clarity.

@fredbi thats great, so can I delete gitregostore from opautils?

@kooomix at least you shouldn't be blocked by my pending PR #76  any longer. I'll rebase that one after your move and keep just what pertains to opa-utils. Cheers.


Creates tile_freeze/unfreeze/set_symbols routines for CUDA API

This pull request addresses the following issues in the CUDA API:

Issue #268 : Create hb_mc_device_tiles_freeze/unfreeze routines that sends packets to a list of tiles to freeze/unfreeze them
Issue #270 : Create hb_mc_device_tiles_set_symbols routine that replaces the large section inside hb_mc_program_load and hb_mc_tile_group_initialize_tiles, sending packets to all tiles in a list to set their binary symbols.
Issue #269 : Removed unnecessary uses of malloc and replaced with stack allocation. Also changed uses of "new" to "malloc"
Issue #267 : Removed unnecessary use of uint8_t in hb_mc_wait_for_tile_group_finish_any, other uses of uint8_t in deprecated functions will be addressed in a separate PR.
Issue #272 : Moved all hb_mc_tile_set_***_symbols from bsg_manycore_tile.cpp/h to bsg_manycore_cuda.cpp/h
Issue #271 : Replaced hb_mc_memcpy_to_device/host to upper case HB_MC_MEMCPY_TO_DEVICE/HOST

Rebased and regression tests pass. Ready to merge.

Does this compile?

Does this compile?

Yeah, it doesn't compile for you?

Does this compile and pass on F1?

@drichmond  @max-ruttenberg , compiles and F1 passes. I believe all issues are addressed now. Let me know what you think, thanks a lot!

Hi Borna, what commits on BSG Manycore and BaseJump is this passing with?

I just tested and it also passes with AGFI image v1.0.1, bsg_manycore commit 8241449, basejump_stl commit 2bb5555

Hmm. Now there's a conflict with Makefile.common

Yeah I fixed it, just running a quick cosim to make sure everything is in check

OK we are good to go!

Thanks Borna!
Good commit messages in this one

Thanks a lot, I'm starting to learn :D


[IOTDB-497] Apache Flink Connector Support

We all know that Apache Flink is one of the most popular stream computing engine, which is widely used in IOT scenes.
This module is the IoTDB integration for Apache Flink, including the IoTDB sink that allows a Flink job to write events into timeseries.
The changes are as following:

Flink IoTDB Sink
User defined IoTSerializationSchema  interface and the default implementation DefaultIoTSerializationSchema
UTs for IoTDB Sink in iotdb-flink-connector/test
Examples for IoTDB Sink in example/flink-example
Documents

@jixuan1989 Could you guys have time to take a look?

@jixuan1989 Could you guys have time to take a look?

Hi, Many Thanks! I will do that asap.

I'd like to invite @sunjincheng121 to have a review together :D

I will update the PR after https://github.com/apache/incubator-iotdb/pull/824 getting merged, to add timeseries checks like this:
        for (String path : options.getTimeseries()) {
            if (!session.checkTimeseriesExists(path)) {
                session.createTimeseries(path, TSDataType.TEXT, TSEncoding.PLAIN, CompressionType.UNCOMPRESSED);
            }
        }

I will update the PR after #824 getting merged, to add timeseries checks like this:
        for (String path : options.getTimeseries()) {
            if (!session.checkTimeseriesExists(path)) {
                session.createTimeseries(path, TSDataType.TEXT, TSEncoding.PLAIN, CompressionType.UNCOMPRESSED);
            }
        }

Hi, adding a check is ok.
I just want to point out that from  v0.9.* on, IoTDB supports create timeseries automatically.
Which means you can directly write "insert into..."
Two things that we need to know if we want to use the feature:

You need to modify the configuration file to claim which level of a path is the "storage group". It is 2 by default, which means if you write data into "root.a.b.c.e", then "root.a" is the storage group. Or, you can use session.setStorageGroup() to create a SG manually and then create timeseries automatically.

If automatically creating time series is enabled, then the data type is inferred by the first data points of a time series. e.g., if the first data is an integer, then it is long; if the first data is a float, then it is double, otherwise, it is TEXT.

In most of our actual scenarios, automatic creation will not be enabled, such as Kafka's topic, because it is not conducive to control, but also may bring some risks.

Great thanks for this valuable PR @vesense
I would like to have the detail review  this weekend.(This week is to busy due to these two weeks are the annual evaluation time)
Best,
Jincheng

@jixuan1989 Addressed your review comments, please take a look again. Thanks.

The CI failure is not related to new changes.

Thanks for the update @vesense
I would like to have look at this PR now... :)

Thanks @jixuan1989 @sunjincheng121 for your comments. I have updated the PR, please take a look again. And very sorry for the delay response, because of my busy work. Usually I don't have enough time to participate in until after 11 p.m. or on weekends.

@WeiZhong94 Thanks for your comments. I will take a look later.

Hi  @vesense , I have click resolve conversation for those comments written by Jincheng and me.
Many thanks for your update.

Thanks @WeiZhong94 @jixuan1989
I have addressed your comments, please take a look again.

Thanks for the update, could you please check the CI issue :)

@sunjincheng121 Fixed the random CI build error.

Thanks for the update @vesense  and great thanks for the review.
+1 to merged from my side after fix the code style problem comment by Wei


Please update GitHub About link to qiskit.org/documentation

Right now, it's only qiskit.org.
To fix this, go to the "Code"/home page of the repo. Then click the wheel on the "about" section on the right.

Thanks!

I am in favor of the link remaining as it is, to https://qiskit.org.  That is the web page for Qiskit, after all.

Yeah, I could be convinced either way. cc @javabster @HuangJunye @1ucian0 any opinion?

sorry I'm confused, when did it get changed away from the docs guide link? I like the docs-guide link being in the about section

@javabster this is qiskit-terra. I took a screenshot of qiskit_sphinx_theme as an example of where you find the "About" section and the little wheel icon. I don't have permission to edit Terra's About, so I added that screenshot to give more clear instructions.
(This issue is one of ~15 I opened today across the ecosystem)

I'm with Jim here, especially since we're about to make this repo the de facto "qiskit" repo.  That link slot is for the repository's "website", rather than specifically its documentation, and we've got links from qiskit.org to the Terra documentation plus the first paragraph of the readme has a big link to the docs.


make TangentialAcceleration and PolarAcceleration respect global flag

Tangential and Polar acceleration didn't respect the global/relative setting in Flame. this fixes https://github.com/libgdx/libgdx/issues/4894
Also fixed a bug in which rotation information wasn't passed to from SpawnInfluencer to ParticleControllers

@Tom-Ski Is there anything I can do to make this pr more easy to review? The actual bug-fixes are about 20 lines, and the majority of the lines changed here are in 3 new test files to demonstrate that the fix works as intended.


The plugin now tries to open pdf on muPDF and fails on Android .

The app url https://play.google.com/store/apps/details?id=de.sitewaerts.cleverdox.viewer doesn't exist.
Readme reference
Android
Due to license restrictions in muPDF, the plugin dispatches to a separate (free) viewer app based on muPDF. If the viewer app is not yet installed, the user may be redirected to the google play app store.
https://play.google.com/store/apps/details?id=de.sitewaerts.cleverdox.viewer
https://github.com/sitewaerts/android-document-viewer

Any fixes?

This plugin is no longer supported.


Improve subscription payload tests and structure

I want to merge this change because generating payloads from subscription needs to be more throughly tested.
This PR changes structure of tests for subscriptions so in the future we can use its fragments and queries throughout whole project.
There are still payloads that needs to be expanded (checkout, order, product) but we decided to add them later in separate prs.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-6516 Improve tests for async webhooks


Prepare release 5.4.3

Motivation:
add the shell scripts to avoid change version error
Modification:
add the shell scripts to avoid change version error
Result:
we can change version by tools/shells

Codecov Report

Merging #74 into master will decrease coverage by 0.55%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #74      +/-   ##
============================================
- Coverage      55.8%   55.24%   -0.56%
+ Complexity      355      352       -3
============================================
  Files            51       51
  Lines          1620     1620
  Branches        274      274
============================================
- Hits            904      895       -9
- Misses          555      561       +6
- Partials        161      164       +3

Impacted Files
Coverage Δ
Complexity Δ

...sofa/rpc/boot/container/ServerConfigContainer.java
78.28% <0%> (-4.55%)
41% <0%> (-3%)

Continue to review full report at Codecov.

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


NVIDIA Jetson TX2

I bought an NVIDIA Jetson TX2
I tried to run openpilot on this, but encountered some problems during deployment. The error is as follows:
scons: Reading SConscript files ...
scons: warning: Calling missing SConscript without error is deprecated.
Transition by adding must_ exist=0 to SConscript calls.
Missing SConscript 'opendbc/can/SConscript'
File "/home/mawei/openpilot/SConstruct", line 324, in
AttributeError: 'Values' object has no attribute 'mpc_ generate':
File "/home/mawei/openpilot/SConstruct", line 335:
SConscript(['selfdrive/controls/lib/lateral_ mpc/SConscript'])
File "/home/mawei/.pyenv/versions/3.8.5/lib/python3.8/site-packages/SCons/Script/ SConscript.py ", line 654:
return method(*args, **kw)
File "/home/mawei/.pyenv/versions/3.8.5/lib/python3.8/site-packages/SCons/Script/ SConscript.py ", line 591:
return _ SConscript( self.fs , *files, **subst_ kw)
File "/home/mawei/.pyenv/versions/3.8.5/lib/python3.8/site-packages/SCons/Script/ SConscript.py ", line 280:
exec(compile(scriptdata, scriptname, 'exec'), call_ stack[-1].globals)
File "/home/mawei/openpilot/selfdrive/controls/lib/lateral_ mpc/SConscript", line 31:
if GetOption('mpc_ generate'):
File "/home/mawei/.pyenv/versions/3.8.5/lib/python3.8/site-packages/SCons/Script/ Main.py ", line 478:
return getattr( OptionsParser.values , name)
File "/home/mawei/.pyenv/versions/3.8.5/lib/python3.8/site-packages/SCons/Script/ SConsOptions.py ", line 112:
return getattr(self.__ dict__['__ defaults__'],  attr)
Can someone help me?

This is not an issue. Our Discussions page would be a better place for this.


Fast way to set matrix to zero

Is there a faster way to set matrix to zero?
I'm using the code like this for now: a = DenseMatrix.zeros(10, 10). But this need to alloc many new arrays. Is there a way to just set the values in a to zero?

a := 0.0, assuming doubles.
On Thu, Jun 4, 2015 at 7:27 PM, Bin Wang<EMAIL_ADDRESS>wrote:

Is there a faster way to set matrix to zero?
I'm using the code like this for now: a = DenseMatrix.zeros(10, 10). But
this need to alloc many new arrays. Is there a way to just set the values
in a to zero?
—
Reply to this email directly or view it on GitHub
https://github.com/scalanlp/breeze/issues/411.


Buffered Channel with overflow capacity doesn't raise

I wanted to test out Buffered Channels but seems like the capacity option is not being used.
This code compiles but it just locks the process without any output.
ch = Channel::Buffered(Int32).new(2)
ch.send 7
ch.send 5
ch.send 6
ch.send 2

loop {
  puts "Yo"
  ch.receive
}

Yes, that's the expected behaviour. Once you try to send the third number it blocks the current coroutine, which is the main and only one, because the capacity is 2.
What you probably want is this:
ch = Channel::Buffered(Int32).new(2)

# Send messages in a different coroutine: if `send` blocks, it will block just this couroutine
spawn do
  ch.send 7
  ch.send 5
  ch.send 6
  ch.send 2
end

# This is on the main coroutine. `receive` blocks until it gets messages from `send`,
# and now it works because two coroutines are communicating
loop {
  puts "Yo"
  ch.receive
}

I thought that it should raise an exception if the capacity is full or over like in Go e.g http://play.golang.org/p/94yoBGi0Zz

In Go it panics because the runtime detects that there's a deadlock. We can do the same, but right now it's not done. In a way, both programs crash: Go with panic, Crystal hangs forever.

The exception seems to be because of a deadlock, not because the channel is full. We might do deadlock detection in the future, but that's a separate topic.

Closing, please open a separate issue if you want deadlock detection.


JS Error when adding a Workload that contains Entities outside of the application scope

Description
When including a Workload that contains entities that are outside of the application scope, the MapLocationDistiller.entityReducer throws a JS error, trying to read the guid from a null object.
Steps to Reproduce

Create a workload in a subaccount
Add an entity from a parent account to that workload
Enable the Workload GeoOps app for the subaccount only
Create a new Map and Location
Add the workload to that Location
Try to access the Map
Result: JS Error Skull & Bones

Expected Behaviour
View the map and workload but without the Entity that is out of scope for that app
Relevant Logs / Console output
Your Environment

NR1 CLI version used:
Browser name and version:
Operating System and version:

Additional context

Starting work on it

Unable to reproduce. I suspect we addressed this on fixes @devfreddy recently implemented. Closing.


Dashboard: Update tooltip text for grid view

updating tooltip text for view toggle to tell users what to expect if they click icon button. Updating viewStyleBar component to match other components in directory - storybook and tests added
Summary
Updating tooltip text for view toggle to tell users what to expect if they click icon button. Now if currently on the grid view it will say "Switch to List View" and if you are on the list view it will say "Switch to Grid View" when you hover over the toggle icon.

Relevant Technical Choices

Text update is in constants/
While here also updated the ViewStyleBar component to have a directory with tests and storybook
Removed redundant label prop that wasn't getting used
Moved ariaLabel up to shared components in bodyViewOptions so we can try to centralize text updates and keep everything in /components sharable.
Added data-testid to icons used in ViewStyleBar to write easier tests.

User-facing changes

On hover of toggle view icons in dashboard (see screenshots attached) you should see the following text:

When in grid view: "Switch to List View"
When in list view: "Switch to Grid View"

Testing Instructions

Verify the above text updates
See that Dashboard/Components/ViewStyleBar works - it has a knob to toggle the view.
See that new unit tests work

Addresses #2398

Looks great!
One question though:
As per #2398 issue description, we should probably also use the same ARIA label.
"Toggle between showing stories as a grid or list." is nice, but I don't see how a screenreader user would know whether they are currently seeing a grid or list view. "Switch to list view" makes it much clearer what the button is about.

Looks great!
One question though:
As per #2398 issue description, we should probably also use the same ARIA label.
"Toggle between showing stories as a grid or list." is nice, but I don't see how a screenreader user would know whether they are currently seeing a grid or list view. "Switch to list view" makes it much clearer what the button is about.

oh yup! i misread the comment conversation and thought that was out of date with the updated tooltip text. will update, thanks for noticing!


Implement smoke test pattern from spike and tag tests to be smoked

Problem statement
E2E tests always querying against Staging data is causing a large number of exceptions and PagerDuty alerts. To prevent this from happening, we want to allow E2E tests to run against different types of data including Local, Mock, Staging, and Live.
What you need to know
Spike to determine pattern of implementation is here
Acceptance criteria

[ ] Merge code from spike and make additional changes for specs that don't require a user flow (i.e. pages with reportId) to be able to run as smoke
[ ] Add smoke tags to tests or groups that are within the Admin -> happy path.
[ ] Smoke tests should only run agains Chrome.

To do

[ ] ...

On track


Update readme

What does this PR do?

Corrects readme file links
Adds development environment info
Remove tests from package.json
Add all contributors bot

@all-contributors please add @xavier506 for infrastructure , doc and projectManagement
@all-contributors please add @tetogomez  for code and review
@all-contributors please add @adriexnet   for code and review
@all-contributors please add @murillojorge for design
@all-contributors please add @PixelNoob  for ideas and content
@all-contributors please add @sergioyuhjtman for ideas and translation
Steps to test

Go to repo
Readme file should contain project contributors and updated links

CheckList

[ ] Follow proper Markdown format
[ ] The content is adequate
[ ] The content is available in both english and spanish
[ ] I Ran a spell check

@all-contributors please add @xavier506 for infrastructure , doc and projectManagement
@all-contributors please add @tetogomez for code and review
@all-contributors please add @adriexnet for code and review
@all-contributors please add @murillojorge for design
@all-contributors please add @PixelNoob for ideas and content
@all-contributors please add @sergioyuhjtman for ideas and translation
Steps to test

@all-contributors please add @xavier506 for infrastructure , doc and projectManagement
@all-contributors please add @tetogomez for code and review
@all-contributors please add @adriexnet for code and review
@all-contributors please add @murillojorge for design
@all-contributors please add @PixelNoob for ideas and content
@all-contributors please add @sergioyuhjtman for ideas and translation

@all-contributors please add @sergioyuhjtman for ideas and translation


support for mkdir?

etcd supports mkdir but libkv does not.
is it intentional?

Hi @streamnsight,
Yes, because Consul does not support mkdir, etcd is the only store making a strict distinction between directories and keys (which is even stronger than the notion of znode in zookeeper, because zk can store a value in a znode).
Current work on etcd APIv3 will get rid of that distinction between keys and directories.
See #20 for more details on that issue.


Fix get_numba_envvar

Fixes #6601

Patch looks good, one comment about missing unittest.main(), would you like to resolve that whilst this file is being changed?


Change watermark timeframe to 3 months

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1276692

After:

/cc @gtanzillo

@imtayadeway We also need to add "region" as a column on the report.

@gtanzillo Just added. Can you check especially column / sort by / group by order?

(showing the region description)

@gtanzillo made the discussed changes and added a screenshot showing one of the new reports. Can you rereview?

Looks good! :+1:


Improve handling of arch-specific dependencies

Provide a detailed description of the proposed feature
For OS-specific dependencies we have on_macos and on_linux (and uses_from_macos ...[, since: ...]). This allows brew audit and brew unbottled to still pick the correct dependency graph even if you are not on that system.
We have no such feature for arch-specific dependencies, which is unfortunately something I'm starting to see used more often.
When implementing this feature, there might be new audit failures to fix first that have previously been missed.
What is the motivation for the feature?
To allow brew audit and brew unbottled to work correctly when you are on the opposite system than the CPU conditional.
Notably our CI only runs audit checks on x86_64 and we ideally never should need to run audit checks on multiple host platforms - just simulate the various target platforms on the same host.
How will the feature be relevant to at least 90% of Homebrew users?
Less issues caused by various things audits catch such as version mismatches.
More correct output on formulae.brew.sh.
What alternatives to the feature have been considered?
Hacking Hardware::CPU.

👍🏻 this is a good idea, we should do this.
To me the goal is having formula.brew.sh be able to be correctly generated for all OSs/archs from any OS/arch.

@Rylan12 @Bo98 was this completed?

I think so! TBH I don't think I've ever looked at this issue before but it seems like it was done for the API stuff 😅
I think this can be closed and individual issues can be opened for things like displaying the information on formulae.brew.sh or changing audit/unbottled if they haven't been done yet

Yes, this was way back when on_intel/arm didn't exist so this was fixed by #13451.
I assume stuff like audits check all variations but haven't checked.


Cross-Site Scripting in markdown-it-katex

the code is not working https://github.com/advisories/GHSA-5ff8-jcf9-fw62

Fixed in v2.


Fix ServerCookieEncoder javadoc, close #7115

Motivation:
ServerCookieEncoder’s javadoc contains some invalid copy-pasting from
ClientCookieEncoder.
Modifications:

As per RFC6265, multiple cookies are sent as separate Set-Cookie
response headers.
Fix code sample

Result:
Proper javadoc

@slandelle is this only for 4.1 ?

It's not only for 4.1. You should be able to cherry-pick on 4.0.

@slandelle thanks
Cherry-picked into 4.1 (d9d0e633dc69b9b93139d822eaa17b97ff3826ab) and 4.0 (5ccedc5e5b)


Heavily CPU usages

I'm developing nostr-relay using relayer.

Apparently the CPU load has not been stable since the most recent version.
For a while after startup, the program runs stably, but after some time, the CPU load suddenly becomes high.
Do you have any idea what is going on?

This is output from the container.
2023/08/10 09:45:33 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal)
2023/08/10 09:51:11 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal)
2023/08/10 09:53:39 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal)
2023/08/10 10:13:34 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 10:13:34 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 10:14:03 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 10:33:44 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 10:41:33 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 11:06:19 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal)
2023/08/10 11:10:02 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal)
2023/08/10 11:10:03 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal)
2023/08/10 11:10:37 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 11:19:16 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 11:27:32 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 11:59:13 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 12:01:57 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close
2023/08/10 12:02:21 nostr-relay: unexpected close error from <IP_ADDRESS>: websocket: close 1000 (normal): Normal close

Maybe some goroutine leak?


Adding-my-resume

Hello, can i add my resume in it ?

sure Go ahead @KAMAL-02

sure Go ahead @KAMAL-02

Hey, I just noticed we should avoid hardcoding the developers' locations. Instead, we can remove it since we already provide our location in the resume section

Nice open a new issue for this I will assign you that issue @KAMAL-02

Okay!

@EuclidStellar can I also add my resume
Please assign me this Or I can make a PR without assign


Investigate possibility of using distribution strategies

Distribution strategies are becoming an integral part of TF 2.0, with several components supporting this way of distributing a computation (including eg tf.keras).
This issue investigates whether secure computations could be expressed as distribution strategies, thereby leveraging more of the built-in TF functionality.

@jvmancuso could you determine what the outcome is of this, not least the comments in https://github.com/tf-encrypted/tf-encrypted/pull/605, and potentially close this up before end of week?

responded to your comments, will add an overview this week with my thoughts!


Does python client support proxy?

Hello,  you know the google APIs are blocked in China, so we can only access these APIs by proxy, but I don't know whether the python client support proxy setup? If yes, please tell me how. If no, could you please add the proxy feature? Thanks!
However, I can set up a global proxy manually by this way, but it a global proxy, other requests would go through with the proxy, which is unnecessary.
import socket
from httplib2 import socks
import google_auth_oauthlib.flow

Socks5 proxy
socket.socket = socks.socksocket
socks.setdefaultproxy(socks.PROXY_TYPE_SOCKS5, "<IP_ADDRESS>", 1086)

flow = google_auth_oauthlib.flow.Flow.from_client_secrets_file(
        CLIENT_SECRETS_FILE, scopes=SCOPES)
flow.authorization_url(
        # Enable offline access so that you can refresh an access token without
        # re-prompting the user for permission. Recommended for web server apps.
        access_type='offline',
        # Enable incremental authorization. Recommended as a best practice.
        include_granted_scopes='true')

Hope there is any better way to handle the issue. 😃

face same problem,
anyone have solutions?

I can visit google.com ( with shadowsocks) while the request time out error is raised if I ping www.googleapis.com

@Folyd, thank you for posting your script! I use Jupyter notebook mostly, and the following snippet from your code helped a lot! For Jupyter notebooks, all interpreters for different notebooks are naturally separated from each other. Then, the solution that you are not satisfied with should work perfectly for Jupyter notebooks.
import socket
from httplib2 import socks
import google_auth_oauthlib.flow

#Socks5 proxy
socket.socket = socks.socksocket
socks.setdefaultproxy(socks.PROXY_TYPE_SOCKS5, "<IP_ADDRESS>", 1086)

PS: more on setting things up for those who stumble into this page on Setting up the Shadowsocks Client on Windows 10 machine ==> just be careful when one first sets up the Shadowsocks client. In the same Edit Server window, one get to choose which Proxy Port to enable. To reuse the code snippet above, one need to change the Proxy Port to 1086. The default Proxy Port is 1080.
I my use case, the code snippet worked with a normal Windows 10 Shadowsocks client and Jupyter notebooks hosted on a WSL session.

I used to proxy via IP chains while I was in China, worked great.

I think you could accomplish this by passing an httplib2.http object with the proxy_info parameter.
See https://httplib2.readthedocs.io/en/latest/libhttplib2.html?highlight=proxy#httplib2.Http and https://httplib2.readthedocs.io/en/latest/libhttplib2.html?highlight=proxy#httplib2.ProxyInfo

Hi @Folyd,
I'm going to close this issue as guidance was provided in this comment. If you're still having trouble please re-open this issue with additional information.

I'm not using shadowsocks, but have the exact same problem, I would thankful if anyone could help me.
I have read the following links, which tell us how to set the proxy on Google APIs:

setting proxy using httplib2shim
client api proxy
official repository for httplib2shim
google proxy support link
google-api-python-client repository

My problem is different from the other questions, since I am using ServiceAccountCredentials method from oauth2client.service_account package. The authentication to the API key is done with json file. (also ServiceAccountCredentials.from_json_keyfile_name method). Whenever I want to request to the google service I want to pass it from http proxy server, unfortunately there isn't any rich documentation for this purpose on the web (or maybe I couldn't find them).
ga_base_config.yml
PROXY:
  proxy_user: '<proxy_user>'
  proxy_pass: '<proxy_password>'
  proxy_host: '<proxy_host>'
  proxy_port: <port_no>

main.py
class GA:
    def __init__(self, root_dir):
        with open(os.path.join(root_dir, 'config/ga_base_config.yml'), 'r') as yaml_file:
            try:
                self.yaml_file = yaml.safe_load(yaml_file)
                print(self.yaml_file)
            except yaml.YAMLError as exc:
                print(exc)

        self.scopes = self.yaml_file['URL']['scope']
        self.proxy_info = self.yaml_file['PROXY']

        self.key_file_location = os.path.join(root_dir, self.yaml_file['KEY_FILE'])

    def initialize_analytics_reporting(self):
        """Initializes an Analytics Reporting API V4 service object.

        Returns:
          An authorized Analytics Reporting API V4 service object.
        """
        credentials = ServiceAccountCredentials.from_json_keyfile_name(
            self.key_file_location,
            self.scopes,
        )
        proxy = httplib2shim.Http(self.proxy_info)
        credentials.authorize(http=proxy)
        analytics = build(
            'analyticsreporting',
            'v4',
            credentials=credentials,
        )
        return analytics

I'm not using shadowsocks, but have the exact same problem, I would thankful if anyone could help me.
I have read the following links, which tell us how to set the proxy on Google APIs:

setting proxy using httplib2shim
client api proxy
official repository for httplib2shim
google proxy support link
google-api-python-client repository

My problem is different from the other questions, since I am using ServiceAccountCredentials method from oauth2client.service_account package. The authentication to the API key is done with json file. (also ServiceAccountCredentials.from_json_keyfile_name method). Whenever I want to request to the google service I want to pass it from http proxy server, unfortunately there isn't any rich documentation for this purpose on the web (or maybe I couldn't find them).
ga_base_config.yml
PROXY:
  proxy_user: '<proxy_user>'
  proxy_pass: '<proxy_password>'
  proxy_host: '<proxy_host>'
  proxy_port: <port_no>

main.py
class GA:
    def __init__(self, root_dir):
        with open(os.path.join(root_dir, 'config/ga_base_config.yml'), 'r') as yaml_file:
            try:
                self.yaml_file = yaml.safe_load(yaml_file)
                print(self.yaml_file)
            except yaml.YAMLError as exc:
                print(exc)

        self.scopes = self.yaml_file['URL']['scope']
        self.proxy_info = self.yaml_file['PROXY']

        self.key_file_location = os.path.join(root_dir, self.yaml_file['KEY_FILE'])

    def initialize_analytics_reporting(self):
        """Initializes an Analytics Reporting API V4 service object.

        Returns:
          An authorized Analytics Reporting API V4 service object.
        """
        credentials = ServiceAccountCredentials.from_json_keyfile_name(
            self.key_file_location,
            self.scopes,
        )
        proxy = httplib2shim.Http(self.proxy_info)
        credentials.authorize(http=proxy)
        analytics = build(
            'analyticsreporting',
            'v4',
            credentials=credentials,
        )
        return analytics

@Folyd, thank you for posting your script! I use Jupyter notebook mostly, and the following snippet from your code helped a lot! For Jupyter notebooks, all interpreters for different notebooks are naturally separated from each other. Then, the solution that you are not satisfied with should work perfectly for Jupyter notebooks.
import socket
from httplib2 import socks
import google_auth_oauthlib.flow

#Socks5 proxy
socket.socket = socks.socksocket
socks.setdefaultproxy(socks.PROXY_TYPE_SOCKS5, "<IP_ADDRESS>", 1086)

PS: more on setting things up for those who stumble into this page on Setting up the Shadowsocks Client on Windows 10 machine ==> just be careful when one first sets up the Shadowsocks client. In the same Edit Server window, one get to choose which Proxy Port to enable. To reuse the code snippet above, one need to change the Proxy Port to 1086. The default Proxy Port is 1080.
I my use case, the code snippet worked with a normal Windows 10 Shadowsocks client and Jupyter notebooks hosted on a WSL session.

I tried to use it that way, but on PFSense's Squid Proxy it's showing:
NONE/000 error:transaction-end-before-headers


Migrate 7 Unary backward ops to TTNN

Unary Backward Tracking Issue : #9874 [ Master issue : #9322 ]
List of ops required to be migrated

[x] clamp_max_bw
[x] hardshrink_bw
[x] softshrink_bw
[x] leaky_relu_bw
[x] elu_bw
[x] celu_bw
[x] rpow_bw

Migrated all the above mentioned ops and submitted PR #10078

Completed and merged to main #10078


Make Request and Response a trait.

Alternative to #42

I don't have a good solution for a registry. I'll revisit that later.


[Cosmos] updating tests for internal pipelines

Pipelines are running into error when trying to re-create resources because they exist, changed method to use create_if_not_exists()

API changes have been detected in azure-cosmos. You can review API changes here


Vault: Clientside implementation

Change log

Add vault docs that explains the security protocol of the vault

Add vault class, which implements the protocol

Add tests for the vault class

TODO:
Connect to gRPC method and test integration
Will remove draft state of this PR once this is tested end to end.

Another idea - maybe it'd be better if we return some randomly generated response even if vsk does not match our records. So in case attacker guess vsk, he won't be able to if vsk is valid without knowing the password.
On the other hand cracking vsk would take some time, so maybe it's just overengineering.

Another idea - maybe it'd be better if we return some randomly generated response even if vsk does not match our records. So in case attacker guess vsk, he won't be able to if vsk is valid without knowing the password.
On the other hand cracking vsk would take some time, so maybe it's just overengineering.

One of the reasons we do this cryptographic flow is that this way we allow users to know if the password they are inputting is right or wrong. If we always return a random vk(vf) wouldn't the user be logged into a new account instead of knowing they have to retype their password?

One of the reasons we do this cryptographic flow is that this way we allow users to know if the password they are inputting is right or wrong. If we always return a random vk(vf) wouldn't the user be logged into a new account instead of knowing they have to retype their password?

He would know he's not able to decrypt the file with used password. I mean it only hardens brute forcing about one more step. But since we'll have API Rate limiting it might be unnecessary complexity - I think even without this step, brute forcing vsk -> password would take so many years.

One of the reasons we do this cryptographic flow is that this way we allow users to know if the password they are inputting is right or wrong. If we always return a random vk(vf) wouldn't the user be logged into a new account instead of knowing they have to retype their password?

He would know he's not able to decrypt the file with used password. I mean it only hardens brute forcing about one more step. But since we'll have API Rate limiting it might be unnecessary complexity - I think even without this step, brute forcing vsk -> password would take so many years.

Exactly. Also the server side will be running 1 million KDF iterations so it takes about 1 second for each attempt 😂

Implemented the service layer that stores and backups the current key pair + mnemonic. Marking this PR as no longer a draft.


Multiplication of complex numbers

Problem: 3+2i, 1-i
Solution: 5-i
Range for Real and Imaginary numbers between -20, 20

Is this open to grab? I would like to try it, please.

Yep, I've assigned you.

Thanks, my code is ready but if it is okay, may I ask a question? Why do we need the Generator class?

@razinc shouldn't you show it's on the argand plane or something like i(iota)?

Hi @ieshaan12,
Sorry, I don't quite catch you. Are you talking about output of the Generator:
25 Multiplication of 2 complex numbers (x + j) (y + j) =  xy + xj + yj -1
or output of the multiplyComplexNumbersFunc()?
('11j * (8+16j) = ', (-176+88j))
Or should I plot both on a plane?
Thanks.

Yeah, more like the second one @razinc

Why do we need the Generator class?

@razinc The generator class is able to hold the properties of a generator such as Title and Id. It makes it a lot easier to manage generators if each generator takes up one line. Class instances are added to genList which makes it super easy to get a generator by Id. I originally didn't have it but realized that creating a class is necessary for management and also adds possibilities for methods to be created.

Hi @ieshaan12. I feel like it would be easier to plot if I import numpy & matplotlib. Is that okay?

@lukew3 Suggestions?

@ieshaan12 I don't know why the plotting is necessary. However, the extent of my experience with imaginary numbers is limited. Currently, there is no way to include a graph in the problem/solution. If it is necessary to check existence of imaginary numbers with numpy or matplotlib it would be fine to import them.

I do plan on making some sort of raw/custom output for parts of problems and solutions that cannot be accurately represented as just a string.

Hi @ieshaan12, should we continue with the plot, or do we wait for @lukew3 to implement his custom output?

Like @lukew3 said, we don't have a way to do the plot thing.
@razinc I think the best output would be of this format: ('11j * (8+16j) = ', (-176+88j))

Can I contribute as well, please?

Why do we need the Generator class?

@razinc The generator class is able to hold the properties of a generator such as Title and Id. It makes it a lot easier to manage generators if each generator takes up one line. Class instances are added to genList which makes it super easy to get a generator by Id. I originally didn't have it but realized that creating a class is necessary for management and also adds possibilities for methods to be created.

Thanks @lukew3 for your explanation.


container hostConfig not checked on container start

On container start, exactly in daemon/start.go, only the hostConfig provided to the API is checked.
If you've previously created a container with a specific hostConfig and then you make changes to the system something could be broken.
I've hit this when unmounting cgroups (memory in my case) but any other hostConfig's field could have this problem I think.
Step to reproduce on an Ubuntu 15.04:
$ docker run -d --memory-swappiness=0 debian:jessie true # memory cgroups is mounted
ba61c3bbe0198643994956aca074976e157ef2521425ac527b1a0b6016e61851
$ sudo umount /sys/fs/cgroup/memory  # umount memory cgroup for instance
$ docker start ba61c3bbe0198643994956aca074976e157ef2521425ac527b1a0b6016e61851
Error response from daemon: Cannot start container ba61c3bbe0198643994956aca074976e157ef2521425ac527b1a0b6016e61851: [8] System error: no such directory for memory.swappiness.
Error: failed to start containers: [ba61c3bbe0198643994956aca074976e157ef2521425ac527b1a0b6016e61851]

This is happening because the hostConfig in the start API is only checked if it's the one provided in the request and not the actual container's hostConfig (see verifyContainerSettings)
Beaware also that verifyContainerSettings will set for instance MemorySwappiness to nil if it detects the system doesn't have memory cgroup enabled.
Should we check hostConfig before starting container (and show a nicer error and put MemorySwappiness in hostConfig to nil) or this libcontainer error is ok to show if you umount cgroups while running? I have a fix to check container's hostConfig on start also
EDIT: Digging deeper, daemon.sysInfo is only populated on daemon start so even calling verifyContainerSettings in container start still shows the error above
ping @LK4D4 (I've seen libcontainer ignores cgroups path not found error in apply_systemd and try to write the file and erroring out above, wondering if that should be fixed as well)

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

@runcom I can't reproduce on the latest master branch, I tried ubuntu 14.04 using upstart and RHEL7 using systemd, what Docker version are you using?

@hqhq master branch on Ubuntu 15.04 :( will try on Fedora 22 later today

@runcom It's weird I can't reproduce it, but for libcontainer part you mentioned, the rule is:

cgroup subsystem not mounted, and you are not using related options, error ignored
cgroup subsystem not mounted, but you assigned related values, error out

So if memory cgroup was not mounted, and --memory-swappiness was assigned, it is expected to see that error msg.
I think fix should be on Docker side, maybe daemon.sysinfo should be updated more often?

@hqhq indeed docker  is fixed (I saw your patch)  butsysinfo is not updated while daemon is running so if you unmount the cgroup while it's running you get that error if your container was started with cgroup's constraints and you try to start it
I think the problem here lies also in having hostConfig in start :/

Working to fix this, I'll send the PR later today


Use a new Github API key for the documentation

Closes: #102
(hopefully)

Hm, looks like Travis didn't download & run rustup.sh
I got this on master, too so it probably doesn't have anything to do with this. Let's wait a bit and see.


Minor Fixes in Series methods docs

Hello @akharche! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file sdc/datatypes/hpat_pandas_series_functions.py:

Line 4549:1: W293 blank line contains whitespace

Also any fixes already exist in PR:
https://github.com/IntelPython/sdc/pull/770

Also some of these changes are in #780

Also #778

It is fully covered by other Pull Requests


Parse event data from ortec

metadata is now loaded in #136 ,
Since Ortec uses numeric identifiers which we can not translate to logical events like "passes" and "shots", we can not parse the event data for now.

Ortec went to betting, not for soccer analytics anymore


Update jquery.state-manager.js to throw informative error

Checks whether a plugin which should be added actually exists.
Currently when a plugin is missing the javascript-execution halts with a cryptic:
Uncaught TypeError: element[pluginName] is not a function (in chrome) which doesn't say what plugin is missing. It actually reads this way.
This way there's also an error thrown which halts javascript-execution, but it reads like this:
Uncaught Error: Plugin <pluginName> is not a valid jQuery-plugin! Where <pluginName> is the name of the plugin missing.
This will help developers to better debug what plugin is missing.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-15098.
Please use this issue to track the state of your pull request.

Thanks for your PR. I changed it to soft-fail on error. So the change has been merged with 96afc383e3a74eb7a785bc8cae1f944b1648c053.


Error: could not decode rune. Windows-1251 (Cyrillic)

Hi, your framework is very cool, it inspired me to make some helpful app for me and my colleagues!
It works fine on Linux, but most employees in my company use Windows and they get error "could not decode rune",
when typing cyrillic symbols, like (ш, б, ж etc).
I find place in source code:
// Translate input into runes. In most cases we'll receive exactly one
// rune, but there are cases, particularly when an input method editor is
// used, where we can receive multiple runes at once.
for i, w := 0, 0; i < len(b); i += w {
	r, width := utf8.DecodeRune(b[i:])
	log.Printf("number bytes: %v, buffer:%v. bytes:%b after decode to rune: %v", numBytes, buf, b, r)
	if r == utf8.RuneError {

		return nil, errors.New("could not decode rune")
	}
	runes = append(runes, r)
	w = width
}

I think problem with windows-1251 (Cyrillic) encode.
They tried set UTF-8 support, like in this issue #153, but it doesn't work.
How can I handle input bytes before framework handle it? Maybe this help me to decode bytes to correct utf8 format?
Or maybe you can help another way?

I think this was fixed in #343. Can you check?

Okey, now app doesn't exits with panic, but when i type it looks like "®¢ ë«ä¤ ®ë¤¢ ãèª®ãè¤ë¢". It's not my language))))). Can I adjust app for work with  windows-1251 (Cyrillic) encode?

Sorry for bothering, but any chances or plans of this being fixed sooner or later?
Some more info from me about related issue.
For example. When I type cyrillic russian "ф" character (CP1251 code: 244) - app missreads it as "ä" (CP1252 code: 228). And "г" (CP1251 code: 227) --> "£" (CP1252 code: 163).
Maybe smth is broken with Code Page detection (CP1251/CP1252) or something.
Looks like on Windows (maybe when russian is used as a system lang?) bubbletea input text is stuck with CP1252 characters only.

Locale set in terminal:  ru_RU.UTF-8.
$ locale
LANG=ru_RU.UTF-8
LC_CTYPE="ru_RU.UTF-8"
LC_NUMERIC="ru_RU.UTF-8"
LC_TIME="ru_RU.UTF-8"
LC_COLLATE="ru_RU.UTF-8"
LC_MONETARY="ru_RU.UTF-8"
LC_MESSAGES="ru_RU.UTF-8"
LC_ALL=

Tried on Windows 10, same with: cmd, git bash, powershell.

Your locale does not match your input encoding.
Have you tried locale ru_RU.CP1252?

Have you tried locale ru_RU.CP1251?

I don't really appreciate the idea of forcing my end-users to mess with their system's or terminal config.

Your locale does not match your input encoding.

My guess is so that bubbletea's input reader implementation might be kinda bugged on Windows. But unfortunately don't really have time to check it myself right now.

Hi everyone! @Dmitrevicz, should be fixed on Windows as of Bubble Tea v0.26.0. I believe your issue should be fixed on Linux now as well @alexey-sderzhikov — mind verifying?

@meowgorithm It looks like this issue didn't present in Linux, and is seemingly confirmed fixed in Windows. It's probably worth closing this issue.


Incorporate language frameworks into azd

JavaScript, Python, and static web apps, have many popular frameworks, e.g. Express, FastAPI, Hugo, that help developers get started building applications. Each of the frameworks has their own distinct way of configuring, building, and deploying their solutions. In many scenarios, these frameworks are the starting point for developers. We will need to determine a subset of language frameworks to "devify" as way to simplify deploying these solutions on Azure.
For Phase 1, we'll need to select one or two language frameworks for each language. Within these, we will look at the popular example applications and select one or two of these examples. In addition, we'll also do the same with static web apps, e.g. Gatsby, Hugo. Although these are JavaScript, they are not the same kind of thing as an Express or FastAPI app.
Phase 1
Fork & devify existing language framework solution
Azure/azure-dev-pr#505,  Azure/azure-dev-pr#506, & Azure/azure-dev-pr#508
Phase 2
Track & manage changed to the community framework solutions
(link to epic)
Phase 3
Automatically "devify" a solution based on a supported template
(link to epic)

I'm going to close this for now in favour of having individual items as demand arises.


Error when scrap suspend user

I will add list with error while scrap user.
Additionally I will create custom errors like UserSuspendedException.


Claw Refinement (Sprint 5)

Third sprint towards arm, focusing on arm mounting and space optimization, we want the claw to be sturdy and be able to work well and fast, without any chance for breaking.
Acceptance Criteria
Claw and arm should be able to take from chamber without interfering or colliding with external parts.
Claw and arm should be able to deposit specimens without the robot turning around and be able to place samples in the highest bucket
Be able to grab specimens from the wall with minimal driver movement, and minimal human player adjustment

Decided to switch to a two servo finger tip

too clunky and decided to remove both the second angle and the roll servo

removed servo brackets to compress it and reduce space and interference


fix eb deploy action

The<EMAIL_ADDRESS>failed to install EB CLI. The repo[1] has fixes however the maintainer does not seem very active.  We try a different action to see if we can get this app deployed to beanstalk.
[1] https://github.com/hmanzur/actions-aws-eb

It looks like there's also an open issue for this action too that links to this upstream issue. From that last link, it looks like there might already be a fix merged to awsebcli and as soon as version 3.20.10 is released/published both actions should be fixed.

It looks like there's also an open issue for this action too that links to this upstream issue. From that last link, it looks like there might already be a fix merged to awsebcli and as soon as version 3.20.10 is released/published both actions should be fixed.

@jesusaurus It looks a valid work around is to install an older version of the EB CLI, i've tested with PR #92


Finished Ruby on Rails capstone project.

For this capstone project I followed the guideliens from Microverse, but I also added the following extra features:

I used Cloudinary to store all avatars and cover images in the cloud.
I used figaro to hide my secret Cloudinary keys.
I added an App rating system which allows users to:

Add apps for everyone to rate
Rate apps
See a calculation of the percentage of Positive vs. Negative app reviews for each app.

I've made the following corrections:

Explain N+1 problem in the presentation video.
Test the relationship between app and opinion.
Test the percentage calculation methods from the app model.
Correct README:  rails db:migrate instead of ruby db:migrate
Corrected link of Claudinary credentials
Deleted unused views and controller methods.
Use only implemented routes.
Sort opinions from newest to oldest on the main page.

In accordance to the following changes requested by the TSEs:

Boakye Forison Addo	First Review	2020-06-16 20:43:43 UTC
You did not explain how you took care of you the N+1 query in your video presentation. You did not test some of the methods in your models, you seem to only focus on validations and relationship. Positive and negative methods in app.rb were not tested and the relationship app and opinion wasn't tested as well.
Shakhawat Hossain	First Review	2020-06-16 21:15:21 UTC	1.
In the Readme you mentioned ruby db:migrate. I think it should be rails db:migrate. 2. The link for Cloudinary credentials provides 404. Please replace the link with the correct one. 3. You did not describe the n+1 issue in your video and did not mention how you resolved it. 4. At this stage, you don't need all routes related to apps. What I mean is if I try to edit an app I can do it, by following the route https://calm-hamlet-19158.herokuapp.com/apps/1/edit. Should I be allowed to do that? I think you should restrict the path using only for this stage.
James Okunlade	First Review	2020-06-16 22:34:01 UTC	1.
Address the n+1 problem and record another short video explaining your solution, add the video link to your PR description for the next reviewer to se. 2. Sort your opinions /ratings according to the newest in your model. 3. You have declared all the seven RESTFul routes for all your resources, define only the ones you're using by using in your routes.

Project approved.


what is $context

Page: /setup/install/providers/kubernetes-v2/
Feedback: in your docs you refer to $context but you never state what this is? Is it a value from my Google account? Is it something that should be used as is? what is this thing?

It's defined a few lines above where it's referenced.
CONTEXT=$(kubectl config current-context)

hal config provider kubernetes account add my-k8s-v2-account \
    --provider-version v2 \
    --context $CONTEXT

So its value is the result of kubectl config current-context run against your cluster.
I'm closing this issue, as answered, but feel free to reopen if you need more information, @xgeoff .


fix error integer overflow with large dataset

Hello,
I was able to use randomForestInfJack to calculate variances for your toy example and a 16000 row dataset, but ran into an error using it on a 217000 row dataset:
Error in if (B^2 > n * nrow(pred)) { :
  missing value where TRUE/FALSE needed
In addition: Warning message:
In n * nrow(pred) : NAs produced by integer overflow

Looking up the issue on stack overflow, figured out it was a simple fix, just needed to change n and nrow(pred) to class numeric. Adding as.numeric to lines 29 and 42 fixed the problem for me.
Thanks for your work - I'm pretty excited to see variance estimates for RF.
-Brian

Thank you!


Add the most anticipated gun in the game. Foam Force.

About the PR

Why / Balance

Rhetorical
Technical details

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Foam Force

Need a better change log

have you checked how it looks in the game? Most likely the weapon in the backpack slot will be drawn on top of the character, even when he is looking at the camera.

have you checked how it looks in the game? Most likely the weapon in the backpack slot will be drawn on top of the character, even when he is looking at the camera.

Good catch! There is supposed to be a strap there I just forgor to delete the front half.

Alright, should be good now. I cleaned up those couple of pixels on the hand in the backpack sprites.

lol wtf, no


Allow using Azure OpenAI

Allows using Azure OpenAI instead of OpenAI (solves #325)
gpt-engineer [project_path] --azure --engine [your_azureopenai_model_name]

To be consistent with Azure OpenAI terminology, the model is called an engine instead.
Requires to set AZURE_OPENAI_ENDPOINT and AZURE_OPENAI_KEY as environment variables.

@herbutant because you opened #325: Could you test the pr for me? I don't have azure openai access. thanks!

@UmerHA , thanks for this.
I made one small change locally to also pull the api_version from ENV because this can also vary (e.g. in my case it is "2023-03-15-preview"). With that it runs like expected.
if not os.getenv("AZURE_OPENAI_VERSION"):
raise ValueError("To use Azure OpenAI version please set a "
"AZURE_OPENAI_VERSION enviroment variable.")
openai.api_version = os.getenv("AZURE_OPENAI_VERSION")

@herbutant agree - have adjusted the readme & made AZURE_OPENAI_VERSION an optional env variables that defaults to 2023-05-15.

I think we need to create a more general abstraction for LLMs if we want to support azure to not make the code messy!
Let me know if you have thoughts

Agree, I think we have 2 options:

write a thin wrapper around LangChain's abstraction (BaseChatModel for chat models / BaseLLM for non-chat models)
create our own simplified abstraction

I would vote for using LangChain, as LLM integration is not this project's purpose and LangChain does that well already. I'd be happy to implement the wrapper, as I'm a LC contributor as well. :)
If you think option 2 is better, I propose the following abstraction:
class AI
	self.model
	self.token_usage_logs = []

	@classmethod
	def create(cls, model_family_name, model_name, ai_config_filename):
		# read ai_config_filename
		# create a new instance with type depending on model_family_name (e.g. new OpenAI, or new AzureOpenAI)
		# 	and pass params from ai_config_filename to it

	@abstractmethod
	def __init__(self, model_name, ai_config_filename, **kwargs):
		# determine model_name or use default (eg gpt-3.5-turbo for OpenAI)
		# load configs (eg api key) from kwargs or from env
		# initialize model (eg  validate configs)

	@abstractmethod
	def _response(self, messages, prompt=None):
		# get response & meta-data (eg token_usage)

	@abstractmethod
	def get_token_usage(self, meta_data):
		# get & return token usage (handle both streaming & non-streaming cases)

	def response(self, messages, prompt=None):  # corresponds to current `next` method
		# logger.debug("starting response")

		# ai_response_object, meta_data = self._response(messages, prompt=None)

		# if is_generator(ai_response_object):  # streaming
		#	stream & print reponse chunks
		# 	ai_response = join(chat)
		# else:  # not streaming
		#	ai_response = ai_response_object[...]
		# 	print(ai_response)

		# logger.debug("response completed")
		# 	messages += [{"role": "assistant", "content": "".join(chat)}]

		# meta_data['answer'] = ai_response
		# self.update_token_count(self.get_token_usage(meta_data))

	def update_token_count
		# udate token usage log

then the OpenAI / AzureOpenAI would look like this:
class OpenAI(AI):
	def __init__():
		# determine model_name or use default (eg gpt-3.5-turbo for OpenAI)
		# initialize model (eg ensure all configs are given)

	@override
	def _response(self, messages, prompt=None):
		# get response & meta-data (ie token_usage)

	@abstractmethod
	def get_token_usage(self, meta_data):
		# get & return token usage (handle both streaming & non-streaming cases)

and
class AzureOpenAI(OpenAI):
	def __init__():
		# determine model_name
		# initialize model (eg ensure all configs are given)

	@override
	def _response(self, messages, prompt=None):
		# get response & meta-data (ie token_usage)

	# get_token_usage(self, meta_data) is same as in OpenAI

This would work for all kinds of models (chat/non-chat, streaming/non-streaming).
Let me know!

Hey appreciate the thinking on this!
Based on discussions in other PRs it looks like we will use langchain abstractions.
Because of that and that we have conflicts I’m closing this for now — feel free to open.


Force correct type for TrynewDot NVP classicRedirect.dismissed

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Problem
Our backend requires that the TryNewDot NVP value of classicRedirect.dismissed is always a bool. The App client suggests that a string can be used in TryNewDot.ts, which has led to bugs and confusion.
Also, we need to fix cases where a string might be being set. For example, the 'Try New Expensify' button.
Solution
Force the type to be bool here

View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Force correct type for TrynewDot NVP classicRedirect.dismissed
What is the root cause of that problem?
we are allowing boolean or string types here
https://github.com/Expensify/App/blob/00a81dc63f265a9f90a29f2d744a70465924f7f1/src/types/onyx/TryNewDot.ts#L12
What changes do you think we should make in order to solve the problem?

Change it to
        dismissed: boolean;

What alternative solutions did you explore? (Optional)

This is being handled by me and @war-in as a solution to a problem we faced here.

Hi, I'm Marcin from Software Mansion and I'm going to work on that issue :)

We are working on more critical issues related to the parent issue


no dynamic prompt menu in webui anymore

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behaviour, include the prompt you used if applicable:

Go to 'webui'
Click on 'txt2img'
Scroll down to 'nothingnis'
See error

Also mention which checkboxes have been set.
Expected behaviour
there should be a menu for dynamic prompts in the webui but its no longer there
Additional context
not sure when it stopped but i know it worked before

Not sure if you're looking for it under scripts, but it moved to its own section in the interface, have a look at this screenshot.

Could you confirm if it is there or not?

I have the same issue - Dynamic Prompts is installed and up to date, but doesn't show up anywhere.

I had the same issue - I solved it, but I am not sure how.
I installed Python 3.11.1, but I'm not sure if that is what made a difference (I was doing a bunch of other, not SD related, stuff - not actively trying to fix this issue).  Regardless, I now have the menu showing up.
Perhaps, @nagaisa , you could try updating/re-installing python and restarting SD to see if that makes a difference?

@bardicone thanks for the update - this suggests that there might be a python version incompatibility. @nagaisa , could you please check in your terminal if there is any error message?

@bardicone python 3.11 is surprising as generally users have an issue starting webui with that version. Is there a reason you need 3.11 or is it just that you get that version automatically installed from the microsoft store?

I just installed the extension through the webui and I too was unable to see the extension's menu.
Turns out restarting the ui with the buttons, either in the extensions or the settings tabs, was not enough.
Problem solved after terminating the console and restarting it. Using Python 3.10.6 by the way.

any error messages in the console?
On Wed, 28 Dec 2022 at 19:11, a56444c @.***> wrote:

I just installed the extension through the webui and I too was unable to
see the extension's menu.
Turns out restarting the ui with the buttons, either in the extensions or
the settings tabs, was not enough.
Problem solved after terminating the console and restarting it. Using
Python 3.10.6 by the way.
—
Reply to this email directly, view it on GitHub
https://github.com/adieyal/sd-dynamic-prompts/issues/119#issuecomment-1366801035,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADGXMW3ZXPXJ5G374KHLDDWPRYCZANCNFSM6AAAAAATCEFTLU
.
You are receiving this because you commented.Message ID:
@.***>

Sorry, I didn't pay attention. No errors since the restart though, basic syntax and wildcards are working fine.

@nagaisa  I would like to close this issue - could you confirm whether this is a still a problem?


Update choices.py

Pull Request
Related Issue

New Behavior

...
Contrast to Current Behavior

...
Discussion: Benefits and Drawbacks

...
Changes to the Documentation

...
Proposed Release Note Entry

Adding IP protocol to choice fields.
...
Double Check

[x ] I have explained my PR according to the information in the comments
or in a linked issue.
[x ] My PR targets the dev branch.

Hi, this is for issue #142 to add IP as a protocol field. Please let me know if anything needs changed.

Howdy,
I also added choice fields for IP/TCP/ICMP to help kickstart #126. I don't know Github enough to split commits across PRs, so apologies for dogpiling this one.
Thanks!
-Jake


Is MapLibre Gl Native 9.6.0 provides support for download the maps offline ?  If yes then please provide documentation for that so we can use it.  OR if you have any example for android source code please share with me. Thanks in advance.

Is MapLibre Gl Native 9.6.0 provides support for download the maps offline ?
If yes then please provide documentation for that so we can use it.  OR if you have any example for android source code please share with me.
Thanks in advance.

#1270


Python 3.12 Support

Added Python 3.12 Support
Updated requirements
Changed dependency-injector to dependency-injector-fork

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.99%. Comparing base (165df06) to head (769b3d2).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   85.98%   85.99%
=======================================
  Files          49       49
  Lines        1577     1578    +1
  Branches      112      113    +1
=======================================
+ Hits         1356     1357    +1
  Misses        191      191
  Partials       30       30

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


⚠ AGEG website (prod) has degraded performance

In 71f5890, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2979 ms

Resolved: AGEG website (prod) performance has improved in 1bd1079.


New empty

This pull request introduces 12 alerts and fixes 1 when merging 7796a01e0ad5bd0b635937c4b5dfaae683f5b0c9 into 90442a150daa219a34e02f12a9eee592320abef4 - view on LGTM.com
new alerts:

12 for Dereferenced variable may be null

fixed alerts:

1 for Dereferenced variable may be null

Comment posted by LGTM.com

This pull request introduces 12 alerts and fixes 1 when merging d327234ab665606c9ce2aef8ab745af658d133fc into 90442a150daa219a34e02f12a9eee592320abef4 - view on LGTM.com
new alerts:

12 for Dereferenced variable may be null

fixed alerts:

1 for Dereferenced variable may be null

Comment posted by LGTM.com

looks good to me

This pull request introduces 6 alerts and fixes 1 when merging<PHONE_NUMBER>5c89e8524bc756468c7f738fee31ab into 90442a150daa219a34e02f12a9eee592320abef4 - view on LGTM.com
new alerts:

6 for Dereferenced variable may be null

fixed alerts:

1 for Dereferenced variable may be null

Comment posted by LGTM.com

This pull request introduces 6 alerts and fixes 1 when merging 68dadd52c3bbc9f8202d5c3cfe67ba9e4eef7b25 into bd02cc3aa537f7666cbee5ac3d70e0147ead07b2 - view on LGTM.com
new alerts:

6 for Dereferenced variable may be null

fixed alerts:

1 for Dereferenced variable may be null

Comment posted by LGTM.com

This pull request introduces 6 alerts and fixes 1 when merging efd59a262e01a6a70324333460b0b9b63acf686a into bd02cc3aa537f7666cbee5ac3d70e0147ead07b2 - view on LGTM.com
new alerts:

6 for Dereferenced variable may be null

fixed alerts:

1 for Dereferenced variable may be null

Comment posted by LGTM.com

This pull request introduces 6 alerts and fixes 1 when merging 4a973a7518844f08f5a0a0a864cfb48f548f0241 into bd02cc3aa537f7666cbee5ac3d70e0147ead07b2 - view on LGTM.com
new alerts:

6 for Dereferenced variable may be null

fixed alerts:

1 for Dereferenced variable may be null

Comment posted by LGTM.com

This pull request introduces 6 alerts and fixes 1 when merging f0477af1db1bd4b8ea7f389ee1c43b87d7500b8d into bd02cc3aa537f7666cbee5ac3d70e0147ead07b2 - view on LGTM.com
new alerts:

6 for Dereferenced variable may be null

fixed alerts:

1 for Dereferenced variable may be null

Comment posted by LGTM.com

This pull request introduces 6 alerts and fixes 1 when merging 65da0ff567f5daa6de8d5e1d8be761e4b1d98027 into bd02cc3aa537f7666cbee5ac3d70e0147ead07b2 - view on LGTM.com
new alerts:

6 for Dereferenced variable may be null

fixed alerts:

1 for Dereferenced variable may be null

Comment posted by LGTM.com


[3.x] Implement onAbort especially for legacy 3.4.2 (3.x branch) without changing logic on 4.x branches

Read first comment

https://codesandbox.io/s/vue-routing-example-pfhhb - Bug example.
Posva don't wanna change 4.0 logic, that's why we should do it only for 3.x, not for 4.0.

Expained one year ago before 4.0 development started, related links:
https://github.com/vuejs/vue-router/issues/3027#issuecomment-596007053
https://github.com/vuejs/vue-router/issues/2956#issue-503374613
So, im not the only one person who undestand that it's not properly designed and should be fixed/added at least in vue 3.x branch.


Enable all data types gemm tutorial

This PR enables fp8 as well as int8 data type and add more configs.

redone in https://github.com/ROCmSoftwarePlatform/triton/pull/456


noencryptwallet does not display seed

Background
when creating a lnd environment with noencryptwallet set, the wallet gets created without displaying a seed.
Your environment

lnd v0.5-beta-rc1
windows 10 64-bit
bitcoind as backend

Steps to reproduce
start a new lnd with --noencryptwallet
Expected behaviour
the wallet should wait for lncli create
Actual behaviour
the wallet creates a wallet.db without showing a seed

That is expected behavior, remove the --noencryptwallet flag if you require a seed.

It's not exactly obvious to users that this is expected behavior. Especially first-time users like me who read https://github.com/lightningnetwork/lnd/blob/master/docs/INSTALL.md#disabling-wallet-encryption and dismiss the this argument must only be used for testing purposes as being overly careful. My expectation was that --noencryptwallet just does what it says without affecting backupability. I only realized much later that I've never seen and therefore never backed up the seed. Maybe it would make sense to stress that part in the documentation to reduce a footgun opportunity.

@jonasnick 100% agree that the naming and docs could be improved. I recall bringing this up in the past that the name does not suit the behavior, though the naming stuck and it was not revisited. I've made a PR to help address the confusion as we flesh out a full plan for deprecation.
#1843


feat: add new parameter called onChange

Add a new paramater called onChange that set a function callback with the string new value as arg;

@laripeanuts Amazing! Thank you so much! I will publish it soon.


[visual-recognition] Build Failed with exit code 65

Archived Failed. Please see the error log below:
error: type 'Face' does not conform to protocol 'Decodable'
public struct Face: Decodable {
^
Swift.Decodable:9:12: note: protocol requires initializer 'init(from:)' with type 'Decodable'
public init(from decoder: Decoder) throws
^
/Users/awsys-dev2/Desktop/ChatApp/Carthage/Checkouts/swift-sdk/Source/VisualRecognitionV3/Models/Face.swift:38:16: note: cannot automatically synthesize 'Decodable' because 'identity' does not have a matching CodingKey and does not have a default value
public var identity: FaceIdentity?
^

@johnlpm Can you pls provide the versions of Xcode and Swift you are using, and whether you are building on MacOs or Linux?

@mkistler I've also been seeing this issue, I can't reproduce it on my machine, but a majority of the people at a few workshops we've been doing have had the problem. The project fails to build when using Carthage as well as when manually building the project. I have Xcode 9.4 (and it works). I think most people with the issue have Xcode 9.4.1

Should be fixed in v0.29.0 (released today).

Elaborating a bit:  according to man xcodebuild:

xcodebuild exits with codes defined by sysexits(3).

and man 3 sysexits says:

 EX_DATAERR (65)       The input data was incorrect in some way.  This should only be used for user's data and not system files.

So it appears that 65 is the expected return code whenever any of the tests fail -- which unfortunately happens semi-regularly because most of the tests hit live service instances.
What should be fixed in v0.29.0 is that if all the tests pass, the xcodebuild rc should be 0 --previously it was 66, because of a missing shared scheme file.

Awesome, thanks :)
I updated to Xcode 9.4.1 and everything is still working for me. I have another workshop on Monday so I'll keep you posted on how that all goes. Thanks again for the help!

Sorry for late response, I'm using objective C on MacOS, I use the old version of SDK and it's working fine right now ..

Building with Carthage worked for 100% of the people at our workshop today (about 30 participants). Thanks for all the help :)

I'm trying to install using carthage in a react native project.
Xcode Version 9.1
Carthage Version 0.31.1
I get the following error:
Build Failed
Task failed with exit code 65:
/usr/bin/xcrun xcodebuild -project /Users/macname/Desktop/REACT_NATIVE/myreactnativeproject/ios/Carthage/Checkouts/swift-sdk/WatsonDeveloperCloud.xcodeproj -scheme VisualRecognitionV3 -configuration Release -derivedDataPath /Users/macname/Library/Caches/org.carthage.CarthageKit/DerivedData/9.1_9B55/swift-sdk/0.36.0 -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive -archivePath /var/folders/gz/9g75r3m148950b69mz7q4h0r0000gn/T/swift-sdk SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO (launched in /Users/macname/Desktop/REACT_NATIVE/myreactnativeproject/ios/Carthage/Checkouts/swift-sdk)


Add lastIndexOf method

Purpose
Add lastIndexOf method to string and array.
Fixes #21236
Fixes #21237
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #21430 into master will decrease coverage by 48.26%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #21430       +/-   ##
===========================================
- Coverage   67.44%   19.17%   -48.27%
===========================================
  Files           5       56       +51
  Lines         129     1528     +1399
  Branches       19      233      +214
===========================================
+ Hits           87      293      +206
- Misses         42     1219     +1177
- Partials        0       16       +16

Impacted Files
Coverage Δ

tool-plugins/vscode/src/utils/logger.ts
78.94% <0%> (ø)

tool-plugins/vscode/src/core/index.ts
100% <0%> (ø)

tool-plugins/vscode/src/utils/rpc/handler.ts
5.88% <0%> (ø)

tool-plugins/vscode/src/debugger/index.ts
0% <0%> (ø)

...-plugins/vscode/src/project-tree-view/activator.ts
0% <0%> (ø)

tool-plugins/vscode/src/telemetry/exceptions.ts
100% <0%> (ø)

...lugins/vscode/src/core/extended-language-client.ts
22.22% <0%> (ø)

tool-plugins/vscode/src/project/cli-cmds/build.ts
0% <0%> (ø)

tool-plugins/vscode/src/extension.ts
0% <0%> (ø)

tool-plugins/vscode/src/telemetry/events.ts
100% <0%> (ø)

... and 41 more

Continue to review full report at Codecov.

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


ignorenick doesn't add to database

MrFishBot and CVNBots added via IRC command, but failed to make it to the ignore_nicks table. Added manually but need to investigate for broken function/sql statement.

Was missing the commit() after INSERT INTO and DELETE FROM statements. Should be resolved pending testing.

Merged, Tested, works.


MaskAcc Always equals 1.0

It seems detection result is ok, but MaskAcc=1.0 and MaskLogLoss=0.0 from start training util end training.
what did I miss?

Did input data has image with only background label infect?

I have got the same problem,I guess maybe the reason for that is  the instances  have  very large areas or masks for your input image.

@xxradon thanks for you reply, I have negative samples which mask is entire image. Do I need to delete these samples?

I delete these negative samples and the result is fine


Fix: Update args with newer versions

update http args
update tessera and quorum versions
fix miner threads arg
remove nousb flag

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Add helper comments for extended-colors when previewing.

Clear and concise description of the problem
description
I extend a color in the tailwind.config.js and use it in my project. When previewing, it displays the color in RGB, the tailwindcss generated results. When developing, I want to check the color to match the figma which produces the HEX instead of RGB.
/** @type {import('tailwindcss').Config} */
export default {
  content: [
    './index.html',
    './src/**/*.{js,ts,jsx,tsx}',
  ],
  theme: {
    extend: {
      colors: {
        "my-color": "#0070E0"
      }
    },
  },
  plugins: [],
}

Suggested solution
suggestion

Could it add some helper comments like above?
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

At present, this extension only detects valid classname. Hover to show the generated css is provided by the official tailwind extension itself. I'm not sure if we can modify its content.

Although very basic, we now support hover and RGB to hex comments. Feel free to let me know if you have any more suggestions.


Rename the distributed jar file

In naming, it's irrelevant to use "java"-hamcrest. Maven, gradle and friends supports only Java libraries, and the complete file name (java-hamcrest.jar) already mentions it's a java library (with the extension).
Also, the name in itself is prone to confusion. When you're used to search for either "hamcrest-core" or "hamcrest-all", in an IDE, you type the group-id, then the first letters ("ham", for instance) and don't get the latest version because it's already been filtered out.
Okay, you're on GitHub, you have several projects for several languages but this doesn't need to be reflected when the context is obvious. So please keep the separation between your project name and what people will have to use on a daily basis.

Fixed in 2.1


 What is the recommended way to insert client_id, secret etc in oauth2_client table that we want to authorize?

Hi!
I recommend you to use CRUD generator (gii module) to make controllers and views.


support for asciinema?

Since this is a somewhat opinionated template, why not load it up with all the goodies?  asciinema is an obvious example.  The fact that I need it for my next presentation is uhh, purely coincidental.

I wonder what's a good generic way to include asciinema. Obvious choice would be to include the integration script in index.html, but I don't know how often that changes upstream, and that JavaScript seems not to be available as a submodule. Ideas?

Not sure, since I haven't gone through the process of using it myself yet.  But if I do, I'll put ideas here :-)

Note to self, this is the JavaScript code that asciinema.org generates, after putting it through a JavaScript beautifier:
! function() {
    function e(e, t) {
        e.parentNode.insertBefore(t, e.nextSibling)
    }

    function t(e, t) {
        function i(e) {
            var i = t.getAttribute("data-" + e);
            return i ? e + "=" + i : void 0
        }
        var a = ["size", "speed", "autoplay", "loop", "theme", "t"];
        return "?" + a.map(i).filter(Boolean).join("&")
    }

    function i(e) {
        var t = document.createElement("a");
        return t.href = e, t
    }

    function a(e) {
        var t = i(e.src);
        return t.protocol + "//" + t.host
    }

    function n(i) {
        function n(e) {
            if (e.origin === s) {
                var t = e.data[0],
                    i = e.data[1],
                    a = c.contentWindow || c;
                e.source == a && "asciicast:size" == t && (c.style.width = "" + i.width + "px", c.style.height = "" + i.height + "px")
            }
        }
        if (!i.dataset.player) {
            var s = a(i),
                l = s + "/api",
                r = i.id.split("-")[1],
                o = document.createElement("div");
            o.id = "asciicast-container-" + r, o.className = "asciicast", o.style.display = "block", o.style.float = "none", o.style.overflow = "hidden", o.style.padding = 0, o.style.margin = "20px 0", e(i, o);
            var c = document.createElement("iframe");
            c.src = l + "/asciicasts/" + r + t(o, i), c.id = "asciicast-iframe-" + r, c.name = "asciicast-iframe-" + r, c.scrolling = "no", c.setAttribute("allowFullScreen", "true"), c.style.overflow = "hidden", c.style.margin = 0, c.style.border = 0, c.style.display = "inline-block", c.style.width = "100%", c.style.float = "none", c.style.visibility = "hidden", c.onload = function() {
                this.style.visibility = "visible"
            }, o.appendChild(c), window.addEventListener("message", n, !1), i.dataset.player = o
        }
    }
    var s = document.querySelectorAll("script[id^='asciicast-']");
    [].forEach.call(s, n)
}();

Hmm, code beautifiers still have a way to go ;-)  As the saying goes, you can't polish a turd ... although perhaps I'm being a little harsh given that it was clearly minified ;-)

Well, I don't really want to copy this into my presentation template, not least because the asciinema-generated JS may change. Just copying the asciicast embed link doesn't work very nicely in Markdown though. I'm considering suggesting this:

Add the asciinema embed script to the bottom of index.html, without the id attribute:

<script type="text/javascript" src="https://asciinema.org/a/13104.js" async></script>

Add just the script placeholder where you want the player to appear:

# Some title
<script id="asciicast-13104"></script>

Neither of the two are very elegant though. Any better ideas?

What's not elegant about the second one?  Looks fine to me.

Except it doesn't seem to work. :/ Will have to come back to this one.


⚠ Bofrost.be has degraded performance

In e7c941c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2373 ms

Resolved: Bofrost.be performance has improved in ba29227.


data: harohar13

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

plz tell me what to do step by step
On Sat, 14 Oct 2023 at 20:53, Sindhu Inti @.***> wrote:

@.**** requested changes on this pull request.
In data/harohar13.json
https://github.com/EddieHubCommunity/BioDrop/pull/9447#discussion_r1359435731
:

@@ -0,0 +1,11 @@
+{

"name": "Harohar",
"bio": "Software engineer,writer",
"links": [
{

 "name": "Follow me on github",

 "url": "https://github.com/harohar13",

 "icon": "github"

Please use icon name as FaGithub
for more info checkout here https://www.biodrop.io/icons
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/BioDrop/pull/9447#pullrequestreview-1678157957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBG6GLCHXJ2P5J2NB6NYGJDX7KVARAVCNFSM6AAAAAA52LV5JOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNZYGE2TOOJVG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

plz tell me what to do step by step
…
On Sat, 14 Oct 2023 at 20:53, Sindhu Inti @.> wrote: @.* requested changes on this pull request. ------------------------------ In data/harohar13.json <#9447 (comment)> : > @@ -0,0 +1,11 @@ +{ + "name": "Harohar", + "bio": "Software engineer,writer", + "links": [ + { + "name": "Follow me on github", + "url": "https://github.com/harohar13", + "icon": "github" Please use icon name as FaGithub for more info checkout here https://www.biodrop.io/icons — Reply to this email directly, view it on GitHub <#9447 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BBG6GLCHXJ2P5J2NB6NYGJDX7KVARAVCNFSM6AAAAAA52LV5JOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNZYGE2TOOJVG4 . You are receiving this because you were mentioned.Message ID: @.***>

Just replace github with FaGithub 👍🏻 in line 8

I have edited the icon a committed changes , but now it is not changed in
the  pull request . And showing 2 workflows awaiting for approval.
On Tue, 17 Oct 2023 at 12:42, Sindhu Inti @.***> wrote:

plz tell me what to do step by step
… <#m_-2919423264051942805_>
On Sat, 14 Oct 2023 at 20:53, Sindhu Inti @.> wrote: @.** requested
changes on this pull request. ------------------------------ In
data/harohar13.json <#9447 (comment)
https://github.com/EddieHubCommunity/BioDrop/pull/9447#discussion_r1359435731>
: > @@ -0,0 +1,11 @@ +{ + "name": "Harohar", + "bio": "Software
engineer,writer", + "links": [ + { + "name": "Follow me on github", +
"url": "https://github.com/harohar13", + "icon": "github" Please use icon
name as FaGithub for more info checkout here https://www.biodrop.io/icons
— Reply to this email directly, view it on GitHub <#9447 (review)
https://github.com/EddieHubCommunity/BioDrop/pull/9447#pullrequestreview-1678157957>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBG6GLCHXJ2P5J2NB6NYGJDX7KVARAVCNFSM6AAAAAA52LV5JOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNZYGE2TOOJVG4
. You are receiving this because you were mentioned.Message ID: @.**>
Just replace github with FaGithub 👍🏻 in line 8
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/BioDrop/pull/9447#issuecomment-1765804477,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBG6GLCXFNTP3QMQ2MFFF3LX7YVW7AVCNFSM6AAAAAA52LV5JOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRVHAYDINBXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

plz help me to change that
On Tue, 17 Oct 2023 at 12:42, Sindhu Inti @.***> wrote:

plz tell me what to do step by step
… <#m_-2919423264051942805_>
On Sat, 14 Oct 2023 at 20:53, Sindhu Inti @.> wrote: @.** requested
changes on this pull request. ------------------------------ In
data/harohar13.json <#9447 (comment)
https://github.com/EddieHubCommunity/BioDrop/pull/9447#discussion_r1359435731>
: > @@ -0,0 +1,11 @@ +{ + "name": "Harohar", + "bio": "Software
engineer,writer", + "links": [ + { + "name": "Follow me on github", +
"url": "https://github.com/harohar13", + "icon": "github" Please use icon
name as FaGithub for more info checkout here https://www.biodrop.io/icons
— Reply to this email directly, view it on GitHub <#9447 (review)
https://github.com/EddieHubCommunity/BioDrop/pull/9447#pullrequestreview-1678157957>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBG6GLCHXJ2P5J2NB6NYGJDX7KVARAVCNFSM6AAAAAA52LV5JOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNZYGE2TOOJVG4
. You are receiving this because you were mentioned.Message ID: @.**>
Just replace github with FaGithub 👍🏻 in line 8
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/BioDrop/pull/9447#issuecomment-1765804477,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBG6GLCXFNTP3QMQ2MFFF3LX7YVW7AVCNFSM6AAAAAA52LV5JOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRVHAYDINBXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Awesome, thanks!
On Wed, Oct 18, 2023, 10:39 PM Priyanshu kumawat @.***>
wrote:

Merged #9447 https://github.com/EddieHubCommunity/BioDrop/pull/9447
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/BioDrop/pull/9447#event-10696713857,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBG6GLFKWDYYXX4N7W7OKH3YAAELFAVCNFSM6AAAAAA52LV5JOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQGY4TMNZRGM4DKNY
.
You are receiving this because you commented.Message ID:
@.***>


Realsense 515 loses ability to transfer by USB 3.0

Hi,
I have a Realsense 515 that is setup in a USB 3.0 system. After a couple of months it stopped reading as a usb 3 and only reads as a usb 2 device. Fortunately, i have an additional unit with me and i plugged it in to check the system, i.e usb cable and pertaining system components and it reads as a usb 3 device. i used the realsense viewer for these tests. what should i do now?
Regards,
Matt

Hi @MaChYD  Is there any difference if you try turning and inserting the micro-sized end of the USB cable in both directions (USB-C cables are two-way insertion at the micro-sized end).

Hi Marty, yeah tried that

Has the camera been stored inactive in a cool dry place such as a drawer or cupboard for more than a couple of weeks before the problem manifested?

yes i think stored for 2 weeks clean airconditioned environment

Storing cameras inactive in this way for a period of 2 weeks or more has been known to cause camera hardware failure in a small number of cases.  Usually the result is an inability for the camera to be detected or for the depth stream to not be able to start.  So your particular issue of USB 3 not being available is milder compared to the failures in those other cases.
In a couple of the cases, the camera restored its functionality on its own after a few weeks.  For example, one that was shipped from India to Intel in California was working normally by the time that it arrived, so environment may be a factor in the self-recovery.   So the best course of action may be to keep testing the camera for a few weeks and if it has not recovered by then then it is probable that it is a non-correctable hardware failure causing the USB issue.

ouh no, thank you for the reply. For my current case reliability is important, how then should i handle this?

You could attempt to return the camera to the retailer that you purchased it from using the Return Material Authorization (RMA) procedure of that particular retailer.
If the camera was purchased from Intel's official RealSense Store then you can contact the store using the form at the link below, selecting Return/Warranty s the topic.
https://www.intel.com/content/www/us/en/forms/support/intel-realsense-store-inquiry.html
If the camera was purchased from a non-Intel retailer then they may not accept a return though as the camera was purchased more than 30 days ago.  It will depend upon the policies of the particular retailer.

return would mean a replacement?

The L515 model is now End of Life (retired) and has been removed from the RealSense Store.  You would have to talk to the retailer, but it is probable that there would not be replacement units available and a refund may be issued instead if the return if accepted.

Hi @MaChYD  Do you have an update about this case that you can provide, please?  Thanks!

Hi Marty,
We are talking to the retailer, will update you when we get an update
Regards,
Matthew
On Wed, 16 Mar 2022 at 16:31, MartyG-RealSense @.***>
wrote:

Hi @MaChYD https://github.com/MaChYD Do you have an update about this
case that you can provide, please? Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/10305#issuecomment-1068862841,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWCFNEE77FYLJG2LKDOUAMDVAGL4VANCNFSM5QLZTX3A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks very much, Matthew.  Please do update me when you have news.  Good luck!

Hi Matthew, do you have an update that you can provide about your return discussions with the retailer, please?  Thanks!

Hi Matthew, do you require further assistance with this case please?  Thanks!

Case closed due to no further comments received.


[Task]: Add tests to check if the Infinite loading of the execution API in edit mode doesn't happen anymore

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Add tests to check if the execution of the API doesn't go to Infinite loading in edit mode when API just has View and execute permission

@ankitakinger - Should this issue be closed? I see the branch merged but, don't know why this is still open.

@infinitetrooper this is to add automated tests. Let's keep it open.