Build errors!

Describe the bug
Branch: Beta
When opening PokemonUnity.sln for the first time I get 1000+ errors in the build solution. "Assembly-CSharp.dll" and "Assembly-CSharp-Editor.dll" not found.
To Reproduce
Steps to reproduce the behavior:
Open project for the first time
Get 1000+ errors
Expected behavior
Should be open with 0 errors.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 Pro
Browser Chrome

Additional context
Add any other context about the problem here.

Development Environment：
Unity version: Unity 2017.4.1f1  (https://github.com/PokemonUnity/PokemonUnity/pull/66)
IDE version: Microsoft Visual Studio 2019

Wow, this was all the way back in Apr, like two month ago, and no one noticed it? Have you tried using the latest pull request? I'm not sure on how much activity this project is getting from any of the other members. Apparently, there's talk of starting over, but in different repositories and in private groups. I've continued this project publicly but on my own name, and I've resolved most of the coding errors and added some improvements wherever I saw needed it. If you still care about pokemon project but using simple programming that's drag-and-drop into unity environment. I believe the repository I host is the only one with that description in c#.


discard gh-pages history

Hi,
I'm converting my doc from travis to github actions and I was wondering the corresponding feature keep-history: false
Now, every time I push to master branch, I get a push to the gh-pages but I don't care about keeping the doc history (this will increase the clone of the main repo)
Thanks a lot!

closing because I realised it's exactly the same flag...


What exactly is Context get/set ?

In the documentation https://honojs.dev/docs/api/context/#csetcget it is only written:

Set the value specified by the key with set and use it later with get.

With a brief example. However, it is not clear what this get/set actually does and how it works. Is it global state? Session based? Something else?

Hey @dagnelies, thanks for opening this issue!
You're right, the documentation is pretty vague on this topic. We are working on a lot of the documentation to make it more descriptive and there's still a lot to get to, but we are happy to receive feedback on what is currently the least clear!
Context here is the context of the current request, so you can define and set keys that have a lifetime of the current request, in order to pass particular variables between middleware.
I'll update the documentation for this soon, and in the meantime we can keep this issue open to track that documentation.

HI @ThatOneBro - Do you have any progress? I mean can I use that to get/set the session of the user? Thank you 😆

I am using the context.set to store the authenticated userId, however when multiple requests are hitting the same worker instance, doing a context.get returns me the wrong userId. It looks like it might not be safe to store mutable data using the context according to https://community.cloudflare.com/t/concurrent-requests-unique-data/148919 and https://community.cloudflare.com/t/is-context-per-worker-execution-unique/127762/3. This means that what you stated above:

allow you to get and set arbitrary key-value pairs that have a lifetime of the current request

might actually not be accurate. Cloudflare already reuses the env object across requests if that hasn't changed (see https://developers.cloudflare.com/workers/runtime-apis/handlers/fetch/#parameters). _vars is a Hono implementation and has private access. Is it possible that Cloudflare reuse of the context instance across request is causing to return _vars values from a previous request?


Refactor/background tasks

No longer implicitly starting threads and tasks when instantiating objects. Instead, start() and stop() functions are added to the source and sink baseclasses to ensure more fine control over behaviour and lifetime of background loops.
To maintain backwards compability and ease of use, start() is lazily run when invoking send() or receive() function, respectively.
Should be end to end tested before merging.

Closes #61, #64, #69.


User "sub" in pre signup trigger

** Which Category is your question related to? **
Auth
** What AWS Services are you utilizing? **
Cognito
** Provide additional details e.g. code snippets **
I am attempting to access the user sub in my pre signup trigger and it appears that it is only accessible if I choose "Email address or phone number" under How do you want your end users to sign in? If I choose "Username" it is not available.
It seems Cognito swaps in the sub for event.userName in my lambda function when using email/phon number auth.
I would ideally like users to be able to sign in with a username as well. Am I out of luck with accessing the sub in my pre signup lambda trigger?

Sorry if this is not the right place for this type of question. I wasn't sure if this might reach the right person. I think it definitely makes sense to expose the sub in preSignup trigger. It seems like the right place to copy the user into a database and the sub allows for a good connection.
I do wish the sub was a uuid v1 instead of v4 so it can be timestamped but I highly doubt that will ever change :)

We will contact the Cognito service team for this, in the meanwhile, you can also create a post in their forum
https://forums.aws.amazon.com/post!default.jspa?forumID=173

@oste
We will inform the Cognito service team about your issue, will keep you updated.
You can so post a new thread in the Cognito forum
https://forums.aws.amazon.com/forum.jspa?forumID=173

thanks @UnleashedMind I actually already cross posted there, here it is https://forums.aws.amazon.com/message.jspa?messageID=915060#915060
Thanks for keeping this one active. Hoping sub can be provided as a param for Pre Signup Triggers.

@oste The Cognito team responded to your request at - https://forums.aws.amazon.com/message.jspa?messageID=915060#915060 and they mentioned that "Sub" is not supported at the moment and provided alternatives. Please correspond on that thread for further questions.


add function to remove tags from head

Hmm maybe I'm missing something, but if we push tags to Headtags inside Headtags.jsx, how will we be able to remove them?

You are absolutely right, I have to find a way to push tags elsewhere and be able to remove them from another package.
In our slack discussion you suggested pushing tags somewhere in the  vulcan:base-components do you have more details on the exact place? (sorry I am still discovering vulcan, any guidance is more than welcome)

Could be anywhere really. If you want, I can merge this is first and then take care of moving where the tags are pushed myself?

If it's easier for you, yes please, this way I see where you will do the push.
Thanks


Version 24.10.0

Hi @lucassaldanha . I thought we were no longer doing versioned docs. Is this PR null?


Binding to wireguard interface not working

Hi,
I have a wireguard interface like this:
3: wg0: <POINTOPOINT,NOARP,UP,LOWER_UP> mtu 1420 qdisc noqueue state UNKNOWN group default qlen 1000
    link/none
    inet6 fd60:1141:233e:2977:d0e:b410:d5b1:171b/64 scope global
       valid_lft forever preferred_lft forever

And I am attempting to publish a container's port on this interface for the other wireguard peers.
So I start ipv6nat like this:
docker run -d --restart=always -v /var/run/docker.sock:/var/run/docker.sock:ro -v /lib/modules:/lib/modules:ro --privileged --net=host robbertkl/ipv6nat
Then I create a network like this:
docker network create --ipv6 --subnet=fd00:dead:beef::/48 -o "com.docker.network.bridge.host_binding_ipv6"=fd60:1141:233e:2977:d0e:b410:d5b1:171b my-network
As you can see, I am binding the network to the address of the wireguard interface.
Next, I run an nginx just for testing:
docker run -d -p 8080:80 --network my-network nginx
I expect to be able to to use curl [fd60:1141:233e:2977:d0e:b410:d5b1:171b]:8080 from a wireguard peer and see the nginx default page. Instead, curl throws this error:
curl: (7) Failed to connect to fd60:1141:233e:2977:d0e:b410:d5b1:171b port 8080: Connection refused
Pinging through wireguard works, so this does not seem to be the problem.
Also netstat -tulpn lists:
tcp6       0      0 :::8080          :::*           LISTEN      4116/docker-proxy

Seems like the port is not bound to the correct IP/interface.
I don't know if it helps: I am on Ubuntu 4.15.0-1021-aws on an aws lightsail instance.
I have exhausted my knowledge here, so I am hoping for some help here.
Thanks!

@max-tet Can you post your output for iptables -nvL and ip6tables -nvL?

I implemented a workaround in the meantime and now I tried to reproduce the setup that caused the problem. But I simply could not do it. In fact, while trying I got it to work in the way I originally planned to. So this is no longer relevant and I am closing the issue.
(If only all bug reports were so easy to solve! :wink: )


Use FrameworkException instead of assert on missing context on CodeIgniter::run()

@paulbalandan @kenjis based on our discussion on on slack, this PR update assert() usage to FrameworkException for missing context in CodeIgniter::run().
Before

After

Checklist:

[x] Securely signed commits

@kenjis oops, just check your PR https://github.com/codeigniter4/CodeIgniter4/pull/6047. I am closing it then.


generate Operations for ONNX.ML domain

Allow ONNX.ML domain in operation generation.
Attached to the previous ONNX operations
ONNX.ML operations are not separated from ONNX. Do we want another dialect? Or put a mark in name?

It is decided to put ML operation as a separate dialect.  PR#97 is the new implementation.


Add default query function reference + pageParam clarity

I added a link to /guides/default-query-function because I was confused what that was referring to when I first started.
I also added a reminder to return the data AND the pageParam, because with useQuery I was used to only returning the data, so I wasn't sure how to properly reference the page in useInfiniteQuery.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support negative lookahead and similar

I tried doing a replace on this: #!(?!/usr/bin/env python) and got "Error: Couldn't parse regex". I'm guessing it's because negative lookahead isn't supported.

The regex library scooter uses doesn't support negative lookahead, which is why you're seeing the error. I think there are other libraries that allow for this so a potential future feature would be to have an additional regex library available for users to use via a CLI flag

Why not switch to a better regex library as default?

I see, that makes sense. Another option is to automatically switch to a
different crate when we identify that certain features, like negative
lookahead, are being used.
On Fri, Nov 29, 2024 at 7:34 PM Tom Schafer @.***>
wrote:

It's generally agreed that the regex https://docs.rs/regex/latest/regex/
crate (i.e. the one that scooter is currently using) is the best for a
bunch of reasons, it is just missing a few features such as negative
lookahead, hence why I'd rather keep that one as the default and add
alternatives via flags
—
Reply to this email directly, view it on GitHub
https://github.com/thomasschafer/scooter/issues/44#issuecomment-2508193276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAN3SVT2RQS7YRHXX46L7T2DCQR7AVCNFSM6AAAAABSXNTZEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBYGE4TGMRXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


libllama.so is 64-bit instead of 32-bit error when trying to load the model on my Samsung Device

Hi,
I tried using Orca mini 3B q4_0 (https://huggingface.co/TheBloke/orca_mini_3B-GGML/tree/main) model with this repo on my Samsung Galaxy M13 Android Device with 6GB Ram.
I get the following error while trying to load the model.

When I install the apk from this repo on my phone, the app loads the model but shuts down immediately after entering the prompt without giving an error message. I don't think this app works with ggml version 3 (like the Orca mini). Try the apk from the fork: https://github.com/dsd/sherpa/releases/tag/2.2.1-dsd2

On the fork above I can use orca mini 7B V3 without issue (on 12gb ram)
About your 32bit error, pretty sure any 32bit android phones are way too weak to run this at decent speed.


parallelization of calibration

Enabling parallel computing will accelerate calibration process.
SciPy Differential Evolution is able to parallelize optimization. Please integrate and test the function and create an example. Therefore simply adjust an existing example.
I will assign Jonas Betscher to this issue asap.
@FWuellhorst Do you have any comment/advice?

Solution
Multiprocessing was achieved by using pymoo as a solver. When using mp, the entire population is handed over to the new method mp_obj in the calibrator class. Here the population is devided into chunks of the same size as the chosen number of CPUs for computing. In the same loop a list with the new parameters is created and handed over to the new method multi_simulate of the SimulationAPI class. This method starts mutliple simultaneous simulations by using pool.starmap of the simulate method. When all simulations are completed a list with the results is returned to mp_obj and logging of the results starts in queue. Having gone through the whole population, objectives are returned to the solver and the next population is presented to start this process all over.
Multiprocessing works for both FmuAPI and DymolaAPI.
This has changed for the user
A new example named e6_multiprocessing_calibration_example has been added and is based on the example e4. Changes are the presented variable n_cpu and new kwargs for using pymoo as a solver.
When using pymoo the chosen framework must be pymoo and the method is the used algorithm.
Right now the preset algorithm is GA (Genetic Algorithm). Changing this will require getting to know the hyperparameters of other algorithms and therefore changing the kwargs.
The README has also been updated, noting that pandas==1.3.5 and tables==3.6.1 have to be used in order to work.
Speed advantage
The speed advantage is highly dependent on the simulation time, as logging does not happen in parallel. Having in mind that simulation time is usually higher by a large number of multiples of the logging time, the speed advantage correlates directly with the used number of CPUs. For example a calibration using n_cpu = 4 should take around a fourth of the time of one using n_cpu = 1.

Hi @FWuellhorst,
I checked the new code and tested its functionality. It worked for all tested scenarios.  Do you have any further ideas for improvement?

@jonasklingebiel1  @JBetscher
First of all thanks for this really nice contribution!
Do you have 30 mins in one of the next days to talk about the changes especially in ebcpy's simulation API? :) I think this is more efficient compared to review in github.

@FWuellhorst
Yes, I'll message you via slack to find time for a call.


Wordpress doesn't recognize WebChat CSS

Hi everyone!
I'm trying to add my Azure Chatbot via Direct Line and this Code to a Wordpress landing page:

I altered and uploaded the WebChat CSS-File (to a FTP server) but when I add the code to a HTML module in a Wordpress page it doesn't seem to recognize the CSS file:

It's supposed to look like this:

What might be the problem here?
Thank you so much in advance, help is really appreciated!

Since this is a question about functionality and not a bug report of feature request, please ask it on Stack Overflow (with tag botframework) so that others can benefit from the answer.


Problem compiling 'ksmbd' and 'omcproxy'

When trying to compile OpenWRT with my version of LuCI, I can see errors reported against the makefiles:
ERROR: please fix feeds/myluci/applications/luci-app-ksmbd/Makefile - see logs/feeds/myluci/applications/luci-app-ksmbd/dump.txt for details
ERROR: please fix feeds/myluci/applications/luci-app-omcproxy/Makefile - see logs/feeds/myluci/applications/luci-app-omcproxy/dump.txt for details

The files are showcasing a problem trying to get access to luci.mk, ex:
Makefile:8: /home/openwrt/feeds/luci/luci.mk: No such file or directory
make[1]: *** No rule to make target '/home/openwrt/feeds/luci/luci.mk'.  Stop.

Checking them out, it seems that they try to include using direct path:
include $(TOPDIR)/feeds/luci/luci.mk

instead of relying on the relative one like every other Makefile:
include ../../luci.mk

Fixed with 2f0ffd9 for master and d365470 for openwrt-21.02


ARROW-2546: [CI] Intermittent npm failures

Upgrade CI npm version beyond 5.7.1 to avoid EINTEGRITY errors, as suggested here: https://github.com/npm/npm/issues/16861#issuecomment-371256089

@trxcllnt let me know if you have any objections to setting the npm version this way. If not I can merge once CI finishes.

d'oh didn't see @xhochy's PR. No particular reason to stick with 5.8, just trying to make a minimal change. I'd be fine merging #2006


UI Artifacts when hovering over different sections or entering text in the editor

VSCode Version:
1.46.1
cd9ea6488829f560dc949a8b2fb789f3cdc05f5d
x64

OS Version: Win 10 Home Version, 64bit (Build 18362)

DirectX Version: 12

Integrated GPU : Intel HD Graphics 630 (drivers updated)

Dedicated GPU : Nvidia GeForce GTX 1050 (drivers updated)

Intuitively I feel that this might be occuring due to some Graphics Settings on my PC, So I have posted other specifications...
I vaguely remember some months ago enabling GPU rendering or some other option through vscode, but I am unable to find that now from settings directly...
Steps to Reproduce:
For eg. If I hover over the file navigator, then other sections, editor and the icons list on the left hand side get covered with those dots...

I disabled GPU rendering and everything is proper now.


feat(demo): Add dashboard

Readme Updated, screen added, description added
The purpose of using public/script/**.js files is just to show the ability to add any kind of technology around the SciChart such as  WebComponents, TS/JS, JS Frameworks, etc.


Tap or toggle don't work on venu 2

Hello,
First of all, congratulations on your work which is the most accomplished of what I have tested.
I followed the procedure and my watch (Venu2) displays the entities that I configured.
But I cannot send a command, neither "tap" nor "toggle"
I tested my long live token with a rest_command and it 's correct.
Can you help me
Thanks
sorry for my bad english. I am French.
my json config: (ihave changed my entities for security)
{   "$schema": "https://raw.githubusercontent.com/house-of-abbey/GarminHomeAssistant/main/config.schema.json",   "title": "Domicile",   "items": [     {       "entity": "script.script_ring_desactivation_alarmemaison",       "name": "Arret Alarme",       "type": "tap"     },     {       "entity": "switch.tasmota10",       "name": "Basc. Porte garage",       "type": "toggle"     },     {       "entity": "switch.tasmota30",       "name": "Ambiance1 Salon",       "type": "toggle"     },     {       "entity": "menu.eclairage_salon",       "name": "Eclairage Salon",       "title": "Salon",       "type": "group",       "items": [         {           "entity": "light.halogenes4",           "name": "Halogene",           "type": "toggle"         },         {           "entity": "light.lamp_leds",           "name": "Lampe Led",           "type": "toggle"         },         {           "entity": "light.lampe_phones",           "name": "Lampe téléphone",           "type": "toggle"         }       ]     },     {       "entity": "script.script_lampe_couleur_pou_raz",       "name": "RAZ Lampe",       "type": "tap"     },     {       "entity": "switch.crnr_tbl_usbs",       "name": "Corner Table USBs",       "type": "toggle"     }   ] }

Just to acknowledge your submission.
{
  "$schema": "https://raw.githubusercontent.com/house-of-abbey/GarminHomeAssistant/main/config.schema.json",
  "title": "Domicile",
  "items": [
    {
      "entity": "script.script_ring_desactivation_alarmemaison",
      "name": "Arret Alarme",
      "type": "tap"
    },
    {
      "entity": "switch.tasmota10",
      "name": "Basc. Porte garage",
      "type": "toggle"
    },
    {
      "entity": "switch.tasmota30",
      "name": "Ambiance1 Salon",
      "type": "toggle"
    },
    {
      "entity": "menu.eclairage_salon",
      "name": "Eclairage Salon",
      "title": "Salon",
      "type": "group",
      "items": [
        {
          "entity": "light.halogenes4",
          "name": "Halogene",
          "type": "toggle"
        },
        {
          "entity": "light.lamp_leds",
          "name": "Lampe Led",
          "type": "toggle"
        },
        {
          "entity": "light.lampe_phones",
          "name": "Lampe téléphone",
          "type": "toggle"
        }
      ]
    },
    {
      "entity": "script.script_lampe_couleur_pou_raz",
      "name": "RAZ Lampe",
      "type": "tap"
    },
    {
      "entity": "switch.crnr_tbl_usbs",
      "name": "Corner Table USBs",
      "type": "toggle"
    }
  ]
}

Checks out in the schema checker (you already knew that). You mention working API key and that the menu does display on your Venu2 (same device as me, so this is the most tested!). As you are getting no actions working, I'm going to go with API URL. Please can you re-test your REST command with a copy & paste of your API URL from your watch settings?
Otherwise the next step would be to screen share somehow so I can get a better feel for what's going on here as I'm worried. This should be working.

Another thought. Does your set up meet this requirement from the README.md?
"It is important to note that your Home Assistant instance will need to be accessible via HTTPS with public SSL or all requests from the Garmin will not work. This cannot be a self-signed certificate, it must be a public certificate (You can get one for free from Let's Encrypt or you can pay for Home Assistant cloud)."
I've missed an opportunity for some error checking on API URLs. That I can fix.

Hello, thank you for your answer, I did some research on my installation, tested the curl for the token, everything is ok and I use a let's encrypt certificate.
when consulting the NGINX proxy manager logs, I noticed that there was a double slash behind api.
I removed the slah in the API settings by putting:
https://<homeassistant>/api
It seems that the error is in the readme, the slash is present .
2 Add the URL for your Home Assistant API, e.g. https://<homeassistant>/api/.
all switches and lights work, it's fine.
automations and scripts do not respond to tap, and do not appear in NGINX logs.
I continue my research...

README.md will be amended, thank you for your insights. Good work.

Good morning,
here is the progress of my various tests:

for scenes, with 'tap' option and service (scene.turn_on) everything works correctly, there is an execution message on the watch.

for scripts, with 'tap' option without service does not work but works well with service (script.turn_on), there is an execution message on the watch.

for automation, with the 'tap' option without service does not work, however it works well with service (automation.trigger), there is no execution message on the watch.

in the case of automations, I was not able to test with automation.turn_on, I do not know how to trigger the reception of the command by API. Maybe I didn't understand the principle well.
If it is feasible, an example would allow me to better understand and increase the processing possibilities.
If I can give my opinion, the new service option is really great, it allows you to personalize via known commands in HOME ASSISTANT.
I'm going to highlight this project on the Home Assistant French forum.
Thank you for everything,
Sincerely.

for automation, with the 'tap' option without service does not work, however it works well with service (automation.trigger), here is no execution message on the watch.

curl -s -X POST \
  -H "Authorization: Bearer ${API_KEY}" \
  -H "Content-Type: application/json" \
  -d '{"entity_id": "automation.turn_off_usbs"}' \
  ${URL}/services/automation/trigger \
  | jq .

Returns the empty JSON array []. Presently I check for an item in the returned array with the desired entity name as part of the confirmation. It should be an array of things that changed, e.g. for a scene. So as the array is empty, the condition to acknowledge is not met by the present code.

I'm looking to close this issue withthe following changes:

All taps must have a service tag. This will be a breaking change so will be done in two parts: firstly to change the schema to prompt people to amend their menus, in a later edition the application will ignore menu items without a service tag.
Implement acknowledgement of automations. These are now able to be both 'toggle's and 'tap's. (Previously they were just 'toggle's)

I hope that meets your expectations @Patrick44fr.


feature request: bump project version command

I'm replacing rye with uv.
(The following articles were referenced: https://github.com/astral-sh/rye/discussions/1342)
I've replaced most of the functionality with uv, or changed to other commands or packages.
But changing the version of a project is both simple and annoying.
In the past, I've changed versions like this(in github action)
rye version <some version>
But now I change it like this
sed -i'' -E “s/version = \”0.0.0\“/version = \”$VERSION\“/g” pyproject.toml
This works, but it's not satisfactory.
I'd like to know if you have any plans to provide functionality like this.
Again, I'm not saying you have to.
It doesn't matter if you don't have it, but it would be nice to have it.
Either way, I'm happy enough with uv for now.

I like the idea of this feature.
It would be nice if it would be able to incorporate features like bump-my-version where you can specify which files to do the version replace and how to search and replace the version strings using settings in the pyproject.toml file.

This is a duplicate of https://github.com/astral-sh/uv/issues/6298


Unify client donations methods

This Pr does the followings:

merges MinimalDonateForm into DonateForm
unifies donation state and move it redux
debounces stripe calls
make the IsDonating consistant between db and redux store.
moves ajax logic to sagas
resolves issues with GA actions

[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] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #38052
Closes #39507
ref #38373

One thing that confused me was why addDonation appeared on the PayPal button, but not elsewhere. Could we change that to addPayPalDonation instead?

we could change it to that. it is just the name of the endpoint that it hits.

we could change it to that. it is just the name of the endpoint that it hits.

No, it's fine. On reflection I like your way better.  It'll become more obvious once there are more donation methods.


Mat-table: giving border to mat-mdc-row doesn't work

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
I wanted to give border for the row, but is is not working
when I give mat-mdc-cell the border works but that doesn't give the effect I wanted
Reproduction
StackBlitz link:
Steps to reproduce:
1.
2.
Actual Behavior
the css I added is not effective
Environment

Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

Can you post what you were trying to do?

I did fixed


Add support for running without zero optimizer sharding

Would be nice to compare footprint with and without zero stage 1. It is currently not implemented https://github.com/jfc4050/dlcalc/blob/8cec28f81017b2142feb0657255a1f4b87e546b6/dlcalc/training_3d.py#L226

hey! sorry but i'm unlikely to get around to this because for vast majority of cases, there's no downside to using the ZeRO optimizer (less memory, less optimizer compute, less checkpoint load/store volume, and equal comm volume).
if you want to calculate it, in you should be able to multiply the optim states by DP to get back to the unpartitioned footprint.


Live video detection and download

Is there anyway to detect live video and also download it?
This is a new Instagram feature and most libraries does not support it.
Unlike the non-official public API, this library has the potential to do this.
I hope this feature will be included.

Addressed in rewrite or no longer relevant.


Button edge-case refinements

A number of button changes here, mostly focused on little edge-case polish.
Changes on the right.

Focused states are important. I think we should maintain the focused border regardless of hover states.

Selected buttons should still have a hover state since you can still interact with them.

Primary button text could use a slightly stronger shadow for increased readability / contrast.

Disabled buttons needed a little contrast. Disabled or not, we should be able to read what's being disabled.

Not stoked about the last two changes. I'm in favor of darkening the text shadow,  but not reversing the direction. The text color change on the disabled state also feels off—maybe a little muddy?

I can revert the shadow direction, but did so to match the light source of all the gradients throughout Primer. Do you just not like it visually or is there some rationale to having the shadow cast opposite the light source?
Darkened the disabled primary button. Is this any better or should I drop it?

The intent of the text-shadow is to illicit an embossed effect of sorts. The text isn't sitting on top of the button, it's inset. Still not a fan of the disabled primary button change :).

@mdo Okie dokie. I've reverted those last two points and updated the screengrabs. I disagree that they should be embossed, but hey, it's your huge mistake, not mine :wink:

Originally I had no text-shadow on them, but that looked off a bit. I wouldn't be opposed to seeing the colors change so we can nuke the text-shadow entirely though—we don't have one on any of our other buttons I think.

I wouldn't be opposed to seeing the colors change

Try these new colors out. I think it improves readability without losing the soul of the original green. It only loses a little fluorescence is all.

Most of these are :ok_hand: by me.
I'm not sure how I feel about darkening the primary buttons though. Most of our brand colors are pretty vibrant and this one feels pretty heavy.

Definitely like most of these changes.
I agree with @jasonlong's point, and I think my only big hang up would be the changes to the primary button color. Maybe it's something we'd use for a few days and wouldn't notice again, but it definitely feels a little out of place in the screenshots.
I would be open to adjusting our colors at some point, as I think the green, red, and orange in particular are pretty bright. But maybe that's a conversation for another PR? It seems like we could get this merged pretty quickly if we moved that to another PR.

:-1: on the green. Everything else looks good.

Perhaps if we flatten the green buttons by getting rid of the top gradient highlight. I love the green as a large swatch, but when it's applied to thin type and buttons with white text, it really lacks contrast.
I think this is a really good example of that.

I think the other stuff is shippable. I just worry that the focus state for buttons might get a little obnoxious in practice. I'll sit with it for a bit.

Perhaps if we flatten the green buttons by getting rid of the top gradient highlight

I'd like to avoid going totally flat, but I perhaps a more subtle gradient? Affordance still seems important, and one flat button seems a little off, too.

Agreed that we should tackle any brand color and UI element tweaks separately. I think we can definitely make some improvements, but there are a lot of use cases to consider.


WIP: toolbox: gpu-operator/diagnose.sh: new script

New script to diagnose GPU Operator issues on a cluster.

/test gpu-operator-e2e


Can't find reset or restart when collision occured

I couldn't find any reset or restart keyboard input for env.
Is there any utils that i don't know?
teleop_key.py example doesent show any Keyborad input for reset too.

Hi,
There is a function robots_reset()  in the env_robot.py to reset the robot states.
And I have added the interface env.reset() to call this function. In addition, for keyboard input, I added the key 'r' to reset. You can check the functions by example teleop_key.py in the newest version.
Thanks

Thx ! ill check!


Update the existing package

I can't find any document about how I can update the existing package. Is it even possible?

Hi @BladeRunner713 Could you clarify what you mean by update the package? To get the latest version of Sui, clone the Devnet or Main branch.

Hi @BladeRunner713 Could you clarify what you mean by update the package? To get the latest version of Sui, clone the Devnet or Main branch.

Hi Randall, I want to know if it's possible the update the package or module I published.

Where is it published? I would think you could update and publish it again. Sorry that I am not understanding your specific question.

Where is it published? I would think you could update and publish it again. Sorry that I am not understanding your specific question.

Suppose I publish a liquidity pool on Sui ledger. Then I realize I have some bugs inside my code. I could publish a new module or package, but how do I deal with these objects on my old module or package?

Ah, OK. Let me get more information about this. Thanks for the additional detail

Looks like they are still working on some of this. There is some info that may be helpful in https://docs.sui.io/devnet/build/move and the additional topics in the same node.


Sending raw ethernet frame with the lan8742a?

Is there a possibility to send raw ethernet frame (IP_RAW) with the lan8742a?
I see that this is possible with the w5xxx already, but would like to do it through the builtin phy on f767.

I believe it's totally possible, but it must be the feature provided by the STM32Ethernet library on which this EthernetWebServer_STM32 relies.
I suggest you open an enhancement request in STM32Ethernet issues.
I'm closing the issue now.
Regards,


does 3.0.0 work on 6.0.9?

considering that 6.1.0 is still rc what would block me from using ngrx forms 3.0.0 on 6.0.9?

Well, it will work just fine with Angular itself but it requires TypeScript 2.8+. 6.0.9 is not officially compatible with such TypeScript versions but it may still work depending on what your application does. I myself have used TypeScript 2.8 successfully with Angular 5.2.9 but I cannot guarantee that there will be no issues. Therefore feel free to try it out on your own risk.
Sadly I did miscalculate on how long it would take Angular to become compatible to TypeScript 2.8 and I had built almost all changes for 3.0.0 on top of the type inference changes so it was too much work to create a version of 3.0.0 that would not require TypeScript 2.8 :(


CSS doesn't link when generating default HTML format

Hi @mattt! Great project. I was poking around and testing the --format html option and noticed the all.css file isn't linking properly to the generated index.html file.
I tried debugging this myself, first by checking index.html where it seems to be linked correctly (however I'm not a web developer):
<link rel="stylesheet" type="text/css" href="all.css" media="all" />

But when I check out the network console in my browser, it looks like all.css is failing:

Any idea what could be happening? I'm using 1.0.0-beta.2.

Oh, I think I may have found the cause.
It appears this base-href line is causing the file search path for all.css be incorrect:
Index.html
<head>
    ...
    <base href="/"/>
    <link rel="stylesheet" type="text/css" href="all.css" media="all" />
</head>

Changing this from <base href="/"/> to <base href=""/> fixes the issue, so I think we just need to swap the ordering of these two lines:
<base href=""/>
<link rel="stylesheet" type="text/css" href="all.css" media="all" />

I created a pull request for this in https://github.com/SwiftDocOrg/swift-doc/pull/96.

https://github.com/SwiftDocOrg/swift-doc/pull/93 is likely a better solution than this PR, so I'll close this in favor 👍

Thanks again for bringing this to our attention, @captainbarbosa. I'll keep this open until after #93 is merged, so that you can confirm the fix for yourself.

Hi @mattt thanks for this great project!
Just wanted to raise that I'm running into this issue as well. I have ´swift-doc 1.0.0-beta.3´ installed on my machine.


Improve mobile display

About 10% of our users visit the site on a mobile or tablet device. That might not sound like much, but it amounts to ~7,000 people this year. The quality of site display on smaller screens is pretty variable. Let's put some thought into improving display for non-desktop devices.

I did some snooping with browserstack and my phone to see how things are. Here are some thoughts on some pages. Lmk what you think @hancush, I can pick any or all of these up anytime:

General

On browserstack devices other than galaxy s21, the top nav bleeds off of the page a bit, which makes it so that the logo and expanded nav items also render slightly off the page. My phone is an s21 and both mine and theirs seems to be fine here.

Homepage:

Could possibly use dividers between listed meetings

Board members listing:

The recent activity section displays fine when showing less, but when showing more, row items get pushed to opposite edges of the container
Related to that: would it be worth considering moving that show more button to the top of that container for mobile? It's potentially a lot to scroll through to get back to the show less button at the bottom if you accidentally clicked it or changed your mind.
The map that says "tap to view details" just takes you to the person detail page for that polygon instead of showing details. If this is a feature we want on mobile, then this might be a priority.

About Us

page scrolls horizontally a bit for board report statuses table

Board report search

After searching once, the collapsed nav disappears. comes back on refresh
Might be nice to have a separator between related search terms for clarity on mobile

Events listing

The top section with the date selector, list button and refresh could be a little more clear. Instructions in the date picker get cutoff and I initially wasn't sure what the bulleted list button was for

Minutes search:

Same as top section of events listing.
The td showing what kind of meeting it was is blank. This also feels like a priority.

Event detail:

The event's heading with details could use some horizontal space

@xmedr Thanks for this! Let's start working through these by section. Want to pick up the General and Homepage items?
Let's also take this opportunity to upgrade to the latest Bootstrap! As there are quite a few changes between legacy and contemporary Bootstrap, I'd like you to reproduce the existing site design but write the new templates from scratch (of course, copying and pasting content and other reusable elements). LMK if this makes sense!
Can you open up a branch called "mobile-updates", then branch off that branch and start your changes? We'll use "mobile-updates" as a long-lived feature dev branch and continue to branch off of it as we move through this list.

@hancush update for you, I was able to get a draft of the General and Homepage upgrades and bootstrap migration on the mobile-adjustments branch, and I've started work on the About Us (was hoping I could knock that out before EOD today, but I'm almost done with that one)

@shrayshray These changes are deployed to the staging site! https://lametro.datamade.us/ Want to take a spin and let us know what you think?

@xmedr to redeploy to staging for Metro review in October.

@shrayshray These changes are redeployed at https://lametro.datamade.us/ and includes the most recent updates at time of writing. That includes things like the alert manager and the Caltrans board members map layer.
Note: The alert manager uses styles based on this upgrade. So on the live site, the Primary and Secondary alerts may not have specific colors until this is brought in. The other alerts should work just fine for now, though.

@xmedr This is looking great to me on my iOS devices, and I have a couple notes below. I asked @camachoo and @neilarellano to check on their devices and add feedback here as well. Notes:

On meeting pages, it looks like there could be some vertical space added between the recording buttons:

Can you add a scroll bar to the list of directors on the mobile version, similar to on the desktop version (screenshot below)? This would make it more clear there are more directors in the list to view.

Can the view PDF and download buttons be emphasized, either with bold or larger font on the Board Report pages?

Is it possible to reissue admin login credentials to the staging site? The credentials I have aren't working to login and see the new alert styles. This is only if you want me to review the new alert styles here, before they are live (but not a huge deal because we'd preview them on live before pushing to users anyway).

@shrayshray Thanks for this! To answer your questions:

Good catch, there absolutely can be some space in between those buttons
So far, the appearance of the scroll bar seems like it's controlled by the mobile OS.  I'll do some more digging on this though to be sure. In case it doesn't pan out, the plus side is that it does show once you start scrolling.
For sure, we can get that emphasized. Below I have an example of both a font size increase and bolded text, what do you think? If it's too much we can just do one or the other.
That'd be useful to for you all to have working logins for the future even, so we'll get that back up

@xmedr Can you provide an update on this?

@shrayshray I've got an email coming your way with new login credentials for the staging site at  https://lametro.datamade.us/ , and changes from your feedback has been redeployed there.
There's also some bonus changes from the first round of accessibility adjustments discussed in #1021. The most notable changes are higher color contrast throughout the site, and the addition of underlines to links to help distinguish them beyond just color.
If you have the time, would you be able to take a look at this by the end of the week? No worries if you folks are busy!

@xmedr on the homepage, could you add additional contrast to the text below the search box? Maybe a white background behind it? The image behind the text obscures it in some views and while the new higher contrast helps, more would be better.

@shrayshray of course! though it looks like you might be looking at a different version of the site. is this maybe the link for the staging site on heroku that was recently shared with you? if so, that unfortunately doesn't have the styling changes on there. sorry for not realizing this would've been confusing sooner!
we'll be keeping the old staging site available until these changes are approved. so for now that's going to be at https://lametro.datamade.us/ and if anything else changes i'll be sure to give an update. so on this link, that text looks something more like:

and i can definitely still increase the contrast more if we need

@xmedr my screenshot was from the live site and it definitely looks better on staging than on live! But Omar requested even more contrast than what we're seeing on staging.

@shrayshray absolutely! here's what it looks like in that white box. and i agree, this does feel way more readable.
also change of plans on that old staging site. we did end up taking that down, and instead we're putting these changes up on the new heroku staging site you were using before, found at https://la-metro-councilmatic-staging.herokuapp.com/. apologies for that bout of switching and thanks for bearing with me on this! 🙇

@xmedr beautiful! Thank you!

@shrayshray absolutely! and of course let me know if you folks find anything else on this or if we're good to go

@neilarellano @camachoo these changes are back on staging for review!

Hi Team,
I noticed when I was going through Board Reports search results that some informational labels were overlapping on Android mobile. Please see below:

@neilarellano thanks for catching that! i've got that fixed, and staging now looks like below. let me know whether you find anything else

@shrayshray Thanks again for signing off on the current set of mobile/accessibility changes! The next two parts are up on the staging site now for you folks to review. Here are some major things to look out for, for each category:
Operable (2 of 4)

Use the keyboard to navigate through various pages, including those with maps.
Confirm that

tab order is predictable.
a bypass block exists at the start of each page, that can skip to the main content of that page.
at no point do you get stuck and cannot tab out of an element or group of elements.
visible focus is maintained for each element tabbed through.
the site is as usable with a keyboard as it is with a mouse, with the exception of map interaction.
there are sufficient accessible alternatives to the maps. The board members listing page has the adjacent members listing table, while individual board member pages have the heading of the map.

Understandable (3 of 4)

Adding the use of a screen reader, navigate through various pages including those with maps.

Note: Some screen readers need a language pack to read different languages. For example, to get Spanish in VoiceOver go to Settings > Accessibility > Open VoiceOver Utility > Speech and click the plus button to add a language.

Confirm that

the screen reader predictably announces each element on the page.
the screen reader announces elements in their appropriate language.
links that open in a new tab/window are announced by the screen reader.

In the meantime, we'll continue work on the last category.

@shrayshray we're close to done with the 4th leg of the a11y changes! though we'll still hold off on bringing it in to the staging site until this previous round of review is done.

Updating this with details from the latest meeting. We’re done with the last category of the a11y changes and just waiting for the previous two to get tested so we can push this last one up. Once that's finished, we can get everything onto the main site!
@shrayshray were you folks able to find any issues with the "operable" and "understandable" categories? If not, I can push the last bit up

@camachoo @neilarellano As mentioned in today's meeting, below is the link to the review app containing the Operable and Understandable changes (instructions for testing listed above), as well as the Robust changes. This environment will be a bit outdated as far as other changes that have been made throughout the site, but will be sufficient to test these particular accessibility adjustments!
Review app - https://la-metro-cou-a11y-robus-n1od02.herokuapp.com/
Robust testing instructions (4 of 4)

Use a programmatic HTML checker like W3's own Nu HTML Checker to compare the html of the review app and the live site on various pages
Confirm that the number of errors and warnings have been reduced.
Note: some of the warnings/errors present are from third party technologies. Many of them can be upgraded to resolve these issues, and those are listed here.

Hi Team! We have confirmed most of the listed updates. Below is a list of comments and questions we have for the line items below:

I am unable to load the search results page on the review app.
We tested read aloud using Edge’s native screen reader. The pages were readable and understandable using the screen reader. Below are a few comments on what may be confusing to our audience when using the screen reader:

About us page’s Board Report Status table section – it may not be clear that this is a table being read aloud. The headings are read but the content of the table may not be understandable.
Pages containing board report numbers have the dashes read as “to” instead of a dash. For example 2024-0001 is read as “two zero two four to zero zero zero one” instead of “two zero two four dash zero zero zero one”. I am unsure if this is something we can do anything about or if this is dependent on the specific reader used.
The agenda PDF is not being read aloud even though the PDF can be tabbed through. I’m not sure if this is expected behavior.
The alert banner that we set is not being read aloud at all in any page where it is visible.

links that open in a new tab/window are announced by the screen reader – Not sure how to test this out, so far the links are just read and does not say if it opens in a new tab or not. What is the expected behavior for this and is there a recommended way of testing this outside of just the native TTS reader on browsers?

Thanks so much for this @neilarellano! I've got an updated version of these changes merged in with the latest changes to the site at: https://la-metro-cou-a11y-robus-rcwgkd.herokuapp.com/
Here's some answers to your comments. If this looks good then we can treat this as a checkpoint to get things merged in and continue work from here!

The search should be working now!
Thanks for catching that on the table! I'll open up a separate issue to fix that after this comes in since the problem also exists on the live site.
You're right that this is specific to each screen reader's settings (eg. mac's voiceover doesn't read it at all by default). Users can specify how they would like their reader to handle dashes and similar characters. Manually coercing that on our side has the potential to make the experience better for some but worse for others. Leaving that up to the user is the best course of action here.
The pdf seems to behave differently for different screen readers, though none of them are particularly good as of now. We have issue #1089 open to look into improving that after these changes come in.
I've made the alerts tabbable to make sure that they will be read!
For links opening in a new tab, we're expecting the reader to say the text in the link followed by "link opens in a new tab". An example would be search results. Do those also not get read for you?

Another way we can see if that phrase should be present is by inspecting the element in your browser and looking for its aria-label. Here's an example of a search result in chrome, with the gray box being a visual representation of what VoiceOver reads:

@neilarellano Whenever you get a chance, let me know how this looks to you!

hi @xmedr
The alert can now be tabbed through but it is not being read by the screen reader.
I do see the aria-label for links that open in new tabs, however, I am still unable to have the "link opens in new tab" be read aloud.
I used Edge's default reader and tried it on Chrome on 2 different screen reader extensions.
The rest looks good.

@neilarellano Got it! I'll do some more testing with Edge and Chrome. Which screen reader extensions are you using on Chrome?

@neilarellano Is the Edge default reader you're using the Read Aloud feature? If not, then I can use whichever one you're using instead. But if so:

It read the alert during my testing, though it skips to the main content by default when it starts. So going back to the alert read it aloud. If you can provide something like a screen recording of the issue happening, that would be a super helpful!
It seems like Read Aloud doesn't support the aria-label tag that we're using to add that new tab context, which is unfortunate since it's a fairly common attribute to use. The alternative to account for Read Aloud would be to add the "new tab" text directly to each button and link, but I'm hesitant to suggest this since it would add more noise to links on each page.

Also for Chrome, I tested using Screen Reader and Screen Reader with On-Off Button and both were able to read the alert and the "link opens in a new tab" portion of the aria-label (though "Screen Reader with On-Off Button" was a bit unwieldy to use). I can definitely still try whichever specific extensions you've tried though to see what's going on with them.

Do we have a full list of accessibility and mobile changes that have been made and not made?  I think there was a full checklist, right @xmedr?

@camachoo Apologies for the delay. Yes, each of the a11y issues is a checklist of items to consider, and each of their attached PRs has a Notes section with specific details on some changes. Here's a list of each issue and its corresponding PR:
Perceivable:

https://github.com/Metro-Records/la-metro-councilmatic/issues/1021
https://github.com/Metro-Records/la-metro-councilmatic/pull/1042

Operable:

https://github.com/Metro-Records/la-metro-councilmatic/issues/1022
https://github.com/Metro-Records/la-metro-councilmatic/pull/1051

Understandable:

https://github.com/Metro-Records/la-metro-councilmatic/issues/1023
https://github.com/Metro-Records/la-metro-councilmatic/pull/1058

Robust:

https://github.com/Metro-Records/la-metro-councilmatic/issues/1024
https://github.com/Metro-Records/la-metro-councilmatic/pull/1088

@neilarellano Let me know if you have any updates/questions on your testing! It would be great if we could bring this in soon.

@xmedr I was able to test the TTS options and was able to get the same results.
It is a shame that Edge's native reader doesn't support aria-label. It did work fine for me when testing through other TTS extensions in Edge and Chrome.
As for the alerts, should these alerts be read first before the site's main content? I think that would be useful in scenarios where we experience technical issues but have important announcements/workarounds through the alerts. Would it be possible to do this? Would there be issues or disadvantages to having the alerts always be selected/read first?

In order to get this issue and its related prs closed and merged, I've opened issue #1172 for this last Edge bug.

Closed by the final a11y category pr #1088 and now pushing these changes to production after successfully testing on staging.
Thanks for all the help on this, team!


Explore onecloud for GPU CI

I've discussed with Vadim a common approach for using the cloud machines. The idea is to build a runner image which we can redeploy after the machine lease is completed.
Will try to make a prototype after Vadim figures out how to update the kernel in the image

need at lest 3x GPU for CI (may need more later)

There were no update from onecloud owners during this week

no update 09/10, found alternative solution internally


Add option n - number of news articles to be downloaded

With default value being 100.
Python main.py 500

Fixed

Can you also update the result set with around 500 articles.
We can freeze this version after you do that. And I can give it to. Dorai to test.


Moving Text entity in A-Frame

https://stackoverflow.com/questions/63063100/moving-text-entity-in-a-frame

I ran into this issue as well, but I don't think it's a super-hands issue. When I made a scene using the grab component from aframe-extras instead of super-hands I ran into the same behavior. I think it is happening in the physics system, as these objects can be moved by other physics objects either. I filed an issue on the physics repo.

I ran into this issue as well, but I don't think it's a super-hands issue. When I made a scene using the grab component from aframe-extras instead of super-hands I ran into the same behavior. I think it is happening in the physics system, as these objects can be moved by other physics objects either. I filed an issue on the physics repo.

Here's an example of a box with text that's movable with super-hands & physics (via mouse): https://glitch.com/edit/#!/past-sophisticated-mojoceratops?path=index.html%3A1%3A1
This is using racyaster for collision detection. I suspect your issues are related to the collision detection & the text component interfering with automatic shape generation. Try using the shape component to explicitly define the collision geometry.

Here's an example of a box with text that's movable with super-hands & physics (via mouse): https://glitch.com/edit/#!/past-sophisticated-mojoceratops?path=index.html%3A1%3A1
This is using racyaster for collision detection. I suspect your issues are related to the collision detection & the text component interfering with automatic shape generation. Try using the shape component to explicitly define the collision geometry.


[FEATURE]: values() should accept readonly values during insert.

Describe what you want
The insert(...).values() call should accept a readonly object or array.  My programming style is very readonly (which easily prevents a entire large category bugs), so not accepting readonly is really inconvenient for me.
I'd suggest replacing this:
    values(value: PgInsertValue<TTable>): PgInsertBase<TTable, TQueryResult>;
    values(values: PgInsertValue<TTable>[]): PgInsertBase<TTable, TQueryResult>;

with something this:
    values(value: DeepReadonly<PgInsertValue<TTable>>): PgInsertBase<TTable, TQueryResult>;
    values(values: readonly DeepReadonly<PgInsertValue<TTable>>[]): PgInsertBase<TTable, TQueryResult>;

using any of the DeepReadonly definitions, e.g. here's a good one from the package ts-essentials:
export declare type Primitive = string | number | boolean | bigint | symbol | undefined | null;
export declare type Builtin = Primitive | Function | Date | Error | RegExp;
/** Like Readonly but recursive */
export declare type DeepReadonly<T> = T extends Builtin
	? T
	: T extends Map<infer K, infer V>
	? ReadonlyMap<DeepReadonly<K>, DeepReadonly<V>>
	: T extends ReadonlyMap<infer K, infer V>
	? ReadonlyMap<DeepReadonly<K>, DeepReadonly<V>>
	: T extends WeakMap<infer K, infer V>
	? WeakMap<DeepReadonly<K>, DeepReadonly<V>>
	: T extends Set<infer U>
	? ReadonlySet<DeepReadonly<U>>
	: T extends ReadonlySet<infer U>
	? ReadonlySet<DeepReadonly<U>>
	: T extends WeakSet<infer U>
	? WeakSet<DeepReadonly<U>>
	: T extends Promise<infer U>
	? Promise<DeepReadonly<U>>
	: T extends {}
	? {
			readonly [K in keyof T]: DeepReadonly<T[K]>;
	  }
	: Readonly<T>;

Seconded, similarly for conditions such as inArray and notInArray

There are many cases where developers would prefer a loose type definition like the current one. This change would also break some projects. This won't be worked on.

@L-Mario564 I'm confused - isn't allowing readonly arrays to be provided actually loosening the definition, compared to them being mutable? We're not suggesting having readonly on the output, only on the input.
function myFunc(input: string[]) {
  // do something with string
}

myFunc(["hello", "world"]); // this works
const testArray: ReadonlyArray<string> = ["hello", "world"];
myFunc(testArray); // this doesn't work: Argument of type 'readonly string[]' is not assignable to parameter of type 'string[]'.

Having the input be ReadonlyArray<string> would still allow users to provide mutable arrays, same as they always have - it would just allow users to provide readonly arrays as well.

The initial comment from OP said:

I'd suggest replacing this:

The word "replacing" is what made me close this issue. It makes sense to add but not to replace existing functionality. If adding this is what's being suggested instead of overwriting the current behavior, then sure, we can consider it. I'll be re-opening this issue due to this clarification.


hosts blocks parts of Google Play and Amazon Kindle Downloads

Hallo
I use the hosts list indirectly by using NetGuard, see Ad Blocking with NetGuard.
I noticed since a while, when I use NetGuard with that hosts file, I  have partly problems to open Google Play (I am on Android 9.0). Some icons would't show, often the download takes time or won't success. If I deactivate NetGuard (using that hosts file) I have no problems.
Similar problems my friend has with Amazon Kindle App on Android. There the Book Downloads arent working with NetGuard.
I cannot exactly tell, which addresses of Google Play or Amazon Kindle are blocked by the hosts list, but I know that it must be the list.
Is there a way to kick out these addresses of the list?
Are they known anyway?
Thanks.
frank

Hi Franc @francwalter.
We will need to know, which variant of our hosts file is used here.
Also, we will need to know which domains to consider and examine.

@francwalter  this was taken from NetGuard's manual:

You'll need to enable filtering and (temporarily) logging for this (you can do this by using the Configure button; check both options) and you'll need to wait until the address appears (you can speed this up by opening some apps with in-app ads). Note that ads are likely being cached, so this may not take effect immediately.

see if you can enable logging and post the blocked domains here

Unfortunately I cannot tell if there is a context between all the blocked addresses and Google Play / Amazon Kindle Downloads. NetGuard does not log the app which is blocked but sets only a "root" as origin of blocked addresses (its netd, running as root).
So I could give a log file of blocked addresses, but I don't know if they are parts of Google Play.
When looking into the log, I didn't see any resemblance of addresses to Google Play. But I am not very skilled at looking into logs ;)

@francwalter  try to reproduce it as close as you can(turn logging on then open Google Play then try to download something) and if possible upload the log file here so we can take a look.Also,would advise to open a issue in the developer's repo as well to make sure that it's not the app itself messing something up.

<IP_ADDRESS> clients.l.google.com
<IP_ADDRESS> clients1.google.com

Is for the google maps app. Try commenting those out.


New URL for Coinbase

Previous URL ended in a 404 when you clicked on "Close account"

Thank you for your contribution!


socks5 proxy support

hi, did you think to add support for socks5 proxies? because there is no any working socks5 proxy library with keep-alive option

Here there is the socks 4/5 implementation, but with socks dependency.
https://github.com/JoshGlazebrook/socks/issues/23#issuecomment-952790344


Seed Published Collections for E2E Test Account

Problem:
Recently we created a new Auth0 account for BE functional tests to use, in order to avoid Auth0 rate limiting. However, the data seeding script is now creating published test collections that belong to the functional test accounts instead of the E2E test account, which results in E2E tests not being able to run certain tests, like revisions
Solution:

Use smoke test account to create seeded published collections for smoke test account

Scheduling for Oct 2 on-call rotation

Use smoke test account to create seeded published collections for smoke test account

Originally only made private collections. Adding published Collections...

@danieljhegeman feels like this ticket is done now? Thank you!


added mockingbot to application list

mockingbot is a prototyping tool built with electron.

Hey @plrthink this looks pretty cool!
Your branch is out of date though. Can you rebase it?

@zeke sure, I would do that

@zeke rebased


Add Bin, Idf, Uf2 support to cargo flash

#1758
This is very rough so far.  In trying to add UF2 support to cargo flash, it helped to bring across some stuff that was only available on probe-rs so far ie. Bin and Idf support.  Is this desirable?
I've 'cheated' with errors for now through unwrap.  Should I create more error types or use anyhow? For now I've just deleted a now unused error type.  44d69bfdbec2d768aa388dbb3f142b3b7cb033af
I've used my own crate for UF2 but I'm not married to it.  I haven't evaluated other options very well yet and my crate does not have very good support for multiple sections.

I am pretty much good to merge this after the open question is resolved we can undraft and get this merged :)


Problem with autocomplete with Django Rest API

In my django project, I needed to use autocomplete with the search for the name of the products.
So I created a Rest API and wanted to use it just like you mentioned in your tutorial https://select2.org/data-sources/ajax . So i was using my rest API as a source to fetch data and I used the javascript code like you gave there just changed the url.
But it's not working.
I have a detailed question here in StackOverflow. Here is the question
I still haven't got any reply.
If you kindly take a look at the question and give a solution it would help a lot. I have been working on it for many hours and couldn't solve it.

This appears to be a usage question, rather than a bug report or feature request.  Please see our contributing guidelines.
Usage questions should be discussed on the forums instead.

Hello, sorry, I should have gone through the guidelines before posting the problems.
I apologize for that. I have created this topic on the forums. Please kindly help.


Play a sound when the sound timer is greater than 0

Currently the sound timer is implemented, and counts down to 0 when an instruction modifies it, however no sound currently plays. The intended functionality of the sound timer is to play a sound whilst the timer is above 0. This sound can be any tone using an OscillatorNode for example.
A good place to start to look into this would be in cpu.js.

What do you mean by

however no sound currently plays

Do you mean a proper bit music or just beeps when it decrements?
I started looking into it

Hey @eugeniumegherea, thanks for taking a look!
Yeah it's just a beep. There's an explanation in the Timers section on this page: http://mattmik.com/files/chip8/mastering/chip8.html
So a sine wave etc would work.

Could you take a look before I make a PR? I've implemented a beep class which plays every 16ms. I was not sure if it should play continuously until it's not 0 or play at each frame. So I made that it plays for each frame

Awesome! I'll try it locally now

Yeah that works really well, actually sounds pretty cool/retro when it plays a short burst per frame


mergeToFile not working

Version information

fluent-ffmpeg version: 2.1.2
ffmpeg version: 3.4.1 (installed using @ffmpeg-installer/ffmpeg)
OS: OSX 10.12.6

Code to reproduce
let deferred = promise.defer(); // Selenium webdriver promise, can be replaced with making a native Promise
let imgList = getFiles(inDir); // Sync get list of image names from a directory
// ... Already done check exist and empty directory, imgList is verified to have correct list of files

let fCommand = ffmpeg();
Promise.resolve(fCommand)
.then((fCmd) => {
    return imgList.reduce((result, img) => {
        result.input(path.join(inDir, img));
        return result;
    }, fCmd);
})
.then((fCmd) => {
    // console.log(fCmd._inputs) // verified to contain all of the inputs
    fCmd.outputFPS(fps)
        .outputFormat('mp4')
        .videoBitrate('1024k')
        .videoCodec('libx264')
        .outputOptions('-pix_fmt yuv420p')
        .on('start', () => {
            console.log('    Creating video...');
        })
        .on('error', (err, stdout, stderr) => {
            console.log('        Error: ' + err.message);
            console.log('        ffmpeg output:\n' + stdout);
            console.log('        ffmpeg stderr:\n' + stderr);
            deferred.reject(err);
        })
        .on('end', () => {
            console.log('        Video created.');
            deferred.resolve('Video created.');
        })
        .mergeToFile(path.join(outDir, 'movie.mp4'), path.join(outDir, 'tmp/'));
});

return deferred.promise;

Expected results
A video containing all images that are in imgList
Observed results
Only create 0 to 1 second video, show up to the second image, eliminating the rest of the list. Terminal output looks fine, no error.
It works ok if I change file names to use for example %03d.png and output(). But my file name also contains varied short description, and I can't use * like ffmpeg. Probably should use something to mark any-string?
Is there a way to just pass the inputDir and file filter like .png?

Closing, no further info given


Point to requirements section in integration section

When setting this up for the first time, I missed the # Requirements section and got a cryptic error:
pandoc paper.md -o generated/paper.pdf -d wg21/data/defaults.yaml --bibliography wg21/data/csl.json
pandoc: xelatex: createProcess: posix_spawnp: illegal operation (Inappropriate ioctl for device)
make: *** [generated/paper.pdf] Error 1

You can determine the cause with a quick google, but I figured it might help folks in the future to get a nudge to install the dependencies before trying to generate their first document.

Hi @nickpdemarco, thanks for pointing this issue out. I decided to pull the requirements section near the top instead.


CLOUD-793 - Set fixed version of jq and yq in Jenkinsfile

CHANGE DESCRIPTION
Problem:
Different tool versions can cause issues with the tests.
Solution:
Fix version.
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are the manifests (crd/bundle) regenerated if needed?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PG version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

demand-backup
passed

init-deploy
passed

monitoring
failure

operator-self-healing
passed

scaling
passed

scheduled-backup
passed

self-healing
passed

start-from-backup
passed

telemetry-transfer
passed

users
passed

We run 10 out of 10

commit: https://github.com/percona/percona-postgresql-operator/pull/488/commits/4dd1e442941504a0e1cbb8f46dc6af9e7355f3ab
image: perconalab/percona-postgresql-operator:PR-488-4dd1e4429

Test name
Status

demand-backup
passed

init-deploy
passed

monitoring
passed

operator-self-healing
passed

scaling
passed

scheduled-backup
passed

self-healing
passed

start-from-backup
passed

telemetry-transfer
passed

users
passed

We run 10 out of 10

commit: https://github.com/percona/percona-postgresql-operator/pull/488/commits/4dd1e442941504a0e1cbb8f46dc6af9e7355f3ab
image: perconalab/percona-postgresql-operator:PR-488-4dd1e4429


Adding PimpleDi plugin

https://packagist.org/packages/rochamarcelo/cake-pimple-di

Are those two in correct alphabetical order? I think they are not.
After fixing this should be good to go for merging.

Sorry,  now it's in correct alphabetical order.

Cool. Sry for being dense here ;) I could also fix this post merge.
PS: What do you think of Allows dependency injection based on Pimple library. That it is a plugin doesnt need to be repeated, it is already the first words in that line.

Sounds good.


Date Helper

The date helper should work with DateTime and related classes to provide our date/time functionality. The list of features within the current helper need to be examined. Not all of them are relevant or the best way to do things now.
MUST integrate with whatever solution is determined for our i18n/i10n integrations to default to converting dates/times to current locale, where applicable.
Exact methods still to be determined.
Development Checklist:

[ ] Component(s)
[ ] ... with PHPdocs
[ ] Unit testing
[ ] ... with >80% coverage
[ ] User guide updated
[ ] Classmap integration?
[ ] Securely signed commits
[ ] Conforms to style guide

Here are the current date helper functions, per the CI3 user guide, with type hints added (with some uncorrected errors, especially in default argument types):

[ ] now(string $timezone = null): int returns time() or the equivalent timestamp for $timezone
[ ] mdate(string $datestr = '', int $time = ''): string Format $time using MySQL-style date codes in $datestr
[ ] standard_date(string $fmt = 'DATE_RFC822', int $time = null): string deprecated, Format $time using the format indicated by $fmt, which is a string indicating one of the DateTime format constants
[ ] local_to_gmt(int $time = ''): int Convert local $time to GMT
[ ] gmt_to_local(int $time = '', string $timezone = 'UTC', bool $dst = false): int Convert GMT to local time
[ ] mysql_to_unix(string $time = ''): int Convert MySQL Timestamp to UNIX timestamp
[ ] unix_to_human(int $time = '', bool $seconds = false, string $fmt = 'us'): string Converts a UNIX timestamp to a "human-readable" timestamp in 12-hour (with AM/PM) format ('us') or 24-hour format ('euro').
[ ] human_to_unix(string $datestr = ''): int Reverse of unix_to_human()
[ ] nice_date(string $bad_date = '', string $format = 'U'): string Attempts to convert $bad_date into a date string in the supplied $format or a UNIX timestamp
[ ] timespan(int $seconds = 1, int $time = '', int $units = 7): string Returns a string indicating the time difference between $seconds and $time using strings from date_lang.php to translate the words years, months, weeks, days, hours, minutes, and seconds.
[ ]  days_in_month(int $month = 0, int $year = ''): int Returns the number of days in a given month/year.
[ ] date_range(int $unix_start = '', int $mixed = '', bool $is_unix = true, string $format = 'Y-m-d'): array Returns an array of date strings with the given format in the specified period
[ ] timezones(string $tz = '') Returns the timezone offset for the CI timezone indicated by $tz, or an associative array of CI timezones and their offsets.
[ ] timezone_menu(string $default = 'UTC', string $class = '', string $name = 'timezones', $attributes = ''): string Returns an HTML select element containing a list of CI timezones/offsets (the list is provided by timezones())

Many of the documented types or default values in either the documentation or the code comments are incorrect. If I have time, I might submit a PR to fix that in CI3.
In my opinion:

If the CI timezones are supported at all in CI4, it should be in the form of a compatibility method/function to convert them to PHP timezones.
The 'us'/'euro' format names for unix_to_human() should be changed, if the function will be kept. The names are misleading, as most would likely believe that they indicate a difference in the placement of the month in the output, and possibly the use of / vs. -, when the only change between the two formats is the use of a 12- or 24-hour format (plus AM/PM in the case of 12-hour).
Since PHP's DateTime accepts a MySQL timestamp in the constructor, the usefulness of mysql_to_unix() is probably limited (there's a note in the code about MySQL 4.1 compatibility, in which case it would likely be best to use DateTime::createFromFormat())
Since human_to_unix() expects the format to be the output of unix_to_human(), the 'euro' format would work directly with the DateTime constructor, while the 'us' format would require DateTime::createFromFormat().
In addition to the individual words, the format of the output of timespan() should be localized
timezone_menu() is probably better suited to an HTML helper/builder, and the arguments can be condensed, since $name and $class can be included in $attributes. It would also be better to call the appropriate HTML helper/builder function/method to build the select element

It's likely that some of the existing code may be more efficient than creating a DateTime object. Since many of these deal with UNIX timestamps and timezone conversions, anyone attempting to rewrite these functions to utilize DateTime should read the PHP manual entries (and associated reader comments) carefully, since the seemingly obvious choice of creating the DateTime in one timezone, then calling $someDateTime->setTimezone($newTimezome)->format('U') to get the new timestamp is not going to work. This may serve as an explanation for the last three lines of the now() function:
$datetime = new DateTime('now', new DateTimeZone($timezone));
sscanf($datetime->format('j-n-Y G:i:s'), '%d-%d-%d %d:%d:%d', $day, $month, $year, $hour, $minute, $second);
return mktime($hour, $minute, $second, $month, $day, $year);

This also might serve as the best argument for keeping some of these functions, or at least for determining what functionality may be useful in a new date helper.
However, the obvious use of the DateTime::setTimezone() method does work when using formats other than UNIX timestamp for the output, which means the extra conversions can be eliminated if you're formatting date/time information for display or for use in a database (unless the database's field requires a UNIX timestamp).

Very good points, Mat. And thanks for pulling all of that together!
And part of the reason why there's no "Help Wanted" tag on this one yet. I'm pretty sure we'll be incorporating PHP's intl extension and making helper methods for that, but we obviously haven't talked about how it's all going to work together just yet, and whether it's all localized by default if we have localized/non-localized versions, etc.

Anybody here interested in Carbon? See http://carbon.nesbot.com/docs.

Carbon's great, but you see the response we get every time we suggest using a third-party library on the forums :)

Oh yeah. Can't deny that one.

Carboon is very good tool and in fact it has everything what we need.
But its a standalone lib using Symfony Translation component which is against the logic we have...
If nobody is working on this helper I can work over it.

I hope you guys see https://github.com/cakephp/chronos.

I'd vote for Carbon as a baseline similar to Kint. Just include minor tweaks to make it CI4.

Chronos looks too complicated but their idea to extend DateTimeImmutable  is ok for CI 4 as we got PHP 7 to look over.. Our solution should be something in the middle..

Chronos has some nice features, but I haven't jumped on the immutability bandwagon just yet. :)
@sv3tli0 I'd love to have someone tackle this.  We need to make sure this works with the Locale, also, if it's set.

Like Carbon/Chronos but simple!
So we can do add operations like so:
$dt = Dtm::create(2012, 1, 31, 0);
echo $dt->addYears(5);                   // 2017-01-31 00:00:00
echo $dt->addYear();                     // 2018-01-31 00:00:00
echo $dt->addYears(-1);                     // 2017-01-31 00:00:00
echo $dt->addYears(-5);                   // 2012-01-31 00:00:00
echo $dt->addMonths(60);                 // 2017-01-31 00:00:00
echo $dt->addMonth();                    // 2017-03-03 00:00:00 equivalent of $dt->month($dt->month + 1); so it wraps
echo $dt->addMonths(-1);                    // 2017-02-03 00:00:00
echo $dt->addMonths(-60);                 // 2012-02-03 00:00:00
echo $dt->addDays(29);                   // 2012-03-03 00:00:00
echo $dt->addDay();                      // 2012-03-04 00:00:00
echo $dt->addDays(-1);                      // 2012-03-03 00:00:00
echo $dt->addDays(-29);                   // 2012-02-03 00:00:00
echo $dt->addWeekdays(4);                // 2012-02-09 00:00:00
echo $dt->addWeekday();                  // 2012-02-10 00:00:00
echo $dt->addWeekdays(-1);                  // 2012-02-09 00:00:00
echo $dt->addWeekdays(-4);                // 2012-02-03 00:00:00
echo $dt->addWeeks(3);                   // 2012-02-24 00:00:00
echo $dt->addWeek();                     // 2012-03-02 00:00:00
echo $dt->addWeeks(-1);                     // 2012-02-24 00:00:00
echo $dt->addWeeks(-3);                   // 2012-02-03 00:00:00
echo $dt->addHours(24);                  // 2012-02-04 00:00:00
echo $dt->addHour();                     // 2012-02-04 01:00:00
echo $dt->addHours(-1);                     // 2012-02-04 00:00:00
echo $dt->addHours(-24);                  // 2012-02-03 00:00:00
echo $dt->addMinutes(61);                // 2012-02-03 01:01:00
echo $dt->addMinute();                   // 2012-02-03 01:02:00
echo $dt->addSeconds(61);                // 2012-02-03 00:01:01
echo $dt->addSecond();                   // 2012-02-03 00:01:02
echo $dt->addSeconds(-1);                   // 2012-02-03 00:01:01
echo $dt->addSeconds(-61);                // 2012-02-03 00:00:00
$dt->year   = 2015;
$dt->month  = 04;
$dt->day    = 21;
$dt->hour   = 22;
$dt->minute = 32;
$dt->second = 5;
PHP does the rest...


Initial loading flicker on "Our Profile" tab

When navigating to the profile tab there is a very short loading state flicker before it's populated.
We could preload the data to avoid this, or the other common solution to problems like these is add a minimum loading time if the view is presented.
https://github.com/subconsciousnetwork/subconscious/assets/5009316/06501a82-83f5-46d5-a2d2-9cdc8578accb

I believe that I (inadvertently) fixed this in https://github.com/subconsciousnetwork/subconscious/pull/990


Fix icons-font example with workspaces

In relation to https://github.com/twbs/examples/pull/114#issuecomment-1492835747
Something like this @XhmikosR?

I tried your branch on StackBlitz but it doesn't seem to work either :/
https://stackblitz.com/github/kyletsang/examples/tree/workspaces-icons/packages/icons-font?file=index.html
Not against such a solution yet, thanks for looking into it!

Oops forgot to check if this worked on StackBlitz.  Not sure of a clean way to do this 😞

No worries! I guess I'll drop the workspaces experiment for the time being :/


Type narrowing error in async function while return type mark as NoReturn

Environment data

Language Server version: 2023.8.41 (23 August 2023) PreRelease
OS and version: win11
Python version (& distribution if applicable, e.g. Anaconda): 3.9

Code Snippet
import asyncio
from typing import NoReturn, Optional

async def raise_exc() -> NoReturn:
    raise Exception()

def get() -> Optional[int]:
    ...

async def main():
    x = get()  # int | None
    if x is None:
        await raise_exc()
    x.as_integer_ratio()  # pylance: "as_integer_ratio" is not a known member of "None"

asyncio.run(main())

Expected behavior
after await raise_exc(), x should be inferred as an integer.
Actual behavior
it is inferred as int | None.
Extra
but in the synchronous function, pylance will provide the correct inference:
from typing import NoReturn, Optional

def raise_exc() -> NoReturn:
    raise Exception()

def get() -> Optional[int]:
    ...

def main():
    x = get()  # int | None
    if x is None:
        raise_exc()
    x.as_integer_ratio()  # int

main()

Could someone on the pylance team please transfer this to the pyright project, since it's a core type checker issue? Thanks!


Why do we average out correlation matrices from different GPUs? Is this mathematically valid?

Thanks for this great work!
I am a bit confused about the computation of the Barlow Twins loss in the multi-gpu setting. If I understand it correctly, each batch is split into smaller minibatches and these are then processed on separage GPUs. Each GPU computes the cross correlation matrix corresponding to its minibatch. The cross correlations between samples on different GPUs are not computed. It is not clear to me, why the different cross correlation matrices are averaged out across GPUs. This creates a mean correlation matrix and this one is then used for loss computation.
Why not compute the loss for each correlation matrix separately and only average out the final loss?
Or even better, why not compute the full cross correlation matrix (i.e. gather all embedding vectors onto one device and computing the cross correlation there?)
I fail to see why summing up correlation matrices is a valid mathematical operation - or is it just an implementation "hack" that makes things easier? I guess since all cross correlation matrices are ideally converging towards identity matrices (as forced by the loss function), avereging them out does not strictly break the convergence - is that the case?
I am not very experienced with distributed deep learning so there may be technical things I don't understand. Thanks for your help.
https://github.com/facebookresearch/barlowtwins/blob/a655214c76c97d0150277b85d16e69328ea52fd9/main.py#L206-L223

Hi, how do we do this on a single GPU? Because torch.distributed seems not to work on a single GPU? I use torch.nn.DataParallel on to run the code on a single GPU.
# empirical cross-correlation matrix
 c = self.bn(z1).T @ self.bn(z2)

 # sum the cross-correlation matrix between all gpus
 c.div_(self.args.batch_size)
 torch.distributed.all_reduce(c)

 on_diag = torch.diagonal(c).add_(-1).pow_(2).sum()
 off_diag = off_diagonal(c).pow_(2).sum()
 loss = on_diag + self.args.lambd * off_diag

Hi @ltnghia .
Is the problem solved? I think the following code may not be needed, right?:
sum the cross-correlation matrix between all gpus
c.div_(self.args.batch_size)
torch.distributed.all_reduce(c)

Or even better, why not compute the full cross correlation matrix (i.e. gather all embedding vectors onto one device and computing the cross correlation there?)

Summing cross correlation matrices (like we do in our code) is equivalent to computing the full cross correlation matrix by gathering all embedding vectors onto one device. They give you exactly the same result.
Think about how to distribute a dot product operation across n machines (computing the cross correlation matrix is basically just a bunch of dot products, one for each pair of features). You could split the vectors into n chunks, compute n smaller dot products (one dot product for each of the n chunks) and sum them to get the final result. Or if you prefer code:
>>> import torch
>>> x = torch.Tensor(8).normal_()
>>> y = torch.Tensor(8).normal_()
>>> torch.allclose(x[:4] @ y[:4] + x[4:] @ y[4:], x @ y)
True

In the above mention, I can't understand why we should divide by batch size (c.div_(self.args.batch_size))
For me,  the code below might be enough.
c = self.bn(z1).T @ self.bn(z2)
torch.distributed.all_reduce(c)
Thanks!


Schema updates

After running gradelw check I've noticed the schemas are updated. Not sure where this is defined, but this should be a separate task that we can run in the CI to keep it up to date.

They are refreshing with the tests.

Are they shipped and used by the plugin, or only by the tests?

the tests are updating the schemas which are used then by the plugin for the IDE. I have seen that 2024 has these schemas already, but i have no idea about other IDE's. They don't harm anyone, so i would keep them for now and focus on more urgend things first