[Taskwarrior] fails to parse task JSON after upgrading to Taskwarrior v3.0, if you leave the old .data files behind

Extension
https://www.raycast.com/yosy2010/raytaskwarrior
Raycast Version
1.75.2
macOS Version
14.5
Description
After upgrading to Taskwarrior v 3.0.2 and exporting/importing tasks to the new database format, the Raycast integration can no longer list my tasks:

The error message is misleading:  the path to task is unchanged and set correctly:

~► which task
/opt/homebrew/bin/task

Instead, I believe that the JSON is no longer parsing correctly with the new version of Taskwarrior.  The error appears to be coming from line 23 in the api.ts file (upstream).
    const data = JSON.parse(stdout) as Task[];
    if (data) tasks = data.sort((a, b) => b.urgency - a.urgency);
  } catch (error) {
    throw new Error("Please make sure you have set the path to task in the settings");

Version 3.0 of Taskwarrior is a breaking change. The format of the task database was changed, but this should be transparent to the extension.
I think what has happened is that the task command will display this warning, which is breaking the extension:
~► task export rc.json.array:on > tasks.json
Found existing '.data' files in /Users/mjl/net/MEGA/task
  Taskwarrior's storage format changed in 3.0, requiring a manual migration.
  See https://github.com/GothenburgBitFactory/taskwarrior/releases.
Configuration override rc.json.array:on

Workaround

Remove the .data files from your taskwarrior data directory

Steps To Reproduce

In a system which has upgraded from Taskwarrior v2.x to v3.x, and has .data files left from the export/import
Run the command "List Active Tasks"
See error in Description
Run the task command executed by the extension from a terminal
See the error message displayed by task
Remove the .data files as advised
Extension working as expected

Current Behaviour
The extension errors with "Please make sure you have set the path to task in the settings" even when the path is correct
Expected Behaviour
The extension should either explicitly state there was a problem parsing the JSON output by task (rather than assume all errors are because of a misconfiguration), or produce the error thrown by the command in some sort of pop-up display, for the user to diagnose and fix.

I have this problem with a fresh install of Taskwarrior (3.0.2). task does not display any errors, there is no .data file I could remove to work around this bug. The path is set correctly in the extension.

@uhkeller Could you try running the task command which the Taskwarrior Raycast extension is running, to see what error it produces? That could reveal what the problem actually is. I suspect that if you don't have a .data file, then you did not upgrade from an earlier Taskwarrior, so you might have a different problem.
task export rc.json.array:on > tasks.json

(disclaimer:  I am not the extension's creator/maintainer)

Thanks for your response, @sinewalker. There is no error running this command, just the override message Configuration override rc.json.array:on. If i disable override messages in .taskrc the message is gone from terminal output put the extension still doesn't work.

How strange!
I wonder whether there is something causing a delay that the extension doesn't wait for? (I'm just guessing now).
Is your data.location on a network volume or something which might be intermittently slow? If so, try moving it locally and see if it helps. If not... I don't know

Strange indeed. No, the data are in the default location on my SSD. I'll try to find some time to properly debug this next week.

I've decided to downgrade to Taskwarrior 2 until taskwarrior-flutter and/or WingTask have caught up with TW 3. This also solves my Raycast problem for now.


Make migration container aware of installed gems

When trying to set up my local environment I realised the migration container crashes because it doesn't know about gems
This fixes it, but I'm not sure why I would be the only one having this issue ? Maybe this container is not used and people just exec a rails db:migrate into the web one ?

I think it's more about people not using the docker version 😆

I usually only dockerize the databases and services, we could have a second docker compose file for that eventually, if that can help new comers

To be honest when i was doing the setup i was thinking if it was not too overkill to make it fully dockerized ahah.
You can edit the current docker-compose.yml and make it simpler with just the external services and the database as you said to be honest.

Ok I can transform this PR into changing the docker config to half dockerized and change the docs accordingly.
Only if you feel this won't harm potential contributors already using this :)

If nobody reacted before you posted your issue, about the container migration, you are free to go i think. And even if some people were running with the previous version is not a big deal, quickly upgradable

Done in #727, closing this !


feat: new camera

This pr adds new qr camera.
Result
Flow:
https://user-images.githubusercontent.com/102026658/233015082-d9b2d290-df4a-40b5-8e31-c57aa0bbc759.mov
Error screen:

@denis-ottofeller Can you check out the build error before merging?


Multiverse plugin management

world based plugin management
As many times plugins may conflict each other due to their function, this may help to enable/disable plugins in certain Worlds.

As I am a regular user of multiverse, many times I face a problem when two features from different plugins conflicts.
For example: In my case, Skywars gave it's custom items(for game join) to players whereas the server also had survival
Also I can't find any plugin like this for spigot 1.16.1 ("per world plugins" is there for spigot 1.8 but it is discontinued for any other update). Hope this request find a good place in this project
Thankyou...

Unfortunately, multiverse should not alter how other plugin works. Usually good plugins will have config to disable their own features in certain worlds. If that plugin uses permissions nodes, you can also use a permission manager like luckperms and its context features to set permission/command access to only certain worlds you want.


Double call to the command prompt doesn't work

The plugin vim-sandwich provide a special type of surrounding which requires to input to value in the command prompt (instant surrounding).
For example, saiwi[command_prompt]foo<CR>[command_prompt]bar<CR> makes a word surrounded by foo and bar.
However, in VSCode, the second command promp does not appear.

neovim sends cmdline_hide event following by cmdline_show. Hiding on first event triggers onHide which sends Esc to neovim


Maxlen property always set to None, when deque item type is provided.

Initial Checks

[X] I have searched GitHub for a duplicate issue and I'm sure this is something new
[X] I have searched Google & StackOverflow for a solution and couldn't find anything
[X] I have read and followed the docs and still think this is a bug
[X] I am confident that the issue is with pydantic (not my code, or another library in the ecosystem like FastAPI or mypy)

Description
The maxlen property of the deque field seems to be set to None when the queue item type is provided.
In the example below I first create an untyped deque and upon instancing everything goes well - the maxlen is set to 15.
However when the model definitions is more specific: deque[int] or Deque[int] then the maxlen value cannot be set.
Example Code
>>> from pydantic import BaseModel
>>> from collections import deque
>>> class D(BaseModel):
...     q: deque = deque(maxlen=10)
...
>>> d = D(q=deque(maxlen=15))
>>> d
D(q=deque([], maxlen=15))
>>>
>>>
>>> class DI(BaseModel):
...     q: deque[int] = deque(maxlen=10)
...
>>> d = DI(q=deque(maxlen=15))
>>> d
DI(q=deque([]))
>>>
>>> from typing import Deque
>>>
>>> class DT(BaseModel):
...     q: Deque[int] = deque(maxlen=10)
...
>>> d = DT(q=deque(maxlen=15))
>>> d
DT(q=deque([]))

Python, Pydantic & OS Version
pydantic version: 1.10.5
pydantic compiled: True
install path: /opt/venv/lib/python3.11/site-packages/pydantic
python version: 3.11.4 (main, Jun 13 2023, 15:08:32) [GCC 10.2.1 20210110]
platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.31
optional deps. installed: ['typing-extensions']

Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .dict() and .json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[X] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

Selected Assignee: @davidhewitt

Some more cases:
>>> from collections import deque
>>> from pydantic import BaseModel
>>> class D(BaseModel):
...     q: deque[int] = deque(maxlen=10)
...
>>> d = D()
>>> d
D(q=deque([], maxlen=10))

>>> d.q = deque(maxlen=25)
>>> d
D(q=deque([], maxlen=25))

So it seems that it is validation issue on model creation only.
I do not know the codebase here but potentially:
https://github.com/pydantic/pydantic/blob/f35c780cc3cf073c400ee038b8b6dd6d7b5703d9/pydantic/v1/fields.py#L954C25-L954C25

Thanks @maciekglowka for this issue 🙏
As you may know, Pydantic V2 is out and we are not actively working on V1.
I've checked your first example code on V2, all of them returning D(q=deque([], maxlen=15))
Same for you second example. In V2 you can have validation on assignment to a deque field like:
from collections import deque
from typing import Deque

from typing_extensions import Annotated
from pydantic import BaseModel, ConfigDict, Field

class D(BaseModel):
    q: Annotated[Deque[int], Field(deque(maxlen=10), max_length=10)]

    model_config = ConfigDict(validate_assignment=True)

d = D()
d.q = deque([1] * 25)

Note: We still accept bug fixes on V1 but it should be an easy fix.

Thanks for checking.
Yeah I've just forked the repo. If I'll be able to fix this I'm gonna make a PR.

@hramezani I've done a very simple fix (PR passes tests) - I am just rewriting the single attribute. Perhaps the rewrite should be less manual? But I am not really sure what happens under the hood in Pydantic.

Fixed in https://github.com/pydantic/pydantic/commit/2aaddf6d692a6a467a401fae5840435c5459aa1b


React survey readme

React survey added to README.md.
Thanks Jason for letting me know to add additional surveys to sub MSAL.js libraries.

Do you want to close your other PR? And only MSAL React, or would you like to add the surveys to the other libraries in the lib folder?

Do you want to close your other PR? And only MSAL React, or would you like to add the surveys to the other libraries in the lib folder?

I need all changes made to each of the README.md files to include the survey. Would you rather me do one massive PR of all the updates?

Do you want to close your other PR? And only MSAL React, or would you like to add the surveys to the other libraries in the lib folder?

I need all changes made to each of the README.md files to include the survey. Would you rather me do one massive PR of all the updates?

@ShannonCanTech Yeah, one PR for all the libraries would be great. Thanks!

@ShannonCanTech What is the status on this PR?


Support dependencies on Snack player

Why
This PR add support for additional Snack dependencies to the remark-snack-player. This allows for fixing missing expo-constants warnings and makes the examples future proof, as Snack moves toward a direction where dependencies are fully optional.

Secondly, this PR makes it possible to support dependencies that are not loaded by default in Snack, such as @react-native-community/slider, which replaces the default Slider component. This in turn fixes web preview for all examples that use the Slider component, which doesn't work on web. Migrating these examples to @react-native-community/slider fixes the web support problem allows for showing the preview in the browser directly.

How

Add support for &dependencies=... query param
Updated snack player docs
Updated snack player tests and added dependencies
Per example updated the "Text style" example, so it also runs on web

Test plan

Tests were updated/extended for the remark snack player
Updated text style example

https://snack.expo.io/@hein_expo/textstyleprops

Hi @Simek! I understand what you are saying and agree with keeping examples to a bare minimum. Was just trying to contribute in making sure stuff work on all platforms. And totally understand that web is outside the maintenance area of this repo. Although it is featured on the introduction page, which makes you wonder about its importance and usefulness. Personally I find the ability to target web as well an amazing feature and as it matures this will only grow.  No worries for deleting the other PRs, they were only a couple mins work. Cheers m8 👍


Embed go-ipfs/kubo via webassembly

Is your feature request related to a problem? Please describe.
I am frustrated by js-ipfs being the only option in regards to the ipfs daemon embedded into the ipfs companion extension.
Describe the solution you'd like
I would like to see go-ipfs/kubo embedded into IPFS companion by compiling it to webassembly. The go compiler supports webassembly as a target, and chrome extensions support webassembly, so IPFS companion should support go-ipfs/kubo. It will bring IPFS companion up to speed in regards with IPFS desktop in regards to Chromebooks, like the one I am stuck with at the moment.
Describe alternatives you've considered
The alternative would be to bring js-ipfs up to speed with the features and transports of go ipfs.
Additional context
Not necessarily.

@Destroyinator69420 Thanks for your contribution, we appreciate the feedback. This is a significant effort and is currently blocked by https://github.com/ipfs/kubo/pull/5878 AFAIK.
As far as js-ipfs supporting everything that Kubo supports, that is unlikely to happen, and we're going a different route. See https://blog.ipfs.tech/state-of-ipfs-in-js/ and https://github.com/ipfs/helia.
We are open to PRs and would also love to see better support here if you can contribute.


Show the amount in $ on account overview and Tx sending screens

Nobody knows how much 0.001Eth is worth.

On an account overview
When sending a Tx: We should do the math for the user and show the equivalent in $ for the amount and the gas.

@Rakhisharma, thanks for proposing your help, no one is working on the issue yet.
I would show the amount in $ like this (suggestion):
Account screen:

USD smaller (and a little ligher?)
Only for ETH, not for tokens

Tx screen:

add "ETH" to the amount
add the USD value at the top (available balance) and for the typed in value
change the GWEI to USD value for gas and add a tooltip (on mouseover) for the selector in GWEI

Thanks for the details @Tbaut.
I am assuming by Ty you mean typed amount. I agree with you but generally, when we send some amount to other account normally we think of how much ETH I need to send. But again when we calculate total amount in dollar for user, it make user experience more rich!
I was going through the codebase, I was able to successfully setup and run the Fether on my local. one question-
What tools you use for debugging? I can see here we are using electron, I am not sure how I can debug UI of Fether. In browser normally developer tools works and for react, we have react developer tool.

I am assuming by Ty you mean typed amount

yes, typo, I meant "Tx" amount.
I'll just copy/paste what I wrote in the chat:
When launching yarn start as explained in the readme you have the ability to see the changes directly (ignore the browser tab that is open and concentrate on the electron window).
Development tools are available just like in a browser when doing Ctrl-shift-i on the Electron window.

@axelchalon, any other tool that is worth mentioning?

nothing else that comes to mind!

@Tbaut thanks for the reply- I think I have made some progress, most probably I will commit my WIP. Right now I am stuck at one point- how to get the values of ETH and US dollar here as it changes every time?  Do we want to use any API to get the updated values every time? Or we already have this in our code base? if yes, please point me out to the relevant code. Thanks :)

@amaurymartiny might also have an opinion :)

@amaurymartiny thank you for the reply. I have created a PR in WIP state. I will take a look into the documentation.

One of the most decentralized way of getting the USD/ETH price is by using the price feed from Maker also see an interesting discussion about it on reddit

@Tbaut @Rakhisharma Is anyone still working on this? Is this the latest WIP branch? https://github.com/paritytech/fether/commit/8697c9453a92c86141a858224e4b10ee5de63e52
Alternatively use BlockScout API and do request for Account > getprice https://blockscout.com/eth/kovan/api_docs

Nobody is working on it but since this is a critical piece (where users can be tricked), it needs to be truly decentralized, or not to be. Relying on 1 API, even open source, is not enough.


ci: fix windows gh actions

resolves #130

Codecov Report

Merging #204 (cfed4fe) into develop (0c850a6) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop      #204   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines          393       393
=========================================
  Hits           393       393

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 0c850a6...cfed4fe. Read the comment docs.


add C implementation for @stdlib/math/base/special/frexp

Resolves #637
Checklist

[x] update readme.md
[x] include.gypi
[x] binding.gyp
[x] include/stdlib/math/base/special/
[ ] manifest.json
[ ] src
[ ] lib
[ ] examples
[ ] benchmark
[ ] test

@kgryte

Closed in reference to #651


Add GradCAM elementwise

Added GradCAM ElementWise implementation as a proposed by this publication: https://www.aaai.org/AAAI21Papers/AAAI-8236.PillaiV.pdf
When tested on a ResNet50 network, and changing the last avg pooling layer for a max pooling layer, GradCAM element wise produces more accurate localizations of object activations (measured by counting overlapping pixels between an object bounding box and the cam activation)

Thanks for the contribution !

Thank you for your guidance!


how to raise the prompt efficiency

thx for your work. since the scene graph need to be generated as the first step, and composition all inputs at the second step, though seem to be effective, the prompt costs raised, especially  when meeting super large size mllms, is there any trick to improve the efficiency for the prompt, maybe v2?
:)

Hello, thanks for your interest in your work. The generation of the scene graph is a crucial part of the method. We do find relatively small scene graphs to work quite well, but nevertheless, there isn't a straightforward way to avoid the computational cost of generating the scene graph.
Here are some ideas though:

Lower the number of max tokens outputted.
Use few-shot prompting (with compact scene graph examples) or an instruction prompt that tells the LLM to keep the scene graph short.

Other than that, I imagine you would need architecture changes like sparse or linear attention. We leave this open to future work that we encourage you or others to pursue.
Hope this helps!


Maximum amount of topics to link to overarching categories

There is a maximum amount of characters that can be linked to a 'domain' (which again: should be overarching category). Can we delete this maximum amount? I can't fill in all the fixed relationships this way.

Fixed for overarch/themes, overarch/topics, and themes/topics relations.
It is now a 'select synomym friendly' selection with autocomplete.
https://landportal.info/taxonomy/term/7340/edit?destination=admin/structure/taxonomy/domains

Hi Jules, I still can't add more concepts. It's giving me the same error. I reached the maximum amount of characters (which is 1024)


Clean up asset classes

Proposed Changes

Updated assert classes to improve option handling

Associated Issue

Fixes  #2435

Architecture changes

Remove this section if not used

Merge Checklist

[x] Tests added/fixed
[ ] Linting Complete

Only PR's with test cases included and passing lint and test pipelines will be reviewed
@ohcnetwork/care-backend-maintainers @ohcnetwork/care-backend-admins

blocking for

#2483

@sainak  made the changes. Please check


V3 core

R: @jeffposnick @addyosmani @gauntface @philipwalton
This is a PR start to build up for V3 so please hammer on this for anything you see and don't like / disagree with.
Notes in terms of changes...........
Eslint
Been slightly more strict on where console is used - this is solely to force the use of a log helper which can add prefixs and colors to helper figure out where logs are coming from in build process and tests.
I moved us off of the gulp-eslint plugin as it couldnt be configured to use our gitignore (i.e. we'd have to duplicate ignores in eslintignore or gulp.src). I checked that we fail the build if there is a lint problem.
Testing
To enable @philipwalton's use case of having access to raw classes I plan on tweaking his approach of creating bundle files where our build process will look for files under test/bundle/node/, test/bundle/browser/ and test/bundle/sw/ this will result in a built bundle consisting of all JS files under this directory and the test runner (which I will add in at a later stage) will basically add scripts from browser/, node/ and sw/ along side the bundles in the respective environment.
Please Comment On This: The bundled tests are currently built and run with no NODE_ENV as well as a NODE_ENV of 'production'. The idea here being that if code changes it's behavior between environments, the tests can check the NODE_ENV and alter their behavior accordingly (i.e. ensure something exists or doesn't exist based on the NODE_ENV). I'm happy to strip this out so we only build one set of tests OR we can use flags to determine what node_env is run OR add multiple gulp tasks (one for prod one for dev). Just wondering what people would want / not want for this.
Gulp
I've switched to gulp 4 so we can get better Promise support.
I took the idea of task runner and altered it into packageRunner, the idea is that it wraps functions that perform the gulp work into an array of functions, one item for each package it should run on (i.e. --project vs no --project). This works well with gulp.series and gulp.parallel - I added a big readme to the file to explain.

Been slightly more strict on where console is used - this is solely to force the use of a log helper which can add prefixs and colors to helper figure out where logs are coming from in build process and tests.

Encouraging use of the log helper class, especially if this makes it clearer for us to strip out particular levels of warnings in production builds at some point, sounds reasonable.

I took the idea of task runner and altered it into packageRunner, the idea is that it wraps functions that perform the gulp work into an array of functions, one item for each package it should run on (i.e. --project vs no --project). This works well with gulp.series and gulp.parallel - I added a big readme to the file to explain.

My main comment here is that re-reading the packageRunner source a few times, it would be nice to see a few more examples of what the typical function being passed to it might look like. I naively assumed it can literally be anything returning a stream.

@addyosmani packageRunner just needs to be given a function that returns a Gulp Stream or a Promise. What that stream does (or promise does) doesn't matter.
The main idea is to just stop any concern or thinking of what the task should be running against.

@gauntface That's much better. Thanks!

This is great, @gauntface! I left a couple of other small questions after another round of review, but I think we're generally good to move ahead.


Can't edit variables in Grafana 7.0.0

Describe the bug
With Grafana 7 i can t edit / add a new variable anymore, instead a form, it s just blank

Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

7.0.0
4.4.4
3.11.0

Same issue here. Other variables work just fine, but for Zabbix datasources, I just get the blank screen.
Grafana: 7.0.0
Zabbix: 5.0.0
Grafana-Zabbix Plugin: 3.11.0

I had the same problem, but with the zabbix problems. The screen is clean, as if there was no incident.
Grafana: 7.0.0;
zabbix: 4.0.20
Plugin: 3.11.0

Fixed in fde56d24e93425132cc11abf5152aa5f681bdf02

Hi everyone!
I do this Fixed in fde56d2 but is not working. Do you have the other alternative?
Zabbix 4.2
Grafana 7.0
With zabbix 4.4 I have de same problem to that zabbix 4.2 in grafana variable.
Thanks!

@dufaustoitu Clear the browser cache :)


Libxml2212

The normal architectures will fail for the first hour. But I wanted to get started on the linux-aarch build.
This is a linux only migration, so I'll probably leave windows and osx unbuilt
Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

on linux aarch:
/home/conda/feedstock_root/build_artifacts/qt-webengine_1702352744401/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pla/bin/../lib/gcc/aarch64-conda-linux-gnu/11.4.0/../../../../aarch64-conda-linux-gnu/bin/ld: /home/conda/feedstock_root/build_artifacts/qt-webengine_1702352744401/_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pla/lib/libQt5WebEngineCore.so:
undefined reference to<EMAIL_ADDRESS>
real fun. i deleted the 1.5 GB log trying to tar it.

linux_.zip
This is now ready for review.
Packages available at: https://anaconda.org/mark.harfouche/qt-webengine/files


"FB:< echo ucmd print"  Example description and successful use?

Has anyone successfully used the shell command?
I added  simple "FB:< echo ucmd printenv serial#"
and I get:
1:2>Start Cmd:FB: ucmd printenv serial#
New USB Device Attached at 1:2
1:2>Fail Failure claim interface(0.201s)
1:2>Fail Failure claim interface(0.201s)
Am I missing something?
I would like to use this to get info so I can generate a MAC address for my boards.
Thanks

run at windows or linux?
is it success if direct use
FB: ucmd printenv serial#

Sorry, it is running on Linux but not working...

Also, everything else works in my scripts...

I duplicate your problem. let me try to fix it

Amazing speed.. It works as advertised.
Thanks

Can you try https://ci.appveyor.com/project/nxpfrankli/mfgtools-kvqcg/builds/33490219/job/43lwh43mq1dwea6q/artifacts?

I pulled the latest git changes and they worked.
Thanks
Floyd Brown
On Fri, Jun 12, 2020 at 2:44 PM nxpfrankli<EMAIL_ADDRESS>wrote:

Can you try
https://ci.appveyor.com/project/nxpfrankli/mfgtools-kvqcg/builds/33490219/job/43lwh43mq1dwea6q/artifacts
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NXPmicro/mfgtools/issues/193#issuecomment-643429668,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHF3BXFKQCPKKAPIULCAJOTRWJZRTANCNFSM4N4OHDNA
.


Can not logout

I can not logout, there is no menu, and if I delete the App, the config account still remains. I using Ubuntu 16.04 Unity.

Are you sure you checked in the right place? Last time I tried, it worked just fine. Just to confirm, the log out button can be found by going to your profile and clicking the "Settings" gear in the top-right corner.

I'm having the same issue, settings gear is missing. I'm on 3.2.3

Yep!, no logout menu.

I have the same problem, I use windows 10.

I have the same problem. I'm waiting for the developers fix this problems or help us to solve this.

Does
cd ~/.config
rm -rf Ramme

at least work?

Force log out by clicking Logout from the tray icon or from the menu, will be available in the next release :wink:

I don't understand for logout from ramme in the win10 and I can not find any .config folder.

I can't logout & switch account on Ramme

Hehe... Since I was on mac, my config folder was hiding in "~/Library/Application Support/Ramme/"
rm -rf "~/Library/Application Support/Ramme/"
The above worked for me.

just delete and extract last state

Fixed yet? (running it using yarn dev and just logged out from the page inside. Now the app does not let me logout - rm -rf ~/.config/Ramme has no effect. Below is the error that is constantly happening).
Uncaught TypeError: Cannot read property 'appendChild' of undefined
    at <anonymous>:1:481
    at EventEmitter.electron.ipcRenderer.on (/home/user/.../ramme/node_modules/electron/dist/resources/electron.asar/renderer…:52)
    at emitMany (events.js:146)
    at EventEmitter.emit (events.js:223)


Sample code for shared contexts

Hey there! I'm trying to implement an app where I share contexts between multiple headless renderers. It seems like it should be possible to do this using glutin via GLAttributes, but when I attempt to use it as I've shown below, I get a compilation error:
extern crate gl;
extern crate glutin;

fn main() {
    let c1 = glutin::HeadlessRendererBuilder::new(640, 480).build().unwrap();
    unsafe { c1.make_current() }.unwrap();
    gl::load_with(|symbol| c1.get_proc_address(symbol) as *const _);

    let mut builder = glutin::HeadlessRendererBuilder::new(640, 480);
    let mut attributes = glutin::GlAttributes::default();
    attributes.sharing = Some(&c1);
    builder.opengl = attributes;
    let c2 = builder.build().unwrap();
    unsafe { c2.make_current() }.unwrap();
    gl::load_with(|symbol| c2.get_proc_address(symbol) as *const _);
}

The error I'm seeing is:
error[E0308]: mismatched types
  --> examples/sharing.rs:15:22
   |
15 |     builder.opengl = attributes;
   |                      ^^^^^^^^^^ expected struct `glutin::platform::platform::headless::HeadlessContext`, found struct `glutin::HeadlessContext`
   |
   = note: expected type `glutin::GlAttributes<&glutin::platform::platform::headless::HeadlessContext>`
   = note:    found type `glutin::GlAttributes<&glutin::HeadlessContext>`

After looking into it, I found glutin::HeadlessContext's .context property, which I believe is of the right type, but doesn't seem to be public. Can anyone point me in the right direction for setting up shared contexts? Once we get it sorted out, I'll gladly contribute back a new example that demonstrates it. Thank you!

This is clearly a problem here: https://github.com/tomaka/glutin/blob/06247af952aa29c0eb5fecf89fd099004e4bc2a0/src/headless.rs#L21
It should be HeadlessContext and not platform::HeadlessContext.

Oh, great! Thank you so much for the quick response. I was going to try and fix it and submit a patch, but I ran into https://github.com/tomaka/glutin/issues/853 when trying to build Glutin locally, and am not an experienced-enough Rust developer to work around that one.

Thanks so much for fixing #853, @tomaka 👍  Now that I've had a chance to dive into the code, I realized that context sharing isn't itself supported on Cocoa yet (#135). I'm definitely too new of a Rust developer to tackle that whole feature, but maybe someday.


playlist stop my bot

Songbird version: (version)
songbird = {version = "0.2.0", features = ["builtin-queue"]}
Rust version (rustc -V): (version)
rustc 1.57.0-nightly (497ee321a 2021-09-09)
Serenity/Twilight version: (version)
serenity = {version = "0.10.7", features = ["client", "framework", "standard_framework", "voice", "rustls_backend"]}
Output of ffmpeg -version, youtube-dl --version (if relevant):
ffmpeg: ffmpeg version 3.4.8-0ubuntu0.2
youtube-dl: 2021.06.06
Description:
Excuse me for google translate from Japanese,
And I'm also a beginner on github.
When I add a playlist song using input :: ytdl (& url) and the first song finishes, all songbird commands stop. Is this a bug?
Also, is there a way to put all the songs in the playlist into the queue at once at this time?

I tried to reproduce something like this last night before implementing #96, but couldn't cause any odd behaviour. Can you explain exactly what commands stop when a track ends?

Also, is there a way to put all the songs in the playlist into the queue at once at this time?

no, to pass --no-playlist to the songbird's ytdl function when calling youtuble-dl.
basically, it is recommended to create your own queue system instead of songbird's builtin queue.


Behaviour when no compartments are defined

For basic packing functionality, it would be handy if we could also pack systems without defining a compartment. Currently this throws an error.
I think the ideal behavior should be:

no compartments defined in the space: Simply pack all segments in the cuboid defined by space[size].
compartments defined in the space:  Current behavior.

This would make it simpler to pack cubic cytoplasms without having to define a "main" compartment. (this seems a bit convoluted?)

I agree that it would be nice to be able to omit the compartment definition.
That does leave the question of defining what compartment a segment belongs to in the "segments" section. Should that default just be called “main” implicitly? Or should an empty/unset compartments field for a segment imply the unset main compartment.
Setting a default compartment is a trivial change but it will only improve the interface if the segment definitions still make sense.
What do you think would be the neatest way of doing that?

I think the easiest way to do this is to always create an implicit 'main' compartment at the start of our routine, which is just the box dimensions. Every segment is then, by default, part of this 'main' compartment.
I'm not sure if this will significantly impact performance. I imagine that having this extra compartment/mask will be noticeable for very large systems. Is this a problem?

The performance matters are not the concern here, in my initial estimate :)
What does slightly worry me is the confusion that comes from a default compartment. But if we do it like you propose, I think, such that

if left unspecified, there is one compartment called “main” with a cuboid shape
if left unspecified, each compartment has one compartment id in its list, which is “main”, but only if the compartment is not specified either

In case the compartment is specified, the implicit default for the segment compartment ids are not valid, and throws a good error about it.
That is all to prevent the following scenario: a specific (set of) compartment(s) is specified yet no segment has any compartment ids. That empty result would be very confusing otherwise.

If you do not set compartments in the space section, but do provide the compartments field in any of the segments in the segments section, you get the following error:
            If no `compartments` are defined in the `space` section,
            compartment selections in segment definitions are not allowed.
            When no compartments are defined in the `space` section, a single
            default compartment is implied. To set each segment's compartment
            to the implied default compartment, remove the `compartments`
            field from all segment definitions.

Though a bit wordy, I do think it communicates what is going on.
With this error in place, I'm comfortable moving forward with implementing this. Almost done :)


Add additional certificate capability

Adds additional capability to add a configmap file containing a PEM certificate bundle.

Mounts certificate to /etc/ssl/certs in both the scanner and server deployments
Adds configurable settings to the values file
Cleans up volumes and volumemounts, wrapping them in an if block

 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.

Hi @ebram-va,
Thank you for your PR, but unfortunately, we are currently not using the master branch.
The "main" branch is 6.5
I guess we already have there what you're looking for

@semyonmor thank you!  You do not have it in 6.5 which is why I originally came here to put in a PR.  I'll close this PR and put it in for 6.5 instead.


Indicate implementation class in REST API

When using the Rest API it is sometimes difficult to know the object that you are working with (Folder, Freestyle project, Maven project, etc.). This PR adds a class element which makes knowing the objects class that are being used by the rest api easier.
@reviewbybees @kohsuke

Question: are there any clients that are coupled to the old schema, which might be broken by adding a new element?  (I am honestly not sure about this -- it's a question I'd ask though.)

@svanoort for proper tracking by GitHub, please comment on the PR’s diff tab, not on component commits.

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be
reviewed by other CloudBees employees before we seek to have the change accepted. If you want to find out
more about our process please see this note

:bug: there is a potential NPE, new tests should be JUnit 4 style and there are other comments added by @jglick that need addressing

When will this PR be completed?

If and when the author decides to work on it again.

Actually it seems you cannot use $class, as this is not a valid NameStartChar for XML output, so that would result in malformed output from XMLDataWriter.

NameStartChar	   ::=   	":" | [A-Z] | "_" | [a-z] | [#xC0-#xD6] | [#xD8-#xF6] | [#xF8-#x2FF] | [#x370-#x37D] | [#x37F-#x1FFF] | [#x200C-#x200D] | [#x2070-#x218F] | [#x2C00-#x2FEF] | [#x3001-#xD7FF] | [#xF900-#xFDCF] | [#xFDF0-#xFFFD] | [#x10000-#xEFFFF]

So, yes, $class would seem to be a :bug: about the only alternative I can think is _class... note that as $ is a valid java name somebody could implement a class with
public Object get$class() {
...
}

Anyway, so it's not like we didn't risk an over-lap with a plugin's data model... what we want is a low risk of overlap... _class or worst case __class__ would probably be safe enough

@stephenc @jglick I can implement with _class if everyone approves. I'll fix the unit tests also to reflect this change.

Well the original class seems fine. (It need not be a Java identifier.) I was striving for consistency with Stapler usage, but since that is not an option, I see no compelling reason for a more exotic-looking attribute.
To avoid extra work while we try to make up our mind, introduce a constant. :-)

FYI - class is a reserved word in ECMA 6 and has a lot of overlap with everything in the ecosystem (HTML, Java, etc.). It would definitely be best to avoid it. I'd +1 the use of _class for this reason.

As noted, using a keyword does not necessarily matter here, since we are just returning XML or JSON…but it might be safer to use something which is a valid Java/JavaScript identifier in case people are doing automatic object bindings or something like that.

Right, or simply using the structure returned, e.g.:
var thisThing = ...
thisThing.class // may fail, would have to use thisThing['class'], which is a pain
thisThing._class // should be fine

Change subsumed by #61


feat: added compile lambda

@varex83 will merge now. If any comments unresolved will free to add, I will target them in new branch


Intro and organizational updates

I rewrote the intro, and made a few other organizational updates.

lgtm as well. @TelegramSam lmk if you need me to merge


chore: update set asset path for coding comps and calcite assets url

@omarkawach can you bump Calcite to 2.8.6?


Using recursive type aliases gives circular reference error

Bug Report
As per TypeScript v3.7 recursive type aliases are valid to use. Trying to migrate from Flow to TypeScript and having issues. This is the flow equivalent and this is what I currently have for TypeScript but it is not an ideal solution. If I remove the generic types it works but that is also not an elegant way of doing this. I asked this on StackOverflow.
Is there a limit to how recursive types work?
🔎 Search Terms

TypeScript Type alias circularly references itself

🕗 Version & Regression Information

Tested on v4.0.2, v4.1.2 and v4.1.3

⏯ Playground Link
Playground link with relevant code
💻 Code

export type IntrospectionType = {
  readonly kind: 'OBJECT';
};

export type IntrospectionListTypeRef<
  T extends IntrospectionTypeRef = IntrospectionTypeRef
> = {
  readonly kind: 'LIST';
  readonly ofType: T;
};

export type IntrospectionNonNullTypeRef<
  T extends IntrospectionTypeRef = IntrospectionTypeRef
> = {
  readonly kind: 'NON_NULL';
  readonly ofType: T;
};

export type IntrospectionTypeRef =
  | IntrospectionNamedTypeRef
  | IntrospectionListTypeRef
  | IntrospectionNonNullTypeRef<
      IntrospectionNamedTypeRef | IntrospectionListTypeRef
    >;

export type IntrospectionNamedTypeRef<
  T extends IntrospectionType = IntrospectionType
> = {
  readonly kind: T['kind'];
};

🙁 Actual behavior
Type alias 'IntrospectionTypeRef' circularly references itself.
🙂 Expected behavior
Flow equivalent works perfectly and this looks right so should work or maybe I am doing something wrong 🤷🏼♂

Recursive type aliases are permitted when they don't create problems, and this creates a problem. One of the problematic recursivenesses is immediate self-instantiation, e.g.
type Box<T> = { content: T };
type SomeBox = Box<SomeBox>;

Basically, you can't create a circularity that results in an instantiation chain of U<T<T<T<T<T<T<T<T<T<T...[infinity].
Interfaces have slightly different rules because they have more concrete restrictions about what kinds of types they can produce (specifically, they're always object types which can be entirely deferred), so you can convert the prior type aliases to interfaces and this will work OK.

Recursive type aliases are permitted when they don't create problems, and this creates a problem. One of the problematic recursivenesses is immediate self-instantiation, e.g.
type Box<T> = { content: T };
type SomeBox = Box<SomeBox>;

Basically, you can't create a circularity that results in an instantiation chain of U<T<T<T<T<T<T<T<T<T<T...[infinity].
Interfaces have slightly different rules because they have more concrete restrictions about what kinds of types they can produce (specifically, they're always object types which can be entirely deferred), so you can convert the prior type aliases to interfaces and this will work OK.


cRIO targets: FPGA Settings

Add context help to FPGA settings.
Modify path control to work with Linux paths if a Linux targets (similar to the hack used on TDMS module path)
Why do we have both a local bitfile path and the remote bitfile path...?

Also, normalize the capitalization on the settings.


Update docs.asp.net and other links

I'm making a quick sweep to shed some docs.asp.net links here and there.
This one includes converting the smart quotes to straight quotes and updating a few other links.

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@mairaw Another thing u should know: [@]danroth27 is cool with the docs.asp.net link. We can go forward or close on that basis. I guessed wrong on this one, but my 💓 was in the right place. https://github.com/aspnet/EntityFramework.Docs/pull/474#issuecomment-330341485

I commented on that PR. Your heart is always in the right place.
I remove all absolute URLs whenever possible. Except when I don't know how the files will also be used like it's the case for the microservices book.


Support ring metrics for asynchronous requests

The Aleph web server, built on top of Netty, gives the option of returning a manifold deferred object in addition to the standard ring response map. This allows the request to be carried out asynchronously.
The current instrument middleware assumes a synchronous implementation. It would be great if async was supported as well.
It wouldn't be much work to get this working. Something along the lines of:
(defn wrap-metrics
  [handler]
  (fn [req]
    (start-metrics)
    (let [resp (handler req)]
      (if (instance? manifold.deferred.Deferred resp)
        (on-realized resp stop-metrics)
        (stop-metrics))
      resp)))

Of course, it means introducing manifold as a dependency. It's a small library though, and only introduces a couple of transitive dependencies.
Happy to submit a PR for this.

This would tie the library to manifold.deferred.Deferred, whatever that is. I'm in favour of introducing a new sub-project.

That's fair enough. It's a shame though as most of the implementation is the same as that of metrics-clojure-ring. Would refactoring metrics-clojure-ring so that metrics-clojure-aleph can pull it in and use the start/stop-metrics functions be a possibility?

Here's my attempt in case someone wants to quickly hack it:
Define an async handler:
(defn handle-request-async [handler metrics request respond raise]
  (let [{:keys [active-requests requests responses
                schemes statuses times request-methods]} metrics]
    (inc! active-requests)
    (let [request-method (:request-method request)
          request-scheme (:scheme request)]
      (mark! requests)
      (mark-in! request-methods request-method)
      (mark-in! schemes request-scheme)
      (let [htimer (tmr/start (times request-method (times :other)))
            callback-fn (fn [cb r]
                          (try
                            (let [^{:tag "int"} status-code (or (:status r) 404)]
                              (do (mark! responses)
                                  (mark-in! statuses (int (/ status-code 100)))
                                  (tmr/stop htimer)
                                  (cb r)))
                           (finally (dec! active-requests))))]
        (handler request
                 (partial callback-fn respond)
                 (partial callback-fn raise))))))

Ensure instrument handles multiple arity
(defn instrument
  ([handler]
   (instrument handler default-registry))
  ([handler ^MetricRegistry reg]
   (fn
     ([request]
      (handle-request handler
                      (ring-metrics reg {:prefix []})
                      request))
     ([request respond raise]
      (handle-request-async handler
                            (ring-metrics reg {:prefix []})
                            request respond raise)))))


setup Travis cron to run integration test daily

I check Travis build history and don't see any build which have integration test. It seems we haven't setup cron to run integration test, do we?
I suggest to setup cron to make sure all tests (unit tests + integration tests) are run fully. In addition, let's have it so that I can check my PR https://github.com/networknt/light-4j/pull/404 run well or not.

Link to setup cron on Travis: https://docs.travis-ci.com/user/cron-jobs/
I think we can setup 2 crons for light-4j, for develop and master branch.

Great ideas. None of us is an expert on Travis and I don't even know it can set up cron-jobs. Internally, we are using light-bot to build and test multiple repositories. Do you know if there is a way to integrate with light-bot from Travis? Thanks.

Travis doesn't support inter-repositories build, so that I don't think it's possible.
However, I come up with another way:

Setup Travis build (unittest only) for each and every light-* repositories. We just do once.
For light-bot (or light-config-test?), we'll setup daily cron-job for this. This cron will run script similar mentioned in this article: https://hiddentao.com/archives/2016/08/29/triggering-travis-ci-build-from-another-projects-build/ (using environment to specify running integration test also), therefore, we just need to setup cron for 1 project only.

 setup Travis cron to run integration test daily


Real-time HTML doesn't show request times while a static one does

I run two almost identical commands:
grep '05/Oct/2020' access_log | LANG=en_US goaccess --log-format '[%d:%t +0200] %h %e "%r" %s %b "%R" "%u" **%^/%D**' --date-format '%d/%b/%Y' --time-format '%H:%M:%S' - -o 2020-10-05-report.htm
and
ssh user@server 'tail -f /var/log/httpd/access_log' | LANG=en_US goaccess --log-format '[%d:%t +0200] %h %e "%r" %s %b "%R" "%u" **%^/%D**' --date-format'%d/%b/%Y' --time-format '%H:%M:%S' --real-time-html -o report.html --ws-url localhost
The first one reads grep output from stdin and writes it to a report file, the second one reads tail output from an ssh session for real-time reporting. The difference is, whichever remote command I try with an ssh session (cat, grep etc.), the following columns are missing on every panel of GoAccess dashboard:
Tx. Amount | Avg. T.S. | Cum. T.S. | Max. T.S.
While they are always present when I source the input locally. Is it some kind of shell or cr/ln mismatch? Or does it somehow misdetect whatever is on the first line and determines the structure of the rest of the file? A typical log line is:
[04/Oct/2020:04:30:34 +0200] <IP_ADDRESS> - "HEAD /icons/apache_pb.gif HTTP/1.0" 301 - "-" "Mozilla/5.0 (compatible; NetcraftSurveyAgent/1.0; +info@netcraft.com)" **0/182**
Both the referrer and the user agent get detected correctly.

any thought on this? This means I can't really use Goaccess for realtime stats, because the most important metric is how long requests take.

I pushed a commit that fixes this. Please feel free to test it out by building from development and let me know if that solves the issue on your end. I'll push it out in the upcoming minor release.
Thanks for reporting it!

Yes, I can confirm it works, thanks for that!


First Set of Animals

The world needs some life so lets inject some basic AI life in here. From the description of the first static world the following are possible wildlife to include:
Fields

Butterflies
Bees
Rabbits
Field Mice

Forest

Birds
Deer
Foxes
Badgers.

[ ] Install Soullink Spawner
[ ] Add pond areas
[ ] Add frog spawner to pond areas
[ ] Add rabbits in grassy areas
[ ] Add foxes in grassy areas
[ ] Add Deer in grassy areas
[ ] Add Birds in forest areas
[ ] Add badgers in forest areas


Update configuration docs about @jest-environment

The @jest-environment docblock is a feature that's coming in Jest version 20.0.0.
This commit adds this information to the documentation.

Thanks! Please take a look at how we do it here: http://facebook.github.io/jest/docs/expect.html#expectstringcontainingstring – mind adjusting your PR to match that?

Codecov Report

Merging #3281 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3281   +/-   ##
=======================================
  Coverage   63.89%   63.89%
=======================================
  Files         176      176
  Lines        6472     6472
  Branches        4        4
=======================================
  Hits         4135     4135
  Misses       2336     2336
  Partials        1        1

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 725edda...be5ba33. Read the comment docs.

I updated it. I hope this is how you imagined. :) Let me know if not.

Thanks!


juliarun test

Coverage remained the same at 100.0% when pulling 71278d6a0b28acd678cc6b3a17ce6c9b0ce1e42c on juliarun:juliarun into 1613be222edfd1b5561eb6b8b51dda9458a40f99 on JuliaLang:master.

Coverage remained the same at 100.0% when pulling 71278d6a0b28acd678cc6b3a17ce6c9b0ce1e42c on juliarun:juliarun into 1613be222edfd1b5561eb6b8b51dda9458a40f99 on JuliaLang:master.

Coverage remained the same at 100.0% when pulling 71278d6a0b28acd678cc6b3a17ce6c9b0ce1e42c on juliarun:juliarun into 1613be222edfd1b5561eb6b8b51dda9458a40f99 on JuliaLang:master.


remove deprecated typography components

Changes
yeet!
I'm working on some repo-wide changes and these were getting in the way.
Testing
N/A
Docs
Not including changeset yet. I think we could have a single changeset at the end saying something like "All previously-deprecated components have been removed".

I'm working on some repo-wide changes and these were getting in the way.

What was the problem?

Oh, do you mean the iui-text-spacing class? iirc we were not happy with the spacing it adds because it is undesirable in most cases and even when it's desirable, it can be too large (constant 22px). But I don't think we have any recommendations

Yeah, when user wants to add padding at the bottom of <Text variant='heading'/>, what is the recommended way to do that. We definitely should document which spacing variables to use with which variant and show some examples how to do that in our docs site.

It's a bit difficult to even come up with a recommendation. We can't just say something like "add margin-bottom: var(--iui-size-m)". The desired value will be different in every case.
Maybe we can just say "use margin or gap with one of the values from itwinui-variables".

Maybe we can just say "use margin or gap with one of the values from itwinui-variables".

Lets move the discussion out of PR. It's documentation issue, not this PR.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 824 ms

Resolved: Bofrost.at performance has improved in da5ca40.


Test doc3 egoitz

Two tests marked as futureWork because the dependency tree is too complex, in "TestDoc3 Paragraph 1" and in "TestDoc3 Paragraph 3".

I have added the comments and fixed the issue with "nmod_for".

What is "especizalided"?  Specialized?  Maybe configured?  For the strings with """ and |, shouldn't we add a .stripMargin?

Sorry for the typo. I have removed that sentence from the comments because it is confusing.

I have solved some conflicts that have arised after merging the last changes in master.

thanks!  Hopefully if mine gets merged first,  new conflicts either won't arrive or will be easy to merge.  If I get to them first I can do it too!

Sorry, people: @EgoLaparra, @bsharpataz.  Guess what happened.

I’ll work on it tonight if @EgoLaparra doesnt beat me to it
On Sat, Mar 24, 2018 at 7:09 PM Keith Alcock<EMAIL_ADDRESS>wrote:

Sorry, people: @EgoLaparra https://github.com/EgoLaparra, @bsharpataz
https://github.com/bsharpataz. Guess what happened.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/213#issuecomment-375939807, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFIniSg8CS7jCR6Z7tw4vUS5f4gqFSc8ks5thvw6gaJpZM4S5F7r
.

As long as 3 people don't work on it!  Thanks.

I'm working on this @bsharpataz. Two of my tests are now failing.

Wow.  Not working alone.  CluLab rocks!  Thanks both.

Ok, solved! :relaxed:


Fuel quickstart with browser wallet

This updates the quickstart guide to use the Fuel browser wallet. I've attached a demo video here to see how it looks.
https://user-images.githubusercontent.com/1857282/222105983-1af2cc7e-1c04-42f3-ac83-7f56fa263718.mp4

cc @sarahschwartz I just updated this PR again with a small update, I think it may be gtg to be reviewed, I think it provides a lot of value for onboarding new devs and a good DX now that the wallet is coming along.

@Dhaiwat10 thx I also want to re-test this with Fuel v3 and the current browser wallet to make sure everything still works ok
@arboleya unsure but IMO having the private keys exposed on the client is something that ultimately should not be referenced at all, but I understand if you all want to keep it until the browser wallet becomes more stable or something.

@dabit3 I fully agree, but will defer it to the @FuelLabs/devrel and @FuelLabs/frontend teams.
They probably know more about the current state of wallet-related things.


chore: add typing to profile

SUMMARY
As titled. There was also some unused state in some components, so I cleaned it up
TEST PLAN
CI, unit tests, manual confirmation of all the tabs on the profile
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

to: @graceguo-supercat @ktmud @suddjian @kristw

ping @graceguo-supercat @ktmud or @suddjian for review, thanks!


volume.activate processes stuck for 6 months, rancher must not try forever on very old processes

Rancher Version: 1.1.4
Docker Version: 1.10.3
OS and where are the hosts located? (cloud, bare metal, etc):
"kernelVersion": "3.10.0-327.13.1.el7.x86_64",
"operatingSystem": "CentOS Linux 7 (Core)",
on Openstack VMs
Setup Details: Rancher server on one host and external mysql on another host.
Environment Type: Cattle
issue:
after months of running rancher we have seen processes stuck forever under admin > processes.
I have managed to remove the agent.reconnect processes that were stuck...but many others are stuck for months not able to delete. For example see image, i am not able to delete the processes volume.activate ...these are 6 months old! maybe these are slowing down the server.
There is no point for rancher to try forever on these.

Steps to Reproduce: Run Rancher for months and you will start seeing accumulated failed processes that are months old and cannot be removed.
Expected:
There is no point for Rancher to try forever smth that has been failing for months ...
rather it should still report them and mark them as "dead", instead of having continuous exceptions in the logs and to avoid performance degradation in the longer run.
related issues:
https://github.com/rancher/rancher/issues/4974
https://github.com/rancher/rancher/issues/5349
https://github.com/rancher/rancher/issues/6374
forum:
https://forums.rancher.com/t/how-to-use-the-processes-information-for-admin-documentation-needed/4161

experiencing the same problem, very annoying. Would love to have a button in UI to kill processes

Is there no official way to stop a stuck process?

Rancher Version: 1.2
Docker Version: 1.2.12
OS and where are the hosts located? (cloud, bare metal, etc): digitalocean server
"kernelVersion": 4.4.0-53-generic,
"operatingSystem": #74-Ubuntu SMP,
Environment Type: Cattle
Same issue here. See link: https://monosnap.com/file/NRRBFWqwLbLx2NfYDsbKgXLM7U0Hor
API links end up in a 404.
How do we get rid of those?

Rancher is an orchestration system which continually reconciles what the state of the world is vs what it should be.  "Processes" are not actual compute processes, but things the orchestration system thinks need to be done.  So "cancelling" a process does not really have any meaning, the disparity between desired and actual needs to be corrected.
Most of these should be fixed in 1.3+ and running the DB cleanup script (see docker run --rm rancher/cleanup-1-1 for usage).


How to share a map with the alliance?

Hello
How can i share a map with the alliance?
I dont see any sharing options under sharing. Only Corp and Private

Alliance maps were not implemented in develop branch so far. I added them.
You probably need to re-login with your chars by CCPs SSO, in order to get the character connected with the alliance data.
I have just tested it. Works fine for me with the new implementation.
If your active character is currently in an alliance, you should be able to see the alliance option under "settings->sharing" now.

Logget out, updated, cleared tmp/ logged in again. Created new map, but still dont see it

Its works, if i manually fill alliance table and assign it to my char.
But it dont happen automaticly

Alliance data is now requested by the CCPs XMLv2 API. Unfortunately CREST does not provide any alliance data on the character endpoint.
You need to re-auth by using the SSO to get the ally data imported and connected with your character.

can confirm, this is working on my installed develop branch. You even see your alliance logo in the cookied character login, nice! :-) :+1:


Support enrolling learner as verified on payment

Pre-Flight checklist

[ ] Screenshots and design review for any changes that affect layout or styling

[ ] Desktop screenshots
[ ] Mobile width screenshots

[ ] Migrations

[ ] Migration is backwards-compatible with current production code

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

[ ] Settings

[ ] New settings are documented and present in app.json
[ ] New settings have reasonable development defaults, if applicable
[ ] Opened issue for DevOps regarding necessary configuration changes to deployed environments

What are the relevant tickets?
(Required)
What's this PR do?
(Required)
How should this be manually tested?
(Required)
Where should the reviewer start?
(Optional)
Any background context you want to provide?
(Optional)
Screenshots (if appropriate)
(Optional)

Codecov Report

Merging #451 (5f83979) into main (f45d19f) will increase coverage by 5.61%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##             main     #451      +/-   ##
==========================================
+ Coverage   89.76%   95.38%   +5.61%
==========================================
  Files         233      130     -103
  Lines        7457     3143    -4314
  Branches      676        0     -676
==========================================
- Hits         6694     2998    -3696
+ Misses        665      145     -520
+ Partials       98        0      -98

Impacted Files
Coverage Δ

static/js/lib/notificationsApi.js
53.06% <0.00%> (-23.41%)
:arrow_down:

static/js/constants.js
100.00% <100.00%> (ø)

courses/api.py

courses/models.py

ecommerce/models.py

ecommerce/urls.py

ecommerce/views/v0/__init__.py

main/constants.py

main/features.py

main/utils.py

... and 95 more

Continue to review full report at Codecov.

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

@jkachel I've rebased this on your PR to pick up the enrollment_mode field, so you'll want to review only my commit in this PR.

LGTM other than the tests failing, which looks like just some simple updates to add in the mode. Otherwise tested successfully as described.


Application crashing with memory leaks

Actual behavior
I have an application that I built with netty version 4.0.40.Final. The application seems to be crashing every hour and when I analyzed the dump file, it looks like most of the leaks are happening because of "io.netty.util.concurrent.SingleThreadEventExecutor". Can someone please help me out with this?
Netty version
4.0.40.Final
JVM version (e.g. java -version)
IBM Java 1.8
OS version (e.g. uname -a)
Windows Server 2012  R2

What you mean by leaks ? Why it crashes ? Also can you share the dump ?

Actually I can't delete th

@normanmaurer : Sorry I was told I can't share the heap dumps. But I think I might have figured out where the problem could be.
Is it ok if I get back to you in a few days? If I can still replicate the issue I will write up a sample program which will emulate the issue so that it would be easier for you to check.

Sounds good

Am 30.05.2018 um 14:10 schrieb Hitesh Kotian<EMAIL_ADDRESS>@normanmaurer : Sorry I was told I can't share the heap dumps. But I think I might have figured out where the problem could be.
Is it ok if I get back to you in a few days? If I can still replicate the issue I will write up a sample program which will emulate the issue so that it would be easier for you to check.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@normanmaurer: I think I have a memory leak issue because the code I'm working on has a scheduled task running using quartz, which essentially does a bunch of cleanups.
I know I can create such sceduled tasks using netty right?
For that would you suggest using the event loop of the parent loop group?
So essentially here is the code snippet I have:
`public void startServer () {
// .... other code here
EventLoopGroup master = new NioEventLoopGroup();
EventLoopGroup slave  = new NioEventLoopGroup(5);
ServerBootstrap bootstrap = new ServerBootstrap();
bootstrap.group(master, slave);
// ... Other bootstrapping logic
master.scheduleAtFixedRate(new Runnable() {
@Override
public void run() {
// .. Running scheduled cleanup tasks
}	 }, 0, 3, TimeUnit.SECONDS);
}
`
Do you think this is fine? Or is this wrong?

@hiteshkotian generally speaking yes you can do this but you will need to be sure you not block in the Runnable as otherwise you will block an EventLoop

cool. Thanks for the info! this will be of great help for us.


ClouDNS Error Cert

Hello, trying to use ClouDNS to make wildcard cert for my *.subdomain.domain.com using DNS challenge option.
But its throwing error
Error: Command failed: certbot certonly --config "/etc/letsencrypt.ini" --cert-name "npm-40" --agree-tos --email<EMAIL_ADDRESS>--domains "*.subdomain.domain.com" --authenticator dns-cloudns --dns-cloudns-credentials "/etc/letsencrypt/credentials/credentials-40"
Traceback (most recent call last):
  File "/usr/local/bin/certbot", line 5, in
    from certbot.main import main
  File "/usr/local/lib/python3.7/dist-packages/certbot/main.py", line 6, in
    from certbot._internal import main as internal_main
  File "/usr/local/lib/python3.7/dist-packages/certbot/_internal/main.py", line 28, in
    from certbot import crypto_util
  File "/usr/local/lib/python3.7/dist-packages/certbot/crypto_util.py", line 42, in
    from certbot import interfaces
  File "/usr/local/lib/python3.7/dist-packages/certbot/interfaces.py", line 21, in
    from acme.client import ClientBase
ImportError: cannot import name 'ClientBase' from 'acme.client' (/usr/local/lib/python3.7/dist-packages/acme/client.py)

    at ChildProcess.exithandler (node:child_process:399:12)
    at ChildProcess.emit (node:events:526:28)
    at maybeClose (node:internal/child_process:1092:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)

Please help me

I have been having this issue as well. Rolling back to previous versions didn't make a difference. I posted details on Reddit, but no one chimed in.

That’ll explain why I didn’t get a response! Thanks for that. I messaged the mods and hopefully they are able to get that converted. I copied the post below so it is here in this issue in case that doesn’t happen.
Original Post:
I'm having an issue with my setup of Nginx Proxy Manager (NPM) being unable issue or renew Let's Encrypt certificates and I'm unable to determine why.
The following error appears in the logs when attempting to issue a certificate, renew a certificate, or on container start.
[12/31/2022] [3:34:30 PM] [SSL      ] › ✖  error     Error: Command failed: certbot renew --non-interactive --quiet --config "/etc/letsencrypt.ini" --preferred-challenges "dns,http" --disable-hook-validation

Traceback (most recent call last):

  File "/usr/local/bin/certbot", line 5, in <module>

    from certbot.main import main

  File "/usr/local/lib/python3.7/dist-packages/certbot/main.py", line 6, in <module>

    from certbot._internal import main as internal_main

  File "/usr/local/lib/python3.7/dist-packages/certbot/_internal/main.py", line 28, in <module>

    from certbot import crypto_util

  File "/usr/local/lib/python3.7/dist-packages/certbot/crypto_util.py", line 42, in <module>

    from certbot import interfaces

  File "/usr/local/lib/python3.7/dist-packages/certbot/interfaces.py", line 21, in <module>

    from acme.client import ClientBase

ImportError: cannot import name 'ClientBase' from 'acme.client' (/usr/local/lib/python3.7/dist-packages/acme/client.py)

    at ChildProcess.exithandler (node:child_process:402:12)

    at ChildProcess.emit (node:events:513:28)

    at maybeClose (node:internal/child_process:1100:16)

    at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

Additionally, inside NPM using the site test functionality results in the following error in the UI:
Communication with the API failed, is NPM running correctly?

In developer tools the error returned is a bad gateway error.
From what I've read these errors are usually caused by not having the ports open to NPM or having poor DNS server options. I've confirmed ports are correctly set, DNS options are good. I have taken the following steps to attempt to resolve this issue:

Confirmed Internet access for the docker container
Confirmed accurate DNS settings for the container by updating packages via the command line inside the container
Verified external access via Port 80 and 443 reaching the container
Completely wiped the container and DB containers to do a new install and the error on startup shows in the log prior to any login

Environment Details:

VMWare ESXi
Photon OS 4 as Docker Host
4 vCPU, 16GB RAM
Portainer 2.16
Docker Engine 20.10.14

Is this a bug in NPM or are there additional steps I can take to isolate an environment issue?

Try to install old version of acme via:
pip install --upgrade --force-reinstall acme==1.32.0

I can confirm that your workaround @Wolfilux to downgrade the Acme package to 1.32.0 worked for me.

I run nginx proxy manager in docker container, so how do I do it?

Normally run that on the vps in console? Or do I need to somehow execute the command in docker container

I try pip install --upgrade --force-reinstall acme==1.32.0 but it didn't work for me

@ririko5834
You should be able to type that into the console in your docker container. I accessed it via Portainer, but Synodocker and Docker directly will allow you to pass commands to the container console.
https://docs.portainer.io/user/docker/containers/console
https://docs.docker.com/engine/reference/commandline/exec/
@rodneyt What about it didn't work? Was it an error or did it downgrade and the issue persisted?

@ririko5834
You should be able to type that into the console in your docker container. I accessed it via Portainer, but Synodocker and Docker directly will allow you to pass commands to the container console.
https://docs.portainer.io/user/docker/containers/console
https://docs.docker.com/engine/reference/commandline/exec/
@rodneyt What about it didn't work? Was it an error or did it downgrade and the issue persisted?

Yes, https://github.com/NginxProxyManager/nginx-proxy-manager/issues/2011#issuecomment-1423022060

@rodneyt Ah I see. Your issue might be different than ours. I have ClouDNS and the above worked for me. From the comment you linked to it appears you're using Cloudflare.

Did it

but getting error

Restarted NPM, but now getting this
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Error communicating with the ClouDNS API: {'status_code': 200, 'success': False, 'payload': {'status': 'Failed', 'status_description': "You don't have access to the HTTP API. Check your plan."}, 'error': "You don't have access to the HTTP API. Check your plan."}
So you need to pay to use that?

I just want to have wildcard certificate for my domain, cloudflare doesn't allow API for .tk domains, so I tried using cloudns.
Is there any supported DNS challenge provider which has free plan with access to API?
Or any different way to make wildcard certificate, like *.mydomain.tk

Recently experienced this myself. Found that the config files were being installed but not the certbot-dns-cloudns dependencies. Checked the plugin creator's Github page and found the following which corroborates what I found:
https://github.com/inventage/certbot-dns-cloudns/issues/2
Followed the steps indicated by 'gregfly' and was able to pull a cert using certbot 2.5.0. Hopefully, the maintainer updates the plugin soon or someone creates a PR and it gets merged.

Can you PR fix for the plugin?


Failed to install v0.0.2-alpha.3

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (4603195 2024-04-10T00:21:30.172477000Z)
What command(s) is the bug in?
installation
Operating System
macOS (Intel)
Describe the bug
Attempting to install on the latest release on MacOS:
git clone https://github.com/matter-labs/foundry-zksync
cd foundry-zksync
git checkout v0.0.2-alpha.3
cargo build --release

Followed by an insanely long output:
error: failed to run custom build command for `librocksdb-sys v0.11.0+8.1.1`

Caused by:
  process didn't exit successfully: `/Users/patrick/code/foundry-zksync/target/release/build/librocksdb-sys-68ab0e7cac38859e/build-script-build` (exit status: 1)
...

I tried to switch to dev and install from there, and am running into this:
error: failed to run custom build command for `openssl-sys v0.9.101`

 stack backtrace:
     0: _rust_begin_unwind
     1: core::panicking::panic_fmt
     2: openssl_src::Build::run_command
     3: openssl_src::Build::build
     4: build_script_main::find_vendored::get_openssl
     5: build_script_main::find_openssl
     6: build_script_main::main
     7: core::ops::function::FnOnce::call_once

@PatrickAlphaC 👋
In the README we have a prerequisite for MacOs users. Can you please do the following:
MacOS Prerequisite - Install make: MacOS users need to ensure make is installed. If not, install it using Homebrew:
brew install make

Then, set the path to GNU make:
export PATH="/opt/homebrew/opt/make/libexec/gnubin:$PATH"
Add this export line to your shell profile (~/.bash_profile, ~/.zshrc, etc.) to make the change permanent.
This should resolve the OpenSSL issue experienced here.

I have make installed and the path exported. I am still running into the same issue.

is homebrew installed in the path /opt/homebrew/

$ which brew
/usr/local/bin/brew

Perhaps this is the issue?

ahh i see I overlooked the intel macos usage.
You will need to update the path to point to your installation path so:
export PATH="/usr/local/opt/make/libexec/gnubin:$PATH"

Yep, that fixed it!
export PATH="/usr/local/opt/make/libexec/gnubin:$PATH"

Thanks!

Relevant PR: #326


Turn autocomplete off on Mobile device keyboards

Avoiding a number of Android IME keyboard issues by turning autocomplete to off in the textarea.
Mostly useful for the monaco-editor.

Looks good :+1: I will merge it to master as soon as we wrap up VS Code's endgame and master is open for business again


Ability to specify Pages in Header

It's awesome that the theme auto-includes a list of pages in the header, but once you get above a certain number of pages, it gets a bit unwieldy.
It also be cool to be able to set the order of the pages. Thinking maybe some sort of array in the config of page paths to include, falling back to all pages if not set.

Cool Idea. Though jekyllrb.com uses a similar technique, it would be cool if we could simply load all .md under /pages and then using docs.yml to define the top navigation for minima as well.
docs:
  - about.md
  - contact-us.md
  - a-random-page.md
  - another-random-page.md

and use liquid (not entirely correct) to set the layout:

{% for doc in docs %}

{% endfor %}

You can do that with something like site.pages | where:"path", path.

Strongly in favor of this feature. I'm currently copying _includes/header.html from the source into my project and adding
{% for my_page in site.pages %}
  {% if my_page.hide_header_link %}{% continue %}{% endif %}

as a workaround. I'd be happy to help out with it.

I'd be happy to help out with it.

@mlovic if you'd like to to begin a pull request to implement this feature, I'd love to see it as well and would be glad to review it. 😄

Is anyone from @jekyll/ecosystem interested in taking a pass at this?

@benbalter Hi, i'm not in the ecosystem group (yet) but if no one else has already started on this i would be glad to help out.

@Crunch09 you can join via https://teams.jekyllrb.com!

@benbalter Do you want this for Minima 2?

@benbalter Can you help me please with adding links to top navigation bar, how can I do this?
Thanks a lot.

@mohsenSy It's explained in the _config.yml file: https://github.com/jekyll/minima/blob/affcd93be22e960afd2be08e6891d99b31bcf920/_config.yml#L17-L20

@DirtyF Looks like that documentation got removed from _config.yml

This option is documented in the main config file...
https://github.com/jekyll/minima/blob/master/_config.yml#L37-L40
... and in the README:
https://github.com/jekyll/minima#customize-navigation-links


Merge code from upstream

[ x] By placing an X in the preceding checkbox, I verify that I have signed the Contributor License Agreement

Merge to my own repository


Add documentation around bumpExpirationFootprint and restoreFootprint usage

Is your feature request related to a problem? Please describe.
Nobody has any idea how to use these operations.
Describe the solution you'd like
A simple end-to-end-esque tutorial on bumping existing storage entries and restoring expired ones.
Describe alternatives you've considered
Pain & suffering.

This is closed / tracked by https://github.com/stellar/soroban-docs/pull/522.


Update shared:akka, shared:core, shared:fs2, ... to 1.3.0

Updates

com.softwaremill.sttp.shared:akka
com.softwaremill.sttp.shared:core
com.softwaremill.sttp.shared:fs2
com.softwaremill.sttp.shared:fs2-ce2
com.softwaremill.sttp.shared:monix
com.softwaremill.sttp.shared:ws
com.softwaremill.sttp.shared:zio

from 1.2.7 to 1.3.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.shared" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #1238.


Too many s

Problem
For various reasons including screen readers and semantic HTML, there should only be one top-level
 element per page.
Solution
AFAIK gisture is just consuming whatever is coming from the raw markdown. It should either take it's first-level heading from the markdown file when it detects one is present. The alternative of just leveling all markdown headings one down (unless already at the bottom-level of
 seems an inferior solution.
I don't know Rust but I might hack on it and see what I can do. No promises though given I would be starting from zero.

Hey @nat-418 thank you for raising this issue. If I understand the problem correctly, you're saying the heading elements in the blog content should not be
 except only the title right?
Now it's just renders the Markdown directly as is, so # Title would be parsed as
Title
, ## Title as
Title
 and so on. So should I render them as any <h{n}> where n should be leveled down from the Markdown input?
Another thing is, it's all user input so in the gist markdown, one can manually enter it like that, in which case, it would cause a lot of confusion.
What do you think?
If as a user, it's a problem for you, I will fix it. :raised_hands:
Also if this is something that can be fixed directly from the HTML (for layouts), the templates are here.
PS: If you want to implement this, there are two places you can change to achieve this. The first one is in the Markdown parser itself (A good reference on the usage] and the second place is where you can configure the lol_html rewriter to do the level down.

Yes, that is correct. I will hack on it a bit and pester you later if I fail to make headway. It's a nice-to-fix, cf. https://www.w3.org/TR/2014/REC-html5-20141028/sections.html#headings-and-section


Riscv compliance test (I-FENCE.I-01.S) corrupting the .text region

I see I-FENCE.I-01.S  fence.i instruction corruption/updating content in .text region.
.text.init and .text are in ROM area, How to move this logic to RAM area (.data segment) ?
364:	0000100f          	fence.i
00000368 <instr_A_dst>:
368:	00000137          	lui	sp,0x0
36c:	0018a023          	sw	ra,0(a7)
370:	0028a223          	sw	sp,4(a7)
374:	0038a423          	sw	gp,8(a7)
378:	00f8a623          	sw	a5,12(a7)

I would first suggest picking the latest set of tests which is named Fencei.S.
The test basically checks if the updates to the instruction memory are properly synchronized with the data memory (which is the primary objective of the instruction Itself). Hence, the test does perform store in the .text regions.
If you have implemented fence.i in your DUT, then to pass this test you will need to load the program in a read-write section of the your test-bench memory. This is specific to your DUT and testbench environment. The test itself doesn't require any changes.

Im going to close this as it appears not to be a test issue; if you disagree, reopen it


addressed dependency vulnerabilities

What?
Addressed vulnerabilities in dependencies. Chose version 10.0.17 for jetty-server as this is compatible with spark-core 2.9.3 and I believe this is what broke the deployment in di-auth-stub-relying-party/pull/238.
Why?
To address dependency vulnerabilities and ensure dependencies are compatible with one another.
Related PRs
https://github.com/alphagov/di-auth-stub-relying-party/pull/240
https://github.com/alphagov/di-auth-stub-relying-party/pull/238

Not a comment on the actual contents here, but when updating a branch with the latest main we prefer to rebase over merge as per the GDS way. Could you undo  the merge and rebase instead? Happy to help if you'd like :-)


Fix URL query parameters in compression hop path when streaming

Fix #3488.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


feat: show the currently command

Description
I'd like to know what commands are currently running, but the DEBUG_SIGN isn't working when I run ni ?
https://github.com/antfu-collective/ni/blob/95d46387734165d461fb917bcda7da2e4177ff3d/src/runner.ts#L17

show the currently command.

Linked Issues
Additional context

? is a zsh operator, you need to do it with ni \?


Support splitting a single item into multiple files

Motivation
To allow generating code for a single entity in multiple files. See a related issue in volo - https://github.com/cloudwego/volo/issues/454
Solution
Introduce a new split mode that can be enabled using the builder

 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.

LGTM. Maybe a test for the workspace mode can be added?

@Millione Hi. I tried adding a test for a workspace, but there are no other tests for it. And my test fails with a panic at pilota-build/src/codegen/workspace.rs:91 because Cargo.toml doesn't exist. I'm not sure how this file should be created when generating the workspace for the first time. Here's how I compile with the workspace mode
      crate::Builder::thrift()
          .ignore_unused(false)
          .compile_with_config(
              vec![IdlService::from_path(source.to_owned())],
              crate::Output::Workspace(target.into()),
          )

Could you please point me to an existing test for a workspace mode, or help me figure out why Cargo.toml is expected to already exist in group_defs?

Could you please point me to an existing test for a workspace mode

Unfortunately, there is no existing test for this, but I'll do my best to help you figure it out.

Why Cargo.toml is expected to already exist in group_defs?

The difference between normal mode and workspace mode is that in workspace mode the code generation doesn't take place in the build phase, but in the run phase. It generates multiple crates and adds all these crates as workspace members to the Cargo.toml file.

I'm not sure how this file should be created when generating the workspace for the first time

In my opinion, we should put the workspace mode test in a separate tests folder, so that it is a package with a binary target and fits into the real usage.

I added tests for the workspace mode, as well as splitting the files in the workspace mode.
I added all the tests from the thrift tests, I'm not sure if so many are needed or if it's too much

I added tests for the workspace mode, as well as splitting the files in the workspace mode.

Fantastic jobs!

I added all the tests from the thrift tests, I'm not sure if so many are needed or if it's too much

Maybe that's a bit too much? I think one thrift test is enough, but the tested thrift files should be well-constructed for the workspace, e.g. A.thrift and B.thrift both use C.thrift struct, and see if the struct is generated in common crates and used correctly by A and B.

A.thrift and B.thrift both use C.thrift struct

Maybe more complex? for example, more layers of nesting mods/namespaces?

Could you maybe write down the .thrift file you have in mind? My knowledge of thrift is quite limited

Sorry for the late reply, I've constructed five idl files.
article.thrift:
include "image.thrift"
include "author.thrift"
include "common.thrift"

namespace rs article

enum Status {
    NORMAL = 0,
    DELETED = 1,
}

struct Article {
    1: required i64 id,
    2: required string title,
    3: required string content,
    4: required author.Author author,
    5: required Status status,
    6: required list<image.Image> images,
    7: required common.CommonData common_data,
}

struct GetArticleRequest {
    1: required i64 id,
}

struct GetArticleResponse {
    1: required Article article,
}

service ArticleService {
    GetArticleResponse GetArticle(1: GetArticleRequest req),
}

author.thrift:
include "image.thrift"
include "common.thrift"

namespace rs author

struct Author {
    1: required i64 id,
    2: required string username,
    3: required string email,
    4: required image.Image avatar,
    5: required common.CommonData common_data,
}

struct GetAuthorRequest {
    1: required i64 id,
}

struct GetAuthorResponse {
    1: required Author author,
}

service AuthorService {
    GetAuthorResponse GetAuthor(1: GetAuthorRequest req),
}

cdn.thrift:
include "common.thrift"

namespace rs article.image.cdn

struct CDN {
    1: required i64 id,
    2: required string url,
    3: required common.CommonData common_data,
}

common.thrift:
namespace rs common

struct CommonData {
    1: required i64 id,
    2: required string name,
    3: required string description,
}

image.thrift:
include "common.thrift"
include "cdn.thrift"

namespace rs article.image

struct Image {
    1: required i64 id,
    2: required string url,
    3: required cdn.CDN cdn,
    4: required common.CommonData common_data,
}

struct GetImageRequest {
    1: required i64 id,
}

struct GetImageResponse {
    1: required Image image,
}

service ImageService {
    GetImageResponse GetImage(1: GetImageRequest req),
}

And there are three services that can be used in workspace mode, in article.thrift, author.thrift, image.thrift.

Thanks again! LGTM now, seems there's still a little format issue, could you please use cargo fmt to fix it?

LGTM, @Millione PTAL

Great, thanks for the reviews!
Now I'd like to use this functionality in volo to fix https://github.com/cloudwego/volo/issues/454
Could you please publish the new (unstable?) version of pilota so that I can use it in volo?

@Millione Maybe we can publish a new version and make split the default behaviour?

@Millione Maybe we can publish a new version and make split the default behaviour?

Get it.

Could you please publish the new (unstable?) version of pilota so that I can use it in volo?

@missingdays Maybe you can use the git dependencies temporarily? Once it is ready in volo, we can release the version in pilota.

@Millione @PureWhiteWu I have submite the pull request to volo with this functionality https://github.com/cloudwego/volo/pull/510


App Release Checklist

Checklist
Cosmetics

[ ] Day color palette
[ ] Night color palette
[ ] Images
[ ] Lottie animations
[ ] No auto-layout warnings

Functionality
Persistent Data

[ ] Keychain stores login data
[ ] Purge persistent data on logout
[ ] Properly update events
[ ] Properly update announcements

Login/Logout

[ ] Login
[ ] Logout
[ ] Keychain auto-login

Home View

[ ] Countdown times
[ ] Upcoming Events
[ ] Happening Now Events

Schedule

[ ] Properly update tableview on tab change
[ ] Pull to refresh
[ ] Favorite/Unfavorite
[ ] Locations appear alphabetically
[ ] Cells push to detail view

TODO: finish list

Outdated issue.


Updated socket to allow communication both ways

For sending messages to socket, add the field:
"dest": "external" or "django"
See updates to dummyRequest or dummyRequestToExternal.py for example
Persistent connections need to send initial message:
{
"conn_type": 1(django) or 2(external)
}
To test:

run ServerSocket.py
run ExternalClienSocket.py & start django server in any order
Run dummyRequest.py (https://github.com/ENG4000-Team-A/capstone-project/pull/172 see here for more info)
Run dummyRequestToExternal.py

@digengill step 3 is not having the dummy message sent sucessfully. Getting the message "ConnectionAbortedError: [WinError 10053] An established connection was aborted by the software in your host machine". Both connections are accepted fine but its when a request is made the server just aborts

@digengill step 3 is not having the dummy message sent sucessfully. Getting the message "ConnectionAbortedError: [WinError 10053] An established connection was aborted by the software in your host machine". Both connections are accepted fine but its when a request is made the server just aborts

If you skip step 3, and do step 4, does step 4 work? Might be the query itself that is the problem, but I didn't get any problem though


Setting "Ignore first part of token name for styles" does not apply to import styles

Describe the bug
The setting "Ignore first part of token name for styles" only affects syncing from token to style, but not the other way around. When using the "Import Styles" option, the setting does not recognize tokens that are used as style source.
To Reproduce
Steps to reproduce the behavior:

Check Figma Tokens setting "Ignore first part of token name for styles"
Create a color token "colors.red.100"
Create a color style "red/100"
Update token
It updates the equivalent style
Update style
Use "Import styles" option
The color style is detected as new, not as an update to an existing token

Expected behavior
The "Import style" option should detect updates to existing tokens when using the "Ignore first part of token name for styles" setting.

I'm afraid we won't be able to support that any time soon as 'colors' could just as well be anything else, or even multiple tokens could match if one is called colors.red.100 and another global.red.100.
We'll provide better support for this with advanced theming which keeps a connection between styles and tokens

I understand that there might not be a 1-to-1 match. But the process already lets you pick which items to import, so couldn't it just include options for each of the *.red.100 matches? Maybe only match on type color for good measure.
Also, doesn't import only ever apply to the currently selected token set? So for your scenario to happen, you would have to had made two red.100 tokens in the same token set. That seems like the above solution producing multiple matches would be uncommon and an expected outcome for those choosing to do that.
Either way, I'm just trying to figure a sustainable way to keep this plugin working smoothly with Figma's native styles. I know someone will inevitably edit a style and be frustrated that it's out-of-sync.

We'll soon be introducing a way to store style connections on a Theme level where we store style id, that way you could name your token however you want after a connection has been established.

Closing this, we shipped Connected styles which allow you to connect styles to tokens. Give it a go! This issue is something we won't be getting to, but we hope the style connection solves that for you.


Treeview - not rendering correctly

Hi I am having this same issue, did you find a resolution?

@ddslawson , @drma-dev  any new news on this topic??

Yes, the fix is to use the latest development package on nuget and not the latest stable. I'm currently using 2.9.0-develop-042.

the project is abandoned. go to another.

@drma-dev , @ddslawson  Thanks a lot


Save base_images_from_dockerfile to /workspaces/source

The reason is that this was failing in buildah remote task because the line
echo "$BASE_IMAGES" > /workspace/base_images_from_dockerfile
was executed on a VM via SSH and it was not restored afterwards. However, /workspaces/source is actually restored via
rsync -ra "$SSH_HOST:$BUILD_DIR/workspaces/source/" "$(workspaces.source.path)/"
https://github.com/redhat-appstudio/build-definitions/blob/main/task/buildah-remote/0.1/buildah-remote.yaml#L326
So storing it in /workspaces/source should make it available in the end.

failure happened in infra-deployments e2e tests in https://github.com/redhat-appstudio/infra-deployments/pull/3515

/retest

/retest

/lgtm

/retest

this PR is supposed to fix this issue that is currently affecting infra-deployments CI

/retest


Implement Response.bytes_stream for wasm

I'm building an application which uses reqwest in combination with the stream feature. This application uses both native and WASM target. Currently Response.bytes_stream is only implemented for non-WASM targets.
There were several attempts to implement this:

https://github.com/seanmonstar/reqwest/pull/1151
https://github.com/seanmonstar/reqwest/pull/1576
https://github.com/seanmonstar/reqwest/pull/1372

The most recent PR is: https://github.com/seanmonstar/reqwest/pull/1576 but it needs to be updated and approved eventually.

The most recent PR is: https://github.com/seanmonstar/reqwest/pull/1576 but it needs to be updated and approved eventually.

This has been done so it just needs to be reviewed.


[SUBMISSION] Instruction Tuning Completed

December 2024 Student Submission
Module Completed

[x] Module 1: Instruction Tuning
[ ] Module 2: Preference Alignment
[ ] Module 3: Parameter-efficient Fine-tuning
[ ] Module 4: Evaluation
[ ] Module 5: Vision-language Models
[ ] Module 6: Synthetic Datasets
[ ] Module 7: Inference
[ ] Module 8: Deployment

Changes Made
Describe what you've done in this PR:

What concepts did you learn?

I discovered best practices for tokenizing and decoding conversational data for language model training, as well as how the Smoltalk and OpenAI's GSM8K datasets can be applied to develop and refine conversational AI models.

What changes or additions did you make?

I added code to demonstrate the usage of chat templates with SmolLM2.
I worked on the examples of how to apply the chat template, tokenize the conversation, and decode the output for both Smoltalk and OpenAI's GSM8K datasets.

Any challenges you faced?

Initially, I had some difficulty understanding the structure of chat templates and how to apply them correctly.
I also needed to learn more about the trl library and its functions for working with chat formats.

Notebooks Added/Modified
List any notebooks you've added or modified:

[ ] Added new example in module_name/students/my_example.ipynb
[ ] Modified existing notebook with additional examples
[x] Added documentation or comments

Checklist

[x] I have read the module materials
[x] My code runs without errors
[ ] I have pushed models and datasets to the huggingface hub
[x] My PR is based on the december_2024 branch

Questions or Discussion Points
Add any questions you have or points you'd like to discuss:

Are there any alternative chat template structures or libraries that could be explored?
What are some best practices for fine-tuning SmolLM2 or similar models on conversational datasets like Smoltalk?

Additional Notes
Any other information that might be helpful for reviewers:

@branley1 Could you review another student's PR, for example: https://github.com/huggingface/smol-course/pull/81 and mention me. The can then review your work, and I'll review too.

Nice work! Two points:

Could you exchange a review with one of the other students's PRs?
could you move your work to the 'student_examples' folder.