Saturate at in split_to call, to avoid panic

Hello,
When working with bytes, I noticed that split_to panics if the at value is larger than or equal to the length: https://docs.rs/bytes/1.0.1/src/bytes/bytes.rs.html#401-423
I propose that this function be modified to remove the assert, and replace line 409:
if at == self.len() {

to instead be:
if at >= self.len() {

This would mean if you have a 5-byte Bytes object and you called .split_to(10), it would be as if you called .split_to(5) on it, i.e. it would take everything that's left.
This also fixes a bug where an at value cannot be equal to self.len(), meaning line 410 never gets executed anyways.
Additionally, this makes for properly-behaving code in release mode for any given at value.
I believe the change would be backwards-compatible for any non-panicking code.
Are there any objections to this change? If not, I could whip up a PR.

I went ahead and created a PR for this: https://github.com/tokio-rs/bytes/pull/469

I went ahead and created a PR for this: https://github.com/tokio-rs/bytes/pull/469

I'm not one of the maintainers, but I believe splitting out of bounds is panicking because std does the same thing.
https://doc.rust-lang.org/nightly/std/primitive.slice.html#panics-17
In general, I feel like these „auto-fixes“ might let a bug go unnoticed for longer than necessary. If you do a split_to(10), you expect to have 10 bytes. You could even rely on that panic to protect some unsafe code accessing these 10 bytes, so in that sense, the change probably is not backwards compatible.

I'm not one of the maintainers, but I believe splitting out of bounds is panicking because std does the same thing.
https://doc.rust-lang.org/nightly/std/primitive.slice.html#panics-17
In general, I feel like these „auto-fixes“ might let a bug go unnoticed for longer than necessary. If you do a split_to(10), you expect to have 10 bytes. You could even rely on that panic to protect some unsafe code accessing these 10 bytes, so in that sense, the change probably is not backwards compatible.

I agree, I don't think this should be done. It's best to follow patterns in std, to surprise less people.

I agree, I don't think this should be done. It's best to follow patterns in std, to surprise less people.

That seems fair. I didn't realize this was a pattern that existed already in std. I still believe it's better to not have it panic, but I'll redirect my concerns elsewhere.
Thanks!

That seems fair. I didn't realize this was a pattern that existed already in std. I still believe it's better to not have it panic, but I'll redirect my concerns elsewhere.
Thanks!


How do I set up for "Contribution" ?

On the GitHub page of dffml, the "Contribution" documentation link is meant to guide the contributors on how to set up the dev. environment and start fixing the bugs. But that link didn't work. Need some help in clarifying those things from my fellow developers or our mentor.

I guess you're looking for this page: https://intel.github.io/dffml/master/contributing/index.html
If that is it then please close the issue, or feel free to ask for more help on gitter chat.


[WIP] Add ability to index individual trials when plotting rasters/histograms

Closes #471 Implements the ability to filter out specific trials either through lists or integers:
net.cell_response.plot_spikes_raster(trial_idx=0)
net.cell_response.plot_spikes_raster(trial_idx=[0, 1])

Codecov Report

Merging #472 (53a93fe) into master (0db52fa) will decrease coverage by 0.18%.
The diff coverage is 94.11%.

:exclamation: Current head 53a93fe differs from pull request most recent head 4614409. Consider uploading reports for the commit 4614409 to get more accurate results

@@            Coverage Diff             @@
##           master     #472      +/-   ##
==========================================
- Coverage   90.46%   90.28%   -0.19%
==========================================
  Files          18       18
  Lines        3378     3386       +8
==========================================
+ Hits         3056     3057       +1
- Misses        322      329       +7

Impacted Files
Coverage Δ

hnn_core/viz.py
83.36% <93.33%> (+0.07%)
:arrow_up:

hnn_core/cell_response.py
84.53% <100.00%> (ø)

hnn_core/parallel_backends.py
80.66% <0.00%> (-1.66%)
:arrow_down:

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 0db52fa...4614409. Read the comment docs.

does this do the job for you @wagdy88 @mkhalil8 ? You can try by doing:
$ git fetch upstream pull/472/head:trial_plots
$ git checkout trial_plots

Once it's merged, just switch to master
$ git checkout master

okay set the PR to [MRG] when ready and don't forget to update whats_new !

All set on my end!

Thanks @ntolley !

Thank you!!

I have re-installed hnn-core to test the new function after it was merged, I used the same code before and after and I encountered this difference, any thoughts what might be causing that? @jasmainak @rythorpe @ntolley

@mkhalil8 we have changed the way seeding is done. I suspect that's what causes the difference. See #462 #454. You will have to adjust your seeds if you want to get the previous results.

@jasmainak I played around with the seeding and re-produced the exact results from before. Thank you

As for plotting rasters of single trials, Yeah it is great. It is working well but I get this warning which does not affect the outcome:
python

hnn_core/viz.py:439: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray.
np.array(cell_response._spike_gids)[trial_idx]

@ntolley do you know if it's safe to fix this by providing dtype='object'? We should ideally not have any deprecation warnings with our code unless intended explicitly.


Added Gemfire repo

Why is this necessary?

Without it, @markfisher and I get this:

[ERROR] Failed to execute goal on project spring-cloud-stream-modules-test-support: Could not resolve dependencies for project org.springframework.cloud.stream.module:spring-cloud-stream-modules-test-support:jar:1.0.0.BUILD-SNAPSHOT: Failed to collect dependencies at com.gemstone.gemfire:gemfire:jar:8.0.0: Failed to read artifact descriptor for com.gemstone.gemfire:gemfire:jar:8.0.0: Could not transfer artifact com.gemstone.gemfire:gemfire:pom:8.0.0 from/to spring-snapshots (http://repo.spring.io/libs-snapshot-local): Failed to transfer file: http://repo.spring.io/libs-snapshot-local/com/gemstone/gemfire/gemfire/8.0.0/gemfire-8.0.0.pom. Return code is: 409, ReasonPhrase: Conflict. -> [Help 1]

Strange, I just built the project and it resolved all the gemfire jars fine.
Can you try changing <url>http://repo.spring.io/libs-snapshot-local</url> to be <url>http://repo.spring.io/snapshot</url> in .settings.xml

This artifact should really be resolved via <url>http://repo.spring.io/libs-release</url> not sure why the snapshot lib pulled it in. Maybe we should put the libs-release repo first?

I just tried it again, and this time it executed just fine without any changes. 😕
Could it be a problem with the repo itself?

I think the repo did the right thing, its not a snapshot release so it shouldn't resolve using a snapshot repo reference. This is the error I get when accessing it using the link above:
status: 409,
message: "The repository 'libs-snapshot-local' rejected the artifact 'libs-snapshot-local:com/gemstone/gemfire/gemfire/8.0.0/gemfire-8.0.0.pom' due to its snapshot/release handling policy."

In that case, should we just move the releases repo to the top of the list?
FWIW, this also worked:
diff --git a/.settings.xml b/.settings.xml
index aad3442..74904ae 100644
--- a/.settings.xml
+++ b/.settings.xml
@@ -17,7 +17,7 @@
                                <repository>
                                        <id>spring-snapshots</id>
                                        <name>Spring Snapshots</name>
-                                       <url>http://repo.spring.io/libs-snapshot-local</url>
+                                       <url>http://repo.spring.io/snapshot</url>
                                        <snapshots>
                                                <enabled>true</enabled>
                                        </snapshots>

Hard to tell unless we can reliably reproduce the issue. Don't think moving the release repo to the top would hurt. Also not sure why we use the full internal repo name instead of just http://repo.spring.io/milestone and http://repo.spring.io/snapshot.

Not entirely clear why we don't simply use http://repo.spring.io/libs-snapshot, which aggregates them all (including the gemstone repo). I'd avoid proliferating the list of repositories as much as possible.

Just tried Marius' suggestion and it looks to work.  Not sure why the spring-cloud project doesn't follow this as well.  Let's ping those folks.

Closed in favor of https://github.com/spring-cloud/spring-cloud-stream-modules/pull/50

So backtracking here.  This PR looks to be the better approach vs. #50.  The reason is that if you include http://repo.spring.io/libs-snapshot then you get spring project dependencies and all their transitive dependencies hosted by libs-snapshot, sort of a wildcard.  If one uses the more selective -local flavored repos you don't pick up any other transitive dependencies other than those in maven central.  That keeps the knowledge of what extra repos are needed more explicit.  The core issue is that gemfire libraries are not in maven central, so need to be referenced some how.  I'll merge this with an additional comment that the gemfire libs are not in maven central.

Merged with a few other minor changes as e157b4bf0d7c5ef1c7f705ece18b0fe6f95663a9


custom scrollbar

please assign me this issue

can u please ellaborate the issue

We have a scroll bar attached with our window to scroll it top bottom , that is of silver colour by default so I want to design it according to the project's theme

can u please ellaborate the issue

can i work on this issue ??

You still want to take up this issue ?

You still want to take up this issue ?

If you allow me I can

Can you please assign me

this issue is alotted to someone else . thanks for showing for taking up this issue


Link color contrast

$nypl-blue (#2799c5) used for links has 3.26:1 color contrast to white.  Ideally we'd have at least 4.5:1 for normal text. Link to WebAIM recommendation

PR: #25


proposal: text/template/parse: add CommentNode to the parse tree

Currently, template comments are ignored in the lexer and do not show in the parse tree.
It makes it difficult to build tools around templates (like code formatter) without re-implementing the parser.
My proposal is to add a CommentNode to the parse tree and skip it in the executer.

I assume this can only be accepted if https://github.com/golang/go/issues/25357 is rejected.

I assume this can only be accepted if #25357 is rejected.

Why? It's a Go2 proposal, and text/template/parse is a public package that's being used outside.

Yes; but if the plan is to eventually hide/remove the template/parse package, considering API changes to it wouldn't make a lot of sense. At the very least, the two proposals are related.

/cc @robpike for thoughts about supporting template-manipulation programs a la gofmt by adding CommentNode.

I think it’s needed for gopls as well  #36911.

I would have made the package internal had that been an option at the time, but I couldn't, and now it's being used by software that wouldn't have been able to, making it harder to maintain and de facto subject to the compatibility rules. From my position, that is a tale of woe. Hyrum's law, which was a problem before Hyrum was even born, bites again.
On the other hand, people do depend on it and the uses are mostly reasonable, so we should probably accept this and close #25357.  I do think though that we should keep the comment that discourages its use, to give us some leverage against unreasonable requests for change.

Based on the discussion above, then, this sounds like a likely accept.
For what it's worth, I might have made regexp/syntax an internal package too, but it has proved so useful in so many other contexts that I'm glad I exposed it.

Sounds good to me. Should we reject https://github.com/golang/go/issues/25357, then? In the past few years it's been made clear that Go2 will be an incremental process instead of one big set of breaking changes, so it seems unlikely to me that we'd hide an entire package at any point.

Thanks for the feedback. I would be happy to add this once it's accepted.

Should we reject #25357, then?

IMHO, the template package should provide an API for tools like gopls to interact with it. Like the go package for Go.
Otherwise, what's the alternative, reimplementing it externally? This is what I do currently, but I expect to replace it once gopls will add support for it (#36911).

No change in consensus, so accepted. Yes, I think we can reject #25357 as well.
For better or worse, this package is the equivalent of go/ast for templates.
In a different timeline we might have made it an internal-only package and
prepared an API that was specifically meant for public consumption, but that's
not what happened.

If this has been accepted, then I take it there's no interest in marking text/template/parse as deprecated?

If this has been accepted, then I take it

Since I already have a working change for this in my fork that I plan to push in the upcoming days, I'll prefer to push it. Thanks

The current CL is adding a new ParseWithOptions function; see #38627.

Hey, CL229398 is currently blocked on review. There's another discussion on #38627 about making the API more generic for changes in the future.
Any chance to get help with the review? I really want to see it shipped with 1.15.


Can not use 'in' operator in traveling aql .

My Environment

ArangoDB Version:
Storage Engine:
Deployment Mode:
Deployment Strategy:
Configuration:
Infrastructure:
Operating System:
Total RAM in your machine:
Disks in use:
Used Package:

Component, Query & Data
Affected feature:

AQL query (if applicable):
//section 1
with DOC
let x = (
  for v,e,p in 2 outbound 'DOC/0' EDGE
    filter p.edges[1]._to == 'DOC/1'
    return p.vertices[1]._id
)
//section 2
for v,e,p in 2 outbound 'DOC/3' EDGE
  filter p.edges[1]._to in x // <--
  return p.vertices[1]._id

AQL explain (if applicable):

Dataset:

// This dataset information is not the practical using dataset (huge size) but for descripting the issue. I am not sure wether this problem is related to the size of dataset .
// collection DOC
[
{_id:'DOC/0'},
{_id:'DOC/1'},
{_id:'DOC/2'},
{_id:'DOC/3'},
{_id:'DOC/4'},
{_id:'DOC/5'},
{_id:'DOC/6'},
{_id:'DOC/7'},
{_id:'DOC/8'},
{_id:'DOC/9'}
]
// edge EDGE
[
{_id:'EDGE/0',_from:'DOC/0',_to:'DOC/5'},
{_id:'EDGE/1',_from:'DOC/0',_to:'DOC/6'},
{_id:'EDGE/2',_from:'DOC/5',_to:'DOC/1'},
{_id:'EDGE/3',_from:'DOC/3',_to:'DOC/7'},
{_id:'EDGE/4',_from:'DOC/3',_to:'DOC/8'},
{_id:'EDGE/5',_from:'DOC/3',_to:'DOC/9'},
{_id:'EDGE/6',_from:'DOC/7',_to:'DOC/5'},
{_id:'EDGE/7',_from:'DOC/8',_to:'DOC/5'},
{_id:'EDGE/8',_from:'DOC/9',_to:'DOC/6'}
]
Size of your Dataset on disk:

Replication Factor & Number of Shards (Cluster only):

Steps to reproduce

Problem:
The AQL is executed correctly, but the returned array is empty.  If I execute section 1 separately I can get array x with its size more than 1. If I copy one of values in array x  (example as 'DOC/5' with index 0 ) and replace the filter condition 'in x ' with 'in ["DOC/5"]' or '== x[0]' , I can get correct result (not empty). But it can not work when using 'in x'.
For given dataset here , the correct result should be ['DOC/7','DOC/8'] and x is ['DOC/5'].
Expected result:
Correct result that not empty like use " == x[0]".
For given dataset here , the correct result should be ['DOC/7','DOC/8'] and x is ['DOC/5'].

Please provide a dataset that reproduces the problem.
What strikes me is that you use a min/max traversal depth of 2 in both cases, but you return the vertex ID from depth 1, whereas the filter condition tests the edge at depth 2. Is that really what you want to do?
//section 1
with DOC
let x = (
  for v,e,p in 2 outbound 'DOC/0' EDGE
    filter p.edges[0]._to == 'DOC/1' (first edge)
    return p.vertices[2]._id // 'DOC/2'? (p.vertices[0] is the start vertex 'DOC/0')
)
//section 2
for v,e,p in 2 outbound 'DOC/3' EDGE
  filter p.edges[ /* ? */ ]._to in x // which of the two edges do you mean to filter on?
  return p.vertices[2]._id

I created a document collection DOC and an edge collection EDGE, then ran the following AQL queries:
for d in [
{_key:'0'},
{_key:'1'},
{_key:'2'},
{_key:'3'},
{_key:'4'},
{_key:'5'},
{_key:'6'},
{_key:'7'},
{_key:'8'},
{_key:'9'}
] insert d into DOC

for d in [
{_key:'0',_from:'DOC/0',_to:'DOC/5'},
{_key:'1',_from:'DOC/0',_to:'DOC/6'},
{_key:'2',_from:'DOC/5',_to:'DOC/1'},
{_key:'3',_from:'DOC/3',_to:'DOC/7'},
{_key:'4',_from:'DOC/3',_to:'DOC/8'},
{_key:'5',_from:'DOC/3',_to:'DOC/9'},
{_key:'6',_from:'DOC/7',_to:'DOC/5'},
{_key:'7',_from:'DOC/8',_to:'DOC/5'},
{_key:'8',_from:'DOC/9',_to:'DOC/6'}
] insert d into EDGE

Note that I modified your provided dataset example to use a _key attribute, not an _id which will be ignored.
In the graph viewer I verified that everything is connected correctly by creating a named graph:

After clicking the "Fetch full graph" button:

Then I ran your query:
with DOC
let x = (
for v,e,p in 2 outbound 'DOC/0' EDGE
filter p.edges[1]._to == 'DOC/1' // the second edge (EDGE/2)
return p.vertices[1]._id // the second vertex (DOC/5)
)
for v,e,p in 2 outbound 'DOC/3' EDGE
filter p.edges[1]._to in x // x == ['DOC/5']
return p.vertices[1]._id // the second vertex (['DOC/7','DOC/8'])

It returns the expected result (DOC 7 and 8):

I used a single server instance of ArangoDB v3.6.1 under Windows for my test.
@TsaiQi Please check that your vertices and edges have the correct _key values. If they do, please let me know which ArangoDB version you are using and in what deployment type and under what OS. Give the latest 3.6.1 a try and see if the problem still occurs there.

Hi @TsaiQi,
Did you have a chance to read Simran's latest reply?

Please check that your vertices and edges have the correct _key values. If they do, please let me know which ArangoDB version you are using and in what deployment type and under what OS. Give the latest 3.6.1 a try and see if the problem still occurs there.

Hi @TsaiQi,
Since we have not heard back from you recently, I am closing this issue for now.
Feel free to reopen this ticket if needed.


#273 (redefine minos.common.Command and minos.common.CommandReply attributes)

Resolves #273

Aaah, thanks @vladyslav-fenchak, you're right, I forgot to set the right branch.


Excel Rendering issue - ViewerJS

Hello,
I am facing an issue with the viewer in the open format of excel. If there are two sheets in the excel file then while rendering the second sheet is getting concatenated after the first one.
We need this to be shown as two sheets. Please let em know if this is possible.
Thanks

Anyobdy?

any Update on this ?


refactor: Override pre-defined profile and devices on device-services

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-compose/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

can't --configDir flag work anymore?

It didn't work for virtual, but still works for modbus.

instead of using environment variable, we should diagnose how to fix the flag
please open an issue in device-virtual

Update TAF script instead. edgex-taf #779


"Protect access to repositories in YAML pipelines" setting affects the usage of the Access Token

Greetings,
It would be good to include the reference to "Protect access to repositories in YAML pipelines" setting, as it affects the scope of the Access Token.
Reference : https://developercommunity.visualstudio.com/t/error-creating-repository-in-azure-devops-tf401027/1653143.
If you are okay, I am happy to update the FAQ with the details and submit a PR.
Please let me know.
Regards,
Vudaya

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

ID: cdd0749b-b681-8192-3849-f38e6fc7f138
Version Independent ID: fb5170d3-2e45-3caf-be17-e012bd523660
Content: Understand job access tokens - Azure Pipelines
Content Source: docs/pipelines/process/access-tokens.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @vijayma
Microsoft Alias: vijayma

@vuvasira -- Vudaya, thank you for your feedback and kind offer.  Please do create that PR.  You are welcome to ask me for any assistance you need.
cc @vijayma for awareness


[500] Issue while updating schema when using partitioning

Hello,
I was trying to create a cube in "manual" mode, and when the data store has partitioning I face the issue "[500] Issue while updating schema" - for example, when I create a measure for a sum, or when I set folder to a contributors count.
Steps to reproduce
import atoti as tt
import pandas as pd

df = pd.DataFrame(data = {"Col1": [1,2], "Col2":["A", "B"]})

session = tt.create_session()
table = session.read_pandas(df, keys= ['Col2'], table_name = "Base", partitioning="hash8(Col2)",)

cube = session.create_cube(table, name = "cube", mode = "manual")

m = cube.measures
list(m)

m['contributors.COUNT'].folder = "Technical"
m['sum'] = tt.agg.sum(table['Col1'])

Actual Result
[500] Issue while updating schema
Expected Result
I want to be able to create measures, and change folders.
Environment

atoti: <IP_ADDRESS>11004165134
Python: 3.9.7
Operating System: darwin

Hello,
The problem seems to come from the fact that you don't have any hierarchies defined. I'm investigating to see why this only happens if the data is partitionned.
Just out of curiosity, what is the reason you are creating/modifying the measures first?

I've created an internal support ticket to track this issue.
The problem occurs when a partitioned field is does not have a level which uses it. I would recommend always creating your hierarchies first to avoid this kind of issue while we wait to see how the core support ticket is handled.

They shouldn't need be, the error is due to a core bug, once it's been resolved you should be able to create hierarchies and measures in whatever order you want.

Hello!
This has been fixed in the next release!
Cheers,
Acriss


Only fail disable firewalld when it's actually installed.

Not all cloud providers install firewalld by default in their templates which breaks this role on those installations.

Thanks for this! Are you going to merge your other changes in as well?


Adding support for Headers and GET params

@tusharghate if you approve, you should update the application version number and re-publish to bower


No module named "evtxtoelk"

Hi, sorry,
maybe it´s a beginner question but I´m trying to figure out how the parser works. I followed the instructions from the "https://dragos.com/blog/industry-news/evtxtoelk-a-python-module-to-load-windows-event-logs-into-elasticsearch/" site and I got the point where I need to load windows logs to elk. I get this message:

ModuleNotFindError: No module named ´evtxtoelk´.

What am I missing? Thanks a lot for any help.

Hi @miranb!
Do you have the module loaded via pip? What environment are you currently working in?

if you are using python 3 then use the python3 environment not python


get mean and std from sampler

Codecov Report

Merging #17 (b299345) into master (d8a0698) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          294       294
=========================================
  Hits           294       294

Impacted Files
Coverage Δ

src/sampler.jl
100.00% <100.00%> (ø)

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 d8a0698...b299345. Read the comment docs.


Prevent duplicate styles

-Updated the regex to ignore files that aren't using *
-Updated regex to better handle semi-colons and spacing
-Updated gitignore for .DS_Store files

Hey @gkiely I'd like to pull this in, just haven't had time to do the merge yet. Feel free to re-open with a clean pull or I can figure out the merge when I have a minute

Hey @mathisonian I'll update this in the next few days.

cool, thanks @gkiely! sorry i haven't gotten to it yet


Remove Handle prefix from handler methods. Remove breaking changes do…

…cs, which are moving to ziti-cmd

Wouldn't it make sense to either use the Handler idiom throughout the codebase (both edge and fabric), or jettison it throughout the codebase?

Wouldn't it make sense to either use the Handler idiom throughout the codebase (both edge and fabric), or jettison it throughout the codebase?

Yes, if they were following the same pattern, but I don't feel they are.  So in the same way that network doesn't have Handle prefixed everywhere, I don't think this code should either.
That probably means they should not be called Handler either, but Controller is longer and Model is kind of overloaded. Can rename them when we come up with a good name :)

That probably means they should not be called Handler either, but Controller is longer and Model is kind of overloaded. Can rename them when we come up with a good name :)

"Controller" is also overloaded.
Just want to keep making sure that we're thinking about the entirety of the architecture, not just the edge.


Access IndexedDB from window

Hi,
I'm trying to access indexedDB from the window object, however I keep getting the below error.
DOMException: Failed to execute 'open' on 'IDBFactory': access to the Indexed Database API is denied in this context.
This is how I am trying to do it.
await page.evaluate(() => {
    const idb = window.indexedDB.open("databaseName");
});

I also set --allow-file-access-from-files as part of the args in the jest-puppeteer.config.js and set the browserContext: 'default'
Still I cannot get it to work! Does anyone have the same problem?

Yeah I think so.

@SilvanTheuma did you find a workaround to this?

@andreisaikouski Have some JS code OUTSIDE of Puppeteer execute the IDB commands that runs parallel to Puppeteer. Keep that code on the DEV env only


Ability to output full list of know entities

Could you add the ability to output either a separate report or a configurable option to output the complete list of known entities, preferably as a csv, with a column alongside each containing the watchman output, i.e. OK, Missing, Unknown, Unavailable

May I ask what problem you are solving by having OK entries in the report?

nothing in particular, just a status for those entities which are confirmed to be in working order as it were

Good idea.
As for now I've 25 entities tracked and don't know why some are not detected as unavailable (and they're unavailable).
They're not all detected as I've more than 25 entities.


Write a Kotlin program to print all prime numbers between two numbers

Description
Write a Kotlin program to print all prime numbers between two numbers
Input  : 10 20
Output : 11 13 17 19

How to contribute

Save the solution in PrintAllPrimeNumbersBetweenTwoNumbers.kt file
Add PrintAllPrimeNumbersBetweenTwoNumbers.kt file in print-all-prime-numbers-between-two-numbers folder

!assign


publish Endpoint bug

Expect : Publish statd data to application insight.
Actual: Failed to publish.
Error: The document is moved. The endpoint 'http' protocol it was publishing throwing a redirection response.
Change : upgrade the applicationinsights dependency to 0.17.2 which change the protocol to https.
The application endpoint doesn't accept http request and the fix was added with release 0.16.0.

The failing part of the build looks like an environment issue and not related to the change proposed.

The Application Insight team no longer maintains this SDK, and it will be archived. Though we cannot officially recommend an alternative, you could try using the "binroon / ApplicationInsights-statsd" repo a fork from a member of the AI User Community.


Cannot connect external cluster

What happened?
[fatal]  check resource availability: discover server resources: unable to retrieve the complete list of ser
ver APIs: admission.certmanager.k8s.io/v1beta1: the server is currently unable to handle the request

What did you expect to happen instead?
Connect external cluster
How can we reproduce the bug? (as minimally and precisely as possible)
 ~/P/@/core  ⎇ rbac $  devspace connect cluster                 7s < понедельник, 27 апреля 2020 г. 16:28:15
[warn]   There is a newer version of DevSpace: v4.12.2. Run `devspace upgrade` to upgrade to the newest version.

? Please enter a cluster name (e.g. my-cluster) handyai-main

? Which kube context do you want to use
 arn:aws:eks:eu-central-1:606350344667:cluster/handyai-main-eu-central-1
[fatal]  check resource availability: discover server resources: unable to retrieve the complete list of ser
ver APIs: admission.certmanager.k8s.io/v1beta1: the server is currently unable to handle the request

Local Environment:

DevSpace Version: [devspace version 4.9.3]
Operating System: mac
Deployment method: helm | kubectl apply

Kubernetes Cluster:

Cloud Provider: aws [EKS]
Kubernetes Version:

Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480
e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:16:51Z", GoVersion:"go1.12.10", Compiler:
"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.9-eks-502bfb", GitCommit:"502bfb383169b124d87848f89e17a04b9fc1f6f0", GitTreeState:"clean", BuildDate:"2020-02-07T01:31:02Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}

Anything else we need to know?

/kind bug

@0x77dev thanks for reporting this issue! The problem is that your cluster apiservices (the apiservice admission.certmanager.k8s.io) cannot be discovered, which is why devspace fails, you can verify this via kubectl get apiservices. Devspace does an initial cluster check before connecting a cluster and if that fails the cluster connect fails otherwise tools such as helm cannot be used in the cluster since they rely on api resource discovery. I suspect that you have a faulty apiservice (which can be deleted via kubectl delete apiservice admission.certmanager.k8s.io) that needs to be fixed before you can connect the cluster. As soon as all apiservices are Available in kubectl get apiservices connecting a cluster should work.

Yeah, so I've just deleted cert manager, that's fixed this problem.


perf(tasks): speed up NAPI parser benchmark build

#2787  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


[Front-End]: Mobile active list element shaking

When you open the dropdown on the last active list with multiple lists active, the element shakes.
I think this has something to do with this CSS setting:
scroll-snap-align: center;

https://github.com/rrickgauer/lists/blob/16cf2f83fbfca1c21e17bb3a7d5289ebd5f05163/src/front-end/lists/static/css/custom/style.scss#L264

I think this has something to do with this CSS setting:
scroll-snap-align: center;

Specifically:
https://github.com/rrickgauer/lists/blob/16cf2f83fbfca1c21e17bb3a7d5289ebd5f05163/src/front-end/lists/static/css/custom/style.scss#L264


DB modeling and diagraming

Task Description

[ ] Model users table
[ ] Model books table
[ ] Model categories table
[ ] Model authors table
[ ] Model reviews table
[ ] Model reading_lists table
[ ] Set relationships between tables

Technical Considerations
Additional Considerations
MoonModeler

https://app.zenhub.com/files/401346050/c32665c1-0f5e-45ca-b745-9c99961a1bc8/download
initial Schema designs

@sohanur-rahman0 Looks great! 👍🏻


-fixes for:beans without a name attribute, "resource" attribute property resolving, anonymous beans are inserted only once

-"resource" attribute property resolving
-beans without a name attribute still to be used with templating: "${" becomes "----" (four times "-") on id and "}" becomes "___" (four times "") on id
-anonymous beans are inserted only once, no exceptions.

Thanks a lot for your help

Thanks for the great code.

Hi, on my machine the unit tests are not working any more. Will fix them before pushing out a release.

Released 1.0.4.
Perhaps you can retest your text replacements.

Thanks for the release.
Everything seems OK on my side.

Also thanks for the nice refactorings


[e2e] Fix Frequent 2FA Code Reuse Issue in github test case

Description
[e2e] Fix Frequent 2FA Code Reuse Issue in github test case
Which issue(s) does this PR fix
https://issues.redhat.com/browse/RHIDP-3237
PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/cherry-pick 1.2.x


Refactor to match TutorTrack

Changed all instances of Person to Student and all instances of Tag to Timeslots

Currently timeslot only checks if its of the form DayOfWeek StartTime-EndTime
Future improvements: Allow DayOfWeek to be Case insensitive
Ensure start time is before end time


How to setup JWT_KEY ?

Hello sir,
Will you please tell me how to setup JWT_KEY?
and elaborate meaning the meaning of
Create a JWT secret key (e.g. using openssl rand -base64 32) and either:
Setup the env variable JWT_KEY
Add it to the configuration file at config/development in app.jwtKey.
i.e. 6th step in guide.
I've setup the JWT but im getting error like JWT_KEY is malformed.
thank you.

Hi,
The JWT secret is used to sign/validate the tokens and, thus, should be a strong and hopefully hard to crack/guess. You can set it to whatever you want, I just gave the suggestion of using the "openssl" CLI tool to generate a random string.
Did you try setting the value of your JWT to other strings and see how it went? Are you setting a string value?

Created JWT using crypto. And testing backend api its simply increadible. Thank you so much for sharing this really very valuable code :1st_place_medal:

This is not issue I want to add OTP registration feature. How should I setup OTP verification in atlas I just want overview. As you have designed this product.

Hi,
One-time password? Like using SMS tokens?

Yes it's something like that
On 14-Jan-2017 4:13 PM, "André Tavares"<EMAIL_ADDRESS>wrote:

Hi,
One-time password? Like using SMS tokens?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/yoonic/atlas/issues/10#issuecomment-272616240, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANaQ0TA4RnNBDoabkFVElqI3qHAl26QPks5rSKbZgaJpZM4LY2M5
.

I just want overview, I will implement it.

I would perhaps update the Login API (POST /api/v1/login) so that, when OTP is enabled:

(Client) Send valid credentials
(Server) Creates OTP and sends it to client
(Server) Returns HTTP 202 Accepted (processing has not been completed)
(Client) Receives OTP
(Client) Send valid credentials + OTP
(Server) Returns valid session token

thank you so much :+1:

Done. Thank you for OTP thing!
I want to publish this app to production.
How should I do it?
I ve customised nicistore too.
I haven't published node app yet.
What is the process for that.

My advice for deployment is to use Docker and I have provided a Dockerfile for both apps (nicistore and atlas).

I have implemented OTP as you advised me to. Now I want to integrate Facebook and Google login how should I do it? Any  idea?

I haven't hosted a nodejs application to server. I got a vps server. Will anybody tell how to host it to production using docker file any tutorial available?

You can either try and use some existing library that integrates with 3rd-party authentication systems or you'll have to read the different flows for each one and integrate them with Atlas.

If you have no prior experience with hosting and docker, perhaps the easy route would be to install the app on the host system, using something like PM2 http://pm2.keymetrics.io/

Thank you sir. It really helps.

@andreftavares When sudo npm run dev the PWT Key it throws:
SyntaxError: /home/.../atlas/config/development.js: Unexpected token (7:72)
Can't even get the app started. @phoenixvk did you encountered the same problem?

What's the value you put there?

Got it fixed. You didn't specify to replace proccess.env with the JWT key but with " ".


Vision Studio sign-in

Lab/Demo: 05-exercise-explore-image-analysis.md
Task: Connect your Cognitive Services account to Vision Studio
Step: 1 and 2

Description of issue
When opening this page for the first time, the student is not signed in, and because of that the Get started with Vision screen is not showing up.
This is what it is shown:

Suggestion:
Ideally, the student would sign in first before getting access to this page.
Also, it is worth noting that if the student has access to different azure directories, the correct one for this exercise should be chosen/confirmed. This is how the instructions would look :

In a web browser, navigate to Vision Studio.
Sign in with your account, and make sure you are using the same directory as the one where you have created the cog-ms-learn-vision resource.
On the Vision Studio home page, select View all resources under the Getting started with Vision heading.

The steps have been updated as suggested.


How to reset value StateProvider on dispose

My case is simple , i want reset value of StateProvider if user not longer in screen . for this , i reset value on dispose. But when dispose method call , i get this error  :
The following assertion was thrown while finalizing the widget tree:
Looking up a deactivated widget's ancestor is unsafe.

At this point the state of the widget's element tree is no longer stable.

To safely refer to a widget's ancestor in its dispose() method, save a reference to the ancestor by calling dependOnInheritedWidgetOfExactType() in the widget's didChangeDependencies() method.

When the exception was thrown, this was the stack
#0      Element._debugCheckStateIsActiveForAncestorLookup.<anonymous closure>
package:flutter/…/widgets/framework.dart:3781
#1      Element._debugCheckStateIsActiveForAncestorLookup
package:flutter/…/widgets/framework.dart:3795
#2      Element.getElementForInheritedWidgetOfExactType
package:flutter/…/widgets/framework.dart:3861
#3      ProviderScope.containerOf
package:flutter_riverpod/src/framework.dart:99
#4      BuildContextX.read

final base64Image = StateProvider((ref) => '');

class _FormDosenScreenState extends State<FormDosenScreen> {
  @override
  void dispose() {
    print('dispose form dosen');
    context.read(base64Image).state = '';
    super.dispose();
  }

My question , how to reset value of StateProvider when no longer in screen ?

Are you tried StateProvider.autoDispose?
https://riverpod.dev/docs/concepts/modifiers/auto_dispose

@rrousselGit  Don't know have this method. Thank's !


Test Case Development

This issue (user story / story) is for the creation of Test Cases that include both Manual and Automation Test Cases. The following is the estimate:

Manual Test cases --> 30 hours
Automation test cases --> 60 hours
Automation test cases include the following activities:
a) Framework design
b) Script development

1.Gone through the test scenarios and found , no access for me . Sent out a mail and awaiting for the access.
2. Preparing Test cases for the New Master Category Screen

Task is completed.


Fix the problem of write loss when the buffer is too large

[BUG]/[FEATURE] title
What changes were proposed in this pull request?
Why are the changes needed?

What are the items that need reviewer attention?
Related issues.
Related pull requests.
How was this patch tested?
/cc @related-reviewer
/assign @main-reviewer

 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.jiadongdong seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

ok
------------------ 原始邮件 ------------------
发件人: "Kent @.>;
发送时间: 2022年9月1日(星期四) 下午3:20
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [alibaba/RemoteShuffleService] Fix the problem of write loss when the buffer is too large (PR #495)
@waitinfuture commented on this pull request.
In server-worker/src/main/scala/com/aliyun/emr/rss/service/deploy/worker/storage/FlushTask.scala:
> @@ -33,7 +33,12 @@ private[worker] class LocalFlushTask(      fileChannel: FileChannel,      notifier: FlushNotifier) extends FlushTask(buffer, notifier) {    override def flush(): Unit = { -    fileChannel.write(buffer.nioBuffers()) +    val buffers = buffer.nioBuffers() +    for (buffer <- buffers) { +      while(buffer.hasRemaining) {
Plz add a whitespace after while
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Unable to build Docker image

I ran update-image.yml on master and it failed while running pipenv sync in the Docker build.
ValueError: "Requirement" object has no field "use_pep517"

Others on the internet have reported that this error comes from a version of PyYAML (which is included in the Pipfile.lock).
I tried re-generating the lock file using pipenv lock locally but it failed.

I haven't dug, so this is a guess, but I'm guessing this issue is restricted to Pipenv and the same version of PyYAML would be fine with standard Pip. If so, that's all the more reason to ditch Pipenv, like we've done elsewhere. If we need strict locking, we can use pip-compile from pip-tools, as elsewhere.


feat(hooks): Make hook enable flag non-default

Removes default implementation for event hooks so that each individual hook is forced to implement the enable/disable flag.
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Failing test in unrelated, merging.


Added public initializer for KeyParams.

Pull Request
Related issue
The KeyParams and KeysQuery structs do not have publicly accessible initializers. This implies that any code outside the MeiliSearch module cannot create new instances of these structs.
This works in direct contrast to examples laid out in the Meilisearch documentation such as when it displays constructing new KeyParams instances to pass to the POST /keys endpoint, which can be seen here, if you select Swift as the client language.
I've simply explicitly added a public initializers to both the KeyParams and KeysQuery structs.

bors try

Hello @mannuch
Thanks for the PR!
Looks like there is a linter issue. Could you fix it?

Hi @mannuch thanks for your contribution!!
I would like to know if you have any tips to test this behavior, so we can prevent that from happening again in the future.

Hi, @mannuch are you still interested in continuing working in this PR?

Yes I am! Sorry I've been busy the past couple weeks, but I will try to implement the necessary changes at the end of the week.
Thanks for your patience!

Thanks again for contributing to Meilisearch :heart:
If you are participating in Hacktoberfest, and you would like to receive some gift from Meilisearch too, please complete this form.


Add "getting started" guide

Add "getting started" guide for new contributors to get their environment set up
Include instructions for installing WSL for users on Windows
Include optional instructions for setting up Sphinx

I'd love to have reviews from anyone and everyone! I have tested this with some happy helpers, but more testing is always welcome. Please feel free to suggest steps I've missed or anything else that might make this better/more helpful.
Note: The Sphinx instructions are not required for working with the ODA, but if we choose to have rendered docs in the future, will need to be moved into the tutorial steps.

Wow, thanks so much @s-makin! I'll mention this in our upcoming office hours - it may be worth mentioning on the forum too, or perhaps after it has been merged if you prefer. Looks ace already though.

Thanks again for this @s-makin! Great work, and a brilliant addition to the repository. I'm going to merge this now - we can always add and edit later, if necessary.


Migrating from signalfx-tracing to splunk-otel-js Tracer Question

Hello,
I am in the process of migrating from signalfx-tracing to splunk-otel-js.
When using signalfx-tracing, the instrumentation entry point returns tracer:
const tracer = require('signalfx-tracing').init({
  // your options here
})

According to the documentation, the new way of doing this is:
const { startTracing } = require('@splunk/otel');

startTracing({
  // your new options here
});

It looks like startTracing does not return anything.
When using the signalfx-tracing library, we depended on the tracer object returned from the init function throughout our application (i.e. tracer.scope() and tracer.startSpan()).
Can you help me understand how to get the tracer object when using splunk-otel-js?
Thanks!
@jtmalinowski

Welcome! splunk-otel-js's purpose it to streamline the configuration process, and help you connect to Splunk Observability quickly, while everything else, like for example manual instrumentation, follows Open Telemetry specification.
A good starting point in your case is https://github.com/open-telemetry/opentelemetry-js-api/blob/main/docs/tracing.md.  It might be a bit counter-intuitive in the beginning, but you need to interact with @opentelemetry/api, rather than with @splunk/otel-js.
This is a very valid issue, and I'll definitely address it in our migration guide, but it may take a couple of days. Also, please do let us know about anything you found unclear in the linked tracing.md guide, so we can improve it in addition to improving our guide.

Thank you for the clarity @jtmalinowski !
My only other question is regarding what the correct order needs to be... Does the @splunk/otel startTracing() function need to be called before interacting with @opentelemetry/api?
i.e.
const { startTracing } = require('@splunk/otel');
startTracing(options);

const { trace } = require('@opentelemetry/api');
const tracer = trace.getTracer('my-application', '0.1.0');

Thanks again for your help.

Yes, that's correct, startTracing() must be called first. That's why we like the node -r tracing.js app.js pattern (like here), where tracing.js calls startTracing() and app.js starts your application. It ensures that instrumentation is executed before the application starts.
The tracer returned by @opentelemetry/api is different, and it's this one. It doesn't have a .scope() method.
I'm assuming that you'd like to access tracer.scope() to connect spans together in a trace. The API to achieve that is described in this paragraph in Open Telemetry docs. Let me highlight the most important parts:
import { context, setSpan, trace } from '@opentelemetry/api';

// creates a span but it doesn't become the active span yet
const span = tracer.startSpan(`GET /user/:id`);

// creates a context which is sort of like scope in Open Tracing
const ctx = setSpan(context.active(), span);

// runs your function, which is getUser in this case
// if you call context.active() within getUser, you'll get the ctx from above
// whatever happens within getUser will be linked to your ctx context
context.with(ctx, getUser, undefined, userId);
// or if getUser returns a promise
await context.with(ctx, getUser, undefined, userId);

// always remember to end the span
span.end();

As you can see the scope from signalfx-tracing behaves differently. The process of migration from signalfx-tracing to @splunk/otel will require more than find-and-replace. We'll be building more example and docs soon.
cc @theletterf @Rauno56

Currently OoO — please report this and other changes through our Docs Jira. Thanks!

Thank you very much for your help! @jtmalinowski

We're always happy to help!


Drag functionality with Scroller

Is there any way to resize the sheet using drag gestures, when using Scroller?

If by resizing you mean changing the snap point of the sheet with scrollable content in it, then yes:

Code: https://github.com/Temzasse/react-modal-sheet/blob/main/example/components/ScrollableSnapPoints.tsx
Demo: https://temzasse.github.io/react-modal-sheet/#/scrollable-snap-points