how to  translate in ts file

tree.component.html:
*ngIf="node.SpaceType !== 'building' && node.SpaceType !== 'portfolio' && node.SpaceType !== 'meter' && node.SpaceType !== 'facility' && node.SpaceType !== 'floor' && node.SpaceType !== 'points'  else test">{{'abc.'+ node?.Name|translate}}
<ng-template #test>{{node?.Name}}
<span *ngIf="node.SpaceType === 'points'" class="point-gripper-icon">

In html file I have applied for ngif to translate which is translating  but I want to write a translate code in ts file and use that in html. How can I ???
i have a two conditions ..                                                                                1) i want to translate when commodity == 'Weather' or 'electricity' or 'steam'.                                    2) i want to translate when spacetype !== 'building' or 'portfolio' or 'meter' or 'points'

This should be on stackoverflow, not in this repository, I suggest you to close this.


trinary: Add canonical-data.json

Added data for trinary exercise.
Included 3 tests around invalid trinary strings.  This is basically a union of all invalid tests that I saw across different tracks and it matches with the exercise description.
This addresses this issue on the todos repo: https://github.com/exercism/todo/issues/158

Ehm, isn't trinary a deprecated exercise? See #279.

Ha!
I just pulled it for ruby yesterday (with a fetch).  After doing the exercise and realizing that I'd also done the Java version, I thought i might as well do the canonical.
So once those are "deprecated", how do we remove them from the existing tracks?
What about adding a DEPRECATED.md file or something in the x-common repo as an indicator?
Either way, just trying to help out.
I'll close this guy.
https://media.giphy.com/media/yyp58Yx3Qk0TK/giphy.gif

I suppose we should close exercism/todo#158 if this is going to be unused

Oh, you didn't need to close it yet. I'll keep it open until @petertseng or @kytrinyx has something to say about.

UNCLOSE!

I take the same position as https://github.com/exercism/x-common/pull/296#issuecomment-234810532 - I don't oppose its addition. I recommend not taking my word as having any more authority than any other track maintainer though! =D (I say that because other commitments prevent me from pushing a cross-track initiative to have all tracks deprecate their existing base-conversion exercises. If there is such an individual pushing the initiative, that individual should recommend that this file not be added!)

So once those are "deprecated", how do we remove them from the existing tracks?

All tracks that have added the all-your-base exercise should add the other base-conversion exercises to the deprecated section of the track config file. This stops exercism fetch from fetching the exercise, unless specifically requested to do so by specifying it. A month and a half ago I made that list of tracks that had alerady done so: https://github.com/exercism/x-common/issues/279#issuecomment-264938769
Global deprecation is performed by adding a .deprecated file in the directory in x-common. Someone will need to remind me what that does. https://github.com/exercism/x-api/issues/139 says it means it doesn't show up in todo lists, please confirm whether that's true, and how it interacts with the per-track deprecation (if it does at all...!)
It will be worthwhile, however, to state the impending deprecation in the todo issues for the remaining two, hexadecimal and octal, so that it is known that other problems can take priority. I have done so.

how much of these should be "what we think the canonical test suite should look like" and how much of it is "what the majority of tests say the canonical test suite is right now"

I think of it as the former. In the investigation, we may discover that the majority test suite is a good one, in which case the two will be the same, but we would generally like to take the opportunity to make sure that the test suite is one we are happy with when we add the tests to this repo.
I know sometimes I want to argue "hey listen, I just submitted this PR to show what the current tests look like. I'm really not interested in making any changes y'all want to suggest. I just want this PR to be over with.", but I do think it is part of the work involved in doing these.
This may be important enough to document.

EBWODP: Clarifying that my "commitments prevent me from pushing a cross-track initiative to have all tracks deprecate their existing base-conversion exercises" are time commitments, not any other sort of commitment (in case anyone was wondering whether it was a moral commitment, a monetary commitment(???), etc.)

Thanks @petertseng . I guess I'll give it a little time.  We can see If @kytrinyx weighs in.  Though based on the comment you pointed to, it sounds like we're leaning towards merging it in, and letting the deprecation (which happens later) take care of dumping it, right?  Is that what I heard?
Interestingly, after reading comments from @Insti here, and above, I want to rewrite the whole thing so that the test descriptions make more sense and maybe there are fewer tests because what are we testing really with all those cases..
But if the exercise is deprecated, then I think... who cares ;)
I'm happy to close this one and leave it unfinished or whatever.
Just let me know.

Unless you really want to, I'd leave this one as it is and not worry too much about improving it. There are many other things that would be a better use of your time.

I'm happy to leave this alone.  And if it gets merged, fine.  If not, fine.

I think based on the above discussion: (1) let's not worry about improving it, but also (2) let's not throw it away. Also, eventually, we should probably take the time to figure out exactly what work needs to happen to properly deprecate everything. Are we missing issues notifying all the various tracks about it?


support integration with strangelove-ventures/ibctest

Should be relatively straight forward to package up atomkraft in a docker container and pass in arguments for it to connect to target nodes that were configured/deployed by https://github.com/strangelove-ventures/ibctest

Thanks! We are planning time this quarter to look into potential integration; @jackzampolin we'll ping you as soon as we learn enough of the IBCTest ourselves first.

@andrey-kuprianov will it be feasible in the current Atomkraft to test IBC modules E2E across two or more networks?
FYI we have been using this framework for running local networks: https://github.com/crypto-com/pystarport#ibc
We then use pytest for manual E2E tests, e.g.: https://github.com/crypto-org-chain/chain-main/blob/master/integration_tests/test_ibc.py#L27
It'd be awesome to integrate Atomkraft, but not sure how much it is currently hardcoded against a single network execution (at least there's a single chain.toml config in Atomkraft).

wow. I was not aware of crypto-com's « starport ». Thanks :pray:
In 2023, we are planning tons of useful features - and IBC is definitely one of them. So we will surely look into pystarport and see what's possible :rocket:


[Bug] BookieRackAffinityMapping is not compatible with custom BookieId

Search before asking

[X] I searched in the issues and found nothing similar.

Version
Pulsar: master
Minimal reproduce step
1.Start bookie server with config property:
bookieId=node1.
2.Then start pulsar with bookkeeper config: reppDnsResolverClass=org.apache.pulsar.zookeeper.ZkBookieRackAffinityMapping
3. The pulsar didn't parse the rack info.
Analyze:
In pulsar, the DefaultBookieAddressResolver use PulsarRegistrationClient, when DefaultBookieAddressResolver resolve, it invoke PulsarRegistrationClient#getBookieServiceInfo.
https://github.com/apache/bookkeeper/blob/8fce024989ea0427877cb8981fcb94f7afadc3c7/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/DefaultBookieAddressResolver.java#L41-L72
PulsarRegistrationClient didn't override the getBookieServiceInfo,
    default CompletableFuture<Versioned<BookieServiceInfo>> getBookieServiceInfo(BookieId bookieId) {
        try {
            BookieServiceInfo bookieServiceInfo = BookieServiceInfoUtils
                    .buildLegacyBookieServiceInfo(bookieId.toString());
            return FutureUtils.value(new Versioned<>(bookieServiceInfo, new LongVersion(-1)));
        } catch (UnknownHostException e) {
            return FutureUtils.exception(e);
        }
    }

If the param bookieId is not match format IP:Port, it will throw an exception.
It will step down to default rack info.
What did you expect to see?
The pulsar parse rack info correctly.
What did you see instead?
The pulsar parse rack info incorrectly.
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@horizonzy Looks like PR https://github.com/apache/pulsar/pull/17922 resolved this issue.


Necro class

adds a basic summon necro template for eldrich,shenk,pindle runs
-no teleport  (only walking, I have no gear to test tele)
-no trav/nihlithak support
-works best if eldritch is run first always
-tested in hell (clvl 80+)
-needs a merc to help first clear for corpse explosion

Left some comments

Reverted the default config files to match the current release. Fixed the accidentally change in bot.py
Added a bunch of stuff to improve shenk+eld runs, and more summoning setup

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f4747a4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #337   +/-   ##
=========================================
  Coverage          ?   27.65%
=========================================
  Files             ?       52
  Lines             ?     4564
  Branches          ?      742
=========================================
  Hits              ?     1262
  Misses            ?     3234
  Partials          ?       68

Continue to review full report at Codecov.

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

Rebased my changes over the latest master to make sure nothing broke, everything seemed to work as expected.


R.generated.swift for multiple targets?

I have tried all the possible thing that I could before posting here. and look into all issues in this repo but couldn't find any clues.
My project has to have multiple target for Swift 3.1 like production, staging, and etc
I've used R.swift since the project is started and it's working fine until last 2-3 days after I have added more Targets in the xcode and it's failed to build on other targets but the original one is still working
Here's the Error that I've got

I have different project LivePopKit to import into our project. That's project is just for Our API, and Model to use in this project.

But My main app is still working fine, just different target is not working. I suspect that it would because of sharing R.generated.swift for 3 different targets? but 3 different targets have exactly the same config because I did duplicated on the original one that's all. the different is just Display name and Bundle Identifier. that's it.
even I did rename my main target to other name and rename my new targets to be the same name as original.  it's still not working. They have error about in R.generated.swift about Could not infer type XXX   XXX is Project name
Do you have any recommend for me to look for?
Thank you

I think I've been able to reproduce your issue in a test project.
In your storyboard, do you have "LivePop" set as an explicit module for the TagsPageController?
If so, try checking the "Inherit From Target" setting:

It's all checked as Inherit From Target

but isn't it weird that only this storyboard that has a problem? and I did picked the module in that storyboard and try to build. it's not working also

I'm still investigated more into it. I strongly believe that it should be a problem with naming thing

Is there -> LivePop.CreateBroadcastViewController? in the generated code?
Checking the box "Inherit From Target" should make R.swift generate: -> CreateBroadcastViewController?
Are you passing any extra arguments to the rswift executable?

@tomlokhorst Thanks for your help.
I just look into all the storyboard from your guide and it's seem that some storyboard has untick the module so it's cause the problem because it's not inherit from target anymore
but I found out new problem again  :\


c8y configuration plugin implementation

Proposed changes

Implementing fs_notify crate for c8y configuration plugin.
This commit contains the changes of #1244 applied on top of it. When reviewing please review only the last commit: 8e5f1f84f440e1f589cad6117b02be34540918e3
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

#1248

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

The code is very neat, and the regression was fixed. Now I see only one thing, "whether we should remove --config-file option or not".
If yes,

The specification must be updated accordingly.

If no,

The code needs to be changed to address the case of --config-file.

Otherwise, this PR cannot get merged :/

Yes, the --config-file option can be removed and the documentation must be updated accordingly.

The clippy warning must be fixed too ;-)

removed the config-file option and made some changes to the reference


Handling a user changing the server url without necessary permissions

Rancher versions:
rancher/server: v2.0.0-rc3
user interface: v2.0.35
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
$ cat /etc/os-release
NAME="Ubuntu"
VERSION="17.10 (Artful Aardvark)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 17.10"
VERSION_ID="17.10"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=artful
UBUNTU_CODENAME=artful

$ uname -r
4.13.0-21-generic

Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
bare-metal
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Currently only has a single rancher/server
Steps to Reproduce:

Log in with user that does not have admin permissions
go to settings --> Rancher Server URL
change server url
click Save Url
Results:
Button changes to Saving, and stays like this.
in my browsers network tab I can see that the request received a 403 response.

Expected Results:
Two ways to go about it I think, both are valid:

Show a clear Forbidden message when a user that does not have the necessary permissions tries to change the server url.
Disable the Save URL button when a user that does not have the necessary permissions to change the server url goes to the settings page. Also include a clear message on why the Save URL button is disabled. Something like "Only administrator level users are allowed to change the Server Url".

Need error message that user does not have permission to save like we do for all other admin pages.

The real answer is they shouldn't see the settings menu item in the first place, but we do not have dynamic updating of the schema in place for the UI to know that they don't have access to change it yet.  So showing an error is fine as a stop-gap, but you eventually shouldn't have been able to get to this situation in the first place.

Error showing up, but the field still looks like it changed.  I think the UI should revert it back so it doesn't look like it changed at all.

Version - 2.0 master 5/14
Verified fixed


Added OnCheckedChangeListener(View,Boolean) to Checkbox and Switch

Hi, this bug is fixed, but I'm fixing more bugs to generate new version


Decide on naming convention for base classes

What
Agree on a naming convention for the base classes for our errors and components and record the decision. A few options have been floated around.
Why
As we may make these classes public, we should be thoughtful about their naming.
Who needs to work on this
Developers, Tech writers?
Who needs to review this
Developers, Tech writers
Done when

[ ] We've reviewed our options for naming these classes and picked on
[ ] We've recorded the convention in our internal docs

For reference, I summarised the current discussions in this (internal) doc: https://docs.google.com/document/d/1D8gQ4zNp3fJeywKuYibH9e0OevH1UVYV215xZ8BfFzY/edit?usp=sharing

@Izabela-16 Yup, we've settled on GOVUKFrontendError and GOVUKFrontendComponent so I'll close the issue as well 😊


Add support for in-memory database

E.g. for unit testing purposes it is often useful to be able to create an in-memory database with Room.inMemoryDatabaseBuilder. This PR adds a corresponding SpatiaRoom.inMemoryDatabaseBuilder function for doing the same with spatia-room.
Apparently in-memory databases cannot be created from an asset, so I needed to remove createFromAsset(templateDb) call and the spatia_db_template.sqlite file. Instead, I added a callback that calls SELECT InitSpatialMetaData(); to initialize the database, which should have the same effect (right?) and also removes the need for shipping this extra 5 MB file with the app.

Hi @johan12345 this is great, I'm going to test it.

I'll let you know when the release has this changes.

@johan12345 here is the new version:
implementation 'com.github.anboralabs:spatia-room:0.2.6'

Awesome, thanks!


v2.0

Description

removed deprecated
CHANGELOG updated for v2.0

edit

removed MessageEvents.disposeEngine and MessageEvents.stopLoop enums alongside _stopLoop()  and _disposeEngine() methods because were used in the old async disposeEngine(). All that work is now done by deinit().

Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[x] 📝 Documentation
[x] 🗑 Chore

Do I need to delete the fix_data.yaml file and the test_fixes directory?

No no, let's keep these. I don't know if they'll still work but it's worth having them around for a while.

In the CHANGELOG I have written the major changes since v1.2.5. Is that enough? Please, take a look if I missed something.

Makes sense to me!

I left the pre versions for reference before deleting them (if you agree to delete them).

I think we can keep those. The people who're still on 1.x will be interested in seeing the path, I think.

Tests passed on Linux, Android and Windows.

I'll try macOS and iOS once it lands.


Wrong Citation

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [o ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

Answer in according to uploaded files

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)
Window11

Versions

Mention any other details that might be useful
I upload my data to Azure Storage.
I can run Azure Chat App correctly.
But, I am disappointed with wrong answer of chatbot.
My data file is written by Korean and English.
My chatbot's answer is not consistent whenever I ask.
Sometimes chatbot answer correctly, sometimes not.
[ Open Source Modification ]

create_search_index() in predocs.py

before :
SearchableField(name="content", type="Edm.String", analyzer_name="en_microsoft"),

after :
SearchableField(name="content", type="Edm.String", analyzer_name="standard.lucene"),

query_prompt_template in chatretrieveread.py

before :
If the question is not in English, translate the question to English before generating the search query.
After :
Please search in the language of the original input of the question, never try to translate it into English.

[ Chatbot provided wrong citation ]

[  Azure Storage Explorer ]
SACC is defined in different file(*Lecture-7.pdf)

How Can I improve Azure Search Service correctness?

Thanks! We'll be in touch soon.

I'm seeing the same issue.

"What is my deductable?"

Answers correctly in the chat
But the citation is incorrect

I also have the same issue. Have anyone try to fix the problem?

@ericthomas1 which of the previous deployments?

Similar issue reported in my case

We should work on predocs functionally...

I think it's an off by one error in the pdf splitting. The pages index beginning at page 0, not page 1.

I am facing a similar issue. The previous file is always shown as citation.

Some cases can be caused because of the bug I described in #370 issue

Any update on this? is #370 the proper way to address? If so, is it because of how the chunking process works?

Still a one off issue atm. Can we expect a fix?

Still waiting for a fix! The issue of incorrect references still exists. Would be nice to know how to address, whether its with some sort of indexing or a Cognitive Search setting, etc.

Hey there, I'm facing the same issue, and it is causing a serious problem with our customers where the bot started showing wrong citations, are there any solutions for this?

@pamelafox Thanks for your response. I have a website, a few adequately named documents, and an FAQ document as references. The citations are made up and happen more frequently now, and I'm still trying to figure out why. Interestingly, the answers are mostly correct, but why is the reference wrong? I also considered deactivating it for now, but I still need to figure out how. I'll try the evaluation and let you know


Https fetching url error

Every time i refresh my novels it always says https fetching url error. I already updated the app nut its no use. I can't refresh my novels but i can download other novels. creepy.

can you open the bakatsuki website from the phone browser?
Have you tried to enable/disable Use App Keystore from Settings -> Update & notification?
please give me more information.

I having the same problem with the main and updates

I can't open up the bakatsuki website on my phone browser

I tried to uninstalling it and then reinstalling it but it still doesn't work

If you cannot open the site in your phone browser, looks like you need to upgrade your android phone.
The new site now forces HTTPS with new encryption method which not supported in old phone.
I cannot do anything about this.


[dvsim] Add a project folder name when scratch root environment variable is set

This PR solves a small issue when the SCRATCH_ROOT environment variable is set, and the
workstation has more than one copy of the repository.
Then we add the repository name to distinguish the output.
For example: the local workstation has two repositories named
opentitan1 and opentitan1, the SCRATCH_ROOT is set to ~/desktop,
then the two scratch roots are: ~/desktop/opentitan1 and
~/desktop/opentitan2.
@tjaychen I hope this solves your issue ;)
Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

I thought for different tools we use different folder names, like prim_alert-sim-dsim/ or prim_alert-lint-dsim/?

Does not seem like this meets the need, I will close it. Thanks everyone.


Sending cookies

Taken from fetch documentation:

Sending cookies
To automatically send cookies for the current domain, the credentials option must be provided:
fetch('/users', {
  credentials: 'same-origin'
})

This option makes fetch behave similar to XMLHttpRequest with regards to cookies. Otherwise, cookies won't get sent, resulting in these requests not preserving the authentication session.
Use the include value to send cookies in a cross-origin resource sharing (CORS) request.
fetch('https://example.com:1234/users', {
  credentials: 'include'
})

This functionality should be added to node-fetch. I'll explain why I need this:
I'm developing an isomoprhic application which uses react, graphql, and relay. This isomorphic application, upon client requests, renders the view, and fetches the required data using relay on the server, while following requests and renders are done on the client.
As you may or may not know, relay uses isomorphic-fetch which uses node-fetch on the server. This makes the cookies not available on my graphql endpoint.
My current solution is to set the cookies manually:
{
  headers:{
    cookie: req.headers.cookie,
  },
}

node-fetch doesn't have a built-in cookie store, so you have to manually specify where the cookies are for node-fetch to be aware of them. Since GitHub's fetch runs in the browser, it does have access to the browser cookie store. CORS of course doesn't apply either.
Yes, we do know about isomorphic-fetch.

And what about adding a cookie store? (#94)

Cookie store is a big can of worms on server-side. While your request to emulate browser is fair, it's quite likely other people will see cookie store and think of different usages.
I do know 2 scenarios where having a cookie store will help (#8 and #94), but I have been trying to leave implementation to third-party, for example, a simple solution was mentioned in #94.
I understand some developers want isomorphism, but not distinguishing client/server side creates serious dependency problems down the road (client/server implementation need to support the same set of features, on an evolving spec). So I lean more towards having familiar API but knowing the gaps between implementations.

I would really want to see the "fetch event" implemented so we can start building cool extensions to node-fetch. Eg stuff like caching, cookie store etc

While events are powerful, I don't think it's something developers are actively looking for, because people tend to 1) prefer simple http requests 2) think in "procedural terms" (which is why yield generators and async/await exists). Having to deal with events sucks, especially server-side.

Let's agree to disagree

I have to agree that events are generally a bad idea server-side. They make it difficult to scope functionality to the context of a specific request. I'd prefer to see something like Koa's middleware that allows composition of functions before/after requests, when implementing extensions.

Is this the same issue as #94?

I am pretty sure the cookie side of things are discussed in #94, and Facebook Relay never had a clear intention to give full server-side rendering support it seems.
As for OP's question, while solution is a bit ugly, it's already outlined here:

https://github.com/denvned/isomorphic-relay#how-to-use
https://github.com/denvned/isomorphic-relay/issues/49

@bitinn Just for the record, I (OP) was the one suggesting the ugly solution on denvned/isomorphic-relay#49. Later, I've posted this issue here to suggest extending node-fetch in order to avoid the not-so-ideal solution mentioned above.


Fix getting started command:toSide

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

In Python extension we use command:toSide in our walkthrough to open a Python file while user is working with the walkthrough: link.
It occupies more than half of the space and requires adjusting the column again:

Compare this to bicep extension where it opens up nicely:

They used their own command:
const document: TextDocument = await workspace.openTextDocument(uri);
return await vscode.window.showTextDocument(
    vscode.ViewColumn.Beside
);

which we cannot use in the Python extension. We have to rely on VSCode command itself: workbench.action.files.newUntitledFile, so a python file can be opened without waiting for extension to activate.
More context: https://github.com/microsoft/vscode/issues/152920
cc/ @luabud

I am not sure who owns this today, I think Logan?

I can verify this occupies half space now, although I think it should be opened to the right side like bicep as the main context is still getting started.

Verified. I agree it should be opened to the right side, @bhavyaus.

I can verify this occupies half space now, although I think it should be opened to the right side like bicep as the main context is still getting started.

Could you please open another issue for that? Thanks.


Docker image for olaper

Just jumped into this project, would be nice to have at least a quick and dirty development docker image. Will try to do one once I retake my explorations but if anyone has got one would be nice to share!

Here's an approach for setting up quickly the foodmart demo for Olaper:
https://github.com/ivansabik/docker-olaper

Thank you, Ivan
i added a link on docker image in doc


Ensure that a terminal failure actually terminates the app

This is a tentative fix for https://github.com/wellcomecollection/platform/issues/4046 and https://github.com/wellcomecollection/platform/issues/3477

It works! 😭


GLIP finetune related error: AttributeError: 'DetDataSample' object has no attribute 'text'

Hi, I have successfully organized my custom dataset and initiated training using the rtmdet model. However, upon transitioning to the GLIP model using the configuration file configs/glip/glip_atss_swin-t_fpn_dyhead_pretrain_obj365-goldg-cc3m-sub.py as base, I encountered the following issues:
Traceback (most recent call last):
  File "~/Code/mmdetection-dev-3.x/tools/train.py", line 138, in <module>
    main()
  File "~/Code/mmdetection-dev-3.x/tools/train.py", line 134, in main
    runner.train()
  File "~/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/runner/runner.py", line 1745, in train
    model = self.train_loop.run()  # type: ignore
  File "~/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/runner/loops.py", line 96, in run
    self.run_epoch()
  File "~/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/runner/loops.py", line 112, in run_epoch
    self.run_iter(idx, data_batch)
  File "~/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/runner/loops.py", line 128, in run_iter
    outputs = self.runner.model.train_step(
  File "~/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/model/base_model/base_model.py", line 114, in train_step
    losses = self._run_forward(data, mode='loss')  # type: ignore
  File "~/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/model/base_model/base_model.py", line 340, in _run_forward
    results = self(**data, mode=mode)
  File "~/anaconda3/envs/openmmlab/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "~/Code/mmdetection-dev-3.x/mmdet/models/detectors/base.py", line 92, in forward
    return self.loss(inputs, data_samples)
  File "~/Code/mmdetection-dev-3.x/mmdet/models/detectors/glip.py", line 270, in loss
    text_prompts = [
  File "~/Code/mmdetection-dev-3.x/mmdet/models/detectors/glip.py", line 271, in <listcomp>
    data_samples.text for data_samples in batch_data_samples
AttributeError: 'DetDataSample' object has no attribute 'text'

I have attempted multiple potential solutions, including transitioning to the official cat dataset referenced in this notebook. Despite these efforts, the error persists.
I would be immensely grateful for any advice or suggestions you might have regarding this issue.
Thank you very much for your assistance. I apologize if my repeated inquiries have caused any inconvenience.

@CDchenlin 请基于我们 funtune 的配置来做哦，你的配置看起来有错误

你解决了嘛，我也有一样的问题，把coco2017放进去就是不行，需要caption

你解决了嘛，我也有一样的问题，把coco2017放进去就是不行，需要caption

遇到同样问题，是因为mmengine、mmcv版本问题吗？

你解决了嘛，我也有一样的问题，把coco2017放进去就是不行，需要caption

遇到同样问题，是因为mmengine、mmcv版本问题吗？

请问解决了吗？

@CDchenlin 请基于我们 funtune 的配置来做哦，你的配置看起来有错误

groundDINO用cat的配置文件来训练提示一样的错误，改了load_from，其余没动还是不行


Dreamview+: localhost:8888 localhost refused the connection.

The Apollo compilation is complete, and Dreamview can also be launched.
./scripts/bootstrap.sh start
[INFO] Module monitor is already running - skipping.
[INFO] Module dreamview is already running - skipping.
However, when entering https://localhost:8888/ in the browser, it shows ：
"localhost refused the connection."`
How can I resolve this problem?someone can help me,Thank you!

./scripts/bootstrap.sh stop
./scripts/bootstrap.sh start


[Engsys][cosmosdb] upgrade runtime dependency priorityqueuejs to v2

Looking at the history, I don't see any functionality changes.

v2
Stop supporting component and browser builds.

but that's just removing the structure used to build and test the package in browsers using components.
Packages impacted by this PR
@azure/cosmosdb
Issues associated with this PR
#19233

API change check
API changes are not detected in this pull request.


Update docs to use the correct branch name for older rails versions

Addresses #2452

Thank you for your contribution, but we do not maintain the previous versions.

I think updating the readme is a reasonable ask here.


⚠ Bofrost.es has degraded performance

In 9cbb19c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1529 ms

Resolved: Bofrost.es performance has improved in 61e37bd after 16 minutes.


Missing documentation

Hi, this is interesting library, but not a lot of documentation. Would it be possible to get basic usage example or at least few guidelines?
For example how to load file steps so that it would be available to my features?

@aisbaa hi, thanks for reaching out. This project is not being worked on any more. There are some ideas of starting work on it again, in which case there will be documentation. Sorry it wasn't there in the first place! If you want to contribute I'd gladly merge any PR.


Replace deprecated/removed File.exists? alias

The alias File.exists? was removed in Ruby 3.2:

https://bugs.ruby-lang.org/issues/17391
https://rubyreferences.github.io/rubychanges/3.2.html#removals

File.exist? has been in Ruby since at least 1.8: https://ruby-doc.org/core-1.8.6/File.html#method-c-exist-3F

Closed in favor of https://github.com/engineyard/ey_config/pull/4.


In case of reorganized block the Bech32UtxoSet recovered to its previous state

I merged it, because I need it. I'll make the changes myself.


Mulitline console output like [1,2] doesn't scroll - cursor is off screen.

If the console output has more than one line it still only scrolls down one line.
The cursor is no longer visible and you have to manually scroll to see what you are typing.

Alright, this has been fixed on master.


Remove redundant initialization

I noticed the run command initializes the tenant, but the runForMultiple function already takes care of that which means when running tenants:run each tenant is initialized twice.
Please double check that I'm not changing something else by removing this.

Ah, nice catch. Seems like that was left over from before we had the runForMultiple() method.

Codecov Report

Merging #775 (36f5ca3) into 3.x (96d9ad1) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.x     #775      +/-   ##
============================================
- Coverage     87.33%   87.30%   -0.04%
  Complexity      383      383
============================================
  Files           103      103
  Lines          1129     1126       -3
============================================
- Hits            986      983       -3
  Misses          143      143

Impacted Files
Coverage Δ

src/Commands/Run.php
100.00% <ø> (ø)

src/Listeners/UpdateSyncedResource.php
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Missing code from pull request

Hey, I did a pull request that got merged a while ago. The following link is for the commit created after the merge.
https://github.com/express-labs/pure-react-carousel/commit/805db5de7f8143679338fc872e917f774ecbe746
It seems like some of the things added during this commit has since been removed, the things missing are the code that prevents clicks on any links in the slider while dragging as well as the dev-example of this feature and the tests.
I just wanted to make sure that this was an intended change or of this has been accidentally removed?
Best Regards,
Christoffer

Hey, I just diffed the master branch with the git commit you posted and the code and tests still seem to still be there.  Can you recheck. Also, can you point out which lines were removed so I know for sure what you mean.  Thanks!

Hey, I've switched to another library so you can can close this issue if you want to.


Please release a new version

So that I can benefit from https://github.com/onelogin/python-saml/commit/54e5be1d8bb7f769c09e5de6b9f05ad6b6fe7e51
Thank you!

I will release a new version next week, I want to add some pending features.

Sounds good, thanks.

@pitbulk any update on a new release?


Request to join the org

Please star the repo to receive an invite.

Because I want to stay on top of everything that goes on in this organization and of course, as a good dev, you have to know about the challenges of this incredible world of programming

Sorry for not replying before, I've changed my mind since. Good luck.


Saving xml documents

I'm curious if there is an alternative to XML DOM Management.SaveXMLDocumentToOutStream? If not could we get functionality to allow the custom xml document to be downloaded/saved by the user?

We provide a method XMLDocument.WriteTo(OutStream). See the documentation at:
https://msdn.microsoft.com/en-us/dynamics-nav/developer/api/xmldocument-class
https://msdn.microsoft.com/en-us/dynamics-nav/developer/api/xmldocument-writeto-outstream-method

Perfect, thanks!


WebGL support

Cannot be used normally in webgl environment.

Hey @zaixiaoqu, thanks for the report!
I've been able to reproduce here the error "Uncaught ...", although I don't get this "abnormal situation has occurred" one.
Maybe WebGL builds don't play nice with the C# Job System, we'll probably need some special handling for WebGL specifically. Shouldn't be too hard, though, running the jobs synchronously instead of async.

Ok, I think I found it. Disabling LOTTIE_THREAD_SUPPORT from the WebGL native build seems to fix the problem =]
I'll create a PR with the fix soon.

Ok, this problem should be fixed in the main branch, now tagged with 1.0.0-preview1.
Feel free to reopen this issue if the problem persists.
Cheers \o/

Thank you so much, I have already tested it, Perfect.


Change app to use eventyay.com instead of next.eventyay.com

The next version of eventyay is life. Please change the app to use eventyay.com.

Already using the new API in app, no changes required


The timeout value is modified/ignored

🐛 Summary
When the user specifies a timeout value, that value is multiplied by 0.4 inexplicably, and the user does not really get an idea that this is occuring.
To reproduce

run findcdn with a timeout variable
inspect the timeout variable at runtime in the domain query commands

Expected behavior
The timeout should be used as specified, or if it is being altered, the user should know about it.

This is fine if we want to just leave it as timeout instead of timeout * 0.4. I think this was as a means to make the timeout equal among all parts but that does not really make any sense (just going based on the comments). Setting to timeout makes the most sense. Going to include this in a PR later.


fatal error: 'mysql/mysql.h' file not found #include <mysql/mysql.h>

> ./setup.py build
checking xmake ...
xmake v2.7.8+20230406, A cross-platform build utility based on Lua
Copyright (C) 2015-present Ruki Wang, tboox.org, xmake.io
                         _
    __  ___ __  __  __ _| | ______
    \ \/ / |  \/  |/ _  | |/ / __ \
     >  <  | \__/ | /_| |   <  ___/
    /_/\_\_|_|  |_|\__ \|_|\_\____|
                         by ruki, xmake.io

    👉  Manual: https://xmake.io/#/getting_started
    🙏  Donate: https://xmake.io/#/sponsor

current python version: 3.10
BOOST_ROOT: /Users/daipei/Code/boost
BOOST_LIB: /Users/daipei/Code/boost/libs
[ 48%]: cache compiling.release hikyuu_cpp/hikyuu/data_driver/kdata/hdf5/H5KDataDriver.cpp
[ 48%]: cache compiling.release hikyuu_cpp/hikyuu/data_driver/kdata/mysql/MySQLKDataDriver.cpp
error: hikyuu_cpp/hikyuu/data_driver/kdata/mysql/../../../utilities/db_connect/mysql/MySQLStatement.h:20:10: fatal error: 'mysql/mysql.h' file not found
#include <mysql/mysql.h>
         ^~~~~~~~~~~~~~~
1 error generated.
  > in hikyuu_cpp/hikyuu/data_driver/kdata/mysql/MySQLKDataDriver.cpp
[ 36%]: cache compiling.release hikyuu_cpp/hikyuu/data_driver/kdata/hdf5/H5KDataDriver.cpp
[ 36%]: cache compiling.release hikyuu_cpp/hikyuu/data_driver/kdata/mysql/MySQLKDataDriver.cpp
error: hikyuu_cpp/hikyuu/data_driver/kdata/mysql/../../../utilities/db_connect/mysql/MySQLStatement.h:20:10: fatal error: 'mysql/mysql.h' file not found
#include <mysql/mysql.h>
         ^~~~~~~~~~~~~~~
1 error generated.
  > in hikyuu_cpp/hikyuu/data_driver/kdata/mysql/MySQLKDataDriver.cpp

brew 安装 mysqlclient


Cast values on update

Shouldn't we use this on update:
// this is from `create()`
let insertion = types.toDefinition(this, object)

I have an error trying to update with a boolean value.

@jonataswalker I've pushed a fix - are you able to pull & test to see if it fixes your problem before I publish? I did add a test that works.

Fix confirmed @citycide. Thanks.

Released in v1.0.0-rc.5.


  I was wondering how this would work with browserify.

My original thought shown below is to use browserify to create the modules, I like this idea because this doesn't populate the global namespace.

app.js

angular
     .module('MyApp', [
          require('./core/module1'),
          require('./feature/module2')
     ])
     .provider(require('./blocks/routehelper'));

routehelper.js

/* @ngInject */
module.exports = function routerHelperProvider($locationProvider, $stateProvider, $urlRouterProvider) {
     (code here)
}

The other way would be using angular modules with browserify to import them to other modules.

app.js

require('./blocks/routehelper');

angular
     .module('MyApp', [
          'blocks.router'
      ]);

./blocks/router.module.js

module.exports = (function(){
     angular.module('blocks.router', ['ui.router'])
          .provider(require('./routehelper'));
})();

might be better on StackOverflow


Exporting tinyllama-1.1b using onnxruntime bf16 crashes

System Info
System information:
Container is Debian12 (mambaorg/micromamba)
Host is RHEL9 / ppc64le
$ cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 12 (bookworm)"
NAME="Debian GNU/Linux"
VERSION_ID="12"
VERSION="12 (bookworm)"
VERSION_CODENAME=bookworm
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

$ uname -a
Linux b8e04f1032bc 5.14.0-362.18.1.el9_3.ppc64le #1 SMP Mon Jan 29 03:48:20 PST 2024 ppc64le GNU/Linux

Python, Optimum & PyTorch version:
$ python3 -V
Python 3.10.9

$ pip3 list installed | grep optimum
optimum            1.18.1

$ micromamba list pytorch
List of packages in environment: "/opt/conda"

  Name             Version  Build                  Channel
─────────────────────────────────────────────────────────────
  _pytorch_select  1.0      cpu_2                  rocketce
  pytorch-base     2.0.1    cpu_py310_pb4.21.12_1  rocketce
  pytorch-cpu      2.0.1    py310_1                rocketce

(base) root@b8e04f1032bc:/tmp# micromamba list onnx
List of packages in environment: "/opt/conda"

  Name         Version  Build                           Channel
──────────────────────────────────────────────────────────────────
  onnx         1.13.1   h25d5be3_py310_pb4.21.12_1      rocketce
  onnxruntime  1.15.1   hd867603_cpu_py310_pb4.21.12_1  rocketce

Who can help?
@michaelbenayoun @JingyaHuang @echarlaix
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)
Converting to fp32 works without issues, fp16 is not possible since I'm on a CPU only system and bf16 throws the following error:
$ optimum-cli export onnx --model TinyLlama/TinyLlama-1.1B-Chat-v1.0 tinyllama-1b_onnx/ --dtype bf16

[...]

In-place op on output of tensor.shape. See https://pytorch.org/docs/master/onnx.html#avoid-inplace-operations-when-using-tensor-shape-in-tracing-mode
================ Diagnostic Run torch.onnx.export version 2.0.1 ================
verbose: False, log level: Level.ERROR
======================= 0 NONE 0 NOTE 0 WARNING 0 ERROR ========================

Saving external data to one file...
Traceback (most recent call last):
  File "/opt/conda/bin/optimum-cli", line 8, in <module>
    sys.exit(main())
  File "/opt/conda/lib/python3.10/site-packages/optimum/commands/optimum_cli.py", line 163, in main
    service.run()
  File "/opt/conda/lib/python3.10/site-packages/optimum/commands/export/onnx.py", line 261, in run
    main_export(
  File "/opt/conda/lib/python3.10/site-packages/optimum/exporters/onnx/__main__.py", line 351, in main_export
    onnx_export_from_model(
  File "/opt/conda/lib/python3.10/site-packages/optimum/exporters/onnx/convert.py", line 1157, in onnx_export_from_model
    _, onnx_outputs = export_models(
  File "/opt/conda/lib/python3.10/site-packages/optimum/exporters/onnx/convert.py", line 768, in export_models
    export(
  File "/opt/conda/lib/python3.10/site-packages/optimum/exporters/onnx/convert.py", line 902, in export
    config.fix_dynamic_axes(output, device=device, input_shapes=input_shapes, dtype=dtype)
  File "/opt/conda/lib/python3.10/site-packages/optimum/exporters/onnx/base.py", line 306, in fix_dynamic_axes
    session = InferenceSession(model_path.as_posix(), providers=providers, sess_options=session_options)
  File "/opt/conda/lib/python3.10/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 383, in __init__
    self._create_inference_session(providers, provider_options, disabled_optimizers)
  File "/opt/conda/lib/python3.10/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 424, in _create_inference_session
    sess = C.InferenceSession(session_options, self._model_path, True, self._read_config_from_model)
onnxruntime.capi.onnxruntime_pybind11_state.InvalidGraph: [ONNXRuntimeError] : 10 : INVALID_GRAPH : Load model from tinyllama-1b_onnx/model.onnx failed:This is an invalid model. Type Error: Type 'tensor(bfloat16)' of input parameter (/model/Constant_34_output_0) of operator (Where) in node (/model/Where_3) is invalid.

Expected behavior
Convert the model properly to bf16

Thank you @mgiessing! It is possible that the ONNX model is valid, but ORT is missing some operators for bf16. It can halso be a bug, I will have a look shortly.

Thank you for having a look - this also happened on my Mac M1 with a more recent ORT version (v1.17.1) and also with a different model (deepset/roberta-base-squad2)

@mgiessing Where (used in https://github.com/huggingface/transformers/blob/caa5c65db1f4db617cdac2ad667ba62edf94dd98/src/transformers/models/llama/modeling_llama.py#L1086) is not implemented for BF16 dtype in ORT https://github.com/microsoft/onnxruntime/blob/v1.17.1/docs/OperatorKernels.md
However it is valid in ONNX standard: https://github.com/onnx/onnx/blob/main/docs/Operators.md#where
I suggest you to open a feature request in ONNX Runtime repository concerning this to add the support. In the meantime, we could patch Transformers code for this to work in BF16 (avoid the Where op in bf16).
See as well https://github.com/huggingface/optimum/issues/1720#issuecomment-1963838333 that is related and that you are likely to hit as well

Thanks for having a look at that :)
I'll try to open a request starting next week to address that issue!


Add primer efficiency evaluation

One of the key questions that keeps getting asked is if primer schemes are working. There are ways from this workflow to determine if primer pairs are working (see https://github.com/UPHL-BioNGS/Cecret#is-there-a-way-to-determine-if-certain-amplicons-are-failing), but these methods require extra steps.
Wouldn't it be awesome if there was something in this workflow that created a graph or something?

https://github.com/UPHL-BioNGS/Cecret/pull/194 now creates some images that get incorporated into multiqc.
I need to still generate images and put them into the readme, but this is a start.

I've decided to spin this idea out into its own repo : https://github.com/erinyoung/ACI
I'm hoping to get this to conda soon


Login improvements and other CLI fixes

Added project creation for a specified namespace.
Fixed project removal.
Closes: #76, #81, #82

By the way, does this fix #95 as well?

By the way, does this fix #95 as well?

It should be fixed now with the latest changes.


improvement not print thread pool error stack trace when shutdown

Proposed changes

when thread pool shutdown, should not print error stack trace, it is very confuse.
arrow flight server should not call shutdown, if it is not enabled, because it will print error stack.
remove service unavailable from thrift because it is useless.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


follow up: massmint - mobile

goes after

#7076

listing mini bar overflow screen on mobile, and "list items" button height doesn't match rest of container
@exezbcz I guess we need a design for this one
How should this bar look on mobile
also, problem on mobile

Originally posted by @daiagi in https://github.com/kodadot/nft-gallery/pull/7076#pullrequestreview-1610268855

will be disabled on mobile since we don't have many UI ways how to select multiple items without overcomplicating it - thus disabled on touch devices

#6941

so mass listing is only on desktop?

@daiagi only on non-touch devices, like i said, we don't have many UI ways how to select multiple items without overcomplicating it

i dont remember if its already disabled on mobile or? Should not be enabled on touchdevices due to the reason above


[Fix]Fix publish may wait timeout because of dead BE

Proposed changes
AckResponseHandler should only accept alive BE, otherwise pubslih may wait timeout if dead Be exists.

run buildall


fix(platform): fixed simulation type labels in project browse controls

Closes #4289

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


Cannot inherit from final ImageSource

Hi
Now in react native 0.75.0.rc-6 ImageSource class is final class which cannot be inherited, so I cannot use react-native-fast-image with my project.
could you fix this issue, if yes when can it be done?
thanks

Hi, your project in Android works correctly, where do you use FastImage ?

No it's not working, Build failed because of Cannot inherit from final ImageSource error

No it's not working, Build failed because of Cannot inherit from final ImageSource error

I understand, I have a completely different error 😃 . From what I've gathered, this library doesn't work on newer versions of React Native; iOS works, but Android doesn't work at all.

@yousefArb I will look in to it for sure . Give me some time i will share the timelines

Hello, after updating the react-native to  0.75.0.rc-7 I have the same error during the run yarn android

link to the issue in the original repo https://github.com/DylanVann/react-native-fast-image/issues/1043

@yousefArb I will be picking this up

@YuriOlepir @yousefArb I will be picking this up

@YuriOlepir @yousefArb https://github.com/facebook/react-native/pull/46092
Meta has changed the class , now its open for inheritance , in newer release this will be fixed automatically

@deepanshushuklad11
Thank you for breathing new life into the old library by fixing issues and adding support for the latest React Native versions—your efforts are truly appreciated!

It's not fixed in the newest version: "react-native": "0.75.2",

It's not fixed in the newest version: "react-native": "0.75.2",

I think there was a miss in the PR by a new PR has been raised to fix https://github.com/facebook/react-native/pull/46152.


Package rustup or update rustc-nightly more often

Issue description
I have to use rustc nightly for one project, but the nightly version of nixpkgs is too old. And we do not ship rustup either, so it is not possible for me to work right now...
Can someone please provide some documentation how to update rustc unstable? That'd be the simplest solution I guess... because right now I do not understand how to update the package...

I was working on a script to do this automatically: https://github.com/NixOS/nixpkgs/pull/21922 ... its not finished yet.

rustup should also work without a nix package in your home.

rustup doesn't work as the downloaded binaries need to be patched. I started adding this functionality to rustup some time ago but never got around finishing it. Can't be done with a simple wrapper as rustup calls into the downloaded binaries to check their versions etc. before exiting.

I don't care anymore because there is the nixpkgs overlay by mozilla now.

For people finding this issue on Google: The overlay @matthiasbeyer mentioned is available at https://github.com/mozilla/nixpkgs-mozilla

Yes. If you find that rustc does not find src or core, have a look here. I don't know how common this is, but I'll leave this here just in case.


Implement Grant Registrar

Adds a GrantRegistrar contract, which keeps track of which addresses are allowed to mint grants, and enables verifying an address for multiple future grants, saving on gas when claiming a grant.
Usage

Deploy GrantRegistrar with World ID details (router, groupId, and externalNullifierHash), the current grant id, and the maximum grant id that will be claimable (more on this last one below).
Deploy RecurringGrantDropWithRegistrar with the address of the GrantRegistrar contract (and the other parameters that were already present on the previous RecurringGrantDrop).
To verify, users must first call GrantRegistrar.verify with the receiver address and a valid World ID proof for that address and maxGrantId.
Then, for the next currentGrandId...maxGrantId grants, users can simply call RecurringGrantDropWithRegistrar.claim(uint256 grantId, address receiver).
Every time a new grant gets deployed, the contract's owner should call GrantRegistrar.setCurrentGrant and GrantRegistrar.setMaxGrantId to update the contract's understanding of the current grant and the max grant users should be able to claim.

Design Considerations

The GrantRegistrar works with both currentGrantId and maxGrantId to allow for rolling registrations. A simpler design would make it so that, if allowing users to verify for $n$ grants at a time, users registering for the first time on grant $n$ would need to re-verify for the next grant.
When verifying their wallet, the provided proof must include both the receiver address and the contract's current maxGrantId in the signal. This prevents re-verifying by simply reusing the previous verification's proof.
For maximum flexibility, the maxGrantId is not computed by adding some constant to the currentGrantId but set dynamically by the contract's owner. This could probably get updated to a constant if we expect grants to always follow the same incrementing pattern.
For simplicity, the new RecurringGrantDropWithRegistrar contract does not include logic for handling reservations. It is expected the currently deployed RecurringGrantDrop contract will continue to handle reservations.

tests?

also: tests would actually be great!


[Q] Compare runs across different job types via the API

For a project, I have a structure similar to the one the grouping tutorial, where runs are grouped together.
Within each group, I have several job_types, such as eval or baseline.
Usually, within these types, only one run is stored.
What I am looking for is a way to "iterate" over this structure like so:

Select a group
Within this group:

Pull data from various job types
Compare the collected data

Proceed to the next group

So far, I do not want to compare between different groups,  only within a group I want to compare between different job_types's runs.
This snippet should give a further idea of what I want to accomplish:
import wandb

api = wandb.Api()

#both runs below are from the same group
runA = api.run("user_name/project_name/runs/123456K")  # eval run, from within eval job type
runB = api.run("user_name/project_name/runs/82921L7")  # baseline, from within baseline job type

compare_two_runs(runA, runB) # pull data from the runs; do actual computation ...

What it misses is a way to iterate over the groups, and then find the runs within a job_type.
How do I best go about this? Doing this manually via the web dashboard is prohibitive, I have a prohibitively large number of groups and runs within them, and more are to be added.

Hey Pascal, the run object that is returned using the public API has job_type and group attributes. So you should be able to do the filtering in the script.
runs = api.runs("entity_name/project_name")
for run in runs:
  print(run.job_type, run.group_name)

Thanks @armanhar for pointing me to this simple solution! It works, and I have discovered many additional helpful attributes!
This is what I used in the end:
for run in runs:
  print(run.job_type, run.group) #it's just group

@phrasenmaeher glad to hear it worked!


Custom Canvas Layer in settings

This is a fix to [Issue]https://github.com/coppolaemilio/dialogic/issues/362)
In the settings editor, you can now choose the layer index which the dialogic's canvas layer will use.
This should prevent dialogues from being rendered behind UI with higher layers.

thank you! I'll try to review it and merge it soon

Thanks again!

@RedXGames. I'm trying to fix #427 right now and wonder if there is a reason why the canvas layer box is a LineEdit instead of a input made for numbers. Any reason or just by accident?


Custom drain flow

Tell us about your request
Add a rollout flag when using drain. It will be used when consolidation and native termination handler (https://github.com/aws/karpenter/pull/2546)  will be ready.
The custom drain flow is like this:

Cordon the node
Do a rolling restart of the deployments that have pods running on the node.
Drain the node.

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Currently when using consolidation feature or aws-node-termination-handler we can result of downtime or heavy performance degradation with the current implementation of kubectl drain
Current drain will terminate all workloads on a node and the scheduler will try to create those workloads on available nodes and if not any Karpenter will provision new node.
Even with PDB there is some level or degradation.
Are you currently working around this issue?
having a custom bash script that implements an alternative to kubectl drain
https://gist.github.com/juliohm1978/1f24f9259399e1e1edf092f1e2c7b089
Additional Context
kubectl drain leads to downtime even with a PodDisruptionBudget https://github.com/kubernetes/kubernetes/issues/48307
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Please open this issue

/reopen

Would be nice to see a proper solution

This issue also affects our single-replica deployments that are accessed through a service. Initially, I considered using a preStop hook, but this solution proved ineffective since it still transitions the pod to a terminating state and immediately removes it from the service endpoints.


Use GH Actions for CI testing

Removes the CircleCI config as well.

The ubuntu tests are failing and for some reason not reported as errors through the GH Checks
PASS dist/test/test/test_python_hello
ERROR:root:code for hash md5 was not found.
Traceback (most recent call last):
  File "/home/runner/.cache/co.zeit.fun/runtimes/python2.7/lib/python2.7/hashlib.py", line 147, in <module>
    globals()[__func_name] = __get_hash(__func_name)
  File "/home/runner/.cache/co.zeit.fun/runtimes/python2.7/lib/python2.7/hashlib.py", line 97, in __get_builtin_constructor
    raise ValueError('unsupported hash type ' + name)
ValueError: unsupported hash type md5
ERROR:root:code for hash sha1 was not found.

@styfle I think they're more of "boot-up errors" that don't affect the running of the Python code, since those hash functions are not used. But still, ya I would like to understand how to fix/avoid that.


zCreateProjection+36^WebTerminal.Installer.1 *WebTerminal.Analytics

WebTerminal-v4.8.0.xml
running on:
Cache for UNIX (Red Hat Enterprise Linux for x86-64) 2017.2.1 (Build 801_3_18583U) Tue Nov 27 2018 06:12:01 EST [HealthShare Modules:Core:15.03.9903 + Linkage Engine:15.03.9903]
this line of code:
do ##class(WebTerminal.Analytics).ReportInstallStatus(err)
causes this error:
zCreateProjection+36^WebTerminal.Installer.1 *WebTerminal.Analytics

Hello! Thanks.
Many people has reported this issue but no one helped me to investigate what's wrong...
Could you please try/answer the following:
0. Import XML and does not compile (untick "compile" in the import popup).

Try to compile all classes in WebTerminal package manually. Does it compiles normally?
Try to compile WebTerminal.Analytics class, and then WebTerminal.Installer class. Does it work now?

The analytics class compiles OK by itself.
The project imports without compiling OK.
The installer class does not compile
From: Nikita Savchenko<EMAIL_ADDRESS>Reply-To: intersystems-community/webterminal<EMAIL_ADDRESS>Date: Thursday, December 13, 2018 at 3:30 PM
To: intersystems-community/webterminal<EMAIL_ADDRESS>Cc: Gramen Tontchev<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [intersystems-community/webterminal] zCreateProjection+36^WebTerminal.Installer.1 *WebTerminal.Analytics (#119)
Hello! Thanks.
Many people has reported this issue but no one helped me to investigate what's wrong...
Could you please try/answer the following:
0. Import XML and does not compile (untick "compile" in the import popup).

Try to compile all classes in WebTerminal package manually. Does it compiles normally?
Try to compile WebTerminal.Analytics class, and then WebTerminal.Installer class. Does it work now?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/intersystems-community/webterminal/issues/119#issuecomment-447110340, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARnUeCO5-zs9vhLJUqFgwHVEC8YRLRpIks5u4rjtgaJpZM4ZSXW5.

Really interesting. You compile WebTerminal.Analytics class on its own, and
then the error says that no WebTerminal.Analytics class is found. I wish I
could play around with your InterSystems setup to find out what is causing
this error to happen. Well, if you are a tech person I would appreciate you
trying to figure this out :)
As a quick workaround to make it work, you can comment out that erroring
line in WebTerminal.Installer class. However, it is much more interesting
to dig in and find out the root case of this problem, as I have never had a
chance to reproduce it...
Thanks you.
On Thu, Dec 13, 2018 at 11:30 PM interhealth<EMAIL_ADDRESS>wrote:

The analytics class compiles OK by itself.
The project imports without compiling OK.
The installer class does not compile
From: Nikita Savchenko<EMAIL_ADDRESS>Reply-To: intersystems-community/webterminal<EMAIL_ADDRESS>Date: Thursday, December 13, 2018 at 3:30 PM
To: intersystems-community/webterminal<EMAIL_ADDRESS>Cc: Gramen Tontchev<EMAIL_ADDRESS>Author <
<EMAIL_ADDRESS>Subject: Re: [intersystems-community/webterminal] zCreateProjection+36^WebTerminal.Installer.1 *WebTerminal.Analytics
(#119)
Hello! Thanks.
Many people has reported this issue but no one helped me to investigate
what's wrong...
Could you please try/answer the following:
0. Import XML and does not compile (untick "compile" in the import popup).

Try to compile all classes in WebTerminal package manually. Does it
compiles normally?
Try to compile WebTerminal.Analytics class, and then
WebTerminal.Installer class. Does it work now?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/intersystems-community/webterminal/issues/119#issuecomment-447110340>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/ARnUeCO5-zs9vhLJUqFgwHVEC8YRLRpIks5u4rjtgaJpZM4ZSXW5>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/issues/119#issuecomment-447127675,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEwhSJeXvxB08NwFfs9WbhoQC8aS0Yfiks5u4scQgaJpZM4ZSXW5
.

All right, now I've finally found an instance where this problem is reproducible.
Furthermore, I've found a source of the problem. Expect the update soon! Again, thanks for reporting the issue.

@interhealth, now this issue is fixed! Please confirm that the latest version (4.8.3) doesn't bring this problem.
Thank you!

Hi Nikita
Fix Confirmed.
Installation completed on 2018.1.1HS.2052.0
Thank you for the followup!
From: Nikita Savchenko<EMAIL_ADDRESS>Reply-To: intersystems-community/webterminal<EMAIL_ADDRESS>Date: Sunday, March 31, 2019 at 4:21 PM
To: intersystems-community/webterminal<EMAIL_ADDRESS>Cc: Gramen Tontchev<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [intersystems-community/webterminal] zCreateProjection+36^WebTerminal.Installer.1 *WebTerminal.Analytics (#119)
@interhealthhttps://github.com/interhealth, now this issue is fixed! Please confirm that the latest versionhttps://intersystems-community.github.io/webterminal/#downloads (4.8.3) doesn't bring this problem.
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/intersystems-community/webterminal/issues/119#issuecomment-478378364, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARnUePZMV8_QbPg8VSbxsm3CIYwpq0rnks5vcRjegaJpZM4ZSXW5.

Awesome! Thanks, @interhealth, for reporting this!


Add NPD endpoints: /debug/pprof, /healthz, /conditions.

For https://github.com/kubernetes/node-problem-detector/issues/76.
This PR:

Added /healthz for health checking. Currently it always returns 200.
Added /conditions to list all internal conditions inside the NPD. This is mainly used for debugging.
Added /debug/pprof pprof endpoint. This will be used for following performance benchmark and optimization.

Note that to get more debug information, we removed the -w ldflag. The binary size only increased from 40MB+ to 60MB+, which should be fine.

This change is

@andyxning I need the endpoint to work on the performance stuff, so send the PR first. Hope you haven't started working on that.

@Random-Liu Yes, i have not spare some time to do this. Since you have done this, i will make a review on this. :)

@andyxning Thanks. I know it's Spring Festival soon. Enjoy yourself! :)

@andyxning Add you as the reviewer. Thanks a lot!

Review status: 0 of 6 files reviewed at latest revision, 7 unresolved discussions, some commit checks failed.

Makefile, line 34 at r2 (raw file):

./bin/node-problem-detector: $(PKG_SOURCES)
	GOOS=linux go build -o bin/node-problem-detector \
	     -ldflags '-extldflags "-static" -X $(PKG)/pkg/version.version=$(VERSION)' \

SGTM

cmd/node_problem_detector.go, line 44 at r2 (raw file):

	printVersion            = flag.Bool("version", false, "Print version information and quit")
	hostnameOverride        = flag.String("hostname-override", "", "Custom node name used to override hostname")
	serverPort              = flag.Int("server-port", 10254, "The port to bind the node problem detector server. Use 0 to disable.")

maybe port and address command line argument are more briefness, as currently we do not have any other http service.
I have grep -r 10254 . in kubernetes, heapster and contrib and find that gcr.io/google_containers/nginx-ingress-controller has used that port. Maybe we should choose another one. BTW, does kubernetes has some standards one port usage.
IIUC, 0 is mainly used to select a port randomly. So, maybe we need to make another command line argument, like http-disabled, to stop npd from starting the http service and the default value for this command line option is false. This seems to be clearer.

cmd/node_problem_detector.go, line 45 at r2 (raw file):

	hostnameOverride        = flag.String("hostname-override", "", "Custom node name used to override hostname")
	serverPort              = flag.Int("server-port", 10254, "The port to bind the node problem detector server. Use 0 to disable.")
	serverAddress           = flag.String("server-address", "<IP_ADDRESS>", "The address to bind the node problem detector server.")

Any special considerations for using <IP_ADDRESS> instead of <IP_ADDRESS>.

cmd/node_problem_detector.go, line 96 at r2 (raw file):

		err := http.ListenAndServe(net.JoinHostPort(*serverAddress, strconv.Itoa(*serverPort)), nil)
		if err != nil {
			glog.Errorf("Failed to start server: %v", err)

glog.Fatalf makes failure fail faster instead of just logging. This error is lost of functionality. :)
Any reasons we need to put http server generating logic to a loop? IIUC, this will success on first try and will last or fail when bind and listen fails where we need to make npd fail fast in order to make users realize that npd can not start and one major functionality is not woking.

cmd/node_problem_detector.go, line 116 at r2 (raw file):

	// Start http server.
	if *serverPort > 0 {

if !httpDisabled {
    startHTTPServer(p)
}

pkg/condition/manager.go, line 62 at r2 (raw file):

type conditionManager struct {
	// The lock protecting the object. Only 2 fields will be accessed by more

we need to refactor the code since #79 has been merged.

pkg/util/http.go, line 23 at r2 (raw file):

	"net/http"
)

SGTM

Comments from Reviewable

@Random-Liu Sorry for the delay. It is Spring Festival holidays. :)

Review status: 0 of 6 files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

pkg/util/http.go, line 24 at r2 (raw file):

ReturnHTTPJson is a helper function to generate json http response.

ReturnHTTPJson generates json http response.

pkg/util/http.go, line 36 at r2 (raw file):

ReturnHTTPError is a helper function to generate error http response.

ReturnHTTPError generates error http response.

Comments from Reviewable

No problem at all. Enjoy your holiday! :)

Comments from Reviewable

Review status: 0 of 6 files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

cmd/node_problem_detector.go, line 44 at r2 (raw file):

maybe port and address command line argument are more briefness, as currently we do not have any other http service.

Will do.

I have grep -r 10254 . in kubernetes, heapster and contrib and find that gcr.io/google_containers/nginx-ingress-controller has used that port. Maybe we should choose another one. BTW, does kubernetes has some standards one port usage.

Good catch. As far as I know, there is no standards for port usage for now. I choose 10254 because it is not used in https://github.com/kubernetes/kubernetes/blob/master/pkg/master/ports/ports.go
I'll use 10256 instead then.

IIUC, 0 is mainly used to select a port randomly. So, maybe we need to make another command line argument, like http-disabled, to stop npd from starting the http service and the default value for this command line option is false. This seems to be clearer.

Actually I copy the code from https://github.com/kubernetes/kubernetes/blob/master/cmd/kubelet/app/options/options.go#L106. I don't think we want to select port randomly, let's just keep the behavior the same with kubelet for now. :)

cmd/node_problem_detector.go, line 45 at r2 (raw file):
Previously, andyxning (Ning Xie) wrote…
Any special considerations for using <IP_ADDRESS> instead of <IP_ADDRESS>.

Actually,  kube-proxy and kubelet use <IP_ADDRESS> as healthz address, <IP_ADDRESS> as server address.
Hm, let's use <IP_ADDRESS> as default, if we need remote access, we can always start NPD with <IP_ADDRESS>. :)

cmd/node_problem_detector.go, line 96 at r2 (raw file):
Previously, andyxning (Ning Xie) wrote…

glog.Fatalf makes failure fail faster instead of just logging. This error is lost of functionality. :)
Any reasons we need to put http server generating logic to a loop? IIUC, this will success on first try and will last or fail when bind and listen fails where we need to make npd fail fast in order to make users realize that npd can not start and one major functionality is not woking.

You are right. Will change to fatal and remove the loop.

Comments from Reviewable

Review status: 0 of 6 files reviewed at latest revision, 9 unresolved discussions.

cmd/node_problem_detector.go, line 44 at r2 (raw file):
Previously, Random-Liu (Lantao Liu) wrote…

maybe port and address command line argument are more briefness, as currently we do not have any other http service.

Will do.

I have grep -r 10254 . in kubernetes, heapster and contrib and find that gcr.io/google_containers/nginx-ingress-controller has used that port. Maybe we should choose another one. BTW, does kubernetes has some standards one port usage.

Good catch. As far as I know, there is no standards for port usage for now. I choose 10254 because it is not used in https://github.com/kubernetes/kubernetes/blob/master/pkg/master/ports/ports.go
I'll use 10256 instead then.

IIUC, 0 is mainly used to select a port randomly. So, maybe we need to make another command line argument, like http-disabled, to stop npd from starting the http service and the default value for this command line option is false. This seems to be clearer.

Actually I copy the code from https://github.com/kubernetes/kubernetes/blob/master/cmd/kubelet/app/options/options.go#L106. I don't think we want to select port randomly, let's just keep the behavior the same with kubelet for now. :)

Done.

cmd/node_problem_detector.go, line 45 at r2 (raw file):
Previously, Random-Liu (Lantao Liu) wrote…
Actually,  kube-proxy and kubelet use <IP_ADDRESS> as healthz address, <IP_ADDRESS> as server address.
Hm, let's use <IP_ADDRESS> as default, if we need remote access, we can always start NPD with <IP_ADDRESS>. :)

Done.

cmd/node_problem_detector.go, line 96 at r2 (raw file):
Previously, Random-Liu (Lantao Liu) wrote…
You are right. Will change to fatal and remove the loop.

Done.

pkg/condition/manager.go, line 62 at r2 (raw file):
Previously, andyxning (Ning Xie) wrote…
we need to refactor the code since #79 has been merged.

Done.

pkg/util/http.go, line 24 at r2 (raw file):
Previously, andyxning (Ning Xie) wrote…

ReturnHTTPJson is a helper function to generate json http response.

ReturnHTTPJson generates json http response.

Done.

pkg/util/http.go, line 36 at r2 (raw file):
Previously, andyxning (Ning Xie) wrote…

ReturnHTTPError is a helper function to generate error http response.

ReturnHTTPError generates error http response.

Done.

Comments from Reviewable

@andyxning Addressed comments. PTAL~

Review status: 0 of 6 files reviewed at latest revision, 6 unresolved discussions.

cmd/node_problem_detector.go, line 116 at r2 (raw file):
Previously, andyxning (Ning Xie) wrote…
if !httpDisabled {
    startHTTPServer(p)
}

Since we need to keep up with kubelet, this should not be modified. :)
LGTM

pkg/condition/manager.go, line 62 at r2 (raw file):
Previously, Random-Liu (Lantao Liu) wrote…
Done.

nit: The lock protecting the object. is useless and can be deleted. WDYT. @Random-Liu

pkg/condition/manager_test.go, line 85 at r3 (raw file):

func TestGetConditions(t *testing.T) {
	m, _, _ := newTestManager()

LGTM

pkg/problemdetector/problem_detector.go, line 81 at r3 (raw file):

	// Add the handler to serve condition http request.
	http.HandleFunc("/conditions", func(w http.ResponseWriter, r *http.Request) {
		util.ReturnHTTPJson(w, p.conditionManager.GetConditions())

LGTM

pkg/util/http.go, line 23 at r2 (raw file):
Previously, andyxning (Ning Xie) wrote…
SGTM

LGTM

Comments from Reviewable

Reviewed 1 of 6 files at r1, 1 of 1 files at r2, 3 of 3 files at r3.
Review status: 5 of 6 files reviewed at latest revision, 6 unresolved discussions.

Comments from Reviewable

@Random-Liu just one nit. Overall LGTM.

pkg/condition/manager.go, line 62 at r2 (raw file):
Previously, andyxning (Ning Xie) wrote…
nit: The lock protecting the object. is useless and can be deleted. WDYT. @Random-Liu

Done.

Comments from Reviewable

@andyxning Thanks for reviewing! Apply LGTM based on https://github.com/kubernetes/node-problem-detector/pull/83#issuecomment-277213866.

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, 5 unresolved discussions.

Comments from Reviewable

/lgtm

Review status: all files reviewed at latest revision, 5 unresolved discussions.

Comments from Reviewable


Meeting Recorder doesn't appear

The meeting recorder doesn't join the meeting. The recorded video stays:

What is the URL that you are passing to the recorder? Is it the V2 link as mentioned here in Point #3
Example: If the meeting URL is https://abcdefghij.execute-api.us-east-1.amazonaws.com/Prod/v2?m=MyMeetingTest, then the encoded URL which needs to be passed to the meeting recorder should look like https%3A%2F%2Fabcdefghij.execute-api.us-east-1.amazonaws.com%2FProd%2Fv2%3Fm%3DMyMeetingTest

@moracarlos feel free to re-open if it's still an issue.


Android on first app installation - facebook::react::Recoverable: Could not open file/data/user/0/com.detoxtesting/files/ReactNativeDevBundle.js: No such file or directory

Description
After I first run the app via react-native run-android or detox build -c android.emu.debug && detox test -c android.emu.debug and click on screen I get red screen and the following error:
facebook::react::Recoverable: Could not open file/data/user/0/com.detoxtesting/files/ReactNativeDevBundle.js: No such file or directory
Try the following to fix the issue:
• Ensure that the packager server is running
• Ensure that your device/emulator is connected to your machine and has USB debugging enabled - run 'adb devices' to see a list of connected devices
• Ensure Airplane Mode is disabled
• If you're on a physical device connected to the same machine, run 'adb reverse tcp:8081 tcp:8081' to forward requests from your device
• If your device is on the same Wi-Fi network, set 'Debug server host & port for device' in 'Dev settings' to your machine's IP address and the port of the local dev server - e.g. <IP_ADDRESS>:8081
facebook::react::Recoverable: Could not open file/data/user/0/com.detoxtesting/files/ReactNativeDevBundle.js: No such file or directory
loadScript
    JSBundleLoader.java:77
runJSBundle
    CatalystInstanceImpl.java:234
createReactContext
    ReactInstanceManager.java:1137
access$900
    ReactInstanceManager.java:113
run
    ReactInstanceManager.java:944
run
    Thread.java:761

If I restart app via simulator then app works, but the tests does not run. iOS build and tests work normally.
Steps to Reproduce
Run app and click on screen.
Detox, Node, Device, Xcode and macOS Versions

Detox: 9.0.4
React Native: 0.52
Node: 8.6.0
Device: Nexus 5X API 25
Xcode: 9.4
macOS: 10.13.6

Device and verbose Detox logs
facebook::react::Recoverable: Could not open file/data/user/0/com.detoxtesting/files/ReactNativeDevBundle.js: No such file or directory

Try the following to fix the issue:
• Ensure that the packager server is running
• Ensure that your device/emulator is connected to your machine and has USB debugging enabled - run 'adb devices' to see a list of connected devices
• Ensure Airplane Mode is disabled
• If you're on a physical device connected to the same machine, run 'adb reverse tcp:8081 tcp:8081' to forward requests from your device
• If your device is on the same Wi-Fi network, set 'Debug server host & port for device' in 'Dev settings' to your machine's IP address and the port of the local dev server - e.g. <IP_ADDRESS>:8081

facebook::react::Recoverable: Could not open file/data/user/0/com.detoxtesting/files/ReactNativeDevBundle.js: No such file or directory
loadScript
    JSBundleLoader.java:77
runJSBundle
    CatalystInstanceImpl.java:234
createReactContext
    ReactInstanceManager.java:1137
access$900
    ReactInstanceManager.java:113
run
    ReactInstanceManager.java:944
run
    Thread.java:761

Is this issue related to detox? Seems like a React Native configuration issue


refactor: apply flake8

Codecov Report

Merging #137 (029c301) into main (7a67e64) will decrease coverage by 0.00%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
- Coverage   89.29%   89.28%   -0.01%
==========================================
  Files          23       23
  Lines        1831     1830       -1
==========================================
- Hits         1635     1634       -1
  Misses        196      196

Impacted Files
Coverage Δ

solnlib/modular_input/modular_input.py
83.04% <0.00%> (ø)

solnlib/splunkenv.py
84.31% <ø> (-0.16%)
:arrow_down:

solnlib/conf_manager.py
92.15% <100.00%> (ø)

solnlib/credentials.py
88.03% <100.00%> (ø)

Continue to review full report at Codecov.

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

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

v4.5.0
GitHub release

Your semantic-release bot :package::rocket:


EliteeMart - E-commerce Platform for Electronics

EliteeMart
An e-commerce platform with a variety of electronic products, including the latest gadgets and accessories, offering a seamless user experience and a wide range of options to choose from.

Website : Visit Elite Emart
The platform features two distinct roles: User and Admin.
🎯 Project Overview

Admin Role:  Full access to manage products, upload new items, and track orders.

**User Role:**Browse and purchase products, with options to search, filter by category, and view detailed product information.

Noted that Restyled App is disabled, so the formatting check failed. Please review and advise if any manual formatting adjustments are needed.


Teensy 4.0 pins placement

Hello,
I downloaded the footprint for teensy but i see that pins placement are not quite correct.

In here 14th pin is on the left side but in teensy website it shows on the right.
I didn't check the size of the footprint yet. I am afraid the size of the footprint is not accurate.
Could you guys fix it ?

The pin numbers on the footprint are not the same as the digital gpio pin numbers that are silkscreen on the teensy.  The footprint numbers were created separately because there are many more physical pins than GPIO pins.


WIP – Fix lodash require issue blocking front end local dev

Hey @jhilde and @kevinrobinson!
Do either of you get this error from Browserify when you run rails server locally?
I've been getting it and it's blocking me from developing on my machine:
Error: Cannot find module 'lodash/isInteger' from '/Users/alexsoble/somerville-teacher-tool/app/assets/javascripts/helpers'

Maybe it's overkill but I decided to just pull lodash into the yarn pipeline, add to package.json, and use import everywhere. If you were having the same error as me, does this seem like a good fix? If not, does this look like a good change anyway?

@alexsoble I can't reproduce on a clean build.  If you rm -rf node_modules and reinstall, do you still see it?

@kevinrobinson Yep, that fixed the error!
Was curious about why the Travis build failed for this and looking into it showed me that we're using lodash inconsistently.
We're using _.pluck, which is exists in 3.10.1 but not 4.17.4.
And we're using _.isInteger, which exists in 4.17.4 but not 3.10.1.
_.isInteger is only used to check for non-numeric values in numeric table sorts, which is a rare edge case and isn't actively throwing errors in the app. So it's not a high-priority fix.
Not urgent at all but I'll aim to land this PR to clean up the inconsistency and guard against those non-numberic values that might creep into number columns and break the sort.

Features to double-check:

graph_helpers.js
Student Report PDF features

Oy upgrading to 4.17.4 is a nightmare, let's say at 3.10.1 and use isNumber instead of isInteger.

@alexsoble Yeah if I remember right there are actually two separate copies right now because of the way we're building both through the asset pipeline and through browserify.  The best long-term solution is to keep chipping away at migrating things away from the asset pipeline (or do a sprint to finish that), and that should get simpler.  lodash does have breaking changes but most I think are mechanical (eg _.pluck(things, 'field') is now just supported by _.map(things, 'field').

Agreed on the mechanical changes!
Tested this branch on an IE VM, looks like everything works good with only one copy of lodash (managed via yarn) including graphs and PDF reports.

Looks good in prod.


The !!stop command stops other guild's running commands

Confirming that this doesn't happen between servers, but rather between text channels in a single server.

Searching deeper into this problem made me realize there was a flaw in the way the threads of CommandRouters were created :
They were created using the command's name + the Guild ID, which is the ID of the server and not the one of the Text Channel, meaning a command run in one text channel could have potentially affected other text channels, such as this very issue.
I changed it so now, threads have a bigger name containing both the Guild ID and the Text Channel ID, meaning that commands will be more isolated than ever.


change comments for environment in sample hooks script

Description of changes
Once, I try to change implementation to match documentation, but, it is not a safe change (https://github.com/aws-amplify/amplify-cli/pull/9347). So, I change comments for environment in sample hooks script. And I add test that implementation for environment in hooks script match documentation.
I also change documentation (https://github.com/aws-amplify/docs/pull/4049).

Issue #, if available

fix #9346
Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[x] Relevant documentation is changed or added (and PR referenced)

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

Codecov Report

Merging #9801 (c5343ca) into master (2bd4172) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9801      +/-   ##
==========================================
- Coverage   53.00%   52.99%   -0.02%
==========================================
  Files         830      830
  Lines       46012    46012
  Branches     9826     9826
==========================================
- Hits        24390    24382       -8
- Misses      19604    19612       +8
  Partials     2018     2018

Impacted Files
Coverage Δ

packages/amplify-cli-core/src/errors/index.ts
72.22% <0.00%> (-13.89%)
:arrow_down:

.../amplify-cli-core/src/state-manager/pathManager.ts
60.33% <0.00%> (-2.48%)
:arrow_down:

...amplify-cli-core/src/state-manager/stateManager.ts
37.91% <0.00%> (-1.65%)
:arrow_down:

packages/amplify-cli-core/src/hooks/hooksMeta.ts
83.78% <0.00%> (+4.05%)
:arrow_up:

Continue to review full report at Codecov.

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


Error: actor.GetOwnedItem is not a function at...

Hi,
Thanks for all your hard work!
I'm using a feature with a "roll macro" attached to it, that looks at the token's class level and then rolls a dice based on that level in DnD 5e. It's a remodelled monk ki macro that I snuck from somewhere. It used to work, but recently, I get this error:
TypeError: actor.getOwnedItem is not a function
at Macro.eval (eval at callScriptMacroFunction (Macros.js:176), :35:20)
at Macro.callScriptMacroFunction [as callScriptFunction] (Macros.js:179)
at Macro.renderMacro [as renderContent] (Macros.js:197)
at Macro.executeMacro (Macros.js:218)
at Workflow.callMacro (workflow.js:875)
at Workflow.callMacros (workflow.js:825)
at Workflow._next (workflow.js:601)
at Workflow.next (workflow.js:334)
at Workflow._next (workflow.js:596)
at Workflow.next (workflow.js:334)
Mousing over the Macros.js seems to dedicate it to Advanced Macros, so I hope I got it right reporting it here. I've attached the data for the feature and the macro. I've tried troubleshooting with different amount of modules enabled, but could only change the error message applied (presumably becaues I accidentally unticked a dependency...), never get it to work again.
Advanced Macros error macro and feature.zip

The error has changed and I cannot reproduce the original, so I assume it is something on my end. Apologies.

Nevermind, I got it to re-produce. I can run the macro fine solo, cannot run the macro as part of a feature.


[BUG] Cannot read property 'documentation' of undefined

Describe the bug
When event is httpApi Cannot read property 'documentation' of undefined exception is raised
To Reproduce
Create a lambda with httpApi event
functions:
myLambda:
...
events:
- httpApi:
  method: GET
  path: hello
  documentation:
    ....
  ...

Expected behavior
No exception raised and open API file generated
Desktop (please complete the following information):
Your Environment Information ---------------------------
Operating System:          linux
Node Version:              14.19.1
Framework Version:         2.72.3 (local)
Plugin Version:            5.5.4
SDK Version:               4.3.2
Components Version:        3.18.2
Additional context
A possible solution, on method createPaths of definitionGenerator.js file there are missing questions mark
const documentation = event.http.documentation || event.httpApi.documentation

should be
const documentation = event.http?.documentation || event.httpApi?.documentation

In fact i think httpApi is not fully managed

@cbm-eheude you're right.  I think I have a fix for this along with some other issues on my other machine.  Will try and get it out tonight or tomorrow

@cbm-eheude I have released a new version.  If you'd like to try that and feel free to let me know any issues you may find with it.

@JaredCE it's working now thanks !

That's great.  I'm sure there are some other gotchas in there that I need
to work through.
Sorry it didn't work first time round.
On Thu, 21 Jul 2022, 18:07 eheude, @.***> wrote:

@JaredCE https://github.com/JaredCE it's working now thanks !
—
Reply to this email directly, view it on GitHub
https://github.com/JaredCE/serverless-openapi-documenter/issues/21#issuecomment-1191734155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF4CYGK45VP4FVRMEDM2SDVVF7VLANCNFSM54ANYDRA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Json gentx for felixschulz team

Can you please close this PR and recreate a new one? We have added a CI test to validate the gentx and it works on new PRs only.


docs: Table operation doc

Which issue does this PR close?
None
What changes are included in this PR?
add table operation document
Are there any user-facing changes?
no
How does this change test
just following the document

ok, I keep Chinese docs in my local repo.


inspector should auto-quit after ten minutes for the desktop version

... in case users leave it running for too long and affect their networks' performance

addressed feedback from Ben

Looks ugly for now because we don't have a transparent icon

Per #173 , copying @cdnicoll


Check if cheatsheets are up to date on CI

This PR resolves #370
Changes

Moved cheatsheet generate script to pkg/cheatsheet/generate.go.
Added pkg/cheatsheet/check.go to validate if cheatsheets are up to date.
Added GHA step to validate if cheatsheets are up to date on ci.yaml.

@jesseduffield This one is ready for review as well!

@jesseduffield Added a PR on lazycore: https://github.com/jesseduffield/lazycore/pull/3. Also, investigating why the failure on windows-latest ci. I saw lazygit doesn't have this problem, so let me know if this is a known issue.

@gusandrioli nice, I've merged that PR. There's a scripts/bump_lazycore.sh script you can run to bump it for you in this repo.
I'm not sure why windows-latest ci would be failing. I am aware of it being slow but not failing

@gusandrioli actually I bet it's because windows uses different line ending characters than unix. I'm not sure how that's possible when the script is only using newlines though

@jesseduffield No luck yet here. Seems like on lazygit it only runs check-codebase on ubuntu machines (and Check Cheatsheet within there). We'd probably have the same issue there. I'll probably separate the jobs here to match that.

Amazing work @gusandrioli ! Merging :)


Interpolation add missing reserved key

Add :exception_handler to the RESERVED_KEYS constant.
Based on https://github.com/svenfuchs/i18n/blob/master/lib/i18n/tests/interpolation.rb
# If no interpolation parameter is not given, I18n should not alter the string.
# This behavior is due to three reasons:
#
#   * Checking interpolation keys in all strings hits performance, badly;
#
#   * This allows us to retrieve untouched values through I18n. For example
#     I could have a middleware that returns I18n lookup results in JSON
#     to be processed through Javascript. Leaving the keys untouched allows
#     the interpolation to happen at the javascript level;
#
#   * Security concerns: if I allow users to translate a web site, they can
#     insert %{} in messages causing the I18n lookup to fail in every request.
#

Passing in a non-interpolation related custom exception handler for a translation would trip the interpolation behaviour when it is unexpected.
It took me a while to understand what was happening, I would suggest to rename the constants toward "interpolation" or move them in lib/i18n/interpolate/ruby.rb.

Thank you! I will look at merging this tomorrow when I am back at work.

@tjoyal Thanks again :) I think moving the constants closer to where they're used would be the right approach. I'd happily review another PR for that.


Procedure that does things and then returns?

Google Blockly has these two kinds of blocks:

They allow me to first do some code and then return a value. This is great for eg. finding things in lists. However, in AppInventor, there are only:

Which means I can either do something, or return a value. This severely limits the procedures a user can create and it seems very unergonomic to me. Is there a walkaround for this? Or would you consider adding the Blockly-style procedure?
Give an example of how this feature would be used

Why doesn't the current App Inventor system address this use case?
I hope it does and I just couldn't find it.
Why is this feature beneficial to App Inventor's educational mission?
Because doing something and then returning is a very basic feature present in virtually every programming language.

Oh, I completely missed the do-result block!

Disregard my last post, then!


SAS tokens instead of keys in app

It would be really cool if you could use a SAS token (generated elsewhere) instead of storing the account key in the application.
I have a backend where I generate SAS tokens but I don't really know what to do with them, so that's pretty much the functionality I am asking for :)

SAS token support is something I would like to add here... But with .NET 4.6 (Experiment) support looming I have held fire on developing this as it looks like we are close to being able to use the full SDKs now.
https://docs.microsoft.com/en-us/sandbox/gamedev/unity/azure-storage-unity

Makes sense. I have already jumped ship to 4.6 myself :)

The experimental Unity Azure SDK has been removed from the web, and from what I saw elsewhere, no longer works anyway.
So here's a quick and dirty modification that works for my needs and will hopefully save someone time:
        public void AuthorizeRequest(StorageServiceClient client, Method method, string resourcePath = "", Dictionary<string, string> queryParams = null, Dictionary<string, string> headers = null, int contentLength = 0)
        {
            AuthorizationHeaders authHeaders = new AuthorizationHeaders(client, method, resourcePath, queryParams, headers, contentLength);
            string stringToSign = authHeaders.ToString();
            this.AddHeader("x-ms-date", authHeaders.MSDate());
            this.AddHeader("x-ms-version", authHeaders.MSVersion());

            // NOTE: you'll need to add string SASParams to the client also
            if (client.SASParams != null)
            {
                this.Request.url = $"{client.Url}{resourcePath}{client.SASParams}";
            }
            else
            {
                string signature = GetSignature(client.Key, stringToSign);
                string authorization = GetAuthorizationHeader(client.Account, signature);
                this.AddHeader("Authorization", authorization);
                Debug.Log("Authorized request url:" + this.Request.url + "\n\nauthorization: \"" + authorization + "\"\nx-ms-date: " + authHeaders.MSDate() + "\nstringToSign:'" + stringToSign + "'");
            }

            if (headers != null)
            {
                this.AddHeaders(headers);
            }