fixed the issue in scrapping.py

#14  fixed the issue with relevant comments as well
please review and merge the PR
-thankyou
kairveeehh

@sanjay-kv please attach levels to both the PRs so that they can be counted in the leaderboard @sanjay-kv

its a minor issue cant add level to these . sorry about that.

its a minor issue cant add level to these . sorry about that.

then you should not assign it and say that before hand , what about the other one??? @sanjay-kv

@kairveeehh It has been communicated well before via documentation and via email, documentation with high changes only given level1. all links changes will be added gssoc


Update the docs regarding OnServerAddPlayer.

Thanks!

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


C++ implementation

This implementation is completely separate from the python implementation. It uses simpler representation than RLE, by just using a list of the indices of all ones (like .nonzero() in numpy).

holy hell this is fast, both in how quick you've made it, and in runtime, very nice.
unfortunately it still seems to slam into a memory wall at n=13, but much less so than the python version.
when would you be happy for full feedback and review on this ? don't want to spam you since this pr is still marked as draft.

As the memory seems to be a main issue, it would optimal to only hold the cubes in memory that are currently needed. The way I see it you don't need a cube after expanding it.

happy to merge this after some cleanup and commenting, would you rather I go through and point stuff out line by line in the review, or just make the changes and pull-request my changes into your repo.
this will not be replacing the python implementation but will be parallel to it.

@NobodyForNothing I'm really not sure where the line is for the memory footprint. At some point the problem will get big. We can try to be efficient, but for "removing cubes after expanding" to be efficient we have to consider other data structures, so removing single items will really decrease size (eg. deque?).
Perhaps a pool for all XYZ structs is more efficient, because for each iteration N there are exactly N x XYZ structs in a Cube.
@bertie2 I think pull-requests to my repo are more efficient than line by line discussion.

What's the current memory footprint e.g. at n=13? I have some servers available ;)
Some of the discussions elsewhere are thinking about how one could shard work into different processes, or distribute across machines. To pull that off we'd probably need to find a way of identifying the minimum set of all cubes from a set of sets of cubes produced on different machines. There's almost certainly a known algorithm for this from big data - maybe future work ;)

What's the current memory footprint e.g. at n=13? I have some servers available ;)

hard to say :) I only have 16G so I can't test everything:
N   SingleTh    4Threads    20 Threads
12  2.5GiB                  10.7GiB
13  10%=9GiB                10%=10GiB  but the base set is already 2.5GiB from N=12

Memory seems to grow linearly with progress...

For what it's worth I've run it single-threaded on my MacBook Pro with 32 GB memory, and N = 13 took ~12 hours (commit cf6e413).
N = 13 || generating new cubes from 18598427 base cubes.
  took 43899.52 s
  num cubes: 138462649

It' currently calculating N = 14 and consumes about ~30GB of memory, though it's rapidly increasing so I suspect it'll hit a wall here soon.
N = 14 || generating new cubes from 138462649 base cubes.
   2%   928 it/s, remaining: 145890s

so I updated the code to use synchronized access to a single set instead of multiple sets when more threads are used. So memory footprint should be more or less the same and not depend on number of threads.
Since all threads use the same resource (the set) they are blocking each other while accessing the set. This will be the limiting factor for speed when using many threads.

open pull request from me @ https://github.com/nsch0e/opencubes/pull/1
gets us pretty close to single threaded performance both in terms of memory usage and speed while running multi threaded.
@mikepound
n=13 is easily doable at 32GB of ram, and we seem to have the same growth pattern as the files, ~~7x each time.
interestingly in my testing the program works REALY WELL with a swap file (at least on my nvme ssd and at n=13) so if you have a server with 32-64GB of ram and a fast NVME SSD, you should be able to allocate a multi-hundred-GB swap file and push n very high. for only a fairly minor slowdown.
running up to n=15 numbers myself now with 32GB of ram and 256GB of swap, n=13 is taking less than 10 minutes.

Thanks to https://github.com/mikepound/opencubes/issues/5#issue-1802085753 for finding it, this paper really gives a lot of useful info: http://kevingong.com/Polyominoes/ParallelPoly.html.
For what it's worth, here are some of my conclusions & thoughts from it:
Do we really want to store all cubes of the record-breaking n, or is just calculating the number enough? If so, then we could try to get clever and differentiate the polycubes into non-intersecting sets - to save on RAM.
One such differentiating metric can be shape (rotated to some canon, e.g. increasing numbers, ensuring it's not mirrored) - but we don't necessarily have to use shape as a metric. And we can use several metrics. But if we can't predict in advance: e.g. for a given cube at level n=14 what metrics the derived cubes at n=16 would fall under, then using several metrics wouldn't be useful; we can just use one metric: modulo of some hashing algorithm (this can be any algorithm different from the one used for putting the elements into the set). And we can change the modulation number depending on how much RAM we have - to control the number of such groups - and thus, the sizes of each set. For the further example, let's say the modulation number k = 1000.
Also, we can only pre-compute cache until a certain level n (max. that our storage capacity allows), and then continue using this cache as a starting point of every run, not caching the subsequent layers. So their information will be lost after finishing the algorithm. Why not cache the next layers? - because their size in MB will be ridiculous (more than we can hold), while the numbers of polycubes will be calculated in parallel on multiple machines - so this size will never actually be help in one place at the same time. We can compute it, but not store it.

Example:
We want to calculate the number for n=17.
The maximum cache that we have capacity to hold is for n=14. This is our starting cache.
So the workflow can be as follows:
distribute the starting cache into several machines, e.g. each of the 5 machines receives a random 20% portion.
for each possible modulo remainder (in our case k=1000, see above, so, for each number from 0 to 999):
    on each machine:
        current_set = empty_set()
        for each subset of cubes of the local portion of starting cache (choose the appropriate subset size - to not overwhelm the RAM in the next step):
            calculate all canonical representations of polycubes with n=17 (this will take several steps, since the starting point is only n=14)
            filter these polycubes: apply some hashing algorithm and calculate the modulo remainder, throw away everything that doesn't match the modulo remainder of the current loop
            add the polycubes into current_set

    // after all the sets are created on all machines:
    merge all the sets from all the machines together
    count the number of elements in the resulting set
    save this integer somewhere (can also save the current modulo remainder with it - for logging)

add up all these integers

since the set contains all the cubes that fall under the metric (modulo remainder in this example), and nothing else - we safely process such sets independently from each other

if one machine finishes early - don't wait for others to finish - export the set to some external place, and go on to the next modulo remainder

If the time is too long:

increase the starting cache, e.g. from n=14 to n=15 (but need more storage). This is "depth vs width balance" idea from the paper
or use smaller modulo remainder (each set will have more elements - need more RAM),
or use more parallelism: not only can we process the same modulo remainder on different machines and at the end just merge all these sets together to count the number of its elements, we can also process different modulo remainders on different machines.

Maybe some further improvements can be made - with ideas both from the paper and people.
Or maybe some things from here are impractical or outdated..
At least, this is the edge of my understanding right now. Thank you for reading!

@VladimirFokow thanks for your thoughts. I can't say I understood them completely. But I can describe the current state of the C++ implementation:
(in the following shape means the bounding box of a polycube)

Each polycube has a rotation invariant representation:

its shape has always the constrain: shape[0] <= shape[1] <= shape[2]

when calculating N we first determine all possible shapes

each shape gets its own hash-set to filter duplicates

expanding one polycube(N) to N+1 can only result in a few shapes

because at most one dimension is increasing while also dimension order must be preserved (shape constraint)

there is a branch select which splits the problem into separate output shapes and then runs each of them after another.

this wastes many polycubes but doesn't need all the results in RAM (only one shape)

currently cachefiles contain all polycubes in flat order without distinction of shape

I think the way towards bigger N needs splitted cachefiles by shape as input
as temporary output from distributed systems the cachefiles should contain blocks by shape so they can be read selectively.

I just did my last 3 small changes and am also happy with this state. Further improvements will have to wait. I have ideas but I think we should provide this as a starting point for others.

This is looking great! For info I have access to some 50 core server with 1/2Tb ram - so if we need some testing doing.. ;)

@mikepound if growth continues as we expect then with 2TB of ram n=16 should be runnable with both this and tjol 's implementation, however n=17 wont.
the next step to get any higher will be to implement direct disk streaming, which I am working on in my own branch, but I want to get this and the rust implementation merged first as they stand so that others have something compatible to work from.

I wouldn’t mind finding out how much RAM my version actually uses for N>14 and if it’s limited by disk speed (writing manageable chunks of the result to disk one by one is all well and good by my method of deduplication – looping through the entire file once for every now block of data – is actually quite slow, who knew)

I have queued some  pull requests on @nsch0e cpp branch:

https://github.com/nsch0e/opencubes/pull/10
https://github.com/nsch0e/opencubes/pull/11

I would like to know are they still candidates to be pulled?
The optimizations had following impact:
./build/cubes -n 12 -t 2 245.05s user 1.88s system 180% cpu 2:16.76 total
vs.
./build/cubes -n 12 -t 2 297.87s user 2.04s system 180% cpu 2:46.07 total

@JATothrim if you raise your changes as pull requests into this repo I will take a look, I think I would merge 10 on the spot, but would want from feedback from @nsch0e before merging 11.

This is looking great! For info I have access to some 50 core server with 1/2Tb ram - so if we need some testing doing.. ;)

@mikepound, not sure where to put this so replying here, the current latest C++ implementation with the split cache files should be able to produce all poly-cubes up to n=16 with 2TB of ram in about 2 days, as long as it also has ~3TB of storage available for the results, table of estimates here:

N
Peak Memory Usage (GB)
Total Output Size (GB)
Growth Factor

10
x
0.01

11
x
0.08
8

12
x
0.638
7.975

13
6
5
7.836

14
27
40.7
8.14

15
216
325.6

16
1728
2604.8

17
13824
20838.4

18
110592
166707.2

19
884736
1333657.6

the data for n=15 + is the estimate, the data for n=10 to n=14 is measured, x indicates i couldn't properly measure the memory usage.
whether this is actually worth running is up to you, at above n=16 it is pretty much required we move to a enumeration technique rather than trying to actually store the cubes.
currently only the rust version can do storage-less enumeration, and https://github.com/datdenkikniet has run it up to n=16.
unfortunately using similar extrapolation, it would take about a year to beat n=18 currently, though possibly faster on your hardware:

N
Time Taken
Growth Factor
Estimated Time (days)

13
1.5

14
13
8.66666666666667

15
110
8.46153846153846

16
970
8.81818181818182

17
7760

5.38888888888889

18
62080

43.1111111111111

19
496640

344.888888888889

20
3973120

2759.11111111111

again need to decide if its worth starting now or we should wait for code improvements, for now the breakthroughs seem to have distinctly slowed down.

This is already an amazing improvement! :) I'd seriously consider running n=16, if only so we can say we did it! 3Tb is a lot, but also not totally out of the question. The RAM is currently an issue, we have 1/2 Tb on our largest server, we may need to wait until we get a few more optimisations, though 75% memory saving may not count as "a few optimisations". I can certainly enumerate n=17, but if we are confident the code is working, all we can do there is verify the correctness of the paper.
I'd argue that enumeration and generation are two separate and entirely worthwhile goals. I.e., that the fact that cubes have been enumerated up to 19 doesn't mean we shouldn't generate a file of cubes up to n=16 (or 17 if we get really lucky with optimisations). Then we stick it one a web app that randomly servers you one of the billions of shapes each time you visit it - next video!

The split cache-files should be some what compressible by just zlib. This would reduce the disk space needed.
I'm working on "cube swapper" system that moves the cube XYZ list representation data out from RAM and instead stores it in disk via memory mapped file. The data is read in only when needed.
A draw back is that this system is currently very slow as flushing the cubes out to disk periodically takes a long time.
As proof-of-concept stage it does work. Used ram for computing the polycubes is minuscule compared to current situation: N=13 run going at 52/66 shapes, the process is showing less than 300MiB RSS. :-)
I noticed @nsch0e has done some experiments compressing the cube representation and I don't know about his plans on it.

so I tried to describe the compression:
PCube Compression
This compression uses the fact that polycubes have to be connected so most of the time we only have to specify a direction for the next cube in the polycube. Only when a string of cubes ends we use a jump instruction to begin the next branch of cubes.
Encoding:
+----------------+-----------+-----------+---------------------+
| 8 bits         | 4 bits    | 4 bits    | ...2*length nibbles |
+----------------+-----------+-----------+---------------------+
| length in Byte | [dir|jmp] | [dir|jmp] | ...                 |
+----------------+-----------+-----------+---------------------+

Nibbles
dir
0b0xxx has its highest bit fixed at zero. Lower 3 bits are the index of the direction.
direction table:
table = [
  { 0,  0,  1},
  { 0,  0, -1},
  { 0,  1,  0},
  { 0, -1,  0},
  { 1,  0,  0},
  {-1,  0,  0}
]

jmp
0b1xxx has its highest bit fixed at one. Lower 3 bits are the reverse jump distance minus 1. So the jump 0b1000 references the second last encoded cube.
jmp instructions can be concatinated to represent bigger jumps. Each following jump instruction shifts previous jump distance by 3 bits.
example:
encoded jump:  1001 1010
jump distance:  001  010 = 0b1010

When a polycube is encoded by an odd number of nibble a jump instruction 0b1000 can be filled at the end, because it doesn't add a cube.

@nsch0e  Thank you for describling the compression scheme. :)


[RFC] Migrate backend to TypeScript

Strategies to migrate the backend to TypeScript.
** Below mention strategies are used by Airbnb.**
Strategy 1: Change file by file.
In this, we change files one by one from .js to .ts and add types/interfaces wherever needed. Here we enable allowJS option that allows and compiles both js and ts files error-free. But now the developer has to write code in both js and ts. we have to repeat until the full project is migrated
Strategy 2: All-in migration! (Airbnb used this)
In this strategy, they used code mods to convert the .js files to .ts in one go and We’ll need to add some any types and @ts-ignorecomments so the project compiles without errors.
Please refer to these links to learn more about the strategy :
https://medium.com/airbnb-engineering/ts-migrate-a-tool-for-migrating-to-typescript-at-scale-cd23bfeb5cc
https://www.youtube.com/watch?v=P-J9Eg7hJwE
Please comment if you know of any better strategies and improvements.

How about a third option, where you get the TS setup ready in the project but still allow js files to run as they were running earlier, now whatever next API is to be developed, that HAS to be written in TS?
Regarding the old API migration, we could literally assign 1 whole parent route to 1 developer and let that developer take care of the children routes and controllers, models, utils, tests etc related to those routes

How about a third option, where you get the TS setup ready in the project but still allow js files to run as they were running earlier, now whatever next API is to be developed, that HAS to be written in TS?
Regarding the old API migration, we could literally assign 1 whole parent route to 1 developer and let that developer take care of the children routes and controllers, models, utils, tests etc related to those routes

Yeah also seems like a good strategy.

I am not a fan of strategy-2.
Strategy-1 seems reasonable, but we need some guidelines, especially for new developers to tackle JS-TS migrations.

I am not a fan of strategy-2.
Strategy-1 seems reasonable, but we need some guidelines, especially for new developers to tackle JS-TS migrations.
If I change a file, do I also have to change all the dependent files to TS? And their dependent files?

No, not immediately, We can add the type of data used from .js dependent files in .d.ts then use types from here and later on create an issue ticket to change these files to TS as well.

Hey, I want to work on this.
Have worked with node ja and have some adequate experience with typescript.
Let me know where I can help.

Closing this issue. We have decided to move with the Strategy 1


GoogleCloudExtension.sln does not build with Visual Studio 2017

I tried opening GoogleCloudExtension.sln in Visual Studio 2017 and it does not build

Fixed with PR #1093


Intermediates.noindex/SEConsoleApp.build/Debug-iphonesimulator/SEConsoleApp-QA.build/Script-0AD1D0E620B9555700AB82AF.sh  /Users/owais/.bash_profile: line 9: -e: command not found /Users/owais/.bash_profile: line 10: /usr/local/opt/asdf/asdf.sh: No such file or directory Command PhaseScriptExecution failed with a nonzero exit code

Description

Steps to reproduce

Expected behavior

Environment

Swift compiler version info
Xcode version info
Deployment target:

any one can please on this issue?

This appears to be a user-related issue, specifically a misconfiguration on your local device.
Specifically, on line 10 of /Users/owais/.bash_profile, there is an attempt to access /usr/local/opt/asdf/asdf.sh, but /usr/local/opt/asdf/asdf.sh does not exist.

@CrazyFanFan how i solve this?


[FEAT] Add cbrt expression

Overview
This PR exposes the cbrt expression to daft.
Changes

Added cbrt to the .pyi and accompanying files inside of daft.
Added basic functionality of cbrts on Series, Float32Array, and Float64Array (all other numeric types get coerced into one of those two array types).
Added a CbrtEvaluator to daft-dsl.

Note
There's a lot of duplication. We'll handle this later.
Todo
Add tests.

Still need to add tests! Am currently adding them. Will be available in the next commit.


Angular 12 production build breaking styles

When building using ng build --prod, or the new ng build --configuration production, a style tag is addded to the head element that forces the body min-width to be 992px!important, and completely breaks the responsive layout.
In an attempt to identify the culprit, I removed styles from angular.json until the min-width wasn't added, and found that node_modules/angular-bootstrap-md/assets/scss/bootstrap/bootstrap.scss was the cause.
Expected behavior
The CSS should render the same as when built without the --prod flag.
Actual behavior
The following CSS is injected into the head, breaking the page. Note the body{min-width:992px!important;} at the end.
@charset "UTF-8";:root{--blue:#007bff;--indigo:#6610f2;--purple:#6f42c1;--pink:#e83e8c;--red:#dc3545;--orange:#fd7e14;--yellow:#ffc107;--green:#28a745;--teal:#20c997;--cyan:#17a2b8;--white:#fff;--gray:#6c757d;--gray-dark:#343a40;--primary:#007bff;--secondary:#6c757d;--success:#28a745;--info:#17a2b8;--warning:#ffc107;--danger:#dc3545;--light:#f8f9fa;--dark:#343a40;--breakpoint-xs:0;--breakpoint-sm:576px;--breakpoint-md:768px;--breakpoint-lg:992px;--breakpoint-xl:1200px;--font-family-sans-serif:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,"Helvetica Neue",Arial,"Noto Sans",sans-serif,"Apple Color Emoji","Segoe UI Emoji","Segoe UI Symbol","Noto Color Emoji";--font-family-monospace:SFMono-Regular,Menlo,Monaco,Consolas,"Liberation Mono","Courier New",monospace;}*,:after,:before{box-sizing:border-box;}html{font-family:sans-serif;line-height:1.15;-webkit-text-size-adjust:100%;-webkit-tap-highlight-color:rgba(0,0,0,0);}body{font-family:-apple-system,BlinkMacSystemFont,Segoe UI,Roboto,Helvetica Neue,Arial,Noto Sans,sans-serif,Apple Color Emoji,Segoe UI Emoji,Segoe UI Symbol,Noto Color Emoji;font-size:1rem;font-weight:400;line-height:1.5;color:#212529;text-align:left;background-color:#fff;}@page {size:a3;}body{min-width:992px!important;}

Your working environment and MDB version information

angular-bootstrap-md - 11.1.0
angular - 12.0.4

Resources (screenshots, code snippets etc.)

package.json
  "dependencies": {
    "@angular/animations": "~12.0.4",
    "@angular/cdk": "^11.2.13",
    "@angular/common": "~12.0.4",
    "@angular/compiler": "~12.0.4",
    "@angular/core": "~12.0.4",
    "@angular/fire": "^6.1.5",
    "@angular/forms": "~12.0.4",
    "@angular/localize": "^12.0.4",
    "@angular/platform-browser": "~12.0.4",
    "@angular/platform-browser-dynamic": "~12.0.4",
    "@angular/router": "~12.0.4",
    "@angular/service-worker": "~12.0.4",
    "@fortawesome/fontawesome-free": "^5.15.3",
    "@types/uuid": "^8.3.0",
    "@zxing/browser": "0.0.9",
    "@zxing/library": "^0.18.6",
    "@zxing/ngx-scanner": "^3.1.3",
    "angular-bootstrap-md": "^11.1.0",
    "angularx-qrcode": "^11.0.0",
    "animate.css": "^4.1.1",
    "eslint": "^7.28.0",
    "firebase": "^8.3.3",
    "hammerjs": "^2.0.8",
    "rxjs": "^7.1.0",
    "tslib": "^2.3.0",
    "uuid": "^8.3.2",
    "zone.js": "~0.11.4"
  },

Yes, you can use min-width: initial!important but this is not a real solution...
You can solve it in the following ways:

Add in your angular.json the propery extractCss: false in build options (build > configurations > your-envirionment > extractCss: false) but its deprecated in Angular 11+

OR

Remove the word node_modules under styles option (maybe scripts too) the your angular.json
node_modules/bootstrap/dist/css/bootstrap.min.css -> bootstrap/dist/css/bootstrap.min.css

Same result for me


telegraf: add exhibitor status metrics

High-level description
Exposes result of exhibitor status endpoint as a telegraf metric.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS-45353 Exhibitor instrumentation

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

@mesosphere-mergebot label Ready For Review


Revert "added: APISanitytest file ,generating test reports"

Reverts contentstack/contentstack-management-java#59
SRE was done hence reverted

Reverting this as the SRE was already done for the release


Added meltano/meltano and meltano/sdk

ℹ Repository information
The repository has:

[x] At least three issues with the good first issue label.
[x] At least 10 contributors.
[x] Detailed setup instructions for the project.
[x] CONTRIBUTING.md
[x] Actively maintained.

Links to issue pages:

https://github.com/meltano/meltano/issues
https://github.com/meltano/sdk/issues

Can I get a maintainer's eyes on this? 🙏


Add logic to handle the case where all the voices are in use

We need to have some logic to handle the case where all the voices are in use.
Either replace the least recent note, or the most quiet note.

Instead, we'll just create enough voices for all notes


Add Microsoft Mouse and Keyboard Center

Added Microsoft Mouse and Keyboard Center
Microsoft Reviewers: Open in CodeFlow

Install fails because on VM.  But no good error populated. I will need to see how we can address.


[MRG]: Add rdf serialization

still the old PR - do not merge yet.
Test notebook:
http://nbviewer.ipython.org/urls/dl.dropbox.com/s/n1876q0fmi1ny3h/RDFtestsuite.ipynb?dl=0

a few more things to finalize:

[ ] support bundles (via trig)
[ ] make the tests run on travis
[ ] check py3 support
[ ] settle an issue with Decimal representation
[ ] skip scruffy round trip tests for now - just ensure they can be read without error for the moment.

these tests fail round trip - need to figure out a way to skip.
FAIL: test_scruffy_end_2 (prov.tests.test_rdf.RoundTripRDFTests)
FAIL: test_scruffy_end_3 (prov.tests.test_rdf.RoundTripRDFTests)
FAIL: test_scruffy_end_4 (prov.tests.test_rdf.RoundTripRDFTests)
FAIL: test_scruffy_generation_2 (prov.tests.test_rdf.RoundTripRDFTests)
FAIL: test_scruffy_invalidation_2 (prov.tests.test_rdf.RoundTripRDFTests)
FAIL: test_scruffy_start_2 (prov.tests.test_rdf.RoundTripRDFTests)
FAIL: test_scruffy_start_3 (prov.tests.test_rdf.RoundTripRDFTests)
FAIL: test_scruffy_start_4 (prov.tests.test_rdf.RoundTripRDFTests)
FAIL: test_scruffy_usage_2 (prov.tests.test_rdf.RoundTripRDFTests)

@trungdong - some of the issues here are unfortunately due to rdflib interactions. but this is good for review. i still need to figure out the python 3 errors, again an interaction with rdflib! and how to suppress the failing scruff tests.

Coverage decreased (-1.3%) to 89.442% when pulling 1debd4b9424912a4e8f2955390e59058002bc61a on satra:enh/rdf-1.x into a556fa767086a2c470b00c7afd8e762f2081b1e0 on trungdong:master.

Coverage decreased (-1.5%) to 89.207% when pulling 1f8955ee04e331793019f053f11e1f2874f74436 on satra:enh/rdf-1.x into 5ddeadeb3887a5338c0a5370c1ead2a70318b633 on trungdong:master.

Coverage decreased (-0.4%) to 90.311% when pulling 5a29a1b60706430057b42b4969b1ac00235ab98f on satra:enh/rdf-1.x into 5ddeadeb3887a5338c0a5370c1ead2a70318b633 on trungdong:master.

Coverage decreased (-0.4%) to 90.311% when pulling 5a29a1b60706430057b42b4969b1ac00235ab98f on satra:enh/rdf-1.x into 5ddeadeb3887a5338c0a5370c1ead2a70318b633 on trungdong:master.

@trungdong - finally got some time to fix and this is ready for merge :)

closes #1

Excellent! Thank you very much @satra!!!
I'll try to get some time this week to clean up the current master branch and will make a new release with RDF support soon.


Add custom return type annotation for subscriptions

Currently it appears that when defining a subscription the return type annotation must be set to typing.AsyncGenerator[str, None], like so:
@strawberry.type
class Subscription:
    @strawberry.subscription
    async def time(self, info) -> typing.AsyncGenerator[str, None]:
        ...

Do we need to use typing.AsyncGenerator as the return type for subscription definitions? I'm not sure that I love that dev experience. Would it be bad practice to use raw types as the return type? Something like:
@strawberry.subscription
async def time(self, info) -> str:
    ...

Or maybe a we could use a custom return type for subscriptions:
@strawberry.subscription
async def time(self, info) -> strawberry.subscription_result(str):
    ...

@jaydenwindle yes, that's a very good point, I don't like that too, not sure what our best options are.
The reason why I was using typing.AsyncGenerator is because MyPy was complaining :D
I'd say I like your last suggestion, but maybe it should look like this:
@strawberry.subscription
async def time(self, info) -> strawberry.SubscriptionResult[str]:
    ...

so it looks more like a python typing type ;)
We should maybe also have a look at MyPy plugins, like the one for dataclasses: https://github.com/python/mypy/blob/master/mypy/plugins/dataclasses.py

I like that option too!
I'll have to look a little deeper into how MyPy does its magic, but I think if we define strawberry.SubscriptionResult as a custom type that's just a wrapper around typing.AsyncGenerator it should keep MyPy from complaining.

Do we still want this @patrick91? Personally I'm fine with AsyncGenerator[str, None] since I see it a lot in code using asyncio. But adding a type alias is still an option of course :)

I think an alias would be nice to have, at least until we get optional type params :D


Add option for java_verifier_tests to load plugins [phab:D1866]

Created by schroederc at 2017-08-14 16:59:13:

schroederc wrote at 2017-08-14 17:21:13:
Rebase

zarko accepted this revision at 2017-08-14 18:04:50

schroederc closed via commit 25888bbe26f6a7dbe15c40860fc0f683f725917d at 2017-08-14 18:07:23

shahms commented inline (context lost during migration) at 2017-08-14 18:11:04:
It seems like just adding a data dependency to the index_compilation target would be better than adding an entire separate target ...


chore(deps): bumps ginkgo and gomega to latest releases

Seeing this repeatedly
#!/bin/bash -eo pipefail
cd ~
pyenv global 3.5.2
curl -s https://packagecloud.io/install/repositories/datawireio/telepresence/script.deb.sh | sudo bash
sudo apt install --no-install-recommends telepresence
telepresence --version
Detected operating system as Ubuntu/xenial.
Checking for curl...
Detected curl...
Checking for gpg...
Detected gpg...
Running apt-get update... done.
Installing apt-transport-https... done.
Installing /etc/apt/sources.list.d/datawireio_telepresence.list...done.
Importing packagecloud gpg key... done.
Running apt-get update... done.

The repository is setup! You can now install packages.
E: Could not get lock /var/lib/dpkg/lock - open (11: Resource temporarily unavailable)
E: Unable to lock the administration directory (/var/lib/dpkg/), is another process using it?
Exited with code 100

With the change of the machine image I keep seeing
Warning: kubectl 1.14.0 may not work correctly with cluster version 1.11.0+d4cacc0 due to the version discrepancy. See https://kubernetes.io/docs/setup/version-skew-policy/ for more information.

With the change of the machine image I keep seeing
Warning: kubectl 1.14.0 may not work correctly with cluster version 1.11.0+d4cacc0 due to the version discrepancy. See https://kubernetes.io/docs/setup/version-skew-policy/ for more information.

Fix coming with new version of the ike orb - see #68


HHH-17068, HHH-18585 move layer-breaking enums

[Please describe here what your change is about]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

https://hibernate.atlassian.net/browse/HHH-17068
https://hibernate.atlassian.net/browse/HHH-18585

@sebersole got time to review this? Apologies for nagging.

Could you run spotless on this? ;)
Otherwise, looks good to me


SWC-47: Implemented multiple test cases, as well fixed certain auth bugs

This commit relates to the comment in SWC-57 , most of the test cases are implemented.

All important test cases relate to "user" are implemented except for user modification (refer to SWC-63).

Modified as requested per comments, please leave feedback and accept merge request.


process.env.BASE_URL does not return a URL

Version
4.5.4
Reproduction link
https://github.com/unikitty37/vue-cli-base-url-demo
Environment info

Environment Info:

  System:
    OS: macOS Mojave 10.14.6
    CPU: (24) x64 Intel(R) Xeon(R) CPU           X5690  @ 3.47GHz
  Binaries:
    Node: 14.4.0 - /usr/local/bin/node
    Yarn: 1.22.4 - /usr/local/bin/yarn
    npm: 6.14.7 - /usr/local/bin/npm
  Browsers:
    Chrome: Not Found
    Edge: Not Found
    Firefox: Not Found
    Safari: 13.1.2
  npmPackages:
    @clampy-js/vue-clampy: ^1.0.6 => 1.0.6
    @testing-library/vue: ^5.0.4 => 5.0.4
    @types/testing-library__vue:  2.0.3
    @vue/babel-helper-vue-jsx-merge-props:  1.0.0
    @vue/babel-plugin-transform-vue-jsx:  1.1.2
    @vue/babel-preset-app:  4.4.6
    @vue/babel-preset-jsx:  1.1.2
    @vue/babel-sugar-functional-vue:  1.1.2
    @vue/babel-sugar-inject-h:  1.1.2
    @vue/babel-sugar-v-model:  1.1.2
    @vue/babel-sugar-v-on:  1.1.2
    @vue/cli-overlay:  4.4.6
    @vue/cli-plugin-babel: ~4.4.0 => 4.4.6
    @vue/cli-plugin-e2e-nightwatch: ~4.4.0 => 4.4.6
    @vue/cli-plugin-eslint: ~4.4.0 => 4.4.6
    @vue/cli-plugin-router: ~4.4.0 => 4.4.6
    @vue/cli-plugin-unit-jest: ~4.4.0 => 4.4.6
    @vue/cli-plugin-vuex: ~4.4.0 => 4.4.6
    @vue/cli-service: ~4.4.0 => 4.4.6
    @vue/cli-shared-utils:  4.4.6
    @vue/component-compiler-utils:  3.2.0
    @vue/preload-webpack-plugin:  1.1.2
    @vue/test-utils: ^1.0.3 => 1.0.3
    @vue/web-component-wrapper:  1.2.0
    eslint-plugin-vue: ^6.2.2 => 6.2.2
    jest-serializer-vue:  2.0.2
    vue: ^2.6.11 => 2.6.11
    vue-eslint-parser:  7.1.0
    vue-hot-reload-api:  2.3.4
    vue-jest:  3.0.6
    vue-loader:  15.9.3
    vue-router: ^3.2.0 => 3.3.4 (3.4.3)
    vue-style-loader:  4.1.2
    vue-template-compiler: ^2.6.11 => 2.6.11
    vue-template-es2015-compiler:  1.9.1
    vuex: ^3.4.0 => 3.5.1
  npmGlobalPackages:
    @vue/cli: 4.4.6

Steps to reproduce

Run yarn serve
Navigate to the local or network URL output by yarn serve (for this example, I'll assume it returns http://localhost:8080/ for the local URL).

What is expected?
The page should read Base URL is http://localhost:8080/.
What is actually happening?
The page reads Base URL is /.

This is not a URL, it's a path. This should really be returned under process.env.BASE_PATH — but I'd argue that process.env.BASE_URL should exist as well, and should return the full http://localhost:8080/ in this case.
I need to create a link that will allow the user to add this site's ICS feed to their Google Calendar easily, but this does not return the base URL:
googleICSFeed () {
  return `https://calendar.google.com/calendar/render?cid=${ process.env.BASE_URL }/calendar.ics`
},

https://cli.vuejs.org/guide/mode-and-env.html#using-env-variables-in-client-side-code does state that BASE_URL returns a path rather than a URL, but it would be much better to document how to actually get this URL, even if the variable is not renamed to reflect what it actually contains…

BASE_URL - this corresponds to the publicPath option in vue.config.js and is the base path your app is deployed at. reference.

You can specify http://localhost:8080/ using publicPath option in vue.config.js.
// vue.config.js

module.exports = {
  publicPath: process.env.NODE_ENV === 'production'
    ? 'http://foobar.com/'
    : 'http://localhost:8080/'
}

Note: our app won't work if BASE_URL is deployed at different origin. AFAIK, we always provide process.env.BASE_URL, but by default it's a path not url.

Thanks — but that's how to set it, not to read it :)

read it ? Just like:
googleICSFeed () {
  return `https://calendar.google.com/calendar/render?cid=${ process.env.BASE_URL }/calendar.ics`
},

Is there any doubt？

I just encountered this same issue and I'm very much with @unikitty37. I introduced some build bugs into a new Docker configuration PR this week that didn't make sense. It was all because of this very poorly named variable.
I wonder how many other Developers this has confused over the years? I'd be willing to bet it's a pretty high number.
URL was standardized by Tim Berners-Lee in 1994 via RFC 1738. It stands for Uniform Resource Locator. It is used to locate a network resource in a uniform way. The different parts of a URL are:
{protocol}://[{user}:{password}@]{host}[:port][{/path}]
A very practical usage of a URL is typing one into a Web Browser to go to a web page.

If you type "https://github.com/vuejs/vue-cli" into your browser you will visit that page, because this is a URL.
If you type "/vuejs/vue-cli" into your browser you will NOT visit that page, because this is the path value of a URL.

Note: Because someone might ask. The only reason this might "seem" to work in the context of developing a single web application is because, in your code, you can link an "href" to a path and your Web Browser's rendering engine will interpret it as an absolute path (if it begins with /) or a relative path (if it does not begin with /) and appropriately inject that path value into the current browser windows URL to form a new URL that can be followed. It's still a path, and not a URL.
The environment variable BASE_URL should be renamed to BASE_PATH. A "new" environment variable named BASE_URL should be re-added. Backward compatible code to figure out which is which (to take the burden off of the current users) should be added as well, with a deprecation warning because this shouldn't be left this way long-term. After a couple of major releases the backward compatible code should be removed. My 2¢.


docs: Update message.md

Minor language corrections
@alequetzalli

Description

...
...
...

Related issue(s)

@leo-epshtein please look into bot message https://github.com/asyncapi/website/pull/2419#issuecomment-1849030765
@alequetzalli your review is needed

The proposed changes do not adhere to standard English grammar, and therefore, I cannot approve this pull request. Thank you for your effort in trying to submit it tho :)
Closing the request.


docs: Extend flow health ADR

Description
Changes proposed in this pull request (what was done and why):

Add an explanation of why we have a special reason for LogPipelines only
Remove the note since we are reflecting BufferFillingUp in a negative condition

Changes refer to particular issues, PRs or documents:

Traceability

[ ] The PR is linked to a GitHub issue.
[x] The PR has a milestone set.
[x] The PR has a respective area and kind label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] If the change is user-facing, the documentation has been adjusted.
[ ] The feature is unit-tested.
[ ] The feature is e2e-tested.

/hold

/unhold


cpuTemperature() always show -1 on raspberry pi

Call cpuTemperature() and it always returns -1 for main and max temperature on raspberry pi.
After checking cpu.js, I found that command "sensors" do NOT trigger error in my raspberry pi and make the result always -1.

Modified codes of cpu.js and fixed the bug. Please check it out. Thanks.

@WayneChang65 Hi, thank you for reporting. I solved it a little bit different now to avoid duplicating code ... Version 4.26.7 just released.


http.Service parseAddress Is the path of the beacon node intercepted, causing some nodes to be inaccessible?

https://github.com/attestantio/go-eth2-client/blob/951cf2c15b346dfa1f9bf1ecf3635e6c8ddf089a/http/service.go#L397-L426
        if baseAddress.Path != "" {
		// Mask the path.
		baseAddress.Path = "xxxxx"
	}

The above code intercepts the path configured beaconAddr, which makes some normal nodes inaccessible.
For example, the following nodes:
https://aaa.com/ethereum2/{token}
It is normal for paths to carry tokens, but such modifications invalidate them.

Hi @duktig666
I took a look at the code you mentioned and I think there is a slight misunderstanding in it's purpose. The function returns two variables; base and baseAddress. base is largely unmodified and is used to connect to the address supplied. As you point out, baseAddress is modified to mask any sensitive information, but should only be used for logs and monitoring purposes.
It might help if you can explain the issue you are seeing related to this? E.g. which URL are you trying to connect to and what is the error message you are getting? Feel free to share some logs (with sensitive information redacted of course).

gm


Custom html of menu item not reflecting in my project.

I have added everything as mentioned in doc for  . i want to add custom html for menu list.
I have used.
<c-item>   <ng-template let-item="item">     <label style="color: #fff;min-width: 150px;"></label>     <img [src]="item.image" style="width: 30px; border: 1px solid #efefef;margin-right: 20px;" />     <label>Capital - </label>   </ng-template> </c-item>
But it is not reflecting in my project.
Please help me to get out of it.
Should i include default.theme.css file then where should i include?

@pshobha97 Can you please provide more information like the version of angular you are using ?

Working as expected. Hence closing.

@pshobha97 Can you please provide more information like the version of angular you are using ?

I am using angular version 4.
The issue is not solved yet.
Please help.


Changed way the primary keys are handled. It is now possible to have a composite primary key

Changed way the primary keys are handled. It is now possible to have a composite primary key

Thanks a lot!

Thanks a lot!


Trouble Creating Pyinstaller EXE - ModuleNotFoundError: No module named 'kivymd.icon_definitions'

Software Versions:
Python: 3.10.4
OS: MacOS (Catalina 10.15.4)
Kivy: 2.2.0
Kivy installation method: Pycharm package installer (PyPi: https://pypi.python.org/simple), also manually installing via cmd, really all possible installation methods for both Kivy and KivyMD
Describe the bug
I'm having trouble making this app into an executable after many hours trying everything. Here's the log:
   File "kivy/lang/parser.py", line 553, in execute_directives
   File "<frozen importlib._bootstrap>", line 1129, in __import__
   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
   File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
   File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
 ModuleNotFoundError: No module named 'kivymd.icon_definitions'

Here's what I tried:
Installing source files from docs
downgrading pyinstaller to 5.6.2
installing master kivymd 1.2.0.dev0
installing kivy 2.1.0. and 2.2.0
manipulating spec files per docs to include imports and hookpaths etc
tried this (editing init.py and hook-kivymd.py): https://stackoverflow.com/questions/69419682/no-module-named-kivymd-effects-when-using-pyinstaller
This is what I changed, maybe this is incorrect? I noticed it's not a folder but a file so the syntax is a little different
    (
        kivymd.iconDefs_path,
        str(kivymd.iconDefs_path),
    ),

I've noticed I've never gotten a kivy_deps folder. Is this for earlier versions of Kivy? I have a /kivy/deps folder instead, and it's empty except the init.py file which is empty.
Could someone maybe walk me through how to properly install everything to make an executable? There's just so much to modify and so many variations with outdated docs, it's very confusing.

fixed by modifying the .spec file, thanks to this post: https://stackoverflow.com/questions/49806573/cant-pyinstall-a-kivy-python-program
added:
from kivymd.icon_definitions import md_icons
in the .spec file which I'm almost positive was the thing that did the trick.
I did also do other things, so if this wasn't the sole fix, I also added what is in that stackoverflow post.
Also ended with these package versions:
Kivymd: 1.1.1
Kivy: 2.2.0
PyInstaller: 5.12.0
which is nearly what I wanted + started with. For some reason the colors are desaturated slightly now though which I think happened after I installed the Kivymd 1.2.0.dev0 version, and it never went back. Not a big deal tho :)
I'll leave open if this isn't the intended working of KivyMD

I was wrong on
from kivymd.icon_definitions import md_icons
being the fix.
I just needed my .spec correct, being the tocs. Here's my minimal .spec file that works after adding the 'tocs' (still don't know what they are):
# -*- mode: python ; coding: utf-8 -*-

import os
from os.path import join
from kivy import kivy_data_dir

block_cipher = None

# assets
kivy_assets_toc = Tree(kivy_data_dir, prefix=join('kivy_install', 'data'))
tocs = [kivy_assets_toc]

a = Analysis(
    ['main.py'],
    pathex=[os.getcwd()],
    binaries=[],
    datas=[],
    hiddenimports=['plyer.platforms.macosx.filechooser'],
    hookspath=[],
    hooksconfig={},
    runtime_hooks=[],
    excludes=[],
    win_no_prefer_redirects=False,
    win_private_assemblies=False,
    cipher=block_cipher,
    noarchive=False,
)
pyz = PYZ(a.pure, a.zipped_data, cipher=block_cipher)

exe = EXE(
    pyz,
    a.scripts,
    a.binaries,
    a.zipfiles,
    a.datas,
    *tocs,
    name='KivyMD_App',
    debug=False,
    bootloader_ignore_signals=False,
    strip=False,
    upx=True,
    upx_exclude=[],
    runtime_tmpdir=None,
    console=True,
    disable_windowed_traceback=False,
    argv_emulation=False,
    target_arch=None,
    codesign_identity=None,
    entitlements_file=None,
)
app = BUNDLE(
    exe,
    name='KivyMD_App.app',
    icon=None,
    bundle_identifier=None,
)

I have the same error:
kivymd.icon_definitions module not found

I have the same error

у меня та же ошибка

True


Unprotected None objects

svviz-run 11.log
$ svviz --export svviz-run11.svg --format svg --verbose 4 demo
Namespace(_parser=ArgumentParser(prog='svviz',
usage='%(prog)s [options] [demo] [ref breakpoint...] [ref vcf]',
description='svviz version 1.5.1', version=None, formatter_class=<class 'argparse.RawTextHelpFormatter'>,
conflict_handler='error', add_help=True), aln_quality=None, aln_score_delta=2, annotations=None, bam=None,
breakpoints=[], context=0, converter=None, dotplots=False, export='svviz-run11.svg', fasta=None, flanks=False,
format='svg', include_supplementary=False, lenient=False, max_deletion_size=None, max_multimapping_similarity=0.95,
max_reads=None, max_size=None, min_mapq=0, no_web=True, open_exported=False, pacbio=False, pair_min_mapq=0, port=None,
processes=None, ref='demo', sample_reads=None, save_reads=None, save_state=None, skip_cigar=False, summary=None,
thicker_lines=False, type=None, verbose=4)
* Sampling reads to calculate Insert Size Distributions *
Using align distance: None
* Loading variant *
* Running for variant 1/1 None *

Traceback (most recent call last):
File "/usr/local/Anaconda/envs_app/svviz/1.5.1/bin/svviz", line 11,
in  sys.exit(main())

File "/usr/local/Anaconda/envs_app/svviz/1.5.0/lib/python2.7/site-packages/svviz/app.py", line 354,
in main
run(sys.argv)

File "/usr/local/Anaconda/envs_app/svviz/1.5.0/lib/python2.7/site-packages/svviz/app.py", line 303,
in run
debug.printDebugInfo(dataHub)

File "/usr/local/Anaconda/envs_app/svviz/1.5.0/lib/python2.7/site-packages/svviz/debug.py", line 12,
in printDebugInfo
for part in dataHub.variant.chromParts(allele):

AttributeError: 'NoneType' object has no attribute 'chromParts'

Explanation
1. The first line shows the command being run

--export:     I need to avoid running svviz all the way up to starting the data viewer
because it doesn't work (must need an ssh tunnel) and I don't want to
present that as a good approach to our cluster users

--format svg: seems most straight-forward format to use

--verbose 4:  The arguments on the command line effect the behavior of svviz

With 1 argument , "demo",
svviz finishes computing and passes to the data viewer which hangs up the
parent shell of svviz.

With 7 arguments , "--export svviz-run11.svg --format svg --verbose 4 demo"
svviz is diverted into the "debug" module and the error is easier to interpret.
The result is the log file "svviz-run11.log" shown above.

2.   I believe that what is going on is this
  *  Because this is a demo session, some internal setup values are initialized to "None".

  *  Some "None" values, in turn, may break code (typically a 'for' loop) when the
    code assumes that some variable has some attribute that it doesn't actually
    have  because the value of the variable is  "None".

  *  I can see some places in the source code that protect against broken code
    caused by a "None" object, by placing the vulnerable code under an 'if' statement
    like:

     if variable is not None:
         USE(variable.attribute)

   *  and I see other places in the source code in which
     there is no 'if' statement protecting code which accesses an attribute
     of a variable which could have the "None" object as its value.
     For example, the code reference in the Traceback from "svviz-run11.log" above.

3. I don't understand how my proposed explanation can be correct
Am I the only svviz user in the world who has run the svviz demo programs as released?

Any insight into this situation would be much appreciated.
Sanford M. Orlow, PhD
Computer Engineer, HPC@NIH
National Institutes of Health, CIT
12 South Dr. 12B/2N207, Bethesda, MD 20892-5680
EMAIL<EMAIL_ADDRESS>

Hrm, I wasn’t intending that “svviz demo” be used with any additional arguments. Sorry there wasn't a better error message — if you check out the latest version from source, it will spit out a more helpful notice.
If you’d like to run the demo but change the arguments, you can do “svviz demo”, type “y” and then hit return to instruct it to download the demo data, and then it will spit out the example command line that it’ll try to run (at this point you can hit ctrl-c to stop it). For example, for the first demo, it’d be:
svviz --type inv -b svviz-examples/example1/son.sorted.bam -b svviz-examples/example1/father.sorted.bam -b svviz-examples/example1/mother.sorted.bam svviz-examples/example1/chr4_short.fa chr4 88847163 88858699
You can copy this command and then edit it. For example, you’d probably want to run this:
svviz --type inv -b svviz-examples/example1/son.sorted.bam -b svviz-examples/example1/father.sorted.bam -b svviz-examples/example1/mother.sorted.bam svviz-examples/example1/chr4_short.fa chr4 88847163 88858699 --export svviz-run11.svg --format svg --verbose 4
Does that make sense?
On Aug 23, 2016, at 7:06 AM, sandoro <notifications<EMAIL_ADDRESS>wrote:
svviz-run 11.log
$ svviz --export svviz-run11.svg --format svg --verbose 4 demo
Namespace(_parser=ArgumentParser(prog='svviz',
usage='%(prog)s [options] [demo] [ref breakpoint...] [ref vcf]',
description='svviz version 1.5.1', version=None, formatter_class=,
conflict_handler='error', add_help=True), aln_quality=None, aln_score_delta=2, annotations=None, bam=None,
breakpoints=[], context=0, converter=None, dotplots=False, export='svviz-run11.svg', fasta=None, flanks=False,
format='svg', include_supplementary=False, lenient=False, max_deletion_size=None, max_multimapping_similarity=0.95,
max_reads=None, max_size=None, min_mapq=0, no_web=True, open_exported=False, pacbio=False, pair_min_mapq=0, port=None,
processes=None, ref='demo', sample_reads=None, save_reads=None, save_state=None, skip_cigar=False, summary=None,
thicker_lines=False, type=None, verbose=4)

Sampling reads to calculate Insert Size Distributions *
Using align distance: None

Loading variant *

Running for variant 1/1 None *

Traceback (most recent call last):
File "/usr/local/Anaconda/envs_app/svviz/1.5.1/bin/svviz", line 11,
in sys.exit(main())
File "/usr/local/Anaconda/envs_app/svviz/1.5.0/lib/python2.7/site-packages/svviz/app.py", line 354,
in main
run(sys.argv)
File "/usr/local/Anaconda/envs_app/svviz/1.5.0/lib/python2.7/site-packages/svviz/app.py", line 303,
in run
debug.printDebugInfo(dataHub)
File "/usr/local/Anaconda/envs_app/svviz/1.5.0/lib/python2.7/site-packages/svviz/debug.py", line 12,
in printDebugInfo
for part in dataHub.variant.chromParts(allele):
AttributeError: 'NoneType' object has no attribute 'chromParts'

Explanation

The first line shows the command being run

--export: I need to avoid running svviz all the way up to starting the data viewer
because it doesn't work (must need an ssh tunnel) and I don't want to
present that as a good approach to our cluster users

--format svg: seems most straight-forward format to use

--verbose 4: The arguments on the command line effect the behavior of svviz

 *   With 1 argument , "demo",

svviz finishes computing and passes to the data viewer which hangs up the
parent shell of svviz.
 *   With 7 arguments , "--export svviz-run11.svg --format svg --verbose 4 demo"

svviz is diverted into the "debug" module and the error is easier to interpret.
The result is the log file "svviz-run11.log" shown above.

I believe that what is going on is this

Because this is a demo session, some internal setup values are initialized to "None".

Some "None" values, in turn, may break code (typically a 'for' loop) when the
code assumes that some variable has some attribute that it doesn't actually
have  because the value of the variable is  "None".

I can see some places in the source code that protect against broken code
caused by a "None" object, by placing the vulnerable code under an 'if' statement
like:
if variable is not None:
USE(variable.attribute)

and I see other places in the source code in which
there is no 'if' statement protecting code which accesses an attribute
of a variable which could have the "None" object as its value.
For example, the code reference in the Traceback from "svviz-run11.log" above.

I don't understand how my proposed explanation can be correct

Am I the only svviz user in the world who has run the svviz demo programs as released?

Any insight into this situation would be much appreciated.
Sanford M. Orlow, PhD
Computer Engineer, HPC@NIH
National Institutes of Health, CIT
12 South Dr. 12B/2N207, Bethesda, MD 20892-5680
EMAIL sandor<EMAIL_ADDRESS>—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/svviz/svviz/issues/44, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAJkLxyI-vRiDB_rROAGlhWEoYqIj0--ks5qiv5mgaJpZM4Jq9up.

closed by 1f3189128ad592609867f5bc533a2c2582b42d91

Noah,
I apologize for not getting back to you about issue #44 for svviz.
Your suggestion about how to run "svviz demo" with additional
svviz command-line options worked perfectly for me and I was able
to complete my testing.
Thanks much for your timely and helpful replies to my questions.
--sandy
HPC@NIH
Sanford M. Orlow, PhD
Computer Engineer, HPC@NIH
National Institutes of Health, CIT
12 South Dr. 12B/2N207, Bethesda, MD 20892-5680
PHONE<PHONE_NUMBER> EMAIL<EMAIL_ADDRESS>Today (10/17/16) at 01:18 -0400, Noah Spies wrote:

closed by 1f3189128ad592609867f5bc533a2c2582b42d91

Thanks for the update, and glad it worked out!
On Oct 17, 2016, at 7:56 AM, sandoro <notifications<EMAIL_ADDRESS>wrote:
Noah,
I apologize for not getting back to you about issue #44 for svviz.
Your suggestion about how to run "svviz demo" with additional
svviz command-line options worked perfectly for me and I was able
to complete my testing.
Thanks much for your timely and helpful replies to my questions.
--sandy
HPC@NIH
Sanford M. Orlow, PhD
Computer Engineer, HPC@NIH
National Institutes of Health, CIT
12 South Dr. 12B/2N207, Bethesda, MD 20892-5680
PHONE<PHONE_NUMBER> EMAIL sandor<EMAIL_ADDRESS>Today (10/17/16) at 01:18 -0400, Noah Spies wrote:

closed by 1f3189128ad592609867f5bc533a2c2582b42d91

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/svviz/svviz/issues/44#issuecomment-254231806, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAJkLyTA2SWG4bjSVM73YhW1BDLet4Ylks5q04yqgaJpZM4Jq9up.


is IS-BERT initialized with a pre-trained BERT's weight? and then undergoes self-supervised training?

Thank you for the brilliant work and sharing it on the web.
Seems all the code are well organized conveniently for users.
Reading thru the docs about training and the paper did not resolve the question of mine.
Is the initialization of the BERT module in IS-BERT is bert-base-uncased or
did you wanted to just specify the architectural aspect but not the pre-trained weight?

Oh, I've just found the code loads pretrained-weights from Huggingface by AutoModel.  IS-BERT/sentence_transformers/models/Transformer.py  Thank you =]


Final entity parenting incorrect

The final entity in the hierarchy seems to be incorrectly parented when a scene is loaded.

Fixed in commit cebc9eb6dd02ef54611944669c6504ddba254a84
The depth of child nodes was not updated in the tree when a node's parent changed.


Check the email text for invitation to book a book. It must be clear that that mail isn't a reservation, but it is only an invitation. We can discuss the mail text togheter

Seems like the reservation button doesn't work properly in all email clients


Received a 500 error when saving a large file to fedora

I was sending a 7GB file to fedora and received the error below.  I was not able to repeat the failure unfortunately...
 #<Ldp::HttpError: STATUS: 500 javax.jcr.ItemExistsException: A node definition that allows same name siblings could not be found for the node "{}content" in workspace "default"
        at org.modeshape.jcr.JcrSession$JcrPreSave.processAfterLocking(JcrSession.java:2422)
        at org.modeshape.jcr.cache.document.WritableSessionCache.runPreSaveAfterLocking(WritableSessionCache.java:530)
        at org.modeshape.jcr.cache.document.WritableSessionCache.save(WritableSessionCache.java:591)
        at org.modeshape.jcr.JcrSession.save(JcrSession.java:1165)
        at org.fcrepo.http.api.FedoraLdp.createOrReplaceObjectRdf(FedoraLdp.java:295)
        at sun.reflect.GeneratedMethodAccessor40.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.glassfish.jersey.server.model.internal.ResourceMethodInvocationHandlerFactory$1.invoke(ResourceMethodInvocationHandlerFactory.java:81)
        at org.glassfish.jersey.server.model.internal.AbstractJavaResourceMeth...>

It looks like you tried to make the identical request twice, hence the "ItemExistsException". Which HTTP method did you use? POST? PUT?

@awoods should that be a 500 error?

Agreed, should probably be a 409 Conflict. But when I try to reproduce the issue I do get a 409. Can you provide any more detail on what actions were performed? If not, I will go ahead and close this ticket as: "Can not reproduce"

Closing this as "Cannot reproduce"


RefSeq annotations for Pdom, Pcan, Dqua

Current coverage is 100.00%

Merging #61 into master will not affect coverage as of 7a53ba7

@@            master     #61   diff @@
======================================
  Files           17      17
  Stmts         1940    1936     -4
  Branches       193     192     -1
  Methods          0       0
======================================
- Hit           1940    1936     -4
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 7a53ba7

Powered by Codecov. Updated on successful CI builds.


remove logging of install-config.yaml

The Azure provisioning PR added logging of the install-config.yaml, which was supposed to have been removed before merging.

Ugh sorry about that.
/lgtm


UI : Page -> Flats : list unnecessary non readable fields and important fields and links between users are missing

UI of Flats page list uuids which is non readable.
Important fields like belonging or owners are missing for flats.
Links between flat and apartments is missing
Data is not comprehensible
Table relationships with users , owners and apartments is not maintained

Fixed the code to list relevant fields as per updates in the backend. Now the flow is also continuos with filter fields and between pages


Add some love to breakpoints

Thanks! Ok, we're ready to pull this in. As long as it's backwards compatible. Can you rename the bp-em-calc to just em-calc and we'll pull it in?

I do not understand. Currently em-calc is a deprecated function and the code in _functions.scss reads like this:
    // OLD EM CALC
    // Deprecated: We'll drop support for this in 5.1.0, use rem-calc()
    @function emCalc($values) {
      @return rem-calc($values);
    }

    // OLD EM CALC
    // Deprecated: We'll drop support for this in 5.1.0, use rem-calc()
    @function em-calc($values) {
      @return rem-calc($values);
    }

If I use the current em-calc for breakpoints I re-introduce REMs, but we did not want REMs for breakpoints. So do you want me to replace the current em-calc function? In case YES, do you want

the code for the old em-calc function? In this case, can you point me to the old code before it was deprecated?
the code from bp-em-calc?
What about the statement "// Deprecated: We'll drop support for this in 5.1.0, use rem-calc()"?

Thanks.

You can keep your code and just rename the function to em-calc. We'll remove the deprecation warning. Cool?

Can't detect any errors. Started from scratch. Result is in PR 6502.


The future of this package

Since the main contributor has lost access to the platform (Discord) that this package is built around (Source: https://github.com/discordjs/discord.js/issues/3440) I would like to know if this package will be further maintained by @appellation or if this package will remain without the main contributor.
Thanks

i agree to this statement, i will still maintain this library and do not intend to drop it


Support for endpoints from dependencies

Please add support for endpoints that are coming from dependencies (from some external .jar file).

Committed to develop, will be released with 0.2.x


Not usable on HiDPI screens (4k monitors) using In-Game Browser

Although the interface looks really nice (great work on that!), it is unfortunately not usable on 4k screens using the in-game browser. The in-game browser has, as far as I could find, no way to zoom, so everything is way too small to be usable on my screen (3840x2160). A zoom function for the map and the entire interface (mainly font-size) would solve this.

I have a 4k screen as well. The whole project was developed and tested in a
4k screen. It is small thats right but i havent seen any problems. The IGB
has no options for zooming. There are no plans for now to increase the font
size. IgB support is deprecated anyways and Ccp does not update the
browser.
Maarten van Gompel<EMAIL_ADDRESS>schrieb am Fr., 19. Feb. 2016
18:15:

Although the interface looks really nice, it is unfortunately not usable
on 4k screens using the in-game browser. The in-game browser has, as far as
I could find, no way to zoom, so everything is way too small to be usable
on my screen (3840x2160). A zoom function for the map and the entire
interface (mainly font-size) would solve this.
—
Reply to this email directly or view it on GitHub
https://github.com/exodus4d/pathfinder/issues/101.


[Questions] Does it support to get profile of kube-apiserver itself ?

like to get profile of kube-apiserver pod.

what I got is timeout ..
Verifying target pod ... ✔
Launching profiler ... ❌
timed out waiting for the condition

Hi @foreversunyao This should work.
I currently have access only to EKS cluster so I can't test it on apiserver, but I did manage to profile other Kubernetes components such as coredns for example:
➜ kubectl flame coredns-86d5cbb4bd-cr9z8 -n kube-system -t 30s --lang go
Verifying target pod ... ✔
Launching profiler ... ✔
Profiling ... ✔
FlameGraph saved to: flamegraph.svg 🔥

It looks like launching the profiler failed, please share kubectl describe pod <flame-pod> so I could investigate further.
Thanks

Seems like a similar failure, I posted flame-pod description in https://github.com/VerizonMedia/kubectl-flame/issues/19 @edeNFed


Support namespace

The current generator did not support namespace models. The patch fixes following issues

The migration file not named correctly if namespace is used in models.

 rails g devise_token_auth:install Sprint::V1::Auth::User auth
create  config/initializers/devise_token_auth.rb
**create  db/migrate/devise_token_auth_create_sprint/v1/auth/20180322050951_users.rb**
create  app/models/sprint/v1/auth/user.rb
insert  app/controllers/application_controller.rb
gsub  config/routes.rb

Class name and table name in migration file is incorrect

class DeviseTokenAuthCreateSprint::V1::Auth::Users < ActiveRecord::Migration[5.1]
  def change
    create_table(:sprint/v1/auth/users) do |t|
      ## Required
      t.string :provider, :null => false, :default => "email"
      t.string :uid, :null => false, :default => ""

snipped
    add_index :sprint/v1/auth/users, :email,                unique: true
    add_index :sprint/v1/auth/users, [:uid, :provider],     unique: true
    add_index :sprint/v1/auth/users, :reset_password_token, unique: true
    add_index :sprint/v1/auth/users, :confirmation_token,   unique: true
    # add_index :sprint/v1/auth/users, :unlock_token,       unique: true

The model file generated needs a fix

    class Sprint::v1::auth::user < ActiveRecord::Base
      # Include default devise modules. Others available are:
      # :confirmable, :lockable, :timeoutable and :omniauthable
      devise :database_authenticatable, :registerable,
             :recoverable, :rememberable, :trackable, :validatable
      include DeviseTokenAuth::Concerns::User
    end

@kaevee ok, but can you add a test for this? Thanks!

Sure. I will. Thanks!

To keep things simple, I have created a separate test for user models with namespace.

Very awesome @kaevee ! Going to just re-run the tests to be sure

Thanks. Lack of namespace support has been bugging me since late 2015. I wonder why I did not take this up all these days.


global axios.interceptors is vulnerable

Hi there! I'm a big fan of what the Digital Service is doing, and was browsing through the code out of interest.
In https://github.com/digitalservicebund/useid-eservice-sdk/blob/7e8906df61e9499720889e0045b34162908b9e27/src/UseIdAPI.ts#L11, and I think in some other places as well, you're using axios.interceptors.request.use from the global axios import. This injects your API Key into the global axios namespace, and it'll also be added onto other axios requests. This includes requests that are made inside other packages that also use axios, and could result in API Keys being sent to endpoints that it shouldn't be sent to. It also breaks if a package consuming the SDK is also using the global interceptors.
To fix it, I'd recommend to use an Axios Instances. It achieves exactly what you're looking for, and doesn't suffer from the same global-namespace problem.

Hi @Skn0tt, thank you for your interest in the project and for your pointing out this issue. We will look into it and take care!
Feel free to create a PR, though, if you feel the urge. 😉

Absolutely! Opened #4 :)


Cannot Load the Nodes，metoo

Traceback (most recent call last):
File "D:\AIGC\ComfyUI_windows_portable\ComfyUI\nodes.py", line 1813, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 940, in exec_module
File "", line 241, in call_with_frames_removed
File "D:\AIGC\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-NegiTools_init.py", line 8, in
from .negi.open_pose_to_point_list import OpenPoseToPointList
File "D:\AIGC\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-NegiTools\negi\open_pose_to_point_list.py", line 6, in
from .repos.controlnet_aux.src.controlnet_aux import OpenposeDetector
File "D:\AIGC\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-NegiTools\negi\repos\controlnet_aux\src\controlnet_aux_init_.py", line 4, in
from .leres import LeresDetector
File "D:\AIGC\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-NegiTools\negi\repos\controlnet_aux\src\controlnet_aux\leres_init_.py", line 13, in
from .pix2pix.models.pix2pix4depth_model import Pix2Pix4DepthModel
ModuleNotFoundError: No module named 'ComfyUI-NegiTools.negi.repos.controlnet_aux.src.controlnet_aux.leres.pix2pix.models'
Cannot import D:\AIGC\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-NegiTools module for custom nodes: No module named 'ComfyUI-NegiTools.negi.repos.controlnet_aux.src.controlnet_aux.leres.pix2pix.models'
Failed to auto update Quality of Life Suit
QualityOfLifeSuit_Omar92_DIR:

me too

Thank you for your feedback.
Release v0.3.2 fixes a bug related to import, so please check with the latest version.
Note: requirements.txt has been updated.
Please comment if the problem is still not resolved. Or, I will close this issue after a week has passed.


Add social card image and favicon

This PR builds off of #351 to add social sharing visuals and additional metadata protocols ~
Favicon

OS Light mode
OS Dark mode
Colorblind users

Social share preview

This is so sick

Well done, @skullface!
BTW when this gets deployed, we need to visit https://developers.facebook.com/tools/debug and re-scrape.

Oh, good call! I was honestly not aware of this. The change is already live! I'll add now.

caution-solidWarnings That Should Be Fixed
Missing Properties
The following required properties are missing: og:url, og:type, fb:app_id

Awesome, I’ll address those in a new PR~


Update mac setup docs

What
Updated mac setup docs
How to review
See the docs
Who can review
Anyone

This is not needed


Introspecting pipeline stage partitioning results

Hi,
First of all, thanks a lot for the great codebase. I'm benefiting significantly from this.
I was wondering what's a good way to look deeper into the how my model is split into multiple stages for pipeline parallelism.
Specifically, currently Merak outputs something like this:
stage=0 layers=14
     0: GraphModule
     1: GraphModule
     2: GraphModule
     3: GraphModule
     4: GraphModule
     5: GraphModule
     6: GraphModule
     7: GraphModule
     8: GraphModule
     9: GraphModule
    10: GraphModule
    11: GraphModule
    12: GraphModule
    13: GraphModule
stage=1 layers=14
    14: GraphModule
    15: GraphModule
    16: GraphModule
    17: GraphModule
    18: GraphModule
    19: GraphModule
    20: GraphModule
    21: GraphModule
    22: GraphModule
    23: GraphModule
    24: GraphModule
    25: GraphModule
    26: GraphModule
    27: GraphModule
  loss: loss_fn

but is there a way to identify, for example, layer 17 is a transformer layer? What is inside all the GraphModule layers, i.e. which points were the original pytorch module split at?
Thanks a lot.

Hello! Thanks for using Merak. We are happy to hear from you!
To see what is inside graphmodule please set the parameter --wall_clock_breakdown True. The information is printed here
We are sorry that the message is in a mess when opening the parameter wall_clock_breakdown. We will go to improve it in the near future~

I just tried it out for a couple models and it works perfectly! Thanks.
Slightly unrelated to the original issue, but if I wanted to influence where to cut these GraphModules, what would be a way to do that? For instance, I just saw that one transformer encoder layer gets split into two different GraphModules for T5, i.e. self attention ends a GraphModule, and then the fully connected layer for that transformer encoder starts the next GraphModule. What would be a way to force each GraphModule to contain exactly one transformer encoder/decoder layer?

Slightly unrelated to the original issue, but if I wanted to influence where to cut these GraphModules, what would be a way to do that? For instance, I just saw that one transformer encoder layer gets split into two different GraphModules for T5, i.e. self attention ends a GraphModule, and then the fully connected layer for that transformer encoder starts the next GraphModule. What would be a way to force each GraphModule to contain exactly one transformer encoder/decoder layer?

Hello @jaywonchung , this is a very interesting question. It is hard to tell the algorithm what is a transformer encoder. The contents of a graphmodule are decided by the number of parameters and the connections between graph nodes.
But setting the parameter shard_count as described here might be a good try.

I see. I reviewed Merak.autoshard.shard_layers._split_layers and indeed it just operates on torch.fx traced nodes and only views them as a chunk of parameters. Yeah I guess it's very tricky to keep this function general across all models and still make it layer name aware.
As per your suggestion, I tried doubling the shard_count from the default, which splitted about twice as more granular than before, and I was able to group together GraphModules belonging to the same transformer layer into the same pipeline stage by hardcoding layer indices as a new partition_strategy here.
https://github.com/HPDL-Group/Merak/blob/e8a2a779fea878be9b778f8a808a192364766f36/Merak/modules/module.py#L376
Not the ideal solution but a hack that works for now. Thanks!

Happy to hear that works for you.
And there might be another way that makes the algorithm layer name aware.
Leaf modules in Merak.trainer could let torch.fx treat a module as a whole. Passing the transformer layer instance could be helpful. The usage is here.
The related code is here and here.
Sorry for the late information. I hope this would also be helpful.

Thanks for the extra information! T5Block would be the right leaf node to use. I tried it like this in the run_t5.py example:
from transformers.models.t5.modeling_t5 import T5Block

trainer = MerakTrainer(
    model=model,
    args=training_args,
    train_dataset=dataset,
    eval_dataset=dataset,
    # Data collator will default to DataCollatorWithPadding, so we change it.
    data_collator=default_data_collator,
    leaf_modules=(T5Block,),
)

But, unfortunately it dies during tracing:
Traceback (most recent call last):
  File "/workspace/merak/examples/language-modeling/run_t5.py", line 93, in <module>
    main()
File "/workspace/merak/examples/language-modeling/run_t5.py", line 86, in main
    train_result = trainer.train()
  File "/workspace/merak/Merak/merak_trainer.py", line 729, in train
    self.create_optimizer_and_scheduler(num_training_steps=max_steps)
  File "/workspace/merak/Merak/merak_trainer.py", line 260, in create_optimizer_and_scheduler
    model, model_layers, input_to_shard_dic = convert_to_sequential(self.model, self.args, self.leaf_modules)
  File "/workspace/merak/Merak/autoshard/convert.py", line 107, in convert_to_sequential
    traced, dummy_inputs = symbolic_trace(
  File "/workspace/merak/Merak/autoshard/convert.py", line 378, in symbolic_trace
    traced_graph = tracer.trace(model, concrete_args=concrete_args)
  File "/workspace/merak/Merak/autoshard/convert.py", line 274, in trace
    graph = torch.fx.Tracer.trace(self, root, concrete_args=concrete_args)
  File "/root/.local/miniconda3/lib/python3.9/site-packages/torch/fx/_symbolic_trace.py", line 615, in trace
    self.create_node('output', 'output', (self.create_arg(fn(*args)),), {},
  File "/root/.local/miniconda3/lib/python3.9/site-packages/transformers/models/t5/modeling_t5.py", line 1611, in forward
    decoder_outputs = self.decoder(
  File "/root/.local/miniconda3/lib/python3.9/site-packages/torch/fx/_symbolic_trace.py", line 604, in module_call_wrapper
    return self.call_module(mod, forward, args, kwargs)
  File "/root/.local/miniconda3/lib/python3.9/site-packages/torch/fx/_symbolic_trace.py", line 422, in call_module
    return forward(*args, **kwargs)
  File "/root/.local/miniconda3/lib/python3.9/site-packages/torch/fx/_symbolic_trace.py", line 600, in forward
    return _orig_module_call(mod, *args, **kwargs)
  File "/root/.local/miniconda3/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/root/.local/miniconda3/lib/python3.9/site-packages/transformers/models/t5/modeling_t5.py", line 912, in forward
    batch_size, seq_length = input_shape
ValueError: too many values to unpack (expected 2)

Removing the leaf_modules line runs fine.


Default password range should be adjusted to comply with NIST standards

Current behavior
  # ==> Configuration for :validatable
  # Range for password length.
config.password_length = 6..128

Expected behavior
  # ==> Configuration for :validatable
  # Range for password length.
config.password_length = 8..1000

The minimum value should be 8 characters. The max value should be much higher than 128. Maybe 255? I don't see any reason why you would want to limit users on the max length (perhaps within 1000 characters), especially if they're using 3rd party password generator tools.
I can file a PR with this change depending on what people decide makes the most sense!

The password length is configurable, so you can change it to what better fit your needs.
Changing this default now can be a pain for backward compatibility, so I think we should keep as it is today.

@tegon How is this a pain for backwards compatibility ?? The defaults that are generated would be for NEW users of devise. This just sets a better standard for future users.

I'm reopening this issue because I feel strongly enough that leaving it at 6 is irresponsible at this point. Please reconsider. Yes you can change it, but most people won't (especially newbies,). Shouldn't we be guiding users into the best practices?
As an aside I believe bcryot is no longer favorable either but we can take about that later...


Different results Robyn 3.6 and 3.7 (same data and hyperparameters))

Project Robyn
I re-ran the same set of data using Robyn 3.6 then re-ran with 3.7.   I received very different results.   Was there a major change in the logic that would cause this?
Thanks.
Describe issue
Describe your issue and provide warnings and error messages.
Provide reproducible example
Issues are often related to custom input data that is difficult to debug without. If necessary, please modify your data to mask real values and share a dataset that is able to reproduce the issue. Please also share your model configuration and exported JSON files if available.
Environment & Robyn version
Make sure you're using the latest Robyn version before you post an issue.

Check and share Robyn version: packageVersion("Robyn")
R version (Please, check and share: sessionInfo() or R.version$version.string)

Hi @tgtod002 we've changed a lot of things around since version 3.6. Check our changelogs based on our releases here.
Basically, we re-coded everything from scratch to migrate from data.tables to R and tidy code format. Results should be reproducible using the same versions, not guaranteed when different versions.
Keep in mind there are several random associated processes that will give different results; that's why we enable the seed parameter to ensure the same randomness when running random-related flows and getting the same outputs when re-run with the same inputs.
Lastly, when you say "very different results", it's not necessarily bad: remember Robyn (nevergrad) will run different scenarios and optimize to minimize the errors, returning thousands of possible models, and providing the best most different models for the user to pick from. They will all be mathematically the best, but not all will make sense necessarily; that's where the user must intervene with his/her knowledge of the business and pick the best one to continue.

Thanks for letting me know.   It makes sense.
Given that, does it make sense to re-run the model every time new versions of Robyn is released?

hi @tgtod002 , by "same hyperparameters" do you mean the same set of ranges at the beginning? if yes, then Bernardo is totally right. Due to the several random processes, even with the same version, you will/should receive different results. Although when your models are all converging, then the candidate clusters from different runs shouldn't be too different. But they won't be identical.
Yes, it's good exercise  to rerun the model after update, because Robyn is still being actively developed. We'll do everything we can to ensure consistency

Please reopen if this issue reoccurs.


If result of getNextRunDate / getPreviousRunDate happens to be during DST change window, exception is thrown

If the result of getPreviousRunDate happens to be within the "missing" hour during DST change, an "Impossible CRON expression" error is thrown.

Isnt that the expected behaviour though? The cron is not valid if the time doesnt exist?

The expected behaviour is to return a valid timestamp. Consider a cron string like "45 */3 * * *" (which is the actual case which triggered the bug). I'm not asking for a time which does not exist, I'm asking for the previous run date and expect cron-expression to figure this out, not fail because it tries to instantiate an invalid DateTime.

I'm trying to get a handle on this, and #148.
With the following test example, wherein we try to set up a "current time" where we will cross a DST boundary, it seems to return the correct timestamp:
What time did the error occur? I've been testing various times on the current master branch and it seems to be returning the expected run times, properly hopping across the DST time switches.


About the test setting of the trained model

Thanks for your contribution of this repository.
I have a doubt. In the original paper, the authors mentioned that the experimental results given are all tested under real attacks, such as JPEG compression with different quality factors, but in this repository, it seems that the trained models are verified and tested on the basis of simulated noise attacks?

Thanks for your answers, I want to know whether you will provide the corresponding code files for testing under the real JPEG compression attack in the future.


Organized schema folder

Renamed folder to "schemas"
Removed unused schemas (TypeBaseNumber is not used but was not removed since it is a basic schema)
Removed "String" schema and used existent "BaseTypeString" instead
Modified "list_path_content" and "list_content_pkg" actors to use a string list that do not contain duplicates
Removed "StringList" schema and used existent "TypeStringList" instead

Following @bertinatto suggestions, this PR is about cleaning up "schema" folder

Thanks for this @artmello the changes look good, however I'd like to propose dropping BaseTypeString and TypeStringList as well, and rather adopting the less contrived names everywhere - String and StringList.
The problem I have with these names is that they're confusing - TypeStringList, BaseTypeString raises, in my mind, the following questions:

...TypeString? Is the value of the string expected to be a type name, like float, int and whatnot?
Base ... ? So is this just an base type for strings and should I use something more concrete? (like UtfString or AsciiString?)

The bottom line is, I find String and StringList nice and straightforward, whereas the other names BaseTypeString and TypeStringList seem to represent a wholly different kind of information.
/cc @bertinatto @mfranczy @vinzenz


Is this repo moved to windows toolkit?

Is this repo still maintained and updated or is it moved to
https://github.com/windows-toolkit/Lottie-Windows
?

It's not the same repo. It's a from-the-ground-up new solution (much better and more performant). You should migrate your code to use that new implementation. This repo is no longer maintained, in favor to that solution.

maybe a good idea would be updating your readme here and point to the new solution?

@sohchatt sent a PR a while ago adding a reference to it:
https://github.com/azchohfi/LottieUWP#lottieuwp

For the official Lottie-Windows project, go here: https://aka.ms/lottie

I would suggest having something much stronger.  It is not clear this is no longer maintained here and they should go to the Toolkit's implementation

@sohchatt sent a PR a while ago adding a reference to it:
https://github.com/azchohfi/LottieUWP#lottieuwp

For the official Lottie-Windows project, go here: https://aka.ms/lottie

In keeping with @crutkas 's suggestion, do you want to include a soft-deprecation message or something similar?


contrib: fix bump-version script on macOS

This fixes non-POSIX conformant usage of expr that is only supported
in the GNU/Linux variant and caused syntax errors on macOS 10.12.1.

I have only tested the changes on macOS, so it would be great if someone could run this on linux before merging, although I'm pretty confident it'll work fine (I compared the linux manpage for expr).

Thanks for doing this, I tested it on a recent fedora and it works for me.


OSError: black-forest-labs/FLUX.1-dev does not appear to have a file named config.json.

Traceback (most recent call last):
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/connection.py", line 199, in _new_conn
sock = connection.create_connection(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/util/connection.py", line 85, in create_connection
raise err
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/util/connection.py", line 73, in create_connection
sock.connect(sa)
OSError: [Errno 101] Network is unreachable
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/connectionpool.py", line 789, in urlopen
response = self._make_request(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/connectionpool.py", line 490, in _make_request
raise new_e
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/connectionpool.py", line 466, in _make_request
self._validate_conn(conn)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/connectionpool.py", line 1095, in _validate_conn
conn.connect()
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/connection.py", line 693, in connect
self.sock = sock = self._new_conn()
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/connection.py", line 214, in _new_conn
raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7fd2b1689990>: Failed to establish a new connection: [Errno 101] Network is unreachable
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/requests/adapters.py", line 667, in send
resp = conn.urlopen(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/connectionpool.py", line 843, in urlopen
retries = retries.increment(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/urllib3/util/retry.py", line 519, in increment
raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='huggingface.co', port=443): Max retries exceeded with url: /black-forest-labs/FLUX.1-dev/resolve/main/transformer/config.json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fd2b1689990>: Failed to establish a new connection: [Errno 101] Network is unreachable'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/file_download.py", line 1376, in _get_metadata_or_catch_error
metadata = get_hf_file_metadata(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py", line 114, in _inner_fn
return fn(*args, **kwargs)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/file_download.py", line 1296, in get_hf_file_metadata
r = _request_wrapper(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/file_download.py", line 277, in _request_wrapper
response = _request_wrapper(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/file_download.py", line 300, in _request_wrapper
response = get_session().request(method=method, url=url, **params)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/requests/sessions.py", line 589, in request
resp = self.send(prep, **send_kwargs)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/requests/sessions.py", line 703, in send
r = adapter.send(request, **kwargs)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/utils/_http.py", line 93, in send
return super().send(request, *args, **kwargs)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/requests/adapters.py", line 700, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: (MaxRetryError("HTTPSConnectionPool(host='huggingface.co', port=443): Max retries exceeded with url: /black-forest-labs/FLUX.1-dev/resolve/main/transformer/config.json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fd2b1689990>: Failed to establish a new connection: [Errno 101] Network is unreachable'))"), '(Request ID: 8b49e1ce-3d54-42ea-b148-1326ddef2ca1)')
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/diffusers/configuration_utils.py", line 379, in load_config
config_file = hf_hub_download(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py", line 114, in _inner_fn
return fn(*args, **kwargs)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/file_download.py", line 862, in hf_hub_download
return _hf_hub_download_to_cache_dir(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/file_download.py", line 969, in _hf_hub_download_to_cache_dir
_raise_on_head_call_error(head_call_error, force_download, local_files_only)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/file_download.py", line 1487, in _raise_on_head_call_error
raise LocalEntryNotFoundError(
huggingface_hub.errors.LocalEntryNotFoundError: An error happened while trying to locate the file on the Hub and we cannot find the requested files in the local cache. Please check your connection and try again or make sure your Internet connection is on.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/mnt/workspace/ai-toolkit/run.py", line 90, in
main()
File "/mnt/workspace/ai-toolkit/run.py", line 86, in main
raise e
File "/mnt/workspace/ai-toolkit/run.py", line 78, in main
job.run()
File "/mnt/workspace/ai-toolkit/jobs/ExtensionJob.py", line 22, in run
process.run()
File "/mnt/workspace/ai-toolkit/jobs/process/BaseSDTrainProcess.py", line 1258, in run
self.sd.load_model()
File "/mnt/workspace/ai-toolkit/toolkit/stable_diffusion_model.py", line 546, in load_model
transformer = FluxTransformer2DModel.from_pretrained(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py", line 114, in _inner_fn
return fn(*args, **kwargs)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/diffusers/models/modeling_utils.py", line 640, in from_pretrained
config, unused_kwargs, commit_hash = cls.load_config(
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py", line 114, in _inner_fn
return fn(*args, **kwargs)
File "/mnt/workspace/ai-toolkit/venv/lib/python3.10/site-packages/diffusers/configuration_utils.py", line 406, in load_config
raise EnvironmentError(
OSError: black-forest-labs/FLUX.1-dev does not appear to have a file named config.json.

I solved the problem by setting up a proxy


runtime expression	$[variables.var] renders as is when not found

while the doc says it renders empty string.
Example pipeline:
steps:
- bash: echo '$[variables.var]'

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

ID: dd7e0bd3-1f7d-d7b6-cc72-5ef63c31b46a
Version Independent ID: dae87abd-b73d-9120-bcdb-6097d4b40f2a
Content: Define variables - Azure Pipelines
Content Source: docs/pipelines/process/variables.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

I wonder if you ran into a bug. Can you test out this YAML? If it doesn't work as expected, please ask a question in our Developer Community. The Developer Community is actively monitored by the engineering team.
trigger: none

pool:
  vmImage: 'ubuntu-latest'

steps:
- bash: echo '$[variables.var]' #output $[variables.var]
- script: echo '$[variables.var]' #output $[variables.var]


added nodejs function

:tada: This PR is included in version 0.5.6 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


RuntimeError: unreachable

This simple ini file wil fail to parse, as it contains an empty comment line
#

https://github.com/editorconfig/editorconfig-core-js/issues/111
https://github.com/renovatebot/renovate/issues/18540

full sample:
#
# .editorconfig
#
# http://editorconfig.org
#
# The EditorConfig project consists of a file format for defining coding styles
# and a collection of text editor plugins that enable editors to read the file
# format and adhere to defined styles.
#

# top-most EditorConfig file
root = true

# Unix-style newlines with a newline ending every file
[*]
end_of_line = lf
indent_style = space
indent_size = 4
insert_final_newline = true

# package.json specific rules
[package.json]
indent_size = 2

# .yml specific rules
[*.yml]
indent_size = 2

...except the ? should have caught that.  Regardless, I can reproduce locally.


Inconsistent phantom type on sha256t::Hash

Just a quick comment... I can't see why there are both TapLeafTag and TapTweakTag used on Hash in the following function:
https://github.com/rust-bitcoin/rust-bitcoin/blob/d662dba8b5d0f28dae1f090703761d4159f6a8d2/bitcoin/src/taproot/mod.rs#L67-L73
Looking into other functions: from_key_and_tweak, combine_node_hashes, they are all using the same phantom type inside. I know it's just a phantom type and does no actual effects, however it just gave me some confusion when I dig into the source piece here. Is this intended somewhat?

This phantom type definitely does have "actual effects". These two hashes are different tagged hashes and will have different output. Their usage is correct according to BIP-341.

Nope. I know they are used to produce "tagged hash", but in this case..
fn from_script(script: &Script, ver: LeafVersion) -> TapLeafHash {
            let mut eng = sha256t::Hash::<TapLeafTag>::engine();
            ver.to_consensus()
                .consensus_encode(&mut eng)
                .expect("engines don't error");
            script
                .consensus_encode(&mut eng)
                .expect("engines don't error");
            let inner = sha256t::Hash::<TapTweakTag>::from_engine(eng);
            TapLeafHash::from_byte_array(inner.to_byte_array())
        }

        fn from_script2(script: &Script, ver: LeafVersion) -> TapLeafHash {
            let mut eng = sha256t::Hash::<TapLeafTag>::engine();
            ver.to_consensus()
                .consensus_encode(&mut eng)
                .expect("engines don't error");
            script
                .consensus_encode(&mut eng)
                .expect("engines don't error");
            // --------------------------------------- CHANGED HERE ----
            let inner = sha256t::Hash::<TapLeafTag>::from_engine(eng);
            TapLeafHash::from_byte_array(inner.to_byte_array())
        }

        let hash1 = from_script(script_hex!("5187"), LeafVersion::TapScript);
        let hash2 = from_script2(script_hex!("5187"), LeafVersion::TapScript);
        assert_eq!(hash1, hash2);

This code passes. The phantom type on let inner = statement does nothing.
From my view, TapTweak tagged hash is only used when generating the final tweak, but is not used when generating the hash of a script leaf.
Am I missing something still?

Wow, cool! Good catch. Yeah, the existing code is just wrong -- though as you say, only in a philosophical sense since Hash::from_engine does not use its tag in any way.
cc https://github.com/rust-bitcoin/rust-bitcoin/issues/1552 (which I also initially struggled to understand :P).
I think we should fix this in the hashes API before hashes 1.0 (and if not, we should have something in our docs saying why we couldn't do it).

The mistake is mine, introduced in commit 39f7dcb816. Fortunately it has no effect, as correctly pointed out by @bczhc. Thanks for bug the report, we should definitely fix this.
FTR the tags are different, if line 68 was changed then the code would break. They just only differ when creating the engine not when finalizing the hash (calling from_engine). In line 72 the hash type doesn't matter as long as its a SHA256 hash (ie. sha256::from_engine would also be fine but still somewhat confusing).
Thanks again for finding my bugs!


coverageReporters option is missing from API docs

I was looking for a way to set additional coverage reporters and found that there is the coverageReporters option available that takes an array of strings. I assume, that those are exactly the same names that Istanbul takes for reporters and that all standard reporters are available? This seems to be correct - by default there are ['json', 'lcov', 'text'] reporters enabled and anything in the coverageReporters overrides those.
Is it intentionally missing from the API docs? Some compatibility issues?

yes. this is directly proxied to istanbul. we should add it to the docs!


Figure out cacheing

Ensure that the nginx cache will be awesome enough so that the transpilation will not be a performance problem

I might just need to do the cache on my own in memory like in Zwitterion. Just use a JS object with keys as path names and values as the file contents

Doesn't really matter, since this won't be used in production, it will just be used for development and then it can create a production build that is static and can be hosted on a CDN.