Link Mapbox API to index.html file

WE NEED the Mapbox API linked to the index.html file SO THAT users can input their location and find the closest movie theatre.

Kyle E now working on.


chore: add new vulnerability aliases to test snapshots

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

Project coverage is 66.16%. Comparing base (62a848e) to head (e79c7e2).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1192      +/-   ##
==========================================
+ Coverage   65.65%   66.16%   +0.50%
==========================================
  Files         165      165
  Lines       13983    10835    -3148
==========================================
- Hits         9181     7169    -2012
+ Misses       4299     3160    -1139
- Partials      503      506       +3

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


MethodArgumentSpaceFixer - new test case

closes: #3067

@nullref could you please confirm the fix ?

I tested it with my minimal test case and the original code I generated it from, both were processed without generating a syntax error.

thanks @keradus :+1:


Document how to test Notifiers

With StateNotifier, testing was pretty easy - I would create an instance (including with some 'seeded' state for testing), do some stuff with it, grab its stream, and validate the events using the standard tool like https://pub.dev/documentation/test/latest/test/emitsDone.html and so on.
I have recently migrated to Notifier, not without effort, and sadly now my tests are broken as I cannot get hold of the stream. I read https://riverpod.dev/docs/essentials/testing and it is very vague about how to actually test stuff, e.g. it doesn't show how to validate multiple state changes. I think I need to do container.listen(myProvider) and collect the states, and compare them afterwards (EDIT: The page actually does mention this approach, sorry for this). Are there any other options?
For comparison:
With StateNotifier:

Create an instance of the notifier.
Invoke methods.
Verify the state using .stream and standard tools for Stream verifications.

With Notifier:

Create an instance of the notifier. If you want to seed the notifier, things get interesting as the state is created using build - I need to add code to build to use my seeded values in tests, or create a 'standard' instance.
Create a ProviderContainer and pass an override of the provider for this notifier to use the seeded one. Remember to dispose (by hand or automatically).
Create a list to collect states.
container.listen(myProvider, callbackAddingStatesToList).
Invoke methods.
Verify the list of states.

I don't want to sound like a grandpa, but IMHO 'it used to be better'. It has all become much more complex, feels like a step backwards. Notice how the list of steps to perform is now twice as long. Are there any plans on making that more streamlined, like a helper that does all of this (I did write one for my project, and possibly more people wrote theirs as well). I'm not sure what you think of the Bloc package, but Felix also provides bloc_test which makes it very straightforward to test Blocs. Or have I simply missed it?
Also, before, it was possible to test the notifier alone, without the container and overrides and providers - I could focus on the logic of the things that has the logic. Now I always need the container.

With regards to using .stream and stream test matchers, I've never recommended using those.
IMO they cause tests to be brittle because those matchers are not precise enough.
The way I do it is use Mockito or something similar, and use its verify/verifyInOrder. That's also what the documentation suggests.
Combined with the createContainer utility I share in the same docs.
So you end-up with:
test('', () async {
  final container = createContainer();
  final listener = ListenerMock<MyState>();

  container.listen(provider, listener.call);

  container.read(provider.notifier).method();

  verify(listener(any(), value));
});

ListenerMock depends on your mock package, but for mockito it is:
class ListenableMock<T> extends Mock {
  void call(T? value, T next);
}

You could even share the first few lines between tests if you wanted to. I don't like the idea of using setup/tearDown to share mocks & stuff, but you could.

Ultimately whatever approach you use, you can create utilities to make it more reusable for your case.
For example, nothing prevents you from making an util that uses a ProviderContainer + ref.listen and create a Stream out of it for testing purposes.

Thanks.
I have never had issues with the test stream matchers, to be honest, but I do acknowledge code that does wild things with streams - mine never does - could expose issues.
I did something similar (an example for one notifier, my helper is a bit more complex as it is 'generic'):
    final notifier = MyNotifier(optionalSeedValue);

    final container = ProviderContainer(
      overrides: [
        myNotifierProvider.overrideWith(() => notifier),
      ],
    );
    addTearDown(container.dispose);

    final states = <MyState>[];
    container.listen(
      myNotifierProvider,
      (previous, next) => states.add(next),
    );

    notifier
      ..doStuff()
      ..doMoreStuff();

    expect(
      states,
      expectedStates,
    );

But that's the point - how many people will write something like this? Is it even correct? I don't know, e.g. I'm not getting the notifier form the container, I'm using the instance that I created directly.
I believe creating a package with such helpers would help a lot - there would be one way to write tests. Again, bloc_test is a very nice example. Or at least document it more, because right now the docs kind of mention some of this stuff, omit some other relevant parts. Ideally there would be more code, like a full example. Your snippets from this thread are helpful and I believe they are not included in the docs.

So the question is: will there ever be a package? I guess this issue could be copied as feature request.
Please close this if you believe the docs are good enough, and there will never be a package with test helpers.

There won't be a separate package.
But the main package may receive more utilities as time goes.
It's more a matter of what utility we'd want.
One thing for certain is that notifiers will have a custom "override" that enables overriding their "build" directly instead of the whole object.
I'll also likely offer a unique constructor on ProviderContainer that works only in tests and autoDispose the container.

Although à syntax like bloc_test is achievable, I won't do it personally.
IMO the reliance on stream matchers and defining variables out of the test promotes bad practices.
If there are specific painpoints or if you have unique suggestions though, I'm happy to look into it

Well, bloc_test doesn't rely on the stream marchers you don't like (last time I checked, it's been a while).
The helpers you mention make sense, can't wait to get them. In the meantime, I still need to migrate to code generation ;)

There won't be a separate package. But the main package may receive more utilities as time goes.
It's more a matter of what utility we'd want.
One thing for certain is that notifiers will have a custom "override" that enables overriding their "build" directly instead of the whole object.
I'll also likely offer a unique constructor on ProviderContainer that works only in tests and autoDispose the container.

Hi, is there any progress of common utilities for tests?
I have custom utilities or use riverpod_test, but ever is better to use official utilities :).


⚠ Bofrost.es has degraded performance

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

HTTP code: 200
Response time: 1530 ms

Resolved: Bofrost.es performance has improved in a2212fc after 6 minutes.


Investigating definition of Unique works pages viewed: metric and time series

What
This Kibana visualisations are inflated by picking up non-works pages
Where
Definition used is
(page.name:work OR page.name:images) AND page.query.id: *, cardinality by page.query.id

Issue opened in error.


copy  Chinese text from  view  got garbled string

change font not help

Thanks for reporting, I have no idea how to debug this, I cannot read Chinese, could you provide a saved logfile and the expected outcome?

just use c#  Trace.WriteLine("some 中文"),
in the view ,  It looks ok,  export  log to file  is also ok,  but  copy to clipboard will became garbled

same garbled in  statusbar:  searching: ...

thanks ! I have tried to reproduce, but it seem to behave diffenent:

Wow, nice work, I'll process this!

you propose to change the use of MultiByteToWideChar  to std:: mbstowcs, and to copy data to the clipboard using the CF_UNICODETEXT flag, both changes sound good. However, I still cannot reproduce the issue.
If you run this program, do the characters show up in debugview++ ? (I see this:

Test program:
showchinesechars.zip

I'm trying very hard to make sense of this, in your example, you used
std::mbstowcs preceeded by setlocale(LC_ALL, "");
This does not make sense to me, I think std::mbstowcs is not affected by setlocale, I think it should be
std::setlocale from <clocale>

Please try how this works:
https://github.com/CobaltFusion/DebugViewPP/releases/tag/v<IP_ADDRESS>
I have tried to simulate the conversions in a test, but I still don't actually 'see' the behavior in debugview++ :) You did uncover a real problem in
basic_stringbuilder& operator<<(const std::wstring& str)

It was very much broken and could crash (now fixed)

I tried to set my windows locale to Chinese, aside from being really difficult to understand my system dialogs, it did not seem to effect debugview

I'm still getting ?? and ???? instead of proper characters.

make sure set windows locale to  chinese,   for windows 10 english version:     Setting:  =>"Time & Language” => "Region & language – Additional date, time & regional settings”=>"Change date, time or number formats" => "Administrative - Change system locale…",  choose:  “Chinese（Simplified, China）",   restart

Did the new version I put up work any better?

I just test  v<IP_ADDRESS> - version,   it act same as  the old version

That is very sad, it spend several hours on that :)
On the bright side:
Changing this, as you suggested, worked:

reopened, to get confirmation that is it actually working correctly now.

yes,  I test it copy text ok,  except select and highlight text length

but status text show:  searching : ....   is still  garbled


Fixed #9796: Initialization of not existing yii\grid\ActionColumn default buttons

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#9796

Also refactored initDefaultButtons method mostly because of copy paste. Otherwise the amount of copy pasted code would be even more after this fix.
Added tests for that and test to ensure that we get correct html in case of using default template.
And fixed code style trifles - last line had extra whitespace after closing bracket and linebreak at the very end of the file was missing.

Added. I also improved testInit() of ActionColumnTest a bit to check array keys instead of just count.

Thank you!

@samdark and @softark , Russian and Japanese translations are broken after this change: https://github.com/yiisoft/yii2/commit/aa6d34a2858b32777b97ae1bf089b20d0f0bcca7

https://github.com/yiisoft/yii2/pull/13497
Reverted the accidental deletion.

see also #13489


Release 0.9.13-incubating.

This change marks 0.9.13-incubating as released, and updates the top-level symbolic links to the latest documentation.
We'll still need to hold off on actually deploying these changes until 2017-08-01 00:01 -0700, as that will be 24 hours since the release artifacts were deployed (and the mirrors need roughly 24 hours to sync). From http://www.apache.org/dev/release.html#release-announcements:

Please ensure that you wait at least 24 hours after uploading a new release before updating the project download page and sending the announcement email(s).  (For time-critical security releases, the download pages script supports bypassing this requirement.)

Also, from http://www.apache.org/dev/release-publishing.html#sync-delay:

Apache uses svnpubsub internally and rsync mirrroring externally. Files committed to the Subversion repository at https://dist.apache.org/repos/dist/ are automatically copied, using svnpubsub, to www.apache.org , and then the external mirrors pick up the files from www.apache.org. It may take up to 24 hours or more for a newly published release to be sync'd to all mirrors. Mirrors have their own schedules. Mirrors are required to check at least once a day, but most will check for updates 2 to 4 times per day.

Looks fine to me.
:

Having never done a website update before...does the merge of this PR need to wait?  That is, are merging this PR and deploying the new web site the same thing, or are they separate steps?

They're separate. To publish the changes, the static site is built using ./build.sh stage and committed to the special detached asf-site branch:
https://github.com/apache/incubator-guacamole-website/blob/master/README.md#publishing-changes
The push to asf-site is normally done by the person who opened the original pull request. Once this is merged to master, I'll take care of the push to asf-site (and the rest of the release announcement steps) once the mirrors have had their 24 hours:
http://guacamole.incubator.apache.org/release-procedures-part4/#update-website
It's safe to merge to master - the important thing is that we refrain from publishing the website via the asf-site branch until the mirrors are ready.

Got it.  Merged to master - will leave website updates to you.


Flutter Null Safety

I have noticed that your repository is awesome and very helpful to others. As you know Flutter has published with null safety. But this repository does not support this feature. Can you please develop this feature so that we can use this more preciously. Thank you for your effort.

+1

I would love it if null-safety supported. I have been using this tool every day

Duplicate of #1673


[Snyk] Security upgrade php from 8.2.10-apache to 8.2.14RC1-apache

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

Dockerfile

We recommend upgrading to php:8.2.14RC1-apache, as this image has only 75 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

721
Out-of-bounds Write SNYK-DEBIAN12-GLIBC-5927132
Proof of Concept

721
Out-of-bounds Write SNYK-DEBIAN12-GLIBC-5927132
Proof of Concept

721
Out-of-bounds Write SNYK-DEBIAN12-GLIBC-5927132
Proof of Concept

721
Out-of-bounds Write SNYK-DEBIAN12-GLIBC-5927132
Proof of Concept

829
Resource Exhaustion SNYK-DEBIAN12-NGHTTP2-5953379
Mature

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Resource Exhaustion

Superfluous now as #12 was recently merged.


DTL response null result with 404 http status DEMO code

Goals of PR
Core changes:

handler support return http status code
return 404 http status code when result is null

Related Issues:

with this we can enable cache safely

Test
 curl "localhost:7878/transaction/index/87?backend=l2" -i

HTTP/1.1 404 Not Found
X-Powered-By: Express
Access-Control-Allow-Origin: *
Content-Type: application/json; charset=utf-8
Content-Length: 33
ETag: W/"21-CMqNFJ1GmGJsYiAIPHNf2iBKJ9E"
Date: Fri, 22 Dec 2023 12:51:18 GMT
Connection: keep-alive
Keep-Alive: timeout=5

{"transaction":null,"batch":null}

➜  curl "localhost:7878/transaction/index/86?backend=l2" -i

HTTP/1.1 200 OK
X-Powered-By: Express
Access-Control-Allow-Origin: *
Content-Type: application/json; charset=utf-8
Content-Length: 1054
ETag: W/"41e-LAO9Ef7iD5peiLs9WMbN9FjpsHw"
Date: Fri, 22 Dec 2023 12:51:24 GMT
Connection: keep-alive
Keep-Alive: timeout=5

{"transaction":{"index":86,"value":"0x0","batchIndex":null,"blockNumber":16005,"timestamp":1703089712,"queueOrigin":"sequencer","confirmed":false,"gasLimit":"0","target":"0x0000000000000000000000000000000000000000","origin":null,"data":"0xf8a8808402faf08082864294deaddeaddeaddeaddeaddeaddeaddeaddead111180b844a9059cbb00000000000000000000000000000500e87ee83a1bfa233512af25a4003836c800000000000000000000000000000000000000000000000000005af3107a400045a045f220e8638eb5f5cdb2259efbdd36533597eae1776d9fdc81d2914d59f60203a03e5202f7743b809bebbe8b8a181ae4f55a472f6b9b5de1ff695c29a102c0037e","decoded":{"sig":{"v":0,"r":"0x45f220e8638eb5f5cdb2259efbdd36533597eae1776d9fdc81d2914d59f60203","s":"0x3e5202f7743b809bebbe8b8a181ae4f55a472f6b9b5de1ff695c29a102c0037e"},"value":"0x0","gasLimit":"34370","gasPrice":"50000000","nonce":"0","target":"0xdeaddeaddeaddeaddeaddeaddeaddeaddead1111","data":"0xa9059cbb00000000000000000000000000000500e87ee83a1bfa233512af25a4003836c800000000000000000000000000000000000000000000000000005af3107a4000"},"queueIndex":null},"batch":null}


Remove trivial setters and getters

Came across this https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#Rh-get
Couldn't not change it!
 Totally agree with the idea of removing them.


Support option to prune empty directories during recursive delete

I much prefer the CLI tool over using straight REST requests, on account of the support for wildcard paths. When I do something like this
jfrog rt del --recursive=true repo1/*/path1/
My intent would be to have the same result as having run
curl -X DEL http://art.example.com/artifactory/repo1/.../path1
for all existing ... directories. Instead all the repo1/*/path1/ folder structures are left in place, with only the files having been deleted. I think a --prune option would be good to combine with --recursive=true

@jgeorgeson,
We started working on an improvement to the delete code that should provide the outcome you're looking for. We hope to add this to the next release.

@jgeorgeson,
We have released a new Jfrog cli version (1.5.1) which supports your request.
It is enabled by default when deleting folder path with wildcard.
Thank you for your suggestion.

Thanks! It seems to be working but the output is a little confusing.
[jgeorgeson@woodhouse: /d01/sandboxes/jgeorgeson/workspaces/jfrog-cli]
$ jfrog rt s "test-deploy-snapshots/test/base1-sub3/*"
[Info:] Searching Artifactory using AQL query: items.find({"repo": "test-deploy-snapshots","$or": [{"$and": [{"path": {"$match":"test/base1-sub3"},"name":{"$match":"*"}}]},{"$and": [{"path": {"$match":"test/base1-sub3/*"},"name":{"$match":"*"}}]}]}).include("name","repo","path","actual_md5","actual_sha1","size")
[Info:] Artifactory response: 200 OK
[Info:] Found 3 artifacts.
[
  {
    "path": "test-deploy-snapshots/test/base1-sub3/index.xml"
  },
  {
    "path": "test-deploy-snapshots/test/base1-sub3/plugins/plugin1.jar"
  },
  {
    "path": "test-deploy-snapshots/test/base1-sub3/plugins/plugin2.jar"
  }
]
[jgeorgeson@woodhouse: /d01/sandboxes/jgeorgeson/workspaces/jfrog-cli]
$ jfrog rt delete --dry-run "test-deploy-snapshots/test/base1-sub3/"
  test-deploy-snapshots/test/base1-sub3/
Are you sure you want to delete the above paths? (y/n): y
[Info:] [Dry run] Deleting: http://art.example.com/artifactory/test-deploy-snapshots/test/base1-sub3/
[jgeorgeson@woodhouse: /d01/sandboxes/jgeorgeson/workspaces/jfrog-cli]
$ jfrog rt delete "test-deploy-snapshots/test/base1-sub3/"
  test-deploy-snapshots/test/base1-sub3/
Are you sure you want to delete the above paths? (y/n): y
[Info:] Deleting: http://art.example.com/artifactory/test-deploy-snapshots/test/base1-sub3/
[Info:] Artifactory response: 204 No Content
[jgeorgeson@woodhouse: /d01/sandboxes/jgeorgeson/workspaces/jfrog-cli]
$ jfrog rt s "test-deploy-snapshots/test/base1-sub3/*"
[Info:] Searching Artifactory using AQL query: items.find({"repo": "test-deploy-snapshots","$or": [{"$and": [{"path": {"$match":"test/base1-sub3"},"name":{"$match":"*"}}]},{"$and": [{"path": {"$match":"test/base1-sub3/*"},"name":{"$match":"*"}}]}]}).include("name","repo","path","actual_md5","actual_sha1","size")
[Info:] Artifactory response: 200 OK
[Info:] Found 0 artifacts.
[]

Looking in the tree browser I can see the whole folder was deleted as desired, but the delete command output made me think nothing had been found (and thus not deleted).

I guess can close issue if you want since the functionality works.

This is still not working for me on 1.5.1:
jfrog rt del --recursive=true  'sandbox/*lookout-server-*' returns 200 and says "Found 0 artifacts".
jfrog rt s --recursive=true  'sandbox/*lookout-server-*'
finds nothing.
But in the GUI I can still see lots of empty folders looking like sandbox/com/lookout/services/lookout-server-audit/1.0.DEV
What gives?

@dimatkach, the reason the empty directories were not removed is because in your scenario, no files had been found and therefore, nothing (including empty directories) was deleted. We would like to further improve the delete command, so that it handles this case as well.

@dimatkach, we enhanced the CLI to support deleting empty folders.
This will be available with our next release of the CLI.

@dimatkach, we enhanced the CLI to support deleting empty folders.
This will be available with our next release of the CLI.

Thanks for c224bfd. Helped me to clean up our Artifactory OSS.

@yagee-de and @jgeorgeson, version 1.7.0 is released and includes this functionality as part of the delete command. I'm therefore closing this issue.

@eyalbe4 I'm using the del command but unfortunately it doesn't clean the folder.  This is the AQL that I use:
{
  "files": [
    {
      "aql": {
        "items.find": {
          "path": { "$match": "${repoPath}" }
        }
      }
    }
  ]
}


License

Thank you for the clear work and easy-to-use code! Would you be able to add a license to the GitHub repo? I would like to use the code defining the TP-Transformer architecture (as long as the license permits), but I don't think I can use it unless the repo has a license that permits such usage.
(Directions for adding a license here: https://docs.github.com/en/github/building-a-strong-community/adding-a-license-to-a-repository)

Hi Tom, certainly. I think the MIT license will do.

Yes, that's perfect - thank you!


fix: support node:test in node 18

closes https://github.com/esbuild-kit/esm-loader/issues/32

Looking forward to this landing, thanks! Can I ask why it ever makes sense to strip off a node:?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Status refactor 5: Change update constructor

Change the update-constructor to take a subject and source.
To be able to document everything correctly, I had to:

add @enum to Member.Status,
Add an interface for member (IMember)

Only 1 minor question about the defensive programming in the Update constructor, but overall this PR LGTM.

well, yay! 👍


LTR direction for indicator view and PinLockView

Hello,
I've just made a couple of changes after I noticed that in my device the PinLockView and IndicatorDots views are shown in RTL because of my native device's language is Hebrew (which is an RTL language), and so the sample app looked like this:  (Don't be worry about the big numeric text sizes and everything in the screenshots, I just changed it in my sample app's view attributes)

And after my submitted changes:

Of course I also checked the results with an LTR based language (English) and it looked the same as in RTL based language (Hebrew).
Thank you for a great library!

Hello Aritra,
Do you have any schedule to when this PR can be merged?
Cheers.

Sorry for getting late into it. I have merged your PR. Will soon update the in jCenter.


Add a _changeBlock attribute on all entities

Graph Node tracks the block range over which a given entity is active. This effectively captures the block when an entity was last changed. This could be surfaced in the graphQL API as an automatically generated field _changeBlock. This field would provide helpful metadata, as well as being useful for filtering or sorting (using the logic applied to other fields).
e.g.
tokens(orderBy: _changeBlock, orderDirection: desc, where: { _changeBlock_gt: 99999 } ) {
    id
    _changeBlock
}

This would also help with polling for new entities & other requested use-cases.

Closely related: https://github.com/graphprotocol/graph-node/issues/295 (confirmations API). The confirmations API might be overkill but it's worth considering use cases before we add an API change that we then need to maintain forever.

Yes there are definitely overlaps, though I think the confirmations API requires quite a lot more functionality (field level details, details on the changes, finalization) which is beyond the scope of this issue.
You do raise a good point on an API change that we then need to maintain forever. I think there is a question whether we want to maintain the _meta pattern that exists at the top level of the subgraph, where _changeBlock is part of _meta (and which leaves more scope for potential extension in the future).

Does this a field a consumer might need as data, or metadata? (needed for app development, or while querying/testing the subgraph API?)
Because I think we can consider creating a spec for thegraph-introspection that exposes those? Just like GraphQL has it's own introspection that starts with __, we can introduce an internal introspection that can be part of every type / schem, and have it's own lifecycle? (so we can decide if we break/maintain it or not, and make a statement that it's an internal API that is subject to changes?)

Interesting - if that is an established pattern that could work, and as you say it has the benefit of flexibility. To answer the first question, this would be useful for app development, for example when polling for new data, or finding recently changed entities - so that does make it a bit more difficult to break anything, once introduced

OK. Revisiting this, I am tempted to opt for the simpler approach described in this issue: https://github.com/graphprotocol/graph-node/issues/1838
This adds this functionality purely as a new filter in the top-level where, rather than as an extension to the entity itself. This means that it is not available to Select, or Orderby.

Closing for now as this was implemented as a filter, we can establish whether adding this information to the entity would be useful


Support name the file with path

if the file is under a dir, support name the file with the full path you specified.

Fixed


Make HTTP server optional

Before, I never really had a use for daemon mode and always "manually" looped cross-seed. Now, with v4, I finally have a reason to use this mode by activating the RSS and search feature.
However, the daemon always starts a HTTP server. In my environment this is not a problem, I simply don't expose the port, but I suspect there are other situations I can't even come up with, in which this isn't a good thing. So, if a user simply wants to use RSS and periodic search in daemon mode, they currently have to have this HTTP server running.
I propose, now that daemon mode has more than just one feature, to make this HTTP server optional.
Thanks for v4 though. Love the new features :D

v4.2.0


Modified truncate_normal with jax.random.truncated_normal

Previously the truncate_normal distribution was sampled from a normal distribution and then clipping the result. This does not yield a continuous distribution. A more reasonable distribution should be like: https://en.wikipedia.org/wiki/Truncated_normal_distribution
This should affect the microphysics simulation. Maybe the previous version is intentional, so I will let experts comment on which behavior is more correct.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

12 of 14 (85.71%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 84.391%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

appletree/randgen.py
12
14
85.71%

Files with Coverage Reduction
New Missed Lines
%

appletree/randgen.py
3
71.03%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
2368

Relevant Lines:
2806

💛  - Coveralls

https://github.com/google/jax/issues/10951
When using jax.random.truncated_normal, we should be careful not to set vmin/vmax too extreme.
But I still think this should be better than using the previous clipping method. The comparison when simulating a truncated normal distribution with mean=0, std=1, vmin=-2, vmax=2 is shown below.

We should revert this PR because NEST also uses the previous truncate_normal in quanta generation. Here NEST defines two types of truncate_normal, one is the non-continuous version (termed rand_gauss with zero_min == true), and the other is the continuous version defined by this PR (termed rand_zero_trunc_gauss). In the quanta generation part, NEST uses rand_gauss so the previous version is more correct.


Current questionnaire is always marked as 'obsolete'

Describe the bug
Every time a 'user update' is triggered when there is already a questionnaire present, the user is informed that this one is obsolete, even though the new one is exactly the same as the current one.
To Reproduce
Steps to reproduce the behavior:

Open app
When questionnaire is available: repeatedly hit the 'refresh' button
Each time user is informed, that the current questionnaire would be 'obsolete'

Expected behavior
An update should only happen, when the currently persisted questionnaire definitely differs from the one to be fetched from the backend.
Screenshots
-/-
Smartphone (please complete the following information):
any device
Additional context
-/-

@GolexPrime @mahvaezi Any thoughts on this?

I think your proposal to just assign the ID from the backend seems likes the best and simplest solution.
Using any kind of constructed identifier needs to be coordinated with the adjacent projects - I think that might be an unnecessary effort right now 😅

This was addressed (I hope) with #79


Don't ignore last block when generating the signature

When generating signatures, if the last remaining block is shorter than blockLen io.ReadFull will return io.ErrUnexpectedEOF and the block will not be processed (signature will be one block too short).
This PR includes the cherry-picked commit originally contributed in https://github.com/balena-os/librsync-go/pull/3 and also a few improvements (namely, enable test cases that will now pass and avoid checking for io.ErrUnexpectedEOF which will not happen anymore after the first commit).

@balena-ci rebase

@balena-ci rebase

@balena-ci rebase


Add /sessioninfo endpoint

Used by browser clients to get a user's scope/email/etc when the cookie is marked HTTP-only

Coverage remained the same at 100.0% when pulling 035359bbb584b48eb1f73816760de2c9929987c3 on ypomortsev:ypomortsev-sessioninfo into e98bbcde92225ae56445edcc599e9097fbc2547c on 30x:master.

Coverage remained the same at 100.0% when pulling 42708cc51c2df545f42efa9a26a0432e95bf2d74 on ypomortsev:ypomortsev-sessioninfo into e98bbcde92225ae56445edcc599e9097fbc2547c on 30x:master.


Use UsernamePasswordCredential in Container

pacakge: azure-identity==1.4.0
Identity E2E test in containers
Live test credentials of azure-identity in container.
Is it possible to avoid "multi-factor authentication" errors when using UsernamePasswordCredential in a container, and how?
Or is it not expected to work in a container?
Thank you.
@jongio

The issue is the AAD tenant's configuration, not the container environment. As the documentation states, this credential won't work with tenants requiring MFA. You can stand up your own tenant, add a user, and test the credential against it, which is what we've done for the test suite.


xmlschema.exceptions.XMLSchemaParseError: duplicated identity constraint

Hello
I try to use this schema definition : http://www.eda.org/xmlschema/spirit/1685-2009
but it failed with rising an exception:
    raise error
xmlschema.exceptions.XMLSchemaParseError: duplicated identity constraint u'{http://www.spiritconsortium.org/XMLSchema/SPIRIT/1685-2009}registerFileRegisterFieldKey':

  <xs:key xmlns:xs="http://www.w3.org/2001/XMLSchema" name="registerFileRegisterFieldKey">
                    <xs:selector xpath="spirit:field" />
                    <xs:field xpath="spirit:name" />
  </xs:key>

Look like there is several node that lead to record what you call "constraint".
My case:
the schema 1685-2009.zip
import xmlschema
my_schema = xmlschema.XMLSchema('1685-2009/component.xsd')

Thomas

More interesting when the archive contain files: 1685-2009.zip

The word "constraint" refers to XML Schema Identity constraints, a feature added in the version 0.9.12.
In the version 0.9.13 the error you reported is fixed but another error persists:
  File "/home/brunato/.virtualenvs/xmlschema3/lib/python3.6/site-packages/xmlschema/components/component.py", line 106, in _parse_error
    raise error
xmlschema.exceptions.XMLSchemaParseError: empty context fields for <XsdKeyref '{http://www.spiritconsortium.org/XMLSchema/SPIRIT/1685-2009}componentChoiceRef' at 0x7f890fbd4710>:

  <xs:keyref xmlns:xs="http://www.w3.org/2001/XMLSchema" name="componentChoiceRef" refer="spirit:componentChoiceKey">
            <xs:selector xpath=".//*" />
            <xs:field xpath="@spirit:choiceRef" />
  </xs:keyref>

A partial rewrite of identity constraint's code is needed (set_context method) because descendant paths (//) doesn't work well with schema components's structure. Maybe it will be done for the next version.
Meanwhile you could use the schema using the lax validation mode:
>>> my_schema = xmlschema.XMLSchema('1685-2009/component.xsd', validation='lax')
>>> my_schema.all_errors
[XMLSchemaParseError("empty context fields for <XsdKeyref '{http://www.spiritconsortium.org/XMLSchema/SPIRIT/1685-2009}componentChoiceRef' at 0x7f890f850b70>:", <XsdKeyref '{http://www.spiritconsortium.org/XMLSchema/SPIRIT/1685-2009}componentChoiceRef' at 0x7f890f850b70>, <Element '{http://www.w3.org/2001/XMLSchema}keyref' at 0x7f890fa5ce58>)]

This bug persists in the version 0.9.14, published today.
Still postponed to a future version.

Should be fixed in version 0.9.17 (tested with an XML instance of the 1685-2009 schema).


[feat] Align implementations of the PBS and the Torque scheduler regarding the use of qstat

Closes #1563
Closes #1473

Codecov Report

Merging #1612 (330c7b4) into master (8b6a23f) will increase coverage by 0.16%.
The diff coverage is 5.45%.

@@            Coverage Diff             @@
##           master    #1612      +/-   ##
==========================================
+ Coverage   87.64%   87.81%   +0.16%
==========================================
  Files          45       45
  Lines        7255     7239      -16
==========================================
- Hits         6359     6357       -2
+ Misses        896      882      -14

Impacted Files
Coverage Δ

reframe/core/schedulers/torque.py
100.00% <ø> (+80.59%)
:arrow_up:

reframe/core/schedulers/pbs.py
46.85% <5.45%> (-17.86%)
:arrow_down:

Continue to review full report at Codecov.

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


add links to API documentation + Update message Update Service Catalog

Description

Added the links to the API documentations
Adjusted the text in the modal to update the catalog
added tooltips where needed.
removed unused method from the URLManager.

closes #4419 and #5122
https://github.com/inmanta/web-console/assets/44098050/39ea9747-2344-49dc-9965-1f70adb53ef2

Processing this pull request

Merged into branches master, iso6 in 0496e9edb208577bcf65124f2ef65f99144c3bbe


Add build test and CI

@astagi
Please enable TravisCI for the project and add the badge.

@ponty done! but I got this error on travis
Using worker: worker-linux-7a054b10-2.bb.travis-ci.org:travis-linux-9
system_info
Build system information
Build language: c
Build image provisioning date and time
Wed Feb  4 18:22:50 UTC 2015
Operating System Details
Distributor ID:	Ubuntu
Description:	Ubuntu 12.04 LTS
Release:	12.04
Codename:	precise
Linux Version
2.6.32-042stab090.5
Cookbooks Version
23bb455 https://github.com/travis-ci/travis-cookbooks/tree/23bb455
GCC version
gcc (Ubuntu/Linaro 4.6.3-1ubuntu5) 4.6.3
Copyright (C) 2011 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
LLVM version
clang version 3.4 (tags/RELEASE_34/final)
Target: x86_64-unknown-linux-gnu
Thread model: posix
Pre-installed Ruby versions
ruby-1.9.3-p551
Pre-installed Node.js versions
v0.10.36
Pre-installed Go versions
1.4.1
Redis version
redis-server 2.8.19
riak version
2.0.2
MongoDB version
MongoDB 2.4.12
CouchDB version
couchdb 1.6.1
Neo4j version
1.9.4
Cassandra version
2.0.9
RabbitMQ Version
3.4.3
ElasticSearch version
1.4.0
Installed Sphinx versions
2.0.10
2.1.9
2.2.6
Default Sphinx version
2.2.6
Installed Firefox version
firefox 31.0esr
PhantomJS version
1.9.8
ant -version
Apache Ant(TM) version 1.8.2 compiled on December 3 2011
mvn -version
Apache Maven 3.2.5 (12a6b3acb947671f09b81f49094c53f426d8cea1; 2014-12-14T17:29:23+00:00)
Maven home: /usr/local/maven
Java version: 1.7.0_76, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-7-oracle/jre
Default locale: en, platform encoding: UTF-8
OS name: "linux", version: "2.6.32-042stab090.5", arch: "amd64", family: "unix"
git.checkout
0.07s$ git clone --depth=50 --branch=master git://github.com/nanpy/nanpy-firmware.git nanpy/nanpy-firmware
Cloning into 'nanpy/nanpy-firmware'...
remote: Counting objects: 221, done.
remote: Compressing objects: 100% (119/119), done.
remote: Total 221 (delta 89), reused 209 (delta 85)
Receiving objects: 100% (221/221), 100.12 KiB | 0 bytes/s, done.
Resolving deltas: 100% (89/89), done.
Checking connectivity... done.
$ cd nanpy/nanpy-firmware
$ git checkout -qf 781f4701e124bed78ea0bcd8cdfdeb6403b75eaf
$ export CC=gcc
$ gcc --version
gcc (Ubuntu/Linaro 4.6.3-1ubuntu5) 4.6.3
Copyright (C) 2011 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
before_install.1
23.75s$ sudo apt-get update -qq
before_install.2
11.45s$ sudo apt-get install -y binutils-avr gcc-avr scons arduino
Reading package lists... Done
Building dependency tree
Reading state information... Done
scons is already the newest version.
The following package was automatically installed and is no longer required:
libgeos-3.2.2
Use 'apt-get autoremove' to remove them.
The following extra packages will be installed:
arduino-core avr-libc avrdude extra-xdg-menus libftdi1 libjna-java
librxtx-java
Suggested packages:
arduino-mk avrdude-doc task-c-devel gcc-doc gcc-4.2 libjna-java-doc
The following NEW packages will be installed:
arduino arduino-core avr-libc avrdude binutils-avr extra-xdg-menus gcc-avr
libftdi1 libjna-java librxtx-java
0 upgraded, 10 newly installed, 0 to remove and 130 not upgraded.
Need to get 19.7 MB of archives.
After this operation, 78.5 MB of additional disk space will be used.
Get:1 http://us.archive.ubuntu.com/ubuntu/ precise/main libftdi1 amd64 0.19-4 [16.8 kB]
Get:2 http://us.archive.ubuntu.com/ubuntu/ precise/universe libjna-java amd64 3.2.7-4 [416 kB]
Get:3 http://us.archive.ubuntu.com/ubuntu/ precise/universe librxtx-java amd64 2.2pre2-10 [174 kB]
Get:4 http://us.archive.ubuntu.com/ubuntu/ precise/universe binutils-avr amd64 2.20.1-2 [4,658 kB]
Get:5 http://us.archive.ubuntu.com/ubuntu/ precise/universe gcc-avr amd64 1:4.5.3-3 [8,267 kB]
Get:6 http://us.archive.ubuntu.com/ubuntu/ precise/universe avrdude amd64 5.11.1-1 [242 kB]
Get:7 http://us.archive.ubuntu.com/ubuntu/ precise/universe avr-libc all 1:1.7.1-2 [4,665 kB]
Get:8 http://us.archive.ubuntu.com/ubuntu/ precise/universe arduino-core all 1:1.0+dfsg-9 [613 kB]
Get:9 http://us.archive.ubuntu.com/ubuntu/ precise/universe arduino all 1:1.0+dfsg-9 [684 kB]
Get:10 http://us.archive.ubuntu.com/ubuntu/ precise/universe extra-xdg-menus all 1.0-4 [12.8 kB]
Fetched 19.7 MB in 6s (2,906 kB/s)
Selecting previously unselected package libftdi1.
(Reading database ... 76763 files and directories currently installed.)
Unpacking libftdi1 (from .../libftdi1_0.19-4_amd64.deb) ...
Selecting previously unselected package libjna-java.
Unpacking libjna-java (from .../libjna-java_3.2.7-4_amd64.deb) ...
Selecting previously unselected package librxtx-java.
Unpacking librxtx-java (from .../librxtx-java_2.2pre2-10_amd64.deb) ...
Selecting previously unselected package binutils-avr.
Unpacking binutils-avr (from .../binutils-avr_2.20.1-2_amd64.deb) ...
Selecting previously unselected package gcc-avr.
Unpacking gcc-avr (from .../gcc-avr_1%3a4.5.3-3_amd64.deb) ...
Selecting previously unselected package avrdude.
Unpacking avrdude (from .../avrdude_5.11.1-1_amd64.deb) ...
Selecting previously unselected package avr-libc.
Unpacking avr-libc (from .../avr-libc_1%3a1.7.1-2_all.deb) ...
Selecting previously unselected package arduino-core.
Unpacking arduino-core (from .../arduino-core_1%3a1.0+dfsg-9_all.deb) ...
Selecting previously unselected package arduino.
Unpacking arduino (from .../arduino_1%3a1.0+dfsg-9_all.deb) ...
Selecting previously unselected package extra-xdg-menus.
Unpacking extra-xdg-menus (from .../extra-xdg-menus_1.0-4_all.deb) ...
Processing triggers for man-db ...
Processing triggers for ccache ...
Updating symlinks in /usr/lib/ccache ...
Processing triggers for hicolor-icon-theme ...
Processing triggers for shared-mime-info ...
Setting up libftdi1 (0.19-4) ...
Setting up libjna-java (3.2.7-4) ...
Setting up librxtx-java (2.2pre2-10) ...
Setting up binutils-avr (2.20.1-2) ...
Setting up gcc-avr (1:4.5.3-3) ...
Setting up avrdude (5.11.1-1) ...
Setting up avr-libc (1:1.7.1-2) ...
Setting up arduino-core (1:1.0+dfsg-9) ...
Setting up arduino (1:1.0+dfsg-9) ...
Setting up extra-xdg-menus (1.0-4) ...
Processing triggers for libc-bin ...
ldconfig deferred processing now taking place
0.02s$ sudo rm /usr/share/arduino/libraries/Robot_Control/Wire.h
rm: cannot remove `/usr/share/arduino/libraries/Robot_Control/Wire.h': No such file or directory
The command "sudo rm /usr/share/arduino/libraries/Robot_Control/Wire.h" failed and exited with 1 during .
Your build has been stopped.

I made little changes to travis.yml, anyway I still get an error in WireClass, looks like endTransmission doesn't accept any parameter (https://travis-ci.org/nanpy/nanpy-firmware)
build/local/WireClass.cpp: In member function 'virtual void nanpy::WireClass::elaborate(nanpy::MethodDescriptor*)':
build/local/WireClass.cpp:31:75: error: no matching function for call to 'TwoWire::requestFrom(int, int, int)'
build/lib_00/Wire/Wire.h:53:13: note: candidates are: uint8_t TwoWire::requestFrom(uint8_t, uint8_t)
build/lib_00/Wire/Wire.h:54:13: note:                 uint8_t TwoWire::requestFrom(int, int)
build/local/WireClass.cpp:39:51: error: no matching function for call to 'TwoWire::endTransmission(int)'
build/lib_00/Wire/Wire.h:52:13: note: candidate is: uint8_t TwoWire::endTransmission()
scons: *** [build/local/WireClass.o] Error 1

Looks like endTransmission doesn't accept parameters
https://github.com/nanpy/nanpy-firmware/blob/master/Nanpy/WireClass.cpp#L39

Passing! I updated Arduino to 1.0.5 in travis config. The Wire interface changed in 1.0.1 and TravisCi has only 1.0.

That's awesome! :) closing...


tidy: Rename Link to URL

#923 : 2  dependent PRs (#925 , #933 )
#993  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Cobalt0s and the rest of your teammates on  Graphite


Wrapping string breaks blame

Hello. When text is wrapped to the new line the blame of agitator is incorrect.
Expected behavior:
Empty lines oposite wrapped lines or so.
https://github.com/emmanueltouzery/agitator.nvim/assets/51912173/d2c94f06-927c-4970-bfb2-03beb6966ee7

yes, i'm aware of that. I don't think it's fixable with the current approach of a split window to show the blame contents.
Maybe it could be doable with some virtual text or something, but I don't see how.
I'd guess other plugins offering a similar feature (for instance fugitive) have the same flaw probably.. if they don't I could try to check exactly how they did it.

@emmanueltouzery Yeah. Fugitive works ok.

hmm I just installed fugitive to check how it does it, but it seems to me to be the same as agitator, with the same flaws. I think it resets the word-wrapping in your buffer when you enable blame... but if you set it back then i can see the problem there too.
Make sure to enable word-wrapping in buffer and scroll to the end and you'll see that the end doesn't line up with fugitive as well.
Maybe agitator could disable word-wrap in your buffer when it's activated? That's probably the difference you see with fugitive, i think.

@emmanueltouzery oh I'm sorry. You are right. Fugitive doesn't work too. I cannot remember why I thought it does.
Auto disable and enabling wrapping sounds good to me. I'm just afraid about auto enabling because someone may close agitator's window manually with :q or :bd and I have no ideas how to track it to reenable wrap behavior.
But in general this solution sounds totally good.

yeah i'm not sure about re-enabling back wrapping. I'm thinking just disabling it when the blame is opened and then not touching it anymore.

@emmanueltouzery
I find some implementation with z-index or so.
https://github.com/tanvirtin/vgit.nvim
Maybe it can help.
Alternatively we could open blame in the new tab.

Hello. Any updates about this issue?

I actually wrote a naive function to compute real line to screen line when there is word-wrapping. but this will break if the user later resizes the window... so i think this is just not worth doing. it'll always be possible to break it. I'll just disable wordwrap when blame is triggered. Just for future reference, the function was:
local function get_win_line_to_buf_line()
    local bufnr = vim.api.nvim_win_get_buf(0)
    local orig_view = vim.fn.winsaveview()

    local res = {}

    local line_count = vim.api.nvim_buf_line_count(bufnr)
    local cur_line = 1
    while cur_line <= line_count do
        vim.api.nvim_win_set_cursor(0, {cur_line, 0})
        res[cur_line] = vim.fn.winline()
        cur_line = cur_line + 1
    end

    vim.fn.winrestview(orig_view)
    return res
end

@emmanueltouzery
Unfortunately break doesn't restore to wrap when I close blame. Totally unusable for me.

i mentioned about that earlier, in https://github.com/emmanueltouzery/agitator.nvim/issues/9#issuecomment-1546732035
probably the best is that you make your own keyboard shortcut on top of these functions here that remembers if line wrapping was set, disables it, calls this blame here, and sets back line wrapping if it was set before. you can totally do this outside of agitator.
maybe you can check what does fugitive do in this case?


Render using multiple threads

Rendering is a CPU bound task which might be problematic for Node js during spikes.
We could use something like https://github.com/xk/node-threads-a-gogo to create a thread pool of workers that render React components to HTML and use the event loop just for I/O.

The solution in node to handle CPU-bound tasks is to scale using separate processes - not threads, which is how hypernova is designed to be used.

We are already using Node.js' cluster module here to achieve the same goal.
I think that Hypernova isn't opinionated on your clustering architecture so you're free to skip out on using cluster and instead implement something that uses threads-a-gogo.
Thanks for sharing though :) It looks like you can probably use threads and skip out on using cluster AND vms since it seems like the code is also sandboxed. I'm not sure what the performance implications would be of using this sort of thing (is this faster than VMs or not?) but it looks interesting.


Optimize allowedIPs for wireguard with CIDRs

I tried to grasp all the details in the commit message.
The idea is to use pod CIDRs when available rather than having to add single IPs/32 or IPs/128 to the allowedIPs in WireGuard. Among the advantages:

less lookup in the kernel trie while processing packets
less updates to the peer configurations in the kernel.

I created a kind cilium (vxlan routing) using this PR, the wg output from a node is:
...
There are still cidrs in allowed ips, are they expected?

Thanks for pointing out, please let me know which was the command you used to install it, as I'm not able to reproduce it.
In any case, I'm moving the CIDRs-related logic into a new separate PR.
In this I'll only handle the difference between previous behavior and tunneling mode, while also refactoring tests and adding a fallback hidden flag to the daemon config to use our old behavior in case something's wrong with the new logic.

/test

/test

/test


[IMP] reload partners of opened orders

This does not work. When I switch from offline to online, the data does not change. And the system shows errors.


Nodes should be depopulated at close

The hashmap of nodes is never depopulated, only inserted.

Nodes now have a weak pointer, and are thus automatically removed.


Mention users in !list and when playing their song

Actually use the discord @mention feature instead of just printing users' Discord nick/name in plaintext

I think this works (haven't tested)!
Alternatively, we could use the mention method on User/Member: https://discordpy.readthedocs.io/en/latest/api.html?highlight=mention#discord.abc.User.mention which may be more future proof.

This is definitely better, I didn't even know this was in the API. I've made the change to User.mention and tested it, so I'm just merging into master now


Create a project which will demonstrate usage of named registrations

Create a project which will lead to named registrations and utilize them
The goal is demonstrate usage of different strategies for the same interface

Implemented with exception of solution

Implemented including solution


Rerun subset of periodic surging experiments that did not make it 9ka

About a third of the periodic surging experiment did not make it the end of the 9ka simulation. The purpose of this PR is to rerun those simulations that failed, with a bit more attention and babysitting this time around.

After the first 3ka chunk eight simulations failed, due to a divergent velocity, and need to removed from the suite of simulations to rerun.  In order to identify the simulations that complete run:
touch run/crmpt12/crmpt12.commands.p2

for i in $(seq 34); do
     if tail "logs/crmpt12/crmpt12_46759282_${i}.out" | grep -q "ELMER SOLVER FINISHED AT:"; then
          sed -n "${i}p" ./run/crmpt12/crmpt12.commands >> run/crmpt12/crmpt12.commands.p2
     fi
done


fix(fix formatted date): fix formatted date

fix formatted date

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

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


how to connect to polygon mumbai testnet ?

We do not support testnets at the moment, but have it planned. ETA is by the end of the year.

@romanankr any update on testnet support? would love to be able to test ankr on goerly 🙏

@romanankr any update on testnet support? would love to be able to test ankr on goerly 🙏

Hey, I've just released the new v0.3.1 version with eth_goerli and avalanche_fuji support enabled. We're also currently working on adding support to polygon_mumbai testnet, ETA is 1-2 weeks.

@romanankr thanks for making the update! any chance we could see this new version integrated into the ankr-react package? that's how i'm using ankr.js atm.

@romanankr thanks for making the update! any chance we could see this new version integrated into the ankr-react package? that's how i'm using ankr.js atm.

hey, unfortunately at the moment we don't have enough resources to keep it updated, but PRs are always welcome 🙂

@romanankr gm, does Ankr itself support Polygon Mumbai? Doesn't seem to be listed here: https://www.ankr.com/rpc/

@romanankr gm, does Ankr itself support Polygon Mumbai? Doesn't seem to be listed here: https://www.ankr.com/rpc/

It does, I believe if you open Polygon chain there's testnet option

Polygon Mumbai is currently supported in v0.5.0


Follow up for JS bindings

Follow-up of #262, fixes a doc generation bug for hacl-star.github.io/

@prosecco @msprotz  you've already reviewed the previous PR


CORS: Add Content-Type to Access-Control-Allow-Headers

Access-Control-Allow-Headers: Content-Type

Added, but I see that this is one of the simple headers which apparently should always be allowed:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Access-Control-Allow-Headers


gson 2.8.9 -> 2.10.1

Test cases look good.
Release notes:
https://github.com/google/gson/releases
Drafting pending further verification.

Tested through. All clear


✨ isRecording() prop for cameraRef

What feature or enhancement are you suggesting?
Right now, you can use:
- cameraRef.current?.startRecording()
- cameraRef.current?.stopRecording()
- cameraRef.current?.pauseRecording()
- cameraRef.current?.resumeRecording()
If you call stopRecording() twice, you get a console.warn that you called it twice.
Would be awesome to be able to control this logic within code like so:
if(cameraRef.current?.isRecording()) { // could also be cameraRef.current?.isRecordingAsync() possibly
    cameraRef.current?.stopRecording() // prevent the warning and also get the status of the camera.
}

What Platforms whould this feature/enhancement affect?
iOS, Android
Alternatives/Workarounds
Right now, you just set a separate state variable when you call:
    cameraRef.current.startRecording({
        flash,
        fileType: "mp4",
        onRecordingError: (error) => {
          console.error("Recording failed!", error)
        },
        onRecordingFinished: (video) => {
          console.log(`Recording successfully finished! ${video.path}`)
          onMediaCaptured(video, "video")
          setCameraRecording(false) // track current state of camera in separate variable
        },
    })
    setCameraRecording(true) // track current state of camera in separate variable

Additional information

[X] I agree to follow this project's Code of Conduct
[X] I searched for similar feature requests in this repository and found none.

@mrousavy that would work great too! Thanks again

Yea but I'm not gonna work on this for a while, maybe some point in the future. If you need this we can of course build this feature for you through my consultancy company, otherwise I'll maybe take a look at this when VisionCamera is a TurboModule :)


incorrect parameter due to typo in line 143

There is an incorrect parameter upload_file in  instamojo.php on line 143,   https://github.com/Instamojo/instamojo-php/blob/master/instamojo.php
if($link['file_upload']) {
$file_upload_json = $this->uploadFile($link['upload_file']);
Both above lines should have the same value 'file_upload'

Thanks @mtrajan. Fixed in 255eefd.

cool ! thanks
Regards,
Rajan
Q: Why is this email five sentences or less?
A: http://five.sentenc.es
On Fri, Apr 10, 2015 at 1:14 PM, Ashwini Chaudhary<EMAIL_ADDRESS>
wrote:

Closed #5 https://github.com/Instamojo/instamojo-php/issues/5.
—
Reply to this email directly or view it on GitHub
https://github.com/Instamojo/instamojo-php/issues/5#event-277876228.


feat(kit): TuiCalendarRange add item property for correctly switch value outside

Closes #8564
Closes #8527

@zhd-dm Don't fork, clone the repository and open a pull request. You have write permissions.

@zhd-dm Don't fork, clone the repository and open a pull request. You have write permissions.

Okay


Python client would wait broker's PING command so that close() even took 42 seconds to complete

2021-06-17 15:20:54,011 - (unknown file)[line:0] - INFO - [persistent://public/default/hamster_school_config-partition-2, process_update_config_0, 132] Unsubscribed successfully
2021-06-17 15:20:54,011 - (unknown file)[line:0] - INFO - Successfully Unsubscribed Consumer - 2 for Subscription - process_update_config_0 for Topic - persistent://public/default/hamster_school_config
2021-06-17 15:20:54,014 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Handling incoming command: SUCCESS
2021-06-17 15:20:54,014 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Received success response from server. req_id: 278
2021-06-17 15:20:54,014 - (unknown file)[line:0] - INFO - [persistent://public/default/hamster_school_config-partition-3, process_update_config_0, 133] Unsubscribed successfully
2021-06-17 15:20:54,014 - (unknown file)[line:0] - INFO - Successfully Unsubscribed Consumer - 3 for Subscription - process_update_config_0 for Topic - persistent://public/default/hamster_school_config
2021-06-17 15:20:54,014 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Handling incoming command: SUCCESS
2021-06-17 15:20:54,014 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Received success response from server. req_id: 279
2021-06-17 15:20:54,014 - (unknown file)[line:0] - INFO - [persistent://public/default/hamster_school_config-partition-4, process_update_config_0, 134] Unsubscribed successfully
2021-06-17 15:20:54,015 - (unknown file)[line:0] - INFO - Successfully Unsubscribed Consumer - 4 for Subscription - process_update_config_0 for Topic - persistent://public/default/hamster_school_config
2021-06-17 15:20:54,015 - (unknown file)[line:0] - DEBUG - Unsubscribed all of the partition consumer for subscription - process_update_config_0
2021-06-17 15:20:54,015 - init.py[line:845] - INFO - Closing Pulsar client
2021-06-17 15:21:09,093 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:1105 -> <IP_ADDRESS>:6650] Handling incoming command: PING
2021-06-17 15:21:09,094 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:1105 -> <IP_ADDRESS>:6650] Replying to ping command
2021-06-17 15:21:09,104 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:1105 -> <IP_ADDRESS>:6650] Sending ping message
2021-06-17 15:21:09,106 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:1105 -> <IP_ADDRESS>:6650] Handling incoming command: PONG
2021-06-17 15:21:09,106 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:1105 -> <IP_ADDRESS>:6650] Received response to ping message
2021-06-17 15:21:09,219 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Handling incoming command: PING
2021-06-17 15:21:09,220 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Replying to ping command
2021-06-17 15:21:09,227 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Sending ping message
2021-06-17 15:21:09,229 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Handling incoming command: PONG
2021-06-17 15:21:09,229 - (unknown file)[line:0] - DEBUG - [<IP_ADDRESS>:56812 -> <IP_ADDRESS>:6650] Received response to ping message
2021-06-17 15:21:34,041 - hamster_consumer.py[line:723] - INFO - process:ConsumerP0,time:42.0277788639
2021-06-17 15:21:34,042 - hamster_consumer.py[line:726] - INFO - Hamster consumer Quit.

The issue had no activity for 30 days, mark with Stale label.

Closed as stale.
The development of the Python client has been permanently moved to https://github.com/apache/pulsar-client-python. If it's still relevant, please open an issue there.


Modify problem upload and add tests

Overview
In Azure Quantum we have two main ways of submitting a problem to a solver:

Uploading the Problem to a Workspace, and then calling submit() supplying the returned URI for where the problem was uploaded:

problem = Problem(name="test", terms = [])
uri = problem.upload(workspace)
job = solver.submit(uri)

Or we can call submit() supplying the Problem, which will upload the problem for the user automatically:

problem = Problem(name="test", terms = [])
job = solver.submit(problem)

Currently the second method uses a default name for the blob, if the submit method receives an instance of a Problem then the blob name is by default set to "inputData":
elif isinstance(problem, Problem):
    name = problem.name
    problem_uri = problem.upload(
        self.workspace,
        compress=compress,
        container_name=container_name,
        blob_name="inputData",
    )

source
In the first method, it uses a formatted version of the (user-defined) Problem name, e.g. "My Optimization Problem" or "Ship Balancing Problem" etc.
blob_name = "{}-{}".format(self.name, uuid.uuid1())

source
In some cases this has resulted issues (due to the naming of the blob) causing the job to fail. To resolve this (and for consistency) this PR adopts the approach used in (2), by using a default name for the blob. Currently there isn't a requirement for this to be user-definable (as it isn't in (2)).
Changes

Modified the upload() method in problem.py to use a default blob_name (done in azure/quantum/optimization/solvers.py also)
Adds a test cases for the upload() method and submit() method
Minor typo fix

/azp run

/azp run

/azp run


A lot of Features Added + Hardware compatibility

need to add gradual boundary direction change to space-mode for hardware feasibility

documentation coming soon


How to disable sign column

Syntastic has this option to disable sign column: let g:syntastic_enable_signs = 0
I'm switching to ale and exptect the same, can you tell me we have similar option?

Should work with set signcolumn=no in .vimrc.


Replace licenseUrl with SPDX license identifier in PowerShell NuGet packages

Summary of the new feature/enhancement
As a developer consuming NuGet packages, I want the license terms of each package to remain readable and immutable after I have reviewed them. However, PowerShell NuGet packages such as Microsoft.PowerShell.Security currently have https://github.com/PowerShell/PowerShell/blob/master/LICENSE.txt as licenseUrl in the package metadata. The license terms referenced by such a URL can be changed, and the URL may stop working if the branch is renamed (as requested in https://github.com/PowerShell/PowerShell/issues/12951).
Proposed technical implementation details (optional)
NuGet has deprecated licenseUrl (https://github.com/NuGet/Announcements/issues/32). The license property should be used instead, presumably here: https://github.com/PowerShell/PowerShell/blob/ab7460365ffdd40e1c3ae67958a27f210c6b594a/tools/packaging/packaging.strings.psd1#L137

We could use <license type="MIT"/>.
/cc @TravisEz13

There are two option in https://docs.microsoft.com/en-us/nuget/reference/msbuild-targets#packing-a-license-expression-or-a-license-file

Set license type
Set license file

Since we have a license file in repo root perhaps second option is better.

iconUrl is also deprecated. We should add icon and keep iconUrl for backward compatibility.

We could use <license type="MIT"/>.
/cc @TravisEz13

This should work for the packages in this repo, but @adityapatwardhan is the expert on compliance here.  We may have to get some changes in other places after we create the package.

iconUrl is also deprecated. We should add icon and keep iconUrl for backward compatibility.

If it's like licenseUrl, you cannot have both.

There are two option in https://docs.microsoft.com/en-us/nuget/reference/msbuild-targets#packing-a-license-expression-or-a-license-file

Set license type
Set license file

Since we have a license file in repo root perhaps second option is better.
For backward compatibility we should keep old code too.

1 is the preferred option.  We have systems that try to parse and 1 works more reliably

The documentation says that both icon and iconUrl can be specified in the same nuspec file. https://docs.microsoft.com/nuget/reference/nuspec#icon

You can specify both icon and iconUrl to maintain backward compatibility with sources that do not support icon. Visual Studio will support icon for packages coming from a folder-based source in a future release.

We only publish to NuGet.org so we no not need "iconUrl".

yes we can use


Idrinth

@w20k would love some feedback about the structure there atm - still not sure if that grouping makes sense.

LGTM

Good thing, that you've started using docs, nice @Idrinth ;)
Though I would consider to overlook this part ->
 idrinth.core.ajax.runHome ( '', function ( file ) {
            var applyRecursive = function ( to, from, func ) {
                for (var prop in from) {
                    if ( from.hasOwnProperty ( prop ) ) {
                        if ( typeof to[prop] === 'string' && typeof from[prop] === 'string' ) {
                            to[prop] = file[prop];
                        } else if ( typeof to[prop] === 'object' && typeof from[prop] === 'object' ) {
                            func ( to[prop], from[prop], func );
                        }
                    }
                }
            };
            applyRecursive ( idrinth.text.data, JSON.parse ( file ), applyRecursive );
        }, idrinth.text.start, idrinth.text.start, null, true );

LGTM


The metrics service is not used in the test-mlflow-image-pipeline in any way.

Description

The metrics service is not used in the test-mlflow-image-pipeline in any way, based on information in https://github.com/opendatahub-io/ai-edge/pull/119#issuecomment-1749253767.
How Has This Been Tested?

Tested manually.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

Thanks, merging.


pull request template URL returns 404Not Found

Describe the bug
pull request template URL in the contribution guid is returning a 404 Not Found error and needs to be updated.
To Reproduce
URL pull request template is 404 Not Found error.
Expected behavior
Fix URL https://github.com/glasskube/glasskube/blob/main/.github/PULL_REQUEST_TEMPLATE.md

Is it OK if I work on fixing this issue and updating the URL?

yes, sure!


Fix non-capturing groups

I'd previously used the wrong syntax, luckily it didn't break things.
I'm sure there are a ton of places this should be changed further. I'll look at that for another PR.
Signed-off-by: kingthorin<EMAIL_ADDRESS>

Codecov Report

Merging #707 (8e8dd0b) into main (9aaa8a5) will increase coverage by 0.01%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #707      +/-   ##
============================================
+ Coverage     92.78%   92.79%   +0.01%
+ Complexity     2578     2576       -2
============================================
  Files           281      281
  Lines          5113     5108       -5
  Branches        529      529
============================================
- Hits           4744     4740       -4
+ Misses          237      235       -2
- Partials        132      133       +1

Impacted Files
Coverage Δ

...ain/java/net/datafaker/providers/base/Barcode.java
85.71% <0.00%> (-4.77%)
:arrow_down:

.../datafaker/transformations/sql/SqlTransformer.java
85.50% <0.00%> (-0.73%)
:arrow_down:

...ain/java/net/datafaker/providers/base/Disease.java
100.00% <0.00%> (ø)

.../java/net/datafaker/service/FakeValuesService.java
84.47% <0.00%> (+0.19%)
:arrow_up:

...n/java/net/datafaker/providers/base/BaseFaker.java
90.99% <0.00%> (+3.16%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Did "we" give up on what-the-diff? Or has it stopped working?

Did "we" give up on what-the-diff? Or has it stopped working?

No available credits at the moment :(

I just applied for an Open Source plan! I hope we could get accepted, I do think these PR summaries are quite helpful!

Okay


Scrollable transfer tab

Description

Create a scrollable component to provide a more intuitive way for users to type in missing courses on transfer tab
Screenshots
Display missing courses for current plan with horizontal scroll component

After typing correct course name, course will be removed from the list

Steps to verify/test this change:
Final Checks:

[ ] Verify successful deployment
[ ] Delete branch

(optional)

[ ] Write tests
[ ] Write documentation

For autocomplete, MaterialUI has a great component to complete that functionality. Maybe worth to take a look


4k to 1080p

Hi.
As my sad little i3 Plex server doesn't have the horsepower to transcode from 4k to 1080 in real time when I am watching on non-4k devices, I was thinking of making a few 1080 rips of 4k content on my i7 box. The problem is I can't work out how to do it. I have used the --720p switch in the past but the PQ looks a little dodgy. Is there an easy way of doing a 1080p rip?
Cheers

@tehbdg I think you mean a 1080p transcoding, right? Because you've already ripped the 4K?
If so, just use the --max-width and --max-height options like this to scale your 4K video into a 1920x1080 boundary:
transcode-video --max-width 1920 --max-height 1080 "/path/to/Movie.mkv"

Does that work for you?

Thanks @donmelton , I will give it a whirl after my current encoding is complete.....

@donmelton That worked perfectly. I have also been trying out the --avbr setting, and it seems to be working out great, I had to reinstall the transcode-video packages using gem as I don't think the docker version has been updated yet. Using x264-10bit and --avbr on a UHD movie I am currently getting an average of 14.8 frames per second encode. Usually on x265-10 bit --simple I get around 3-4 fps.
Obviously the files work out to be bigger. I just did a comparison using Once Upon A Deadpool 1080p. The original Remux was around 35gb.
--AVBR x264 1080p is 9.289 gb
Your old system worked out to be 15.4 gb
--simple x265 is 9.226 gb
That's pretty amazing.

@tehbdg I think you mean a 1080p transcoding, right? Because you've already ripped the 4K?

Sorry, I missed that. Yes I have already ripped the movies, now I want to transcode.

@tehbdg I'm glad that worked for you! And that you're using and enjoying the new --avbr ratecontrol option. Thanks.
I'll close this now since your original issue is now resolved but feel free to comment again here or open new issues if you have additional questions.


Changing release version

Please could we get this pushed to Clojars? BTW the README.md has old coords in it.


main: set TCMUR_DEV_FLAG_IS_OPEN on successfully added devices

Otherwise the device will not properly be closed when removed.

https://github.com/open-iscsi/tcmu-runner/issues/279

Test packages: https://3.chacra.ceph.com/r/tcmu-runner/wip-open-flag/7efb6ad614acd2949f36e6cfd4d4fd9782deb0b5/centos/7/flavors/default/x86_64/


25 enhancement create default user files

This PR adds a task & accompanying documentation for adding files to a user's home directory. It assumes any files added this way will always live at /home/username/filename without nested directories.
@franklinkim I'm sorry, but I have no idea how to make your tests work. I whipped up a small vagrant-ansible test to test my changes. Download it, cd into it, and run Vagrant up (it'll pull down HEAD of my branch and run the playbook).
Let me know if you'd like to see any changes.

Hi, thanks for the PR.
I did some small changes:

Renamed user_files to home_files to make it more clear
Added a check for missing home_files attributes
Set the correct home directory

I'm testing this role through docker and the Makefile:
make ubuntu14.04
make ubuntu16.04
...


replace reinit with apiCommand

This covers cave_static and cave_app

Fixed in static with: c20ff8059cc04b0e93ccb2d0d8cd3eb3151bee3a
Fixed in app with: https://github.com/MIT-CAVE/cave_app/commit/59860e0bc4af54e8e34985f43150f700996c517a


Naming inconsistencies

Possible changes for the next major release:

[ ] change Z to something else, perhaps just z? Capital letter is poor Python style and inconsistent with other lowercase variables.
[ ] O and x0 both used for origin in different contexts.

Hey, now there is a Renamed trait to aid backwards compatibility so we don't have to wait for the next major release!


Enable triggering postcommit via specific file

Workaround #28909
Enable to trigger a PostCommit from pull request by touching a file named "<workflow_name>.json" (/json to get around with RAT PreCommit), as a simple workaround of #28909.
In this way the last piece of gap (comment trigger) is partially fixed and we can shutdown all Jenkins jobs.
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Oh, we should probably do the same for benchmarks as well

Oh, we should probably do the same for benchmarks as well

The hope is we can still have the comment trigger in the near future, which is more convenient for developers and reviewers (reviewer can also run the test by comment), as @volatilemolotov working on GitHub App approach. If decided this being long time solution then we can add it to load tests and performance tests


chore: promote golang-ok to version 0.0.6

this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

golang-ok
Changes in version 0.0.6
Chores

release 0.0.6 (jenkins-x-bot)
add variables (jenkins-x-bot)

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Example in documentation does not compile

Hi, I attempted to copy and paste the example from the purescript-spec documentation and run it under PureScript 0.12.5, but the code does not compile. Here is the error:
Error found:
in module Main
at src/Main.purs:35:8 - 49:51 (line 35, column 8 - line 49, column 51)

  Could not match type

    Effect

  with type

    Aff

while trying to match type Effect Unit
  with type Aff t0
while checking that expression (run [ consoleReporter
                                    ]
                               )
                               ((describe "purescript-spec") ((discard (...)) (\$__unused ->
                                                                                ...
                                                                              )
                                                             )
                               )
  has type Aff t0
in value declaration main

where t0 is an unknown type

See https://github.com/purescript/documentation/blob/master/errors/TypesDoNotUnify.md for more information,
or to contribute content related to this error.

This is for the code here: https://purescript-spec.github.io/purescript-spec/#full-example, i.e.
module Main where

import Prelude

import Data.Time.Duration (Milliseconds(..))
import Effect (Effect)
import Effect.Aff (launchAff_, delay)
import Test.Spec (pending, describe, it)
import Test.Spec.Assertions (shouldEqual)
import Test.Spec.Reporter.Console (consoleReporter)
import Test.Spec.Runner (run)

main :: Effect Unit
main = launchAff_ $ run [consoleReporter] do
  describe "purescript-spec" do
    describe "Attributes" do
      it "awesome" do
        let isAwesome = true
        isAwesome `shouldEqual` true
      pending "feature complete"
    describe "Features" do
      it "runs in NodeJS" $ pure unit
      it "runs in the browser" $ pure unit
      it "supports streaming reporters" $ pure unit
      it "supports async specs" do
        res <- delay (Milliseconds 100.0) $> "Alligator"
        res `shouldEqual` "Alligator"
      it "is PureScript 0.12.x compatible" $ pure unit

I believe the correct example would just be to not to launchAff_? It looks like that works.

Also, this example:
it "adds 1 and 1" do
  1 + 1 `shouldEqual` 2

Is missing parentheses, because shouldEqual has higher precedence than +. It should probably be:
it "adds 1 and 1" do
  (1 + 1) `shouldEqual` 2

Which version of purescript-spec are you running?

This is v3.1.0.

recent version is  v4.0.0-rc1
https://github.com/purescript-spec/purescript-spec/blob/v4.0.0-rc1/src/Test/Spec/Runner.purs#L192-L198
in v3.1.0, run returns Effect so no need for launchAff_ https://github.com/purescript-spec/purescript-spec/blob/v3.1.0/src/Test/Spec/Runner.purs#L173-L177


[ENH] Investigate potential integration with modin project

Brief Description

I would like to investigate whether janitor would work well with modin as a backend.
Modin is a (currently Linux/OSX only) replacement for Pandas that purports to be much faster. It has most, but not all, of the pandas API, but for what it has it should be a one-for-one replacement.
This would be just some investigative work to see if I could even get this to work in the first place.

Worthwhile to point this to: #394
Also: https://modin.readthedocs.io/en/latest/architecture.html
This may or may not introduce a complication:

Currently, each partition’s memory format is a pandas DataFrame. In the future, we will support additional in-memory formats for the backend, namely Arrow tables.

Hey @anzelpwj! Greetings from chilly Cambridge :smile:. Wanted to know where you’re at with this issue? I’m just reviewing old issues from the issue tracker at the moment.

Apologies @ericmjl - kid started daycare late August and we've been dealing with the daycare plague and other sundries for the past several months. I'd like to get started again contributing to PyJanitor (was inspired by going to PyDataLA and doing a tiny sprint there), but would definitely understand if someone else is raring to take this ticket. Should hopefully have some time clear up after next week though.

Okay, so I've done a review of modin documentation, with a particular eye to which methods they state don't have full support link. None of our code uses methods that are complete unsupported, which is good. This leaves two other categories:

Code that "defaults to Pandas" (modin dataframe is converted to pandas dataframe, routine is run, dataframe is converted back to modin. Code will work, but there will be some overhead).
APIs with partial implementation.

As for the DataFrame APIs we call:
Partial support:

append: "Not fully optimized" (so probably not a concern).
fillna: If a dataframe is given for the fill value, this becomes a default-to-pandas routine (unlikely to be a big problem).
query: Local variables not supported. As best I can tell, since query is kind of an eval function, it means that if we define a variable in the string it's an issue. Maybe we might want to throw up a warning for people running filter_on?

Defaults to pandas:

assign
combine_first
drop_duplicates
duplicated
to_csv (and only run in a few tests anyway...)

The biggest difficulty I'm having is seeing if we can create an equivalent to pandas_flavor for modin. That's going to be what I look into next...

Okay, sent them a message to their listserv, will see what I hear back.

Got a response!

We do have ways to create dataframe methods, but it's in early stages and still kind of hacky.
There's a tutorial here: https://github.com/ucbrise/risecamp/blob/risecamp2019/modin/tutorial_notebooks/exercise_3.ipynb.
Let me know if it works for you.

Not sure if it's going to be easier to try and create a pandas/pandas-flavor like fix for Modin or create something on our end now.

Hi everyone, is this issue still in demand - or it became less relevant for whatever reasons?
I am trying to understand a) does it make sense to play with modin b) does it make sense to target pyjanitor  in the process of playing with modin


How does auth work?

From the README:

It authenticates using the GitHub provided OIDC token for the pipeline run.

Could you please elaborate how auth works?

Hey @guettli,
when this action is invoked, we call the GitHub Actions OIDC endpoint to generate an OIDC token signed by GitHub Actions. This token identifies the pipeline (as well as the Repo, Org, Pipeline Creator).
This token is then being passed to TPS. TPS validates that the OIDC token matches GitHub Actions signature, and then matches the org from the token to a Hetzner account. A new project is then created in the Hetzner Account and the projects token is returned.
Hope this answers your question.


Fix Set-PnPListItem not saving taxonomy values when using -Batch parameter

Type

[x] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
Fixes #4475
What is in this Pull Request ?
Fix Set-PnPListItem not saving taxonomy values when using -Batch parameter. The -Batch parameter has a different code path when using -Batch and when passing terms in format TermGroup|TermSet|Term1 instead of a guid.
The issues fixed related to -Batch and taxonomy terms passed with the format TermGroup|TermSet|Term1 are:

fieldValueCollection.Values.Add(field.NewFieldTaxonomyValue(itemId, label)); was never called, so the term was never included in the updates
batch.CacheTerm(termGuid.ToString(), termGuid, label); used termGuid after Guid.TryParse(arrayItem as string, out termGuid) failed, meaning it was always an empty guid 00000000-0000-0000-0000-000000000000, causing any later iteration to return an empty cached term
batch.CacheTerm() now uses as key the format TermGroup|TermSet|Term1 in addition to term id, making the cache useful when all terms are in that format. Before the cache worked only with guid format.

Example of running the script:

Script sample similar to the one from #4475
$list = 'List with taxonomy'
$ID = 1
# Store the terms in an array
$terms = @()
$terms += 'This is a test|Column with taxonomy|Test 1'
$terms += 'This is a test|Column with taxonomy|Test 1'
$terms += 'This is a test|Column with taxonomy|Test 2'
$terms += "271a4dae-c22f-4ba4-b79c-50dad60bd8c3"

$batch = New-PnPBatch

Set-PnPListItem -List $list -Identity $ID -UpdateType "UpdateOverwriteVersion" -Values @{"Column_x0020_with_x0020_taxonomy" = $terms; "Title" = (Get-Date).ToLongTimeString()} -Batch $batch

Invoke-PnPBatch -Batch $batch

Thanks @jackpoz , good catch, merged the fix.


Download/upload file through RAML endpoint

Please add possibility to download and upload file by using RAML endpoints definition.

When it comes to download, there is a way to do it.
Define a raml part like this:
get:
  responses:
    200:
      body:
        application/octet-stream:
          type: file

and you will get decorator generated like this:
@RequestMapping(value = "", method = RequestMethod.GET)
public ResponseEntity<?> getObject() {
	return this.returnControllerDelegate.getObject();
}

So you got return type: ResponseEntity<?> and you can return a byte[] from your implementation.
This is more like a workaround and it's not ideal since generated code should enforce you to return a file (or byte[]).


Fix packaging in unix

Ignore build output and runtime configuration for TestPlatform.CLI and TestPlatform.Build packaging projects.
Fix build.sh to use netcoreapp1.0 publish for testhost.

@codito,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Foundry 8.9 causes layout issues.

Looks like something about the latest upgrade to 8.9 does some layout changes. The current speaker display element fills up available vertical space.
After experimenting it looks like a new flexcol class is taking priority. Changing the .illandril-chat-enhancements--currentSpeaker css flex declaration to be !important fixes the issue:
.illandril-chat-enhancements--currentSpeaker {
  flex: 0 !important;
  padding: 3px 0.5em;
}

Alternatively providing a more specific declaration including the new flexcol class works as well:
.flexcol > .illandril-chat-enhancements--currentSpeaker {
  flex: 0;
  padding: 3px 0.5em;
}

Is there an ETA on when this might be fixed? I don't know how to fix this on my own.

I should have time to get a new release out with a fix by the end of this weekend.

@Artetsu in the meantime you can manually edit the CSS declaration as mentioned in the issue. You can open it in a normal text editor, should be the first text in the file. It'll be the styles.css file in the modules\illandril-chat-enhancements in your Foundry install folder.

I managed to free up a few spare minutes before bed - it should be fixed tonight.


Add FAQ to README (new_layout branch)

This is a work in progress to add a FAQ to the README, and refactor it a bit. There are still some TODOs that should be taken care of (mostly prominent links to installers/install instructions for other platforms), and it could be tidied up a bit. A table of all available versions could be added (like I did on https://github.com/RequestPolicyContinued/requestpolicy/wiki). The screenshots need to be updated to reflect the new layout.
Please let me know if something should be added/removed/changed.

I'm planning to add a seperate file FAQ and keep all the questions there.
Anyway for now we can keep those in the readme file.
Here's a list with tasks for the README.md file:

Add slackbuild
Update images with new UI
Fix the overall layout

Nice :+1:


Location for saved lighting profiles

I want to secure the lighting profiles that i created for my logitech 810 on dropbox, but i can't seem to find where they're located =_0

Open: %appdata%/Aurora/ there are folders with *.json files. These files are the profiles. :wink:


docker/vpp-cri/Dockerfile: use alpine as base

This PR modifies the cri Docker image to be based on the alpine Docker image. This makes the image a lot smaller (39 MB together with #657).

Coverage remained the same at 75.243% when pulling 15937ab9528fefcb8d93a94d380fa2d822131872 on unclejack:switch_cri_image_to_alpine into a9290714f8e947364408f60bc6137b311dc5e8e6 on contiv:master.

Hi,
as far as I remember, CRI was not able to run under alpine - since the Dockershim was using some Linux features that were not present there. I don not remeber what it was exactly, but the log of the CRI was full of errors.
Make sure the CRI actaully works with this.

docker logs contiv-cri
Before that, you need to have it running / installed by
bash <(curl -s https://raw.githubusercontent.com/contiv/vpp/master/k8s/cri-install.sh)

Closing this, as CRI has been disabled for now by https://github.com/contiv/vpp/pull/729


chore(test) Make all the template tests work offline!!!

Signed-off-by: jeromesimeon<EMAIL_ADDRESS>Description
I've been losing my mind on failed test runs unable to download models from the models repository. This PR makes the template tests go offline.
Changes

Add all necessary models in template tests
Make the model loader use offline mode

Flags

We got to do something about the stability / availability of the published models some day
Tested by actually going offline!

Coverage remained the same at 87.553% when pulling 7a52e4575bbb79d27b6665d6d2246ad7616ac1ed on jeromesimeon:js-test-offline into 579f162b84ae07791cd5f28fcc152372069cb732 on accordproject:master.


add ability to copy the events

It will be useful to have the possibility to copy a specific event.

@shiramax by copy you mean as system-wide clipboard? i.e. long-press and select "copy text"?

yes


Method to make binding the class kwarg easier

Its common to want to intercept the class kwarg like this:
  def input(*, class: "hi", **, &)
    super(*, class: tokens(keyword(class:)))
  end

Phlex could have a method that makes this possible like this:
    def keyword(**bindings)
      if bindings.size > 1
        bindings.values
      else
        bindings.values.first
      end
    end

Not sure if keyword is the right method name. Some possible method names are:
bind
keyword
keywords
value
values
grab

Previous attempt at https://github.com/phlex-ruby/phlex/pull/717 broke the way tokens behaved and was less composable. I believe this approach solves both of those problems.

I quite like this with the name grab. @willcosgrove what are your thoughts?

I like grab too. It is solving an interesting problem. I'm kind of surprised I haven't run into a need for it yet 🤔
In the example given, I would just use mix and not have any need to grab out the class kwarg.

Oh, of course you can already use mix for this. I’m trying to be very careful about what features we introduce in core, but I don’t think this represents a significant increase in complexity or maintenance burden. It’s the kind of thing that’s useful if you need it and disappears if you don’t, which I like.

Even though you can technically use mix, with **mix(:class), I think something like this is quite a bit simpler when all you’re doing is passing along the class.
def foo(class:)
  div(class: [grab(:class), "some other classes here"])
end


Use Dotman for setup script

After Dotman is completed, the contents of Dotfile.rb on branch wip/dotman will replace the current Python setup script setup.py.

No, it won't. That branch doesn't even exist anymore (because I just deleted it, woops).


Show project list default  view in list column

Describe the bug
Company, Service, Period, Due Date, Status, PM
yeh order chahiye project me
Screenshots


How to properly install?

I'm following the exact installation steps and I got a can't found vue-tour component. Please update the installation process with these step. By the way, how can it be fixed?

Can you check if you don't miss any import?
import Vue from 'vue'
import App from './App.vue'
import VueTour from 'vue-tour'

require('vue-tour/dist/vue-tour.css')

Vue.use(VueTour)

new Vue({
  render: h => h(App)
}).$mount('#app')

i restarted the app and it work, so, the library work, thanks!


Issue 553

Issue
Closes #553
Intent
StreamConfig should be inherited to target from root level, for all actions in sasjs web
Also resolve unhandled promise rejections in main.ts
Implementation

Updated main.ts,  unhandled promise rejections were due to findTargetInConfiugaration.
Passed down inherited StreamConfig to all actions in src/commands/web/index.ts
Added test for sasjs web, with configuration having streamConfig at root level and not at target.

Checks

[x] Code is formatted correctly (npm run lint:fix).
[x] Any new functionality has been unit tested.
[x] All unit tests are passing (npm test).
[ ] All CI checks are green.
[ ] JSDoc comments have been added or updated.
[x] Reviewer is assigned.

@saadjutt01 / @allanbowe The PR title needs to be changed to make a release happen.

@saadjutt01 / @allanbowe The PR title needs to be changed to make a release happen.

updated


no_log set to JINJA expression failing due to expecting bool

I had a task with no_log set to a variable, which wasn't behaving as expected.
    - name: Update yum cache
      yum:
        list: available
        update_cache: yes
      no_log: no_log_package

The no_log_package variable was set in defaults/main.yml as no_log_package: false and I would see failures in these tasks saying that no_log is set t true
fatal: [asa-co7-498617816]: FAILED! => {"censored": "the output has been hidden due to the fact that 'no_log: true' was specified for this result"}

I found I could get rid of this by changing from a variable to a JINJA expression
    - name: Update yum cache
      yum:
        list: available
        update_cache: yes
      no_log: "{{ no_log_package | bool }}"

And passing values for no_log_package with -e on the command line worked. However I now see language server errors highlighted for this task

If I'm indeed using bad coding here to allow toggling no_log I'm happy to change my code per a suggested correct solution. Otherwise how can I configure the language server to ignore this?

Actually it seems the language server doesn't like no_log: no_log_package syntax either, same error there. So it appears the schema only accepts a boolean literal for no_log?


APP-258: Add timestamp field in the log scheme

Can one of the admins verify this patch?

@kaa-leeroy test this please


[Feature Request] Add left delim comment before leading space

Add a new option NERDCommentBeforeLeadingWhitespace or something to allow behavior of AddLeftDelim to be changed to

function! s:AddLeftDelim(delim, theLine) abort
  return substitute(a:theLine, '^\(\s*\)', a:delim . '\1' , '')
endfunction

Should this option be turned on by default? I personally find aligned comment strings more pleasant.


pin config links for v0.6 to v0.6-branch

Fixes https://github.com/kubeflow/website/issues/1070
/hold
Until the existing_arrikto rename is cherry picked in v0.6-branch: https://github.com/kubeflow/kubeflow/pull/3884
/assign @sarahmaddox
/cc @abhi-g @lluunn
Signed-off-by: Yannis Zarkadas<EMAIL_ADDRESS>

This change is

Thanks for updating this Yannis. Please see my comment about using a shortcode variable. That way with the branching of the site, it will auto update to the correct branch.

Since we are pinning to a specific tag, there is no need to wait for kubeflow/kubeflow#3884
That change will be picked up in v0.6.2.
@sarahmaddox @cspavlou I am reverting the filename back to kfctl_existing_arrikto.v0.6.yaml because that's the name in v0.6.1.
When v0.6.2 is out, we should make it kfctl_existing_arrikto.yaml again.

Preview:

https://deploy-preview-1071--competent-brattain-de2d6d.netlify.com/docs/aws/deploy/install-kubeflow/
https://deploy-preview-1071--competent-brattain-de2d6d.netlify.com/docs/started/k8s/kfctl-existing-arrikto/
https://deploy-preview-1071--competent-brattain-de2d6d.netlify.com/docs/started/k8s/kfctl-k8s-istio/

Thanks!
/hold cancel
/lgtm
/approve


Project model

Adds Admin::Project model. It has name, landing page slug and landing page content at the moment.
Main features:

activities, external activities and investigations can be assigned to project.
material search results can be filtered by project.
project can have its own landing page, available at custom ULR defined by slug (portal.concord.org/<project-slug>).

Hopefully it's well tested, including cucumber tests.

This is looking great.

Thanks Noah!
@psndcsrv, do you want to double check or should I merge?

That extra table definition in db/schema.rb was the only thing I noticed. Everything else looks good!

Thanks! Fixed, so I'm merging it.


Pass slug of newest live dos framework to index

For this story on Pivotal: https://www.pivotaltracker.com/story/show/139283877
Currently the home page has links to create DOS opportunities for the different lots. The framework slug used in the url is hardcoded to digital-outcomes-and-specialists.
When DOS2 goes live this slug needs to be digital-outcomes-and-specialists-2.
This PR updates the buyer app to check for all live DOS frameworks and passes the slug of the most recent (judged by the greatest ID) to the index template to correctly render the links. This handles the edge case of when we have two DOS frameworks live (the short period when transitioning that DOS1 is live at the same time as DOS 2).
FUNCTIONAL TESTS:
Those on the master branch will break when we transition, but this is due to briefs that are created as part of the flow being on the DOS1 framework. Can easily be updated.
Those on the jenkins branch also break a bit. I'm investigating why...
UPDATE:
Functional tests will break on master and live only when DOS 1 is expired. They'll be fine of DOS1 and DOS2 are live. They break when DOS1 is expired due to a mix of being unable to create briefs on an expired framework, and hardcoded references to dos1 links (especially in the admin app). Should be fairly straightforward to fix, I've got a list on my machine of where/why they're breaking.

👍


Toolkit: bail out early if credentials are not defined

Seems like currently even if there are no credentials defined in the default chain, the toolkit will still execute Looking up default account ID from STS which will time out and only then a message is displayed that credentials are not found.
We should be able to bail out early if there are no credentials defined in the chain (we should make sure that plugins can still do their job)

This might be useful for an optimization:
https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/identify_ec2_instances.html
https://docs.aws.amazon.com/AWSEC2/latest/WindowsGuide/identify_ec2_instances.html
On Lambda we get environment variables prepopulated.
Don't know about ECS.

I've just run into this issue myself (from #702), I tend not to have default creds set, but use a profile (which I hadn't specified), takes forever to eventually get the timeout from <IP_ADDRESS> being unreachable.
A couple of questions:

As mentioned above, could we detect in a better way when in a 'cloud like' environment, and only then attempt to use the 'magic IP'
Could we turn down the timeout on the connect request to something smaller (eg. 5sec), which would presumably be more than enough to hit an 'internal address' when running on a cloud-like environment
How does a command such as aws sts get-caller-identity handle this sort of situation? As for me, it figures out instantly that I don't have creds defined:

⇒  aws sts get-caller-identity
Unable to locate credentials. You can configure credentials by running "aws configure".