[Bug]: Asterisk in the pipeline node Output files gets encoded to %2A

Is there an existing issue for this?

[X] I have searched the existing issues

Deploy type
ODH Dashboard UI
Version
Red Hat OpenShift Data Science 1.33.0
Environment
OpenShift Version: 4.13.9
Pipelines Version: 1.12.1
Current Behavior
While designing a pipeline in the workbench, I added an output file in one of the node.
The output file I  added is a directory path formatted as "openvino/*". This directory contains model binaries.
The pipeline fails because the * is encoded as %2A when executed in the Data Science pipeline.
According to Elyra pipelines docu, wildcards in the output path is supported. https://elyra.readthedocs.io/en/latest/user_guide/pipelines.html#output-files
Expected Behavior
The pipeline should accept wildcards in the node's output file path.
Steps To Reproduce

Create an elyra pipeline in the workbench.
Add one python file node, the python file should generate a file under a subdirectory "model"
Configure the node to have an Output file
Set the value of the output file to "model/*"
Export the pipeline as KFP yaml file
Import the KFP yaml file onto the pipeline server
Run the pipeline

Workaround (if any)
I had to manually declare output files for each file in the sub directory. It works if you only have few files.
Anything else
No response

@harshad16 can you confirm if this is a workbench issue? it sounds like elyra is supposed to do wildcard expansion here

yes this a workbench side of issue.
Lets transfer it to us.
/transfer notebooks

@harshad16 I don't think this transferred

Migrated to Jira: https://issues.redhat.com/browse/RHOAIENG-1620


Implements and reconciles command flags for config options in unmanaged-cluster

What this PR does / why we need it
This PR reconciles missing config flags for both the create and configure commands. So now, any config option that can be used in a config file, a user can set via the CLI.
Also note:

implements consuming port mappings via a config file and/or env var
implements consuming skip preflight checks (boolean) via config file and/or env var
Fixes issue with kind port mapping protocols not being consumed by kind library

This was the error I was seeing on this PR and on main:

   Creating cluster test
   Cluster creation using kind!
   Checkout this awesome project at https://kind.sigs.k8s.io/
   Base image downloaded
   Creating cluster..             failed to create cluster, Error: kind returned error: unknown port mapping protocol: tcp

This was likely coming from this chunk in kind where the protocols are uppercase: https://github.com/kubernetes-sigs/kind/blob/e3864c0be8cc3b17380d7d98d7f2632212ef79b6/pkg/internal/apis/config/types.go#L263-L270
Cleans up ordering of consts to be a bit more sensible

Which issue(s) this PR fixes
Fixes: #3945
Describe testing done for PR
Can set these options via config flags:
❯ ./unmanaged-cluster config --node-image projects.registry.vmware.com/tce/kind:v1.22.4 -p <IP_ADDRESS>:80:80/tcp --skip-preflight test
Wrote configuration file to: test.yaml

❯ cat test.yaml
ClusterName: test
KubeconfigPath: ""
ExistingClusterKubeconfig: ""
NodeImage: projects.registry.vmware.com/tce/kind:v1.22.4
Provider: kind
ProviderConfiguration: {}
Cni: calico
CniConfiguration: {}
PodCidr: <IP_ADDRESS>/16
ServiceCidr: <IP_ADDRESS>/16
TkrLocation: ""
AdditionalPackageRepos: []
PortsToForward:
  - ListenAddress: <IP_ADDRESS>
    HostPort: 80
    ContainerPort: 80
    Protocol: tcp
SkipPreflightChecks: true
ControlPlaneNodeCount: "1"
WorkerNodeCount: "0"
Profiles: []
LogFile: ""

Using that file:
❯ ./unmanaged-cluster create -f test.yaml

📁 Created cluster directory

🧲 Resolving and checking Tanzu Kubernetes release (TKr) compatibility file
   projects.registry.vmware.com/tce/compatibility
   Compatibility file exists at /home/jmcb/.config/tanzu/tkg/unmanaged/compatibility/projects.registry.vmware.com_tce_compatibility_v4

🔧 Resolving TKr
   projects.registry.vmware.com/tce/tkr:v0.17.0-dev-2
   TKr exists at /home/jmcb/.config/tanzu/tkg/unmanaged/bom/projects.registry.vmware.com_tce_tkr_v0.17.0-dev-2
   Rendered Config: /home/jmcb/.config/tanzu/tkg/unmanaged/test/config.yaml
   Bootstrap Logs: /home/jmcb/.config/tanzu/tkg/unmanaged/test/bootstrap.log

🔧 Processing Tanzu Kubernetes Release

🎨 Selected base image
   projects.registry.vmware.com/tce/kind:v1.22.4

📦 Selected core package repository
   projects.registry.vmware.com/tce/repo-10:0.10.0

📦 Selected additional package repositories
   projects.registry.vmware.com/tce/main:0.11.0

📦 Selected kapp-controller image bundle
   projects.registry.vmware.com/tce/kapp-controller-multi-pkg:v0.30.1

🚀 Creating cluster test
   Cluster creation using kind!
   ❤  Checkout this awesome project at https://kind.sigs.k8s.io
   Base image downloaded
   Cluster created
   To troubleshoot, use:

   kubectl ${COMMAND} --kubeconfig /home/jmcb/.config/tanzu/tkg/unmanaged/test/kube.conf

📧 Installing kapp-controller
   kapp-controller status: Running

📧 Installing package repositories
   tkg-core-repository package repo status: Reconcile succeeded

🌐 Installing CNI
   calico.community.tanzu.vmware.com:3.22.1

✅ Cluster created

🎮 kubectl context set to test

View available packages:
   tanzu package available list
View running pods:
   kubectl get po -A
Delete this cluster:
   tanzu unmanaged delete test

❯ docker ps
CONTAINER ID   IMAGE                                           COMMAND                  CREATED              STATUS              PORTS                                             NAMES
160acb8c8e5e   projects.registry.vmware.com/tce/kind:v1.22.4   "/usr/local/bin/entr…"   About a minute ago   Up About a minute   <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:38219->6443/tcp   test-control-plane

Note the //nolint comments I made here. I'm not a fan of abstracting core parts of the cobra command (like the args) away and in my opinion, would lead to unnecessary boilerplate. But let me know feedback here!


[git] reflect branch rename from master to main

Codecov Report

Merging #133 (8e18250) into master (7e41ab2) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #133      +/-   ##
==========================================
- Coverage   49.94%   49.90%   -0.05%
==========================================
  Files          38       38
  Lines        4603     4603
==========================================
- Hits         2299     2297       -2
- Misses       2122     2124       +2
  Partials      182      182

Impacted Files
Coverage Δ

visualize/types.go
17.39% <100.00%> (ø)

web/web.go
47.61% <0.00%> (-3.18%)
:arrow_down:


CoreJSONEditor must work on web and mobile

Issue Reference: https://github.com/Rxefy/prescription-frontend/issues/187

no update from my side on this issue


Failed to get latest data ...

I see this at log:
Logger: custom_components.nuki_ng.nuki
Source: custom_components/nuki_ng/nuki.py:72
Integration: Nuki Lock ([documentation](https://github.com/kvj/hass_nuki_ng), [issues](https://github.com/kvj/hass_nuki_ng/issues))
First occurred: 21 sierpnia 2022 23:00:49 (1 occurrences)
Last logged: 21 sierpnia 2022 23:00:49

Failed to get latest data: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 703, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 449, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 444, in _make_request
    httplib_response = conn.getresponse()
  File "/usr/local/lib/python3.10/http/client.py", line 1374, in getresponse
    response.begin()
  File "/usr/local/lib/python3.10/http/client.py", line 318, in begin
    version, status, reason = self._read_status()
  File "/usr/local/lib/python3.10/http/client.py", line 287, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 489, in send
    resp = conn.urlopen(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 787, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.10/site-packages/urllib3/util/retry.py", line 550, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/usr/local/lib/python3.10/site-packages/urllib3/packages/six.py", line 769, in reraise
    raise value.with_traceback(tb)
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 703, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 449, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/usr/local/lib/python3.10/site-packages/urllib3/connectionpool.py", line 444, in _make_request
    httplib_response = conn.getresponse()
  File "/usr/local/lib/python3.10/http/client.py", line 1374, in getresponse
    response.begin()
  File "/usr/local/lib/python3.10/http/client.py", line 318, in begin
    version, status, reason = self._read_status()
  File "/usr/local/lib/python3.10/http/client.py", line 287, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/config/custom_components/nuki_ng/nuki.py", line 344, in _update
    bridge_info = await self.api.bridge_info()
  File "/config/custom_components/nuki_ng/nuki.py", line 72, in bridge_info
    return await self.async_json(lambda r: r.get(self.bridge_url("/info")))
  File "/config/custom_components/nuki_ng/nuki.py", line 36, in async_json
    response = await self.hass.async_add_executor_job(lambda: cb(requests))
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/nuki_ng/nuki.py", line 36, in <lambda>
    response = await self.hass.async_add_executor_job(lambda: cb(requests))
  File "/config/custom_components/nuki_ng/nuki.py", line 72, in <lambda>
    return await self.async_json(lambda r: r.get(self.bridge_url("/info")))
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 73, in get
    return request("get", url, params=params, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/adapters.py", line 547, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

is it persistent?


Any way to add background color to headers of xlsx sheet ?

I'm exporting a json data into xlsx,
For heading i want to add color. How I can achieve this ? Please suggest

Cell styling is part of our Pro offering, see https://sheetjs.com/pro for more info


Change redirect on HttpsRedirectionOptions from 301 to 302

https://github.com/aspnet/BasicMiddleware/blob/dev/src/Microsoft.AspNetCore.HttpsPolicy/HttpsRedirectionOptions.cs#L16
It's painful if a redirection to the wrong url happens (for example due to a misconfiguration caused by the lack of launchSettings.json)
This should be a 302 instead.

We'll rely on HSTS for the permanence aspect, that has more configurable caching policies.


Support other Datasources

In nds_power.py, setup_tables uses get_schemas to provide all schemas. Incase the tpcds data is not generated using this tool , one of the folder dbgen_version would not be present.
This causes an error unless the following code block is commented:
https://github.com/NVIDIA/spark-rapids-benchmarks/blob/952444f89b8384201859aa094d2f50b8a24ba22e/nds/nds_transcode.py#L66-L71
It would be nice to avoid this constraint so that other datasources can be used as well.

Thanks @parthosa for the feedback. I'm new to TPC-DS and this repo is intended only for the official TPC-DS run originally.
I take a look at this dbgen_version table, looks like it's only used to show the meta info for "data gen", not used in queries. I'm fine to remove it.
Btw, could you share what "other Datasources" could be except the official tool?

There could be other ways to generate the TPCDS data that does not create the dbgen_version folder. Since this folder is not used in queries, it should not cause hurdle while running queries that use data from these other ways.
I was using an S3 bucket having TPCDS data that did not have the dbgen_version folder.


After updating Vtube Studio, the software stopped working and the icon disappeared

After updating Vtube Studio today, the logo disappeared and the software is no longer functioning properly. what should I do?

Also having this issue

Go to <Your VTube Studio Folder>\BepInEx\config and open the BepInEx.cfg file in notepad. Search for HideManagerGameObject and change false to true then save the file. The line should look like this HideManagerGameObject = true.

That worked perfectly, thank you!
On Wed, Jul 24, 2024 at 1:26 PM wtfblub @.***> wrote:

Go to \BepInEx\config and open the BepInEx.cfg
file in notepad. Search for HideManagerGameObject and change false to true
then save the file. The line should look like this HideManagerGameObject
= true.
—
Reply to this email directly, view it on GitHub
https://github.com/wtfblub/Trashy/issues/10#issuecomment-2248653750, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AYN43A5WHNJESHLHGDXRUPDZN7WWRAVCNFSM6AAAAABLARMVDCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBYGY2TGNZVGA
.
You are receiving this because you commented.Message ID:
@.***>


Problems integrating Helpdesk html with project

Hi, I'm having some trouble integrating helpdesk's html with my project. I'd like to add my project's sidebar to helpdesk and edit the header. For some reason though, the HTML of helpdesk in my project doesn't match the demo's HTML or the code in the GitHub repo, so I'm at a bit of a loss. I'm somewhat new to Django apps (helpdesk was added by another member of my team), so I could be missing something, but could someone point me in the right direction on how to get this to work?
Unedited, my page looks like this:

I can add some header edits and helpdesk's own sidebar code to files named navigation.html and base.html. The result looks like this:

The HTML appears to be wholly different from helpdesk's, and I'm not sure why, or where it's coming from.

Any pointers on where the issue might be coming from would be much appreciated!

Offhand, I'd guess you might be mixing the version 0.2 and version 0.3 html
templates. The demo website uses the 0.3 version beta, check out:
http://django-helpdesk-demo.herokuapp.com/ and see if that's closer to what
you're seeing in terms of the sidebar to confirm. 0.3 hasn't been
officially release yet but is in a beta state. it's what you'd see in the
master branch.
On Wed, Apr 28, 2021 at 2:06 PM ebeers-png @.***> wrote:

Hi, I'm having some trouble integrating helpdesk's html with my project.
I'd like to add my project's sidebar to helpdesk and edit the header. For
some reason though, the HTML of helpdesk in my project doesn't match the
demo's HTML or the code in the GitHub repo, so I'm at a bit of a loss. I'm
somewhat new to Django apps (helpdesk was added by another member of my
team), so I could be missing something, but could someone point me in the
right direction on how to get this to work?
Unedited, my page looks like this:
[image: image]
https://user-images.githubusercontent.com/53191386/116450648-8a28fe80-a829-11eb-8223-31903ef5b476.png
I can add some header edits and helpdesk's own sidebar code to files named
navigation.html and base.html. The result looks like this:
[image: image]
https://user-images.githubusercontent.com/53191386/116451200-310d9a80-a82a-11eb-84b0-c7c12d98aee7.png
The HTML appears to be wholly different from helpdesk's, and I'm not sure
why, or where it's coming from.
[image: image]
https://user-images.githubusercontent.com/53191386/116451406-6c0fce00-a82a-11eb-9ea4-ff8d4741ce28.png
Any pointers on where the issue might be coming from would be much
appreciated!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/django-helpdesk/django-helpdesk/issues/960, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEKHU7C2BXOCT5X7YGSTU3TLBFDBANCNFSM43XSIIVQ
.

Yup, you're right! It looks like I was searching GitHub in the wrong places for the 0.2 code that we're using. Hopefully it'll be smooth sailing from here - thanks so much for the quick response!


load_model('path_to_model') not working

While trying to load model using load_model(), I am getting this error. ValueError: Unknown initializer: EfficientConv2DKernelInitializer
I followed the steps written on your readme.

While trying to load model using load_model(), I am getting this error. ValueError: Unknown initializer: EfficientConv2DKernelInitializer
I followed the steps written on your readme.

If you update the lib a few days ago installing it from source, please rool back, they are not compatible.
Try pip install -U efficientnet==0.0.4 and try to load model again.

Thank you, it worked. I uninstalled the latest version and installed it again using the command you mentioned above.

Hi,
I was installed efficientnet==0.0.4, but got same error.
tensorflow 1.12.0
keras 2.2.4
please help , thanks

Hi,
I was installed efficientnet==0.0.4, but got same error.
tensorflow 1.12.0
keras 2.2.4
please help , thanks

@Eton try this
from keras.models import load_model
import efficientnet
_ = efficientnet.model.EfficientNetB5() #which ever architecture you trained on, I used B5
model = load_model('model/model_weights_efficient.h5')

What has happend to version 0.0.4?
When I run RUN pip install -U git+https://github.com/qubvel/efficientnet==0.0.4
i get:
ERROR: Command errored out with exit status 128: git clone -q https://github.com/qubvel/efficientnet==0.0.4 'C:\Users<..>\AppData\Local\Temp\pip-req-build-l4nz2h8r' Check the logs for full command output.

I think the command should be: pip install -U<EMAIL_ADDRESS>


[Feature Request] Quilt Support

I and I would imagine many others would like Quilt Mod Loader support to be added when creating modpacks.

I've already implemented this, just waiting for Quilt's beta release to push the code :)


Tokenizing

Need to research and implement tokenizing of PBASIC code.
Options include:

Use emscripten to compile tokenizer to JS
Rewrite the tokenizer in JS
Send files to a server that tokenizes using C tokenizer

@PropGit can you include David in on this thread?  I'd like to see his steps and results related to emscripten.

I prompted David to jump in here.
I should note that there's a fourth possible option that David mentioned; A JavaScript-based x86 emulator/VM.
In an earlier conversation, David said:
There is a very simple open-source implementation here: https://github.com/copy/v86.
Also, someone wrote a JavaScript program to interpret the assembly mnemonics: http://timhatch.com/projects/jsassembly/index.html and it didn't take much.  (Click view source after opening that last link.)
As far as I know the JavaScript VM is interpreting at least a modern 32-bit instruction set.  It is really fast too, it only takes 60 seconds to boot Linux: http://bellard.org/jslinux/.  (That implementation uses an i586 Linux kernel).
The last link is a closed-source project unrelated to the Github implimentation I linked to earlier.  The one on Github has a bootable Linux system here: http://copy.sh/v86/?profile=linux26 and several other operating systems here: http://copy.sh/v86/.  It looks like at least two people have written a Linux-booting x86 VM in JavaScript.  The closed source project has a page about it: http://bellard.org/jslinux/tech.html that mentions that "the x86 emulator can be modified to provide an API to load x86 dynamic libraries and to provide a js-ctypes like API to call the C/C++ functions from javascript."  Also, on the closed-source implementation, try running "time tcc -o hello hello.c" then "./hello".

For some background on Emscripten:
Most multi-language compilers, like GCC and LLVM, compile from a given language to an intermediary language then from the intermediary language to the target architecture.  Emscripten compiles the input code using the LLVM compiler, but instead of providing a target architecture, it grabs the compiled intermediary language, (also called LLVM) and wraps a JavaScript interpreter around it.
The performance and complexity would be on par with using one of the aforementioned x86 interpreter VMs with our existing x86 binaries.  There is more much support available for Emscripten than the x86 interpreters, so it would likely be the easier route of the VM options.
In neither case can you modify the code once it has been compiled.  Performance probably won't be a problem, but a native JavaScript translation of the code will likely be several times faster.
My preferences between our various options are as follows:

Translate the tokenizer code from C to JavaScript
Use an Emscripten VM to interpret the existing C code
Use an x86 interpreter with existing binaries
Send the files to a server for remote compilation

I would prefer we not go with the last option, as it would take a significant investment on our part to create the remote compile system and to keep it secure, it could be a significant liability if anything goes wrong on our end, it will require constant network uptime for us and the user, it creates a poor user experience, and it can be difficult to use behind a firewall.
— David Carrier
Parallax Inc.

TLDR we have a working but not perfect prototype but were blocked on https://github.com/parallaxinc/ChromeIDE/issues/34
Last week wewere able to run emscripten against the tokenizer.cpp file and generate javascript. Full details in this gist https://gist.github.com/jacobrosenthal/92889db5223891a985a8
However there appears to be no official support for passing a struct back. I'm faking it by passing a big uint8 array in but the memory locations are a little odd on the return trip. Probably because of struct packing http://www.catb.org/esr/structure-packing/
This makes me think my solution wouldnt build consistently on 100% of architectures -- which is probably why theres no official support. That said, nobody else NEEDS to build it once you release it.
@phated has work he can publish on how to add the stampapp example to the build. We could use that to pull out primitive data structures like just the resulting code, and safely pass that every time. @PropGit perhaps we can discuss whats needed or have your team decide on a conversion of that struct to a primitive type like an array or array of arrays?
We noticed another minor problem in the Version command in javascript. I was working off version 123 which is stored as a uint8t and returned properly to javascript, @phated was working on 130 which returned as -126, presumably rolling over. Maybe needs to be filed with emscripten people?
Also @PropGit is there a struct member for size of the eeprom block? When programming Im currently just pulling out all 2048 bytes from memory but bs2 is unhappy to be sent 0s.

Doing some more digging today, we found the figure on page 7 of the "Using the PBASIC Tokenizer Library v1.2.pdf" that outlines the struct member alignment.  Based on this, we see 5 bytes of spacing in the first 11 bytes of the memory structure.  We've come up with the following JavaScript code to demonstrate the spacing is correct:
var buf = new Buffer(result);
console.log('succeeded', buf[0]);
console.log('error', Module.Pointer_stringify(buf[4] + (buf[5]<<8) + (buf[6]<<16) + (buf[7]<<24)));
console.log('DebugFlag', buf[8]);
console.log('TargetModule', buf[9]);
console.log('TargetStart', buf[12] + (buf[13]<<8) + (buf[14]<<16) + (buf[15]<<24));

Given this and the fact that the struct elements are 6500 bytes but the full struct is 6508 bytes, we believe the last 3 bytes of spacing are after PacketCount (a one byte field).  Is this correct @PropGit??  If so, I believe we have enough information to change the goal from server compiling to in-browser compilation. Let's have a call about this tomorrow.

I'm faking it by passing a big uint8 array in but the memory locations are a little odd on the return trip.

The tokenizer itself just needs the caller to pass it an address of a long-aligned (4-byte aligned) contiguous memory block (of proper size in bytes) that it can write to.  The tokenizer will write to it based on the designated structure element positions (as indicated in the document) and with byte padding in-between some structure elements as needed to ensure the next logical data element is aligned on a 4-byte boundary if, and only if, it's element(s) unit size is greater than 1.  As long as you can ensure the memory is long-aligned (multiple-of-4 address) and is the proper size (rounded up in bytes to fill a multiple of 4), it should work on every platform that it is compiled on.

We noticed another minor problem in the Version command in javascript. I was working off version 123 which is stored as a uint8t and returned properly to javascript, @phated was working on 130 which returned as -126, presumably rolling over. Maybe needs to be filed with emscripten people?

This is concerning.  Seems like an emscripten problem.

Also @PropGit is there a struct member for size of the eeprom block? When programming Im currently just pulling out all 2048 bytes from memory but bs2 is unhappy to be sent 0s.

The EEPROM block is only intended for generating an memory map (a visual for the user), in which case, all 2048 bytes are necessary always.  It's the PacketBuffer you're after.  The PacketCount field just before the PacketBuffer indicates the number of 18-byte packets to read from the PacketBuffer (starting at element 0).  Each 18-byte packet is exactly what needs to be sent to the BASIC Stamp; they include the EEPROM block number and checksum values as well as 16-bytes of data.

Given this and the fact that the struct elements are 6500 bytes but the full struct is 6508 bytes, we believe the last 3 bytes of spacing are after PacketCount (a one byte field). Is this correct @PropGit??

Well, I think you're almost right.  Because VarCounts is long-aligned, PacketCount is long-aligned, but PacketBuffer is not long-aligned (because it's a byte array)... but, it's a multiple of 4 bytes in length, leaving it 3 bytes short of filling the last long of the buffer.  Thus, PacketBuffer element 0 appears immediately after the PacketCount byte, but after PacketBuffer element 2303, there are 3 more bytes needed to pad to the next long-aligned memory location.  So, if the memory you're allocating is based on a unit of 4 bytes in size, it would naturally end up 6508 bytes in length.  If the memory is allocated as units of bytes, it can be 6505 bytes in length; but you must ensure the buffer is somehow long-aligned so that you can access the character pointers and ints that make up parts of the structure.

If so, I believe we have enough information to change the goal from server compiling to in-browser compilation.

!!!!!! REALLY ?!?!?  That's awesome!  Using Emscripten-ized PBASIC Tokenizer?  Wow!  That's so fantastic!

Let's have a call about this tomorrow.

Yes, let's have a call about it - unless I just gave you all the answers you need to proceed.  I think I'll be available until as late as 3 pm PT.

I forgot to say that there's a built-in function for testing the structure alignment (and field alignment), called TestRecAlignment.  It's mentioned on after the table on page 7, and described in more detail on pages 9 and 10.  You should call that function and then verify that the data returned in the buffer you delivered matches what's indicated in the table on page 10.

We noticed another minor problem in the Version command in javascript. I was working off version 123 which is stored as a uint8t and returned properly to javascript, @phated was working on 130 which returned as -126, presumably rolling over. Maybe needs to be filed with emscripten people?

This is concerning. Seems like an emscripten problem.

Wait... maybe not.  I looked at how Version is defined.  I wrote in the document that Version returns an int.  But in the code it's written as STDAPI which is defined in the header as uint8_t if being compiled on Linux.  I don't know why it's defined like this (and can't remember if I did it or if a developer who was helping me with porting issues did it... but may have been me)... now I think it should be a 32-bit unsigned integer value.
Maybe the problem isn't really emscripten?

It's the PacketBuffer you're after

You're right, I just misnamed it because I wasnt sure of the memory mapping at first.

Given this and the fact that the struct elements are 6500 bytes but the full struct is 6508 bytes, we believe the last 3 bytes of spacing are after PacketCount (a one byte field). Is this correct @PropGit??

Now that weve mapped it all up its looking like the last 3 bytes of padding are actually AFTER PacketBuffer. The tests should confirm this.
If you like, you can take a look at the branch Im working on in https://github.com/parallaxinc/PBASIC-Tokenizer/tree/emscripten

I forgot to say that there's a built-in function for testing the structure alignment

My current feeling is theres isn't a use case for writing anything around that function atm as we need to write tests to catch any errors in our code and that should catch anything at that level as well.

Maybe the problem isn't really emscripten?

Seems like youd probably like it to be bigger than uint8_t so you have some version headroom, but uint8_t still properly holds up to 255 so it seems like emscripten.

@PropGit TestRecAlignment turned out to be a very nice test actually. I THINK the only thing missing currently is the Source-to-Token-Reference Structure and the version problem. Ive added them both as issues to that repo to be dealt with time permitting.

This has been merged to master


create comments model

const blogSchema = new Schema({
authors: [
{
type: String,
},
],
description: {
type: String,
required: true,
},
},
link: {
type: String,
},
title: {
type: String,
required: true,
},
});
refs userID for author


Rename terraboard; name already in use for Terraform UI

I strongly suggest naming this to avoid naming conflicts with another Terraform UI, http://github.com/camptocamp/terraboard. This current naming will cause a lot of confusion.

Can you suggest a new name?


Are volunteering positions eligible for the job board?

In the PyBaMM team we are about to advertise some volunteering positions (i.e. unpaid) for a fundraising lead, a communications lead and a community manager. Are unpaid positions eligible to be posted in the board or is it only for paid positions?
More details on the positions: https://github.com/pybamm-team/pybamm.org/blob/10fed6efb535aa75d1597546b4d29c77cc9b9f3c/content/news/2024/07/15/role-adverts.md

As long as it explicitly states that it is a volunteer position, it should be fine.


Update app template style

When a developer starts edge for the first time, they will inevitably see the default template. I think that the latter should serve the purpose of a book cover or a business card in that it should convey values like robustness, code organization, good design, and everything that edge is. For this reason, I feel that having a template that is not just a centered div with some heavy box shadows, suggests that edge is well reasoned, without leaving anything out ... In other words, it just gives the developer a better impression of edge. Of course this comes from my personal experience with edge and you might not share any of those ideas 💡

Why I think the changes are

better.
Current design:

Thick box shadows
Page doesn't look as clean as it could
Small width for main container
Adding any html element without touching the template makes it look squashed
Centered main container
It gives me the feel of a popup rather than a web app

Proposed design:

Minimal box shadows
Cleaner design
-Bigger container width
More room for html elements
Navbar
To give a website feel (see React default app for example)


Provide convenience scripts to verify TAP/metadata file archive is 'good'

To alleviate any manual effort, we should provide some convenience/utility scripts that will verify the AQAvit results so that vendors can check that their results are good (and binaries ready to publish).
The features can include:

basic checking of TAP files (if any 'not ok' statements present, report on the target name)
checking that the release info matches the release file of the binary being published
check that the SHAs look 'correct'

This can be done in stages.  There can also be enhancements to the Jenkins scripts to gather the Rerun in Grinder with failed targets links for jobs with non-passing status.

There are many different pre-built, open-source consumers we can use as the basis for this work.
As a quick and dirty initial start, one can use the prove utility that comes with Perl.  In order to assess failures in TAP files in current working directory, one could run:
 prove -f -t *.tap
or for verbose mode and searching recursively in a directory called temurin, then this:
prove -v -t *.tap -r ./temurin
If one wanted a pretty

Several of features that we may want to include:

check java version is expected in all TAP files
list all level.group per platform to ensure all AQAvit verification required targets are included. This should handle different folder structures and parallelization.
list num of passed tests in each level.group to ensure the minimal number of passed tests
provide the ability to remove unwanted TAP files. Vendors may run extra test targets and they may not want to include them in the results.zip

Part of this has been delivered via #4277 (thanks Lan!).  We can add additional checks into the newly added aqaTap.sh file.


HTML Project Demos Describe Themselves as Such

Is your feature request related to a problem? Please describe.
Some learners seem to think that they need to replicate the demo projects for the Responsive Web Design certification exactly.
Describe the solution you'd like
Perhaps we should add some text to the demo projects like
DEMO - YOU SHOULD NOT EXACTLY REPLICATE THIS DEMO PROJECT
or
DEMO - YOU SHOULD MAKE A SIMILAR PROJECT WITH DIFFERENT CONTENT
Describe alternatives you've considered
N/A
Additional context
No response

I prefer the simpler language:
DEMO - YOU SHOULD MAKE A SIMILAR PROJECT WITH DIFFERENT CONTENT

Sounds good to me. I think wordsmithing to something simple that communicates the intent is ideal, especially with how multilingual our audience is.

yeah, I agree with the change.
Hopefully that will clear up the misunderstanding on having to replicate the projects

I think
DEMO : YOU SHOULD MAKE A SIMILAR PROJECT WITH DIFFERENT CONTENT

Can we not have all caps, we can make it bold if needed.

Yes, we can do that.
On Sat, 5 Aug 2023, 19:33 Muhammed Mustafa, @.***>
wrote:

Can we not have all caps, we can make it bold if needed.
—
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/51152#issuecomment-1666514739,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAB6NQZUSSDGSHQYFVCJAJ3XTZHDHANCNFSM6AAAAAA3ERN7RI
.
You are receiving this because you commented.Message ID:
@.***>

I thought I saw something similar to that warning on those projects.
There's already
Fulfill the user stories and pass all the tests below to complete this project. Give it your own personal style. Happy Coding!
I'm surprised campers are getting confused by this. I mean it's nice to have extra warnings but what was the original malfunction?

The 'original malfunction' is that people still regularly exactly clone the projects. That makes it harder to tell at a glance between copying and making their own projects.

Sorry Jeremy. I wanted to know why campers thought it was acceptable to completely clone the projects. I wanted to know if it was them not reading that instruction or if the wording was too ambiguous; and to try to offer my own two cents on a possible solution.

Ah, I misinterpreted 'original malfunction' as 'problem' but I think perhaps you are meaning what I would call 'root cause'?
I think the message isn't clear and direct and obvious enough.

Yes

Thanks, all. @jeremylt is correct here. The main thing is people don't read.
We should go with this exact notice. Its terse, clear, and specific:
DEMO - YOU SHOULD MAKE A SIMILAR PROJECT WITH DIFFERENT CONTENT

I mostly used all caps because I forget that bold is an option. I'm all for any emphasis technique.

Just a curiosity @jeremylt (or anyone), are we sure campers are confused and not deliberately cloning the project?

I have seen a lot of campers who think that they need to duplicate the project, not create their own.

Its not super infrequent (and usually with someone who appears to have english as a second language) that I run into a camper that's surprised when I tell them they can make the project about anything they want and it only has to match functional requirements.

I agree

I can confirm! When I followed the course I had the impression that we should try replicate the demo. I am surprised to find out😲

My first thought is to always replicate the project. If I can make a suggestion, maybe something that combines both sentences.
Try not to replicate this demo project and create a similar project with your own content.

I'm always an advocate for smaller, simpler sentences whenever possible.

Its not super infrequent (and usually with someone who appears to have english as a second language) that I run into a camper that's surprised when I tell them they can make the project about anything they want and it only has to match functional requirements.

I was under the impression that we should try to replicate the demo, but I’m surprised to learn that we shouldn’t..
if I can make a suggestion then :-
No need to copy, Make a similar project, Get Creative!

I think actually want to say 'do not copy' instead of 'sometimes copy'

I am going to go ahead and close this since it looks like a PR was merged a while back to resolve the issue.


My game isn't working

I don't know what's happening. Could someone fix it?
https://codesandbox.io/s/physics-playground-b4cs6?file=/index.html

I fixed it!


refactor(headless-client): vendor uptime_lib

This does the same thing as #5621 without removing the library, since it will now compile against whatever version of windows we need
We could do the same with hostname, either vendor or ask upstream to bump deps, and then windows 0.52.0 should be gone.

Do we really need to know the system uptime for this functionality? To be honest, I'd prefer a different design to vendoring.
What is the problem that we are trying to solve here?

I'd like to know whether the IPC service / GUI started near the same time Windows started or later on. That would tell us whether it likely auto-started (maybe intended behavior or maybe they rebooted to fix a problem in the previous run) or was manually started (maybe it was just upgraded or installed, or maybe they had to manually restart the service)
Since the IPC service hang is fixed, we could get rid of all the heartbeat stuff, but I'd like to log system uptime at least once at process start.

I'd like to know whether the IPC service / GUI started near the same time Windows started or later on. That would tell us whether it likely auto-started (maybe intended behavior or maybe they rebooted to fix a problem in the previous run) or was manually started (maybe it was just upgraded or installed, or maybe they had to manually restart the service)
Since the IPC service hang is fixed, we could get rid of all the heartbeat stuff, but I'd like to log system uptime at least once at process start.

Logging a message on startup does the same thing no? You'll see the timestamp in the log as to when it started. I don't really understand what system uptime has to do with this.

I want to know whether the IPC service auto-started or not. If the logs show, e.g. that one instance did not shut down cleanly, and the next instance started up at a high system uptime, that means the process crashed and was restarted.
If we did not know the system uptime in that situation, it would look the same as if the computer lost power and the whole system restarted

I want to know whether the IPC service auto-started or not. If the logs show, e.g. that one instance did not shut down cleanly, and the next instance started up at a high system uptime, that means the process crashed and was restarted.
If we did not know the system uptime in that situation, it would look the same as if the computer lost power and the whole system restarted

Okay that makes sense, thanks for explaining :)


initialize_with for create_list

I'm looking for a way to customize the way lists of objects are created. For example, if I want to create 5 objects, and I have a service class called SomeService that will create n of those objects, how would I override the create_list functionality to call that service?
There's intitialize_with, which works for creating a single item in a custom way:
FactoryGirl.define do
  factory :some_custom_object, class: String do
    skip_create

    initialize_with do
       new('blah')
    end
  end
end

But that doesn't allow me to bulk-create a bunch Here's an example of what I'm trying to do:
FactoryGirl.define do
  factory :some_custom_object, class: String do
    skip_create

    initialize_with_for_create_list do
       SomeService.run('blah', n)
    end
  end
end

Sorry if something like this already exists. I searched around and couldn't find anything.

I know it has been a while, but I don't believe anything like this exists. Just out of curiosity, it you already have a service for creating lists of objects, why do you need to use create_list?

@composerinteralia Great question! It's been a while since I opened this, and I honestly don't even remember the initial use-case anymore. I'm going to go ahead and close this.


Update Prefabs for ROS 2 Frame Editor Component

Update prefabs and code to use the ROS2FrameEditorComponent (https://github.com/o3de/o3de-extras/pull/593).

@arturkamieniecki what is the difference between #5 and #6 ?

Closing in favor https://github.com/RobotecAI/o3de-otto-robots-gem/pull/7


[BUG]

OpenIM Server Version
openim/openim-server:release-v3.8
Operating System and CPU Architecture
Linux (AMD)
Deployment Method
Docker Deployment
Bug Description and Steps to Reproduce
docker方式部署成功，服务启动正常。客户端请
求接口http://localhost:10008/chat/client_config/get  报404 page not found

Screenshots Link

The correct route should be: /client_config/get


Jade templates included file wrong spacing

How do you include a Jade (or any template language) with this library and keep the correct white space? File is being included correctly, but white space is wrong.

file-include does not response for your indents,
You can save firstline empty and then use indent based of the parent template. I don't know is it good for Jade.

I'll add a test for this, white space should be included correctly.

@coderhaoxin It's correctly, but parent indents are not inherited.

but parent indents are not inherited.

yes, it should

Need to pass third argument in include with parent information.
It could be useful for error tracing. I will realize this in MT.

@TrySound +1

@stoplion I solved a similar problem with Jade, but it was messy. http://stackoverflow.com/a/29043553/845717.
@TrySound should I open an issue regarding gulp-file-include's ability to support multiple includes on the same line? If my slim templates compile the @@ directives to the following HTML can we make it "just work"?
@@include('./dist/templates/_header.html')@@include('./dist/templates/_nav.html')

I think this must work in Meta-templater. I will test this ASAP

@bradgreens It's works!

@coderhaoxin let's make a list of meta-templater requirements for using it as dependancy

@bradgreens I made some fixes in parser. Try it again, please.

@stoplion I added indent option. You can try it with jade.


Added informaiton about layouts

Love the project, saved me a lot of time!
Had some issues with my multi-page application, so thought it would help by sharing my discoveries.

@modbender So what do you think?

Hmm, it works for both SPA and multi but sure why not.


Fix the unit test testUseOfVendedBinaryTool to run on Apple Silicon

This failed because the test fixture (which actually uses a script as a build tool) didn't list arm64-apple-macosx in its list of triples.
Also rename the subpath of the tool to not suggest that the triple has to be part of the subpath.
rdar://76421504

@swift-ci please smoke test


Remove Swift PM Artifacts to avoid Generated Schemes in Xcode

All Swift Packages vended by Apple do not currently check in their .swiftpm directories. The .swiftpm directory is an artifact that gets created when a user opens a package within Xcode. By checking in the .swiftpm directory, we risk having a backwards incompatible change in the future affect the auto-generation of .xcschemes in Xcode when adding this package as a dependency in another Xcode project.
Currently, when adding swift-collections as a package dependency within an Xcode project, Xcode uses the .xcscheme files located in the package's .swiftpm/xcode/xcshareddata directory as an indication that it should surface those as schemes within a top level project. This clutters the scheme list for an Xcode project when a user is most likely to prefer only seeing targets/schemes they're actively working on, not dependency targets/schemes.
TL;DR:
This pull request removes the whole .swiftpm directory and adds .swiftpm to the .gitignore list to prevent unneeded xcscheme generation within a top level Xcode project that pulls in swift-collections as a dependency.
Checklist

[x] I've read the Contribution Guidelines
[x] My contributions are licensed under the Swift license.
[x] I've followed the coding style of the rest of the project.
[ ] I've added tests covering all new code paths my change adds to the project (if appropriate).
[ ] I've added benchmarks covering new functionality (if appropriate).
[x] I've verified that my change does not break any existing tests or introduce unexplained benchmark regressions.
[ ] I've updated the documentation if necessary.

Hm; I was relying on these schemes to customize test execution when this package is tested under Xcode. On the other hand, if these schemes are polluting dependent projects, then that seems like a pretty convincing argument for removing them.
Cc @neonichu What's SwiftPM's official guidance here?

@lorentey I'm so sorry about that! I know how important it is to have an environment that's easy to work with and feels turn key when opening the package. 😢
The good thing about xcschemes is that you can still use them locally on your end as long as you copy the schemes over before pulling from main. Any subsequent uses of it will still stick around since it's in gitignore so you can still have your custom test execution settings!
Also, hi friend @neonichu ❤

I don't know of any official guidance for this, but personally, I would not check in any schemes for a package, for the reasons @hectormatos2011 listed.
I think you can create per-user versions of the schemes by unchecking "Shared" which might be a compromise to still getting whatever you need for local development.

@neonichu you definitely can create per-user schemes. I just tested it on my machine and it sticks around even after git pull. @lorentey if you uncheck "Shared" in the Manage Schemes window, it will store those schemes in a separate folder that won't get deleted after pulling this branch

Fair enough. Having these schemes show up in dependents is reason enough to remove these. We have a dev workspace in the Utils folder that can hold these schemes just as well, without messing up downstream configurations.

@swift-ci test


Possible duplicate templates?

So we have both:
https://github.com/projecthydra-labs/hyrax/blob/master/app/views/hyrax/base/_attributes.html.erb
and
https://github.com/projecthydra-labs/hyrax/blob/master/app/views/hyrax/base/_metadata.html.erb
which share an awful lot of code. Do we still need both? If so, might it make sense to extract common bits into a new partial so we have fewer places to update code?
@projecthydra-labs/hyrax-code-reviewers

Yes, I think we can remove the former.  I only see it used by app/views/hyrax/single_use_links_viewer/show.html.erb

Closing this, as these now have substantive differences. The shared code seems to be in attribute_rows.


Add plain_acl_bak.xml for reset at test case start.

Add plain_acl_bak.xml for reset at test case start.

#4624


feat: support logout cmd

What kind of change does this PR introduce?
feature
What is the current behavior?
1688

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

19 of 35 (54.29%)  changed or added relevant lines in 3 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 58.139%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/logout/logout.go
11
13
84.62%

cmd/logout.go
3
10
30.0%

internal/utils/access_token.go
5
12
41.67%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.31%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
5797

Relevant Lines:
9971

💛  - Coveralls

Lovely, thanks @S96EA. Added few changes myself so you don't have to go back and forth anymore :)
Please apply those to this PR and we're good to go - https://gist.github.com/kamilogorek/cf2cacf7ff0cce8b91d65734df776fad

Lovely, thanks @S96EA. Added few changes myself so you don't have to go back and forth anymore :) Please apply those to this PR and we're good to go - https://gist.github.com/kamilogorek/cf2cacf7ff0cce8b91d65734df776fad
(changed default to false as its destructive operation, fixed actual/expected order in tests, simplified error check and used separate error value for not logged in)

I'm sincerely appreciative of your support. Thanks again.

LGTM! @sweatybridge feel free to merge anytime.


Feature request for support example attribute for elements

Dear Developer(s),
I want to ask you to develop PHPSwaggerGen to support example attribute for swagger.
What I mean?

responses:
200:
description: 'Succes data request'
schema:
properties:
code:
type: string
example: "SUCCESS"
message:
type: string
example: "Your request was successfully!"

More about this attribute: https://swagger.io/docs/specification/adding-examples/
Would be nice to have this feature too. Thank you!

I think that would be quite doable. Will look into it somewhere in the coming weeks.
Do you know if the SwaggerUI can handle examples.

Thank you very much! SwaggerUI can handle examples.
Check here in the example:
http://petstore.swagger.io/?_ga=2.136686704.1540059357.1509039902-598612444.1509039902
And here is the json for that: http://petstore.swagger.io/v2/swagger.json
The "example" shows up in the UI when you see eg. the "Add a new pet to the store" body options object.

I've added support for example to all types and to the Response context.
Note that PHPSwaggerGen bases off the 2.0 spec. Relevant page on examples is here: https://swagger.io/docs/specification/2-0/adding-examples/.
The page you linked to was for OAS3. I may support 3.0 in the future, but currently have no plans to do so.

Thank you very much!


Using 'negative numbers' to specify how many versions behind the current version to notify not working correctly.

The comment in the options on this page says, you can use "Negative numbers [to] specify how much versions behind current version to notify."
However, this doesn't appear to be working consistently - I've been using browserstack to test this and I have set -1 for all browsers. So technically it should show the message on any browser that is 1 version behind the current version.
In reality, here are the results I have found:
Current Firefox version : 55
Doesn't notify on version: 54
Shows notification on version: 53
Current Chrome version: 61
Doesn't notify on version: 60
Doesn't notify on version: 59
Shows notification on version: 58
Current Safari version: 10.1
Shows notification on version: 9.1

As you already found out the issue is that the current versions where not updated in the last weeks. I usually do this once a month.
Give me some time to upload it. It'll be done today.

Thanks for the quick response 👍

Sorry, the version number fix is not yet online. I have some trouble updating it. I hope that it works tomorrow.

Using 'negative numbers' still isn't working as it should. I thought updating the the browsers.json file in https://github.com/browser-update/browser-update/pull/361 was going to fix this? Is there something else causing an issue?

I know! Sorry.
I fixed the stuff also in the other places where it is needed.
However, I'm stuggling with clearing our sever's cache.
If you are using the npm version it should work as expected right now.
I'm hoping to get it working today.

I'm not using the NPM version unfortunately. Good luck with it 👍

Finally! It should work now! (Make sure to clear your cash if it does not when you're testing)

Tested on FF, IE, Chrome and Safari, all LGTM 👍  Thanks for this!


Move PageReader to embulk-api

PageReader is now considered as a part of Embulk API. Although calling its constructor has been the only way to construct PageReader from plugins, PageReader should no longer provide its own constructor for plugins in embulk-api. Instead, Exec is going to have Exec.getPageReader.

v0.10: PageReader's constructor is marked as @Deprecated, and Exec.getPageReader is added. But Embulk v0.9-compatible plugins still have to call the constructor because Embulk v0.9 has only the constructor, no Exec.getPageReader.
v0.11: PageReader's constructor will still be kept, but plugins should start using Exec.getPageReader instead.
v1.0: PageReader's constructor would be removed at some early point of v1.0.

Thanks!


Add support for numeric datatypes to featherread and featherwrite

Overview
These changes add support for numeric datatypes to featherread and featherwrite.
Datatypes Support

Unsigned Integer Datatypes: uint8, uint16, uint32, uint64
Signed Integer Datatypes: int8, int16, int32, int64
Floating Point Datatypes: single, double

Validity (Null) Bitmap Support
Although MATLAB does not support missing values for integer types, featherreadmex and featherwritemex are able to round trip the validity bitmap (null bitmap) to/from a Feather file. When reading integer columns from a Feather file which contain missing values, featherread casts the data to double and then sets missing values to NaN.
Metadata Support
Support for the following MATLAB table metadata has been added:

table description (table.Properties.Description)
table variable names (table.Properties.VariableNames)

Tests
Round trip tests for all numeric datatypes have been added in tfeather.m and tfeathermex.m. To run the tests, featherreadmex and featherwritemex must be added to the MATLAB path. See README.md for more instructions on running the code and associated tests.

It looks like all the feedback has been addressed for this code review. I am going to merge this.
Thanks @rdmello, @jeremy-hughes, and @sihuiliu for all of your feedback!


Add Rabbit Escape

Please could you add my game Rabbit Escape ?
It's a puzzle/action game of rescuing rabbits, inspired by Lemmings and Pingus.

I added it. Is it rather beta or rather mature. Version 0.13 seems to indicate rather more beta, but >2000 commits on the repository rather the opposite.

Thank you!  It is stable - the <1.0 version number indicates the fact that we want to implement some UI improvements before we consider it "ready".


add example app using sinatra and pg

Codecov Report
Base: 97.67% // Head: 97.67% // No change to project coverage :thumbsup:

Coverage data is based on head (f078c88) compared to base (228ed6c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #75   +/-   ##
=======================================
  Coverage   97.67%   97.67%
=======================================
  Files          27       27
  Lines         517      517
=======================================
  Hits          505      505
  Misses         12       12

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

needs a rebase, otherwise no blockers, but take a look at my suggestions


Updating for merge/edit

Odd to see this, but merging the pull request (from upstream to current fork).


Update documentdb-sql-query-metrics.md

@mimig1

@rafats - Please open all new PRs in the private repo. Waiting on confirmation from @arramac on the change. Not sure if there's another value other than -1 (and less than 0) that it could be set to.

it can be any value, not just 0 or -1. The behavior is actually explained later in the article.


Added the Introduction to the Intermediate Algorithm Scripting Challenges

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #34401

@RandellDawson Done

@thecodingaviator I am fine with the changes.  @cmccormack What do you think?


[RFC] What would be most valuable for Informal Systems?

Now that all the packages are at least in PR (only tendermint-rs is proving difficult) I am wondering what the greatest value-add besides reproducible builds would be. I think there are a couple nice things that we can get from nix, but its unclear which ones I should prioritize, here are a couple suggestions:

Integration Tests: NixOS Testing Library is a pretty cool project that makes it easy to write integration tests using qemu virtualization. This seems like a great fit for integration testing relayers using gaia to spin up a couple local chains. You can find a nice article on integration testing with VM's using NixOS Testing Library here
Docker Images: Nix Docker Tools provide a really nice way to generate docker images from nix derivations. You get the reproducibility of nix (for the software that you are providing via nix) and you can provide it to devs in the familiar docker format. There could be an image per package, or a single image providing all of them, or a collection of packages that make sense to provide in concert, I would have to talk to someone about what makes the most sense. You can find a nice article on creating docker images with nix here
AMI's: Nix makes it really easy to bake software into AWS ami's, using AWS kind of feels wrong given the values of this community, but its an option if you think it would be useful for relayers!
Development Environments: This one doesn't follow from as clearly from the packaging work that has been happening in this repo, but it might be useful to informal to be able to provide consistent, 1 command development environments for developers at informal. I would be happy to build some of those nix shells in this repo, and then whoever wants to use them could clone the repo and enter a nix shell from there!

If you can think of some other way that we can start making this repo useful, please comment!

Thanks for the initiative! I think the most valuable feature we can deliver is to use Nix for end-to-end testing. I have described it in more details in #15.
As of now, the best tools we have is Gaia Manager and the dev-env scripts, which provide pretty basic functionality in managing test chains. We can potentially write Nix modules that have the same features, but most importantly they need to be as intuitive to use as the current tools so that users are willing to adopt Nix.
For us to officially adopt Nix for end-to-end testing, we will have to convince the others that the solution offered by Nix is much better than alternatives like Docker or even plain Bash. Ideally users should only need to install Nix on a fresh machine and be able to run all end-to-end tests from the get go.
It is also important that users are able to write new chain configurations to be used by the end-to-end tests. It can potentially be challenging if users need to learn the Nix language to edit the configuration. We may also want to investigate whether users can edit the configuration in simpler languages like JSON, YAML, or TOML.

@soareschen I seem to remember that informal configures some of their legal contracts with dhall? Perhaps this tool https://github.com/dhall-lang/dhall-haskell/tree/master/dhall-nix would provide an easier interface for configuration?

The use of Dhall was experimental, and I'm not sure there is widespread use of it yet. A potential issue for Dhall is that the learning curve is still pretty steep, and support for features like recursive types are limited. We may want to try out simpler solutions like YAML first, and then consider switching to more advanced languages like Dhall or Nix when the need arise to configure complicated setups.

It seems that end to end testing is the initial priority. Closing by https://github.com/informalsystems/cosmos.nix/issues/15.


hugo: 0.57.2 -> 0.58.0

Release notes: https://gohugo.io/news/0.58.0-relnotes/

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

@GrahamcOfBorg build hugo


Nist College

Nist College,
Nayabasti Nala Road, Banepa 45210
There are two domains of this college,
1.nistbanepa.edu.np
2.nistcollege.edu.np
both of them are workable, and I have updated both in this pull request. I hope this PR will be merged

@Sandipbgn First stage of verification passed, official domain is nistbanepa.edu.np as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@philipto Thank you for your information. Waiting for final decision

@Sandipbgn Thank you for your patience while waiting for a review and processing your request. Pull request merged.


What are the goals of URL handling in http-types?

Is the goal to provide some URL handling utilities (like extract the path from a bytestring, which http-types already provides) or to eventually provide comprehensive URL handling (e.g. extracting the scheme and fragment from a URL as well)?
Clarifying this would help users understand the difference between http-types and other libraries like uri-bytestring and network-uri.

The goal is to provide only the parsing / generating methods for URIs
needed by a typical HTTP server or client. Non-HTTP URIs are out of scope.
Ian Jeffries<EMAIL_ADDRESS>schrieb am So., 18. Okt. 2015 um
22:46 Uhr:

Is the goal to provide some URL handling utilities (like extract the path
from a bytestring, which http-types already provides) or to eventually
provide comprehensive URL handling (e.g. extracting the scheme and fragment
from a URL as well)?
Clarifying this would help users understand the difference between
http-types and other libraries like uri-bytestring and network-uri.
—
Reply to this email directly or view it on GitHub
https://github.com/aristidb/http-types/issues/61.

Oh, that makes total sense. Thanks, and feel free to close this!


use appropriate find_package calls for HIP

resolves #303
Summary of proposed changes:

change find_package( hip ... calls to find_package( HIP ... to match the FindHIP.cmake file as installed by hip-base.
change export dep from hip to HIP

Thanks for this, @acxz; I'll run this through CI and take a look.

@malcolmroberts sounds good

Lowercase "hip" is the correct name as the cmake config file is named hip-config.cmake. The problem in #303 is most likely due to something else. It may very likely be that the CMAKE_PREFIX_PATH is not set correctly as it as it mentions in the error:
Add the installation prefix of "hip" to CMAKE_PREFIX_PATH or set "hip_DIR"
  to a directory containing one of the above files.  If "hip" provides a
  separate development package or SDK, be sure it has been installed.

Which means you would need to set CMAKE_PREFIX_PATH to /opt/rocm/hip(but it needs to come before /opt/rocm because the symlinking in hip is broken).

ls /opt/rocm/hip/cmake/
FindHIP  FindHIP.cmake  FindROCR.cmake  FindROCT.cmake
But we use "HIP" for the find name convention and it is case sensitive.

@pfultz2 my understanding is that find_package(.) and DEPENDS PACKAGE ... needs to be the same case as the Find<..>.cmake file, which is FindHIP.cmake in our case.
find_package(.) only throws warnings when using a different case name, but DEPENDS PACKAGE ... causes a failure when the find_dependency(hip) line is created in the exported cmake file for rocfft.
Manually renaming the FindHIP.cmake file to Findhip.cmake get's me farther along but still errors out.
The cmake config file being named lowercased hip-.. does not matter.

But we use "HIP" for the find name convention and it is case sensitive.

@TorreZuk I am a bit confused by what you mean here?

My comment followed Paul's so was directed at him, @acxz I was agreeing with you.   Unless CMAKE_FIND_PACKAGE_PREFER_CONFIG is used we use the name  FindHIP.cmake (pasted the ls output) as the name so should use DEPENDS PACKAGE HIP.

my understanding is that find_package(.) and DEPENDS PACKAGE ... needs to be the same case as the Find<..>.cmake file, which is FindHIP.cmake in our case.

This should not be looking for FindHIP.cmake as this doesn't provide the hip::host and hip::device targets which are needed. It should be looking for the hip-config.cmake file.
If you are using FindHIP with find_package(...) then you will probably need to rename it to avoid this conflict.
Ideally, it would be best to have hip-config.cmake provide the functionality in FindHIP.cmake. This would avoid this naming conflict, and user's wouldn't need to copy FindHIP.cmake into their project. However, @mangupta strongly opposed this. So it seems the best option is to rename it when copying it into your project.

This should not be looking for FindHIP.cmake as this doesn't provide the hip::host and hip::device targets which are needed. It should be looking for the hip-config.cmake file.

Can you provide proof with this statement? I strongly believe find_dependency is looking for FindHIP.cmake and this is confirmed when I change the file name to Findhip.cmake and the find_dependecy call works.

If you are using FindHIP with find_package(...) then you will probably need to rename it to avoid this conflict.

I am using the FindHIP.cmake installed by hip-base and adding the location to my CMAKE_MODULE_PATH. Expecting users to change a system level file is not ideal at all.

Ideally, it would be best to have hip-config.cmake provide the functionality in FindHIP.cmake.

Doesn't FindHIP.cmake call hip-config.cmake?

This would avoid this naming conflict, and user's wouldn't need to copy FindHIP.cmake into their project.

What exactly is the naming conflict here? Copying FindHIP.cmake will still not fix the issue, since it needs to be Findhip.cmake to work with how the HIP dependency is currently exported. I am proposing we change our hip calls to HIP so that Findhip.cmake is not needed and upstream FindHIP.cmake can be used as is.

ls /opt/rocm/hip/cmake/
FindHIP FindHIP.cmake FindROCR.cmake FindROCT.cmake
But we use "HIP" for the find name convention and it is case sensitive.

Other teams have used that convention for the Find modules, but since this is only to support upstream projects you can name them whatever you want.(for example ROCR provides hsa-runtime-config.cmake so the find_package and Find module names do not need to match).
However, math libs and miopen need to find use hip-config.cmake because it uses the hip::host and hip::device targets to build with hip. This is why we call with the lowercasefind_package(hip) to also make it clear we are looking for hip-config.cmake and not FindHIP.cmake.

However, math libs and miopen need to find use hip-config.cmake because it uses the hip::host and hip::device targets to build with hip. This is why we call with the lowercasefind_package(hip) to also make it clear we are looking for hip-config.cmake and not FindHIP.cmake.

hmm I see so the lower level issue is that FindHIP.cmake is not exporting all the targets in hip-config.cmake.

Can you provide proof with this statement? I strongly believe find_dependency is looking for FindHIP.cmake and this is confirmed when I change the file name to Findhip.cmake and the find_dependecy call works.

Well first, there is most likely a bug in cmake when one uses an uppercase Find module with a lowercase package config file. However, these cmake files are definitely looking for hip-config.cmake as they need the hip::host and hip::device function.

I am using the FindHIP.cmake installed by hip-base and adding the location to my CMAKE_MODULE_PATH. Expecting users to change a system level file is not ideal at all.

Find modules are to support upstream projects not downstream projects. That is the project has a Find module in its own repo and uses that module, so there is no need to change a system level file.
Find modules are not meant to support downstream projects. Trying to force it to support downstream projects through CMAKE_MODULE_PATH is problematic as it breaks cmake's standard searching mechanism for dependencies(ie CMAKE_PREFIX_PATH or <PackageName>_DIR.

Doesn't FindHIP.cmake invoke the call to hip-config.cmake in the user cmake project?

No it doesn't.

What exactly is the naming conflict here? Copying FindHIP.cmake will still not fix the issue, since it needs to be Findhip.cmake to work with how the HIP dependency is currently exported. I am proposing we change our hip calls to HIP so that Findhip.cmake is not needed and upstream FindHIP.cmake can be used as is.

For projects that need to use special cmake functions like hip_add_library, they should copy FindHIP.cmake into their project and rename it to something like FindHIP2.cmake and then call find_package(HIP2). This way the projects can call find_dependency(hip) without an ambiguity.

hmm I see so the lower level issue is that FindHIP.cmake is not exporting all the targets in hip-config.cmake.

Actually its the reverse. hip-config.cmake needs to include the functionality in FindHIP.cmake. hip-config.cmake is what is available to support downstream find_package(hip).

Gotcha thanks for explaining @pfultz2
Well then this seems to be a user issue then. When calling cmake I (as well as downstream projects like pytorch should ensure that hip-config.cmake is used over FindHIP.cmake. Is that correct?

Well then this seems to be a user issue then. When calling cmake I (as well as downstream projects like pytorch should ensure that hip-config.cmake is used over FindHIP.cmake. Is that correct?

Yes, but some downstream projects(like pytorch) use the hip_add_library cmake functions which currently hip-config.cmake doesn't provide. So we need to update hip-config.cmake to provide this same functionality.

I see, then an issue and corresponding PR should be made over at the HIP repo about this.

Should this issue be close?

We have separately added these changes; I am going to close this PR.

Thanks @malcolmroberts !


Add contributing page codifying C++ components development

See https://trello.com/c/8l4Rgg6a/547-codify-c-components-development-process
Following the release of OME Files, this page documents the current guidelines for the C++ repositories including source control management, build system and development lifecycle.

NB: the last section i.e. bumping the release to the next patch number immediately after release has not been enforced so far. As this was discussed as part of the OME Files 0.5.0 release process and might be our best solution so far (see also https://gitlab.kitware.com/cmake/cmake/issues/16716), let us agree on the current statu quo and review it when needed.

No objections to anything pushed to this state (i.e. 2554acfcb5d2f650d1f7c834eac2c1f6971d6ed1) but of course, in releasing, I didn't few if any of the mentioned steps myself.

@manics pushed 3 commits which should address all your comments.


[camera] Provide default exposure point if its null

Some devices, particularly veux and peux misbehave if [CONTROL_AE_REGIONS] are set with null value.
We should instead return the default value of the exposure point.
Fix attempt for https://github.com/flutter/flutter/issues/105200
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[ ] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Thank you for your contribution. I'm going to close this PR for now since there are outstanding comments, just to get this off our PR review queue. Please don't hesitate to submit a new PR if you have the time to address the review comments. Thanks!


Map Fixer does not fix service layer

If the URL to a web service moves, map fixer cannot fix it.

https://github.com/AKROGIS/MapFixer/issues/4


Added new rule to alert on traffic mirroring events in AWS cloudtrail; tests included and pack updated

…; includes tests and updated aws pack
Background

Changes

Testing

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


Mup with nginx doesnt load the app

Hi,
I created a nginx sites-enabled conf file as
server {
listen                *:80;
server_name          todo.cefware.com
access_log            /var/log/nginx/todo.cefware.com.access.log;
error_log             /var/log/nginx/todo.cefware.com.error.log;
location / {
proxy_pass http://<IP_ADDRESS>:3001;
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection 'upgrade';
proxy_set_header X-Forwarded-For $remote_addr;
}
}
After that i created a symlink to /Sites-enabled forlder
when i go to http://todo.cefware.com:3001/ works fine
but doesn'tload the app in http://todo.cefware.com

Thanks i solved it the issue was due to semicolon i have created a bash script which will run automatically point to the application...


how to push to download

Operating system and version:
nvm debug output:

nvm ls output:

How did you install nvm?

What steps did you perform?
What happened?
What did you expect to happen?
Is there anything in any of your profile files that modifies the PATH?

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

The install script is documented in the readme; there's no "download button".

Thank you
On Thu, Aug 19, 2021, 12:15 PM Jordan Harband @.***>
wrote:

Closed #2561 https://github.com/nvm-sh/nvm/issues/2561.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nvm-sh/nvm/issues/2561#event-5182565923, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AR6KIW62Q2CHDZBNNTAJMELT5U3ZXANCNFSM5COTBL5Q
.


add self check tests

Codecov Report

Merging #260 (a211b6e) into avi/inf (cc71a00) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           avi/inf     #260   +/-   ##
========================================
  Coverage    88.51%   88.51%
========================================
  Files           10       10
  Lines         2176     2176
========================================
  Hits          1926     1926
  Misses         250      250

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 cc71a00...a211b6e. Read the comment docs.


Updated jenkins file to run smoke test

Added a new stage "Run SmokeTest" to jenkins file.
In this stage it will pull all dependency binaries from sdo components
and run the smoke test configured in sdo-test repository.
Smoke test logs will be archived under SmokeTest_Log file in jenkins
Signed-off-by: Mareddy, Deepthi<EMAIL_ADDRESS>

@dmareddx @AdithyaBaglody  Do we need this PR after PR #3 has been merged?

@dmareddx @AdithyaBaglody Do we need this PR after PR #3 has been merged?

This is a duplicate of PR#11 and this can be closed