Mudrod-135 Archive to use zip for windows compatibility

The server was having problems starting on windows machines because the javaSVMWithSGDModel was being compressed as a tar.gz
This pull request changes to use a .zip format for compression so that javaSVMWithSGDModel can be decompressed on windows/mac/linux
Fixes https://github.com/mudrod/mudrod/issues/135

@quintinali I made a few more minor changes; can you test one more time on your windows laptop and if it works I'll merge.

@fgreg  I tested the code on my computer and it worked well.


[circleci] normalize machine types, normalize shell env setup, re-enable code coverage - no move env right now.   not good.

Motivation
Make the executor names transparent.    Make the shell setup readable.   Re-enable code coverage.
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
CI, and a test branch for coverage, will post back here with it.
Test-1 branch build:
https://app.circleci.com/pipelines/github/diem/diem/36379/workflows/cf8cf36d-8bf1-448b-bba9-b6ecb2ae1be0/jobs/270030
Let's see if it completes
Related PRs
Young's pr.

Currently takes about 3 minutes to build install sccache.  Would be great to get x build pulled out the critical path and binaries stored somewhere, same with sccache.   As far as I can tell this isn't supported in any way by cargo -- we'd need a tool like rustup.

At this point move tools properly install (21 seconds) and coverage is a couple of hours in to its run.    This should be good, please stamp so I can land in a few hours.

Testing coverage back to correct values:
https://app.circleci.com/pipelines/github/diem/diem/36411/workflows/2bc4d1eb-4706-47ee-80a6-9724a4e132c5/jobs/270382
Currently takes 4 hrs 20 minutes, reminder that circleci has a hard cut at 5 hrs.
Notes:

don't try to bump the version of ubuntu we're running on -- this will need more investigation.
Set timeout for standard jobs to 70 minutes
move lang tooling/testing/coverage is back:

/priority high

/land

/cherry-pick release-1.1

/cherry-pick release-1.1

@sausagee :exclamation: cherry-pick failed, possibly due to conflicts. You can perform the cherry-pick yourself by running the following commands:
git fetch https://github.com/diem/diem.git release-1.1 088de0f3ac539b7cb93c47ca261d7a4d4e5e7cf5
git checkout release-1.1
git cherry-pick 7373f3dad1fc2fa3862df807f180a3e24042d10e..088de0f3ac539b7cb93c47ca261d7a4d4e5e7cf5

@sausagee :exclamation: cherry-pick failed, possibly due to conflicts. You can perform the cherry-pick yourself by running the following commands:
git fetch https://github.com/diem/diem.git release-1.1 088de0f3ac539b7cb93c47ca261d7a4d4e5e7cf5
git checkout release-1.1
git cherry-pick 7373f3dad1fc2fa3862df807f180a3e24042d10e..088de0f3ac539b7cb93c47ca261d7a4d4e5e7cf5


How to define options dynamically?

How to define directory option dynamically?
i18n(app, {
  directory: path.join(__dirname, 'config', 'locales'),
  locales: ['en', 'pt-BR'],
  modes: ['header']
})

Do you have a full example?


[Karyu densetsu] - defense and translate fixes

Changes / Comments
Fixing the Defense calculation and some translation issues.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Hello @danielsousat ! Your Pull Request includes changes to translation files. We will not deny your Pull Request, but we would prefer sheet contributors use Crowdin for translations. Please email Cassie at<EMAIL_ADDRESS>and she will get you an invite to our Crowdin

Ok, noted for future updates.


BRS-410: filter out published alerts

Jira Ticket:
BRS-410
Jira Ticket URL:
https://bcparksdigital.atlassian.net/browse/BRS-410
Description:
Only show advisories for published parks on the alerts page. There was an existing strapi filter, so API behaviour hasn't changed.

Looks good, just one question. Does the api call that 3rd parties use show all protected areas including the "unpublished" ones (point 2 in the ticket AC)? Maybe add an example of an api call demo'ing this to the ticket?

@DayKirby-OXD oh, that's a very good point, thanks. I'll add a note explaining that.


Issue with k8s.io/docs/admin/cluster-large/

Elastic and Kibana links are broken

[ ] Feature Request
[x] Bug Report

Problem:
Proposed Solution:
Page to Update:
http://kubernetes.io/...

/assign


State persistance

Is it possible to persist the state of the runtime so that it could be used in a cloud function for handling HTTP requests?

I'm not exactly sure what you mean. Do you mean saving the state of the project so it can be loaded/resumed again as if you were to save the project in the Scratch editor? Or do you mean hosting a single project on another webpage?

Some context: I'm experimenting using your scratch interpreter/runtime to run some non-visual projects in a shared web server instance (essentially using scratch to create the logic). To support multiple users, the runtime state has to be persisted and restored for each incoming HTTP request since I don't want to dedicate a runtime instance for each user.
This use case is of course very different from what you have designed it for. I'm using scratch ask/say for doing inputs/outputs. I have a hack where I replay the previous inputs to get the runtime back to the previous state for each incoming request. Having a way to persist the runtime state would be much more efficient.

phosphorus wasn't designed to be used as a library (sorry!), which makes things like this tricky, but you should be able to inspect the Stage instance and save/restore any object state you need from that.
Execution state is more complicated, but the only non-serializable thing in the runtime is the continuations. As long as you can guarantee that you're restoring execution state in exactly the same project, you can convert continuations to a sprite name and an index in that sprite's fns array. (Technically, the functions are serializable as long as you deserialize them with P.runtime.scopedEval, but that would inflate the size of the serialized state considerably.)
If you just want to store the state in memory, it's probably better to have an instance per user and cut out all of the drawing/display code, since that's really the only CPU- or memory-intensive thing.

Appreciate the thoughtful feedback for my crazy idea :)


Noblox cookie invalid

So what im trying to do is make a discord bot that pays people with a command. In my main code i set the cookie and in my code that is used to login, i used the nbx.cookielogin feature. It keeps saying that my cookie is invalid when i just made it.
If anyone would like to help me, please friend me on discord (AGR#2073) and i will be grateful

You must use setCookie not cookieLogin


CCI using 20 instead of input

The CCI is using the number 20 instead of the input, which is an issue for anyone who wants to use a CCI period other than 20. I change it manually to reference the CCI period I needed to use by replacing the number 20 with input.period and it worked, although there is probably some other way you prefer.
It needs to be changed to use the input provided by the programmer in these two places:
https://github.com/anandanand84/technicalindicators/blob/718cabf3d763109bb844569d348a3054b3d9d9e6/lib/oscillators/CCI.js#L40
https://github.com/anandanand84/technicalindicators/blob/718cabf3d763109bb844569d348a3054b3d9d9e6/dist/index.js#L1889

Hi, please make this fix.

Fixed and published

Thank you!


Provide lib/noStyles which doesn't need less-loader

I'm trying to use this in a project where I already use lodash and don't
want to add less-loader. Before this PR, my options were:

require('react-telephone-input/dist/react-telephone-input')
This includes a lot of lodash that I already include in my app.
require('react-telephone-input')
This requires that I add less-loader because index.js requires a
less file. I'd rather not do that for one dependency.

This commit introduces an alternative way to require:
require('react-telephone-input/lib/noStyles')
This allows applications to use this project without having to duplicate
lodash dependencies, and without having to run less-loader.
Fixes #34

Note that I didn't include built files in this PR. Let me know if you'd prefer that I added them.

Thanks @dmnd. Appreciate it.
I am getting merge conflicts and don't know enough git to resolve it in a pull request. Can you please send another pull request or educate me on how to resolve the merge conflicts on the pull request?

Let me rebase on master for you. One minute.

I feel the default import should be without the less file and with less should be a special file.  Like
require('react-telephone-input/lib/withStyles')

What do you say?

Yeah, I agree with that, but it's a breaking change, so you'd have to release version 3.

Yup. Ok, i will merge this change of yours, cut a minor release.
Then change it so that without less is the default export and cut a major release.

Ok just bear with me while I figure out this merge conflict :smile:

Ok, rebased.
Personally I wouldn't bother with releasing the minor release if I were you. Just merge, change it so that no styles is the default, and then release.
I don't think there's a use case for staying on v2 but wanting the no styles. Anyone who wants that should upgrade to 3.

That makes sense.

Awesome, thanks for merging so promptly! :bow:

Have cut a new release 3.0.0. Let me know if it works ok for you.
And thanks for this. I wanted to remove the lessify dependency for a long time.


[Bug]: Vite does not support shortcut keys and does not prompt press h for help

This seems to require vite support. At present, the plugin cannot know that the service is successfully started

v0.0.7


Update cache initial values with write-only accesses in CachedState::count_actual_storage_changes

Closes #1002
Depends on #1012 & #1011
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #1009 (0796ce4) into get_class_hash_at_zero_by_default (f6f7cf6) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@                          Coverage Diff                          @@
##           get_class_hash_at_zero_by_default    #1009      +/-   ##
=====================================================================
+ Coverage                              90.56%   90.58%   +0.02%
=====================================================================
  Files                                     54       54
  Lines                                  13797    13831      +34
=====================================================================
+ Hits                                   12495    12529      +34
  Misses                                  1302     1302

Files Changed
Coverage Δ

src/core/errors/state_errors.rs
50.00% <ø> (ø)

src/state/cached_state.rs
92.20% <100.00%> (+0.48%)
:arrow_up:


Modify user agent syntax

Update user agent builder syntax while building user agent header string

This isn't correct. I think we'll need to handle the # on the AWS side here.
For example the addProductName function calls
builder.AddKeyValue(aws.SDKName, aws.SDKVersion)

This would now be formatted as aws-sdk-go-v2#1.2.3 but it should still be aws-sdk-go/1.2.3. I think we'll have to handle the logic here instead. I suggest we update the tests on the AWS side and make the appropriate changes there instead.
I think we can also probably handle the valid chars on the AWS side and just leave smithy-go UA abstraction alone. That will be the least invasive change and keep the changes isolated to AWS.
The other question I have is do we implement support for the app ID related settings anywhere?

Nope, I plan to put app ID setting support in another PR cause we need to correct the UA syntax first.

Can we close this if we aren't going to use it?


Update RC date for 2.17.0

Description
Update RC date for 2.17.0
Issues Resolved
See https://github.com/opensearch-project/opensearch-build/issues/4908#issuecomment-2329863750
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@nateynateynate @kolchfa-aws @krisfreedain  Leaving up to you to merge when you see fit.


config.md: args of Hooks should be optional

Signed-off-by: Ma Shimiao<EMAIL_ADDRESS>

LGTM

LGTM

On Wed, Feb 15, 2017 at 02:12:48PM -0800, Mrunal Patel wrote:

Merged #685.

My reading of the language this landed is that:
{
…
"hooks": {
"prestart": [
{
"path": "/foo/bar"
}
]
}
}
would invoke /foo/bar with an empty/unset argument list.  Something
like:
$ head *.c
==> x.c <==
#include <stdio.h>
int main(int argc, char **argv) {
printf("invoked with argc = %d\n", argc);
return 0;
}
==> y.c <==
#include <unistd.h>
int main(int argc, char **argv) {
execv("./x", NULL);
return 1;
}
$ gcc -o x x.c
$ gcc -o y y.c
$ ./y
invoked with argc = 0
Was that the intention?
Or was the intention to restore the old Go semantics where ‘args’
falls back to ‘[path]’ if ‘args’ was unset 1?  If so, we probably
need a follow-up PR that updates the ‘args’ documentation to specify
the fallback.


Add aliases to multiple commands.

Changes
Aliases Added

Added "getadmin" as an alias for the "GetScript" command. :getadmin is more commonly used than :getscript.
Added "dban" as an alias for the "DirectBan" command. Quickens command usage.
Added "dunban" as an alias for the "UnDirectBan" command. Quickens command usage.

Credits Added

Added "@AidenJamesYt" to "Contributors" as an Open Source Contributor.

this is not contributor worthy

Added "getadmin" as an alias for the "GetScript" command. :getadmin is more commonly used than :getscript.

This is confusing and potentially conflicting

Added "dban" as an alias for the "DirectBan" command. Quickens command usage.
Added "dunban" as an alias for the "UnDirectBan" command. Quickens command usage.

Is this really necessary?


Activity subgrid - description field is rendering as HTML text

After upgrade to V 8,
In portal description column of Activity subgrid  is rendering raw HTML.

By compairing with older version , there seems to be difference with the way html and text content has been handled in entity-grid.js.
Latest (V.8)
var html = $this.formatAttributeValue(attribute, attribute.AttributeMetadata, j, record, data);
if (html.isHtml) {
$td.html(html.value);
} else {
$td.text(attribute.DisplayValue);
}
Older Version(7.)**
var html = $this.formatAttributeValue(attribute, attribute.AttributeMetadata, j, record) || attribute.DisplayValue;
var $td = $("")
.attr("data-type", attribute.Type)
.attr("data-attribute", attribute.Name)
.attr("data-value", typeof attribute.Value === 'object' ? JSON.stringify(attribute.Value) : attribute.Value)
.html(html);
please let me know if its bug ?

@mawankhade do you have an update on this?

sorry for delayed response.
No I did not get chance to try it on online portal.
My guess on current implementation is  they are assuming that user will not show email body column in subgrid and that's why they are not handling email body type column as HTML.
for now we have modified code to handle every content as HTML and this seems to be working for us.
Thanks,
Mahesh

If you find this works differently in online portals and want to contribute your local code change feel free to add more information to this issue. I will close this issue for now.


Add overloads for jsonOf and as

To allow specifying explicit decoders.

@OlegYch had some objections to using overloads on Gitter: https://gitter.im/http4s/http4s?at=59a489fc162adb6d2e437871

I tried implementing JsonDecoder and decodeJson.
https://github.com/aeons/http4s/commit/a82b3b75bf93d235a7dc389a78f0dbb99dac48ce

Where do we stand on this?  Are we leaning toward a82b3b7?

I think it works well, and I don't mind specialcasing JSON, since its so common.

Okay, can you get that commit into a PR that passes?

I'll just force push into this one.

Added some tests and instances for argonaut and json4s as well.

See #1396


[4.X] use json stringify instead of regex in table field

REFS: #2840

A new inspection was created.


Random Furniture Items are White (Missing Material Color)

Sorry for the lack of error message but I couldn't find anything in the log about why this might be happening.  It happens with no other mods installed except prepatcher.
If it helps, here's the furniture items I've noticed.  Most of everything else is fine.

Slab Beds
Small Research Bench
Stonecutter Bench
Artist Bench

Can you post a hugslib log?

Sure what is the log location?

No log location, it just lets you generate one. Install Hugslib mod, enable
dev mode, press ctrl-f12 or ctrl-shift-f12
вт, 6 февр. 2024 г., 00:06 Slemaire-PNI @.***>:

Sure what is the log location?
—
Reply to this email directly, view it on GitHub
https://github.com/Taranchuk/FasterGameLoading/issues/3#issuecomment-1928192770,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF5SDFJSK25YZ5F3JC2GX6DYSFJX3AVCNFSM6AAAAABC223QAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYGE4TENZXGA
.
You are receiving this because you commented.Message ID:
@.***>

Hugslib says unable to establish connection, probably regional or a firewall somewhere.  Is it just the player.log file it uploads?  Here it is if it's helpful.
Player.log
Pic

I misspoke earlier, the one mod enabled was Harmony.  And the problem is with harmony - when harmony isn't in the modlist everything works fine.  This only happens with Harmony and only when the second option is enabled.

What second option?
вт, 6 февр. 2024 г., 02:55 Slemaire-PNI @.***>:

Hugslib says unable to establish connection, probably regional or a
firewall somewhere. Is it just the player.log file it uploads? Here it is
if it's helpful.
Player.log
https://github.com/Taranchuk/FasterGameLoading/files/14173399/Player.log
Pic
Untitled.png (view on web)
https://github.com/Taranchuk/FasterGameLoading/assets/60244073/846a63f9-f3bd-4473-882b-55bff069b94b
http://url
I misspoke earlier, the one mod enabled was Harmony. And the problem is
with harmony - when harmony isn't in the modlist everything works fine.
This only happens with Harmony and only when the second option is enabled.
—
Reply to this email directly, view it on GitHub
https://github.com/Taranchuk/FasterGameLoading/issues/3#issuecomment-1928582039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF5SDFPTWNYTA5Z4HX35GCLYSF5QNAVCNFSM6AAAAABC223QAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYGU4DEMBTHE
.
You are receiving this because you commented.Message ID:
@.***>

The second feature of faster game loading, in it's options.
On Mon, Feb 5, 2024, 5:13 p.m. Taranchuk @.***> wrote:

What second option?
вт, 6 февр. 2024 г., 02:55 Slemaire-PNI @.***>:

Hugslib says unable to establish connection, probably regional or a
firewall somewhere. Is it just the player.log file it uploads? Here it
is
if it's helpful.
Player.log
<
https://github.com/Taranchuk/FasterGameLoading/files/14173399/Player.log>
Pic
Untitled.png (view on web)
<
https://github.com/Taranchuk/FasterGameLoading/assets/60244073/846a63f9-f3bd-4473-882b-55bff069b94b>

http://url
I misspoke earlier, the one mod enabled was Harmony. And the problem is
with harmony - when harmony isn't in the modlist everything works fine.
This only happens with Harmony and only when the second option is
enabled.
—
Reply to this email directly, view it on GitHub
<
https://github.com/Taranchuk/FasterGameLoading/issues/3#issuecomment-1928582039>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AF5SDFPTWNYTA5Z4HX35GCLYSF5QNAVCNFSM6AAAAABC223QAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYGU4DEMBTHE>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Taranchuk/FasterGameLoading/issues/3#issuecomment-1928596659,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLUA2P2ND5EWBPXP5I2IPLYSF7TDAVCNFSM6AAAAABC223QAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYGU4TMNRVHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Okay, what's the name of the option? It's been a while really since I worked on the mod

Okay, the log looks weird, I suppose it's not a steam RW version? Is it a pirate version? Could you try to make sure you are on the latest game version and that the game is installed correctly?

It's the latest version.  I got this from a key site that's why it's not in
the steam path.  I haven't seen any updates yet, this is the latest
download.
On Mon, Feb 5, 2024 at 6:20 PM Taranchuk @.***> wrote:

Okay, the log looks weird, I suppose it's not a steam RW version? Is it a
pirate version? Could you try to make sure you are on the latest game
version and that the game is installed correctly?
—
Reply to this email directly, view it on GitHub
https://github.com/Taranchuk/FasterGameLoading/issues/3#issuecomment-1928662396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLUA2MUVFPO2KY5PDZC2WDYSGHQRAVCNFSM6AAAAABC223QAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYGY3DEMZZGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

So I can't reproduce this issue on my end. With a minimal mod order and with all the mod options enabled.

I would say, issue lies in your game version/install.


Use a different data structure other than array for more reliability

Check out lines https://github.com/JAYKRISHNAN/calculator_using_javascript/blob/master/src/calculator.js#L24-L26. They rely on the absolute position of elements in an array to work. You can switch to something more robust which does not depend on something like position of an element in an array.

used hash instead

Let's use the correct technical term - object literal (not hash)


Train fail

Hello, Have you face the same problem below?
File "train.py", line 306, in
sample(ep, it)
File "train.py", line 251, in run
im.imwrite(sample, '%s/Epoch-%d_Iter-%d.jpg' % (save_dir, epoch, iter))
...
'images should be in the range of %s!' % (l + ',' + r)
AssertionError: images should be in the range of [-1.0,1.0]!
It's my download data problem?

@GilbertTam What modification did you made?

I just run train ,no parameters change
when epoch is 15 will fail

@GilbertTam Do the images in the output folder make sense? I just rerun the code but haven't got any error.

hi,i have the same problem,no parameters change.
run in cpu mode,It's problem?
python 3.6.9
tensorflow-gpu 1.15.0
cuda 10.2
cudnn 7.6
error output:
Inner Epoch Loop:  40%|█████████████████████████████████████████████████████████| 99/250 [06:59<10:40,  4.24s/it]
Epoch Loop:   3%|████                                                                                | 14/500 [4:09:19<144:15:05, 1068.53s/it]
Traceback (most recent call last):
  File "train.py", line 306, in <module>
    sample(ep, it)
  File "train.py", line 251, in run
    im.imwrite(sample, '%s/Epoch-%d_Iter-%d.jpg' % (save_dir, epoch, iter))
  File "/home/git_prj/AttGAN-Cartoon-Tensorflow/imlib/basic.py", line 23, in imwrite
    iio.imsave(path, dtype.im2uint(image), quality=quality, **plugin_args)
  File "/home/git_prj/AttGAN-Cartoon-Tensorflow/imlib/dtype.py", line 51, in im2uint
    return to_range(images, 0, 255, np.uint8)
  File "/home/git_prj/AttGAN-Cartoon-Tensorflow/imlib/dtype.py", line 32, in to_range
    _check(images, [np.float32, np.float64], -1.0, 1.0)
  File "/home/git_prj/AttGAN-Cartoon-Tensorflow/imlib/dtype.py", line 27, in _check
    '`images` should be in the range of %s!' % (l + ',' + r)
AssertionError: `images` should be in the range of [-1.0,1.0]!

On this issue,I changed it here ,Then it run.
# File "train.py"
# run(epoch, iter)
try:
    im.imwrite(sample, '%s/Epoch-%d_Iter-%d.jpg' %
                       (save_dir, epoch, iter))
except:
    print('ERROR: Could not save %s/Epoch-%d_Iter-%d.jpg' %
                  (save_dir, epoch, iter))

@millionsofluo I am still confused by this problem. Is there any good results in the output folder.

@LynnHo Yes,This folder has good results,Very strange.
Epoch-11_Iter-249.jpg  Epoch-2_Iter-99.jpg   Epoch-9_Iter-149.jpg
Epoch-19_Iter-49.jpg   Epoch-4_Iter-199.jpg
Epoch-21_Iter-149.jpg  Epoch-7_Iter-49.jpg


Add PyUnfold to conda-forge

This will give users the option to conda install PyUnfold

See conda-forge/staged-recipes#5792

It's in there https://anaconda.org/conda-forge/pyunfold :tada:


Deduplication when using ReplicatedMergeTree with delete+insert incremental strategy

Describe the bug
Steps to reproduce

build an incremental model using Replicated merge tree and delete+insert strategy
populate the model as full refresh
re-run as incremental
re=run as incremental

Expected behaviour
step 4 should delete the records inserted in step 3 and reinsert them
Actual behaviour
The records are not reinserted in step 4
Explanation
It looks like when you insert exactly the same set of records in the incremental update, the replication deduplication mechanism kicks in and ignores the new block of rows.
I am working around this by adding an alter table... modify setting hook, but it is not ideal. It would be nice if I could specify this as a setting when the table is created, or alternatively that it is set automatically when using the delete+insert strategy, as otherwise it wouldnt be guaranteed to work.

Just to clarify, the problem is that in Step 4, the lightweight  DELETE occurs, but the reinsert does not because of built in ClickHouse deduplication, so those records are just missing?
It seems reasonable to disable deduplication for the destination table, I'll take a look.

Yep thats what seems to happen.

For the short term it seems like intead of using a hook you should be able to add the insert_deduplicate: 0 into the model config settings, which should be automatically picked up by this code:
 @available
    def get_model_settings(self, model):
        settings = model['config'].get('settings', dict())
        res = []
        for key in settings:
            res.append(f' {key}={settings[key]}')
        return '' if len(res) == 0 else 'SETTINGS ' + ', '.join(res) + '\n'

Is that a fix you can validate quickly?
My current thought is to add insert_deduplicate: 0 automatically into this method as a fix, and have an override configuration value if the user really wants to leave ClickHouse deduplication on.

Ah that looks like a neater solution. Give me 15mins to try it out.

16:44:09  Database Error in model int_sales__daily__external2 (models/intermediate/sales/int_sales__daily__external2.sql)
16:44:09    :HTTPDriver for http://clickhouse-prod.dragonite.venatusmedia.com:8123 returned response code 404)
16:44:09     Code: 115. DB::Exception: There was an error on [chi-repl-04-23-ch-prod-0-1:9000]: Code: 115. DB::Exception: Unknown setting insert_deduplicate: for storage ReplicatedMergeTree. (UNKNOWN_SETTING) (version <IP_ADDRESS> (official build)). (UNKNOWN_SETTING) (version <IP_ADDRESS> (official build))

For reference I added it like this :
{{
  config(
    ...
    settings={
      'insert_deduplicate':0
    }
  )
}}

Oh, these are table level settings... I figured these were settings on the INSERT?

Yeah the above works... thanks... I wasnt aware you could set the table SETTINGS this way.

Probably need the settings option in the readme?

cool, that seems like a reasonable solution.  And yeah, I think that needs to be called out better in the documentation.

Still maybe worth defaulting to this option for this strategy, do you think?

Yes, I think it's a good default for any of the dbt incremental strategies.  The ClickHouse automatic deduplication is sort of an unexpected gotcha in such a high level tool.


New stage describing how to model protein families in pathways

From a curation report issue, how to best model protein families in pathways. Either use Pfam id on a single node representing the protein family, or list out all members of the protein family (if feasible).

Having InterPro identifiers work would be great anyway. But what I read at the Interpro website is that they will host PFAM which sounds different from using Interpro identifiers instead. Does anybody know how that really works.

It seems like PFAM is still actively providing content, but this will be found via the interprot webpage only: https://xfam.wordpress.com/2022/08/04/pfam-website-decommission/
In the interprot search, you can then see a list of results coming from the different sources.

For this particular case, the best match I could find is this InterPro identifier, for Ribosomal protein S6 kinase:
https://www.ebi.ac.uk/interpro/entry/InterPro/IPR016238/
I added a stage here: https://academy.wikipathways.org/stages/draw-protein-families/ (not yet integrated in the path). Please review.

Looks good! Only the upload doesn't work yet, is that intended? I got the following error: Oops! That doesn't look quite right. Please try again. Incorrect number of objects: 5 detected, 0 expected.
Are there plans to to include the data mapping at some point? Would be great if the family could be connected to the actual proteins somehow.

Thanks @danidi! There was a typo in the gpml validation, it is fixed now.
For the data mapping, there is no plan to make that work as far as I know. These instructions were only meant to solve the issue raised in the curation report, basically the alternative to leaving it empty. I can to add a comment to the task that data mapping from individual proteins that are part of the family won't work, and maybe also describe the alternate approach of adding individual proteins as a stack of nodes off to the side of the pathway (like we do with other groupings or genes/proteins)?

On second thought, Im not sure this should be a stage in the Academy. Although the idea to use an Interpro ID instead of leaving the xref blank is still valid for individual cases (for example the original question by Javi), it's potentially counter-intuitive and confusing as a stage in the Academy since it doesn't enable data mapping at all (at least in PathVisio, or in a straight-forward way in Cytoscape). We can keep this issue open for discussion, but Im not going to add the stage to the path for now.

I think that that is fine for now. But it is one of the ideas that often come up in discussions about sequencing data to functionally evaluate sequencing data from multi-species mixtures, e.g. microbiome samples. If we can assign motifs in. sequences to functional protein motifs, and through that to pathways we could in principle evaluate the functionality or the functional capacity of such a mixture without assigning the sequences to species or complete genes. Of course we do not even have complete methods for that yet indeed.


io.seata.core.rpc.netty.NettyRemotingServer#registerProcessor代码建议优化

Why you need it?
Is your feature request related to a problem? Please describe in details
将硬性编码转成用SPI进行解耦和扩展,使用getMessageTypes()方法来获取类型,进行注册到processorTable中
How it could be?
A clear and concise description of what you want to happen. You can explain more about input of the feature, and output of it.
List<RemotingProcessor> processors = ServicLoader.load(RemotingProcessor.class); .....
Other related information
Add any other context or screenshots about the feature request here.

你这么做的目的是什么？

为了进行扩展和版本迭代不需要改动原来代码,只需要更改SPI的文件,减少硬性编码

server 端的 spi 意义不大吧。

@juckTom So according to the figure above ，we need 5 types of SPI to load?


Add Navigation methods for InterSystems Objects Indexed properties

InterSystems Objects has the feature to get the Id of instance for a property value if the property has Unique Index.
e.g.:
ClassMethod IndexOpen(val, concurrency, sc As %Status)
and
ClassMethod IndexExists(val, Output id) As %Boolean
Learn more in this article: https://community.intersystems.com/post/useful-auto-generated-methods
This works only for Unique indexes.
It'd be great if we have auto-generated methods for navigation on instances which have same values for the property with the option to get the all the id for the property value.

We'll need navigational methods such as:
First(), Next(), Previous() and Last() both on the IdKey and on Indices.
So something like:
##class(Person).Next(IdKey)
##class(Person).Next(KeyValue)
Multi-level indices would then be a little harder
I've discussed this many years ago with ISC developers but they were very reluctant to implement this. The argument was to use SQL for this. (I didn't/don't agree)

@hermanslagman What is the point of having First, Next, Previous and Last for IdKey, if it is unique and they all should return the same value?
I'm curious now, how one can add methods into the system class like %Persistent or wherever the methods for working with indices are realized?

see the article on this topic https://community.intersystems.com/post/indexnameopen-and-navigation-methods-non-unique-indexes


CheckMK - Grafana - Process Discovery

Hi,
I have been using CheckMK and the corresponding CheckMK-Grafana Plugin for visualizing a sort of Network/Host Monitoring. And now i ran into a problem.
I’ve added the checks for some processes on our Windows Server 2019 machines. While the processes are running everything works as expected and i can visualize the Check in Grafana. (Green for process is running). Once the process is not running, i would assume that the check would go red, since there is no process running but grafana throws an exception that it “cannot read property “meta” of undefined”.
i tried the enforced service and it is working more consistently, but there is still 1 problem.
If the process is running and you select the metric within Grafana, the number of processes is at position 5 or so within the possible metrics. If the process is not running there is only 1 metric left and so Grafana can not find the 5th metric within the service and and therefore throws another exception. “e is undefined”.
We’re running on CheckMK-Raw-2.0.0p7.
Kind Regards
Kai Jakobsen

I have also opened up a troubleshooting post in the CheckMK Forum.
https://forum.checkmk.com/t/checkmk-grafana-process-discovery/26247/4

Can you still reproduce this issue with Checkmk >= 2.1.0 and the current stable release of this plugin?


Response Query Feature Not Implemented?

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
The Response Query feature does not seem to be implemented. According to the docs this feature should exist. I attempted to use it in the Post Response Script and it throws the following error
Error invoking remote method 'send-http-request': TypeError: res is not a function

Looking through the source code I could not find where this is implemented. It looks like the res object does not have this capability based on the script or testing paths which the docs suggest it should be implemented. This also might be a matter of the docs not being accurate either.
.bru file to reproduce the bug
No response
Screenshots/Live demo link

Looks like there is a PR out for this; 3710. The docs might be ahead of the feature.

Closing as duplicate; https://github.com/usebruno/bruno/issues/481


Add HTML download attribute to Davrods links

Feature
Assaysheets inline file links as well as links of the collection "list files" popup should set the HTML "download" attribute to signal browsers not to try opening a potentially huge file as a new tab. The download attribute is recognized by all major browsers. Probably sensible to exclude PDF files as many browsers have embedded PDF viewers.
Useful because
Users can simply click RAW data file links in SODAR instead of Rightclick > Save as

I see some issues with this:

The links can point to other servers than DavRods, which means we'll need to check and modify each link
Davrods also hosts a lot of small text files, HTML reports etc which may be linked and the user may actually want to see in the browser
If the file name is used as a label, the user can see it's a raw data file. At that point, right click and "save as" should not bee too much of an effort.


Borehole Drilling Method: minor fixes

These changes look major due to RDF reformatting but that are extremely minor: I have just fixed a small issue with an orphan notation for Diamond Core Drilling and added a namespace prefix for CGI terms to make the vocab source more readable.

@nicholascar After merging and deploying to DEV and PROD, I noticed that the new scripts get rid of the concept list

I am going to revert until the issue is solved


V1.0.2

Coverage remained the same at 100.0% when pulling 1d240dfc06d6e4f02c07523137e3506dd8cf50d2 on v1.0.2 into c9f6f39f15a9c6643974aade0bd8a2bd5302e77e on master.


Fix renv conflicts

Fixing renv conflicts and adding the required nphRCT package

Hey @sharonav8
The pull request you created for CAMIS is great, but one slight issue was that there was a number of versioning conflicts regarding renv. This was likely due to the fact CAMIS was temporarily migrated to R 4.3.1 to make it easier for us at Novartis to replicate the CAMIS R environment from within DaVinci during the Hackathon, however, this has since been reverted back to R 4.4.0, which means renv has created a number of versioning conflicts within your PR.
Another small issue is that the nphRCT package required in your script was not recorded.
Don't worry too much about this as renv is quite tricky - especially if you've not come across it before - and fixing this issue definitely deviates from the main focus of this Hackathon which is Git, GitHub and open-source contribution!
So in order to fix this I've gone ahead and fixed the conflicts and added the missing packages to the renv.lock file.
If you accept this pull request, it should update your pull request in the main CAMIS repository and we can see if these fixes help!
Any issues or confusion let us know in the Hackathon Teams channel!


Update api 21 and update version lib

When i execute gradlew build, there is an error, please check it.


{AppService} Fix linter error in _github_oauth.py

Description
For unknown reason, Pylint 2.3.0 fails to find linter errors in _github_oauth.py added by https://github.com/Azure/azure-cli/pull/17826.
After bumping Pylint 2.8.0 (https://github.com/Azure/azure-cli/pull/17861), those errors are revealed.
https://dev.azure.com/azure-sdk/public/_build/results?buildId=880573&view=logs&j=36dd4138-4d53-5e46-00d9-e5cd9744be05&t=1cf3879c-0a42-5ccd-7693-7a4781d739d8&l=107
src/azure-cli/azure/cli/command_modules/appservice/_github_oauth.py:20:28: W0613: Unused argument 'cmd' (unused-argument)

AppService


Bukkit 1.10.x - 1.12.x and 1.13.2 and performance / readability improvements

This PR makes a variety of changes to the plugin in order to have it function for 1.13.2 and onward without requiring any further updates.

Adds support for Minecraft 1.13.2+ (ONLY THE LATEST VERSION)

This is made possible by a Bukkit PR merged this morning (February 28th)

Update the Java version to Java 8 (it has become standard in the Minecraft community). Near 100% of servers use it, or otherwise they've updated to Java 9+
Various changes to the pom.xml(s)

Removed the repository directing to an illegal mirror of the CraftBukkit and Spigot server jars

To obtain the server jars, BuildTools should be run and it will be locally installed to the .m2 repository

Removed redundant <type>jar</type> declarations
Removed the ../target build directory as it is deprecated by Maven and unsupported by Eclipse
Removed unnecessary dependencies on the Spigot API and Bukkit API. Spigot-exclusive APIs were not used and Bukkit is shaded into CraftBukkit
Removed redundant declarations of groupId in children poms
Update the Maven Shade Plugin from 2.4.3 to 3.2.1
Update generic dependencies to 1.13.2 (as version-exclusive features are not used, but having the latest version retains project maintainability)

Added WildAnimalHandlerSafe as a default implementation used by 1.13.2+ servers... this uses the new API methods

The following are changes made exclusively to specific classes

WildSex.java:

Declare lastMateAnimals as a Set rather than a Set for memory leak reasons and clear it on disable
Let wildSexTask be a BukkitTask rather than an int. These scheduler methods are deprecated, therefore BukkitRunnable instances should be used instead
Removed some unnecessary fields that were not used outside of the onEnable() method
Ensure no NullPointerExceptions are thrown on disable for unsupported servers

WildSexTask.java:

Added a ELIGIBLE_MATES_BUFFER List constant to buffer entities rather than creating new ArrayList instances for every iteration.

Slightly increases performance and reduces the overhead of ArrayList resizing by defining an initial size of 64

Unnested numerous conditions... better readability
Use streams in place of for loops where it makes sense (i.e. entity filtering and buffering)
Extract the eligible mate condition to a method
Use Math#max() instead of ternary in #getReproductionProbability()

WildSexTaskListener.java:

Remove the unencapsulated retrieval of lastMateAnimals and use proper methods instead
Declare variables for father and mother
WildAnimal.java:
Declared a LOVEMODE_TICK_DURATION constant for readability sake in version-specific implementations

This PR also removes the dependency-reduced-pom.xml which was left in the repository by mistake. This is an IDE-specific file and should not be present.

The reason the Travis build is not working properly is due to the local dependency on CraftBukkit and Spigot, though this should be excused for the sake of complying with Minecraft's EULA of not distributing the server jar (which is being done by the mirror repository). I would advise finding an alternative such as a private repository for Travis, but the server jar should not, under any circumstance, be distributed by a Maven repository.

This is a very comprehensive PR and I really appreciate the time you put into it - I am not around as much as I used to be so I really appreciate the contribution.
The changes mostly look good to me and you're definitely right about the pom.xml dependencies. I am also happy that Spigot finally appears to have set up a proper API for the love mode, saving us the trouble of adding updates without new features just for new version matches. Have you had a chance to test this with any of the earlier versions? It definitely will compile but I'm not knowledgeable as to what happens when the plugin depends on an API level at compile time that is not available at runtime. If there are any issues regarding this it can be fixed by putting this futureproof version as a subproject like all of the NMS ones.
Before we merge this PR though, I would like to take a couple of days to try and find time to integrate BuildTools into the travis flow so that it can build the plugin without needing to find the server jar on a maven repository. This should be fairly straightforward but it's important for this to function right since automatic releases on GitHub are set up using travis.
Also, I wanted to point out that if you are interested in becoming the plugin's official maintainer (or at least a contributor), I would be happy to add you as a contributor onto the repo and the plugin page.

I am also happy that Spigot finally appears to have set up a proper API for the love mode, saving us the trouble of adding updates without new features just for new version matches
It's something I pull requested to Bukkit for this very reason. Someone on the SpigotMC forums was looking to update this plugin themselves to 1.13 so I figured I'd take the liberty to do it for them and have it be future proof so others wouldn't have similar issues

It definitely will compile but I'm not knowledgeable as to what happens when the plugin depends on an API level at compile time that is not available at runtime
If for some reason the plugin is run on a version of Bukkit that do not have this API, (say 1.13.0, 1.13.1 or early versions of 1.13.2) because these people will inevitably exist, the plugin will throw a NoSuchMethodException. While not ideal, Bukkit doesn't really provide a friendly way of checking API version by commits. Yes, there is Bukkit#getBukkitVersion(), but this returns a commit hash which isn't exactly easy to compare whether one precedes the other. I'm open to changing this if a solution is provided, but as far as I'm aware it's not possible.

That being said, server owners are expected to update their servers regularly (hence the reason there is a server startup delay for server builds older than a few days).

Also, I wanted to point out that if you are interested in becoming the plugin's official maintainer (or at least a contributor), I would be happy to add you as a contributor onto the repo and the plugin page.
In truth, I'm not sure the project will require a maintainer if this PR were to be merged. Due to this project's simplicity and the fact that there is now a proper API for this, it's not likely it will break in the future. Bukkit strives to be as forwards compatible as possible.

Good job, Choco. I am not around as often as I used to be to help Sultan also. Many people request this plugin from me still on Bukkit and Spigot and I'm sure they will be ingratiated to see updates.

Sorry, I realized I dropped the ball on this again, I have some time this weekend that I intend to use towards this end.
I believe I know how to leverage the current setup to use the Bukkit API for newer versions and our makeshift API for older ones. Will try that, and once that is done we should hopefully have full backward and forward compatibility.


Couldn’t sign you in error for Enterprise Gmail

Application version: 3.1
OS: MacOS

Steps to Reproduce:
I cannot get Enterprise Gmail account to work.
Non-enterprise Gmail works fine.
Other enterprise google apps (calendar, chat) work fine.
Screenshots

still occuring on 3.2


fix: npmignore not needed as we use pkg.files

.npmignore isn't used when pkg.files is defined

Excellent!


use ActiveSupport::HashWithIndifferentAccess if available

similar to https://github.com/binarylogic/settingslogic/pull/52, but preserves 'classic' hash if method with_indifferent_access not available

@aphilippi Is it possible to relax the activesupport requirement? Also a good time to update to 4.2


Feat: Add support for Azure OpenAI api key

Problem Statement
For azure openai api, authentication by api key is not supported.
Proposed Solution
I propose to support this use case by adding small modification to the function below.

promptwizard/glue/common/llm/llm_mgr.py > call_api()

Development Consideration

Please let me know if there's anything I need to take into account, besides modifying the file above.

Related Issuses

https://github.com/microsoft/PromptWizard/issues/18

Hi,
Thanks for your comments. By changing the following function any LLM calling resource can be used https://github.com/microsoft/PromptWizard/blob/60504476f8231a7cb0fc3182101a86014e3b73ee/promptwizard/glue/common/llm/llm_mgr.py#L17


v1.6.0 fails to add directory to archive

After the v1.6.0 update, I have been unable to add a directory to the archive with--include. This command:
cargo lambda deploy --include target/site

produces the following erorr:
Error:   × failed to add directory `target/site` to zip file
  ╰─▶ invalid Zip archive: Duplicate filename

The directory wasn't previously added since the archive is crated with this command. This has also worked for previous versions, so it seems to be a regression in v1.6.0.
If more information is required, I can put together a minimal reproduction.

Ugh, thanks for opening this issue. I think it should be easy to reproduce with this information, but I’ll let you know what I find.

Sorry about the trouble. I've identified the problem it should be solved in #756. I'll release a new version tomorrow morning PT with that patch.

Oh awesome, thanks!

I just released 1.6.1 with the fix: https://github.com/cargo-lambda/cargo-lambda/releases/tag/v1.6.1


Add cargo deny to project itself

Add cargo-deny to the repository itself to make sure that compiles stay fast and that all dependencies are compatible with the license.

Closed by #31.


GDAX trade exit date 1970

Note: this is the technical bug tracker, please use other platforms for getting support and starting a (non technical) discussion. See the getting help page for details.
I'm submitting a ...
[ X] bug report
[ ] question about the decisions made in the repository
Action taken (what you did)
GDAX live trading with the
tulip-macd strategy
Expected result (what you hoped would happen)
a round-trip with an exit date greater than start date
Actual result (unexpected outcome)
exit date 1970...??

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

Would you be able to attach the logs (you can comment out amounts and such)? Thanks!

Here's the tradelog:
2018-07-31-07-05-tradebot-285063869057231.log

Interesting, order creation was rejected at some point. Definitely a bug in
Gekko's handling of rejected orders.
As to why it was rejected, it's not clear from the logs but: was there
anyone or anything also trading on gdax at the time this happened? Also,
what kind of internet connection do you have & in what country are you
based?
On Wed, Aug 1, 2018 at 3:18 AM, abmbc<EMAIL_ADDRESS>wrote:

Here's the tradelog:
2018-07-31-07-05-tradebot-285063869057231.log
https://github.com/askmike/gekko/files/2246766/2018-07-31-07-05-tradebot-285063869057231.log
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/2390#issuecomment-409336682, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MD9bBgpb2vgyg8G1-F_tcc-0_DrHyks5uMK2dgaJpZM4Vn3TR
.

--
PGP key at keybase.io/mikevanrossum
https://keybase.io/mikevanrossum/key.asc

I live in Britain, my internet connection is wireless to my router which leads out via DSL
I was monitoring the exchange on the same PC from my coinbasePro account, other than that there was nothing else.

Thanks for the response. I will definitely have a look why Gekko did not
process the order rejection!
On Wed, Aug 1, 2018 at 5:20 PM, abmbc<EMAIL_ADDRESS>wrote:

I live in Britain, my internet connection is wireless to my router which
leads out via DSL
I was monitoring the exchange on the same PC from my coinbasePro account,
other than that there was nothing else.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/2390#issuecomment-409510437, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MD4BtRW_zi0JWLNQgfwOA7dmpSr4Nks5uMXLKgaJpZM4Vn3TR
.

--
PGP key at keybase.io/mikevanrossum
https://keybase.io/mikevanrossum/key.asc


Payment error: txn-mempool-conflict (Quick Mode)

In the wallet quick mode, sending two transactions in a row quickly can result in txn-mempool-conflict:

This was fixed, is it still occuring?
What indexer version did you use?

Not 100% sure, but I think it was blockcore and that runs latest version. It might have been one of the other 3 that is available though...

Is this the change? This is not released yet: https://github.com/block-core/blockcore-indexer/commit/26e504b83e1610abb20b1b4dcb677430a4868e4d

This is still an issue, the client appears to receive utxos from indexer that is in mem-pool and attempting to use that for a new tx.

Ah so its an indexer issue?


Fix small typing mistake

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Adding option to dump mappings to files

without this feature we're missing an option to print all mappings registered for servers
with this feature it will be much easier to debug what was registered in which http server stub. The mappings will be dumped to files with name artifactName_registeredPort
fixes #355

+1

Codecov Report

Merging #356 into master will increase coverage by 0.07%.
The diff coverage is 77.77%.

@@             Coverage Diff              @@
##             master     #356      +/-   ##
============================================
+ Coverage     59.35%   59.42%   +0.07%
- Complexity     1603     1616      +13
============================================
  Files           221      221
  Lines          6731     6773      +42
  Branches        889      894       +5
============================================
+ Hits           3995     4025      +30
- Misses         2235     2244       +9
- Partials        501      504       +3

Impacted Files
Coverage Δ
Complexity Δ

.../cloud/contract/stubrunner/NoOpHttpServerStub.java
66.66% <0%> (-8.34%)
6 <0> (ø)

...loud/contract/stubrunner/junit/StubRunnerRule.java
48.8% <100%> (+1.24%)
11 <1> (+1)
:arrow_up:

.../contract/stubrunner/StubRunnerOptionsBuilder.java
86.04% <100%> (+0.33%)
32 <1> (+1)
:arrow_up:

...act/stubrunner/spring/StubRunnerConfiguration.java
86.48% <100%> (+0.37%)
8 <0> (ø)
:arrow_down:

...ntract/stubrunner/spring/StubRunnerProperties.java
53.84% <100%> (+0.65%)
18 <3> (+2)
:arrow_up:

...nner/provider/wiremock/WireMockHttpServerStub.java
79.68% <100%> (+2.1%)
26 <3> (+3)
:arrow_up:

...ramework/cloud/contract/stubrunner/StubServer.java
89.28% <100%> (+0.39%)
12 <1> (+1)
:arrow_up:

.../cloud/contract/stubrunner/StubRunnerExecutor.java
81.1% <100%> (+0.14%)
52 <1> (+1)
:arrow_up:

...ramework/cloud/contract/stubrunner/StubRunner.java
77.55% <63.15%> (-9.55%)
13 <1> (+2)

...k/cloud/contract/stubrunner/StubRunnerOptions.java
60.41% <66.66%> (-0.05%)
14 <2> (+2)

... and 1 more

Continue to review full report at Codecov.

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


adding user 1001 throws EACCES permission issue

I tried adding       options: --user 1001 to ensure commit name / information are shown on my cypress cloud runs.
But doing so, throws an EACCES permission issue.
jobs:
  cypress:
    runs-on: default-visable
    container:
      image: cypress/browsers:node-18.14.1-chrome-110.0.5481.96-1-ff-109.0-edge-110.0.1587.41-1
      options: --user 1001
    env:
      GITHUB_TOKEN: ${{ secrets.PACKAGES_READ_ONLY_TOKEN_GITHUB }}
      PUBLIC_API_URL: "https://api.staging.visable.io"
      IS_CYPRESS: "true"
      BASE_URL: "https://www.wlw-staging.de"
      REQUESTS_INTERNAL_API: "https://requests-service.internal.wlw-1.staging.visable.cloud/requests_service/internal_api"
      RECAPTCHA_SITE_KEY: "6LeIxAcTAAAAAJcZVRqyHh71UMIEGNQ_MXjiZKhI"
      RECAPTCHA_SECRET_KEY: "6LeIxAcTAAAAAGG-vFI1TnRWxMZNFuojJ4WifJWe"
    steps:
      - uses: pnpm/action-setup@v3
        with:
          version: 8.15.3
      - name: Define proper Checkout REF
        shell: bash
        run: echo "ref=$(echo ${GITHUB_HEAD_REF:-${GITHUB_REF}})" >> $GITHUB_OUTPUT
        id: extract_ref
      - uses: actions/checkout@v4
        with:
          ref: "${{ steps.extract_ref.outputs.ref }}"
      - name: setup github packages authentication
        run: 'echo "//npm.pkg.github.com/:_authToken=${{ secrets.PACKAGES_READ_ONLY_TOKEN_GITHUB }}" > ~/.npmrc'
      - uses: cypress-io/github-action@v6
        id: cypress
        env:
          GITHUB_TOKEN: ${{ secrets.PACKAGES_READ_ONLY_TOKEN_GITHUB }}
          COMMIT_INFO_BRANCH: ${{ github.ref_name }}
          CYPRESS_RECORD_KEY: ${{ secrets.CYPRESS_RECORD_KEY }}
        with:
          record: true
          build: pnpm build
          start: pnpm start
          browser: chrome
      - name: cleanup repo files
        if: always()
        run: find . -mindepth 1 -delete

I also tried to do what was suggested in https://github.com/cypress-io/github-action/issues/1129#issuecomment-1944303998 add options: --user 1001 but I get permissions issues.

The Docker image section of the documentation says:

Include options: --user 1001 to avoid permissions issues.

This is missing from the workflow posted above.

sure, trying that just to make it clearer:
jobs:
  cypress:
    runs-on: default-visable
    container:
      image: cypress/browsers:node-18.14.1-chrome-110.0.5481.96-1-ff-109.0-edge-110.0.1587.41-1
      options: --user 1001
    env:
      GITHUB_TOKEN: ${{ secrets.PACKAGES_READ_ONLY_TOKEN_GITHUB }}
      PUBLIC_API_URL: "https://api.staging.visable.io"
      IS_CYPRESS: "true"
      BASE_URL: "https://www.wlw-staging.de"
      REQUESTS_INTERNAL_API: "https://requests-service.internal.wlw-1.staging.visable.cloud/requests_service/internal_api"
      RECAPTCHA_SITE_KEY: "6LeIxAcTAAAAAJcZVRqyHh71UMIEGNQ_MXjiZKhI"
      RECAPTCHA_SECRET_KEY: "6LeIxAcTAAAAAGG-vFI1TnRWxMZNFuojJ4WifJWe"
    steps:
      - uses: pnpm/action-setup@v3
        with:
          version: 8.15.3
      - name: Define proper Checkout REF
        shell: bash
        run: echo "ref=$(echo ${GITHUB_HEAD_REF:-${GITHUB_REF}})" >> $GITHUB_OUTPUT
        id: extract_ref
      - uses: actions/checkout@v4
        with:
          ref: "${{ steps.extract_ref.outputs.ref }}"
      - name: setup github packages authentication
        run: 'echo "//npm.pkg.github.com/:_authToken=${{ secrets.PACKAGES_READ_ONLY_TOKEN_GITHUB }}" > ~/.npmrc'
      - uses: cypress-io/github-action@v6
        id: cypress
        env:
          GITHUB_TOKEN: ${{ secrets.PACKAGES_READ_ONLY_TOKEN_GITHUB }}
          COMMIT_INFO_BRANCH: ${{ github.ref_name }}
          CYPRESS_RECORD_KEY: ${{ secrets.CYPRESS_RECORD_KEY }}
        with:
          record: true
          build: pnpm build
          start: pnpm start
          browser: chrome
      - name: cleanup repo files
        if: always()
        run: find . -mindepth 1 -delete

ci output:
0s
Run pnpm/action-setup@v3
/usr/bin/docker exec  d7472bbeaff8694ddc8da[1](https://github.com/visable-dev/requests-frontend/actions/runs/9001862118/job/24728966119#step:3:1)641958f6af298fe18bad1f83ffbe64038a2fd1006f sh -c "cat /etc/*release | grep ^ID"
Error: EACCES: permission denied, open '/__w/_temp/_runner_file_commands/save_state_c5aa5db7-4546-4d2d-95eb-fc1f342[16](https://github.com/visable-dev/requests-frontend/actions/runs/9001862118/job/24728966119#step:3:17)615'
    at Object.open (node:internal/fs/sync:78:18)
    at Object.openSync (node:fs:565:17)
    at Object.writeFileSync (node:fs:2288:35)
    at Object.appendFileSync (node:fs:2350:6)
    at Object.issueFileCommand (/__w/_actions/pnpm/action-setup/v3/dist/index.js:1:14731)
    at saveState (/__w/_actions/pnpm/action-setup/v3/dist/index.js:1:12889)
    at main (/__w/_actions/pnpm/action-setup/v3/dist/index.js:1:425)
    at 276 (/__w/_actions/pnpm/action-setup/v3/dist/index.js:1:539)
    at __nccwpck_require__ (/__w/_actions/pnpm/action-setup/v3/dist/index.js:10:243127)
    at /__w/_actions/pnpm/action-setup/v3/dist/index.js:10:243312 {
  errno: -13,
  code: 'EACCES',
  syscall: 'open',
  path: '/__w/_temp/_runner_file_commands/save_state_c5aa5db7-4546-4d2d-95eb-fc1f34[21](https://github.com/visable-dev/requests-frontend/actions/runs/9001862118/job/24728966119#step:3:22)6615'
}
Error: Error: EACCES: permission denied, open '/__w/_temp/_runner_file_commands/save_state_c5aa5db7-4546-4d2d-95eb-fc1f34216615'

@gkatsanos

The issue about installing pnpm in your runner is a different one.
It seems that you had a problem with this before, that you reported in https://github.com/cypress-io/github-action/issues/947

how do we proceed?

@MikeMcC399 coming back to this, it seems the permissions issue disappears when I remove:
          record: true

from the workflow.
Shall I create a new issue or shall I modify the existing one?

@gkatsanos
This seems to be an ownership / permissions issue on your self-hosted runner default-visable
I tried to reproduce your problem on a GitHub-hosted runner ubuntu-22.04 (with options: --user 1001) using latest versions and everything worked correctly.
Are you able to run a test of your workflow on ubuntu-22.04  instead of  default-visable?

I'm not a devops expert, but I'm pretty sure a generic ubuntu runner won't work in our infra. Our runners are not running inside docker by default. They are ephemeral EC2 instances with docker installed.

just in case I missed something, is there a documentation you can link with regards to how to display the commit name / or the branch name in the description of the cypress cloud run? I've read about different github env variables that should be passed but I may be forgetting something

@gkatsanos These are the docs for how we get git info for Cloud runs: https://docs.cypress.io/guides/continuous-integration/introduction#Git-information

I'm still not seeing the commit message as shown in the screenshot of the description of my issue. Any clues as to why?

I'm still not seeing the commit message as shown in the screenshot of the description of my issue. Any clues as to why? I'm using:
           COMMIT_INFO_MESSAGE: ${{ github.event.head_commit.message }}

@gkatsanos

I'm not a devops expert, but I'm pretty sure a generic ubuntu runner won't work in our infra. Our runners are not running inside docker by default. They are ephemeral EC2 instances with docker installed.

I don't really understand this. Normally you would just change
runs-on: default-visable

in your workflow to
runs-on: ubuntu-22.04

and GitHub Action takes care of the rest. But perhaps I am missing something fundamental here about your setup?

@MikeMcC399

I was able to reproduce / fix the issue and it seems to be related to the runs-on value.
I guess I need to see with our devops team if we could/should use the github public runners - maybe there's cost involved.

by the way, setting record: true occasionally hangs the test run, so I decided to not use it at all. Any clue if there's other documented similar issues?

@gkatsanos
If your workflow runs successfully on a GitHub-hosted runner and it fails on a self-hosted runner then you would need to look at how your self-hosted runner is set up. Unfortunately that is out-of-scope for this Cypress GitHub Actions repo to provide any specific information about self-hosted runners. The GitHub documentation About self-hosted runners appears not to cover the subject of directory ownership and permissions. The topic generally only becomes a problem when a Docker image is run inside a runner.  You can find a mention of a related issue in the cypress-docker-images README where it notes:

The GitHub Actions Runner creates the /github/home ($HOME) directory with non-root ownership 1001 (runner).

@gkatsanos

by the way, setting record: true occasionally hangs the test run, so I decided to not use cypress cloud after all. Any clue if there's other documented similar issues?

This is a different topic and generally it is not something which the Cypress GitHub Action has any influence on.
Cypress Cloud has its own different support channel which is open to customers on paid plans.
Community support is available independent of whether you use the Free plan or one of the paid plans. You can access this through  (click on button). If you want to discuss your issue on Discord be prepared to provide more details about configuration, versions, where it hangs, how it hangs, etc. otherwise the question is too vague to give any meaningful response.

@gkatsanos

by the way, setting record: true occasionally hangs the test run, so I decided to not use cypress cloud after all. Any clue if there's other documented similar issues?

This is a different topic and generally it is not something which the Cypress GitHub Action has any influence on.
Cypress Cloud has its own different support channel which is open to customers on paid plans.
Community support is available independent of whether you use the Free plan or one of the paid plans. You can access this through  (click on button). If you want to discuss your issue on Discord be prepared to provide more details about configuration, versions, where it hangs, how it hangs, etc. otherwise the question is too vague to give any meaningful response.

thank you. I'm not sure the product offering is such that deserves any more investment in time. I was merely evaluating it.
I'll try to see what's wrong with the self-hosted runners though as this might be a blocker down the line.

Closing this issue now as the permissions issue on the GitHub self-hosted runner default-visable is not something which can be solved by changes to the Cypress GitHub Action.


java.lang.NullPointerException: Attempt to invoke virtual method 'nl.siegmann.epublib.domain.TableOfContents

I have used the following dependency in our app build.gradle
compile 'com.folioreader:folioreader:0.2.5'
But am getting the following error while run my app.
"java.lang.NullPointerException: Attempt to invoke virtual method 'nl.siegmann.epublib.domain.TableOfContents nl.siegmann.epublib.domain.Book.getTableOfContents()'"
please help me to resolve this issue.  i have tried your previous versions also.  Am able to run your sample app , in it all lib projects are directly added.

Does anybody know how to fix this problem

Hi, you have to allow Write permission

With write permission I get the same error

@vikydev with new version 0.3.1, it should be fixed.. can you please confirm and close the ticket?


Is it possible to customize timeouts the same way as in aiohttp?

Is it possible to customize timeouts the same way as in aiohttp? Like timeout = aiohttp.ClientTimeout(total=10, connect=2, sock_connect=2)
I noticed that if I use session in both aiohttp and curl-cffii, if there are many proxies with long timeout, then all subsequent requests start to get more time to execute and I would like to weed out such slow proxies immediately as in aiohttp, how can I do it in curl-cffii?

Actually, you can set separate connect_timeout and read_timeout just like original requests.
from curl_cffi import requests
requests.get("https://example.com", timeout=(1, 3))

These two values use libcurl's CURLOPT_CONNECT_TIMEOUT_MS and CURLOPT_TIMEOUT_MS respectively.

What about in an asynchronous session?

The same.
r00t-Taurus @.***>于2023年10月27日 周五17:31写道：

What about in an asynchronous session?
—
Reply to this email directly, view it on GitHub
https://github.com/yifeikong/curl_cffi/issues/148#issuecomment-1782600478,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH4ZX53HWHJFW4RQHE5IZ3YBN5QBAVCNFSM6AAAAAA6SA3SQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBSGYYDANBXHA
.
You are receiving this because you were assigned.Message ID:
@.***>

it doesn't work for me, the timeout error doesn't pop up if you specify through ()

Can you describe you use case in aiohttp and what you are expecting for curl_cffi in a few lines of code?

async def main():
    async with requests.AsyncSession() as session:
        t1_start = time.perf_counter()
        req = await session.get(
            "https://www.google.com/",
            headers=headers,
            proxies={"https": "http://proxy"},
            impersonate="safari15_5",
            timeout=0.3,
        )
        print(time.perf_counter() - t1_start)

if __name__ == "__main__":
    import asyncio

    asyncio.run(main())

if you do this, the timeout error is shown
if changed timeout to timeout=(0.3, 0.3),then the timeout error does not appear for 0.3s

Hmm, I'm not getting you point here.
If I use timeout=1, the error is:
'Failed to connect to proxy port 1080 after 754 ms: Operation timed out'.

If I use timeout=(0.3,1), the error is:
'Failed to connect to proxy port 1080 after 228 ms: Operation timed out'

which aligns with the timeouts given.

yes, sorry, everything works, it was about the proxy and the fact that I mixed up the parameters in places


The Name and Hex properties are empty on WindowsPowerShell

I've tried the example given in https://twitter.com/JamesBru/status/1761091548383707161.
All I get (both in Windows PowerShell 5.1.19041.4046 and in PowerShell 7.4.1) is:

(OS is Windows 10 build 10.0.19041.4046)

Thank you for the reminder....
To mark this module as only working on core.
Most of the Emoji will not render on non-core, and the escape sequence is not supported.

It also fails with 7.4.1!
Here's what $error has to say:

@sba923 Still looks very weird.  I don't think oh-my-posh is doing it, but trying without that might work.
The other guess I have is execution policy (it could be affecting the types files more silently than I would like).

Launching oh-my-posh with the same arguments as from my profile doesn't break Get-Emoji:

The place that is complaining is related to colorization / style, and might be tripped if $psStyle was set to an unexpected value.
If this is the case, you should also see colorization errors if you load up Posh
If you do, then the actual issue is in EZOut (which builds formatters for all of these things)

Does this shed some light on the issue?

Yeah and no.  Based off of what you've already sent I already know it's related to $psStyle and how colorization is handled.
But not why it's broken, or how.
Can you poke at $psStyle a bit when it's not broken and see what might be going on?

$PSStyle is identical in broken and non-broken sessions.
Posh doesn't break Get-Emoji's output.

And the culprit is...🥁
Set-StrictMode -Version latest

Glad we've found the culprit!
You should be able to reproduce this in EZOut by creating any formatter that uses -Style* parameters:
https://twitter.com/SBarizien/status/1766043978640585185 (just add -StyleProperty to your example)

Fix confirmed, good job!


chore: start program with alt screen instead of init

As described by the godocs, instead of calling tea.EnterAltScreen  in the init method, the correct way is to initialize the program with the tea.WithAltScreen

Thank you for the contribution! As mentioned in Discord, we are working on merging

testing some big bug fixes right now, so we aren't going to be merging anything new until after there's another release (which is a wip right now). 👍

Thank you for the contribution!


unable to see alure reports from jenkins

What are you trying to achieve?
Hi,
I am trying to run with codecept and allure reports. the issue that locally I can see all logs, but when I am trying to run with jenkins and plugin allure 2.7 then I cant see logs.
What do you get instead?
allure-results does not exists
Report successfully generated to c:....allure-report
Allure report was successfully generated.
I didnt understand what should be 2 params from plugin of allure
Results ->what it means? if my codeceptjs generate it to output folder...
Report path --?

Allure plugin has only outputDir param.
So yes, make a report, make it available to Jenkins, and that should work fine

when we execute using chunks codeceptjs creates individual folders with individual xml n png(for failures). how to put together all reports together in allure ?

@Vaahin Try separating your instances and treat them as unique. In CI/CD like Jenkins, create a new view, create a job for each suite you want to run. In that view you can then choose what jobs you want to see. In this case each suite would run in their own container/machine/vm and you will always have one report generated.

@Vaahin First: you must to use uniqueScreenshotNames: true, in codeceptjs config.
After tests you should copy all of xml and png files to allure-report directory (you must to create this dir), then use allure results: [[path: 'allure-report']]


HAL: Upstream from OVL works

Hey, I've been working on ATSAME70 support specifically in a repo as part of some contracting work (OVL => OneVariable UG Labs).
Since I've been focused on just this one chip, and a relatively limited set of features, it's probably not ready to merge 1:1, but just in case someone is interested, the repo is here, and I've published the crate under the name ovl-atsamx7x-hal for now, until this work gets upstreamed.
So far, I have the following components implemented:

EFC - basically just the ability to set wait states
GMAC - some corners rounded, really only tested to work with the KSZ8061 PHY, but capable of "doing ethernet" and has a smoltcp implementation capable of maintaining a TCP connection. Only supports one set of pins ATM
PIOx - generally all 5 ports are supported, and some pin functionality is working (mostly setting ABCD peripheral modes, and output pins)
PMC - Pretty rudimentary support, but enough to set some prescalers. I mostly use it to reach a single clock config (300MHz CPU clock, 150MHz MCK)
Serial - I haven't really touched the upstream code here, I'm not using serial
SPI - Basic "transfer" API support for the SPI0 peripheral. Only supports one set of pins ATM
WDT - Basically just enough to shut it off

Feel free to take any of this work if you want/need it before I get a chance to upstream! I've kept the 0BSD license, so you can do what you'd like.

Oh, there are also quite a few working examples in the same70-bringup project, available here.

Thanks for the notice. I'll take a more thourough look at it when able.

@tmplt is there anything from that @jamesmunns work here that wasn't in #25?

Only the GMAC and groundhog::RollingTimer implementations. I'll push a WIP PR for the former tomorrow. I do not consider the latter in-scope for the HAL currently.

Superseded by #33.

The #33 is a PR, as well as #32 and #31 which are continuation of work from #24.


fix(clerk-js): Correctly set idle card state on add mfa error

Description

Set the card to the idle state after success or error.

Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[x] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Fix table row spacing in Chrome.

This will mess with the ability to style by changing cell width/height, and I'd like to avoid changing that if possible. What does this do, exactly?

Currently, the demos on your website look like this on Chrome on my machine (but they look fine on Firefox)!

Adding just one of the two (display: block, or height: 40px) doesn't work; it has to be both. Apparently this has to do with Chrome's default rendering of tables (maybe from a recent update?), and it's annoying to work around.
Re: setting height to 40, currently we say that .puzzle-entry td has width and height 40px. If you want, you can make a CSS variable for example .puzzle-entry { --grid-size: 40px; } and then use the variable elsewhere. You can also change the variable value with JS if that's needed.
Could you give an example of where the tr height would not be equal to the td height?

I like the CSS variable approach. TBH I've never seen a CSS variable defined anywhere other than :root or else I would have done this sooner as it's better for eventual hex grids.
That said, I don't want to make the change without updating the styling reference plus all demos that change cell sizes, including documenting the breaking change. I can get that done by the end of this weekend (if not today) if you'd like.

That would be great. Sorry, I wasn't aware of anywhere that cell sizes were
changed; I trust you to update the demos accordingly!
On Fri, Oct 27, 2023, 11:56 AM Kenny Young @.***> wrote:

I like the CSS variable approach. TBH I've never seen a CSS variable
defined anywhere other than :root or else I would have done this sooner
as it's better for eventual hex grids.
That said, I don't want to make the change without updating the styling
reference plus all demos that change cell sizes, including documenting the
breaking change. I can get that done by the end of this weekend (if not
today) if you'd like.
—
Reply to this email directly, view it on GitHub
https://github.com/tabascq/PuzzleJS/pull/28#issuecomment-1783369535, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHFLOUZC5SYYJLQD6MQD7YLYBP7VRAVCNFSM6AAAAAA6R44AP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGM3DSNJTGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Please check out PR #29; if that works for you, then I will merge that one and close this one.

Closing in favor of #29.


unmockedModulePathPatterns not matching module paths properly with npm 3

Here's a part of jest key in my package.json:
"jest": {
    "scriptPreprocessor": "<rootDir>/node_modules/babel-jest",
    "unmockedModulePathPatterns": [
      "/history",
      "/react",
      "/react-bootstrap",
      ...

I'm on node 5.0.0. With npm 2.14.10 after deleting node_modules and doing a fresh npm install, my jest test passes:
$ rm -rf node_modules
$ npm install
$ npm --version
2.14.10
$ npm test -- src/javascript/components/FilePicker/__tests__/FilePicker-test.js
...
7 tests passed (7 total in 1 test suite, run time 3.705s)

As, there's an issue with npm 3.x as I'll demonstrate, I added a couple console.log lines in HasteModuleLoader.js in _shouldMock method right before the for loop that checks the regular expressions for files to ignore for mocking.
For npm 2.14.10, this is the  relevant output:
modulePath: /Users/myuname/projects/myproj/frontend/node_modules/react-bootstrap/node_modules/uncontrollable/index.js
realPath: /Users/myuname/projects/myproj/frontend/node_modules/react-bootstrap/node_modules/uncontrollable/index.js
modulePath: /Users/myuname/projects/myproj/frontend/node_modules/react-bootstrap/node_modules/uncontrollable/createUncontrollable.js
realPath: /Users/myuname/projects/myproj/frontend/node_modules/react-bootstrap/node_modules/uncontrollable/createUncontrollable.js
modulePath: /Users/myuname/projects/myproj/frontend/node_modules/react-bootstrap/node_modules/uncontrollable/utils.js
realPath: /Users/myuname/projects/myproj/frontend/node_modules/react-bootstrap/node_modules/uncontrollable/utils.js

Now, I upgrade to npm 3.3.12 and do a fresh npm install:
$ sudo npm install npm -g
$ npm --version
3.3.12
$ rm -rf node_modules && npm install
...

I now run the test exactly as before and get an error:
$ npm test -- src/javascript/components/FilePicker/__tests__/FilePicker-test.js
...
 FAIL  src/javascript/components/FilePicker/__tests__/FilePicker-test.js
● Runtime Error
TypeError: /Users/myuname/projects/myproj/frontend/src/javascript/components/FilePicker/__tests__/FilePicker-test.js: /Users/myuname/projects/myproj/frontend/src/javascript/components/FilePicker/FilePicker.jsx: /Users/myuname/projects/myproj/frontend/src/javascript/components/FilePicker/Breadcrumbs.jsx: /Users/myuname/projects/myproj/frontend/node_modules/react-bootstrap/lib/index.js: /Users/myuname/projects/myproj/frontend/node_modules/react-bootstrap/lib/Dropdown.js: _uncontrollable2.default is not a function
npm ERR! Test failed.  See above for more details.

Looking at the output of my console.logs (which I had to add again since I did a fresh npm install), I noticed that the module paths are totally different:
modulePath: /Users/myuname/projects/myproj/frontend/node_modules/uncontrollable/index.js
realPath: /Users/myuname/projects/myproj/frontend/node_modules/uncontrollable/index.js
modulePath: /Users/myuname/projects/myproj/frontend/node_modules/uncontrollable/createUncontrollable.js
realPath: /Users/myuname/projects/myproj/frontend/node_modules/uncontrollable/createUncontrollable.js
modulePath: /Users/myuname/projects/myproj/frontend/node_modules/uncontrollable/utils.js
realPath: /Users/myuname/projects/myproj/frontend/node_modules/uncontrollable/utils.js

My /react-bootstrap in unmockedModulePathPatterns no longer matches thess paths.
Also, uncontrollable is still only used by react-bootstrap:
$ npm list uncontrollable
<EMAIL_ADDRESS>/Users/myuname/projects/myproj/frontend
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>

Hey! Can you try to add uncontrollable to the unmocked module list? The problem is that npm3 uses a flat copy of node_modules and jest currently doesn't know how to deal with that. I will fix this eventually!
This is tracked in #554

Yup, that should do it for now. My apologies. I didn't see #554.

No worries. This is definitely confusing and should probably be part of the documentation for now. How do you feel about making a small pull request for the docs that adds information about unmocking with npm3? :)

Sure, can do :)
Gezim Hoxha
Entrepreneur
780.802.4837(C)
http://ca.linkedin.com/in/gezimhoxha
http://namez.com/profiles/2394-gezim-hoxha/?autoPlay=true
On Tue, Nov 10, 2015 at 4:14 PM, Christoph Pojer<EMAIL_ADDRESS>wrote:

No worries. This is definitely confusing and should probably be part of
the documentation for now. How do you feel about making a small pull
request for the docs that adds information about unmocking with npm3? :)
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/jest/issues/602#issuecomment-155598326.

I think it makes sense somewhere around here: https://github.com/facebook/jest/blob/master/docs/API.md#configunmockedmodulepathpatterns-array ?

It's here #604. I wanted to go into more detail on how to determine what needs to be mocked when you upgrade to npm 3. Thought about having a Troubleshooting section somewhere?

Yeah I think that would be fantastic. I'm actually planning to revamp how jest's config and setup works and with that I also wanna make the jest website more beginner friendly! Some people (including me :) ) are hanging out on discord – if you wanna join us and be involved, feel free to do so: https://facebook.github.io/jest/support.html :)


Is it possible to move the cross in the menu to the right side?

It's on the left side by defualt and i havent found anything in the documents on how to move it. So i'm wondering if it's possible and if so, how?

@Andekl Where do you want to move it ?
We have right prop to move the sidebar to right side

@mbj36 Sorry maybe i wasent clear enough.
I didnt mean that i want to move the whole sidebar to the right (that i have already done). I want to move the cross inside the sidebar to the right side of the sidebar

Can you please post a visual pic with indication of what you want ?
pic1 -  what it looks like
pic2 -  what you want

If you do style="right: 0px" this should move cross button to the right of the sidebar

@mbj36 I tried to add it in both .cross-style and .bm-cross and it just moved the cross slightly to the right


[Enhancement] "Leads To" pop up unnecessary majority of the time

When a new wormhole signature is added, Type pops up, then Leads Top pops up. However, in a large amount of cases, Leads To is useless to fill in as that system typically doesn't exist on the map yet. Sometimes it does like in the case of a return wormhole or if connecting to a pre-existing system.
I would prefer it to not automatically pop up as it requires making an extra click to remove the focus away from it.

I had the same concerns about to many popovers. My intention was to reduce the initial click to open it for edit.
AFAIK, the "Leads to" popover auto opens for existing signatures only (check the *.gif below).
After submit, the next focused cell is determined by its value. If there is no value set -> cell gets focused, popover opens. Cells where no value can be set (e.g. group combat can not have a leads to system) are skipped.

Hint: You can use arrow keys left, right, up and down for navigation inside the table. return can be used to open a popover, or to add, delete a row, esc can be used to close a popover,
You should be able working with the signature table without any mouse interaction. Arrow keys can also be used to select options.

Example:


Visualize model

Hi,
I am trying to use your code. I am using launch_default.launch, but the terminal remains saying "Connecting to camera ... ". Rostopic list indicates that the kinfu/raycast_image is there, but I cannot see any image. Should I setup rviz to do it?  I have tried with an asus xtion pro live and an Orbbec Astra S, but the laser projectors do not seem to be working.
Could you please guide me a bit, how can I get a 3D model with your code?
Thanks,

Hi,
I encountered the same problem as yours. Did you solve the problem and get it working? Could you please guide me? Thanks.

kinfu/raycast_image contains a rendered image of the model and normals. If you want to get 3D model as mesh or pointcloud you need to run something like marching cubes using the data from /get_tsdf

Thanks! The method to get point cloud seems helpful. I will check that up. Before that, I need to solve the connection problem. I am running this on Jetson TX2 and using Structure Sensor as mentioned in the readme, but it does not recognize my camera, I am getting the same problem as " I am using launch_default.launch, but the terminal remains saying "Connecting to camera ... ". Rostopic list indicates that the kinfu/raycast_image is there, but I cannot see any image." Should I edit some comminication related settings in the package? Thanks!


startup bundle level support

For master and R7 branch, goal is to ensure maven plugin startup bundle order is respected (since start level is not yet mapped in the config)

last commit cherry-pick on master and karaf-4.2.x and rebase on #995


Removed unused code

Codecov Report

Merging #302 (b0dceee) into master (d6c1dbb) will increase coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #302      +/-   ##
============================================
+ Coverage     73.64%   73.73%   +0.09%
  Complexity      760      760
============================================
  Files           105      105
  Lines          2436     2433       -3
  Branches        325      325
============================================
  Hits           1794     1794
+ Misses          519      516       -3
  Partials        123      123

Impacted Files
Coverage Δ

...c/main/java/seedu/address/model/person/Person.java
85.84% <ø> (+1.58%)
:arrow_up:

src/main/java/seedu/address/ui/MainWindow.java
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 d6c1dbb...b0dceee. Read the comment docs.


Update draco3dgltf

Update gltfjsx
Fixes #153

Thanks @ggozad it's works even with node 18 version, how did you solve it?

thanks! and sorry for the wait, i don't get github notifications, everything relevant gets drowned out by hundreds of repos i was auto subscribed to. :-S

Haha no worries, I am in the same situation 😂

thanks! and sorry for the wait, i don't get github notifications, everything relevant gets drowned out by hundreds of repos i was auto subscribed to. :-S

Tahnk you very much 💯


Fix #2214. Support narrowing with typeof in switch condition.

Enable narrowing support for typeof switch conditions. Fixes #2214
Implement narrow-by-assertion in switch with typeof, intending to replicate the equivalent behaviour with if-then-else.
@mhegazy Is there any possibility of this making the 3.0 milestone?
Basic Example:
function assertNever(x: never) { return x; }

function switchTypeOf(x: number | string | { x: boolean }): boolean {
    switch (typeof x) {
        case 'number': return x > 0;
        case 'string': return x === "narrowed";
        case 'object': return x.x;
    }
    return assertNever(x);
}

Exhaustiveness Checking:
Switch exhaustiveness checking is extended to support typeof.
function exhaustiveSwitch(x: number | ((x: string) => number) | symbol): number {
    switch (typeof x) {
        case 'number': return x;
        case 'function': return x('narrowed')
        case 'symbol': return 0;
    }
    // No implicit return
}

Order sensitive narrowing:
Repeated cases do not get the same narrowing.
function redundantCases(x: number | string | { x: boolean }): boolean {
    switch (typeof x) {
        case 'number': return x > 0;
        case 'string': return x === "narrowed";
        case 'object': return x.x;
        case 'number':
        case 'object': return assertNever(x);
        case 'string': return assertNever(x);
    }
    return assertNever(x);
}

Comparison with if-then-else.
1. Constrained parameters are not narrowed to never in default / else
function constraintNarrowingSwitch<T extends number | string | boolean>(x: T): boolean {
    switch (typeof x) {
        case 'number': return x > 0;              // x : T & number
        case 'string': return x === "narrowed";   // x : T & string
        case 'boolean': return x;                 // x : (T & true) | (T & false)
    }
    return assertNever(x); // <-- Error
    /* Argument of type 'T' is not assignable to parameter of type 'never'.
       Type 'string | number | boolean' is not assignable to type 'never'.
       Type 'string' is not assignable to type 'never'. [2345]
    */
}

function constraintNarrowingIf<T extends number | string | boolean>(x: T): boolean {
    if (typeof x === 'number') {
        return x > 0;
    }
    if (typeof x === 'string') {
        return x === "narrowed";
    }
    if (typeof x === 'boolean') {
        return x;
    }
    return assertNever(x); // <-- Error
    /* Argument of type 'T' is not assignable to parameter of type 'never'.
       Type 'string | number | boolean' is not assignable to type 'never'.
       Type 'string' is not assignable to type 'never'. [2345]
    */
}

2. Unions of constrained generic parameters.
type L = (x: number) => number;
type R = { x: string }

function genericUnionSwitch<T extends L, U extends R>(x: T | U) {
    switch (typeof x) {
        case 'function': return x; // x: T extends L
        case 'object': return x;   // x: U extends R
    }
}

function genericUnionIf<T extends L, U extends R>(x: T | U) {
    if (typeof x === 'function') {
        return x;  // x: T extends L
    }
    if (typeof x === 'object') {
        return x   // x: U extends R
    }
}

Code Review
The code comments are primarily there for review purposes, they should probably be taken out afterwards. A few things to signpost for the review:

The function getSwitchClauseTypeOfWitnesses performs no caching. The choice for this was because the cache type is Type[], the types of the clause values. In the context of typeof the types of the clause values are less important, rather what they represent. There didn't seem to be much use for caching the types. I'd like to get feedback on this, I'm not sure my decision is correct.
The function isExhaustiveSwitchStatement is updated to handle switch statements with typeof conditions.
Can (and should?) narrowBySwitchOnTypeOf be moved outside of the scope of reference flow typing?

This PR does not update isExhaustiveSwitchStatement to handle typeof conditions but I don't think it would be too hard to change. If so, I think I'll probably want to try and cache the statement types.

It it possible to use Extract and Exclude for narrowing types of the form T extends A | B | C?

ping @mhegazy @RyanCavanaugh
Sorry for the ping, just wondering if this got lost under a pile of other stuff.
I'm not expecting anyone to drop what they're doing and look at this; I just want to make sure that it's clear I'm still actively wanting to try and merge this.

@weswigham can you please review this change

After making some changes the CI failed due to strictNullChecks that weren't enabled when I initially developed the fix.
The source of the problem was using undefined to mark the default case in the list of types tested against. E.g.
switch(typeof x) {
    case "number": ...
    case "string"...
    default: ...
}

is represented as the array ["number", "string", undefined]. I'm not sure it's possible to completely ignore the default case, as you need to know where it is to correctly fix the clauseStart and clauseEnd. However, I have tried to limit the used of undefined by removing it once all the necessary information is computed.
I'm not convinced it's the most elegant solution, and would be more than happy to readdress the issue based on any feedback.

@weswigham can you take another look.

I'm currently implementing a RemoteData pattern in TS and I think this would greatly help to simplify my fold method.

I'm looking to add some more tests to this. If anyone has some examples that would be great!

Pinging @RyanCavanaugh for visibility because I think mhegazy was the only one watching this and I believe they've left the TS team.

@weswigham can you review the current iteration? Thanks

@jack-williams @RyanCavanaugh  even though there's no conflicts, a merge with master in the branch prior to us pulling it in would be good - a few months passing may mean someone's added a test this may affect.

@weswigham @RyanCavanaugh Thanks! Master merged. CI failed on node 6 because of missing jake, I'm not sure if this was caused by my changes. Just let me know if I need fix anything.


Cannot interact with GraphQL toolbar

I have installed the plugin, and in the tool window, my types appear in the schema discovery summary. However, when I open the graphql file where my queries are defined, the only button that I can click on the toolbar is the "Open Configuration File" button. What step have I missed in setting things up? My code can be found here.

@spartanhooah hi! The disabled buttons you see are related to making queries and schema introspection from the remote URL. If you need this, you should specify a URL in the config. There are two ways to do this 1, 2.
That's a UX issue, maybe we need to hide those disabled buttons to prevent confusion.


iOS Simulator: sync not working as expected

Just installed the package in our app, running in simulator with logging enabled.
The docs state that the data is flushed every 60 seconds, but at least in Simulator nothing happens - no timer or so. But when I send the app to background, the data is eventually flushed and visible in Mixpanel's dashboard.
It looks like the app has to be sent to background at least once to start the timer for flushing the data

I have facing the same issue, on iOS, events are only recorded when I send the app to background. Otherwise I dont see anything in Mixpanel's dashboard.
On Android, I don't see any events tracked at all, not even after sending the app to background

Sorry for the late response and Thanks for bringing it up! There was an issue that might cause the iOS flush timer to stop working. It's been fixed in v1.3.5! Now it should flush for every 60s by default. I'm closing this issue now, please feel free to reopen if the problem still exists.


App.directory and App.load_kv broken in python 3.10

Python: 3.10+
Kivy: 2.2.1

In Python 3.10, inspect.getfile() may also return OSError('source code not available') in some circumstances, making App.directory and App.load_kv throw this exception.
Fix:
Replace in these two functions except TypeError by except (TypeError, OSError)

That behavior does not seem to be documented in https://docs.python.org/3.10/library/inspect.html#inspect.getfile
Can you please share some more context? (When is this happening? , o you have some logs?)

Stackoverflow suggests:

The source code is probably a compiled c binary and not python source code, so it is not showing up in inspect.getsource

This is the inspect.getfile source code in Python 3.10, showing it may throw OSError in some cases:
def getfile(object):
    """Work out which source or compiled file an object was defined in."""
    if ismodule(object):
        if getattr(object, '__file__', None):
            return object.__file__
        raise TypeError('{!r} is a built-in module'.format(object))
    if isclass(object):
        if hasattr(object, '__module__'):
            module = sys.modules.get(object.__module__)
            if getattr(module, '__file__', None):
                return module.__file__
            if object.__module__ == '__main__':
                raise OSError('source code not available')
        raise TypeError('{!r} is a built-in class'.format(object))
    if ismethod(object):
        object = object.__func__
    if isfunction(object):
        object = object.__code__
    if istraceback(object):
        object = object.tb_frame
    if isframe(object):
        object = object.f_code
    if iscode(object):
        return object.co_filename
    raise TypeError('module, class, method, function, traceback, frame, or '
                    'code object was expected, got {}'.format(
                    type(object).__name__))

@yves-surrel
That scenario appears to be: The object you pass getfile is a class that is in a module that isn't in a file but is __main__.
Where is your subclass of App? Can you produce a minimal faulty program we can try?