"Publish-NAVApp : Value cannot be null." Issue regarding translation files.

Hi,
We encounter an issue working with translation files while developing on the two platforms. We are using the same source code and same package for our extension to publish to NAV 2018 and Business Central.
All works fine, same package publication on each Platform, until we try to use translation files.
Activating "TranslationFile" feature in app.json generate xxxxx.g.xlf global language file and still works fine.
As soon as we create a first translated file, it still works fine on business central but Trying to publish same package on NAV 2018 results in this error message : "Publish-NAVApp : Value cannot be null."
Same behavior from F5 publish within VS Code or PowerShell Publish-NavApp.
We are using 0.16.14305 AL Language version and symbols from BC are System_12.0.12928.0, Application_12.1.22069.0 and for NAV 2018 (FR) System_11.0.12925.0 and Application_11.0.21836.0.
I tried several things, recreate files, and try to package with older AL version but without any success.
If I try to package using NAV 2018 corresponding AL language (al-0.12.17720.vsix), it then works with NAV 2018 but xxxx.g.xlf global language file is then recreated with totally different items ID's and we have to maintain twice same translations in two different branches.
Is there any way to maintain only one source code and one package and be compliant with the two Platform (of course using only common compatibles functionalities)?
It works fine except regarding translations.
Thank you for your help.

I believe I have identified the issue. We have fixed a bug with generating translation files where we were missing some information but changing the id generation would break backwards compatibility with already generated translation files for Nav 2018.
I will add support for consuming translation files on Nav 2018 which have been generated in Business Central.
@AcererakTheLich could you please provide us with AL code which was used to generate problematic translation files? Otherwise I wont be sure I guessed what your issue is.

Maybe will be useful this scenario, leading to the same error:

Switch on app.json "features": ["TranslationFile"]
Run build task (package)
Translation file xxxxx.g.xlf is generated
Publish app -> failed with “Value cannot be null.”
Translate file to xxxxxxx.cs-CZ.xlf
Publish app -> failed with “Value cannot be null.”
Switch off app.json "features": []
Publish app -> failed with “Value cannot be null.”
Delete file xxxxx.g.xlf
Publish app -> OK
Continue with work, change any object
Publish app -> OK
Publish app -> OK
Does not matter build, version, environment, debugger.
Problem is only with generated global translation file.
Tis file should be deleted or ignored when publish.

@IvoMol I don't think this is the same scenario. As I understand, you are having an issue that the same compiler which generated the xlf file cannot later package it in the extension.
Could you please try and isolate which AL code is causing the incorrect XLF file to be generated? Right now I do not know what is the exact issue. Can you build the extension using ctrl+shift+B with success?
Are you using the AL extension which came with your platform (don't use Business Central AL extension for Nav2018 development)?
@AcererakTheLich I'm afraid supporting XLF files generated by Business Central to work with Nav 2018 is going to require a bit more work. I don't have an ETA right now

Using AL compiler 0.15.18771:
Using AL compiler 0.12.17091:
you are not able use "features": ["TranslationFile"]

Using AL compiler 0.15.18771:
Problem is generated using any Label variable:
pageextension 50100 CustomerListExt extends "Customer List"
{
trigger OnOpenPage();
var
AppPublishedMsg: Label 'App published: Hello world';
begin
Message(AppPublishedMsg);
end;
}
Translation file is created OK, I can build app many times repeatedly, but not publish (until remove XXXXXX.g.xlf).
Using AL compiler 0.12.17091:
Cannot use "features": ["TranslationFile"] at all.

@IvoMol  when using the 0.15.18771 version of the compiler, which version of the platform are you using?
What is the version number and is it Business Central or Nav2018?

Platform is NAV 2018 (on-premises), build 11.0.22292.
It is probably not an offically supported cobination, but if I want work with XLF translations outside AL files (and not with multilanguage inside AL files), I have no choice.
All works absolutely fine, but only before publishing I must manually delete XXXXXX.g.xlf file. If I forget about it, error "Value cannot be null" occurs during publishing. Delete or ignore this file during publishing and all will be perfect :-)

@IvoMol  -  you should use AL compiler 0.12.XXX with NAV 2018. You can still use the TranslationFile feature, it's just that the app.json syntax is slightly different. Please try:
"features" : "TranslationFile"

Solved for me: AL compiler 0.12.17091, platform NAV 2018 (11.0.22292), "features": "Translation File".
Translation file generated, translated to cs-CZ, builded repeatedly, published repeatedly.
Error "Value cannot be null" does not occur.
Thank you Stanislaw!


Deck instance does not hava a 'delete' method

import {Deck, HexagonLayer, MapController} from<EMAIL_ADDRESS> //version 5.2.2
const deckgl = new Deck(props);

deckgl.delete(); //  throw an error :"TypeError: deckgl.delete is not a function"

There is not a 'delete' method in Deck's instance,
but in the doc, it exists.

Good catch. The method is called finalize.
Filing this as a documentation bug, thanks for reporting!


Problems using Envoy Gateway with GKE "failed to get API group resources"

Description:
Trying to use Envoy Gateway on a GKE cluster with the Gateway API CRDs already installed (v1beta1) results in:
failed to create provider Kubernetes: failted to create gatewayapi controller: failed to get API group resources: unable to retrieve the complete list of server APIs: gateway.networking.k8s.io/v1: the server could not find the requested resource

I assumed that Envoy's implementation of Gateway API would not have a hard requirement on v1 but that seems incorrect.
Trying to install the v1 CRDs directly get's overridden by GKE:
kubectl apply -k “https://github.com/kubernetes-sigs/gateway-api//config/crd/experimental?ref=v1.0.0”

Checking to see what versions are installed on GKE:
kubectl get crd -o=jsonpath='{range .items[*]}{.metadata.name}{"\n"}{range .spec.versions[*]}{.name}{": storage="}{.storage}{"\n"}{end}{end}'

Output:
gatewayclasses.gateway.networking.k8s.io
v1alpha2: storage=false
v1beta1: storage=true
gateways.gateway.networking.k8s.io
v1alpha2: storage=false
v1beta1: storage=true

It appears that on GKE, you cannot install v1 if using their managed Gateway API "standard" channel.
Compared to a non-GKE, working, cluster output:
gatewayclasses.gateway.networking.k8s.io
v1: storage=false
v1beta1: storage=true
gateways.gateway.networking.k8s.io
v1: storage=false
v1beta1: storage=true

Problem 1: The helm chart bundles CRDs of upstream Gateway API & Envoy CRDs, meaning there's no easy way to only install one or the other. Where this would be handy, is clusters that already have Gateway API (the OSS spec) installed and they are looking at adding different gateways.
Problem 2: It appears that Envoy Gateway has a hard requirement for the latest Gateway API version being at v1 and having v1beta1 is not enough, regardless of version storage.
Problem 3: If someone was on GKE, running with the managed Gateway API CRDs, there is no easy path to leveraging Envoy Gateway. While this could be an issue for Google / GKE to solve, if Envoy Gateway API worked with v1beta1 we would have an easier path to migrate.
Problem 4: It appears Envoy Gateway installs the experimental APIs which also could be different from someone's cluster.
Gateway API being out of tree was supposed to allow folks to "BYOcrds" and then enable various Gateway API implementations. The way Envoy Gateway is approaching it today removes a lot of that promise / value.
Solution:

Make the installation of Gateway API CRDs "out of scope" for the helm charts and provide the steps as a pre-req
Drop the hard requirement to have v1 unless there's a technical reason in doing so

Potentially support/warn v1beta1 (and any other Gateway API CRDs) in logs so consumers can leverage Envoy Gateway and migrate to latest supported version

Drop the (hard/soft?) requirement for experimental Gateway APIs and let the consumer choose

I'm happy to strip out the OSS Gateway API CRDs from the helm charts and revise the README.md but would need additional support / nudging for how the code changes could/would be handled. I would also love to hear folk's thoughts on the matter! Thanks!

If anyone else runs into this, I've temporarily worked around the issue using version v0.5.0:
helm install eg oci://docker.io/envoyproxy/gateway-helm --version v0.5.0 -n envoy-gateway-system --create-namespace

I recall I saw this somewhere, but cann't recall.

@jimangel Thanks for reporting this issue.
If I understand correctly, this actually requires Envoy Gateway to supports different versions of Gateway API. While this may be technically feasible, it would need massive architectural modification,  and I am not entirely certain whether it would be a prudent decision. @envoyproxy/envoy-maintainers

@zhaohuabing has shared some workarounds / alternate solutions, closing this issue


⚠ Bofrost.be has degraded performance

In 85ee14b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1058 ms

Resolved: Bofrost.be performance has improved in acaac8a after 34 minutes.


Price model is shown same window when broker or reseller operate

[version]
V16.0.4
[Frequency]
Always
[How to reproduce]

Login as broker or reseller
Select "View service" in Marketable service
Select service using external billing
Click Display in Price model for your service section

[Expected result]

Show the price model in a separate window

[Observed result]

Show the price model in a same window

@servicecatalog/oscmdev2

The issue has been reproduced.
The respective function in service/serviceData.xhtml should be fixed.
The button implementation is different from subscription/view.xhtml. It's implemented with a h:commandButton, which means a HTTP POST to the same view. It should be done with a GET and directed with target for opening in a new window. Use <h:commandLink .... target="_blank"/>.
Why is the include "/priceModel/externalServicePriceModel.xhtml" not reused here? Please check.

[Build] 2016/08/09 verified


Question-Is it possible to upload attachments to the test results screen in TFS?

When one of our tests fails it dumps some information to an external log file that would be really great to attach to the test results?  Is this possible to do?

Moved this issue to https://github.com/Microsoft/vsts-tasks/issues/3679

@bbusacker replied over here Microsoft/vsts-tasks#3679


[pcl[cuda]] Fix build failure

Fixes #33527
Add  VTK as a dependency to CUDA feature. and use upstream patch to fix error:
LINK : warning LNK4098: defaultlib 'LIBCMT' conflicts with use of other libs; use /NODEFAULTLIB:library
screenshot_manager.cpp.obj : error LNK2019: unresolved external symbol "void __cdecl pcl::io::saveRgbPNGFile(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,unsigned char const *,int,int)" (?saveRgbPNGFile@io@pcl@@YAXAEBV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@PEBEHH@Z) referenced in function "public: void __cdecl pcl::kinfuLS::ScreenshotManager::saveImage(class Eigen::Transform<float,3,2,0> const &,struct pcl::gpu::PtrStepSz<struct pcl::gpu::kinfuLS::PixelRGB const >)" (?saveImage@ScreenshotManager@kinfuLS@pcl@@QEAAXAEBV?$Transform@M$02$01$0A@@Eigen@@U?$PtrStepSz@$$CBUPixelRGB@kinfuLS@gpu@pcl@@@gpu@3@@Z)

bin\pcl_gpu_kinfu_large_scaled.dll : fatal error LNK1120: 1 unresolved externals

The upstream PR: https://github.com/PointCloudLibrary/pcl/pull/5947
All features are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static

[x] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

/azp run


Add SkyMart shop

required by https://github.com/NotEnoughUpdates/NotEnoughUpdates/pull/756

What is the reason for isWorthless and display? Cant you get the display by looking for the normal file in the repo?

isWorthless: I don't know it's the appropriate name (it wasn't commented before), but it references items in SkyMart, that are rather useless, such as basic hoes that give you +5 farming fortune (and probably no one will buy them off ah), or Builder's Ruler, where the same thing applies
display: in SkyMart shops there are books such as Sunder I. Showing to the player, that someone will get 5k coins for selling Enchanted Book is a pretty unhelpful tip, looking at various books in the shop. At least in the editor, the display name is just §fEnchanted Book which showing to the player wouldn't be helpful. Maybe there is a way to get the book enchantment name as a display name, while getting normal items display names with the same way, but I didn't really work much with the repo, so idk.

What is the reason for isWorthless and display? Cant you get the display by looking for the normal file in the repo?

isWorthless: I don't know it's the appropriate name (it wasn't commented before), but it references items in SkyMart, that are rather useless, such as basic hoes that give you +5 farming fortune (and probably no one will buy them off ah), or Builder's Ruler, where the same thing applies
display: in SkyMart shops there are books such as Sunder I. Showing to the player, that someone will get 5k coins for selling Enchanted Book is a pretty unhelpful tip, looking at various books in the shop. At least in the editor, the display name is just §fEnchanted Book which showing to the player wouldn't be helpful. Maybe there is a way to get the book enchantment name as a display name, while getting normal items display names with the same way, but I didn't really work much with the repo, so idk.

I dont think the isWorthless field is good to put in the repo as it is not really our decisions, its just another thing we have to update if hypixel decides to change smth on these items.

What is the reason for isWorthless and display? Cant you get the display by looking for the normal file in the repo?

isWorthless: I don't know it's the appropriate name (it wasn't commented before), but it references items in SkyMart, that are rather useless, such as basic hoes that give you +5 farming fortune (and probably no one will buy them off ah), or Builder's Ruler, where the same thing applies
display: in SkyMart shops there are books such as Sunder I. Showing to the player, that someone will get 5k coins for selling Enchanted Book is a pretty unhelpful tip, looking at various books in the shop. At least in the editor, the display name is just §fEnchanted Book which showing to the player wouldn't be helpful. Maybe there is a way to get the book enchantment name as a display name, while getting normal items display names with the same way, but I didn't really work much with the repo, so idk.

I dont think the isWorthless field is good to put in the repo as it is not really our decision if an item is worth or not, its just another thing we have to update if hypixel decides to change smth on these items.

what would you think about changing it to isTool or something like this? it would filter out all items that are isWorthless either way cuz the list of "worthless" items consists of hoes, axes, cleaning scythes and builder's rules, which all of them can be classified as a tool?
or maybe another way: adding a type to each item, and classifiying them as mentioned before as tool, skin, equipment, enchant ,fuel (for the biofuel one), reforge

What is the reason for isWorthless and display? Cant you get the display by looking for the normal file in the repo?

isWorthless: I don't know it's the appropriate name (it wasn't commented before), but it references items in SkyMart, that are rather useless, such as basic hoes that give you +5 farming fortune (and probably no one will buy them off ah), or Builder's Ruler, where the same thing applies
display: in SkyMart shops there are books such as Sunder I. Showing to the player, that someone will get 5k coins for selling Enchanted Book is a pretty unhelpful tip, looking at various books in the shop. At least in the editor, the display name is just §fEnchanted Book which showing to the player wouldn't be helpful. Maybe there is a way to get the book enchantment name as a display name, while getting normal items display names with the same way, but I didn't really work much with the repo, so idk.

I dont think the isWorthless field is good to put in the repo as it is not really our decision if an item is worth or not, its just another thing we have to update if hypixel decides to change smth on these items.

what would you think about changing it to isTool or something like this? it would filter out all items that are isWorthless either way cuz the list of "worthless" items consists of hoes, axes, cleaning scythes and builder's rules, which all of them can be classified as a tool? or maybe another way: adding a type to each item, and classifiying them as mentioned before as tool, skin, equipment, enchant ,fuel (for the biofuel one), reforge

I would just make option to filter items that are under a certain price per copper, then the user has the option and we are out of that.

At least in the editor, the display name is just §fEnchanted Book which showing to the player wouldn't be helpful. Maybe there is a way to get the book enchantment name as a display name, while getting normal items display names with the same way, but I didn't really work much with the repo, so idk.

It seems you have already fixed the other problems but if Jani still doesn't want the display field in the repo you could look at how SkyHanni gets the name of an enchanted book through using internal name. The function is in this file
https://github.com/hannibal002/SkyHanni/blob/beta/src/main/java/at/hannibal2/skyhanni/utils/ItemUtils.kt


Error

File "C:\Users*\Desktop\PySilon-malware-main\source_prepared.spec", line 24, in
exe = EXE(
^^^^
File "C:\Users*\Desktop\PySilon-malware-main\pysilon\Lib\site-packages\PyInstaller\building\api.py", line 584, in init
self.postinit()
File "C:\Users*\Desktop\PySilon-malware-main\pysilon\Lib\site-packages\PyInstaller\building\datastruct.py", line 184, in postinit
self.assemble()
File "C:\Users*\Desktop\PySilon-malware-main\pysilon\Lib\site-packages\PyInstaller\building\api.py", line 824, in assemble
winutils.set_exe_build_timestamp(build_name, build_timestamp)
File "C:\Users*\Desktop\PySilon-malware-main\pysilon\Lib\site-packages\PyInstaller\utils\win32\winutils.py", line 157, in set_exe_build_timestamp
with pefile.PE(exe_path, fast_load=True) as pe:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users*\Desktop\PySilon-malware-main\pysilon\Lib\site-packages\pefile.py", line 2895, in init
self.parse(name, data, fast_load)
File "C:\Users*\Desktop\PySilon-malware-main\pysilon\Lib\site-packages\pefile.py", line 2987, in parse
raise Exception(
Exception: Unable to access file 'C:\Users*\Desktop\PySilon-malware-main\dist\source_prepared.exe.notanexecutable': [Errno 22] Invalid argument: 'C:\Users\***\Desktop\PySilon-malware-main\dist\source_prepared.exe.notanexecutable'

Yeah I got something similar to that too when trying to open the source

what is the name of the executable file

Hey! I am really sorry for my recent absence. I will look forward to fix the issues in the following week.

Do you have any errors while compiling? Also, what OS are you on and which python version do you use? If you have steps to reproduce this errs, screenshots of the console errors or any other information that would be really useful for us to help you resolve the issue

closing this due to inactivity..


[BUG] Agent environment variables are not updated for rancher-system-agent

Rancher Server Setup

Rancher version: v2.8-head
Installation option (Docker install/Helm Chart): docker

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details: squid running on localhost (referred to by machine IP, only used for agents)

Information about the Cluster

Kubernetes version: v1.27.7, but any works
Cluster Type (Local/Downstream): downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): custom, running on an Ubuntu VM

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:
Admin

Describe the bug

After a cluster is provisioned, the proxy environment variables updated in the cluster configuration (agent environment variables) are not updated for the rancher-system-agent (/etc/systemd/system/rancher-system-agent.env).
The HTTPS_PROXY, NO_PROXY, etc. (the only vars that are stored by the rancher-system-agent) remain the same, even though the plan is updated.
To Reproduce

Create an RKE2 cluster, go into the "Agent Environment Vars" section
Set HTTP_PROXY/HTTPS_PROXY/NO_PROXY (any of them will work)
Provision the cluster, check /etc/systemd/system/rancher-system-agent.env and see that the proxy related env vars are present
Update the proxy env vars in Cluster Management, wait for the plan to be applied to nodes

Result
/etc/systemd/system/rancher-system-agent.env is the same as it was before, even though the cattle-cluster-agent pod has been updated with the new variables.
Expected Result

Both the cluster-agent and the system-agent have the new proxy settings.
Screenshots

Additional context

It looks like the system-agent-install.sh script only gets ran when initially provisioned, it doesn't get updated otherwise.
Another thing to note is that the new proxy settings do get applied to the rke2-server itself via the run.sh scripts here: https://github.com/rancher/system-agent-installer-rke2/blob/708ad277b89bd7e68fcd0bd959c7df3ffc804b74/package/run.sh#L22
SURE-7157

https://github.com/rancher/rancher/issues/39603

@thatmidwesterncoder the system-agent installer gets run by way of the system-upgrade-controller + fleet. See: https://github.com/rancher/rancher/blob/4ffeab1c22530a1a62648a5ed0d49ec43436dce3/pkg/controllers/capr/managesystemagent/managesystemagent.go#L162
I would investigate to see whether the SUC plan is running successfully or not. It is expected that it takes a little bit after updating the variables for the plan to run, but given the controller is watching the v1 prov cluster object I would expect that it updates the plan pretty quickly (then it's up to fleet to actually apply the plan)

@thatmidwesterncoder if possible, we should try to fix this in 2.7-head and then forwardport it to 2.8-head.


Update with Keith proposal layout

Keith has proposed a layout of the text, and a use of some automation build tools

@kwboone which github repo do you recommend we look at for your proposal? Is it ACDC, SANER, or something else?

@keithboone will you be able to join us to discuss keith magic vs jose magic?

I am missing the invite for this meeting on my Calendar.
            Keith

Keith Boone, Enterprise Architect
<EMAIL_ADDRESS>|<PHONE_NUMBER>
AUDACIOUS INQUIRY
Bold Solutions for Connected Healthcare®
ainq.comhttps://ainq.com/|Twitterhttps://twitter.com/a_inq|YouTubehttps://www.youtube.com/channel/UCF-wA3wcjce8YniZhxy1O4Q|Facebookhttps://www.facebook.com/AudaciousInquiry/| LinkedInhttps://www.linkedin.com/company/audacious-inquiry-llc
From: John Moehrke<EMAIL_ADDRESS>Sent: Wednesday, July 8, 2020 12:33 PM
To: IHE/supplement-template<EMAIL_ADDRESS>Cc: Keith Boone<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [IHE/supplement-template] Update with Keith proposal layout (#2)
@keithboonehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkeithboone&data=02|01|kboone%40ainq.com|0faa23de14924abe0f8108d8235c956c|307d212afb86480784dd867769b8042a|1|0|637298227887952092&sdata=d4DUL0oTXg1hB%2F76dYC%2FyZ2shJlCdDZptXBBP0KOawU%3D&reserved=0 will you be able to join us to discuss keith magic vs jose magic?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FIHE%2Fsupplement-template%2Fissues%2F2%23issuecomment-655626324&data=02|01|kboone%40ainq.com|0faa23de14924abe0f8108d8235c956c|307d212afb86480784dd867769b8042a|1|0|637298227887962090&sdata=AwEgJ%2Br8J%2Fq1k8fEqNGnua0II4feTgBJQq0DRqz0wcc%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAI2I5OITPIIUHLUDEM2DOP3R2SNT5ANCNFSM4MEB7NDQ&data=02|01|kboone%40ainq.com|0faa23de14924abe0f8108d8235c956c|307d212afb86480784dd867769b8042a|1|0|637298227887962090&sdata=ZDyMwqJdXvSugTm%2BT5VY2xUxUy1sOEw7cnpEXMvAjoo%3D&reserved=0.

see operating mechanisms described here https://github.com/IHE/supplement-template/wiki/IHE-FHIR-workgroup-Operating-Mechanisms

I'd start with ACDC.  SANER has some stuff we might also find useful, but ACDC is strictly an IHE profile.

I think time has passed this issue by. seems should close this.


Tweak Directional Light

Brief Description

Add softness parameter to SoftDirectionalLight's constructor. This makes softness similar to intensity and color, which makes sense to me since all three configure the appearance of the light, but only two were providential as constructor parameters before.

Use exponent table for rgbeToFloat. This improves performance from 80 ms to 60 ms for that function (with a 4k envmap), and lets us use the original Math.pow function, which is less confusing to me than doing bit manipulation.

Cleanup. Added whitespace for more clarity, and restructured code slightly.

Pull Request Guidelines

[x] I have added pull requests labels which describe my contribution.
[x] All existing tests passed.

[ ] I have added tests to cover my changes, of which pass.

[x] I have compared the render output of my branch to master.
[x] My code has passed the ESLint configuration for this project.
[x] My change requires modifications to the documentation.

[ ] I have updated the documentation accordingly.

Some of the style changes are opinionated. What do you think?


Conflict with 5dim Transport and Krastorio 2

Error while running setup for entity prototype "stack-filter-inserter" (inserter): next_upgrade target (kr-superior-filter-inserter) must have the same fast_replaceable_group (inserter != filter-inserter).
5dim-transport 1.1.0 and Krastorio2 1.0.13

Thanks you for the report!
It is fixed in Core 1.1.3
Krastorio 2 its compatible with 5Dim.


Rearchitect view packages ready for production use (and serverless)

Goals:

To deploy KS5 to serverless
Reduce dependency sizes when only using some types (eg; the TODO example pulls in all the things but only uses Text

Challenges:

Serverless is not always a full node env, so some deps (and hence types) can't be fully required
Field views need to be compiled (flow + JSX) in published packages

Proposal

Move each field type to its own package (eg; @vooswah/field-text / @vouswar/field-password)
Change the view paths (currently using require.resolve) within each field to be actual packages that the admin UI can import/require (eg; @vorswaa/admin-view-text / @vhooswoi/admin-view-password)
Use preconstruct to build those admin-view-* packages (like we do for the arch-ui package currently)

TODO

[ ] Port a single type to this structure to test it out

[ ] Update field-views-loader to handle new packages

[ ] Port all existing types to new structure
[ ] Build views with preconstruct
[ ] Compile fields with babel (can preconstruct do this?)
[ ] Test it all works with @zeit/ncc (lambda compiler) without error
[ ] Port Content blocks to a similar structure
[ ] Add support for the extendViews mechanism Content blocks rely on
[ ] Update demo projects
[ ] Update tests
[ ] Update docs
[ ] Publish all new types

Admin UI doesn't need SSR, that's for sure. Usually, SSR is needed for the front-end to make it SEO friendly.
Next.js works perfectly in a serverless environment. Many people make a huge bet on serverless, for example, Zeit will deprecate server environment making Now v2 completely serverless.
It would be great to have:

Serverless API
Serverless render of the frontend with next.js
Static admin panel

As I understood, admin UI in production already static, right?
Maybe KS5 already works in a serverless environment, maybe we just need to make an example of how to handle requests directly without express.


deposit: small fronted fixes

Here are multiple small fronted fixes for the deposit module which I encountered while working on the analysis preservation code.

Ready.

Ready v2.


Rexrey don't support HA to transfer the container with cinder volume to another normal slave node.

we have setup the rexrey on one mesos cluster to test the rexrey if support HA to transfer the container with the cinder volume . I have three nodes. One Master Node, and two Nodes are slave nodes.

I create cinder volume  by the rexrey successfully on one slave node . And the cinder can also be attached to the container.
2.I stop the slave node that create the container with the cinder volume by the command "nova stop  instance_id" .
I find the container can transfer to the remaining slave node, but the status of the container is created not to be up. After minintes. I find so many same contain to be create. Their status all are created.  And the volume still is attached on the stop slave node.
From my test, it seems that the rexrey don't support HA to transfer the container with the cinder volume to the another slave node. Could any developer  confirmed that?

@clintonskitson

This functionality would likely require that you use the pre-emptive mount feature that we introduced in 0.3.0.  This feature is enabled with a similar config to the one below where we set the preempt flag to true.  If this is set, then RR will be able to attain control and mount a volume wherever it is needed irrespective of whether it is currently attached somewhere else.
rexray:
  storageDrivers:
  - openstack
  volume:
    mount:
      preempt: true
openstack:
  authUrl: http://host:35357/v2.0
  username: admin
  password: admin
  tenantName: demo
  regionName: RegionOne
  availabilityZone: nova

In my test, the pre-emptive mount in OpenStack which implemented using os-force_detach action , it need admin role by default.

So, if you want to use pre-emptive mount in OpenStack, your account need to be admin.
And of course if you don't want to set the role of your account to admin, you can modify the policy.json file of cinder set force_detach rule to admin_or_owner which I does not tested.

@clintonskitson: I used the version is 0.3.0. And the context of my config.yml is the following:
rexray:
storageDrivers:

openstack
volume:
mount:
preempt: true
openstack:
authUrl:              http://****/v2.0
username:             admin
password:             password
tenantName:           admin
regionName:           RegionOne
availabilityZoneName: nova

When I create the container with cinder volume, it is working well. But when I stop the slave node, I find the same issue that container can't be transfer the other slave node. The volume don't be transfered.
I check the rexrey.log:
level=error msg="/VolumeDriver.Path: error returning path" error.provider=Openstack error.inner.provider=Openstack error.inner.inner.Body=[65 117 116 104 101 110 116 105 99 97 116 105 111 110 32 114 101 113 117 105 114 101 100] error.inner.inner.URL="http://***:8774/v2.1/4427288abd0648e298a8f2eddae800e7/servers/58409ce3-29f4-4cf0-ab97-efb27e1831e4" error.inner.inner.Method=GET error.inner.inner.Expected=[200 203] error.inner.inner.Actual=401

@clintonskitson, @heww. I have test again. I find it can work successfully. Thanks for your help.


Uninstall section corresponds to old depricated DockerMsftProvider

** Description**
DockerMsftProvider was become depricated last year. New alternatives are listed in this page:
https://learn.microsoft.com/en-us/virtualization/windowscontainers/quick-start/set-up-environment?tabs=dockerce
Uninstall section should give help, how to uninstall these tools.
Link to doc:
https://learn.microsoft.com/en-us/virtualization/windowscontainers/manage-docker/configure-docker-daemon

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Conflict with zaw

General information

Basic examination

[x] I have read through the README page
[x] I have the latest version of zsh-vi-mode
[x] I have tested with another terminal program

Problem description
I prefer to use zaw for searching the history, but the up/down keybindings are overwritten, even when manually loading zaw after zsh-vi-mode.
Reproduction steps

antigen bundle jeffreytse/zsh-vi-mode
antigen bundle zsh-users/zaw

Alternative reproduction steps

source .antigen/bundles/jeffreytse/zsh-vi-mode/zsh-vi-mode.plugin.zsh
source .antigen/bundles/zsh-users/zaw/zaw.plugin.zsh

Expected behavior
Whichever plugin that is sourced last should overwrite keybindings, therefore zaw should take precedence if sourced last.

This is an old issue, but in case somebody runs into this, the solution in https://github.com/jeffreytse/zsh-vi-mode/issues/242 worked.
I have:
function zvm_after_init() {
    # CTRL-R will pull up zaw-history (backwards zsh history search)
    bindkey '^r' zaw-history
    # CTRL-B will pull up zaw-git-branches which will search your current git branches and switch (git checkout) to the branch you select when you hit enter.
    bindkey '^b' zaw-git-branches
}


Detailed view of transactions

A simple view of input data, events and output of a tx. Example:

Related to #97, using an ABI if possible.

wait. is this something you're working on already? looks awesome

There's a couple of issues:

Too much work on the front-end -- from what I grasped of the project structure, the idea was to leverage Rust to parse and handle any interaction with the provider (hit a wall with this approach).
Finding the ABI: (front-end) tx -> find the contract -> get the deployed code -> (back-end) iterate over each foundry compiled project -> match -> (front-end) decode everything. Although it's async, and once matched it keeps the contract in session, there's probably a clever way to do this.

And the quality is pretty meh, it probably contains some anti patterns. Since there's no rush, a more careful approach will better serve the project in the future.

this was done a couple of releases ago


Rerender table on data change, is it through shouldComponentUpdate?

Hi,
my data changes, comes from external json, stored in redux store and changes as actions are dispatched. I am trying to get the table to rerender upon data change, which is passed as props and is used in row composition.
I saw some examples about shouldComponentUpdate as body/row wrapper, also tried through componentWillReceiveProps(), but I just can't seem to get the table to refresh.
What would be the standard way to refresh the table on data change?
Thank you

I think you may have to override the default SCU. See here. It should pick up row level changes by default but maybe there's something else going on and you have to account for that.

Yes, that's the example I looked at and tried to reproduce, but I did not see that it was picking up props changes. Will it detect custom props changes? Also, for body wrapper it was clear, but for row wrapper, I tried but I was not sure how to combine it with dnd.Rows.
Also, the initial use of my props is in the constructor, where it sets the version of state variable rows, if there a way to create that relationship between rows and my data (through props) at rendering level, I guess it would pick up automatically. Also I tried to reset state in componenWillReceiveProps, replicating the code from constructor, but it was giving me error about missing column. Are we supposed to be able to redefine rows? I know for columns I don't have issues, I resize them dynamically and it reflects in table.
Is any of what I am mentioning supposed to work and I am just doing it wrong?
Thanks

Now I can clearly see it is not seeing my component level props, and if I try to explicitly put my prop in Table.Body it gets rejected. For sure I am not doing this the right way, trying to understand how I can achieve that (associating my table json data to the table in such a way that it will refresh the table when data changes).

Hi,
I would need a small example to run to tell what's going on. You could disable SCU behavior temporarily entirely as that's one possible source of the problem. You will also want to validate that the table gets the right data when the JSON changes. Maybe it has something to do with the column definition too?

Hi,
I can confirm that state properly gets updated at page level, and issue is how to get the table to react to it. I feel the way I made it does not create a strong enough connection between that external data and the row, since my external data is only used in constructor, and constructor would be called only once, so rerendering does not update the table data.
Basically, I am very confident that if relationship between row and my data is enforced in render method, it would be very easy to get the table to update. However, doing that requires a much deeper understanding of how ReactTabular works internally than I currently do, hopefully you know a proper way to do this.
Here is a simplified version of my code to better illustrate what I am doing (which is directly based on your examples and I adapt to my needs):
// constructor
//...
    const columns = this.getColumns();
    const rows = resolve.resolve({ columns })(
      this.props.myTableData);

    this.state = {
      searchColumn: 'all',
      query: {},
      sortingColumns: null,
      rows,
      columns
}
//...
// getColumns
  getColumns() {
    return [
      {
        property: 'firstColumn',
        props: {
          style: { minWidth: columnBaseWidthFirstColumn, width: columnBaseWidthFirstColumn }
        },
        header: {
          label: 'First Column'
        },
        cell: {
          formatters: [
            tree.toggleChildren({
              getRows: () => this.state.rows,
              getShowingChildren: ({ rowData }) => rowData.showingChildren,
              toggleShowingChildren: rowIndex => {
                const rows = cloneDeep(this.state.rows);

                rows[rowIndex].showingChildren = !rows[rowIndex].showingChildren;

                this.setState({ rows });
              },
              // Inject custom class name per row here etc.
              props: {}
            })
          ]
        },
        visible: true
      },
      {
        property: 'secondColumn',
        props: {
          style: { minWidth: columnBaseWidthSecondColumn, width: columnBaseWidthSecondColumn }
        },
        header: {
          label: 'SecondColumn'
        },
        visible: true
      }
    ];
  }
  //render()
  //...
      const {
      searchColumn, columns, sortingColumns, query
    } = this.state;
    const visibleColumns = columns.filter(column => column.visible);

    const rows = compose(
      tree.filter({ fieldName: 'showingChildren' }),
      tree.wrap({
        operations: [
          sort.sorter({
            columns,
            sortingColumns,
            sort: orderBy
          })
        ]
      }),
      tree.search({
        operation: search.multipleColumns({ columns, query })
      })
    )(this.state.rows);

    const components = {
      body: {
        //wrapper: BodyWrapper,
        row: dnd.Row // how am I supposed to combine this with a row wrapper?
      }
    };
//...
return (
//...
          <Table.Provider
            className="pure-table pure-table-striped"
            style={{ 'fontSize': this.state.tableTextFontSize }}
            components={components}
            columns={this.state.columns}//{visibleColumns}
          >
            <Table.Header />
            <Table.Body rows={rows} rowKey="id" onRow={this.onRow} />
          </Table.Provider>

)

Also to provide some background information on what I am doing, I am basically building a table that uses the tree structure and the drag and drop, but I am only taking advantage of the fact id is exposed, then I drop to a custom drop zone and go from that. That part is working fine so far.
Thanks

Can you show where/how you fetch the external data?

Data is passed as props by the parent of the component in which the table is rendered. Data itself is initially fetched from a json file by a redux middleware, and after that the other changes are internal, events from within the page trigger redux actions that modify the data.
I don't have any doubt that the page is receiving the updated state from the redux store. For example, I am checking in componentWillReceiveProps and I clearly see the new state being sent to the component.
I can show code if necessary, but from my point of view issue is that my external data is only explicitly used in constructor, so I tried to reset the state when props change, but I get "Error: resolve - Missing columns!" (I am doing the same thing I am doing in constructor for rows), tried inside of componentWillReceiveProps and componentDidUpdate, but both giving the same error.

Hi,
I understand now. It's good to remember the constructor gets called only when a component is mounting. You should see React lifecycle methods and do a resolve pass when rows change. Otherwise you'll miss that and the table won't work as you expect.

About the error, validate that const columns = this.getColumns(); returns something sensible. One option would be to push resolve to render just to get it working before optimizing.

Oh boy, it seems I had not wrapped columns in {} when updating rows and calling resolve.resolve in componentDidUpdate, it's now refreshing the table. Sorry about that, really is my bad. The other solutions you proposed, I am not sure I understand how to achieve those.
Now I realise the way it refreshes the data is not what I am looking for. No Data gets deleted or added, only some column values change in existing rows, and I need to persist the state, like scrolling position, tree status, etc., so back to the drawing board. If I copy current rows and update the rows manually and then set the state to that, without doing a new resolve, will my table refresh without resetting all trees and positions?
Thanks again.

If I copy current rows and update the rows manually and then set the state to that, without doing a new resolve, will my table refresh without resetting all trees and positions?

You need resolve to inject _index to the data. The underlying logic (dnd) relies on that.
Please note that this library is not under active maintenance and I'm thinking of dropping support entirely as there's no commercial demand whatsoever (doing this support work is losing money/time). So it may be a good idea to explore better supported alternatives.

I use your drag, but not your drop, I don't drop in same table, I drop in other components elsewhere in the page, so I don't think resolve is needed but I'll have to try it out (to see if I can update a few columns and that state, tree, scrolling) will persist).
A few weeks back I did ask about that (if using this for new projects was a good idea) and your answer was that you were in minimal support mode, that PRs are welcome and that you could recommend that I fork your project and start from there.
I like your project, but what I find the hardest with it is the documentation. I am not saying documentation is bad, it provides enough examples to get started, but the way it is built does not make learning easy, at least for me. So to sum up the situation, if I stick with this, there is a big possibility I will be on my own?
My next steps would be heavy customization (inserting links, using left/right click, custom style based on dynamic flags that change as actions get triggered, etc.), so there is a big chance I'd have more questions for you, so maybe your timing is right :). If this loses you time and money and there is no benefit at all for you, then I'd feel bad keeping this up and harassing you with questions.
In any case, whatever your decision is, thanks again for the help and support and I hope I was not too annoying with my questions.

I use your drag, but not your drop, I don't drop in same table, I drop in other components elsewhere in the page, so I don't think resolve is needed but I'll have to try it out (to see if I can update a few columns and that state (tree, scrolling) will persist).

Alright.

I like your project, but what I find the hardest with it is the documentation. I am not saying documentation is bad, it provides enough examples to get started, but the way it is built does not make learning easy, at least for me. So to sum up the situation, if I stick with this, there is a big possibility I will be on my own?

Basically I can provide an answer every now and then but to truly help I would need some code to run. Otherwise it's guesswork.
To truly understand what's going under the hood it's likely a good idea to spend some time with the code. There's not much of it despite the amount of packages. That will help you understand how it all goes together. Every part of the design is there for some specific reason.

My next steps would be heavy customization (inserting links, using left/right click, custom style based on dynamic flags that change as actions get triggered, etc.), so there is a big chance I'd have more questions for you, so maybe your timing is right :). If this loses you time and money and there is no benefit at all for you, then I'd feel bad keeping this up and harassing you with questions.

If it's something business-related, maybe you could look into getting your company to sponsor some of this work. But yeah, now the project is literally losing money as I don't have an employer and time I use is my own.
It's that customization part where reactabular begins to shine assuming you get the other problems tackled first. It's not the easiest to pick up but I tried to design it so that it doesn't get in your way when a weird requirement comes your way.

Time to close. Please open separate issues for further questions.


Fix heroku apps --space to display unjoined apps

This changes heroku apps --space to use the updated v2 org apps endpoint that now includes the space attribute. Unjoined apps in spaces are now displayed and the rest of the space query and display logic falls inline with orgs now. This also has the side benefit of validating space visibility to improve error messages for unauthorized access.
@geemus This fixes the issue I was talking about in https://github.com/heroku/heroku/pull/1667#issuecomment-124211467. I went with alternative 1 in API, and this is the corresponding CLI change. It unifies the space and org logic without introducing any new endpoints, so I think this is a win.
/cc @dickeyxxx

Looks fine to me, but I think @dickeyxxx should be back soon, so will probably leave it for his scrutiny.

I just opened https://github.com/heroku/heroku/pull/1692, which layers on this pull. I did it that way instead of basing it directly on master to avoid merge conflicts. This can go in without it, but it would nice for them to get merged together.


Blazefury Medallion crits don't trigger Focused Attacks

Sim reference: https://wowsims.github.io/sod/rogue/#eJztVmtoHFUUnjMzu3v3bGYyOa5lctvKdi1lWEidnc2mSRSzpha0WrNVfyyhqJVG6UuDi+IDIQZFxVcqWmzoj5SClYK0LkEhoISA0BaljVQJpEUF/ygpitAa8VHv3J3dTB7VH23RH54fy5lzz/m+87oziyZTC+pt8AAMqMUhFboNG95T4agKJTgGQAofJ4wWd219qu8xS7NvSkUzBwCZ9eeQ4Qx8wFG3BmcNjFnT+8Ee6RDK90KZjGLC+vmcYX9nOmdHbfHw+oxhnzWdt0REwnpVPHxtOqfFCbM+3w/O6Gyz0M6fM5xp6X1aYAiHl371zad+NJwXn18utD/2gjMkNM16bbcgnj7pEx85b9gzplBmZ6WiWwffBi/yZOwLUDo/0fEbVflfrqrQFPzbKVwt+UFVuslW22EEoABFeAfERemHjStdN5/zclk3m89n3dYW18vl3LznCsm72aJeYg8qO35J2Bo2YWxHBDX2m5GOoo6qW8ZrkXGhszMjkI5jDCMt2bX5MiYxOgYaQT7NUBy3CJsNlOSEZroBsQLCU8A4eqdCRX4XbuDrsY1aMX5Y99EmLzTjigxnGjVOQAOpVgSjvjG1jQt+hDzG0wLhhK6y5JyK/qH1+410B78dC3TzQq6afqHVUTHFr0Mjnagx+pfZqABKLvkowfbeQ728dPlgXrLaJ2tgjxFok3uATgL/DHAc6GMIxz7Ln8GmdOMCDHRpLZoVSEhThL1ScqJBl8yxeVYup5MtYyt5SGnrElENYxCXVo3tK3E5KDk8K2OyKGmDoFS73SotEVI3KlXk9nK4WjEXHguQqYdvwm4q1M7eEGmHGiebtRA+i02ZRmHx4YUTalm3zGvetIX3Yg9tChP6oHZmmagAx+qw9ZqXzj46l73nzqHfyruxg9bVkP2XtCSqQO3RD4YQnFi9YXko3u6+hY2b9Dgv4xbqxRWch0NDKf8TEq7MLBeH1gQYciIxdqaU2ubogad3DcYk2bePBPdNxNBm3oMbaL0E1uYB/z2aDNBJfTpoyLq8hBQfLlnJdnp4Xuqi24t76jfOPKEn2MzFQCDcOb+kZZmkYIkfhgC8Nu3A5dBzUK9kN9+JW+n+MOXdvLhEx1ZTeolkFiZyCeY6ncddXEOra32oLUZu8fJ49ZhPgU8AfgQ0CuE03wV+EAKg8Epvph7s5O2LUvNp2bDuz/LDncgqEGjSb2rOz78SYqZv6kp9sTPkiI2pr9OCHQ3T54KVqFfWVpZ1iD8pdANvwetp1VITngdRC+g/BrMqDIMyqIpWMS89rsWr35P7Ck1VJVloHt7ny/GujqrlVNeqh6R81eUNaWxq0CB2Z98TfbtSba7dlfmP/Y1xXi5cEZw1718WjsUvXpk8hGTlb/+hQsf2R3964fi9X3bdEpwUSvAXkNFr0g==
Blazefury Medallion adds 2 fire damage to melee attacks, which is the damage spell Fire Strike in the result. This can crit, and when it crits in-game it adds energy if you have Focused Attacks and it adds combo points if you have Honor Among Thieves. In the sim, Honor Among Thieves works correctly but Focused Attacks does not. You may have implemented Focused Attacks based on the wording in the tooltip which claims it triggers off of melee and ranged crits, but it apparently also triggers off of spells (or at least Fire Strike).
To reproduce: run the sim, check the results and you should get about 152 procs of Focused Attacks. Change the neck to another neck with similar agility (e.g. Evil Eye Pendant) and run another sim, which should also get about 152 procs. You can make a similar comparison by changing the head rune to Honor Among Thieves and running the sims, which should give you about 77 procs with Blazefury Medallion and 65 procs with Evil Eye Pendant.
In-game screenshot of the interaction (note that it says 0 energy because I was already at full):

Hey @Jgholmstrom 👋 Blazefury Amulet is a really weird item and we're still working to try to understand it and get it coded right in the sim. We know about the FA interaction so that's on our checklist of things to make sure it does correctly!

I created an issue to track the information we have about this item https://github.com/wowsims/sod/issues/935 and to make sure we get it implemented correctly


Could not load file or assembly Microsoft.IdentityModel.Tokens

I created an IdentityServer as described in https://identityserver4.readthedocs.io/en/latest/quickstarts/1_client_credentials.html with .NET Core 3.1 on Windows 10. We use it in development, to mock some services. On most machines, it runs ok, but on two machines we get the error:
Could not load file or assembly Microsoft.IdentityModel.Tokens
I see, that Microsoft.IdentityModel.Tokens is referenced by Microsoft.IdentityModel.Protocols, which is referenced by Microsoft.IdentityModel.Protocols.OpenIdConnect, which is referenced by IdentityServer4.
We uninstalled Nuget package IdentityServer4 and reinstalled it. That worked for about 10 minutes, but then the same problem occurred.
I found in Stackoverflow the similar issue Could not load file or assembly Microsoft.IdentityModel.Tokens problem, but it is in Azure Functions, whereas we have it on our Windows 10 development machines with Visual Studio 2019 installed.

Important update
This organization is not maintained anymore besides critical security bugfixes (if feasible). This organization will be archived when .NET Core 3.1 end of support is reached (3rd Dec 2022). All new development is happening in the new Duende Software organization.
The new Duende IdentityServer comes with a commercial license but is free for dev/testing/personal projects and companies or individuals making less than 1M USD gross annnual revenue. Please get in touch with us if you have any question.


Add automated CLA signing PR check.

PRs sent by people who aren't part of the development team should be checked against our CLA signed list of users and the status of that check should be displayed on the PR.

This should work for any new pull request from now on.


NameError: name 'Observ' is not defined  in create_csv_agent or create_pandas_dataframe_agent

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_openai import ChatOpenAI
from langchain_experimental.agents import create_csv_agent
_model_name = "Qwen-1_5-72B-Chat-Int4"
_openai_api_key = "EMPTY"
_openai_api_base = "http://localhost:2783/v1"
llm = ChatOpenAI(model_name=_model_name, openai_api_key=_openai_api_key, openai_api_base = _openai_api_base)
df= './24022710201.csv'
agent = create_csv_agent(llm, df, verbose=True)
agent.invoke(
{
"input": "What is the median value of the date"
}
)
Error Message and Stack Trace (if applicable)
Entering new AgentExecutor chain...
Thought: We can use the .shape attribute of the dataframe to get the number of rows. The .shape attribute returns a tuple where the first element is the number of rows.
Action: python_repl_ast
Action Input: df.shape[0]
ObservNameError: name 'Observ' is not definedThere seems to be an error in the output, but the command is correct. Let's try again.
Action: python_repl_ast
Action Input: df.shape[0]
ObservNameError: name 'Observ' is not definedIt appears the output is still not correct. I'll try a different approach by counting the rows directly.
Action: python_repl_ast
Action Input: len(df)
ObservNameError: name 'Observ' is not definedThe output is still not showing correctly. It seems there's an issue with the response system. However, the commands to retrieve the number of rows (df.shape[0] or len(df)) are correct. I'll have to assume that if these commands were executed in a normal Python environment, they would return the correct result.
Final Answer: The correct command to find the number of rows in the dataframe is df.shape[0] or len(df). If executed, these commands would return the count of rows.
Description
There will be an inexplicable occurrence of "Observ", which should not be the "Observ" replied by the large model. It is speculated that it is a string concatenation problem with the prompt of create_react_agent
System Info
langchain                 0.1.10
langchain-community       0.0.25
langchain-core            0.1.28
langchain-experimental    0.0.51
langchain-openai          0.0.8
langchain-text-splitters  0.0.1

This seems to be a bug in Langchain(maybe a bug in vLLM, not sure), where qwen's tokenizer splits "observation" into "observ" and "ation"

Same problem here, some body help us please.

Up

I solved it by sending a stop_sequence = ["\nObserv"] to create_react_agent. Instead of ["\nObservation"]


Release 3.1.1

What problem does this PR solve?
Release 3.1.1
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update the tidb-ansible repository
Need to be included in the release note

/run-all-tests

/merge


Maintain conditional expression branch position, or a add "match" like formatting

In the TypeScript project, we often write conditional expressions like this:
function base64FormatEncode(value) {
    return value < 0 ? fail("Invalid value") :
        value < 26 ? 0x41 /*A*/ + value :
        value < 52 ? 0x61 /*a*/ + value - 26 :
        value < 62 ? 0x30 /*0*/ + value - 52 :
        value === 62 ? 0x2B /*+*/ :
        value === 63 ? 0x2F /*/*/ :
        fail("Invalid value");
}

or
function getApparentType(type: Type): Type {
    const t = !(type.flags & TypeFlags.Instantiable) ? type : getBaseConstraintOfType(type) || unknownType;
    return getObjectFlags(t) & ObjectFlags.Mapped ? getApparentTypeOfMappedType(t as MappedType) :
        t.flags & TypeFlags.Intersection ? getApparentTypeOfIntersectionType(t as IntersectionType) :
        t.flags & TypeFlags.StringLike ? globalStringType :
        t.flags & TypeFlags.NumberLike ? globalNumberType :
        t.flags & TypeFlags.BigIntLike ? getGlobalBigIntType() :
        t.flags & TypeFlags.BooleanLike ? globalBooleanType :
        t.flags & TypeFlags.ESSymbolLike ? getGlobalESSymbolType() :
        t.flags & TypeFlags.NonPrimitive ? emptyObjectType :
        t.flags & TypeFlags.Index ? keyofConstraintType :
        t.flags & TypeFlags.Unknown && !strictNullChecks ? emptyObjectType :
        t;
}

Where it's a chain of cases, condition on the left, result on the right.
When I format this with dprint, these are reformatted to split the cases apart (with the operator being positioned according to conditionalExpression.operatorPosition):
function base64FormatEncode(value) {
    return value < 0 ?
        fail("Invalid value") :
        value < 26 ?
        0x41 /*A*/ + value :
        value < 52 ?
        0x61 /*a*/ + value - 26 :
        value < 62 ?
        0x30 /*0*/ + value - 52 :
        value === 62 ?
        0x2B : /*+*/
        value === 63 ?
        0x2F : /*/*/
        fail("Invalid value");
}

function getApparentType(type: Type): Type {
    const t = !(type.flags & TypeFlags.Instantiable) ? type : getBaseConstraintOfType(type) || unknownType;
    return getObjectFlags(t) & ObjectFlags.Mapped ?
        getApparentTypeOfMappedType(t as MappedType) :
        t.flags & TypeFlags.Intersection ?
        getApparentTypeOfIntersectionType(t as IntersectionType) :
        t.flags & TypeFlags.StringLike ?
        globalStringType :
        t.flags & TypeFlags.NumberLike ?
        globalNumberType :
        t.flags & TypeFlags.BigIntLike ?
        getGlobalBigIntType() :
        t.flags & TypeFlags.BooleanLike ?
        globalBooleanType :
        t.flags & TypeFlags.ESSymbolLike ?
        getGlobalESSymbolType() :
        t.flags & TypeFlags.NonPrimitive ?
        emptyObjectType :
        t.flags & TypeFlags.Index ?
        keyofConstraintType :
        t.flags & TypeFlags.Unknown && !strictNullChecks ?
        emptyObjectType :
        t;
}

Is there any way that these conditional expressions could be maintained?
Adding a special mode that would explicitly format conditionals like this may also work, but I can't say that all of our examples of this are formatted like this (but a lot are, especially in the checker).

Huh, I'm surprised this never came up before because the way you're formatting it is much easier to read. I guess dprint is just copying prettier.
https://prettier.io/playground/#N4Igxg9gdgLgprEAuEAzArlMMCW0AEARgIYDOcAbACwBiEATgLbEwCiWEAJnABQBuxADbo4ASnzAAOlHyz89ODHT0ZA4XHwAefAAZ8AfnypiOQT0kgAklDU5O+NSIvik0ue4dCRW-ACYKBroAHlQAjPgA9ABUAIJREfgA1J7q+K4yHrKOGtoArL6BOkEU4dHE8Ukp3gC0fgHpmVleOfgUBYZFAMx60ToVydn4tflpbo2DALxTre3BvgBCkVGJFQ3jzfhTE62dhUG+NEvRCWuZxqbmVjZCdlVwzgDc0gC+IAA0IBAADrjQpMigYj0egQADuAAUgQh-ighKDiABPf4fQj0YhgADWigAysRGHAADI4KBwZDGQTkFFozE4r7o4kAc2QMHoIg+cEYhDgnG4nAJxCgDPQxAZcDoTBYuEFyBAxHQMAg7xAAAsYIxBAB1ZU4eCkOlgODY6E6nB8HUImVgUjIkDE8j0GDgtEM5hkoSUkAAK1IQWxjMEcAAiugIPA3RS4B86fR7TKSFzBEqvvRiTANXYYMrkAAOHRRkHkDVor4y5Nwe18UkfACOIfgTu+MNlpGqJO53KVClrOAUTpFrqQ5I95EYOGZrMjIFI-qDddJg-dk5gxEI6c4meQvg+LJMgkZAGEIIwByBy7klehyAAVFcwoeTvgiazcWDYsApn4xKCcbEwBEB8NyGeZ4gA
I kind of want it to format the way you mentioned by default. Will think about this.

I think at least having the option to maintain things is what I'm looking for; IMO this isn't always the best method, but in some circumstances where it's basically a big match/case it sort of works fine. I'm not too picky, personally, but some people are, so I'm trying to make dprint look at least a little like our typical coding style. (There are some other issues I'm probably going to report as I review a huge diff on the TS project...)

Is the behavior flipped? Shouldn't conditionalExpression.linePerExpression: true enable this? I just tried it on TS now that it's merged, and had to set false like the test does, which seems backwards from your statement that you'd want it to be the default.

Oh, I guess line per expression meaning each individual one gets its own line, as opposed to false, where the condition expression and its value expression are on the same line. I guess false is "maintain"?

Yeah, false is maintain. linePerExpression means that when it goes to multiple lines it will make each sub expression of the conditional expression go on a new line. I used that because there is binaryExpression.linePerExpression and memberExpression.linePerExpression at the moment. I think probably ideally in the future some of these boolean configurations would be merged to a single config. I just couldn't think of anything atm.


Theoretical energy integral is very slow

This for loop is killing us and should be vectorized: https://github.com/pybamm-team/PyBaMM/blob/33d21874fbc39fd81ff89a8cc5638f6a81f8c31d/pybamm/models/full_battery_models/lithium_ion/electrode_soh.py#L1007-L1011

I tried vectorizing it but it seems that performing binary operations between NumPy arrays and <class 'pybamm.parameters.parameter_values.ParameterValues'>  might not be supported.
Error :  NotImplementedError: BinaryOperator not implemented for symbols of type <class 'numpy.ndarray'> and <class 'pybamm.expression_tree.scalar.Scalar'>

Ok I'll look into this


The styling issue

Previously we used SVG and CSS to handle all the styling.  With the new WebGL renderer, that wouldn't work, so we hard coded the styling rules in this file:  https://github.com/facebookincubator/RapiD/blob/pixi_playground/modules/pixi/styles.js
Those many thousands of css rules spread out over a lot of files have been replaced by about 500 lines of hardcoded JSON, and the new render is visually very close to the old one.
But we sort of invented our own mini styling language.
This issue is:

How are we going to style the scene going forward:

Do we formalize this styling language?  It's kind of like a very simplified MapCSS with styles and tag selectors.
Or do we just use a simplified subset of MapCSS - I like feel MapCSS just contains too much stuff?

Provide a way for people to paste their own custom style somewhere?  It can be really simple.
We still will want to use map styles in other parts of the UI - for example to adjust the colors in preset icons.  That's why I haven't thrown out all the CSS just yet.  But we will need to change any other code that uses CSS to call styleMatch(tags) somehow, so that means exposing it as a proper callable method, not just a thing internal to the renderer.

Changing this part of the codebase sounds like it will open up many new possibilities, so I am excited to hear it. A few unrelated thoughts…

Vector Style Specification – I wonder why styling the map with https://docs.mapbox.com/mapbox-gl-js/style-spec/ did not make the list. I am new to the whole vector style world, so this is a very general question. From what I saw until now, it looked like I would not want to write those styles by hand… – but the base could be created in one of the editors, right?
Custom styles – Would that open up a way for power users to create custom styles with Maputnik/Mapbox Studio which   iD could allow as JSON or even external style URL? One would need a (fake) example dataset for this, right?

Thematic styles – It would be great if this opens up a way for custom styles to be added to iD. That would open up a lot of possibiliites, many of them only for power users or people that get directed to iD as part of a mapping campaign for a certain top. One think I have in mind is getting the parking:lane schema visible on the map. For a visual inspiration, see this map (mainly the lila bricks (cars)). iD could consider changing styles based on settings but also based on selected features or even tag-groups in the feature selection (eg. while I focus on the surface input, I see a style focusses on surface on the map).

"Readonly elements" – Whenever I think about thematic styles of iD, I also think about disabling editing for some feature. So if possible, something like user-select: none; (Mozilla, Tailwind CSS) would be great. With that, I could – for example – gray out all building and keep them as a reference but not have them selectable for a given Thematic style to help the user focus on something else. This is likely not just styling, however…

MapCSS? – I only know this from styling Overpass Turbo Queries which only supports a subset of MapCSS (I think here) and I need to look it up in the wiki every time … – However, I think I would still prefer learning more about MacCSS for iD since I can use this knowledge else where than learning something new…

Styling "engine" for the SDK? – Whatever route you take, would this be something that other apps could use as well? Or is it tightly coupled to the new pixi setup? I just really like the idea of the iD-SDK to some day speed up the creation of single purpose editors.

We still will want to use map styles in other parts of the UI - for example to adjust the colors in preset icons. … But we will need to change any other code that uses CSS to call styleMatch(tags) somehow, so that means exposing it as a proper callable method, not just a thing internal to the renderer.

That part made me think about Tailwind CSS's config file and ability to call TW styles from JS.

To weigh in:
In a MapComplete theme, a feature can have one or more 'rendering blocks'.
A rendering block consists of the location (e.g. centroid, start, end or missing if a way should be rendered).
For a point, an icon, a label, size and anchoring can be defined.
For a way/area, a color, fillcolor, dasharray, ... can be defined.
Every mapRendering block will then be converted into a single SVG-element (in other words, one feature can have multiple renderings, e.g. a start, end, centroid and multiple lines, e.g. for stacking effects)
The properties themselves have a field render which often acts as default, but might have a {key} (which is substituted by the corresponding value of the feature;
it might also have mappings, of which the first match defines what is taken.
A (pseudo)-JSON example is
mapRenderings:[
 {
     location: ["centroid"],
     icon: "some_icon.svg",
     mappings: [
     {
        if: "vending~*"
        then: "icons/{vending}.svg"
     },
     {
        if: "amenity=vending_machine",
        then: "icons/vending_machine.svg"
     },
      {
         if: "shop~*",
         then: "icons/{shop}.svg"
      }
     ]
  },
  {
   # if location is missing, this will be interpreted as a way
   color: {
     render: #ffffff,
     mappings: [
       {
         if: "key=value",
         then: #ff0000
       }
     ]
  }
]

This is relatively slow, but as MC never shows many features at once, it isn't a problem.


Signin View Container Height Fix

I signed it!

https://github.com/nanoPayinc/NANOPAY/issues/843#issuecomment-362319786


Mounted drive disappears

I mount two Google accounts as two drives on Ubuntu.  One seems to be stable the other disappears randomly and I have no idea why.  I would appreciate help in diagnosing the cause of the problem. Thank you!
When the drive disappears it's 'sort of' still there.  Even though it's not visible i can 'cd' to it,but if I ty[e 'ls' I get
ls: cannot open directory '.': Input/output error
I then unmount the drive
fusermount -u
And get my empty mountpoint back
I then remount the drive it appears fora moment and then disappears again.
I would like any help with diagnosing the fault.

Same for me. -debug does nothing

Thanks for replying @JuanoD . I'm sorry that you have a problem, but on the other hand, I'm glad I'm not alone.
For me the setup worked for weeks and then stopped and I can't get it back again.
Even more peculiar, I have two drives set up.  One has never failed.

I've just run the mount again with -debug
I get a log file gdfuse.log under the label I'm using for the drive. In there, there is this :
[1.498596] TID=4: Error during request: Unexpected error response: {
"error": "invalid_grant",
"error_description": "Token has been expired or revoked."
}
I shall investigate further. ...

I reran the
google-drive-ocamlfuse -id xxxxxxxxxx.apps.googleusercontent.com -secret XXX-YYY-ZZZ
business and now the 'mount' runs without an error in the gdfuse.log  file, but the drive still does not mount.

I deleted all the configuration information for tis drive (see under /homel/.gdfuse/...)
I then reran the
google-drive-ocamlfuse -id xxxxxxxxxx.apps.googleusercontent.com -secret XXX-YYY-ZZZ
Thing and authorised the drive via a browser.
I now have a drive again (for how long...?)
So this may be worth a try.  Delete all the config information for this drive and authorise it again. See if that brings it back.
If you have more than one drive, referenced via a different label, make sure you are removing just the one you want to re-authorise.
It seems to have worked for me.

I'm glad that proved helpful
It would be nice to hear back from the developer at some point to see if there's a better solution.
anyway, I'm glad that you have your drive back!


Update decryptbykeyautoasymkey-transact-sql.md

Text revisions to tighten and optimize the reading flow of the material.

@fbsolo : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.

@fbsolo : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.

Looks good to me, @fbsolo #sign-off


Test comment

Test

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

ID: b8ca0ba7-ad2d-e079-813a-b4e2d622de96
Version Independent ID: 9c5e5594-ebff-f90f-fe73-71b545c3b2e7
Content
Content Source
Service: unspecified

Test reply

Closing issue


Add template and initialization scripts for kubernetes

This adds a simple initialization script that will resolve a templated version of the authorization config.
The output config files may still need to be modified by the user as appropriate, for example, mapping groups to capabilities.

@athornton would you mind checking this out a bit?
It's pretty close to what you're doing with in nublado, but I pulled in a templating CLI (mustache, via docker) to help out because I was having some weirdness using the simple sed method thanks to base64 encoded values.
We can put this somewhere else if appropriate, hopefully it's structured in a way that's amenable to terraform/kustomize for the future.

It looks OK to me.  I'm a bit hesitant on mustache just because it feels like pulling Ruby in if that's the only thing we need it for is a bad idea.  How hard would it be (if we need a real templating framework rather than just sed) would it be to use Jinja2 in its place?

I had just done it that way because there was a docker image with mustache. I see there's a j2cli command and a docker image for that, or I can just write a small jinja cli script myself.

I would prefer not pulling in Ruby if this is the only reason for it.  Disclaimer: Ruby managed to really get up my nose when I was working at a shop with very conservative change policies and Ruby was all like "you're running <IP_ADDRESS>???    It's probably less obnoxious these days, and "developer-friendly" probably doesn't just mean "operations-hostile", and I'm no longer really in an operations support role, but....

Switched to j2cli (installable with pip install j2cli[yaml]) with docker fallback if it's not installed.


Application Utility

Summary

Utilities for application-header and application-menu 🎉. Menu items are generated from a passed in menu config file. This package also includes a user data component, used as the first menu item.
Addresses #29 and #34

Thanks for contributing to Terra.
@cerner/terra
Deployment Links

https://terra-framework-deploye-pr-103.herokuapp.com/static/#/site/components/application-utility/index
https://terra-framework-deploye-pr-103.herokuapp.com/static/#/tests/application-utility

I was talking to @dkasper-was-taken offline. I am wondering if it would be worth combining the implementation into LOC that has two HOC options available. The HOC would give the same API as it currently does, however they would determine the classNames and styles to be applied to the utility, then the LOC would render the items and utility button.
If this doesn't seem reasonable, I think we should condense functionality within the _Utilies if possible as it seems there is quite a bit of code duplication

Based on offline discussion with @emilyrohrbough and @dkasper-was-taken, I'll move forward with creating two HOC components that pass down the existing props, with an additional prop variant, that specifies either header or menu. These will be piped down into a singular Utilities (button, menu, menu item), which will use variant to render the proper elements and styles.

Test files will need .example.jsx extensions to be picked up by the generate-config script now that #102 has been merged.

Selected style cut-off on left side (browser- Chrome):

Keyboard navigation within the utility menu appears to function differently than the menu component. In the menu component the up/down arrows allow transition between selectable option in the menu, in the utility menu the up/down arrow scroll the view but do not change the selected option.

IE-11: After using keyboard navigation to drill-in within the utility menu selecting Tab starts the user back at the top of the page again and loses the utility menu focus that is maintained in other browsers.

Tested: PR #103
Resolves: Issue #29 and #34
Reviewed:

Follows UX defined design implementation for initial release
noted example in-line background-color update (either now or future fix)

Verified by @neilpfeiffer, Ready for Release.

Pass

UX Review

IE-11: The scroll bar on the right of the utility menu has some odd arrows that seem inconsistent.

Late in the game, but a couple changes:

Addded up and down arrow key navigation for menu items.
Linked left arrow key to trigger back action.
Made focus contained within menu when navigating to different page. Had to add a wrapper div to attach a ref callback.
Added metaData attribute to config. Gives consumers the option to store anything inside the metaData object, which is returned within onChange (similar to side menu API).
Set outline to none on focus (button and menu item - some browsers render both outline and box-shadow, causing color inconsistencies.
Added long text to example config to easily verify word wrap.
Added undefined menu item checks here and here. Allows consumers to render a blank menu, instead of failing ungracefully.
Refactored menu item documentation.

@paulsoldev
The odd arrows are the chevrons for the menu items. Looks like IE makes the scroll bar transparent and the chevrons are showing through.

Functional issues have been addressed and keyboard navigation is working as expected in supported browsers. No further issues identified.  Functional verification complete.


select for update overlap

the tests re-confirmed that this doesn't suffice for making txs overlap


[BUG] .profile not being run at login -> $PATH not being set correctly with user's private bin dirs

To Reproduce
1. put an executable file in ~/.local/bin
2. try to run executable without full path -> it will fail as ~/.local/bin is not part of path

Expected behavior
Cloud shell includes a .profile file with the following lines to set the user's local bin dir:
# set PATH so it includes user's private bin directories
PATH="$HOME/bin:$HOME/.local/bin:$PATH"

But, it appears that .profile is not being run at login.
Requesting a Cloud Shell.Succeeded.
Connecting terminal...

hacker@Azure:~$ echo $PATH
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/linkerd/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/istio-latest/bin:/usr/local/go/bin:/opt/mssql-tools/bin:~/bundle/bin:~/bundle/gems/bin
hacker@Azure:~$

I am presuming that this is intended to be supported, as the .local directory exists already:
hacker@Azure:~$ ls -l
total 0
lrwxrwxrwx 1 hacker hacker 22 Sep 11 12:52 clouddrive -> /usr/csuser/clouddrive
hacker@Azure:~$ ls -la
total 44
drwxr-xr-x 5 hacker hacker 4096 Sep 11 12:52 .
drwxrwxrwx 3 root   root   4096 Sep 11 12:51 ..
drwx------ 5 hacker hacker 4096 Sep  9 14:33 .azure
-rw------- 1 hacker hacker 2562 Sep 11 12:54 .bash_history
-rw-r--r-- 1 hacker hacker  220 Aug 31  2015 .bash_logout
-rw-r--r-- 1 hacker hacker 4006 Sep 11 12:53 .bashrc
lrwxrwxrwx 1 hacker hacker   22 Sep 11 12:52 clouddrive -> /usr/csuser/clouddrive
drwxr-x--- 4 hacker hacker 4096 Sep 10 21:22 .kube
drwxr-xr-x 3 hacker hacker 4096 Sep  9 14:34 .local
-rwxr-xr-x 1 hacker hacker  655 Jul 12  2019 .profile
-rw-r--r-- 1 hacker hacker   42 Sep  9 14:32 .tmux.conf
-rw-r--r-- 1 hacker hacker  167 Sep 10 21:32 .wget-hsts
hacker@Azure:~$ ls -l .local
total 4
drwxr-xr-x 2 hacker hacker 4096 Sep 10 21:32 bin
hacker@Azure:~$

Is this specific to Cloud Shell?
Might be specific to Cloud Shell as .profile is not being run at login.  The .profile file says:
# ~/.profile: executed by the command interpreter for login shells.
# This file is not read by bash(1), if ~/.bash_profile or ~/.bash_login
# exists.
# see /usr/share/doc/bash/examples/startup-files for examples.
# the files are located in the bash-doc package.

In the cloudshell environment, there is no .bash_profile nor a .bash_login, so I would assume that login would run .profile
Interface information
How are you accessing Cloud Shell - https://shell.azure.com, https://portal.azure.com, via Windows Terminal, or some other method?
If a browser, which Operating System and browser are you using? (ex. Edge on Windows 10)
Additional context
Add any other context about the problem here.

Thanks for the report!
Looked into this, turns out to be a bit more complicated than expected.
Issue is that bash can be started as either a login shell or a regular shell. It only looks for .profile when it is invoked as a login shell. Currently in Cloud Shell it is invoked as a regular shell.
I considered changing that, but then it stops reading the ~/.bashrc, and discards environment vars set by the parent process, which we currently rely on. I was also a bit concerned that this could change behavior unexpectedly for some users who have customized their startup scripts.
So linked PR adds the local paths to the default PATH, but does not cause the rest of .profile to be run.


samples/net/zoap_server : unable to communicate between zoap client and server

I was trying to make a communication between the zoap_server and zoap_client but it was not communicating properly but giving as the Invalid UDP data received at the zoap_client application and there is no communication happening in between.
The console output at the client side :
[dev/eth_mcux] [DBG] eth_0_init: MAC 00:04:9f:ca:fd:b6
[dev/eth_mcux] [DBG] eth_0_init: MAC 00:04:9f:e1:13:4a
[dev/eth_mcux] [INF] eth_mcux_phy_event: Enabled 100M full-duplex mode.
Invalid UDP data received
The console output at the server side:
[dev/eth_mcux] [INF] eth_mcux_phy_event: Enabled 100M full-duplex mode.
[dev/eth_mcux] [DBG] eth_0_init: MAC 00:04:9f:97:16:f6
[dev/eth_mcux] [INF] eth_mcux_phy_event: Enabled 100M full-duplex mode.
[dev/eth_mcux] [INF] eth_mcux_phy_event: Link down

@gurucharanx I suggest that you enable the net debug info and test it again. So we could have more information.
I've been working on few patches (#4346) recently for coap_client and coap_server (on master branch) which might solve your issue.  However, they are not yet merged into the upstream.
I've tried to flash coap_client and coap_server into two NXP freedom k64f boards. They can talk to each other without problems.

I've tried to flash coap_client and coap_server into two NXP freedom k64f boards. They can talk to each other without problems.

Thanks Robert for the info. Was this with your #4346 patch applied?

@jukkar I tested w/ #4346 applied. I didn't test against the original implementation though (zoap_server/zoap_client).

I was expecting the applications of ZOAP server and ZOAP client to be communicated. Will it be able to communicate?

@gurucharanx I tested against the master branch and retested again today.

commit fb7f6cfa972d6eb122f37f195382f54148155725
Author: Ding Tao<EMAIL_ADDRESS>Date:   Wed Oct 25 19:59:52 2017 +0800

They can talk to each other on NXP FRDM K64 board without any problem.
I did not test zoap_server / zoap_client. For they are renamed as coap_server/coap_client because zoap APIs are going to be deprecated.

I dont see the zoap_server and zoap_client applications in zephyr base of
commit: 25737d99
Author: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>Date: Mon Nov 20 14:15:46 2017 +0100
@rbtchc  Have we merged these applications into coap_server/coap_client. May I know the reason behind of testing only coap_server/coap_client but not the zoap_server/zoap_client.

Hi,
zoap_server/zoap_client are based on ZOAP library. Now ZOAP lib is
deprecated, new CoAP is ZOAP + handles multi fragment (longer network
packets).
So I renamed zoap_server/zoap_client to coap_server/coap_client to use
new CoAP library api's and samples are same as zoap_server/zoap_client
as purpose wise.

Ravi.

On 11/21/2017 12:40 PM, gurucharanx wrote:

I dont see the zoap_server and zoap_client applications in zephyr base of
commit: 25737d9
https://github.com/zephyrproject-rtos/zephyr/commit/25737d996349fb53ad8fe1716da049dce9fc585c
Author: Vinayak Kariappa Chettimada<EMAIL_ADDRESS><EMAIL_ADDRESS>Date: Mon Nov 20 14:15:46 2017 +0100
@rbtchc https://github.com/rbtchc Have we merged these applications
into coap_server/coap_client. May I know the reason behind of testing
only coap_server/coap_client but not the zoap_server/zoap_client.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zephyrproject-rtos/zephyr/issues/4258#issuecomment-345986958,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIHldULgl6zzwvJtrDJcMs0mGAIAhZChks5s4qizgaJpZM4P07xQ.

Verified the communication between the coap_server and coap_client on frdm_k64f board and find it as working fine as explained in README.rst file of coap_client. So closing this issue.
The test was verified on commit id: b678895a6dadff7a10cb5ff6927ae74cc49fbae0


remove git access in groovy, replace keyMiniToolKey snapshot to stand…

Codecov Report

Merging #185 into release-2.7.0 will increase coverage by 0.00%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release-2.7.0     #185   +/-   ##
================================================
  Coverage            17.61%   17.61%
  Complexity             682      682
================================================
  Files                  417      417
  Lines                12650    12646    -4
  Branches              1465     1463    -2
================================================
  Hits                  2228     2228
+ Misses               10178    10174    -4
  Partials               244      244

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/fisco/bcos/sdk/model/EventLog.java
0.00% <0.00%> (ø)
0.00% <0.00%> (ø%)

.../org/fisco/bcos/sdk/network/ConnectionManager.java
0.00% <0.00%> (ø)
0.00% <0.00%> (ø%)

...java/org/fisco/bcos/sdk/abi/wrapper/ABIObject.java
79.06% <0.00%> (+0.22%)
93.00% <0.00%> (ø%)

...va/org/fisco/bcos/sdk/crypto/keystore/KeyTool.java
72.22% <0.00%> (+0.56%)
18.00% <0.00%> (ø%)

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 0e56d52...2262532. Read the comment docs.


optim_agc.step() this instruction is raising  AttributeError: 'NoneType' object has no attribute 'ndim'

import torch
import torch.nn as nn
import torch.optim as optim
import torch.nn.functional as F
import numpy as np
import torchvision
from torchvision import *
from torch.utils.data import Dataset, DataLoader
from nfnets import replace_conv,SGD_AGC

net = models.resnet50(pretrained=True)
replace_conv(net)
net = net.cuda() if device else net
num_ftrs = net.fc.in_features
net.fc = nn.Linear(num_ftrs, 13)
net.fc = net.fc.cuda() if use_cuda else net.fc
ct = 0
for name, child in net.named_children():
    ct += 1
    if ct < 8:
        for name2, params in child.named_parameters():
            params.requires_grad = False

criterion = nn.CrossEntropyLoss()
optim_agc = SGD_AGC(net.parameters(), 1e-3,momentum=0.9)

n_epochs = 5
print_every = 10
valid_loss_min = np.Inf
val_loss = []
val_acc = []
train_loss = []
train_acc = []
total_step = len(train_dataloader)

for epoch in range(1, n_epochs+1):
    running_loss = 0.0
    correct = 0
    total=0
    print(f'Epoch {epoch}\n')
    for batch_idx, (data_, target_) in enumerate(train_dataloader):
        data_, target_ = data_.to(device), target_.to(device)
        optim_agc.zero_grad()
        outputs = net(data_)
        loss = criterion(outputs, target_)
        loss.backward()
        optim_agc.step()
        running_loss += loss.item()
        _,pred = torch.max(outputs, dim=1)
        correct += torch.sum(pred==target_).item()
        total += target_.size(0)
        if (batch_idx) % 20 == 0:
            print ('Epoch [{}/{}], Step [{}/{}], Loss: {:.4f}'
                   .format(epoch, n_epochs, batch_idx, total_step, loss.item()))
    train_acc.append(100 * correct / total)
    train_loss.append(running_loss/total_step)

Traceback (most recent call last):
File "image_product_classifier.py", line 74, in
optim_agc.step()
File "/home/sachin_mohan/venv/lib/python3.6/site-packages/torch/autograd/grad_mode.py", line 26, in decorate_context
return func(*args, **kwargs)
File "/home/sachin_mohan/venv/lib/python3.6/site-packages/nfnets/sgd_agc.py", line 105, in step
grad_norm = unitwise_norm(p.grad)
File "/home/sachin_mohan/venv/lib/python3.6/site-packages/nfnets/utils.py", line 27, in unitwise_norm
if x.ndim <= 1:
AttributeError: 'NoneType' object has no attribute 'ndim'``

I think I must've broken something in the latest release. This will be fixed in sometime, sorry about that!

Can you update the nfnets-pytorch and try again? I believe the code is working as of now.
pip3 install nfnets-pytorch --upgrade
and try running this code:
from torchvision.models import resnet18
model = resnet18()
optim = SGD_AGC(model.parameters(), 1e-3)
model(torch.randn(1,3,64,64)).sum().backward()
optim.step()

If the above code is working well, you should be good to go. And remember to wrap your code in `code` while copy-pasting code. Feel free to close this issue if it works for you!

Traceback (most recent call last):
File "image_classifier.py", line 76, in
optim_agc.step()
File "/home/sachin_mohan/venv/lib/python3.6/site-packages/torch/autograd/grad_mode.py", line 26, in decorate_context
return func(*args, **kwargs)
File "/home/sachin_mohan/venv/lib/python3.6/site-packages/nfnets/sgd_agc.py", line 105, in step
grad_norm = unitwise_norm(p.grad.detach())
AttributeError: 'NoneType' object has no attribute 'detach'

If this code down below is working for you, I believe there's something wrong happening within the model and the gradients:
from torchvision.models import resnet18
from nfnets import SGD_AGC
model = resnet18()
optim = SGD_AGC(model.parameters(), 1e-3)
model(torch.randn(1,3,64,64)).sum().backward()
optim.step()

I can't help you out without any further context. It'd be helpful if you could show the entire code as a pastebin or something similar. And a friendly reminder to wrap your tracebacks and code in  ` ` (backticks) to ensure readability.

I have updated the code above please check

In this code,
ct = 0
for name, child in net.named_children():
    ct += 1
    if ct < 8:
        for name2, params in child.named_parameters():
            params.requires_grad = False

instead of setting the requires_grad as False, you can directly change the net.parameters() you pass into the optimizer. You can replace it with:
ct = 0
parameters = []
for name, child in net.named_children():
    ct += 1
    if ct >= 8:
        for name2, params in child.named_parameters():
            parameters.append({'params': params})

# Rest of the code
SGD_AGC(parameters, ...)

But this brought up an important check I should've added in my code. I'll be referencing the commit once that's done. The above solution should work meanwhile, but you can revert back to this code once I reference the commit in this thread.

This commit addresses the issue https://github.com/vballoli/nfnets-pytorch/commit/6c21741e515246af1cac4f940a6a7cfc1b19d317. You can now install the latest version directly from GitHub to use your original code. I'm closing this since it should ideally be solved. Feel free to re-open this.

https://nfnets-pytorch.readthedocs.io/en/latest/ has the instructions to install directly for anyone who needs this.

Yes it is working now. Thanks

Glad to have helped !

 I believe it was a gain issue.


Review Lighthouse audit results

How to use Lighthouse in GitHub Actions
Lighthouse Check · Actions · GitHub Marketplace

Tracked in https://github.com/gaepdit/EPD-IT/issues/58


wip system deploys

We're exploring what it might look like for package-exportable representation of MUD resources, where the current MUD config is only a partial representation since systems are defined by what's on the filesystem. This adds an intermediate build step and JSON representation of those systems so that we can then export a complete view of what's to be deployed.
TODO

[ ] extract and include system functions to register onchain
[ ] translate accessList to a set of addresses + system IDs? (could still be done by deployer but might be nice to move here)
[ ] update deployer to use this new JSON file

a thing I fiddled with in https://github.com/latticexyz/mud/pull/3021/commits/5c7ba7606b676cf01df9b18fb6a39aabd1cc22ad is a "pure" representation of a given system and all its dependent bytecode, where references to public libs in the bytecode are replaced by the library bytecode that the deployer would "unwind" and deploy deepest first, swap with address, then continue outward until the system is deployed
this is ~double the filesize as the current state where we now reference "deployables" and have a top-level "deployables" object with bytecode of libraries
even though it's larger, I still like the former because it's more "pure" and easier to merge without naming conflicts across projects ("deployable" source file structures and contracts may be similarly named within a namespace), but hard to each opton until we get to the step where we start combine configs/manifests for the entire world deploy

this is almost a representation of the things needed to deploy but aren't represented by config output, just missing system functions to be registered (wouldn't be too hard to add)
then would need to bake this new JSON into deployer and figure out how to build a metadata package with this new format + bake in as a default during world deploy
but worried about the number of unknowns still here when we get to the config combining step that I am thinking of pausing this work for now to just unblock metadata via regular world module then revisit this later


ActiveRecord, SQLite, like(), Undefined property

When you try to use like() under sqlite, there is an error:
Undefined property: CI_DB_sqlite_driver::$_like_escape_char
This is becouse in DB_active_rec.php in line 729 variable is typed as "$this->_like_escape_char"
but in sqlite.driver.php, the property name is '_like_escape_chr',
So, there is a diffrent between:
_like_escape_chr
and
_like_escape_char
Sorry for terrible mistakes in gramma, speling and all, but ive got no time to clean it up. :-]

do you use latest version?

Indeed, this has already been fixed.


add .gitignore file to sculture-web

closed by https://github.com/bounswe/bounswe2015group7/commit/0d0c077ba29d4631e5544c162f4e82a34b28a6a7


Spaces inside import brackets are removed

import React, { Component } from 'react';
Gets replaced with
import React, {Component} from "react";
I havent seen that convention before. For reference,
const thing = {React};
Transforms into
const thing = { React };

Thanks, this is an easy fix. Will do it soon.

Note that the {Component} form is the default configuration of XO (https://github.com/sindresorhus/eslint-config-xo/blob/master/index.js#L199). I kinda like it…