v4 feature request: support lerna packages and yarn workspaces

So the new approach to using typescript's paths property for monorepo works nicely.
The truth is, though, that for a real world monorepo setup, I wouldn't use typescript paths at all. My monorepos are set up with lerna and yarn workspaces, which symlinks the source code from child package to a parent's node_modules directory. The parent references the child package in their package.json as 'just another dependency'.
This allows each dependency to be treated as an independent package and has many benefits, including being able to automatically calculate change logs and semver and navigate dependency trees. In fact, yarn workspaces already keeps the dependency tree for us.
The downside to this, is that to fuse-box, the child packages look like 'just another dependency', and fuse-box can't tell the difference between our monorepo packages and those installed in node_modules through npm.
The result of this is that fuse-box builds everything correctly, but we get no watch or hmr on our local monorepo dependencies.
Ideally, we would update usages of ´resolver.ts´ to allow for yarn workspace and lerna workspace information to be read, so that we can detect when dependencies are local, and treat them the same as though they were included in tsconfig paths.

To clarify, if I combine yarn workspaces with paths in tsconfig, everything seems to work fine.

Is this with yarn v1 or v2? Afaik paths in tsconfig doesn't work with Yarn v2.

Hmm, I'm still on yarn 1... I'm not familiar with yarn 2 and how it might change the monorepo game.
Basically, I think yarn workspaces are the correct way to do deps between packages in monorepos. It opens up lots of cool stuff like automatic dependency walking when generating semver from commits etc.
The thing is that fuse-box won't listen to changes inside packages linked via yarn workspaces because they just look like any other node_module. So I have to add paths in tsconfig as well to get fuse-box to work.
The drawback then, is that I no longer actually need to add the referred package as a dependency to my code. I want to maintain that need for explicit declaration of deps, so that semver can be calculated, and if we need to split up the monorepo, any package can be pulled out and replaced with npm or verdaccio on a whim.

To achieve this, basically all fuse-box would have to do is look up the workspaces properties for lerna/yarn from package.json or lerna.json, then iterate the package names from each of them - and it'd have a list of packages that it should watch when encountered. Seems like small potatoes. I'd take a look myself if I could get some guidance on where to start.

@Martaver sounds like a good idea. baseUrl already does that btw. Let's have a chat about it on slack.

I've been working a lot on this.  I have a branch where I think I have these problems resolved, and I'm just testing this now.  I thought I'd give some information on how some of this is solved.

My monorepos are set up with lerna and yarn workspaces, which symlinks the source code from child package to a parent's node_modules directory. The parent references the child package in their package.json as 'just another dependency'.

The biggest reason that this wasn't working before was fuse-box's module resolution not looking above the app root.  With #1820 (merged in @next), this part is resolved.  Or almost...

The thing is that fuse-box won't listen to changes inside packages linked via yarn workspaces because they just look like any other node_module

Yes and no... since these are symlinks in those workspaces strategies.  And since node always dereferences symlinks, node does not actually see their paths as containing a node_modules.  So we just need to make fuse-box dereference these like node does.  But currently it does not - that's the "almost" part.  I have a fix for that coming.
This solves the watcher issue for me because even though workspaces are imported "through" node_modules, fuse-box can tell they are not actually "in" node_modules and they can trigger the watch.

This allows each dependency to be treated as an independent package and has many benefits, including being able to automatically calculate change logs and semver and navigate dependency trees. In fact, yarn workspaces already keeps the dependency tree for us.

Yes, this is one of the main goals, but it isn't enough of its own.  If these were simple JavaScript with no transformation step, then this would be enough.  But with TypeScript you'll run into all kinds of compatibility issues because conventional tools expect packages to export .js not .ts.  Things like the "main" entry point, or separating src from dist cause problems.  E.g. dist/file.js is expected to be built and exist and so you're expected to import x from "greatpkg/dist/file" not import x from "greatpkg/src/file".
TypeScript paths can sort of rescue you here by rewriting some of these things, but it runs into its own problems.
So TypeScript's own solution to this problem is to have tsconfig.json files reference other tsconfig.json files.  That's described here: https://www.typescriptlang.org/docs/handbook/project-references.html.
When you import greatpkg/dist/file it uses Node.js's resolution algorithm to find the greatpkg part.  And then it uses the tsconfig.json to translate the dist/file part into src/file.ts.
Furthermore, tsc --build from the top-level package will automatically incrementally build child packages.  And Visual Studio Code also knows about this and so it also doesn't complain about dist not existing.
You can see an example of a monorepo setup using tsconfig references here: https://github.com/andrewthauer/ts-monorepo .  But this one is using Webpack.  I want to do that with fuse-box.  I have this working in my branch right now.

The drawback then, is that I no longer actually need to add the referred package as a dependency to my code. I want to maintain that need for explicit declaration of deps

Yes, this!  This is a big problem.
Unfortunately, though, what you'll find is that fuse-box isn't causing this problem and in fact can't do much about it.  This is a fundamental flaw in how Node.js's node_modules resolution has always worked, combined with how those workspace strategies are designed to use it.
Specifically, Node.js's module resolution algorithm does not look at declarations (package.json) at all!  It only looks at what is in node_modules.  So when the workspaces hoist all "declared" dependencies into a shared node_modules it makes it so that as soon as one workspace "declares" a dependency, all workspaces can "import" it.  This flaw in Node.js's resolution algorithm is one of the worst flaws of Node.js.
Yarn 2 solves this by replacing node_modules entirely with Plug'n'Play (PnP) (which is just genius and awesome, btw).  PnP will not let you import a dependency you have not declared in your package.json.  The great news is that fuse-box does support PnP in @next.  But PnP is still hard to recommend using at this moment now because TypeScript still doesn't support it natively and requires some hacks to work.  I'm using it, though, and have made sure that it works with my ts references changes in fuse-box.  Hopefully the TypeScript team adds support soon.

Appreciate the detailed break-down! It's really useful to hear how other people are tackling this.
PnP is probably the future and with a bit more adoption, it should be a use case we can safely pour time and energy into. Personally, I haven't played with it yet, and I don't know whether it's going to actually make life easier for the monorepo scene...
With regards to the non-PnP, though, I don't understand why we can't just update resolver.ts to read yarn or lerna workspaces... yarn is already doing all the hard work for us.
I don't think it's wise to make fuse-box do any special treatment of symlinks. That sounds like it's opening pandora's box.

The reason workspaces work in the node ecosystem isn't because everyone went out and added support, it's because workspaces were designed to take advantage of the way the node ecosystem already works.
Node doesn't know anything about workspaces, and neither does much of anything else.  The whole point is that they don't have to.  They just have to keep doing what they were doing.  To make workspaces work, you just have to ask "how is everyone else doing it" and the answer is "they just do what node does" because that is what the workspaces expect - it's how they were designed.

I don't think it's wise to make fuse-box do any special treatment of symlinks. That sounds like it's opening pandora's box.

Exactly.  Node and everything else dereferences symlinks.  Fuse-box doesn't.  So fuse-box is already doing special treatment.  And this special treatment actually breaks things1.  And so I am agreeing with you here and saying that fuse-box should start dereferencing symlinks like everyone else.

With regards to the non-PnP, though, I don't understand why we can't just update resolver.ts to read yarn or lerna workspaces from their config files...

Because the config files are instructions for the installer not the resolver.  Resolvers don't read workspaces from the config file.

...yarn is already doing all the hard work for us.

Exactly, yarn is already doing all of the hard work.  That's why resolvers don't read workspaces from their config files.  The "hard work" that yarn does is to set up node_modules for the resolvers so that they can just work like they always have worked.  Resolvers don't need to know anything about workspaces.  They just have to read node_modules like everyone else.
The "like everyone else" part is important.  This is what fuse-box was not doing before #1820 and why it didn't work with monorepos.  To do it "like everyone else" here just means "like node does it" because they all copy node.
So it isn't that we can't update resolver.ts to read yarn or lerna workspaces.  It's that it doesn't help anything.  There's no new information to be found in the workspace config that is useful for a resolver.

1 Not dereferencing makes local things look non-local because it leaves "node_modules" in the path.  Making things look non-local breaks typescript project references and also makes fuse-box's watcher ignore them.

I should clarify that I'm not saying that fuse-box shouldn't know anything about workspaces.  Just the resolver shouldn't need to.  I would like to see fuse-box be zero config as much as possible, and this means that if it could detect a workspace, it could set up somethings automatically that today you have to manually reconfigure (such as the watcher, for example).  So workspace info isn't useless... it just isn't a resolver thing.

So essentially what you're saying is that fuse-box should watch those folders anyway, because if it dereferences those symlinks it will see that they're not actually in node_modules (by inspecting their path), and start to watch them?
So then in the case of typescript paths, TS is letting our folders look like node_modules, but they don't exist in node_modules. Fuse-box fails at trying to resolve them, which is why it needs the information about paths from tsconfig.
This all makes perfect sense.
I think one benefit that project references + tsc has over other configurations is that each 'project' can have its own tsconfig. However I'm pretty sure fuse-box reads only a single, root tsconfig. To 'perfect' the monorepo scenario, we would really need fuse-box to handle nested tsconfigs. What do you think? Pinging @nchanged too...

I think that's a great solution by the way!

@Martaver fuse isn't just reading one tsconfig, if a package contains one and it points to a typescript source it will respect the paths defined, However it won't try too hard to find the config, the resolver just checks a tsconfig next to the package.json file
I was hoping we could improve on that without perf losses

You could check the package.json for entries and also check for tsconfigs in the folders between the entry point back up to the root.

Closed due to inactivity.


Update schema docs

Update schema documentation with examples for Image, BBox, Text, Audio and ClassLabel

@thisiseshan Mind resolving the few conflicts?

Strange, we're still getting linting errors. Copying @imshashank in case he can help!

I fixed a few linting errors. @AbhinavTuli could you take a look at it please.

Thank you @kristinagrig06

Also, regarding the tests that have failed, merging master into your branch should resolve that.

Yeah I thought so too but a "Getting started with Audio" example should provide more in depth context. I believe @sparkingdark is working on it.

@thisiseshan these are great additions!

Thank you, wouldn't have been possible without support from @AbhinavTuli ✨


/en-US/docs/Web/API/DocumentFragment/querySelectorAll

Request type

[ ] New documentation
[x] Correction or update

Details

Typo:
documentframgment -> documentfragment

Fixed, thanks for filing!


time.Sleep returns immediately for very long durations (integer overflow?)

go1.13.5
amd64p32

If Duration is at least around MaxInt64 / 4, time.Sleep returns immediately. This doesn't happen at MaxInt64 / 8.
https://play.golang.org/p/nC3KiDrY_vr
What did you expect to see?
The script should start sleeping.
What did you see instead?
time.Sleep returns immediately.

It works fine for me on go version go1.13.5 linux/amd64. That is, the program starts sleeping, and it doesn't seem to stop.


Recursively generate all HTML doc files from Enso files.

Pull Request Description

Important Notes

Checklist
Please include the following checklist in your PR:

[ ] The documentation has been updated if necessary.
[ ] The code has been tested where possible.

Oh, also please add a CI step to run prettier. You can take the configuration and ignore file from the enso repo.

Also, please change the name of your CI to Doc Viewer, so we have Doc Viewer / build.


update gpg --quick-add-key commands

gpg --quick-add-key seems to be only accepting fingerprints and rejecting key ID-s:
$ export KEYID=0x450944965A788EE4
$ gpg --quick-add-key "$KEYID" rsa4096 auth 2y
gpg: "0x450944965A788EE4" is not a fingerprint

so we should extract the fingerprint from the following:
$ gpg --fingerprint $KEYID
pub   rsa4096/0x450944965A788EE4 2024-03-04 [C]
      Key fingerprint = 6768 2AB6 06DF 7C9A 8BB6  C2D6 4509 4496 5A78 8EE4
uid                   [ultimate] Test User<EMAIL_ADDRESS>

...and the most format-agnostic grep pattern I could think of was to find the first occurrence of groups of hexadecimal numbers and spaces, and then truncate the result.

Fixed for now. I'll give this a try on several system and update as needed. Thanks again @Xronophobe


Run some type of function will meet error

I use the uhdm to translate the orv64 SystemVerilog file into the verilog file, and want to do synthesis by yosys, the orv64 gitlab adress is here: https://gitlab.com/picorio/OpenSource/hardware/orv64
But there is some error when doing this translate , when the function include following format(Variables include output)， the code where use this function will meet error. Here is an example in the orv64
function automatic void lru8_update_lru(
  output logic [6:0] new_lru,
  input logic [6:0] old_lru,
  input logic [2:0] way_id
);
  new_lru = old_lru;
  new_lru[6] = ~old_lru[6];
  if (way_id[2])
    new_lru[5] = ~old_lru[5];
  else
    new_lru[4] = ~old_lru[4];

  new_lru[way_id[2:1]]  = ~old_lru[way_id[2:1]];

endfunction

and will meet this error

the function screenshot is this

what we can do is change the source code where use this function, and expand this function where it is used, and this problem will be sovled.

Thank you for reporting this, I have reproduced the error and we will investigate it.

This issue is closed, was it fixed?

I've just retested with current plugin, the issue is still present. Reopening.


cannot escape interactive map teleport

i tryed using the interactive map to teleport to keven when he ran off but when i try to turn off the interactive map it wont let me

You are probably playing on a resolution > 1080p which from what I remember can have this problem. Anyway this mod menu is outdated and you shouldn't really use it but this instead. But right now all mods have problems so it's better to wait for a fix.


User-Api: add getMyProjects (#1233)

see #1233

@BlueTeck and @fguillot
Where are the unit tests?

@fguillot will add them


[Task] Play the attack animation dependent on the currently equipped weapon

Description
Task: #200
Feature: #150
This task involves harmonising the various animations which will be created for each weapon, ensuring that the corresponding attack animation is played dependent on the weapon that is equipped.
Example (optional)

Attack animation: If dagger level one is equipped, play the corresponding animation for dagger level one.

Dependencies

[x] #178 - Animation for weapons needs to be created before it can be implemented into the game.

Milestones
List of steps that need to be completed for this task.

[x] Create function/method that tests to see what weapon is currently equipped
[x] Based on the weapon equipped, place a still image of the weapon on the player's hand
[x] Based on the weapon that is equipped, play the associated attack animation for that weapon when SPACE is pressed
[x] Only play attack animation if the player attack status is current
[x] Dispose animation when weapon is unequipped
[ ] Write visual test inspection guide

Completion Deadline: Sept. xx
Documentation

Attack function

Member

Lachlan Benson (@LachlanBenson ) (Lachlan.Benson#4926)
Eugene Chong (@eugene2341 ) (gene#0005)
Nisha Vashist (@NishaVashist0 ) (Nisha Vashist #4270)

Attack animation now plays based on the weapon that the player is holding in their hand
Still to do - make a still image appear of the weapon before pressing attack

Attack animation now plays when weapon is equipped. Figured this was acceptable rather than a still image - just means the player tests out the weapon when they equip it.
Would like assistance from someone with disposing of the weapon atlas upon "unequip" or "drop"

hey @PeterOu8 are you able to assist with the final coding-related item for this task?

[ ] Dispose animation when weapon is unequipped

Sure thing!

The animation will be correctly disposed of after unequipping now.

Currently, animation is played based on weapon, plus the associated buff effect. will need to see how this works with stationary animation.

The animation will be correctly disposed of after unequipping now.

thanks Peter!


Add support for new Worksection layout

Please remember the Contributing Guidelines :heart:
:star2: What does this PR do?
Adds support for new Worksection layout. Fixes integration 🤷♂
:bug: Recommendations for testing
All changes should be tested across Chrome and Firefox.
Button should be visible in few places:
When hover over task item in tasks list

On task details page

On task details modal view
to view it you need to click on item in tasks list (BUT NOT ON TITLE 💥)

:memo: Links to relevant issues or information
Closes #1496

Thanks for review 🙇

Small project popdown

The project dropdown is really small, so filtering doesn't visually render and at first I thought no projects were populated

Create an issue https://github.com/toggl/toggl-button/issues/1606


⚖ Register Validator:

Network Choice
nemeton-1
Who are you?
Moniker : Coinstamp
We are glad to be here. Coinstamp is a validator mainly on Cosmos.
Our Mainnet Nodes :
Rebus- https://rebus.explorers.guru/validator/rebusvaloper1jk394yhsr3c0am9fezu7luqz3hvs2ey0dvej33
Meme-https://ping.pub/meme/staking/memevaloper1xvr6cjp9xec4ar87ecljp7mhwcqeuzr3s842c2
Genesis-L1-https://ping.pub/genesisl1/staking/genesisvaloper10rx8lxl33tuz08jf94z7ullf6lxx7mg8rdl3aj
Assetmantle-https://assetmantle.explorers.guru/validator/mantlevaloper1f6u85enm6dfq8y7rqyfq2550eztcjh9u4kkvda
Stride-https://stride.explorers.guru/validator/stridevaloper10ucf07jl82595s8l2ncayf6lpzz32ln4whjug3
Lambda- https://explorer.nodestake.top/lambda/staking/lambvaloper1xx7kwxn959r35a7s5tmd66c5e6g2hzkexhfvv8
Jackal- https://explorer.nodestake.top/jackal/staking/jklvaloper150939ssmyfjxy3px29fujflnlyv89mt2gg5qzg
About us:
💻 Website :  https://coinstamp.me/
💻 Twitter : https://twitter.com/coinstamp_
💻 Youtube : https://www.youtube.com/c/CoinStamp
💻 Discord : kivanc | Coin Stamp#4854
gentx
{"body":{"messages":[{"@type":"/cosmos.staking.v1beta1.MsgCreateValidator","description":{"moniker":"Coinstamp","identity":"DC593D952394A19A","website":"https://coinstamp.me/","security_contact":"info@coinstamp.me","details":"linktr.ee/coinstamp"},"commission":{"rate":"0.050000000000000000","max_rate":"0.100000000000000000","max_change_rate":"0.010000000000000000"},"min_self_delegation":"1","delegator_address":"okp41azhqqyzdl2vpux2yd4v66um59qfuzw45ug450g","validator_address":"okp4valoper1azhqqyzdl2vpux2yd4v66um59qfuzw45f09a3f","pubkey":{"@type":"/cosmos.crypto.ed25519.PubKey","key":"SaqPIxcDVi4T4jE1A5L6R+ProgTqFh1RPzgeIyRvtXE="},"value":{"denom":"uknow","amount":"10000000000"}}],"memo":"da8e2423cb90fba519e685aa47669eb861ea18c4@<IP_ADDRESS>:26656","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[{"public_key":{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"A9rccqa4NWaMx2oD+RZzynI9OFPL10cFHFVc4Qec1SlN"},"mode_info":{"single":{"mode":"SIGN_MODE_DIRECT"}},"sequence":"0"}],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""},"tip":null},"signatures":["5tx8pDi8bfqgYL6asPyDGNellwTwydTzVP6zdquM47kCVlKUvclaDYVD1/PsWhZviCCu/M9jr0axjjHpruGMVw=="]}

😉 Here is the corresponding PR: https://github.com/okp4/networks/pull/326


[bug] [docs] Images not rendering

Hey, I just noticed that the paths are misconfigured in the docs, example:

Github has some issues with the paths on the asciiDoc format. I'll fix it

@pr3mar It's fixed now. Check out the images https://github.com/amejiarosario/dsa.js/blob/master/book/chapters/linked-list.adoc


New Release before new LoL patch

Would it be possible to have a new release before the new patch hits the servers?
The addition of Rift Herald is quite noteworthy, and that means we can start collecting its data since the start of the patch.

Hey - should be putting out a release once I finish the tournament API soon (tm). Probably right after the holidays.

That would be great!

Riot doesn't provide patchnotes and API changes before they go live, so I guess there is no chance to release a new version before the patch hits the servers?!
But if one follows the PBE cycle maybe one can guess the new changes.

Yeah,
But they are following something close to a 6 weeks release cycle, so we
know when we can expect a new patch.
I guess the time estimate Rob did is fine!
I have the code ready and tested, I just need the release before the
deployment so not to much too much with virtualenvs on the servers

new release is finally done


List area inside another list area not working

This example produces the second list not working:
list area 1

    list-item11
    list-item12
    ...

    list area 2
        list-item21
        list-item22
        ...
    list area 2 end

list area 1 end

Just check this codepen

Sorry, but List.js doesn't not support nested lists. :cry:


MQTT won't initialize.

Home Assistant release (hass --version):
0.34.4
Python release (python3 --version):
3.4.1
Component/platform:
MQTT
Description of problem:
Nothing has changed in regards to my MQTT set up but after upgrading to 34.4, it no longer initializes.
Traceback (if applicable):
16-12-16 02:27:57 homeassistant.components.mqtt: Can't connect to the broker. Please check your settings and the broker itself
Traceback (most recent call last):
  File "/srv/hass/hass_venv/lib/python3.4/site-packages/homeassistant/components/mqtt/__init__.py", line 261, in setup
    birth_message)
  File "/srv/hass/hass_venv/lib/python3.4/site-packages/homeassistant/components/mqtt/__init__.py", line 350, in __init__
    self._mqttc.connect(broker, port, keepalive)
  File "/home/hass/.homeassistant/deps/paho/mqtt/client.py", line 686, in connect
    return self.reconnect()
  File "/home/hass/.homeassistant/deps/paho/mqtt/client.py", line 808, in reconnect
    sock = socket.create_connection((self._host, self._port), source_address=(self._bind_address, 0))
  File "/usr/lib/python3.4/socket.py", line 509, in create_connection
    raise err
  File "/usr/lib/python3.4/socket.py", line 500, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused
16-12-16 02:27:57 homeassistant.bootstrap: component mqtt failed to initialize

Additional info:

I successfully use MQTT with hat version. So it's not a general problem. Also Connection refused in socket.py looks more like either a network-related error (firewall?) or a not-(yet-)running mqtt server.

This is all self contained on a RPI using the AiO Script that hasn't changed. :/  The connection refused is because Hass fails in setting up the MQTT server. :(

My guess is that this is related to https://github.com/home-assistant/fabric-home-assistant/issues/32

After some outstanding chatroom troubleshooting, it looked like my DB got corrupted somehow.
sudo mv /var/lib/mosquitto/mosquitto.db /var/lib/mosquitto/mosquitto.db.bak
and then restarting the pi fixed it.

This seemed to happen again.  not sure if this is something that should be built into the component or not.


iOS webkit crash on libgif-js:load after multiple GIF files

Calling the libgif load() function on multiple gif files will eventually crash iOS webkit based browsers like chrome or safari.
This occurs in both buzzfeed/libgif-js and talkr-app/gif-talkr, and is likeley a bug in webkit's canvas implementation.  You can't reproduce the problem on desktop browsers, or Android, and the number of gif files loaded (12 or so) is not sufficient to cause out-of memory problems on iOS.  Reproduced on iOS 10.3.3 and iOS 11 Beta.
On chrome, you get "Aw Snap", while one safari, you get "A problem occurred with this webpage so it was reloaded".  This crashes the javascript console/debugger, so it is very tricky to troubleshoot.
There isn't much to do about this other than hope Apple fixes this issue, and keep the number of active, loaded SuperGif objects to a minimum.  Below is some HTML that reproduces the issue:
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">

<head>
    <script src="./libgif.js"></script>
</head>

<body>
    <div id="container">
    </div>
</body>
<script>
var container = document.getElementById("container")

var gif = "https://i.imgur.com/eBzTN0U.gif"
var supArray = []

function addSup(onLoaded) {
    imgElement = document.createElement('img');
    imgElement.src = gif;
    container.appendChild(imgElement);
    var sup = new SuperGif({
        gif: imgElement
    })
    supArray.push(sup);

    sup.load(function() {
        onLoaded();
    })
}

function onSUPLoaded() {
    if (supArray.length < 40) {
        addSup(onSUPLoaded)
    }
}
addSup(onSUPLoaded);
</script>

</html>

webkit has strict requirements for memory usage and will crash when using too much memory.  The above case simply consumes memory until this happens.
Calling sup.destroy() when you are finished using a SuperGif object will internally pause all looping animations, allowing the frame buffer to be cleared with garbage collection.  It will also immediately set the frames array to [] to immediately free the memory.
Note:  Deleting a parent HTML element is not sufficient for freeing SuperGif memory. You must call destroy() first.

here is the safari bug https://bugs.webkit.org/show_bug.cgi?id=172533


Users log-in details

Hi
Is it possible to have separate log-ins to YOURLS for different users?

Duplicate #1416


Added Reference URL to stats modal

@kenancasey all should be good now to review, squash, and merge


Build Failed after linking

I got this error when running react-native run-ios
CompileC /Users/Applica/Sites/applica_jobs/applicajobs-frontend/ios/build/Build/Intermediates/RNVectorIcons.build/Debug-iphonesimulator/RNVectorIcons.build/Objects-normal/x86_64/RNVectorIconsManager.o RNVectorIconsManager/RNVectorIconsManager.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)

I received the same error after linking on ios

@BrianGorter078 lets wait for them to help us.

Provide versions of library and RN you're using.

@Eldelshell "react-native": "0.39.2", "react-native-vector-icons": "^4.0.0"

You need to upgrade RN to 0.40 or use the previous version of react-native-vector-icons 3.0.0

Ok will try, Ive also changed some imports in the ios code and than it worked. But I dont think that is the desired behaviour

@Eldelshell thank you so much!


MNT: Re-rendered with conda-smithy 2.1.1

To fix #2, which I originally and incorrectly attempted in #3.

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

@ocefpaf, I think I finally got this resolved.
Moving forward, should I await your/somebody's approval before merging (even though I have been granted merge permission)?
Thanks again for your patience and sorry again for my myriad mistakes.  I've now learned the ropes and so hopefully future builds will be relatively seamless.

@ocefpaf https://github.com/ocefpaf, I think I finally got this
resolved.
Awesome!
Moving forward, should I await your/somebody's approval before merging
(even though I have been granted merge permission)?
It is fair game to self merge in the feedstock. But feel free to bug me for
reviews if you are in doubt.
Thanks again for your patience and sorry again for my myriad mistakes. I've
now learned the ropes and so hopefully future builds will be relatively
seamless.
I see those as a documentation problem on our side and not your fault. With
that said feel free to write down thar experience as a doc page for
conda-forge ; -p

I see those as a documentation problem on our side and not your fault. With
that said feel free to write down thar experience as a doc page for
conda-forge ; -p

Ok, I will try to make some time to do this.  Thanks again.


dataclass not recognized in fields function when it is not applied as decorator

Environment data

Language Server version: 2023.9.10
OS and version: Windows 10
Python version: 3.10.9 Anaconda

Repro Steps
Code Snippet
from dataclasses import dataclass, fields

class Test:
    ...

Test = dataclass(Test)

f = fields(Test) # -> Error!

Screenshot:

Error message:
Argument of type "type[Test]" cannot be assigned to parameter "class_or_instance" of type "DataclassInstance | type[DataclassInstance]" in function "fields"
  Type "type[Test]" cannot be assigned to type "DataclassInstance | type[DataclassInstance]"
    "__dataclass_fields__" is not present
    "Test" is incompatible with protocol "DataclassInstance"
    Type "type[Test]" cannot be assigned to type "type[DataclassInstance]"
      "__dataclass_fields__" is not present

Expected behavior
The same as using dataclass as decorator (raises no error).
from dataclasses import dataclass, fields

@dataclass
class Test:
    ...

f = fields(Test) # -> No Error

The handling of dataclass within a static analysis tool requires significant special-casing because some parameters of the dataclass must be determined statically. For example, passing init=False or frozen=True, the type checker needs interpret those values and treat the decorated class differently. It uses this information to simulate all of the "magic" that is performed at runtime by the dataclass function. This "magic" synthesizes a bunch of attributes and methods for the decorated class, but the exact behavior depends on the parameters that are used. For these reasons, you must use the dataclass call as a class decorator if you want it to work with static type checkers.
If you don't want to use static type checking, you can disable it by setting python.analysis.typeCheckingMode to "off", and you will not see any type-related errors.
Note that the dataclass function modifies the original class, so if you use Test1 = dataclass(Test2), the Test2 class will be modified, and a copy of the modified class will be passed back and assigned to Test1. That means the result of applying the class decorator directly to the class versus calling dataclass(Test) after the class has been defined is the same.


Reduce memory usage of Analyzer test suite

Change Description
Reduce memory usage of Presidio Analyzer test suite (excluding Azure AI language recognizer tests) down to ~6GB from ~8GB. When run locally the tests runtime went from ~60 seconds to ~25 seconds.
Memray memory usgae graphs
Before

After

Changes

Removed duplicate nlp.load call on each iteration of Stanza and transformers recognizer tests.
Paramatized test_when_analyze_with_unsupported_language_must_match tests to avoid calling AnalyzerEngine five times in single test
Mocked nlp engine in test_when_analyze_with_unsupported_language_must_match as test doesn't require nlp model to be loaded
Removed unused fixtures - pytest-unused-fixtures plugin used to detect them

Issue reference
This PR fixes issue #1427
Checklist

[x] I have reviewed the contribution guidelines
[x] I have signed the CLA (if required)
[x] My code includes unit tests
[x] All unit tests and lint checks pass locally
[ ] My PR contains documentation updates / additions if required

@microsoft-github-policy-service agree

Thanks!

/azp run

/azp run

/azp run


Implement new collective page sections data structure

As discussed in https://github.com/opencollective/opencollective/issues/3698, particularly this comment https://github.com/opencollective/opencollective/issues/3698#issuecomment-735729599, we need to:

implement the new data structure for the collective page section categories/sections/widgets

settings: {
  collectivePage: {
    sections: [
      { type: 'SECTION', section: 'GOALS' }, // a.k.a. a 'widget'
      {
        type: 'SECTION_CATEGORY',
        category: 'CONNECT',
        sections: [
          { section: CONVERSATIONS, isEnabledSettingKey: 'features.conversations.enabled' },
          { section: UPDATES, isEnabledSettingKey: 'features.updates.enabled' },
        ]
      }
  ]
}

and migrate the current settings for collectives to this structure. This includes considering that some collectives have differing information about what is enabled/disabled, i.e. Conversations in settings.features.conversations vs settings.collectivePage.sections...

I'm gonna start by introducing a convertSectionsToNewFormat helper in the frontend to make sure we can start working with them before everything is migrated.

Base implementation done in https://github.com/opencollective/opencollective-frontend/pull/5497. Added support for this new format for editing profile page in https://github.com/opencollective/opencollective-frontend/pull/5505 and implemented for the collective page + navbar in https://github.com/opencollective/opencollective-frontend/pull/5530


crashes NodeRed is directory is not available in proxy options

Just got this message in logs:
`[error] [alexa-remote-account:alexa] Error: EISDIR: illegal operation on a directory, read
at Server.setupListenHandle [as _listen2] (net.js:1270:14)
at listenInCluster (net.js:1318:12)
at Server.listen (net.js:1405:7)
at Function.listen (/data/node_modules/express/lib/application.js:618:24)
at Object.initAmazonProxy (/data/node_modules/alexa-cookie2/lib/proxy.js:303:22)
at Object.generateAlexaCookie (/data/node_modules/alexa-cookie2/alexa-cookie.js:337:21)
at AlexaRemote.generateCookie (/data/node_modules/node-red-contrib-alexa-remote2/tools/tools.js:11:26)
at getCookie (/data/node_modules/alexa-remote2/alexa-remote.js:98:22)
at AlexaRemote.init (/data/node_modules/alexa-remote2/alexa-remote.js:133:9)
at AlexaRemoteAccountNode._initAlexaFromObject (/data/node_modules/node-red-contrib-alexa-remote2/nodes/alexa-remote-account.js:109:15)
at fs.readFile (/data/node_modules/node-red-contrib-alexa-remote2/nodes/alexa-remote-account.js:192:38)
at FSReqWrap.readFileAfterClose [as oncomplete] (internal/fs/read_file_context.js:37:12)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: node $NODE_OPTIONS node_modules/node-red/red.js -v $FLOWS "--userDir" "/data"
npm ERR! Exit status 1`
I had to comment out line 192 in /data/node_modules/node-red-contrib-alexa-remote2/nodes/alexa-remote-account.js to get NR up and running as I use Docker and safe mode is a problem ;-)

Thanks for the report!
I'm not familiar with docker and its safe mode.
I don't understand why fs.readFile would throw?
And doesn't the stack trace show that fs.readFile was successful?
I'm confused.

Hi - it isnt related to docker, its just that with docker it isnt possible to start NodeRed without running the flow (ie the new safe mode).
I tried to setup the Proxy without understanding what I was doing :).... It didnot have read access to the directory (I think) so it errored out and crashed NodeRed. What happens if you use a directory that doesnt exist on normal NodeRed?.
I'm using the legacy username and password method at the moment and would like to change to rpxy. Is there a how-to?

If i don't have read access or it doesn't exist it calls node.warn(...) so i get a [warn] in the console and the debug tab.
There is a tutorial in the readme for the proxy method.
I'm don't know how i can prevent the crash you encounter. I can not reproduce it...

ok, got a little further using a different configuration. if I setup a docker using bridge mode so that it gets its own IP (say <IP_ADDRESS>) but my web browser on my machine (<IP_ADDRESS>), I cannot use localhost:3456. If I edit the url to change it to the server ip <IP_ADDRESS>, if keeps changing back to localhost.
I think that this proxy (at the moment) will only work for local installations.....

I have tested the proxy method before from different devices in the local network and that worked.
Where does it keep changing back to localhost? Are you running at least v2.2.0? You have to enter <IP_ADDRESS> in the This IP field.

Thanks, silly mistake by me. Will try and document how to crash it.... I managed to get the temporary login working (This IP field!) but cant get it to store the data so it is lost on redeploy.
The File Path field - does this need filename and directory or just the path?.
I think the problem I have is with directory permissions due to the docker technology.
many thanks

The path is to a file for storage, so a relative alexa-cookie.json or absolute /home/pi/alexa-cookie.json.
You can also save the cookie in your own way as described in the Alexa Init node info like this for example:
[{"id":"323608dd.6ee4c8","type":"inject","z":"ba312d1.b9842d","name":"","topic":"","payload":"alexa","payloadType":"flow","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":120,"y":80,"wires":[["50b011bc.1d8b1"]]},{"id":"415d7558.7fbcfc","type":"change","z":"ba312d1.b9842d","name":"","rules":[{"t":"set","p":"alexa","pt":"flow","to":"payload","tot":"msg"}],"action":"","property":"","from":"","to":"","reg":false,"x":410,"y":80,"wires":[["afc17e7c.70e288"]]},{"id":"afc17e7c.70e288","type":"debug","z":"ba312d1.b9842d","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"false","x":570,"y":80,"wires":[]},{"id":"50b011bc.1d8b1","type":"alexa-remote-init","z":"ba312d1.b9842d","name":"","account":"","x":260,"y":80,"wires":[["415d7558.7fbcfc"]]}]

There is also a thread about saving the cookie yourself in this issue.

thanks - will switch to the other thread!


parseMultiServerAddress is deprecated... but ssb-ref uses it?

I got the deprecated api used: ssb-ref.parseMultiServerAddress message recently and looked at the stack, only to find that the offending package was ssb-ref 😢
in ssb-ref:
type() calls isInvite() calls isMultiServerInvite() calls parseMultiServerAddress()
should this function be marked as deprecated if the package itself is using it? if the goal is to warn others from calling parseMultiServerAddress, one idea is to keep the deprecation warning on parseMultiServerAddress, but have parseMultiServerAddress call _parseMultiServerAddress (move the logic there), and _parseMultiServerAddress can be called internally without warning.

one idea is to keep the deprecation warning on parseMultiServerAddress, but have parseMultiServerAddress call _parseMultiServerAddress (move the logic there), and _parseMultiServerAddress can be called internally without warning.

I think that's a great idea, I'd merge that PR. :tada:

Opened #29

Resolved by #29!


[sentry_flutter] sentry_flutter only one version in 'pub.dev'

sentry_flutter versions: https://pub.dev/packages/sentry_flutter/versions
Only 1 version: 4.0.0-alpha.2. Alpha meaning not safe.I can't use safe 3.x version
Compare dio versions: https://pub.dev/packages/dio/versions
Dio has all versions.

@chvin sentry_flutter is a new package and it's built on top of the https://pub.dev/packages/sentry/versions package, which does have more versions.
our docs https://docs.sentry.io/platforms/flutter/ mention that this is a Prerelease.
so feel free to wait a bit more and use once it's GA.


LogisticsGoodsReceiptById: quantities

Enumerations for quantities.context in  LogisticsGoodsReceiptById should only be Shipped.
Remove enumerations Confirmed, Invoiced, Ordered and Produced.

Done, see commit 611a38d.


ENH: make BcolzMinuteBarWriter.write take iterable

Updates the BcolzMinuteBarWriter.write api to allow users to pass their
data as a stream instead of requiring that they loop over their data
externally. This matches the API presented by BcolzDailyBarWriter.

Coverage increased (+0.005%) to 84.204% when pulling b24a02791651b0ab5f7a98adf478aa49d56b38de on minute-bar-write into 842c47b32861ca5d264ab5f0b2c7031aa3b27906 on master.

@ehebert @llllllllll is this still something we're planning on merging?

oh, I thought this was merged, I would still like to merge this. I can rebase and merge this later assuming this still looks good.

This code is on master, it was merged in the wiki loader pr.


How to exactly use response compression with the isolated worker model?

What version of .NET does your existing project use?
.NET 6
What version of .NET are you attempting to target?
.NET 8
Description
I have a hard time figuring out how to add response compression to the new isolated worker model. Can someone please explain?
Project configuration and dependencies
No response
Link to a repository that reproduces the issue
No response

I would also love to know how to do this.  context.Services.AddResponseCompression is there on ConfigureFunctionsWebApplication, but there's no equivalent to the ASP.NET 8 app.UseResponseCompression.

@fabiocav  would be glad if you could give some guidance or direct it to someone who can.  currently I only managed to make it explizit using this code snippet: https://gist.github.com/cajuncoding/a4a7b590986fd848b5040da83979796c


Depreciation

'sendAsynchronousRequest(_:queue:completionHandler:)' was deprecated in iOS 9.0: Use [NSURLSession dataTaskWithRequest:completionHandler:] (see NSURLSession.h
Could you please update your code since this is deprecated for a while now? I have tried it my own, but couldn't get it running...
Errors like that slow down the building process in big Xcode projects.

Fixed with last commit.


error indexing three concatenated files

    Hi Matt,

Another issue that is probably related:
echo "line 1" | gzip > concat.gz
echo "line 2" | gzip >> concat.gz
echo "line 1" | gzip >> concat.gz
zindex concat.gz --regex 'line ([0-9]+)' --numeric --unique
ERROR: Failed to index line 3: 'line 1' - Error handling index match '1' - constraint failed
Again, this only happens for concatenated gzip files.
Thanks!
René

Op 4 sep. 2017, om 16:54 heeft Matt Godbolt<EMAIL_ADDRESS>het volgende geschreven:
Thanks for the report! I'll take a look.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mattgodbolt/zindex/issues/30#issuecomment-326981555, or mute the thread https://github.com/notifications/unsubscribe-auth/AAca2RqglXsvcrB3BLiC9r8lmnrSy_3Hks5sfA8_gaJpZM4PL12m.

Either drop the --unique or echo "line 3" instead! This is working as intended (amazingly!)


Build pipeline failed with azure

Hi,
Thanks for sharing. I have tried to setup the build pipeline in azure with azure container registery and get below error:
Step 6/19 : RUN poetry install
 ---> Running in f06e7d5e44e6

  RuntimeError

  Poetry could not find a pyproject.toml file in /backend or its parents

  at /usr/local/lib/python3.10/site-packages/poetry/core/factory.py:369 in locate
      365│             if poetry_file.exists():
      366│                 return poetry_file
      367│
      368│         else:
    → 369│             raise RuntimeError(
      370│                 "Poetry could not find a pyproject.toml file in {} or its parents".format(
      371│                     cwd
      372│                 )
      373│             )
The command '/bin/sh -c poetry install' returned a non-zero code: 1
##[error]The command '/bin/sh -c poetry install' returned a non-zero code: 1
##[error]The process '/usr/bin/docker' failed with exit code 1
Finishing: Build and push an image to container registry

My azure-pipelines.yml file looks like this ( I haven't figure out how to add the unit tests and deploy to the tasks yet):
trigger:
- main

resources:
- repo: self

variables:
  # Container registry service connection established during pipeline creation
  dockerRegistryServiceConnection: 'xxxxxx'
  imageRepository: 'someimg'
  containerRegistry: 'somereporeg.azurecr.io'
  dockerfilePath: '$(Build.SourcesDirectory)/deploy/docker/Dockerfile'
  tag: '$(Build.BuildId)'

  # Agent VM image name
  vmImageName: 'ubuntu-latest'

stages:
- stage: Build
  displayName: Build and push stage
  jobs:
  - job: Build
    displayName: Build
    pool:
      vmImage: $(vmImageName)
    steps:
    - task: Docker@2
      displayName: Build and push an image to container registry
      inputs:
        command: buildAndPush
        repository: $(imageRepository)
        dockerfile: $(dockerfilePath)
        containerRegistry: $(dockerRegistryServiceConnection)
        tags: |
          $(tag)

any reason why?
Thank you

Hi,
It looks like while trying to build the image, the context is missing (docker can't find the Python files and the poetry config file), I believe adding the 'buildContext' argument, pointing to the correct path, should do the job. Give it a try:
trigger:
- main

variables:
  # Container registry service connection established during pipeline creation
  dockerRegistryServiceConnection: 'xxxxxx'
  imageRepository: 'someimg'
  containerRegistry: 'somereporeg.azurecr.io'
  dockerfilePath: '$(Build.SourcesDirectory)/deploy/docker/Dockerfile'
  tag: '$(Build.BuildId)'

  # Agent VM image name
  vmImageName: 'ubuntu-latest'

stages:
- stage: Build
  displayName: Build and push stage
  jobs:
  - job: Build
    displayName: Build
    pool:
      vmImage: $(vmImageName)
    steps:
    - task: Docker@2
      displayName: Build and push an image to container registry
      inputs:
        command: buildAndPush
        repository: $(imageRepository)
        dockerfile: $(dockerfilePath)
        containerRegistry: $(dockerRegistryServiceConnection)
        buildContext: $(Build.SourcesDirectory)
        tags: |
          $(tag)

Additionally:
As you are using the Azure Container Registry, you'll also need to log into it to push you images there, before your buildAndPush task, you should add the login task, and reference your service connection.
steps:
- task: Docker@2
  displayName: Login to ACR
  inputs:
    command: login
    containerRegistry: dockerRegistryServiceConnection1

Hi, It looks like while trying to build the image, the context is missing (docker can't find the Python files and the poetry config file), I believe adding the 'buildContext' argument, pointing to the correct path, should do the job. Give it a try:
trigger:
- main

variables:
  # Container registry service connection established during pipeline creation
  dockerRegistryServiceConnection: 'xxxxxx'
  imageRepository: 'someimg'
  containerRegistry: 'somereporeg.azurecr.io'
  dockerfilePath: '$(Build.SourcesDirectory)/deploy/docker/Dockerfile'
  tag: '$(Build.BuildId)'

  # Agent VM image name
  vmImageName: 'ubuntu-latest'

stages:
- stage: Build
  displayName: Build and push stage
  jobs:
  - job: Build
    displayName: Build
    pool:
      vmImage: $(vmImageName)
    steps:
    - task: Docker@2
      displayName: Build and push an image to container registry
      inputs:
        command: buildAndPush
        repository: $(imageRepository)
        dockerfile: $(dockerfilePath)
        containerRegistry: $(dockerRegistryServiceConnection)
        buildContext: $(Build.SourcesDirectory)
        tags: |
          $(tag)

Thanks I have tried, but still got the same error.
Installing collected packages: webencodings, pylev, ptyprocess, msgpack, lockfile, distlib, urllib3, tomlkit, six, shellingham, pyparsing, pycparser, poetry-core, platformdirs, pkginfo, pexpect, pastel, jeepney, idna, filelock, crashtest, charset-normalizer, certifi, cachy, virtualenv, requests, packaging, html5lib, clikit, cffi, requests-toolbelt, cryptography, cleo, cachecontrol, SecretStorage, keyring, poetry
Successfully installed SecretStorage-3.3.2 cachecontrol-0.12.11 cachy-0.3.0 certifi-2022.6.15 cffi-1.15.1 charset-normalizer-2.1.0 cleo-0.8.1 clikit-0.6.2 crashtest-0.3.1 cryptography-37.0.4 distlib-0.3.5 filelock-3.7.1 html5lib-1.1 idna-3.3 jeepney-0.8.0 keyring-23.7.0 lockfile-0.12.2 msgpack-1.0.4 packaging-20.9 pastel-0.2.1 pexpect-4.8.0 pkginfo-1.8.3 platformdirs-2.5.2 poetry-1.1.14 poetry-core-1.0.8 ptyprocess-0.7.0 pycparser-2.21 pylev-1.4.0 pyparsing-3.0.9 requests-2.28.1 requests-toolbelt-0.9.1 shellingham-1.4.0 six-1.16.0 tomlkit-0.11.1 urllib3-1.26.10 virtualenv-20.15.1 webencodings-0.5.1
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
WARNING: You are using pip version 22.0.4; however, version 22.2 is available.
You should consider upgrading via the '/usr/local/bin/python -m pip install --upgrade pip' command.
Removing intermediate container 47cff53bc6e3
 ---> 76fcaf310cd6
Step 6/19 : RUN poetry install
 ---> Running in 0e0b879b32b4

  RuntimeError

  Poetry could not find a pyproject.toml file in /backend or its parents

  at /usr/local/lib/python3.10/site-packages/poetry/core/factory.py:369 in locate
      365│             if poetry_file.exists():
      366│                 return poetry_file
      367│
      368│         else:
    → 369│             raise RuntimeError(
      370│                 "Poetry could not find a pyproject.toml file in {} or its parents".format(
      371│                     cwd
      372│                 )
      373│             )
The command '/bin/sh -c poetry install' returned a non-zero code: 1
##[error]The command '/bin/sh -c poetry install' returned a non-zero code: 1
##[error]The process '/usr/bin/docker' failed with exit code 1
Finishing: Build and push an image to container registry

Below is the content of the Dockerfile:
FROM python:3.10.4-slim-bullseye

RUN mkdir backend
WORKDIR /backend
COPY . /backend
RUN pip install poetry
RUN poetry install
EXPOSE 80
ENTRYPOINT ["poetry","run","uvicorn","api.main_api:app","--host","<IP_ADDRESS>", "--port", "80"]

Additionally: As you are using the Azure Container Registry, you'll also need to log into it to push you images there, before your buildAndPush task, you should add the login task, and reference your service connection.
steps:
- task: Docker@2
  displayName: Login to ACR
  inputs:
    command: login
    containerRegistry: dockerRegistryServiceConnection1

I'm using "Azure DevOps Organization" with service connections configured to connect to my azure subscription and container registry. do I still need to add the login step?
Thanks again!

This is the project file structure


Dynamically change xlim/ylim on the fly?

Is there a way to dynamically adjust the xlim/ylim on the fly like this:
https://www.reddit.com/r/dataisbeautiful/comments/ickvfq/oc_two_thousand_years_of_global_temperatures_in/

I wrote my own package for this:
https://github.com/ahuang11/ahlive


Custom Validation Inside DataGrid Inplace Edit

In EditForm we have EditContext that is possible to have custom validation by deriving from ComponentBase, But in the DataGrid we don't have a wrapping Form so we should use Validation Controls. So my question is, Is there any way to do custom validation inside DataGrid Inplace edit controls?

RadzenDataGrid injects an EditContext too. Perhaps you can use it with CascadingParameter.


ValueError: Invalid values for outdated_data [enginedefinitions], must be a subset of [enginedefinition, configuration, sanitization]

Language: python
MDCore Server version: 4.19.2
API version: 4.18.0
Making a call to file_analysis_get in the Analysis Api from the SDK can throw an error if engines are out of date. Looking at the source for analysis_result.py, that is where the error is generated. The Server is responding with an unexpected value (an extra 's' at the end of enginedefinitions).
The openapi spec specifies that the correct expected value should be enginedefintion, meaning the server is not following the published API.

I see this was fixed in commit  ef87f0f1db35b23d3c93438a9807edbfc08da2ff. Closing.


Where is jquery.peekabar.min.js.map

Hello
I get this when using the script
"DevTools failed to load SourceMap: Could not load content for https://www.example.com/templates/path/to/js/jquery.peekabar.min.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE"
what am I doing wrong?
I m loading the files normally
/templates/path/to/js/jquery.peekabar.min.js
/templates/path/to/css/jquery.peekabar.min.css
thanks

Hello @campones,
Thanks for using this library. In the source files you downloaded, there should be a jquery.peekabar.min.js.map file as well. If you put that in the same js folder, you should not get this warning.

Yes I saw it now. I downloaded this a very long time ago .. I have version 1.0.1
And as I am not familiar with NPM I do not know how to upgrade. but this is okay for me.
I saw the file and will put it on  my server
thanks for your work, it's great


connection to 'ws://localhost:8000/eel?page=main.html' failed: Could not decode a text frame as UTF-8.

Eel version
0.12.2
Describe the bug
Occasionally loses WebSocket connection because of this error message connection to 'ws://localhost:8000/eel?page=main.html' failed: Could not decode a text frame as UTF-8.. This is happening when sending pretty large arrays to the front end
To Reproduce
send (1024x256) multidimensional array from python backend to the javascript front end like so  eel.sendData(arr) until the error is produced (will not happen first time, seems to be random)
Expected behavior
Websocket does not close because of encoding error
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: windows
Browser Chrome

By the way this appears to be caused by the javascript side calling eel before the last eel call has completed. I solved it by implementing a mutex on the javascript front end but Im sure there is a better way to do this in eel backend

I am running into this same issue. Could you describe more how you implemented the mutex on the JavaScript front end?

I'll look into this


Proovread Output

Hello Author,
I have tried using proovread for one of our low coverage pacbio data. My question is related to proovread output. When I submit pacbio reads ( Number of pacbio reads = 5889). From the application, I get
pb-001.parameter.log
pb-001.ignored.tsv
pb-001.untrimmed.fq
pb-001.chim.tsv
5630181 Feb 21 23:14 pb-001.trimmed.fq
pb-001.trimmed.fa
I counted the Number of reads for pb-001.untrimmed.fq and pb-001.trimmed.fq.
For: pb-001.untrimmed.fq
Total No of reads = 4815
For: pb-001.trimmed.fq
Total No of reads = 2607
If I understand correctly both of the files have corrected reads. However the No of reads vary and the N50 vary quiet significantly. My question would be, which file of reads should I use for downstream analysis for example: denovo Assembly or Gap filling.
Thanks.

Untrimmed reads comprise uncorrected (regions that did not have
any/enough Illumina reads mapped) and corrected regions
u: uncorrected, C: corrected
untrimmed read:
uuuuCCCCCCCuuuuCCCCCCCCCCCCCCuuuu
results in trimmed reads:
CCCCCCC
CCCCCCCCCCCCCC
You will have to choose, which one to use based on application: Since
uncorrected regions are likely to have > 20% error, you should use
trimmed reads for assembly (which is error sensitive). If you want to
scaffold / map reads to identify structural differences, untrimmed reads
might work as well, yet uncorrected regions might also be chimeric
joins, so using trimmed reads is always safer.
Trimming chops up reads in smaller parts, hence N50 drops can be
considerable - cutting a read in the middle halves the N50, although it
might just have removed a couple of bases
Getting less reads in trimmed than untrimmed output is worrisome. I
suspect, that you loose a lot of data during trimming (too short pieces
<500bp are also removed). This suggests that correction does not work
well on your data. If you can send me the complete log and some more
information about your project, I might be able to give you some more
advice. You can contact me directly by email if you like (see profile
for address)
Cheers
Thomas
On 22.02.2016 16:07, rakeshponnala wrote:

Hello Author,
I have tried using proovread for one of our low coverage pacbio data. My
question is related to proovread output. When I submit pacbio reads (
Number of pacbio reads = 5889). From the application, I get
pb-001.parameter.log
pb-001.ignored.tsv
pb-001.untrimmed.fq
pb-001.chim.tsv
5630181 Feb 21 23:14 pb-001.trimmed.fq
pb-001.trimmed.fa
I counted the Number of reads for pb-001.untrimmed.fq and
pb-001.trimmed.fq.
For: pb-001.untrimmed.fq
Total No of reads = 4815
For: pb-001.trimmed.fq
Total No of reads = 2607
If I understand correctly both of the files have corrected reads.
However the No of reads vary and the N50 vary quiet significantly. My
question would be, which file of reads should I use for downstream
analysis for example: denovo Assembly or Gap filling.
Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/BioInf-Wuerzburg/proovread/issues/45.

Thanks Thomas. I have emailed you the log file.


Update Horizons

Releases are still building so will restart travis jobs when they're ready.

Closing in favour of https://github.com/stellar/docker-stellar-core-horizon/pull/79


Eliminate a reference cycle caused by the Spitfire placeholder_cache.

Uses weakref.proxy() when adding methods to the cache in order to avoid reference cycles.

Added a test and then added a fix for the issue you found.

I'm curious about the performance impact of this change as resolve_placeholder is usually a hot function. Are there any benchmarks?

I ran the benchmark scripts a few times and didn't notice any glaring differences.

I see. Does it include methods specifically? Maybe it's not be worth caching self.methods at all - if a getattr() is faster than an isinstance()+weakref().


Add WinGet configuration

Added .configurations/configuration.dsc.yaml
Update README.md

[x] I have signed the Contributor License Agreement.
[ ] This pull request is related to an issue.

Microsoft Reviewers: Open in CodeFlow

/azp run


vaclient gives misleading error if server not running

Hi,
I have ubuntu 18.04. I followed the get started doc and ran ./build.sh , i verified the image was pulled successfully.
when i start using vaclient i get below issue, the command issued was "vaclient list-pipelines" "vaclient list-models" but getting below error
WARNING: Published ports are discarded when using host network mode
HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /pipelines (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fda55d4a490>: Failed to establish a new connection: [Errno 111] Connection refused'))
Got empty response retrieving pipelines
request help on this to resolve and move forward

Make sure the REST running is running before you run vaclient. The vaclient error message is not very helpful - we plan to improve this in our next release.

Fixed in v0.6.0-beta


composition can be seeded with multiple arguments

Related to https://github.com/rackt/redux/issues/632#issuecomment-157361578
This allows us to provide our composition with multiple arguments.
It does so by pulling off the last function from our list, calling it with the provided args, and using the result as the initialValue for reduceRight (which we then call on the remaining functions). Unfortunately, this does make our simple one liner a little more complex
Let me know if you see a cleaner approach.

Hey @Bjvanminnen, If you need a multiple-arg compose in your app could you not use lodash's compose?

I could, and am fine with doing so. However, it also seems reasonable to expect the compose provided by redux to support this functionality.

Watch out for an empty function list. The way compose is written in this PR, compose()(x) will throw an error.
Currently redux's compose()(x) returns x, lodash's, _.compose()(x, y) returns x.

I would argue that neither redux nor lodash's current behavior really makes sense. I can't think of any good reason one would call compose with no functions, but if you did, I would think it would make more sense for the result to be () => undefined instead of x => x or (x, y) => x.
That said, I understand the value in not changing existing behavior where there's not a clearly better behavior. I will update my PR and add a test for this, hopefully later this evening.

Latest commit ensures that we don't change existing behavior for case where no funcs are passed.

Stopped trying to do this as a one-liner, and I think it makes it a lot easier to follow.

Would this be a breaking change?

It shouldn't be.
If given no funcs, we return the first argument, same as before.
If given funcs and a single argument, behavior should be identical to before.
Only new behavior is that if given multiple arguments, we pass all of them to the last func instead of ignoring the additional arguments.

Out in 3.0.6, thanks!


Any way to stop/abort reading a file in the eachLine method?

I need to stop reading a file once it gets to certain line & just stop whatever else is going on in the eachLine part. How can I do that? I've read through the docs and searched on the internet but found nothing that works. Please tell me it's possible to do

Can't you just return false? The docs state: If the callback returns false, reading will stop and the file will be closed.


Add images/diagrams to the Camera Tutorial

I think the tutorial about the Camera would benefit from diagrams / images showing objects in some 2d-plot.
This is quite some effort, so I'd say this is low-priority (you can label it as such).
( Camera Wiki )

@EtienneBruines I looked at your tutorial, this is awesome work!!!

:+1: for @EtienneBruines
On Wed, Oct 28, 2015 at 6:59 AM Mateusz Szczap<EMAIL_ADDRESS>wrote:

@EtienneBruines https://github.com/EtienneBruines I looked at your
tutorial, this is awesome work!!!
—
Reply to this email directly or view it on GitHub
https://github.com/paked/engi/issues/31#issuecomment-151625683.

We no longer have a "camera" tutorial.

We do, but it's outdated. See https://github.com/EngoEngine/engoengine.github.io/issues/17


Update the REST-based libraries user agent

This uses new functionality in GAX, hence the project.json changes.
Fixes #355.

LGTM


Consul 0.6.4 Support

Version bump.

Feel free to merge https://github.com/vkhatri/chef-consul-ng/pull/15 prior to get passing tests :)

@CpuID Sorry for the delay to get to this PR.
Could you please rebase with the master? PR now has conflicts after merging PR 13.

@vkhatri I'll fix the failures shortly

@vkhatri tests pass, good for review again.

@CpuID 👍


Add --fields option  to pass onto rbw list

First off, thanks for both rbw and rofi-rbw. I was looking to integrate Bitwarden with Rofi, but the official NodeJS client was much slower than I wanted.
This PR's title is self-explanatory I think.

Hey, thanks for the PR! :)
If I read this right, this will show additional fields in the list and not just the folder and the name. Do you think we should do some additional formatting to make this nicer to read?

Hi @davidatbu , are you still working on this? If not, I'd like to play with formatting the fields.

Thinking about this change, I'm wondering what exactly the use case for it is. Is it to distinguish between similarly named accounts? Or is it because you want to search for usernames or URLs?

Not sure if you're still reading this, but I had another go at this.
First off, rbw doesn't allow many fields in list - apart from name and folder, there are only username and id. I'd argue that id is not interesting at all for rofi-rbw.
So, we're left with possibly showing the username. I can see some use cases where that would be helpful. I implemented a draft in https://github.com/fdw/rofi-rbw/tree/show-user , where the username is shown in a table-like view. It's not too complicated (but not too pretty, either). What do you think?
In regards to making this configurable: As the only possible field is user, I'm not sure a flag would be much help. It would certainly add complexity, but would it clutter that much?


Signal to stop a container.

Allow to set the signal to stop a container in docker run. Use --stop-signal with docker-run to set the default signal the container will use to exit.
Add STOP command to the builder to allow image creator to set the default exit signal for their programs.

TODO:

[x] Tests
[x] More docs
[x] Shell completions

Signed-off-by: David Calavera<EMAIL_ADDRESS>

Seems ok to me.

Not sure we should be checking for proper signals on this.  Since the signal table seems to be often out of date.

Not sure we should be checking for proper signals on this. Since the signal table seems to be often out of date.

I think we could support both. I agree that we might not include all the signals in our table, but remembering a signal name is easier than remembering a number in some cases. I think we should check if the flag is a number and pass it through in that case.

Ok, as long as we support numbers.

Design LGTM.

I added some tests here and fixed the documentation. Please, take a look.

Any plans to support setting a default for this at the image level?

(Especially since when packaging software that needs a non-standard "stop signal", the person packaging it knows better than the users do what that signal might be. :smile:)

@tianon there are plans for that, not in this PR though.

LGTM from a Windows perspective.

@tianon I've added the stop command to the builder in a second command here. Please, take a look.

Can you add a test for testing the actual stop signal (with it not being SIGTERM)?

@cpuguy83 I added several tests that don't use SIGTERM already, are you looking for something else more specific?

@cpuguy83 I've extracted the logic to choose the signal to a function, that way we can just use unit tests to make sure we send the right signal without complicating the integration much. :smile_cat:

Can we add a test with a trap inside shell script just make sure some signal sending logic change doesn't break this.

@tonistiigi test added

Please, take a look

The Dockerfile change is a bit weird... I get @tianon's point, but can also end up being quite surprising unless the image setting that is the main image.
Also aren't we putting a moratorium on adding new Dockerfile syntax?

IMO, if we don't have a way to define this property at the image level,
then the usefulness of it is diminished quite a bit, since the user
experience will have to be something like "to be able to stop this
container properly, you need to append this extra flag when you run it".

@cpuguy83 we had some discussion with @crosbymichael and @icecrime about this, and we agreed that this is more useful if it's at the image level, as @tianon mentions.

LGTM but needs a rebase

rebased

LGTM :+1: :heart:

Before merging, docs may need some extra information on what formats are supported (names signals, numbers) -

@thaJeztah I've added some info about the valid formats.

@calavera thanks! sorry I didn't notice earlier, looks like you didn't update the CLI docs;

https://github.com/docker/docker/blob/master/docs/reference/commandline/create.md
https://github.com/docker/docker/blob/master/docs/reference/commandline/run.md

(For the commandline/run.md file, you can add the same description you used in builder.md)

@thaJeztah I've changed those docs. Thanks for checking.

PS: failures in CI completely unrelated.

Hm, forgot about these; the new option also needs to be added to;
https://github.com/docker/docker/blob/master/docs/reference/api/docker_remote_api_v1.21.md#create-a-container
(In the JSON example, and the list of JSON options)
And in the JSON output example of https://github.com/docker/docker/blob/master/docs/reference/api/docker_remote_api_v1.21.md#inspect-a-container
Almost there! :heart:

@thaJeztah I've added those missing fields to the api already.

I've added those missing fields to the api already.

Hm, did  I really scroll over them yesterday? (picks up his hat of eternal shame)
The option is missing from the list of JSON parameters though; somewhere around here; files#diff-3acbcaa129d9a6972e140d1b4c32cc27R253
Sorry for being a PITA (I heard mayor improvements are coming to the API docs :innocent:)
LGTM otherwise, thanks @calavera

The option is missing from the list of JSON parameters though;

@thaJeztah fixed. Thanks for looking.

LGTM, thanks!
ping @SvenDowideit for review :)

@SvenDowideit, @moxiegirl Please, take a look.

@calavera given the addition to the Dockerfile syntax, does this need wider approval before merging? I saw some concerns raised on the build-ARGS PR (which was approved before the freeze on builder)

@thaJeztah we already went through that

LGTM

Nice one!

Can I use a custom command replace signal ?
for example use ./bin/solr stop to stop a solr container

You could docker exec into the container to execute this.
docker exec CONTAINID solr stop

Using the atomic command you can add a STOP label which would execute this command

@rhatdan yes,I know I can use exec to do it. Docker has a default start command,why not have a default stop command?
How can I use STOP label ?Can you give me a example?Thanks

@loveai88 Docker starts a process, it interacts with that process by using standard unix signals.
I'd recommend using something like runit (or even a custom bash script, really) which allow you to specify these things.

atomic command is a wrapper around docker that uses LABEL embedded in the container image to tell it how to start, stop, install, uninstall, scan, verify ... the images.
https://github.com/projectatomic/atomic
Here are the generic Labels that atomic command and others can use.
https://github.com/projectatomic/ContainerApplicationGenericLabels
Basically you could add a label in your dockerfile.
LABEL STOP docker exec ${NAME} solr stop
Then users could execute
atomic stop CONTAINERID
atomic would execute this command.

Similarly if you wanted a special

@rhatdan :+1:


Add A(H)BCD tabs to Compendium Browser

This PR adds Ancestries, Heritages, Backgrounds, Classes, and Deities to the compendium browser and additionally redirects the search buttons of the Character page on the Character sheet to the corresponding compendium browser tabs. This allows A(H)BCD content that is stored in compendiums to be loaded into the compendium browser to browse and filter during character building instead of searching through a myriad of compendia to find available content.
There is one issue that I ran into that I don't know how to solve. src/module/apps/compendium-browser/index.ts throws errors in the Tagify section starting at line 642. I tried a myriad of fixes but couldn't figure it out - this happened after removing the traits filter from the new A(H)BCD tabs. I don't know why this is happening, I thought the objectHasKey method in the if statement at line 631 would handle this, but it appears not.
I realize this is an extremely large PR and probably has a lot of places where the code can be optimized and/or doesn't follow the system's standards, so take as much time as necessary to review. There's no rush on this, just a project I wanted to work on.

There are several failing CI checks found in the Details link here in the PR. Please take a look at those and resolve them

Fixed the linting issue.
Also... maybe fixed the Tagify errors? Seems kind of janky to me but maybe it's the right way, I don't know.

I'm seeing errors when I enable a module that contains compendium packs with valid items inside of it (class, feats, spells, etc) and enable those packs in the settings of the compendium browser. I will make a new world and test more slowly. If you want, you may install PF2E Lifeblade and/or PF2E Companion Compendia to try to reproduce the issue.
Here's a copy of the error from the console:
pf2e.mjs:1341 TypeError: Cannot read properties of undefined (reading 'toString')
    at CompendiumBrowserClassTab.loadData (pf2e.mjs:1676:75999)
    at async CompendiumBrowserClassTab.init (pf2e.mjs:1676:50897)
    at async CompendiumBrowser.loadTab (pf2e.mjs:1676:112592)
    at async HTMLFormElement.sheetHandler (pf2e.mjs:1341:441966)

Screenshot?

Here are some of it working as expected (without any 3rd party content):

It works if the compendium only has one type of item in it (i.e. only classes, only backgrounds, etc). I saw that Lifeblade has effects as well as classes, etc. in them (I'm having trouble downloading Companion Compendia right now... probably a github issue so I can't test, but I'm guessing it probably is the same way). I thought this was a design philosophy to have a unique compendium for each item type. As soon as I added an effect to my class compendium, I got the same error as you. I didn't test with mixed compendia!

This commit should fix the error you're seeing, and I added a warning if any of the index fields are missing to be more in line with the other tabs.

This commit should fix the error you're seeing, and I added a warning if any of the index fields are missing to be more in line with the other tabs.

It seems as though that fixed everything except Classes from being added as packs in settings. I tested on a new world. If you...

open browser dev tools
enable either of the modules above
open the compendium browser
go to settings
enable the checkbox for either module under Classes
click save

this will throw an error to the console. It appears to be the same error but from a different line number. So, I assume we made it past the earlier calls for ancestry, background, etc and now we're stuck on a later call to deal with Classes. Everything else can be enabled in the settings and shows up fine.
Thank you for all of your work on this!

 I can successfully add packs from various modules and everything seems to come up normally and filter correctly! Huzzah!
Thank you for your work on this, @mysurvive 😄

for posterity

Upon further testing, I see some deities missing from the browser. Some examples: Aakriti, Abraxas, Adanye. Opening the pf2e.deities pack next to the browser is a good way to see what's missing and what was found. I double-checked that the browser wasn't excluding sources (in settings and in filters).

I found the problem - I will get a commit in later tonight or tomorrow to fix it.

Unfortunately, I'm not a fan of how many entries are there at the top. I don't know how to tackle it though. One idea I had was to create different contexts somehow, and have bestiary/hazards into a completely separate context, but I don't think that'd be enough.

Would a tab for "Character Building" be a better option, with tabs for A(H)BCD (like the settings tab has)? I have a commit that I've been working on that does that, but it's not working yet.
I think this is an important enhancement for the compendium browser, so I'm willing to keep pecking at it until it's in a good place.

Does this look better @CarlosFdez ?

Fixed the CSS too

I don't think we're going this route. It appears the tide is more turning towards rewriting it in svelte, and using special sheets for PC side stuff: https://github.com/foundryvtt/pf2e/pull/16896
As part of whatever inevitable rewrite it'll get, I would personally like it if we split actors from items since the use case is different, but that's a road we'll cross when we get there.
I do like the sub-tab for ABCD items though, and I'll pitch that as part of the rewrite. Thank you for your work here, as it did at spark discussion on what we want and don't want.


hexo is not defined

ReferenceError: hexo is not defined
    at Object.<anonymous> (/Users/hainuo/hexoblog/node_modules/hexo-tag-asset-res/lib/renderAsset.js:4:13)
......

when i run hexo s or hexo d -g   ,it show this

I am just trying it for the first time and I am having this problem as well ..


Compute hinted class hash

I see in the code that compute_hinted_class_hash is being calculated as the keccak of this string r#"{"abi": contract_class.abi, "program": contract_class.program}"#.
Should it be changed to reflect the actual information of abi and program of the json artifact

Hi! We just fixed it in: https://github.com/lambdaclass/starknet_in_rust/pull/681

Yes, it is changed but I think in the abi section the struct is missing state_mutability property

Yes, it is changed but I think in the abi section the struct is missing state_mutability property

Along with the cairo-vm program, we are also storing the compiled program in a serde_json::Value, which has the stateMutability field in  each object of the ABI, so it should be there on the deserialization. But let us know if you realize that is not right. Thanks!
@marioiordanov

@SantiagoPittella with this (https://gist.github.com/marioiordanov/646683d1fe073e1682fbed5794c168dc) Cairo 0 contract the pythonic Cairo lang produces the following hash 0x4d07e40e93398ed3c76981e72dd1fd22557a78ce36c0515f679e27f0bb5bc5f, but compute_deprecated_class_hash gives this one 0x5b5e230e0920e1a18ed8c4360388f042a98b91914f774d7b2cac2633303646d

you can take a look at this implementation https://github.com/0xSpaceShard/starknet-devnet-rs/blob/2dfcf6c5f5b76338836b5767bc00ed8079796be7/crates/types/src/contract_class.rs#L106

@SantiagoPittella @juanbono I saw your changes to the ContractClass compute_deprecated_class_hash. The problem is that you rely on starknet_api and in its ABI they don't take in mind the stateMutability field

Hi! I added a test for the class hash that you mention and looks like it's working. Are you using an specific version of starknet_in_rust or main?
The tests can be found here:
https://github.com/lambdaclass/starknet_in_rust/pull/801

@SantiagoPittella with this (https://gist.github.com/marioiordanov/646683d1fe073e1682fbed5794c168dc) Cairo 0 contract the pythonic Cairo lang produces the following hash 0x4d07e40e93398ed3c76981e72dd1fd22557a78ce36c0515f679e27f0bb5bc5f, but compute_deprecated_class_hash gives this one 0x5b5e230e0920e1a18ed8c4360388f042a98b91914f774d7b2cac2633303646d

Regarding this:
I look in mainnet for the class_hash 0x4d07e40e93398ed3c76981e72dd1fd22557a78ce36c0515f679e27f0bb5bc5f and used the json from the mainnet:
curl -s https://alpha-mainnet.starknet.io/feeder_gateway/get_class_by_hash\?classHash\=0x04d07e40e93398ed3c76981e72dd1fd22557a78ce36c0515f679e27f0bb5bc5f

And our class_hash calculation worked correctly.
With the one from your link didn't. But it was different from the one from mainnet.


public is always added to constants despite the visibility_required rule

Bug report
It seems visibility_required always adds public to constants in spite of a set rule.
Code to Format
<?php
class Test {
    const FOO = 'bar';
}

with the following rule
    'visibility_required' => [
        ['elements' => ['method', 'property']],  // `const` is omitted
    ],

Actual Result
<?php

class Test
{
    public const FOO = 'bar';
}

Expected Result
<?php

class Test
{
    const FOO = 'bar';
}

Environment

PHP 7.4.19
PHP 8.0.9
PHP CS Fixer 3.6.0 Roe Deer by Fabien Potencier and Dariusz Ruminski.

Hi and thanks for reporting,
I cannot replicate the issue on latest master, however I did notice that your configuration is a bit off.
The option is wrapped in a second array, I think it should be:
    'visibility_required' =>
        ['elements' => ['method', 'property']],  // `const` is omitted
    ,

Can you update the config and re-run the tool with -vvv added and share the output?

Indeed, the posted rule had an error but even though it is corrected as follows, still I get the same result.
    'visibility_required' => [
        'elements' => ['method', 'property'],
    ],

Sorry, it is working fine. I was merging the rule with the default options and it was the cause.

Thanks for letting us know :+1:   : )


Installing stuck.

Hello
When I try to bash . /setup  im stuck on ngrok step
[~] Instaling ngrok ..
[~] Successfuly installed
[~] Inflating ngrok
And nothing happened .  Just stuck in there
I'm using termux non rooted device
I have already installed  ngrok and using ath key in it

Hello
When I try to bash . /setup im stuck on ngrok step
[~] Instaling ngrok ..
[~] Successfuly installed
[~] Inflating ngrok
And nothing happened . Just stuck in there
I'm using termux non rooted device
I have already installed ngrok and using ath key in it

stiil facing same issue??

run :

bash -x setup

provide me screenshot of it.

It's solved  thank you for helping

Same error how can i fix this??

Same error how can i fix this??

and you provide a screenshot?

Comment it on #27
"Dear users,
Shark is already enhanced for its installation please check your environment, is up-to-date?
If not check our readme Android section.
Thanking You
@Bhaviktutorials


index.js code not being injected into index.html

I am inheriting a CRA app (not ejected) and when I run npm start, the web server starts, the code is processed and shows errors/warnings in log if any exist, but the page that pops up only serves the index.html as it is in the public folder without any <script> tags added to it. The project is using "react-scripts": "3.4.1".
A brand new CRA app works just fine, so not sure what is causing this issue for my project. Would anyone be able to guide me in troubleshooting this?

I should note that npm run build works just fine. The index.html file the the build folder does include the bundled code.
I also tried ejecting and running npm run start', same issue. The index.html` file is served as is without the included javascript app.

Compare whats different between your code and a new CRA app.
by the way, this is probably better served on StackOverflow

As far as the index.html and react-scripts versions, nothing is different. Not sure what else to compare since its react-scripts that doing the heavy lifting. I'll close this issue and try StackOverflow. Thanks for the suggestion.


Subscribe to the Low-Code / No-Code Service in SAP BTP

Tutorials: https://developers.sap.com/tutorials/appgyver-subscribe-service.html
Hi, experts I have tried to find Low-code services in booster as well as service market place in my free BTP account<EMAIL_ADDRESS>but it does not exists. Does it not available with Free Trial? Please guide.

Hi @SMG1985 - this service is only available with an active SAP BTP EU10 account with entitlements, as noted in the prerequisites:

Access to an SAP BTP account in EU10 with Low-Code / No-Code entitlements

While this means that you can't complete the mission with a free account, you can still create applications (and follow most of the tutorials) by signing up for free at www.appgyver.com. The layout and features in this edition are very similar, though there may be slight changes to the location of buttons and the colour scheme.

Thank you for your quick response and guidance.
On Thu 18 Nov, 2021, 21:54 Tom Beck, @.***> wrote:

Hi @SMG1985 https://github.com/SMG1985 - this service is only available
with an active SAP BTP EU10 account with entitlements, as noted in the
prerequisites:

Access to an SAP BTP account in EU10 with Low-Code / No-Code
entitlements

While this means that you can't complete the mission with a free account,
you can still create applications (and follow most of the tutorials) by
signing up for free at www.appgyver.com. The layout and features in this
edition are very similar, though there may be slight changes to the
location of buttons and the colour scheme.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SAPDocuments/Tutorials/issues/7221#issuecomment-973025159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUKVLJTGNZLXO5UHGSQODLLUMUSEBANCNFSM5IJHZFTQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Left click on tray icon should do something

As the title says, left clicking on the tray icon does nothing. At least on (Arch) linux. Right click shows the context menu and you can then hide/show the main window just fine, like quitting the application, but a one-click action would be faster, if the main window is hidden, one (or two) click shows it and brings it to front, and vice versa, if it is shown, then one left click on the tray icon should hide it.
Speaking about the context menu, it would be nice to disable the uneeded option, i.e: hide option disabled when the main window is hidden, and show option hidden when the window is opened, or bring it to front.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

+1

+1

+1

The tray icon changes depending on the operation system, but on Windows if you left click the app will always open if not already, if you right click you get a menu with quit and show/hide (depending of if its open or not).
On mac you just get the menu show/hide & quit if you left or right click.
Is there anything else that is needed, or could be improved?

hey @alexbrazier take a look on https://github.com/RocketChat/Rocket.Chat.Electron/pull/544, I added some extra validation, but it already works just fine.
on Mac/Linux there's no difference between left or right clicks, it always will pop the context menu


[Question] - How should we change the MaterialDesignTheme?

Sry for leaving an Issue, I could not find an email adress.
The MessageBox does not react to changes in Primary oder Accent Color of MaterialDesign. So how can we get the MessageBox to adopt the current theming?

I actually implemented a Solution. If you would like to give me permission I could open a Pull Request :)


Add Visual Studio 2019 themes with semantic colors

Adding color themes that show off the semantic highlighting and closely match Visual Studio 2019 editor colors.

@ryanbrandenburg would it make sense to include Razor's semantic goodness here?

My current understand is that we will not need to make edits to themes to support Razor Semantic Colorization, we should be able to just use the default scopes.

@ryanbrandenburg @NTaylorMullen What we found was that colors were not defined for the scopes we were using which caused VS Code to fall back to the textmate grammar scopes. So to better show off the semantic colors we wanted to include themes that really made it apparent.


Epa air quality index automation changes made with manifest file.

Epa air quality index automation changes made with manifest file.

Can you add a README for this individual import in the subdirectory?

Can you add a README for this individual import in the subdirectory?

I have added the Readme file[1] individually for AirQualityIndex
[1] /data/scripts/us_epa/airqualityindex/README.md


Added ctrl + g to group visuals

You can use cntrl + G to group visuals. This is very helpful for creating bookmarks and keeping various elements on the canvas grouped together

@pawarbi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@pawarbi - thank you very much for your contribution! I have verified this keyboard shortcut and approved your changes.

@kfollis : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Instructions for enabling stdout logs no longer work

The instructions for enabling the stdout logs that appears on this page are no longer followable in the Azure Portal.  The first three steps involve a convoluted method of reaching Kudu that includes a round-about method that doesn't work anymore.  I offer the following instructions as a replacement:

Open the Azure Portal for your web app.
Search at the top of the menu on the left side of the page.  Type ‘kudu’ into the search box.
Click on Advanced Tools, then click the word 'Go ->' to open the Kudu diagnostic console.
From the menu at the top, select 'Debug Console', and then CMD
Navigate to ‘site\wwwroot’ and find the web.config file.  (Usually at the bottom of the list.)
Click the pencil icon next to the web.config file.
Set stdoutLogEnabled to true and change the stdoutLogFile path to: \?%home%\LogFiles\stdout.
Select Save to save the updated web.config file.
Make a request to the app.
Return to the Kudu diagnostic console (Steps 1 -4 above)
Select the LogFiles folder and select the pencil icon for the stdout log with the latest modification date.

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

ID: 4a2a7104-1d14-2e8a-0984-d92aa311c108
Version Independent ID: 5e3c0546-c579-afb9-5289-b14532c127eb
Content: Troubleshoot ASP.NET Core on Azure App Service and IIS
Content Source: aspnetcore/test/troubleshoot-azure-iis.md
Product: aspnet-core
Technology: aspnetcore-test
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@rhamlett  your instructions look good but changing to the  stdoutLogFile path to: \?%home%\LogFiles\stdout didn't produce a log file.
cc @davidebbo

That's a typo, it should be \\?\%home%\LogFiles\stdout (two slashes). This is the same as the current instructions at the Kudu level. Seems what changed is the Azure portal's Diagnose UI which broke the early parts of the instructions. Those instructions were too complex anyway, and what @rhamlett suggests is simpler.
cc @suwatch as I'm no longer on the team.


Add strong validation via pydantic for all events

WIP:

[x] Fix typing
[ ] Add test coverage

There are a lot of typing issues to work through but wanted to put this out there for initial review. We have strong typing for all event data which could be used in the future to validate events as they come in. I also learned a lot about the relationship between dataclasses, pydantic, and TypedDict's, and the conclusion I have come to is that we should use pydantic everywhere as a replacement for those classes because we will get stronger typing capabilities and won't lose any features

We don't need Python 3.8 but I don't think we can assume Python 3.9.2, just Python 3.9.0.

@MartinHjelmare it seems that all of the missed coverage is related to importing TypedDict from typing instead of typing_extension. But shouldn't our 3.8 tests hit that code path?
This PR is theoretically ready, but we should probably do some extensive testing against an actual server to make sure our event schemas make sense because validation is now running in the path of event handling. If we don't want to do that/de-risk this PR, I can remove that validation step, but the advantage of it is that we will catch mistyping in our models. I hit multiple typing bugs that I was able to fix by running the tests with event validation in place

I have been running this on my production instance for the last 24+ hours and everything seems to be OK. Because I am testing this against production, I am unable to test all the inclusion/exclusion events, but given that we have test coverage for every event type I feel pretty good about the schemas created here


tare calibration API

im trying to get the tare calibration to work. im using rs2_run_tare_calibration API with latest firmware and sdk;however, I always get calibration table which points to NULL. I was wondering if anyone tried this API and whether im missing some steps here.
according to the error::handle function one or more parameters are invalid. please note that im using default json by passing nullptr. same for callback functions
``
std::future<const rs2_raw_data_buffer*> RealSenseDevice::startCalibration(const float ground_truth_mm, const void *jsonContent,int contentSize,rs2_update_progress_callback_ptr progressCallback, const int timeout) {
auto future = std::async(std::launch::async,this,ground_truth_mm,jsonContent,contentSize,progressCallback,timeout->const rs2_raw_data_buffer*{
rs2_error* e = nullptr;
auto calibrationTable = rs2_run_tare_calibration(device.get().get(),20000,nullptr,0,nullptr,nullptr,timeout*10,&e);
rs2::error::handle(e);
if(calibrationTable!= nullptr){
Log::Debug("SUCCESSSSSSSSSSSSSSSSSS!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!") << Log::Endl;
} else
Log::Debug("FAILEDDDDDDDDDDDDDDDDDDDDDDDDDDDDD!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!") << Log::Endl;
return calibrationTable;
});
return future;
}

Hi @eliranno
This API is being used in the Viewer, and, indirectly, from a python example I believe.
You are using modern C++ in your App, why are you calling the C-API? (I'm just curious)
Are you getting any error from the call? I assume you are running it during streaming? Did you manage to run self-cal successfully?

Hi Surgey,
Thank you for your reply.
Yes, my application is written in modern c++. I have tried using both C API
and also cast the rs2_device to auto_calibeated_device and then call the
tare function. Both failed to execute. Is there a better way of calling it
? Also, I was told that in order to run the tare calibration the device
configuration must be set to 90fsp which I have not tried yet.
Thanks for your help
On Sat, Jan 11, 2020, 9:44 AM Sergey Dorodnicov<EMAIL_ADDRESS>wrote:

Hi @eliranno https://github.com/eliranno
This API is being used in the Viewer
https://github.com/IntelRealSense/librealsense/blob/master/common/on-chip-calib.cpp#L319-L323,
and, indirectly, from a python example I believe.
You are using modern C++ in your App, why are you calling the C-API? (I'm
just curious)
Are you getting any error from the call? I assume you are running it
during streaming? Did you manage to run self-cal successfully?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/5620?email_source=notifications&email_token=ABR4T4B2ECPOUM2BENGXREDQ5HLLRA5CNFSM4KFHCX5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIWDMGA#issuecomment-573322776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABR4T4HPHLXM45DAYAGHLETQ5HLLRANCNFSM4KFHCX5A
.

@eliranno What's the error details when you get execution failure? Could you please provide more logs or snapshots? Thanks!

@eliranno Could you please provide more details if you still need support on this? Thanks!

@eliranno Could you please update if you still need support on this? Thanks!

@eliranno Could you please update? Thanks!

@eliranno Sorry we didn't get response from your side for weeks. Will close the issue at this point. Please feel free to create new one if you still have issues or questions. Thanks!


Refactor the code for upload widgets.

@shubhamdhama  FYI.

I've updated the PR, but I'm still working on further possibilities.
@timabbott @shubhamdhama FYI.

This is great, merged, thanks @pragatiagrawal31!  A few notes on things that could be further cleaned up:

Rather than having the {{theme_mode}}-settings class on the button, we could just use a .closest("realm-logo-section"), so that only the outer ID differs between the two widgets.
I think the whole upload_realm_icon code path could likely also be deduplicated with upload_ream_logo in a similar way (just with an additional case in the big conditionals).
I'm less sure about this, but I think ultimately we could just have an image_upload_widget.js code and template, and share that logic and code with the user's personal avatar as well.  That would likely be significantly more messy on the CSS/template end, since there's fairly different styling, but the JS logic should be possible to deduplicate (importing the better UX we have on the avatar of the "click and widget" and "X icon" rather than big upload buttons in the process).  Something to work towards.

Related issue #11612


WIP: Release cert-manager v1.8.0

Follow up to the test release v1.8.0-rc1

https://github.com/k8s-operatorhub/community-operators/pull/1232
https://github.com/redhat-openshift-ecosystem/community-operators-prod/pull/1215
https://operatorhub.io/operator/cert-manager/stable/cert-manager.v1.8.0
https://artifacthub.io/packages/olm/community-operators/cert-manager/1.8.0

@jahrlin Please take a look at this. I've simply removed the -rc1 suffix and published again to operatorhub.io and redhat-operator hub (see PR links in the description).
Without the -rc1 suffix, the build scripts automatically add this release to both the candidate and stable release channels.


[Issue-2416] Update balance service

Related issue(s)

#2416

Is your feature request related to a problem(s)? Please describe.

Update balance service

Describe the solution you'd like

Move all balance functions from subscription, state to balance service
Optimize balance service and resolve some problems
Handle case cached data (balance and earning)

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-2777--sw-web-runner.netlify.app