az sf cluster create fails when providing a pfx file with AttributeError: 'NoneType' object has no attribute 'get'

Describe the bug
When using an existing key vault and certificate the az sf cluster create command fails with  AttributeError: 'NoneType' object has no attribute 'get'. Please find the stack  trace below:
ERROR: 'NoneType' object has no attribute 'get'
Traceback (most recent call last):
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\knack\knack\cli.py", line 197, in invoke
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\azure-cli-core\azure\cli\core\commands\__init__.py", line 373, in execute
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\six\six.py", line 693, in reraise
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\azure-cli-core\azure\cli\core\commands\__init__.py", line 347, in execute
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\azure-cli-core\azure\cli\core\commands\__init__.py", line 182, in __call__
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\azure-cli-core\azure\cli\core\__init__.py", line 440, in default_command_handler
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\azure-cli-servicefabric\azure\cli\command_modules\servicefabric\custom.py", line 204, in new_cluster
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\azure-cli-servicefabric\azure\cli\command_modules\servicefabric\custom.py", line 1119, in _create_certificate
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-jvxpk9ku\azure-cli-servicefabric\azure\cli\command_modules\servicefabric\custom.py", line 1635, in _create_keyvault
AttributeError: 'NoneType' object has no attribute 'get'

To Reproduce

Create a ResourceGroup named "MyServiceFabric" and Key Vault in that ResourceGroup named "MyServiceFabric-SF-Keyvault"
Add a certificate to the KeyVault named "MyServiceFabric" with the subject set to "CN=MyServiceFabric.southcentralus.cloudapp.azure.com"
Execute the following Powershell script

# Variables
$ResourceGroupName="MyServiceFabric"
$ClusterName= "$ResourceGroupName".Replace('.','')
$Location="southcentralus"
$Subject="$ClusterName.$Location.cloudapp.azure.com"
$VaultName="$ClusterName-SF-Keyvault"

# #GetKeys From the KeyVault
$VmPassword = "VmPassword"
$VmUserName = "VmUserName"

az keyvault secret download --encoding base64 --vault-name $VaultName --name $ClusterName --file cert.pfx

az sf cluster create --location $Location --resource-group $ResourceGroupName --certificate-file ./cert.pfx `
  --cluster-name $ClusterName --cluster-size 5 --os WindowsServer2016Datacenter --vault-name $VaultName `
  --vm-password $VmPassword --vm-user-name $VmUserName --debug

Expected behavior
Existing Key Vault and provided certificate is used and the service fabric cluster is created or a clear error message is displayed with a description on how to resolve the issue
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build)
MSI
CLI version (az --version)
azure-cli (2.0.50)

acr (2.1.8)
acs (2.3.11)
advisor (2.0.0)
ams (0.3.0)
appservice (0.2.6)
backup (1.2.1)
batch (3.4.1)
batchai (0.4.4)
billing (0.2.0)
botservice (0.1.1)
cdn (0.2.0)
cloud (2.1.0)
cognitiveservices (0.2.3)
command-modules-nspkg (2.0.2)
configure (2.0.19)
consumption (0.4.0)
container (0.3.8)
core (2.0.50)
cosmosdb (0.2.3)
dla (0.2.3)
dls (0.1.4)
dms (0.1.1)
eventgrid (0.2.0)
eventhubs (0.3.1)
extension (0.2.3)
feedback (2.1.4)
find (0.2.12)
hdinsight (0.1.0)
interactive (0.4.0)
iot (0.3.4)
iotcentral (0.1.3)
keyvault (2.2.6)
lab (0.1.3)
maps (0.3.2)
monitor (0.2.6)
network (2.2.8)
nspkg (3.0.3)
policyinsights (0.1.0)
profile (2.1.2)
rdbms (0.3.4)
redis (0.3.2)
relay (0.1.2)
reservations (0.4.0)
resource (2.1.6)
role (2.1.9)
search (0.1.1)
servicebus (0.3.2)
servicefabric (0.1.7)
signalr (1.0.0)
sql (2.1.5)
storage (2.2.4)
telemetry (1.0.0)
vm (2.2.7)

Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\akrone\.azure\cliextensions'

Python (Windows) 3.6.6 (v3.6.6:4cf1f54eb7, Jun 27 2018, 02:47:15) [MSC v.1900 32 bit (Intel)]

OS version
Windows 10 64-bit (10.0 Build 16299)
Shell Type (e.g. bash, cmd.exe, Bash on Windows)
PowerShell
Additional context
Add any other context about the problem here.

I am running into the same issue, but with parameters set to generate a cert rather than supplying one that already exists. In fact I was just following a tutorial... https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-quickstart-containers-linux

Thanks for the feedback. @a-santamaria is currently investigating.


Chatto and live data

Hi,
Is there any example code with live data.... ?  I am facing difficulties in understand how to send data and to work with FakeDataSource
please help!!

Hi @albarq. FakeDataSource is implementation of ChatDataSourceProtocol in example app. In your own app you should have your own implementation of ChatDataSourceProtocol that loads data from your servers. When data is loaded you should trigger delegate call chatDataSourceDidUpdate().
Let me show you an example. In Badoo we have BMAChatViewController which is subclass of BaseChatViewController. Also, we have our own implementation of ChatDataSourceProtocol called BMAChatViewModel. It is responsible for loading data from server and from cache. When data loaded either from cache or server BMAChatViewModel calls self.delegate?.chatDataSourceDidUpdate(). That's it.
class BMAChatViewController: BaseChatViewController {
    override func viewDidLoad() {
        ...
        self.viewModel = BMAChatViewModel()
        self.dataSource = self.viewModel
        self.viewModel.load()
    }
}

class BMAChatViewModel: ChatDataSourceProtocol {
    func load() {
        // load data from cache or server
        self.delegate?.chatDataSourceDidUpdate()
    }
}

This is very simplified example, but it should help you to understand what to do

Thank you for your explanation ..  for me i have finished the parts of loading all messages .. but my problems are:
1- how to send new messages to server ...
2- how  do live updating  for current received messages...
3- how to update current status of message
4- Right now i am using alamofire ... am I right?  or do I need to switch to socket.io?

Chatto is UI framework. It's not responsible for client-server communication. Each app should implement it separately. It's up to you what solution to use. Regarding your questions from UI point of view:

Please, check DemoChatViewController in example app.

    private func createTextInputItem() -> TextChatInputItem {
        let item = TextChatInputItem()
        item.textInputHandler = { [weak self] text in
            self?.dataSource.addTextMessage(text)
        }
        return item
    }

In textInputHandler you can handle a new text message (same for PhotoChatInputItem and photoInputHandler parameter for photo message). It's up to you how to deliver this message to server.

Whenever your data is updated you need to notify view controller that data source changed using self.delegate?.chatDataSourceDidUpdate(). Everything else will be handled automatically.

Your data source should track message status and update appropriate properties on your model. For example, when you start sending message to the server you can change status property of your MessageModelProtocol implementation to .sending value. Whenever state changed just call self.delegate?.chatDataSourceDidUpdate()

Again, it's up to you. In Badoo app we are using sockets, because it allows you to have two way communication between client and server.


Bump terser from 4.8.0 to 4.8.1, 5.14.0 to 5.14.2

Let me know if anything is missing

@mergifyio backport release/3.2.x


$NODE_PATH is not being set, so it can not find global modules

For whatever reasons, nvm is not setting correct NODE_PATH, and hence my builds can not find the globablly installed modules.
my app does this require('ibrik') but it fails, even if the ibrik is installed globally
I think this is same problem as http://stackoverflow.com/questions/27876557/node-js-configuring-node-path-with-nvm
Am i supposed to set it manually ?
MacBook-Pro:coffee sudhir$ nvm ls
->       v6.3.1
default -> 6.3.1 (-> v6.3.1)
node -> stable (-> v6.3.1) (default)
stable -> 6.3 (-> v6.3.1) (default)
iojs -> N/A (default)
lts/* -> lts/argon (-> N/A)
lts/argon -> v4.4.7 (-> N/A)
MacBook-Pro:coffee sudhir$ nvm use 6.3.1
Now using node v6.3.1 (npm v3.10.3)
MacBook-Pro:coffee sudhir$ which npm
/Users/sudhir/.nvm/versions/node/v6.3.1/bin/npm
MacBook-Pro:coffee sudhir$ which node
/Users/sudhir/.nvm/versions/node/v6.3.1/bin/node
MacBook-Pro:coffee sudhir$ echo $NODE_PATH
<EMPTY>

MacBook-Pro:coffee sudhir$ npm ls -g -depth 0
/Users/sudhir/.nvm/versions/node/v6.3.1/lib
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
This is my exact problem - https://github.com/karma-runner/karma-cli/issues/22
Not sure if its problem because of nvm or its a karma bug

This is intentional. NODE_PATH should never be set, and global modules both are not requireable by default, and should never be required. Something you require is a dependency, and thus, must be installed locally.

Doesnt work even when i have ibrik installed locally..

@snimavat when it's locally installed, you should use an npm run-script so that it sets up the PATH for you. Alternatively, ./node_modules/.bin/ibrik should work.

I will tryout and let u knw, this is the only module which doesnt work... other locally installed works just fine..

@snimavat according to<EMAIL_ADDRESS>there's no "main" - so that entire package is not requireable. It is only a command-line tool.

Crazy.. i am not a node expert, just trying to get coverage for the coffee files, and karma coverage examples used it the same way as i posted above...
https://github.com/karma-runner/karma-coverage/blob/master/examples/coffee/karma.conf.coffee
This is strange, but this setup works on other machine..
Note sure where this issue belongs to.. here, or karma, or karma-coverage..

It looks like v1 did in fact have a "main" -<EMAIL_ADDRESS>- so I think you're just using the wrong version of ibrik. I'd file the issue on karma-coverage - it's definitely not related to nvm.

Thansk for clarifying, karma-coverage asks to use ibrik 2.. i will try to get help from them.. thanks

question on this - shouldn't / couldn't NODE_PATH get set for repl usage?

otherwise you can't really include global modules easily when using a REPL with nvm unless you do some hackery. sorry to reanimate a dead thread...

@illegalnumbers you shouldn't be including global modules ever under any circumstances. If you want to play with something in a repl, install it locally, and then it'll be available to require.

why though? sometimes it does make sense to install and use global dependencies outside of the scope of a project especially if you're using it in a repl. there are many points of common functionality from parsing files of specific types to outputting in specific ways that could be common.  it feels rather arbitrary to say to never use a part of npm.

No, it never does make sense to do that; if it's required, it's a dependency, and it should be installed locally.

local to what though that's the question - it's not a project it's just a repl. it's a global usage area

What use case do you have for playing with require that isn't going to end up being a project?

exploratory testing a new module, using something for a script that isn't part of the project (parsing  files with a syntax that is less terrible than doing bash scripting), etc

npm install --no-save new-module && node; as for the script, if it requires anything, it'd need a package.json to be able to be reproducible on another machine, so it's a bad idea to use globals for that too.

TIL --no-save which i think would fulfill what I needed for that particular use case and true, but I guess the prerequisite to what I was thinking would be that you aren't putting as part of the project and just running it as a one off

@maxatwork if it’s requireable in a project, it is a dependency of a project, full stop. Separately, global things are never requireable, only runnable as command line tools. If an editor wants to do something absurd like require something that isn’t a dependency of the editor, it’d be the editor’s responsibility to employ hacks to make that work, not nvm’s.

I found this thread because I have a requirement to use the appdynamics node module. The npm install for that module pulls a version specific to the os. This breaks my shrinkwrap file because it hashes the module and it's always different between windows, mac, and linux. I can either install it globally, or I have to set the user-agent when I build so that it pulls the version for the server I'm deploying to. It's a mess. I was exploring using require with a global module for this specific package.

@dustinaleksiuk typically using a shrinkwrap file also means every consumer has to be on the same OS (when you have compiled deps).

Appreciate the concise schooling on global vs. local modules. Especially late at night and doubly so when trying to use global packages when nvm was designed specifically to only locally define the node environment.


Error: ER_NOT_SUPPORTED_AUTH_MODE: Client does not support authentication protocol requested by server; consider upgrading MySQL client

Hi i have some problems to access to my database the error is in the title. My auth plugin is mysql_native_password.
Can someone help me ?
EDIT : nvm I fixed it

how did you fix it ?


No volumes listed since the update to 0.7.0

Since I updated longhorn from Rancher UI from version 0.6.2 to 0.7.0, no more volumes are listed in Longhorn.
On the other hand, there are still the instance-manager-r-*** and instance-manager-e-*** pods of version 0.6.2 (in addition to those of version 0.7.0).
How to import volumes from the previous version? Have I forgotten a step for the update?

We update our API group version from alpha to beta, but we also have an upgrader to migrate all Longhorn related resources automatically during the upgrade. Can you check if those Longhorn volumes are migrated to the new version via the command:
kubectl -n longhorn-system get volumes.longhorn.io

You can also check if the old Longhorn volumes are still there:
kubectl -n longhorn-system get volumes.longhorn.rancher.io

Volumes do not seem to have been migrated.
First command returns:
No resources found.

Second command returns:
pvc-10912e5b-a2ea-11e9-a3b6-e41f13804639   138d
pvc-1174f0ef-a2ea-11e9-a3b6-e41f13804639   138d
pvc-14fe5f5b-c298-11e9-a3b6-e41f13804639   98d
pvc-2eb7abc4-9c11-11e9-a3b6-e41f13804639   147d
pvc-7f5bf0a9-9e39-11e9-a3b6-e41f13804639   144d
pvc-a79597712b8911e9                       290d
pvc-a8be48c52b8911e9                       290d
pvc-eeb881dd-9c08-11e9-a3b6-e41f13804639   147d

How to manually execute the upgrader?

@JRial95 The upgrader should run automatically. Can you take a look at manager logs to see if you can see things related to the Upgrade at the beginning of the log?

The longhorn-manager has nothing specific to the update in its logs. However, they contain these warnings:
time="2019-11-26T08:16:00Z" level=warning msg="Can't find instance manager for pod instance-manager-r-*, may be deleted"
time="2019-11-26T08:16:00Z" level=warning msg="Can't find instance manager for pod instance-manager-e-*, may be deleted"

During the update, I had the following problem:
Error starting manager: settings.longhorn.io \"crd-api-version\" is forbidden: User \"system:serviceaccount:longhorn-system:longhorn-service-account\" cannot get resource \"settings\" in API group

I solved it by manually updating the ClusterRole according to this file: longhorn/deploy/longhorn.yaml.

@JRial95 crd-api-version is the field used for upgrade, so unable to access it is the reason the upgrade process wasn't started. There is something wrong with your upgrade since Helm should update the permission for settings.longhorn.io (and other CRDs).
I am not sure what went wrong in the upgrade stopped the right RBAC to be deployed, but maybe worth a double-check.
After you update the ClusterRole, you can delete all Longhorn Manager pods (not the daemonset) and they should be recreated and the upgrade process should be triggered.
Let me know if you figured out what's happened to the RBAC rules.

I deleted all longhorn-manager pods but nothing new happened. There are always these warnings:
time="2019-11-26T08:16:00Z" level=warning msg="Can't find instance manager for pod instance-manager-r-*, may be deleted"
time="2019-11-26T08:16:00Z" level=warning msg="Can't find instance manager for pod instance-manager-e-*, may be deleted"

Here are the messages displayed when the pod is launched:
time="2019-11-26T18:10:39Z" level=info msg="Event(v1.ObjectReference{Kind:\"Node\", Namespace:\"longhorn-system\", Name:\"k8s\", UID:\"8167be7c-e1b9-4a56-9155-8a5f0ca2f38d\", APIVersion:\"longhorn.io/v1beta1\", ResourceVersion:\"106531828\", FieldPath:\"\"}): type: 'Normal' reason: 'Ready' Node k8s is ready"
time="2019-11-26T18:10:39Z" level=debug msg="Requeue longhorn-system/k8s due to conflict: Operation cannot be fulfilled on nodes.longhorn.io \"k8s\": the object has been modified; please apply your changes to the latest version and try again"
time="2019-11-26T18:10:40Z" level=info msg="Event(v1.ObjectReference{Kind:\"Node\", Namespace:\"longhorn-system\", Name:\"kubernode4\", UID:\"014a7520-a177-48d3-850a-4bfc3493cdee\", APIVersion:\"longhorn.io/v1beta1\", ResourceVersion:\"106531801\", FieldPath:\"\"}): type: 'Normal' reason: 'Ready' Node kubernode4 is ready"
time="2019-11-26T18:10:40Z" level=debug msg="Requeue longhorn-system/kubernode4 due to conflict: Operation cannot be fulfilled on nodes.longhorn.io \"kubernode4\": the object has been modified; please apply your changes to the latest version and try again"
time="2019-11-26T18:10:41Z" level=info msg="Event(v1.ObjectReference{Kind:\"Node\", Namespace:\"longhorn-system\", Name:\"kubernode3\", UID:\"2db0e3f7-4d00-4ce3-8d9c-1d0ab37edf85\", APIVersion:\"longhorn.io/v1beta1\", ResourceVersion:\"106531867\", FieldPath:\"\"}): type: 'Normal' reason: 'Ready' Node kubernode3 is ready"
time="2019-11-26T18:10:41Z" level=debug msg="Requeue longhorn-system/kubernode3 due to conflict: Operation cannot be fulfilled on nodes.longhorn.io \"kubernode3\": the object has been modified; please apply your changes to the latest version and try again"

NOTE: After the update, the longhorn-engine version 0.6.2 was not recognized by Longhorn. So I tried to create a new one (also version 0.6.2) from the Longhorn GUI hoping that the volumes would be bound to the new engine. Unfortunately, it didn't work, so I deleted the engine (also from the Longhorn GUI) but it also deleted the engine that worked with the old Longhorn version. Have I lost my data?

@JRial95 I don't think you lost your data since according to your description, Longhorn v0.7.0 hasn't recognized the old volume yet.
That error information shouldn't cause any trouble.
To check if old CRDs are still there, run
kubectl -n longhorn-system get volumes.longhorn.rancher.io

You should see if the old CRDs are still there.
And
kubectl -n longhorn-system get replicas.longhorn.rancher.io

Should give you the location of your replica data in the field at NodeID with DiskPath.
We can try the upgrade process again once you verified your v0.6.2 CRDs are intact.

Remove the manager daemonset

kubectl delete -f https://raw.githubusercontent.com/longhorn/longhorn-manager/master/deploy/install/02-components/01-manager.yaml

Clear the setting to retrigger the upgrade process

kubectl -n longhorn-system delete settings.longhorn.io crd-api-version

Re-create the manager daemonset

kubectl create -f https://raw.githubusercontent.com/longhorn/longhorn-manager/master/deploy/install/02-components/01-manager.yaml

Monitor the log, it should say something like Upgrade from v1alpha1

I checked and my old CRDs are still there.
I followed your steps but the logs say that an update is not needed:
time="2019-11-26T20:11:10Z" level=info msg="Start overwriting built-in settings with customized values"
time="2019-11-26T20:11:10Z" level=info msg="New upgrade leader elected: kubernode1"
time="2019-11-26T20:11:36Z" level=info msg="Start the upgrade process"
time="2019-11-26T20:11:36Z" level=info msg="No upgrade is needed"
time="2019-11-26T20:11:36Z" level=info msg="Upgrade completed"

Unfortunately, it didn't work and the logs still display these messages:
time="2019-11-26T20:11:36Z" level=warning msg="Can't find instance manager for pod instance-manager-e-***, may be deleted"
time="2019-11-26T20:11:36Z" level=warning msg="Can't find instance manager for pod instance-manager-r-***, may be deleted"

Those logs are fine.
How many nodes/managers are there? Can you check the log of all of them?
The following command should at least retriggered the upgrade process in one of them.
kubectl -n longhorn-system delete settings.longhorn.io crd-api-version

I restarted the update again with the steps you described and found in the logs this warning:
level=warning msg="Cannot verify current CRD version, assume it's not v1alpha1: unable to verify if version matches v1alpha1: settings.longhorn.rancher.io \"default-engine-image\" is forbidden: User \"system:serviceaccount:longhorn-system:longhorn-service-account\" cannot get resource \"settings\" in API group \"longhorn.rancher.io\" in the namespace \"longhorn-system\""

So I certainly didn't copy the whole ClusterRole configuration when I did the update manually 🤦♂
After correctly applying the ClusterRole configuration, the update was successfully completed.
@yasker Thank you for your help.

@yasker I have the same issues except neither my new or old crd's are there and I ran into an issue before hand where I had to run this: https://github.com/longhorn/longhorn/blob/master/docs/upgrade-from-v0.6.2-to-v0.7.0.md#error-kind-customresourcedefinition-with-the-name-xxx-already-exists-in-the-cluster-and-wasnt-defined-in-the-previous-release
My volumes and pvc's still exist and they have the new "volume.beta.kubernetes.io/storage-provisioner: driver.longhorn.io" but I cannot see them in the UI and I get this when trying to restart one of my containers:

AttachVolume.Attach failed for volume "pvc-f522a0b9-b919-11e9-ac62-005056ac7a40" : rpc error: code = NotFound desc = ControllerPublishVolume: the volume pvc-f522a0b9-b919-11e9-ac62-005056ac7a40 not exists

That claim exists though

@ColonelBundy You cannot find old CRDs according to https://github.com/longhorn/longhorn/issues/912#issuecomment-558782473 ? That's weird. Those must be removed manually using the steps in https://github.com/longhorn/longhorn/blob/master/docs/upgrade-from-v0.6.2-to-v0.7.0.md#post-upgrade

@yasker I figured out i accidently removed the old crd's so I ran restore of my nodes and redid the upgrade and it worked.


side panel and menu icon now animated with css transforms

Following the merge of #37, made some quick adjustments on how the side panel and menu icons are being animated.
Something I learned last year after I worked on Hikari V2 is how css transforms are actually being handled by the computer instead of the browser, which plays a big role in performance optimisation.
More here: http://blog.teamtreehouse.com/increase-your-sites-performance-with-hardware-accelerated-css
Now, I want to do v2.5 :D
PR if you want to check. Will merge in 24 hours.

Nice! :+1:

nice!


Inability to Drag Columns on Tablets, Mobiles, or Touch Devices

material-react-table version
v2.11.1
react & react-dom versions
v18.2.0
Describe the bug and the steps to reproduce it
While column dragging functions as expected on laptops or PCs using a mouse cursor, it is not achievable on mobile or tablet devices due to the absence of a cursor. Touch interactions do not enable column movement or dragging.
I've attached the video for your ready reference.
Thanks in advance.
Screenshots or Videos (Optional)
https://github.com/KevinVandy/material-react-table/assets/153281510/5a90060c-7c82-42bc-bec6-5fc25b6a8cb7
Do you intend to try to help solve this bug with your own PR?
No, because I do not know how
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

duplicate of #577


Core library: F*: support for the Default trait

hax doesn't seem to support Rust's Default trait. Would it be possible to add support for it?
Originally posted by @ROMemories in https://github.com/future-proof-iot/RIOT-rs/pull/205#discussion_r1534117627

What exactly is failing here? I just tried a simple example with a custom Default implementation, that works.
Can you post a reproducer for the issue?

Oh, interesting. Let me re-open and investigate.


Web site down

Not directly connected to the repository. But the web site http://www.padrinorb.com/ is down.
I can't stop wondering if the project is now officially dead.

Hmm thanks for bringing this up, the entire website is open source here: https://github.com/padrino/padrino-web. It's been a while, I forget who manages the dns pointing to the github pages for the site @wikimatze @DAddYE I am happy to take over the domain costs to keep the site running as well if I don't already manage it and just forgot.

Thank you guys for pointing this out, my bank issued a new card this year which prevented the automatic renewal of the domain. I updated the details and we should be good.

Looks like it’s back up now, thanks Davide :pray:


Parse ansible '#jinja2:' overrides

This change allows to use ansible templates as is

Is there documentation around what this format is? I've never seen this before.

Is there documentation around what this format is? I've never seen this before.

https://docs.ansible.com/ansible/latest/collections/ansible/builtin/template_module.html
Also, you can override jinja2 settings by adding a special header to template file. i.e. #jinja2:variable_start_string:'[%', variable_end_string:'%]', trim_blocks: False which changes the variable interpolation markers to [% var %] instead of {{ var }}. This is the best way to prevent evaluation of things that look like, but should not be Jinja2.

Basically, any jinja2 environment variables can be set this way


NoClassDefFoundError

I got this error on using the Float Action Button
java.lang.NoClassDefFoundError: com.nineoldandroids.view.ViewHelper
at com.gc.materialdesign.views.ButtonFloat$1.run(ButtonFloat.java:108)

Importing nineOldAndroid makes the app function but that shouldn't happen

Ran into this a moment ago. Importing NineOldAndroids Fixed it for me too. Maybe documentation is outdated...

I ran into this as well. Adding compile 'com.nineoldandroids:library:2.4.+' to the gradle file made it work.

same error here, and thx @raghuveer-s , that line solved my problem :D

Just add:
compile 'com.nineoldandroids:library:2.4.+'


Project References page not available from left-hand Handbook menu

Looks like it is there, under Project Configuration?

I guess this can be closed.


oxipng: 4.0.0 -> 4.0.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/oxipng/versions.
meta.description for oxipng is: "A multithreaded lossless PNG compression optimizer"
meta.homepage for oxipng is: "https://github.com/shssoichiro/oxipng"
meta.changelog for oxipng is: ""
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 4.0.1 with grep in /nix/store/sk7z7mn78fbp1amv4vljmp099vb3grsx-oxipng-4.0.1

directory tree listing: https://gist.github.com/fe02fbaa2a60566825645aff8be62e22

du listing: https://gist.github.com/6345583b7de74a5d6b882a28e5b90c30

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
oxipng

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sk7z7mn78fbp1amv4vljmp099vb3grsx-oxipng-4.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oxipng https://github.com/r-ryantm/nixpkgs/archive/797cf968a25e24e669d6291e2615cc7249c29ed8.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sk7z7mn78fbp1amv4vljmp099vb3grsx-oxipng-4.0.1
ls -la /nix/store/sk7z7mn78fbp1amv4vljmp099vb3grsx-oxipng-4.0.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

oxipng

Maintainer pings
cc @dywedir for testing.

Result of nixpkgs-review pr 105323 run on x86_64-darwin 1

1 package built:

oxipng


Update to Agents v6

Use SVectors instead of NTuples
Define microbes with the @agent keyword (Closes #50)
General API improvements

Currently tracking Agents#main because still unreleased

Need to merge even though agents v6 is not out yet


Kernel compilation failure

Hi all,
I tried to build the image at the current HEAD "7cfb0e892822f339e8a0c1f0207426cc388daa23" in the branch master, but I get a compilation error in the kernel recipe because it is looking for the commit "7f26b4456f70f9909c19936d550cf7c5dc47e1a5" which is not present in the kernel repo.
Patching the recipe to work with the current Linux version "4.19.27" it works fine.
Follows the patch, can you please verify?
diff --git a/recipes-kernel/linux/linux-raspberrypi_4.19.bb b/recipes-kernel/linux/linux-raspberrypi_4.19.bb
index 28aed98..6ebf838 100644
--- a/recipes-kernel/linux/linux-raspberrypi_4.19.bb
+++ b/recipes-kernel/linux/linux-raspberrypi_4.19.bb
@@ -1,9 +1,9 @@
FILESEXTRAPATHS_prepend := "${THISDIR}/linux-raspberrypi:"
-LINUX_VERSION ?= "4.19.25"
+LINUX_VERSION ?= "4.19.27"
LINUX_RPI_BRANCH ?= "rpi-4.19.y"
-SRCREV = "7f26b4456f70f9909c19936d550cf7c5dc47e1a5"
+SRCREV = "c0e09b3420442df016ee7906985535d644481e4b"
SRC_URI = "
git://github.com/raspberrypi/linux.git;protocol=git;branch=${LINUX_RPI_BRANCH}
"

@stefano-vallarino looks good. Push a PR. Mind the commit log (check git log for reference https://github.com/agherzan/meta-raspberrypi/commit/7cfb0e892822f339e8a0c1f0207426cc388daa23).

This should be fixed now in master. Thanks for reporting it.


Add regex & command prefix filters to reaction cog

Add a command prefix check to prevent commands from firing the spooky reaction cog
Translate the spooky reaction check into a regular expression

Prevents matching of substrings (e.g. "halloween" matching "halloweenify"
Allows for flexible matching of certain reactions (e.g. "spoooooky" fires the "spooky" reaction)

At the moment the bot is responding to its own messages, so if a command response contains a trigger word it drops a reaction on it. This seems like a fitting PR to address that in.

Short-circuit added


feat: add more condition and make cpl concurrent install

Fix #85

@CodiumAI-Agent

Preparing review...

PR Analysis

🎯 Main theme: Adding more conditions and making component plan concurrent install
🔍 Description and title: Yes
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
✨ Minimal and focused: Yes, the PR is focused on enhancing the component plan by adding more conditions and making the installation concurrent.
🔒 Security concerns: No, the PR does not introduce any apparent security issues.

PR Feedback

💡 General PR suggestions: The PR is generally well structured and follows good coding practices. However, it lacks tests for the new functionality. It is highly recommended to add tests to ensure the new code works as expected and does not break in the future.

🤖 Code suggestions:

relevant file: api/v1alpha1/componentplan.go
suggestion content: Consider using a more descriptive name than 'Actioned' for the condition type. It would be more informative if the name could reflect the state it represents. [medium]

relevant file: controllers/componentplan_controller.go
suggestion content: The error handling could be improved. Instead of logging the error and continuing the execution, consider returning the error and stopping the execution. This would make the error handling more robust and prevent potential issues down the line. [important]

relevant file: controllers/componentplan_controller.go
suggestion content: The function 'isHelmDoing' could be renamed to something more descriptive like 'isHelmOperationInProgress'. This would make the code more readable and maintainable. [medium]

relevant file: controllers/componentplan_controller.go
suggestion content: Consider adding comments to the code to explain the logic behind the operations. This would make the code easier to understand and maintain. [medium]

How to use

Tag me in a comment '@CodiumAI-Agent' to ask for a new review after you update the PR.
You can also tag me and ask any question, for example '@CodiumAI-Agent is the PR ready for merge?'

@CodiumAI-Agent

Preparing review...

PR Analysis

🎯 Main theme: Enhancement of the ComponentPlan functionality
🔍 Description and title: Yes
📌 Type of PR: Enhancement
🧪 Relevant tests added: Yes
✨ Minimal and focused: Yes, the PR is focused on enhancing the ComponentPlan functionality by adding more conditions and making the component plan concurrent install.
🔒 Security concerns: No, the PR does not introduce any obvious security concerns. The changes are mostly related to the internal logic of the ComponentPlan and do not involve any sensitive operations such as handling user input or interacting with external systems.

PR Feedback

💡 General PR suggestions: The PR is generally well-structured and follows good coding practices. However, there are some areas where the code could be improved or clarified.

🤖 Code suggestions:

relevant file: controllers/componentplan_controller.go
suggestion content: Consider adding more comments to the code to explain the logic behind the changes, especially in the Reconcile function where there are several conditions and branches. This will make the code easier to understand for other developers. [medium]

relevant file: api/v1alpha1/componentplan.go
suggestion content: The function componentPlanCondition has a switch case that is no longer used and can be removed. This will make the code cleaner and easier to read. [important]

relevant file: controllers/componentplan_controller.go
suggestion content: In the Reconcile function, there are several places where the error handling could be improved. Instead of just logging the error and returning, consider adding more context to the error message using fmt.Errorf before returning it. This will make debugging easier if an error occurs. [important]

relevant file: controllers/componentplan_controller.go
suggestion content: In the Reconcile function, the error returned by helm.GetLastRelease is not handled properly. If an error occurs, the function should return immediately instead of continuing to execute the rest of the code. [important]

How to use

Tag me in a comment '@CodiumAI-Agent' to ask for a new review after you update the PR.
You can also tag me and ask any question, for example '@CodiumAI-Agent is the PR ready for merge?'


[WIP] Code review and potential bug

Just fixing some minor things in the code.
I was trying to convert the distributed QPE example to the new qubit circuit style and I may have uncovered a bug.
A simpler use case is implemented in the phase_kickback.py file.  Essentially when I try to access the second qubit on a QPU, it isn't able to find it for some reason.
I'll continue to investigate.

The latest push fixes the bug, but I've uncovered another issue. When a CNOT is performed in EQSN, the qubit threads are merged and the qubit object is stored in the first object (I think) and the other is lost. I think this is a bug and I'll have to investigate it further.
What this means is that if I perform a CNOT between qubits with the same target and different controls, the second time I run it, it fails. I may have a work around for this temporarily.


xidlehook: 0.6.0 -> 0.6.1

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build xidlehook

Success on aarch64-linux (full log)
Attempted: xidlehook
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/xdcs5z61j97gjm3l76j3ljz8awh8bjdy-xidlehook-0.6.1
shrinking /nix/store/xdcs5z61j97gjm3l76j3ljz8awh8bjdy-xidlehook-0.6.1/bin/xidlehook
shrinking RPATHs of ELF executables and libraries in /nix/store/xdcs5z61j97gjm3l76j3ljz8awh8bjdy-xidlehook-0.6.1
shrinking /nix/store/xdcs5z61j97gjm3l76j3ljz8awh8bjdy-xidlehook-0.6.1/bin/xidlehook
strip is /nix/store/6dpnd5aniypn8124mmy8f88s4mq2zl07-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/xdcs5z61j97gjm3l76j3ljz8awh8bjdy-xidlehook-0.6.1/bin
patching script interpreter paths in /nix/store/xdcs5z61j97gjm3l76j3ljz8awh8bjdy-xidlehook-0.6.1
checking for references to /build in /nix/store/xdcs5z61j97gjm3l76j3ljz8awh8bjdy-xidlehook-0.6.1...
/nix/store/xdcs5z61j97gjm3l76j3ljz8awh8bjdy-xidlehook-0.6.1

Success on x86_64-linux (full log)
Attempted: xidlehook
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1
shrinking /nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1/bin/xidlehook
shrinking RPATHs of ELF executables and libraries in /nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1
shrinking /nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1/bin/xidlehook
strip is /nix/store/rpbg8gmqxhz8g61p1plz5d2srs84pvmv-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1/bin
patching script interpreter paths in /nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1
checking for references to /build in /nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1...
warning: working around a Linux kernel bug by creating a hole of 1822720 bytes in ‘/nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1/bin/xidlehook’
/nix/store/dr0h0ga7p8yq6g59c37isajsfxawxwvl-xidlehook-0.6.1

Success on x86_64-darwin (full log)
Attempted: xidlehook
Partial log (click to expand)
   Compiling failure v0.1.3
   Compiling xidlehook v0.6.1 (/private/tmp/nix-build-xidlehook-0.6.1.drv-0/source)
    Finished release [optimized] target(s) in 1m 14s
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/pvbcgygcjvvyxk38dr38d0rc3a8f4zjn-xidlehook-0.6.1
strip is /nix/store/rk3p99jbj4idl3dkkg1rxja5gz0qgs98-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/pvbcgygcjvvyxk38dr38d0rc3a8f4zjn-xidlehook-0.6.1/bin
patching script interpreter paths in /nix/store/pvbcgygcjvvyxk38dr38d0rc3a8f4zjn-xidlehook-0.6.1
/nix/store/pvbcgygcjvvyxk38dr38d0rc3a8f4zjn-xidlehook-0.6.1


[security] Transaction  Argument runtime verify

We need to verify the transaction argument in runtime to match verifier #152

Tracked by #202


Add support for cyclic dependency in union type definitions

Purpose
This PR adds support for cyclic dependency in union type definitions
Fixes #23690
Note: Opening as a Draft PR, more tests needs to be added.
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closed and re-opened because github workflow has been updated.

Closed and re-opened because github workflow has been updated.

This example crashes.

This is resolved.

Closing this PR, since @KRVPerera is working on a fix on top of this PR.


Exclude methods with missing references when enumerating the methods of a TypeDef

Fixes dart-windows/win32#721
@timsneath It's this annoying exception again, which we've encountered before at  https://github.com/dart-windows/win32/issues/697#issuecomment-1541500049.
In that pull request, we disabled the failed test to mitigate the issue. However, it seems that we missed a code snippet in tool\win32gen\bin\generate.dart, which is causing the exception described in dart-windows/win32#721.
To address this issue, this PR catches the exceptions that occur when parsing methods due to missing references, and it also prints some information about the problematic methods:

What are your thoughts on this approach?

@timsneath  Could you help with this pull request to unblock https://github.com/dart-windows/win32/issues/721? Thanks!


E2E test workflow

Done:
This is one attempt to bring the e2e tests for the native part (ios) to the CI related to #3 .
The tests are made using appium.
The e2eios  workflow builds the application using the expo server and exports it as an ipa ready for ios simulator.
We download the ipa from the expo's server and we unzip it. The ipa is unzipped (appium fails loading it triggering an error during the verification of the package, if you run appium in macOS locally you don't need to unzip it because the system does it magically for you 🤬).
We install appium and we run  yarn e2e:ios:run:ci
this command start appium and run the tests concurrently
the test script runs
In this repository https://github.com/brainrepo/expo-ci-proto, you can find my first attempt.
Here the CI workflow https://github.com/brainrepo/expo-ci-proto/runs/2095445200?check_suite_focus=true
if you check line 1467 and 1468 in run appium tests step you can see that the test runs
[e2e-t] 2021-03-12T13:58:59.279Z INFO webdriver: RESULT Open up App.js to start working on your app!
[e2e-t] Message: <Open up App.js to start working on your app!>

To improve:

The download ipa name is hardcoded in the test file
The tests should be done using a test framework and the test response have to be more readable.
Actually, the test doesn't end because we need to switch off appium graceful.

To evaluate
This workflow relies strongly on expo server, the builds steps are doing using expo servers which are so slow. Can be interesting to become completely autonomous using turtle-cli.
Please note: to be completely independent we need to have a web server with https, to allow turtle-cli to retrieve the js payloads and the assets. Turtle-cli is a copy of expo build system, you can use it by giving the expo credentials (in this case you have to upload the payloads and assets in their servers) or you can publish in your webserver passing to turtle-cli the url. In this last case I think that we will be completely autonomous.
https://docs.expo.io/distribution/hosting-your-app/#hosting-your-static-files

Supersesed by new work


Added additional examples used for disconnected cluster deployments

Added examples of existing parameters required for disconnected clusters to the "install-config-example.yaml" file.

Do you happen to know if these work? I believe I left them out of the example when I first wrote this because disconnected installations didn't work with Hypershift, but I haven't tried in a while

Hi LoganThe added examples do work, but to make disconnected clusters work you need a couple extra additions to the helm chart.in order to make the HCP worker node use the correct mirror sources there are a couple extra post sync scripts that need to be added. I will be creating another pull request to add the extra variables once we have tested it a bit more and formatted it better.P.s. Thank you for creating the helm chart, it has saved us a lot of time and pain. Johannes H. T. JohansenAssociate cloud consultantRed HatOn 25 Jun 2024 04.02, Logan McNaughton @.***> wrote:
Do you happen to know if these work? I believe I left them out of the example when I first wrote this because disconnected installations didn't work with Hypershift, but I haven't tried in a while
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


infra for returning MockResponse

instead of moving through the whole pipeline of urllib, I feel like it'll be much better to return a Response directly.
This PR works towards making this happen.

Codecov Report

Merging #1 (28a1fd7) into main (e101bc2) will increase coverage by 1.41%.
The diff coverage is 83.63%.

@@            Coverage Diff             @@
##             main       #1      +/-   ##
==========================================
+ Coverage   85.09%   86.50%   +1.41%
==========================================
  Files          10       10
  Lines         483      378     -105
==========================================
- Hits          411      327      -84
+ Misses         72       51      -21

Impacted Files
Coverage Δ

tests/test_urllib3.py
100.00% <ø> (ø)

pytest_response/app.py
83.89% <75.75%> (-4.46%)
:arrow_down:

pytest_response/interceptors/requests_quick.py
83.33% <83.33%> (ø)

pytest_response/interceptors/urllib_quick.py
86.15% <86.15%> (ø)

pytest_response/database.py
80.64% <100.00%> (+0.31%)
:arrow_up:

pytest_response/plugin.py
95.00% <100.00%> (-0.46%)
:arrow_down:

pytest_response/interceptors/urllib3.py

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 e101bc2...28a1fd7. Read the comment docs.


Investigate alternatives to responsiveness of tables

Right now we have horizontal scrolling on smaller devices which works fine, but there could be improvments.
There are other suggestions here that may be useful to implement.
The box shadow technique mentioned on here is clean: https://www.456bereastreet.com/archive/201309/responsive_scrollable_tables/
Also see:
https://community.digital.gov.au/t/table/92/2
and:
https://github.com/alphagov/govuk-design-system-backlog/issues/61#issuecomment-430300205

I wanted to try and avoid using javascript for box shadows. Here is what I came up with:
Variation 1
As per the gov uk tables post above. Collapse each row in the table into a summary list:
https://codepen.io/sukhrajghuman/pen/VJEyJe?editors=1100
Variation 2
Make use of CSS gradients to add shadows in outer container edges. This doesn't work well for striped tables
https://codepen.io/sukhrajghuman/pen/VJEXMB

Variation 1
Collapse each row in the table into a summary list:
https://codepen.io/sukhrajghuman/pen/VJEyJe?editors=1100
Variation 2
Make use of CSS gradients to add shadows in outer container edges. This doesn’t work well for striped tables
https://codepen.io/sukhrajghuman/pen/VJEXMB
Summary of questions from Design Guild:

Could the striping for variation 2 be removed when the shadows are in use?
What if the shadows in variation 2 were white gradients instead?


Handle exception for invalid column names containing .

Issue #, if available:
#99
Description of changes:
Columns with dot in their name are throwing an Analysis Exception
Based on analysis done , the exception is being thrown by the core scala deeque library
Have proposed to handle this exception to notify the caller that a column name with "." should not be passed
the exception handled is added to all check methods and implemented via a static method
Local Tests were passing and have added an additional UT test_invalidColumnException to verify the behaviour
*version = "1.0.1"
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

hi @chenliu0831  thanks for the review
In that case this PR handles this exception and notifies the caller to not pass a column with a "." so that he can handle it outside this library

hi @chenliu0831 thanks for the review In that case this PR handles this exception and notifies the caller to not pass a column with a "." so that he can handle it outside this library

@sidharthbolar right I think I understand the purpose, but column with dot is a valid usage for structured column (JSON type). Do you mind add a test to verify if that works?
Looks like you actually want a better Pythonic error (than the vanilla Spark error) when such column with dot does not exist? If so, how about add a helper to check column existence instead?


Plug leak in perf buffers

Small change to remove a memory leak in the perf buffers. Otherwise, the missing perf_reader_free() triggers ASAN leak check.
Checklist

[ ] Language changes are updated in docs/reference_guide.md
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

Nice!


Add a Coil utils module and a KmpFileFetcher

Coil is a well-known network image loading library use by many people. In Coil, you can extend the image loading pipeline by adding, among others, custom fetchers that will express how to download an image given any model.
This PR adds a custom Coil Fetcher to be able to pass a KmpFile to Coil and it will display the local image gracefully.

Should we document this somewhere? Otherwise, it'll be hard to find.

Yes this should be documented here:
https://github.com/MohamedRejeb/Calf/blob/main/docs/filepicker.md


Support .HEIC and .MOV (HEVC encoded) files

Currently, files with extension .HEIC, .MOV, and .mov are ignored and not included in the output. Such files are commonly generated by Apple devices such as iPhones.

Thanks for the suggestion @inakaboy. I've pushed an update which adds support for HEIC and MOV file extensions out of the box, and also adds a config.ts file which allows the list of supported extensions to be easily customised.

Great, thanks, amazingly quick! Actually I forgot to mention that yesterday I started working on a fork to add more file types, but your solution is more elegant than mine anyway. Thanks very much!


fix(ui): :bug: fix exported image tab styles for only single tab

Strips out accent css classes of WindowTab and TerminalWindowTabList after cloning
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/riccardoperra/codeimage/blob/main/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Exported image was having active tab styles even when single tab was present
Issue Number: 278
What is the new behavior?
This will export code image without active tab styles if there is only a single tab present
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
Here is screenshot of image having only single tab

Here is one with multiple tabs

Cool! I am totally newbie to solidjs, will try to do that thing

Hi @piyushjohnson, thank you for this contribution.
Unfortunately this fix cannot work in production build because Vanilla extract minify the generated classes..so you are not able to use querySelector (this is also not a recommended approach while using SolidJS)
Since the dom-export export an image in base of how it’s rendered in the DOM , my suggestion is to apply a css class (even with the data-tabs=1 selector, but it could also be directly :first-child:last-child in the .css.ts) in order to override the active style or override it.
feel free to ask if you have any question


No support for typescript application

Hi! this api looks nice, but I can't use it because it doesn't have a type declaration file, for typescript project.

Hi @slacerda85,
I would be very happy if you could contribute to make this possible.
Right now I don't believe I have enough knowledge in typescript to support it.


Update README.md

After some testing, newer versions of python 3 and windows sdk are not supported.

It's a bit surprising that new Windows SDKs are not supported. Do you happen to have the compiler error messages?

Yes, after switching to an older version of SDK, it builds successfully.

How did you change the Windows SDK version? Did you generate a visual studio solution and manually retarget the solution?

Ah I see. Thanks for the PR!


A tool to extract all rules matching a host (Address Groups included, and multi VSYS)?

Hi,
I was wondering if there is any way to "extract" all the rules from XML snapshot that matches a HOST.
I mean, exporting the XML and searching, for example, the rules that matches the host "<IP_ADDRESS>/32" is easy, but with the Address Groups involved... that can become a hard task.
Also, any way that handle multi VSYS configurations?
Thak you very muchfor the effort.

Hi,
This library is multi-vsys and device-group aware. What you describe is perfectly doable within a custom script using my library (example-xxxx.php scripts shows you the basic skeletons of a script).
Also note that the utility utils/rules-edit.php cannot 'export xml' but can filter rules on almost any parameter (included network matches) and you can then tag, delete, disable/whatever action is supported.

I've been trying with that and I'm getting this error:

** ERROR  ** *   xml  element  was not found

With the example .XML is working finee, the error is when using my own .XML.
This is the part of your XML (I guess) is used  to locate the "vsys1":

no
normal
no

And  this is the part of my XML used to locate  "vsys1":

As you can see, both are quite  different, so, you know where the error can be?
(I can't past you  my entire   because is  300.000 lines long!
Thanks again,

You're mixing Panorama templates and PANOS vsys I think.
My scripts work on full configs of the product, not parts of them. Just load the whole config (Panorama or PANOS/Firewall) into the scripts.

Hi,
This is just what I'm doing here. I make an snapshot of the Panorama
configuration, then an export and this is the .XML file I pass to the
script.
Marcel Sánchez Toledano
2015-03-31 18:49 GMT+02:00 Christophe<EMAIL_ADDRESS>
You're mixing Panorama templates and PANOS vsys I think.
My scripts work on full configs of the product, not parts of them. Just
load the whole config (Panorama or PANOS/Firewall) into the scripts.
—
Reply to this email directly or view it on GitHub
https://github.com/PaloAltoNetworks-BD/pan-configurator/issues/2#issuecomment-88168542
.

what is the full error message ?

This is the full  error message:

Thanks,

You are trying to load a Panorama configuration with a PANConf object that is meant to be used for Firewall configs.
Please use PanoramaConf object instead.

Hi again,
My mistake, working as expected now with PanoramaConf.
With Addresses and Services is working perfectly fine, but  I'm having problems dealing with Rules:
$countGroups2 = $sub->addressStore->count();

$countGroups3 = $sub->serviceStore->count();

$countGroups4 = $sub->ruleStore->count();

ERROR * * Died on user notice or warning!! Error: Undefined property: DeviceGroup::$ruleStore

rules are in a different store for each time:
rule->securityRules
rule->natRules
rule->decryptionRules
all of them are RuleStore class.
please have a look at the /doc/index.html to find variables and their class names. Also example-basics-panos.php should help you to get a picture of the different places to look at.

Hi, following your advice I've been able to run my tests. But I have one doubt, with: ServiceRuleContainer class. I cant find any example using that class! Maybe you can help me with that..
Actually, what I need to use is a particular method: hasObjectRecursive( $my_address ). If I pass and Address name, will this method find a Rule which have an Address Group (and that Address Group contains this Address).

Hi,
I've been able to run some tests with your help. But I have the following problem:
I have all the AddressGroups in an array. Can I use a foreach to iterate all this AddressGroups and check if any of this has (as Source or Destination) an Address?
$object = "NAME_<IP_ADDRESS>";
$adressGroup->hasObjectRecursive( Address|AddressGroup $object )
This will look inside every AddressGroup? Looking for Addresses?
Can't find any example with hasObjectRecursive.
Thanks!

yes, do a foreach loop for each object you want to test.
this function is 'recursive' because it will look inside the rule's groups
if object is there (and inside the groups of the groups ... it's nested).
but be careful, it doesn't look 'inside' an AddressGroup you provide as an
argument, it is not the purpose. It looks inside groups that are part of
the AddressRuleContainer.
On Sat, Apr 4, 2015 at 10:05 PM, msanchezt<EMAIL_ADDRESS>wrote:

Hi,
I've been able to run some tests with your help. But I have the following
problem:
I have all the AddressGroups in an array. Can I use a foreach to iterate
all this AddressGroups and check if any of this has (as Source or
Destination) an Address?
$object = "NAME_<IP_ADDRESS>";
$adressGroup->hasObjectRecursive( Address|AddressGroup $object )
This will look inside every AddressGroup? Looking for Addresses?
Can't find any example with hasObjectRecursive.
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/PaloAltoNetworks-BD/pan-configurator/issues/2#issuecomment-89653401
.

Sorry, I see the first part of your answer, but not the second part...
What I'm doing is iterating an array with all the AddressGroups to find which ones contains a certain Address. Then, I will iterate each Rule to find which rules contains this Address or Addresses Groups. That way I can make an export of all the rules related with, for example, a host in particular.
But you are talking about "AddressRuleContainer", actually I don't really understand what is the exact purpose of this class. Any example where it's used? Can you give me a brief explanation about it? Because maybe this class could save me some time and some lines of code!

AddressRuleContainer is used by rules to manage source and destination
objects.
you can access them from $rule->source and $rule->destination
in fact all my example-xxx script are using them as you can see :)
anyway, let's imagine your scenario where you are looking for SecurityRules
using an object, you have 3 or 4 ways to do what you are looking for:
version 1 : use rule filter
$objectNameToLookFor = 'server2';
$objectionLocation = 'device-group1';
$panoramaFile = 'config.xml';
$panorama = new PanoramaConf();
$panorama->load_from_file($panoramaFile);
$deviceGroup = $panorama->findDeviceGroup($objectLocation);
// the following will select all rules from device group matching a
specific query
$rules = $deviceGroup->securityRule->rules("(from has.recursive
{$objectNameToLookFor}) or (to has.recursive {$objectNameToLookFor})" );
// now do your you want with the rules:
foreach( $rules as $rule )
{
// do something with $rule here
}
version 2 : do filtering by yourself in the code, dont use filter
$objectNameToLookFor = 'server2';
$objectionLocation = 'device-group1';
$panoramaFile = 'config.xml';
$panorama = new PanoramaConf();
$panorama->load_from_file($panoramaFile);
// look for specific device group
$deviceGroup = $panorama->findDeviceGroup($objectLocation);
// look for specific object
$object = $deviceGroup->addressStore->find($objectNameToLookFor);
if( $object === null )
die("ERROR : object '$objectNameToLookFor' not found \n");
$rules = $deviceGroup->securityRule->rules( );
foreach( $rules as $rule )
{
if( $rule->source->hasObjectRecursive($object) ||
$rule->destination->hasObjectRecursive($object) )
{
/// do something with $rule here
}
}
On Sun, Apr 5, 2015 at 6:44 PM, msanchezt<EMAIL_ADDRESS>wrote:

Sorry, I see the first part of your answer, but not the second part...
What I'm doing is iterating an array with all the AddressGroups to find
which ones contains a certain Address. Then, I will iterate each Rule to
find which rules contains this Address or Addresses Groups. That way I can
make an export of all the rules related with, for example, a host in
particular.
But you are talking about "AddressRuleContainer", actually I don't really
understand what is the exact purpose of this class. Any example where it's
used? Can you give me a brief explanation about it? Because maybe this
class could save me some time and some lines of code!
—
Reply to this email directly or view it on GitHub
https://github.com/PaloAltoNetworks-BD/pan-configurator/issues/2#issuecomment-89805512
.


Exponential Processing Time

Strings of format (/a)n require exponential time to be highlighted.
Some clients using HLJS, such as RocketChat, freeze attempting to process strings potentially taking millions of years to process, due to this.
Example:

\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a\/a

hljs.initHighlightingOnLoad();

Is this still an issue?  Any details on which grammar is actually at fault here would be helpful.  Or a PR with a test to reproduce the issue.

Ping.

If any beginning would like to see if this is still a reproducible issue in 2019 that'd be greatly appreciated.

@dnl-blkv https://jsfiddle.net/ajoshguy/fp82xa7k/
I can't reproduce this.  If this is still an issue please provide an updated jsfiddle that reproduces the issue and we'll be happy to look into it!


Impliment less percise and consolidated velocity / position in smart networking

Velocity updates don't need to be done by actual velocity updates, rather it can just be done by the player sending what is moving towards + rotation (4 bit movement flags + rotation). The position doesn't need to be very accurate either, and can just be a relative change. The rotation can be the same.
1 byte per position axis (3 bytes vs 24 bytes), the same for rotation (2 bytes vs 16).
This would reduce velocity updates to 3 bytes (from 24 bytes), remove 16 bytes + header from rotation updates when there is also a velocity change, and reduce position update size by 21 bytes.

I estimate ~60% reduction in traffic from this when including packet headers in the calculation.


Model binding to Controller Properties

@harshgMSFT please update this comment with the suggested design.
Including how we deal with [FromBody] appearing twice, is there a TryUpdateModel for the controller? What we do about validation?

Initial thoughts:
Representation :
Controller properties are treated as additional action parameters which are common for all actions.
This gives us a single place where we can bind all controller properties ( which are interesting, more on this later). along with actual action parameter. This will ensure things like validation and handling of multiple [FromBody] attributes is uniform and is at one place ( model binding phase).
Also since the information about the controller properties (of interest) will be part of Action descriptor it will also be available to api explorer.
Since action parameters are passed as a dictionary to any action filters we will have a custom dictionary with source as the controller object. This will ensure any modifications to the dictionary reflect on the controller instance.
This would also mean any conflicts in name would be disastrous as they will override each other's value. I would recommend this dictionary to be case sensitive ( which is the case currently) to avoid name clashes.
Discovery of Controller properties:
The discovery happens when action descriptor is created, viz. in ControllerActionDescriptorBuilder
There are four categories of public properties which can be defined on the controller

Those without any Model binding related attributes (eg From*, ModelBinder etc). and without any Activate attribute.
Those with only activate.
Those with only model binding related attribute.
Those with both activate and model binding attributes.

Case 1 and 3 should be considered as action parameters while case 2 and 4 should not.
Case 4 is weird and we should consider it as an invalid scenario and throw.
Also we should provide an opt out mechanism for controller properties which need to be excluded from model binding. We can consider this to be similar to [BindNever].

Decisions :

The controller properties are not present in the actionArguments dictionary. If a filter needs it, he can get it via controller object.
Activate attribute will be no longer applicable for Controller ( will still be used for view components etc). The activate concept for controllers will be replaced by a new attribute like FromBinding (or ModelBinderAttribute.. the jury is still out on that.).
Controller properties are opt in.
Controller Properties will be added to actionDescriptor.Parameters  collection.
Validation, ApiExplorer, Errors for multiple FromBody, work as they look at actionDecriptor for parameters.

adeb1ba194ba24663bb41c954c0acbe12c494ced


handling Kubernetes Errors

The StatusError that is returned when a call from kubernetes doesn't succeed would be useful.
Currently its just being decoded https://github.com/obmarg/kazan/blob/master/lib/kazan/client/imp.ex#L160

Looking into this a tiny bit more, it seems like the StatusError contains a metaStatus, which is actually in the kube specs, and the kazan struct is here so it might be possible for kazan to parse one of these out on error...

If your curious, I’ve opened a ticket on the kubernetes api machinery to have these errors in the OpenAPI spec


Automatically upgrade old filters instead of requiring —force

Came across a case recently where someone hadn't used git-lfs for a while and suddenly got errors on git lfs install of the form:
filter.lfs.clean attribute should be "git-lfs clean --%f" but is "git-lfs clean %f"

This surprised me because I thought we had upgrade paths for our config, but turns out that was only for hooks. So rather than making them run git lfs install --force for something we already know about, I introduced the same concept as for hooks, a list of upgradeable values we used to use.
While I was at this I removed the attribute upgrade path for git-media, since it was vague and over-general really and I don't think it's necessary any more? I can put it back if there was a specific reason it's still there but looked like a relic to me.

Awesome, thanks for adding this!
The git-media stuff is way outdated. I don't think there's any need to find every filter we used during development :)

This looks awesome. Thanks!


Complete set of letters in the filter?

James raised the question whether we should have complete set of letter in the filter or we should only have letters for available countries?

I think we should have all letters as missing letters only confuse user.
a) Why is V missing? I need data for Vietnam
b) Aha there is not dataset for Vietnam

I think it should be obvious that the letter is missing because no countries match, e.g. an e-commerce website doesn't list product categories it doesn't offer in a dropdown, but I'm open to the possibility that this is not obvious.

I think there is precedence with filtering with letter that all letters of the alphabet are included (https://uxdesign.cc/making-lists-easy-e7c5d971d1f7). We should stick with that metal model. Making a letter non-clickable, should tell the user that there is no data and that we have not forgotten a letter

agree. Nicely described in #19

New development requirement #40 based on this discussion


[FEATURE] Visualization of API

Is your feature request related to a problem? Please describe
it would be nice to have a CLI option to just list the api without making any requests to any server.
why?

sometimes the swagger docs can be huge and really not human-readable
while writing hooks it is good to quickly get the paths from a schemathesis cli command
serves as a human readable overview of the api, this being an api tool
easy to follow the sequence of the api links without running actual requests

as all this openapi data is parsed already, good to have this visual on the data.
Describe the solution you'd like
good to have the links for each api followed and listed
ok to have the response codes listed
ok to have 'operation-id's of the api listed
something like this , maybe! or can as well omit the operationId and responses
GET /items operationId:getItems responses: [200,4XX,5XX]
POST /items operationId:createItem responses: [200,4XX,5XX]
    -> GET /items/{itemId}
GET /items/{itemId} operationId:getItem responses: [200,4XX,5XX]

in above example, post is linked to get
maybe some command at top level like st view instead of st run
st view openapidocs.yaml
no need to specify server and no need for hypothesis engine to run
Describe alternatives you've considered
currently i am using this workaround
st run -v -b http://dummyhost --dry-run my-swagger.yaml --hypothesis-phases=explicit
note: http://dummyhost here is literal and not a real server, just to hack the behavior
Additional context
dredd has a similar option
https://dredd.org/en/latest/usage-cli.html?highlight=--names#cmdoption-names
sample output of dredd openapi20.yaml http://dummyhost --names for your reference
info: items > /items > Get all items in the bucket list > 200 > application/json
skip: GET (200) /items?year=2024
skip: GET (401) /items?year=2024
skip: GET (404) /items?year=2024
skip: GET (500) /items?year=2024
info: items > /items > Add a new item to the list > 201 > application/json
skip: POST (201) /items
skip: POST (400) /items
skip: POST (401) /items
skip: POST (500) /items
info: items > /items/{itemId} > Retrieve a bucket list item > 200 > application/json
skip: GET (200) /items/415feabd
skip: GET (400) /items/415feabd
skip: GET (401) /items/415feabd
skip: GET (500) /items/415feabd

notes:
no openapi-links information
no operationId listed
has response codes, paths and descriptions too
it uses example values of query and path parameters in the output
dredd --version
dredd v14.1.0

Interesting idea! It also reminds me of pytest's --setup-only. I think st view would be the right direction, however, the output format requires some more work, especially for large schemas. I don't have the bandwidth for this right now, but I'll check how dredd & pytest do this kind of thing (and maybe will check some other tools too) in a few weeks.
P.S. I think you can omit -b when you use --dry-run.


Move ZIO Interop documentation into the microsite

Moving interop docs to the microsite as per #201

Something wrong with the build. Could you take a look please?

The build for 2.11.12 timed out after the tests passed. Not sure what happened there right now, but I'll be able to have a look tomorrow.

Codecov Report

Merging #203 into master will decrease coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   64.27%   64.02%   -0.26%
==========================================
  Files          26       26
  Lines        1170     1184      +14
  Branches      149      149
==========================================
+ Hits          752      758       +6
- Misses        418      426       +8

Impacted Files
Coverage Δ

core/shared/src/main/scala/scalaz/zio/Queue.scala
97.14% <0%> (-2.86%)
:arrow_down:

core/shared/src/main/scala/scalaz/zio/Retry.scala
31.86% <0%> (-0.72%)
:arrow_down:

...d/src/main/scala/scalaz/zio/interop/scalaz72.scala
90.9% <0%> (+0.43%)
:arrow_up:

...e/shared/src/main/scala/scalaz/zio/KleisliIO.scala
74.02% <0%> (+0.69%)
:arrow_up:

core/jvm/src/main/scala/scalaz/zio/RTS.scala
90.7% <0%> (+0.75%)
:arrow_up:

Continue to review full report at Codecov.

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

I re-triggered the build and it passes now. So merging.
Thanks a lot for your help!

@meras  Not a microsite expert here, but interop docs don't seem to be anywhere.

I can publish this later today. I have yet to implement automatic publishing, that is correct....


Fix/(curriculum): calorie counter step 94

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #53109

please do not make changes to non english files.
Your changes should only affect the english file.
Please review this, before a review can be done


added a white line around the favicon

The binary file is not visible, put an screen shot in order to see the changes please 😄

We also make it with a hook that depends of the nav theme. Anyway, thank u for the contribution! We appreciate it a lot! 😁


Delete deprecated mbtiles impl

Towards #1715.
@shobhitagarwal1612 PTAL?

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Libedit: update and adopt

Description of changes

Picking https://github.com/NixOS/nixpkgs/pull/278049
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] Fits CONTRIBUTING.md.

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

Phew!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3210

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3435


Add show_banner trait to control the banner display

Fixes https://github.com/jupyter/nbclassic/issues/242
Depends on https://github.com/jupyter/notebook_shim/pull/29 Once a new notebook_shim release is cut, we need o bump that dependency version in this PR.

A new notebook_shim has been released, this is now ready for review.

Thank you @echarles


Rebase on Kueue v0.8.0 - Sync aug22 2024

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

/retest

/retest-required


Handling different input and output number of channels in AdaptiveConv

In this line shouldn't we pass x to gap, otherwise if our input and output number of channels is different this will throw error, because x1 will have different number of channels than x and fc1 layers maps from number of input channels to number of output channels.
gap = self.gap(x1)

Please let me know  if I am missing something.

Hi! Many thanks for your advice, I didn't notice this mistake, I have fix this bug in new commit.


Would be nice to have more documentation on buttons and modals

As title says - Unless I am completely missing it I can find barely to no information on how to use buttons or modals. A few explanations and/or examples is all I would ask. I have figured out a few things by looking through source code but I don't think thats really a good general solution

+1, agree! It works great, if you know how to use it.

Well, based on issue activity etc. I honestly thought that this project would no longer be used by anyone. Thus I've discarded my plans on writing docs. However if you guys still have demand I could start working on docs again


The Japanese version of 'Front-End-Design-Checklist

Hi @thedaviddias,
I'd like to translate 'Front-End-Design-Checklist' into Japanese. Is that OK with you?
The Japanese version link address is Front-End-Design-Checklist.
Thanks!

@53able I'm sorry, I forgot to answer you. Of course, feel free to translate the FEDC into Japanese 😉


SICK MultiScan 100 LiDAR

I have a problem getting all the 16 layers from the SICK MultiScan 100 LiDAR sensor. I used the default sick_multiscan.launch file and change the parameter
But still i see the callback triggered with msg->width : 2700 and msg->height : 1. I expect the height to be 16 but i get always 1. Any idea ?

This is correct and intended. The scan data of different layers are exported in cartesian coordinates. All scan data in a point cloud message are flattened and concatenated into a data array of height 1. See https://github.com/SICKAG/sick_scan_xd/blob/master/doc/sick_scan_api/sick_scan_api.md#usage-example for details about the memory layout of the pointcloud data.
The scan data of a Multiscan lidar are multiplexed in segments. Therefore, two different kinds of point clouds are published:

Segment point clouds contain the scan data of a single segment (typically 30 or 60 degree segments)
Full frame point clouds, which concatenate all segments within a full frame.

I.e. the full frame point cloud contains all the scan data of a 360 degree scan. See https://github.com/SICKAG/sick_scan_xd/blob/master/doc/sick_scan_segment_xd.md#pointcloud-memory-layout for details of Multiscan pointcloud messages.
There are 2 possible ways to get the layer resp. the elevation of a point, if required:

The polar point cloud publishes the fields range, azimuth, elevation and intensity. Each layer corresponds to an elevation angle.
Elevation and azimuth can be calculated from the cartesian point cloud. See https://github.com/SICKAG/sick_scan_xd/issues/175 for further information.


Support multi-page app by marking html-webpack-plugin entries as lazy

Currently, this plugin only deals with dynamic imports, but in real life, a common cause of slow compilation is huge multi-page apps (an example app: https://github.com/GodStream/multi-vue-cli3.0-bug).
By adding html-webpack-plguin entries to the lazyModules list, there will be great improvement to those apps. And this feature seems quite in line with this plugin's goal.

@liximomo
It starts with error after upgrading.
ERROR in   Error: webpack-internal:///./node_modules/lazy-compile-webpack-plugin/lib/loaders/api.js:22
  var originTitle = document.title;
                    ^
  ReferenceError: document is not defined

  - api.js:22 eval
    [.]/[lazy-compile-webpack-plugin]/lib/loaders/api.js:22:19

  - index.pug:108 Object../node_modules/lazy-compile-webpack-plugin/lib/loaders/api.js
    /Users/JounQin/Workspaces/GitHub/pkgr/packages/webpack/index.pug:108:1

  - index.pug:21 __webpack_require__
    /Users/JounQin/Workspaces/GitHub/pkgr/packages/webpack/index.pug:21:30

  - index.pug:3 eval
    [.]/[html-webpack-plugin]/lib/loader.js!./packages/webpack/index.pug:3:11

  - index.pug:97 Object../node_modules/html-webpack-plugin/lib/loader.js!./packages/webpack/index.pug
    /Users/JounQin/Workspaces/GitHub/pkgr/packages/webpack/index.pug:97:1

  - index.pug:21 __webpack_require__
    /Users/JounQin/Workspaces/GitHub/pkgr/packages/webpack/index.pug:21:30

  - index.pug:85
    /Users/JounQin/Workspaces/GitHub/pkgr/packages/webpack/index.pug:85:18

  - index.pug:88
    /Users/JounQin/Workspaces/GitHub/pkgr/packages/webpack/index.pug:88:10

  - index.js:247 HtmlWebpackPlugin.evaluateCompilationResult
    [pkgr]/[html-webpack-plugin]/index.js:247:28

@JounQin Please go to #5


Add "license" and "licenses" to package.json

Specifying your license in package.json makes it a lot easier for your users to find and comply with GNU Affero licenses.
Also giving licenses (an array of objects with type and url keys) helps  users whose legal departments request both the license name and the text. See this example from async:
  "licenses": [{
    "type": "MIT",
    "url": "https://github.com/caolan/async/raw/master/LICENSE"
  }]

Though licenses isn't specified in the npm package.json spec, it is supported by the NPM site, as you'll see if you check async's NPM page.
Thanks!

The license field is actually in the package spec: https://docs.npmjs.com/files/package.json#license

If interested, I can create a PR for this.

I have seen there is little progress on this project over the past few years.  My project, the net-snmp project, depends on this project, so I have cloned it and migrated (i.e. copied) all the outstanding issues and pull requests.
I will be releasing a new version of my version of this project under the asn1-ber namespace on npmjs.org by the end of the week which should address this particular item (I've already pushed the current code to this namespace to reserve it).
I am now tracking all changes under my own copy at asn1-ber.  Note that this is not a "fork" but a clone and fresh commit to a new repository.  I have of course retained all copyrights and attributed back the original author.
There are quite a few projects dependant on this project, and I want to make sure there is momentum on the issues and feature requests reported.

This issue has been addressed in my version of this project which has now been published to npm:
https://www.npmjs.com/package/asn1-ber


Replace groupAttr recursively

Description
This PR is an approach of replacing groupAttrs recursively. This allows to use constructs in groups like some_object.groupAttr1.sub_component or even more complex once like some_object.groupAttr1.intermediate_stuff.groupAttr2.sub_component.
This addresses request like the following:

https://forum.iobroker.net/topic/15880/nutzung-von-attributen-in-gruppierungen
https://forum.iobroker.net/topic/31402/vis-groupattr-einen-ordner-von-objekten-übergeben

Imo this would be a great enhancement for the use of groups when generating a visualisation.
Open Points
Duplicated code
The function replaceGroupAttr is duplicated in the files www/js/vis.js and www/js/visUtils.js. I don't know where you would put such pieces of code to share. Please let me know and I will fix it.
Replaced groupAttr not evaluated on first loading of vis
I made a test on a local setup where I used this approach to display the position of a window shutter. There is a problem when the vis page is first loaded, it looks like the position is not evaluated. When the position of the window shutter is changed the data gets interpreted correctly.
It's also possible that this has nothing to do with the changes from this PR (but I doubt it, which would mean the PR is not perfect).
The edit vis page is always showing the correct things (position is evaluated instantly).
Maybe some experts here have an idea what the problem of this could be.
Let me know if I shall provide more information around this test.
New to contributing to ioBroker.vis
I am new to contributing to ioBroker.vis so please guide me where I am not following your rules.

@GermanBluefox any chance that you could take a look at this any give some inputs on whats needed to continue?

Thank you!

Thanks for merging!


Add Map.of and OrderedMap.of

So it is consistent with other collections.

Could you explain how you expect these static methods to act via example code?
The same named static method on List or Stack accepts a variable list of arguments and builds a collection like List.of(1, 2, 3), as a convenience to the slightly noisier List([1, 2, 3]).

Well, instead of
Map([["a", 3], ["b", 4], ["c", 5]])

you would be able to write
Map.of(["a", 3], ["b", 4], ["c", 5])

Just for convenience, it would be nice if every collection has of and it behaved like this:
Coll.of(...val) => Coll(val);

Just curious why you would prefer typing Coll.of(...val) instead of Coll(val)?

We can do significantly more optimizations with Coll(val), including structural sharing, which can't be done with Coll.of(...val) so I'm nervous about encouraging that sort of use.

Of course, I would not type Coll.of(...val). That was just implementation code:)
The same way I would prefer to type List.of(1, 2, 3) instead of List([1, 2, 3]), I would also prefer typing Map.of(['a', 1], ['b', 2], ['c', 3]) instead of Map([['a', 1], ['b', 2], ['c', 3]]). I don't see much difference between those two cases and I just want them to be consistent.

I'm actually leaning towards removing the of method in a future version. It was added for parity with Array.of but notably Array.of serves a purpose that isn't particularly relevant for List.

It looks like the conclusion here is to leave as is, so I am going to close this issue. Thanks for bringing this up and sparking some discussion!


docs: fix surrounding text after changes in #1477

Just found out that some text mentions that reducer uses slice to create a new array. Once we changed it in #1477, we should change surrounding text as well.
Does new text look good for you?
I really should more carefully read the text before sending that PR. Sorry about that.

That looks great, thank you!


Added motor control support to foxy

I ported this from melodic to foxy. Tested on Create2

Apologies, I overlooked this PR. I've already gone ahead and ported this to Foxy (https://github.com/AutonomyLab/create_robot/pull/97).


Fix WebSocket duplication in handleServerMessagesGuard

This is part 2 of 3 in an effort to fix wwWallet/wallet-ecosystem#31; part 1 is #133.
Before, a new WebSocket was created on every render of handleServerMessagesGuard, and event listeners bound to all of them. This results in lots of sockets being opened - often about 8 on initial page load, and then another for each page navigation.
This changes the hook to keep the WebSocket instance in a React ref, so that the same socket instance is used consistently for the lifetime of the wrapped component. The startup procedure is also tweaked to unbind the startup message handlers and replace them with the long-term message handler.
This depends on the changes in #133 because the initialization logic now only runs once, and therefore needs to be able to react when the appToken state changes. This is much easier to do using the useSessionStorage hook than using cookies.

Oh, and this is also the change that automatically makes you reauthenticate if you open a new tab, because the appToken won't exist in the new tab session storage, and therefore <PrivateRoute> now redirects you to the login page to reauthenticate. :slightly_smiling_face:


Handle client FIN as request abort

#1139
This part of the change is mostly to avoid test failures in HttpsTests.EmptyRequestLoggedAsInformation. After the change to handle FIN as abort, this test started failing consistently because the OnConnectionAsync continuation would run after the abort had taken place, the connection had ended and the server had been disposed - at that point ReadInputAsync would try to lease a block from an already disposed memory pool, triggering the assert and failing the entire test run.
@halter73 @Tratcher @pakrym

@halter73 Added code to reset the RequestAborted token before the last write in each response.

Updated.

Ping.

We might want to perf test this because we are adding another check that is per-write.

No impact on plaintext benchmark:

dev
PR

1173992.5 RPS
1170175.3

Will work on microbenchmark.

Just fixed a race in ResponseTests.ResponseBodyNotWrittenOnHeadResponseAndLoggedOnlyOnce.

Write() microbenchmarks
dev

Method
Mean
StdDev
Gen 0
Allocated

Write
39.8272 ns
0.4220 ns
-
0 B

WriteChunked
40.1696 ns
0.5210 ns
-
0 B

WriteAsync
76.2597 ns
0.7999 ns
0.0068
63 B

WriteAsyncChunked
76.5476 ns
0.8251 ns
0.0070
63 B

WriteAsyncAwaited
110.1888 ns
0.3927 ns
0.0037
63 B

WriteAsyncAwaitedChunked
111.9686 ns
1.3847 ns
0.0033
63 B

ProduceEnd
35.1201 ns
0.4583 ns
0.0086
63 B

ProduceEndChunked
35.3245 ns
0.3355 ns
0.0084
63 B

This change

Method
Mean
StdDev
Gen 0
Allocated

Write
57.9786 ns
0.1677 ns
-
0 B

WriteChunked
58.1488 ns
0.1471 ns
-
0 B

WriteAsync
91.0255 ns
0.2082 ns
0.0033
63 B

WriteAsyncChunked
93.8709 ns
1.3749 ns
0.0034
63 B

WriteAsyncAwaited
126.9702 ns
0.8035 ns
0.0034
63 B

WriteAsyncAwaitedChunked
129.1539 ns
2.0402 ns
0.0040
63 B

ProduceEnd
36.8412 ns
1.3556 ns
0.0085
63 B

ProduceEndChunked
35.2432 ns
0.2778 ns
0.0086
63 B

:shipit: *crosses fingers*


Source distributions

I'd like to bump support for hosted distribution archives in case a specific environment (e.g. Mac M1 python3.10) needs pip to build it's own wheel.

It’s a bug for missing macOS Python 3.10 wheels and shall be fixed in the next release. https://github.com/ringsaturn/tzfpy/compare/v0.15.3...main
Wheels and tar.gz should not be included in repo but built in GitHub Actions instead.

I may release a new version within a week and will close this PR after that.

Wheels and tar.gz should not be included in repo but built in GitHub Actions instead.

That makes sense, thanks for clarifying!

@miki5799 I have released v0.15.4 and you can install Python 3.10 M1 Mac wheel from PyPI.
https://pypi.org/project/tzfpy/0.15.4/#files

Thank you very much! Installation was successful.

@ringsaturn Could you host an Apple Silicon wheel for v0.15.3, since a dependency needs that specific version? Thanks in advance

@ringsaturn Could you host an Apple Silicon wheel for v0.15.3, since a dependency needs that specific version? Thanks in advance

Sorry to tell you that I don't have to do this and wheel files are built in GitHub Actions not from my local computer. I suggest you bump you tzfpy version to fix this since there are not breaking changes between v0.15.3 to current latest v0.15.5.
Or, you can try install it via tzfpy's conda distribution, because in conda tzfpy use different build steps so macOS Arm wheels are not missing in v0.15.3


[Issue Nours312/ng-cordova-oauth#1] update oauth_callback

This will not work because the Apache Cordova InAppBrowser needs to know when to close.  Switching to oob will leave too many loose ends.

Yes, I know... my push is an error.. sorry for that !..
But when I use http://localhost/calback, twitter-api said :
Desktop applications only support the oauth_callback value 'oob'
When I use oob the response is a code ..
I'll try to find a better solution.

Alright, let me know.
Oauth with hybrid development is always a pain, but I'll take any help I can get :-)


Allow for custom values when using touch

First of all, thanks for creating this gem! Works really great for me :+1:
I do have a small issue when using the touch: true option though. One rails application that I am maintaining store timestamps is integers, i.e. unix timestamps. Don't ask me why, it was like that when I got there :sweat_smile:
This causes some issues when trying to use touch to update timestamps through counter culture. We have a model similar to product, but created_at and updated_at are not actually date times but decimals.
class Product
  counter_culture :category, touch: true
end

This causes an error when trying to update counters:
Mysql2::Error: Incorrect decimal value: '2020-11-02 15:55:11' for column 'updated_at' at row 1

As a possible solution, I'd propose to add another option touch_with that allows one to specify the formatting of the used timestamp:
class Product
  counter_culture :category, touch: true, touch_with:  touch_with: current_time_from_proper_timezone.to_i
end

Now, I realise that I am probably the only person in the whole wide world with this problem, and looking at previous PRs regarding this functionality I totally understand if something like this is outside the scope of this project.
What would still help in this regard is a slight restructuring of counter.rb to allow for easier monkey patching, e.g.
...
....
if touch
   update_timestamps(obj)
end

private

update_timestamps(obj)
  current_time = obj.send(:current_time_from_proper_timezone)
  timestamp_columns = obj.send(:timestamp_attributes_for_update_in_model)
  if touch != true
     # starting in Rails 6 this is frozen
     timestamp_columns = timestamp_columns.dup
     timestamp_columns << touch
  end
  timestamp_columns.each do |timestamp_column|
    updates << "#{timestamp_column} = '#{current_time.to_formatted_s(:db)}'"
  end
end

Please let me know what you think. I'd be happy to open a PR in any instance :slightly_smiling_face:

Haha, well, that's fun. Gotta love old applications.
It seems to me that this is pretty niche. I think maybe the minimal change would be to add a method here, something like _counter_culture_current_time_for_touch that defaults to current_time.to_formatted_s(:db) and then you could simply override that in the relevant class? Or is this something that you'd need to have column by column, not just class by class?

Thanks for getting back to me!

_counter_culture_current_time_for_touch that defaults to current_time.to_formatted_s(:db) and then you could simply override that in the relevant class?

You are right, that definitely works and is an even smaller change than the one I proposed.

Or is this something that you'd need to have column by column, not just class by class?

No, luckily this is only needed on class by class :grin:

Closing for inactivity.


Kazakh(kk)

added translation to Kazakh(kk)

Please add this as fast as you can 👍

sorry be late

Thank you. Can you please upload it to pub dev?
чт, 9 янв. 2020 г. в 20:59, 刘彦博<EMAIL_ADDRESS>
sorry be late
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Realank/flutter_datetime_picker/pull/97?email_source=notifications&email_token=ALG46G7AG6XQFLDE5JVN6BLQ443TVA5CNFSM4KEDD5WKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIQSV5A#issuecomment-572599028,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALG46G4WG7CWDNU3CFOEX5TQ443TVANCNFSM4KEDD5WA
.


Uninitialized constant

In a user activerecord model (done this way so I can have additional checks):
def check_email
      email_address = ::EmailAddress.new(email)
      if false == email_address.valid?(email)
        errors.add(:email, 'invalid email address provided')
      end
end

Error: Uninitialized constant EmailAddress
If I require 'email_address' at the top of the file my server crashes on start up with no error message (obviously the gem is in my Gemfile). Any idea what could be going on?
lloyd@xps13:~/Sites/api$ ruby -v
ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux]
lloyd@xps13:~/Sites/api$ rails -v
Rails 4.2.10

Also runs file in development. I can start server with RAILS_ENV set to production or staging just fine. Only on staging server is this causing an issue.
Server setup:
$:~$ ./bin/rails --version
Rails 4.2.10
$:~/$ ruby --version
ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux]

If I open rails console on the staging server all is well:
$:~$ RAILS_ENV=staging ./bin/rails c
I, [2018-07-25T16:49:07.934395 #6551]  INFO -- sentry: ** [Raven] Raven 2.7.3 ready to catch errors
Loading staging environment (Rails 4.2.10)
irb(main):001:0> require 'email_address'
=> false
irb(main):004:0> e =<EMAIL_ADDRESS>=> #<EmailAddress::Address:0x2aceeb927ce8<EMAIL_ADDRESS>irb(main):005:0> e.valid?
=> false
irb(main):007:0> e.error
=> "Domain name not registered"

$:~/olio/current$ grep email_address Gemfile.lock
    email_address (0.1.11)
  email_address

Hi,
I'm not sure this is an issue with the gem.  Can you clarify or paste a failure case?
Also runs file (fine?) in development.
I can start server with RAILS_ENV set to production or staging just fine.
Only on staging server is this causing an issue.
If I open rails console on the staging server all is well

In rails, you shouldn't have to require the gem in your file, it will load automatically. You would only need to require ruby standard libraries not already loaded by rails, for example:
require 'fileutils'

In your Gemfile, does the "gem 'email_address" occur inside of a "group" block where it won't be loaded for certain environments? Also, does "bundle install" run cleanly?
Is there a startup error or other useful information logged in ./log/staging.log ?

Apologies, a couple of typos haven't helped.
In my gemfile the gem isn't in a block (is available in all environments). Also bundle install runs cleanly.
On my development machine: all works ok.
On staging: EmailAddress not defined (hence me trying require, I'm aware it shouldn't be needed). However, if I open the ruby console on the staging server EmailAddress is available.
Nothing useful in the log, I just see the server constantly restarting when including require. If I don't have the require then on staging I get not defined error.
Haven't seen this issue with other gems. I thought it was a conflict with validates_email_format_of but that's not the case.

I knew something weird was going on! I restarted the server this morning and its fixed everything. Thanks for a great gem :+1:


not working asp.net core 3.1

hello,
Is there an example running on net.core version 3.1?

I guess there is no example running on asp.net core


Migration tool for testnet return 500 error

https://migration.web.test-helium.com/migrate/6pJypiTMEHCke91hrw3Dh36b69PFEwfMzAXZRek5PsTg?limit=1000&offset=0
returns
{
"statusCode": 500,
"code": "28000",
"error": "Internal Server Error",
"message": "PAM authentication failed for user \"web\""
}

This appears to no longer be happening.