support authentication for extension endpoints

Currently extensions do not have support authentication. This is insecure. We should add support for setting an auth token.

So for example, a bot.yml file would look like this?:
bot_name: "trivia_bot"
extensions:
  type: REST
  url: http://localhost:8770
  token: my-extensions-token

And extension.ServeREST should check for the token in all calls, right?

Yep that seems like the easiest way. Then the serve endpoints would check for a bearer token in the HTTP header.


Add match on both ‘link’ and ‘Link’ header keys

What does this do?
It fixes the problem with missing critical information for product records on the app install, like handle, title, variants, and price=0. It happens that Shopify now uses variations of the link key name, either Link or link (most probably they just switched from one to another) in API response to GET all products. That leads to an improper stream of products when the first 50 products are getting in and receiving all the information, and the rest might not. This PR is to handle this situation.
How does this change work?
Setting the header key name to a downcase and checking with that rather than matching it on 'Link'.
How do you manually test this?
I changed elixir-uso to use local elixir-shopifyapi and updated it with this solution.
I generated the elixir-shopifyapi app again and restarted the Phoenix server.
Then, I installed the dev app on the shop and made sure that all products were now added to the database and had all fields filled with information about them.
When should this be merged?
Sooner the better

Hey, @orbit-apps/devs, I need your help to validate the approach on this one, please. Thanks 🙏

I need your help to validate the approach on this one, please. Thanks 🙏

What you have here definitely works! Nice job wrapping your brain around the for comprehension - definitely not elixir's easy mode :D
But am I the only one that thinks this function is kinda weird, in that it handles the response having multiple "link" headers, and maybe not all of them have the pagination data we need? Wish the commit/PR had more info on why the implementing dev thought that was necessary. Or maybe this is documented behaviour of the pagination headers?
Either way, I'm definitely not asking you to rebuild the whole function :D
There is definitely an opportunity to add a test here that you may want to take, tho.
Also, this is a case where the commit message subject you have is the "what", not the "why", so another pass there would be nice. Another thing to keep in mind is that the code you are touching here is generic pagination functionality that could be used for any resource for any reason, so we probably don't want to reference the details of your particular usage in the commit message body (or the PR description, for that matter).

Might be worth adding that http headers are case-insensitive
So it's fairly common to only work with the lowercase version to avoid confusion, which is what it looks like shopify-api-js is doing

Might be worth adding that http headers are case-insensitive
So it's fairly common to only work with the lowercase version to avoid confusion, which is what it looks like shopify-api-js is doing

Great point @Waynenabors. We should almost certainly be downcasing all our headers early in the response handling process. Out of scope for this work, but maybe a TODO somewhere? Or a new issue? Thoughts @hez?

We dug at this yesterday and implementation here is probably cribbed from Plug here. We could write our own case-insensitive header find function, but that exceeds this bug patch, I mean unless @IuriiZaruba feels up to the task (PS I could 🍐 with ya on it).

Thank you, Hez; let's have this merged and work on the function for headers on our 1 on 1 time.


CryptoService is unsafe by design

CryptoService is not using the "secret" parameter(password) set on init(...) method for ciphering or deciphering. Instead, it's using a salt and a iv parameter that are returned plain ( hex encoded ) at the end of the encrypted text.
Instead of using the provided "secret", the code is generating a SecretKeySpec from the salt.
Salt and IV parameters can be easily obtained:
            byte[] salt = cypherText[-32..-1].decodeHex()
            byte[] iv = cypherText[-64..-33].decodeHex()

The secret key can then be easily regenerated:
        SecretKeyFactory factory = SecretKeyFactory.getInstance("PBKDF2WithHmacSHA1")
        KeySpec spec = new PBEKeySpec(secret.toCharArray(), salt, 20000, 128)
        SecretKey tmp = factory.generateSecret(spec)
        SecretKeySpec secretKey= new SecretKeySpec(tmp.getEncoded(), "AES")

And then, the text can be decrypted ( without knowin the password set to the CryptoService!! )
        Cipher cipher = Cipher.getInstance("AES/CBC/PKCS5Padding")
        cipher.init(Cipher.DECRYPT_MODE, secretKey, new IvParameterSpec(iv))
        byte[] decValue = cipher.doFinal(text)
        return new String(decValue)

On it, let me see...

Mmm... I don't think this is right.
The generateKey method uses the salt AND the secret:
KeySpec spec = new PBEKeySpec(secret.toCharArray(), salt, 20000, 128)
In fact, you use that exact line on your example also! So, in order to regenerate the key, you need the secret.
You can easily test it. First, encrypt a text with the secret "secret1". Then, change the secret to "secret2". Then, try to decrypt the text. You won't be able.

I think you are confused by the unfortunate choice of variable names. It is true that the cipher uses a local variable called "secret". In order to avoid misinterpretations, it would be better to rename the local variable of the line that you mention to:
SecretKeySpec secretKey = generateKey(salt)
Anyway, if you look inside the "generateKey" function, on the line 45 of the file, you can see that the key is generated using the "secret" attribute, not the local variable. Again, it would be less confusing to use "this", so we can change that line to:
KeySpec spec = new PBEKeySpec(this.secret.toCharArray(), salt, 20000, 128)
I hope it is more clear now.


Adds includeSolutions option to 'flow list' command

Added new option --includeSolutions to include flows that are contained within a solution.
Added some logic to remove duplicates, which was not included in the previous PR. Context: https://github.com/pnp/cli-microsoft365/issues/4865#issuecomment-1546366473

Closes #5058

ready to merege 🚀

merged manually. Awesome work 👏


Plasticity sometimes converges to zero displacement

The related bug report for ContactMechanics is here: https://github.com/ContactEngineering/ContactMechanics/issues/65

It turns out this is a simple but pretty serious bug in our plastic contact logic. The problem is a follows: We try to automatically determine a set of displacements such that contact area and force are roughly equally spaced in log-log plot. This involves a heuristics that jumps back and force in displacement. Subsequent calculations are include plastic deformation from prior runs. This would not be a problem when simply moving into contact (you can do that for example by providing manually a range of pressures), but it is a problem if the code jumps out of contact occasionally. The solution is to reset the plastic deformation before each subsequent contact step.
This contact logic has now been implemented as a pipeline function in ContactMechanics, see here https://github.com/ContactEngineering/ContactMechanics/pull/66, where this bug has been fixed.


def has HasError and HasCleanup

And, remove componentItem struct from translate/to_runner.go

get method name from def?


Running with "Test Impact Analysis"

First of all, sorry, this is more a question than an issue or anything else, but I'm curious whether someone has already tried combining kotlintest with https://github.com/rpau/junit4git? I'm not saying I'm expecting it not to work, but I wanted to know about any previous experience before I spend the time to give it a try.

Interesting.
Never heard of it! I'm not even sure how it would work with KotlinTest

Might be a good idea for a feature for 3.3 ?

Maybe.
But I'm not sure if many people use that Junit4git. There are very few stars in Github and very few forks.
I'm afraid to put too much effort into adding this feature and nobody using it after all :x

Should be easy enough to add though.

Never heard of it!
But I'm not sure if many people use that Junit4git.

See, @Kerooker, how could you use it if you (and other people) have never heard of it 😉 More seriously, you indeed need to know what you're looking for to find this sort of thing, or you need to be following Martin Fowler's blog. But the idea makes absolute sense to pursue, IMO.
Anyway, my hope indeed is that it's easy to use with kotlintest. Note that I'm not necessarily asking for adding generic Test Impact Analysis support to kotlintest. I'm rather interested to see what it takes to make the existing junit4git project work smoothly with kotlintest.

That project can't work as it uses JUnit specific stuff.

Well, and so does kotlintest (if using kotlintest-runner-junit5), or? Ideally, I would have expected it to "just work" when using both kotlintest-runner-junit5 and junit5git artifacts. If that's not the case for whatever reason, what would be the path to a potential implementation? A new kotlintest-runner-junit5?

Don't forget there's junit5 platform and junit5-jupiter. The latter is what you use if you're writing junit tests, and what provides things like the junit test rules or whatever they're called now.

@Kerooker, please always add a rationale when closing issues.

Oh, this one was by mistake.
I'll reopen, sorry


Soft fail when trying to send to same domain

Hi guys,
We've changed our Postal setup, removing the relay we had, to a new deploy in a new data center. I got the DNS working correctly -- I can send e-mails from my domain to gmail, perfectly.
But if I try to send to the same domain, I get a soft fail:
Temporary SMTP delivery error when sending to mx.mydomain.redacted (ipv6:of:my:public:mx)
452 4.5.2 Could not resolve sender domain.
Time: 0.03s
Support Ref: JO3UUCVL
The VM is using Google's public DNS, it has IPv6 support enabled, but no address (except fe80::). The domain is correctly resolved to the IPv6 address in the error message. Everything is configured as the default in postal.yml, except the DNS address entries.
Should it happen like this, or do I have to alter any other configuration that I'm missing so far?
Thanks!

I would presume that if the VM does not have a real and public IPv6 address, it will not be able to communicate to mx.mydomain.redacted.
I would work with your VM provider to prove, fix or remove the IPv6 connection.


Game crashes when used remote filesystem for resources.

When I load remote resources, the game crashes on loading time.
I narrow down and found when net response is data type in FileAccessNetworkClient::_thread_func(), and its FileAccessNetwork fa memory is corrupt, such as fa->page_size is 0, or fa pointer is incorrect.
	FileAccessNetwork *fa=NULL;

	if (response!=FileAccessNetwork::RESPONSE_DATA) {
		ERR_FAIL_COND(!accesses.has(id));
	}

	if (accesses.has(id))
		fa=accesses[id];  //Get FA

...
fa->_set_block(offset,block);  // APP crashed in this function.

No answer, so archiving as there isn't much that can be done here if it can't be reproduced.


ObjectAL for iPhone to AudioKit

ObjectAL for iPhone is deprecated and not in swift programming language and we have decided to remove and try audio kit so we are looking suggestion for moving ObjectAL for iPhone to AudioKit.
Here is the link for more reference
https://github.com/kstenerud/ObjectAL-for-iPhone
and it provided below main features and

Play, Pause, Stop, Loop
Volume, Pitch, Pan, Mute
Maybe some "power user" stuff, such as preloading and session control

We wanted to know whether that is possible in AudioKit before moving to AudioKit.
Can you help me with this?

You can definitely use AudioKit for those features. I'm not sure how this is an "issue" with AudioKit though. Would you like to come to our AudioKit Slack group and perhaps find people who can help you? If so, give me your email address.

Thanks for quick reply. I can provide email address. Can you send invitation to
<EMAIL_ADDRESS>


Caching documentation isn't up to date

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
In the documentation of the caching mechanism, there is GetSession which isn't implemented in the V12.13.2. There is only a watch function implemented in the interface. But it isn't clear if we still have to have a reference to the session Disposable
Here is the link to the file:
https://github.com/ChilliCream/hotchocolate/blob/bbfc832472e9a4630db4a4883a939e2a760375d1/website/src/docs/strawberryshake/v12/caching/index.md?plain=1#L58
Steps to reproduce

Go to Documentation
if you try to implement the piece of code will not work due to the fact that GetSession isn't implemented anymore

Relevant log output
there isn't a relevant output

Additional Context?
Modify the Documentation
Product
Strawberry Shake
Version
12.13.2

GetSession in this example is a generated piece ... this might have a totally different name in your schema.


Update run-programs-automatically.md

The run these programs at user logon is saved elsewhere.

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

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


Discussion on balancing for option swapping

There's been some discussion on discord already, but good to have it in a more lasting form. Some points to cover:

Should we allow users to fully customize left click ops?
Should we allow users to fully customize shift click ops?
Is swapping ops on inventory items okay?
Is swapping ops on equipped items okay?
We want to avoid changing XP rates, but how far do we go with that? Do we just not allow specific items to be op swapped, or do we extend that to teleportation and utility items too since they have an indirect effect on XP rates?
What could be done to mitigate the effects of changing XP rates?
What precedents have been set by other clients that we could follow?

My personal opinion on the whole thing is we should just do whatever we want with it, barring obvious overpowered stuff.
If Jagex doesn't like any of the more questionable ones, they will have us change it or they will change things on their end to make it stop working.

OSB already has had customizable shift click options, which I would very much like this in RuneLite as well. I've mentioned before that if there's nobody working on it by next week I'll probably pick it up myself.
As for equipment tab clicks, those items have different options from their inventory ones, e.g. ardy cloak has "teleport" in inv which opens an interface where you select your destination, and has the destinations in the equipment tab.
I'm not against making a plugin to swap the ops of equipment, as long as it does not give the user an unfair advantage exp/hr wise. Swapping ops for specific items which don't give an unfair advantage is fine tbh.
When the menu-entry plugin was added there was also a discussion on what options would be allowed since the same applies there. As long as we're not giving users an unfair advantage, it will be fine.

Also commenting on "if jagex doesn't like it, they will have us change it" that would be correct but I'd rather avoid having jagex contact us at all, and us not pushing the boundaries of what they deem to be "OP"

Yeah there's a balance we need to find between doing too much and having Jagex get upset with us, and crippling our features unnecessarily for fear of Jagex getting upset with us when they don't actually mind.
I'd say a happy medium in this case could be to allow fully user customizable shift-click ops in the inventory and equipment panel, and for the rest of the game change the left click ops only in places that aren't too OP. (Left click Trade on a shopkeeper npc is probably fine, left click Buy 50 is probably not. Left click Pickpocket on npcs with a Talk-to op is debatable, but other clients already offer that.)

I mentioned this on discord earlier, but I think a good starting point for the boundaries of "op" is to disallow 1-click ops for actions where rapid, repeated use gives an obvious advantage. Shop buying commands would fall under that category, and so would pickpocketing (whether we allow this because other clients do is the next question).
By contrast, a 1-click op on an item the player might use no more than once every ~20 seconds wouldn't really save them any gameticks.

Yeah, that's a fair distinction to make. Going by how often you click the option is probably the best way to decide which ones we want to not allow or put restrictions on.

Shift click config is a must have for high level players. It helps with click intensive skill methods.
As for how far Runelite should go. I would just mimic Osbuddy when it comes to questionable things like this. Since jagex has the hard on for osb.


Custom background shape/image in launcher icon

It would be handy to be able use a custom background shape. Likewise how you atm can have a custom foreground image.

With the move to adaptive icons this is basically obsolete :-/


An error occurred when using defineProps<>.

Version
3.2.22
Reproduction link
sfc.vuejs.org/
Steps to reproduce
Create one interface file with typescript type.
Import the file from the component(. vue) that definesProps.
Define props as follows.
import type { GroupTitleLargePageData, GroupTitleLargeProps } from "@/interfaces/components/groups/GroupTitleLargePageTypes";
const props = withDefaults(defineProps(),{});
------------------- error ----------------------------
[plugin:vite:vue] [@vue/compiler-sfc] type argument passed to defineProps() must be a literal type, or a reference to an interface or literal type.
D:/project/hamonica/workspace/hamonica-fe-type/src/components/groups/GroupTitleLarge.vu
14 |  const props =  withDefaults(defineProps(),{});
|                                          ^^^^^^^^^^^^^^^^^^^^
What is expected?
I want to call and use the props interface through import.
What is actually happening?
------------------- error ----------------------------
[plugin:vite:vue] [@vue/compiler-sfc] type argument passed to defineProps() must be a literal type, or a reference to an interface or literal type.
D:/project/hamonica/workspace/hamonica-fe-type/src/components/groups/GroupTitleLarge.vu
14 |  const props =  withDefaults(defineProps(),{});
|                                          ^^^^^^^^^^^^^^^^^^^^

i use vite + typescript

duplicate of #4294

look: https://github.com/vuejs/vue-next/issues/4294#issuecomment-896082769

@ygj6  Okay, I got it. I was told it might be supported later. May I know the schedule?

duplicate of #4294

Has it been fixed? I meet the same problem @v3.2.13

no. I know it's not supported yet.
-----Original Message-----
From: @.>
To: @.>;
Cc: @.>; @.>;
Sent: 2022-03-27 (일) 16:06:49 (GMT+09:00)
Subject: Re: [vuejs/core] An error occurred when using defineProps<>. (Issue #4982)
duplicate of #4294
Has it been fixed? I meet the same problem @v3.2.13
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


RET-4926: New Counter for WA Task - Test Version

Jira link (if applicable)
https://tools.hmcts.net/jira/browse/RET-4926
Change description
Add a counter so that the ET3 Processing Task would only trigger once.
Checklist

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

Closing as not being used afterall


Error when opening a sidebar

I'm using this rendering configuration
    rendering = {
        max_buffer_width = 20,

        left_sidebar = {
            filetype = 'NvimTree',
            components = {
                text = 'NvimTree',
                hl = {
                    fg = get_hex('Normal', 'fg'),
                    bg = get_hex('NvimTreeNormal', 'bg'),
                    style = 'bold',
                },
            },
        },
    },

Whenever I open NvimTree (with NvimTreeToggle), I get the below error.
E5108: Error executing lua ...ack/packer/start/nvim-cokeline/lua/cokeline/sidebars.lua:46: attempt to index a nil value
stack traceback:
	...ack/packer/start/nvim-cokeline/lua/cokeline/sidebars.lua:46: in function 'get_sidebar_components'
	...ack/packer/start/nvim-cokeline/lua/cokeline/sidebars.lua:85: in function 'get_left_right_sidebar_components'
	...ck/packer/start/nvim-cokeline/lua/cokeline/rendering.lua:186: in function <...ck/packer/start/nvim-cokeline/lua/cokeline/rendering.lua:172>

The error is coming from the second of these lines:
    components[#components].text = components[#components].text
      .. str_rep(" ", space_left)

After this error is dismissed the bufferline goes back to default, but works as configured before the error is triggered.

Should be fixed in 0.3.0. Note that the sidebar api has changed, see the CHANGELOG for details. Let me know if you still have issues.

Seems to work now, and the consistency with the other components field is very nice!


Use brackets instead of test

A neat alternative to using test is putting brackets around the file you want to test (along with the attribute for the test). Has the same effect as test.

Shouldn't there be spaces around brackets?

Oh yeah sry, I'll make a fix for it.


Image upload plugin missing WIDTH & HEIGHT

Hi
Although the images uploaded on the browser are mostly the optimized ones. But we should still have an option of specifying the image width and height as a good HTML code ethic. This way even if someone doesn't want to optimize the image, he can make minor changes on the real image size through the view source of the editor.
Thanks
Sandy

Don in the new merge #663

Nope, you only have fixed for core image, not upload plugin ^^ But a copy paste should works :)


Rewrite reorder fields assist to use mutable syntax trees

This also instead uses Either to use the typed RecordPat and RecordExpr nodes, this unfortunately gives a bit of code duplication

bors r+

changelog internal


Modify add test

Codecov Report

Merging #2 (6b5f32b) into main (186b689) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   90.39%   90.40%
=======================================
  Files         144      144
  Lines        8539     8543    +4
=======================================
+ Hits         7719     7723    +4
  Misses        820      820

Impacted Files
Coverage Δ

src/data/literal.rs
83.69% <100.00%> (+0.28%)
:arrow_up:

Continue to review full report at Codecov.

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


E2s changing back to the old version when loading a saved and (seemingly) updated version

So basically, sometimes whenever I'm workin on something e2 related and I'm changing the e2 to something new, once I save it its all done, and then I load the save, the e2 is back to the old e2 before I changed it! Thing is, this has only really happened to me in multiplayer (and I only play on one server), and it seems no one else has ever complained about it. Unless its probably an internet related issue, then dang.

Check your data folder, specifically the expression2 folder. Maybe the files are in read-only mode which would prevent gmod from writing to them. If so, disable read-only mode. After pressing the save button in the e2 editor, you can open that file in an external editor to make sure it was saved properly.

It sounds like he's updating the chip, but after pasting with the updated code saved, it's going back to the old code.
One problem that has been known to cause this is trying to set an already existing entity modifier. I don't think I've seen this issue occur with e2 before though.


Can not login after upgrade to 0.38

Home Assistant release (hass --version):
0.38
Python release (python3 --version):
3.5.3
Component/platform:
http
Description of problem:
After upgrading to the 0.38 release I can not log into Home Assistant anymore. All I got is the password promt and after entering the password I am getting a spinning circle and the text "Connecting" below it. This also happens even if I turn off the auth/password feature.
In the syslog there are some entries like:
INFO:homeassistant.components.http:Serving /api/websocket to <IP_ADDRESS> (auth: True)
Expected:
Successful login and full funtional Home Assistant web interface.
Problem-relevant configuration.yaml entries and steps to reproduce:
http:
  server_port: 8080
  server_host: <IP_ADDRESS>

What happens if your clean the cache of your browser and a new try?

May be related to https://github.com/home-assistant/home-assistant/issues/5887
It is fixed in 0.38.1

Cleaning the cache doesn't help.
I just upgraded to 0.38.1 and the problem isn't solved.

Do you use a self-signed SSL certificate? I think I have the same issue but I have resolved it temporarily by removing my SSL certificate.

No I currently don't use SSL.
I noticed that after a while the circle disappears and the password/login dialogue shows up, although I deleted the password line in the  config.

Using 0.38.1, I can reproduce the same problem @bsqrl has, when using a self-signed SSL certificate. Disabling it, login works.

I forgot a very important detail in my configuration: I use nginx as a reverse proxy like it's described in the official documentation (except the SSL/TLS part).
It turns out that this seems to be the problem for me. If I change the binding interface and connect to Home Assistant directly, it works just fine.

I'm also having the same issue as described by @bsqrl - I have a Squid reverse proxy, and no SSL.
As above, connecting directly to x.x.x.x:8123 works fine

I'm having a very similar issue but it's only from one computer (tried chrome, firefox, IE). I put in my password and it spins as if logging in and then says 'Unable to connect". I've tried my domain mask and the ip address directly. I have no SSL or any other proxy settings on my raspPI. Works fine from other computers I've tried and my android phone.

I corrected this by changing how I had my Proxy set up. I followed this example configuration and it started working. https://home-assistant.io/ecosystem/nginx/

Issue has been resolved for me, it wasn't an update that fixed it but rather I upgraded the computer in question from win 7 to Windows 10 and it started working again. So likely a bad configuration on my machine.


error

https://github.com/user-attachments/assets/a9f9629f-b14a-4eba-a5f8-6bfc707d42b7
Please tell me what the problem is, acodex works but still gives an error, maybe I didn’t download some additional package?
Or is there another option to solve this problem?
HarmonyOS 3.1.0

Both are different things , you can't combine both.

Btw this is plugin specific issues, open it there

I didn't understand you

I mean python plugin and AcodeX plugin is different, you can't combine them .
If you have installed module in AcodeX then run your code through Acode using command like python


Add Dockerfile

Dockerfile for building on-dhcp-proxy image.

*** BUILD #170 ***

test this please

:+1:

@RackHD/corecommitters @heckj

+1


sysapps: Rewrite the check for UDPSocket::SendTo's return value.

Drop the check for whether it has returned the number of bytes we are
trying to send (which includes a comparison between a size_t and an
int) since it does not make much sense. We only need to contemplate 3
different cases:

SendTo returns ERR_IO_PENDING, which means it is still transmitting
all the data.
SendTo returns another negative value, so we just bail out.
SendTo returns a positive value, which means everything was
transmitted, otherwise ERR_IO_PENDING would have been returned.

@darktears @heke123

LGTM

LGTM
I don't think base::checked_cast is unacceptable, Line 270 has another base::checked_cast.

Nobody said it's unacceptable, it's just unnecessary if the code can be rewritten in a more clear manner.


1437 logging reduction

Pull Request Overview

Removed a load of supurflous trace that was making debugging really difficult
There is almost certainly more that can be removed but this covers a large percentage of the logging

Coverage is low intentionally as the code that is 'new' is the logging code that is now disabled so is INTENTIONALLY not run


Interface Connections not displaying

When viewing the "Interface Connections" page, no connections are being displayed for interfaces that are connected to each other.
I have something like this
DEVICEA -- INTERFACE -- CIRCUIT -- INTERFACE -- DEVICEB
Both devices show the interface is attached to a remote device over a circuit.

This is expected behavior. Logically, interface connections are represented as
Interface <-- InterfaceConnection --> Interface

whereas circuit connections are represented as
Interface <-- CircuitTermination <-- Circuit --> CircuitTermination --> Interface

This alternate model is necessary because each circuit termination may or may not be attached to a particular interface.
While I understand your reasoning, the two are very different types of connections, so I don't think it makes sense to include both in the same list. This is probably better solved by enhancing the circuit import/export ability (see #815).

OK Thanks, I was wondering if I was going mad, there was a bug or I was doing something wrong :)
Maybe something one could look in to adding in the future for additional reporting/auditing.


How's FixMatch compared to supervised methods?

Thanks for sharing the great work.
It might be interesting to compare FixMatch with supervised learning methods, given the same labeled data and model structure, to further demonstrate the power of SSL.
I appreciate if you could comment on the idea or release experimental results on such comparison.
Regards,

I'm not quite sure I follow the question. Do you want to know how well our methods do in comparison to a fully supervised baseline that uses, say, just 40 or just 250 images? This is given in Table 9 and Table 10.
If instead you mean fully supervised on the entire dataset, then these numbers are reported in our ReMixMatch paper Appendix Section 4.1
To begin, we train a fully-supervised baseline to measure the highest
accuracy we could hope to obtain with our training pipeline. The experiments we perform use the
same model and training algorithm, so these baselines are valid for all discussed SSL techniques. On
CIFAR-10, we obtain an fully-supervised error rate of 4.25% using weak flip + crop augmentation,
which drops to 3.62% using AutoAugment and 3.91% using CTAugment. Similarly, on SVHN we
obtain 2.70% error using weak (flip) augmentation and 2.31% and 2.16% using AutoAugment and
CTAugment respectively. While AutoAugment performs slightly better on CIFAR-10 and slightly
worse on SVHN compared to CTAugment, it is not our intent to design a better augmentation
strategy; just one that can be used without a pre-training or tuning of hyper-parameters.

Actually I meant both. Thanks for the considerate response.
I implemented FixMatch with RandAugment in my own codebase and on a classification task, with real world data and imbalanced distribution (4 classes, majority:minority up to 50:1). The training dataset is composed of balanced/imbalanced labeled data and imbalanced unlabeled data (sampled randomly from the entire dataset).
I tried a coarse grid search for hyperparameters (weight decay, tao, lr, lambda_u, range of augmentations), but got no luck. FixMatch didn't really help to improve classification performance compared to a purely supervised model. I have a few observations:

With too strong augmentation, the model cannot learn much from pseudo labels. Given large training epochs, the "unlabeled loss" multiplied by lambda_u, i.e. L_u * lambda_u, still maintains a high value and oscillates fiercely, despite that the "labeled loss" quickly converges. L_u * lambda_u and L_s start from the same order of magnitude, but differ by 1 - 2 orders of magnitude in the end.

With balanced labeled data the training set, FixMatch achieves very similar precision but worse recall compared to naive supervised training. With imbalanced labeled data, FixMatch gives better precision but significantly worse recall.

Here are my guesses, please correct me if I'm wrong.
First, FixMatch should start the training process with a good or at least mediocre pre-trained model to prevent low quality pseudo labels introducing too much noise to the learning process. Second, FixMatch seems to have problems adapting to imbalanced classification tasks. The majority class is quickly overfitted. The randomly sampled unlabeled data cannot provide much useful info via consistency regularization and pseudo labeling. Third, hyperparameter tuning (maybe among all, augmentation range tuning is the most important) is vital and difficult in adaptation.
I appreciate any comment or suggestion for improvements. Thank you very much.

Yeah, unbalanced datasets are so far something that SSL really struggles with. We've found distribution matching from ReMixMatch can help here, but I don't know of any answers. My guess is this is less of any issue with fixmatch-as-expected, and more of an open research question for how to solve unaligned data distributions better.


version 0.6.0 is buggy

I have a solution with both .net and .net core projects. And we have several nuget repos (nuget.org and a private one). Here is my code:
let nugetRestore = require('gulp-nuget-restore');
let { restore } = require('gulp-dotnet-cli');
...
gulp.src("123.sln")
.pipe(nugetRestore())
.pipe(restore())
.pipe(msbuild({ ... }))
previous version 0.5.0 works fine. Latest one - 0.6.0 doing something wrong, after its nugetRestore(), packages are conflicting each other and build not working.

I forgot to update the release notes for 0.6.0. That contained an update to nuget.exe to 4.1.0. Could this be a breaking change in nuget?

If you make a repo in github I'd be happy to help you debug this


Evan/commit to extended data2

Description
the same branch as #540, but trying with a different branch to see if it fixes the CI
Closes: #XXX

it wasn't needed, as the CI magically fixed itself in #540


Update bibtex to reflect team growth

cc @BenjaminBossan @SunMarc 🤗

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Fix: replaced button in the form

I'm doing refactoring now, later I will add a unit tests in a separate commit

in the next commit, I will add onSubmit and new tests

@maxvektor


Embedding - set scatter colour when 'color' (by obs/var) is not used

[ x] Additional function parameters / changed functionality / changed defaults?
[ ] New analysis tool: A simple analysis tool you have been using and are missing in sc.tools?
[ ] New plotting function: A kind of plot you would like to seein sc.pl?
[ ] External tools: Do you know an existing package that should go into sc.external.*?
[ ] Other?

Add parameter for setting embedding/UMAP colour when the "color" parameter is not used - e.g. set one colour for all points. Currently, the only option to set colour for all cells seems to be to add additional constant obs feature and modify uns to set the colour for the new feature.

Soon, you can do:
import scanpy as sc
pbmc = sc.datasets.pbmc68k_reduced()
sc.pl.umap(pbmc, na_color="purple")

Is this what you're looking for?

Soon, you can do:
import scanpy as sc
pbmc = sc.datasets.pbmc68k_reduced()
sc.pl.umap(pbmc, na_color="purple")

Is this what you're looking for?

@Hrovatin, I think this has been addressed. Let me know if I should reopen this

@Hrovatin, I think this has been addressed. Let me know if I should reopen this


Add Code Review to Contributing.md #117

Fixes #117
Checklist

[ ] I have made a material change to the repo (functionality, testing, spelling, grammar)
[ ] I have read the [Contribution Guide] and my PR follows them.
[ ] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation about the functionality in the appropriate .md file
[ ] I have added in line documentation to the code I modified

Short description of what this PR does:

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

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

Thanks for the review @hugovk!

Hello @derekneuland,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


certificate errors all over

hi j.
just tried your app and getting x509 errors while trying to run it.
~/scripts/gocode/bin$ ./weather
Http request to https://geocode.jessfraz.com/forecast failed: Post https://geocode.jessfraz.com/forecast: x509: certificate signed by unknown authority (possibly because of "x509: cannot verify signature: algorithm unimplemented" while trying to verify candidate authority certificate "COMODO ECC Certification Authority")

same issue here.

probably the go version should work if you install from  source


#include <sys/mman.h> not found

I was trying to do  something strange but i'm unable to include sys/mman.h file . When compiling with this header file i got error file not found. Can somebody help me ??

Generally speaking if it's not part of the C or C++ standard libraries, the header is not available.  In the case of mman.h, it will not be available because nothing in it is useful in the context of a contract.  Memory mapped files are not allowed because using them can have nondeterministic results.  Contracts are not allowed access to the filesystem, the network, or to real time clocks.


keyclu wont launch at login

Hi there,
I have a problem, in keyclu setting the launch at login option is on, and even if i add keyclue to the launch at login in the system preference of my mac, after rebooting my mac it won't launch and keyclu is deleted from the list in my system preferences.
Any idea what's up ?

Hello ObedH26,
Thanks for reporting the issue. Could you please let me know which macOS version you're currently running?

Good evening, looking at my safari history i got it from github one month ago redirected from this specific website : https://sergii.tatarenkov.name/keyclu/support/
i am running macOS Sonoma 14.0 on an M1 pro
Have a good one thanks for reaching me

Hello, I've attempted to reproduce the issue, but unfortunately, I couldn't replicate it in VM. My research online indicates that this issue might be related to the operating system itself, and it may require adjusting certain permissions or removing specific files (depending on your specific case). I'm sorry that I couldn't provide a solution, but feel free to ask if you have any other questions or need assistance with something else.

Hey, thanks for your answer, no problem ill let know here if i find a solution

I have exactly the same problem, also running Sonoma on M1 Pro.


[ContractKit] Add retry and backoff around fetch metadata

Description
Add retry and backoff around fetching metadata. Seen a few occasions where the services hosting validator metadata will return transient errors, such as connection resets, in which case an attestation attempt from Valora or a status check from the metadata crawler will fail and not retry.
Tested
Locally
Backwards compatibility
Yes

Can we do this at a calling site? This to me doesn't seem like the library should make that choice

Can we do this at a calling site? This to me doesn't seem like the library should make that choice

I think it complicates the calling service (for example CK uses this to figure out if a set of issuers are valid -- and the calling service would then have to retry ALL of them, if one metadata fetch failed).  Plus I feel that the library is a fine place to work around transient failures. That's exactly what the the blockchain tx pool does, for instance.  Looks like alchemy have extended that philosophy to web3 -- would be interesting to consider that.

for example CK uses this to figure out if a set of issuers are valid -- and the calling service would then have to retry ALL of them, if one metadata fetch failed

I'm not sure how this applies? Retrying with asyncWithBackoff seems quite easy to use?
Fwiw, not a strong opinion on my side

I'm not sure how this applies? Retrying with asyncWithBackoff seems quite easy to use?

It's easy, but if I wrap say getActionableAttestations with asyncWithBackoff from the calling client, then if I request say 3 attestations and 1 call to  fetchFromURL fails, then I need to re-make more view calls and all 3 calls to fetchFromURL. Does that make sense?

That makes sense. How about an optional parameter specifying the retry logic of network calls within?


update pdf document loaders' metadata source to url for online pdf

Description: Update 5 pdf document loaders in langchain.document_loaders.pdf, to store a url in the metadata (instead of a temporary, local file path) if the user provides a web path to a pdf: PyPDFium2Loader, PDFMinerLoader, PDFMinerPDFasHTMLLoader, PyMuPDFLoader, and PDFPlumberLoader were updated.

The updates follow the approach used to update PyPDFLoader for the same behavior in #12092
The PyMuPDFLoader changes required additional work in updating langchain.document_loaders.parsers.pdf.PyMuPDFParser to be able to process either an io.BufferedReader (from local pdf) or io.BytesIO (from online pdf)
The PDFMinerPDFasHTMLLoader change used a simpler approach since the metadata is assigned by the loader and not the parser

Issue: Fixes #7034
Dependencies: None

# PyPDFium2Loader example:
# old behavior
>>> from langchain.document_loaders import PyPDFium2Loader
>>> loader = PyPDFium2Loader('https://arxiv.org/pdf/1706.03762.pdf')
>>> docs = loader.load()
>>> docs[0].metadata
{'source': '/var/folders/7z/d5dt407n673drh1f5cm8spj40000gn/T/tmpm5oqa92f/tmp.pdf', 'page': 0}

# new behavior
>>> from langchain.document_loaders import PyPDFium2Loader
>>> loader = PyPDFium2Loader('https://arxiv.org/pdf/1706.03762.pdf')
>>> docs = loader.load()
>>> docs[0].metadata
{'source': 'https://arxiv.org/pdf/1706.03762.pdf', 'page': 0}

@eyurtsev: you reviewed and merged #12092 about a month ago.  This PR has similar enhancements to other pdf document loaders.  Requesting a review or a pointer in the right direction for contributing these updates if tagging you isn't the right way.  I can separate out some or all of the document loaders in this update if the team doesn't want everything in here or everything at once.  Thank you!


Rerender to try to fix windows linking issue

It appears the last render didn't successfully build for windows. See https://github.com/conda-forge/gdal-feedstock/issues/616 for details.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Pint fails when using new PHP 8.3 features

Pint Version
1.13
PHP Version
8.3
Description
Pint fails when using new PHP 8.3 features like:

readonly class
typed constants

Steps To Reproduce

Add readonly before class
readonly class FacebookAdAccountService implements IntegrationAdAccountService
Add type to constant:
const string VIEW_STORES = 'view stores';

readonly works for me, typed constants not. We'll get the dependencies updated for tomorrow's release but in this is mainly a PHP CS Fixer issue.


cli.js not found when installing locally

With reference to this commit: https://github.com/nearprotocol/near-shell/commit/84c126831f34301ae51e833dcf854efb50540917 the shell command has been split into cli.js and near.
However, when installing this package, only the near file is copied into node_modules/.bin:
https://github.com/nearprotocol/near-shell/blob/master/package.json#L22
As a result, the cli.js file is not found when executing near shell commands.

I believe this is fixed now https://github.com/nearprotocol/near-shell/commit/5aded68991682d30426e1d3cd7f3a05e6fb05c2c, feel free to re-open if not.


Starting and shutting down accesslog with kmeshctl

What type of PR is this?
/kind enhancement

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #925
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/cc @YaoZengzeng  PTAL

/lgtm
/approve


Add performance testing scripts and job for release workflow

Motivation
It'd be very useful to monitor our perfomance metrics on releasing candidates. It could help to catch performance issues.
TODO

[ ] check scrits (AWS2AWS)
[ ] prepare scripts for githunb workflow
[ ] add perormance testing step on releasing (kind2kind)

The action plan:

[x] Run MSM perf test scripts manually with AWS2AWS scenario
[ ] Add vl3 test

Adopt deployment configs from MSM testing ~4h
Run the test manually to ensure that it works ~4h

[ ] Add interdomain kernel2wireguard2kernel test

Adopt deployment configs from MSM testing ~4h
Run the test manually to ensure that it works ~4h

[ ] Add test scripts and deployments to this repo ~2h
[ ] Add CI configs to run test scripts ~2h
[ ] Trigger CI testing to ensure that everything works ~1h

Seems like done


[Select] Add ability to position as popper

Fixes #1247

Does this also contain a fix for non-modal mode?

@Andrii-Antoniuk, it might be possible in the new positioning mode now, but will be in another PR. We'll prioritise releasing this change for now.

I didn't review the positioning logic from the previous implementation as I assume nothing changed there?

No it hasn't, if you look at each commit you should be able to see it's just moving stuff around.

@andy-hook I've just realised also that I made onPlaced a private prop for Select but haven't touched the other components that use Popper so these would have it as public.
Do you think it's worth adding to the public API? Or should I omit it from the other primitives too?

Do you think it's worth adding to the public API? Or should I omit it from the other primitives too?

We should omit it until needed

When do you have a release date? :)  Thanks for your work! :heart:

It should be released tomorrow if all goes to plan!


WIP: Debug PIP Download Issue.

High Level Description

We'll to know the reason, why we need https://github.com/dcos/dcos/pull/1451.

This was helpful in debugging.  The underlying reason we had a failure was, we had "several" but certain specific set of packages for azure in dcos-image
        # Pins taken from 'azure==2.0.0rc4'
        'msrest==0.4.0',
        'msrestazure==0.4.1',
        'azure-storage==0.32.0',
        'azure-mgmt-network==0.30.0rc4',
        'azure-mgmt-resource==0.30.0rc4',

In this 'azure-storage==0.32.0' had a open azure-common dependency, and picked up the latest version of azure-common from internet.
[00:09:49] :		 [setup] Collecting azure-common (from azure-storage==0.32.0->dcos-image==0.1)
[00:09:49] :		 [setup]   1 location(s) to search for versions of azure-common:
[00:09:49] :		 [setup]   * https://pypi.python.org/simple/azure-common/
[00:09:49] :		 [setup]   Getting page https://pypi.python.org/simple/azure-common/
[00:09:49] :		 [setup]   Looking up "https://pypi.python.org/simple/azure-common/" in the cache
[00:09:49] :		 [setup]   No cache entry available
[00:09:49] :		 [setup]   "GET /simple/azure-common/ HTTP/1.1" 200 1460
[00:09:49] :		 [setup]   Updating cache with response from "https://pypi.python.org/simple/azure-common/"
[00:09:49] :		 [setup]   Caching b/c date exists and max-age > 0
[00:09:49] :		 [setup]   Analyzing links from page https://pypi.python.org/simple/azure-common/
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/00/39/7b915a03e1a64415c81a8b8f317556182592327d2b62812ef2b19e71b378/azure-common-1.1.4.zip#md5=290a03b3a74417ad2f8c58c2e20ae233 (from https://pypi.python.org/simple/azure-common/), version: 1.1.4
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/08/6d/6f9c6484f6c3615244a86b01a806e5869b407316b0330350856f84460119/azure_common-1.1.1-py2.py3-none-any.whl#md5=12ccfe20d1f68e8809c7918232411efc (from https://pypi.python.org/simple/azure-common/), version: 1.1.1
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/32/10/b353b328deb80e81a9288f0c684e4bbc6b022ec29110c4f90d265b07a371/azure_common-1.1.4-py2.py3-none-any.whl#md5=478d067e700938f874c0d4132e19a767 (from https://pypi.python.org/simple/azure-common/), version: 1.1.4
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/47/a5/c46855c1e85e1d295dd9ee949db40164f7be95f17a0c31d93eecd6241405/azure_common-1.0.0rc2-py2.py3-none-any.whl#md5=7b11eef0e9767328c2f8a6f5ff1b3a13 (from https://pypi.python.org/simple/azure-common/), version: 1.0.0rc2
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/48/89/017df9bed55925de004076937ca168d3a03aa0250fbda7826ec3c0ac714f/azure-common-1.1.3.zip#md5=13471213f49400c5d483865c96558ab9 (from https://pypi.python.org/simple/azure-common/), version: 1.1.3
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/4f/97/6522ac1a9466b186430b0d23dd4babb6e743cf41a1ec08fc75cdb0b3bf5e/azure_common-1.0.0-py2.py3-none-any.whl#md5=0ab5dfd6a8e9551f8eaedf095681de2a (from https://pypi.python.org/simple/azure-common/), version: 1.0.0
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/52/5a/19c32c4f550bd419283992cecc2f58010952b6c6aa2321b821daf777658a/azure-common-1.1.0.zip#md5=69981dd96c5036977babe6352d1902a3 (from https://pypi.python.org/simple/azure-common/), version: 1.1.0
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/55/8a/8c7be55fc54f8d5b20d5cb2e1d730dad0511845d3dcac3c989a2a5352ec7/azure-common-1.1.2.zip#md5=07d40312ecad5e85f8d9a2f1d667daab (from https://pypi.python.org/simple/azure-common/), version: 1.1.2
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/56/a4/6d57b76434552d9f78463d2eea0cf9519870d322407bbb50f02e990917e6/azure_common-1.1.5-py2.py3-none-any.whl#md5=63d2de635b244126329718ed168627a3 (from https://pypi.python.org/simple/azure-common/), version: 1.1.5
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/64/d1/1e736971bc17b806384c0c37d9bb3c76312166c20d015b4d925834316f67/azure_common-1.1.0-py2.py3-none-any.whl#md5=0d2678f8b36e4d88bf814159345af31c (from https://pypi.python.org/simple/azure-common/), version: 1.1.0
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/76/91/5603bc2193d2c862cdd61005123ed198931304b6829e9a92302122f6c789/azure-common-1.0.0rc1.zip#md5=ed1234f7ced20635fc96bfa65d8404fc (from https://pypi.python.org/simple/azure-common/), version: 1.0.0rc1
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/7c/36/d1323c5fdd32d4e871f096ef13af83cd65920e2fcbe639258b71e2c1fd6e/azure-common-1.0.0rc2.zip#md5=5dea490f9f48bf19b8fca04de5d3eae7 (from https://pypi.python.org/simple/azure-common/), version: 1.0.0rc2
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/92/24/add0aa9e79f71a2edc238861ea79536552ce57a331ea1a1a87facccb5ce1/azure-common-1.1.5.zip#md5=e7f3653aedea989754c642a1de561a3b (from https://pypi.python.org/simple/azure-common/), version: 1.1.5
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/a2/2a/9a9799909d0cca299449558d5fd1a1f50abe348c05497772dbfba916c7be/azure_common-1.1.2-py2.py3-none-any.whl#md5=5f1ae39f589a1a53656dc370c67802d1 (from https://pypi.python.org/simple/azure-common/), version: 1.1.2
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/a8/89/b69b74372420ff08291e2ac86ff54b15e3b07132817ff1a138cfc5b70933/azure-common-1.0.0.zip#md5=936f523f8f0e11780fc0b8a87d56d390 (from https://pypi.python.org/simple/azure-common/), version: 1.0.0
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/e4/14/45947940e25ab095b93490dc592b73b2985c2748bee8e9b2f5b2608f610d/azure_common-1.1.3-py2.py3-none-any.whl#md5=22d748f056f7bad87f2627822952f8d7 (from https://pypi.python.org/simple/azure-common/), version: 1.1.3
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/eb/54/8bcc998ca877c2b1120bcfd3ca740f6222d41102d6a43a8209d4bfc820dd/azure_common-1.0.0rc1-py2.py3-none-any.whl#md5=29545f42f6fcc1adb602fdd34c59bdcf (from https://pypi.python.org/simple/azure-common/), version: 1.0.0rc1
[00:09:49] :		 [setup]     Found link https://pypi.python.org/packages/f6/3d/9f433a71a81bbd19d48d14fffe1771f08f0bdbec3404afcf20bfa1f0b571/azure-common-1.1.1.zip#md5=3e3768095fff759ef7faa14b9dbf8bd4 (from https://pypi.python.org/simple/azure-common/), version: 1.1.1
[00:09:49] :		 [setup]   Using version 1.1.5 (newest of versions: 1.0.0, 1.1.0, 1.1.1, 1.1.2, 1.1.3, 1.1.4, 1.1.5)

Note the last line for the winner: 1.1.5

Where as in dcos-image 1.0, azure-mgmt-resource==0.30.0rc4 had a specific dependency, azure-common[autorest]==1.1.4.

It worked previously because 1.1.4 was the latest version.

When the entire azure-sdk sdk got updated, 1.1.5 became the latest version. Our pinning of azure-storage got 1.1.5 where as, the pinning of azure-mgmt-resource was looking for 1.1.4 and failed.

If we see this problem again, I think, a good idea will be depend upon higher level package like azure==2.0.0rc6


Grafana dashboard: Go Prometheus client library

Process status published by Go Prometheus client library, e.g. memory used, fds open, GC details

/lgtm


make server/test/test.html work properly with latest qunit version

Using the latest version of qunit it generate some errors on server/test/test.html

Since QUnit v1.16 there is a global failure verification:
			else {
				QUnit.test( "global failure", extend(function() {
					QUnit.pushFailure( error, filePath + ":" + linerNr );
				}, { validTest: true } ) );
			}

It's possible to comment these lines.
The error happens on Chrome browser but not on Firefox, it seems to be a chrome plugin related issue.

It's not related to wwidd/NDVM.
problem with Chrome and Qunit


Float trapped explicitly: losing precision

Float.div(Float.fromInt(4999999999999999890753), Float.fromInt(Int.pow(10, 8)));
Call was rejected:
Request ID: 81f4f619948b3a19e47d912a644f8772e60decc32ea9179f622f3cb22c56e735
Reject code:
Reject text: Canister 3c7jb-myaaa-aaaab-qacoa-cai trapped explicitly: losing precision

Float
type Float = Prim.Types.Float
64-bit floating point numbers.

Is float only 64 bit enough for business?
Float trapped explicitly: losing  #precision

This smells like https://github.com/dfinity/motoko-base/issues/272; maybe that fix has not yet reached you

An older version of Motoko converted Ints to Floats via Int64, which indeed caused problems. Please make sure that you have the most recent dfx.

I used Motoko Playground to test whether Motoko Playground also uses the old version of the SDK
Motoko Playground

Motoko playground's default base library is only at dfx 0.8.0, which is the latest tag on motoko-base repo. To get base from master, you can click "Add" button at the "Packages" section and reimport "base". Your example works after the reimport.


Preserve content-type when proxying to workers

This is necessary for conveyal/r5#459 to work properly.
The workers can generate responses in several formats including GeoTIFF, proprietary binary grid, or JSON. We want to preserve that information when the response is repeated by the broker back to the UI.

Codecov Report

Merging #183 into dev will decrease coverage by 0.45%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##                dev     #183      +/-   ##
============================================
- Coverage     25.59%   25.13%   -0.46%
  Complexity      103      103
============================================
  Files            58       58
  Lines          2161     2224      +63
  Branches        195      210      +15
============================================
+ Hits            553      559       +6
- Misses         1574     1629      +55
- Partials         34       36       +2

Impacted Files
Coverage Δ
Complexity Δ

...om/conveyal/taui/controllers/WorkerController.java
22.91% <0%> (+0.46%)
6 <0> (ø)
:arrow_down:

...om/conveyal/taui/controllers/BundleController.java
47.59% <0%> (-19.34%)
14% <0%> (ø)

...n/java/com/conveyal/taui/AnalysisServerConfig.java
74.07% <0%> (-3.48%)
3% <0%> (ø)

Continue to review full report at Codecov.

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

If the responses are GZIPed we definitely need to add that Content-Encoding. It looks like that was just removed.

This has been observed to work on staging, merging to dev.


Can send EOS but not IQ tokens (ERROR unable_to_find_key) on 0.5.0RC1

The test case was IQ tokens, repeated multiple times. The same tokens with the same wallet sent fine on 0.4.1. No more details available at the moment.

Thanks! We were able to reproduce and will have it fixed in the next release.


Create a non-React version

@ilan-gold : Would this have been for Indiana? Is this issue still relevant?

We are actively working on integrating VIV into the CCF-UI as a library now. It seems to be going pretty well. @axdanbol can speak to any problems he needs addressed, but those should probably be reported as separate issues at this point.

Thanks. Closing; Let us know if anything comes up.


Consider where to store configuration file (home dir vs work dir)

Received from user feedback
The original design was built around the use case with localization manager. You as localization manager have multiple projects, each one in a separate folder. Each project may have different settings. Smartling credentials are just part of settings. Also, settings may contain file patterns and upload\download configuration. So it's convenient to keep CLI configuration right in a project folder.
One of the feedbacks contained following steps:

Install CLI
Run init command
Switch to folder with test files or create such folder
Run files push command

User got the error that user identifier is not configured. He added user into command line, then he got the error about user secret and so on. So he didn't realize that smartling.yml was created in his home dir, but CLI looks for it in work dir.
DoD:
CLI must work from project root and subfolders. People expect behavior like in git. You can run git pull from root folder (where .git exists) or from any subfolder.
It means that CLI must look for config file in the current folder or parent folders

We continue to use the same order for looking context as described in #31. But we extend the last item - where to search for smartling.yml
If we didn't find the mandatory configuration in Env variables or command line, and we cannot find config in the current or parent folders then write error:ERROR: Cannot find the mandatory configuration "user_id"

The current or any of the parent directories don't contain "smartling.yml"
Please either specify:
- Environment variable "SMARTLING_USER_ID"
- Command line option "--user"
- Or set "user_id" option in the configuration file:
smartling.yml:
  user_id: "PUT_VALUE_HERE"

Updated description after an internal conversation. We continue to use the same approach but make error more user friendly

@dimitrystd: Check it out. Also, -vv flag can be used to look where CLI is looking for config file and which one is used.

Reopened. Added comments in commit but don't see them now.
I cannot pass basic flow
$ smartling-cli init
ERROR: no configuration file "smartling.yml" found

Ensure, that config file exists either in the current directory or in any parent directory.

@dimitrystd: Check it out.

Verified


[Tests] Pass stdout and stderr in error message when it's asserted to be empty

Motivation
When a test fails, it's hard to debug the problem if the output causing the failure isn't returned in the error message.
Modifications
Pass stdout and stderr in error message when it's asserted to be empty

/pulsarbot run-failure-checks


Fail to read syndication link of mastodon when more than one syndication

I'm using wordpress syndication link to try to syndicate to mastodon, but failed.
The post url is https://jacky.seezone.net/2021/01/13/19328/
If I use 'Discover' at brid.gy, it would also fail to fetch.
The log https://brid.gy/log?start_time=1610507656&key=agdicmlkLWd5clILEg1QdWJsaXNoZWRQYWdlIitodHRwczovL2phY2t5LnNlZXpvbmUubmV0LzIwMjEvMDEvMTMvMTkzMjgvDAsSB1B1Ymxpc2gYgICI3OCDkAoM
In the html there are two syndication links (twitter & mastodon), twitter one can be read but not the mastodon one. Is there something wrong with the markup?

Another one where I only tick post to mastodon, the syndication link can be read but the post to mastodon is 403. Don't know if it's server problem or bridgy problem?
https://brid.gy/log?start_time=1610514319&key=agdicmlkLWd5clILEg1QdWJsaXNoZWRQYWdlIitodHRwczovL2phY2t5LnNlZXpvbmUubmV0LzIwMjEvMDEvMTMvMTkzMzIvDAsSB1B1Ymxpc2gYgICInOTSgQoM

Another one where I only tick post to mastodon, the syndication link can be read but the post to mastodon is 403. Don't know if it's server problem or bridgy problem?
https://brid.gy/log?start_time=1610514319&key=agdicmlkLWd5clILEg1QdWJsaXNoZWRQYWdlIitodHRwczovL2phY2t5LnNlZXpvbmUubmV0LzIwMjEvMDEvMTMvMTkzMzIvDAsSB1B1Ymxpc2gYgICInOTSgQoM

Posting mastodon alone now works, after I have disabled publishing and then enable again.
Posting both twitter & mastodon doesn't work

Posting mastodon alone now works, after I have disabled publishing and then enable again.
Posting both twitter & mastodon doesn't work

Sorry for the trouble, and thanks for all the details!
The consistent issue I'm seeing here is, when you use Bridgy Publish to syndicate a post to both Twitter and Mastodon, Bridgy sees the Twitter publish link (https://brid.gy/publish/twitter) but not the Mastodon publish link (https://brid.gy/publish/mastodon). I do see the Mastodon publish link later though, when I look in a browser.
https://jacky.seezone.net/2021/01/13/19346/ is a good example. The publish logs for both silos (Twitter, Mastodon) only show the Twitter publish link.
Your site's markup is fine, and I'm pretty confident in Bridgy's HTML and mf2 parsing. Is your site maybe sending the webmentions to trigger Bridgy Publish before the Mastodon publish link gets included?
You might try turning off your automatic webmention sending, post a post, check that both publish links are there, and then send each publish webmention manually to see if they both work then.

Sorry for the trouble, and thanks for all the details!
The consistent issue I'm seeing here is, when you use Bridgy Publish to syndicate a post to both Twitter and Mastodon, Bridgy sees the Twitter publish link (https://brid.gy/publish/twitter) but not the Mastodon publish link (https://brid.gy/publish/mastodon). I do see the Mastodon publish link later though, when I look in a browser.
https://jacky.seezone.net/2021/01/13/19346/ is a good example. The publish logs for both silos (Twitter, Mastodon) only show the Twitter publish link.
Your site's markup is fine, and I'm pretty confident in Bridgy's HTML and mf2 parsing. Is your site maybe sending the webmentions to trigger Bridgy Publish before the Mastodon publish link gets included?
You might try turning off your automatic webmention sending, post a post, check that both publish links are there, and then send each publish webmention manually to see if they both work then.

I tried to make a post contains both publish links and send webmention individually via https://telegraph.p3k.io .
They all succeeded. (I also tried different orders, i.e. twitter first or mastodon first)
https://jacky.seezone.net/2021/01/14/19350/
https://telegraph.p3k.io/webmention/15NgaFyfzePG2oqYAX/details
https://telegraph.p3k.io/webmention/15NmYbmK472zFrfu8H/details
https://jacky.seezone.net/2021/01/14/19352/
https://telegraph.p3k.io/webmention/15B2NkrX7KjAg6R1Wf/details
https://telegraph.p3k.io/webmention/15etJIrXWKZBES3UYX/details
For the sending part in wordpress, it's handled by syndication links plugin, can't quite sure on whether the publish link  is intact when sending webmention. Here is the syndication_log it captured for the failed case:
array (
  0 =>
  array (
    'twitter-bridgy' => true,
    'mastodon-bridgy' =>
    WP_Error::__set_state(array(
       'errors' =>
      array (
        'mastodon-bridgy_publish_error' =>
        array (
          0 => 'Unknown Mastodon via Bridgy Error',
        ),
      ),
       'error_data' =>
      array (
        'mastodon-bridgy_publish_error' =>
        array (
          'status' => 400,
          'data' =>
          (object) array(
             'error' => 'Couldn\'t find link to brid.gy/publish/mastodon',
          ),
        ),
      ),
       'additional_data' =>
      array (
      ),
    )),
  ),
)

I tried to make a post contains both publish links and send webmention individually via https://telegraph.p3k.io .
They all succeeded. (I also tried different orders, i.e. twitter first or mastodon first)
https://jacky.seezone.net/2021/01/14/19350/
https://telegraph.p3k.io/webmention/15NgaFyfzePG2oqYAX/details
https://telegraph.p3k.io/webmention/15NmYbmK472zFrfu8H/details
https://jacky.seezone.net/2021/01/14/19352/
https://telegraph.p3k.io/webmention/15B2NkrX7KjAg6R1Wf/details
https://telegraph.p3k.io/webmention/15etJIrXWKZBES3UYX/details
For the sending part in wordpress, it's handled by syndication links plugin, can't quite sure on whether the publish link  is intact when sending webmention. Here is the syndication_log it captured for the failed case:
array (
  0 =>
  array (
    'twitter-bridgy' => true,
    'mastodon-bridgy' =>
    WP_Error::__set_state(array(
       'errors' =>
      array (
        'mastodon-bridgy_publish_error' =>
        array (
          0 => 'Unknown Mastodon via Bridgy Error',
        ),
      ),
       'error_data' =>
      array (
        'mastodon-bridgy_publish_error' =>
        array (
          'status' => 400,
          'data' =>
          (object) array(
             'error' => 'Couldn\'t find link to brid.gy/publish/mastodon',
          ),
        ),
      ),
       'additional_data' =>
      array (
      ),
    )),
  ),
)

I tried to make a post contains both publish links and send webmention individually via https://telegraph.p3k.io .
They all succeeded. (I also tried different orders, i.e. twitter first or mastodon first)

Thanks! That's a key clue. cc @dshanske for the Syndication Links plugin.

I tried to make a post contains both publish links and send webmention individually via https://telegraph.p3k.io .
They all succeeded. (I also tried different orders, i.e. twitter first or mastodon first)

Thanks! That's a key clue. cc @dshanske for the Syndication Links plugin.

Evidence seems to point to an issue with Syndication Links or similar on your site's side (friendly ping @dshanske), so I'm tentatively closing here. Feel free to reopen!

For the record, I found that it's the cache busting problem.
https://github.com/dshanske/syndication-links/issues/140


pydeck: Specify semver range for @deck.gl/jupyter-widget

For #3640
Use a semver range instead of updating the exact version manually

Background

Change List

Coverage remained the same at 80.804% when pulling 3e10b5e9d146ebef4e79f17d2494f4b6c1989ed0 on ajd/use-semver into<PHONE_NUMBER>24d5d6e9a110fe09837d806fe2b6e7 on master.

Rolled into #4096


Add specify start director for ssh

Resolving ask in issue https://github.com/MicrosoftDocs/terminal/issues/25

Docs Build status updates of commit 1a2f538:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

TerminalDocs/tutorials/ssh.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Cannot Export to FFMpegServer

With the following settings on the recorder on Chrome 68 / Win 7, recording to ffmpegserver does not work:
const recorder = new CCapture({
            verbose: true,
            display: true,
            framerate: 60,
            format: 'ffmpegserver'
        });

This is the error message:
CCapture.all.min.js:1 Uncaught ReferenceError: FFMpegServer is not defined
    at new h (CCapture.all.min.js:1)
    at new d (CCapture.all.min.js:1)
    at index.html?_ijt=6pi7u7o15u5s36rgjnelgq4s00:9

Have you included the ffmpegserver lib? Follow the instructions here https://github.com/greggman/ffmpegserver.js


Remove dead project file

Fixes #4085
The topic uses the project file at first-mvc-app-xplat, so it looks like the sample at tutorials/first-mvc-app-mac/start-mvc/sample can be removed, which isn't really a sample anyway ... it's just a project file.

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

sry ... didn't spot that you self-assigned. I'm looking for things I can knock out on our way to <=220 issues. .... What do they say there? ... "Just do it and ask for forgiveness later??" lol

@Rick-Anderson Please review.


Fix timezone inconsistencies

Summary
Addresses some unexpected behavior when the site's timezone differs from the user's timezone.
Overall I tried to compare this to how dates are handled in the "All Stories" view (aka classic WP list view) and the Gutenberg block editor, ensuring that the results are the same everywhere.
Relevant Technical Choices

Does not use the REST API response's date_gmt anywhere anymore, but date itself instead
Dates aren't immediately parsed as such when loading the story, but instead only when needed for formatting or comparison.
This was previously the case in the dashboard.
Do not use relative date formatting if the date is today, but in the future.
Otherwise you'll get "Published in an hour", which is verry misleading.

To-do

User-facing changes
No user-facing changes, just dates that now should be displayed correctly.
Testing Instructions
Scenario A: Timezone Moscow

Set your site's timezone to Moscow
Create a new story
In the Document panel, see that the publish date is displayed in Moscow timezone
The "Publish" button should read "Publish", not "Schedule"
Publish the story
Go to "My Stories" and see that story date is in Moscow timezone
Go to "All Stories" and see that story date is in Moscow timezone as well

To double check, publish a new blog post as well, and see how dates are displayed there.
Scenario B: Timezone Honolulu

Set your site's timezone to Honolulu
Create a new story
In the Document panel, see that the publish date is displayed in Honolulu timezone
The "Publish" button should read "Publish", not "Schedule"
Publish the story
Go to "My Stories" and see that story date is in Honolulu timezone
Go to "All Stories" and see that story date is in Honolulu timezone as well

To double check, publish a new blog post as well, and see how dates are displayed there.

Fixes #5522

+1 to this. Seems WordPress core doesn't use gmt by local to display post date. See get_post_time.

+1 to this. Seems WordPress core doesn't use gmt by local to display post date. See get_post_time.

This is looking good. A couple of questions and tweaks.

This is looking good. A couple of questions and tweaks.

Really it's been 2 minutes and I've just changed my timezone so shouldn't it say 6:22 AM since the story has been published the same amount of time?

In theory, yes. But since WP does not store date information with timezones, all we know is that is was published at 7:22 PM. That's what we show to the user.
What I am aiming for here is mainly consistency. If the "All Stories" page says 7:22 PM, so should we. Otherwise users will be very confused.

Really it's been 2 minutes and I've just changed my timezone so shouldn't it say 6:22 AM since the story has been published the same amount of time?

In theory, yes. But since WP does not store date information with timezones, all we know is that is was published at 7:22 PM. That's what we show to the user.
What I am aiming for here is mainly consistency. If the "All Stories" page says 7:22 PM, so should we. Otherwise users will be very confused.

Really it's been 2 minutes and I've just changed my timezone so shouldn't it say 6:22 AM since the story has been published the same amount of time?

In theory, yes. But since WP does not store date information with timezones, all we know is that is was published at 7:22 PM. That's what we show to the user.
What I am aiming for here is mainly consistency. If the "All Stories" page says 7:22 PM, so should we. Otherwise users will be very confused.

ahhh I see now, ok thank you for elaborating.

Really it's been 2 minutes and I've just changed my timezone so shouldn't it say 6:22 AM since the story has been published the same amount of time?

In theory, yes. But since WP does not store date information with timezones, all we know is that is was published at 7:22 PM. That's what we show to the user.
What I am aiming for here is mainly consistency. If the "All Stories" page says 7:22 PM, so should we. Otherwise users will be very confused.

ahhh I see now, ok thank you for elaborating.

Sorry, one more clarification.
I'm in Honolulu still, which is 3 hours behind MST where I actually am, which is known by my browser (not WordPress). My relative time for this shows "published 3 hours ago" for my honolulu stories that I just created.
I don't think anything is wrong with your logic, the 3 hours matches the time displayed in all stories, it just seems like the relative time display has potential for adding confusion here/ if continuity is the goal, should we abandon the relative timing and just display the true time?
(It's also causing my sort on dashboard for "last modified" to be weird, the initial moscow story which is published 7:22pm is showing up as the most recently modified even though i've made honolulu stories after the moscow story- which is probably an edge case we don't need to worry about)

Sorry, one more clarification.
I'm in Honolulu still, which is 3 hours behind MST where I actually am, which is known by my browser (not WordPress). My relative time for this shows "published 3 hours ago" for my honolulu stories that I just created.
I don't think anything is wrong with your logic, the 3 hours matches the time displayed in all stories, it just seems like the relative time display has potential for adding confusion here/ if continuity is the goal, should we abandon the relative timing and just display the true time?
(It's also causing my sort on dashboard for "last modified" to be weird, the initial moscow story which is published 7:22pm is showing up as the most recently modified even though i've made honolulu stories after the moscow story- which is probably an edge case we don't need to worry about)

@BrittanyIRL What‘s your timezone in WP? It is all relative to that

@BrittanyIRL What‘s your timezone in WP? It is all relative to that

Sorry, one more clarification.
I'm in Honolulu still, which is 3 hours behind MST where I actually am, which is known by my browser (not WordPress). My relative time for this shows "published 3 hours ago" for my honolulu stories that I just created.
I don't think anything is wrong with your logic, the 3 hours matches the time displayed in all stories, it just seems like the relative time display has potential for adding confusion here/ if continuity is the goal, should we abandon the relative timing and just display the true time?
(It's also causing my sort on dashboard for "last modified" to be weird, the initial moscow story which is published 7:22pm is showing up as the most recently modified even though i've made honolulu stories after the moscow story- which is probably an edge case we don't need to worry about)

@BrittanyIRL we could perhaps continue to use date_gmt for the relative time?

Sorry, one more clarification.
I'm in Honolulu still, which is 3 hours behind MST where I actually am, which is known by my browser (not WordPress). My relative time for this shows "published 3 hours ago" for my honolulu stories that I just created.
I don't think anything is wrong with your logic, the 3 hours matches the time displayed in all stories, it just seems like the relative time display has potential for adding confusion here/ if continuity is the goal, should we abandon the relative timing and just display the true time?
(It's also causing my sort on dashboard for "last modified" to be weird, the initial moscow story which is published 7:22pm is showing up as the most recently modified even though i've made honolulu stories after the moscow story- which is probably an edge case we don't need to worry about)

@BrittanyIRL we could perhaps continue to use date_gmt for the relative time?


Wire primitive and wire inliner

Fixes #158.
Adds a new std_wire primitive and a new pass wire-inliner. The new inlining
pass acts as an alternate to hole-inliner that instantiates wires to hold
the value of each group's go and done instead of manually inlining the
go and done expressions.
I expect that this will probably make icarus-verilog and Verilator happier
because they don't need to do evaluate as many expressions as before.

All right, things look correct so I'm going to go ahead and merge this. I'll open a different PR/issue to discuss making this the default inlining strategy because I expect verilator to perform better with it and it would solve some of the problems @cgyurgyik ran into when trying to generate really big NTT kernels (https://github.com/verilator/verilator/issues/2771).


Reporobot doesn't appear to actually be editing my file

RepoRobot won't edit any changes you make. It will process the pull request you make next and add your name to the list of people who completed Git-it.


Improve error messages when Ecto.Query.from is not imported

I ran into the issue where I didn't import the from/2 function from Ecto.Query and compilation was failing on queries with interpolated expressions as a result.
Here's was my initial query:
Repo.one(from u in User,
                 where: u.username == ^email_or_username or u.email == ^email_or_username,
                 limit: 1)

And the error I was getting:
== Compilation error on file web/plugs/auth.ex ==
** (CompileError) web/plugs/auth.ex:31: cannot use ^email_or_username outside of match clauses
    (elixir) expanding macro: Kernel.or/2
    web/plugs/auth.ex:31: Rumbl.login_with_password/4
    (elixir) lib/kernel/parallel_compiler.ex:100: anonymous fn/4 in Kernel.ParallelCompiler.spawn_compilers/8

Then I replaced my interpolated variable with a string and I got this error:
== Compilation error on file web/plugs/auth.ex ==
** (CompileError) web/plugs/auth.ex:30: function from/2 undefined
    (stdlib) lists.erl:1337: :lists.foreach/2
    (stdlib) erl_eval.erl:669: :erl_eval.do_apply/6
    (elixir) lib/kernel/parallel_compiler.ex:100: anonymous fn/4 in Kernel.ParallelCompiler.spawn_compilers/8

This obviously solved my initial problem. It didn't take me too long to figure it out but still I'm wondering if we can make the experience nicer I imagine this must happen quite frequently. If you think there is a way I'm happy to dig into it.

There is nothing we can do. Because there is no macro, the compiler will try to expand everything as usual, running into invalid expressions.

It makes sense, thanks for the explanation @josevalim


cannot satisfy -package-id stm-2.4.4-2526ff89874f899372b2e4f544bb03cd

Hello
This is probably a more generic problem than with this specific package, but as I am not sure where/how to report this, asking help here.
Using Mac OS X ElCapitan (yeah, the one giving a lot of trouble). I have ghc/ghci working (at least it seems), but when installing transforers-base I get:
Building library...
creating dist/build
/usr/local/bin/ghc --make -fbuilding-cabal-package -O -static -dynamic-too -dynosuf dyn_o -dynhisuf dyn_hi -outputdir dist/build -odir dist/build -hidir dist/build -stubdir dist/build -i -idist/build -isrc -idist/build/autogen -Idist/build/autogen -Idist/build '-optP-DHS_TRANSFORMERS_BASE__ORPHANS=1' -optP-include -optPdist/build/autogen/cabal_macros.h -this-package-key trans_88TAFm21vtn2NrYUppy50c -hide-all-packages -package-db dist/package.conf.inplace -package-id base-<IP_ADDRESS>-075aa0db10075facc5aaa59a7991ca2f -package-id stm-2.4.4-2526ff89874f899372b2e4f544bb03cd -package-id transformers-<IP_ADDRESS>-21dcbf13c43f5d8cf6a1f54dee6c5bff -package-id transformers-compat-<IP_ADDRESS>-3ca5cbcec233c17da785d5f27705643c -XHaskell98 Control.Monad.Base -Wall
<command line>: cannot satisfy -package-id stm-2.4.4-2526ff89874f899372b2e4f544bb03cd
    (use -v for more information)
Updating world file...
cabal: Error: some packages failed to install:
transformers-base-0.4.4 failed during the building phase. The exception was:
ExitFailure 1

THanks

Seems like a problem with the package database (perhaps after a GHC upgrade). Try to remove dist and reconfigure.