Influxd remote restore: error updating meta: Response did not contain the proper header tag.

Both hosts (local and remote) are using the InfluxDB 1.6 Docker container.
Steps to reproduce:

Deploy an InfluxDB container on a remote instance
Deploy an InfluxDB container on your local machine
Create a backup on your local machine
Restore your backup to the remote instance: influxd restore -portable -host $hostmachine:$hostport $(pwd)

Expected behavior: Backup is restored on the remote instance
Actual behavior: The following error occurs:
2018/10/18 07:45:41 error updating meta: Response did not contain the proper header tag.
restore: Response did not contain the proper header tag.

Configuration issue on my end.

For me it was one line that was commented in influxd.conf
# Bind address to use for the RPC service for backup and restore.
# bind-address = "<IP_ADDRESS>:8088"

I removed "# " from the "bind-address" line and replaced the <IP_ADDRESS> address with hostname.
In my environment hostname is resolved to the correct IP address, YMMV.
Restarted influxd and after that I could restore remotely.
Why I left this comment? I wanted to unfrak this error message

For what it's worth, solution was pretty much the same for me. In my case I was using a Docker container, in specific the Docker Compose sandbox, where port 8088 needed to be opened manually for backup/restore. The default port 8086 for the API was already OK.


Documentation on Hackage is wrong

For some reason, the hscolour-ified source code on Hackage is wrong; e.g. the source for the SP module still has the old version of spfoldr (even though the actual Haddock documentation is correct).
On a related note, the tag in the source-repository this section needs to be updated.

This should be satisfied by #8


improve name assignment

One of the fundamentals of variables is that they are stored before they're loaded.
This can happen implicitly in the setup of frames or explicitly through STORE_FAST (for locals).
This PR does two things:

it looks at assigning names when variables are set. This implicitly prefers outer scopes for variables that are passed to calls, which is good.
as we also have occasions where the variable is not passed to calls but obtained otherwise, we avoid internal functions (e.g. when a variable is obtained though an attribute or item lookup).

Quite likely, it would be even better to make a stronger preference for outer scopes, maybe by overwriting name assignments if the stack depth of the current assignment is smaller than the previous one. I'll leave that for the next PR though.

Does this PR have any effect on the argument names for the computation trace from https://github.com/Lightning-AI/lightning-thunder/issues/657#issuecomment-2208317779? The expected variable names were suggested here: https://github.com/Lightning-AI/lightning-thunder/issues/657#issuecomment-2274264003.

Yes, it becomes
def computation(param, t_0_1, step_t, t_1_1):

Again, this isn't "the full solution", but to my mind, it's a strict improvement.


[WIP] add support for link|linkat

Please first review https://github.com/nix-rust/nix/pull/754

see https://github.com/nix-rust/nix/pull/751#issuecomment-333878207


Force update of iframe height after click.

Is there a method of calling a function onclick that can reset the iframe size based on current content?
I have used the scroll to function which has worked a treat, however the iframe height is then reset to 0px. When i resize the browser it then shows the correct height.
I was hoping to call another function to reset the height. or determine what is causing the iframes height to be reset to 0px

Hi James, same problem for me, but in diferent context, i open a iframe inside a bootstrap modal and this script didn´t work, mabye some conflict with bootstrap css or js. So, looking for internet finally find two lines of code that did the work(but without this script). hope you can solve your problem
function resizeModalFrame(idFrame) {     fheight = document.getElementById(idFrame).contentWindow.document.body.scrollHeight;     $('#'+idFrame).css('height',fheight+'px');  }
then when i open the modal with bootstrap i call this function. Same thing when i change a combo option and get dinamic content throught ajax(after put dinamic content via innerHTML), and works perfect.

Hi coyarzun2013 I dont think this will resolve my issue, the iframeresizer script is doing exactly what i need it to however the content within the iframe is changing shape which is why i need to amend the height of the parent iframe from within the iframe on click.

Just call the parentIFrame.size() method. See docs

Hi David, I have called the scrollto method but the iframe height is then
reset to 0 px. Upon physically resizing the browser the iframe height is
updated. I have tried the following:-
window.parentIFrame.scrollTo(0,0);
parentIFrame.size();
return false;
but the iframe height doesn't refresh.
Your script is working perfectly, and has solved a massive issue that we
have had for a number of years.
however due to how well this script is working, when our content changes
people can no longer see shorter content, hense trying to get the parent
window to scroll to top.
Your advice is greatly appreciated.
On Wed, 20 Apr 2016, 22:00 David J. Bradshaw<EMAIL_ADDRESS>wrote:

Just call the parentIFrame.size() method. See docs
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/iframe-resizer/issues/344#issuecomment-212604257

It's most likely a CSS issue in you code, but odd that resizing the browser fixes things. Impossible to say what without seeing it.  I will test fully tomorrow. Thank you for your advice so far.
I could pm you across a link but as it is a production server I would prefer not to send publicly.

Sadly this doesn't fix the issue. On a mobile the iframe height also disappears once window.parentIFrame.scrollTo(0,0) is called. Once the device is rotated, the iframe resizes to the content height.

Can you post the entire log output

Closed due to no response


More Precise Exception Handling for Item Rename Operation

As per this documentation, we now handle the following cases during the rename operation to provide specific user feedback through the standard content dialog:

ArgumentException - equivalent of E_INVALIDARG
PathTooLongException - equivalent of ERROR_FILENAME_EXCED_RANGE
FileNotFoundException - equivalent of ERROR_FILE_NOT_FOUND

I may have messed up the resources, so I'll have to revert the merge conflict resolution when I get a chance.

/azp run


weston: Refresh no-input-device patch for Morty

[Temporary fix; already merged in OE-Core Morty's staging.]
The included patch, backported from Weston master (and OE-Core
master since Weston 1.11.1, b6864b1), allows it to run without
any input device at launch. An ini option is introduced for
this purpose, so there is no behavioral change.
Related change in weston.ini:
[core]
require-input=true
Default is true; setting it false allows Weston to run
without a keyboard or mouse, which is handy for automated
environments.
Signed-off-by: Daniel Díaz<EMAIL_ADDRESS>

Can you queue up a new PR or revert for when it gets out of staging?


Feature Request: Read information from embeds

Chat stream does a bang-up job of reading information from notes added to a canvas. However, sometimes canvases are put together with references to just one section of a note, by creating a card and referencing the section like this:
![[My Cool Note#Chapter 1]]

Within the canvas, this has the effect of rendering that part of the note in the card. However, it seems Chat Stream doesn't have any current way of resolving those details, so they're quietly left out of the context handed to the LLM. It would be neat if these links within cards were supported.

Fixed in 1.4.3. Thanks for the request!


Add take methods to DataArray and Variable

These are often way faster than numpy's array indexing. The following works for xray.Variable:
def take(self, indices, dimension):
    axis = self.get_axis_num(dimension)
    values = self.values.take(indices, axis)
    return self.__array_wrap__(values)

NumPy's indexing methods now have comparable speed to take. This wouldn't be very useful and would just clutter the API.


Anchor tags are stripped from URLs

Configure a beacon to have a URL with a trailing anchor tag, e.g. 'http://cnn.com#test'
Find the beacon in the Physical Web app, you'll see CNN but notice that the URL looses the tag
Just to be sure, click on the URL and you'll notice that you go to cnn.com

Turns out that if your url ends in the form "...com/x#test" this is the fail case. If you change it to "...com/x/#test" it then works. So the bug isn't stripping ALL anchor tags

It might be the web server redirecting us.  Clicking the CNN link above in desktop Chrome actually navigates to http://www.cnn.com/#test.  I get the same behavior for other sites:
http://cnn.com#test      -> http://www.cnn.com/#test
http://www.cnn.com#test  -> http://www.cnn.com/#test
http://jenson.org/m#0000 -> http://jenson.org/m/#0000

Chrome is smart enough to preserve the anchor tag but it looks like PWS is dropping it when the URL is resolved.

Indeed:
$ curl -k -s -H "Content-Type: application/json" -d '{"objects":[{"url":"http://cnn.com#test"}]}' http://url-caster.appspot.com/resolve-scan | python -m json.tool
{
    "metadata": [
        {
            "description": "View the latest breaking news and daily news for U.S., world, weather, entertainment, politics and health at CNN.com.",
            "displayUrl": "http://www.cnn.com/",
            "icon": "https://url-caster.appspot.com/favicon?url=http%3A%2F%2Fi.cdn.turner.com%2Fcnn%2F.e%2Fimg%2F3.0%2Fglobal%2Fmisc%2Fapple-touch-icon.png",
            "id": "http://cnn.com#test",
            "rank": 1000,
            "title": "Breaking News, Daily News and Videos - CNN.com",
            "url": "http://www.cnn.com/"
        }
    ]
}

Okay, I think Matt is right.  This is just an interesting case of our url fetcher dropping the hash thinking it isn't needed.
When I set a goo.gl short link to redirect to cnn.com#test, I get this curl reply:
curl -v http://goo.gl/ucDOPb
... (TEXT OMITTED)
< HTTP/1.1 301 Moved Permanently
... (TEXT OMITTED)
< Location: http://cnn.com/#test

But when I curl cnn.com/#test directly, I get:
curl -v http://cnn.com/#test
... (TEXT OMITTED)
< HTTP/1.1 301 Moved Permanently
... (TEXT OMITTED)
< Location: http://www.cnn.com/

goo.gl is returning a hash even though it won't affect the content you fetch, because its trying to actively inform the browser that the user intended to leave the hash in.
ccn.com is dropping the hash because it shouldn't affect the content being fetch.
The python/GAE urlfetch util we are using is apparently not doing anything special to track hashes across redirects, and just blindly following Location: headers.
I think in order to fix this we may need to manually append the hash at each redirection if it hasn't been preserved.  Alternatively, we may only need to append the hash in the final REST reply, since technically I think the hash should not affect the page content returned from a fetch (unlike get variables).

Is it clear we need to work around the foibles of a redirector? We could just say 'don't use a redirector that drops anchor tags' I'm a bit out of my depth but I'm concerned we may be working too hard to fix this. Once I understoond that my WP installation was redirecting, I was perfectly fine using "/#tag" for my URL

So I think I have a fix which will do, but since it changes the way we return urls we should test with care.
Our existing tests pass and my light pass of manual tests seem ok, but I'd only push this to dev PWS if accepted and try to get everyone to test it out locally for a while.
(As an aside, this work helped me uncover another bug with groupid which I patched.  Thankfully that wasn't pushed to the stable server yet, but it shows that we aren't testing DEV enough).

Actually, I think I may have introduced a bug.  This PR will always persist the initiall fragment, but that breaks redirectors which introduce new fragments.
I'll change the PR to only overwrite the fragment if we end up without a fragment, and started with something.

Using /#tag vs #tag for the URL does not appear to make a difference, at least not for cnn.com.
Fixing the goo.gl case is actually more intricate than I expected, because we need to now introduce hash at each redirection level and not just at the start..

PR has been updated, now works with http://cnn.com#test,  http://goo.gl/ucDOPb,  and http://www.cnn.com/#test
$ curl -k -s -H "Content-Type: application/json" -d '{"objects":[{"url":"http://cnn.com#test"},{"url":"http://goo.gl/ucDOPb"},{"url":"http://www.cnn.com/#test"}]}' http://localhost:8080/resolve-scan | python -m json.tool
{
    "metadata": [
        {
            "description": "View the latest breaking news and daily news for U.S., world, weather, entertainment, politics and health at CNN.com.",
            "displayUrl": "http://www.cnn.com/#test",
            "groupid": "1af1c209dba64b7c",
            "icon": "https://url-caster.appspot.com/favicon?url=http%3A%2F%2Fi.cdn.turner.com%2Fcnn%2F.e%2Fimg%2F3.0%2Fglobal%2Fmisc%2Fapple-touch-icon.png",
            "id": "http://cnn.com#test",
            "rank": 1000,
            "title": "Breaking News, Daily News and Videos - CNN.com",
            "url": "http://www.cnn.com/#test"
        },
        {
            "description": "View the latest breaking news and daily news for U.S., world, weather, entertainment, politics and health at CNN.com.",
            "displayUrl": "http://www.cnn.com/#test",
            "groupid": "1af1c209dba64b7c",
            "icon": "https://url-caster.appspot.com/favicon?url=http%3A%2F%2Fi.cdn.turner.com%2Fcnn%2F.e%2Fimg%2F3.0%2Fglobal%2Fmisc%2Fapple-touch-icon.png",
            "id": "http://goo.gl/ucDOPb",
            "rank": 1000,
            "title": "Breaking News, Daily News and Videos - CNN.com",
            "url": "http://www.cnn.com/#test"
        },
        {
            "description": "View the latest breaking news and daily news for U.S., world, weather, entertainment, politics and health at CNN.com.",
            "displayUrl": "http://www.cnn.com/#test",
            "groupid": "1af1c209dba64b7c",
            "icon": "https://url-caster.appspot.com/favicon?url=http%3A%2F%2Fi.cdn.turner.com%2Fcnn%2F.e%2Fimg%2F3.0%2Fglobal%2Fmisc%2Fapple-touch-icon.png",
            "id": "http://www.cnn.com/#test",
            "rank": 1000,
            "title": "Breaking News, Daily News and Videos - CNN.com",
            "url": "http://www.cnn.com/#test"
        }
    ]
}

This bug still seems to exist.. Using the Goo.gl shortner, I can create a shortened URL that does appear to contain a #tag, but through physical web, the #tag is dropped.  The PW validator OK's the URL as well... I am using Chrome on iOS 10.


Add .d.ts

Thanks for the continuous OSS development.
Is your feature request related to a problem? Please describe.
I use react-datepicker along with typescript in my work. However, @types/react-datepicker is currently in a separate repository, so new features of react-datepicker are not immediately available when they are released.
Describe the solution you'd like
I would like to distribute d.ts files in this repository so that the latest react-datepicker functionality is available in typescript without @types/react-datepicker.
Describe alternatives you've considered
Wait for the latest @types/react-datepicker to be released. Or create a PR.
We also considered rewriting this entire repository to TS, but due to the scale of the project, we felt it would not be practical.
Additional context
We need to introduce a check of d.ts by CI with reference to @types/react-datepicker.

refs #4000

refs #4700


#81: Fix error handling in body writer

The error handling in the msgWriter.writeBody() method was not working properly. We basically overwrote the mw.err with nil if the function that followed after a failed write attempt was successful again
This patch fixes #81

Codecov Report

Merging #82 (47eb7b5) into main (1ff87db) will decrease coverage by 0.88%.
The diff coverage is 52.00%.

@@            Coverage Diff             @@
##             main      #82      +/-   ##
==========================================
- Coverage   83.85%   82.96%   -0.89%
==========================================
  Files          13       13
  Lines        1381     1397      +16
==========================================
+ Hits         1158     1159       +1
- Misses        153      163      +10
- Partials       70       75       +5

Impacted Files
Coverage Δ

msgwriter.go
86.08% <52.00%> (-4.57%)
:arrow_down:

client.go
73.58% <0.00%> (-0.81%)
:arrow_down:

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


Update to 1.10 config.example error

Issue Summary
Cant start ghost without config.example.js file after upgrade.
Steps to Reproduce

upgrade ghost js
start ghostjs
ghost failing to start
reupload config.example.js
starting ghost normally

I allready have my old config.js
Technical details:

Ghost Version: 1.10
Node Version: latest 4
Browser/OS: Internet Explorer 6
Database: flat

Hey @dyanakiev Sorry to hear you are having trouble and thanks for reporting.
This is caused by our submodule gscan, which uses ghost-ignition.
The ghost-ignition module requires a config.example file, see https://github.com/TryGhost/Ignition/blob/master/lib/config/index.js#L4.
I will create an issue in the ghost-ignition module.

FWIW as a short term fix, I worked around this by ensuring the CWD (WorkingDirectory in a systemd unit file) was set to the Ghost subdirectory (in my case, roughly, a setting likeWorkingDirectory=/opt/ghostd-wrapper/node_modules/ghost/ :)).

Closing, because we have fixed this issue in ghost-ignition and updated gscan to 0.0.13, see https://github.com/TryGhost/Ghost/pull/7328


code with pure js instead of coffeescript?

Hi, sorry if i'm not seeing this in the docs/closed issues but is there a way to code roots with pure js instead of coffeescript?
Much thanks in advance :)

almost forgot to ask, is there support for babeljs or es6?

Hey @adampotocki you can convert all of your scripts into pure javascript with the expection of app.coffee (if i'm not mistaken). If you want to use any other languages within roots checkout accord. Babel is supported in accord so you can use it!

Adding a little clarity, roots uses accord to compile everything. All you need to do is install the language you'd like to use via npm and roots will automatically pick it up and compile it for you :grinning:

now way :smile:! thank you!

No problem, if you have any other questions let us know! :smile:

I tried to add assets/js/main.js and inside main.js I put console.log("heyy"); but it doesn't work at all, am I missing something?

@christiansakai did you load the file on to your page in the html?

@jenius Yes I did, in fact, main.coffee is automatically converted to main.js am I right? the index.html's main.js is already there from the main.coffee

@christiansakai yes it is. You can check the public folder if you want to see the compiled files. Probably there was an issue with the path. Have you checked the console?


Ford Account Lockout

Hi all,
It seems Ford has started cracking down on third party services taking advantage of their apis.
Recently I was locked out of my account and after speaking to Ford support they told me it was because of an unauthorized third party service using my credentials, which would be connected-car SDK's.
I am curious if anyone else has experienced this issue?

I've encountered the same issue with multiple accounts. Calling Ford's APIs from a scheduled job is a sure fire way to get locked out...
Did Ford unlock your account?

I've encountered the same issue with multiple accounts. Calling Ford's APIs from a scheduled job is a sure fire way to get locked out...
Did Ford unlock your account?

Not yet...they've asked me to contact the developer team directly. In your experience did they have you reach out to the development team?
I believe the reason for the account lock was attempting to login and generate an access token and refresh token too quickly. I use iOS shortcuts on my phone to start my truck, instead of building the use refresh token login, I just generated a new access token every time I wanted to run a command like start, lock, unlock, etc.

Not yet...they've asked me to contact the developer team directly. In your experience did they have you reach out to the development team?
I believe the reason for the account lock was attempting to login and generate an access token and refresh token too quickly. I use iOS shortcuts on my phone to start my truck, instead of building the use refresh token login, I just generated a new access token every time I wanted to run a command like start, lock, unlock, etc.

Apparently, support was communicating with the dev/security team. I was told that they could not unlock my account, so I requested it be deleted so that I could just remake the account. Support replied and said my account had been deleted and closed the issue, but my account was not deleted... It's been crickets ever since.
I also use iOS shortcuts in the same manner, but I don't believe they're the culprit for me.

Yeah, I was communicating with the support team as well but then they told me to reach out to the development team to speak more about unlocking my account.
I went ahead and tried to find a solution and it seems like updating the User-Agent to the latest version did the trick in my case.
I created a new account, used the latest FordPass UA and then redid my iOS Shortcuts to not force a new login every time.
So far so good.

I'd guess you're probably right about the login/User-Agent being the problem. Multiple login attempts coming from an outdated UA definitely looks suspicious.
Let me know if you're able to get Ford to unlock your account. I'd like to get my original account back...


ggshield: 1.32.2 -> 1.33.0

Diff: https://github.com/GitGuardian/ggshield/compare/refs/tags/v1.32.2...v1.33.0
Changelog: https://github.com/GitGuardian/ggshield/blob/1.33.0/CHANGELOG.md

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354912

x86_64-linux

:white_check_mark: 2 packages built:

ggshield
ggshield.dist


Tests fail due to

Reported by Greg Coogan:

Getting this error when running tests in Production on “AddCampaignMembersByReportBatchableTest”:
System.UnexpectedException: No more than one executeBatch can be called from within a test method. Please make sure the iterable returned from your start method matches the batch size, resulting in one executeBatch invocation.
Stack Trace: External entry point
Do we have a fix?

Yes, that test class runs the report Add_Contacts_To_Campaign_Test. You will need to modify its filters to ensure only a handful of records are found.
Unfortunately, with the Analytics API in Apex the tests always run with “seeAllData = true” regardless if we want it to or not. To control the amount of data the test crunches on you can widen or narrow the filters on that report.

I'll look at modifying the batch class to check if Test.isRunningTest() and ensure iterable only has no more than one execute's worth of records.

What do you mean by "modify its filters to ensure only a handful of records are found."?  The report only pulls 1 record for me, so I'm not sure how to narrow the results down any further.
Thanks!

Hi @kristin44hubbard,
One reason why the unit test may fail is if the batch job executes more than once (testing limit imposed by Salesforce). When the test runs it runs with access to all data so my hypothesis is that despite when you look at the report in Salesforce and see only 1 record that when the test runs it might see 201+ records.
Can you confirm if the org physically has more than 1 contact, or more specifically, more than 200 contacts?
If it does have more than 1 contact, customize the test report to have criteria that ensures that only 1 contact would ever come back in the report.
Make sense?

@DouglasCAyers Thank you so much for the explanation!  I updated the report to only pull one contact record, but I'm still getting the errors:
System.AssertException: Assertion Failed: Expected: 2, Actual: 1
Stack Trace: Class.AddCampaignMembersByReportBatchableTest.test_batchable: line 98, column 1
AND
System.UnexpectedException: No more than one executeBatch can be called from within a test method. Please make sure the iterable returned from your start method matches the batch size, resulting in one executeBatch invocation.
Stack Trace: External entry point
Method Name: test_batchable
Class Name: AddCampaignMembersByReportBatchableTest


Fix: "Failed to create MMKV instance" after reload/fast refresh with Expo Dev Client

Fixes #576 and #566

Please review and merge @mrousavy 🚀


Add easy to use examples to webapp

Create buttons on top of the text field on the webapp that allow users to try out the site with premade examples.

We could try having examples for a news article, a school essay and something else. I can help source these if you'd like.


Make the "opensource" tag more lightweight

The current "opensource" tag (located in //src/licences/opensource.toml) is rendered as a big chunk of text by Discord, because it is too "big".
I think it would be easier and clearer for users to simply remove the brief definition of an open source license and, instead, adding a link to the formal definition from the Open Source Initiative (OSI) (https://opensource.org/osd)
This would also make it possible to add documentation about the differences between "open-source" according to the Free Software Foundation (FSF) and OSI

I will work on this tomorrow (I think), let me known if you have suggestions to improve the content of the tag


Z3 as a sort

This is an initial pass at making Z3 available from inside egglog. I thought it wise to seek comments / advice before continuing to fill it out because I had to perform violence in a couple places.
The high level z3 bindings expose a lifetime parameter that associates the datatype with the lifetime of the z3 context. I found it hard/impossible to get that to work while keeping an indexset of the z3 asts inside the Z3Sort.
There are a couple choices:

Trying to thread Z3Sort<'ctx>. This was not working. Since Z3Sort needs to hold a reference to Context, there is a circularity in the data structure.
Rebuild the high level z3 bindings from z3_sys to allow for hashmap storage of asts. This will largely be highly replicated from what the z3 crate already does
Use 'static to remove the lifetime parameter from z3::ast::Dynamic. mem::transmute will be needed at a few certain spots because the context is not actually static. I don't see how these asts could escape the lifetime of the context as used here, but this doesn't feel that good. There could also perhaps be a memory leak.
Actually have a static global Z3 context available something like Symbol.
Don't use bindings at all. Construct ast/s-expressions and call smt solver via a string interface. This has the advantage of safety, parallelism, and extending to solvers beyond z3. Replicatng an smtlib ast is tiresome though. This also may enable using  regular egglog datatypes.

Some other considerations:

Is it better to have a sort Z3Sort, Z3Bool, or (Z3 bool). One does want to define user defined sorts in z3 eventually, which it is hard to see how that latter two could support.
Should one always call simplify before hashconsing the z3 expression. This may improve termination and deduplicate the database.
Parallelism and non blocking. check-sat calls are going to become very expensive. I note that an interesting possible capability of egglog is using eids as promises/futures from external systems.
I also added unsafe Sync and Send traits for the Z3Sort. I am uncertain what Z3's promises are in regards to thread safety nor how to protect it if it isn't thread safe. My understanding is that egg-smol is currently not multi-threaded.
z3 should be guarded under a feature at least since it will probably be too annoying to get the wasm build to work. Or perhaps some mechanism to install third party sorts without having to fork the code base.

I would prefer to keep this as a separate crate until we think it's applicable to package it with egglog under a feature. That said, egglog should definitely have featureful enough extensions to do this, so we should make enough things public that it's possible to do that.
Regarding the options above, I think the best thing think is to hide the lifetime and just use 'static and some unsafe code. This is a pretty common pattern, and it's a little cleaner than mucking around with raw pointers. We'll have to do something to prevent these things from moving across threads though.

Makes sense. I'll close this.
A thing I don't like that much is if I make this an external crate that uses egg-smol as a library, that crate will need to create it's own copy of the executable already in egg-smol. Then for further capabilities, you'll need to write wrapper executables that add in all the capabilities you want.
The design space as I understand it is:

egg-smol accepts all kinds of weirdo extras. This creates a lot of maintenance burden and bloats dependencies, but also is uncomplicated and kitchen sink for users. Extras could be guarded under cargo.toml features.
Some mechanism (?) in egg-smol to dynamically load extra features (by finding some kind of .so on a path perhaps? importing some wasm file with some kind of interface?). Souffle user-defined functors are somewhat like this.
egg-smol as a library and external crate libraries and then if you want to combine features, you need to write a small executable wrapper project. This is actually fairly simple, but maybe to make it even easier some of the functionality in main.rs should be pushed into the library to avoid duplication. The more I think about it, this is probably the right approach. If someone wants to, they can maintain a kitchen sink distro.

I like option 3. It’s the closest to what egg did as well.


Add support for AdditionalDateClassesFunc to MudDateRangePicker

I also fixed the rather bad formatting of SimpleMudDatePickerTest.razor, hope that is ok. The date range one was fine 🤷♂
I can squash the commits if required - thought it might be preferred to have them separate?
Note DatePickerTests.AdditionalDateClassesFunc_ClassIsAdded() was a false positive test before - it was passing because it wasn't actually doing any checks.
Description
Fixes #6202
Allows for using AdditionalDateClassesFunc to apply additional classes to date picker days.
How Has This Been Tested?
New tests
Updated existing
Visually (uncommitted)

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[X] I've added relevant tests.

Why does the @if (IsDateDisabledFunc != null) exist? It is perfectly fine for the IsDateDisabledFunc to be null? This just duplicates the markup for no reason. Same in the Range one. Is this just a relic from a previous version where that check was required?

When will this bugfix be released?

Fixed conflicts in the PR.

Oh sorry I saw your comment but forgot to address it when I got home.

Thanks @EricEzaM!


Bug when saving a grade, the grade can disappear

I think this may have something to do with the new saving mechanism (or the ONE_PASS_RENDER).
Sometimes the message gets stuck on
Saving...
and the disk icon or green tick appears, then the grade disappears.
In the log, newGradeAdjusted gets reported as null which means that the blank grade was sent to the backend service for saving, even though I type a number.
Hard to reproduce, may just be some code out of sync in my browser.

On second sitting, I can't seem to replicate this one.  @steveswinsburg, have you seen it pop up again?

Not lately but will let you know if I do 👌

Ok this is now happening constantly on Firefox. Doesn't appear to be doing it in Chrome.
What happens is I enter a grade then press enter/click away, the grade disappears and the Saving.... stays there.

Boo.. I'm not seeing that locally or on sandbox1 using Firefox on both OSX or Linux.  Let me guess.. are you on Windows? I'll see if I can track down a VM.

Windows?  I'm on OS X. I'll try a different computer
and report in.
On Wed, May 25, 2016 at 1:55 AM, Payten Giles<EMAIL_ADDRESS>wrote:

Boo.. I'm not seeing that locally or on sandbox1 using Firefox on both OSX
or Linux. Let me guess.. are you on Windows? I'll see if I can track down a
VM.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/sakaiproject/sakai/issues/2531#issuecomment-221317522

Ha! :)  Thanks

Haven't seen this for a while, may have been an local environment issue.

Closing this one out.


Query port wrong

Hey, you set the query Port on line 107 as "?Port=" instead of "?QueryPort="   ...
so both Ports are set as "?Port="

fixed


Allow bus name per NextForwarded attribute

In some cases when command is dispatched into multiple buses, in one case it makes sense to forward for next processing, while in other - not to do so.

It would make sense not to use the attribute in this case, but return instance of NextForwarded from the conversion handler instead


Beta endpoint

Added an '/beta' endpoint to all current schemas for data of an ongoing beta version of elite.

This might not be retro-compatible with listener just checking for test part in schema?

But which listener does only listen to test and expects beta data? I think a clear distinction between actual beta data and just a simple test message outweigh a maybe break in old software.
Also a test message may be syntactical correct but the content might not be.

I was actually thinking about the reverse.. :D
Listener that just check for "test" in schema header to reject them, before any other proceedings.
I'm not against it, but repercussions needs to be checked.
At least on the big three, and may be wait after the actual beta not to screw anyone, and it's in place or next beta.
This will also need to readme edit and wiki? Need to check once at the office!

Listeners should normaly reject it with some sort of 'schema unkown' otherwise the listener is severely broken anyway.
The readme is just a link to the wiki and I don't know how to include the wiki into a pull request (I would have done it otherwise). Is that even possible?

Why not bump the version of the schema? I have heard of people asking for a schema bumping in the EDCD server a few times now. This will also let people migrate over slowly. I like the idea of having a beta endpoint separate from the test endpoint.

It's just a new endpoint, no schema change is done in itself, so no version bump.

Yeah, but I understand why this could break existing stuff. If someone is filtering out the test endpoint instead of listening to the non test endpoint particularly, like I am doing, they will start getting beta data which can cause a lot of trouble. Hence my recommendation to bump the version.


It's time to create a middleware for CORS, I will create it when I have time.

I like the issue name 👍🏻


Code cleanup

Simplified the code with relative paths instead of the Python pathlib module (I know that the pathlib module is the safer way to do paths, but the way I had it when I joined them negated it anyway because it would still mess up any Windows installs).

Do I need to do this?


Add machinery for Python bindings generation

@davidegorbani what is the status of this activity?

I haven't done it yet; for the moment it is not a high priority issue so I will work on it in the future.

Starting from next week we start implementing the bindings

First PR: https://github.com/ami-iit/biomechanical-analysis-framework/pull/48

The bindings for the HumanIK class were added with #52, I will add the bindings for the dynamics with a new PR.

Closing as the bindings for the class HumanID were added with #53.

CC @LudovicaDanovaro


Add an FAQ section for incoming webhooks

Whoops. I just read further in the customer channel and saw @icelander you were gonna work on this. Feel free to edit, or resubmit a separate PR if you had something else in mind

Just to add - this is good, but we may want to specifically note how to send as a DM to yourself.
For our one customer's use case, it sounds like they have the webhooks created by the Admin, and as a result it's being sent to the DM channel between the admin and the user instead of the user's personal DM channel. I'm not sure how common that is, but it might be an issue for teams where creating integrations is restricted to admins.

Maybe @icelander you can help with Lindsay's feedback?

I created a new pull request with this information that includes how to send to a specific DM channel: #1907

Thanks @icelander - is it ok to close off this PR from JB then?

https://github.com/mattermost/docs/pull/1907 covers this


Fixed #2822, #3013, #3126, #3015, #2826

Positive Test Case

[Enter positive test case here]

Fixes #2822
Fixes #3013
Fixes #3126
Fixes #3015
Fixes #2826
Fixes #3054

@Lizabrazil can you quickly verify/approve this PR?


Got HTTP error [code=400] When Downloading the problems

Got HTTP error [code=400] When Downloading the problems

Downloading problems
Downloading questions titles
[ERROR] http error [code=400]

I found a bug in leetcode.cn.js->plugin.getProblemsTitle->opts.body->query, maybe this argument is wrong. The response from http servers is:
{
  errors: [
    {
      message: 'Cannot query field "question" on type ' +
        '"AppliedTranslationNode". Did you mean ' +
        '"questionId"?',
      locations: [Array]
    }
  ]
}


rdkit-sys

I am on a Mac and I want to support building specific version of RDKit for various platforms (MacOS ARM/Intel, Linux ARM/Intel). So I have under taken a path to easier integration with rdkit-sys, check it out: https://crates.io/crates/rdkit-sys

I am going to politely steal your cffi ideas, it's all new to me but I think I can figure out how to get fully cargo-ify it.

happy to report I used your code well in the repo. now I have a self-building crate with your very convenient molecule implementation on top of the low level bindings.

HI xrl,  I am glad you find the code useful! Actually, I planned (and also started) to have something like an rdkit-sys in order to get rid of the compiled .so in the repo but did not find the time to finish this. Pretty cool that this is now available!

Hi xrl, I am glad you find the code useful! Actually, I planned (and also started) to have something like an rdkit-sys in order to get rid of the compiled .so in the repo but did not find the time to finish this. Pretty cool that this is now available!


Make install docker friendly

Fixes: https://github.com/replicate/cog/issues/1783
Adds venv to gitignore (not strictly part of this issue but noticed it on my system)
Add support for specifying the INSTALL_DIR via environment variable outside of the script to prevent a read
Allow the user to specify SUDO_CMD via environment variable outside of the script incase they are operating on a docker that doesn't contain it
Downgrades the docker run check to a warning, in docker this can't be running. I also feel this should be a runtime check rather than an install check.
Updates the docs to add a docker install instruction.

Thanks, @8W9aG! This is a nice improvement. Only change I had was renaming SUDO_CMD to SUDO to match the convention in overrides for Makefile. Lemme know if you think this would cause a problem for anyone.

Not at all, great change! I think it's ready to be merged now so let me know if its looking good.


Upgrade to go 1.20 and update required dependencies + point to compatible MCAD

What changes have been made

Updating the codeflare-operator's dependencies and go version to 1.20 to solve a High CVE.
There is a single function used by MCAD where the signature is changed due to these updated dependencies, causing conflicts and errors. This PR will also point the CFO to use the newly created branch in MCAD with the updated dependencies and signature updates.
PR in MCAD repo to be merged in a new branch named go-1.20 and to be consumed by the CFO: https://github.com/project-codeflare/multi-cluster-app-dispatcher/pull/697
Verification steps

e2e tests should pass.

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests

/hold until this PR is merged: https://github.com/project-codeflare/multi-cluster-app-dispatcher/pull/697 and I have updated the CFO to point to that branch.

/lgtm

/lgtm

/unhold all tests are now passing.


XMLHttpRequest Keeps Failing In React Native 0.64.0

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
Since I upgraded to RN 0.64.0 Ajax a lot of the times fail to make a successful connection to my API, I'm currently using Axios and have two different versions installed on two separate RN 0.64.0 projects and they both behave the same way, a new project is using Axios 0.21.1 while an old one using Axios 0.19.2, I've been facing this issue for the past 4 weeks and have not been able to figure out what's going on or maybe I don't know where to look, please can anyone help with either a debugging technique or if they've experienced this and have a solution, share it here? This happens on both iOS and Android.
I'm also thinking this might be related to Flipper but I'm not sure, have removed all installations and initialization of Flipper in the project but the issue persists.
React Native version:
System:
    OS: macOS 11.3
    CPU: (8) x64 Intel(R) Core(TM) i5-8279U CPU @ 2.40GHz
    Memory: 33.72 MB / 8.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 12.16.1 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.13.4 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.1 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
    Android SDK:
      API Levels: 23, 28, 29
      Build Tools: 28.0.3, 29.0.2, 29.0.3
      System Images: android-24 | Google Play Intel x86 Atom, android-25 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom, android-29 | Google APIs Intel x86 Atom, android-29 | Google Play Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 4.2 AI-202.76<IP_ADDRESS>22048
    Xcode: 12.5/12E262 - /usr/bin/xcodebuild
  Languages:
    Java: 1.8.0_242 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.1 => 17.0.1
    react-native: 0.64.0 => 0.64.0
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Setup a fresh React Native 0.64.0 project.
Install Axios.
Try making multiple requests to a functioning API.

Expected Results
Expected the network request to always go through so long as the API is running, but a lot of the times I keep getting a Network Error message although Axios shows this when it cannot connect to the server you are calling. This wasn't the case when I was on RN 0.62
Snack, code example, screenshot, or link to a repository:
import Axios from 'axios';
...
Axios.get('/a/functioning/url').then(handleSuccess).catch(handleError)
...

Same here

This went away without any update of any kind, the issue came from my internet service provider, I will be closing this for now.
If you experience this try doing the following.

Try a different internet service provider.
If you tried 1 and it doesn't work check that you're not connected to a sub-ISP, some ISPs are actually Sub-ISPs that are connected to the same ISP. [ISP = Internet Service Provider, Sub-ISP = Sub Internet Service Provider].
If you are connecting through a VPN or have any network configuration other than your network password try turning it off.

Hope this helps.
cc @iMJumpmxn


link to token page for AEX-X classified tokens

Please describe the problem that your request should resolve.
can't easily access the token page for an AEX-9 token from contract detail view

Describe the solution you'd like
please provide a way to access the token page from the contract detail view in case it is classified as AEX-9 / AEX-141 (once the page exists)
Is it already possible to achieve the same outcome in any other way? If so, how?
not really, as token search in token view is missing for unlisted tokens. more experienced users can copy the contractId and past it into the /tokens/ url 😄

Solution: add a new row with token image and symbol

Hola! It could be a row or an icon like we did in the Token pages. Either way would look great.

The problem of the first one is that you don't actually see the token symbol, which in my opinion could be misleading

Do we implement the second option then?

@danilosierrac I forgot to tag you


[0.18.0] DruidSegmentReader should work if timestamp is specified as a dimension (#9530)

Backports the following commits to 0.18.0:

DruidSegmentReader should work if timestamp is specified as a dimension (#9530)

Clean cherry-pick


search workspaces by id

Description

You need an API that can query workspaces by list of workspace ids.
Related Issue :

How Has This Been Tested?

URL: /api/worksapces
Method: GET
param:
 - workspaceIds : Workspace ID list using separator (,)
   (ex: ws-00,ws-03)

Sample Response
URL: /api/workspaces?workspaceIds=ws-00,ws-03
{
    "_embedded": {
        "workspaces": [
            {
                "name": "Admin Workspace",
                "id": "ws-00",
                "active": true,
                "description": "",
                "owner": {
                    "type": "user",
                    "organizations": [],
                    "username": "admin",
                    "fullName": "Administrator",
                    "email"<EMAIL_ADDRESS>                },
                "published": false,
                "createdTime": "2020-07-17T00:00:00.000Z",
                "modifiedTime": "2020-07-17T00:00:00.000Z",
                "lastAccessedTime": "2021-01-25T04:24:44.656Z",
                "publicType": "PRIVATE",
                "_links": {
                    "self": {
                        "href": "http://localhost:8180/api/workspaces/ws-00"
                    },
                    "workspace": {
                        "href": "http://localhost:8180/api/workspaces/ws-00{?projection}",
                        "templated": true
                    },
                    "dataConnections": {
                        "href": "http://localhost:8180/api/workspaces/ws-00/connections"
                    },
                    "books": {
                        "href": "http://localhost:8180/api/workspaces/ws-00/books"
                    },
                    "connectors": {
                        "href": "http://localhost:8180/api/workspaces/ws-00/connectors"
                    },
                    "roleSets": {
                        "href": "http://localhost:8180/api/workspaces/ws-00/rolesets"
                    },
                    "dataSources": {
                        "href": "http://localhost:8180/api/workspaces/ws-00/datasources"
                    }
                }
            },
            {
                "name": "Shared Workspace",
                "id": "ws-03",
                "active": true,
                "description": "",
                "owner": {
                    "type": "user",
                    "organizations": [],
                    "username": "polaris",
                    "fullName": "Polaris",
                    "email"<EMAIL_ADDRESS>                },
                "published": false,
                "createdTime": "2020-07-17T00:00:00.000Z",
                "modifiedTime": "2020-07-17T00:00:00.000Z",
                "lastAccessedTime": "2021-01-25T04:24:33.659Z",
                "publicType": "SHARED",
                "_links": {
                    "self": {
                        "href": "http://localhost:8180/api/workspaces/ws-03"
                    },
                    "workspace": {
                        "href": "http://localhost:8180/api/workspaces/ws-03{?projection}",
                        "templated": true
                    },
                    "dataConnections": {
                        "href": "http://localhost:8180/api/workspaces/ws-03/connections"
                    },
                    "books": {
                        "href": "http://localhost:8180/api/workspaces/ws-03/books"
                    },
                    "connectors": {
                        "href": "http://localhost:8180/api/workspaces/ws-03/connectors"
                    },
                    "roleSets": {
                        "href": "http://localhost:8180/api/workspaces/ws-03/rolesets"
                    },
                    "dataSources": {
                        "href": "http://localhost:8180/api/workspaces/ws-03/datasources"
                    }
                }
            }
        ]
    },
    "_links": {
        "self": {
            "href": "http://localhost:8180/api/workspaces?workspaceIds=ws-00,ws-03&page=0&size=20"
        }
    },
    "page": {
        "size": 20,
        "totalElements": 2,
        "totalPages": 1,
        "number": 0
    },
    "workspace": {
        "linked": 0,
        "total": 2
    }
}

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 1

deploy to 1


Version 3.17 won't allow a comma in a table's RowKey

Which service(blob, file, queue, table) does this issue concern?
Table
Which version of the Azurite was used?
3.17
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
https://hub.docker.com/_/microsoft-azure-storage-azurite
What's the Node.js version?
As in docker image
What problem was encountered?
Inserting a row where the RowKey has a comma produces InvalidInput error
Steps to reproduce the issue?
Using Storage Explorer, try adding a row to a table with a comma in the RowKey (nb. this issue is not specific to using Storage Explorer as client).
Have you found a mitigation/solution?
Pinning to 3.16 for now, which is fine. (Note that real storage accounts are also fine with comma in RowKey.)

Thanks for raising this, I am looking into it.


Add installation instructions and usage instructions

Add instructions for installing the library and command line program. This will be important once #2 is completed.
We should also use that as an opportunity to update the usage instructions if we can.

This is all set.


strategy for searching with only name in the command line tool

So far, searching with the following combination(s) is implemented in the command line tool:

'person_id' which exists in ReadAct
Wikipedia link
Wikidata id

This issue is aiming to discuss about the strategy for searching with only family_name, name_lang, and first_name.
For example, user has given a person entry with only family_name,  first_name, and name_lang, and nothing else, our weighting mechanism wouldn't work if there are no comparable information.
What we can choose are:

Require users to input more information in this case
Choose the first returned Q-id if we can find any person entities by SPARQL querying with name.

What do you think? @duncdrum

The current strategy is to return the first found.
See 423bbe0

Returning the first match seems the way to go, I would however augment Person.note with something like "uncertain match"

Implemented.


Unable to proxy stream through OpenVPN

Describe the bug
This isn't so much a bug as it is a request for help with troubleshooting. There's nothing wrong with xteve I don't think but I'm wanting to figure out what is going wrong.
Server (please complete the following information):

OS FreeBSD (TrueNAS jail)
Arch Intel 64 Bit
xTeVe Version and Build 2.0.0 Build 0200

To Reproduce
Steps to reproduce the behavior:

Set up OpenVPN in the jail to stream from Australia
Verify all connections are using the VPN except LAN connections
Set xteve to use an Australian IPTV list (publicly available and not obtained illegally), in particular one from here
Attempt to stream one of the channel 7 feeds which are geoblocked.
The stream cannot play

I have verified that accessing the stream directly via my VPN on the very same server connection works through my phone using the native VPN app but it doesn't appear to be working via OpenVPN in the jail. Specifically it gives a 403 error indicating I'm not authorised even though I'm connected the same way as on my phone.
Expected behavior
The stream plays.
Desktop (please complete the following information):

All clients

Additional context
So I realise this is not something wrong with xteve itself but I figure others might have seen this issue or you may be able to help with what might be wrong. I just can't figure out why I'm getting this 403 error.

I tried this with a completely different IPTV list in the UK with a UK connection and get the same thing even though it worked via my phone directly with VPN. I'm curious how anyone is able to circumvent geo restrictions at all.


Move engine configurations to their own file.

Flutter configurations became very complex with a lot of duplication.
This pr refactors the engine builder configs as functions to avoid
duplication and improve readability.
Bug:
https://github.com/flutter/flutter/issues/56419

LGTM with a nit to add a comment to config/lib/repos.star about the fake repo stubs

Actually I got the repo urls back to the chromium external because it would've affected the prod pollers and modified the cq_groups to replace "." with "_" in the name generated from the url.


tests: logging: log_stack: Adjust log stack limits for SPARC

Recent changes c0d7e10 increased stack usage for SPARC platform. Adjusting values for immediate mode.
Fixing instead of reverting as proposed in #43158.

I checked that stack increase (~100 bytes) is due to calling a function through pointer instead of calling it directly (shocked!).


Change login flow (experimental)

This is the flow (more or less):
clasp login: logs you in using the default clasp credentials. Saves .clasprc.json to your ~ directory.
clasp login --creds: logs you in using the file credentials.json in the directory that you are running the command in. Saves .clasprc.json to your current directory.
clasp login --creds other_creds.json: logs you in using the credentials in other_creds.json. Saves .clasprc.json to your current directory.
Your credentials file should look like this:
{"installed":{"client_id":"239267426989-275aglft6htcfsdj7t5d3csdlogchamh.apps.googleusercontent.com","project_id":"project-id-xxxxxxxxx","auth_uri":"https://accounts.google.com/o/oauth2/auth","token_uri":"https://accounts.google.com/o/oauth2/token","auth_provider_x509_cert_url":"https://www.googleapis.com/oauth2/v1/certs","client_secret":"xxxxxxxx","redirect_uris":["urn:ietf:wg:oauth:2.0:oob","http://localhost"]}}

Signed-off-by: campionfellin<EMAIL_ADDRESS>Related to #204

[x] npm run test succeeds.
[x] npm run lint succeeds.
[ ] Appropriate changes to README are included in PR.

Please test before merging.

Coverage decreased (-0.05%) to 15.691% when pulling 64b5301ccef7dcc99a2d9690c6d52db708975e08 on campionfellin:new-login into bd177b435ba4dc7c9d2117f24ae15b004b07b9b5 on google:master.

(also note that npm run docs did not work for some reason to update the docs with this)

@campionfellin - would this help with https://github.com/google/clasp/pull/28?

For now, you still have to use the browser to confirm the login. What this does is allow you to authenticate with your own Oauth client, rather than the default for clasp. This helps with things like clasp run which need more permissions than the default.

Will update README in separate PR.

Ok, after some testing with it, I think the best solution is to remove the default value and make it so if the user uses --creds they must supply a credentials file. The reasoning is this:
If we default it to ./credentials.json, or anything really, when the user uses clasp login without specifying --creds, the options.creds variable is still set to ./credentials.json. Thus, if they have that file in their directory, but want to login globally with the default (old) clasp credentials, they can't.
@grant what do you think?

Agreed, that makes sense. Let's not default.

Ok, now here is what happens:
clasp login : works as it used to, using the default clasp credentials.
clasp login --creds : gives error error: option '--creds <file>' argument missing
clasp login --creds creds.json (when creds.json exists and is formatted properly) : logs in user with those credentials, saves ./.clasprc.json
clasp login --creds creds.json (when creds.json doesn't exist): Credentials file not found.
clasp login --creds creds.json (when creds.json exists, but not right format) : Incorrect credentials file format.

Important to note when we update the readme:
If you use your own credentials from GCP, you must enable both the Apps Script API and Drive API.

It would be cool if you added @example with the comment you wrote above, so it lasts.

Added simple examples, I think the more detailed "what happens if I do this..." should be in the README under troubleshooting.

I'll try to release this week with these changes.


whereis on Linux

whereis on Linux works like this:
whereis ibd2sdi
ibd2sdi: /usr/bin/ibd2sdi /usr/share/man/man1/ibd2sdi.1.gz

so the script crashes with:
Traceback (most recent call last):
  File "sdi2sql.py", line 182, in <module>
    ibd2sql(config)
  File "sdi2sql.py", line 52, in ibd2sql
    sdi_result = subprocess.run([ibd2sdi_path, '--dump-file=' + sdi_path, ibd_path], stdout=subprocess.PIPE)
  File "/usr/lib/python3.8/subprocess.py", line 493, in run
    with Popen(*popenargs, **kwargs) as process:
  File "/usr/lib/python3.8/subprocess.py", line 858, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib/python3.8/subprocess.py", line 1704, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'ibd2sdi: /usr/bin/ibd2sdi /usr/share/man/man1/ibd2sdi.1.gz'

Thanks for your feedback.
I fixed this bug in commit: a9cfa9a4ea203d7faddc8058c99525883fb62275
But I don't have suitable environment to test it.
So, some more feedback is welcome if there is still a probleam


CrossValidation does not work with newer numpy.

Issue Description
What did you expect to see happen?
The tests to pass on newer versions of libraries.
Failed on:
numpy 1.13.1
h5py 2.7.0
scipy 0.19.1
sklearn 0.19.0
matplotlib 2.0.2

What did you see instead?
framework/PostProcessors/CrossValidations.leaveOneLabelOut: Exception in thread 0:
framework/PostProcessors/CrossValidations.leaveOneLabelOut: Traceback (most recent call last):
framework/PostProcessors/CrossValidations.leaveOneLabelOut:   File "/home/jjc/miniconda2/lib/python2.7/threading.py", line 801, in __bootstrap_inner
framework/PostProcessors/CrossValidations.leaveOneLabelOut:     self.run()
framework/PostProcessors/CrossValidations.leaveOneLabelOut:   File "/home/jjc/miniconda2/lib/python2.7/threading.py", line 754, in run
framework/PostProcessors/CrossValidations.leaveOneLabelOut:     self.__target(*self.__args, **self.__kwargs)
framework/PostProcessors/CrossValidations.leaveOneLabelOut:   File "/home/jjc/sub/raven/framework/Runners/SharedMemoryRunner.py", line 134, in <lambda>
framework/PostProcessors/CrossValidations.leaveOneLabelOut:     self.thread = threading.Thread(target = lambda q, *arg : q.append(self.functionToRun(*arg)), name = self.identifier, args=(self.subque,)+tuple(self.args))
framework/PostProcessors/CrossValidations.leaveOneLabelOut:   File "/home/jjc/sub/raven/framework/Models/PostProcessor.py", line 208, in evaluateSample
framework/PostProcessors/CrossValidations.leaveOneLabelOut:     returnValue = (Input, self.interface.run(Input))
framework/PostProcessors/CrossValidations.leaveOneLabelOut:   File "/home/jjc/sub/raven/framework/PostProcessors/CrossValidation.py", line 250, in run
framework/PostProcessors/CrossValidations.leaveOneLabelOut:     inputDict, self.CVEstimator = self.inputToInternal(inputIn, full = True)
framework/PostProcessors/CrossValidations.leaveOneLabelOut:   File "/home/jjc/sub/raven/framework/PostProcessors/CrossValidation.py", line 218, in inputToInternal
framework/PostProcessors/CrossValidations.leaveOneLabelOut:     if newInput.values().count(None) != 0:
framework/PostProcessors/CrossValidations.leaveOneLabelOut: ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

Do you have a suggested fix for the development team?
Same general fix as bd14004ad10147654bdc8403dd7e7c37afc2ff17
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement? improvement
[x] 2. Is it tagged with a priority: critical, normal or minor? normal
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements? n/a
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.  (Causes crash, not wrong results) not a defect. New version compatibility
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.) yes

Issue Reviewed by @joshua-cogliati-inl on 2017-Sept-13

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed? not a defect
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.) not a defect
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)? not a defect
[x] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)? not a defect
[x] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided? #351

Issue Closure Checklist Reviewed by @alfoa on 2017-Sept-13

Closure check list passed...


Dear maintainer. This is a nice work. Congrats! It's pleasing to hear that the project is evolving.

Dear maintainer. This is a nice work. Congrats! It's pleasing to hear that the project is evolving.
I have doubts about the use of some functions when generating the SQS. I don't know if there is a better channel for the present discussion; please let me know. Anyway, here is my issue: I'm trying to generate the SQS for a ZnO Wurtizite structure with some Zn atoms changed by Ag. I created a 3x3x2 supercell with 72 atoms and instructed the code to add 3 Ag atoms to the Zn sites. I read in the documentation the necessity of adding pair_weights, but I didn't get how to define the atoms' order to build the matrix. Here is my yaml input:
structure:
  supercell: [3, 3, 2]
  file: ZnO.cif
mode: random
iterations: 1e7
shell_weights:
  2: 1.0
  3: 0.5
  4: 0.25
  5: 0.2
  6: 0.1667
  7: 0.1429
#which: Zn
composition:
  Zn:
    Zn: 33
  Ag:
    Zn: 3
  O:
    O: 36
prefactor_mode: mul
prefactors: 0.5
pair_weights:
  #   O      Ag     Zn
  - [ 0.0,   0.0 , 0.0] # O
  - [ 0.0,   0.0,  0.5 ] # Ag
  - [ 0.0,   0.5,  0.0 ] # Zn

I'm using 0.5 following the example. Does it hold even for the unbalanced number of atoms (33 Zn and 3 Ag)?
Should I always include the parameter "supercell" in the input to force the code to generate structures as supercells? I mean, in the ATAT SQS code, we just need to inform the number of atoms, not necessarily the supercell.
Thanks in advance.
Originally posted by @anibalbezerra in https://github.com/dgehringer/sqsgenerator/discussions/48#discussioncomment-11472293

Hi @anibalbezerra
I have transferred your question to this issue!
Indeed @dhoulek is right. The matrix indices are referring to the ordinal numbers of the atomic species in ascending order.
I admit however, that it might not be straightforward to find. The information you are looking for is in the target_ojective parameter setting . There you have to open the card A note on array input interpretation. There is just a reference to it in the [pair_weights](A note on array input interpretation) parameter's section.
There is also a note named Input array interpretations at very to of the input parameter documentation page which should help.
Coming back to your question!
The supercell argument's default value is 1,1,1. So if your file ZnO.cif contains 72 atoms you can leave the supercell argument away. In case it has 4 atoms you need to specify the supercell. The supercell tag is meant as a convenience function.
As you are turning of in your example the sub lattice - sub lattice interactions. I'd suggest the following simplified YAML (assuming ZnO.cif contains 4 atoms) ($4 \times 3 \times 3 \times 2 = 72$ atoms)
structure:
  supercell: [3, 3, 2]  # just delete this line in case ZnO.cif has 72 atoms
  file: ZnO.cif
mode: random
iterations: 1e7
shell_weights:
  2: 1.0
  3: 0.5
  4: 0.25
  5: 0.2
  6: 0.1667
  7: 0.1429
which: Zn
composition:
  Zn: 33
  Ag: 3

I hope that helps!
Let me know if this does what you expect
Best regards


¿Could you make this library compatible with Compose Multiplatform?

@Dhakron I am working on a Compose Multiplatform version of this library. It will probaply be released in its own GitHub repository and will be linked in this repositories readme.


Edit description phrasing in brew-desc.rb

Edited phrasing of descriptions to make them more “glanceable”.
I did this by fixing some inconsistencies.  In doing so, I also introduced some inconsistencies of my own—but IMO they are much less…I suppose “disruptive” is the word?
I realize that’s an odd word to use, since we’re talking about short, one-line descriptions.  However, since this formula tends be used by people who are in “exploration mode” on the command line, I think it’s important.
In particular, I’ll emphasize these changes:

Capitalization.  If you’re talking about the protocol, it’s “SSH”.  If you’re talking about the program by its name, it’s “ssh”.
Leading/trailing spaces.  Fixed a few leading and trailing spaces sprinkled throughout the descriptions.
Brevity. A huge number of descriptions lead with “Tool for…” or “Utility to…”.  If I could delete this and find a concise rephrasing, I did.

I did not do this when…:

A formula installed “tools” or “utilities”—plural.  In the former case, a single “tool” or “utility” usually installs a program with the same name as the formula itself.  On the other hand, when a single brew install «formula» results in multiple new commands, I left the description intact to explicitly communicate this.
I couldn’t come up with a suitable rewrite.

It’s “OS X”!  Not “OSX”. Not “Mac OS X” (not for many years, anyway!).
Oxford comma. This is a matter of personal preference, but I prefer the Oxford comma. It more closely resembles how people would read the text aloud—and (for me, at least) seems to read a little easier.
Written in «language».  Where possible, I moved this from the beginning or middle of a description to the end, and put it after a semi-colon.

  But most of the time, people who read the description are wondering what a piece of software is, not what it’s written in.  Therefore, I put it at the end, so that the first thing a person reads is what the software is.
I did not do this when the description read, e.g., “A C++ library for performing…”.  That’s because it’s ambiguous whether this means the library is written in C++, or whether it’s merely intended for other C++ code.  (In other words, it could be “A C++ library” that’s written in C.  In this case, it would be inaccurate to say “written in C++” for the description.)

In general, I also tweaked phrasing here and there, if I thought it would “read easier”.

I think that covers it.  Let me know if missed anything.
Thanks!

Initial response: Thank you! This is the kind of thing I've meant to do for years but simply haven't carved out enough time.  But I will certainly get to this sometime this week.

Ok, I pulled this, but I had to edit it a ton. Some notes for future work:

Please don't overpunctuate. Two things jumped out at me:
+ Descriptions like this do not need a comma before and: "Scan log files, and ban IPs showing malicious signs"
+ The descriptions are too short ever to need semicolons. There's almost always an easier way.
If something actually is a C or C++ library, it should say that right at the start of the description (that is essential information for a potential user).
However, if something is just a tool written in C, C++, Java, Python, Go, Ruby or whatever, then I agree with you that it's not essential. But I went even further than you: I removed nearly all of those. I say "nearly" because I'm sure I missed some or was not 100% consistent. C'est la vie.
I finally decided to make a rule: in this file, everything is now command-line, not command line or commandline. Whew.

Thanks so much for the overall cleanups though!

Got it!  Thanks for sharing your rationale.
I will miss the ability to see what some projects are written in.  While it’s definitely second-class info, sometimes it’s still something I’d try to search for.
Why? Maybe I want to customize something and I don’t know C, so I want to search for something in Python; or maybe I want to look at the source code to learn how to do something; or maybe I’m looking for something written in Ruby so that it can talk to some code that I’m writing on my own machine.
I respect your decision to remove it, and I do concede the secondary nature of that information—I just wanted to communicate that, secondary or not, there’s still some value there.  Still, I’m glad I was able to contribute some of my edits.  :)
In any case, if I make another Pull Request like this in the future and you’re only on board with some of my changes, you can just ask me to adjust my PR and I’ll make the changes.  Might save you a little work.  ;)
Thanks!

I will miss the ability to see what some projects are written in. While it’s definitely second-class info, sometimes it’s still something I’d try to search for.
Why? Maybe I want to customize something and I don’t know C, so I want to search for something in Python; or maybe I want to look at the source code to learn how to do something; or maybe I’m looking for something written in Ruby so that it can talk to some code that I’m writing on my own machine.

Honestly, I agree. I may add them all back soon. I don't know. I'm having editor's remorse now! In any case, I'll keep your thoughts in mind.


wavesurfer.backend.getPeaks() returns array with NULL

Hello,
I'm urgently looking for a way to save the waveform on serverside.
getPeaks() doesn#t work for me . Any suggestions ? This might be something that
should be important in all use cases.
Wavesurfer.js version(s):
2.03
Browser version(s):
Chrome
Code needed to reproduce the issue:
  wavesurfer.on('waveform-ready', function () {

                var dataJSON = wavesurfer.backend.getPeaks(960);
                console.log (JSON.stringify(dataJSON));
                var dataImg = wavesurfer.exportImage();
}

Any suggestions on this issue? A'm I doing somethig wrong or is this a bug?

I am able to retrieve the peaks fine on Firefox using 2.0.4, e.g. using arbitrary value of 300:
var length = 300;
var start = 0;
var end = 300;

backend.getPeaks(length, start, end);

// returns [0.019927978515625, -0.02044677734375, 0.0419921875, -0.022216796875, 0.03094482421875, -0.032928466796875, etc]

@thijstriemstra this is still a big issue on v2 latest version and not even working on chrome v64
and getPeaks return either an array of null
or something like
[17654.68571428571: 0, 17655.68571428571: 0]
so please test it on chrome too before closing this and most people use chrome not firefox!
thanks

my bad this last code you posted works fine :)
var length = 300;
var start = 0;
var end = 300;
wavesurfer.backend.getPeaks(length, start, end);

@thijstriemstra and @x5engine
hi, i have a similar problem, i`m using the following library

below is my code where i was returning the audio peaks in an interval, 3 days ago it was working normally, until yesterday it was working normally,
today it just stopped returning some value and is returning an empty array

   <script>
  $(document).ready(function () {

    var wavesurfer = WaveSurfer.create({
      container: document.querySelector('#waveform'),
      backend: 'MediaElement',
      waveColor: 'black',
      progressColor: 'gray',
    });

    wavesurfer.load('http://ia902606.us.archive.org/35/items/shortpoetry_047_librivox/song_cjrg_teasdale_64kb.mp3');

    wavesurfer.on('ready', function () {

      var length = 300;
      var start = 0;
      var end = 300;

      console.log(wavesurfer.backend.getPeaks(length, start, end));

    });

  });
</script>

could someone tell me what am i doing wrong? I spent 6 hours today trying to solve it and I couldn't
how should i install or import the library to use it? thank you, i'll wait

could someone tell me what am i doing wrong?

you're using a really old version of wavesurfer. Use a 3.x version instead.

@thijstriemstra
even including the minified script: <script src="https://unpkg.com/wavesurfer.js"></script> the problem remains
below is my code where i was returning the audio peaks in an interval, 3 days ago it was working normally, until yesterday it was working normally,
today it just stopped returning some value and is returning an empty array
if i remove the backend option from the setup it does not render my audio on the screen
  var wavesurfer = WaveSurfer.create({
      container: document.querySelector('#waveform'),
      // backend: 'MediaElement',
      waveColor: 'black',
      progressColor: 'gray',
    });

I will explain my application better, I am sending a file through a form so I pass the file (path)
$audio = asset('original/'. $file->getClientOriginalName ());
to the return view
return view ('result')-> with('audio', $audio);
and i'm rendering it this way
  $(document).ready(function() {

    var wavesurfer = WaveSurfer.create({
      container: document.querySelector('#waveform'),
      backend: 'MediaElement',
      waveColor: 'black',
      progressColor: 'gray',
    });

    var audioElement = document.createElement('audio')

    audioElement.autoplay = true
    audioElement.loop = true
    audioElement.crossOrigin = 'anonymous'
    audioElement.src = '{!! $audio !!}'

    wavesurfer.load(audioElement);

    wavesurfer.on('ready', function() {

      var length  = wavesurfer.getDuration();
      var start   = 0;
      var end     = length;

      console.log(wavesurfer.backend.getPeaks(length, start, end));

    });

  });

and in this way, using the backend when creating the WaveSurfer the waves are rendered on screen, if I remove them, the peaks appear but the waves on the screen do not


OSSM-6131 Split Test suites on integration and e2e

I need to submit another PR to change the make target in the integration test run

@jwendell PR updated. Waiting to this PR to be merge to run again the job: https://github.com/openshift/release/pull/50050


Update phf to 0.11.0

As mentioned in #289 a transitive dependency, via phf, was flagged by Windows Defender. The dependency could be removed in phf, and this change updates the version of phf in palette to one that doesn't depend on the removed dependency. phf is not part of the public API and can thus be patched as a non-breaking change.
I'm treating this as a security patch and making an exception from the general MSRV rule. Also clarifying in print that this exception may have to happen. It may still support older versions when the named feature is disabled, unless they are eagerly parsing the phf crate.
Closed Issues

Fixes #289, by changing to a newer phf version.

bors r+

I'm not going to bother updating the benchmark test for this.


Let cancel() return if block successfully was cancelled

Unfortunately the dispatch_block_testcancel() doesn't behave as expected. It just return whether or not you called dispatch_block_cancel() on a block, not if the block was successfully cancelled before being dispatched to a queue.
If the behavior of dispatch_block_testcancel() changes, the solution would look as follows:
func cancel() -> Bool {
    dispatch_block_cancel(block) // Cancel block
    let success = dispatch_block_testcancel(block) // Test if block was successfully cancelled
    return success != 0
}

Can't be achieved.


Migration pod controller

Splits the migration logic from the VM lifecycle controller and moves it to migrate.go

Looks good. It's not immediately clear to me why every functional test failed.


[Feat. request] Headset-only 'seated play' support

TL;DR - Headset-only as in no motion controllers, locking the crosshair to the middle of the screen or something similar.
Some people can't afford proper VR and thus do hacky stuff with Google Cardboard headsets. It would be pretty awesome if this mod was compatible with a seated headset + keyboard and mouse setup, so that playing on Google Cardboard was made viable.
I'm nearly sure Vivecraft supports that, but it's kinda odd and literally unplayable on an iGPU (Intel's OpenGL drivers have shoddy OpenGL VR support, afaik) so I'm hoping MCXR gets that feature since it seems really promising + fabric performance mods FTW.
You can achieve an unplayable (as in unable to interact with anything) but still cool seated setup in MCXR by combining it with a controller via LambdaControls/MidnightControls, and that's pretty much the only reason I'm requesting proper support.

Yeah that's not happening unless you do it yourself

Yeah that's not happening unless you do it yourself, I'm sure it's simple enough though just pojav launcher and looking

Sir, this is MCXR not questcraft

Yeah that's not happening unless you do it yourself, I'm sure it's simple enough though just pojav launcher and looking

What? Even if i used Pojav, that feature still isn't in MCXR. And even if it was, making use of it on a phone would be nearly impossible.
Besides, i play Minecraft on my computer, not my phone.

When the new input system is done this should be easily doable

When the new input system is done this should be easily doable

Any idea of roughly when that might be?


Jupyter Notebook to reproduce graphics is 404

Thanks for the great work.
Its will be great to have access to the notebook with code for fitness graphics from /WANNRelease/prettyNeatWann/

i linked it to the correct file. the notebook was in the WANN folder (and uses the sample run in that same folder). sorry about that.


Bug: diffArrays gives wrong result on array of booleans

const jsdiff = require('diff')
const a1 = [true,true,true]
const a2 = [true,false,true]
const diff = jsdiff.diffArrays(a1, a2)

outputs
[ { count: 2, value: [ true, true ] },
  { count: 1, added: undefined, removed: true, value: [ true ] } ]

which is wrong! i'd expect something like
[ { count: 1, value: [ true ] },
  { count: 1, added: undefined, removed: true, value: [ true ] }
  { count: 1, added: true, removed: undefined, value: [ false ] }
  { count: 1, value: [ true ] } ]

In addition to false any falsey primitive value including number and string:
const a1 = [1,2,3]
const a2 = [1,0,3]

[ { count: 1, value: [ 1 ] },
  { count: 1, added: undefined, removed: true, value: [ 2 ] },
  { count: 1, value: [ 3 ] } ]

const a1 = ['a','b','c']
const a2 = ['a','','c']

[ { count: 1, value: [ 'a' ] },
  { count: 1, added: undefined, removed: true, value: [ 'b' ] },
  { count: 1, value: [ 'c' ] } ]

It looks like a result of removeEmpty in https://github.com/kpdecker/jsdiff/blob/master/src/diff/base.js#L27-L28
Removing falsey values seems to contradict the description: diffs two arrays, comparing each item for strict equality (===)
Does it make sense to override another method in array.js like:
arrayDiff.removeEmpty = function(value) {
  return value;
};

@wvanderdeijl Are you interested to follow up on this or prefer that I go for it?

I won't have time for this over the weekend or beginning next. So feel free to take a crack at it. Otherwise I could have a look late next week

Thank you for quick reply. I submitted a pull request. Take is as appreciation, not impatience.
When you do have time, am happy to get your thought about change to clone arrays in https://github.com/kpdecker/jsdiff/commit/d5003de462121e0fa7e44c7d59d0ec42a394719a
I wrote a test to confirm that if the input is arrays for which every item is equal, then the output value property refers to an input array.
Is the goal to avoid aliasing input to output, that is, provide value semantics like strings?
If yes, then what do you think about

tokenize can make the copy in one place at the beginning
join can return the value to avoid unneeded copying, especially after splice

If no, then it seems like the very original code for both methods to return the value is okay.


Make CITResewn work on 1.20.3 without breaking old versions :)

I did testing and nothing seems to be broken.
The only thing that changed is that we need to capture the field now and make it a field our self.

could you make a built version if that's fine?

Crashes whenever launching for reason "Mixin transformation of net.minecraft.class_4599 failed" BufferBuilderStorage requires an argument of int maxBlockBuildersPoolSize, this doesn't seem to implement that. Why doesn't it? Beyond me, never worked in mods before. Not sure where to set it, but this seems to be the problem.

I think you want to update to 1.20.4 while you are at it...

If anyone's interested, I published my personal build for 1.20.4 here: https://github.com/dicedpixels/CITResewn/releases/tag/1.20.4-1.1.4-dicedpixels. Everything should work.

If anyone's interested, I published my personal build for 1.20.4 here: https://github.com/dicedpixels/CITResewn/releases/tag/1.20.4-1.1.4-dicedpixels. Everything should work.

This seems to work perfectly, would be nice if you opened a PR and it got merged. It would also close #358

@dicedpixels please could you open a PR?
It would also close #361

@BenignPigeon I'd rather wait for the author. Even if I make a PR, the author needs to merge it. For now, my build should suffice.

@dicedpixels but for the author it might be easier to merge a PR rather than update manually the code.
But anyway I agree that still we need to wait for the author in both cases, just maybe with the PR it might be faster.

Completely see your point dicedpixels and i think Nyphet has a good point too it it will probably be easier for the author to merge changes rather than have to do it manually

Oh hey I just saw this, wait let me fix this real quick

Emissive CIT not working on Handheld items

Emissive CIT not working on Handheld items

Did it work on older versions? I don't think emissives were ever a part of CITR.
https://github.com/SHsuperCM/CITResewn/issues/123#issuecomment-1077023538

maybe theyre not part of it, but it worked on 1.20/1.20.1

maybe theyre not part of it, but it worked on 1.20/1.20.1

Can you provide a resource pack? I want to double check.

@lucksducks
Immersive textures is not and never was something CITR implemented. If you had emissive textures then another mod added it.
Apologies for this *very* over-extended delay with this update, I'll try to take a look soon.

@SHsuperCM , sorry for the ping, could I at least get your permission to ship @dicedpixels 's build in my modpack please?
I guess I might need @dicedpixels permission too.

@SHsuperCM , sorry for the ping, could I at least get your permission to ship @dicedpixels 's build in my modpack please? I guess I might need @dicedpixels permission too.

Go ahead. Please give credit to the original author. My fork is just a few changes to get it working on 1.20.4.

@lucksducks Emissive textures is not and never was something CITR implemented. If you had emissive textures then another mod added it.
Apologies for this *very* over-extended delay with this update, I'll try to take a look soon.

Please merge other PRs as well when merging this

@SHsuperCM , sorry for the ping, could I at least get your permission to ship @dicedpixels 's build in my modpack please? I guess I might need @dicedpixels permission too.

Go ahead. Please give credit to the original author. My fork is just a few changes to get it working on 1.20.4.

Can you merge other PR's fixes as well for your 1.20.4 build?

@SHsuperCM , sorry for the ping, could I at least get your permission to ship @dicedpixels 's build in my modpack please? I guess I might need @dicedpixels permission too.

Go ahead. Please give credit to the original author. My fork is just a few changes to get it working on 1.20.4.

Can you merge other PR's fixes as well for your 1.20.4 build?

Will have to think about this. My knowledge of the mod's internal workings is limited and I'd rather not merge anything that's not approved by the original author.

@SHsuperCM , sorry for the ping, could I at least get your permission to ship @dicedpixels 's build in my modpack please? I guess I might need @dicedpixels permission too.

Go ahead. Please give credit to the original author. My fork is just a few changes to get it working on 1.20.4.

Can you merge other PR's fixes as well for your 1.20.4 build?

Will have to think about this. My knowledge of the mod's internal workings is limited and I'd rather not merge anything that's not approved by the original author.

Can you make an 1.20.4 build with the JSON Lag Patch?

@SHsuperCM , sorry for the ping, could I at least get your permission to ship @dicedpixels 's build in my modpack please? I guess I might need @dicedpixels permission too.

Go ahead. Please give credit to the original author. My fork is just a few changes to get it working on 1.20.4.

Can you merge other PR's fixes as well for your 1.20.4 build?

Will have to think about this. My knowledge of the mod's internal workings is limited and I'd rather not merge anything that's not approved by the original author.

Can you make an 1.20.4 build with the JSON Lag Patch?

I cherry-picked that PR a while back and found out it broke some RPs. I'd like to keep the current version as stable as possible.

Hi @SHsuperCM , any update on this? As I said before, could you please either release a 1.20.4 version or give me permission to include dicepixels' build into my modpack?
Thanks

@Crystal-Spider

The project is open source and licensed under the MIT license. This means you don't need permission to use the mod or fork.

Permission is provided through the MIT license. I personally used this mod in my minecraft modpack TEDS Plus

Superseded by upstream.
This PR would've broken other 1.20 versions as well which is part of what was stopping it from being merged.
As a note for the future as well, try to keep the impact of PRs you make as small as possible, this PR breaks a few of things that it shouldnt even come in contact with.
@Crystal-Spider  As others said, I made CIT Resewn use an open license for a reason. everyone is free to do whatever they wish with the source code. (ofc I'd appreciate credits regardless)

A 1.20.4 release will come soon as I check a few things before publishing.


Fix race condition in subnet deletion API

This PR proposes two issue fix

Fix concurrency issue when multiple subnets in the same VPC are deleted at the same time (tracked by Issue #662)
Fix subnet non-gateway port count issues in subnet deletion API

@kevin-zhonghao Could you also review this PR? This PR is to address a concurrency issue #662. Basically when we try to access db multiple times or multiple tables, we will need to ensure the set of operations are in a transaction, otherwise we will hit race condition like this one when the level of concurrency increases.

@kevin-zhonghao Could you also review this PR? This PR is to address a concurrency issue #662. Basically when we try to access db multiple times or multiple tables, we will need to ensure the set of operations are in a transaction, otherwise we will hit race condition like this one when the level of concurrency increases.

Sure, working on it.


Extract SQL queries to a properties file

This is the first step towards allowing different target databases.
It is a precursor to fixing #3
@wisechengyi @KidDaedalus

@KidDaedalus please take another look when you have a chance.

just started looking. the tests seem to be failing?
$ gr integrationTest

> Task :integrationTest

com.gradle.exportapi.BuildDaoIntegrationTest > shouldInsertBuild FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

com.gradle.exportapi.CustomValueDaoIntegrationTest > shouldInsertCustomValue FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

com.gradle.exportapi.SchemaIntegrationTest > shouldDropAndCreateSchema FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

com.gradle.exportapi.TaskDaoIntegrationTest > shouldInsertTask FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

com.gradle.exportapi.TestDaoIntegrationTest > shouldInsertTest FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

5 tests completed, 5 failed

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':integrationTest'.
> There were failing tests. See the report at: file:///Users/yic/workspace/ge-export/build/reports/tests/integrationTest/index.html

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

@wisechengyi you will need to create a test-db-info.properties pointing to your test DB.  Note that the test will destroy any data in the DB, so don't do your normal one.


Fix failing service tests for [bintray npmlicense]

Closes #2343 #2348
Ref: #1359

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS


Lists all Functions deployed in a given namespace using API does not work

Describe the bug
Context: the function workers are not in the brokers.
When invoking the service URL of a function worker to get the list of deployed functions in a given namespace, the response is 404 Not Found (while there are functions deployed).
To Reproduce
Steps to reproduce the behavior:

Run a function worker
Deploy a function using the admin url of the function worker
Get the list of currently deployed functions in a namespace (pulsar-admin --admin-url http://my-host:6750 functions list --namespace my-namespace --tenant my-tenant)
The response is HTTP 404 Not Found

Also test with curl, same result.
If the worker run in a broker, the response is correct.
The other pulsar-admin commands such as functions create are working.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
Desktop (please complete the following information):

OS: MacOS

Additional context

Pulsar Version: 2.3.2
Java Version: 1.8.0_202

@bbonnin Have you tried the latest Pulsar version?

Hi, with Pulsar 2.5.2, now I can see all the functions. :)

Hi, with Pulsar 2.5.2, now, I can list the functions. :)


[NFT] Allow user transfer NFT

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

User can transfer NFT with receiving address and custom network fee

Tested cases:

[ ] EggForce
[ ] Hammer
[ ] Casper Punk
[ ] Transfer Casper Punk to new account

hi @hoailinh1210 , please help to create improvement task with your concern about high gas fee while using session code for CEP78. Thanks.


Bug: cannot install with<EMAIL_ADDRESS>

Info

Tool
Version

Plugin
v2.0.0

Vue
v3.2.0

Node
v18

OS
mac

Input
npm install

Output or Error
npm ERR! Could not resolve dependency:
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/vue
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@gtm-support/vue-gtm
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/node/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/node/.npm/_logs/2023-03-03T21_14_33_526Z-debug-0.log

Expected Output

Additional Context
I am migrating a Vue2 repo to Vue3 and I currently have my vue package version set to ^3.2.0 and @gtm-support/vue-gtm set to ^2.0.0.
When I run npm install I receive the error seen above. In the package.json if your repo you list a devDep for vue at ^3.2.40 and a peerDep at ^3.2.0. I have to set my vue version to ^3.2.40 to avoid the error.

any updates? I'm migrating nuxt2 to nuxt3 and found the same issue(vue version 3.3.4)
npm ERR! Found:
npm ERR! node_modules/vue
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@gtm-support/vue-gtm
npm ERR<EMAIL_ADDRESS>from the root project

I assume I need to set https://github.com/gtm-support/vue-gtm/blob/18fd3374ac4a970c12aaa1cae97ae43b82a0fd6f/package.json#L90 to >= instead of ^
Will do when I find the time

@Shinigami92 I tested it, and it works! thanks:) I'm in the middle of the migration to Nuxt3, so I will report any additional issues if occur


Make PGUtilsSpec unmixable

Mixing in PGUtilsSpec caused database setup and teardown to happen
twice, which probabilistically killed the Jenkins build. Now PGUtilsSpec
is final so that's impossible. Changes to ThumbnailsSpec are also
included since that was the offender.

+1


Token Access Request Confirmation

[x] I read and understood how to enable logging

Issue / Steps to reproduce the problem
Im looking to confirm I am using this right. I've checked the examples and docs but cannot find a complete example of what Im trying to do.
Using the quick start 5 project as an example, I have added extra ApiResources, I've then added 2 scopes to the Api Resource.
The user request has been set to this resrouce name, api2. I want the API to confirm the user has read access from API2. So the client is set to be allowed api2, read scopes. api2 is requested buy i dont seem to be able to confirm they have read? If I requested api read for example it would confirm, but if someone doesnt have read access it would give invalid scope.. Is there no setup that can be configured where a client is given read access scoped to a certain API resource, it confirms what the client can and cant do as part of that API??
Im either having to reqest just the API2, then add claims to a user that states what claims they have but then all claims come through on the user.. their access to api1 etc. I simply want to request a key for an API, then have details of what that user can do with reagards to that resource. Is this possible?
Relevant parts of the log file
<log goes here>

AllowedScopes are what a client has access to, not a user. You do the authorization check in the API for the user. IdentityServer does not have a way to determine if a user is allowed, and that's not its job (by design).


Host or network get operation not displaying expanded information

Thanks for this amazing work, this package helps me lot.
I am struggling with searching for the host or network using the name ( for example :  test (object name) and returning the expanded details ( like Name, value, description and id)
I tried to pass the expanded = True but no luck.
It only output below information
INFO:root:GET success. Object with name: "test" and id: "0050569D-669F-0ed3-0000-146028913744" fetched from FMC.
Appreciate any guidance
Thanks

That "output" is just a "syslog" message.  If you want to see the object's information you'll need to use the Python print() function.  print(host1.name) for example.

Thank you for quick reply
I am using below commands:
    name = input ('Enter IP Host Name:- ')
    iphost = fmcapi.Hosts(fmc=fmc1,name = name)
    iphost.get()

Should I add below
print(iphost.name)?
Thanks again

Yep!  That will output the value for the "name" variable for the iphost Class object.

Thanks a lot


Error when seed subsampling is set to zero

If no subset of core points is specified for seed detection, all core points should be used under consideration of a subsampling parameter. This is implemented as seed_candidates_curr = range(0, self.analysis.distances_for_compute.shape[0], self.seed_subsampling) here.
If seed_subsampling is set to zero, no subsampling should be performed. However, the 0 value causes an error in the range function. If no subsampling is to be done, the argument must be 1 here.

Probably a user error, no subsampling should be 1. Please check if this is something we should handle (does it make sense to expect 0 for no subsampling) or implement error information for the user.