Confusing expression

the content in you page
"When virtual machine instances or other compute resources attempt to communicate on a TCP connection that doesn't exist, they send TCP reset packets. An example is connections that have reached idle timeout. The next packet received will return a TCP reset to the private IP address of the virtual machine to signal and force connection closure. The public side of a NAT gateway doesn't generate TCP reset packets or any other traffic. Only traffic produced by the customer's virtual network is emitted."
It is very confusing. It is hard to understand what you are saying?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 6dce7dff-6689-f91b-64b7-2817b9d52e5e
Version Independent ID: 70276322-e104-0cc1-bc5a-ebe5d754fb43
Content: What is Azure Virtual Network NAT?
Content Source: articles/virtual-network/nat-gateway/nat-overview.md
Service: virtual-network
Sub-service: nat
GitHub Login: @asudbring
Microsoft Alias: allensu

@byguo2018
Thanks for your feedback! We will investigate and update as appropriate.

@byguo2018 , We have assigned this to the content owner/author for review.
@asudbring , for your review.

@byguo2018 thanks for the feedback on this bullet point from the NAT Gateway Overview article. I agree that the description in that bullet was written in a confusing way. I have rewritten that bullet to provide more clarity which is now published here: https://learn.microsoft.com/en-us/azure/virtual-network/nat-gateway/nat-overview#outbound-connectivity.
For context, the purpose of that bullet was to broadly explain how TCP reset is unidirectional with NAT GW. When NAT GW receives traffic from an endpoint on a connection flow that does not exist, it will send a TCP RST packet to indicate to the endpoint that the connection no longer exists.

@byguo2018 glad to hear it. If there are no other questions in relation to this, can we close this issue?

Yes.
Thanks.
Bruce Guo
发件人: Aimee Littleton @.>
发送时间: Saturday, February 4, 2023 2:13 AM
收件人: MicrosoftDocs/azure-docs @.>
抄送: Bruce Guo @.>; Mention @.>
主题: Re: [MicrosoftDocs/azure-docs] Confusing expression (Issue #104308)
@byguo2018https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbyguo2018&data=05|01|byguo%40microsoft.com|aba7266715224401d5a008db06123b13|72f988bf86f141af91ab2d7cd011db47|1|0|638110447624840832|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=%2FnR4LPkUh%2BayYJjXCr%2Fps2aFyGv%2FqEz5Wev60tP9hiE%3D&reserved=0 glad to hear it. If there are no other questions in relation to this, can we close this issue?
―
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F104308%23issuecomment-1416228045&data=05|01|byguo%40microsoft.com|aba7266715224401d5a008db06123b13|72f988bf86f141af91ab2d7cd011db47|1|0|638110447624997065|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Jjtq%2Bmg4UPTlUVz5C6%2ByN1QW8erlRS6KJ3FKdPrgpHU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAK343JOSDGHBPD3XCT4IKUDWVVDBJANCNFSM6AAAAAAUHBC22I&data=05|01|byguo%40microsoft.com|aba7266715224401d5a008db06123b13|72f988bf86f141af91ab2d7cd011db47|1|0|638110447624997065|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=54%2FvWVeT%2BdpSrReeixQKEWwCqwzomnGwVbaivYUMveo%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

#please-close


SocketException running on AspNet Core on Linux

This code works fine on windows, but throws this error on linux.
I'm suspecting I might be missing some .deb's on the linux box, but not sure how to find out which..
Can you help me?
Thanks
Exception:
Unhandled Exception: System.Net.Sockets.SocketException: Protocol not available
at System.Net.Sockets.Socket.UpdateStatusAfterSocketErrorAndThrowException(SocketError error, String callerName)
at System.Net.Sockets.Socket.SetSocketOption(SocketOptionLevel optionLevel, SocketOptionName optionName, Int32 optionValue, Boolean silent)
at System.Net.Sockets.Socket.SetSocketOption(SocketOptionLevel optionLevel, SocketOptionName optionName, Int32 optionValue)
at Rssdp.SocketFactory.CreateUdpMulticastSocket(Int32 multicastTimeToLive, Int32 localPort)
at Rssdp.Infrastructure.SsdpCommunicationsServer.BeginListeningForBroadcasts()
at Rssdp.Infrastructure.SsdpDevicePublisherBase..ctor(ISsdpCommunicationsServer communicationsServer, String osName, String osVersion, ISsdpLogger log)
at TessarolliSolutions.Zeus.Program.Main(String[] args) in C:\VSTS\TS\src\Zeus API\Zeus\Program.cs:line 69
Code:
_Publisher = new SsdpDevicePublisher(new SsdpCommunicationsServer(new SocketFactory(HardwareInfo.GetIPAddress().ToString())));
_Publisher.StandardsMode = SsdpStandardsMode.Relaxed;
_Publisher.AddDevice(PublishDevice());

Hi,
The only protocol I'm aware this needs is UDP. I'm afraid I'm not familiar with Linux, 'debs' or mono, and have no device(s) on which to test this so I'm not much help.
If there is a 'deb' to add UDP support then I guess try installing that, otherwise perhaps try Googling for that error details and mono and .net.

Sorry, that is not totally accurate... UDP is used primarily, but also HTTP which also requires TCP. In the error above it looks like (multicast) UDP is what is trying to be accessed though.


WIP: Add support for Kubernetes

Describe in detail the problem you had and how this PR fixes it
Fixes #60 by adding support for Kubernetes. This deploys code-server, Contour for ingress controller, and Cert Manager for Let's Encrypt certificates, with persistent storage with AWS EBS volumes. Also, I left the Dockerfile I used for golang development. If there's a public push of this somewhere it would be great to document how you can create base images with the right tooling up front.
This is also somewhat opiononated in that it's using AWS EBS volumes, but can be adapted for many use cases. I wasn't sure how that might get organized within the repo.
Signed-off-by: Steve Sloka<EMAIL_ADDRESS>Is there an open issue you can link to?
#60
// cc @jbeda

This looks awesome! I wonder if it could be improved by possibly integrating or using parts of our current Dockerfile as we currently do some building?

@NGTmeaty Sounds good! I just updated to use the example Dockerfile. I should probably add some docs to this as well. Let me see if I can pull those together so it's more clear to folks.

Awesome! +1

It could be nice to have an official helm chart.

Please do not rely on EBS only, make it vendor agnostic

It doesn't, just an example. You could reference any storage class and it would work.

I could integrate kustomize to allow for customization. Could pick some clouds and show examples (e.g. AWS, GCP, Azure). What would you suggest?

@stevesloka since its an example, might as well use default storage class for PVC or hostMount, then have providers documented if they want to

I think the ingress controller and TLS should not be part of this. Depending on the cluster setup, those are unwanted extras in most cases.

Is this ready to go? Happy to merge this in.

@nhooyr I can add some docs on a follow-up PR.

Awesome, sounds good man.

@kylecarbs can you disable the signed commits requirement?


Does archaius now support sending only property changes only during polling?

Are you asking if the property changes are updated through push or poll? Currently, it's through polling at the polling interval.

No.  My question is about if archaius can poll only the properties that
have been changed.  For example, if I have 100 properties and only 3
changes, archaius will only get 3 new values at the next polling interval
instead of all 100.
Thanks
On Wednesday, July 6, 2016, Howard Yuan<EMAIL_ADDRESS>wrote:

Are you asking if the property changes are updated through push or poll?
Currently, it's through polling at the polling interval.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Netflix/archaius/issues/425#issuecomment-230916217,
or mute the thread
https://github.com/notifications/unsubscribe/ADHgLABlZyYKQ9lqSvjYPsTru-uhGLQpks5qTCFagaJpZM4JGbs_
.

I see. Currently, no. Current polling configuration source will get everything to the client side and then find out difference between the current client state and polled results, then only publish those differences into the configuration. This is to distribute the load between server and client and make server stateless.


Catch double "/" in any resolver functions

These are causing identity issues currently.

Fixed in upcoming batch of changes.


eos_designs incorrectely adds mtu 9214 for Mlag interface Vlan4094 on CCS-722/750

Issue Summary
switch models 722 and 750 do not support MTU settings per interface, but only global.
for example:
eos_designs adds mtu 9214 to MLAG peer-interface VLAN4094, which is not supported on these platforms.
It throws an error message and MLAG will stay inactive.
Which component(s) of AVD impacted
eos_designs
How do you run AVD ?
Ansible CLI (with virtual-env or native python)
Steps to reproduce
just render mlag setting and push ist against two CCS-750/722 switches.

Relevant log output
interface Vlan4094
   description MLAG_PEER
   mtu 9214
   no autostate
   ip address <IP_ADDRESS>/31

access-04-111-01.LAN.MU__14:23:34(config-if-Vl4094)#mtu 9214% Unavailable command (not supported on this hardware platform)

domain-id                          :   ACCESS-04-111_01_02
local-interface                    :              Vlan4094
peer-address                       :          <IP_ADDRESS>
peer-link                          :        Port-Channel51
peer-config                        :

MLAG Status:
state                              :              Inactive
negotiation status                 :            Connecting
peer-link status                   :                    Up
local-int status                   :                    Up
system-id                          :     00:00:00:00:00:00
dual-primary detection             :              Disabled
dual-primary interface errdisabled :                 False

### Contributing Guide

- [X] I agree to follow this project's Code of Conduct

I guess we should set the global mtu setting instead, to whatever value we have in p2p_uplinks_mtu.

@kmueller68 thinking more about this, if the devices only support a single MTU value, it would lead to user SVIs also using the global value, which may not be the intention.
I can implement a knob under the platform settings for this, but I am not sure how we should decide on which MTU to set as global.

@ClausHolbechArista I know about a similar topic, because I ran into the same on platforms of another vendor.
It has been a challenge to first set the mtu globally to 9xxx and then reduce each single svi interface back to mtu=1500.
But even this would not work on 750/722 since we would never be able to set mtu back to 1500 on single interfaces.
But in this case, we are talking about MLAG control plane traffic between MLAG peers only.
There will never be sent any user data traffic between Vlan4094 interfaces, right?
Ethernet mtu by default is 9214 on 750/722.
So, switched traffic accross MLAG peerlink supports jumboframes anyways.
Why are you currently setting the mtu for MLAG control plane svi Vlan4094 to p2p_uplinks_mtu?
Is there a technical case, which I didnt get so far?
I have not found anything in the manual that MLAG peer svi mandatory needs to have jumboframes enabled.
This would mean, that we could leave Vlan4094 mtu at 1500.
I see only one usecase to increase the global mtu on 750/722 to jumboframes:
If 750/722 are part of an EVPN fabric as MLAG l3leafs and have overlay iBGP peering in the vrfs.
What is your opnion on this?

Why are you currently setting the mtu for MLAG control plane svi Vlan4094 to p2p_uplinks_mtu?

I don't know the history of why, but I agree it should not be required as long as you don't have transit traffic on that SVI (it is possible to reuse the same SVI for iBGP).
I am working on a fix that implements these schema changes for eos_designs:
keys:
  platform_settings:
<snip>
        feature_support:
          type: dict
          keys:
            per_interface_mtu:
              type: bool
              default: true
              description: |
                Disables configuration of per interface MTU for p2p links, MLAG SVIs and Network Services.
                Effectively this means that all settings regarding inteface MTU will be ignored.
                Platforms without support for per interface MTU can use a single default interface MTU setting. Set this via "default_interface_mtu"
  default_interface_mtu:
    documentation_options:
      table: system-settings
    type: int
    convert_types:
      - str
    description: Default interface MTU configured on EOS under "interface defaults"
    min: 68
    max: 65535

In the above I have only allowed default_interface_mtu as a root key, and not under the platform_settings, but maybe I should add it there too.

I think that it would be good to have it under plattform_settings too.
Is it intened that the default platform settings are not shown in the avd.sh documentation or did I miss it anywhere else?
https://avd.sh/en/stable/roles/eos_designs/docs/tables/platform-settings.html?h=platform_settings

We need to figure out if we can change the default values now, or just provide the knobs, so you can change them yourself. It depends a bit if these mtu restrictions cover all devices under 722XP and 750X. @carlbuchmann @mpergament

@ClausHolbechArista
I have tested your PR with the following config.
question:
When I am setting this mtu things for 722XP it is appended to the default platform_settings, right?
It is not deleting any of the default settings?
LAN_ACCESS.yml
# special mtu settings for 722xpm switches supporting global mtu only
platform_settings:
  - platforms:
    - 722XP
    default_interface_mtu: 8000
    feature_support:
      per_interface_mtu: false

LAN_POD_08.yml
    ACCESS-04-111_01_02:
      platform: 722XP
      mlag_interfaces: "{{ mlag_interfaces_access_48 }}"
      uplink_interfaces: "{{ uplink_interfaces_access_48_mlag }}"
      uplink_switch_interfaces: ['Ethernet25']
      nodes:
        access-04-111-01.LAN.MU:
          id: 42
          # disabled to avoid generation of oob config
          # mgmt_ip: x.x.x.x/24
          uplink_switches: ['distri-08-01.LAN.MU']

        access-04-111-02.LAN.MU:
          id: 43
          # disabled to avoid generation of oob config
          # mgmt_ip: x.x.x.x/24
          uplink_switches: ['distri-08-02.LAN.MU']

gives me the following cli config for the two switches:
!
interface defaults
   mtu 8000
!

question:
When I am setting this mtu things for 722XP it is appended to the default platform_settings, right?
It is not deleting any of the default settings?

Unfortunately it replaces the default platform_settings, so you have to copy the defaults (the big dictionary you can find in the docs using the (+) on the YAML page) and then add your extra setting.
Hopefully your changes should also show removal of the MTU under the interfaces, so the MLAG issue goes away.

@ClausHolbechArista
I am sorry...
Where would you add the following?
- platforms:
  - 722XP
  default_interface_mtu: 8000
  feature_support:
    per_interface_mtu: false

:)
Please try to add this to the fabric group vars file:
platform_settings:
# Custom platform settings
- platforms:
  - 722XP
  default_interface_mtu: 8000
  feature_support:
    per_interface_mtu: false

# DEFAULT ENTRIES copied from docs
- feature_support:
    queue_monitor_length_notify: false
  platforms:
  - default
  reload_delay:
    mlag: 300
    non_mlag: 330
- feature_support:
    queue_monitor_length_notify: false
  platforms:
  - 7050X3
  - 720XP
  - 722XP
  reload_delay:
    mlag: 300
    non_mlag: 330
  trident_forwarding_table_partition: flexible exact-match 16384 l2-shared 98304 l3-shared
    131072
- lag_hardware_only: true
  platforms:
  - 7280R
  - 7280R2
  - 7020R
  reload_delay:
    mlag: 900
    non_mlag: 1020
  tcam_profile: vxlan-routing
- platforms:
  - 7280R3
  reload_delay:
    mlag: 900
    non_mlag: 1020
- lag_hardware_only: true
  management_interface: Management0
  platforms:
  - 7500R
  - 7500R2
  reload_delay:
    mlag: 900
    non_mlag: 1020
  tcam_profile: vxlan-routing
- management_interface: Management0
  platforms:
  - 7500R3
  - 7800R3
  reload_delay:
    mlag: 900
    non_mlag: 1020
- management_interface: Management0
  platforms:
  - 7368X4
  reload_delay:
    mlag: 300
    non_mlag: 330
- management_interface: Management0
  platforms:
  - 7300X3
  reload_delay:
    mlag: 1200
    non_mlag: 1320
  trident_forwarding_table_partition: flexible exact-match 16384 l2-shared 98304 l3-shared
    131072
- feature_support:
    bgp_update_wait_for_convergence: false
    bgp_update_wait_install: false
    interface_storm_control: false
    queue_monitor_length_notify: false
  platforms:
  - VEOS
  - VEOS-LAB
  - vEOS
  - vEOS-lab
  reload_delay:
    mlag: 300
    non_mlag: 330
- feature_support:
    bgp_update_wait_for_convergence: false
    bgp_update_wait_install: false
    interface_storm_control: false
    queue_monitor_length_notify: false
  management_interface: Management0
  platforms:
  - CEOS
  - cEOS
  - ceos
  - cEOSLab
  reload_delay:
    mlag: 300
    non_mlag: 330

@ClausHolbechArista I tried this in different group_vars files but with this I get the mtu settings but miss the mlag reload_delay settings for 722xp switches.

Right. I just copied from your example. I have updated above to include the default settings and adding yours.

@ClausHolbechArista
Thank you very much for your patience.
It is working as expected now

Thank you Karl for reporting and testing this fix. Have a nice weekend :)

@ClausHolbechArista
I please would like to come back to you on the discussion on the integration of this 722/750 specialty into the default platform_settings.
Now I have to copy the AVD default platform_settings to my local repo and add the custom settings for 722/750.
But from now on I have to take care on the defaut platform_settings continously if anything is changed in the AVDs and have to copy it manually to my repo.
Anyone using 722/750 switches with AVD eos_designs in a MLAG setup will hit this topic since it is a common hardware limitation and will need to get around it.
Could you please have a discussion internally if you find a common sense to add it to the defaults?

@kmueller68 Thank you for sharing your concerns here. I agree 100%, but the issue is that this issue concerns a subset of several platform families. It depends on the internal chip.
I will talk to the team and get back.


ModuleNotFoundError: No module named 'torch_sparse'

I want to ask the torch_sparse vision？It's not show in requirments,txt,I want to rebuilt this programs.

Hi @Picrew, thanks for the catch. You should be able to install the correct version of torch_sparse through this command: pip install pyg_lib torch_scatter torch_sparse torch_cluster torch_spline_conv -f https://data.pyg.org/whl/torch-1.13.0+cpu.html (here I selected the torch version compatible with the one specified in requirements.txt, however, the code should easily extend to e.g., cuda-enabled versions)
Hopefully this helps


Roll engine 4e9e69d2b392..f001ea29f1b9 (1 commits)

https://github.com/flutter/engine/compare/4e9e69d2b392...f001ea29f1b9
git log 4e9e69d2b392..f001ea29f1b9 --first-parent --oneline
2020-01-08<EMAIL_ADDRESS>adding windows platform to felt. fixing signal (flutter/engine#15111)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Trybots failed. These were the failed builds: https://github.com/flutter/flutter/runs/379783227 , https://github.com/flutter/flutter/runs/379783226

Commit queue failed; closing this roll.


can you make it compatible with react 18?

can you make it compatible with react 18, please?

hi, i'm going to create a new branch for this scope. As soon as possible I will try to update react.

@royaltics-solutions new branch react-18 is published. The dependencies has been upgraded from react 17 to react 18.
P.S. In the future, I will switch from class to functional components.


Installation Problem „gn Not found“

Hi, I am currently installing matter to test out some prototypes for our company. I am facing an error, in vs code when I try to build the lighting app for an esp 32 s3: cmake error command gn not found. Yet all previous tests in the terminal are without any errors. The error occurs on both a M1 MacBook Pro and an old Intel MacBook Pro. Currently I am installing an Ubuntu virtual machine to test it out.
I am able to use the gn command in the terminal, after I started matter.
can anyone please help me? I am not a professional yet I still need it in order to do some tests
thank you in advance

Franz

I found a solution, I forgot to „export“ as described in the ESP32 application usage guide. Now I am running into another error: ModuleNotFoundError: No Module named ‚lark‘. I will try to find another solution.


Embedding layer and Constraints

Dear everyone,
what follows is not really a bug, but IMO it can be considered a sort of counter-intuitive behavior.
When enforcing unitary norm constraints on an embedding layer, the constraints are enforced on the columns of the embedding matrix (the embedding vector dimension) instead of the rows (the no. of embedding vectors).
For instance consider the following setting, in which we have two embedding vectors of size 10:
import numpy as np
from keras.models import Sequential
from keras.layers.embeddings import Embedding
from keras.layers.core import Flatten, TimeDistributedDense, Activation

from keras.constraints import unitnorm

model = Sequential()
embedding_layer = Embedding(input_dim=2, output_dim=10, input_length=1, W_constraint=unitnorm())
model.add(embedding_layer)
model.add(Flatten())
model.add(Dense(output_dim=1))
model.add(Activation('sigmoid'))

model.compile(loss='binary_crossentropy', optimizer='adagrad')
model.fit(np.array([[0]]), np.array([0]), batch_size=1, nb_epoch=5, verbose=0)

embedding_matrix = embedding_layer.params[0].get_value()

print(embedding_matrix.shape)
print(np.linalg.norm(embedding_matrix[1, :]))
print(np.linalg.norm(embedding_matrix[:, 1]))

The unitary norm is enforced on each column of the matrix; in my opinion, it could make sense to enforce it on its rows (the actual embedding vectors).
(2, 10)
2.25984
1.0

At the moment I'm working around this by using a custom Constraint:
class FixedNorm(Constraint):
    def __init__(self, m=1.):
        self.m = m

    def __call__(self, p):
        p = K.transpose(p)
        unit_norm = p / (K.sqrt(K.sum(K.square(p), axis=0)) + 1e-7)
        unit_norm = K.transpose(unit_norm)
        return unit_norm * self.m

    def get_config(self):
        return {'name': self.__class__.__name__, 'm': self.m}

Just to be clear, the current constraint is that each word vector have unit length, right?
Rather than the ith dimension of all the word vectors is unit?
The second case seems crazy to me. If you have 10k word vectors in your embedding, each word vector would be nearly the zero vector. Why would that be good?
I just want to make sure that the current behavior is the first, rather than the second.

Ok, after experimenting, it is indeed the second way. So strange. Maybe I'm missing something, but I don't see how that's ever useful. Thanks @pminervini for bringing it to my attention and offering a solution! Don't forget to make the necessary imports:
from keras.constraints import Constraint

from keras import backend as K

Thanks @tstandley, I'm glad I was not hallucinating.
I think in some architectures it's really useful to impose constraints on the embedding vector layer, for avoiding trivial solutions for the loss minimization problem (e.g. you can increase the distance between two vectors by simply increasing their norm).

Glad to see this discussion. It was really helpful to implement some representation learning tasks of knowledge graph.

Same! All the problems I've run into were because I was implementing KG embedding models :) (the last one was an issue with cascades of Merge layers)
I'm glad this topic is getting attention; feel free to contact me if you feel like you want to discuss something!

Yes. I am trying to implement some basic models of KG embedding such as TransE with Keras. Hope it can reproduce the results in the paper.

Actually you can achieve much better results than the original NIPS'13 paper, by simply switching from SGD to Adagrad.
Some up-to-date numbers (and models) are available here: http://jmlr.org/proceedings/papers/v48/trouillon16.pdf

@pminervini Where did you code your custom constraint? is it in the same file. In my custom constraint and while using yours also I get error TypeError: Error when calling the metaclass bases
list() takes at most 1 argument (3 given)

@eshijia  Can you share your TransE implementation in Keras here please??

@mdmustafizurrahman You can check this repo.  That is not a perfect implementation. I just rewrote some codes based on some QA models. Maybe that repo doesn't fit the current release of Keras. I don't have enough extra time to implement the perfect Keras version of TransX. Recently, there is an implementation in PyTorch, I think maybe that is a better reference than the Tensorflow implementations. Hope to see your implementation in Keras.

@eshijia Hey thanks for sharing this. And sorry for my late reply I did not check earlier.


[Feature Request] Add a compile error if entry function returns any value.

🚀 Feature Request
Add a compile error if entry function returns any value.
Motivation
For aptos network, it is illegal to have the entry function return any values. E.g.
module Dev::HelloWorld {
    public(script) fun hello(): u64 { 5 }
}

$ aptos move run --function-id default::HelloWorld::hello
{
  "Error": "API error: transaction execution failed: Transaction Executed and Committed with Error INVALID_MAIN_FUNCTION_SIGNATURE"
}

This error can be reported at compile level.
Pitch
Describe the solution you'd like
Add a compile error when entry function returns some values.
Are you willing to open a pull request? (See CONTRIBUTING)
Don't think I am capable of handle this error.

The core Move language intentionally does not restrict input and output parameters of transactions. What is allowed here depends on the integration of Move. Errors should be reported on the Aptos compiler integration side. Please open this bug again in the Aptos Repo.


Can markdown-oxide be added to the Mason registry?

This would make it easy to configure alongside other LSPs directly inside neovim: https://github.com/mason-org/mason-registry

So I opened two PRs for this. One here https://github.com/mason-org/mason-registry/pull/4683 and one here https://github.com/neovim/nvim-lspconfig/pull/3030
Note that even once these are merged, you will still need to configure nvim cmp yourself. I am looking into options to make this automatic (let me know if you have any ideas!)

I stopped maintaining my own configuration and switched to https://github.com/LazyVim/LazyVim, so my best idea would be a lazyvim extra that can be configured as such:
{ import = "lazyvim.plugins.extras.lang.markdown_oxide" }

Then that would handle installation and LSP setup. This could either be contributed to LazyVim or distributed as a standalone package.

Great idea!
On Mon, Feb 26, 2024 at 11:18 AM Ken Powers @.***>
wrote:

I stopped maintaining my own configuration and switched to
https://github.com/LazyVim/LazyVim, so my best idea would be a lazyvim
extra that can be configured as such:
{ import = "lazyvim.plugins.extras.lang.markdown_oxide" }
Then that would handle installation and LSP setup. This could either be
contributed to LazyVim or distributed as a standalone package.
—
Reply to this email directly, view it on GitHub
https://github.com/Feel-ix-343/markdown-oxide/issues/1#issuecomment-1964551433,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVGUVS3J3LMCNWDUJGJ62XLYVSYUXAVCNFSM6AAAAABDRYYB2GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRUGU2TCNBTGM
.
You are receiving this because you commented.Message ID:
@.***>

Update: So it seems that the mason registry is not very active in PR reviews. I am sad about this because I think this is the best way to install LSs for neovim on all platforms.
For now, the package is listed on the AUR as markdown-oxide-git (paru{/yay} -S markdown-oxide-git and if you are not on arch, you can always compile from source.
I am working on getting this into other package managers now as well

done!
(development is too active for a binary release, but this should work for now)
https://github.com/mason-org/mason-registry/pull/4683

Awesome, thank you!


How to load the fine-tuned lora adapter and the downloaded model from the local directory for inference?

Hello,
Thank you so much for such great work!
I need help with loading the lora adapter which had finished fine-tuning process.
Here is my code:
model, tokenizer = FastLanguageModel.from_pretrained(
    model_name = "/path /to /unsloth_model/llama-3-8b-bnb-4bit", # load model either from the local directory or the specific huggingface repo
    max_seq_length = max_seq_length,
    dtype = dtype,
    load_in_4bit = load_in_4bit,
    local_files_only = True,
)
model = FastLanguageModel.get_peft_model(
    model,
    r = 16, # Choose any number > 0 ! Suggested 8, 16, 32, 64, 128
    target_modules = ["q_proj", "k_proj", "v_proj", "o_proj",
                      "gate_proj", "up_proj", "down_proj",],
    lora_alpha = 16,
    lora_dropout = 0, # Supports any, but = 0 is optimized
    bias = "none",    # Supports any, but = "none" is optimized
    # [NEW] "unsloth" uses 30% less VRAM, fits 2x larger batch sizes!
    use_gradient_checkpointing = "unsloth", # True or "unsloth" for very long context
    random_state = 4848,
    use_rslora = False,  # We support rank stabilized LoRA
    loftq_config = None, # And LoftQ
)
...
model.save_pretrained("/path /to /save /the /lora /adapter")
tokenizer.save_pretrained("/path /to /save /the /lora /adapter")

if True:
    from unsloth import FastLanguageModel
    model, tokenizer = FastLanguageModel.from_pretrained(
        model_name = "/path /to /save /the /lora /adapter", # YOUR MODEL YOU USED FOR TRAINING
        max_seq_length = max_seq_length,
        dtype = dtype,
        load_in_4bit = load_in_4bit,
    )
    FastLanguageModel.for_inference(model) # Enable native 2x faster inference
...

The message popped up for loading the saved lora adapter:

So, i can see the logic behind of the "FastLanguageModel.from_pretrained()" is to download the base model when "model_name" pointes to the directory of the lora adapter.
My Question is: what if I had already downloaded the base model, after finishing off the fine-tuning process and saving the corresponding lora adapter, how to load the saved lora adapter and the saved based model from the local directory for inference?
Thanks in advance! Looking foraward to any response!

If you're in Colab or Kaggle, the model gets auto deleted since it uses a lot of disk space, hence it needs to redownload it - it should be fine for larger disks


[11.x] Add --environment option to the schedule:list command

Add --environment option to the schedule:list command to only show tasks for the current environment.
Scheduled tasks can limit the task to certain environments using  ->environments($env), Currently the schedule:list command will show tasks for all environments which makes it hard to know which tasks will run for a specific environment. Other users shared this concern in this discussion https://github.com/laravel/framework/discussions/49143
When the --environment option is used it will filter the tasks to those that are defined for the current environment. You can also combine this with the --env option to see what tasks would run for a different environment.
# list tasks for ALL environments
php artisan schedule:list

# list tasks for current environment
php artisan schedule:list --environment

# list tasks for the production environment
php artisan schedule:list --environment --env=production

This change is backwards compatible because it is a new option, and the default behavior is not changed.
I also thought about updating the verbose output to show the environments but that could be done in a separate PR if desired.

I would honestly maybe just make this the default.


This lines prevents from files to result in not responding process.

All files besides gl1_gradient uses data files. This line should fix the problem.

I prefer to set up working directory instead. However, since it eliminates one of the most common pitfalls I'm merging it.


Sync master

Codecov Report

Merging #202 into master will increase coverage by 0.22%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #202      +/-   ##
============================================
+ Coverage     56.81%   57.04%   +0.22%
- Complexity     2174     5620    +3446
============================================
  Files           203      392     +189
  Lines          9226    22165   +12939
  Branches       1101     3027    +1926
============================================
+ Hits           5242    12644    +7402
- Misses         3255     7262    +4007
- Partials        729     2259    +1530

Impacted Files
Coverage Δ
Complexity Δ

...t/subscription/adapters/ResourceActionAdapter.java
44.11% <0%> (-26.48%)
4% <0%> (-2%)

...connect/subscription/ResourceActionParameters.java
34.88% <0%> (-6.98%)
7% <0%> (-2%)

...scribe/model/AsynchronousResponseNotification.java
37.2% <0%> (-4.66%)
8% <0%> (-3%)

...n/java/com/arm/mbed/cloud/sdk/DeviceDirectory.java
71.42% <0%> (-3.29%)
16% <0%> (+1%)

...om/arm/mbed/cloud/sdk/connect/model/EndPoints.java
85% <0%> (-2.88%)
8% <0%> (-9%)

...ud/sdk/devicedirectory/adapters/DeviceAdapter.java
66.93% <0%> (-2.06%)
19% <0%> (+1%)

src/main/java/com/arm/mbed/cloud/sdk/Billing.java
61.22% <0%> (-1.55%)
21% <0%> (+1%)

src/main/java/com/arm/mbed/cloud/sdk/Update.java
69.06% <0%> (-1.31%)
20% <0%> (+1%)

...ud/sdk/accountmanagement/adapters/UserAdapter.java
72.05% <0%> (-1.08%)
8% <0%> (-1%)

...ccountmanagement/adapters/LoginHistoryAdapter.java
54.54% <0%> (-1.02%)
3% <0%> (ø)

... and 423 more

Continue to review full report at Codecov.

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


Multiple Galleries

Is it possible to have multiple galleries? For example, a wedding gallery and a portrait gallery.
I can think of maybe two ways which might answer this.

Name the files according to their grouping and save in image folder. Use get.resource or resources.match.

Duplicate the current gallery on the theme and rename folder, links and components.

Any ideas of which is best?
Thanks for your time and for producing the theme. It's amazing.

We can take advantage of Hugo's Content Types to achieve this. If I want to add a new gallery called "portraits" at the ultimate URL "https://example.com/portraits/", I can simply create a new content folder that looks like this:
content/
 └── portraits/
    └── images/
    |   ├── file_1.jpg
    |   ├── file_2.jpg
    |   └── file_3.jpg
    └── _index.md

The front matter of _index.md would look like:
---
title: "Portraits"
type: "gallery"
url: "/portraits"
---

That's it!
I've added to the README to hopefully make this more clear. Thanks for the question @DigiJo!

Hi I followed these instructions, including file structure and editing _index.md but new galleries are only displaying images in the "gallery" folder. Am I missing something? Thanks

@erikparr Doesyour site repo live on GitHub? If so, would you link it please so I can take a look?
Also, please make sure you have the latest version of the theme.


[Documentation]: Update docs to include select/deselect behaviour for Details List

**Is this request for a new component, or additional functionality for an existing component? **
Existing component, DetailsList
Have you validated that your desired feature or component is not already in the backlog?
Yes
Describe the component and functionality you would like to see in the starter kit
Could the Details List documentation be updated with an example of how to handle de-select events on a DetailsList (e.g. on select add something to a collection, on remove remove something from a collection).
Both Paging as well as Search on the DetailsList clears the components SelectedItems so quite often you need to maintain what's been selected in a collection, and an example of how to do that would be great to have in the documentation.
Describe alternatives you've considered
Trying to figure it out through trial and error.

In addition, selections in general. I have a column in my collection, IsSelected, to have the ability to programmatically select a row. But I would like to update that column to reflect a change by the user, if they select a new row. The problem is when I try and update the underlying collection based on what the user selected, it causes the OnRowSelectionChange to get called over and over and then the list, even though a row is selected, the List.Selection property is Blank(). So now I have no isea what is selected.


Inlay variable hints should support including numeric separators

Inlay variable hints should include numeric separators (_) to match the style of the definition.
Current behavior:

const number: 123456 = 123_456;
const bigint: 123456n = 123_456n;

Desired behavior:
const number: 123_456 = 123_456;
const bigint: 123_456n = 123_456n;

Keywords:

typescript.inlayHints.variableTypes.enabled
javascript.inlayHints.variableTypes.enabled
numeric separator
number separator

I think it's almost same as https://github.com/microsoft/TypeScript/issues/45550
Both has TS displaying "calculated" value as type.


SPIKE: Filesystem watcher solution

We need to evaluate a solution for file system watcher. A notable one in the Golang space is github.com/fsnotify/fsnotify.
How to evaluate

we create a Go program that uses fsnotify lib.
we compile it for 2 major platforms, Linux, mac.
we test the program on all mentioned platforms.
we check the performance of the watcher

a whole podinfo repo + directory structure
msdemo yaml files + directory structure

Out of scope
We don't currently support windows. We don't necessarily don't want to support windows, but don't test it here.

Definitions

Local Directory is the directory of a Git repository on our local machine.
Bucket is an S3-compatible bucket managed by a feature implemented in this epic.
Object is an object inside a Bucket.  An object consists of the key and the value.  In our implementation, an Object represents a copy of a file in the Local Directory.  The key of each Object is the relative path of a file, whose content is kept as the value.
We don't have a semantic of Directory in the Bucket, and a Directory can be seen as a common prefix string of some keys.


Implement IoT device registry management API authorization

@dejanb looks like an OK approach to me :)

@enmasse-ci run tests profile=shared-iot

@enmasse-ci run tests profile=shared-iot

@enmasse-ci run tests profile=shared-iot

@enmasse-ci run tests profile=shared-iot

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: shared-iot
TEST_CASE:
TOTAL: 26
PASS: 23
FAIL: 0
SKIP: 3

@lulf @ctron squashed commits, this is now ready for merging IMHO


Change default font weight

400 is not a good default value for font weight. According to fontconfig source, the weight for regular font should be 80.

I don't think this is correct. There must be a scale conversion somewhere because 400 is actually a quite standard number to use for "normal" weight, and this works on my systems to using font families with dozens of weights to select the "normal" one. Dropping down to 300 in some font families will start selecting light or thin variants.

I don't think this is correct. There must be a scale conversion somewhere because 400 is actually a quite standard number to use for "normal" weight, and this works on my systems to using font families with dozens of weights to select the "normal" one. Dropping down to 300 in some font families will start selecting light or thin variants.

Thanks for commenting on this! After I read your comment, I digged deeper into the source codes. The conversion from OpenType weight (regular=400) to fontconfig weight (regular=80) is defined in the source code of fontconfig (see here, the FC_WEIGHT_REGULAR macro is defined to 80), so that when fontconfig loads a OpenType font, it will convert its font weight value (as defined here) to fontconfig's scale. However, an issue in SILE could be, it directly passes the font information in SILE.settings to justenoughfontconfig to get the font face (from here and eventually to here). If we assume the font weight used in SILE is OpenType standard font weight (400 for regular), we must convert the weight into fontconfig weight before send it to justenoughfontconfig._face.
Let me correct this pull request to convert the font weight before send it to fontconfig. I think I will do that in justenoughfontconfig.c.

Found out the weight is already converted in justenoughfontconfig.c. Let me close this PR. Maybe it's my fonts.conf file where something get wrong. Great typesetting software!


Warhammer End Times - Vermintide: Black screen on start

After starting the game I ended with black screen. I hear the sound but nothing else. Even the DXVK_HUD disappeared after few seconds.
Software information
System: Fedora 27 x86_64
Game: Warhammer End Times - Vermintide - Steam version
System information

GPU: GeForce GTX 960
Driver: 390.48-1.fc27.x86_64
Wine version: wine-staging 3.6
DXVK version: 0.42

Log files

d3d11.log: vermintide_d3d11.log
dxgi.log: vermintide_dxgi.log

Please test with latest master, the binary releases do not reflect the current state of development.
Anyway, this error indicates that dxvk is doing something illegal on the Vulkan side of things:
err:   DxvkDevice: Command buffer submission failed: VK_ERROR_DEVICE_LOST
Please export VK_INSTANCE_LAYERS=VK_LAYER_LUNARG_standard_validation and write both the standard output and error streams to a file. You might have to run the steam client with all that stuff enabled if the game launches itself through steam.
Or try to record an apitrace as requested in the issue template........

I tried it with the 1bcae90d and the game is now starting, but it crashed when switched to borderless fullscreen.
I'm adding the game log when running with borderless fullscreen and export VK_INSTANCE_LAYERS=VK_LAYER_LUNARG_standard_validation
Log files

d3d11.log: vermintide_d3d11.log
dxgi.log: vermintide_dxgi.log
game.log: vermintide.log

I noticed the same issue with black screen on start when the game isn't started in windowed mode and resolution 1280x720 (this is probably some default configuration).
I tried to start it in fullscreen, borderless fullscreen and in resolution 1920x1080. Encountered the same issue as above.
I updated the wine-staging to version 3.7 and used DXVK commit: https://github.com/doitsujin/dxvk/commit/ba53cf92acbdf60458e69ec7fcbabc1e597ae73d
##Log files

d3d11.log:  vermintide_d3d11.log
dxgi.log: vermintide_dxgi.log

I want to record an apitrace. Is there a way to temporary disable DXVK in wineprefix?
Or should I ran the apitrace with DXVK?

In order to use wined3d instead of dxvk, launch the game (or better, Steam) with:
export WINEDLLOVERRIDES=d3d11,dxgi=b
If that doesn't work (wined3d has its own issues), you can try recording one with DXVK.

After few retries I was able to get into game with DXVK enabled, no luck without it.
The game crashed when I tried to change the display mode to fullscreen.
I hope this will help to find the issue.

vermintide-apitrace-dxvk.log : vermintide-apitrace.log

Those logs are sadly completely useless. I won't be able to debug the issue if apitrace doesn't work.

seems like fixed
played a round without any problems

@MEXAHOTABOP
Did you tried to change the resolution?
This issue happens after change of resolution and switching to/from fullscreen.

oh i read only first post
i alerdy remove the game so test it later

Ping?

https://github.com/ValveSoftware/Proton/issues/1085

Closing then.


can't use minified library standalone

I downloaded the minified javascript directly from the site at this url:
http://verbnurbs.com/js/verb.min.js
But, when i require in this library in node, I get a stack like:
Error: Cannot find module 'numeric'
  at Function.Module._resolveFilename (module.js:338:15)
  at Function.Module._load (module.js:280:25)
  at Module.require (module.js:364:17)
  at require (module.js:380:17)
  at Object.<anonymous> (/Users/bdwyer/git/vecdraw/lib/verb.js:2:258)

Also, AFAICT, 'numeric' isn't listed as a dependency in package.json

Sorry for the late reply on this. numeric is no longer a dependency in 2.0.0. I'll be updating this source very soon.


Support TypeConverterAttribute on bound properties

It's very difficult to add a custom TypeConverter for framework types to have them bind correctly within ConfigurationBinder. The documentation for TypeConverterAttribute specifies that the attribute can be attached to a property, and that property will then use the specified TypeConverter during serialization/deserialization. Unfortunately, ConfigurationBinder doesn't take the property's attributes into consideration during BindProperty, instead passing the binding off to BindInstance with just the type of the property.
Presently, the only way to use custom TypeConverters is to register them using TypeDescriptor.AddAttributes before any configuration binding occurs, which is inconvenient if your library requires custom TypeConverters for standard types.

@ajcvickers We should retriage this, as I think TypeConverters might be a good option for how we deal with custom config binding of optoins in general.

Also look into general binding for options + config


Javascript API

Can we discuss a JS api for users who use something else than turbolinks ? Currently, the init/destroy functions are not exposed in any reusable global.
Also, would it be possible to enable/disable features from the init/destroy functions ? Ie. keep every
For the moment we're using a forked override of material.forms.js which is ok because it's a small script but it would be great to have a more upgrade-friendly solution in the future.
Would you be interested in reviewing design proposals ?

V2 branch addresses this issue by switching to google-mdc. Google library has standard in init/destroy live-cycle. And auto activation from data-... attribute.

So the problem not in turbolinks by itself. But in the missing component model and component live-cycle.
Without turbolinks this would arises in the dynamic form parts (ex formset add)
And generally, complex frontend requires some sort of components for js and css. Google-mdc addresses both issues with BEM for css, common Es6 base class for js component, and standard for component auto-wiring
https://github.com/material-components/material-components-web/blob/master/packages/mdc-auto-init/README.md

Thanks !
Found it: https://material.io/components/web/catalog/auto-init/
It's great ! Is v2 also using materializecss ?
Do you think we can switch to v2 right now if we're just using the forms part of django-material ?

V2 would be google-mdc based (but still with jQuery 3d-party  components)
I'm expect v2 open source would be useful around February 2018, and pro at April.
Currently v2 only has textinput/checkbox/select widgets implemented.

Found my answer: "This is django-material version 2 rewrite, replaces MaterializeCSS library with Google MDC. Work in progress."
Thanks !


⚠ Bofrost.at has degraded performance

In 53ad9a4, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 916 ms

Resolved: Bofrost.at performance has improved in 0241019 after 7 minutes.


Modify the template to separate ASP.NET Core and ASP.NET

Separated the ASP.NET and ASP.NET Core templates into separate template files. Both are now visible on the New Project dialog.
Fixes #883.

You should add an explanation of what you did for this PR, not just reference the bug. Referencing the bug only explains what you fixed. I am also interested on how you fixed the issue.

Removed a bunch of switch statements by separating the ASP.NET and ASP.NET Core dialog UI and view models into separate classes.
PTAL.

PTAL.

Just took a quick peek at this and didn't fully read everything yet but my general thoughts:

This is a little more than a bug fix and we should probably put this into a feature branch for a release.  At this point it may be good to start one for this quarter and we can dump all the fixes from our fixit to it?
Adding the headers clearly something we should do but I think it will be easier and cleaner if we do this for all files at once.

@ivannaranjo  @ILMTitan Thoughts?

@ILMTitan LGTM


Engine not capturing mouse when going fullscreen and back

In master going fullscreen doesn't capture the mouse pointer, one has to click (which may fire unwanted actions)
It was meant to be fixed by !856:

https://github.com/DaemonEngine/Daemon/pull/856

but we are currently running #922:

https://github.com/DaemonEngine/Daemon/pull/922

I don't know if it's a regression in that rewrite, or if it's a bug hidden somewhere else in our code, or a bug in libraries like SDL.

We also not capture the mouse when leaving fullscreen.

Reverting b434381ad5517d672a4e60b191ca773ada01719b (Simplify mouse mode handling) doesn't fix the mouse capture when toggling fullscreen.
I assume we have a kind of race condition: toggling fullscreen can be very slow, and the game is not (re)focused yet when we request the mouse pointer.

Fixed by #1146?

Yes


erdtree: 1.3.0 -> 1.6.0, add figsoda as a maintainer

Diff: https://github.com/solidiquis/erdtree/compare/v1.3.0...v1.6.0
Changelog: https://github.com/solidiquis/erdtree/releases/tag/v1.6.0
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 222244 run on x86_64-linux 1

1 package built:

erdtree


feat(snapshot::aufs): config root_path

It solves the aufs part of the #4514 and #4513 .
https://github.com/containerd/aufs/pull/21 needs to be merged along with this.
Signed-off-by: Siddharth Yadav<EMAIL_ADDRESS>

Please revendor github.com/containerd/aufs

@AkihiroSuda what do you mean by "revendor github.com/containerd/aufs"?

@sedflix update the commit that's in the vendor.conf file at the root of the repository; https://github.com/containerd/containerd/blob/master/vendor.conf#L106
After that run vndr github.com/containerd/aufs (you can install the vndr tool from https://github.com/LK4D4/vndr)
Running that should fetch the new version of the aufs source code into the vendor directory.
After that, add both vendor.conf and the vendor directory, amend your commit, and force push to update your PR

Latest commit for containerd/aufs looks to be https://github.com/containerd/aufs/commit/dab0cbea06f43329c07667afe1a70411ad555a86
(dab0cbea06f43329c07667afe1a70411ad555a86 )

@thaJeztah thank you so much for the detailed explanation. 😊


Little CPU spikes in LV2 version but not in VST3 version

Linux / Debian Based
Not sure if this means anything but something I noticed.
When using Ardour and looking at the DSP % meter in main toolbar, I notice that when using the LV2 version of Sfizz certain little CPU spikes occur when....
-First inserting the plugin into Ardour MIDI track
-Loading up an instrument into Sfizz
DSP will jump up about 5-9% when actions like these occur and then slowly drop back down.
I compared this to the VST3 version and the VST3 version does not do this at all.
Was just curious why this would be, just was not sure if it was just due to the formats being different or if the LV2 version maybe has some kind of issue?
Does anyone else notice this?
Thank You

I moved this to the other sfizz-gui Github page


Create simple ASP .NET Core Web API application

Created based on VS project template ASP.NET Core Web API


Remove Unnecessary "tss"

Why is this here?

Denied. Commit linting failed.
I will use this commit, however I will not merge this PR.


Extends Glider DAC checks

Adds the following checks
1. CF
  a. Missing or invalid standard_name attribute
  b. Missing or invalid units attribute
  c. No _FillValue
2. Missing Required Variables
3. Missing or incorrect Dimensions
4. File has profile data
   a. Check time array
     i. Values should be monotonically increasing or decreasing with the
        profile
   b. Check depth array
     i. abs(Z0 - Zend) is non-negligible
   c. Profile has data
     i. Pull out time & depth arrays
     ii. Remove all rows for which the row has at least 1 _FillValue
     iii. Number of remaining rows >= 2

Also fixes an issue with dtype comparison where differing endianness would cause an issue even if the number of bytes and basic type (i.e. int, floating point) were the same.
Refactors tests to correct issues with score counts.  Many of the tests are rather arbitrary, simply checking that a given result has achieved a particular score and it may be desirable to rewrite them in the future.  In the interim, this fixes test failures.

closing in favor of https://github.com/ioos/cc-plugin-glider/pull/29


Font issues

A) Please consider hosting them on your own site, instead of phoning to google.
B) Consider minimizing the fonts, by taking out unnecessary glyphs (if possible)
C) Whats up with the copy pasted font-faces in the CSS?

a) I will consider doing that
b) no, I don't wanna end up in a situation where I need a glyph but I stripped it out cause I thought I wouldn't need it.
c) it saves one layer of HTTP requests to google, yes, I benchmarked.


fix: checkpoint cef format issue

Handle checkpoint CEF log.
CEF format sample-
Jan 11 10:25:39 host CEF:Version|Device Vendor|Device Product|Device Version|Device Event Class ID|Name|Severity|[Extension]

Checkpoint format sample-
time=1611840576|hostname=r81-t279-leui-main-take-2|severity=Critical|confidence_level=High|product=Anti Malware New Anti Virus|action=Detect|ifdir=inbound|loguid= .....

In the case of Checkpoint CEF is missing the header. Hence generic CEF regex does not work for checkpoint logs.

@dhruvp-crest can you please verify the above with checkpoint TA

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

v3.0.4
GitHub release

Your semantic-release bot :package::rocket:


Dots in URL lead to 404 (regression?)

⚠ IMPORTANT ⚠ Please do not ignore this template. If you do, your issue will be closed immediately.

[x] Read the docs.
[x] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[x] This is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

Describe the bug
URLs with dots are leading to a 404 error instead of correctly displaying the corresponding page.
Note that it works fine with a project created with Vue CLI but it doesn't work with a project created with @vite/create-app (both using Vue 3 with Vue Router 4).
I guess this is a regression from https://github.com/vitejs/vite/issues/130 but many things have change with Vite 2 (especially project structure) so it's hard for me to identify why the bug reappeared (but with some guidance I'm happy to help).
Reproduction

🔴 Repro created with @vite/create-app (not working): https://github.com/Deckluhm/vite-dots-url-bug
🟢 Repro created with Vue CLI (working): https://github.com/Deckluhm/vue-cli-dots-url-bug

Run projects and go to http://localhost:3000/just.a.test (or a similar URL).
System Info

vite version: 2.0.1 (with @vitejs/plugin-vue 1.1.4)
Operating System: Ubuntu 20.10
Node version: 15.10.0
Package manager (npm/yarn/pnpm) and version: Yarn 1.22.10

I'm experiencing the same issue when using vite version 2.0.5.

There was an issue created about this already here: https://github.com/vitejs/vite/issues/2245
But it was closed because of the use of a workaround with a plugin, that may also help you while this is open.
Maybe there is a limitation I'm not aware of, but this looks like a bug, and the other issue was wrongly closed.

Thank you @matias-capeletto. I can confirm that it works with the plugin.

Hi! Did anyone solve problem with serving vite app with dots in several parts like
http://localhost:3000/j.smith/d.porter
I have a combined url with dots, in case of
http://localhost:3000/j.smith
vite-plugin-rewrite-all works great but when I have another one
http://localhost:3000/j.smith/d.porter
I still have a 404 not found

@TLIQ In short, the workaround provided via the Vite plugin https://www.npmjs.com/package/vite-plugin-rewrite-all seems to work for people.

The behaviour seems to be because vite assumes that if the final segment of the path contains .,  the user meant to navigate to a file. This is a pretty bad assumption to take because files without . are valid too (although, uncommon) and modern SPAs generally expect all paths except static assets to be routed to index.html so client-side routing can take effect.
One solution could be to check existance in the file system, and if the file is not there, serve index.html, similar to try_files is often configured in nginx to serve prod apps. With such an mechanism, no option like https://github.com/vitejs/vite/pull/2634 is necessary as it'll work correctly in all cases, but it will involve one additional stat call on each request.

Hi, I have the same problem. In my case, it's PocketBase adding a JWT token to the URL which I have no control over.

Hi, I have the same problem. In my case, it's PocketBase adding a JWT token to the URL which I have no control over.

Same problem till now

Hey everyone, ran into the same issue today. If you want a workaround that doesn't involve installing random "fix things" packages such as vite-plugin-rewrite-all, here is a simple middleware function you can add that works for me:
import fs from 'fs';

const dotPathFixPlugin = () => ({
  name: 'dot-path-fix-plugin',
  configureServer: (server) => {
    server.middlewares.use((req,_,next) => {
      const reqPath = req.url.split('?', 2)[0];
      if(!req.url.startsWith('/@') && !fs.existsSync(`.${reqPath}`)) {
          req.url = '/';
      }
      next();
    });
  }
});

Which just checks if the request is a path for an existing file (or special dev server path), and if not just rewrites it to /.
In your vite.config.js file you simply use it:
export default defineConfig({
  plugins: [
    // your other plugins
    dotPathFixPlugin()
  ],
  // rest of the stuff
}

And sure, there are "performance" implications to checking if a file exists, but as long as we are just talking a development server, I cannot see how this is a practical concern. I have to admit I never understood the modern obsession with ultra-fast / hmr, etc. when it means we have to sacrifice application design in favor of a good developer experience.

Hey everyone, ran into the same issue today. If you want a workaround that doesn't involve installing random "fix things" packages such as vite-plugin-rewrite-all, here is a simple middleware function you can add that works for me:

Interesting solution. Can you clarify what the /@ check is for? Is that some vite-internal routing prefix?

The Vite dev server requires the disableDotRule: true option, like in #2634.
But the Vite preview server requires the ignores: false option here: https://github.com/vitejs/vite/blob/57276b723749e647b81f0a67c8c7b69303f95cf7/packages/vite/src/node/preview.ts#L120-L134

Hey @silverwind, regarding the /@ paths:
I believe they may be a Solid.js specific behavior. In Solid the JS code for connecting to the HMR system is served on URL /@solid-refresh - they are neither a component or a file in node_modules as far as I can tell, but a special path used inside of 'vite-plugin-solid' to serve up this file.
Other implementations may not need this.

I don't know if that will be useful to someone, but I needed the rewrite rule to be applied in vite preview mode as well. Here what I used :
function dontRewritePath() {
    return {
        name: 'dont-rewrite-path',
        configureServer(server) {
            return () => {
                server.middlewares.use(
                    history({
                        // verbose: true, // Uncomment if you need to debug
                        disableDotRule: true,
                        rewrites: [{ from: /.*\.ext$/, to: () => '/index.html' }],
                    }),
                )
            }
        },
        configurePreviewServer(server) {
            server.middlewares.use(
                history({
                    // verbose: true, // Uncomment if you need to debug
                    disableDotRule: undefined,
                    rewrites: [{ from: /.*\.ext$/, to: '/index.html' }],
                }),
            )
        },
    }
}

I am not a js dev therefore I cannot explain why I used different syntaxes. It works !

Thanks Nicholas for the suggested fix!
This was really hard to find because being new to Vite, I didn't realize the dot in the URL was the issue. I just thought every path served from the dev server needed to correspond to a file, so I was looking for some sort of "how to do client-side routing with Vite" guide, to tell me how to define dynamic paths...

Given that . are valid characters and URLs and that https://www.npmjs.com/package/vite-plugin-rewrite-all has now almost 30k downloads weekly (plus people using a raw plugin in their vite config). I think it would be great not to leave this hanging too much longer. The longer is stays open, the longer people keep finding workarounds and not use dots in their paths, despite being totally valid

This took me a fair amount of to debug, and ended up resolving the issue with the above package mentioned (https://www.npmjs.com/package/vite-plugin-rewrite-all). Would love to see this resolved so we don't have to rely on external packages/plugins and continue using valid urls with ..

Thanks sapphi-red and patak-dev! This fixes the problem for me when switching to 5.0.0-beta.0

I've faced the same issue on vite/4.4.9. When the release date of 5.0.0 ?

https://github.com/vitejs/vite/issues/4344
https://github.com/vitejs/vite/issues/5632

having this issue with<EMAIL_ADDRESS>

It's not fixed in Vite 4 as it's a breaking change. You can start using the Vite 5 beta to fix it. Vite 5 stable will come around mid October.

it's a breaking change

How so ?


Add a translation option to require a specific version (or version range) for MSVC tool when building on Windows

Right now generator authors can pin to a specific MSVC tool version or Windows SDK version either by:

Running their generator from a Visual Studio Developer Command Prompt with a specific -vcvars_ver/-winsdk_version
Setting the VCToolsInstallDir/WindowsSdkDir environment variables before building the library, IE:

Environment.SetEnvironmentVariable("VCToolsInstallDir", @"C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.28.29910\");

However this is not super intuitive or discoverable, it'd be nice if Biohazrd provided official means of specifying a MSVC tools version preference.
We already do this for the Windows SDK in the private Win32 and DirectX generators, see InfectedWin32.Generator.WindowsSdkHelper.
This would enable a handful of useful features:

Makes builds more reproducible (minor differences in Windows SDK versions have caused translation churn between my desktop and laptop before.)
Allows using older MSVC versions when they deprecate/break our version of Clang. (See https://github.com/InfectedLibraries/Biohazrd/issues/176 or https://github.com/InfectedLibraries/Biohazrd/issues/98)
Allows smarter discovery of the MSVC toolchain. (Clang always uses the newest Visual Studio version, including preview versions. If you have 2019 with the C++ workload and 2019 preview without, Clang just fails.)

When we fix this, we should remove the workaround added in https://github.com/InfectedLibraries/Biohazrd/issues/98 and change things to avoid the affected versions and error if they're the only ones available. (Could probably be a warning, but the issue affects most non-trivial libraries and having it be noticeable is probably better than not.)

For the sake of limiting bus factor, here's WindowsSdkHelper mentioned above: https://gist.github.com/PathogenDavid/29794af32ac1c88fdd489d27a2debb39
It doesn't actually override what Clang uses using environment variables but instead uses --no-standard-includes and -isystem based on the logic Clang uses internally.
Usage looks like this:
// We currently target Windows SDK 10.0.14393.0, which is Windows 10 1607
WindowsSdkHelper sdk = new("10.0.14393.0");
TranslatedLibraryBuilder builder = new()
{
    Options = new TranslationOptions()
    {
        SystemHeadersAreAlwaysOutOfScope = false
    }
};
sdk.ConfigureBuilder(builder);

builder.AddFile(sdk.GetHeaderFilePath("um", "Windows.h"));
builder.AddFile(sdk.GetHeaderFilePath("um", "ShellScalingApi.h"));
// ... etc


[Question]

Is there a way to specify max file size when creating createSimpleFileLogger?

There is not a way to limit the size but a good place to look would be the rolling file appender.  It currently rolls on a specific time interval, hour, minute day, week, etc.  But it would be easy to simply roll based on file size.  so when a log file gets to a certain threshold, it gets rotated out.
feel free to fork and enhance and then submit a pull request if you want it merged in.  thanks...


CSHRCP-549 - amended brand logos

Meging it on trust , have a look though and ignore or correct the test which ever is the best option for now in another PR

I think CI is down as a whole


Update to Angular 16

As we are using an outdated version of Angular, we should update to the latest version in an incremental fashion.
This issue refers to the update from Angular 15 to Angular 16.

End of life for Angular 16 is in about a month: https://endoflife.date/angular
Shouldn't we upgrade to Angular 18, in order to be on a supported version?

End of life for Angular 16 is in about a month: https://endoflife.date/angular
Shouldn't we upgrade to Angular 18, in order to be on a supported version?

Yes we should, but the updates are not easy, and I would like to update only one version with each PR.


Update configure-proxy-internet.md

Made changes to the note as it is required in a scenario of offboarding a server from one tenant and onboarding to another tenant. Documentation may need to be updated to reflect this as a required value..
fixes#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/11085

@JohanFreelancer9 Please copy review

@mjcaparas Please merge this PR. Thanks


Audio Switcher 1.8 - favorite device forgotten?

Audio Switcher: <IP_ADDRESS>
OS: Windows7 Pro x64
Audio devices: Yamaha RX-V483 AVR connected via HDMI, Headphones
Audio drivers: Realtek HD Audio <IP_ADDRESS>99
I've noticed that sometimes Audio Switcher seems to "forget" its favorite setting for the AVR.
So, when I turn it on, AS doesn't allow quick switch (just stays on the headphones).
And when I check the program settings, the AVR device lost its star marking and I have to re-add it.
Do you know what could be the cause?
Can you try to fix this, please?

Thanks.
It's a limitation in Windows that's causing the device to be treated as a "new" device every time it's connected.
I use an internal identifier for the device to determine which devices are/aren't favourites, and reconnecting some devices causes them to get a new id, and the favourite device lost.
Sadly there's no way to change this default behaviour.
In the latest version, I enabled quick switching between all active devices when there are no favourites set.
The work around is to disable un-used devices, and you can quick switch on all other non favourite devices.


Move StrvGetType into internal namespace

Fixed #736

Thank you for your PR. This looks good to me :+1:. You just need to rebase your branch on the most recent version of the main branch so I can merge it.


This does not fix @hurricane642 issue, so I'll have another bug to find


Update README.md with new wiki domain URL

What type of PR is this?
Add one of the following kinds:

cleanup

Duplicate to #365


When adding an invalid Graph to the FederatedStore the graph may still be created in Accumulo

When an invalid Graph is added to the FederatedStore the graph is first initialised, then validated. This means that if the Graph is backed by Accumulo/HBase the graph may still be created in Accumulo, even though an exception is later thrown and the user is given an error message.
This means the graph is added to Accumulo, but not the FederatedStore. In the future a different graph could be added with a different schema, but with the same graphId. The federated store will assume this is ok and Accumulo will just assume the schemas match and will attempt to use the wrong schema. This will later cause issues in serialisation/aggregation within Accumulo.

Migration Steps
Some of the APIs in the FederatedStore have been changed in order to fix this bug. Any users of this store should be aware that this Store is still experimental (as documented in the README) and breaking APIs may occur.
When graphs are added the FederatedStore, the methods now take a GraphSerialisable object instead of a Graph object.

Merged into develop.


[pulsar-client-cpp] Fix memory leak caused by not being executed ClientConnection destructor

Motivation
Currently, closing a C++ Pulsar client object does not execute the destructor for the generated ClientConnection object. As a result, memory usage increases in applications that repeatedly create and close clients.
This pull-request is a continuation of #5246.
Modifications
The reason why the ClientConnection destructor is not executed is that the reference count of the shared pointer does not become 0 even when the ClientConnection object is closed. As a result of the investigation, it was found that if the shared pointer executor_ held by the ClientConnection object is reset, the reference count becomes 0 and the destructor is executed.
--- a/pulsar-client-cpp/lib/ClientConnection.cc
+++ b/pulsar-client-cpp/lib/ClientConnection.cc
@@ -1385,6 +1385,10 @@ void ClientConnection::close() {
     if (tlsSocket_) {
         tlsSocket_->lowest_layer().close();
     }
+
+    if (executor_) {
+        executor_.reset();
+    }
 }

 bool ClientConnection::isClosed() const { return state_ == Disconnected; }

However, with this fix alone, executing the following code causes an "Illegal instruction" error and the program crashes.
https://gist.github.com/massakam/ebd169caff3db802eee5001b0d06c980
The exact cause is unknown, but this issue doesn't seem to occur if closing ClientConnection before closing ExecutorServiceProvider. So, I changed the ConnectionPool destructor added in #5246 to the close() method and call it before closing ExecutorServiceProvider.

rerun cpp tests

rerun cpp tests
rerun integration tests

rerun java8 tests


Update simplenote.py submodule for rewritten commit.

The current master references simplenote.py @ d71dd84daeb05857f0403b412b8cb1a8d6bc076d, which appears to have been rewritten to 6d0610875add4af3a295ed1649d5a4b5b8e5ce41.
I have attempted to fix that.

Thank you VERY much for spotting this. Turns out I'd mucked it up even
more than this so some further corrections were required (I had to
retag simplenote.py which then meant a further update to the
submodule).


Add neclib, xarray, necstdb to pyproject.toml

Motivation

Use @KaoruNishikawa 's high-spec machine to resolve dependencies

What should be realised
Enable
import xarray
import necstdb
import neclib

@r-yamada1998
Now neclib only supports python >=3.8.
Is the support for Python 3.7 needed for THIS package?
Adding support for the Python versions in neclib is also possible.

@KaoruNishikawa
In fact, supportation of 3.7 or earlier is just a temporary implementation. No need to support older versions of python.
Let's purge 3.7.


show embed link to everyone

Clean up admin checks for links
Show embed option to all users if certificate is published

Coverage increased (+0.03%) to 81.82% when pulling 5ba46f81ff367f3ccf3050c59c372bdf506b6c91 on fix-show-embed-link-to-admins into 86ba558255642d8fd09568d3b8f9b5c387c51851 on staging.


Add API for outstanstding and overdue invoices report

Notion card
https://www.notion.so/saeloun/Create-API-s-for-outstanding-and-overdue-invoices-report-03baa78301bc4ea692fe54be6243444c
Summary

Preview

Type of change
Please delete options that are not relevant.

[ ] 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
not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[x] I have manually tested all workflows
[ ] I have performed a self-review of my own code
[x] I have added automated tests for my code

@shalapatil Please check the review comments from Keshav and resolve them as required.

@keshavbiswa Please check the review comments that have been incorporated by Shalaka. @shalapatil Please mark the resolved conversations as resolved .


[Feature Request] Display setting to align to top

Would be great to have a setting to align the screen to the top or bottom. Right now the screen is always centered.
This would make it possible to play vertically with attached controllers on mobile like in this image.

Good idea, will try to get something in the next release


chore(helm): stop writing on root file system

Description
(this PR replaces #657 because I messed up my branches, sorry)
Writing on the root filesystem fails when readOnlyRootFilesystem is set, which is the case in some environments, such as OpenShift. This PR fixes this.
It also removes one permission we used to ask for, because it looks unused. 🤞
Depends on #668  being merged and released
Fixes FL-1000
Checklist

[x] changelog was updated with notable changes
[x] documentation was updated

/e2e

End to end tests: :hourglass_flowing_sand: STARTING

/e2e --help

Usage: /e2e [options] [help]

/e2e may appear anywhere as long as it is on its own line

Options:
  --refs <value>                                         Extra refs (branch or tag) with format REPO=GIT_REF,REPO=GIT_REF.
  Supported repositories: hlf-k8s, orchestrator, substra-backend, substra-frontend, substra-tools, substrafl, substra, substra-tests, substra-ci.
  Example: /e2e --refs substra-backend=some_branch,orchestrator=some_tag (default: {})
  --tests-to-run, --tests <tests-to-run>                 Comma-separated list of tests to run. Valid options: sdk,substrafl,frontend or NONE. (default: "sdk")
  --benchmarks-to-run, --benchmarks <benchmarks-to-run>  Comma-separated list of workflows tests to run. Valid options: mnist, camelyon or NONE. (default: "NONE")
  --orchestrator-mode, --mode <orchestrator-mode>        Comma-separated list of orchestrator modes to run tests for. Valid options: standalone,distributed (default: "standalone,distributed")
  -h, --help                                             display help for command

/e2e --tests sdk,substrafl,frontend

End to end tests: :hourglass_flowing_sand: STARTING

/e2e --tests sdk,substrafl,frontend

End to end tests: :hourglass_flowing_sand: STARTING

/e2e --tests sdk
just keep removing tests until it passes

/e2e --tests sdk
please?

End to end tests: :hourglass_flowing_sand: STARTING

/e2e --tests sdk,frontend

/e2e --tests sdk,frontend


Improve mirror node information flow

Problem
The mirror node information needs to be included where relavent in the docs. The mirror node content needs to be reviewed for accuracy.
Solution
Update mirror node documentation
Alternatives
No response

The mirror node section was reviewed #329


Improve typing

Done: Reverted the ifs that were added and added a comment about the negative score items.


Accessing the Facebook Graph to retrieve more info.

Dear All,
Probably I am in the completely wrong location to ask question about this nice piece of code but I am wondering if it is possible to retrieve other information about the user, maybe group posts etc?
Due to issues with the PHP SDK from Facebook I am not able to use those routines to authenticate users and so I tried this code. It worked with some little changes almost within 5 minutes.
(Please advice me when I am in the wrong location, or asking the wrong type of questions. I am completely new here and like to learn.)
For now thank you all for the great work and please enjoy your weekend!

Hey @MrCoding! Once you obtain an access token from the Facebook provider, you can make calls to the Graph API using any HTTP client you like (e.g. Guzzle HTTP Client or file_get_contents()).
Here's an example of getting photos of the authenticated user with file_get_contents():
$url = 'https://graph.facebook.com/v2.4/me/photos?access_token='.$accessToken;
$res = file_get_contents($url);
$data = json_decode($res, true);

var_dump($data);

Hope that helps!


make --set more liberal

be more liberal in accepting names for qx environment variables via the --set option. Now everything but space and = are allowed in the names.
parse the value as js expression.

3 approvals, merging, thank you @oetiker !


Refactor the model

Discussed in https://github.com/TulipaEnergy/TulipaEnergyModel.jl/discussions/688

Originally posted by datejada July  1, 2024
Input data names

Double-check the is_seasonal name, since it is used by more assets than the storage (use_timeframe?)
#622

create_input_dataframes

[x] #702
[x] #705

Verify if we can use only one df (without using a dict) for assets_profiles and assets_timeframe_profiles (better after having a pipeline with timeframe); have a pipeline example before changing that.
table_tree stores all the data in dfs

 create_internal_structures

[ ] #706

Use DuckDB directly to calculate the weight as sum of the mapping in the RepresentativePeriod.

[ ] Clean the graph such that we only store the data that are not created before to avoid duplication, e.g., only leave names for flows/edges.

[ ] Have an internal table instead of a internal dictionary for the rep_periods_partitions. This allows more DuckDB integration.
Example:graph["Midgard"].rep_periods_partitions = Dict(
    1 => [1:3, 4:5, 6:12],
    2 => [1:1, 2:2, ... 6:6],
)
asset | rep_period | k | begin | end
"Mid" | 1          | 1 |     1 |   3
"Mid" | 1          | 2 |     4 |   5
"Mid" | 1          | 3 |     6 |  12

[ ] Double-check that where we need rep_periods profiles, maybe just call DuckDB, instead of creating all of them apriori.

compute_constraints_partitions

[ ] Rename constraint partitions to be closer to the constraint name itself (e.g., power or balance)

compute_rp_partitions

This function needs a full refactor when changing to table (DuckDB)

Big change

solve_model and solve_model!

[ ] Store the solution into tables (a.k.a. DuckDB) instead of the structure solution and the graph.

Clean the solution structure to (maybe?) don't store any vector/array (only keep objective function and some extra info). Even delete it?
Create a new table for the investments (assets, and flows)

create_model

comment -> if we do change all the partitioning to dataframe including the constraints partition computation, then construct_dataframes is not necessary.

[ ] constraints_partitions from Dict to df

When using tables the filters will be on the tables not in the graph

[ ] Change sets and names to be verbose (e.g., Ai -> assets_to_invest)
[ ] Double check the constraint dataframes AffExpr are correctly created with the aggregation in the model. Is unique working as intended?

add_expression_terms_intra_rp_contraints will need a refactor, and it will determine changes we need before and after
Notes on code structure:

constraints partition: lowest/highest -> to get the correct partition;
sum of workspace expressions: sum/unique -> to get the correct expression
profile_aggregation functions are in the table in the documentation

AND THEN:

[ ] Rename input data to make more sense

[ ] How to include Multi-Year?

[ ] Separate the model data from the scenario data

solve_model and solve_model!

[ ]  Store the solution into tables (a.k.a. DuckDB) instead of the structure solution and the graph.

@abelsiqueira @datejada  Can I try tackling this one? Or should I wait?

@clizbe you can do it, it's mostly independent from the rest.


Word object model

Patterned on the new Excel object model article as per plan.
We will add a graphic like the one in the Excel article in a later PR.

Docs Build status updates of commit 3dcb4a8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-js-api-common-api.png
:white_check_mark:Succeeded

docs/reference/overview/word-add-ins-reference-overview.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

docs/word/word-add-ins-core-concepts.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 364afda:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-js-api-common-api.png
:white_check_mark:Succeeded

docs/reference/overview/word-add-ins-reference-overview.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

docs/word/word-add-ins-core-concepts.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit da3a1fd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-js-api-common-api.png
:white_check_mark:Succeeded

docs/reference/overview/word-add-ins-reference-overview.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

docs/word/word-add-ins-core-concepts.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Build error on windows 10 x64 and node.js 11

><EMAIL_ADDRESS>install C:\Users\llc\AppData\Roaming\npm\node_modules\sqlite3
> node-pre-gyp install --fallback-to-build

node-pre-gyp WARN Using request for node-pre-gyp https download
node-pre-gyp WARN Tried to download(403): https://mapbox-node-binary.s3.amazonaws.com/sqlite3/v4.0.3/node-v67-win32-x64.tar.gz
node-pre-gyp WARN Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v67 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp WARN Pre-built binaries not installable for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v67 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp WARN Hit error Connection closed while downloading tarball file
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppBuild.targets(312,5): error MSB3491: Could not writ
e lines to file "Release\obj\action_before_build\action_b.1409F021.tlog\action_before_build.lastbuildstate". The proces
s cannot access the file 'C:\Users\llc\AppData\Roaming\npm\node_modules\sqlite3\build\deps\Release\obj\action_before_bu
ild\action_b.1409F021.tlog\action_before_build.lastbuildstate' because it is being used by another process. [C:\Users\l
lc\AppData\Roaming\npm\node_modules\sqlite3\build\deps\action_before_build.vcxproj]
  unpack_sqlite_dep
gyp ERR! build error
gyp ERR! stack Error: `C:\Program Files (x86)\MSBuild\14.0\bin\msbuild.exe` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Users\llc\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\build.js:262:23)
gyp ERR! stack     at ChildProcess.emit (events.js:182:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:240:12)
gyp ERR! System Windows_NT 10.0.14393
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\llc\\AppData\\Roaming\\npm\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "build" "--fallback-to-build" "--module=C:\\Users\\llc\\AppData\\Roaming\\npm\\node_modules\\sqlite3\\lib\\binding\\node-v67-win32-x64\\node_sqlite3.node" "--module_name=node_sqlite3" "--module_path=C:\\Users\\llc\\AppData\\Roaming\\npm\\node_modules\\sqlite3\\lib\\binding\\node-v67-win32-x64" "--napi_version=3" "--node_abi_napi=napi"
gyp ERR! cwd C:\Users\llc\AppData\Roaming\npm\node_modules\sqlite3
gyp ERR! node -v v11.0.0
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok

NodeJS 11 is not yet supported.

How about 10.13.0?

Closing in favor of #1063


Can i set a offset?

I have some thing floating on the screen like header. I want to set a offset to scroll more and show the content behind the floating header out.

Maybe someone can implement this and take some code from here: https://github.com/yiminghe/dom-scroll-into-view/blob/master/src/dom-scroll-into-view.js

Also wondering this same thing!

Maybe implement similar to:
https://github.com/yiminghe/dom-scroll-into-view/blob/master/src/dom-scroll-into-view.js#L46

Thanks to @iwangulenko and #127 this is now possible 😄

Hi all.
I've read the readme and it says:

offset
This was always a buggy feature and warned against using in v1 as it might get dropped. It's much safer to use CSS wrapper elements for this kind of thing.

https://github.com/stipsan/scroll-into-view-if-needed#offset
Seems that this feature has been removed in v2.
Shouldn't this issue be reopened then?
There seems to be no way to use this library on websites with position: fixed "floating" headers.

Actually, seems this like not an issue: center should be used (not top) and then it doesn't matter whether there's a floating header or not.
There're issues when the input field itself is higher than the screen height (for example, a big JSON editor) but that can be worked around by setting the input max height less than screen height.
So ignore the above comment.


Not able to get location when the app is background.

i have granted background location permission but not able to received any location updates in background.

please close this issue i m able to get location in background every 10 mins thanks for reply.

is it possiable to get location every 3 mins using pending intent in Fused Location Api in android 13?


feat(validation): use aurelia-validation 0.12.x

@RWOverdijk

@RWOverdijk that should get in. how to proceed. it's not protected (though probably should), hence i could merge (though i shouldn't)


fix policy enforce in KinD env.

Install config-policy-controller to make sure policies can be enforced in KinD clusters.
Signed-off-by: morvencao<EMAIL_ADDRESS>

/LGTM

/lgtm


Remove bool returns from check functions

Submisison Checklist

[x] Run unit tests: ./runTests.py test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary:

Changed all return types from bool to void for the check_* functions.
Updated doxygen documentation.
Updated calls that expected a bool return.

Intended Effect:
Make the math library cleaner.
I didn't change any of the implementation.
How to Verify:
Look at the diff. Lots of files, but it should be easy to see what's going on.
Side Effects:
Places that call the check functions and expect a boolean return will fail to compile. I found all the places in the math library. There may be some places upstream that will need to be fixed.
Documentation:
Included in the updates to the doxygen documentation.
Reviewer Suggestions:
Anyone.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

The failure's all warnings.

Thanks.
On Fri, Sep 9, 2016 at 9:21 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

The failure's all warnings.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/391#issuecomment-246078853, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZ_Fy4Rbq0W-G9PfWlj-VOadiQVjduJks5qogYigaJpZM4J5lyn
.

This looks fantastic.  It's a huge simplification of the error-checking code and the changes in the probability functions are simplifying, too.  Thanks for updating all the doc and for all the reformatting.

Thanks for taking a look. Hopefully I got all the compiler warnings and good catch on the redundant check. It was removed.

This affects Stan upstream and it passes these tests now. I'm going to submit a pull request to deal with this in stan-dev/stan.

I just pushed a branch on Stan that works with this change. I'm merging this pull request and Stan will be updated asap.


Airdrop: Implement ignored tests

Motivation
The Airdrop application's ignored tests can be implemented as the issue linked in the comments are closed.
// NOTE: This test is ignored as it uses the Fuel-Merkle crate. There is currently an
// incompatability with the Fuel-Merkle crate and the Sway-Libs Merkle Proof library.
// The issue can be tracked here: https://github.com/FuelLabs/sway/issues/2594

This issue is dependent on https://github.com/FuelLabs/sway-libs/pull/81 and a new release of sway libs.

Done.


Feat: Added pagination and secrets support to infoblox provider

This PR contains feats to infoblox provider:

Pagination support to get more that 1000 entries by zone. https://github.com/rancher/rancher/issues/10317
Updated url's to wapi v1.5 to support pagination.
Rancher secret support as an option to INFOBLOX_PASSWORD env variable. If INFOBLOX_PASSWORD is empty, will use INFOBLOX_SECRET.

Thanks for the review @alena1108 . Updated PR with your comments.


don't modify URLs in layout template

meetup.com event links break if there are more than one trailing slash. this doesn't affect the photos URL for some reason.
For example:

https://www.meetup.com/Berlin-Hack-and-Tell/events/252068900  (works)
https://www.meetup.com/Berlin-Hack-and-Tell/events/252068900/ (works)
https://www.meetup.com/Berlin-Hack-and-Tell/events/252068900// (fails)
https://www.meetup.com/Berlin-Hack-and-Tell/events/252068900// (fails)

Good point @vog . I've updated the PR.

Found another issue when checking the links. The edit link points to the gh-pages branch.
For example,
https://github.com/berlin-hack-and-tell/berlinhackandtell.rocks/blob/gh-pages/_posts/2018-06-26-no62-summer-ish-hacks.md
Should be,
https://github.com/berlin-hack-and-tell/berlinhackandtell.rocks/blob/master/_posts/2018-06-26-no62-summer-ish-hacks.md

:+1:


But in Firewall rules

There is a bug in firewall rules that adds good addresses on the block list.
I have jus tested and only one successful iPhone connection caused IP being added to drop list. The issue is not consistent however. It seems that when devices redial quickly they get added to the list.
More leeway is needed.

Does anyone have any idea how to solve this?

Exactly.
I am also having problems with SCRIPTs sending alerts via email. Does anyone know how to solve?


feat: 🎸 add maxCharacters prop to SQFormTextarea

✅ Closes: #104
Same as #103 but on the textarea component. For better readability I made the value of characterCounter a string instead of wrapping it in <small> because the label on the "outlined" variant is already pretty small.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


(feat): Improved the UI of favboards

As Silvia has asked, I created a new button which redirects to Search Page when you are on the boards page and also moved the boards button on the top of the search page.

@dobri1408 there are a few issues, see my screenshots:

Also this feature is only available and visible for logged in users, but now the icons are always visible (even on the homepage):


Support for Customized Description

While using Nexus staging repositories it creates the repository name with profile-name-. It becomes difficult to search for specific artifact in the Nexus Staging repositories GUI View belonging to same group.
With support for customized description user can override this with artifact name and spot the repository without any issue.
Currently description is being defaulted to "Created by de.marcphilipp.nexus-publish Gradle plugin"

Support for updating description is already added in nexus staging plugin which we will be using to close the staging repository so don't think we need it here.


[Bug] azure-configuration branch is missing

Describe the bug
Workflow "Check step 2, Set up the Azure environment" fails at the job. Create azure-configuration pull request with a error:
pull request create failed: GraphQL: Head sha can't be blank, Base sha can't be blank, No commits between main and azure-configuration, Head ref must be a branch (createPullRequest)
Error: Process completed with exit code 1.

To Reproduce
Steps to reproduce the behavior:
Follow Readme.md guide at the step 2.16 commit checks will fail and you will not be able to proceed the course.
Additional context
at this step workflow should create a PR from azure-configuration branch which is currently missing in the this template

See the discussion board for more on this: https://github.com/orgs/skills/discussions/14
It's not that the branch is missing. There is a missing yaml file (spinup-destroy.yml) and a missing commit of that file to the azure-configuration branch. This means that the head and base refs point to the same commit when trying to create the PR. That will throw the error you got because you can't create a PR with no commits in it. There must be a difference in history between HEAD and base refs.
I think everyone is waiting for the Skills team to take another look at this lab, because there are a number of issues with it that need to be fixed (See dicussion thread about it.)

Thanks for the information, is there any timeline for the bug fix? Thanks！

@heiskr / @emilyistoofunky any thoughts about this ?

We're open to pull requests. I believe a contractor ported this course from a previous Learning Lab version:

https://github.com/githubtraining/continuous-delivery-azure
https://github.com/githubtraining/continuous-delivery-azure-template

I'll need to defer to @skills/skills-content-reviewers for questions on course content.

We're now in December. Any chance this lab will be corrected anytime soon?

Any updates?

This exercise should be removed from the learning path at https://learn.microsoft.com/en-us/training/modules/github-actions-cd/3-create-workflow-deploy-azure
until it is fixed.

Thank you for opening this issue. I'm closing this issue out per https://github.com/skills/continuous-delivery-azure/pull/20#issuecomment-1445303419
The GitHub Skills folks don't control or influence the content on Microsoft Learn, just fyi.


New sessions should be created by a ws request from the static build and not as part of the rest request to the /app page.

Closing with: 2ad755ba57b7495419ad755aee6f2e74b0f3e047