Adjust the DESCRIBE AS SDL and AS TEXT logic.

Change the logic when DESCRIBE command shows the original expression.
Basically if the normalized expression appears to only introduce a
default module that is the same as the module the object belongs to then
SDL and TEXT versions of describe can safely use the original short
expression. Otherwise, a normalized expression must be given to avoid
confusion.
Fixes: #1254

I think we should do a simpler thing:

for SDL to always output the normalized expression.
for TEXT to always output the original expression.
for TEXT VERBOSE to always output the normalized expression.

The specific version matters a lot for indexes, so regardless of SDL, DDL or TEXT we would want to output the original expr version for them so that it can actually be used to refer to the real index.
We can do as you suggested otherwise for the different modes.

What's up with this PR?

So is the consensus then:

for SDL to always output the normalized expression.
for TEXT to always output the original expression.
for TEXT VERBOSE to always output the normalized expression.
for indexes always output the original, regardless of the mode since they are identified by the original text.

for TEXT to always output the original expression.

@elprans should we do that? Maybe we should always output the normalized one because it would reflect the correct state of the schema including renames etc.?

because it would reflect the correct state of the schema including renames etc.?

The "original" here does not necessarily mean the initial text, it just means that the expression is unmangled by the injected WITH block.  Renames will alter all expressions where the renamed object appears.

for indexes always output the original, regardless of the mode since they are identified by the original text.

Constraints also.

@vpetrovykh Please finalize this PR.

I've rebased the PR and made constraints obey the same rule as indexes w.r.t. preserving and showing the original expression at all times because they are basically defined by it.
This causes test_get_migration_multi_module_03 to fail due to a default module being injected into the finalized version of the expression and that default module is inferred to be different in SDL and DDL. This is an inherent ambiguity that 99% of the time doesn't matter as expressions for indexes or constraints typically involve pointers directly on the object and some constants and thus don't rely on a specific default module (WITH MODULE foo injected into the expression in the process of normalization). This seems more of an issue with how the expressions are normalized internally than an actual issue with grammar.

@vpetrovykh I rebased on top of my "DDL identity" work that's in master, so index and constraint hacks are no longer necessary.  Not sure whether we still want to merge this given the pending normalization rework.

This appears to be no longer necessary due to #1522


How to test updated Python code?

I see @RRMaximiliano has made a lot of updates on the Python code. However, when I run the run file, I still see the old version of outputs -- I assume this is because I have the earlier version installed. What do I need to do to use the updated Python functions?

remove current Stata linter installing
add github folder to top of adopath
restart Stata

maybe there's a way to update .py files too everytime we run net install ... to try code from a development branch?

Hardcode linter version in both ado and py and check that they are the same before running. Throw an error if it's not.

The easiest fix is to simply remove the .pkg file from this repo, and only support SSC installation

I'm changing the name of this issue so it's more accurate with what we're correcting.
Regarding the test of updated Python code, updated Python scripts will be loaded by Stata after updating the package with net install ... or ssc install ... and restarting the Stata session


examples: ADMM Lasso: fix missing variables

Similar to #324, this PR fixes an issue in the ADMM Lasso example where variables gamma and NUM_PROCS were previously undefined.
@SteveDiamond Looks like I may have been mistaken about how many examples needed updating. At the moment, it seems only ADMM Lasso had the same issue.

Coverage remained the same at 89.342% when pulling 347965c0eb37e4c0406a622f256f08f784a77a58 on dplarson:fix_examples into be172685c1e7198e14ded8361f0ac99c85b18a52 on cvxgrp:master.


fix: ci to delete the stale namepaces

Description

Make sure that we fetch the creds for dev cluster before deleting the namespaces

:zap:  Two deployments have been created for the Point-of-Sale application. You may access and test them at: :zap:

MySQL DB backed deployment: <IP_ADDRESS>
Embedded H2 DB backed deployment: <IP_ADDRESS>


Permissions are not documented

Is your feature request related to a problem? Please describe.
I tried to use lfs-warnings for pull_request events from forks of a public repo. I got "Resource not accessible by integration" error.
PR in question: https://github.com/FerretDB/FerretDB/pull/1371
Describe the solution you'd like
lfs-warnings should document what permissions it needs.

The action tries to create the PR label if it doesn't exist already. PRs don't have access to do do that.
https://github.com/ActionsDesk/lfs-warning/blob/e5f9a4c21f4bee104db7c0f23954dde59e5df909/src/index.ts#L163-L169
You can work around this error by pre-creating the label manually.

I created use-git-lfs label: https://github.com/FerretDB/FerretDB/labels
I configured lfs-warning to use it: https://github.com/FerretDB/FerretDB/blob/cd52bc76204357a615dc8a896e54c92bf2b3ebb1/.github/workflows/docs.yml#L41-L47
I created test PR: https://github.com/FerretDB/FerretDB/pull/1797
It still fails with the same error message: https://github.com/FerretDB/FerretDB/actions/runs/3914014480/jobs/6690585846
I guess it fails later while trying to add a label and comment to PR. That functionality is also not available to PRs from forks. But at least it produces a file list, even if the last error message is confusing.

Maybe pull_request_target is the only solution. See official docs, it says This event allows your workflow to do things like label or comment on pull requests from forks.

I fixed the error by adding the pull-requests: write permission:
jobs:
  check:
    name: Check
    runs-on: ubuntu-latest
    permissions:
      # required for ppremk/lfs-warning
      pull-requests: write
    steps:
      ...

Hi authors @AlekSi, thanks so much for developing this great tool. I am working on a public repository and have tried to integrate your tool into our testing framework. I keep getting the same error discussed above: Error: Resource not accessible by integration. I've tried the solution discussed above, but haven't been able to figure it out. Could you advise how to solve this error? The PR can be found here.


[CELEBORN-789] Set default value of flushBuffer's max components to 256

What changes were proposed in this pull request?
Set default value of celeborn.worker.push.compositeBuffer.maxComponents to 256, to be aligned with 0.2.1-incubating version.
Why are the changes needed?
Default 16 is too small, and causes severe GC and CPU high load.
Does this PR introduce any user-facing change?
No, it's internal config.
How was this patch tested?
Passes GA.

cc @pan3793

maybe 64 is fine, let me test

My test shows that 128~256 works better in low memory cases (2g off-heap memory per worker)
When lower than 128, the CPU cost is very high on CompositeBuffer compaction, and there is a high probability that "direct memory usage" < "disk buffer size", since pause is judged by "direct memory usage", OOM happens frequently.
Even with 128, sometimes "direct memory usage" < "disk buffer size"
Direct memory usage: 1476.0 MiB/2.0 GiB, disk buffer size: 1850.7 MiB

On the other hand, with the growth of "maxComponents", memory fragmentation grows rapidly too.
My final choice is 192. But I think setting 128 in default should be fine considering most users will give it a larger resource than me.

My test shows that 128~256 works better in low memory cases (2g off-heap memory per worker)
When lower than 128, the CPU cost is very high on CompositeBuffer compaction, and there is a high probability that "direct memory usage" < "disk buffer size", since pause is judged by "direct memory usage", OOM happens frequently.
Even with 128, sometimes "direct memory usage" < "disk buffer size"
Direct memory usage: 1476.0 MiB/2.0 GiB, disk buffer size: 1850.7 MiB

On the other hand, with the growth of "maxComponents", memory fragmentation grows rapidly too.
My final choice is 192. But I think setting 128 in default should be fine considering most users will give it a larger resource than me.

Thanks @pan3793 for the detailed profiling! I have changed the default value to 128 according to your suggestion. I also agree that optimal settings depend on environment and workloads.


[TypeSpec Validation] Improve error message when PR is missing file

The error message when a PR is missing a file (like a generated swagger) is not very readable:

Executing rule: GitDiff
Rule GitDiff failed
{"not_added":["specification/contosowidgetmanager/data-plane/Azure.Contoso.WidgetManager/preview/2022-11-01-preview/widgets.json"],"conflicted":[],"created":[],"deleted":[],"modified":[],"renamed":[],"files":[{"path":"specification/contosowidgetmanager/data-plane/Azure.Contoso.WidgetManager/preview/2022-11-01-preview/widgets.json","index":"?","working_dir":"?"}],"staged":[],"ahead":0,"behind":0,"current":"test-missing-swagger","tracking":"origin/test-missing-swagger","detached":false

We should print a nice human-readable error rather than just dumping the git results object as JSON.

Dup of #27254


replace -Zno-trans with cargo check

It looks like -Zno-trans tends to cause build failures when there are build.rs steps involved in building so using -Zno-trans makes rustw unusable on many projects.

Yes, it does, that is the motivation for implementing cargo check.


Add class.rst

Object Oriented Programming hasn't been mentioned in the wiki yet and this might help understanding how classes work in the modified version of Squirrel Respawn uses.
I've added everything I know about Squirrel's classes.

I did the thing.

So, theres a pretty big reason it hasn't been mentioned: it really shouldn't be used, because IIRC there is a total max of like, 60 classes and Respawn already uses 58 or so of them
of course, if you test more classes and it works, ignore me.

@laundmo Looks good to me, feel free to merge if you agree.


[7.x] Replace placeholder for dots and asterisks in validator

This PR fixes: https://github.com/laravel/framework/issues/33357

Can we update a test for this?

@taylorotwell test added


When receiving returns the shipping cost is not being accounted for in the refund amount

In admin:

Click request return
Add return shipping method
Create return
Click receive return
Notice that the shipping cost is not accounted for in the refund currency-input box

Admin should show the refund amount as it is stored in the database and store operator should be able to change the refund amount between 0 (min) to cost of products (max).

Duplicate of #522


Develop: Create Notification and NotificationSubscription data objects

As a developer of the application, I need Notification and NotificationSubscription data objects to work with so that all developers aren't putting in temporary workaround code.
Acceptance Criteria
The following data objects are available for developers to reference:

Add Notification object

Timestamp
EmailID (foreign key)
MID (individual mail piece ID)
NotificationSubscriptionKeyword (foreign key)

Add NotificationSubscription object

Keyword (string) (primary key to avoid duplicates)

Objects are ideally added to the database unless it is not yet ready.
Document any relevant content in the Programmer guide (draft thoughts of usage and importantly the "Why")
Updates SRS and STP (and other docs if necessary)
Add/Update relevant unit tests

Completed.


Support for several licence files

Hello,
Following issue https://github.com/Esri/arcgis-powershell-dsc/issues/130, I realized that it is not possible to authorize an ArcGIS Server with several roles.
Indeed, if you generated a .prvc with 2 roles and 1 extension (eg: ArcGIS server standard + Image Server + Network analyst), only 1 role and the extension are activated by the licence manager (it is the same when it is done manually I believe).
I thought I would duplicate the file and switch the order of the 2 roles in the file but realized it is not possible to specify an array of LicenseFilePath.
What is the best strategy ?
Thanks

It's usually not recommended to have the same server perform multiple roles.
However the license manager should accept all information provided in the license file.
Are you observing consistent behavior where it does not apply?

Thanks for the reply.
Yes, I am observing consistent behavior where it does not apply. Also with the Wizard.
My prvc is like ending like that:
// Features and authorization numbers
ArcGIS Server=ECPAAAAAAAAA
ArcGIS Server=ECPBBBBBBBBBB
Network Analyst Server=ECPCCCCCCCCC
The second role is not installed. If I switch the order, then it is the other one which is not authorized.
But the extension is always successfully authorized.
Thanks

what version are you observing this behavior with?

10.6.1

let me check with out licensing team and see what behavior is expected.

I spoke to our licensing experts and the workaround for this scenario (since our automation does not support a sequence of multiple licensing files) is to make the Feature name unique. In your case rename the second occurrence of "ArcGIS Server"  to "ArcGIS Image Server" or "ArcGIS Server2".  The name does not matter in almost all scenarios. Its the activation numbers that are really the key piece of information
@CViles can provide the exact details and correct me if necessary

@nshampur Many thanks for your quick reply and fix 🥇

@Biboba actually please use "ArcGIS Server2" rather than "ArcGIS Image Server"!
The Software Authorization Wizard is designed to only authorize a single core product at a time, which effectively means that the last core product in the provisioning file is the only one used.  This is why only one of your current roles is being authorized.  The provisioning name "ArcGIS Image Server" is also a valid core product name, which would result in the exact same issue continuing to occur!

@nshampur  and @CViles, I confirm that udating the variable name works.
With the following config file, ArcGIS Server role, Image Server role and Network analyst extension were authorized in one shot:
// Features and authorization numbers
ArcGIS Server=ECPAAAAAAAAA
ArcGIS Server1=ECPBBBBBBBBBB
Network Analyst Server=ECPCCCCCCCCC
Thanks

Closed as resolved


[SPARK-13309][SQL] Fix type inference issue with CSV data

Fix type inference issue for sparse CSV data - https://issues.apache.org/jira/browse/SPARK-13309

Can one of the admins verify this patch?

cc @HyukjinKwon  want to review this one?

Test build #2542 has started for PR 11194 at commit 5edaa2a.

Test build #2542 has finished for PR 11194 at commit 5edaa2a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

@rxin Sure.

Overall, it looks good to me. This was already merged in https://github.com/databricks/spark-csv/pull/261 and the logic looks identical.

Actually, I have had a thought that we might have to make a class such as TestCSVData for dataset for testing (similarly with TestJsonData for JSON datasource) or a class like CSVTest (similarly with OrcTest fpr ORC datasource) rather than adding test CSV files for everytime.
I think this might better be done in another PR. If you agree on this, I will create an issue and PR for this.

test this please

Yes, could be done. I personally prefer to keep code and data separate. This way:

Changing the data does not require us to compile the code.
Reading through the source code is friendly if it does not contain data in between.

@HyukjinKwon @rxin Is this waiting on me? Just want to confirm it I am expected to add anything more.

Could we please merge this?

Sorry for the delay. I'm merging this in master. Thanks!

@rxin @HyukjinKwon thank you.


V2 - Tree Benefits Calculator - create test case

Create a test case.

In data/species.json add a new field: 'iTreeCode'.

For species "ID": "190". Add new field called: "iTreeCode" = 'THPL'

In Dev firebase db create a field that saves the 'iTreeCode'. Based on the mapping in the species.json

@https://github.com/blahutkam is on this


Cucumber messages testcasestarted platform

Adds a new Platform field to TestCaseStarted so consumers can know more about the platform where the tests ran. The main usecase for this is analytics.
Supersedes #616

Superseded by #626

Closed the wrong issue


Task Notification Sounds

can the notifications have a sound?
i tend to ignore notification on my setup but sounds really work well on me. lack of sound causes me to not see/ignore notifications and lose track of stuff

hi, @ivan-lednev thanks providing  such a good plugin for free.
Is there such kind of feature. If it is how can we enable. It sound really helps us to be 	conscious of task has started.

Hello, @livan3li! There is an option to trigger notifications on task start. On my Windows machine this produces a sound. What kind of computer do you use?

Hello, @livan3li! There is an option to trigger notifications on task start. On my Windows machine this produces a sound. What kind of computer do you use?

I'm on Linux. Where is that option? Is it in Obsidian or in Windows?

This option is inside Day Planner settings in Obsidian

This option is inside Day Planner settings in Obsidian

But there is only on option which is Task Notification and it doesn't produce any sound on my machine. I'm using gnome DE.

I don't have a Linux machine to debug and fix this, so marking it as help wanted.

I don't have a Linux machine to debug and fix this, so marking it as help wanted.

Alright, thank you. I appreciate your help.

I have been investigating this issue, and it appears to be a common problem in Linux. For example, this issue occurred in Signal, where the developer resolved it by implementing a solution also used in statusbar-pomo-obsidian, which involves playing a sound along with the notification.


preprocess  ERROR

Who can tell me where this path can be changed?
I have installed this package, but I don't know where to change the path below.
Thank you!!!

@JingsenZhang
That is interesting.
I haven't test my code under Windows system.
The extension name (.de_core_news_sm) is different from mine (.de).
Please check if the corpus files are under the data folder (.data\multi30k\).
To set your own path to your corpus files, set up the parameters root and exts in the splits function call. doc
Hope this helps. Feel free to ask if you have other problems.
Best,
Yu-Hsiang

@jadore801120
Thank you!
The shortcut was not available when I installed 'en' and 'de' on Windows, like this

When i use 'en' in code,it says it is not available.So I try to replace 'en' in the code with en_core_web_sm,
however,under the data folder ,it is 'train.en' not 'train.en_core_web_sm'

It feels like a contradiction.

@JingsenZhang
I install the spacy model with python -m spacy download en; python -m spacy download de.
It seems that you install with python -m spacy download en_core_web_sm.
In this case, the extension name and the spacy model name are inconsistent.
Feel free to split it into 2 different variables (don't just use opt.lang_src to indicate both file extension name and spacy model).
Hope this helps.
Yu-Hsiang

@jadore801120
Thank you for your help !!! And I have performed all the actions you listed.

How can I use this model, for example to complete a small task or test its effect.
Thank you again!

I also have some preprocess error when I try to run the code in the pycharm
like this


Update alter-role-transact-sql.md

ALTER ROLE only supported for Azure Synapse Analytics Serverless SQL Pool.
This is not mentioned in Syntax. (Since customers copy / paste the syntax from the public documents, this needs to be specified under syntax comments.

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

@VanMSFT – Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

@WilliamDAssafMSFT - Hey William, can you please verify and approve? Thanks!
#assign:WilliamDAssafMSFT

Hi @charithcaldera thanks for taking the time to make a suggestion. In this case we shouldn't just provide this Applies To information in a code comment - those aren't translated - but in the Arguments/Remarks, or if it doesn't apply to a whole product, potentially in the Applies To line and moniker section of the metadata.
However, I just tested and I was able to create and rename a role in a dedicated SQL pool in both standalone and in a Synapse workspace. The article appears to be correct in that you cannot add a member in a dedicated SQL pool in both standalone and in a Synapse workspace via the ALTER ROLE. Can you be more specific as to what platforms and what behavior is not supported?

Hi @charithcaldera thanks for taking the time to make a suggestion. In this case we shouldn't just provide this Applies To information in a code comment - those aren't translated - but in the Arguments/Remarks, or if it doesn't apply to a whole product, potentially in the Applies To line and moniker section of the metadata.
However, I just tested and I was able to create and rename a role in a dedicated SQL pool in both standalone and in a Synapse workspace. The article appears to be correct in that you cannot add a member in a dedicated SQL pool in both standalone and in a Synapse workspace via the ALTER ROLE. Can you be more specific as to what platforms and what behavior is not supported?

Hi @WilliamDAssafMSFT ,
Thank you for the clarification.
"ALTER ROLE ADD MEMBER / DROP MEMBER" does not work with Synapse Dedicated SQL Pools.
As you mentioned, "ALTER ROLE ALTER NAME" works as expected.
I noticed already its mentioned in the document already, but I was having the concern about the syntax area, which you already answered it.
I will close this PR since there is no further actions.
Thank you @WilliamDAssafMSFT.


[BUG] setting calendar description "status: completed: moves task to odd time with weird title

❗❗❗ Please have a close look at the logs before posting them here. Amend the
logs if you see your username or password there.
To Reproduce
Note, I am using scheduled time

enter task in tw, scheduled for today at some time
sync
in Gcal, edit event descripts, change to `status: completed"
sync

Expected Behavior
event updates with check mark emoji, w/o moving the scheduled time
📸 Screenshots

This is dealt with in another resolved issue - https://github.com/bergercookie/syncall/issues/69
Closing this


Update qs package dependency for request 2.88.2

What
node-zendesk requires request which is deprecated. In turn request depends on qs v6.5.2 which contains vulnerability - https://github.com/alphagov/paas-admin/security/dependabot/29
To mitigate, we use the npm overrides property and force a patched version of qs for request only.
where request is used
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
Before
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>deduped
    └──<EMAIL_ADDRESS>deduped

After
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>deduped
├──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>deduped
    └──<EMAIL_ADDRESS>deduped

How to review

tests pass
zendesk tickets still get created - > currently deployed to dev03

Who can review
not @kr8n3r

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

the only thing to do here @paroxp would be to replace node-zendesk with node18 fetch or axios. node-zendesk itself isn't deprecated, but cannot wait on them to rewrite and remove request


identifierFormat: null causes AD FS error on metadata importing

When identifierFormat: null the passport-saml generated metadata with a tag <NameIDFormat/> which causes an error in AD FS: The value 'NameIDFormat' must be an absolute URI.
Setting the identifierFormat: urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified allows importing the metadata file successfully.
Suggestion: when identifierFormat is set to null, generate urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified

@ahavriluk You mean you have a pull request ready on your local machine? Just push it your own Github account, which already have permission to push to. If you cloned this project, you may need to set up a second remote for your account. Review Github documentation on creating pull requests.

Thanks, I adjusted this and got past that AD FS import error.  On invoking the init to do the SamlRequest POST ADFS is giving an error though:

The relying party "https://example.com/saml/consume" is not configured with SAML Assertion Consumer Services
Relying party: https://example.com/saml/consume
This request Failed.
User Action
Use the AD FS Management snap-in to configure one or more Assertion Consumer Services for this relying party.

Any idea how to resolve this?   I was not sure with this library how to generate assertions/claims in the AuthRequest or setup and convey to IDP that we want to get a set of attributes back like email... ?
How to use this: "attributeConsumingServiceIndex" . ? (I just put it to "1" and it didn't help.

Did you setup Relying Party end points in ADFS?


Wrap & Optimize rest bin collator

What problem does this PR solve?
Issue Number: close #5294
Problem Summary:
What is changed and how it works?
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/merge


Phantomjs executable needs to be in PATH

root@kali:~/shuriken# python shuriken_xss.py -u http://dramaonline.pk/search.php?q={xss} -p wordlist.txt
Traceback (most recent call last):
  File "shuriken_xss.py", line 198, in <module>
    Shuriken().main()
  File "shuriken_xss.py", line 37, in __init__
    self.browser = Browser("phantomjs")
  File "/usr/local/lib/python2.7/dist-packages/splinter/browser.py", line 63, in Browser
    return driver(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/splinter/driver/webdriver/phantomjs.py", line 38, in __init__
    self.driver = PhantomJS(desired_capabilities=capabilities, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/phantomjs/webdriver.py", line 52, in __init__
    self.service.start()
  File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/common/service.py", line 81, in start
    os.path.basename(self.path), self.start_error_message)
selenium.common.exceptions.WebDriverException: Message: 'phantomjs' executable needs to be in PATH.

Oh sorry, its something that needs to be downloaded. My bad 👍
Btw you should mention the requirements in your repo.

Hey! Thanks for flagging this up, I just merged a commit made by @paralax which addresses this issue. Now, you should be able to get all the dependencies via pip so it should be easier to get going with Shuriken. Hope you enjoy using the tool and please let me know if you have any additional ideas for improvements!

@shogunlab I have done everything your readme tells a user to do but shuriken is taking forever to scan.
Here's a short video I have made regarding the issue: http://shuriken.vidmeup.com/

I'll investigate the issue and see if there's anything I can add to improve the speed.

If you are getting this error, you need 'geckodriver'.
You can find how to get it by googling: 'install geckodriver on windows/linux/mac'.


MidHook x86: ESP is offset by -8

Hey, I'm trying to move some hooks that originally used injector's MakeInline, which works in a similar manner, to use midhook instead.
For demo purposes, the hook only replicates original code.
Original game code:
.text:00A200DC      F3 0F 11 4C 24 60           movss   dword ptr [esp+60h], xmm1
injector version:
void operator()(injector::reg_pack& regs)
{
    static float f = 0.0f;
    _asm { movss f, xmm1 }
    *(float*)(regs.esp + 0x60) = f;
}

MakeNOP(0xA200DC, 6); // because injector does that
safetyhook::create_mid(0xA200DC,
	[](SafetyHookContext& regs)
	{
		*(float*)(regs.esp + 0x60) = regs.xmm1.f32[0];
	});

Crashes later in that function.
I've compared the esp value and determined that doing
safetyhook::create_mid(0xA200DC,
	[](SafetyHookContext& regs)
	{
		*(float*)(regs.esp + 0x60 + 8) = regs.xmm1.f32[0];
	});

works fine.
I am not sure if this is intended or not, so let me know.
v0.2.0 was used to compile.

https://github.com/cursey/safetyhook/blob/v0.2.0/include/safetyhook/context.hpp#L21-L23
Maybe this new note has something to do with it.
There was also some information in #46 & #47 about it. But I'm sure you already poked around these already.

@ThirteenAG can you try #49 and see if it fixes this behavior?

@ThirteenAG can you try #49 and see if it fixes this behavior?

Yes, works. On an unrelated note, but at the same location, I can't seem to get rid of asm:
static float f = 0.0f;
_asm { movss f, xmm1 } // f == -0.000668681168

That way the hook works as intended.
If I do
static float f = 0.0f;
f = regs.xmm1.f32[0];

Then it stops working, probably because of this:

Other hooks work fine, only this one behaves this way, so I'm not sure it's safetyhook's issue.
Surrounding code for more context:
.text:00A200B3 12C E8 38 26 00 00                                call    sub_A226F0
.text:00A200B8 104 F3 0F 10 44 24 2C                             movss   xmm0, [esp+100h+var_D4]
.text:00A200BE 104 0F 57 C9                                      xorps   xmm1, xmm1
.text:00A200C1 104 0F 2E C1                                      ucomiss xmm0, xmm1
.text:00A200C4 104 9F                                            lahf
.text:00A200C5 104 F6 C4 44                                      test    ah, 44h
.text:00A200C8 104 7B 12                                         jnp     short loc_A200DC
.text:00A200CA 104 F3 0F 10 4C 24 48                             movss   xmm1, [esp+100h+var_B8]
.text:00A200D0 104 F3 0F 5C 8E 08 03 00 00                       subss   xmm1, dword ptr [esi+308h]
.text:00A200D8 104 F3 0F 5E C8                                   divss   xmm1, xmm0
.text:00A200DC
.text:00A200DC                                   loc_A200DC:                             ; CODE XREF: CCamFollowPed::sub_A1F9D0(void)+6F8↑j
>>> .text:00A200DC 104 F3 0F 11 4C 24 60                             movss   dword ptr [esp+60h], xmm1
.text:00A200E2 104 85 DB                                         test    ebx, ebx
.text:00A200E4 104 74 1B                                         jz      short loc_A20101
.text:00A200E6 104 F6 86 8C 03 00 00 04                          test    byte ptr [esi+38Ch], 4
.text:00A200ED 104 74 12                                         jz      short loc_A20101
.text:00A200EF 104 8D 44 24 18                                   lea     eax, [esp+100h+var_E8]
.text:00A200F3 104 50                                            push    eax
.text:00A200F4 108 8D 44 24 2C                                   lea     eax, [esp+104h+var_D8]
.text:00A200F8 108 50                                            push    eax
.text:00A200F9 10C 53                                            push    ebx
.text:00A200FA 110 8B CE                                         mov     ecx, esi
.text:00A200FC 110 E8 4F DD FF FF                                call    sub_A1DE50

From the look of it, it seems like xmm1 has been captured incorrectly? Are you able to provide a minimal test that reproduces this issue?

Was it working before #49? Are you able to provide a minimal test that reproduces this issue?

No, it wasn't. I'll prepare an example.

I think that should be it:
#include <iostream>
#include <safetyhook.hpp>

__declspec(noinline) void test() {
    std::cout << "test\n";
}

__declspec(noinline) int add_42(int a) {

    static float f = 0.111f;
    _asm { movss xmm1, f } // f = regs.xmm1.f32[0];
    _asm { test    ebx, ebx }

    _asm call dword ptr ds : [test]

    return a + 42;
}

void hooked_test(SafetyHookContext& regs) {

    static volatile float f1 = 0.0f;
    static volatile float f2 = 0.0f;
    _asm { movss f1, xmm1 }
    f2 = regs.xmm1.f32[0];

    regs.eax = 1337;
}

SafetyHookMid g_hook{};

int main() {
    g_hook = safetyhook::create_mid(&test, hooked_test);

    std::cout << add_42(3) << "\n";

    std::cout << add_42(4) << "\n";

    return 0;
}

Thanks for the test @ThirteenAG . Can you try #50 and let me know if it fixes it for you?

Thanks for the test @ThirteenAG . Can you try #50 and let me know if it fixes it for you?

Works like a charm, thank you!


Adding CF metadata

Change Summary
This is work towards adding CF metadata to xwrf
Related issue number

Closes #15

Checklist

[x] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable

Changes still in development:

[ ] Add custom dictionary option to backend

Do any of you have comments regarding using a yaml file to store this information? It seemed intuitive to me, but is this good form?

Hi everyone, so I had a bit of time and implemented the configuration in donfig. First of all, I hope I am using the package correctly. I'm not terribly experienced with dask configurations, so please correct me if I should have done it differently :)
Secondly, I simply implemented a preliminary structure into the yaml file which is probably not ideal. If any of you have ideas on how to improve this, they are very welcome!

Cheers for the review, I just added the changes you requested :)

Cheers for the review, I just added the changes you requested :)

Great 👍🏽! Is this ready to transition from draft to a ready PR or are you still planning to add more work to it?

I think it's ready for merging if you are satisfied :+1:

If we think of any way to improve this in the future, we can just open new PRs I guess

Thanks for putting this together!


Typo "Teams Room"→"Teams Rooms"

https://learn.microsoft.com/en-us/microsoftteams/rooms/data-and-privacy-info
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/blob/live/Teams/rooms/data-and-privacy-info.md
#PingMSFTDocs

@altsou
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Release 0.9.0

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Release of 0.9.0

Give a summary of what the PR does, explaining any non-trivial design decisions

Other references

If we're on ghcr back on openhands, we need to re-search for "opendevin" in the codebase.
Also, slack links still go to opendevin as well. @rbren


python 3.7

I'm getting a suppressed traceback though:

Full tox -e py37 output
▶ tox -e py37
GLOB sdist-make: /Users/peterbe/dev/PYTHON/hyper/setup.py
py37 inst-nodeps: /Users/peterbe/dev/PYTHON/hyper/.tox/.tmp/package/1/hyper-0.8.0.dev0.zip
py37 installed: apipkg==1.5,atomicwrites==1.3.0,attrs==19.1.0,brotlipy==0.7.0,certifi==2019.3.9,cffi==1.12.3,chardet==3.0.4,coverage==4.5.3,execnet==1.6.0,h2==2.6.2,hpack==3.0.0,hyper==0.8.0.dev0,hyperframe==3.2.0,idna==2.8,mock==2.0.0,more-itertools==7.0.0,pbr==5.1.3,pluggy==0.9.0,py==1.8.0,pycparser==2.19,pytest==4.4.1,pytest-cov==2.6.1,pytest-forked==1.0.2,pytest-xdist==1.28.0,requests==2.21.0,rfc3986==1.3.1,six==1.12.0,urllib3==1.24.2
py37 run-test-pre: PYTHONHASHSEED='2726258995'
py37 run-test: commands[0] | coverage run -m py.test /Users/peterbe/dev/PYTHON/hyper/test/
================================================================================================ test session starts ================================================================================================
platform darwin -- Python 3.7.3, pytest-4.4.1, py-1.8.0, pluggy-0.9.0
cachedir: .tox/py37/.pytest_cache
rootdir: /Users/peterbe/dev/PYTHON/hyper
plugins: xdist-1.28.0, forked-1.0.2, cov-2.6.1
collected 303 items
test/test_SSLContext.py .....                                                                                [  1%]
test/test_abstraction.py ....                                                                                [  2%]
test/test_cli.py ................................                                                                                [ 13%]
test/test_headers.py ............................                                                                                [ 22%]
test/test_http11.py .............................................................                                                                                [ 42%]
test/test_http20.py .                                                                                [ 43%]
test/test_hyper.py ....Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/threading.py", line 917, in _bootstrap_inner
self.run()
File "/Users/peterbe/dev/PYTHON/hyper/test/server.py", line 103, in run
self.server = self._start_server()
File "/Users/peterbe/dev/PYTHON/hyper/test/server.py", line 96, in _start_server
self.socket_handler(sock)
File "/Users/peterbe/dev/PYTHON/hyper/test/test_http20.py", line 21, in socket_handler
sock = listener.accept()[0]
File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/ssl.py", line 1166, in accept
server_side=True)
File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/ssl.py", line 412, in wrap_socket
session=session
File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/ssl.py", line 853, in _create
self.do_handshake()
File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/ssl.py", line 1117, in do_handshake
self._sslobj.do_handshake()
ssl.SSLEOFError: EOF occurred in violation of protocol (_ssl.c:1056)
....................................................................................                                                                                [ 72%]
test/test_hyper_SSLContext.py s                                                                                [ 72%]
test/test_import.py s.                                                                                [ 73%]
test/test_integration.py ...................................                                                                                [ 84%]
test/test_integration_http11.py ............                                                                                [ 88%]
test/test_parser.py ....                                                                                [ 90%]
test/test_socket.py .................                                                                                [ 95%]
test/test_ssl_socket.py ...                                                                                [ 96%]
test/test_windowmanager.py ..........                                                                                [100%]
================================================================================================= warnings summary ==================================================================================================
hyper/http11/connection.py:13
hyper/http11/connection.py:13
/Users/peterbe/dev/PYTHON/hyper/hyper/http11/connection.py:13: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
from collections import Iterable, Mapping
.tox/py37/lib/python3.7/site-packages/hyperframe/flags.py:14
/Users/peterbe/dev/PYTHON/hyper/.tox/py37/lib/python3.7/site-packages/hyperframe/flags.py:14: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
class Flags(collections.MutableSet):
hyper/common/headers.py:13
/Users/peterbe/dev/PYTHON/hyper/hyper/common/headers.py:13: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
class HTTPHeaderMap(collections.MutableMapping):
test/test_hyper.py::TestHyperConnection::test_closed_connections_are_reset
/Users/peterbe/dev/PYTHON/hyper/hyper/http20/connection.py:465: DeprecationWarning: The 'warn' method is deprecated, use 'warning' instead
log.warn("GoAway frame could not be sent: %s" % e)
test/test_hyper.py::TestServerPush::test_reset_pushed_streams_when_push_disabled
test/test_hyper.py::TestUpgradingPush::test_reset_pushed_streams_when_push_disabled
/Users/peterbe/dev/PYTHON/hyper/hyper/http20/connection.py:843: DeprecationWarning: The 'warn' method is deprecated, use 'warning' instead
stream_id, e)
-- Docs: https://docs.pytest.org/en/latest/warnings.html
================================================================================ 301 passed, 2 skipped, 7 warnings in 10.24 seconds =================================================================================
py37 run-test: commands[1] | coverage report
Name                         Stmts   Miss  Cover   Missing
hyper/init.py               13      0   100%
hyper/cli.py                   143      0   100%
hyper/common/init.py         1      0   100%
hyper/common/bufsocket.py       89      0   100%
hyper/common/connection.py      34      0   100%
hyper/common/decoder.py         21      0   100%
hyper/common/exceptions.py      25      0   100%
hyper/common/headers.py         94      0   100%
hyper/common/util.py            27      0   100%
hyper/contrib.py                77      0   100%
hyper/http11/init.py         1      0   100%
hyper/http11/connection.py     200      0   100%
hyper/http11/parser.py          35      0   100%
hyper/http11/response.py       146      0   100%
hyper/http20/init.py         1      0   100%
hyper/http20/connection.py     291      0   100%
hyper/http20/errors.py          23      0   100%
hyper/http20/exceptions.py      14      0   100%
hyper/http20/response.py        91      0   100%
hyper/http20/stream.py         131      0   100%
hyper/http20/util.py            21      0   100%
hyper/http20/window.py          27      0   100%
hyper/tls.py                    47      0   100%
TOTAL                         1552      0   100%
______________________________________________________________________________________________________ summary ______________________________________________________________________________________________________
py37: commands succeeded
congratulations :)

For the record, to make that warning go away you just...
▶ pip install ndg-httpsclient
Collecting ndg-httpsclient
...
Requirement already satisfied: pycparser in /Users/peterbe/virtualenvs/hyper/lib/python3.7/site-packages (from cffi!=1.11.3,>=1.8->cryptography>=2.3->PyOpenSSL->ndg-httpsclient) (2.19)
Installing collected packages: asn1crypto, cryptography, PyOpenSSL, pyasn1, ndg-httpsclient
Successfully installed PyOpenSSL-19.0.0 asn1crypto-0.24.0 cryptography-2.6.1 ndg-httpsclient-0.5.1 pyasn1-0.4.5

So what does that mean? That ndg-httpsclient needs to be mentioned as a requirement in setup.py?

Also, I didn't do any extensive manual usage/testing BEFORE adding ngd-httpsclient (and its peers) but I was able to use hyper for a simple GET before on a https:// site.


Add option to convert between pure and non-pure nodes

Especially useful for call method nodes because who wants to have a full execution node for .ToString().

Done in https://github.com/RobinKa/netprints/commit/d03b88e486df50f5cd9f69f0f61929782fef87cb


About the memory,Thanks

Hello, thank you for your wonderful work, please ask how much GPU memory is needed for the first and second stages of training in this project? And the number of GPUs used and training days?

It depends on batch size, frame_num, img_width, activation_checkpointing, mixed_precision type, distributed_type, etc. For stage1, it needs 10-20GB. Stage2 typically needs 40-80GB

Thank you for your reply. When using the NVIDIA A800, how about the training days and GPU count following the default settings？
Since I only have two A6000s, I would like to inquire about the details and evaluate the training possibility of experiments conducted on my devices.
Thanks again！

It depends on batch size, frame_num, img_width, activation_checkpointing, mixed_precision type, distributed_type, etc. For stage1, it needs 10-20GB. Stage2 typically needs 40-80GB

Thank you for your reply. When using the NVIDIA A800, how about the training days and GPU count following the default settings？ Since I only have two A6000s, I would like to inquire about the details and evaluate the training possibility of experiments conducted on my devices. Thanks again！

8 A800s cost ~a week.

hi ,i also have some question about memory, i found it's not enough to train 32 frames use 40G memory, i want to ask that the memory cost ~, and it is possible to train 32 frames use 40 memory ?


Branch for testing staking with farm validators using lockups

Branch for testing staking with farm validators using lockups:

Create an account on the render pull request preview
Deploy a lockup to that account using https://testnet.lockup.tech/, (change wallet.testnet.near.org to near-wallet-pr-2588.onrender.com when logging in and signing transactions)
Stake with one of the following validators:
zentriav2.factory.colorpalette.testnet
domanodes.factory.colorpalette.testnet

No longer necessary, testnet LOCKUP_ACCOUNT_ID_SUFFIX is updated to lockup.devnet on render and #2627 updates the defaults


refactor: grapqhl request logging and error handling

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.56%. Comparing base (ed53a65) to head (f27f4b2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
- Coverage   97.63%   97.56%   -0.07%
==========================================
  Files          11       11
  Lines         296      288       -8
  Branches       57       55       -2
==========================================
- Hits          289      281       -8
  Misses          7        7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


GetX 5 GetMiddleware almost all functions like onPageBuilt run twice

With nested navigation in example_nav2 example project, they will all run twice more.
Codes:
class AppPages {
  AppPages._();

  static const initial = Routes.home;

  static final routes = [
    GetPage(
      middlewares: [MainMiddleware()],
      preventDuplicates: true,
      name: _Paths.home,
      page: () => const HomeView(),
      bindings: [
        HomeBinding(),
      ],
      title: null,
      participatesInRootNavigator: true,
      children: [
        GetPage(
          name: _Paths.dashboard,
          page: () => const DashboardView(),
          bindings: [
            DashboardBinding(),
          ],
        ),
        GetPage(
          // middlewares: [
          //   //only enter this route when authed
          //   EnsureAuthMiddleware(),
          // ],
          name: _Paths.profile,
          page: () => const ProfileView(),
          title: 'Profile',
          transition: Transition.size,
          bindings: [ProfileBinding()],
        ),
        GetPage(
          name: _Paths.products,
          page: () => const ProductsView(),
          title: 'Products',
          transition: Transition.cupertino,
          showCupertinoParallax: true,
          // participatesInRootNavigator: true,
          bindings: [ProductsBinding(), ProductDetailsBinding()],
          children: [
            GetPage(
              name: _Paths.settings,
              page: () => const SettingsView(),
              participatesInRootNavigator: true,
              bindings: [
                SettingsBinding(),
              ],
            ),
            GetPage(
              name: _Paths.productDetails,
              transition: Transition.cupertino,
              showCupertinoParallax: true,
              page: () => const ProductDetailsView(),
              participatesInRootNavigator: true,
              bindings: const [],
              // middlewares: [
              //   //only enter this route when authed
              //   EnsureAuthMiddleware(),
              // ],
            ),
          ],
        ),
      ],
    ),
    GetPage(
      name: _Paths.settings,
      page: () => const SettingsView(),
      bindings: [
        SettingsBinding(),
      ],
    ),
  ];
}

class MainMiddleware extends GetMiddleware {
  @override
  void onPageDispose() {
    log('MainMiddleware onPageDispose');
    super.onPageDispose();
  }

  @override
  Widget onPageBuilt(Widget page) {
    log('MainMiddleware onPageBuilt');
    return super.onPageBuilt(page);
  }

  @override
  GetPage? onPageCalled(GetPage? page) {
    log('MainMiddleware onPageCalled');
    return super.onPageCalled(page);
  }

  @override
  List<R>? onBindingsStart<R>(List<R>? bindings) {
    log('MainMiddleware onBindingsStart');
    return super.onBindingsStart(bindings);
  }

  @override
  GetPageBuilder? onPageBuildStart(GetPageBuilder? page) {
    log('MainMiddleware onPageBuildStart');
    return super.onPageBuildStart(page);
  }
}

Screenshots:

And onPageDispose also run twice more!!!

This was introduced by Nested Bindings. When someone directly accesses a nested route in the browser, the parent's dependencies and middleware must be loaded.
However, you're right, it should only start once, so I'm going to change the api to initialize Bindings only once.

This was introduced by Nested Bindings. When someone directly accesses a nested route in the browser, the parent's dependencies and middleware must be loaded.
However, you're right, it should only start once, so I'm going to change the api to initialize Bindings only once.

Thanks, so you mean that mb next version of GetX will change this?

Yeah, definitely.

Yeah, definitely.

Thanks for your help!
Btw, could you please help with this issuse? Or mb there is something wrong with my nested routes.
https://github.com/jonataslaw/getx/issues/3175


install issue with 1.2.2 and angular 2.0.0-rc.5

Hi,
I have trouble installing v 1.2.2 with angular2 rc5 (with angular cli)
all steps has been performed (npm install, system-config modifications, declaring the provider, importing CookieService)
the Typescript files compiles without any issue but I have this error @app loading in the browser
zone.js:101 GET http://localhost:4200/vendor/angular2-cookie/core 404 (Not Found)
I checked the dist\vendor\angular2-cookie folder and it s here (core.js + other stuff inside)
Find a lot of similar issues while googling so I guess the issue is more related to changes in angular rather than the angular cookie module itself, anyway I post the issue here because other users may experience this issue.
I posted the question to stackoverflow as well http://stackoverflow.com/questions/39268655/importing-angular2-cookie-in-angular2-rc5-project

Answered on SO.


lottie: incorrect rendering, elements appear upside down

Hello team
some elements in the provided lottie are being rendered upside down (rotated 180deg), any ideas ?
lottie file -> reportLoadingDone.json
thorvg -> v0.14.8
related issue -> https://github.com/LottieFiles/dotlottie-web/issues/351

seems related to expressions.

curlprit:
"var $bm_rt;\nvar $bm_rt;\nvar amp, freq, decay, n, n, t, t, v;\namp = 0.06;\nfreq = 2;\ndecay = 4;\n$bm_rt = $bm_rt = n = 0;\nif (numKeys > 0) {\n    $bm_rt = $bm_rt = n = nearestKey(time).index;\n    if (key(n).time > time) {\n        n--;\n    }\n}\nif (n == 0) {\n    $bm_rt = $bm_rt = t = 0;\n} else {\n    $bm_rt = $bm_rt = t = sub(time, key(n).time);\n}\nif (n > 0) {\n    v = velocityAtTime(sub(key(n).time, div(thisComp.frameDuration, 10)));\n    $bm_rt = $bm_rt = add(value, div(mul(mul(v, amp), Math.sin(mul(mul(mul(freq, t), 2), Math.PI))), Math.exp(mul(decay, t))));\n} else {\n    $bm_rt = $bm_rt = value;\n}"

sample.json

@hermet

from https://helpx.adobe.com/after-effects/using/expression-language-reference.html:

key(index)
Return type: Key or MarkerKey.

Argument type: index is a Number.

Returns the Key or MarkerKey object by number. For example, key(1) returns the first keyframe.

we count frames from 0, but in expressions the first index should be 1 (maybe it is enough in _key() decrease the key value and in _nearestKey() increase the index value?)
also support for floats in _velocityAtTime is missing

@mgrudzinska yes, I know the issue details!

the fix will be in v0.15


Added code-of-conduct file #190

I've added a code-of-conduct file. Please revise it.

Thank you @sujeevan81  but we already merged another pull request for this.
My apologies for the delay in getting back to you


Rename useHttpForHttps to portMapping

LGTM, you can merge.

Because this is an API-breaking change we should document it. I would propose to add an "upgrade-notes.md" file similar to what we are doing for OpenLayers (https://github.com/openlayers/ol3/blob/master/changelog/upgrade-notes.md).

Like this?
https://github.com/mapfish/mapfish-print/pull/352


feat(postinstall): allow environment variable to be in the .npmrc

Pull Request Test Coverage Report for Build 138

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 137:
0.0%

Covered Lines:
386

Relevant Lines:
386

💛  - Coveralls

Hi,
thanks for your submission. As the variable name doesn't comply to the linting rules (snake_case), I'd want it to be accessed using the bracket operator.
I'd be very happy if you could either use the bracket operator here, or change the existing access to env to use a dot to stay consistent.
Thanks


[Mining] Monero / Tari block mined notifications

Blocked by: #188

the above notifications should be shown as modals, with backdrop
user interaction is needed for the notification to disappear (either "got it" button or backdrop click)
if any other modal is currently open (e.g. settings), the notification should wait - once the current modal is closed, the new one appears after short delay (300~500ms ?), effectively making a stack of popups waiting for the user
the same principle explained above should apply when there are multiple different mining notifications, so:
if there are 3 blocks mined of the same type (XTR/XMR), the text should combine them "3 blocks were mined" etc.
but if there are blocks of different type XTR & XMR, we should not cmbine them, they should stack as explained with modals)
we should show a text OS notification about mined blocks, example notification:

[Title] Tari Launchpad: Fantaritastic!
[Body] You've just mined a Tari block!

tarnas14 commented:
internal discussion:

should they pop up like modals to take the whole screen and user attention?
do they auto-close or is user interaction necessary (either "Got it" or clicking backdrop)?
what if the user is currently in a modal (like settings) - do we obscure everything to show them that a block was mined?
what if we mine multiple blocks in succession in a short time before user interacts with the previous notification - do they stack?
do we show an OS notification about mined blocks?

~ answer in description

tarnas14 commented:
@CjS77 @alex1307
what is the event that we should listen to for mining notifications?

CjS77 commented:
Doesn't the design have pop-ups / notifications for whenever you mine a new block?

CjS77 commented:

This is still to be written

CjS77 commented:
https://github.com/Altalogy/tari/issues/155

tarnas14 commented:

Doesn't the design have pop-ups / notifications for whenever you mine a new block?

yes, they're designed - included in the description, also figured out OS notifications
thanks for the backend issue

tarnas14 commented:
preparing the frontend while backend is unblocking


Add flutter_lints package and fix linter offenses

Hi, great project 👍
I've just thought that it would be nice to use flutter_lints just like the default Flutter skeleton template

Thanks for fixing those linter issues @karniv00l!


allow the proxy to be a deployment (instead of a daemonset)

added parameters :

controllers.proxyAddress
controllers.proxyPort
proxy.kind

Combined with a hostIp service, this allows to have only one (or a few) instance(s) of the proxy, accessible through that IP, instead of one instance per node. For this to work, you need to use the hostIp directly (containerd cannot query the DNS for the service IP, because the DNS lives inside the cluster).
Example of service definition :
apiVersion: v1
kind: Service
metadata:
  name: kuik-proxy
  namespace: kuik-system
spec:
  ports:
  - name: registry
    port: 7439
    protocol: TCP
    targetPort: 8082
  selector:
    app.kubernetes.io/component: proxy
  type: ClusterIP
  clusterIP: "<IP_ADDRESS>"

Example of values :
controllers:
  proxyAddress: "<IP_ADDRESS>"
  proxyPort: 7439
proxy:
  kind: Deployment
  hostPort: ""
  hostIp: ""

Finally, containerd will try to access the registry as https instead of http, to convince it to use http, configure a mirror like so :
version = 2

[plugins]
    ...
    [plugins."io.containerd.grpc.v1.cri".registry]

      [plugins."io.containerd.grpc.v1.cri".registry.mirrors]

        [plugins."io.containerd.grpc.v1.cri".registry.mirrors."<IP_ADDRESS>:7439"]
          endpoint = ["http://<IP_ADDRESS>:7439"]

Hi @zig, as this is not the path we want to follow, and the linked "performance" issue was tackled down, I'll close this ticket.
As we might missed something, feel free to open an other ticket and detail the value brought by the config you proposed.


res.buffer is not a function in bun

Describe the bug
I am migrating my web app to bun, and it seems the dropbox SDK usage fails to download files.
To Reproduce

Download bun. I'm using 1.0.30.
Install Dropbox SDK JS - bun install --save-dev dropbox
Using this code:

// dropbox.ts

const dropbox = new Dropbox({accessToken: process.env.DROPBOX_ACCESS_TOKEN, fetch: fetch});

const result = await dropbox.filesListFolder({path: "/products", recursive: true})

dropbox.filesDownload({path: "/path/to/file.jpg"}).then(
    {result} => {
        console.debug(result.name)
    }
).catch(err => {
    console.error(err)
})

Run it using bun dropbox.ts

Expected Behavior
I expect it to run, and print the name of my file. The file exists in the path, and is accessible to the app (the node JS app finds it and has access to it; moreover, if I use a non-existent file, I get a 409 error, and I don't in this case).
Actual Behavior
I get an error, like this:
62 |     if ((0, _utils.isWindowOrWorker)()) {
63 |       res.blob().then(function (data) {
64 |         return resolve(data);
65 |       });
66 |     } else {
67 |       res.buffer().then(function (data) {
           ^
TypeError: res.buffer is not a function. (In 'res.buffer()', 'res.buffer' is undefined)
      at /project/node_modules/dropbox/cjs/src/response.js:67:7
      at new Promise (:1:21)
      at parseDownloadResponse (/project/node_modules/dropbox/cjs/src/response.js:61:10)
      at /project/node_modules/dropbox/cjs/src/dropbox.js:146:20

Screenshots
N/A
Versions

What version of the SDK are you using? 10.34.0
What version of the language are you using? Whatever Bun's using
Are you using Javascript or Typescript? TypeScript
What platform are you using? (if applicable) Bun 1.0.30

Additional context
None.

I think I've figured it out. It seems that Bun's fetch has a different API than node-fetch; However, it still seems to make sense to fix this problem, so I hope it'll remain open.
To locally fix it (for me it worked at least) :

Install node-fetch: bun install --save-dev node-fetch
Use node-fetch instead of Bun's fetch, meaning add an import fetch from "node-fetch", which should sort it out for the example I offered in the original post.

Thanks for the detailed report! I'll ask the team to look into addressing this.


Improved reporting when missing mime types

While debugging the "invalid codec.mimeType" problem (#29) I found it useful to report the actual identifier of the missing mime type. Oddly the problem went away when I compiled with this change (i.e. I can now target the single armeabi-v7a ABI without seeing the problem at runtime. I doubt it was this change that fixed it, probably just compiling in a different environment.

@rawnsley It's a nice improvement, but it's the code in https://github.com/versatica/libmediasoupclient,  it would be nice the move this PR into libmediasoupclient.


Get more responses than expected

Steps to reproduce
Tell us about your environment:

Puppeteer version:1.3.0
Platform / OS version: Windows 7
URLs (if applicable):
Node.js version: v8.9.4

What steps will reproduce the problem?
I writed a simple code to show the problem:
var puppeteer =require('puppeteer')
var counter ={
    req:0,
    res:0
};

(async function () {
    var browser = await puppeteer.launch({
        headless:false,
        devtools:true,
    })
    var page = await browser.newPage();

    //listen on "request"
    page.on("request",async function (req){
        if(req.url().match("fanyi.baidu.com/v2transapi")) {
            counter.req ++
            console.log("req:",counter.req,"res",counter.res);
        }
    })
    //listenon "response"
    page.on("response",async function (res){
        if(res.url().match("fanyi.baidu.com/v2transapi")) {
            counter.res ++
            console.log("req:",counter.req,"res",counter.res ,"length", (await res.text()).length);
        }
    })

    await page.goto("https://fanyi.baidu.com/#en/zh/hello")
    //await browser.close()
})()

// expected result :
   req :1 res : 0
   req :1 res : 1  length 64411

//actully:
    req: 1 res 0
    (node:16084) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error:
    Protocol error (Network.getResponseBody): No resource with given identifier found undefined
    (node:16084) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the
    future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit
    code.
    req: 2 res 1
    req: 2 res 2 length 64411

I want to catch the request whose URL contain "fanyi.baidu.com/v2transapi" .
And I have catch it manually in my Chrome Dev Tool  (without using puppeteer) , There is only one request contain URL "fanyi.baidu.com/v2transapi"
However , when I use putteteer , I catch two request and  two response .
And I have tried other sites . Some of them have the same problem , and some sites even get this:
// some sites
  req :1 res : 0
  req :1 res : 1
  req :1 res : 2 // response is more than request!

I have noticed that in API document said:

If request fails at some point, then instead of 'requestfinished' event (and possibly instead of 'response' event), the 'requestfailed' event is emitted.

I don't know the cause of my problem and if it's exactly the situation said in the document .  any response/help will be much appreciated.

Thanks for reading !

After reading tht source code  and Chrome DevTools Protocol . I thoughtthat is a problem caused by Chrome DevTools Protocol rather than puppeteer .
But I'm not sure about that . any response/help will be much appreciated . Thanks

And I have catch it manually in my Chrome Dev Tool (without using puppeteer) , There is only one request contain URL "fanyi.baidu.com/v2transapi"
However , when I use putteteer , I catch two request and two response .

@jiladahe1997 it looks like the website reloads itself if it runs for the first time for the user. That's why there are two requests to the URL you mention.
You can see this behavior with devtools:

Open chrome Incognito mode
Open devtools
In Network panel, click "preserve log" checkbox. This way the network panel will preserve requests/responses across navigations
navigate to the https://fanyi.baidu.com/#en/zh/hello

And I have tried other sites . Some of them have the same problem , and some sites even get this:
... // response is more than request!

This is hard to judge; if you have a repro, please re-file a separate issue. We'll take a look.


Add Arbitrum Flux Token logo

Add Arbitrum Flux Token logo

@jiro-ono please help us merge it. thanks

Added in another commit


Unable to use the same entities in different navigations

When using the same entities in different navigations the entity relation of the existing navigation item is removed.
First navigation item:

Second navigation item:

Is there something I'm doing wrong?

Hi @JelmerV-WFC,
That's the case of polymorphic relations, it's overriding the relation like page - item. Those multiple navs we're not supporting from the level of UI yet. That's planned for 1.1 version.

Hi @cyp3rius!
Alright! Do you know if there is a workaround for the time being?

The question if you really want to differ the navigations for footer and header :) What I can guide you is to create the Auditory keys like header, footer and add relevant ones to the each item of your navigation. Than on the client app or the backend part of it you might filter out the items you're not interested in.
That's not a perfect solution, but a workaround that has helped couple of people already.

@JelmerV-WFC Hello! How did you create two navigation? I don't see any select fields

I had the same question as @welch08 How can we create more than one navigation? I can't find any related on the docs. Any guide about this? Thanks.

@shyr1punk planned release for Friday 22nd Oct, last tests are conducted.

Any updates on the planned release of this feature? I am waiting on this and i18n Navigation before I can use this plugin.

We've released version 1.1.0 where polymorphic relations has been removed and provided support for multiple navigations (enabled it in native Content Manager).


Add jobs description

Each job's spec should include description of the job to quickly get understanding of what this job is for.

Fixed by https://github.com/cloudfoundry-community/logsearch-boshrelease/commit/a7d32c4d1270f978def566d3fe4a996efa71515b


[Platform]: fix chiplist spacing in widgets

[Platform]: fix chiplist spacing in widgets
Adjust the spacing around chips in the chiplist component.
Description
Fix the spacing around chips in the Chiplist component. Also fix the spacing between the label and the chips in the widgets for Cancer Hallmarks, Cancer Gene Census, IntoGen, ClinVar
Issue: N/A
Deploy preview: https://deploy-preview-208--ot-platform.netlify.app/
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Widgets and chiplist can be viewed here:

[ ] https://deploy-preview-208--ot-platform.netlify.app/target/ENSG00000171862
[ ] https://deploy-preview-208--ot-platform.netlify.app/evidence/ENSG00000171862/EFO_0000519

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

Thanks @carcruz - do you mean to only have the ChipList component in the ui package, and delete the one in sections/src/components ?

@LucaFumis  Corretto

To move quicker for the release, we will let address !important additions as an indepent work https://github.com/opentargets/issues/issues/3062


Using vscode as EDITOR in WSL doesn't work to write git commit messages

VSCode Version: 1.12.2
OS Version: 10.0.15063 Build 15063
WSL Ubuntu 14.04

Steps to Reproduce:

In Bash, export EDITOR='code -w -n'
In a git repository, make changes.

Investigating further, it appears that the path being passed to VSCode isn't correct:

That's the Linux path (i.e. starting with /mnt/c/...), but over in Windows-land it should just be C:\Users\jacob\...). So somehow between the git -> EDITOR -> code handoff the correct path is getting messed up in some way?

Not sure if vscode should actually fix this, as it is not a good idea to modify linux files using windows apps and tools (https://blogs.msdn.microsoft.com/commandline/2016/11/17/do-not-change-linux-files-using-windows-apps-and-tools/)
But one can have a simple workaround this problem:

https://github.com/Milly/wslpath
This is a script to convert linux path to windows path

A simple wrapper script to launch code with converted path:
~/bin/editor

#!/bin/bash

#extract last argument (the file path)
for last; do true; done

#get all the initial command arguments
all="${@:1:$(($#-1))}"

#launch code with windows path
code $all `wslpath.sh -w $last`

git commit:
EDITOR="/home/ferreus/bin/editor -w -n" git commit

This is not in our control.


Improve task list

fix issues with number vertical alignment and panel top spacing on mobile
add click tracking for panel links
code update to rename the parts of a task list, now contains groups each containing steps, instead of steps containing sections

Best to work through this one by commits - the last commit is a lot of find and replace for variable names.
Before:

After:

Component guide link: https://govuk-static-pr-1182.herokuapp.com/component-guide/task_list
Trello card: https://trello.com/c/hjuxqyuT/275-visual-and-code-tweaks-to-the-task-list
Tested on iphone 6s, Google Nexus 6, iphone 6, Samsung Galaxy s5, ipad air 2, Google nexus 7, Samsung galaxy tab 4, Google Pixel
IE8, IE9, IE10, FF 56, Opera 48, Chrome latest, Safari 5.1, Edge 16

The new vertical spacing fix on mobile looks different to what you get normally on desktop? Should these be vertically centred too?

@vanitabarrett Good spot, no you're right, the numbers should always be vertically centered. What browser/device/screen size/task list was that?

@andysellick Just Chrome on Mac, viewing the small task list in the component guide (herokuapp link in PR above).

@vanitabarrett odd that it looked fine for me. Have had another crack at it, very difficult to get it perfect cross browser (might be slightly off on iphone but little I can do about that), but should be better now.

This looks good to me 👍


Reset callbacks when Cartridge roles hot reload

After #341  callbacks don't reset too. It seems like an unexpected behavior.
At least with cartridge, because each:
myapp.router> require("cartridge.roles").reload()
myapp.router> metrics = require("metrics")

adds 16 callbacks to:
https://github.com/tarantool/metrics/blob/b17a3234daff03481d15efa9a0c24e0cf56888ed/metrics/registry.lua#L13
16, 32, 48...
Old callbacks may be outdated or use outdated data. If we call them, it can lead to unexpected errors.

Let's reset callbacks


gconv: The type conversion does not meet expectations

Go version
go 1.22.6 windows/amd
GoFrame version
2.7.4
Can this bug be reproduced with the latest release?
Option Yes
What did you do?

What did you see happen?
浮点数类型的负数字符串，在执行类型转换后，负数变正了
What did you expect to see?
能正常转换正负浮点数

@gqcn Found the float was not using minus in string
https://github.com/gogf/gf/blob/8c6db247b26e59c930e7fe07a1be6eb9e74cd991/util/gconv/gconv_int.go#L98
https://github.com/gogf/gf/blob/8c6db247b26e59c930e7fe07a1be6eb9e74cd991/util/gconv/gconv_int.go#L125-L129


Fix toc accessibility issue

There is a bug in volto that can be tested on https://demo.plone.org/toc-issue
If you press on a Toc element( with the keyboard) the view will change on the element, but the focus will not move
Actual behaviour:
bug.webm
Expected behaviour
my fix.webm
I fixed the issue with the following approach:
First I made the headings keyboard accessible, and then I listen to the route hash change, when the hash URL had been added I manually changed the focus via js

I have a feeling that the solution is too much and it can potentially cause issues. Better to use react-router-hash-link with smooth scroll, on the TOC links?

@tiberiuichim Thank you for your input, I like your solution so much. I didn't know about react-router-hash-link, and I checked online, and it s also moving the focus. It is perfect! I will start implementing this first thing on Monday.

@tiberiuichim I have implemented the approach that you suggested.


Boolean methods

Description
This pull request only change some methods in PyCrypoBot class.
This update methods that should return a boolean instead of an int to clarify the code.
Type of change
Please make your selection.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] This change requires a new dependency
[ ] Code Maintainability / comments
[x] Code Refactoring / future-proofing

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added pytest unit tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

This looks like a nice update, thanks.
I just want to confirm it doesn't conflict with this:
https://github.com/whittlem/pycryptobot/pull/204

I'm pretty sure it doesn't but I will wait @dthevenin to confirm.

Let’s merge that PR and I will rebase mine

Thanks for the help reviewing. Merging this one now.


How about mocking sockets with asyncio libs ?

Using mocket with trio, I get :
TypeError: descriptor 'send' requires a '_socket.socket' object but received a 'MocketSocket'

Is there any known way for this to work ?

Please add more information, maybe a few line you are using for testing that case. I am not familiar with trio. From what I can infer, it seems they did not implement their module with duck typing in mind.

I think so, it's probably more following the modern python3 gradual typing mindset...
I am using https://github.com/theelous3/asks (asynchronous http client lib) to test it (I'm trying to mock sockets in there) and I made this quick script for checking:
import pytest
import json
import asks
from mocket import Mocketizer
from mocket.mockhttp import Entry

asks.init('trio')

@pytest.mark.trio
async def test_connection_httpbin():
    url_to_mock = 'http://httpbin.org/get'

    Entry.single_register(
        Entry.GET,
        url_to_mock,
        body=json.dumps({"args": {}}),
        headers={'content-type': 'application/json'}
    )

    with Mocketizer():
        session = asks.Session()
        resp = await session.get(url_to_mock,)

    assert "args" in resp.json() and resp["args"] == {}

if __name__ == '__main__':
    pytest.main(['-s', __file__])

It crashes, probably because :
sock = <socket.socket [closed] fd=-1, family=AddressFamily.AF_UNIX, type=SocketKind.SOCK_STREAM, proto=0>

    async def wait_socket_readable(sock):
        if type(sock) != _stdlib_socket.socket:
>           raise TypeError("need a socket")
E           TypeError: need a socket

../../../.local/share/virtualenvs/replies-YhL4Uusr/lib/python3.7/site-packages/trio/_core/__init__.py:44: TypeError

More investigation is needed here, but I guess my question becomes, couldn't 'MockerSocket' be a usual 'socket' to pass this kind of checks ?

Thanks for the hint. Actually, imho -of course-, there is no reason to check the type when dealing with socket objects, since there is one and only one implementation of that in the Python world, and the result is just breaking all the mocking libraries using the approach I used (httpretty to name another one, even more popular than mocket). Out of curiosity, I've just tried to run the tests with MocketSocketinheriting fromsocketinstead ofobject` and I had a failure, so this is not an easy fix like you expect.

Just to add an information, mocket is compliant with popular asyncio libraries ad aiohttp. You can verify it easily from the test suite, so the issue title is definitely wrong.

Oh thank I hadn't notice that, it's a good reference.
I'll have a look and compare trio and asyncio for this usecase...

Thank you very much for your contribution. I am not 100% sure I should do something to make MocketSocket inheriting from socket.socket. I see it as quite dangerous, complicating the debugging of possible issues due to the fact that MocketSocket acts as a socket following the duck typing principles. I would invest more time in advocating the removal of this check in those libraries.

Hey, main author of Trio here. I wrote some more notes here: https://github.com/python-trio/trio/issues/170#issuecomment-410907876
Trio does check that the sockets you pass in are socket.socket objects – this is to avoid things like allowing a ssl.socket, which is a subclass of socket.socket but with different and incompatible behavior (from the point of view of an async library like trio). But, this isn't actually a problem at all, because mocket also patches socket.socket to refer to its special object, so that test passes :-). AFAICT there are two issues:

Because of a strange quirk, trio tries calling the real socket.socket methods on the object that get passed in, and these get confused because they require a real-honest-to-goodness socket.socket object. (Hence the "descriptior 'send'" bit the original bug report.) There's more discussion of this in the thread linked above.

Since Trio is an async library, it needs to be able to pass sockets to low-level OS primitives like epoll_ctl and select. It looks like mocket is not set up to support this, so even if we fixed the descriptor issue it still wouldn't work? I see what looks like an attempt to support this in MocketSocket.fileno() calling os.pipe, but I don't see how it can work (maybe it assumes that no-one ever calls fileno more than once? also it obviously only supports Unix, since on Windows pipe and sockets are totally incompatible – socket.socketpair might work better).

However, trio does have explicit support for socket mocking at a slightly higher level: if you have a class that implements the trio socket API (which is basically the regular socket API except the blocking functions are marked as async), then you can tell trio to use that instead of regular sockets. I wonder if there would be any way to implement a class like this, while reusing mocket's existing code for handling http requests etc.?


Migrating Open Targets Google Analytics to GA4

Background
Currently, we are using Universal Google Analytics GA360; however, this version will stop tracking data on October 1, 2023 and will be replaced by Google Analytics 4 (GA4)
Tasks

[] We should migrate OT GA360, including all the current projects and functionalities, to GA4.

Current GA360 link here: https://analytics.google.com/analytics/web/?authuser=1#/report-home/a101860681w176880194p175737841
Acceptance tests
How do we know the task is complete?

When all our tracking data are migrated and functional on GA4

Link from Google Support with instructions on how to perform the migration:
Make the switch to Google Analytics 4

@carcruz , would you like to have a chat about this?

Successfully migrated so far to GA4 (through Google Tag Manager):

Genetics Portal;
Open Targets Platform

There is a blocker for migration of the following pages from UA to GA4:
Platform Documentation
Genetics Portal Documentation
Open Targats Community Page (Discourse)
Open Targets Blog (Ghost?)
For the above, I need to paste a piece of code into the source code + GitHub commit in order to "tag" them for migration.
I am however not familiar with the source code location for these pages in the Open Targets repository and need some help to identify them please @carcruz @HelenaCornu @d0choa.
Code to paste in the  section of the above web pages:

I have now successfully migrated our Google Analytics to Google Analytics GA4 for all our tagged/tracked websites.


Update actionResponses.js

This PR updates an old link to an old version of our community handbook. Thanks!
Changes

Screenshots
(prefer animated gif)
Checklist

[ ] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

Fixes #

@misspee007 @mide358 I'm not sure who is actively working on this project anymore but if you can merge this (or have the current maintainers do it) I'd be most grateful!  :pray: 💖

@misspee007 @mide358 I'm not sure who is actively working on this project anymore but if you can merge this (or have the current maintainers do it) I'd be most grateful! 🙏 💖

Hey Elizabeth! That would be me. I'll look into this right away.


fix: removed form-validation-error attribute on TextArea component

Description
Ref: #12348
Removed x-on:form-validation-error.window as it is not needed for autosize bug issue

Thanks, @emamalias!
Reference: https://github.com/filamentphp/filament/pull/12348#discussion_r1568402627.

What about x-intersect.once="render()"?

What about x-intersect.once="render()"?

Yep. I think it is safe to remove. I was hesitant to remove as it might help the performance especially for long forms?

I am returning the intersect. #8193


delete this (checking will be performed in the backend)

The puzzle 9-a09e895e from #9 has to be resolved:
https://github.com/ammaratef45/Attendance/blob/6458784661c60ff240bc92347d7306a070af4ac8/attendance/lib/homePage/home_page_viewmodel.dart#L101-L101
The puzzle was created by Ammar Atef on 16-Apr-19.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@ammaratef45/z please, pay attention to this issue

The puzzle 9-a09e895e has disappeared from the source code, that's why I closed this issue.

Job gh:ammaratef45/attendance#49 is not assigned, can't get performer

This job is not in scope


Issue Flashing new Version of Novostella LED Flood Light

I succesfully flashed such lights 2 weeks ago. This time, it doesnt work. The light stops blinking, but nothing changes. It might be the same issue as in https://github.com/ct-Open-Source/tuya-convert/issues/483
PSK shows the following:
ID: 0242416f68626d6436614739314946523173106c1b5b6c27c3886116567b35e847ffa4b2ab4ab64e1704d1f260d7b85a31
PSK: 165bbd09dc3281cb53b181846b026bc958b4283dee0dbdb43341b1fc155d34b5
could not establish sslpsk socket: [SSL: DECRYPTION_FAILED_OR_BAD_RECORD_MAC] decryption failed or bad$
I added all logs bellow. I dont know what to do next - help would be great!
pi@raspberrypi:~/tuya-convert $ ./start_flash.sh
tuya-convert v2.4.4
Checking for network interface wlan0... Found.
Checking UDP port 53... Available.
Checking UDP port 67... Available.
Checking TCP port 80... Available.
Checking TCP port 443... Available.
Checking UDP port 6666... Available.
Checking UDP port 6667... Available.
Checking TCP port 1883... Available.
Checking TCP port 8886... Available.
Starting AP in a screen
Starting web server in a screen
Starting Mosquitto in a screen
Starting PSK frontend in a screen
Starting Tuya Discovery in a screen
======================================================
IMPORTANT

Connect any other device (a smartphone or something) to the WIFI vtrust-flash
This step is IMPORTANT otherwise the smartconfig may not work!
Put your IoT device in autoconfig/smartconfig/pairing mode (LED will blink fast). This i            s usually done by pressing and holding the primary button of the device
Make sure nothing else is plugged into your IoT device while attempting to flash.
Press ENTER to continue

======================================================
Starting smart config pairing procedure
Waiting for the device to install the intermediate firmware
Put device in EZ config mode (blinking fast)
Sending SSID                  vtrust-flash
Sending wifiPassword
Sending token                 00000000
Sending secret                0101
................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
..................
SmartConfig complete.
Resending SmartConfig Packets
..................
SmartConfig complete.
Resending SmartConfig Packets
..................
SmartConfig complete.
Resending SmartConfig Packets
.............
Device did not appear with the intermediate firmware
Check the *.log files in the scripts folder
Do you want to try flashing another device? [y/N] n
Cleaning up...
Closing AP
Exiting...
%%%%%%%%%%%%%  smarthack-wifi.log
System info
94acc3b
Linux raspberrypi 4.19.118-v7+ #1311 SMP Mon Apr 27 14:21:24 BST 2020 armv7l GNU/Linux
OpenSSL 1.1.1d  10 Sep 2019
Dnsmasq version 2.80  Copyright (c) 2000-2018 Simon Kelley
Compile time options: IPv6 GNU-getopt DBus i18n IDN DHCP DHCPv6 no-Lua TFTP conntrack ipse$
This software comes with ABSOLUTELY NO WARRANTY.
Dnsmasq is free software, and you are welcome to redistribute it
under the terms of the GNU General Public License, version 2 or 3.
hostapd v2.8-devel
User space daemon for IEEE 802.11 AP management,
IEEE 802.1X/WPA/WPA2/EAP/RADIUS Authenticator
Copyright (c) 2002-2019, Jouni Malinen<EMAIL_ADDRESS>and contributors
Python 3.7.3
Attempting to stop wpa_supplicant
Configuring AP interface...
RTNETLINK answers: File exists
Starting DNSMASQ server...
Starting AP on wlan0...

Listening on <IP_ADDRESS>:80
^CReceived SIGINT, exiting...
Listening on <IP_ADDRESS>:80
^CReceived SIGINT, exiting...
^CListening on <IP_ADDRESS>:80
^CReceived SIGINT, exiting...
Listening on <IP_ADDRESS>:80
^CReceived SIGINT, exiting...
%%%%%%%%%%%%%  smarthack-web.mqtt
1594848935: mosquitto version 1.5.7 starting
1594848935: Using default config.
1594848935: Opening ipv4 listen socket on port 1883.
1594848935: Opening ipv6 listen socket on port 1883.
^C1594849064: mosquitto version 1.5.7 terminating
1594849292: mosquitto version 1.5.7 starting
1594849292: Using default config.
1594849292: Opening ipv4 listen socket on port 1883.
1594849292: Opening ipv6 listen socket on port 1883.
^C1594849711: mosquitto version 1.5.7 terminating
1594849743: mosquitto version 1.5.7 starting
1594849743: Using default config.
1594849743: Opening ipv4 listen socket on port 1883.
1594849743: Opening ipv6 listen socket on port 1883.
^C^C1594849753: mosquitto version 1.5.7 terminating
1594850644: mosquitto version 1.5.7 starting
1594850644: Using default config.
1594850644: Opening ipv4 listen socket on port 1883.
1594850644: Opening ipv6 listen socket on port 1883.
^C1594850769: mosquitto version 1.5.7 terminating
1594851053: mosquitto version 1.5.7 starting
1594851053: Using default config.
1594851053: Opening ipv4 listen socket on port 1883.
%%%%%%%%%%%%%  smarthack-web.udp
Listening for Tuya broadcast on UDP 6666
Listening for encrypted Tuya broadcast on UDP 6667
^CListening for Tuya broadcast on UDP 6666
Listening for encrypted Tuya broadcast on UDP 6667
^CListening for Tuya broadcast on UDP 6666
Listening for encrypted Tuya broadcast on UDP 6667
^C^CListening for Tuya broadcast on UDP 6666
Listening for encrypted Tuya broadcast on UDP 6667
^CListening for Tuya broadcast on UDP 6666
Listening for encrypted Tuya broadcast on UDP 6667
^C
%%%%%%%%%%%% nano scripts/smarthack-psk.log
new client on port 443 from <IP_ADDRESS>:57209
could not establish sslpsk socket: [SSL: NO_SHARED_CIPHER] no shared cipher (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:57210
could not establish sslpsk socket: [SSL: NO_SHARED_CIPHER] no shared cipher (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:57211
could not establish sslpsk socket: [SSL: WRONG_SSL_VERSION] wrong ssl version (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:57212
could not establish sslpsk socket: [SSL: NO_SHARED_CIPHER] no shared cipher (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:57213
could not establish sslpsk socket: [SSL: NO_SHARED_CIPHER] no shared cipher (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:57214
could not establish sslpsk socket: [SSL: WRONG_SSL_VERSION] wrong ssl version (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:57218
could not establish sslpsk socket: [SSL: NO_SHARED_CIPHER] no shared cipher (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:57219
could not establish sslpsk socket: [SSL: NO_SHARED_CIPHER] no shared cipher (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:57220
could not establish sslpsk socket: [SSL: WRONG_SSL_VERSION] wrong ssl version (_ssl.c:1056)
don't panic this is probably just your phone!
new client on port 443 from <IP_ADDRESS>:32635
ID: 0242416f68626d6436614739314946523173106c1b5b6c27c3886116567b35e847ffa4b2ab4ab64e1704d1f260d7b85a31
PSK: 165bbd09dc3281cb53b181846b026bc958b4283dee0dbdb43341b1fc155d34b5
could not establish sslpsk socket: [SSL: DECRYPTION_FAILED_OR_BAD_RECORD_MAC] decryption failed or bad$
new client on port 443 from <IP_ADDRESS>:30070
ID: 0242416f68626d6436614739314946523173106c1b5b6c27c3886116567b35e847ffa4b2ab4ab64e1704d1f260d7b85a31
PSK: 165bbd09dc3281cb53b181846b026bc958b4283dee0dbdb43341b1fc155d34b5
could not establish sslpsk socket: [SSL: DECRYPTION_FAILED_OR_BAD_RECORD_MAC] decryption failed or bad$
new client on port 443 from <IP_ADDRESS>:2563
ID: 0242416f68626d6436614739314946523173106c1b5b6c27c3886116567b35e847ffa4b2ab4ab64e1704d1f260d7b85a31

IP <IP_ADDRESS> is my phone, <IP_ADDRESS> is the light in question

Currently not possible to flash OTA. See #483

I love these lights. I bought 6 and flashed them successfully. Where'd you buy them from where the firmware was changed to not support OTA? I might buy some to just convert them for the future.

From Amazon DE

@kueblc Having the same issue. I've seen you suggest using Wireshark while configuring with API, do you have general instruction for this or any other advice for capturing the necessary data? Im an application developer by day and its been years since I've had to use Wireshark but happy to help in any way possible

@kueblc Having the same issue. I've seen you suggest using Wireshark while configuring with API, do you have general instruction for this or any other advice for capturing the necessary data? Im an application developer by day and its been years since I've had to use Wireshark but happy to help in any way possible

Hi @TATERxxWARDY, I appreciate your willingness to help. We currently have more than enough data to look at right now, unless you happen to have a device of the same model with older firmware.

@kueblc I do not, sorry


[Azure Analysis Services] Add firewall rules and fix autorest errors

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@taiwu,
Thanks for your contribution.  To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
The agreement will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

No modification for AutorestCI/azure-sdk-for-python

No modification for AutorestCI/azure-sdk-for-node

 https://github.com/Azure/azure-sdk-for-ruby/pull/909


Every app on Avalonia.iOS crashes with <IP_ADDRESS> Xamarin.iOS SDK

Describe the bug
Recently I have updated my VS to the 16.9 version. I ran an app that uses Avalonia.iOS. Each VS update updates an SDK on my macOS.This time it was updated to <IP_ADDRESS>. And sadly Avalonia doesn't work at the moment with this version.

To Reproduce
Just try to update your VS and run ControlCatalog
Desktop (please complete the following information):

OS: Win 10
Version 10.0

Was this on the simulator or a physical device?

Was this on the simulator or a physical device?

simulator

I would try referencing HarfBuzzSharp directly in your project

Any news on this one? I am on macos and not being able to use Avalonia.iOS. Tried running the samples in the source code - same error. Adding HarfBuzzSharp directly does not seem to solve the issue.

I get similar problem - trying to launch ContentGallery.iOS - targeting a simulator.
Using all latest atm - VS 2022 (17.0.5), Xam.iOS 14 and macOS on Monterey with XCode 13.2.1

I investigated what's wrong with it, and found workarounds:

It's not an issue if you use NET6-iOS SDK
It's not an issue if you use old Xamarin SDK, but with MSBuild.SDK.Extras (possibly also works with Xamarin.Legacy.Sdk)
With old style csproj projects it doesn't work.

If you have old-style project, workaround would be to upgrade it to either net6-SDK or use MSBuild.SDK.Extras/Xamarin.Legacy.Sdk.
Later approach should be less-impactful:

Set SDK on the  to MSBuild.SDK.Extras or Xamarin.Legacy.Sdk.
Remove all default  entries that reference xamarin SDK (it will be automatically referenced by SDK from above).
Remove all  entries from csproj, because they will be added automatically.
You can remove all transitive project/package dependencies now, if you have them with old csproj.

Samples can be found in Xamarin.Legacy.Sdk or MSBuildSdkExtras repos. Also this repro from old solved issue is yet another working sample (don't forget to change target from mac to ios).

In my opinion this issue should be closed, because legacy SDK problems is out of our control. Instead new issue should be created at xamarin-macios repository (if anybody can create minimal repro preferable without Avalonia usage, but with HarfBuzz).
Our ControlCatalog.iOS and XamlControlGallery projects will be updated with workaround from above.

Not an issue with net6 target which is now available


--rm option for kontena app build command

When building a service via kontena app build <service_name, it appears that kontena leaves the previous image on a successful build. This previous image is untagged, and in some (most?) cases, won't be used anymore. As a convenience, it would be nice to be able to specify an --rm flag or similar to the build command which would instruct kontena to remove the untagged/old image upon successful build.

closing as app commands are now fully deprecated and moved to separate plugin


Groovy syntax highlighting needs work

(Original issue 879 created by catalan42 on 2013-06-07T20:21:25.509905+00:00)
Hi - I found Pygments through creating a Gist at GitHub.  Unfortunately, the lexer for Groovy needs improvement.
Here is an example of the current Pygments output for a Groovy source file:  https://gist.github.com/catalan42/5730957   You can see that the shebang on the first line really thows things off.  The 2nd half of the file is better, but not nearly as good as it should be.
Here is an example of good syntax highlighting for Groovy as done by GVim:  https://plus.google.com/u/0/109111276108676937221/posts/AiFrtcHUUUZ
The Vim syntax file which defines the highlighting for Groovy can be found here:  http://www.vim.org/scripts/script.php?script_id=945
I was originally thinking I'd try to submit a pull request to update the Pygments lexer, but I must admit that I'm having a tough time understand the first thing about your system! I have found the documentation at http://pygments.org/docs/lexerdevelopment/  very difficult to understand.

(Original issue 879 last updated on 2016-06-01T07:03:50.050998+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by catalan42 on 2013-06-07T20:22:17.386747+00:00)
The Vim syntax file for Groovy source code

(Original comment by tshatch on 2016-06-01T07:02:04.186423+00:00)
The gist text now highlights correctly using Pygments ~2.2.


can't install from repo

I'm getting this error:
output/output.go:11:2<EMAIL_ADDRESS>(replaced by ./agent-plugins-grpc): reading agent-plugins-grpc/go.mod: open /go/src/github.com/deepfence/SecretScanner/agent-plugins-grpc/go.mod: no such file or directory

@dcu Can you add the commands you ran?

I tried both, go get and go install

@dcu We recommend you to use docker to build SecretScanner and deploy as per documentation:
./bootstrap.sh
docker build --rm=true --tag=deepfenceio/deepfence_secret_scanner:latest -f Dockerfile .


Add Sheringham Sixth Form

http://sheringhamsixthform.co.uk/

@georgePadolsey Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!

@georgePadolsey Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!